Subject: Re: [PATCH v3] git-latexdiff: new command in contrib, to use latexdiff and Git

From: Matthieu Moy <[email-masked]>
Date: Thu, 16 Feb 2012 15:15:27 +0100

Jakub Narebski <[email-masked]> writes:

> Nb. I think it would be good to put detecting PDF viewer in its own
> function, don't you?

I normally like functions, but for such a linear and small piece of
code, I think it adds more confusion that clarity to write

do_fo () {
   actual stuff
}

do_foo

than the actual stuff alone.

-- 
Matthieu Moy
http://www-verimag.imag.fr/~moy/