Formatting changes #611
Merged
Formatting changes #611
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- auto single-spacing can be switched off w/o losing KW formatting - KW formatting is performed always to either UPPER, CaMeL or lower case
- reworked "Code Layout" dialog - reworked config parsing for related options
- code export now usable w/o code layout
- looks wild, but it's mainly rewording of the command line help output and reordering CASEs to follow the help order - adapted parsing for the new KW related layout options - some minor bugfixes (wrong interpreted INSTR result and a type mismatch)
- updated existing tests to new behavior - added new tests for different KW cases
RhoSigma-QB64
added
bug
RhoSigma-QB64
requested review from
grymmjack,
mkilgore,
flukiluke,
SteveMcNeill,
a740g and
QB64Cobalt
- greetings to Murphy
- damn I hate this testing stuff - more fixing tests than fixing QB64pe - and every time you've to wait another 20 minutes
- and of course the search & replace devil wants to play with us too - no, that really isn't funny anymore
a740g
reviewed
Feb 20, 2025
a740g
approved these changes
Feb 20, 2025
RhoSigma-QB64
commented
Feb 20, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR addresses complains and suggestions by @QB64Cobalt on Discord https://discord.com/channels/975381912350752819/975389657212805160/1339150044942700606
The keywords formatting is now detached from the Auto Layout process, so one can switch off layouting to be able to space and align code on own desire, but still get proper UPPER, CaMeL or lower case formatting of the keywords.
For review I suggest to ignore whitespace changes especially in files qb64pe.bas and format.bas