- User Since
- May 9 2017, 8:22 AM (76 w, 10 h)
Jun 13 2018
Jun 6 2018
- Sometimes you refer to "fixits", sometimes "fix-its" and some times "FixIts". Unify to what is already there.
- The term "completion items" is new so far. Use "completions" for consistency.
Jun 1 2018
May 30 2018
I've stumbled about this bug too and was looking into it and then I saw the mail about this change being submitted :)
May 29 2018
Generates build errors here:
May 16 2018
If this is fine now, please submit as I don't have the permissions to do so.
Addressed inline nit.
May 11 2018
May 8 2018
May 7 2018
OK, the issue with the test dependencies I've had is resolved by https://reviews.llvm.org/D46514
I have no permissions to submit, please do this for me.
Moved the test to "Index".
May 3 2018
Do I miss something? I've uploaded a new diff/version and state is still "(X) Requested Changes to Prior Diff".
Apr 27 2018
OK, to skip all function bodies in the preamble except for template functions and functions within a template class, I've amended the previous diff with:
Trying to format the diff in the previous comment:
Reduction to skip-in-preamble-only functionality.
Apr 26 2018
Added a test and clarified scenario in commit message.
Apr 25 2018
only clang-format fixes
Apr 24 2018
Addressed inline comments.
Apr 20 2018
Apr 19 2018
This one still applies and tests pass.
Sorry for the delay, I think I'll come back to this one soon.
@ilya: Using SkipFunctionBodies_AllExceptTemplates for the preamble might be also useful for clangd.
Apr 17 2018
Huch, seems already submitted. Ignore :>
Feb 7 2018
Jan 31 2018
Abandoning since the change is already in.
Jan 19 2018
Hmm, nothing changed. So how to get rid of this correctly. Is "Abandon Revision" the right thing to do?
Jan 16 2018
This was one was already submitted/merged by yvvan. Hmm, this review does not reflect that yet?
I can merge this for you.
Please add me as reviewer in any follow-up patches and we can turn them around more quickly.
Jan 14 2018
Can you submit this for me? I don't have the permissions.
Addressed inline comment.
Jan 12 2018
What about this? :)
Added assert() for getter/setter.
It might be worth adding some very simple get/set tests to ensure that properties are set as intended.
Jan 11 2018
Used macros as in a previous version to make it less verbose and error prone.
Could one use an enum to get/set different properties of the policy?
I've seen other C-API's (for Linear and Quadratic programming) follow a similar approach quite extensibly.
It would significantly reduce the size of the API.
Jan 10 2018
Rebased only, please review.
Jan 2 2018
Had to rebase.
New year, new hope - ping :)
Rebased and renamed the counter variable only.
Hmm, could libclang profit from something like this, too? (or does it already?)
Dec 14 2017
Ping..."Please submit as I don't have the permissions for this."
Ping... "Please submit as I don't have the permissions."
Dec 11 2017
Addressed Ilya's comments.
Thanks for the review. Please submit as I don't have the permissions for this.
Dec 8 2017
Dec 6 2017
Dec 1 2017
Nov 30 2017
Nov 28 2017
Rebaed and clarified the documentation only.
Ping III - is there anything I can do to get this reviewed faster? 3 weeks passed.
Could you elaborate on "the client will be able to use it to generate a reproducer for the crash"? Having the json file, what would I need to do in order to reproduce the crash?