Details
-
Bug
-
Resolution: Fixed
-
P5
-
12, 24
-
b09
Description
There is a suspicious to the Math.ceil method with integer argument. Calling Math.ceil with integer argument is useless - it will just return the same passed value.
https://github.com/openjdk/jdk/blob/d10afa26e5c59475e49b353ed34e8e85d0615d92/src/java.desktop/share/classes/javax/swing/text/html/StyleSheet.java#L2394
javax.swing.text.html.StyleSheet.ListPainter#drawShape
int y = Math.max(ay, ay + (int)Math.ceil(ah/2));
'ah' is 'int'
Seems we should remove it.
https://github.com/openjdk/jdk/blob/d10afa26e5c59475e49b353ed34e8e85d0615d92/src/java.desktop/share/classes/javax/swing/text/html/StyleSheet.java#L2394
javax.swing.text.html.StyleSheet.ListPainter#drawShape
int y = Math.max(ay, ay + (int)Math.ceil(ah/2));
'ah' is 'int'
Seems we should remove it.
Attachments
Issue Links
- relates to
-
JDK-8202013 JEditorPane shows large HTML unordered list bullets
- Resolved
- links to
-
Commit(master) openjdk/jdk/7e21d4c1
-
Review(master) openjdk/jdk/20358