Re: [protobuf] typedef support

2017-01-18 Thread Arpit Baldeva
s submessages. Feel free to file an issue, though, if you > think you can make a good argument for why the feature is worth it. > > On Tue, Jan 17, 2017 at 3:29 PM, Arpit Baldeva <abald...@gmail.com> wrote: > >> Should I file a GitHub issue for this? I am not sure what the right

Re: [protobuf] typedef support

2017-01-17 Thread Arpit Baldeva
Should I file a GitHub issue for this? I am not sure what the right protocol is. Thanks. On Wednesday, November 23, 2016 at 1:04:07 PM UTC-8, Arpit Baldeva wrote: > > Thanks for the info. Typedef support would be pretty much useful for > regular use cases in C/C++ languages. Fo

Re: [protobuf] Protobuf documentation

2016-09-23 Thread Arpit Baldeva
As a little background, I am a new user to protobuf. Naturally, I am trying latest releases. And I see no mention on the page that this syntax may not be supported in latest version. So it ends up being odd that the first example I see does not compile with latest release :). -- You received

Re: [protobuf] Protobuf documentation

2016-09-23 Thread Arpit Baldeva
Can you elaborate what you mean by version 3 of protobuf supporting both proto 2 and 3? I was using latest pre-built protoc compiler and it balked at this sample. Arpit -- You received this message because you are subscribed to the Google Groups "Protocol Buffers" group. To unsubscribe from

Re: [protobuf] Protobuf documentation

2016-09-26 Thread Arpit Baldeva
you saw? I tried compiling the example just now > and it worked just fine, although protoc did log a warning that I needed to > set the syntax. > > On Fri, Sep 23, 2016 at 8:43 PM, Arpit Baldeva <abal...@gmail.com > > wrote: > >> As a little background, I am

Re: [protobuf] protobuf::Any Message vs MessageLite interface

2016-10-07 Thread Arpit Baldeva
Thanks for the info. I feel like without pack/unpack/Is method, the utility of Any will diminish. For example, the rpc status proto (https://github.com/googleapis/googleapis/blob/master/google/rpc/status.proto) uses repeated Any field. It'd not be possible to write code like one described

[protobuf] protobuf::Any Message vs MessageLite interface

2016-10-04 Thread Arpit Baldeva
Hi, I was wondering if there is any reason that Any type is implemented in terms of Message interface rather than MessageLite? It'd seem to me that all this class needs to work correctly is to use GetTypeName() on MessageLite interface. My motivation is, of course, to be able to use Any type

[protobuf] Protobuf documentation

2016-09-22 Thread Arpit Baldeva
Hi, The documentation here - https://developers.google.com/protocol-buffers/docs/overview shows following: optional PhoneType type = 2 [default = HOME]; However, default value can no longer be specified in proto3. I'd suggest to remove this and keep the documentation up to date with the

[protobuf] additional map keys

2016-11-21 Thread Arpit Baldeva
Hi, Are there any plans to support 1. Enums as keys in a map 2. Case-insensitive keys in a map Thanks Arpit -- You received this message because you are subscribed to the Google Groups "Protocol Buffers" group. To unsubscribe from this group and stop receiving emails from it, send an email

[protobuf] typedef support

2016-11-21 Thread Arpit Baldeva
Hi, Are there any plans to introduce typedef support in the Proto files? Thanks Arpit -- You received this message because you are subscribed to the Google Groups "Protocol Buffers" group. To unsubscribe from this group and stop receiving emails from it, send an email to

[protobuf] file import via another import

2017-01-13 Thread Arpit Baldeva
Hi, It seems like the import directive in a proto file does not import the import dependencies of the file being imported. Easier to explain by example here. Say you have 3 files - a.proto, b.proto, c.proto If b.proto imports a.proto and c.proto imports b.proto, c.proto does not

[protobuf] extending field and message options

2017-03-13 Thread Arpit Baldeva
Hi, I have my proto file like following. package example; message ExFieldOptions { map meta_data = 1; } extend google.protobuf.FieldOptions { ExFieldOptions exOptions = 4245; } extend google.protobuf.MessageOptions { ExFieldOptions exOptions = 4245; } When I

[protobuf] package attribute

2017-03-06 Thread Arpit Baldeva
Hi, While working with proto files, I realized that I can place the package attribute anywhere in the file and all the messages (even the ones above it) get put in the namespace generated due to the package attribute (C++). Is that a 'feature' I can rely upon ? In other words, ProtoC does not

Re: [protobuf] Why doesn't protobuf C++ include smart pointers to submessages?

2017-07-24 Thread Arpit Baldeva
A weak_ptr needs a shared_ptr to begin with so all the submessages would end up being a shared_ptr (creating an impression that they are meant for shared ownership). In fact, I feel that Protobuf could perhaps change the generated code to use unique_ptr instead of it's raw pointers to more

Re: [protobuf] Next release tentative due date(3.4.0)

2017-07-24 Thread Arpit Baldeva
Thanks. In general, is there a release schedule? Like once a quarter, once every X weeks etc? On Friday, July 21, 2017 at 2:46:34 PM UTC-7, Bo Yang wrote: > > https://github.com/google/protobuf/milestone/20 > ETA is end of this month. > > On Thu, Jul 20, 2017 at 1:26 PM Arpi

[protobuf] Next release tentative due date(3.4.0)

2017-07-20 Thread Arpit Baldeva
Hi, Is there a release schedule page that I can follow for upcoming release dates? I am interested in knowing the next release due date (tentative is fine). Thanks. -- You received this message because you are subscribed to the Google Groups "Protocol Buffers" group. To unsubscribe from

[protobuf] Re: Json object hash

2017-07-05 Thread Arpit Baldeva
A list as a 'value type' in not supported for map. I have not worked with JavaScript and protobuf but I believe in the other languages, it will be a code generation error. In general, I have had the similar problems you have when you need to come up with a particular json format which protobuf

[protobuf] protobuf js custom options

2017-10-06 Thread Arpit Baldeva
Hi, Are custom options supported with protobuf JS? I have my proto files that work fine with C++/C# so I don't think it is a syntax issue. However, when I try to run the grpc sample (I am not sure it is a grpc issue though), I get an error similar to

[protobuf] C++ - Protobuf descriptor static memory leaks

2017-11-09 Thread Arpit Baldeva
Hi, Currently, protobuf generated code makes allocations in the static descriptor objects. For example, valgrind reports Memcheck:Leak fun:malloc fun:_ZN20PassthroughAllocator5AllocEmPKcj fun:_Znwm

Re: [protobuf] Re: protobuf js custom options

2018-06-06 Thread Arpit Baldeva
Sorry, I did not investigate it further (I don't currently have a use case of it - I was just testing the support). On Wed, Jun 6, 2018 at 10:26 AM, wrote: > Hi Arpit, > > Did you have any luck? > > > On Friday, October 6, 2017 at 3:24:09 PM UTC-7, Arpit Balde

[protobuf] protoc input dependency checking (--dependency_out)

2018-01-04 Thread Arpit Baldeva
Hi, I tried using --dependency_out option and ran into a few issues. 1. It does not work with multiple files input. Are there any plans to add that support? 2. I noticed that the dependency file includes the whole dependency chain from other proto files. I am not sure if it is