[Libreoffice] [PATCH] [PUSHED] should fix bugs fdo#36080 and fdo#34666

Cedric Bosdonnat cedric.bosdonnat.ooo at free.fr
Fri Apr 22 02:41:36 PDT 2011


forgotten PUSHED flag again

On Fri, 2011-04-22 at 11:38 +0200, Cedric Bosdonnat wrote:
> Hello Anthony,
> 
> Many thanks for your patch. I just pushed it in the master branch. Keep
> providing good patches like this!
> 
> Regards,
> 
> On Tue, 2011-04-19 at 15:27 +0300, Anthony Durity wrote:
> > Hello good people,
> > 
> > 
> > If someone related to this area could give this patch the once over.
> > Please find attached patch.
> > 
> > 
> > This is (teensy tiny) patch that should fix an annoying HTML filter
> > "feature".
> > The problem is is that an xhtml style close tag <br/> <title/> is not
> > recognized
> > is a html tag which is as it should be. But there was some code that
> > would spew
> > the contents (inner HTML as it were) of the tag into the Writer
> > document. This
> > makes sense but not in the HEAD section. So I found where I think the
> > check
> > should go and stuck it in. May break other stuff but seems to be the
> > correct logic
> > from where I am sitting.
> > 
> > 
> > Uncovered a slightly nastier issue fdo#36390 while implementing this
> > fix. If anybody
> > is concerned it is if a tag is formed like <br /> or <title /> (note
> > the extra space this
> > time!) then the HTML parser recognizes them as valid tags when they
> > are obviously
> > not with an interesting set of consequences depending on the tag.
> > 
> > 
> > Regards,
> >     Anthony
> > _______________________________________________
> > LibreOffice mailing list
> > LibreOffice at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/libreoffice
> 

-- 
Cédric Bosdonnat
LibreOffice hacker
http://documentfoundation.org
OOo Eclipse Integration developer
http://cedric.bosdonnat.free.fr



More information about the LibreOffice mailing list