Re: 'load' functions in cpp unittest - are they unused?
On Tue, 2012-08-21 at 19:42 +0700, Korrawit Pruegsanusak wrote: sd/qa/unit/regression-test.cxx (line #80) sc/qa/unit/subsequent_filters-test.cxx (line #246) which I think they are unused. Is it ok to remove both of them? [1] http://opengrok.libreoffice.org/search?q=loadproject=coredefs=refs=path=-java+-idl+-osl+qa+cxxhist= I think they're pure virtuals so it'll fail to build with them removed ?, could stub them I suppose, but they'd still exist. C. ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: 'load' functions in cpp unittest - are they unused?
Hello Caolán, First, sorry for a late reply. On Fri, Aug 17, 2012 at 11:33 PM, Caolán McNamara caol...@redhat.com wrote: On Fri, 2012-08-17 at 23:19 +0700, Korrawit Pruegsanusak wrote: Hello all, While I tried to create unittest with UNO API for fdo#47434 [1], I came across a function 'load' defined in sd/qa/unit/regression-test.cxx line #80: virtual bool load( const rtl::OUString rFilter, const rtl::OUString rURL, const rtl::OUString rUserData); and the function code is defined in line #239-253. Also, opengrok [2] showed this 'load' functions *seem* to be unused, because the result shown only the function definitions, without function call. SdFiltersTest inherits from test::FiltersTest and that has a FiltersTest::testDir which calls recursiveScan which calls the virtual load. In this specific case the SdFiltersTest::load isn't being used. But that's more an oversight and someone should get around to filling a dir like sw/qa/core/data/ww8 and do the same as SwFiltersTest::testCVEs for sd Thanks for the info. :-) Anyway, this sd/qa/unit/regression-test.cxx doesn't have 'testDir' function. This 'testDir', though, is in sd/qa/unit/filters-test.cxx. So, I browsed through each file in [1] and found that: the files that have 'load' function will also have 'testDir'. *Except* only two files: sd/qa/unit/regression-test.cxx (line #80) sc/qa/unit/subsequent_filters-test.cxx (line #246) which I think they are unused. Is it ok to remove both of them? [1] http://opengrok.libreoffice.org/search?q=loadproject=coredefs=refs=path=-java+-idl+-osl+qa+cxxhist= Best Regards, -- Korrawit Pruegsanusak ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
'load' functions in cpp unittest - are they unused?
Hello all, While I tried to create unittest with UNO API for fdo#47434 [1], I came across a function 'load' defined in sd/qa/unit/regression-test.cxx line #80: virtual bool load( const rtl::OUString rFilter, const rtl::OUString rURL, const rtl::OUString rUserData); and the function code is defined in line #239-253. Also, opengrok [2] showed this 'load' functions *seem* to be unused, because the result shown only the function definitions, without function call. Are these functions are actually unused? They are defined 'virtual', so are there any classes that inherit them? I might have missed something, and probably I had. Sorry if I used incorrect word or if this question sound dumb. [1] https://gerrit.libreoffice.org/421 [2] http://opengrok.libreoffice.org/search?q=loadproject=coredefs=refs=path=-java+-idl+-osl+qa+cxxhist= Have a nice weekend everyone :-) Best Regards, -- Korrawit Pruegsanusak ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: 'load' functions in cpp unittest - are they unused?
On Fri, 2012-08-17 at 23:19 +0700, Korrawit Pruegsanusak wrote: Hello all, While I tried to create unittest with UNO API for fdo#47434 [1], I came across a function 'load' defined in sd/qa/unit/regression-test.cxx line #80: virtual bool load( const rtl::OUString rFilter, const rtl::OUString rURL, const rtl::OUString rUserData); and the function code is defined in line #239-253. Also, opengrok [2] showed this 'load' functions *seem* to be unused, because the result shown only the function definitions, without function call. SdFiltersTest inherits from test::FiltersTest and that has a FiltersTest::testDir which calls recursiveScan which calls the virtual load. In this specific case the SdFiltersTest::load isn't being used. But that's more an oversight and someone should get around to filling a dir like sw/qa/core/data/ww8 and do the same as SwFiltersTest::testCVEs for sd C. ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice