Implemented decoding of ZigZag-encoded varints.

[moritzuehling]

The parser breaks with the netcode from the 13501-update. This fixes it.

Source for the implementation: The official parser

[taylorfinnell]

Sweet! it should be this.varInt32 not reader.ReadVarInt. I think the 13501 update causes this line to be called too

jsgo/lib/jsgo.js

Line 1215 in 8434927

return stream.bits(prop.numBits); // hmm

. I don't think it's implemented the same as the official implementation either. Trying to dig into it

[moritzuehling]

@taylorfinnell I just fixed it, mind trying it again?

[moritzuehling]

@taylorfinnell And even if the line is called it reads the right amount of bits - the working code looks like this:

int CBitRead::ReadSBitLong( int numbits )
{
    int nRet = ReadUBitLong( numbits );
    // sign extend
    return ( nRet << ( 32 - numbits ) ) >> ( 32 - numbits );
}

The right amout of bits is read, so it shouldn't bring the parser in a crashing state.

[taylorfinnell]

Thanks! I gave it a shot, still crashing on my demo. Here is the demo I am using. http://www.megafileupload.com/9muE/nerf_update.dem I am still looking into it. I also used the implementation of ReadSBitLong that you put above with no luck

[HarasimowiczKamil]

should fix #7