PeachCompiler

PeachCompiler copied to clipboard

Resource Leak in some function

1

In file compiler.c at function compile_file there is a resource and memory leak in case you are able to create a process but unable to create a lexer. In the...

assism

EXPRESSION_LOGICAL_OR not in EXPRESSION_UNINHERITABLE_FLAGS

1

Hello Daniel, In the EXPRESSION_UNINHERITABLE_FLAGS macro function you added "EXPRESSION_LOGICAL_AND" but not "EXPRESSION_LOGICAL_OR". Is this a bug or intended. Kind Regards

fedi-nabli

Preprocessor elif not implemented

1

Hello Daniel, Just wanted to point out that you forgot to implement elif preprocessor token. Kind Regards,

fedi-nabli

For loop generation error when inside another function

1

I found a bug, when I place a for loop in another function besides main it thinks it needs a ; after the loop part and gives a segmentation fault....

fedi-nabli

Allocating size of pointer instead the size of struct at lecture 85

1

At the end of lecture 85, the compiler outputs this warning message: > ./parser.c:109:39: warning: 'memset' call operates on objects of type 'struct parser_history_switch' while the size is based on...

fedi-nabli

I found an issue when I call printf to output values of array items.

2

printf outputs addresses of array items, not values. Have you got a schedule to fix issues? C code: int printf(const char* str, ...); int main() { int i; int arr[3];...

zzdmfk

__ignore_typecheck__ in lexer is verified for "__ignore_typecheck"

1

In the lexer lecture when we created the `is_keyword` function, we check for `__ignore_typecheck` and in the parser we check for `__ignore_typecheck__` , was this a typo or made intentially?

fedi-nabli

Issue about calculating secondary data type size.

3

When I declared variables like below: short int a; long int b; Data type sizes were calculated wrong In function: ``` void parser_datatype_adjust_size_for_secondary(struct datatype* datatype, struct token* datatype_secondary_token) { if(!datatype_secondary_token)...

zzdmfk

An issue about secondary allowed data type in function parser_datatype_is_secondary_allowed_for_type().

2

bool parser_datatype_is_secondary_allowed_for_type(const char* type) { return S_EQ(type, "long") //i.e long int, long long || S_EQ(type, "short"); //short int || S_EQ(type, "double") //???? || S_EQ(type, "float"); //???? } double and float...

zzdmfk

Swtich modulas

1

Minor spelling fixes. I have ensured the code still compiles properly Feel free to simply close this pull request if you don't want it, I won't take any offense.

yegct