Skip to content

Comments

Fix compiler error that fails to honour post-increment order in inlin…#3

Open
Bodmer wants to merge 2 commits intorichgel999:masterfrom
Bodmer:master
Open

Fix compiler error that fails to honour post-increment order in inlin…#3
Bodmer wants to merge 2 commits intorichgel999:masterfrom
Bodmer:master

Conversation

@Bodmer
Copy link

@Bodmer Bodmer commented Feb 24, 2022

The fix in consistent with other functions.

The problem was corupted images cause by the compiler inlining code and not putting post-inrement in correct place at end of inline code.

…ed code

The fix in consistent with other functions.

The problem was corupted images cause by the compiler inlining code and not putting post-inrement in correct place at end of inline code.
@FREEWING-JP
Copy link

It resolved this warning with Clang compiler !
Thank you !

picojpeg/picojpeg.c:1737:13: warning: unsequenced modification and access to 'pDstG' [-Wunsequenced]
      *pDstG++ = subAndClamp(pDstG[0], cbG);
            ^                ~~~~~

FREEWING-JP added a commit to FREEWING-JP/HackBGRT that referenced this pull request Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants