Question on dubious code

Caolán McNamara caolanm at redhat.com
Mon Oct 7 06:35:25 PDT 2013


On Sat, 2013-10-05 at 19:45 +0200, Matteo Casalin wrote:
> looks wrong to me since aUserData cannot start with both aProt and 
> ".". I had a look at older revisions, but the logic remains the same at 
> least from when it was integrated in 2005 with commit 
> 4fdb2d2cb198e7ea85d1293333d96a6c26b50323

So, the commit where the problem appears seems to be
710a553683cef16be38fa8ef35b1dc88fb607ba4 and references #i35133#, but I
think that's a further typo and the real bug was #i35113# which has as
its 2nd last comment "document will be attached later" and they never
were :-)

> Does anybody knows ho to deal with this (the comparison with "./" should 
> be done after aProt, is completely unnecessary, or something else)?

Well, the intent was presumably to see if the token
*following* ...Package: started with "./" but seeing as its been 9 years
since that code was introduced I recommend you just delete the
impossible-to-reach if block.

C.




More information about the LibreOffice mailing list