summaryrefslogtreecommitdiff
path: root/io
AgeCommit message (Collapse)Author
2014-12-16io: Use appropriate OUString functions on string constantsStephan Bergmann
Change-Id: Ib05e759a1223d71e738940f6f9298ac593d94a27
2014-11-26Fix C4716: 'anonymous namespace'::Input::readBytes : must return a valueTor Lillqvist
Change-Id: I1d660da4cdd3f612ac1478e37e613446d4253fae
2014-11-24Unit test for fdo#79941 (handle short reads)Stephan Bergmann
...done as a subsequentcheck as doing it as a BootstrapFixtureBase (which is the easiest way) makes it depend on later modules in the dependency chain. Change-Id: I9588bae409b38aa373ccfa855042f598b6e2bb2b
2014-11-21fdo#79941: Properly handle short readsStephan Bergmann
Change-Id: Ie762ff272410d809382f98a1d446946b566df747
2014-11-18cppu: clean up public headers with include-what-you-useMichael Stahl
Unfortunately iwyu gets quite confused by the weird cyclic dependencies between various foo.h/foo.hxx and cppumaker generated headers, so it's not obvious if any improvement here is realistic... Change-Id: I0bc66f98b146712e28cabc18d56c11c08418c721
2014-11-17sal: clean up public headers with include-what-you-useMichael Stahl
Sadly cannot forward declare "struct {...} TimeValue;". rtl/(u)?string.hxx still include sal/log.hxx but removing osl/diagnose.h was painful enough for now... Change-Id: Id41e17f3870c4f24c53ce7b11f2c40a3d14d1f05
2014-10-29remove unnecessary 'using namespace rtl' declarationsNoel Grandin
It turns out that almost none of them were necessary. Change-Id: I1311ed28409c682b57ea8d149bcbaf2c49133e83 Reviewed-on: https://gerrit.libreoffice.org/12133 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-09-29loplugin: cstylecastNoel Grandin
Change-Id: I53b69a488c70769cbb841db519bc28fd211dc087
2014-08-25set names on a bunch more threads...Michael Stahl
Change-Id: I4c2f2f0bcce52839033876ec991529721e06a3c8
2014-06-05various: remove SAL_THROW macroNoel Grandin
Change-Id: I9464179a736b91f5bc32eb5e5e89b3b4e3da3494
2014-06-02fdo#68849: Add header guards to all include filesJens Carl
added header guards for directories basebmp/, chart2/, cppuhelper/, include/test/, io/test/, sax/test, shell/, writerfilter/, and xmlhelp/ Change-Id: I0e29a9b75c26d71f58aa98986b52f6d3b46015a6 Reviewed-on: https://gerrit.libreoffice.org/9615 Reviewed-by: Thomas Arnhold <thomas@arnhold.org> Tested-by: Thomas Arnhold <thomas@arnhold.org>
2014-05-30fix . in #defineNoel Grandin
as a consequence of commit 9e3cc6b47444f988a68ca3ddcad779ed0480b00c "fdo#68849: Add header guards to all include files" Change-Id: Ibabf0d88ff4d928311ad5079ac3aa1fd6a2c1e6d
2014-05-30fdo#68849: Add header guards to all include filesJens Carl
Added header guard to files in directories i18npool/, include/, and io/ Change-Id: I7ba5b5f0ffdf5dedb8fd40e8efa06936e7c23591 Reviewed-on: https://gerrit.libreoffice.org/9558 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-05-29remove more unnecesary OUString constructor useNoel Grandin
when throwing exceptions Change-Id: I6edfb6b6745499f802b0e3c0e096a36fb7c32aac
2014-05-27remove unnecessary use of Reference constructor in throwNoel Grandin
Convert code like this: throw IOException("xx", Reference< XInterface >(static_cast<OWeakObject*>(this)) ); to this: throw IOException("xx", static_cast<OWeakObject*>(this) ); Change-Id: Ife9f645f0f1810a8e80219126193015502c43dbb
2014-05-23Prefer cppu::UnoType<T>::get() to ::getCppuType((T*)0) part20Julien Nabet
Change-Id: If87cdfb2c605254f6d69baa4ca5aec09091caa68
2014-05-23remove boilerplate in UNO Exception constructor callsNoel Grandin
Now that we have default values for Exception constructor params, remove lots of boilerplate code. Change-Id: I620bd641eecfed38e6123873b3b94aaf47922e74
2014-04-23io: sal_Bool->boolNoel Grandin
Change-Id: I136c98cbe15d1c6629028392a9573066320b3a04
2014-04-22second test should be DataOutputStreamStephan Bergmann
...as <https://gerrit.libreoffice.org/#/c/9007/> "cppcheck: multiCondition, second test should be DataOutputStream" found out for an effective duplicate of this code that got removed with the previous commit "Remove dead test code." Change-Id: Ib7b088f065def2d5e954837d22178521f85c49fe
2014-04-18Typo: s/shure/sureTor Lillqvist
Change-Id: I1c55d3ee8be06deb6d48a6ad5cd30d170c8b9dbb
2014-04-01Explicitly mark overriding destructors as "virtual"Stephan Bergmann
It appears that the C++ standard allows overriding destructors to be marked "override," but at least some MSVC versions complain about it, so at least make sure such destructors are explicitly marked "virtual." Change-Id: I0e1cafa7584fd16ebdce61f569eae2373a71b0a1
2014-03-26First batch of adding SAL_OVERRRIDE to overriding function declarationsStephan Bergmann
...mostly done with a rewriting Clang plugin, with just some manual tweaking necessary to fix poor macro usage. Change-Id: I71fa20213e86be10de332ece0aa273239df7b61a
2014-03-12coverity#707937 Uninitialized pointer fieldCaolán McNamara
Change-Id: I25b92686b08137a5292de620d4e348019c21a739
2014-03-12coverity#707936 Uninitialized pointer fieldCaolán McNamara
Change-Id: I0ae177349408fd49832056fbc4c300e8e77b2417
2014-03-08coverity#707939 Uninitialized scalar fieldCaolán McNamara
Change-Id: I5cbb4e2fef4361085840c914e26bb494566a4600
2014-03-08coverity#707938 Uninitialized scalar fieldCaolán McNamara
Change-Id: Ic76e70a33ed33a94e06faf0168e9a740c736a19e
2014-02-26cppuhelper: retrofit std::exception into overriding exception specsStephan Bergmann
Change-Id: I56e32131b7991ee9948ce46765632eb823d463b3
2014-02-26Remove visual noise from ioAlexander Wilms
Change-Id: I46f8c4f793596b0e6feb2db7b9302b9b6eb9644b Reviewed-on: https://gerrit.libreoffice.org/8273 Tested-by: Caolán McNamara <caolanm@redhat.com> Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2014-02-23Remove unneccessary commentsAlexander Wilms
Change-Id: I939160ae72fecbe3d4a60ce755730bd4c38497fb Reviewed-on: https://gerrit.libreoffice.org/8182 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2014-01-21bool improvementsStephan Bergmann
Change-Id: Ia6f86e38cb46a5f7ac634ad0afb66701811e6758
2013-12-17Adapt all (non-extension, SharedLibrary) .components to environment="..."Stephan Bergmann
Change-Id: I56f38bd786f3a026cb2908f28540dc9c4003af83
2013-12-09fdo#54938: Convert io to use cppu::supportsServiceMarcos Paulo de Souza
Change-Id: Id53289a68352558b039e0993b687d0447cea8036 Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
2013-11-19remove unnecessary use of OUString constructor when assigningNoel Grandin
change code like aStr = OUString("xxxx"); to aStr = "xxxx"; Change-Id: Ib981a5cc735677ec5dba76ef9279a107d22e99d4
2013-11-19remove most use of RTL_CONSTASCII_USTRINGPARAM macroNoel Grandin
This is largely unnecessary when working with OUString Change-Id: I3cf4d68357a43665d01162ef4a2d5346a45da9be
2013-11-11convert OUString 0==compareToAscii to equalsAsciiNoel Grandin
Convert code like: 0 == aStr.compareToAscii("XXX") to aStr.equalsAscii("XXX") which is both clearer and faster. Change-Id: I2e906d7d38494db38eb292702fadb781b1251e07
2013-11-11remove unnecessary use of OUString constructorNoel Grandin
Change-Id: Ifb220af71857ddacd64e8204fb6d3e4aad8eef71
2013-11-05fdo#60698: Merge all libs of io...Marcos Paulo de Souza
creating the libio. All these libraries are always used together, so we can put them together in one single library. This save almost 500 kb of the size of the final library. Change-Id: Ib32fec36cc4eb80ca646ce472c1f1bcdd98ac62b Reviewed-on: https://gerrit.libreoffice.org/6567 Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Stephan Bergmann <sbergman@redhat.com>
2013-11-04remove redundant calls to OUString constructor in if expressionNoel Grandin
Convert code like: if( aStr == OUString("xxxx") ) to this: if( aStr == "xxxx" ) Change-Id: I8d201f048477731eff590fb988259ef0935c080c
2013-10-22Bin comments that claim to say why some header is includedTor Lillqvist
They are practically always useless, often misleading or obsolete. Change-Id: I2d32182a31349c9fb3b982498fd22d93e84c0c0c
2013-10-22Resolves: #i75731# check output stream and...Jürgen Schmidt
throw IOExceptioin if it is not intialized (cherry picked from commit cf17f922f01bd75643749a7fd01a03f8167bccee) Change-Id: I07f6def513ef54b736689bf22565645f0c313982
2013-10-22fdo#54938: Adapt supportsService implementations..Marcos Paulo de Souza
to cppu::supportsService Change-Id: I0b03d3910f094f2183bf9859db9d5bcaf35d1f14 Reviewed-on: https://gerrit.libreoffice.org/6370 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Stephan Bergmann <sbergman@redhat.com>
2013-10-04fdo#46374 replace stardiv in comments where applicableThomas Arnhold
Change-Id: Icd07fba84735325408741ad10864c63ea32a6481
2013-08-21finish deprecation of O(U)String::valueOf()Luboš Luňák
Compiler plugin to replace with matching number(), boolean() or OUString ctor, ran it, few manual tweaks, mark as really deprecated. Change-Id: I4a79bdbcf4c460d21e73b635d2bd3725c22876b2
2013-08-12Mark as constTakeshi Abe
Change-Id: I9277052ee9c6197cef7a46bd8f3ab9c5515c2502
2013-07-01Fix memory leak in OTextInputStream.Mark Wielaard
Delete mpBuffer when done in destructor or replaced in implResizeBuffer. Change-Id: Ide1b44f10a482ce9bb6135fb879e62e0a8ccea29 Reviewed-on: https://gerrit.libreoffice.org/4637 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2013-06-29remove OUString wrap for string literalsThomas Arnhold
For some functions and all kinds of Exceptions. CannotConvertException CloseVetoException DisposedException EmptyUndoStackException ErrorCodeIOException Exception GridInvalidDataException GridInvalidModelException IOException IllegalAccessException IllegalArgumentException IllegalTypeException IndexOutOfBoundsException NoMasterException NoSuchElementException NoSupportException PropertyVetoException RuntimeException SAXException ScannerException StorageWrappedTargetException UnsupportedFlavorException VetoException WrappedTargetException ZipIOException throwGenericSQLException throwIllegallArgumentException createInstance createInstanceWithContext forName getByName getPackageManager getPropertyValue getUnpackedValueOrDefault getValueByName hasPropertyByName openKey setName setPropertyValue supportsService bash command: for i in `cat list`; do git grep "$i\s*(\s*OUString(\s*\"" -- '*.[hc]xx' | cut -d ':' -f1 | sort -u | xargs sed -i -e "s/\(\<$i\s*(\)\s*OUString(\s*\(\"[^\")\\]*\"\)\s*)\s*/\1\2/g" -e "s/\($i.*\)\"+ /\1\" + /g"; done Change-Id: Iaf8e641b0abf28c082906014f87a183517630535 Reviewed-on: https://gerrit.libreoffice.org/4624 Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org> Reviewed-by: Thomas Arnhold <thomas@arnhold.org> Tested-by: Thomas Arnhold <thomas@arnhold.org>
2013-05-06remove usage of RTL_CONSTASCII_USTRINGPARAMLuboš Luňák
Mechanical removal of usage together with OUString ctor, done by compiler plugin. Change-Id: I554227f76df0dac620b1b46fca32516f78b462c5
2013-04-22Move to MPLv2 license headers, with ESC decision and author's permission.Michael Meeks
2013-04-19Remove any remaining remnants of unused component_canUnload featureStephan Bergmann
Change-Id: I635fd006b5ba9f5fb0091748ee2ff9bc3c1a2d2a
2013-04-19remove unused component_canUnload functionsMatúš Kukan
Change-Id: Id3f41e2a620c47bb848718d0fc994739be2d64fc