[Libreoffice] Rewiew needed for pythonscript.py patch (new try)

Jan Holesovsky kendy at suse.cz
Thu Mar 17 03:41:01 PDT 2011


Hi Julien,

On 2011-03-14 at 13:20 +0000, Caolán McNamara wrote:

> > I added this because of a warning (error) with pychecker. On the
> > contrary of the other blocks, the variable uri "wasn't" declared and
> > initialized whereas it was used in this block.
> 
> Ah, i see it now, its a use of that "uri" inside the log.debug line.
> Indeed, if logging was turned on then replaceByName would fail due to a
> missing uri, so its a good fix after all

Seems to me that there are no objections [and for me after a brief look
looks good too], so please go ahead, and push :-)

Regards,
Kendy



More information about the LibreOffice mailing list