I think you're right about the pull request.  I just saw your email and 
went to issue the next pull request but the icon doesn't appear.  It 
seems your merge this evening has included both sets of changes.

On 19/06/2013 6:53 PM, Andrew Robinson wrote:
>
> In src/Mod/Cam2/App/TPG/CppTPGFactory.cpp:
>
> > @@ -61,6 +66,11 @@ void CppTPGFactoryInst::scanPlugins()
> >  	QString path = QString::fromAscii(App::GetApplication().Config()["AppHomePath"].c_str());
> >  	path.append(QString::fromAscii("Mod/Cam/CppTPG/"));
> >
> > +	/*
> > +	// NOTE: We cannot delete the plugins here because pointers to them are still held by
> > +	// parent objects (CppTPGDescriptorWrapper).  As an alternative, we're going to check each file by name and only
> > +	// load those that we have not previously seen.
> > +
>
> Cool, I think If I accept the pull request it will bring in any new 
> items you have added since making the request as well, (at least that 
> was what it did for me a few months back). I just had a quick look at 
> the CPP Plugin stuff and two things came to mind: (1) the reference 
> counter starts off at 1 so its possible that the 
> CppTPGDescriptorWrapper is just using that copy; this is probably a 
> bad design decision which we should probably address, and (2) I think 
> the CppTPGDescriptorWrapper may not be needed anymore since the Delete 
> descriptor library function has been removed though this will require 
> a clearer mind that one that has been at work all day. I think I will 
> just accept your request (after I build/run test it) and have a look 
> at improving the code on the weekend.
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/andrewjrobinson/FreeCAD_sf_master/pull/5/files#r4768051>.
>