Building on Windows

[lucasvinbr]

After setting up the symlinks, I've successfully built and run the project on Linux, but I've run into some issues when building with visual studio on Windows:

• I get this C1128 error when compiling BindFunctions_Urho3d.cpp , but adding the /bigobj command line argument solves it. Would it be necessary to break the generated file into multiple ones in this case?
• I had to add a `#ifdef SendMessage

  undef SendMessage

  endif` to the BindConnection.cpp file, just like in Urho's Connection.cpp file, because apparently Windows has that define overriding things. It could be added via the binding generator, right?

• After solving the points above, I'm getting multiple messages No constructor could take the source type, or constructor overload resolution was ambiguous in some files, followed by C2440 errors in those same files and lines. -- The error message in all of them is can't convert from "unkown" to "sol::this_state". -- Here's my build log; it's in portuguese, but the line numbers are there, so maybe it could help: build-out.txt . -- The one pattern I've noticed is that the overloads the compiler complained about seem to be the ones that have parameters with default values for them, like Node's CreateChild with CreateMode mode = REPLICATED

Thanks!

[SirNate0]

Thank you for attempting to build it for Windows.

1. It seems to me that just enabling /bigobj might be better. I believe Microsoft does the same for UWP apps because their template heavy code also exceeds the limit.
2. Yes, it can be added to the binding generator.
3. I think your final point is probably the answer, though I've only looked at a single function. I believe the issue comes down to this line (below), though the error message isn't very clear and points to the end of the lambda instead of the beginning. I think it's probably because there is a default argument before the sol::this_state argument (that has no default). Possibly the presence of the default argument at all. In any case, I think removing the default arguments should be doable and advisable.

https://github.com/SirNate0/U3D-Doxygened-Lua-Bindings/blob/4929104893196c05c5b6cfbbb5b30133bd97306f/generator/generated/Graphics/BindCustomGeometry.cpp#L117

[lucasvinbr]

Alright, I've finally got it to build on Windows! I removed the default arguments, added /bigobj, and then had to add skipping of members with no name, because BindShaderParameter was being generated with 2 empty member variables (I think it was because of the "union" keyword being used there?). Would it be interesting to add these changes as a PR?

[SirNate0]

I'd appreciate that, please do.