From 46bab17e6d48c2279f4698d46f01db5404b0ba6f Mon Sep 17 00:00:00 2001 From: Noel Power Date: Fri, 8 Mar 2013 17:40:13 +0000 Subject: remove duplicate file loading from basic_coverage ( now in MacroSnippet ) Change-Id: I1b19fec59419575fcf09d79986b6bac73ea51c9a --- basic/qa/cppunit/basic_coverage.cxx | 37 +++++-------------------------------- 1 file changed, 5 insertions(+), 32 deletions(-) (limited to 'basic') diff --git a/basic/qa/cppunit/basic_coverage.cxx b/basic/qa/cppunit/basic_coverage.cxx index fff9aee011ca..8d540e4a3820 100644 --- a/basic/qa/cppunit/basic_coverage.cxx +++ b/basic/qa/cppunit/basic_coverage.cxx @@ -24,8 +24,7 @@ private: int m_nb_tests_skipped; OUString m_sCurrentTest; void process_directory(OUString sDirName); - void process_file(OUString sFileName); - void run_test(OUString sFileName, OUString sCode); + void run_test(OUString sFileName); void test_start(OUString /* sFileName */); void test_failed(void); void test_success(void); @@ -78,10 +77,11 @@ void Coverage::test_success() fprintf(stderr,"%s,PASS\n", rtl::OUStringToOString( m_sCurrentTest, RTL_TEXTENCODING_UTF8 ).getStr() ); } -void Coverage::run_test(OUString /*sFileName*/, OUString sCode) +void Coverage::run_test(OUString sFileURL) { bool result = false; - MacroSnippet testMacro( sCode ); + MacroSnippet testMacro; + testMacro.LoadSourceFromFile( sFileURL ); testMacro.Compile(); if( !testMacro.HasError() ) { @@ -101,33 +101,6 @@ void Coverage::run_test(OUString /*sFileName*/, OUString sCode) } } -void Coverage::process_file(OUString sFileName) -{ - osl::File aFile(sFileName); - - test_start(sFileName); - if(osl::FileBase::E_None == aFile.open(osl_File_OpenFlag_Read)) - { - sal_uInt64 size; - sal_uInt64 size_read; - if(osl::FileBase::E_None == aFile.getSize(size)) - { - void* buffer = calloc(1, size+1); - CPPUNIT_ASSERT(buffer); - if(osl::FileBase::E_None == aFile.read( buffer, size, size_read)) - { - if(size == size_read) - { - OUString sCode((sal_Char*)buffer, size, RTL_TEXTENCODING_UTF8); - run_test(sFileName, sCode); - return; - } - } - } - } - test_failed(); -} - void Coverage::process_directory(OUString sDirName) { osl::Directory aDir(sDirName); @@ -141,7 +114,7 @@ void Coverage::process_directory(OUString sDirName) aItem.getFileStatus(aFileStatus); if(aFileStatus.isRegular()) { - process_file(aFileStatus.getFileURL()); + run_test(aFileStatus.getFileURL()); } } } -- cgit