You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by el...@apache.org on 2015/07/28 06:59:08 UTC
svn commit: r1692993 - in
/directory/studio/trunk/plugins/ldifeditor/src/main/java/org/apache/directory/studio/ldifeditor/editor:
actions/AbstractOpenValueEditorAction.java
reconciler/LdifReconcilingStrategy.java
Author: elecharny
Date: Tue Jul 28 04:59:08 2015
New Revision: 1692993
URL: http://svn.apache.org/r1692993
Log:
o Removed useless checked on x != null when && ( x instanceof X )
o Same for check of x == null when && !( x instanceof X )
Modified:
directory/studio/trunk/plugins/ldifeditor/src/main/java/org/apache/directory/studio/ldifeditor/editor/actions/AbstractOpenValueEditorAction.java
directory/studio/trunk/plugins/ldifeditor/src/main/java/org/apache/directory/studio/ldifeditor/editor/reconciler/LdifReconcilingStrategy.java
Modified: directory/studio/trunk/plugins/ldifeditor/src/main/java/org/apache/directory/studio/ldifeditor/editor/actions/AbstractOpenValueEditorAction.java
URL: http://svn.apache.org/viewvc/directory/studio/trunk/plugins/ldifeditor/src/main/java/org/apache/directory/studio/ldifeditor/editor/actions/AbstractOpenValueEditorAction.java?rev=1692993&r1=1692992&r2=1692993&view=diff
==============================================================================
--- directory/studio/trunk/plugins/ldifeditor/src/main/java/org/apache/directory/studio/ldifeditor/editor/actions/AbstractOpenValueEditorAction.java (original)
+++ directory/studio/trunk/plugins/ldifeditor/src/main/java/org/apache/directory/studio/ldifeditor/editor/actions/AbstractOpenValueEditorAction.java Tue Jul 28 04:59:08 2015
@@ -87,11 +87,12 @@ public abstract class AbstractOpenValueE
cellEditor.activate();
Object newValue = cellEditor.getValue();
- if ( newValue != null && newValue instanceof String || newValue instanceof byte[] )
+ if ( ( newValue instanceof String ) || ( newValue instanceof byte[] ) )
{
IDocument document = editor.getDocumentProvider().getDocument( editor.getEditorInput() );
LdifValueLineBase newLine;
+
if ( line instanceof LdifControlLine )
{
LdifControlLine oldControlLine = ( LdifControlLine ) line;
Modified: directory/studio/trunk/plugins/ldifeditor/src/main/java/org/apache/directory/studio/ldifeditor/editor/reconciler/LdifReconcilingStrategy.java
URL: http://svn.apache.org/viewvc/directory/studio/trunk/plugins/ldifeditor/src/main/java/org/apache/directory/studio/ldifeditor/editor/reconciler/LdifReconcilingStrategy.java?rev=1692993&r1=1692992&r2=1692993&view=diff
==============================================================================
--- directory/studio/trunk/plugins/ldifeditor/src/main/java/org/apache/directory/studio/ldifeditor/editor/reconciler/LdifReconcilingStrategy.java (original)
+++ directory/studio/trunk/plugins/ldifeditor/src/main/java/org/apache/directory/studio/ldifeditor/editor/reconciler/LdifReconcilingStrategy.java Tue Jul 28 04:59:08 2015
@@ -109,7 +109,7 @@ public class LdifReconcilingStrategy imp
// notify outline
IContentOutlinePage outline = ( IContentOutlinePage ) editor.getAdapter( IContentOutlinePage.class );
- if ( outline != null && outline instanceof LdifOutlinePage )
+ if ( outline instanceof LdifOutlinePage )
{
( ( LdifOutlinePage ) outline ).refresh();
}