[Po4a-devel]Re: [Po4a-commits] po4a/lib/Locale/Po4a TransTractor.pm,1.62,1.63

Nicolas François nicolas.francois@centraliens.net
Sun, 3 Apr 2005 15:15:35 +0200


On Sun, Apr 03, 2005 at 01:27:25AM +0200, Denis Barbier wrote:
> On Sat, Apr 02, 2005 at 09:57:15PM +0000, Nicolas FRAN??OIS wrote:
> > Update of /cvsroot/po4a/po4a/lib/Locale/Po4a
> > In directory haydn:/tmp/cvs-serv27444/lib/Locale/Po4a
> > 
> > Modified Files:
> > 	TransTractor.pm 
> > Log Message:
> > If a comment is "0", it is evaluated to false and thus ignored.
> > Use defined rather than the string evaluation.
> 
> Martin has another solution for this problem, which is IMO better:
> he checks if length($foo) > 0.

I'm not sure it is needed here, because $options{'comment'}||"" was just
used to make sure that $options{'comment'} is defined.

-- 
Nekral