Hello All,

sorry for chiming in here so late.

On Sun, Nov 14, 2021 at 10:12:05PM -0500, Carl Marcum wrote:

> Hi All,
> 
> On 11/14/21 5:05 PM, Matthias Seidel wrote:
> > Hi Carl,
> > 
> > Am 14.11.21 um 22:45 schrieb Carl Marcum:
> > > Hi Matthias,
> > > 
> > > 
> > > On 11/14/21 11:44 AM, Matthias Seidel wrote:
> > > > Hi Carl,
> > > > 
> > > > Am 14.11.21 um 17:36 schrieb Matthias Seidel:
> > > > > Hi Carl,
> > > > > 
> > > > > Am 14.11.21 um 17:31 schrieb Carl Marcum:
> > > > > > Hi Matthias and All,
> > > > > > 
> > > > > > On 11/14/21 9:10 AM, Matthias Seidel wrote:
> > > > > > > Hi Carl,
> > > > > > > 
> > > > > > > I only have a VM available with Xubuntu 18.04.6 and AOO42X, but
> > > > > > > have no
> > > > > > > problems opening and saving *.STW.
> > > > > > Okay I'll build AOO42X and see if I have same issues as trunk.
> > > > > > 
> > > > > > I recompiled trunk with
> > > > > > --enable-crashdump=yes \
> > > > > > --enable-dbgutil \
> > > > > > 
> > > > > > and now I get a better error dialog for opening these file types:
> > > > > > testSaveNewSTC.stc
> > > > > > testSaveNewSXC.sxc
> > > > > > ------------------
> > > > > > Error: uno exception caught while importing:
> > > > > > attempt to insert duplicate XML tag attribute: xmlns:svg
> > > > > >   From File
> > > > > > <build-path-to>/openoffice/main/sc/source/filter/xml/xmlwrap.cxx at
> > > > > > Line 365
> > > > > > Abort ? (Yes=abort / No=ignore / Cancel=core dump)
> > > > > I remember that we have a commit in trunk but not in AOO42X:
> > > > > 
> > > > > https://github.com/apache/openoffice/pull/122
> > > > > 
> > > > > Until now it has not been tested... Maybe that is the problem?
> > > > I can confirm Read-Errors with AOO450 (trunk) on Windows with STW and
> > > > SXW.
> > > > No problems with AOO420 (AOO42X).
> > > > 
> > > > This merge may need to be reverted...
> > > 
> > > Thanks for checking.
> > > 
> > > It might be better at this point to fix the code.
> > That's why I asked for testing... ;-)
> > > I'll do some bisecting and try to narrow it down.
> > I am not sure if the code was ready, Arrigo might know better.
> 
> I bisected it down to this one where it starts giving me the error which
> makes sense according the the commit message:
> 
> ebd28ad3a7fedadb378ee1596803008e8c6b383b is the first bad commit
> commit ebd28ad3a7fedadb378ee1596803008e8c6b383b
> 
>     Raise an exception when attempting to add duplicate attributes

Yes, that's my commit. It is supposed to avoid duplicated attributes
when _saving_ files. I am surprised you see this error while _opening_
them.

I could not really understand how to reproduce this problem. Could you
please send me a RTFM or a more precise link to the
file/script/whatever?

Reverting that commit is of course possible. But it would be like
sweeping dust under the carpet. Duplicated XML attributes are just
illegal, and we should rather fix the code requesting them. IMHO, of
course.

Thank you in advance and best regards,
-- 
Arrigo

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org
For additional commands, e-mail: dev-h...@openoffice.apache.org

Reply via email to