diff --git a/icu4j/src/com/ibm/icu/text/Bidi.java b/icu4j/src/com/ibm/icu/text/Bidi.java index 52d5eb9db95..f6792e323d2 100644 --- a/icu4j/src/com/ibm/icu/text/Bidi.java +++ b/icu4j/src/com/ibm/icu/text/Bidi.java @@ -1,4 +1,4 @@ -//##header J2SE15 +//##header J2SE14 /* ******************************************************************************* * Copyright (C) 2001-2008, International Business Machines @@ -4027,8 +4027,12 @@ public class Bidi { * *
* Note that in right-to-left runs, code like this places - * modifier letters before base characters and second surrogates - * before first ones. + * second surrogates before first ones (which is generally a bad idea) + * and combining characters before base characters. + *
+ * Use of {@link #writeReordered}
, optionally with the
+ * {@link #KEEP_BASE_COMBINING}
option, can be considered in
+ * order to avoid these issues.
*
* @param runIndex is the number of the run in visual order, in the
* range [0..countRuns()-1]
.
@@ -4078,8 +4082,11 @@ public class Bidi {
* such as {@link #OPTION_INSERT_MARKS} and {@link #OPTION_REMOVE_CONTROLS}.
*
* Note that in right-to-left runs, this mapping places
- * modifier letters before base characters and second surrogates
- * before first ones.
+ * second surrogates before first ones (which is generally a bad idea)
+ * and combining characters before base characters.
+ * Use of {@link #writeReordered}
, optionally with the
+ * {@link #KEEP_BASE_COMBINING}
option can be considered instead
+ * of using the mapping, in order to avoid these issues.
*
* @param logicalIndex is the index of a character in the text.
*
@@ -4176,6 +4183,13 @@ public class Bidi {
* REMOVE_BIDI_CONTROLS
, the visual positions returned may not
* be correct. It is advised to use, when possible, reordering options
* such as {@link #OPTION_INSERT_MARKS} and {@link #OPTION_REMOVE_CONTROLS}.
+ *
+ * Note that in right-to-left runs, this mapping places
+ * second surrogates before first ones (which is generally a bad idea)
+ * and combining characters before base characters.
+ * Use of {@link #writeReordered}
, optionally with the
+ * {@link #KEEP_BASE_COMBINING}
option can be considered instead
+ * of using the mapping, in order to avoid these issues.
*
* @return an array of getProcessedLength()
* indexes which will reflect the reordering of the characters.
@@ -4823,7 +4837,7 @@ public class Bidi {
* since the setPara()
call.