Two equally valid preferences.

Code view:

A painting is a person's account of this edit as text, recorded independently of the node mapping - a rendering often has several equally defensible answers where the mapping has one. Deletions and insertions carry no position on the opposite side in a painting, so that panel draws no caret for them.

Before

…lines 1–283 unchanged (283 lines)
284 else if (!("yes".equalsIgnoreCase(value)
285 || "true".equalsIgnoreCase(value)
286 || "1".equalsIgnoreCase(value)))
287 {
288 // if the value is not yes, true or 1 then don't add the
289 // option to the CommandLine
290 break;
291 }
292
293 cmd.addOption(opt);
294 }
295 }
296 }
…lines 297–411 unchanged (115 lines)

After

…lines 1–283 unchanged (283 lines)
284 else if (!("yes".equalsIgnoreCase(value)
285 || "true".equalsIgnoreCase(value)
286 || "1".equalsIgnoreCase(value)))
287 {
288 // if the value is not yes, true or 1 then don't add the
289 // option to the CommandLine
290 continue;
291 }
292
293 cmd.addOption(opt);
294 }
295 }
296 }
…lines 297–411 unchanged (115 lines)

Before

program
block_comment "/**\n * Licensed to the Apache Software Foundation (ASF) unde"...
package_declaration
package "package"
scoped_identifier
scoped_identifier
scoped_identifier
identifier "org"
. "."
identifier "apache"
. "."
identifier "commons"
. "."
identifier "cli"
; ";"
import_declaration
import "import"
scoped_identifier
scoped_identifier
identifier "java"
. "."
identifier "util"
. "."
identifier "ArrayList"
; ";"
import_declaration
import "import"
scoped_identifier
scoped_identifier
identifier "java"
. "."
identifier "util"
. "."
identifier "Arrays"
; ";"
import_declaration
import "import"
scoped_identifier
scoped_identifier
identifier "java"
. "."
identifier "util"
. "."
identifier "Enumeration"
; ";"
import_declaration
import "import"
scoped_identifier
scoped_identifier
identifier "java"
. "."
identifier "util"
. "."
identifier "Iterator"
; ";"
import_declaration
import "import"
scoped_identifier
scoped_identifier
identifier "java"
. "."
identifier "util"
. "."
identifier "List"
; ";"
import_declaration
import "import"
scoped_identifier
scoped_identifier
identifier "java"
. "."
identifier "util"
. "."
identifier "ListIterator"
; ";"
import_declaration
import "import"
scoped_identifier
scoped_identifier
identifier "java"
. "."
identifier "util"
. "."
identifier "Properties"
; ";"
block_comment "/**\n * <code>Parser</code> creates {@link CommandLine}s.\n *\n"...
class_declaration
modifiers
public "public"
abstract "abstract"
class "class"
identifier "Parser"
super_interfaces
implements "implements"
type_list
type_identifier "CommandLineParser"
class_body
{ "{"
block_comment "/** commandline instance */"
field_declaration
modifiers
protected "protected"
type_identifier "CommandLine"
variable_declarator
identifier "cmd"
; ";"
block_comment "/** current Options */"
field_declaration
modifiers
private "private"
type_identifier "Options"
variable_declarator
identifier "options"
; ";"
block_comment "/** list of required options strings */"
field_declaration
modifiers
private "private"
type_identifier "List"
variable_declarator
identifier "requiredOptions"
; ";"
method_declaration (+46 nodes collapsed)
method_declaration
modifiers
protected "protected"
type_identifier "Options"
identifier "getOptions"
formal_parameters
( "("
) ")"
block
{ "{"
return_statement
return "return"
identifier "options"
; ";"
} "}"
method_declaration
modifiers
protected "protected"
type_identifier "List"
identifier "getRequiredOptions"
formal_parameters
( "("
) ")"
block
{ "{"
return_statement
return "return"
identifier "requiredOptions"
; ";"
} "}"
block_comment "/**\n * Subclasses must implement this method to reduce\n "...
method_declaration (+32 nodes collapsed)
block_comment "/**\n * Parses the specified <code>arguments</code> based"...
method_declaration (+40 nodes collapsed)
block_comment "/**\n * Parse the arguments according to the specified op"...
method_declaration (+44 nodes collapsed)
block_comment "/**\n * Parses the specified <code>arguments</code>\n "...
method_declaration (+44 nodes collapsed)
block_comment "/**\n * Parse the arguments according to the specified op"...
method_declaration (+555 nodes collapsed)
block_comment "/**\n * Sets the values of Options using the values in <c"...
method_declaration
modifiers
protected "protected"
void_type "void"
identifier "processProperties"
formal_parameters
( "("
formal_parameter
type_identifier "Properties"
identifier "properties"
) ")"
block
{ "{"
if_statement
if "if"
parenthesized_expression
( "("
binary_expression
identifier "properties"
== "=="
null_literal "null"
) ")"
block
{ "{"
return_statement
return "return"
; ";"
} "}"
for_statement
for "for"
( "("
local_variable_declaration
type_identifier "Enumeration"
variable_declarator
identifier "e"
= "="
method_invocation
identifier "properties"
. "."
identifier "propertyNames"
argument_list
( "("
) ")"
; ";"
method_invocation
identifier "e"
. "."
identifier "hasMoreElements"
argument_list
( "("
) ")"
; ";"
) ")"
block
{ "{"
local_variable_declaration
type_identifier "String"
variable_declarator
identifier "option"
= "="
method_invocation
method_invocation
identifier "e"
. "."
identifier "nextElement"
argument_list
( "("
) ")"
. "."
identifier "toString"
argument_list
( "("
) ")"
; ";"
if_statement
if "if"
parenthesized_expression
( "("
unary_expression
! "!"
method_invocation
identifier "cmd"
. "."
identifier "hasOption"
argument_list
( "("
identifier "option"
) ")"
) ")"
block
{ "{"
local_variable_declaration
type_identifier "Option"
variable_declarator
identifier "opt"
= "="
method_invocation
method_invocation
identifier "getOptions"
argument_list
( "("
) ")"
. "."
identifier "getOption"
argument_list
( "("
identifier "option"
) ")"
; ";"
line_comment "// get the value from the properties instance"
local_variable_declaration
type_identifier "String"
variable_declarator
identifier "value"
= "="
method_invocation
identifier "properties"
. "."
identifier "getProperty"
argument_list
( "("
identifier "option"
) ")"
; ";"
if_statement
if "if"
parenthesized_expression
( "("
method_invocation
identifier "opt"
. "."
identifier "hasArg"
argument_list
( "("
) ")"
) ")"
block (+63 nodes collapsed)
else "else"
if_statement
if "if"
parenthesized_expression (+45 nodes collapsed)
block
{ "{"
line_comment "// if the value is not yes, true or 1 then don't add the"
line_comment "// option to the CommandLine"
break_statement
break "break"
; ";"
} "}"
expression_statement
method_invocation
identifier "cmd"
. "."
identifier "addOption"
argument_list
( "("
identifier "opt"
) ")"
; ";"
} "}"
} "}"
} "}"
block_comment "/**\n * Throws a {@link MissingOptionException} if all of"...
method_declaration (+50 nodes collapsed)
block_comment "/**\n * <p>Process the argument values for the specified "...
method_declaration (+185 nodes collapsed)
block_comment "/**\n * Process the Option specified by <code>arg</code> "...
method_declaration (+254 nodes collapsed)
} "}"

