Fixed a bug in the ReverseDocComparatorSource where it wasn't actually reversing (missing a negative sign).
Fixed BoboTestCase.DoTest to output the IDs for the result the same way it does for the input.
Fixed bug in BoboTestCase.TestPath() - the first test should be in ascending order.
Fixed bug in CombinedFacetAccessible.GetFacets() where the test was being made for a null string instead of a null or empty one.
Reformatted the bit shift code to ensure that the value is cast to uint before being shifted.
Aligned the ListMerger variable names with the original source.
Fixed bug in FacetHandlerTest.TestNegativeLoad where it was not possible for the test to pass even with the expected result.
Removed the catch statement and added using statements to the BoboTestCase.TestMultiSort() method.
Made the BrowseHit.Comparable and BrowseResult.SortCollector properties into fields so they could be marked with the NonSerialized attribute (which fixes the TestMultiSort test).
Fixed CombinedFacetAccessible.GetFacets(). The facet iterator could return either null or empty string in a no-op scenario. This was fixed earlier in just one place, but there were other places where the check was still being made with null.
Added missing documentation to the DynamicTimeRangeFacetHandler.
Fixed bug in FacetHitcountComparatorFactory where comparison logic was incorrect.
Fixed bug in format function of TermNumberList.
Fixed bug in TestCustomFacetSortComparatorFactory that was causing the TestCustomFacetSort test to fail.
Fixed the constructor of FacetTermQuery to accept IDictionary instead of Dictionary and fixed the value property so it returns the correct result.
Removed some of the unnecessary parenthesis from the BoboTestCase tests.
Fixed bug in the BoboDocScorer where it was attempting to lookup the values using the formatted values, but was matching against raw values.
Fixed the TermLongList tests - a) The original tests were using the elements property. b) There was no Arrays.Equals method to test the equality between 2 arrays.
Fixed bug in CombinedFacetIterator where result of FacetIterator.Next(int) was being checked against null, but in fact it could also be an empty string.
Refactored FacetDataCache, MultiValueFacetDataCache, and MultiValueWithWeightFacetDataCache so they no longer have interfaces. Since they didn't need to be generic, it is better to align with the original source.