clang-format pass over new files, for good measure

Clean up getConversionRatesInfo to prepare for review.

Move testGetConversionRateInfo to UnitsDataTest

Rename getunitsdata.{cpp,h} to unitsdata.{cpp,h}

Merge pull request #21 from hugovdm/units_router_testfix

Remove failing volt-ampere test case, consider it non-sensical enough.

Remove the volt-ampere test case, consider it non-sensical enough.

Merge pull request #20 from hugovdm/units_router

Fix getConversionRateInfo compound-base-unit calculation bug

Comment out a test that is failing.

Add consistency checks for input and output units.

Fix getConversionRateInfo bug.

Merge pull request #19 from younies/units_router_younies

remove non-needed comment

remove non-needed comment

Merge pull request #18 from hugovdm/units_router

Fix for base unit complexity != 1. Eg: volume -> cubic-meter.

Fix for base unit complexity != 1. Eg: volume -> cubic-meter.

Merge pull request #17 from hugovdm/units_router

Load data for compound units

Also rename source to sourceUnit (matching baseUnit).

Use 'baseUnit' for conversion pivot unit rather than 'target'.

Implement getConversionRatesInfo, plus cleanups&improvements.

Merge pull request #16 from hugovdm/units_router_quickfix

Fixes to make complexunitsconverter and unitconverter compile again.

Fixes to make complexunitsconverter and unitconverter compile again.

Merge pull request #15 from younies/units_router_younies

Merge sffc/units_staging branch

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

Merge branch 'unit-id' into units-staging

Merge branch 'master' into unit-id

ICU-20920 Add support for CLDR 37 unit identifiers in ICU4C

See #964

ICU-20976 GCC 8 fixes phase 1

Some initial fixes for GCC 8

- setup a GCC 8 buildbot with -Wextra
- rewrite ucol_sit to use CharString
- workaround for gcc7+ on mac
  see https://github.com/arbor-sim/arbor/issues/562#issuecomment-409970434
- fix ucnv_2022 strcpy site

Code review feedback

Merge pull request #14 from younies/units_router_younies

Comment out broken tests

fix the testing

Merge pull request #13 from younies/units_router_younies

Hook up the data into units router

hook up the data to units router

ICU-20956 Fix monetary symbol getters in DecimalFormat

See #987

Merge pull request #12 from younies/units_router_younies

delete the hard coded test cases

delete the hard coded test cases

Merge pull request #11 from younies/units_router_younies

Move getUnitsData to a separate file

Move getUnitsData to a separate file

Reply to code review feedback

Reply to code review feedback

ICU-21024 RBBI Table Builder / Coverity CID 1460598 "Dodgy Code" issue

Fix the issue identified by Coverity.
The problem was in code handling the mapping from the table build time
representation of a set of status values for an RBBI rule to the corresponding
status data as saved in a binary RBBI rule file.

The problem was benign, the rbbi data built by the incorrect code would
would still operate correctly, although it might not byte-for-byte match
that built by ICU4C. (The problem was in Java only.)

Merge pull request #10 from younies/tryingdouble

pass the data to unit converter

Merge branch 'units_router' into tryingdouble

merge tryingdouble branch

use CharString for internal data

Merge pull request #9 from younies/tryingdouble

move enum::constants to unitconverter.cpp and remove unnecessarily us…

Merge pull request #8 from younies/units_router_younies

Add Todo for Hugo

Merge branch 'tryingdouble' of github.com:younies/icu into tryingdouble

use CharString for internal data

Merge branch 'units_router' into tryingdouble

move enum::constants to unitconverter.cpp and remove unnecessarily use of CharString

Merge branch 'units_router' of github.com:younies/icu into units_router_younies

add a todo(hugo) for getUnitsData

Merge pull request #7 from hugovdm/younies-units_router-convert

Add string length checks to testGetUnitsData.

Add string length checks to testGetUnitsData.

Merge remote-tracking branch 'origin/unit_header' into units_router

