Developer forums (C::B DEVELOPMENT STRICTLY!) > Development

Code::Blocks, Scintilla and Fedora

<< < (8/9) > >>

Loaden:

--- Quote from: MortenMacFly on November 08, 2010, 08:13:01 pm ---So again my question: What's the issue with it?

--- End quote ---
This will increase the difficulty of maintenance.
I think we should change the scintilla/wxScintilla as little as possible.

Loaden:

--- Quote from: Loaden on November 08, 2010, 04:42:20 pm ---
--- Quote from: jens on November 08, 2010, 12:21:17 pm ---Can you test my new patch, works for me on XP SP3, wx2.8.10 and TDM gcc 4.5.0 (just for the core project-file at the moment).

--- End quote ---
Well done! testing passed.
And have a issue, I think we need change the lib name from "libwxscintilla.a" to "libcbscintilla.a" or "libwxscintilla_cb.a".
Because in wx2.9.x, the wxscintilla library named "libwxscintilla.a" too. :)

--- End quote ---
Any comments about the library name?

BTW, I am testting in XPSP3, if build wxScintilla as static library, the core (not include contrib) result is : 16.5MB.
And, If build wxScintilla as shared library, the core reulst is 16.5 MB too.
In fact, only codeblocks.dll size have changed.

--- Code: ---DLL Name static build shared build
codeblocks.dll 4.76MB 3.62MB

--- End code ---

Loaden:
Here is a patch for wx2.9.2, build wxScintilla as static library too. :lol:

Biplab:
If a struct is inside struct and we want to initialize it to zero we need to enclose it with { } Thus gcc was throwing warning on the original code.

So correct form should be -

--- Code: ---struct foo f = { {0}, 0};
--- End code ---

M$VC not being standard compliant can ignore such code. :)

MortenMacFly:

--- Quote from: Biplab on November 09, 2010, 03:37:26 am ---
--- Code: ---struct foo f = { {0}, 0};
--- End code ---

--- End quote ---
That sounds reasonable to me. If that removes the error and is standard compliant we can go for it. However, we would still have these portions in the code patched than... ;-)

Navigation

[0] Message Index

[#] Next page

[*] Previous page