Re: XLegacyFastParser - Errors

On Mon, Jul 11, 2016 at 1:48 PM, Michael Meeks <michael.meeks@collabora.com>
wrote:
> On Mon, 2016-07-11 at 10:22 +0530, Mohammed Abdul Azeem wrote:
>
> > 2. SwUiWriterTest::testAutoCorr
> > It fails with an uncaught exception of type
> > com.sun.star.container.NoSuchElementException.
> > It fails at sw/qa/extras/uiwriter/uiwriter.cxx:1034, so I assume these
> > statements
> > pWrtShell->Insert("Bolt");
> > pWrtShell->AutoCorrect(corr, cIns);
> > at line 1032 and 1033 is failing for some reason.
> > During the execution of those statements, LegacyParser is called with
> > these two filters as document handlers.
> > com.sun.star.comp.Writer.XMLStylesImporter
> > com.sun.star.comp.Writer.XMLContentImporter
>
> A quick git grep:
>
> xmloff/source/transform/OOo2Oasis.cxx-OOO_IMPORTER(
> XMLWriterContentImportOOO,
> xmloff/source/transform/OOo2Oasis.cxx:
> "com.sun.star.comp.Writer.XMLContentImporter",
> xmloff/source/transform/OOo2Oasis.cxx-
> "com.sun.star.comp.Writer.XMLOasisContentImporter" )
>
>         Suggests that there is some sort of wrapper / renaming (?)
>
> sw/source/filter/xml/xmlimp.cxx-extern "C" SAL_DLLPUBLIC_EXPORT
> css::uno::XInterface* SAL_CALL
> sw/source/filter/xml/xmlimp.cxx:com_sun_star_comp_Writer_XMLOasisContentImporter_get_implementation(css::uno::XComponentContext*
> context,
> sw/source/filter/xml/xmlimp.cxx-        css::uno::Sequence<css::uno::Any>
> const &)
>
>         Might be a good place to look,
>
> At some point of time ManifestReader::readManifestSequence is called,
which calls the parser. But our new parser is not emitting any events for
this source, while XParser parses the manifest file and emits some events.
I am not sure why this is happening. I have attached the diff -u of outputs
of events emitted by both the parsers during SwUiWriterTest::testAutoCorr
test.

Any help is appreciated.

Thanks,
Azeem

--- xparser.txt 2016-07-13 00:22:37.251741561 +0530
+++ legacyfastparser.txt        2016-07-13 00:22:38.419716423 +0530
@@ -699,160 +699,10 @@
 Attribs: script:name = Module1
 Attribs: script:language = StarBasic
 characters: REM  *****  BASIC  *****
-characters: 
 
-characters: 
-
-characters: Sub Main
-characters: 
-
-characters: 
-
-characters: End Sub
-startElement: manifest:manifest
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = /
-Attribs: manifest:media-type = 
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = 4xx/styles.xml
-Attribs: manifest:media-type = text/xml
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = 4xx/manifest.rdf
-Attribs: manifest:media-type = application/rdf+xml
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = 4xx/content.xml
-Attribs: manifest:media-type = text/xml
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = 4xx/
-Attribs: manifest:media-type = application/vnd.oasis.opendocument.text
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = Bolt/content.xml
-Attribs: manifest:media-type = text/xml
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = Bolt/manifest.rdf
-Attribs: manifest:media-type = application/rdf+xml
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = Bolt/styles.xml
-Attribs: manifest:media-type = text/xml
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = Bolt/
-Attribs: manifest:media-type = application/vnd.oasis.opendocument.text
-characters: 
-
-startElement: manifest:manifest
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = /
-Attribs: manifest:media-type = 
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = 4xx/styles.xml
-Attribs: manifest:media-type = text/xml
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = 4xx/manifest.rdf
-Attribs: manifest:media-type = application/rdf+xml
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = 4xx/content.xml
-Attribs: manifest:media-type = text/xml
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = 4xx/
-Attribs: manifest:media-type = application/vnd.oasis.opendocument.text
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = Bolt/content.xml
-Attribs: manifest:media-type = text/xml
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = Bolt/manifest.rdf
-Attribs: manifest:media-type = application/rdf+xml
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = Bolt/styles.xml
-Attribs: manifest:media-type = text/xml
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = Bolt/
-Attribs: manifest:media-type = application/vnd.oasis.opendocument.text
-characters: 
-
-startElement: manifest:manifest
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = /
-Attribs: manifest:media-type = 
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = 4xx/styles.xml
-Attribs: manifest:media-type = text/xml
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = 4xx/manifest.rdf
-Attribs: manifest:media-type = application/rdf+xml
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = 4xx/content.xml
-Attribs: manifest:media-type = text/xml
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = 4xx/
-Attribs: manifest:media-type = application/vnd.oasis.opendocument.text
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = Bolt/content.xml
-Attribs: manifest:media-type = text/xml
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = Bolt/manifest.rdf
-Attribs: manifest:media-type = application/rdf+xml
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = Bolt/styles.xml
-Attribs: manifest:media-type = text/xml
-characters: 
-
-startElement: manifest:file-entry
-Attribs: manifest:full-path = Bolt/
-Attribs: manifest:media-type = application/vnd.oasis.opendocument.text
-characters: 
+Sub Main
 
+End Sub
 startElement: office:document-styles
 Attribs: xmlns:office = urn:oasis:names:tc:opendocument:xmlns:office:1.0
 Attribs: xmlns:style = urn:oasis:names:tc:opendocument:xmlns:style:1.0
@@ -1223,12 +1073,12 @@
 startElement: text:p
 Attribs: text:style-name = P1
 characters: Bolt
-/home/azeem/libreoffice/sw/qa/extras/uiwriter/uiwriter.cxx:1035:SwUiWriterTest::testAutoCorr
-assertion failed
-- Expression: false
+unknown:0:SwUiWriterTest::testAutoCorr
+An uncaught exception of type com.sun.star.container.NoSuchElementException
+- 
 
-SwUiWriterTest::testAutoCorr finished in: 5569ms
-uiwriter.cxx:1035:Assertion
+SwUiWriterTest::testAutoCorr finished in: 269ms
+##Failure Location unknown## : Error
 Test name: SwUiWriterTest::testAutoCorr
-assertion failed
-- Expression: false
+An uncaught exception of type com.sun.star.container.NoSuchElementException
+-