Exception raised while writting commit message

[angoca]

What steps will reproduce the problem?

1. I loaded JOSM with 8 GB memmory: javaws -Xmx8000M ~/Desktop/josm.jnlp
2. I mapped a restaurant (house + node)
3. I commit with a similar commit to a previous commit. The message included an accent on e.

What is the expected result?

What happens instead?

Please provide any additional information below. Attach a screenshot if possible.

URL:https://josm.openstreetmap.de/svn/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2022-04-04 22:26:00 +0200 (Mon, 04 Apr 2022)
Build-Date:2022-04-05 01:30:55
Revision:18427
Relative:URL: ^/trunk

Identification: JOSM/1.5 (18427 en) Mac OS X 10.15.7
OS Build number: Mac OS X 10.15.7 (19H1824)
Memory Usage: 5037 MB / 7200 MB (2462 MB allocated, but free)
Java version: 1.8.0_333-b02, Oracle Corporation, Java HotSpot(TM) 64-Bit Server VM
Look and Feel: com.apple.laf.AquaLookAndFeel
Screen: Display 69731270 1440×900 (scaling 1.00×1.00)
Maximum Screen Size: 1440×900
Best cursor sizes: 16×16→16×16, 32×32→32×32
System property file.encoding: UTF-8
System property sun.jnu.encoding: UTF-8
Locale info: en_CO
Numbers with default locale: 1234567890 -> 1234567890
VM arguments: [-Djava.security.policy=file:<java.home>/lib/security/javaws.policy, -DtrustProxy=true, -Djnlpx.home=<java.home>/bin, -Djava.security.manager, -Djnlpx.origFilenameArg=${HOME}/Desktop/josm.jnlp, -Djnlpx.remove=false, -Dsun.awt.warmup=true, -Djava.util.Arrays.useLegacyMergeSort=true, -Djnlpx.heapsize=4096m,8100m, -Dmacosx.jnlpx.dock.name=JOSM, -Dmacosx.jnlpx.dock.icon=${HOME}/Library/Application Support/Oracle/Java/Deployment/cache/6.0/25/4c122699-7a1c6b48.icns, -Djnlpx.jvm="<java.home>/bin/java"]
Dataset consistency test: No problems found

Plugins:
+ FastDraw (35893)
+ Mapillary (2.0.0-beta.6-3-g192a88c)
+ PicLayer (1.0.2)
+ apache-commons (35924)
+ apache-http (35924)
+ buildings_tools (35951)
+ continuosDownload (99)
+ ejml (35924)
+ geotools (35956)
+ imagery-xml-bounds (35893)
+ imagery_offset_db (35893)
+ jaxb (35952)
+ jna (35924)
+ jts (35924)
+ log4j (35924)
+ opendata (35910)
+ photo_geotagging (35933)
+ pt_assistant (1ff2e15)
+ reverter (35893)
+ tageditor (35893)
+ terracer (35893)
+ turnlanes-tagging (v0.0.5)
+ turnrestrictions (35893)
+ undelete (35893)
+ utilsplugin2 (35951)
+ wms-turbo-challenge2 (35939)

Tagging presets:
+ https://josm.openstreetmap.de/josmfile?page=Presets/SITP-Transmilenio&zip=1

Map paint styles:
+ https://josm.openstreetmap.de/josmfile?page=Styles/PublicTransport&zip=1
+ https://josm.openstreetmap.de/josmfile?page=Styles/NumberedCycleNodeNetworks&zip=1
+ resource://mapcss/Mapillary.mapcss

Validator rules:
+ https://raw.githubusercontent.com/MaptimeBogota/ColombianJosmValidator/main/Rules_Colombian-specific.validator.mapcss

