Modify

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#3371 closed defect (fixed)

Fatal freeze trying to enter a changeset comment

Reported by: Claudius Henrichs <claudius.h@…> Owned by: team
Priority: critical Component: Core
Version: tested Keywords:
Cc:

Description

How to reproduce:
Prerequisite: Have some changesets uploaded with comments before

  • Try to upload a changeset
  • Delete the prefilled changeset comment
  • Enter The first letter of a previously used changeset comment

Result:

  • Complete UI freeze
  • Java VM CPU usage peaks to 100%
  • Java VM memore usage peaks to all available memory

(I was using farsi UTF8 characters in some of the changeset comments. Not sure if this is related.)

Version 2021 - Present in last tested rev 1981 as well
Plugins:
waydownloader 16963
wmsplugin 17278
remotecontrol 16965

Attachments (2)

faulty_upload_comment_history (979 bytes) - added by Claudius Henrichs <claudius.h@…> 4 years ago.
faulty line of JOSM preferences
josm.debug.txt (3.6 KB) - added by katpatuka 4 years ago.
josm debug output when editing relation with Thai characters

Download all attachments as: .zip

Change History (17)

comment:1 Changed 4 years ago by Gubaer

  • Owner changed from team to Claudius Henrichs <claudius.h@…>
  • Status changed from new to needinfo

sound like #2542 which has been fixed months ago.

What's your environment? OS? WM? Java VM?

comment:2 Changed 4 years ago by Gubaer

Is this related to #2747? Are you working with russian characters?

comment:3 Changed 4 years ago by Claudius Henrichs <claudius.h@…>

WindowsVista 64bit
Sun Java 1.6.0_15

Unfortunately I cannot kill the locked/looped JOSM to get more information. What I tried was starting JOSM from cmd and hitting Ctrl+Break with the frozen JOSM window focussed which did nothing.

I've been using farsi (similar to arabic script) characters in my comments before.

comment:4 Changed 4 years ago by Claudius Henrichs <claudius.h@…>

Deleting upload.comment.history in my JOSM Preferences "solved" this issue for me.

I'm attaching the content of this line here so you can reproduce. I think this bug is a major issue as it's affecting all those emerging OSM regions that are using UTF-8.

Changed 4 years ago by Claudius Henrichs <claudius.h@…>

faulty line of JOSM preferences

comment:5 Changed 4 years ago by stoecker

Ticket #3136 has been marked as a duplicate of this ticket.

comment:6 Changed 4 years ago by stoecker

  • Owner changed from Claudius Henrichs <claudius.h@…> to team
  • Status changed from needinfo to new

comment:7 Changed 4 years ago by katpatuka

confirmed. I had the same problem with Thai characters in the comment field - after deleting upload.comment.history no freezing anymore.

Changed 4 years ago by katpatuka

josm debug output when editing relation with Thai characters

comment:8 Changed 4 years ago by jttt

I can confirm the freeze with java 1.6.0_15 and faulty_upload_comment_history

comment:9 Changed 4 years ago by jttt

It looks like freeze is caused by EventConsumingPlainDocument in SuggestingJHistoryComboBox. When I don't enable consuming events then josm doesn't freeze.

comment:10 Changed 4 years ago by stoecker

Can't we replace the comment history with the AutoComplete code, so we don't have two places with equal code?

comment:11 Changed 4 years ago by jttt

I didn't know we have two autocompletion combos. I think they can be merged, SuggestingJHistoryComboBox has some extra features (or rather convenience methods) but AutoCompleteComboBox works fine with multilanguagual texts (tested on faulty_upload_comment_history)

comment:12 Changed 4 years ago by stoecker

What extra stuff does SuggestingJHistoryComboBox have?

AutoComplete has one problem with multi-sign input methods. There is a bu report about this. But otherwise it works very good after the last rework I did (which was a very hard job to get all working as expected).

comment:13 Changed 4 years ago by Gubaer

(In [2218]) see #3371: Fatal freeze trying to enter a changeset comment

comment:14 Changed 4 years ago by Gubaer

  • Resolution set to fixed
  • Status changed from new to closed

I've removed SuggestingJHistoryComboBox. It's replaced by a HistoryComboBox which combines the features of AutoCompletionComboBox wit the features of the ComboBoxHistory.

In my env, JOSM doesn't freeze any more with the faulty upload comment history.

comment:15 Changed 4 years ago by Claudius Henrichs <claudius.h@…>

Played around a bit and it seems to be fixed finally in v2221. Thanks a lot.

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed .
as The resolution will be set. Next status will be 'closed'.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.