Page MenuHomePhabricator

puremourning (Ben Jackson)
User

Projects

User does not belong to any projects.

User Details

User Since
Aug 22 2015, 6:53 PM (223 w, 5 d)

Recent Activity

Aug 14 2019

puremourning added a comment to D66254: Correct include suggestion when search path includes symlink.

Tests?

Per my comment: I haven't added new tests to HeaderSearchTest.cc because the InMemoryFileSystem doesn't support symlinks and I didn't want to try and implement that for this patch. Let me know thoughts on alternative ways to regression test it.

Aug 14 2019, 2:21 PM · Restricted Project
puremourning added a comment to D66254: Correct include suggestion when search path includes symlink.

Tests?

Aug 14 2019, 2:13 PM · Restricted Project
puremourning added a comment to D66254: Correct include suggestion when search path includes symlink.

FYI this fixes the issue in https://github.com/clangd/clangd/issues/124

Aug 14 2019, 1:57 PM · Restricted Project
puremourning created D66254: Correct include suggestion when search path includes symlink.
Aug 14 2019, 1:57 PM · Restricted Project

Aug 10 2019

puremourning created D66056: Add 2 tests which cover the hover result for auto.
Aug 10 2019, 8:11 AM · Restricted Project

Aug 9 2019

puremourning added a comment to D66038: [Support] heavyweight_hardware_concurrency uses affinity when counting cores fails, and never returns 0.

LGTM :)

Aug 9 2019, 4:00 PM · Restricted Project, Restricted Project
puremourning added a comment to D66031: clangd: use -j for background index pool.

Thanks! Want me to land this for you?

Aug 9 2019, 3:49 PM · Restricted Project, Restricted Project
puremourning updated the diff for D66031: clangd: use -j for background index pool.

Rebase on master

Aug 9 2019, 3:22 PM · Restricted Project, Restricted Project
puremourning added inline comments to D66031: clangd: use -j for background index pool.
Aug 9 2019, 3:16 PM · Restricted Project, Restricted Project
puremourning updated the diff for D66031: clangd: use -j for background index pool.

Always use physical cores rather than logical cores for best performance.

Aug 9 2019, 3:13 PM · Restricted Project, Restricted Project
puremourning added inline comments to D66031: clangd: use -j for background index pool.
Aug 9 2019, 2:44 PM · Restricted Project, Restricted Project
puremourning updated the diff for D66031: clangd: use -j for background index pool.

When -sync -background-index supplied, use 1 thread for background index.

Aug 9 2019, 2:44 PM · Restricted Project, Restricted Project
puremourning created D66031: clangd: use -j for background index pool.
Aug 9 2019, 2:00 PM · Restricted Project, Restricted Project

Sep 4 2017

puremourning added a comment to D37282: clangd: Tolerate additional headers.

Looks good and ready to land. Thanks for this change.
Do you have commit rights to llvm repo?

Sep 4 2017, 4:43 AM

Sep 2 2017

puremourning added a comment to D37282: clangd: Tolerate additional headers.

So I think this is ready now. Anything more you need from me?

Sep 2 2017, 10:14 AM

Aug 31 2017

puremourning updated the diff for D37282: clangd: Tolerate additional headers.

Validate that the duplicate message is printed.

Aug 31 2017, 2:34 PM
puremourning updated subscribers of D37282: clangd: Tolerate additional headers.
Aug 31 2017, 2:15 PM
puremourning updated the diff for D37282: clangd: Tolerate additional headers.

Tidy logging: missing newlines.

Aug 31 2017, 2:09 PM
puremourning updated the diff for D37282: clangd: Tolerate additional headers.

Keep ContentLength within the loop by using an inner loop to read headers.
Update tests to check stderr correctly.
Add some useful diagnostics to the log.

Aug 31 2017, 2:06 PM

Aug 30 2017

puremourning added a comment to D37282: clangd: Tolerate additional headers.

Thanks for the review! Few quick responses.

Aug 30 2017, 5:35 AM

Aug 29 2017

puremourning created D37282: clangd: Tolerate additional headers.
Aug 29 2017, 4:09 PM

Sep 7 2016

puremourning updated subscribers of D24192: [clang-refactor] introducing clang-refactor.
Sep 7 2016, 1:15 AM

Jul 11 2016

puremourning added a comment to D22129: [clang-rename] add documentation.

Couple of drive-by comments. Thanks for doing this.

Jul 11 2016, 3:19 PM

Aug 22 2015

puremourning added a comment to D11976: [libclang] Return deduced type for auto type, not the one written in the source..

I can confirm that this patch fixes a number test cases found in ycmd (a code-completion server). Specifically these ones marked as 'sic'

Aug 22 2015, 7:16 PM
puremourning updated subscribers of D11976: [libclang] Return deduced type for auto type, not the one written in the source..
Aug 22 2015, 6:54 PM