Region.clone() creates identifiers with a null string instead of null identifiers

[matiash]

The problem seems to be in the Region(Region otherRegion) copy constructor:

    for (int i = 0; i < otherRegion.mIdentifiers.size(); i++) {
        mIdentifiers.add(new Identifier(otherRegion.mIdentifiers.get(i)));
    }

This means that for each position that the original region had no identifier, as in the basic example:

new Region("myRangingUniqueId", null, null, null);

the cloned copy (for example, produced when calling startRangingBeaconsInRegion()) will have an Identifier with a null string instead.

A possible fix would be to change the copying loop into:

    for (int i = 0; i < otherRegion.mIdentifiers.size(); i++) {
        Identifier otherIdentifier = otherRegion.mIdentifiers.get(i);
        mIdentifiers.add(otherIdentifier != null ? new Identifier(otherIdentifier) : null);
    }

[davidgyoung]

Fixed in e9e3f80