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

New event - cbEVT_EDITOR_SAVE_ALL

(1/2) > >>

oBFusCATed:
Hello,

Is it a good idea the addition of cbEVT_EDITOR_SAVE_ALL?
For example, this event will be fired when the compiler starts to build the project (currently it calls project->SaveAllFiles() :( )  every other plugin/core needs to save all files.
I want this event in order to make the wxSmith plugin save the *.wxs file, when the user compiles the project.
At the moment when you hit compile in a wxSmith project the *.wxs is not saved, which is a bit disturbing and not very user friendly.

MortenMacFly:

--- Quote from: oBFusCATed on February 16, 2010, 10:33:56 pm ---Is it a good idea the addition of cbEVT_EDITOR_SAVE_ALL?

--- End quote ---
Why don't you simply use cbEVT_COMPILER_STARTED?

oBFusCATed:
That was my first idea, but adding the event SAVE_ALL sounds more logical and robust (can save all files on other events, not sure which :) ).

Maybe the event name should be other, because cbEVT_EDITOR_* is for the editors... probably cbEVT_PROJECT_SAVE_FILES is better one.

MortenMacFly:

--- Quote from: oBFusCATed on February 17, 2010, 08:14:19 am ---That was my first idea, but adding the event SAVE_ALL sounds more logical and robust (can save all files on other events, not sure which :) ).

--- End quote ---
Make it cbEVT_PROJECT_SAVE_ALL_FILES then to comply with the method's name.

oBFusCATed:
OK, One more thing - should I fire it from the inside of project->SaveAllFiles() or I need to modify all places that call SaveAllFiles() to fire the event?

Navigation

[0] Message Index

[#] Next page