After

program
block_comment "/**\n * Licensed to the Apache Software Foundation (ASF) unde"...
package_declaration
package "package"
scoped_identifier
scoped_identifier
scoped_identifier
identifier "org"
. "."
identifier "apache"
. "."
identifier "commons"
. "."
identifier "cli"
; ";"
import_declaration
import "import"
scoped_identifier
scoped_identifier
identifier "java"
. "."
identifier "util"
. "."
identifier "ArrayList"
; ";"
import_declaration
import "import"
scoped_identifier
scoped_identifier
identifier "java"
. "."
identifier "util"
. "."
identifier "Arrays"
; ";"
import_declaration
import "import"
scoped_identifier
scoped_identifier
identifier "java"
. "."
identifier "util"
. "."
identifier "Enumeration"
; ";"
import_declaration
import "import"
scoped_identifier
scoped_identifier
identifier "java"
. "."
identifier "util"
. "."
identifier "Iterator"
; ";"
import_declaration
import "import"
scoped_identifier
scoped_identifier
identifier "java"
. "."
identifier "util"
. "."
identifier "List"
; ";"
import_declaration
import "import"
scoped_identifier
scoped_identifier
identifier "java"
. "."
identifier "util"
. "."
identifier "ListIterator"
; ";"
import_declaration
import "import"
scoped_identifier
scoped_identifier
identifier "java"
. "."
identifier "util"
. "."
identifier "Properties"
; ";"
block_comment "/**\n * <code>Parser</code> creates {@link CommandLine}s.\n *\n"...
class_declaration
modifiers
public "public"
abstract "abstract"
class "class"
identifier "Parser"
super_interfaces
implements "implements"
type_list
type_identifier "CommandLineParser"
class_body
{ "{"
block_comment "/** commandline instance */"
field_declaration
modifiers
protected "protected"
type_identifier "CommandLine"
variable_declarator
identifier "cmd"
; ";"
block_comment "/** current Options */"
field_declaration
modifiers
private "private"
type_identifier "Options"
variable_declarator
identifier "options"
; ";"
block_comment "/** list of required options strings */"
field_declaration
modifiers
private "private"
type_identifier "List"
variable_declarator
identifier "requiredOptions"
; ";"
method_declaration (+46 nodes collapsed)
method_declaration
modifiers
protected "protected"
type_identifier "Options"
identifier "getOptions"
formal_parameters
( "("
) ")"
block
{ "{"
return_statement
return "return"
identifier "options"
; ";"
} "}"
method_declaration
modifiers
protected "protected"
type_identifier "List"
identifier "getRequiredOptions"
formal_parameters
( "("
) ")"
block
{ "{"
return_statement
return "return"
identifier "requiredOptions"
; ";"
} "}"
block_comment "/**\n * Subclasses must implement this method to reduce\n "...
method_declaration (+32 nodes collapsed)
block_comment "/**\n * Parses the specified <code>arguments</code> based"...
method_declaration (+40 nodes collapsed)
block_comment "/**\n * Parse the arguments according to the specified op"...
method_declaration (+44 nodes collapsed)
block_comment "/**\n * Parses the specified <code>arguments</code>\n "...
method_declaration (+44 nodes collapsed)
block_comment "/**\n * Parse the arguments according to the specified op"...
method_declaration (+555 nodes collapsed)
block_comment "/**\n * Sets the values of Options using the values in <c"...
method_declaration
modifiers
protected "protected"
void_type "void"
identifier "processProperties"
formal_parameters
( "("
formal_parameter
type_identifier "Properties"
identifier "properties"
) ")"
block
{ "{"
if_statement
if "if"
parenthesized_expression
( "("
binary_expression
identifier "properties"
== "=="
null_literal "null"
) ")"
block
{ "{"
return_statement
return "return"
; ";"
} "}"
for_statement
for "for"
( "("
local_variable_declaration
type_identifier "Enumeration"
variable_declarator
identifier "e"
= "="
method_invocation
identifier "properties"
. "."
identifier "propertyNames"
argument_list
( "("
) ")"
; ";"
method_invocation
identifier "e"
. "."
identifier "hasMoreElements"
argument_list
( "("
) ")"
; ";"
) ")"
block
{ "{"
local_variable_declaration
type_identifier "String"
variable_declarator
identifier "option"
= "="
method_invocation
method_invocation
identifier "e"
. "."
identifier "nextElement"
argument_list
( "("
) ")"
. "."
identifier "toString"
argument_list
( "("
) ")"
; ";"
if_statement
if "if"
parenthesized_expression
( "("
unary_expression
! "!"
method_invocation
identifier "cmd"
. "."
identifier "hasOption"
argument_list
( "("
identifier "option"
) ")"
) ")"
block
{ "{"
local_variable_declaration
type_identifier "Option"
variable_declarator
identifier "opt"
= "="
method_invocation
method_invocation
identifier "getOptions"
argument_list
( "("
) ")"
. "."
identifier "getOption"
argument_list
( "("
identifier "option"
) ")"
; ";"
line_comment "// get the value from the properties instance"
local_variable_declaration
type_identifier "String"
variable_declarator
identifier "value"
= "="
method_invocation
identifier "properties"
. "."
identifier "getProperty"
argument_list
( "("
identifier "option"
) ")"
; ";"
if_statement
if "if"
parenthesized_expression
( "("
method_invocation
identifier "opt"
. "."
identifier "hasArg"
argument_list
( "("
) ")"
) ")"
block (+63 nodes collapsed)
else "else"
if_statement
if "if"
parenthesized_expression (+45 nodes collapsed)
block
{ "{"
line_comment "// if the value is not yes, true or 1 then don't add the"
line_comment "// option to the CommandLine"
continue_statement
continue "continue"
; ";"
} "}"
expression_statement
method_invocation
identifier "cmd"
. "."
identifier "addOption"
argument_list
( "("
identifier "opt"
) ")"
; ";"
} "}"
} "}"
} "}"
block_comment "/**\n * Throws a {@link MissingOptionException} if all of"...
method_declaration (+50 nodes collapsed)
block_comment "/**\n * <p>Process the argument values for the specified "...
method_declaration (+185 nodes collapsed)
block_comment "/**\n * Process the Option specified by <code>arg</code> "...
method_declaration (+254 nodes collapsed)
} "}"