same as last mail (sorry), but with one important typo corrected ....


On Mon, 09 Feb 2015 22:35:52 +0100, Richard Hipp <d...@sqlite.org> wrote:

Probably we would be well-served to clean this all up. But first all
the committers have to agree on a particular style to use.  Then we
have to hunt down and change every place that prints out a SHA1 hash.
And, so far, everybody has felt that it works well enough as it is and
nobody has put in the effort to clean things up.

thanks for clarifying this (partly ...). I understand that this will not be
high on anyones todo list but I definitely agree that cleaning this up
would
be good. and the specific problem I ran into was not so
much different (fixed) lengths at different places, but rather the somewhat
strange (for me, that is) extension beyond the 10 chars in some cases
(which
are those, as you explained to me, where no [a-f] is occuring in the first
10 chars).

simple question: could _that_ be switched off consistently for timeline
and finfo withoud causing
havoc somewhere else in fossil (i.e. use the current behaviour of `finfo
-b' to use `%.10s')?
this looks like two places (at most), where it would need to be fixed,
right?

and in case you wonder, why this bit me at all: the script in question
maps sha1 hashes (as displayed
by timeline) to incremental checkin numbers which in turn can be used in
calls to `diff' etc.
I still believe that despite the fact that such numbers only make sense
locally (are usually NOT unique across different checkouts), they
significantly facilitate everyday "local" tasks such as checking a diff
between revision n and n+1, for instance. easier to memorize and type,
too, for modestly sized projects (at least everything below 10^4 checkins
...). but I know that the devs didn't like this some 2-3 years ago, so
I'll not propose this again as a useful change to the UI ;-)

thx/j

--
Using Opera's revolutionary email client: http://www.opera.com/mail/
_______________________________________________
fossil-users mailing list
fossil-users@lists.fossil-scm.org
http://lists.fossil-scm.org:8080/cgi-bin/mailman/listinfo/fossil-users

Reply via email to