From: SourceForge.net <no...@so...> - 2011-02-04 14:41:33
|
Patches item #3163114, was opened at 2011-01-21 04:08 Message generated for change (Comment added) made by egonw You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=3163114&group_id=20024 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: cdk-1.4.x Group: Needs Review >Status: Closed >Resolution: Fixed Priority: 5 Private: No Submitted By: Rajarshi Guha (rajarshi) Assigned to: Egon Willighagen (egonw) Summary: Cache parsed SMARTS queries Initial Comment: A simple LRU caching mechanism to avoid reparsing previously used SMARTS queries ---------------------------------------------------------------------- >Comment By: Egon Willighagen (egonw) Date: 2011-02-04 15:41 Message: Applied to cdk-1.4.x: https://github.com/cdk/cdk/commit/c3a768d356ddd6998901e81d5cccef67ec02f605 ---------------------------------------------------------------------- Comment By: Rajarshi Guha (rajarshi) Date: 2011-01-23 23:14 Message: 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. ---------------------------------------------------------------------- Comment By: Egon Willighagen (egonw) Date: 2011-01-23 21:07 Message: 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... ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=320024&aid=3163114&group_id=20024 |