Age | Commit message (Collapse) | Author |
|
Change-Id: Ib48a12e902f2311c295b2007f08f44dee28f431d
Reviewed-on: https://gerrit.libreoffice.org/3499
Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>
|
|
Since we no longer support the old Apple SDK using gcc-4.0.1, we can
remove the cruft to work around its problems. Woohoo.
Change-Id: Idf275e76449443f1f0314e75dab993f213a77eb7
|
|
...inspired by comments to <https://gerrit.libreoffice.org/#/c/3068/>
"String::AppendAscii cleanup in dbaccess," but it quickly becomes apparent that
lots of code rely on s[s.getLength()] == 0, so live with a weakened precondition
check for now.
Change-Id: Ifad96c706b14433df4a084ab8054b32433b8b5b6
|
|
Change-Id: I1c5b66888baac8aa1bc99c06579e0ef3638a4877
|
|
Change-Id: Ic908cb5cc9169ab1baae2c1c52070adfb9afba39
|
|
Change-Id: I3a41036ec03cdaefea94b19dbedf59e5a5e37dc8
|
|
http://lists.freedesktop.org/archives/libreoffice/2013-March/047769.html
Change-Id: I81ed4500878ff3193e028410a1f0205e28d17fc3
|
|
Change-Id: Ic511a4a399ae2d75de6050ac8e3f77ee13ec8275
|
|
Changed all occurrences of 'charcter' found by git grep. All of them
were used in comments only, so it should not break anything.
Change-Id: Ief2c00d929ae7972c55a4920fc0eaa8d6b18ab82
Reviewed-on: https://gerrit.libreoffice.org/2372
Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
Tested-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
|
|
Change-Id: I6542470c2d4b27de25e263651e25baf85c7d018d
|
|
Change-Id: I586fca181f63999a8f2d19d600b69f6788fd61d9
Reviewed-on: https://gerrit.libreoffice.org/1992
Reviewed-by: Luboš Luňák <l.lunak@suse.cz>
Tested-by: Luboš Luňák <l.lunak@suse.cz>
|
|
|
|
...instead of sal_Bool; this would cause C4800 performance warnings with MSVC,
but we disable them anyway.
Also, added unit tests that are actually executed.
Change-Id: Ib405132565918be72d93b3fc24180edcb6e565c7
|
|
valueOf(?) has been superseded by number(?).
O(U)String::valueOf(sal_Bool) has been deprecated without providing an
alternative method. Use boolean() method as a replacement for it.
Change-Id: I7eb74574fa8f1a3cf007cb0cbc32e681007ffb92
Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Change-Id: I2051e161219d424d2c2b69faf6f939cfe21fa5f7
Reviewed-on: https://gerrit.libreoffice.org/1980
Reviewed-by: Luboš Luňák <l.lunak@suse.cz>
Tested-by: Luboš Luňák <l.lunak@suse.cz>
|
|
- add sal_uInt64 valueOf helper to handle its full value range
- group deprecated valueOf() together
- forward to the number() taking the largest type instead of repeating
the same code every time
- various doc improvements:
- add missing @since
- do not refer to non-existent number() overloads in docs
- "use number" - "huh, of course I use a number?"
- "code your own" - my own function? why?
- + or += operators are not, strictly speaking, replacements for valueOf()
Change-Id: Ib138a06c4ac4365cfffc534e6ab115d55180a70d
|
|
This reverts commit 1efe9a15f86a7a9dc08b57fd1dd12336522ba515, which is obsoleted
by 488823a140217e393298bc83e75084041a85ed45 "Remove -Wsign-promo."
|
|
Now 'OUString()' and '""' should be exactly the same, and thus the latter better :).
Change-Id: If42b2ca5889967aaa8ccdaf531b45c75c12d54fd
|
|
Change-Id: Ie0bd47a16af56b7d3ae070d5b82ec46ed21834c0
|
|
I actually find the warning to be obsolete nonsense (the manpage talks
about cfront), but at least for now.
Change-Id: I4390b1779f6f8de062afd441f6855cb760adf93d
|
|
Change-Id: I3961ec27d23915b038c3a803dc71e7d937ca331c
|
|
API CHANGE: Adds new methods (several overloads)
OString::number()
OUString::number()
and marks all of the existing fromValue() methods as deprecated.
The purpose of this change is to clean up call sites
by hiding the necessary casts.
The casts are necessary because of overload resolution rules which are
somewhat vague about which methods to choose when using integer types.
See mailing list discussion here:
http://nabble.documentfoundation.org/replacing-OUString-valueOf-static-cast-lt-sal-Int32-gt-td4027989.html
Subject: "replacing OUString::valueOf(static_cast<sal_Int32>) ??"
Change-Id: Id3d150a6525eb0334e41e2ec6640bb06cd790b43
Reviewed-on: https://gerrit.libreoffice.org/1625
Reviewed-by: Luboš Luňák <l.lunak@suse.cz>
Tested-by: Luboš Luňák <l.lunak@suse.cz>
|
|
Change-Id: Id5ca706aa3fd1831990db2310933c6b94ca376cb
|
|
Change-Id: I30754e4d0cb0c490e6c125ba034e2e0eac3751e7
Reviewed-on: https://gerrit.libreoffice.org/1297
Reviewed-by: Michael Stahl <mstahl@redhat.com>
Tested-by: Michael Stahl <mstahl@redhat.com>
|
|
Change-Id: I3e8d8f123aaa43ee3721fae6f220a8c5f959a0ea
|
|
Not much point in building a string instance that will be copied elsewhere immediatelly.
Change-Id: I38b7d3696f2c619e6424eb3959b00cd2c7738c47
|
|
Change-Id: Ie5026c116f9cc7d5b10981d53881f8a0d7986a68
|
|
Change-Id: Ib816e8126193a3477fca1334d7526743da0d4423
|
|
Change-Id: I27bf6d5466313a7e4a1363f481b2bd7eab142a5c
|
|
... possible loss of data.
Minimal fix that inserts casts when calling strlen; other option would
be to change all length() to return size_t...
Change-Id: I5d1e0b0c7175a1957aad3acd89884fcd37089454
|
|
the asserts I've added need string.h , which was not available in that case
Change-Id: If8dbedcb48400e5edd0bde0214640c2c392fde2b
|
|
Change-Id: Ice9c6974f44be3bc4c9b3533de2a9beb5b146ca5
|
|
It's better to detect corner cases this way rather than get possibly
incorrect results. And strlen() should be easy to optimize out
for string literals.
Change-Id: Id762e256207668a0cbefe4e13d5f2067f373e783
|
|
Operator+ now, instead of requiring O(U)String operands and returning
another O(U)String object, keeps a track of the whole concatenation
operation using temporary O(U)StringConcat objects and performs
the whole operation only at the very end.
Change-Id: I94b3348300a137498514d26e96459c1698328520
|
|
Corrected types and grammer errors in comments
Change-Id: I47ce77a8d36d71677720be6dd80594ec76970376
|
|
String used to do some operation by modifying itself
whereas OUString never does that and when a modificaiton is needed
it create a new copy.
so it is very easy when one convert String code to OUString code to
miss stuff like
sString.ToUpperCase()
which need to be converted into
sString = sString.toAsciiUpperCase()
and not
sString.toAsciiUpperCase()
This patch make the compiler generate a warning in that later _wrong_ case
Change-Id: I4a9c0b4c7d0b75ad8850ac23b86e8508a334f5fe
Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
|
|
Change-Id: I40005ef4fad4d44314ec2fb2881fec82e926970e
|
|
Change-Id: Ie02dc3511e262a3b13df38ddbe0b3136f291ac9e
|
|
Patch contributed by Herbert Duerr:
#i118662# remove berkeleyDB from module xmlhelp (author=orwitt)
http://svn.apache.org/viewvc?view=revision&revision=1213188
#i119141# remove ISCII converter for now
http://svn.apache.org/viewvc?view=revision&revision=1306246
make exceptions for cppunittester verbose
http://svn.apache.org/viewvc?view=revision&revision=1174831
Patches contributed by Pedro Giffuni:
Avoid some uses of non portable #!/bin/bash in shell scripts.
http://svn.apache.org/viewvc?view=revision&revision=1235297
Patch contributed by Oliver-Rainer Wittmann
88652: applied patch, remove unicows deps
http://svn.apache.org/viewvc?view=revision&revision=1177585
drop OS/2 code, remove in-line assembler ARM atomics,
and obsolete armarch header.
|
|
Change-Id: Iefbe31af8d6ec421c6006423b6ee3a6375e3d453
|
|
Change-Id: If5c4c0b59aa91ec3775e2c15c4edb57bd7ac9aa7
|
|
Change-Id: I532558f5502898e036e01d64faa854c68e7109c9
|
|
This method will be needed for forthcoming String->OUStringBuffer
conversions.
Change-Id: I001099baaca5cd402aebcd15c031d9060286a8f9
|
|
Change-Id: I420229dea6c5b3e45cec5989897bb31654851e32
|
|
Change-Id: I03bb4db5931932280e368012cbaee6bef2854dd6
|
|
It apparently works that way, so there's no need to have
an #include loop.
Change-Id: I58d4f0461c14637872a139f0fbfb78f2a99fe28a
|
|
Change-Id: I1befdc4c9772c4bf66ee12bf0d2d4aa303903099
|
|
There's match(), with the second argument defaulting to 0, which
does the same, but that's pretty non-obvious.
Change-Id: Idd4de9388f53e1b5dc5d48446d1001af32c0af3d
|
|
...instead of sal_Bool, to work around MSVC warning C4805: unsafe mix of type
'sal_Bool' and type 'bool'.
Change-Id: Ia3477539ccc23aa9daec49d633f023d414f2b4bf
|
|
http://lists.freedesktop.org/archives/libreoffice/2012-April/029940.html
The RTL_USING #define (set by gbuild for anything that's not public
API) allows to use such classes simply by their name, without having
to use the namespace or do explicit using rtl::OUString (which half
of the sources do anyway).
Change-Id: I7edaf12cd278489cdc1d5ff782f0a86361c13c0a
|