New issue
Advanced search Search tips

Issue 647760 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Sep 2016
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

git cl format || git cl presubmit does not handle spacing for empty curly brackets properly

Project Member Reported by ekaramad@chromium.org, Sep 16 2016

Issue description

Version: 55.0.2861.0
OS: All

What steps will reproduce the problem?
(1) In a header file in blink define a method as void foo() {};
(2) Run 'git cl format'
(3) the method changes to void foo(){};
(4) Run 'git cl upload' after committing.

What is the expected output?
Presubmit should pass.

What do you see instead?
Presubmit fails complaining about the missing space between () and {}.

 
Sorry but I did not know where to report this bug to so I assigned it to you since I found your name on chromium.org. Thanks!

Comment 2 by mar...@chromium.org, Sep 16 2016

Owner: thakis@chromium.org
-> thakis so he can forward to someone else, I've never used clang-format.

Comment 3 by thakis@chromium.org, Sep 16 2016

Remove the ; after {} and it'll work.
thakis@ is right! I shouldn't have even put ; there. Thanks!

Comment 5 by thakis@chromium.org, Sep 20 2016

Status: WontFix (was: Assigned)
cool :-)

Comment 6 by thakis@chromium.org, Sep 20 2016

Labels: clang-format

Sign in to add a comment