#311 Cache parsed SMARTS queries

Accepted
closed
cdk-1.4.x (181)
5
2012-10-28
2011-01-21
Rajarshi Guha
No

A simple LRU caching mechanism to avoid reparsing previously used SMARTS queries

Discussion

  • I assume the changes to the JavaDoc is only whitespace? You know my usual reply on these things... :) You did update the JavaDoc to reflect the cache?

    Can the cache be sized via a get/set method? Did not see one, but may have overlooked it because of all the JavaDoc changes ...

    Anyways, otherwise, looks clean to me... I am now compiling, if online, please ping on JavaDoc...

     
  • Rajarshi Guha
    Rajarshi Guha
    2011-01-23

    Was on a flight when I saw the email - don't know what happened to the docs, but I updated the patch to fix the Jaavdocs and add a setter for the cache size.