Page MenuHomePhabricator

benhamilton (Ben Hamilton)Administrator
User

Projects

User does not belong to any projects.

User Details

User Since
Oct 16 2017, 8:34 AM (83 w, 6 d)
Roles
Administrator

Recent Activity

Fri, May 17

benhamilton requested changes to D62045: Revise the google-objc-global-variable-declaration check to match the style guide..
Fri, May 17, 11:45 AM · Restricted Project

Mon, Apr 29

benhamilton added inline comments to D61267: Update Phabricator.rst.
Mon, Apr 29, 2:33 PM · Unknown Object (Project)
benhamilton added inline comments to D61267: Update Phabricator.rst.
Mon, Apr 29, 2:32 PM · Unknown Object (Project)

Apr 22 2019

benhamilton committed rGa282bde69e37: [ASTMatchers] Introduce Objective-C matchers `isClassMessage`, `isClassMethod`… (authored by benhamilton).
[ASTMatchers] Introduce Objective-C matchers `isClassMessage`, `isClassMethod`…
Apr 22 2019, 10:54 AM
benhamilton committed rL358904: [ASTMatchers] Introduce Objective-C matchers `isClassMessage`, `isClassMethod`….
[ASTMatchers] Introduce Objective-C matchers `isClassMessage`, `isClassMethod`…
Apr 22 2019, 10:52 AM
benhamilton committed rC358904: [ASTMatchers] Introduce Objective-C matchers `isClassMessage`, `isClassMethod`….
[ASTMatchers] Introduce Objective-C matchers `isClassMessage`, `isClassMethod`…
Apr 22 2019, 10:52 AM
benhamilton closed D60920: [ASTMatchers] Introduce Objective-C matchers `isClassMessage`, `isClassMethod`, and `isInstanceMethod`.
Apr 22 2019, 10:52 AM · Restricted Project

Apr 19 2019

benhamilton accepted D60920: [ASTMatchers] Introduce Objective-C matchers `isClassMessage`, `isClassMethod`, and `isInstanceMethod`.

LGTM!

Apr 19 2019, 2:11 PM · Restricted Project

Feb 8 2019

benhamilton committed rG30b7d09d7a31: [Format/ObjC] Fix [foo bar]->baz formatting as lambda arrow (authored by benhamilton).
[Format/ObjC] Fix [foo bar]->baz formatting as lambda arrow
Feb 8 2019, 7:58 AM
benhamilton committed rL353531: [Format/ObjC] Fix [foo bar]->baz formatting as lambda arrow.
[Format/ObjC] Fix [foo bar]->baz formatting as lambda arrow
Feb 8 2019, 7:55 AM
benhamilton committed rC353531: [Format/ObjC] Fix [foo bar]->baz formatting as lambda arrow.
[Format/ObjC] Fix [foo bar]->baz formatting as lambda arrow
Feb 8 2019, 7:55 AM
benhamilton closed D57923: [Format/ObjC] Fix [foo bar]->baz formatting as lambda arrow.
Feb 8 2019, 7:54 AM · Restricted Project

Feb 7 2019

benhamilton updated the diff for D57923: [Format/ObjC] Fix [foo bar]->baz formatting as lambda arrow.

Tidy up code.

Feb 7 2019, 1:25 PM · Restricted Project
benhamilton created D57923: [Format/ObjC] Fix [foo bar]->baz formatting as lambda arrow.
Feb 7 2019, 1:24 PM · Restricted Project

Feb 1 2019

benhamilton abandoned D57623: [please ignore, v2] Clang test revision for new Herald rules for Github monorepo.
Feb 1 2019, 2:54 PM · Restricted Project
benhamilton created D57623: [please ignore, v2] Clang test revision for new Herald rules for Github monorepo.
Feb 1 2019, 2:54 PM · Restricted Project
benhamilton abandoned D57622: [please ignore, v2] Test revision for new Herald rules for Github monorepo.
Feb 1 2019, 2:53 PM · Unknown Object (Project)
benhamilton created D57622: [please ignore, v2] Test revision for new Herald rules for Github monorepo.
Feb 1 2019, 2:53 PM · Unknown Object (Project)

Jan 31 2019

