Nebukam / PCGExtendedToolkit

PCGEx is a free (libre) Unreal 5 plugin that expands PCG capabilities. It offers a variety of high-performance nodes; with an edge for building relational graphs (Freeform, Delaunay, Voronoi, MST etc) and pathfinding; and much more.
MIT License
91 stars 10 forks source link

linux build issue #44

Closed tychotic closed 1 month ago

tychotic commented 1 month ago

Received this error building for linux. Is it a known issue? Any reason not to make the fix suggested in the error messsage? ("use reference type...")

^~~~~~~~~~~~~~~ 1 error generated. [144/169] Compile Module.ShooterCoreRuntime.cpp [145/169] Compile Module.PCGExtendedToolkit.6.cpp In file included from /src/game/Plugins/PCGExtendedToolkit/Intermediate/Build/Linux/x64/UnrealEditor/Development/PCGExtendedToolkit/Module.PCGExtendedToolkit.6.cpp:30: /src/game/Plugins/PCGExtendedToolkit/Source/PCGExtendedToolkit/Private/Data/PCGExGraphDefinition.cpp:248:33: error: loop variable 'Socket' creates a copy from type 'const PCGExGraph::FSocket' [-Werror,-Wrange-loop-construct] for (const PCGExGraph::FSocket Socket : SocketMapping->Sockets) ^ /src/game/Plugins/PCGExtendedToolkit/Source/PCGExtendedToolkit/Private/Data/PCGExGraphDefinition.cpp:248:7: note: use reference type 'const PCGExGraph::FSocket &' to prevent copying for (const PCGExGraph::FSocket Socket : SocketMapping->Sockets) ^~~~~~~~~~ &

Nebukam commented 1 month ago

Hey @tychotic ! Nope, never tried to compile on Linux, but the error is clearly an oversight on my part; no idea why Rider didn't catch it since it's usually keen on that kind of stuff :/

Thanks for the report <3

tychotic commented 1 month ago

Thanks for the fast fix!! Huge fan, learning a lot from your code here.

On Tue, May 28, 2024 at 9:56 PM Tim @.***> wrote:

Hey @tychotic https://github.com/tychotic ! Nope, never tried to compile on Linux, but the error is clearly an oversight on my part; no idea why Rider didn't catch it since it's usually keen on that kind of stuff :/

Thanks for the report <3

— Reply to this email directly, view it on GitHub https://github.com/Nebukam/PCGExtendedToolkit/issues/44#issuecomment-2136511541, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUHSCL5UQNQXE3UDDSK5BHDZEVNWPAVCNFSM6AAAAABIOEMPLWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZWGUYTCNJUGE . You are receiving this because you were mentioned.Message ID: @.***>