Skip to content
Snippets Groups Projects
  • Martin Probst's avatar
    ac4917bc
    clang-format: [JS] fix indenting bound functions. · ac4917bc
    Martin Probst authored
    Summary:
    The previous fix to force build style wrapping if the previous token is a closing parenthesis broke a peculiar pattern where users parenthesize the function declaration in a bind call:
        fn((function() { ... }).bind(this));
    
    This restores the previous behaviour by reverting that change, but narrowing the special case for unindenting closing parentheses to those followed by semicolons and opening braces, i.e. immediate calls and function declarations.
    
    Reviewers: djasper
    
    Subscribers: cfe-commits, klimek
    
    Differential Revision: https://reviews.llvm.org/D33640
    
    git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@304135 91177308-0d34-0410-b5e6-96231b3b80d8
    ac4917bc
    History
    clang-format: [JS] fix indenting bound functions.
    Martin Probst authored
    Summary:
    The previous fix to force build style wrapping if the previous token is a closing parenthesis broke a peculiar pattern where users parenthesize the function declaration in a bind call:
        fn((function() { ... }).bind(this));
    
    This restores the previous behaviour by reverting that change, but narrowing the special case for unindenting closing parentheses to those followed by semicolons and opening braces, i.e. immediate calls and function declarations.
    
    Reviewers: djasper
    
    Subscribers: cfe-commits, klimek
    
    Differential Revision: https://reviews.llvm.org/D33640
    
    git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@304135 91177308-0d34-0410-b5e6-96231b3b80d8
Code owners
Assign users and groups as approvers for specific file changes. Learn more.