test/regression/115.stmt - Issue 846133002: Put conditional operators on the left.

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: test/regression/115.stmt

Issue 846133002: Put conditional operators on the left. (Closed) Base URL: https://github.com/dart-lang/dart_style.git@master

Patch Set: Update version. Created 5 years, 11 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View unified diff | Download patch

« no previous file with comments | « test/regression/112.stmt ('k') | test/regression/122.unit » ('j') | no next file with comments »

OLD	NEW
1 >>>	1 >>>

2 assert(	2 assert(

3 // Indexing send-sets have an argument for the index.	3 // Indexing send-sets have an argument for the index.

4 (selector.isIndexSet ? 1 : 0) +	4 (selector.isIndexSet ? 1 : 0) +

5 // Non-increment send-sets have one more argument.	5 // Non-increment send-sets have one more argument.

6 (ast.Operator.INCREMENT_OPERATORS.contains(op.source) ? 0 :	6 (ast.Operator.INCREMENT_OPERATORS.contains(op.source) ? 0 :

7 1) == node.argumentCount());	7 1) == node.argumentCount());

8 <<<	8 <<<

9 assert(	9 assert(

10 // Indexing send-sets have an argument for the index.	10 // Indexing send-sets have an argument for the index.

11 (selector.isIndexSet ? 1 : 0) +	11 (selector.isIndexSet ? 1 : 0) +

12 // Non-increment send-sets have one more argument.	12 // Non-increment send-sets have one more argument.

13 (ast.Operator.INCREMENT_OPERATORS.contains(op.source) ? 0 : 1) ==	13 (ast.Operator.INCREMENT_OPERATORS.contains(op.source) ? 0 : 1) ==

14 node.argumentCount());	14 node.argumentCount());

15 >>> (indent 4)	15 >>> (indent 4)

16 assert(	16 assert(

17 // Indexing send-sets have an argument for the index.	17 // Indexing send-sets have an argument for the index.

18 (selector.isIndexSet ? 1 : 0) +	18 (selector.isIndexSet ? 1 : 0) +

19 // Non-increment send-sets have one more argument.	19 // Non-increment send-sets have one more argument.

20 (ast.Operator.INCREMENT_OPERATORS.contains(op.source) ? 0 :	20 (ast.Operator.INCREMENT_OPERATORS.contains(op.source) ? 0 :

21 1) == node.argumentCount());	21 1) == node.argumentCount());

22 <<<	22 <<<

23 assert(	23 assert(

24 // Indexing send-sets have an argument for the index.	24 // Indexing send-sets have an argument for the index.

25 (selector.isIndexSet ? 1 : 0) +	25 (selector.isIndexSet ? 1 : 0) +

26 // Non-increment send-sets have one more argument.	26 // Non-increment send-sets have one more argument.

27 (ast.Operator.INCREMENT_OPERATORS.contains(op.source) ?	27 (ast.Operator.INCREMENT_OPERATORS.contains(op.source)

28 0 :	28 ? 0

29 1) == node.argumentCount());	29 : 1) == node.argumentCount());

OLD	NEW