flatc converts a protobuf 'bytes' field to 'string' in fbs schema file

[cdkamat]

I have a protobuf schema with

optional bytes fingerprint = 1;

I am using flatc --proto to convert it to a fbs schema.

The schema generated has

fingerprint:string;

I was expecting a

fingerprint: [byte];

In the parser code, I see that the mapping is

 { "bytes", BASE_TYPE_STRING },

https://github.com/google/flatbuffers/blob/master/src/idl_parser.cpp#L1729

Is the conversion to string intentional? If the bytes in proto were used to store binary data, will they be compatible with the generated fbs ?

[ghost]

I agree, [byte] would be better. Under C++, string is about the same, but in other languages it isn't. A PR to fix this would be welcome.

[ghost]

Fixed: 69776b9

[cdkamat]

Thanks for fixing this.