[Bug 37977] New: Multiple applications using VBScript/JScript need scripting host 'WScript.Quit' implementation

https://bugs.winehq.org/show_bug.cgi?id=37977

Bug ID: 37977
           Summary: Multiple applications using VBScript/JScript need
                    scripting host 'WScript.Quit' implementation
           Product: Wine
           Version: 1.7.35
          Hardware: x86
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: wscript
          Assignee: [e-mail]
          Reporter: [e-mail]
      Distribution: ---

Hello folks,

similar to bug 36466

Actually not implementing scripting host 'Wscript.Quit' has more severe
consequences since many scripts use 'On Error Resume Next'.

Those simply continue execution with intended behaviour (in various cases
ending up in a crash)

--- snip ---
On Error Resume Next

If WScript.Arguments.Count <> 1 Then
 Wscript.Echo "need one argument!"
 Wscript.Quit 127
End If

Wscript.Echo "not reached"
--- snip ---

Regards

[Bug 37977] Multiple applications using VBScript/JScript need scripting host 'WScript.Quit' implementation

https://bugs.winehq.org/show_bug.cgi?id=37977

Anastasius Focht  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |source

--- Comment #1 from Anastasius Focht  ---
Hello folks,

revisiting, still present.

https://source.winehq.org/git/wine.git/blob/d6f527af14d0e9f9ea629024433e3735...

--- snip ---
 225 static HRESULT WINAPI Host_Quit(IHost *iface, int ExitCode)
 226 {
 227     WINE_FIXME("(%d)\n", ExitCode);
 228     return E_NOTIMPL;
 229 }
--- snip ---

Correction to my initial comment: "intended behaviour" -> "unintended
behaviour" if it was not obvious ;-)

$ wine --version
wine-1.7.54-179-ga0d0d0d

Regards

[Bug 37977] Multiple applications using VBScript/ JScript need scripting host 'WScript.Quit' implementation

https://bugs.winehq.org/show_bug.cgi?id=37977

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #2 from [e-mail] ---
https://source.winehq.org/patches/data/123760

Has a patch that didnt merge in current form.

[Bug 37977] Multiple applications using VBScript/ JScript need scripting host 'WScript.Quit' implementation

https://bugs.winehq.org/show_bug.cgi?id=37977

Jacek Caban  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
      Fixed by SHA1|                            |341bb46fc693273b357842bdbe3
                   |                            |260c017aa0318
             Status|NEW                         |RESOLVED
                 CC|                            |[e-mail]
         Resolution|---                         |FIXED

--- Comment #3 from Jacek Caban  ---
The fix is in Git.

[Bug 37977] Multiple applications using VBScript/ JScript need scripting host 'WScript.Quit' implementation

https://bugs.winehq.org/show_bug.cgi?id=37977

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #4 from Alexandre Julliard  ---
Closing bugs fixed in 1.9.15.