.clang-format - Issue 697893002: clang-format/java: Tweak style a bit.

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: .clang-format

Issue 697893002: clang-format/java: Tweak style a bit. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Created 6 years, 1 month 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

OLD	NEW
1 # Defines the Chromium style for automatic reformatting.	1 # Defines the Chromium style for automatic reformatting.

2 # http://clang.llvm.org/docs/ClangFormatStyleOptions.html	2 # http://clang.llvm.org/docs/ClangFormatStyleOptions.html

3 BasedOnStyle: Chromium	3 BasedOnStyle: Chromium

4 # This defaults to 'Auto'. Explicitly set it for a while, so that	4 # This defaults to 'Auto'. Explicitly set it for a while, so that

5 # 'vector<vector<int> >' in existing files gets formatted to	5 # 'vector<vector<int> >' in existing files gets formatted to

6 # 'vector<vector<int>>'. ('Auto' means that clang-format will only use	6 # 'vector<vector<int>>'. ('Auto' means that clang-format will only use

7 # 'int>>' if the file already contains at least one such instance.)	7 # 'int>>' if the file already contains at least one such instance.)

8 Standard: Cpp11	8 Standard: Cpp11

9	9

10 ---	10 ---

11 Language: Java	11 Language: Java

12 # See http://crbug.com/429372 for information about this section.	12 # See http://crbug.com/429372 for information about this section.

13 IndentWidth: 4	13 IndentWidth: 4

14 ContinuationIndentWidth: 8	14 ContinuationIndentWidth: 8

15 ColumnLimit: 100	15 ColumnLimit: 100

	16 AllowShortBlocksOnASingleLine: false

	17 AllowShortFunctionsOnASingleLine: false
	cjhopman 2014/11/02 22:48:35 Should the value here be one of SFS_NONE,SFS_INLIN Should the value here be one of SFS_NONE,SFS_INLINE,SFS_ALL? Nico 2014/11/02 23:07:53 Oh, good point. It used to be a bool, and false is Show quoted text On 2014/11/02 22:48:35, cjhopman wrote: > Should the value here be one of SFS_NONE,SFS_INLINE,SFS_ALL? Oh, good point. It used to be a bool, and false is mapped to None (see bottom of https://www.mail-archive.com/cfe-commits@cs.uiuc.edu/msg94160.html) but it's cleaner to use a documented value. Changed, thanks.
16 AllowShortIfStatementsOnASingleLine: true	18 AllowShortIfStatementsOnASingleLine: true

	19 Cpp11BracedListStyle: false # but see http://llvm.org/PR21457

OLD	NEW

« no previous file with comments | « no previous file | no next file » | no next file with comments »