[jira] [Updated] (DIRAPI-167) Make LdapConnection extend java.io.Closeable

["Barney Barumba (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/DIRAPI-167?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Barney Barumba updated DIRAPI-167:
----------------------------------

    Attachment: DIRAPI-167-directory-client-api.patch

There's already a comment in the code that suggests the devs are aware of this:

{{// TODO: why do connect() and close() return a boolean? What is the difference between false and an Exception?}}

Anyway, as you said, the change is pretty simple, so here's a patch to save a few lines of typing if the devs decide to go ahead and make this change. This only addresses the {{close}} part, not the {{connect}} part.

Cheers,

> Make LdapConnection extend java.io.Closeable
> --------------------------------------------
>
>                 Key: DIRAPI-167
>                 URL: https://issues.apache.org/jira/browse/DIRAPI-167
>             Project: Directory Client API
>          Issue Type: Improvement
>            Reporter: lucas theisen
>            Priority: Minor
>         Attachments: DIRAPI-167-directory-client-api.patch
>
>
> This very small modification would allow us to leverage the Java 7 try-with-resources feature.  The proper close method is already part of LdapConnection, so the modification is extremely trivial.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)