Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

unqualified call to 'std::move' UE5.1 and MacOS #375

Closed
beuman opened this issue May 14, 2023 · 8 comments
Closed

unqualified call to 'std::move' UE5.1 and MacOS #375

beuman opened this issue May 14, 2023 · 8 comments

Comments

@beuman
Copy link

beuman commented May 14, 2023

Hello, I am having the following issue when trying to build the project wich was working fine few days ago :

sio_message.h(208, 38): unqualified call to 'std::move' [-Werror,-Wunqualified-std-cast-call]

I get this error everywhere this function is called. Interesting to note there's also another error :

WindowsPlatformApplicationMisc.h(42, 5): 'WINDOWS_USE_FEATURE_APPLICATIONMISC_CLASS' is not defined, evaluates to 0 [-Werror,-Wundef]

I have latest macOS, Xcode (only used to generate project files), and Rider version. Using UE5.1.

@beuman
Copy link
Author

beuman commented May 14, 2023

From my Windows computer, when trying to package for Android I also get the following issue :
WindowsPlatformApplicationMisc.h(42, 5): 'WINDOWS_USE_FEATURE_APPLICATIONMISC_CLASS' is not defined, evaluates to 0 [-Werror,-Wundef]

@getnamo
Copy link
Owner

getnamo commented May 14, 2023

Which plugin version are you trying to build/package?

@beuman
Copy link
Author

beuman commented May 15, 2023

Using the master branch :

"FileVersion": 3,
"Version": 1,
"VersionName": "2.5.8",
"EngineVersion": "5.1"

@getnamo
Copy link
Owner

getnamo commented May 15, 2023

Looks like it complains about: https://github.com/getnamo/SocketIOClient-Unreal/blob/master/Source/SocketIOLib/Public/sio_message.h#L208 and the second line is not referenced inside the plugin so may not be related.

For the first error you can try to remove the move() wrapper around v so it should just be _v(v). I've never seen the issue before but it's possible a new compiler version for Mac might be more strict, this is typically a warning and not an error though and usually compiles through otherwise.

Edit: I believe the correct fix is std::move(v) to remove the warning.

@getnamo
Copy link
Owner

getnamo commented May 15, 2023

See if https://github.com/getnamo/SocketIOClient-Unreal/tree/Stdmove fixed the warning error for you

@staskjs
Copy link
Contributor

staskjs commented Jul 11, 2023

I had the same issue after upgrading from 5.1 to 5.2. Adding std:: to move helped.

@mikeseese
Copy link
Contributor

This error also happens for cross compiling for Linux. I implemented the changes in stdmove myself and that did fix the compilation errors.

@mikeseese mikeseese mentioned this issue Sep 6, 2023
@mikeseese
Copy link
Contributor

This was fixed in #376 and should be closed

@getnamo getnamo closed this as completed Oct 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants