Date: prev next · Thread: first prev next last
2013 Archives by date, by thread · List index


On 10/07/2013 03:35 PM, Caolán McNamara wrote:

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.



Sorry for the long delay, fixed now with fc357b79e71577916e8d1970c6419619b950bd1c

Many thanks for the review!

Cheers
Matteo

Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.