The new directive doesn't seem to work here. Looks like DefaultLevel is
never
changed:
core/parseconfig.inc:pc_default: cmp ax,[DefaultLevel]
core/parseconfig.inc:DefaultLevel dw 0 ; The current
level of default
core/ui.inc: cmp word [DefaultLevel],1 ; Active UI statement?
- Sebastian
diff --git a/core/parseconfig.inc b/core/parseconfig.inc
index 2fb26fd..fcf18d9 100644
--- a/core/parseconfig.inc
+++ b/core/parseconfig.inc
@@ -22,6 +22,7 @@
;
pc_default: cmp ax,[DefaultLevel]
jb .skip
+ mov [DefaultLevel],ax
mov di,default_cmd
call getline
mov byte [di-1],0 ; null-terminate
Sebastian Herbszt wrote:> The new directive doesn't seem to work here. Looks like DefaultLevel is never > changed: > > core/parseconfig.inc:pc_default: cmp ax,[DefaultLevel] > core/parseconfig.inc:DefaultLevel dw 0 ; The current level of default > core/ui.inc: cmp word [DefaultLevel],1 ; Active UI statement? >Indeed. I didn't mean to push it out on the main branch yet, actually; and given that the kkpxe stuff seems broken, too, I'm probably going to reset the main branch and clean this stuff up. The UI work will continue on the "ui" branch for now. -hpa