• Changed ParameterInfoMap to ParameterInfoArray.
  • Not sure why refit was using a dictionary as an array?
• Changed Headers, HeaderParameterMap, PropertyParameterMap, QueryParameterMap, AttachmentNameMap to only allocate a dictionary when needed.
  • Sharing a mutable Empty dictionary is a little risky but I dont see why we'd ever want to mutate the collection.
• Addded EmptyDictionary<TKey, TValue>.get() to create static empty dictionaries. I would prefer to use ImmutableDictionary.Empty and change Dictionary to IReadOnlyDictionary but they aren't available in . Net Framework.
  • This is a little messy but It saves 72 bytes per dictionary instance.
• bodyParamEnumerable and authorizeParamsEnumerable now use a tuple instead of an anonymous class.
• Used an integer HeaderCollectionParameterIndex instead of the hashset HeaderCollectionParameterMap
  • Another strange one, refit explicitly prevents adding more than 1 item to the hashset, it seemed a little redundant 😄.

We could probably get away with using arrays instead of Dictionary<int, TSomething> they would use less memory in most cases and would have faster lookups for smaller numbers.

Saves 4Kb of memory, I might be able to do more after profiling.

Original

Changes