Page MenuHomePhabricator

mprobst (Martin Probst)
User

Projects

User does not belong to any projects.

User Details

User Since
Feb 18 2015, 5:19 AM (330 w, 1 d)

Recent Activity

Fri, Jun 11

mprobst committed rG63042d46bb0c: clang-format: [JS] don't sort named imports if off. (authored by mprobst).
clang-format: [JS] don't sort named imports if off.
Fri, Jun 11, 3:03 AM
mprobst closed D104101: clang-format: [JS] don't sort named imports if off..
Fri, Jun 11, 3:03 AM · Restricted Project
mprobst requested review of D104101: clang-format: [JS] don't sort named imports if off..
Fri, Jun 11, 2:45 AM · Restricted Project

Apr 30 2021

mprobst committed rGb2780cd744ea: clang-format: [JS] handle "off" in imports (authored by mprobst).
clang-format: [JS] handle "off" in imports
Apr 30 2021, 5:19 AM
mprobst closed D101515: clang-format: [JS] handle "off" in imports.
Apr 30 2021, 5:19 AM · Restricted Project
mprobst updated the diff for D101515: clang-format: [JS] handle "off" in imports.
  • review comments
Apr 30 2021, 5:16 AM · Restricted Project
mprobst added a comment to D101515: clang-format: [JS] handle "off" in imports.

added test for merging

Apr 30 2021, 2:34 AM · Restricted Project
mprobst updated the diff for D101515: clang-format: [JS] handle "off" in imports.
  • test for avoiding merges
Apr 30 2021, 2:33 AM · Restricted Project
mprobst updated the diff for D101515: clang-format: [JS] handle "off" in imports.
  • review comments
Apr 30 2021, 2:33 AM · Restricted Project
mprobst added a comment to D101515: clang-format: [JS] handle "off" in imports.

PTAL, addressed review comments.

Apr 30 2021, 2:32 AM · Restricted Project

Apr 29 2021

mprobst requested review of D101515: clang-format: [JS] handle "off" in imports.
Apr 29 2021, 2:38 AM · Restricted Project

Apr 22 2021

mprobst committed rGfbc6f42dbee5: clang-format: [JS] do not merge side-effect imports. (authored by mprobst).
clang-format: [JS] do not merge side-effect imports.
Apr 22 2021, 1:37 AM
mprobst closed D101020: clang-format: [JS] do not merge side-effect imports..
Apr 22 2021, 1:37 AM · Restricted Project

Apr 21 2021

mprobst requested review of D101020: clang-format: [JS] do not merge side-effect imports..
Apr 21 2021, 10:54 PM · Restricted Project
mprobst committed rG70ae843d9980: clang-format: [JS] do not wrap after `asserts` (authored by mprobst).
clang-format: [JS] do not wrap after `asserts`
Apr 21 2021, 7:34 AM
mprobst closed D100953: clang-format: [JS] do not wrap after `asserts`.
Apr 21 2021, 7:34 AM · Restricted Project
mprobst requested review of D100953: clang-format: [JS] do not wrap after `asserts`.
Apr 21 2021, 6:45 AM · Restricted Project

Apr 20 2021

mprobst committed rG3d4a6037ff46: clang-format: [JS] do not merge imports and exports. (authored by mprobst).
clang-format: [JS] do not merge imports and exports.
Apr 20 2021, 4:08 AM
mprobst closed D100752: clang-format: [JS] do not merge imports and exports..
Apr 20 2021, 4:08 AM · Restricted Project

Apr 19 2021

mprobst requested review of D100752: clang-format: [JS] do not merge imports and exports..
Apr 19 2021, 4:33 AM · Restricted Project

Apr 14 2021

mprobst committed rG2fe4909748b5: clang-format: fix undefined behavior. (authored by mprobst).
clang-format: fix undefined behavior.
Apr 14 2021, 11:11 AM
mprobst committed rG4d195f1b4dd6: review comments (authored by mprobst).
review comments
Apr 14 2021, 8:21 AM
mprobst committed rGd45df0d29f70: clang-format: [JS] merge import lines. (authored by mprobst).
clang-format: [JS] merge import lines.
Apr 14 2021, 8:20 AM
mprobst closed D100466: clang-format: [JS] merge import lines..
Apr 14 2021, 8:20 AM · Restricted Project
mprobst added inline comments to D100466: clang-format: [JS] merge import lines..
Apr 14 2021, 8:06 AM · Restricted Project
mprobst updated the diff for D100466: clang-format: [JS] merge import lines..

address review comments.

Apr 14 2021, 8:04 AM · Restricted Project
mprobst requested review of D100466: clang-format: [JS] merge import lines..
Apr 14 2021, 4:44 AM · Restricted Project

Mar 30 2021

