adesutherland / CREXX

REXX Language implementation
Other
13 stars 3 forks source link

context->debug_mode not defined in F0038 #279

Closed Peter-Jacob closed 2 years ago

Peter-Jacob commented 2 years ago

I just upgraded to the last F0038 version, and all my DEBUG macro calls don't compile with: Use of undeclared identifier 'context'

define DEBUG(...) if (context->debug_mode) fprintf(stderr, __VA_ARGS__)

For the moment I can bypass the problem by changing the macro, but what is the solution?

Peter-Jacob commented 2 years ago

it seems there is more broken:

C:\Users\PeterJ\CLionProjects\CREXX_F030\interpreter\rxvmintp.c:131:16: error: conflicting types for 'run'
 RX_FLATTEN int run(int num_modules, module *program, int argc, char *argv[],
                ^~~
In file included from C:\Users\PeterJ\CLionProjects\CREXX_F030\interpreter\rxvmintp.c:18:
C:\Users\PeterJ\CLionProjects\CREXX_F030\interpreter\rxvmintp.h:215:5: note: previous declaration of 'run' was here
 int run(rxvm_context *context, int argc, char *argv[]);
     ^~~
C:\Users\PeterJ\CLionProjects\CREXX_F030\interpreter\rxvmintp.c: In function 'run':
C:\Users\PeterJ\CLionProjects\CREXX_F030\interpreter\rxvmintp.c:296:63: warning: assignment discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
             program[mod_index].segment.binary[j].impl_address =
                                                               ^
...
...
rvjansen commented 2 years ago

Did not look at it, but in my book 'previous declaration was' always warrants a fresh git clone and checkout of the branch

rvjansen commented 2 years ago

yes, I can confirm, I pulled on a machine I did not use for a while and had exactly the same errors. Did a fresh git clone git@github.com:adesutherland/CREXX.git crexx-f0038, cd crexx-f0038, git checkout feature/f0038, and then in my crexx_release directory, cmake -G Ninja -DCMAKE_BUILD_TYPE=Release ../crexx-f0038 && ninja && ctest --verbose ... works like a charm.

rvjansen commented 2 years ago

tentatively closing this issue as a (sort-of) duplicate of https://github.com/adesutherland/CREXX/issues/273 - reopen if you disagree.