Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compiler failure when M_USE_SMALL_NAME is disabled #132

Closed
BrianSipos opened this issue Feb 3, 2025 · 2 comments
Closed

Compiler failure when M_USE_SMALL_NAME is disabled #132

BrianSipos opened this issue Feb 3, 2025 · 2 comments
Assignees
Labels

Comments

@BrianSipos
Copy link

There may be other similar errors, but the first one I encountered with the latest main branch is in m-dict.h:172 the use of a bare ARRAY_DEF instead of M_ARRAY_DEF.

mlib/m-dict.h

Line 172 in e1a4d35

ARRAY_DEF(m_array_index, m_indexhash_t, M_POD_OPLIST)

It could be good to design and build the unit tests with M_USE_SMALL_NAME of zero to ensure there aren't other similar problems.

@P-p-H-d
Copy link
Owner

P-p-H-d commented Feb 3, 2025

Thanks.
I have detected a few more issue like this in the code and fixed them in master.
I'll code a test case that uses M_USE_SMALL_NAME.

@P-p-H-d P-p-H-d self-assigned this Feb 3, 2025
@P-p-H-d P-p-H-d added the bug label Feb 3, 2025
@P-p-H-d
Copy link
Owner

P-p-H-d commented Mar 1, 2025

I have written a test case to check for this which detected a few more issues in the code.
Everything has been fixed in master.
Thanks again for your bug report!

@P-p-H-d P-p-H-d closed this as completed Mar 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants