[PATCH v3 0/3] MR2727: hrtfapo: Add CreateHrtfApo() stub.
Needed by Britannia VR: Out of Your Mind. -- v3: hrtfapo: Add CreateHrtfApo() stub. hrtfapo: Add stub DLL. include: Add hrtfapoapi.idl file. https://gitlab.winehq.org/wine/wine/-/merge_requests/2727
From: Mohamad Al-Jaf <mohamadaljaf(a)gmail.com> --- include/Makefile.in | 1 + include/hrtfapoapi.idl | 108 +++++++++++++++++++++++++++++++++++++++++ 2 files changed, 109 insertions(+) create mode 100644 include/hrtfapoapi.idl diff --git a/include/Makefile.in b/include/Makefile.in index c957fed884c..0dae4cab2bf 100644 --- a/include/Makefile.in +++ b/include/Makefile.in @@ -340,6 +340,7 @@ SOURCES = \ highlevelmonitorconfigurationapi.h \ hlguids.h \ hlink.idl \ + hrtfapoapi.idl \ hstring.idl \ htiface.idl \ htiframe.idl \ diff --git a/include/hrtfapoapi.idl b/include/hrtfapoapi.idl new file mode 100644 index 00000000000..a62b968aeec --- /dev/null +++ b/include/hrtfapoapi.idl @@ -0,0 +1,108 @@ +/* + * Copyright (C) 2023 Mohamad Al-Jaf + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA + */ + +import "xapo.idl"; + +cpp_quote("#include <float.h>") + +cpp_quote("#define HRTF_MAX_GAIN_LIMIT 12.0f") +cpp_quote("#define HRTF_MIN_GAIN_LIMIT -96.0f") +cpp_quote("#define HRTF_MIN_UNITY_GAIN_DISTANCE 0.05f") +cpp_quote("#define HRTF_DEFAULT_UNITY_GAIN_DISTANCE 1.0f") +cpp_quote("#define HRTF_DEFAULT_CUTOFF_DISTANCE FLT_MAX") + +typedef struct HrtfPosition +{ + float x; + float y; + float z; +} HrtfPosition; + +typedef struct HrtfOrientation +{ + float element[9]; +} HrtfOrientation; + +typedef enum HrtfDirectivityType +{ + OmniDirectional = 0, + Cardioid, + Cone, +} HrtfDirectivityType; + +typedef enum HrtfEnvironment +{ + Small = 0, + Medium, + Large, + Outdoors, +} HrtfEnvironment; + +typedef struct HrtfDirectivity +{ + HrtfDirectivityType type; + float scaling; +} HrtfDirectivity; + +typedef struct HrtfDirectivityCardioid +{ + HrtfDirectivity directivity; + float order; +} HrtfDirectivityCardioid; + +typedef struct HrtfDirectivityCone +{ + HrtfDirectivity directivity; + float innerAngle; + float outerAngle; +} HrtfDirectivityCone; + +typedef enum HrtfDistanceDecayType +{ + NaturalDecay = 0, + CustomDecay, +} HrtfDistanceDecayType; + +typedef struct HrtfDistanceDecay +{ + HrtfDistanceDecayType type; + float maxGain; + float minGain; + float unityGainDistance; + float cutoffDistance; +} HrtfDistanceDecay; + +typedef struct HrtfApoInit +{ + HrtfDistanceDecay *distanceDecay; + HrtfDirectivity *directivity; +} HrtfApoInit; + +cpp_quote("STDAPI CreateHrtfApo(const HrtfApoInit *init, IXAPO **xapo);") + +[ + object, + uuid(15b3cd66-e9de-4464-b6e6-2bc3cf63d455) +] +interface IXAPOHrtfParameters : IUnknown +{ + HRESULT SetSourcePosition(const HrtfPosition *position); + HRESULT SetSourceOrientation(const HrtfOrientation *orientation); + HRESULT SetSourceGain(float gain); + HRESULT SetEnvironment(HrtfEnvironment environment); +} -- GitLab https://gitlab.winehq.org/wine/wine/-/merge_requests/2727
From: Mohamad Al-Jaf <mohamadaljaf(a)gmail.com> --- configure.ac | 1 + dlls/hrtfapo/Makefile.in | 1 + dlls/hrtfapo/hrtfapo.spec | 5 +++++ 3 files changed, 7 insertions(+) create mode 100644 dlls/hrtfapo/Makefile.in create mode 100644 dlls/hrtfapo/hrtfapo.spec diff --git a/configure.ac b/configure.ac index e96131953c4..fda8130de16 100644 --- a/configure.ac +++ b/configure.ac @@ -2629,6 +2629,7 @@ WINE_CONFIG_MAKEFILE(dlls/hlink) WINE_CONFIG_MAKEFILE(dlls/hlink/tests) WINE_CONFIG_MAKEFILE(dlls/hnetcfg) WINE_CONFIG_MAKEFILE(dlls/hnetcfg/tests) +WINE_CONFIG_MAKEFILE(dlls/hrtfapo) WINE_CONFIG_MAKEFILE(dlls/http.sys) WINE_CONFIG_MAKEFILE(dlls/httpapi) WINE_CONFIG_MAKEFILE(dlls/httpapi/tests) diff --git a/dlls/hrtfapo/Makefile.in b/dlls/hrtfapo/Makefile.in new file mode 100644 index 00000000000..c2a24658436 --- /dev/null +++ b/dlls/hrtfapo/Makefile.in @@ -0,0 +1 @@ +MODULE = hrtfapo.dll diff --git a/dlls/hrtfapo/hrtfapo.spec b/dlls/hrtfapo/hrtfapo.spec new file mode 100644 index 00000000000..d9395a66c7e --- /dev/null +++ b/dlls/hrtfapo/hrtfapo.spec @@ -0,0 +1,5 @@ +@ stub CreateHrtfApo +@ stub CreateHrtfApoWithDatasetType +@ stub CreateHrtfEngineFactory +@ stub IsHrtfApoAvailable +@ stub GetHrtfEngineMinFrameCount -- GitLab https://gitlab.winehq.org/wine/wine/-/merge_requests/2727
From: Mohamad Al-Jaf <mohamadaljaf(a)gmail.com> Needed by Britannia VR: Out of Your Mind. --- dlls/hrtfapo/Makefile.in | 3 +++ dlls/hrtfapo/hrtfapo.spec | 2 +- dlls/hrtfapo/main.c | 29 +++++++++++++++++++++++++++++ 3 files changed, 33 insertions(+), 1 deletion(-) create mode 100644 dlls/hrtfapo/main.c diff --git a/dlls/hrtfapo/Makefile.in b/dlls/hrtfapo/Makefile.in index c2a24658436..16a745e0fca 100644 --- a/dlls/hrtfapo/Makefile.in +++ b/dlls/hrtfapo/Makefile.in @@ -1 +1,4 @@ MODULE = hrtfapo.dll + +C_SRCS = \ + main.c diff --git a/dlls/hrtfapo/hrtfapo.spec b/dlls/hrtfapo/hrtfapo.spec index d9395a66c7e..42248518619 100644 --- a/dlls/hrtfapo/hrtfapo.spec +++ b/dlls/hrtfapo/hrtfapo.spec @@ -1,4 +1,4 @@ -@ stub CreateHrtfApo +@ stdcall CreateHrtfApo(ptr ptr) @ stub CreateHrtfApoWithDatasetType @ stub CreateHrtfEngineFactory @ stub IsHrtfApoAvailable diff --git a/dlls/hrtfapo/main.c b/dlls/hrtfapo/main.c new file mode 100644 index 00000000000..711f0bc9c29 --- /dev/null +++ b/dlls/hrtfapo/main.c @@ -0,0 +1,29 @@ +/* + * Copyright (C) 2023 Mohamad Al-Jaf + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA + */ + +#include "hrtfapoapi.h" + +#include "wine/debug.h" + +WINE_DEFAULT_DEBUG_CHANNEL(xaudio2); + +HRESULT WINAPI CreateHrtfApo(const HrtfApoInit *init, IXAPO **xapo) +{ + FIXME("(%p, %p): stub!\n", init, xapo); + return E_NOTIMPL; +} -- GitLab https://gitlab.winehq.org/wine/wine/-/merge_requests/2727
Hi, It looks like your patch introduced the new failures shown below. Please investigate and fix them before resubmitting your patch. If they are not new, fixing them anyway would help a lot. Otherwise please ask for the known failures list to be updated. The tests also ran into some preexisting test failures. If you know how to fix them that would be helpful. See the TestBot job for the details: The full results can be found at: https://testbot.winehq.org/JobDetails.pl?Key=132292 Your paranoid android. === debian11 (32 bit report) === mshtml: script.c:937: Test failed: L"/events.html: extern_http_script_rs = loading,loaded, expected loading,loaded,complete, or complete,"
On Mon May 1 04:10:47 2023 +0000, Zebediah Figura wrote:
You can declare this directly in the IDL. That doesn't work, it results in this error:
include/hrtfapoapi.idl:96:7: error: type 'STDAPI' not found in global namespace STDAPI CreateHrtfApo(const HrtfApoInit *init, IXAPO **xapo); ^ make[1]: *** [Makefile:162013: include/hrtfapoapi.h] Error 1 Even with just `HRESULT WINAPI`, which is missing `EXTERN_C`, it fails to build. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/2727#note_31609
On Mon May 1 05:15:20 2023 +0000, Mohamad Al-Jaf wrote:
changed this line in [version 3 of the diff](/wine/wine/-/merge_requests/2727/diffs?diff_id=44798&start_sha=9df20c6afeae1319043dde38de82646ab1c27508#fc268a76b18cbdc1e7786bcf5b0d56231811d975_22_23) I hope this style is how you wanted it.
-- https://gitlab.winehq.org/wine/wine/-/merge_requests/2727#note_31610
On Mon May 1 05:17:36 2023 +0000, Mohamad Al-Jaf wrote:
That doesn't work, it results in this error: include/hrtfapoapi.idl:96:7: error: type 'STDAPI' not found in global namespace STDAPI CreateHrtfApo(const HrtfApoInit *init, IXAPO **xapo); ^ make[1]: *** [Makefile:162013: include/hrtfapoapi.h] Error 1 Even with just `HRESULT WINAPI`, which is missing `EXTERN_C`, it fails to build. You can use __stdcall instead of WINAPI.
-- https://gitlab.winehq.org/wine/wine/-/merge_requests/2727#note_31611
participants (4)
-
Marvin -
Mohamad Al-Jaf -
Mohamad Al-Jaf (＠maljaf)
-
Zebediah Figura (＠zfigura)