mprobst committed rG2b30bd2be0a8: clang-format: [JS] do not collapse - - to --. (authored by mprobst).
clang-format: [JS] do not collapse - - to --.
Mar 30 2021, 5:32 AM
mprobst closed D99495: clang-format: [JS] do not collapse - - to --..
Mar 30 2021, 5:32 AM · Restricted Project

Mar 29 2021

mprobst requested review of D99495: clang-format: [JS] do not collapse - - to --..
Mar 29 2021, 4:16 AM · Restricted Project

Nov 10 2020

mprobst added a comment to D91078: [clang-format] do not break before @tags in JS comments.

For posterity: there was a question whether this is legal syntax in JSDoc in the first place (e.g. whether it should be escaped). That remains unclear - Closure Compiler parser however does ignore JSDoc tags if the line does not start with an @ tag. So the safe thing is to do what this revision does, avoid wrapping.

Nov 10 2020, 12:29 AM · Restricted Project, Restricted Project
mprobst accepted D91078: [clang-format] do not break before @tags in JS comments.
Nov 10 2020, 12:28 AM · Restricted Project, Restricted Project
mprobst committed rG16212b8b3e4f: clang-format: [JS] support new assignment operators. (authored by mprobst).
clang-format: [JS] support new assignment operators.
Nov 10 2020, 12:27 AM
mprobst closed D91132: clang-format: [JS] support new assignment operators..
Nov 10 2020, 12:27 AM · Restricted Project
mprobst updated the diff for D91132: clang-format: [JS] support new assignment operators..
  • fix comment
Nov 10 2020, 12:22 AM · Restricted Project
mprobst requested review of D91132: clang-format: [JS] support new assignment operators..
Nov 10 2020, 12:18 AM · Restricted Project

Nov 9 2020

mprobst added a comment to D91078: [clang-format] do not break before @tags in JS comments.

See my comment on the upstream bug, let's chat a bit about whether we need this.

Nov 9 2020, 7:51 AM · Restricted Project, Restricted Project

Nov 6 2020

