[dpdk-dev] [PATCH] improve git diff
2016-11-09 16:44, Thomas Monjalon: > Sometimes git does not print the name of the function being changed > after @@. It happens especially after a goto label which is not indented. > Giving a hint about the languages of files .c, .h and .py > will improve hunk headers of "git diff" rendering. > > Signed-off-by: Thomas Monjalon Applied
[dpdk-dev] [PATCH] improve git diff
2016-11-11 17:28, Ferruh Yigit: > On 11/11/2016 4:21 PM, Thomas Monjalon wrote: > > 2016-11-11 11:22, Ferruh Yigit: > >> On 11/9/2016 3:44 PM, Thomas Monjalon wrote: > >>> Sometimes git does not print the name of the function being changed > >>> after @@. It happens especially after a goto label which is not indented. > >>> Giving a hint about the languages of files .c, .h and .py > >>> will improve hunk headers of "git diff" rendering. > >>> > >>> Signed-off-by: Thomas Monjalon > > [...] > >>> --- /dev/null > >>> +++ b/.gitattributes > >>> @@ -0,0 +1,3 @@ > >>> +*.c diff=cpp > >>> +*.h diff=cpp > >> > >> Can't git auto detect to use C/C++ language diff use for .c/.h files? > > > > No > > > >> Do you have a sample that generates bad hunk header, just to test? > > > > Yes, you'll find a lot of them with "git log -p | grep '@@.*:'" > > Example: > > git show bb6722f | grep '@@.*:' > > Without the patch, it is a goto label in the hunk header. > > > > You are right, I was expecting better from git J Sometimes, less is more :)
[dpdk-dev] [PATCH] improve git diff
On 11/11/2016 4:21 PM, Thomas Monjalon wrote: > 2016-11-11 11:22, Ferruh Yigit: >> On 11/9/2016 3:44 PM, Thomas Monjalon wrote: >>> Sometimes git does not print the name of the function being changed >>> after @@. It happens especially after a goto label which is not indented. >>> Giving a hint about the languages of files .c, .h and .py >>> will improve hunk headers of "git diff" rendering. >>> >>> Signed-off-by: Thomas Monjalon > [...] >>> --- /dev/null >>> +++ b/.gitattributes >>> @@ -0,0 +1,3 @@ >>> +*.c diff=cpp >>> +*.h diff=cpp >> >> Can't git auto detect to use C/C++ language diff use for .c/.h files? > > No > >> Do you have a sample that generates bad hunk header, just to test? > > Yes, you'll find a lot of them with "git log -p | grep '@@.*:'" > Example: > git show bb6722f | grep '@@.*:' > Without the patch, it is a goto label in the hunk header. > You are right, I was expecting better from git J
[dpdk-dev] [PATCH] improve git diff
2016-11-11 11:22, Ferruh Yigit: > On 11/9/2016 3:44 PM, Thomas Monjalon wrote: > > Sometimes git does not print the name of the function being changed > > after @@. It happens especially after a goto label which is not indented. > > Giving a hint about the languages of files .c, .h and .py > > will improve hunk headers of "git diff" rendering. > > > > Signed-off-by: Thomas Monjalon [...] > > --- /dev/null > > +++ b/.gitattributes > > @@ -0,0 +1,3 @@ > > +*.c diff=cpp > > +*.h diff=cpp > > Can't git auto detect to use C/C++ language diff use for .c/.h files? No > Do you have a sample that generates bad hunk header, just to test? Yes, you'll find a lot of them with "git log -p | grep '@@.*:'" Example: git show bb6722f | grep '@@.*:' Without the patch, it is a goto label in the hunk header.
[dpdk-dev] [PATCH] improve git diff
On 11/9/2016 3:44 PM, Thomas Monjalon wrote: > Sometimes git does not print the name of the function being changed > after @@. It happens especially after a goto label which is not indented. > Giving a hint about the languages of files .c, .h and .py > will improve hunk headers of "git diff" rendering. > > Signed-off-by: Thomas Monjalon > --- > .gitattributes | 3 +++ > 1 file changed, 3 insertions(+) > create mode 100644 .gitattributes > > diff --git a/.gitattributes b/.gitattributes > new file mode 100644 > index 000..fe555f8 > --- /dev/null > +++ b/.gitattributes > @@ -0,0 +1,3 @@ > +*.c diff=cpp > +*.h diff=cpp Can't git auto detect to use C/C++ language diff use for .c/.h files? Do you have a sample that generates bad hunk header, just to test? > +*.py diff=python >
[dpdk-dev] [PATCH] improve git diff
Sometimes git does not print the name of the function being changed after @@. It happens especially after a goto label which is not indented. Giving a hint about the languages of files .c, .h and .py will improve hunk headers of "git diff" rendering. Signed-off-by: Thomas Monjalon --- .gitattributes | 3 +++ 1 file changed, 3 insertions(+) create mode 100644 .gitattributes diff --git a/.gitattributes b/.gitattributes new file mode 100644 index 000..fe555f8 --- /dev/null +++ b/.gitattributes @@ -0,0 +1,3 @@ +*.c diff=cpp +*.h diff=cpp +*.py diff=python -- 2.7.0