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

[Bug] #pragma pattern_limit is bugged #1618

Open
1 task
FireNX70 opened this issue Mar 25, 2024 · 5 comments
Open
1 task

[Bug] #pragma pattern_limit is bugged #1618

FireNX70 opened this issue Mar 25, 2024 · 5 comments
Labels
bug Something isn't working

Comments

@FireNX70
Copy link
Contributor

Operating System

Linux

What's the issue you encountered?

#pragma pattern_limit only appears to work on the first file opened. On other files, it is simply ignored. If the first file is closed, it still doesn't work for the other files. Closing all files but one will not make it work for the only file either.

How can the issue be reproduced?

Open at least two files and try to change the pattern limit (through #pragma pattern_limit) on any file besides the one that was opened first.

ImHex Version

1.33.2

ImHex Build Type

  • Nightly or built from sources

Installation type

Both AppImage and .deb package (Ubuntu 23.04, but it works fine on Debian 12)

Additional context?

Both this and #1617 are already broken on 1.33.1, so these aren't recent bugs.

@FireNX70 FireNX70 added the bug Something isn't working label Mar 25, 2024
@ZwipZwapZapony
Copy link

This also affects #pragma array_limit, #pragma debug, #pragma eval_depth, and likely any other pragma. This doesn't affect #define, so it doesn't just apply to the pre-processor in general. "Extras" > "Settings" > "General" > "Patterns" > "Sync pattern source code between providers" doesn't affect this.

@paxcut
Copy link
Contributor

paxcut commented Apr 9, 2024

I tested this bug on 1.33.0 (master branch) which is what the nightly gives for windows and I couldn't reproduce this, so it could be a release branch only bug or less likely a linux only problem.

@mandar1jn
Copy link

mandar1jn commented Apr 9, 2024

this is not a linux-specific issue as I have been running into the same thing on windows

@paxcut
Copy link
Contributor

paxcut commented Apr 9, 2024

It looks like it is a known problem on the release branch (1.33.1 and 1.33.2) the bug was fixed on master branch (1.33.0) but hasn't been applied to release branch according to posts on discord.

@FireNX70
Copy link
Contributor Author

I can confirm that #pragma pattern_limit is fixed in recent nightlies. I haven't tested the other pragmas mentioned by @ZwipZwapZapony (I'm assuming it's the same bug affecting these) and the fix hasn't made it into a release yet, so I'll leave the bug report open for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants