Hi Erik, > On 31 May 2019, at 18:15, Erik Joelsson <erik.joels...@oracle.com> wrote: > > Hello Robin, > > On 2019-05-31 05:26, Robin Westberg wrote: >> Hi Erik, >> >>> On 29 May 2019, at 17:22, Erik Joelsson <erik.joels...@oracle.com> wrote: >>> >>> Thanks, looks good! >> Thanks for reviewing! Unfortunately I had to do a few more general changes >> to get the “ccls" indexer working properly as well, hope you don’t mind >> taking another look: >> >> Webrevs: >> - Full: http://cr.openjdk.java.net/~rwestberg/8223678/webrev.03/ >> - Inc: http://cr.openjdk.java.net/~rwestberg/8223678/webrev.02-03/ > > Looks good! > > I sure appreciate adding tests when modifying the hairier utility macros.
Thanks again for reviewing! I did a small tweak to the settings of one of the optional indexers (ccls), perhaps not something that needs a detailed review, but here’s the final version that I’m thinking of pushing: Webrev: - Full: https://cr.openjdk.java.net/~rwestberg/8223678/webrev.04/ - Inc: http://cr.openjdk.java.net/~rwestberg/8223678/webrev.03-04/ Best regards, Robin > > /Erik > >> Best regards, >> Robin >> >>> /Erik >>> >>> On 2019-05-29 06:52, Robin Westberg wrote: >>>> Hi Erik, >>>> >>>> Thanks for taking a look! >>>> >>>>> On 28 May 2019, at 15:52, Erik Joelsson <erik.joels...@oracle.com> wrote: >>>>> >>>>> Hello Robin, >>>>> >>>>> Looks good. >>>>> >>>>> Adding of ide.md is very nice, but could you try to format it like >>>>> testing.md and building.md in regards to line lengths? I believe we are >>>>> trying for 80 chars in those to make them read reasonably in a standard >>>>> editor. >>>> Sure, did a bit of reflowing and updated some formatting in order to look >>>> more like the other .md files. >>>> >>>> Webrevs: >>>> - Full: https://cr.openjdk.java.net/~rwestberg/8223678/webrev.02/ >>>> - Inc: https://cr.openjdk.java.net/~rwestberg/8223678/webrev.01-02/ >>>> >>>> Best regards, >>>> Robin >>>> >>>>> /Erik >>>>> >>>>> On 2019-05-27 09:03, Robin Westberg wrote: >>>>>> Hi all, >>>>>> >>>>>> Please review this change that adds build system support for generating >>>>>> a Visual Studio Code workspace configured for working with the JDK >>>>>> native code. It configures the default C/C++ IntelliSense Engine to >>>>>> allow code completion/navigation and similar features. It also >>>>>> configures two executable targets (gtestLauncher and java) that can be >>>>>> built and debugged from the IDE. >>>>>> >>>>>> The main target is "make vscode-project”, additional information can be >>>>>> found in doc/ide.[md|html]. >>>>>> >>>>>> Issue: >>>>>> https://bugs.openjdk.java.net/browse/JDK-8223678 >>>>>> >>>>>> Webrev: >>>>>> https://cr.openjdk.java.net/~rwestberg/8223678/webrev.01/ >>>>>> >>>>>> Testing: >>>>>> Manual testing on Linux, MacOS and Windows >>>>>> >>>>>> Thanks Erik Joelsson for taking a look at an earlier version of it! >>>>>> >>>>>> Best regards, >>>>>> Robin