On Fri, Oct 14, 2011 at 12:04 PM, My Th <rei4...@gmail.com> wrote: > Here is a patch for plugin constructors. It is a bit big because I > decompacted the macros to make them readable, but there is really only > one change - an "if" statement which checks if the ID is already in the > Map() and only if not then it assigns "this" to Map() and PluginMap(). > > With this patch there is no need for any of the previous patches. > > I'm not sure if anyone has relied on the availability of the new > instances in plugin map, but AFAIK this is the simplest way to make > MakeNewInstance() work and be thread safe. > > I'm curious to hear opinions from others :)
This patch fixes up the problems I've been seeing. Let's see what the people with more plugin experience think. Dave ------------------------------------------------------------------------------ All the data continuously generated in your IT infrastructure contains a definitive record of customers, application performance, security threats, fraudulent activity and more. Splunk takes this data and makes sense of it. Business sense. IT sense. Common sense. http://p.sf.net/sfu/splunk-d2d-oct _______________________________________________ OpenBabel-Devel mailing list OpenBabel-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openbabel-devel
