Skip to content

Commit 5f391fb

Browse files
committed
a little simplification...
1 parent fb8f473 commit 5f391fb

File tree

1 file changed

+10
-14
lines changed

1 file changed

+10
-14
lines changed

src/org/argmap/client/ViewPropEdit.java

Lines changed: 10 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -536,10 +536,12 @@ else if (charCode == KeyCodes.KEY_BACKSPACE
536536
&& !isLink() && !isTopLevel()) {
537537
if (getChildCount() == 0) {
538538
/* merge With Previous Sibling */
539-
mergePropsAndDeleteOne(preceedingSibling, this, true);
539+
mergePropsAndDeleteOne(preceedingSibling, this,
540+
preceedingSibling, this);
540541
} else if (preceedingSibling.getChildCount() == 0) {
541542
/* merge Previous Sibling With This */
542-
mergePropsAndDeleteOne(preceedingSibling, this, false);
543+
mergePropsAndDeleteOne(preceedingSibling, this, this,
544+
preceedingSibling);
543545
} else {
544546
/* do nothing; ignore keystroke */
545547
}
@@ -579,10 +581,12 @@ else if (charCode == KeyCodes.KEY_DELETE
579581
&& !isLink() && !isTopLevel()) {
580582
if (getChildCount() == 0) {
581583
/* merge With Subsequent Sibling */
582-
mergePropsAndDeleteOne(this, followingSibling, false);
584+
mergePropsAndDeleteOne(this, followingSibling,
585+
followingSibling, this);
583586
} else if (followingSibling.getChildCount() == 0) {
584587
/* merge Subsequent Sibling With This */
585-
mergePropsAndDeleteOne(this, followingSibling, true);
588+
mergePropsAndDeleteOne(this, followingSibling, this,
589+
followingSibling);
586590
} else {
587591
/* do nothing; ignore keystroke */
588592
}
@@ -612,16 +616,8 @@ else if (charCode == KeyCodes.KEY_DELETE
612616
}
613617

614618
private void mergePropsAndDeleteOne(ViewPropEdit firstProp,
615-
ViewPropEdit secondProp, boolean keepFirst) {
616-
ViewPropEdit keepProp;
617-
ViewPropEdit deleteProp;
618-
if (keepFirst) {
619-
keepProp = firstProp;
620-
deleteProp = secondProp;
621-
} else {
622-
keepProp = secondProp;
623-
deleteProp = firstProp;
624-
}
619+
ViewPropEdit secondProp, ViewPropEdit keepProp,
620+
ViewPropEdit deleteProp) {
625621
String firstText = firstProp.textArea.getText();
626622
keepProp.textArea.setText(firstText + secondProp.textArea.getText());
627623
keepProp.textArea.setCursorPos(firstText.length());

0 commit comments

Comments
 (0)