benhamilton committed rL352757: Update monorepo .arcconfig with new project callsign..
Update monorepo .arcconfig with new project callsign.
Jan 31 2019, 9:57 AM
benhamilton abandoned D57513: [please ignore] Clang test revision for new Herald rules for Github monorepo.
Jan 31 2019, 7:23 AM · Restricted Project
benhamilton abandoned D57512: [please ignore] Test revision for new Herald rules for Github monorepo.
Jan 31 2019, 7:23 AM
benhamilton updated the diff for D57513: [please ignore] Clang test revision for new Herald rules for Github monorepo.
  • Another update
Jan 31 2019, 7:21 AM · Restricted Project
benhamilton created D57513: [please ignore] Clang test revision for new Herald rules for Github monorepo.
Jan 31 2019, 7:11 AM · Restricted Project
benhamilton created D57512: [please ignore] Test revision for new Herald rules for Github monorepo.
Jan 31 2019, 7:08 AM

Jan 30 2019

benhamilton committed rC352622: [clang-format] Fix line parsing for noexcept lambdas.
[clang-format] Fix line parsing for noexcept lambdas
Jan 30 2019, 5:54 AM
benhamilton committed rL352622: [clang-format] Fix line parsing for noexcept lambdas.
[clang-format] Fix line parsing for noexcept lambdas
Jan 30 2019, 5:54 AM
benhamilton closed D56909: [clang-format] Fix line parsing for noexcept lambdas.
Jan 30 2019, 5:54 AM
benhamilton added a comment to D56909: [clang-format] Fix line parsing for noexcept lambdas.

So sorry for the delay. I've landed this diff as rC352622.

Jan 30 2019, 5:54 AM
benhamilton requested changes to D55545: Allow IncludeSorter to use #import for Objective-C files.
Jan 30 2019, 5:53 AM

Jan 24 2019

benhamilton accepted D56226: [clang-format] square parens that are followed by '=' are not Objective-C message sends.
Jan 24 2019, 1:14 PM

Jan 23 2019

benhamilton accepted D56896: Update property prefix regex to allow numbers..
Jan 23 2019, 9:34 AM · Restricted Project
benhamilton accepted D56909: [clang-format] Fix line parsing for noexcept lambdas.
Jan 23 2019, 9:34 AM

Jan 7 2019

benhamilton added inline comments to D56226: [clang-format] square parens that are followed by '=' are not Objective-C message sends.
Jan 7 2019, 9:27 AM

Dec 13 2018

benhamilton added inline comments to D55545: Allow IncludeSorter to use #import for Objective-C files.
Dec 13 2018, 10:35 AM

Dec 12 2018

benhamilton added inline comments to D55482: [clang-tidy] Improve google-objc-function-naming diagnostics 📙.
Dec 12 2018, 8:42 AM

Dec 10 2018

benhamilton accepted D55482: [clang-tidy] Improve google-objc-function-naming diagnostics 📙.

Thanks, just minor suggestions.

Dec 10 2018, 3:47 PM

Nov 30 2018

benhamilton added a comment to D55101: [clang-tidy] Ignore namespaced and C++ member functions in google-objc-function-naming check 🙈.

Would you be okay with landing this fix and if we get any further reports for Objective-C++ sources then we can suppress it in all C++/Objective-C++ sources? I think there is enough value to enforcing the naming conventions on non-namespaced C functions in Objective-C++ to justify a simple followup fix. If other issues are reported after this then I also agree that enforcement in Objective-C++ sources may incur more overhead than it's worth.

Nov 30 2018, 2:43 PM
benhamilton added a comment to D55101: [clang-tidy] Ignore namespaced and C++ member functions in google-objc-function-naming check 🙈.

IMHO we should just disable this check entirely for C++ files (including Objective-C++).

Nov 30 2018, 8:17 AM

Nov 21 2018

benhamilton accepted D54795: [clang-format] Do not treat asm clobber [ as ObjCExpr, refined.
Nov 21 2018, 8:39 AM

Nov 12 2018

benhamilton accepted D51575: [clang-tidy/checks] Implement a clang-tidy check to verify Google Objective-C function naming conventions 📜.
Nov 12 2018, 8:38 AM · Restricted Project

Nov 9 2018

benhamilton accepted D54288: Fix ClangFormat issue of recognizing ObjC subscript as C++ attributes when message target is a result of a C-style method..
Nov 9 2018, 3:05 PM

Nov 7 2018

benhamilton added inline comments to D51575: [clang-tidy/checks] Implement a clang-tidy check to verify Google Objective-C function naming conventions 📜.
Nov 7 2018, 12:20 PM · Restricted Project

Nov 5 2018

benhamilton accepted D54111: [clang-format] Do not treat the asm clobber [ as ObjCExpr.

In the diff description, please fix the typo: Do not threat the asm clobber -> Do not treat the asm clobber

Nov 5 2018, 9:20 AM
benhamilton committed rL346144: [Format] Add debugging to ObjC language guesser.
[Format] Add debugging to ObjC language guesser
Nov 5 2018, 9:02 AM
benhamilton committed rC346144: [Format] Add debugging to ObjC language guesser.
[Format] Add debugging to ObjC language guesser
Nov 5 2018, 9:01 AM
benhamilton closed D54110: [Format] Add debugging to ObjC language guesser.
Nov 5 2018, 9:01 AM
benhamilton created D54110: [Format] Add debugging to ObjC language guesser.
Nov 5 2018, 8:51 AM

Nov 1 2018

benhamilton accepted D53955: Fix the issue that not recognizing single acronym with prefix as ObjC property name..
Nov 1 2018, 10:02 AM

Oct 12 2018

benhamilton committed rL344406: [clang-format] Fix BraceWrapping AfterFunction for ObjC methods.
[clang-format] Fix BraceWrapping AfterFunction for ObjC methods
Oct 12 2018, 12:45 PM
benhamilton committed rC344406: [clang-format] Fix BraceWrapping AfterFunction for ObjC methods.
[clang-format] Fix BraceWrapping AfterFunction for ObjC methods
Oct 12 2018, 12:45 PM
benhamilton added a comment to D53197: [clang-format] Fix BraceWrapping AfterFunction for ObjC methods.

@benhamilton Could you land this patch?

Oct 12 2018, 12:44 PM
benhamilton closed D53197: [clang-format] Fix BraceWrapping AfterFunction for ObjC methods.
Oct 12 2018, 12:44 PM
benhamilton accepted D53197: [clang-format] Fix BraceWrapping AfterFunction for ObjC methods.

Thanks!

Oct 12 2018, 8:11 AM

Sep 10 2018

benhamilton accepted D51832: [clang-tidy/checks] Update objc-property-declaration check to allow arbitrary acronyms and initialisms 🔧.

This is fine, but please update the comments (and docs?) to make it clear that we no longer enforce camelCase but allow aRBiTraRYcAsE now.

Sep 10 2018, 8:16 AM

Sep 7 2018

benhamilton committed rL341721: [clang-tidy/ObjC] Update list of acronyms in PropertyDeclarationCheck.
[clang-tidy/ObjC] Update list of acronyms in PropertyDeclarationCheck
Sep 7 2018, 3:06 PM
benhamilton committed rCTE341721: [clang-tidy/ObjC] Update list of acronyms in PropertyDeclarationCheck.
[clang-tidy/ObjC] Update list of acronyms in PropertyDeclarationCheck
Sep 7 2018, 3:06 PM
benhamilton committed rCTE341720: [clang-tidy/ObjC] Update list of acronyms in PropertyDeclarationCheck.
[clang-tidy/ObjC] Update list of acronyms in PropertyDeclarationCheck
Sep 7 2018, 3:06 PM
benhamilton committed rL341720: [clang-tidy/ObjC] Update list of acronyms in PropertyDeclarationCheck.
[clang-tidy/ObjC] Update list of acronyms in PropertyDeclarationCheck
Sep 7 2018, 3:06 PM
benhamilton closed D51819: [clang-tidy/ObjC] Update list of acronyms in PropertyDeclarationCheck.
Sep 7 2018, 3:06 PM
benhamilton updated the diff for D51819: [clang-tidy/ObjC] Update list of acronyms in PropertyDeclarationCheck.
  • Update docs.
Sep 7 2018, 3:05 PM
benhamilton created D51819: [clang-tidy/ObjC] Update list of acronyms in PropertyDeclarationCheck.
Sep 7 2018, 2:48 PM

Sep 5 2018

benhamilton added inline comments to D51575: [clang-tidy/checks] Implement a clang-tidy check to verify Google Objective-C function naming conventions 📜.
Sep 5 2018, 2:09 PM · Restricted Project
benhamilton requested changes to D51575: [clang-tidy/checks] Implement a clang-tidy check to verify Google Objective-C function naming conventions 📜.

Thanks for this! Let's consolidate this with the property name checker (either simplify the logic there and allow arBiTRAryCapSAnYWHere or apply the same registered acronym logic here).

Sep 5 2018, 9:11 AM · Restricted Project

Jul 12 2018

benhamilton committed rL336919: [clang-tidy/ObjC] Add SQL to list of acronyms.
[clang-tidy/ObjC] Add SQL to list of acronyms
Jul 12 2018, 10:38 AM
benhamilton committed rCTE336919: [clang-tidy/ObjC] Add SQL to list of acronyms.
[clang-tidy/ObjC] Add SQL to list of acronyms
Jul 12 2018, 10:38 AM
benhamilton closed D49190: [clang-tidy/ObjC] Add SQL to list of acronyms.
Jul 12 2018, 10:38 AM

Jul 11 2018

benhamilton created D49190: [clang-tidy/ObjC] Add SQL to list of acronyms.
Jul 11 2018, 8:12 AM

Jul 9 2018

benhamilton accepted D46274: [Support] Harden JSON against invalid UTF-8..

Thanks!

Jul 9 2018, 9:06 AM
benhamilton added inline comments to D46274: [Support] Harden JSON against invalid UTF-8..
Jul 9 2018, 7:57 AM

Jul 2 2018

benhamilton accepted D48718: [clang-format/ObjC] Prohibit breaking after a bracket opening ObjC method expression.

Sorry for the confusion. If you prefer me to add this test and modify in later commit I'll do it.

Jul 2 2018, 10:01 AM
benhamilton accepted D48716: [clang-format/ObjC] Fix counting selector name parts for ObjC.

General comment to changes D48716, D48718, D48719, D48720 (which are the split of D48352):

Jul 2 2018, 10:00 AM
benhamilton added a child revision for D48719: [clang-format/ObjC] Improve split priorities for ObjC methods: D48720: [clang-format/ObjC] Put ObjC method arguments into one line when they fit.
Jul 2 2018, 9:59 AM
benhamilton added a parent revision for D48720: [clang-format/ObjC] Put ObjC method arguments into one line when they fit: D48719: [clang-format/ObjC] Improve split priorities for ObjC methods.
Jul 2 2018, 9:59 AM
benhamilton added a parent revision for D48719: [clang-format/ObjC] Improve split priorities for ObjC methods: D48718: [clang-format/ObjC] Prohibit breaking after a bracket opening ObjC method expression.
Jul 2 2018, 9:59 AM
benhamilton added a child revision for D48718: [clang-format/ObjC] Prohibit breaking after a bracket opening ObjC method expression: D48719: [clang-format/ObjC] Improve split priorities for ObjC methods.
Jul 2 2018, 9:59 AM
benhamilton added a child revision for D48716: [clang-format/ObjC] Fix counting selector name parts for ObjC: D48718: [clang-format/ObjC] Prohibit breaking after a bracket opening ObjC method expression.
Jul 2 2018, 9:58 AM
benhamilton added a parent revision for D48718: [clang-format/ObjC] Prohibit breaking after a bracket opening ObjC method expression: D48716: [clang-format/ObjC] Fix counting selector name parts for ObjC.
Jul 2 2018, 9:58 AM

Jun 29 2018

benhamilton accepted D48720: [clang-format/ObjC] Put ObjC method arguments into one line when they fit.
Jun 29 2018, 12:58 PM
benhamilton accepted D48719: [clang-format/ObjC] Improve split priorities for ObjC methods.

Nice improvement!

Jun 29 2018, 12:29 PM
benhamilton committed rC335983: [clang-format/ObjC] Fix NS_SWIFT_NAME(foo(bar:baz:)) after ObjC method decl.
[clang-format/ObjC] Fix NS_SWIFT_NAME(foo(bar:baz:)) after ObjC method decl
Jun 29 2018, 8:31 AM
benhamilton committed rL335983: [clang-format/ObjC] Fix NS_SWIFT_NAME(foo(bar:baz:)) after ObjC method decl.
[clang-format/ObjC] Fix NS_SWIFT_NAME(foo(bar:baz:)) after ObjC method decl
Jun 29 2018, 8:31 AM
benhamilton closed D48679: [clang-format/ObjC] Fix NS_SWIFT_NAME(foo(bar:baz:)) after ObjC method decl.
Jun 29 2018, 8:31 AM

Jun 28 2018

benhamilton requested changes to D48718: [clang-format/ObjC] Prohibit breaking after a bracket opening ObjC method expression.

Can you add a test, please?

Jun 28 2018, 8:26 AM
benhamilton requested changes to D48716: [clang-format/ObjC] Fix counting selector name parts for ObjC.

Count selector parts also for method declarations.

Jun 28 2018, 8:26 AM

Jun 27 2018

benhamilton added a reviewer for D48679: [clang-format/ObjC] Fix NS_SWIFT_NAME(foo(bar:baz:)) after ObjC method decl: jolesiak.
Jun 27 2018, 4:06 PM
benhamilton created D48679: [clang-format/ObjC] Fix NS_SWIFT_NAME(foo(bar:baz:)) after ObjC method decl.
Jun 27 2018, 3:51 PM
benhamilton committed rCTE335770: [clang-tidy/ObjC] Add hashing algorithm acronyms to objc-property-declaration.
[clang-tidy/ObjC] Add hashing algorithm acronyms to objc-property-declaration
Jun 27 2018, 12:18 PM
benhamilton committed rL335770: [clang-tidy/ObjC] Add hashing algorithm acronyms to objc-property-declaration.
[clang-tidy/ObjC] Add hashing algorithm acronyms to objc-property-declaration
Jun 27 2018, 12:17 PM
benhamilton closed D48652: [clang-tidy/ObjC] Add hashing algorithm acronyms to objc-property-declaration.
Jun 27 2018, 12:17 PM
benhamilton requested changes to D48352: [clang-format] Improve ObjC method expressions formatting.

It's really hard to understand reviews which change 4 different things.

Jun 27 2018, 9:37 AM
benhamilton created D48652: [clang-tidy/ObjC] Add hashing algorithm acronyms to objc-property-declaration.
Jun 27 2018, 9:03 AM

Jun 14 2018

benhamilton committed rC334739: [clang-format] Disable AlwaysBreakBeforeMultilineStrings in Google style for….
[clang-format] Disable AlwaysBreakBeforeMultilineStrings in Google style for…
Jun 14 2018, 10:34 AM
benhamilton committed rL334739: [clang-format] Disable AlwaysBreakBeforeMultilineStrings in Google style for….
[clang-format] Disable AlwaysBreakBeforeMultilineStrings in Google style for…
Jun 14 2018, 10:34 AM
benhamilton closed D47393: [clang-format] Disable AlwaysBreakBeforeMultilineStrings in Google style for Objective-C 📜.
Jun 14 2018, 10:34 AM
benhamilton added a comment to D47393: [clang-format] Disable AlwaysBreakBeforeMultilineStrings in Google style for Objective-C 📜.

I'll just land it for you.

Jun 14 2018, 10:33 AM

Jun 11 2018

benhamilton accepted D48039: - Add "AV" as new default acronym. - Add support for "I" and "A" in lowerCamelCase pattern.
Jun 11 2018, 12:08 PM

Jun 7 2018

benhamilton committed rCTE334238: [checks/property-decls] Fix comment in clang-tidy/objc/PropertyDeclarationCheck..
[checks/property-decls] Fix comment in clang-tidy/objc/PropertyDeclarationCheck.
Jun 7 2018, 2:35 PM
benhamilton committed rL334238: [checks/property-decls] Fix comment in clang-tidy/objc/PropertyDeclarationCheck..
[checks/property-decls] Fix comment in clang-tidy/objc/PropertyDeclarationCheck.
Jun 7 2018, 2:35 PM
benhamilton closed D46922: [checks/property-decls] Fix comment in clang-tidy/objc/PropertyDeclarationCheck.cpp ✍️.
Jun 7 2018, 2:35 PM
benhamilton added a comment to D46922: [checks/property-decls] Fix comment in clang-tidy/objc/PropertyDeclarationCheck.cpp ✍️.

Is it possible to get someone to land this for me? I don't believe I have access to land it myself.

Jun 7 2018, 2:35 PM