[Dominik] [0.16.36] Ctrl-backspace triggers temporary stall.

This subforum contains all the issues which we already resolved.
Post Reply
jercos
Manual Inserter
Manual Inserter
Posts: 2
Joined: Wed May 03, 2017 3:32 am
Contact:

[Dominik] [0.16.36] Ctrl-backspace triggers temporary stall.

Post by jercos »

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.
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.
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.

jercos
Manual Inserter
Manual Inserter
Posts: 2
Joined: Wed May 03, 2017 3:32 am
Contact:

Re: [0.16.36] Ctrl-backspace triggers temporary stall.

Post by jercos »

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.

Dominik
Former Staff
Former Staff
Posts: 658
Joined: Sat Oct 12, 2013 9:08 am
Contact:

Re: [Dominik][0.16.36] Ctrl-backspace triggers temporary stall.

Post by Dominik »

Yeah, totally sounds like some open cycle.

User avatar
wheybags
Former Staff
Former Staff
Posts: 328
Joined: Fri Jun 02, 2017 1:50 pm
Contact:

Re: [Dominik] [0.16.36] Ctrl-backspace triggers temporary stall.

Post by wheybags »

This was already fixed in 0.16.37.
Thanks for the report!

Dominik
Former Staff
Former Staff
Posts: 658
Joined: Sat Oct 12, 2013 9:08 am
Contact:

Re: [Dominik] [0.16.36] Ctrl-backspace triggers temporary stall.

Post by Dominik »

A fine piece of work done today.

Post Reply

Return to “Resolved Problems and Bugs”