mprobst accepted D90908: [clang-format] do not break before { in JS comments.
Nov 6 2020, 1:00 AM · Restricted Project, Restricted Project

Apr 6 2020

mprobst committed rG92201505cdec: clang-format: [JS] handle pseudo-keywords. (authored by mprobst).
clang-format: [JS] handle pseudo-keywords.
Apr 6 2020, 8:39 AM
mprobst closed D77548: clang-format: [JS] handle pseudo-keywords..
Apr 6 2020, 8:39 AM · Restricted Project
mprobst created D77548: clang-format: [JS] handle pseudo-keywords..
Apr 6 2020, 7:00 AM · Restricted Project

Apr 3 2020

mprobst committed rG146d685cd657: clang-format: [JS] detect C++ keywords. (authored by mprobst).
clang-format: [JS] detect C++ keywords.
Apr 3 2020, 5:53 AM
mprobst closed D77311: clang-format: [JS] detect C++ keywords..
Apr 3 2020, 5:52 AM · Restricted Project, Restricted Project
mprobst added inline comments to D77311: clang-format: [JS] detect C++ keywords..
Apr 3 2020, 5:20 AM · Restricted Project, Restricted Project
mprobst updated the diff for D77311: clang-format: [JS] detect C++ keywords..
  • fix formatting
Apr 3 2020, 3:43 AM · Restricted Project, Restricted Project
mprobst added a comment to D77311: clang-format: [JS] detect C++ keywords..

Thanks for the feedback. Indeed, whitelisting the JS keywords is better, but we still needed a way to blacklist all C++ keywords then - Krasimir's suggestion with the #include did the trick, thanks.

Apr 3 2020, 3:43 AM · Restricted Project, Restricted Project
mprobst updated the diff for D77311: clang-format: [JS] detect C++ keywords..
  • - improve handling of keywors - rather than blacklisting all C++
Apr 3 2020, 3:43 AM · Restricted Project, Restricted Project

Apr 2 2020

mprobst created D77311: clang-format: [JS] detect C++ keywords..
Apr 2 2020, 8:06 AM · Restricted Project, Restricted Project

Feb 3 2020

mprobst added a comment to rGdc04c54fc1f6: clang-format: [JS] document InsertTrailingCommas..

That has landed a while ago in https://reviews.llvm.org/rGa324fcf1ae6, not sure what's up with the differential revision not closing. Will do manually.

Feb 3 2020, 1:05 AM
mprobst closed D73354: clang-format: insert trailing commas into containers..

This has landed as https://reviews.llvm.org/rGa324fcf1ae6 (not sure why this hasn't closed automatically).

Feb 3 2020, 1:05 AM · Restricted Project, Restricted Project

Feb 2 2020

mprobst committed rGdc04c54fc1f6: clang-format: [JS] document InsertTrailingCommas. (authored by mprobst).
clang-format: [JS] document InsertTrailingCommas.
Feb 2 2020, 11:52 PM
mprobst closed D73768: clang-format: [JS] document InsertTrailingCommas..
Feb 2 2020, 11:52 PM · Restricted Project

Jan 31 2020

mprobst created D73768: clang-format: [JS] document InsertTrailingCommas..
Jan 31 2020, 1:27 AM · Restricted Project

Jan 29 2020

mprobst committed rGa324fcf1ae62: clang-format: insert trailing commas into containers. (authored by mprobst).
clang-format: insert trailing commas into containers.
Jan 29 2020, 4:28 AM
mprobst added inline comments to D73354: clang-format: insert trailing commas into containers..
Jan 29 2020, 4:28 AM · Restricted Project, Restricted Project
mprobst added inline comments to D73354: clang-format: insert trailing commas into containers..
Jan 29 2020, 3:49 AM · Restricted Project, Restricted Project

Jan 28 2020

mprobst updated the diff for D73354: clang-format: insert trailing commas into containers..
  • - only run comma insertion for JavaScript.
  • review fixes
  • Fix col limit
  • test for comma insertion
  • - validate options, reject bin packing + trailing commas
Jan 28 2020, 7:17 AM · Restricted Project, Restricted Project
mprobst added a comment to D73354: clang-format: insert trailing commas into containers..

PTAL.

Jan 28 2020, 7:17 AM · Restricted Project, Restricted Project
mprobst added inline comments to D73335: clang-format: [JS] options for arrow functions..
Jan 28 2020, 6:05 AM · Restricted Project, Restricted Project
mprobst added inline comments to D73354: clang-format: insert trailing commas into containers..
Jan 28 2020, 6:04 AM · Restricted Project, Restricted Project
mprobst updated the diff for D73354: clang-format: insert trailing commas into containers..
  • - only run comma insertion for JavaScript.
  • review fixes
  • Fix col limit
  • test for comma insertion
Jan 28 2020, 5:54 AM · Restricted Project, Restricted Project

Jan 27 2020

mprobst committed rG02656f29abda: clang-format: [JS] options for arrow functions. (authored by mprobst).
clang-format: [JS] options for arrow functions.
Jan 27 2020, 7:28 AM
mprobst closed D73335: clang-format: [JS] options for arrow functions..
Jan 27 2020, 7:28 AM · Restricted Project, Restricted Project

Jan 24 2020

mprobst updated the diff for D73335: clang-format: [JS] options for arrow functions..
  • Disable arrow functions on single lines by default.
Jan 24 2020, 7:55 AM · Restricted Project, Restricted Project
mprobst added a comment to D73335: clang-format: [JS] options for arrow functions..

If that's the case, consider adding the option to the style in a follow-up patch:

GoogleStyle.AllowShortLambdasOnASingleLine = FormatStyle::SLS_All;

that would have an advantage of making clear which test case diffs are result of the implementation and which are result of the style flip.

Jan 24 2020, 7:55 AM · Restricted Project, Restricted Project
mprobst updated the diff for D73354: clang-format: insert trailing commas into containers..
  • - only run comma insertion for JavaScript.
Jan 24 2020, 7:35 AM · Restricted Project, Restricted Project
mprobst added a comment to D73354: clang-format: insert trailing commas into containers..

This is required by JavaScript style, but this change as is optimistically runs the tool for any language. I think that's probably not what we want here initially, just throwing it out there. WDYT?

Jan 24 2020, 7:27 AM · Restricted Project, Restricted Project
mprobst created D73354: clang-format: insert trailing commas into containers..
Jan 24 2020, 7:26 AM · Restricted Project, Restricted Project
mprobst added a comment to D73335: clang-format: [JS] options for arrow functions..

FYI I've started a conversation on whether SLS_All should be the default Google style. But the option I think is useful in either case.

Jan 24 2020, 2:46 AM · Restricted Project, Restricted Project
mprobst created D73335: clang-format: [JS] options for arrow functions..
Jan 24 2020, 2:35 AM · Restricted Project, Restricted Project

Jan 20 2020

mprobst committed rG4c9d6914453d: clang-format: [JS] fix `??` opreator wrapping. (authored by mprobst).
clang-format: [JS] fix `??` opreator wrapping.
Jan 20 2020, 8:08 AM
mprobst closed D73026: clang-format: [JS] fix `??` opreator wrapping..
Jan 20 2020, 8:08 AM · Restricted Project, Restricted Project
mprobst created D73026: clang-format: [JS] fix `??` opreator wrapping..
Jan 20 2020, 3:30 AM · Restricted Project, Restricted Project

Jan 17 2020

mprobst committed rG9835cf159014: clang-format: [JS] pragmas for tslint, tsc. (authored by mprobst).
clang-format: [JS] pragmas for tslint, tsc.
Jan 17 2020, 4:44 AM
mprobst closed D72907: clang-format: [JS] pragmas for tslint, tsc..
Jan 17 2020, 4:44 AM · Restricted Project
mprobst accepted D72911: clang-format: fix spacing in `operator const char*()`.
Jan 17 2020, 4:01 AM · Restricted Project
mprobst created D72907: clang-format: [JS] pragmas for tslint, tsc..
Jan 17 2020, 1:48 AM · Restricted Project
mprobst committed rG0734fb21ed5e: clang-format: [JS] Handle more keyword-named methods. (authored by mprobst).
clang-format: [JS] Handle more keyword-named methods.
Jan 17 2020, 1:19 AM
mprobst closed D72827: clang-format: [JS] Handle keyword-named methods..
Jan 17 2020, 1:19 AM · Restricted Project, Restricted Project
mprobst added a comment to D72827: clang-format: [JS] Handle keyword-named methods..

I've added tests and a fix for try/catch and if/else.

Jan 17 2020, 1:09 AM · Restricted Project, Restricted Project
mprobst added a comment to D72827: clang-format: [JS] Handle keyword-named methods..

How about if and try? Is there a list somewhere for all such valid identifiers?

Jan 17 2020, 1:09 AM · Restricted Project, Restricted Project

Jan 16 2020

mprobst created D72827: clang-format: [JS] Handle keyword-named methods..
Jan 16 2020, 2:54 AM · Restricted Project, Restricted Project
mprobst committed rG736a3802124b: clang-format: [JS] tests for async wrapping. (authored by mprobst).
clang-format: [JS] tests for async wrapping.
Jan 16 2020, 2:27 AM
mprobst closed D70377: clang-format: [JS] tests for async wrapping..
Jan 16 2020, 2:26 AM · Restricted Project

Nov 18 2019

mprobst created D70377: clang-format: [JS] tests for async wrapping..
Nov 18 2019, 1:12 AM · Restricted Project

Nov 11 2019

mprobst committed rG6bcd8d4a18fc: clang-format: [JS] test declared fields. (authored by mprobst).
clang-format: [JS] test declared fields.
Nov 11 2019, 7:36 AM
mprobst committed rGa7638d384983: clang-format: [JS] support null operators. (authored by mprobst).
clang-format: [JS] support null operators.
Nov 11 2019, 7:36 AM
mprobst closed D69972: clang-format: [JS] test declared fields..
Nov 11 2019, 7:36 AM · Restricted Project, Restricted Project
mprobst closed D69971: clang-format: [JS] support null operators..
Nov 11 2019, 7:36 AM · Restricted Project, Restricted Project

Nov 8 2019

mprobst updated the diff for D69971: clang-format: [JS] support null operators..

Add tests for .?(foo) and .?[foo].

Nov 8 2019, 10:35 AM · Restricted Project, Restricted Project

Nov 7 2019

mprobst created D69972: clang-format: [JS] test declared fields..
Nov 7 2019, 2:54 PM · Restricted Project, Restricted Project
mprobst created D69971: clang-format: [JS] support null operators..
Nov 7 2019, 2:54 PM · Restricted Project, Restricted Project

Aug 26 2019

mprobst committed rL369916: clang-format: [JS] handle `as const`..
clang-format: [JS] handle `as const`.
Aug 26 2019, 8:47 AM
mprobst committed rG5836472ac488: clang-format: [JS] handle `as const`. (authored by mprobst).
clang-format: [JS] handle `as const`.
Aug 26 2019, 8:39 AM
mprobst closed D66736: clang-format: [JS] handle `as const`..
Aug 26 2019, 8:39 AM · Restricted Project
mprobst created D66736: clang-format: [JS] handle `as const`..
Aug 26 2019, 5:55 AM · Restricted Project

Mar 25 2019

mprobst removed a reviewer for D59684: [clang-format] [PR41187] moves Java import statements to the wrong location if code contains statements that start with the word import: mprobst.
Mar 25 2019, 4:24 AM · Restricted Project, Restricted Project

Mar 19 2019

mprobst committed rG26a484f479d5: [clang-format] [JS] handle private members. (authored by mprobst).
[clang-format] [JS] handle private members.
Mar 19 2019, 5:29 AM
mprobst committed rC356449: [clang-format] [JS] handle private members..
[clang-format] [JS] handle private members.
Mar 19 2019, 5:28 AM
mprobst committed rL356449: [clang-format] [JS] handle private members..
[clang-format] [JS] handle private members.
Mar 19 2019, 5:28 AM