Last errors/warnings:
- 36540.782 E: Handled by bug report queue: java.lang.NullPointerException
- 36540.790 E: Handled by bug report queue: java.lang.NullPointerException
- 36540.960 E: Handled by bug report queue: java.lang.NullPointerException
- 36552.220 E: Handled by bug report queue: java.lang.IllegalArgumentException: null XORColor
- 36552.781 E: Handled by bug report queue: java.lang.IllegalArgumentException: null XORColor
- 36553.096 E: Handled by bug report queue: java.lang.IllegalArgumentException: null XORColor
- 36553.388 E: Handled by bug report queue: java.lang.IllegalArgumentException: null XORColor
- 36553.616 E: Handled by bug report queue: java.lang.IllegalArgumentException: null XORColor
- 36553.863 E: Handled by bug report queue: java.lang.IllegalArgumentException: null XORColor
- 36557.746 E: Handled by bug report queue: java.lang.IllegalArgumentException: null XORColor



=== REPORTED CRASH DATA ===
BugReportExceptionHandler#handleException:
No data collected.

Warning issued by: BugReportExceptionHandler#handleException

=== STACK TRACE ===
Thread: AWT-EventQueue-2 (46) of javawsApplicationThreadGroup
java.lang.IllegalArgumentException: null XORColor
	at sun.java2d.SunGraphics2D.setXORMode(SunGraphics2D.java:2080)
	at javax.swing.text.JTextComponent$ComposedTextCaret.paint(JTextComponent.java:4972)
	at javax.swing.plaf.basic.BasicTextUI.paintSafely(BasicTextUI.java:748)
	at com.apple.laf.AquaTextFieldUI.paintSafely(AquaTextFieldUI.java:107)
	at javax.swing.plaf.basic.BasicTextUI.paint(BasicTextUI.java:887)
	at javax.swing.plaf.basic.BasicTextUI.update(BasicTextUI.java:866)
	at javax.swing.JComponent.paintComponent(JComponent.java:774)
	at org.openstreetmap.josm.gui.widgets.JosmTextField.paintComponent(JosmTextField.java:274)
	at javax.swing.JComponent.paint(JComponent.java:1050)
	at javax.swing.JComponent.paintChildren(JComponent.java:883)
	at javax.swing.JComponent.paint(JComponent.java:1059)
	at javax.swing.JComponent.paintChildren(JComponent.java:883)
	at javax.swing.JComponent.paint(JComponent.java:1059)
	at javax.swing.JComponent.paintToOffscreen(JComponent.java:5213)
	at javax.swing.RepaintManager$PaintManager.paintDoubleBuffered(RepaintManager.java:1579)
	at javax.swing.RepaintManager$PaintManager.paint(RepaintManager.java:1502)
	at javax.swing.RepaintManager.paint(RepaintManager.java:1272)
	at javax.swing.JComponent._paintImmediately(JComponent.java:5161)
	at javax.swing.JComponent.paintImmediately(JComponent.java:4972)
	at javax.swing.RepaintManager$4.run(RepaintManager.java:831)
	at javax.swing.RepaintManager$4.run(RepaintManager.java:814)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:74)
	at javax.swing.RepaintManager.paintDirtyRegions(RepaintManager.java:814)
	at javax.swing.RepaintManager.paintDirtyRegions(RepaintManager.java:789)
	at javax.swing.RepaintManager.prePaintDirtyRegions(RepaintManager.java:738)
	at javax.swing.RepaintManager.access$1200(RepaintManager.java:64)
	at javax.swing.RepaintManager$ProcessingRunnable.run(RepaintManager.java:1732)
	at java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:311)
	at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:758)
	at java.awt.EventQueue.access$500(EventQueue.java:97)
	at java.awt.EventQueue$3.run(EventQueue.java:709)
	at java.awt.EventQueue$3.run(EventQueue.java:703)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:74)
	at java.awt.EventQueue.dispatchEvent(EventQueue.java:728)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:205)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:116)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:109)
	at java.awt.WaitDispatchSupport$2.run(WaitDispatchSupport.java:190)
	at java.awt.WaitDispatchSupport$4.run(WaitDispatchSupport.java:235)
	at java.awt.WaitDispatchSupport$4.run(WaitDispatchSupport.java:233)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.awt.WaitDispatchSupport.enter(WaitDispatchSupport.java:233)
	at java.awt.Dialog.show(Dialog.java:1084)
	at java.awt.Component.show(Component.java:1671)
	at java.awt.Component.setVisible(Component.java:1623)
	at java.awt.Window.setVisible(Window.java:1014)
	at java.awt.Dialog.setVisible(Dialog.java:1005)
	at org.openstreetmap.josm.gui.io.UploadDialog.setVisible(UploadDialog.java:354)
	at org.openstreetmap.josm.actions.UploadAction.uploadData(UploadAction.java:245)
	at org.openstreetmap.josm.actions.UploadAction.actionPerformed(UploadAction.java:296)
	at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:2022)
	at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2348)
	at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:402)
	at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:259)
	at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:262)
	at java.awt.AWTEventMulticaster.mouseReleased(AWTEventMulticaster.java:289)
	at java.awt.Component.processMouseEvent(Component.java:6539)
	at javax.swing.JComponent.processMouseEvent(JComponent.java:3318)
	at java.awt.Component.processEvent(Component.java:6304)
	at java.awt.Container.processEvent(Container.java:2239)
	at java.awt.Component.dispatchEventImpl(Component.java:4889)
	at java.awt.Container.dispatchEventImpl(Container.java:2297)
	at java.awt.Component.dispatchEvent(Component.java:4711)
	at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4904)
	at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4535)
	at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4476)
	at java.awt.Container.dispatchEventImpl(Container.java:2283)
	at java.awt.Window.dispatchEventImpl(Window.java:2746)
	at java.awt.Component.dispatchEvent(Component.java:4711)
	at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:760)
	at java.awt.EventQueue.access$500(EventQueue.java:97)
	at java.awt.EventQueue$3.run(EventQueue.java:709)
	at java.awt.EventQueue$3.run(EventQueue.java:703)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:74)
	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:84)
	at java.awt.EventQueue$4.run(EventQueue.java:733)
	at java.awt.EventQueue$4.run(EventQueue.java:731)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:74)
	at java.awt.EventQueue.dispatchEvent(EventQueue.java:730)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:205)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:116)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:105)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:93)
	at java.awt.EventDispatchThread.run(EventDispatchThread.java:82)

