Merge pull request #44 from younies/units_router_younies

Merge sffc/units-staging branch

Merge branch 'units-staging' of github.com:sffc/icu into units_router_younies

Merge pull request #43 from younies/units_router_younies

fix units router and allow the test cases to compare double numbers w…

fix units router and allow the test cases to compare double numbers with precision

Merge pull request #23 from younies/unit_header

add usage to number formatter settings header

Merge pull request #45 from hugovdm/usagefix

Support usage component dropping, and more

Drop unnecessary reference to "draft API".

Add an error check, as per code review.

Support usage component dropping, and more:

- Improve efficiency by not constructing UnitPreferenceMetadata instance
  in binarySearch: reuse an instance passed in.
- Fix stale documentation. (Doc freshness is hard! ;-)
- getPreferenceMetadataIndex actually returns -1 upon failure.

Add test cases for dropping last component(s) of usage.

ICU 68 (not 67). Add U_HIDE_DRAFT_API ifdef guards.

Some more documentation tweaks.

Clarify the documentation.

Implement full output testing for unitPreferencesTest.txt tests.

Add error-handling to avoid assertion failure.

Merge pull request #42 from younies/unit_header_staging

Add missing comments ...

add a note about  with sub usage

add a note about  with empty string

Merge pull request #43 from hugovdm/getUnitCategory

Add getUnitCategory()

Merge branch 'units-staging' into getUnitCategory

TODO(CLDR-13787) for improving inverse handling.

Add unit tests, improve docs, add a FIXME for consumption-inverse.

Merge branch 'units-staging' into units_router

Merge pull request #42 from hugovdm/UnitPreferences

UnitPreferences class in unitsdata.cpp

Merge pull request #41 from hugovdm/units_router_prefstest

Test UnitsRouter constructor and fix category/preferences handling

Add getUnitCategory.

fixup! Use UnitPreferences, drop getUnitsData and related.

Start hooking up UnitsRouter and unitPreferencesTest.txt.

Add getUnitCategory.

Use UnitPreferences, drop getUnitsData and related.

Merge PR#42 into units_router_unitprefs (branch 'UnitPreferences')

(PR still open.)

const implemented: const UnitPreferences *const

Merge pull request #40 from younies/units_router_younies

Merge sffc/units-staging

merge units-staging 3

merge units-staging

merge units-staging

Code review feedback incorporated.

Merge branch 'units-staging' into UnitPreferences

Merge pull request #21 from younies/tryingdouble

Add unit converter.

const annotations for getPreferencesFor() and thus getAlias()

Implement slicing via `UnitPreference **&outPreferences`.

(Still multiple TODOs for make things cleaner/neater.)

Merge pull request #39 from younies/tryingdouble_staging

add a TODO to load the constants\' values from the units data

add a TODO to load the constants\' values from the units data

Merge pull request #38 from younies/tryingdouble_staging

fix substituteConstans function

fix substituteConstans function

Merge pull request #37 from younies/tryingdouble_staging

add substitution of the constant to `Factor`

add substitution of the constant to

Merge pull request #36 from younies/tryingdouble_staging

Fix Shane's comment about unitconverter.cpp

fix UnitConverter

Reduce number of binary searches again, resulting in more complexity.

More readable binarySearch at 4x worst-case cost.

With this hopefully more readable version, looking for an unknown
usage and region will result in the binary search being repeated four
times.

The previous version did not use a metadata instance comparator so
that individual components of the metadata key could be compared. A
comparator that returns status information on which of the components
had been found would be able to improve on this.

One improvement to error reporting.

Merge pull request #35 from younies/tryingdouble_staging

Fix dependencies and small nits

fix dependices and small nits

First set of code review improvements.

Fix dependencies.txt, add documentation, add appropriate UMemory inheritence.

Merge remote-tracking branch 'sffc/units-staging' into units_router

Merge pull request #34 from younies/tryingdouble_staging

Remove extractUnitInfo from UnitConverter's helpers

remove extra spaces from the factors

UnitPreferences class in unitsdata.cpp

remove extractUnitInfo from UnitConverter's helpers

Merge pull request #33 from hugovdm/tryingdouble_unitsTestTxt

testConversions: test UnitConverter against unitsTest.txt test cases

Merge pull request #41 from hugovdm/units-staging

Update unitsTest.txt and unitPreferencesTest.txt from current CLDR master

Update unitsTest.txt and unitPreferencesTest.txt from current CLDR master.

testConversions: test UnitConverter against unitsTest.txt test cases

Merge pull request #32 from hugovdm/tryingdouble

Reinstate assertEqualsNear that got lost in merge.

Reinstate assertEqualsNear that got lost in merge.

- Code compiles, `make check` returns OK.

merge sffc/units-staging

Merge pull request #39 from hugovdm/unitsTestTxt

testConversions: test convertibility of unitsTest.txt test cases.

Add IntlTest::assertNotEquals() for int32_t.

Remove hard-coded unit tests that were pulled out of unitsTest.txt.

Add some comments to UnitsTestContext.

Remove the 'FIXME_skip_tests' bool and branch.

Uncomment pound-force-per-square-inch test in unitsTest.txt

testConversions: test convertibility of unitsTest.txt test cases.

Post-merge: update code to make "make check" pass.

* MaybeStackVector<ConversionRateInfo> -> ConversionRates.
* Inherit from UMemory to make MaybeStackVector happy (UnitConverter
  and ConverterPreference).
* Hack around getUnitsData's bad interface.

Code is still bad though, unit tests don't cover everything:

* getUnitsData does not work correctly, and various things are marked
  with "WIP" comments (Work In Progress).

Merge branch 'tryingdouble' into units_router

Complicated manual merge of branch 'units-staging' into tryingdouble

This adds a ConversionRates::getInternalList() that needs to be
removed again, by improving APIs appropriately.

Merge pull request #32 from younies/check_convirtible_units

Check convertible units

add units_extra to unitsformatter dependencies

Merge pull request #31 from hugovdm/check_convirtible_units

Enable "testConversions" test again

Merge branch 'check_convirtible_units' into check_convirtible_units

fix unitConverter and add ConversionRates class

Fix make check: dependencies.txt, fix units_extra deps

Remove a getUnitsData test failure, but the function is still broken.

Merge branch 'tryingdouble' into units_router

Tweaked during merge: use the "getAllConversionInfo" function.

Merge branch 'units-staging' into tryingdouble

Conflict resolution: uncomment unit tests that pass now, but were
previously disabled by commenting-out within units-staging.

Enable "testConversions" test again.

With MeasureUnit fixes merged, these tests seem to pass now.

fix dependencies.txt

fix the header

merge sffc/units-staging

Merge branch 'units-staging' of github.com:sffc/icu into unit_header

Merge pull request #31 from hugovdm/units-staging-getConversionRatesInfo

Add unitsdata.cpp, getConversionRatesInfo, and unit tests.

Merge pull request #30 from hugovdm/check_convertible_units_identifierfixes

Fix the broken testConversionCapability unit identifiers.

Fix the broken testConversionCapability unit identifiers.

Remove testGetConversionRateInfo.

Fix deps: use UMemory as base class for it's new().

dependencies.txt attempt #1: where's "operator new(unsigned long)" coming from?

Nit: remove two unneeded #includes

Comment out broken test cases as per younies' 476ca805c