From bea720fa7081b69197ba48d6ee5c9d12e3b3dde0 Mon Sep 17 00:00:00 2001 From: David Inglis Date: Thu, 18 Mar 2004 15:10:24 +0000 Subject: [PATCH] fixed NPE on init --- .../refactoring/actions/RenameRefactoringAction.java | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/core/org.eclipse.cdt.ui/refactor/org/eclipse/cdt/internal/ui/refactoring/actions/RenameRefactoringAction.java b/core/org.eclipse.cdt.ui/refactor/org/eclipse/cdt/internal/ui/refactoring/actions/RenameRefactoringAction.java index a41fbcc52d9..432bb70a5af 100644 --- a/core/org.eclipse.cdt.ui/refactor/org/eclipse/cdt/internal/ui/refactoring/actions/RenameRefactoringAction.java +++ b/core/org.eclipse.cdt.ui/refactor/org/eclipse/cdt/internal/ui/refactoring/actions/RenameRefactoringAction.java @@ -63,11 +63,12 @@ public class RenameRefactoringAction extends SelectionDispatchAction { } if((element == null) || (element instanceof ITranslationUnit)){ enable = false; - } - ITextSelection textSelection= (ITextSelection)fEditor.getSelectionProvider().getSelection(); - if( (((CElement)element).getIdStartPos() != textSelection.getOffset()) - || (((CElement)element).getIdLength() != textSelection.getLength())) { - enable = false; + } else { + ITextSelection textSelection= (ITextSelection)fEditor.getSelectionProvider().getSelection(); + if( (((CElement)element).getIdStartPos() != textSelection.getOffset()) + || (((CElement)element).getIdLength() != textSelection.getLength())) { + enable = false; + } } setEnabled(enable); }