|
From: Alon Bar-L. <alo...@gm...> - 2014-05-18 10:23:50
|
On Sun, May 18, 2014 at 12:28 PM, Neil A. Wilson <nei...@un...> wrote: > On 05/17/2014 06:59 AM, Alon Bar-Lev wrote: >> >> Hello, >> >> Priority: LOW, nice to have. >> >> As most of the classes are final because of reason previously >> discussed, it is impossible to attach user data to important >> instances, such as LDAPConnection and LDAPConnectionPool and get these >> when callback is invoked, such as async. >> >> I suggest to add Map<Object, Object> to at least >> LDAPConnectionOptions, LDAPConnection, LDAPConnectionPool if not for >> LDAPInterface, to allow application to attach own data and state to >> these artifacts, so these will be available at every point within >> application that receives these instances. > > > I don't really see any substantial value in providing this. It adds > unnecessary clutter to the API without providing a capability that you can't > get just as easily by maintaining a map in your own code. OK, thanks, although usually you can extend a class and then get the required information out of it when it is available in a callback. I am unsure how it can be done without wrapping every class of SDK that requires attachment of data. > > > >> >> Regards, >> Alon Bar-Lev. >> >> >> ------------------------------------------------------------------------------ >> "Accelerate Dev Cycles with Automated Cross-Browser Testing - For FREE >> Instantly run your Selenium tests across 300+ browser/OS combos. >> Get unparalleled scalability from the best Selenium testing platform >> available >> Simple to use. Nothing to install. Get started now for free." >> http://p.sf.net/sfu/SauceLabs >> _______________________________________________ >> ldap-sdk-discuss mailing list >> lda...@li... >> https://lists.sourceforge.net/lists/listinfo/ldap-sdk-discuss >> > > > > ------------------------------------------------------------------------------ > "Accelerate Dev Cycles with Automated Cross-Browser Testing - For FREE > Instantly run your Selenium tests across 300+ browser/OS combos. > Get unparalleled scalability from the best Selenium testing platform > available > Simple to use. Nothing to install. Get started now for free." > http://p.sf.net/sfu/SauceLabs > _______________________________________________ > ldap-sdk-discuss mailing list > lda...@li... > https://lists.sourceforge.net/lists/listinfo/ldap-sdk-discuss > |
