diff options
author | Noel Grandin <noel@peralex.com> | 2015-07-13 16:17:00 +0200 |
---|---|---|
committer | Noel Grandin <noelgrandin@gmail.com> | 2015-08-03 06:37:16 +0000 |
commit | 2660d24a07866e083c5135ea263030f3e3a2e729 (patch) | |
tree | 0089d6018d4fc33a7fde955e585e77191cdd258b /sax/qa/cppunit/parser.cxx | |
parent | baba1d14766282bd2c592bffd79ed69f9078cfe1 (diff) |
new loplugin: refcounting
This was a feature requested by mmeeks, as a result of
tdf#92611.
It validates that things that extend XInterface are not
directly heap/stack-allocated, but have their lifecycle managed
via css::uno::Reference or rtl::Reference.
Change-Id: I28e3b8b236f6a4a56d0a6d6f26ad54e44b36e692
Reviewed-on: https://gerrit.libreoffice.org/16924
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
Diffstat (limited to 'sax/qa/cppunit/parser.cxx')
-rw-r--r-- | sax/qa/cppunit/parser.cxx | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/sax/qa/cppunit/parser.cxx b/sax/qa/cppunit/parser.cxx index 824d5ecd7078..b781d9e1a6e1 100644 --- a/sax/qa/cppunit/parser.cxx +++ b/sax/qa/cppunit/parser.cxx @@ -45,7 +45,7 @@ public: class ParserTest: public test::BootstrapFixture { InputSource maInput; - sax_fastparser::FastSaxParser maParser; + uno::Reference< sax_fastparser::FastSaxParser > mxParser; uno::Reference< XFastDocumentHandler > mxDocumentHandler; uno::Reference< DummyTokenHandler > mxTokenHandler; @@ -67,7 +67,8 @@ void ParserTest::setUp() { test::BootstrapFixture::setUp(); mxTokenHandler.set( new DummyTokenHandler() ); - maParser.setTokenHandler( mxTokenHandler ); + mxParser.set( new sax_fastparser::FastSaxParser() ); + mxParser->setTokenHandler( mxTokenHandler.get() ); } void ParserTest::tearDown() @@ -89,13 +90,13 @@ uno::Reference< io::XInputStream > ParserTest::createStream(const OString& sInpu void ParserTest::parse() { maInput.aInputStream = createStream("<a>...<b />..</a>"); - maParser.parseStream( maInput ); + mxParser->parseStream( maInput ); maInput.aInputStream = createStream("<b></a>"); bool bException = false; try { - maParser.parseStream( maInput ); + mxParser->parseStream( maInput ); } catch (const SAXParseException &) { |