Re: [Cdk-devel] [ cdk-Patches-3163114 ] Cache parsed SMARTS queries

[Egon W. <ego...@gm...>]

On Mon, Jan 24, 2011 at 8:26 AM, Andrew Dalke <da...@da...> wrote:
> Wouldn't it be easier to just make the PubchemSubstructure implementation use actual SMARTS objects, rather than go through a cache that it can't control?

Is it an idea to separate the cache into it's own class
(ISMARTSFooCache; yeah, I have an interface fetish), and have the tool
using the SMARTSQueryTool set that instead?

Egon

-- 
Dr E.L. Willighagen
Postdoctoral Researcher
Institutet för miljömedicin
Karolinska Institutet
Homepage: http://egonw.github.com/
LinkedIn: http://se.linkedin.com/in/egonw
Blog: http://chem-bla-ics.blogspot.com/
PubList: http://www.citeulike.org/user/egonw/tag/papers