On Wed, 2010-11-03 at 17:16 +0100, Gert Faller wrote:
Hi,
this may be the last for module 'writer'.
Excellent, all pushed, thanks.
Remains 'sw/source/ui/vba/vbacheckbox.cxx:78'.
When I google I find :
#define ECMA_FORMCHECKBOX_CHECKED checked in
'/xmloff/inc/xmloff/ecmaflds.hxx'
but
On Tue, 2010-11-02 at 18:44 +0100, Gert Faller wrote:
Hi,
with this one, module 'impress' should be rather clean of that.
Heh, good, liked the ? : rework to keep me pacified :-). Thanks for
this.
As an aside, stylistically I'd prefer
rtl::OUString sFoo(RTL_CONSTASCII_USTRINGPARAM(apple));
On Mon, 2010-11-01 at 20:28 +0100, Gert Faller wrote:
Hi,
more of that...
Mostly good, though createFromAscii_7.patch has...
OUString::createFromAscii( GetWindowState( WINDOWSTATE_MASK_POS ).GetBuffer() )
);
that one can't be changed to use RTL_CONSTASCII_USTRINGPARAM because
Selon Caolán McNamara caol...@redhat.com:
Mostly good, though createFromAscii_7.patch has...
OUString::createFromAscii( GetWindowState( WINDOWSTATE_MASK_POS ).GetBuffer()
) );
that one can't be changed to use RTL_CONSTASCII_USTRINGPARAM because
GetWindowState is a method rather than a
On Sat, 2010-10-30 at 22:08 +0100, Caolán McNamara wrote:
old: OUString sFoo(OUString::createFromAscii(bBool ? true : false));
new: OUString sFoo(RTL_CONSTASCII_USTRINGPARAM(bBool ? true :
false));
would be expanded as...
OUString sFoo(bBool ? true : false, ((sal_Int32)(sizeof(bBool ?
On Sat, 2010-10-30 at 17:08 +0200, Gert Faller wrote:
Hi,
a first try on that.
Excellent, these are little mini-optimizations and they all help.
Not sure if I let the original source (for control)
Nah, don't bother keeping the old content as a comment, we can look at
the git history to see
Selon Caolán McNamara caol...@redhat.com:
You have two typos in there however, i.e.
- com.sun.star.chart.BarDiagram
+ com.sun.star.chart.BarDiagra
- com.sun.star.chart2.ScatterChartType
+ com.sun.star.chart2.ScatterChartTyp
in those two cases the trailing letter of the original string
On Sat, 2010-10-30 at 18:16 +0200, Gert Faller wrote:
I've got 2388 files with 'createFromAscii'. That's a lot.
The wiki says that this macro is 'faster'. Does it mean at running time ?
Yes, ever so slightly, sal/inc/rtl/ustring.hxx has
OUString::createFromAscii in it, and that calls
OK, thanks.
Now, this only works when you can compute the length of the string at
compile time, i.e. its a string literal, so
OUString::createFromAscii(foo) can be trivially replaced, while
OUString::createFromAscii(pSomething) should be left alone.
C.
Yes, of course, I was just
On Sat, Oct 30, 2010 at 10:35 PM, Gert Faller gertfal...@aliceadsl.fr wrote:
OK, thanks.
Now, this only works when you can compute the length of the string at
compile time, i.e. its a string literal, so
OUString::createFromAscii(foo) can be trivially replaced, while
On Sat, 2010-10-30 at 22:23 +0200, Gert Faller wrote:
Hi,
here are some more of them.
Great, thanks.
Looking at this createFromAscii_4.patch reminds me of something else.
Nothing to do with you, the code was this way already, but
rtl::OUString foo =
11 matches
Mail list logo