Abdelrazak Younes <[EMAIL PROTECTED]> writes:

| Lars Gullik Bjønnes wrote:
| > Abdelrazak Younes <[EMAIL PROTECTED]> writes:
| > | This simple patch provides a nice speedup without any visual side
| > effect.
| > | | Comments? Objection?
| > No, objections.
| 
| What's that? "No, I have an objection" or "No objection." ?

d/,/

| 
| >  | Abdel.
| > | Index: unicode.C
| > | ===================================================================
| > | --- unicode.C     (revision 14849)
| > | +++ unicode.C     (working copy)
| > | @@ -49,7 +49,7 @@
| > |  |        char ICONV_CONST * inbuf = const_cast<char ICONV_CONST
| > *>(&buf[0]);
| > |   size_t inbytesleft = buf.size();
| > | - char out[1000] = { 0 };
| > | + static char out[1000];
| > Did you see if you get the same speedup if you just disable the
| > initialization "{ 0 };"?
| 
| No I didn't try. You reckon that the compiler will optimize that
| automatically?

No, but that the initialization might be the cause of the slowdown.

-- 
        Lgb

Reply via email to