- User Since
- Feb 15 2017, 1:27 PM (61 w, 4 d)
Mar 14 2018
Mar 13 2018
Feb 12 2018
@ioeric I'm working on a few other priorities over the next few weeks, sorry, but should get back to this relatively soon after that.
I would just land it, but I expect some downstream breakage I want to make sure I have time to fix.
Jan 26 2018
Looks good to me!
Jan 18 2018
- Applied the various refactorings suggested by @ioeric
- Extended c-index-test with a new option to print out the collected unit indexing data, and
- Added tests for the unit indexing functionality using the new option
- Fixed formatting
@ioeric I should have an updated patch up shortly with your inline comments addressed + new tests. Thanks again for reviewing!
Dec 19 2017
This needs to be reworked to remove its dependence on blocks.
Updated to account for the most recent changes to Part 1
Fix out of date header comment in FileIndexData.h
Dec 18 2017
I've refactored the indexing/dependency data collection out from the writing with the new IndexUnitDataConsumer class, and made other smaller changes to address the feedback from @ioeric.
Dec 12 2017
Thanks for taking another look @ioeric – I'll work through your comments and update.
Dec 7 2017
Worked through the comments from @ioeric and split the code for writing out the collected indexing data into a separate patch.
Nov 27 2017
Thanks for the feedback @malaperle!
Nov 9 2017
Nov 6 2017
Based on @arphaman's feedback:
- Pulled the index store related diagnostics out into their own category/diagnostic group
- Removed the CLANG_PROJECT_INDEX_PATH env var check.
- Swapped "/" used in a few places as a separator/root with the equivalent llvm::sys::path call.
- Fixed the typo/convention/documentation issues and simplifications pointed out so far
Oct 31 2017
Thanks @arphaman! I'll work through your comments and update.
I've split out the parts for reading and managing the store data (the IndexStore and DirectoryWatcher libraries) into a separate patch, so this one is now just about the -index-store-path option and everything necessary to write out the index data to the provided path.
Oct 18 2017
Aug 11 2017
May 19 2017
Ah sorry, missed the "Decl::" part of your recommendation. I'll move it.
May 18 2017
Mar 21 2017
Thanks for this – looks good to me!
Mar 17 2017
Mar 13 2017
Mar 7 2017
Mar 6 2017
Feb 23 2017
Feb 16 2017
Remove pointless cast.
Also report an implicit property definition (with relation to the ivar), getter and setter when the @synthesize is implicit.