search

leejw51 / protobuf-net

Automatically exported from code.google.com/p/protobuf-net
0 stars 0 forks source link

Serializer.GetProto<T> outputs fixed64 for double? #108

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 

Using r278, I believe that Serializer.GetProto<T> is turning this:

    [ProtoContract]
    public class OutputPoint
    {
        /// <summary>
        /// X the ProtoMember
        /// </summary>
        [ProtoMember(1)]
        public double X { get; set; }

        /// <summary>
        /// Y the ProtoMember
        /// </summary>
        [ProtoMember(2)]
        public double Y { get; set; }

into a .proto file with:

message StylusPoint {
  optional fixed64 X = 1;
  optional fixed64 Y = 2;

I was expecting to see 'double' rather than 'fixed64'.

Original issue reported on code.google.com by joseph.j...@gmail.com on 20 May 2010 at 11:42

GoogleCodeExporter commented 9 years ago 
We are planning to use protobuf-net in our project. And this issue is stopping 
us from using it. Any plans to resolve this soon?

Original comment by arvind.g...@gmail.com on 26 Aug 2010 at 1:39

GoogleCodeExporter commented 9 years ago 
Same issue for float as well.

Original comment by arvind.g...@gmail.com on 26 Aug 2010 at 1:40

GoogleCodeExporter commented 9 years ago 
Hi, I'm having the same issue here, any chance this can get fixed in the near 
future.

Original comment by jacob.so...@gmail.com on 26 Aug 2010 at 3:10

GoogleCodeExporter commented 9 years ago 
I've changed this in source (r349) in the v1 branch. Due to machine changes, 
I'll need to setup a new VM to deploy a new dll; can you build from source for 
now?

Original comment by marc.gravell on 26 Aug 2010 at 6:44

GoogleCodeExporter commented 9 years ago 
I've now run into this too.  I guess I'll have to go figure out how to get the 
branch.

Original comment by bryanliv...@gmail.com on 26 Jan 2011 at 8:41