[Bug 56139] New: vbscript: Dictionary does not allow storing Undefined
https://bugs.winehq.org/show_bug.cgi?id=56139 Bug ID: 56139 Summary: vbscript: Dictionary does not allow storing Undefined Product: Wine Version: 8.21 Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: vbscript Assignee: wine-bugs(a)winehq.org Reporter: francisdb(a)gmail.com Distribution: --- Example code below, min the commented KEY1 const Dim pinEvents Set pinEvents = CreateObject("Scripting.Dictionary") 'Const KEY1 = "one" pinEvents.Add KEY1, "hello" WScript.Echo pinEvents(KEY1) WScript.Echo pinEvents(Undefined) Prints on windows [Running] cscript //Nologo "test.vbs" hello hello [Done] exited with code=0 in 0.057 seconds On Linux we get no output wine cscript ./test.vbs This is because the code ends up in a FIXME section at https://gitlab.winehq.org/wine/wine/-/blob/master/dlls/scrrun/dictionary.c#L... -- 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=56139 --- Comment #1 from francisdb <francisdb(a)gmail.com> --- min -> mind -- 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=56139 francisdb <francisdb(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|vbscript: Dictionary does |vbscript: Dictionary does |not allow storing Undefined |not allow storing at key | |Undefined -- 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=56139 Jason Millard <jsm174(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jsm174(a)gmail.com -- 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=56139 Nikolay Sivov <bunglehead(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Ever confirmed|0 |1 Component|vbscript |scrrun Status|UNCONFIRMED |NEW --- Comment #2 from Nikolay Sivov <bunglehead(a)gmail.com> --- Thanks, I sent a fix for this: https://gitlab.winehq.org/wine/wine/-/merge_requests/4778 -- 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=56139 --- Comment #3 from francisdb <francisdb(a)gmail.com> --- That was quick, thanks! -- 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=56139 Nikolay Sivov <bunglehead(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|vbscript: Dictionary does |scrrun: Dictionary does not |not allow storing at key |allow storing at key |Undefined |Undefined -- 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=56139 --- Comment #4 from Nikolay Sivov <bunglehead(a)gmail.com> --- This's been merged now, will be out with wine 9.1, or you could build from current source to test. -- 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=56139 Robert Wilhelm <sloper42(a)yahoo.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |sloper42(a)yahoo.com Fixed by SHA1| |564b796e3605f8f622c47f3588b | |22deef4f323a4 Resolution|--- |FIXED Status|NEW |RESOLVED --- Comment #5 from Robert Wilhelm <sloper42(a)yahoo.com> --- This is fixed by Nikolay´s https://gitlab.winehq.org/wine/wine/-/commit/564b796e3605f8f622c47f3588b22de... -- 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=56139 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #6 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 9.3. -- 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=56139 Michael Stefaniuc <mstefani(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |9.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.
http://bugs.winehq.org/show_bug.cgi?id=56139 Michael Stefaniuc <mstefani(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|9.0.x |--- --- Comment #7 from Michael Stefaniuc <mstefani(a)winehq.org> --- Removing the 9.0.x milestone from bug fixes included in 9.0.1. -- 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 (1)
-
WineHQ Bugzilla