Workspace loading and Active Project

[daniloz]

you mentioned a plugin you use for developement: "DisplayEvents-plugin". Where do I find this one to use it myself?

Search the forums. ;-)

I also indexed it in the plugins page about a month ago.

And here is the patch for the DisplayEvents plugin, so we also see the new event...
(sorry about the "hardcoded" paths, I don't know how to do it better - suggestions are welcome)

[oBFusCATed]

You're mixing tabs and spaces, C::B source is all spaces and you've not increased the SDK version numbers,
but you've changed the API/ABI for the ProjectManager. The SDK version numbers are in cbplugin.h.


Edit:
For the patch generation, goto the root of the working copy and execute "svn diff > my.patch".
This is the most robust way to generate patches. I've seen tortoisesvn mess up the patches it creates.

[MortenMacFly]

If you do what I said here:

If you really need an event for the case "workspace completely loaded", provide a patch accordingly. It should be really simple to implement. Create an event in sdk_events{.cpp/.h} and instrument the method "ProjectManager::LoadWorkspace" accordingly.

You wouldn't even need to change the API interface. I think putting this event in WorkspaceChanged is the wrong place.

[daniloz]

If you do what I said here:

If you really need an event for the case "workspace completely loaded", provide a patch accordingly. It should be really simple to implement. Create an event in sdk_events{.cpp/.h} and instrument the method "ProjectManager::LoadWorkspace" accordingly.

You wouldn't even need to change the API interface. I think putting this event in WorkspaceChanged is the wrong place.

Hey, Morten, sorry.... I haven't really understood what you've suggested. Now I do...
I'll change my implementation on Monday (hopefully) and provide a new patch as soon as available...
And maybe I'll change the event name to cbEVT_WORKSPACE_LOADING_COMPLETE, that would be more uniform with the other events names...

[MortenMacFly]

And maybe I'll change the event name to cbEVT_WORKSPACE_LOADING_COMPLETE, that would be more uniform with the other events names...

True... I'll wait patiently...

[daniloz]

And maybe I'll change the event name to cbEVT_WORKSPACE_LOADING_COMPLETE, that would be more uniform with the other events names...

True... I'll wait patiently...

I don't want to keep you wanting, so here it is...
The patch for cbEVT_WORKSPACE_LOADING_COMPLETE and for the DisplayEvents plugin.

@jens: any better way do do a patch for DisplayEvents than having a strange path for the original and patched versions?