summaryrefslogtreecommitdiffstats
path: root/sal
Commit message (Collapse)AuthorAgeFilesLines
* some more clang warningsCaolán McNamara2012-10-221-0/+5
| | | | Change-Id: I4359784279875dc9dac99bc4d2db95dccf094b20
* More useless code, "#if 0" blocks in lingucomponent, sal, and rscMarcos Paulo de Souza2012-10-172-45/+0
| | | | | Change-Id: I1dda0f2b3bc2bb4a4a877c160026e53a90471d54 Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
* loplugin: some log, indent and unused variable warningsCaolán McNamara2012-10-131-0/+5
| | | | | | fix up some indents, remove some unused OUStrings and add some log areas Change-Id: I5c50807aff7a726b03b72522975d9b75e6685b9b
* compiler check to compare SAL_WARN/LOG areas against sal/inc/sal/log-areas.doxLuboš Luňák2012-10-131-0/+142
| | | | | | | Some of the areas are guesses I've added after seeing them, whoever feels reponsible for whichever part of the code feel free to adjust them. Change-Id: I2192de84d51cc2bc7c28fa84019d38b465985d15
* use consistently sal.osl and sal.rtl as log areasLuboš Luňák2012-10-139-52/+58
| | | | Change-Id: Ib5e606283d3d37c38e9729c79c4531807a1419d3
* mark lcl_ functions static or rename them if they are not local at allLuboš Luňák2012-10-121-3/+3
| | | | | | http://lists.freedesktop.org/archives/libreoffice/2012-October/039639.html Change-Id: I1a0e436051d48e7f6224d6f0fc602347df2d4df1
* loplugin: more unused OUStrings and so forthCaolán McNamara2012-10-111-17/+0
| | | | Change-Id: I4f3a51e1fd3ddca9442022a7134306fbf32e13ae
* Be consistent with surrounding codeTor Lillqvist2012-10-111-2/+4
| | | | Change-Id: Ice88c3912c2fd0d99156acaa8e15518acab3b55b
* WaE: 'fn' may be used uninitialized in this functionTor Lillqvist2012-10-111-1/+1
| | | | Change-Id: Ib918bd537b30fe5dc48396fc7e952147003e3b19
* Use DISABLE_DYNLOADING on AndroidTor Lillqvist2012-10-115-750/+104
| | | | | | | | IN this branch these changes are not conditional. Unclear yet whether this is what we finally will want to use or not. Maybe should make these changes conditional and do this stuff in master instead? Change-Id: I379d570a0e00648d295c675fd90eba6594ba3182
* avoid unused variable warnings for vars used for RTL_LOGFILE_CONTEXT_TRACExLuboš Luňák2012-10-101-3/+3
| | | | Change-Id: I8847192f61b2fca696a1ec6fbad65026e1ffdb43
* loplugin: unused O[U]StringsCaolán McNamara2012-10-103-14/+0
| | | | Change-Id: I020149a3073d8479887d108465cf5d3b727588d7
* Add a 'fromIndex' parameter to OUString::replaceAllNoel Grandin2012-10-094-4/+38
| | | | | | | This method will be needed for forthcoming String->OUStringBuffer conversions. Change-Id: I001099baaca5cd402aebcd15c031d9060286a8f9
* mark our string classes as SAL_WARN_UNUSEDLuboš Luňák2012-10-094-4/+4
| | | | Change-Id: I420229dea6c5b3e45cec5989897bb31654851e32
* compiler check for unused variablesLuboš Luňák2012-10-091-0/+20
| | | | | | | This is for variables that the compiler itself cannot figure out (e.g. non-trivial ctors). The classes need to be marked manually. Change-Id: I0109972e11e20578b1adc32065f701a871ee21aa
* When DISABLE_DYNLOADING don't even provide any module loading/unloading APITor Lillqvist2012-10-075-10/+33
| | | | | | | | | | | | Will then cause compilation errors where they are used, which will be noticed and taken care of. (The code chanaged to either use direct linking instead, when it makes sense, or to just bypass the functionality that requires dynamic loading.) Much better than waiting until run-time to notice where dynamic loading is attempted. Change-Id: Ib0cb5a2524b5c63f8e27670e7d72e37ce2a8e6e9
* Version name/dependency cleanup towards 3.7Stephan Bergmann2012-10-041-1/+1
| | | | Change-Id: Ib5da7703bf48713093bc6a3380facafd0013e039
* Revert "add rtl::OUStringBuffer::append(bool)"Markus Mohrhard2012-10-031-19/+0
| | | | This reverts commit 563fa900ba22bf83dfa58e67807ed0337f810576.
* add rtl::OUStringBuffer::append(bool)Markus Mohrhard2012-10-031-0/+19
| | | | Change-Id: I2cbb4d126b27c5cb7a1be3606807fcbda25d3e72
* fix indentationMarkus Mohrhard2012-10-031-7/+7
| | | | Change-Id: I43bad91f6afe5b55c6e2b6bf952cc0291d5bdd10
* xml2cmp no longer usedStephan Bergmann2012-10-031-1/+1
| | | | Change-Id: Idba65371b8778521bc767fe4893340cf13a8ff3b
* update string copy semantics to be undefined in a non-crashing way.Michael Meeks2012-10-026-48/+87
| | | | Change-Id: I03bb4db5931932280e368012cbaee6bef2854dd6
* Remove internal usage of rtl/memory.h in sal moduleArnaud Versini2012-10-018-14/+8
| | | | | | | Change-Id: I43650c6f4a66058e73945851a6990555e42b8ac2 Reviewed-on: https://gerrit.libreoffice.org/744 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
* Seems that the iOS-specific crack is not needed any longerTor Lillqvist2012-10-011-10/+0
| | | | Change-Id: I91a7f74e55b6ad8780a3a0920a22b6a7264b7b88
* gbuild: invert handling of standard system libraries:Michael Stahl2012-09-2834-66/+2
| | | | | | | Always link in gb_STDLIBS, except when the library explicitly opts out with gb_LinkTarget_disable_standard_system_libs. Change-Id: I489a99114fbfa46d0421a27cf6c7b899dc268a4a
* gbuild: gb_Library_PLAINLIBS_NONE cleanup for WNT:Michael Stahl2012-09-282-10/+11
| | | | | | | add a new gb_LinkTarget_use_system_win32_libs to abstract different linker options on MSVC and GCC. Change-Id: Ic9bf2545f59bf7871e6fc06b290c486ddfbec03d
* gbuild: gb_Library_PLAINLIBS_NONE cleanup for unxgcc/solarisMichael Stahl2012-09-281-7/+7
| | | | Change-Id: Ia7f64bdd0fd81c5dcc08d828db4602b65e2da949
* gbuild: gb_Library_PLAINLIBS_NONE cleanup for MacMichael Stahl2012-09-281-5/+8
| | | | Change-Id: I66f8229e186e312ed3242695db9ef0768ab4d9a0
* gbuild: clean up Mac OS X framework handling:Michael Stahl2012-09-281-3/+3
| | | | | | | | | | | There are currently 3 different mechanisms being used for frameworks, which is of course intolerable so we invent a 4th one and standardize on it: gb_LinkTarget_use_darwin_frameworks (This doesn't mean using add_libs or externals was wrong, it was just inconsistent... and i don't see an obvious benefit of using externals here) Change-Id: I5de9020402c87e7236c6a358c47f02fa56642d3d
* gbuild: replace direct gb_STDLIBS use with ...Michael Stahl2012-09-2833-34/+69
| | | | | | ... new gb_LinkTarget_add_standard_system_libs Change-Id: Ib2bc843098db3d8c6822b45a3d21724e67f57d69
* gbuild: split uwinapi out of gb_STDLIBSMichael Stahl2012-09-2832-0/+32
| | | | Change-Id: I53316e0b9369d806197bccb42cf22d3497af43e7
* move layout diagnostics to vcl.layout channelCaolán McNamara2012-09-281-0/+1
| | | | Change-Id: Ib33d91ea56219036182d30fdd3dc2159ce32a48c
* sal: uri.cxx: deploy some assertsMichael Stahl2012-09-261-19/+15
| | | | Change-Id: I1d755086295f5a8cd7acf56204402b95fe228d2d
* use intrinsict to inline InterlockCount() on Windows tooNorbert Thiebaud2012-09-252-0/+6
| | | | | | | Change-Id: I93949cc37821c5306514c8ce2f21519550f33775 Reviewed-on: https://gerrit.libreoffice.org/672 Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com> Tested-by: Norbert Thiebaud <nthiebaud@gmail.com>
* replace remaining InterlockedCount() with inlined versionNorbert Thiebaud2012-09-259-20/+20
| | | | | | | Change-Id: Ifcfa48fc87f905a91470a5b0fd597b02f220784c Reviewed-on: https://gerrit.libreoffice.org/671 Tested-by: Norbert Thiebaud <nthiebaud@gmail.com> Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com>
* dependency on makefile needs full pathDavid Tardon2012-09-221-1/+1
| | | | Change-Id: I0733abb5c736ab393259fd6a005a89b887304f10
* Guard against lack of slashes in dli_fnameTor Lillqvist2012-09-211-1/+4
| | | | Change-Id: Id9c452d9c1034f2e7969a9eae7588f2eb81a8813
* fix system-clucene/-boost build with gcc 4.4Rene Engelhard2012-09-191-0/+3
| | | | Change-Id: I95d23e6728571b3f3a6421a05fec814f7c5d059c
* dung out no longer needed initUCBHelper methods; thanks to sberg.Michael Meeks2012-09-191-21/+0
|
* deprecate oustringostreaminserter.hxxNorbert Thiebaud2012-09-185-31/+14
| | | | | | | | | | | | | | | | | | | the intent of this header has canged over time. now it is already systematically included with ustring.hxx and the operator overload it provide fit nicely there... Just to be safe, since that include as been added to the api during the 3.5 timeframe and therefore is already in 'production' the header remain and simply attempt to include ustring.hxx but a warning is issued indicating that this header should not be used anymore... in a couple of major release we will thenr emove it completely All internal users of that header are converted. Change-Id: I8934c55f089e29d78c0f5649b7c87b2ecf024bad Reviewed-on: https://gerrit.libreoffice.org/634 Tested-by: Norbert Thiebaud <nthiebaud@gmail.com> Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com>
* move ostream OUString operator<< directly to ustring.hxxLuboš Luňák2012-09-182-46/+34
| | | | | | | It apparently works that way, so there's no need to have an #include loop. Change-Id: I58d4f0461c14637872a139f0fbfb78f2a99fe28a
* migrate some of the biggest consumer of osl_*InterlockedCount to osl_atomicNorbert Thiebaud2012-09-172-4/+4
| | | | | | | Change-Id: I0e6992afbeffaf3b993e6630fb396d93012890e0 Reviewed-on: https://gerrit.libreoffice.org/632 Tested-by: Norbert Thiebaud <nthiebaud@gmail.com> Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com>
* Some comment clean upStephan Bergmann2012-09-171-13/+14
| | | | Change-Id: I33fbe68b17e9a1c457b37c6d81619f2df67fbe8d
* add osl_atomic_* api to allow for inlined atomic increment/decrementNorbert Thiebaud2012-09-171-0/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | atomic increment/decrement is provided by osl_increment/decrementInterlockedCount() but that is a exported dll function, so it cannot be inlined. valgrind analysis of a run, loading a medium sized spreadsheet, shows that these 2 functions were called 3.5 millions times for a total cost of 55 millions of instructions... a cost of 8 instructions per call, which is at least a 300% overhead since an atomic inc/dec is 2 instructions iow we could save about 1% of the total instruction count of that run(4.6B) We cannot change the existing api, as this would break ABI. but we can add a new api. and migrate internal user to it. osl_atomic_decrement/osl_atomic_increment do the same task, than osl_*IntelockedCount() but do that inlined if possible. Note that this version only optimize the case GCC with atomic built-in. but support for other case should not be very hard. follows-up patches will replace the use of the osl_*InterlockedCount() in the product with their osl_atomic_* equivalent. Change-Id: If4dcbf94ea6f62eb6d55d30613fe65878ffb8023 Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
* update unused code listCaolán McNamara2012-09-171-40/+0
| | | | Change-Id: I4d6ad11fbe68420931e88defa20afbef290d4d24
* Include header with prototypes of defined functionsStephan Bergmann2012-09-172-4/+6
| | | | Change-Id: Ia28c36b1627ef944adecd5c8a55c150bd0f9aa02
* export sal init and deinit functionsDavid Tardon2012-09-162-4/+4
| | | | | | | Linking of cppunittester (and any other executable that uses SAL_IMPLEMENT_MAIN or SAL_IMPLEMENT_MAIN_WITH_ARGS) fails otherwise. Change-Id: I778941ab2069819789f96ab04f364ea61cf4a3cb
* Let --enable-assert-always-abort affect setting NDEBUG directlyStephan Bergmann2012-09-151-17/+0
| | | | | | | | | | | For one, assert.h is designed to be includeable multiple times with changing NDEBUG settings, so it is not robust to include it early in sal/macros.h with "correct" NDEBUG settings and potentially include it again later. For another, there is #ifndef NDEBUG code providing functionality used exclusively within assert calls, which must be compiled with the same NDEBUG-setting as the relevant #include <assert.h>. Change-Id: I7b2f9c85f8e2155051274757c64162ed5a5e9d1b
* Include saldllapi.h explicitly where neededStephan Bergmann2012-09-1547-79/+189
| | | | Change-Id: I7ba421479cd332e059ed9da1c6090e2df420347e
* Revert "saldllapi.h is really not included outside of sal itself"Stephan Bergmann2012-09-156-5/+26
| | | | | | | | | | | | | | This reverts commit 2dfe34ce0efef6ec0412130a32f755657710363d: * sal/saldllapi.h is about something different than sal/types.h (it is rather a mistake of cf77e8a0b9dc26d5007c76388c3f09231f048bdd "sal: add visibility symbols" to shortcut the #include of sal/saldllapi.h into sal/types.h instead of the headers that actually need it), not every header needs to include sal/types.h * sal/config.h is the header to always include first (not sal/types.h) Change-Id: I217f2540197ddb682c6c00e529b812b04b327d73
generated by cgit (git 2.34.1) at 2024-08-23 02:25:05 +0000