Hi Matthias

Crashes with all PPT files I opened (but not with PPTX)

Best.
Pedro


> On 06/10/2025 5:15 PM WEST Matthias Seidel <matthias.sei...@hamburg.de> wrote:
> 
>  
> Hi all,
> 
> I just uploaded new Windows builds for AOO42X:
> 
> https://drive.google.com/drive/folders/18cd7sMdBEthJ_jNgxliJnX57OuJeYMDR
> 
> Please test and provide feedback.
> 
> Thanks!
> 
> Matthias
> 
> Am 10.06.25 um 13:35 schrieb Matthias Seidel:
> > Hi Damjan,
> >
> > For AOO42X I have reverted #1890034 and cherry-picked your last 4 
> > commits and I don't see any problem now.
> >
> > Thanks again!
> >
> > Regards,
> >
> >    Matthias
> >
> > Am 08.06.25 um 12:25 schrieb Damjan Jovanovic:
> >> On Sun, Jun 8, 2025 at 9:18 AM Damjan Jovanovic <dam...@apache.org> 
> >> wrote:
> >>
> >>>
> >>> On Sat, Jun 7, 2025 at 2:01 PM Matthias Seidel 
> >>> <matthias.sei...@hamburg.de>
> >>> wrote:
> >>>
> >>>> Hi Damjan,
> >>>>
> >>>> Am 07.06.25 um 13:47 schrieb Damjan Jovanovic:
> >>>>> On Fri, Jun 6, 2025 at 8:32 PM Matthias Seidel <
> >>>> matthias.sei...@hamburg.de>
> >>>>> wrote:
> >>>>>
> >>>>>> Hi Damjan,
> >>>>>>
> >>>>>> Am 06.06.25 um 20:15 schrieb Damjan Jovanovic:
> >>>>>>> On Fri, Jun 6, 2025 at 7:55 PM Matthias Seidel <
> >>>>>> matthias.sei...@hamburg.de>
> >>>>>>> wrote:
> >>>>>>>
> >>>>>>>> Hi Arrigo, Damjan,
> >>>>>>>>
> >>>>>>>> Am 05.06.25 um 22:45 schrieb Arrigo Marchiori:
> >>>>>>>>> Hello All,
> >>>>>>>>>
> >>>>>>>>> On Thu, Jun 05, 2025 at 10:33:52PM +0200, Matthias Seidel wrote:
> >>>>>>>>>
> >>>>>>>>>> Hi Damjan,
> >>>>>>>>>>
> >>>>>>>>>> That sounds really good!
> >>>>>>>>> Yes! It makes the code easier to read, and getting rid of 
> >>>>>>>>> outdated
> >>>>>>>>> constructs is A Good Thing.
> >>>>>>>>>
> >>>>>>>>>> I would love to join, but I (still) don't understand enough of
> >>>> it...
> >>>>>>>>>> Would that be something to cherry-pick for AOO42X?
> >>>>>>>>> FWIW I think so.
> >>>>>>>> Done! (first 4 commits...)
> >>>>>>>>
> >>>>>>>> Unfortunately, I now get a crash when opening an ODP 
> >>>>>>>> presentation on
> >>>>>>>> Windows...
> >>>>>>>>
> >>>>>>>>
> >>>>>>> Oh no! I'll investigate over the weekend.
> >>>>>> I get no error message and no crashdump...
> >>>>>>
> >>>>>> But it looks like it does not happen with every presentation.
> >>>>>>
> >>>>>>
> >>>>> Hi Matthias
> >>>>>
> >>>>> I cannot reproduce this on the latest trunk so far, but I will 
> >>>>> continue
> >>>> to
> >>>>> test.
> >>>>>
> >>>>> All .odp documents I found load without crashing, as does a new one I
> >>>> made
> >>>>> containing text and shapes.
> >>>>> One of my patches was to the linguistic module, but the api.i18n 
> >>>>> tests
> >>>> pass
> >>>>> and (the not yet pushed) api.linguistic2 tests pass.
> >>>>> bvt and fvt tests largely pass.
> >>>>>
> >>>>> My build flags were:
> >>>>> --with-dmake-url=
> >>>>> https://github.com/jimjag/dmake/archive/v4.13.1/dmake-4.13.1.tar.gz
> >>>>> --with-ant-home=/cygdrive/c/apache-ant-1.9.6
> >>>>> --with-junit=/cygdrive/c/packages/junit-4.13.2.jar
> >>>>> --with-epm-url=
> >>>>> https://sourceforge.net/projects/oooextras.mirror/files/epm-3.7.tar.gz 
> >>>>>
> >>>>> --disable-directx
> >>>>> --with-jdk-home="/cygdrive/c/Program Files/Java/jdk1.7.0"
> >>>>> --enable-verbose
> >>>>> --enable-pdfimport
> >>>>> --enable-dbgutil
> >>>>> --enable-crashdump
> >>>>> --with-hamcrest-core=/cygdrive/c/packages/hamcrest-core-1.3.jar
> >>>>> --with-package-format=installed
> >>>>> --with-nasm-home="/cygdrive/c/Program Files/NASM"
> >>>>> --enable-category-b
> >>>>> --with-mozilla-build=/cygdrive/c/mozilla-build
> >>>>>
> >>>>> Your crash could be due to some specific difference, eg. fonts,
> >>>> languages
> >>>>> and locale, images, shapes, macros, etc.
> >>>>>
> >>>>> Please send me the documents that crash, if possible.
> >>>>> Share your build flags.
> >>>> I simply use the presentation wizard to create a presentation from
> >>>> template.
> >>>>
> >>>> After "create" AOO crashes immediately.
> >>>>
> >>>> I can only test on Windows and build with the configure I use for 
> >>>> years:
> >>>>
> >>>>
> >>>> https://drive.google.com/file/d/1F8BlOJLbeE92ao6chdzJ6BUgCdP9iwng/view?usp=drive_link
> >>>>  
> >>>>
> >>>>
> >>>> Regards,
> >>>>
> >>>>      Matthias
> >>>>
> >>>>
> >>> I still cannot reproduce that Windows crash with trunk, but I have 
> >>> found
> >>> another way to crash it with the unit tests:
> >>> ./run
> >>> -Dopenoffice.home="../main/instsetoo_native/unxfbsdx/Apache_OpenOffice/installed/install/en-US/openoffice4"
> >>>  
> >>>
> >>> -tp fvt.uno.sd
> >>> will crash in certain tests that didn't crash before.
> >>>
> >>> Reverting the following commit stops it from crashing:
> >>>
> >>> commit 1890034f73947872dc88f59af6226fed5174e2b5
> >>> Author: Damjan Jovanovic <dam...@apache.org>
> >>> Date:   Thu Jun 5 19:09:02 2025 +0200
> >>>
> >>>      Port main/linguistic component_getFactory() to the component 
> >>> context
> >>> API.
> >>>
> >>>      Patch by: me
> >>>
> >>> I am investigating why. That commit seemed straightforward and 
> >>> unlikely to
> >>> cause problems, but maybe the extensive queryInterface() calls in
> >>> main/linguistic/source/gciterator.cxx behave differently with the new
> >>> service manager.
> >>>
> >>>
> >>>
> >> Reverting changes to gciterator.* and lngopt.cxx (and of course patching
> >> lngreg.cxx to call them the old way while still calling the others 
> >> the new
> >> way) did not help.
> >>
> >> Reverting changes to gciterator.* and lngopt.cxx and lngreg.cxx, 
> >> completely
> >> fixed the test crashes.
> >>
> >> Reverting only the changes to lngreg.cxx also completely fixed the test
> >> crashes, which tells us that the regression is in this simple
> >> harmless-looking part of the change:
> >> git show 1890034f73947872dc88f59af6226fed5174e2b5
> >> main/linguistic/source/lngsvcmgr.cxx
> >> (and of course the related changes to 
> >> main/linguistic/source/lngreg.cxx).
> >>

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

Reply via email to