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

Codeblocks ported to sqrat

<< < (6/11) > >>

BlueHazzard:

--- Quote from: LETARTARE on April 16, 2014, 09:47:57 am ---Hello,
I unfortunately do not currently have the time to test your excellent work.
It's a pity, because I have "wxTranslatepluginForCB" transpose.
I recouped "https://github.com/bluehazzard/codeblocks_sf" by Tortoise.
Sincerely

--- End quote ---
remember if you want to try your plugin you have to add the line

--- Code: ---info = PluginInfo();
--- End code ---
in your constructor and to run the converter script like i described in my first post.

BlueHazzard:

--- Quote from: oBFusCATed on April 16, 2014, 11:41:47 pm ---Two patches, so far:
http://cmpt.benbmp.org/codeblocks/patches/sqrat/0001-scripting-no-pch-build-fix.patch
http://cmpt.benbmp.org/codeblocks/patches/sqrat/0002-scripting-Fix-compilation-it-seems-that-Play-doesn-t.patch

--- End quote ---

I'm not at my main pc so i can't test. Thanks for the work, because i never did something with PCH...


--- Quote from: oBFusCATed on April 16, 2014, 11:41:47 pm ---1. There is an include for sqplus in a lot of places, still.
2. sqrat/sqrat looks ugly, I'd prefer if you remove one of the levels.
3. It will be good if all public headers for scripting could be included without the need of additional paths
4. Please keep filename only with lower case letters and underscores.

--- End quote ---

1. Hmm... i will search and remove them
2. indeed this is really ugly, but i don't wanted to change the sqrat code (the code in sqrat.h) and leave it simple to update sqrat, by only coping the sqrat folder
3. It shouldn't be necessary to use additional paths outside the scripting internal stuff. Inside scripting (like the wx thing) i used additional paths, because it was a lot work to keep in sync all include directories over all project files. I will again look into it. Thanks for pointing it out.
4. Will look into it,


--- Quote from: oBFusCATed on April 16, 2014, 11:41:47 pm ---Is there any real reason you have to bind the wxXXXBase class? This is an implementation detail the wx user shouldn't care!

--- End quote ---
I tried to compile the code on linux today and hit the same errors. Sqrat needs the base classes for its internal type checking. I think there are needed some #ifdef because some wx controls don't have a base on linux... Still some work... ;(

greetings

oBFusCATed:

--- Quote from: BlueHazzard on April 19, 2014, 10:53:02 pm ---2. indeed this is really ugly, but i don't wanted to change the sqrat code (the code in sqrat.h) and leave it simple to update sqrat, by only coping the sqrat folder

--- End quote ---
I don't see what the problem will be. Now you have sqrat/sqrat.h and sqrat/sqrat/* if you remove the top folder you'll have sqrat.h and sqrat/* and I don't see how this will make the updating of sqrat tough and problematic.

BlueHazzard:
just uploaded a fix for the unix build... (not pretty but functional, and i don't think there is a better solution, because of the different implementation on every platform...)

greetings

oBFusCATed:

--- Code: ---/home/obfuscated/projects/codeblocks/git/src/sdk/menuitemsmanager.cpp|118|warning: deleting object of abstract class type ‘Menu_point_base’ which has non-virtual destructor will cause undefined behaviour [-Wdelete-non-virtual-dtor]|

--- End code ---
This is pretty bad warning/error, please fix!

Edit:

--- Code: ---/home/obfuscated/projects/codeblocks/git/src/sdk/menuitemsmanager.cpp|201|warning: unused variable ‘sub_menu’ [-Wunused-variable]|

--- End code ---
This one seems like a bug, too.

Navigation

[0] Message Index

[#] Next page

[*] Previous page