robotdotnet / NetworkTables

FRC NetworkTables for .NET. This is all old code and should not be used anymore.
3 stars 5 forks source link

Adds fast bitconverter code #36

Closed ThadHouse closed 8 years ago

ThadHouse commented 8 years ago

Previously always involved allocations, which was slow and had a lot of memory allocations. New code uses unsafe code, but is MUCH faster.

ThadHouse commented 8 years ago

I did some performance testing on this, and it actually does help with a lot of keys, as allocations are reduced immensely while sending data.

ThadHouse commented 8 years ago

This way still had the performance benefits, but no unsafe code. So I like this better.

codecov-io commented 8 years ago

Current coverage is 63.55% (diff: 30.20%)

Merging #36 into master will decrease coverage by 0.49%

@@             master        #36   diff @@
==========================================
  Files            40         42     +2   
  Lines          6823       6914    +91   
  Methods         803        817    +14   
  Messages          0          0          
  Branches        899        900     +1   
==========================================
+ Hits           4370       4394    +24   
- Misses         2184       2250    +66   
- Partials        269        270     +1   

Powered by Codecov. Last update 6c62ba0...5ab4a1e