< prev index next >

src/java.base/share/classes/javax/security/auth/callback/ConfirmationCallback.java

Print this page
rev 51919 : 8215281: Use String.isEmpty() when applicable in java.base
Reviewed-by: dfuchs, alanb

*** 246,256 **** options == null || options.length == 0 || defaultOption < 0 || defaultOption >= options.length) throw new IllegalArgumentException(); for (int i = 0; i < options.length; i++) { ! if (options[i] == null || options[i].length() == 0) throw new IllegalArgumentException(); } this.messageType = messageType; this.options = options; --- 246,256 ---- options == null || options.length == 0 || defaultOption < 0 || defaultOption >= options.length) throw new IllegalArgumentException(); for (int i = 0; i < options.length; i++) { ! if (options[i] == null || options[i].isEmpty()) throw new IllegalArgumentException(); } this.messageType = messageType; this.options = options;
*** 292,302 **** * {@code optionType}. */ public ConfirmationCallback(String prompt, int messageType, int optionType, int defaultOption) { ! if (prompt == null || prompt.length() == 0 || messageType < INFORMATION || messageType > ERROR || optionType < YES_NO_OPTION || optionType > OK_CANCEL_OPTION) throw new IllegalArgumentException(); switch (optionType) { --- 292,302 ---- * {@code optionType}. */ public ConfirmationCallback(String prompt, int messageType, int optionType, int defaultOption) { ! if (prompt == null || prompt.isEmpty() || messageType < INFORMATION || messageType > ERROR || optionType < YES_NO_OPTION || optionType > OK_CANCEL_OPTION) throw new IllegalArgumentException(); switch (optionType) {
*** 355,372 **** * {@code options}. */ public ConfirmationCallback(String prompt, int messageType, String[] options, int defaultOption) { ! if (prompt == null || prompt.length() == 0 || messageType < INFORMATION || messageType > ERROR || options == null || options.length == 0 || defaultOption < 0 || defaultOption >= options.length) throw new IllegalArgumentException(); for (int i = 0; i < options.length; i++) { ! if (options[i] == null || options[i].length() == 0) throw new IllegalArgumentException(); } this.prompt = prompt; this.messageType = messageType; --- 355,372 ---- * {@code options}. */ public ConfirmationCallback(String prompt, int messageType, String[] options, int defaultOption) { ! if (prompt == null || prompt.isEmpty() || messageType < INFORMATION || messageType > ERROR || options == null || options.length == 0 || defaultOption < 0 || defaultOption >= options.length) throw new IllegalArgumentException(); for (int i = 0; i < options.length; i++) { ! if (options[i] == null || options[i].isEmpty()) throw new IllegalArgumentException(); } this.prompt = prompt; this.messageType = messageType;
< prev index next >