From 71f0cd45eff088d26309564d6766003bb788a04b Mon Sep 17 00:00:00 2001 From: Soonil Nagarkar Date: Fri, 30 Jan 2015 11:29:42 -0800 Subject: Update Location.convert() javadocs to reflect their inconsistency with locales. The Location.convert() methods do not invert each other as might be expected. Changing this would introduce breaking changes, so I've updated the javadocs to make this clearer. Bug: 13280976 Change-Id: If4bd3c83d5fb67915450849ca471aabc27544dac --- location/java/android/location/Location.java | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) (limited to 'location') diff --git a/location/java/android/location/Location.java b/location/java/android/location/Location.java index fcf222b..bf3387b 100644 --- a/location/java/android/location/Location.java +++ b/location/java/android/location/Location.java @@ -170,6 +170,9 @@ public class Location implements Parcelable { * Converts a coordinate to a String representation. The outputType * may be one of FORMAT_DEGREES, FORMAT_MINUTES, or FORMAT_SECONDS. * The coordinate must be a valid double between -180.0 and 180.0. + * This conversion is performed in a method that is dependent on the + * default locale, and so is not guaranteed to round-trip with + * {@link #convert(String)}. * * @throws IllegalArgumentException if coordinate is less than * -180.0, greater than 180.0, or is not a number. @@ -217,7 +220,9 @@ public class Location implements Parcelable { /** * Converts a String in one of the formats described by * FORMAT_DEGREES, FORMAT_MINUTES, or FORMAT_SECONDS into a - * double. + * double. This conversion is performed in a locale agnostic + * method, and so is not guaranteed to round-trip with + * {@link #convert(double, int)}. * * @throws NullPointerException if coordinate is null * @throws IllegalArgumentException if the coordinate is not -- cgit v1.1