- User Since
- Apr 25 2017, 12:43 AM (21 w, 4 d)
Wed, Sep 20
This cannot be implemented where we currently call breakProtrudingToken(), since this function starts by 'creating' the BreakableToken and dealing with meany corner cases: so this should be done before in any case.
But the code at the end of breakProtrudingToken() is actually very similar to what you propose.
Tue, Sep 19
Remove Reflow from LineState, and perform the decision again during reconstruction phase.
I am still trying to get to the bottom of this assertion, any hint where to look for?
For one thing, we need to update the state to store the "decision" of the reflowing mode, which is performed only in DryRun=true mode, to avoid doing the computation multiple times.
Replace sorted list with hashtable.
Wed, Sep 13
Split diff: handle only statements in here, namespace macros will be moved to another one.
Fix review comments, before splitting the commit.
Reorder the functions to minimize diff.
Tue, Sep 12
Rebase to master to fix merge issue
Mon, Sep 11
Jul 28 2017
Jul 24 2017
Jul 23 2017
Address review commentsx
Address review comments
Jul 18 2017
So, there are two things in this patch: Statement macros and namespace macros. Lets break this out and handle them individually. They really aren't related that much.
Add more tests
Jul 17 2017
Jul 13 2017
Move code out of optimizer, directly into ContinuationIndenter::breakProtrudingToken(), to minimize impact on performance.
Comment reflowing of breakable items which break the limit will be slightly slower, since we now consider the two options; however this change has no impact on the performance of processing non-breakable items, and may actually increase the operformance of processing breakable items which do not need to be reflowed.
Jul 12 2017
I renamed the option to AlignAfterOperator, is it acceptable now?
(I also thought of UnindentOperator, but I think it is better to keep the notion of alignment)
Rename option to AlignAfterOperator
Jul 4 2017
Jun 30 2017
Jun 29 2017
Jun 28 2017
Jun 26 2017
Merge SplitEmptyClass/Struct/Union options into a single SplitEmptyRecord option.
Complete refactor to make the processing much more generic
I don't know if some style would want different styles, and I agree with you on principle; but since the brace wrapping is already configured for each kind of record, I choose to keep things consistent and have flags for each kind of record.
But I can merge the options, and keep only SplitEmptyRecord and SplitEmptyNamespace, if you really think it is better.
Jun 23 2017
Jun 21 2017
Fix according to review comments
Jun 20 2017
Enable merging records for Mozilla style
Rebase & fix indentation when newline is inserted after return or equal.
This style is used in the Skia project.
Jun 19 2017
Fix case where the content fits on a line, by wrapping after each comma, like this:
Fix review comments
Jun 16 2017
remove special case after assignment
Fix incorrect labelling as ObjCMethodExpr
Do not binpack designated array initializers, just like designated member initializers
Jun 15 2017
fix unit tests
This patch is probably not complete, though it works fine in all situations I could think of: nested initializers, "short" statement (properly merged), column layout is still performed when needed...
Jun 14 2017
Seems there is a special exception to BinPacking in case of designated member initializers.
Should BinPacking be disabled also in case of designated array initializers?
@krasimir : ping
Move tests that add or fix namespace end comments to NamespaceEndCommentsFixerTest.cpp
Make tests more readable
Jun 13 2017
- make "compacted" namespaces always add at most one level of indentation
- compact only namespaces which both start and end on consecutive lines
Jun 12 2017