[BUG - Release] Game__Debug__Win64 does not successfully build

[UE4SS]

Describe the bug GenericPlatformMath.hpp @ L410: error C2760: syntax error: 'UE_DEBUG_SECTION' was unexpected here; expected '{'

To Reproduce Using my command line: xmake f --profilerFlavor=Tracy --cxxflags="-DSTATS=0" -m "Game__Debug__Win64" -y && xmake Untested with the default command line or VS, but I'd be surprised if the problem didn't exist with those.

Expected behavior The Game__Debug__Win64 target is expected to successfully build.

Desktop (please complete the following information):

• OS: Win10 VM

[UE4SS]

I'm not sure how to properly fix this, but defining DO_ENSURE=0 bypasses the problem.

[Buckminsterfullerene02]

Perhaps fix could be bundled into #494?

Edit: Build is successful on that branch already. I installed into DRG and it works 👍

[UE4SS]

Perhaps fix could be bundled into #494?

Edit: Build is successful on that branch already. I installed into DRG and it works 👍

Did you test #494 with the latest Unreal submodule ? I'm pretty sure this problem was caused by a recent PR: https://github.com/Re-UE4SS/UEPseudo/pull/87 and #494 is based on an older commit so I just want to confirm that you tested with the latest Unreal submodule.

[Buckminsterfullerene02]

Perhaps fix could be bundled into #494? Edit: Build is successful on that branch already. I installed into DRG and it works 👍

Did you test #494 with the latest Unreal submodule ? I'm pretty sure this problem was caused by a recent PR: Re-UE4SS/UEPseudo#87 and #494 is based on an older commit so I just want to confirm that you tested with the latest Unreal submodule.

Okay I merged #494's UEPseudo branch in with main locally and the debug build does indeed fail. Did not notice that #494 is based off an older commit.

[bitonality]

Perhaps fix could be bundled into #494? Edit: Build is successful on that branch already. I installed into DRG and it works 👍

Did you test #494 with the latest Unreal submodule ? I'm pretty sure this problem was caused by a recent PR: Re-UE4SS/UEPseudo#87 and #494 is based on an older commit so I just want to confirm that you tested with the latest Unreal submodule.

Okay I merged #494's UEPseudo branch in with main locally and the debug build does indeed fail. Did not notice that #494 is based off an older commit.

I can try to update and see if the change is something easily bundle-able. Will know in a couple of hours

[bitonality]

#define UE_ENSURE_IMPL(Capture, Always, InExpression, ...) \
            (LIKELY(!!(InExpression)) || (DispatchCheckVerify<bool>([Capture] () FORCENOINLINE UE_DEBUG_SECTION \
            { \
                static bool bExecuted = false; \
                if ((!bExecuted || Always) && FPlatformMisc::IsEnsureAllowed()) \
                { \
                    bExecuted = true; \
                    FDebug::OptionallyLogFormattedEnsureMessageReturningFalse(true, #InExpression, __FILE__, __LINE__, ##__VA_ARGS__); \
                    if (!FPlatformMisc::IsDebuggerPresent()) \
                    { \
                        FPlatformMisc::PromptForRemoteDebugging(true); \
                        return false; \
                    } \
                    return true; \
                } \
                return false; \
            }) && ([] () { PLATFORM_BREAK(); } (), false)))

https://github.com/Re-UE4SS/UEPseudo/pull/87 introduced a breaking change whenever we compile in xxxDEBUGxxx mode due to a missing UE_DEBUG_SECTION def as well as unimplemented

FPlatformMisc:IsEnsureAllowed() FPlatformMisc:IsDebuggerPresent() FPlatformMisc::PromptForRemoteDebugging(bool)

I will not be making a fix in #494 since it's out of scope of that change.

[bitonality]

https://github.com/Re-UE4SS/UEPseudo/pull/91