[jira] [Commented] (DIRKRB-224) Enhance kadmin to support delete principal

["Lin Chen (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/DIRKRB-224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14504425#comment-14504425 ] 

Lin Chen commented on DIRKRB-224:
---------------------------------

Hi Jiajia, 
    The work looks great! A minor thing:
{code}
} else if (reply.equals("no") || reply.equals("NO") || reply.equals("n") || reply.equals("N")) {
    return;
} else {
    System.err.println("unknow request, fail to delete the principal");
    return;
}
{code}
I am not sure, is {{return}} useful here?


> Enhance kadmin to support delete principal
> ------------------------------------------
>
>                 Key: DIRKRB-224
>                 URL: https://issues.apache.org/jira/browse/DIRKRB-224
>             Project: Directory Kerberos
>          Issue Type: New Feature
>            Reporter: Jiajia Li
>            Assignee: Jiajia Li
>         Attachments: DIRKRB-224-V1.patch
>
>
> Enhance kadmin to support delete principal.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)