[Bug 43630] New: Altium Designer Installer - Richedit control shows rtf code instead of text
https://bugs.winehq.org/show_bug.cgi?id=43630 Bug ID: 43630 Summary: Altium Designer Installer - Richedit control shows rtf code instead of text Product: Wine Version: 2.15 Hardware: x86 URL: https://s3.amazonaws.com/altium-release-manager/Altium _Designer_17/AltiumDesignerSetup_17_1_6.exe OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: richedit Assignee: wine-bugs(a)winehq.org Reporter: dark.shadow4(a)web.de Distribution: --- Just download the attached installer, click 'Next' and change the language to German. It doesn't show German text, but richedit coded text. winetricks "riched20" works as workaround. Side note: Currently hard to test, since it's blocked by Bug 43608/Bug 43577. But it's still existent in wine-2.15 if you account for the regression. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=43630 Fabian Maurer <dark.shadow4(a)web.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |download Depends on| |43608, 43577 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=43630 --- Comment #1 from Fabian Maurer <dark.shadow4(a)web.de> --- Created attachment 59045 --> https://bugs.winehq.org/attachment.cgi?id=59045 Screenshot showing the bug -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=43630 --- Comment #2 from Nikolay Sivov <bunglehead(a)gmail.com> --- Please attach +richedit,+tid log. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=43630 --- Comment #3 from Fabian Maurer <dark.shadow4(a)web.de> --- Created attachment 59046 --> https://bugs.winehq.org/attachment.cgi?id=59046 Log with WINEDEBUG=+richedit,+tid Attached requested log. But I figured it would be easier to reproduce from the download itself. For the future, should I always attach logs? -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=43630 Bug 43630 depends on bug 43608, which changed state. Bug 43608 Summary: Altium Designer: Drop down menu entry not clickable since Wine 2.13 https://bugs.winehq.org/show_bug.cgi?id=43608 What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Resolution|--- |DUPLICATE -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=43630 alex freudenberg <afb(a)mailbox.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |afb(a)mailbox.org -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=43630 Bug 43630 depends on bug 43577, which changed state. Bug 43577 Summary: unit test: DBGrid inplace editor dropdown does not work and worked on version 2.0.2 https://bugs.winehq.org/show_bug.cgi?id=43577 What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Resolution|--- |FIXED -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=43630 Jeff D. Hanson <jhansonxi(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jhansonxi(a)gmail.com --- Comment #4 from Jeff D. Hanson <jhansonxi(a)gmail.com> --- Confirming on Wine 3.19 staging, Xubuntu 16.04 x86-64. It affects several versions of the installer and the affected language differs. Changing to a faulty language results in coded text, high CPU load, and large memory usage. https://s3.amazonaws.com/altium-install/Altium_Designer_14/AltiumDesignerSet... https://s3.amazonaws.com/altium-release-manager/Altium_Designer_15/AltiumDes... https://s3.amazonaws.com/altium-release-manager/Altium_Designer_16/AltiumDes... https://s3.amazonaws.com/altium-release-manager/Altium_Designer_17/AltiumDes... https://s3.amazonaws.com/altium-release-manager/Altium_Designer_18/AltiumDes... Affects German and Japanese EULAs on v17. Affects English and Japanese EULAs on v14 and v15. All EULA languages are normal on v16 and v18. Files are located in user temp directory during installation. Info from file command (v17): EULA_CHINESE.rtf: Rich Text Format data, version 1, unknown character set EULA_ENGLISH.rtf: Rich Text Format data, version 1, unknown character set EULA_GERMAN.rtf: Rich Text Format data, version 1, unknown character set EULA_JAPANESE.rtf: Rich Text Format data, version 1, unknown character set All v17 files load in native LibreOffice without problems. Some experimentation using the v15 and v17 German EULA RTF files showed that the RTF files are what trigger the problem. If I replace the German file on the v15 installer with the v17 version then the problem occurs on v15. If I replace the German file on v17 with the v15 version then the problem does not occur on v17. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=43630 --- Comment #5 from Jeff D. Hanson <jhansonxi(a)gmail.com> --- Created attachment 62743 --> https://bugs.winehq.org/attachment.cgi?id=62743 v15 German EULA RTF from v15 installer that loads normally. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=43630 --- Comment #6 from Jeff D. Hanson <jhansonxi(a)gmail.com> --- Created attachment 62744 --> https://bugs.winehq.org/attachment.cgi?id=62744 v17 German EULA RTF from v17 installer that fails to display. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=43630 --- Comment #7 from Fabian Maurer <dark.shadow4(a)web.de> --- Thanks for investigation. Also tested a bit today, and that's what I found: The program uses EM_STREAMIN to read in the file. When using the German rtf, the program calls EM_STREAMIN twice - first with rtf, but then it reads it as text (wParam is SF_TEXT). When adding a hack to skip the first iteration, it indeed does display fine. Problem is AFAIK due to a faulty rtf reader, still no clue why the program calls EM_STREAMIN again though. As said, it first does read the rtf properly, then throws everything away and reads it as plain text. Maybe some error state inside the rich edit control? Either way, I'm not entirely sure what causes the issue, but removing all occurrences of '\r' in the rtf file seems to solve the issue. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=43630 --- Comment #8 from Fabian Maurer <dark.shadow4(a)web.de> --- Okay, I think I've gotten to the bottom of it. I'll send in a test (and hopefully a fix) soon, the tests should explain it better than I could here. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=43630 --- Comment #9 from Fabian Maurer <dark.shadow4(a)web.de> --- Sent in a patch here: https://source.winehq.org/patches/data/153776 Short Explanation: The program uses EM_STREAMIN to send the text to the richedit control. Problem is, sometimes (randomly, AFAIK) it sends a buffer with a terminating \0. Wine interprets this as EOF, while on windows it just results in a space. Program notices that the parsing failed, and adds the whole rtf markup as plain text. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=43630 Fabian Maurer <dark.shadow4(a)web.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |4600169fd4eedcffb1dee3c146c | |d0b9cd56f619e Status|UNCONFIRMED |RESOLVED Resolution|--- |FIXED --- Comment #10 from Fabian Maurer <dark.shadow4(a)web.de> --- Fixed by https://source.winehq.org/git/wine.git/?a=commit;h=4600169fd4eedcffb1dee3c14... -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=43630 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #11 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 3.21. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=43630 Michael Stefaniuc <mstefani(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |3.0.x -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=43630 Michael Stefaniuc <mstefani(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|3.0.x |--- --- Comment #12 from Michael Stefaniuc <mstefani(a)winehq.org> --- Removing the 3.0.x milestone from bug fixes included in 3.0.5. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=43630 Anastasius Focht <focht(a)gmx.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |Installer URL|https://s3.amazonaws.com/al |https://web.archive.org/web |tium-release-manager/Altium |/20170829043647/https://s3. |_Designer_17/AltiumDesigner |amazonaws.com/altium-releas |Setup_17_1_6.exe |e-manager/Altium_Designer_1 | |7/AltiumDesignerSetup_17_1_ | |6.exe CC| |focht(a)gmx.net -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
participants (2)
-
wine-bugs＠winehq.org -
WineHQ Bugzilla