commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Rory Winston (JIRA)" <>
Subject [jira] Commented: (NET-310) FTPCommand conversion to use enum
Date Sat, 03 Apr 2010 08:51:27 GMT


Rory Winston commented on NET-310:

I dont like the approach of using the ordinals, its too fragile and can lead to the same problems
as using the current integer-based constants. If we are going to use enums, I would do it
in a 3.0 release and ditch the integer-based command ordinal compatability.

> FTPCommand conversion to use enum
> ---------------------------------
>                 Key: NET-310
>                 URL:
>             Project: Commons Net
>          Issue Type: Improvement
>            Reporter: Sebb
>            Priority: Minor
>         Attachments: FTPCommandEnum.patch
> Seems to me it would be sensible to replace the integer values in the FTPCommand class
with an enum.
> I propose creating a new enum class (called FTPCommands) which contains the enum and
aliases. The existing FTPCommand class is then rewritten using the enum ordinals; and the
FTP class is changed to use the enums internally. Existing public methods which use an integer
for the command id are deprecated.
> If there are any further commands added to the set lter, they should only be added to
the enum, and no corresponding ints should be defined.
> See attached patch (to follow). I've not fixed the FTP subclasses yet as I wanted to
get agreement on the approach first.
> Note that the new class name FTPCommands is very similar to the old class FTPCommand;
of course that can be changed.
> Comments? Suggestions?

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message