[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Half-baked unused features.
From: |
Reinhold Kainhofer |
Subject: |
Re: Half-baked unused features. |
Date: |
Mon, 16 Aug 2010 11:12:00 +0300 |
User-agent: |
KMail/1.13.5 (Linux/2.6.32-24-generic; KDE/4.4.5; i686; ; ) |
Am Sonntag 15 August 2010, um 17:15:55 schrieb David Kastrup:
> Not the process per se, but try doing this on Rietveld. Those are lots
> of changes in small files. For every single change, you need to tell
> the web interface to show you the file difference. You look at it, it
> looks ok. Now you need to navigate back to the list of changed files,
> remember which file you just looked at, select the next file in the
> list, navigate to its change overview.
Huh? Have you ever really looked at the pages???
Rietveld has a next/previous file link, and it has even keyboard shortcuts (j
for next, k for previous file) so that you don't even need to use your mouse...
> This is the sort of change you want to review and comment on in one
> piece. Linearly. Without being forced to do hundreds of mouse clicks.
> The scroll wheel should be all you need unless you want to comment.
You can always use "Download raw patchset" and have your browser open that
link in e.g. kompare automatically....
Cheers,
Reinhold
--
------------------------------------------------------------------
Reinhold Kainhofer, Vienna University of Technology, Austria
email: address@hidden, http://reinhold.kainhofer.com/
* Financial and Actuarial Mathematics, TU Wien, http://www.fam.tuwien.ac.at/
* Edition Kainhofer Music Publishing, http://www.edition-kainhofer.com/
* LilyPond music typesetting software, http://www.lilypond.org/
signature.asc
Description: This is a digitally signed message part.
- Review: remove markup function aliasing mechanism (was: Half-baked unused features.), (continued)
- Review: remove markup function aliasing mechanism (was: Half-baked unused features.), David Kastrup, 2010/08/15
- Review: start work on markup doc and code (was: Half-baked unused features.), David Kastrup, 2010/08/15
- Re: Half-baked unused features., David Kastrup, 2010/08/15
- Re: Half-baked unused features., Carl Sorensen, 2010/08/15
- Re: Half-baked unused features., David Kastrup, 2010/08/15
- git merge drivers (was: Half-baked unused features.), Ralf Wildenhues, 2010/08/16
- Re: Half-baked unused features., Graham Percival, 2010/08/15
- Re: Half-baked unused features., David Kastrup, 2010/08/15
- Re: Half-baked unused features., Carl Sorensen, 2010/08/15
- Re: Half-baked unused features., Graham Percival, 2010/08/15
- Re: Half-baked unused features.,
Reinhold Kainhofer <=
Re: Half-baked unused features., Werner LEMBERG, 2010/08/15
Re: Half-baked unused features., Reinhold Kainhofer, 2010/08/16