WerWolv / EdiZon

💡 A homebrew save management, editing tool and memory trainer for Horizon (Nintendo Switch)
GNU General Public License v2.0
1.16k stars 109 forks source link

Build warnings & freeze issues using a selfmade config. #9

Closed borntohonk closed 6 years ago

borntohonk commented 6 years ago

Currently getting these warnings compiling the master branch.

lua_save_parser.cpp
aarch64-none-elf-g++ -MMD -MP -MF /home/ch/Downloads/EdiZon/build/lua_save_parser.d -g -Wall -O3 -ffunction-sections -march=armv8-a -mtune=cortex-a57 -mtp=soft -fPIE  -DVERSION_MAJOR=1 -DVERSION_MINOR=2 -DVERSION_MICRO=1 -I/home/ch/Downloads/EdiZon/include -I/home/ch/Downloads/EdiZon/include/lua -I/opt/devkitpro/portlibs/switch/include -I/opt/devkitpro/libnx/include -I/home/ch/Downloads/EdiZon/build -D__SWITCH__ -I/opt/devkitpro/libnx/external/bsd/include -fno-rtti -fexceptions -std=gnu++17 -c /home/ch/Downloads/EdiZon/source/lua_save_parser.cpp -o lua_save_parser.o 
EdiZon/source/lua_save_parser.cpp:25:13: warning: 'void stackDump(lua_State*)' defined but not used [-Wunused-function]
 static void stackDump (lua_State *L) {
             ^~~~~~~~~
EdiZon/source/theme.c: In function 'setTheme':
EdiZon/source/theme.c:25:19: warning: missing braces around initializer [-Wmissing-braces]
       currTheme = (theme_t) {
                   ^
EdiZon/source/theme.c:44:19: warning: missing braces around initializer [-Wmissing-braces]
       currTheme = (theme_t) {
                   ^

Aswell as I can't seem to get this working, it gets to the menu where to select save, but once selecting a save the console freezes up.

https://github.com/borntohonk/EdiZon_ConfigsAndScripts/commit/b34f516d360333f3e1a0841eaa44899afd60acc3

WerWolv commented 6 years ago

These are all just warnings and can be ignored. stackDump as for debugging Lua and the second warning with theme_t is a bug in GCC that causes these faulty warnings.

The freeze gets caused because you're missing the category tag on all of your items. Check out the Wiki to find out how it works.

WerWolv commented 6 years ago

@borntohonk I just fixed the config and it works perfectly now. Thanks for the offsets and values! Modified version: https://transfer.sh/KqCWW/01007EF00011E000.json

borntohonk commented 6 years ago

o-oh well i PR'd a functional one, but yours is more descriptive.