Re: [PATCH v2 00/10] Add 'ls-files --debug-json' to dump the index in json

2019-06-26 Thread Johannes Schindelin
Hi Ramsay, On Tue, 25 Jun 2019, Ramsay Jones wrote: > On 25/06/2019 15:10, Johannes Schindelin wrote: > > Hi Duy, > [snip] > > >> Again our experiences differ. Mine is mostly about extensions, > >> probably because I had to work on them more often. For normal entries > >> "ls-files --debug" gives

Re: [PATCH v2 00/10] Add 'ls-files --debug-json' to dump the index in json

2019-06-25 Thread Ramsay Jones
On 25/06/2019 15:10, Johannes Schindelin wrote: > Hi Duy, [snip] >> Again our experiences differ. Mine is mostly about extensions, >> probably because I had to work on them more often. For normal entries >> "ls-files --debug" gives you 99% what's in the index file already. > > Like the device.

Re: [PATCH v2 00/10] Add 'ls-files --debug-json' to dump the index in json

2019-06-25 Thread Johannes Schindelin
Hi Duy, On Tue, 25 Jun 2019, Duy Nguyen wrote: > On Tue, Jun 25, 2019 at 6:27 PM Johannes Schindelin > wrote: > > > > On Tue, 25 Jun 2019, Duy Nguyen wrote: > > > > > On Tue, Jun 25, 2019 at 1:00 AM Johannes Schindelin > > > wrote: > > > > > - extension location is printed, in case you need to

Re: [PATCH v2 00/10] Add 'ls-files --debug-json' to dump the index in json

2019-06-25 Thread Duy Nguyen
On Tue, Jun 25, 2019 at 6:27 PM Johannes Schindelin wrote: > > Hi Duy, > > On Tue, 25 Jun 2019, Duy Nguyen wrote: > > > On Tue, Jun 25, 2019 at 1:00 AM Johannes Schindelin > > wrote: > > > > - extension location is printed, in case you need to decode the > > > > extension by yourself (previousl

Re: [PATCH v2 00/10] Add 'ls-files --debug-json' to dump the index in json

2019-06-25 Thread Johannes Schindelin
Hi Duy, On Tue, 25 Jun 2019, Duy Nguyen wrote: > On Tue, Jun 25, 2019 at 1:00 AM Johannes Schindelin > wrote: > > > - extension location is printed, in case you need to decode the > > > extension by yourself (previously only the size is printed) > > > - all extensions are printed in the same o

Re: [PATCH v2 00/10] Add 'ls-files --debug-json' to dump the index in json

2019-06-25 Thread Thomas Gummerer
On 06/25, Duy Nguyen wrote: > On Tue, Jun 25, 2019 at 1:00 AM Johannes Schindelin > wrote: > > especially when we offer this as a better way > > for 3rd-party applications to interact with Git (which I think will be the > > use case for this feature that will be _far_ more common than using it for

Re: [PATCH v2 00/10] Add 'ls-files --debug-json' to dump the index in json

2019-06-25 Thread Duy Nguyen
On Tue, Jun 25, 2019 at 1:00 AM Johannes Schindelin wrote: > > - extension location is printed, in case you need to decode the > > extension by yourself (previously only the size is printed) > > - all extensions are printed in the same order they appear in the file > > (previously eoie and ieo

Re: [PATCH v2 00/10] Add 'ls-files --debug-json' to dump the index in json

2019-06-24 Thread Jeff Hostetler
On 6/24/2019 2:00 PM, Johannes Schindelin wrote: Hi Duy, On Mon, 24 Jun 2019, Nguyễn Thái Ngọc Duy wrote: - json field names now use '_' instead of '.' to be friendlier to some languages. I stick to underscore_name instead of camelCase because the former is closer to what we use Thi

Re: [PATCH v2 00/10] Add 'ls-files --debug-json' to dump the index in json

2019-06-24 Thread Johannes Schindelin
Hi Duy, On Mon, 24 Jun 2019, Nguyễn Thái Ngọc Duy wrote: > - json field names now use '_' instead of '.' to be friendlier to some > languages. I stick to underscore_name instead of camelCase because > the former is closer to what we use This is not a good reason. People who are used to read

[PATCH v2 00/10] Add 'ls-files --debug-json' to dump the index in json

2019-06-24 Thread Nguyễn Thái Ngọc Duy
v2 main changes: - --json is renamed --debug-json - json field names now use '_' instead of '.' to be friendlier to some languages. I stick to underscore_name instead of camelCase because the former is closer to what we use - extension location is printed, in case you need to decode the exte