Skip to content

Conversation

@nbriggs
Copy link
Collaborator

@nbriggs nbriggs commented Dec 21, 2020

All uses of the defined symbol, which are intended to be the same symbol, must be spelled in the same way.

@waywardmonkeys
Copy link
Contributor

The defined symbol being fixed here is actually RESWAPPEDCODESTREAM, so the commit text and PR title should be updated to reflect that.

@nbriggs nbriggs changed the title Fix typo #ifdef BYTESWAPPEDDCODE -- should be BYTESWAPPEDCODE Fix typo #ifdef RESWAPPEDDCODESTREAM -- should be RESWAPPEDCODESTREAM Dec 21, 2020
@nbriggs nbriggs force-pushed the fix-typo-RESWAPPEDD branch from 66ab676 to acc72da Compare December 21, 2020 16:12
@waywardmonkeys
Copy link
Contributor

LGTM

@nbriggs nbriggs merged commit b901038 into master Dec 22, 2020
@nbriggs nbriggs deleted the fix-typo-RESWAPPEDD branch December 27, 2020 05:09
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.

3 participants