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.

Sign up for GitHub

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

Jump to bottom

add -D_GLIBCXX_ASSERTIONS to CXXFLAGS #793

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

add -D_GLIBCXX_ASSERTIONS to CXXFLAGS #793

wants to merge 1 commit into from

Commits on Apr 4, 2023

  1. add -D_GLIBCXX_ASSERTIONS to CXXFLAGS 

    Many linux distributions have switched this on by default, but TR does
not have it on by default and so bugs are slipping through and hitting
users.  Turn on bounds checking by default so that bugs are found
quicker.

This WILL break users.  I'm sure of it.  It will cause program exit when
running broken code, example being robotastic#783
The performance penalty of this is minimal (none on modern distros where
this is already teh default) and the bounds checking causing failure is
better than simply running objectively wrong code.
    @ZeroChaos-
    ZeroChaos- committed Apr 4, 2023
    Configuration menu
    Copy the full SHA
    1bac505 View commit details
    Browse the repository at this point in the history