Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Applied [ HEAD ] Re: [cdt-patch] Applied [HEAD] Parser_SymbolTable branch merged to HEAD


Thanks for the patch, I'm in too deep right now to run any of the exploratory "search the filesystem" suites. :-)
LineNumberTest.java is no longer references from our AutomatedSuite.java test harness, so as far as I am concerned, it is harmless.  The only way you can run it is to run the whole project as a Junit plugin, which for some reasons has different behaviour on different JDKs (i.e. will work on some, on other ones it prompts for a TestCase class).  

JohnC

p.s.  nice hotmail address :-)



"Victor Mozgin" <vic_mozgin@xxxxxxxxxxx>
Sent by: cdt-patch-admin@xxxxxxxxxxx

06/13/2003 02:08 PM

Please respond to
cdt-patch@xxxxxxxxxxx

To
<cdt-patch@xxxxxxxxxxx>
cc
Subject
Re: [cdt-patch] Applied [HEAD] Parser_SymbolTable branch merged to HEAD





            Here is the fix for broken AutomatedTest (parsing with NullParserCallback throws a NullPointerException, as NullParserCallback doesn't implement ISourceElementRequester).
            That still leaves 1 failure - testDOMLineNos() in LineNumberTest.java. I assume this is an expected failure (DOM still doesn't support line numbers directly, right?)
 
            /Vic
----- Original Message -----
From: John Camelon
To: cdt-patch@xxxxxxxxxxx
Sent: Friday, June 13, 2003 5:05 PM
Subject: [cdt-patch] Applied [HEAD] Parser_SymbolTable branch merged to HEAD


Major restructuring work includes:
       - introduction of public interfaces of parser for Search/Indexer teams

       - restructuring of parser architecture to deal with footprint issues

       - restructuring of parser logic so to not break outline view/quickParse while we get full parse working.  


We still have a long way to go to get full parsing working.


JohnC


[attachment "pst_fix_test.patch" deleted by John Camelon/Ottawa/IBM] [attachment "pst_fix_core.patch" deleted by John Camelon/Ottawa/IBM]


Back to the top