[taylor.smock]

Java version: 1.8.0_333-b02, Oracle Corporation, Java HotSpot(TM) 64-Bit Server VM
VM arguments: [-Djava.security.policy=file:<java.home>/lib/security/javaws.policy, -DtrustProxy=true, -Djnlpx.home=<java.home>/bin, -Djava.security.manager, -Djnlpx.origFilenameArg=${HOME}/Desktop/josm.jnlp, -Djnlpx.remove=false, -Dsun.awt.warmup=true, -Djava.util.Arrays.useLegacyMergeSort=true, -Djnlpx.heapsize=4096m,8100m, -Dmacosx.jnlpx.dock.name=JOSM, -Dmacosx.jnlpx.dock.icon=${HOME}/Library/Application Support/Oracle/Java/Deployment/cache/6.0/25/4c122699-7a1c6b48.icns, -Djnlpx.jvm="<java.home>/bin/java"]

@angoca: Can you please update to ​OpenWebStart?

Anyway, I think

src/org/openstreetmap/josm/gui/io/UploadTextComponentValidator.java

@@ -3 +3 @@
 
 import static org.openstreetmap.josm.tools.I18n.tr;
 
+import java.awt.Color;
 import java.util.Collection;
 import java.util.Collections;
 import java.util.Objects;
@@ -21 +22 @@
  */
 abstract class UploadTextComponentValidator extends AbstractTextComponentValidator {
     private final JLabel feedback;
+    private final Color originalBackgroundColor;
     protected boolean uploadRejected;
 
     UploadTextComponentValidator(JTextComponent tc, JLabel feedback) {
         super(tc);
         this.feedback = feedback;
         this.feedback.setOpaque(true);
+        this.originalBackgroundColor = this.feedback.getBackground();
         validate();
     }
 
@@ -37 +40 @@
         if (!Objects.equals(msg, feedback.getText())) {
             feedback.setText(msg);
             feedback.setForeground(VALID_COLOR);
-            feedback.setBackground(null);
+            feedback.setBackground(this.originalBackgroundColor);
             feedback.setBorder(null);
         }
     }

will fix the problem, but I'll need to see if I can reproduce to make certain that the fix works.