Merge branch 'units_router' of github.com:younies/icu into units_router

fix building units router branch

Merge pull request #6 from hugovdm/younies-units_router-convert

Various resource-loading code and documentation improvements.

Various resource-loading code and documentation improvements.

Merge pull request #5 from hugovdm/younies-units_router-convert

Hook up data-driven unit conversion tests

Avoid adding duplicate conversion rate data.

Also includes some cosmetic and readability improvements.

ICU-21016 Special handling of Spanish and Hebrew list format until CLDR get the data

See #1043

Little fixes to unitconverter.cpp

Hook up data-driven unit conversion tests: triggers segfault

Contains a "FIXME_*" boolean for selecting between fprintf and
constructing a UnitConverter, because the UnitConverter constructor is
currently segfaulting.

Also contains some clean-up of testConversions code.

merge tryingdouble branch

Merge pull request #4 from hugovdm/younies_units_router_resources

Add units resources data loading. (Unit test is WIP.)

Remove namespace hacks. Trust code review will deal with 'work in progress'.

Fix default geq: 1, not 0.

Merge branch 'units_router' into younies_units_router_resources

remove std::string

add unitsroutertest.cpp

address Shane's comments on number_formatter.h

Add skeleton for testing units router

Strip down ConversionRateInfo to only four CharStrings.

WIP: Add base conversion rates for all preferences: with dupes.

ICU-21013 Fix OOM not checked in vtzone.cpp

Also change NULL to nullptr

Move wip_units_resource_loader.* code into unitsrouter.*

Delete old wip_units_resources code.

Merge remote-tracking branch 'younies/units_router' into wip_younies_units_router_resources

ICU-21019 x86: Add .note.gnu.property section for Intel CET

Intel Control-flow Enforcement Technology (CET):

https://software.intel.com/en-us/articles/intel-sdm

contains shadow stack (SHSTK) and indirect branch tracking (IBT).  When
CET is enabled, ELF object files must be marked with .note.gnu.property
section.  GCC provides <cet.h> which can be included in assembly codes
to generate CET maker when compiling with -fcf-protection.

Working parsing and returning of units data.

WIP: c-style ures_* handling.

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

fix comments

add getOutputUnit to FormattedNumber

address Shane's comments on number_formatter.h

ICU-21020 Document unchecked exception in various number formatter classes

ICU-20978 Reduce unnecessary building for tools when cross-building

ICU-20916 late computation of roundedThreshold to ensure it is up to date

Change "+" to "-and-"

Update simple unit identifiers

Rename TempSingleUnit -> SingleUnitImpl

Revert changes to LocalArray (localpointer.h)

address Shane's comments2

add another constructor to ComplexUnitConverter

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

ICU-20605 travis: make dist

- run 'make dist' after the linux clang build

ICU-20797 fix UBS compilation error and UBS in test code

Two issues here:

- fix 2 build issue in i18n when compiling with clang++ -fsanitize=undefined
the following two symbols were not exported (and they should be):
  typeinfo for icu::CollationCacheEntry
  typeinfo for icu::numparse::impl::CodePointMatcher

- remove undefined behavior warning in NumberFormatTestTuple.. minor, but very annoying
when repeated many times during every test run. Tends to mask real errors.

> numberformattesttuple.cpp:319:5: runtime error: member access within null pointer of type 'NumberFormatTestTuple'

ICU-20600 build icu-data-bin files

make dist will now produce:
- icu4c-67.1-data-bin-b.zip
- icu4c-67.1-data-bin-l.zip

ICU-20910 C API Change Report Tool updates

- This commit was used to generate ICU4C 65's report
- This commit was inadvertently left out of [ICU-20162]

Changes:
- fixup copyright messages
- code cleanup
- fix unreplaced '$rightVer' in born stable APIs

Merge pull request #26 from hugovdm/units-staging-preferences

Parse unitPreferencesTest.txt preparing to run data-driven tests