Re: Bug or User Interface Fault? Temporary file names used in "Compare Revisions"

Alf Lacis wrote:

> This is not a software "bug", but it *is* a fault in the design of the
> user-interface:
>
> It's impossible to tell from the temporary file names used in "Compare
> Revisions" which is the newer file, or even which revisions they actually
> are.
>
> If you have several 'diffs' opened on the screen, it's really easy to lose
> track, not only of the revisions, but even which files they refer to!
>
>>From Repository Browser:
>
> Repository Browser | R.H.C. | Show Log | <'Select' two Revisions> | R.H.C.
> | Compare Revisions
>
> You get two files to compare (I left out the directory paths):
> svn001.tmp.i
> svn000.tmp.i
>
> At least the file names should reflect what versions you are looking at.

Why? Those are temporary files. And we must create temporary files for
that. Also we can't remove those files automatically, because they're
used by the diff-viewer. And of course, you can diff the same files
multiple times with different revisions, so the filenames *must* be unique.

If I do the exact steps you mention above, TortoiseMerge nicely shows
the filenames and the revision numbers they're from, not the temp file
names. I don't know which diff tool you use, but if it can't handle the
name params TSVN can pass to them, then it's your diff tools fault, not
TSVN's.

> The files' names should contain the revision, as well as the original file
> name: something like:
> 1569.move_long.c
> 1751.move_long.c

Now you diff 'move_long.c' from revision 1569 with revision 1900, and
the diff would fail because '1569.move_long.c' already exists.

> They are already in a 'temporary' directory ("C:\temp\" in my case), so
> you don't really need the ".tmp" part of the filename. And what on earth
> is the ".i" for?

The ".i" would be the file extension of the file. We do that because
some diff viewers need the real file extension to work properly.

[snip signature and *8* quotes]
What the hell is that for? I don't mind signatures or quotes at the end
of the mails, as long as they're small in size (common understanding is
that they must not exceed four lines). But you really just broke every
record: 8 quotes!

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.tigris.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: dev-help@tortoisesvn.tigris.org