[Bug 42658] New: Playstation Now needs msvcr120.dll.? Alloc@Concurrency@@YAPAXI@Z
https://bugs.winehq.org/show_bug.cgi?id=42658 Bug ID: 42658 Summary: Playstation Now needs msvcr120.dll.?Alloc(a)Concurrency@@YAPAXI(a)Z Product: Wine Version: 2.3 Hardware: x86 URL: https://www.playstation.com/en-us/explore/playstationn ow/ OS: Linux Status: NEW Keywords: download Severity: normal Priority: P2 Component: msvcrt Assignee: wine-bugs(a)winehq.org Reporter: austinenglish(a)gmail.com Distribution: Gentoo Follow up to bug 39685 -- 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=42658 Gijs Vermeulen <acescopezz(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |acescopezz(a)gmail.com --- Comment #1 from Gijs Vermeulen <acescopezz(a)gmail.com> --- Created attachment 57594 --> https://bugs.winehq.org/attachment.cgi?id=57594 stub test Could you try this patch? (it's my first, so if it doesn't work, please forgive me) -- 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=42658 Gijs Vermeulen <acescopezz(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #57594|0 |1 is obsolete| | --- Comment #2 from Gijs Vermeulen <acescopezz(a)gmail.com> --- Created attachment 57595 --> https://bugs.winehq.org/attachment.cgi?id=57595 stub test v2 The other patch was obviously wrong, I changed it a bit. (Could still be wrong) -- 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=42658 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |patch --- Comment #3 from Austin English <austinenglish(a)gmail.com> --- (In reply to Gijs Vermeulen from comment #2)
Created attachment 57595 [details] stub test v2
The other patch was obviously wrong, I changed it a bit. (Could still be wrong)
Causes some compiler warnings: ../msvcrt/lock.c: In function 'Alloc': ../msvcrt/lock.c:616:5: warning: 'return' with a value, in function returning void return p; ^ but yes, it appears to work (at least, it gets to the same point as native now, bug 42649) -- 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=42658 --- Comment #4 from Gijs Vermeulen <acescopezz(a)gmail.com> --- (In reply to Austin English from comment #3)
(In reply to Gijs Vermeulen from comment #2)
Created attachment 57595 [details] stub test v2
The other patch was obviously wrong, I changed it a bit. (Could still be wrong)
Causes some compiler warnings: ../msvcrt/lock.c: In function 'Alloc': ../msvcrt/lock.c:616:5: warning: 'return' with a value, in function returning void return p; ^
but yes, it appears to work (at least, it gets to the same point as native now, bug 42649)
According to MSDN the function is void, but returns a pointer. I don't know how I could fix the warning. https://msdn.microsoft.com/en-us/library/dd492420(vs.100).aspx -- 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=42658 --- Comment #5 from Bruno Jesus <00cpxxx(a)gmail.com> --- (In reply to Gijs Vermeulen from comment #4)
According to MSDN the function is void, but returns a pointer. I don't know how I could fix the warning. https://msdn.microsoft.com/en-us/library/dd492420(vs.100).aspx
The function returns void pointer, not void. _CRTIMP void * __cdecl Alloc( size_t _NumBytes ); -- 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=42658 Gijs Vermeulen <acescopezz(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #57595|0 |1 is obsolete| | --- Comment #6 from Gijs Vermeulen <acescopezz(a)gmail.com> --- Created attachment 57597 --> https://bugs.winehq.org/attachment.cgi?id=57597 stub test v3 (In reply to Bruno Jesus from comment #5)
The function returns void pointer, not void.
_CRTIMP void * __cdecl Alloc( size_t _NumBytes );
Thank you, I changed this but reverted it like an idiot. This patch should take care of the warning. -- 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=42658 --- Comment #7 from Austin English <austinenglish(a)gmail.com> --- (In reply to Gijs Vermeulen from comment #6)
Created attachment 57597 [details] stub test v3
(In reply to Bruno Jesus from comment #5)
The function returns void pointer, not void.
_CRTIMP void * __cdecl Alloc( size_t _NumBytes );
Thank you, I changed this but reverted it like an idiot. This patch should take care of the warning.
Yep, warnings are gone, thanks. But then it has a different failure: msvcr120.dll.?_NewCollection(a)_AsyncTaskCollection@details(a)Concurrency@@SAPAV123(a)PAV_CancellationTokenState@23@@Z Others it needs: ./psnowlauncher.exe imports following stub symbols: msvcr120:??0invalid_operation(a)Concurrency@@QAE(a)PBD@Z msvcr120:?Free(a)Concurrency@@YAXPAX(a)Z msvcr120:?GetCurrentThreadId(a)platform@details(a)Concurrency@@YAJXZ msvcr120:?_Cancel(a)_TaskCollection@details(a)Concurrency@@QAEXXZ msvcr120:?_GetCurrentInlineDepth(a)_StackGuard@details(a)Concurrency@@CAAAIXZ msvcr120:?_NewCollection(a)_AsyncTaskCollection@details(a)Concurrency@@SAPAV123(a)PAV_CancellationTokenState@23@@Z msvcr120:?_Oversubscribe(a)_Context@details(a)Concurrency@@SAX_N(a)Z msvcr120:?_ReportUnobservedException(a)details@Concurrency@@YAXXZ msvcr120:?_RunAndWait(a)_TaskCollection@details(a)Concurrency@@QAG?AW4_TaskCollectionStatus(a)23@PAV_UnrealizedChore(a)23@@Z msvcr120:?_Schedule(a)_TaskCollection@details(a)Concurrency@@QAEXPAV_UnrealizedChore(a)23@@Z msvcr120:?_ScheduleTask(a)_CurrentScheduler@details(a)Concurrency@@SAXP6AXPAX(a)Z0@Z msvcr120:?__ExceptionPtrAssign@@YAXPAXPBX(a)Z msvcr120:__crtUnhandledException -- 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=42658 --- Comment #8 from Gijs Vermeulen <acescopezz(a)gmail.com> --- This bug should be fixed by: https://source.winehq.org/git/wine.git/commit/32a4bec9099676ce1ade3fbd287119... -- 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=42658 --- Comment #9 from Austin English <austinenglish(a)gmail.com> --- (In reply to Gijs Vermeulen from comment #8)
This bug should be fixed by: https://source.winehq.org/git/wine.git/commit/ 32a4bec9099676ce1ade3fbd2871195f9596686c
Yep, thanks. Next is bug 42680. -- 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=42658 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Fixed by SHA1| |32a4bec9099676ce1ade3fbd287 | |1195f9596686c Resolution|--- |FIXED --- Comment #10 from Austin English <austinenglish(a)gmail.com> --- (In reply to Austin English from comment #9)
(In reply to Gijs Vermeulen from comment #8)
This bug should be fixed by: https://source.winehq.org/git/wine.git/commit/ 32a4bec9099676ce1ade3fbd2871195f9596686c
Yep, thanks. Next is bug 42680.
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=42658 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 2.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=42658 Michael Stefaniuc <mstefani(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |2.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=42658 Michael Stefaniuc <mstefani(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|2.0.x |--- --- Comment #12 from Michael Stefaniuc <mstefani(a)winehq.org> --- Removing the 2.0.x milestone from bugs included in 2.0.2. -- 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)
-
wine-bugs＠winehq.org