Affects all text fields tested, is reliable (not intermittent), fields tested are the console; inventory search; blueprint string import; and locomotive color. Affects at least steam-distributed binaries on Windows 7 x64.
To reproduce, with text cursor focus in any empty text field, press control-backspace (delete previous word). The field must be completely clear, control-backspace when the cursor is at the left of a field containing some text (even a single space) performs as expected rather than triggering the bug.
The result? The client takes 100% CPU for several seconds (seems variable between around 3 to 10) and many times will stall long enough to get dropped from the server. Updates are not painted during the stall, as shown in screenshot.
I expect the client to do nothing on receiving control-backspace on an empty text field.
[Dominik] [0.16.36] Ctrl-backspace triggers temporary stall.
[Dominik] [0.16.36] Ctrl-backspace triggers temporary stall.
- Attachments
-
- factorio-control-backspace.log
- Log, though nothing appears in it at the event itself, this clearly indicates I'm not using any mods I guess.
- (13.87 KiB) Downloaded 45 times
-
- Screenshot of smearing task manager over the stalling Factorio instance.
- obligatory.png (696.63 KiB) Viewed 1311 times
Last edited by jercos on Mon Apr 23, 2018 8:59 am, edited 1 time in total.
Re: [0.16.36] Ctrl-backspace triggers temporary stall.
This worries me because of the implication that it might be under-running the text buffer searching for a word-ending or something... on some systems it could crash instead of stalling if that's the case.
Re: [Dominik][0.16.36] Ctrl-backspace triggers temporary stall.
Yeah, totally sounds like some open cycle.
Re: [Dominik] [0.16.36] Ctrl-backspace triggers temporary stall.
This was already fixed in 0.16.37.
Thanks for the report!
Thanks for the report!
Re: [Dominik] [0.16.36] Ctrl-backspace triggers temporary stall.
A fine piece of work done today.