Skip to content

Refactor List operator[] to prevent compiler warnings#94

Merged
DanielaOrtner merged 1 commit into
RebelToolbox:mainfrom
madmiraal:fix-compiler-c4715
Aug 24, 2024
Merged

Refactor List operator[] to prevent compiler warnings#94
DanielaOrtner merged 1 commit into
RebelToolbox:mainfrom
madmiraal:fix-compiler-c4715

Conversation

@madmiraal
Copy link
Copy Markdown
Contributor

Prevents GCC compiler throwing: control reaches end of non-void function.
Prevents Visual Studio throwing C4715: not all control paths return a value:

RebelEngine\RebelEngine\core\list.h(450) : warning C4715: 'List<long,DefaultAllocator>::operator[]': not all control paths return a value

Rebel version of godotengine/godot#36097

Prevents GCC compiler throwing: control reaches end of
non-void function.
Prevents Visual Studio throwing C4715: not all control paths
return a value.
@madmiraal madmiraal added the PR Type: Bug Fix Your current game should now work as expected. label Aug 20, 2024
@DanielaOrtner DanielaOrtner self-requested a review August 24, 2024 10:21
Copy link
Copy Markdown
Contributor

@DanielaOrtner DanielaOrtner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At long last, this fix is in :)

@DanielaOrtner DanielaOrtner merged commit f5a7ec3 into RebelToolbox:main Aug 24, 2024
@madmiraal madmiraal deleted the fix-compiler-c4715 branch August 25, 2024 06:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR Type: Bug Fix Your current game should now work as expected.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants