gbdev/rgbds
1
0
Fork 0
mirror of https://github.com/gbdev/rgbds.git synced 2025-11-20 18:22:07 +00:00
Code Issues Packages Projects Releases Wiki Activity

Add a test case for overlapping IF/ENDC and REPT/ENDR

Browse Source 
This trips an asserton in the Rust rewrite because of its different
conditional stack design
This commit is contained in:
Rangi42
2025-09-25 11:39:49 -04:00
committed by Rangi
parent 0670c03bc2
commit 96b953fe51
4 changed files with 13 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
test/asm/misleading-indentation.asm Normal file
View File

@@ -0,0 +1,4 @@
IF 1
REPT 1
ENDC ; misleading indentation!
ENDR

2
test/asm/misleading-indentation.err Normal file
View File

@@ -0,0 +1,2 @@
FATAL: Found `ENDC` outside of a conditional (not after an `IF`/`ELIF`/`ELSE` block)
at misleading-indentation.asm::REPT~1(3) <- misleading-indentation.asm(2)

8
test/asm/test.sh
View File

@@ -62,7 +62,7 @@ fi
for i in *.asm notexist.asm; do
flags=${i%.asm}.flags
RGBASMFLAGS="-Weverything -B collapse"
RGBASMFLAGS="-Weverything -Bcollapse"
if [ -f "$flags" ]; then
RGBASMFLAGS="$RGBASMFLAGS $(head -n 1 "$flags")" # Allow other lines to serve as comments
fi
@@ -163,7 +163,7 @@ done
# These tests do their own thing
i="continues-after-missing-include"
RGBASMFLAGS="-Weverything -B collapse -M - -MG -MC"
RGBASMFLAGS="-Weverything -Bcollapse -M - -MG -MC"
# Piping the .asm file to rgbasm would not make sense for dependency generation,
# so just test the normal variant
(( tests++ ))
@@ -193,9 +193,9 @@ i="state-file"
if which cygpath &>/dev/null; then
# MinGW translates path names before passing them as command-line arguments,
# but does not do so when they are prefixed, so we have to do it ourselves.
RGBASMFLAGS="-Weverything -B collapse -s all:$(cygpath -w "$o")"
RGBASMFLAGS="-Weverything -Bcollapse -s all:$(cygpath -w "$o")"
else
RGBASMFLAGS="-Weverything -B collapse -s all:$o"
RGBASMFLAGS="-Weverything -Bcollapse -s all:$o"
fi
for variant in '' '.pipe'; do
(( tests++ ))

6
test/link/test.sh
View File

@@ -73,7 +73,7 @@ tryCmpRomSize () {
}
rgblinkQuiet () {
out="$(env "$RGBLINK" -Weverything -B collapse "$@")" || return $?
out="$(env "$RGBLINK" -Weverything -Bcollapse "$@")" || return $?
if [[ -n "$out" ]]; then
echo "$bold${red}Linking shouldn't produce anything on stdout!${rescolors}${resbold}"
false
@@ -266,8 +266,8 @@ evaluateTest
test="pipeline"
startTest
continueTest
("$RGBASM" -Weverything -B collapse -o - - | \
"$RGBLINK" -Weverything -B collapse -o - - | \
("$RGBASM" -Weverything -Bcollapse -o - - | \
"$RGBLINK" -Weverything -Bcollapse -o - - | \
"$RGBFIX" -Weverything -v -p 0xff -) < "$test"/a.asm > "$gbtemp"
# This test does not trim its output with 'dd' because it needs to verify the correct output size
tryCmp "$test"/out.gb "$gbtemp"