csharp support (proto3) about s2client-proto HOT 4 CLOSED

We've tried to keep the syntax compatible with both proto2 and proto3. Do you get errors when using the proto3 compiler?

from s2client-proto.

proto3 using zero based Enums;
data.proto contains:
enum Target {
None = 1; // Does not require a target.
Point = 2; // Requires a target position.
Unit = 3; // Requires a unit to target. Given by position using feature layers.
PointOrUnit = 4; // Requires either a target point or target unit.
PointOrNone = 5; // Requires either a target point or no target. (eg. building add-ons)
}
Im getting compile error.
Looks like if i manualy change None = 0 and etc. it will confuse system. Didnt it?

from s2client-proto.

Better to add Unknown = 0;

from s2client-proto.

I would be fine with adding an Unknown entry to our enums for compatibility. Are there any other proto3 errors?

from s2client-proto.