add debug build target #4664
Closed
add debug build target #4664
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…fix time display bugs, reverse readme changes
…g to graphics, consistency in naming gens
* newer generation soundproof heal bell interactions * Fix bugs and rewrite AnyPartyMemberStatused * add missing check, tests, clean up * fix ai code and rename battler for clarity
* Add followers to readme & update migration script readme * Apply suggestions from code review Co-authored-by: Eduardo Quezada <[email protected]> * Update README.md Co-authored-by: Eduardo Quezada <[email protected]> --------- Co-authored-by: Eduardo Quezada <[email protected]>
* Tera Shell AI calcs * agbcc
`src/data/map_group_count.h` is currently in `.gitignore`. However, git does not ignore a file if that file is already in the index. To actually ignore the file, the file must be deleted and that deletion must be committed. In addition, the Makefile did not previously clean `src/data/map_group_count.h`, nor know how to generate `src/data/map_group_count.h`. It did generate the file as a side-effect of a different rule, but if asked to generate, it did not know how to.
* Rename files to PSV * Rename functions and defines * Add missing newlines * Integrate feedback
…pr/upcoming/bwPopupMergeFix # Conflicts: # include/field_weather.h # include/rtc.h # include/strings.h # src/field_weather.c # src/map_name_popup.c # src/menu.c # src/rtc.c # src/strings.c
Co-authored-by: Bassoonian <[email protected]>
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adds a debug build target
Description
make debugwill build separate artifacts inbuild/(modern-)debugand enable-Ogand-g.for agbcc it will enable
-O1and-gAlso adds the option to run any build with
NOOPT=1remove any optimization flagssets all debug menu options in
config/debug.hto false by default. the debug target turns all of these options on by default.To achieve this, any preproc directive regarding these menus has been removed and integrated in the regular control flow.
Also fixes an error that reveals when building with debug (Werror=maybe-uninitialized)
Discord contact info
u8.salem