Put the debug check condition always. Follow it religiously. On Fri, Jan 9, 2015 at 5:09 PM, Gayan Gunawardana <ga...@wso2.com> wrote:
> Hi All, > > Currently I am going through user core exception handling and identifying > changes required to do. > > In user core [1] there are couple of places debug logs exists without > isDebugEnabled check. > > Ex:getSearchBaseDirectoryContext(), setUserClaims() > > Is there any rule for places where we can ignore isDebugEnabled check or > should we do isDebugEnabled check always? > > [1] > https://github.com/wso2-dev/carbon4-kernel/blob/master/core/org.wso2.carbon.user.core/src/main/java/org/wso2/carbon/user/core/ldap/ReadWriteLDAPUserStoreManager.java > -- > Gayan Gunawardana > Software Engineer; WSO2 Inc.; http://wso2.com/ > Email: ga...@wso2.com > Mobile: +94 (71) 8020933 > -- *Afkham Azeez* Director of Architecture; WSO2, Inc.; http://wso2.com Member; Apache Software Foundation; http://www.apache.org/ * <http://www.apache.org/>* *email: **az...@wso2.com* <az...@wso2.com> * cell: +94 77 3320919blog: **http://blog.afkham.org* <http://blog.afkham.org> *twitter: **http://twitter.com/afkham_azeez* <http://twitter.com/afkham_azeez> *linked-in: **http://lk.linkedin.com/in/afkhamazeez <http://lk.linkedin.com/in/afkhamazeez>* *Lean . Enterprise . Middleware*
_______________________________________________ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev
