MartinKanters edited a comment on pull request #639: URL: https://github.com/apache/maven/pull/639#issuecomment-1008035176
> RU sure you don't need to perform lookup on each call (instead to keep the map in final member?) This is a singleton, so the map you lookup in constructor, once inited, would not change.... Ah, then it is indeed best to lookup the map once and store it in an member variable. Good suggestion! -- edit: just realized you meant exactly the opposite. Anyway, changed it accordingly. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
