Author: adietish Date: 2011-06-01 16:06:28 -0400 (Wed, 01 Jun 2011) New Revision: 31746 Modified: trunk/as/plugins/org.jboss.ide.eclipse.as.core/jbosscore/org/jboss/ide/eclipse/as/core/server/internal/DeployableServerBehavior.java Log: [JBIDE-9054] changed the statements (no functional change!) so that the variable that is checked is always to the left -> far better readability Modified: trunk/as/plugins/org.jboss.ide.eclipse.as.core/jbosscore/org/jboss/ide/eclipse/as/core/server/internal/DeployableServerBehavior.java =================================================================== --- trunk/as/plugins/org.jboss.ide.eclipse.as.core/jbosscore/org/jboss/ide/eclipse/as/core/server/internal/DeployableServerBehavior.java 2011-06-01 19:52:23 UTC (rev 31745) +++ trunk/as/plugins/org.jboss.ide.eclipse.as.core/jbosscore/org/jboss/ide/eclipse/as/core/server/internal/DeployableServerBehavior.java 2011-06-01 20:06:28 UTC (rev 31746) @@ -102,13 +102,17 @@ } public int getPublishType(int kind, int deltaKind, int modulePublishState) { - if( ServerBehaviourDelegate.ADDED == deltaKind ) + if( deltaKind == ServerBehaviourDelegate.ADDED ) return IJBossServerPublisher.FULL_PUBLISH; - if (ServerBehaviourDelegate.REMOVED == deltaKind) { + else if (deltaKind == ServerBehaviourDelegate.REMOVED) { return IJBossServerPublisher.REMOVE_PUBLISH; - } else if (kind == IServer.PUBLISH_FULL || modulePublishState == IServer.PUBLISH_STATE_FULL || kind == IServer.PUBLISH_CLEAN ) { + } else if (kind == IServer.PUBLISH_FULL + || modulePublishState == IServer.PUBLISH_STATE_FULL + || kind == IServer.PUBLISH_CLEAN ) { return IJBossServerPublisher.FULL_PUBLISH; - } else if (kind == IServer.PUBLISH_INCREMENTAL || modulePublishState == IServer.PUBLISH_STATE_INCREMENTAL || kind == IServer.PUBLISH_AUTO) { + } else if (kind == IServer.PUBLISH_INCREMENTAL + || modulePublishState == IServer.PUBLISH_STATE_INCREMENTAL + || kind == IServer.PUBLISH_AUTO) { if( ServerBehaviourDelegate.CHANGED == deltaKind ) return IJBossServerPublisher.INCREMENTAL_PUBLISH; }