login
Header Space

 
 

Re: [PATCH] gitweb: Extra columns in blame

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Petr Baudis <pasky@...>
Cc: <git@...>
Date: Friday, May 18, 2007 - 5:01 pm

Good job, except that I think you should also show the filename
especially as you seem to run with -C (I haven't looked at the
code yet, though).

E.g.

http://repo.or.cz/w/linux-2.6.git?a=blame_incremental;f=block/ll_rw_blk.c;h=6b5173ac81...

the first few hunks that came from 1da177e4 are from a different
file, drivers/block/ll_rw_blk.c.

Also the incremental thing using JavaScript does not seem to
work for me incrementally for some reason, although if I wait
long enough I get the fully blamed picture that seems to match
nonincremental one.  While I am waiting, the browser goes silent
and does not even let me switch to other tabs, so it is not all
that useful to me in its current shape.






-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] gitweb: Extra columns in blame, Petr Baudis, (Fri May 18, 3:17 pm)
Re: [PATCH] gitweb: Extra columns in blame, Junio C Hamano, (Fri May 18, 5:01 pm)
Re: [PATCH] gitweb: Extra columns in blame, Petr Baudis, (Fri May 18, 6:19 pm)
Re: [PATCH] gitweb: Extra columns in blame, Junio C Hamano, (Fri May 18, 8:18 pm)
Re: [PATCH] gitweb: Extra columns in blame, A Large Angry SCM, (Fri May 18, 10:10 pm)
speck-geostationary