Bug 160749

Can't set a breakpoint on the first line of an editor

1 files changed, 21 insertions, 25 deletions

diff --git a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/actions/ToggleBreakpointAction.java b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/actions/ToggleBreakpointAction.java
index 071942652..a1aa400ba 100644
--- a/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/actions/ToggleBreakpointAction.java
+++ b/org.eclipse.debug.ui/ui/org/eclipse/debug/ui/actions/ToggleBreakpointAction.java
@@ -177,33 +177,29 @@ public class ToggleBreakpointAction extends Action implements IUpdate {
 			}
 			if (adapter != null) {
 				int line = fRulerInfo.getLineOfLastMouseButtonActivity();
-				
-				// Test if line is valid 
-				if (line > 0) {
-					/*
-					 * XXX: remove once the following bug is fixed:
-					 * 		https://bugs.eclipse.org/bugs/show_bug.cgi?id=99234
-					 */ 
-					if (line < document.getNumberOfLines()) {
-						try {
-							IRegion region = document.getLineInformation(line);
-							ITextSelection selection = new TextSelection(document, region.getOffset(), 0);
-							if (adapter instanceof IToggleBreakpointsTargetExtension) {
-								IToggleBreakpointsTargetExtension extension = (IToggleBreakpointsTargetExtension) adapter;
-								if (extension.canToggleBreakpoints(fPart, selection)) {
-									setEnabled(true);
-									return;
-								}
+				/*
+				 * XXX: remove once the following bug is fixed:
+				 * 		https://bugs.eclipse.org/bugs/show_bug.cgi?id=99234
+				 */ 
+				if (line > -1 & line < document.getNumberOfLines()) {
+					try {
+						IRegion region = document.getLineInformation(line);
+						ITextSelection selection = new TextSelection(document, region.getOffset(), 0);
+						if (adapter instanceof IToggleBreakpointsTargetExtension) {
+							IToggleBreakpointsTargetExtension extension = (IToggleBreakpointsTargetExtension) adapter;
+							if (extension.canToggleBreakpoints(fPart, selection)) {
+								setEnabled(true);
+								return;
 							}
-							if (adapter.canToggleLineBreakpoints(fPart, selection) |
-								adapter.canToggleWatchpoints(fPart, selection) |
-								adapter.canToggleMethodBreakpoints(fPart, selection)) {
-									setEnabled(true);
-									return;
-							}
-						} catch (BadLocationException e) {
-							reportException(e);
 						}
+						if (adapter.canToggleLineBreakpoints(fPart, selection) |
+							adapter.canToggleWatchpoints(fPart, selection) |
+							adapter.canToggleMethodBreakpoints(fPart, selection)) {
+								setEnabled(true);
+								return;
+						}
+					} catch (BadLocationException e) {
+						reportException(e);
 					}
 				}
 			}