auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public void testComplexListArrayElementResetWithNull() { DocumentModel doc = session.createDocumentModel("/", "mydoc", "MyDocType"); doc.setPropertyValue("complexlist", <START> (Serializable) Collections.singletonList(Collections.singletonMap("array", new String[]{"foo"}))); <END> doc = session.createDocument(doc); session.save(); doc.setPropertyValue("complexlist", (Serializable) Collections.singletonList(Collections.singletonMap("array", null))); doc = session.saveDocument(doc); session.save(); @SuppressWarnings("unchecked") List<Map<String, Serializable>> list = (List<Map<String, Serializable>>) doc.getPropertyValue("complexlist"); assertEquals(1, list.size()); Map<String, Serializable> map = list.get(0); Object[] array = (Object[]) map.get("array"); assertNull(array); }</code><technical_language>I this a cleanup, longer. in future move Java 9's List.of anyway... in formatting string array needs spaces sides of braces, before</technical_language>

Target:
public void testComplexListArrayElementResetWithNull() { DocumentModel doc = session.createDocumentModel("/", "mydoc", "MyDocType"); doc.setPropertyValue("complexlist", (Serializable) Arrays.asList(Collections.singletonMap("array", new String[] { "foo" }))); doc = session.createDocument(doc); session.save(); doc.setPropertyValue("complexlist", (Serializable) Arrays.asList(Collections.singletonMap("array", null))); doc = session.saveDocument(doc); session.save(); @SuppressWarnings("unchecked") List<Map<String, Serializable>> list = (List<Map<String, Serializable>>) doc.getPropertyValue("complexlist"); assertEquals(1, list.size()); Map<String, Serializable> map = list.get(0); Object[] array = (Object[]) map.get("array"); assertNull(array); }
Source:
<code>private void initJobsService() { RxSession rxSession = new RxSessionImpl(session); jobsService = new JobsServiceImpl(); jobsService.setMetricsService(metricsService); jobsService.setSession(rxSession); <START> if (schedulerMode.equals("test")) { <END> scheduler = new TestScheduler(rxSession); } else { scheduler = new SchedulerImpl(rxSession); } jobsService.setScheduler(scheduler); jobsService.start(); }</code><technical_language>overlay this class in context of tests only? in main jaxrs code base</technical_language>

Target:
private void initJobsService() { RxSession rxSession = new RxSessionImpl(session); jobsService = new JobsServiceImpl(); jobsService.setMetricsService(metricsService); jobsService.setSession(rxSession); scheduler = new JobSchedulerFactory().getJobScheduler(rxSession); jobsService.setScheduler(scheduler); jobsService.start(); }
Source:
<code>public boolean isValid(VdsNetworkInterface iface, ConstraintValidatorContext context) { NetworkBootProtocol bootProtocol = iface.getBootProtocol(); String address = iface.getAddress(); if (bootProtocol != null && bootProtocol == STATIC_IP) { if (isNullOrEmpty(address)) { context.disableDefaultConstraintViolation(); context.buildConstraintViolationWithTemplate("NETWORK_ADDR_MANDATORY_IN_STATIC_IP") .addNode("address").addConstraintViolation(); return false; } } if (!isEmpty(iface.getBondName()) && !validateSlave(iface)) { context.disableDefaultConstraintViolation(); context.buildConstraintViolationWithTemplate("SLAVE_INTERFACE_IS_MISCONFIGURED").addConstraintViolation(); return false; } <START> if (validateLabel(iface)) { <END> context.disableDefaultConstraintViolation(); context.buildConstraintViolationWithTemplate("IMPROPER_INTERFACE_IS_LABELED").addConstraintViolation(); } return true; }</code><technical_language>meant negate condition</technical_language>

Target:
public boolean isValid(VdsNetworkInterface iface, ConstraintValidatorContext context) { NetworkBootProtocol bootProtocol = iface.getBootProtocol(); String address = iface.getAddress(); if (bootProtocol != null && bootProtocol == STATIC_IP) { if (isNullOrEmpty(address)) { context.disableDefaultConstraintViolation(); context.buildConstraintViolationWithTemplate("NETWORK_ADDR_MANDATORY_IN_STATIC_IP") .addNode("address").addConstraintViolation(); return false; } } if (!isEmpty(iface.getBondName()) && !validateSlave(iface)) { context.disableDefaultConstraintViolation(); context.buildConstraintViolationWithTemplate("SLAVE_INTERFACE_IS_MISCONFIGURED").addConstraintViolation(); return false; } if (!validateLabel(iface)) { context.disableDefaultConstraintViolation(); context.buildConstraintViolationWithTemplate("IMPROPER_INTERFACE_IS_LABELED").addConstraintViolation(); } return true; }
Source:
<code>static void queryWithQueryOptions(DatabaseClient dbClient) { try (ResultSet resultSet = dbClient .singleUse() .executeQuery( Statement .newBuilder("SELECT SingerId, AlbumId, AlbumTitle FROM Albums") <START> .withQueryOptions(QueryOptions.newBuilder().setOptimizerVersion("latest").build()) <END> .build())) { while (resultSet.next()) { System.out.printf( "%d %d %s\n", resultSet.getLong(0), resultSet.getLong(1), resultSet.getString(2)); } } }</code><technical_language>here, I useful show set version other default. suggestion .withQueryOptions(QueryOptions.newBuilder().setOptimizerVersion("1").build())</technical_language>

Target:
static void queryWithQueryOptions(DatabaseClient dbClient) { try (ResultSet resultSet = dbClient .singleUse() .executeQuery( Statement .newBuilder("SELECT SingerId, AlbumId, AlbumTitle FROM Albums") .withQueryOptions(QueryOptions.newBuilder().setOptimizerVersion("1").build()) .build())) { while (resultSet.next()) { System.out.printf( "%d %d %s\n", resultSet.getLong(0), resultSet.getLong(1), resultSet.getString(2)); } } }
Source:
<code>private boolean isLegalParameterName(final String parameterName) { <START> return !(parameterName.equals(NAME) || parameterName.equals(OP) || parameterName.equals(OP_ADDR) || parameterName.equals(OPERATION_HEADERS)); <END> }</code><technical_language>@lvydra Please remove 'parameterName.equals(NAME) 'from this. apologies; I listed 'name' invalid key in JIRA description. corrected JIRA description</technical_language>

Target:
private boolean isLegalParameterName(final String parameterName) { return !(parameterName.equals(OP) || parameterName.equals(OP_ADDR) || parameterName.equals(OPERATION_HEADERS)); }
Source:
<code>public void recoverWorkflowInstancesFromDeadNodes() { List<InstanceInfo> instances = jdbc.query( "select id, state from nflow_workflow where executor_id in (select id from nflow_executor where " + executorGroupCondition + " and id <> " + getExecutorId() + " and expires < current_timestamp)", new RowMapper<InstanceInfo>() { @Override public InstanceInfo mapRow(ResultSet rs, int rowNum) throws SQLException { InstanceInfo instance = new InstanceInfo(); instance.id = rs.getInt("id"); instance.state = rs.getString("state"); return instance; } }); for (InstanceInfo instance : instances) { <START> workflowInstanceDao.recoverWorkflowInstance(instance); <END> } }</code><technical_language>create action in ExexcutorDao? pass (long instanceId, WorkflowInstanceAction action) arguments instanceDao. instanceDao InstanceInfo object all</technical_language>

Target:
public void recoverWorkflowInstancesFromDeadNodes() { List<InstanceInfo> instances = jdbc.query( "select id, state from nflow_workflow where executor_id in (select id from nflow_executor where " + executorGroupCondition + " and id <> " + getExecutorId() + " and expires < current_timestamp)", new RowMapper<InstanceInfo>() { @Override public InstanceInfo mapRow(ResultSet rs, int rowNum) throws SQLException { InstanceInfo instance = new InstanceInfo(); instance.id = rs.getInt("id"); instance.state = rs.getString("state"); return instance; } }); for (InstanceInfo instance : instances) { WorkflowInstanceAction action = new WorkflowInstanceAction.Builder().setExecutionStart(now()).setExecutionEnd(now()) .setType(recovery).setState(instance.state).setStateText("Recovered").setWorkflowInstanceId(instance.id).build(); workflowInstanceDao.recoverWorkflowInstance(instance.id, action); } }
Source:
<code>public void test() throws InterruptedException { connect(); click(jq("@button").eq(0)); waitResponse(); <START> assertEquals(jq(".z-button").outerHeight(), jq(".z-groupbox-content").height()); <END> }</code><technical_language>jq(".z-button:eq(0)") jq(".z-groupbox-content:eq(0)")</technical_language>

Target:
public void test() throws InterruptedException { connect(); JQuery btn = jq("@button").eq(0); click(btn); waitResponse(); assertEquals(btn.outerHeight(), jq(".z-groupbox-content").eq(0).height()); }
Source:
<code>public void run(IAction action) { <START> boolean performAction = MessageDialog.openConfirm(getShell(), "Discard Local Changes", "This will discard all local changes for the selected resources. Are you sure you want to do this ?"); <END> if (performAction){ performDiscardChanges(); } }</code><technical_language>messages need internationalised well</technical_language>

Target:
public void run(IAction action) { boolean performAction = MessageDialog.openConfirm(getShell(), UIText.DiscardChangesAction_confirmActionTitle, UIText.DiscardChangesAction_confirmActionMessage); if (performAction){ performDiscardChanges(); } }
Source:
<code>public final String generateUnusedStyle(final String commandId) { final Object[] existingImages = (Object[]) imagesById.get(commandId); if (existingImages == null) { return null; } final Set existingStyles = new HashSet(3); for (final Object styledImages : existingImages) { if (styledImages instanceof ImageDescriptor) { existingStyles.add(null); } else if (styledImages instanceof Map) { final Map styleMap = (Map) styledImages; existingStyles.addAll(styleMap.keySet()); } } if (!existingStyles.contains(null)) { return null; } StringBuilder generatedStyle = new StringBuilder("AUTOGEN:::"); int index = 0; while (existingStyles.contains(generatedStyle)) { <START> generatedStyle.append((index++ % 10)); <END> } return generatedStyle.toString(); }</code><technical_language>some parenthesis? Also, extracting index++ in a separate line</technical_language>

Target:
public final String generateUnusedStyle(final String commandId) { final Object[] existingImages = (Object[]) imagesById.get(commandId); if (existingImages == null) { return null; } final Set existingStyles = new HashSet(3); for (final Object styledImages : existingImages) { if (styledImages instanceof ImageDescriptor) { existingStyles.add(null); } else if (styledImages instanceof Map) { final Map styleMap = (Map) styledImages; existingStyles.addAll(styleMap.keySet()); } } if (!existingStyles.contains(null)) { return null; } StringBuilder generatedStyle = new StringBuilder("AUTOGEN:::"); int index = 0; while (existingStyles.contains(generatedStyle)) { generatedStyle.append(index % 10); index++; } return generatedStyle.toString(); }
Source:
<code>public List<IEventDeclaration> getEventDeclarations() { if( fEventUnsetId != null){ return ImmutableList.of(fEventUnsetId); } return <START> fEvents; <END> }</code><technical_language>wrap this immutable</technical_language>

Target:
public List<IEventDeclaration> getEventDeclarations() { return ImmutableList.copyOf(fEvents); }
Source:
<code>public boolean isReadOnly(ELContext elContext, @Nullable Object base, Object property) throws ELException { if (base == null) { String beanName = property.toString(); WebApplicationContext wac = getWebApplicationContext(elContext); <START> return wac.containsBean(beanName); <END> } return false; }</code><technical_language>proposed changed is technically correct, please revert this for sake of readability</technical_language>

Target:
public boolean isReadOnly(ELContext elContext, @Nullable Object base, Object property) throws ELException { if (base == null) { String beanName = property.toString(); WebApplicationContext wac = getWebApplicationContext(elContext); if (wac.containsBean(beanName)) { return true; } } return false; }
Source:
<code>private static void ttyCheck() { final Console console = System.console(); if(console != null) { System.out.println( <START> "WARNING: Are you running agent.jar from an interactive console?\n" + <END> "If so, you are probably using it incorrectly.\n" + "See <LINK_0>"); } }</code><technical_language>avoid ".jar" here, Remotig lib another name && packaged WAR/*.jnlp/whatever</technical_language>

Target:
private static void ttyCheck() { final Console console = System.console(); if(console != null) { System.out.println( "WARNING: Are you running agent from an interactive console?\n" + "If so, you are probably using it incorrectly.\n" + "See <LINK_1>"); } }
Source:
<code>public void testProperties() { assertTrue(module instanceof TestStateSystemModule); TestStateSystemModule mod = (TestStateSystemModule) module; Map<String, String> properties = mod.getProperties(); <START> assertEquals(mod.getBackendName() ,properties.get(Messages.TmfStateSystemAnalysisModule_PropertiesBackend)); <END> }</code><technical_language>fix spacing</technical_language>

Target:
public void testProperties() { assertTrue(module instanceof TestStateSystemModule); TestStateSystemModule mod = (TestStateSystemModule) module; Map<String, String> properties = mod.getProperties(); assertEquals(mod.getBackendName(), properties.get(Messages.TmfStateSystemAnalysisModule_PropertiesBackend)); assertEquals(mod.getId(), properties.get(org.eclipse.tracecompass.tmf.core.analysis.Messages.TmfAbstractAnalysisModule_LabelId)); }
Source:
<code>public void testParsePartitionsFromFiles_wrong_type() { String root = "hdfs://host:123/partition_test/web_sales5/"; String[] spaths = { root + "c=3.14/ws_sold_date_sk=__HIVE_DEFAULT_PARTITION__/part-00780.c002.snappy.parquet", root + "c=3.14/ws_sold_date_sk=HELLO/part-00780.c002.snappy.parquet" }; Path basePath = new Path( "hdfs://host:123/partition_test/web_sales5/" ); HashSet<Path> basePaths = new HashSet<>(); basePaths.add(basePath); List<Path> files = Arrays.stream(spaths).map(Path::new).collect(toList()); StructType s = new StructType(); s = s.add("ws_sold_date_sk", DataTypes.DoubleType); try { com.splicemachine.spark.splicemachine.PartitionSpec ps = SparkExternalTableUtil.parsePartitionsFromFiles( files, true, basePaths, s,null); Assert.fail("no exception"); } catch(Exception e) { <START> <END> } }</code><technical_language>nice check for a specific Exception / Error code</technical_language>

Target:
public void testParsePartitionsFromFiles_wrong_type() { String root = "hdfs://host:123/partition_test/web_sales5/"; String[] spaths = { root + "c=3.14/ws_sold_date_sk=__HIVE_DEFAULT_PARTITION__/part-00780.c002.snappy.parquet", root + "c=3.14/ws_sold_date_sk=HELLO/part-00780.c002.snappy.parquet" }; Path basePath = new Path( "hdfs://host:123/partition_test/web_sales5/" ); HashSet<Path> basePaths = new HashSet<>(); basePaths.add(basePath); List<Path> files = Arrays.stream(spaths).map(Path::new).collect(toList()); StructType s = new StructType(); s = s.add("ws_sold_date_sk", DataTypes.DoubleType); try { SparkExternalTableUtil.parsePartitionsFromFiles(files, true, basePaths, s,null); Assert.fail("no exception"); } catch( Exception e) { Assert.assertEquals(e.toString(), "java.lang.IllegalArgumentException: Column ws_sold_date_sk is double, can't parse HELLO"); } }
Source:
<code>public void should_ReturnNull_When_AnnotatedId() throws NoSuchMethodException { String error = explicitNullableTypeChecker .checkValueForAnnotatedElement(null, getClass().getMethod("methodWithIdAnnotation")); Assert.assertNull("Method with @Id annotation should allow null value", error); error = explicitNullableTypeChecker.checkValueForAnnotatedElement( 1l, getClass().getMethod("methodWithIdAnnotation")); <START> Assert.assertNull("Method with @Id annotation should allow null value", <END> error); }</code><technical_language>This assert is related a non-null 1l above, message incorrect me. suggestion Assert.assertNull("Method @Id annotation allow non-null value",</technical_language>

Target:
public void should_ReturnNull_When_AnnotatedId() throws NoSuchMethodException { String error = explicitNullableTypeChecker .checkValueForAnnotatedElement(null, getClass().getMethod("methodWithIdAnnotation")); Assert.assertNull("Method with @Id annotation should allow null value", error); error = explicitNullableTypeChecker.checkValueForAnnotatedElement( 1l, getClass().getMethod("methodWithIdAnnotation")); Assert.assertNull("Method with @Id annotation should allow non-null value", error); }
Source:
<code>public void initiate(ChannelHandlerContext ctx) throws Exception { loggerNet.debug("RLPX protocol activated"); nodeId = myKey.getNodeId(); <START> handshake = new EncryptionHandshake(ECKey.fromNodeId(nodeId).getPubKeyPoint()); <END> Object msg; if (config.eip8()) { AuthInitiateMessageV4 initiateMessage = handshake.createAuthInitiateV4(myKey); initiatePacket = handshake.encryptAuthInitiateV4(initiateMessage); msg = initiateMessage; } else { AuthInitiateMessage initiateMessage = handshake.createAuthInitiate(null, myKey); initiatePacket = handshake.encryptAuthMessage(initiateMessage); msg = initiateMessage; } final ByteBuf byteBufMsg = ctx.alloc().buffer(initiatePacket.length); byteBufMsg.writeBytes(initiatePacket); ctx.writeAndFlush(byteBufMsg).sync(); channel.getNodeStatistics().rlpxAuthMessagesSent.add(); if (loggerNet.isDebugEnabled()) loggerNet.debug("To: \t{} \tSend: \t{}", ctx.channel().remoteAddress(), msg); }</code><technical_language>remoteId</technical_language>

Target:
public void initiate(ChannelHandlerContext ctx) throws Exception { loggerNet.debug("RLPX protocol activated"); nodeId = myKey.getNodeId(); handshake = new EncryptionHandshake(ECKey.fromNodeId(this.remoteId).getPubKeyPoint()); Object msg; if (config.eip8()) { AuthInitiateMessageV4 initiateMessage = handshake.createAuthInitiateV4(myKey); initiatePacket = handshake.encryptAuthInitiateV4(initiateMessage); msg = initiateMessage; } else { AuthInitiateMessage initiateMessage = handshake.createAuthInitiate(null, myKey); initiatePacket = handshake.encryptAuthMessage(initiateMessage); msg = initiateMessage; } final ByteBuf byteBufMsg = ctx.alloc().buffer(initiatePacket.length); byteBufMsg.writeBytes(initiatePacket); ctx.writeAndFlush(byteBufMsg).sync(); channel.getNodeStatistics().rlpxAuthMessagesSent.add(); if (loggerNet.isDebugEnabled()) loggerNet.debug("To: \t{} \tSend: \t{}", ctx.channel().remoteAddress(), msg); }
Source:
<code>private boolean isSupportedType(String type) { boolean supported = false; switch (type) { case "ear": case "war": case "rar": case "eba": case "esa": case "liberty-assembly": supported = true; break; case "jar": <START> if (installThinProject) { <END> supported = true; } break; default: break; } return supported; }</code><technical_language>For cases 'thin' is supported, I if this method catch that? for example, case "ear": check for if (installThinProject) break setting supported = true;</technical_language>

Target:
private boolean isSupportedType(String type) { boolean supported = false; switch (type) { case "ear": case "rar": case "eba": case "esa": case "liberty-assembly": if(installThinProject) { log.warn("thin-project cannot be configured for installAppPackages for the " + type + " packaging type"); } else { supported = true; } break; case "war": supported = true; break; case "jar": if (installThinProject) { supported = true; } break; default: break; } return supported; }
Source:
<code>public void onActivityResult(int requestCode, int resultCode, final Intent data) { super.onActivityResult(requestCode, resultCode, data); if (requestCode == ACTIVITY_REQUEST_DESCRIPTION_EDIT && resultCode == RESULT_OK) { FeedbackUtil.showMessage(this, data.getSerializableExtra(INTENT_EXTRA_ACTION) == ADD_CAPTION ? getString(R.string.description_edit_success_saved_image_caption_snackbar) : getString(R.string.description_edit_success_saved_image_caption_in_lang_snackbar, app.language().getAppLanguageLocalizedName(targetLanguageCode))); layOutGalleryDescription(); setResult(ACTIVITY_RESULT_IMAGE_CAPTION_ADDED); } if (requestCode == ACTIVITY_REQUEST_ADD_IMAGE_TAGS && resultCode == RESULT_OK) { FeedbackUtil.showMessage(this, getString(R.string.description_edit_success_saved_image_tags_snackbar)); layOutGalleryDescription(); <START> setResult(ACTIVITY_RESULT_IMAGE_CAPTION_ADDED); <END> } }</code><technical_language>ACTIVITY_RESULT_IMAGE_TAG_ADDED? (or removed?)</technical_language>

Target:
public void onActivityResult(int requestCode, int resultCode, final Intent data) { super.onActivityResult(requestCode, resultCode, data); if ((requestCode == ACTIVITY_REQUEST_DESCRIPTION_EDIT || requestCode == ACTIVITY_REQUEST_ADD_IMAGE_TAGS) && resultCode == RESULT_OK) { ABTestSuggestedEditsSnackbarFunnel abTestFunnel = new ABTestSuggestedEditsSnackbarFunnel(); DescriptionEditActivity.Action action = (data != null && data.hasExtra(INTENT_EXTRA_ACTION)) ? (DescriptionEditActivity.Action) data.getSerializableExtra(INTENT_EXTRA_ACTION) : (requestCode == ACTIVITY_REQUEST_ADD_IMAGE_TAGS) ? ADD_IMAGE_TAGS : null; Snackbar snackbar = FeedbackUtil.makeSnackbar(this, action == ADD_CAPTION ? getString(R.string.description_edit_success_saved_image_caption_snackbar) : (action == TRANSLATE_CAPTION) ? getString(R.string.description_edit_success_saved_image_caption_in_lang_snackbar, app.language().getAppLanguageLocalizedName(targetLanguageCode)) : getString(R.string.description_edit_success_saved_image_tags_snackbar), FeedbackUtil.LENGTH_DEFAULT); if (abTestFunnel.shouldSeeSnackbarAction() && action != null) { snackbar.setAction(R.string.suggested_edits_tasks_onboarding_get_started, view -> startActivity(SuggestionsActivity.newIntent(this, action))); } snackbar.show(); abTestFunnel.logSnackbarShown(); layOutGalleryDescription(); setResult((requestCode == ACTIVITY_REQUEST_DESCRIPTION_EDIT) ? ACTIVITY_RESULT_IMAGE_CAPTION_ADDED : ACTIVITY_REQUEST_ADD_IMAGE_TAGS); } }
Source:
<code>public void redirectHomePage() { <START> Helpers.running(Helpers.fakeApplication(), () -> { <END> Result result = Helpers.route(application, controllers.routes.HomeController.index()); assertThat(result.status(), equalTo(Helpers.SEE_OTHER)); assertThat(result.redirectLocation(), equalTo("/computers")); }); }</code><technical_language>need application built setup method this fakeApplication here. In other words, Helpers.running removed test "plain"</technical_language>

Target:
public void redirectHomePage() { Result result = Helpers.route(application, controllers.routes.HomeController.index()); assertThat(result.status(), equalTo(Helpers.SEE_OTHER)); assertThat(result.redirectLocation().get(), equalTo("/computers")); }
Source:
<code>public boolean equals(Object obj) { if(!(obj instanceof GlusterVolumeProfileInfo)) { return false; } GlusterVolumeProfileInfo profileInfo = (GlusterVolumeProfileInfo) obj; <START> if(!ObjectUtils.objectsEqual(getVolumeId(), profileInfo.getVolumeId())){ <END> return false; } return true; }</code><technical_language>need check lists equal well</technical_language>

Target:
public boolean equals(Object obj) { if(!(obj instanceof GlusterVolumeProfileInfo)) { return false; } GlusterVolumeProfileInfo profileInfo = (GlusterVolumeProfileInfo) obj; if(!ObjectUtils.objectsEqual(getVolumeId(), profileInfo.getVolumeId())){ return false; } if(!ObjectUtils.objectsEqual(getBrickProfileDetails(), profileInfo.getBrickProfileDetails())) { return false; } if(!ObjectUtils.objectsEqual(getNfsProfileDetails(), profileInfo.getNfsProfileDetails())) { return false; } return true; }
Source:
<code>protected void modelIncomplete(boolean isGlobalRequest) { if (isGlobalRequest) { Object input = getInput(); <START> if (input != null && input instanceof TmfStatisticsTreeNode) { <END> TmfStatisticsTreeRootFactory.removeStatTreeRoot(getTreeID()); } resetUpdateSynchronization(); sendPendingUpdate(); } else { resetTimeRangeValue(); } refresh(); waitCursor(false); }</code><technical_language>check is redundant, "instanceof" checks for null also</technical_language>

Target:
protected void modelIncomplete(boolean isGlobalRequest) { if (isGlobalRequest) { Object input = getInput(); if (input instanceof TmfStatisticsTreeNode) { TmfStatisticsTreeRootFactory.removeStatTreeRoot(getTreeID()); } resetUpdateSynchronization(); sendPendingUpdate(); } else { resetTimeRangeValue(); } refresh(); waitCursor(false); }
Source:
<code>protected boolean acquireLockInternal() { if (context.getParentLock() != null) { final EngineLock requiredLock = buildLock(); <START> return context.getParentLock().covers(requiredLock, getLockProperties().getScope()); <END> } if (context.getLock() == null) { EngineLock lock = buildLock(); if (lock != null) { Pair<Boolean, Set<String>> lockAcquireResult = getLockManager().acquireLock(lock); if (lockAcquireResult.getFirst()) { log.info("Lock Acquired to object '{}'", lock); context.withLock(lock); } else { log.info("Failed to Acquire Lock to object '{}'", lock); getReturnValue().getValidationMessages() .addAll(extractVariableDeclarations(lockAcquireResult.getSecond())); return false; } } } return true; }</code><technical_language>reduced context.getParentLock().covers(buildLock(), getLockProperties().getScope());</technical_language>

Target:
protected boolean acquireLockInternal() { if (context.getParentLock() != null) { return context.getParentLock().covers(buildLock(), getLockProperties().getScope()); } if (context.getLock() == null) { EngineLock lock = buildLock(); if (lock != null) { Pair<Boolean, Set<String>> lockAcquireResult = getLockManager().acquireLock(lock); if (lockAcquireResult.getFirst()) { log.info("Lock Acquired to object '{}'", lock); context.withLock(lock); } else { log.info("Failed to Acquire Lock to object '{}'", lock); getReturnValue().getValidationMessages() .addAll(extractVariableDeclarations(lockAcquireResult.getSecond())); return false; } } } return true; }
Source:
<code>public void login(String username, String password) { <START> usernameField.sendKeys(username); <END> passwordField.sendKeys(password); signInButton.click(); }</code><technical_language>wait for elements interaction</technical_language>

Target:
public void login(String username, String password) { wait.forElementVisible(usernameField); wait.forElementVisible(passwordField); wait.forElementClickable(signInButton); usernameField.sendKeys(username); passwordField.sendKeys(password); signInButton.click(); }
Source:
<code>private void handleSaveFailure (Throwable e) { Log.e(App.LOG_TAG, "Could not save tree", e); safeDismiss(saveDialog); <START> Toast.makeText(App.getAppInstance(), "Could not save tree", Toast.LENGTH_SHORT).show(); <END> }</code><technical_language>want put this text in a strings resource</technical_language>

Target:
private void handleSaveFailure (Throwable e) { String msg = getString(R.string.save_tree_failure); Log.e(App.LOG_TAG, msg, e); safeDismiss(saveDialog); Toast.makeText(App.getAppInstance(), msg, Toast.LENGTH_SHORT).show(); }
Source:
<code>protected void init() { if (!initialized) { Profiler profiler = Profiler.create(LOG).startInfo("JavaClasspath initialization"); initialized = true; binaries = getFilesFromProperty(JavaClasspathProperties.SONAR_JAVA_BINARIES); List<File> libraries = getFilesFromProperty(JavaClasspathProperties.SONAR_JAVA_LIBRARIES); if (binaries.isEmpty() && libraries.isEmpty() && useDeprecatedProperties()) { <START> throw new AnalysisException("sonar.binaries and sonar.libraries are deprecated since version 2.5 of sonar-java-plugin, please use sonar.java.binaries and sonar.java.libraries instead"); <END> } elements = Lists.newArrayList(binaries); if(libraries.isEmpty()) { LOG.warn("Bytecode of dependencies was not provided for analysis of source files, " + "you might end up with less precise results. Bytecode can be provided using sonar.java.libraries property"); } elements.addAll(libraries); profiler.stopInfo(); } }</code><technical_language>I change message to: sonar.binaries sonar.libraries supported version 4.0 of sonar-java-plugin. Please sonar.java.binaries sonar.java.libraries instead</technical_language>

Target:
protected void init() { if (!initialized) { Profiler profiler = Profiler.create(LOG).startInfo("JavaClasspath initialization"); initialized = true; binaries = getFilesFromProperty(JavaClasspathProperties.SONAR_JAVA_BINARIES); List<File> libraries = getFilesFromProperty(JavaClasspathProperties.SONAR_JAVA_LIBRARIES); if (binaries.isEmpty() && libraries.isEmpty() && useDeprecatedProperties()) { throw new AnalysisException( "sonar.binaries and sonar.libraries are not supported since version 4.0 of sonar-java-plugin, please use sonar.java.binaries and sonar.java.libraries instead"); } elements = Lists.newArrayList(binaries); if(libraries.isEmpty()) { LOG.warn("Bytecode of dependencies was not provided for analysis of source files, " + "you might end up with less precise results. Bytecode can be provided using sonar.java.libraries property"); } elements.addAll(libraries); profiler.stopInfo(); } }
Source:
<code>private void setOptionalDependencies(TargetPlatformConfiguration result, Xpp3Dom resolverDom) { String value = getStringValue(resolverDom.getChild("optionalDependencies")); <START> if (null == value) { <END> return; } else switch (value) { case OPTIONAL_RESOLUTION_REQUIRE: result.setOptionalResolutionAction(OptionalResolutionAction.REQUIRE); break; case OPTIONAL_RESOLUTION_IGNORE: result.setOptionalResolutionAction(OptionalResolutionAction.IGNORE); break; default: throw new BuildFailureException("Illegal value of <optionalDependencies> dependency resolution parameter: " + value); } }</code><technical_language>in other class, invert condition for readability</technical_language>

Target:
private void setOptionalDependencies(TargetPlatformConfiguration result, Xpp3Dom resolverDom) { String value = getStringValue(resolverDom.getChild("optionalDependencies")); if (value == null) { return; } else switch (value) { case OPTIONAL_RESOLUTION_REQUIRE: result.setOptionalResolutionAction(OptionalResolutionAction.REQUIRE); break; case OPTIONAL_RESOLUTION_IGNORE: result.setOptionalResolutionAction(OptionalResolutionAction.IGNORE); break; default: throw new BuildFailureException("Illegal value of <optionalDependencies> dependency resolution parameter: " + value); } }
Source:
<code>private void updateDiskImageDynamic() { VDSReturnValue ret = runVdsCommand( VDSCommandType.GetImageInfo, <START> new GetImageInfoVDSCommandParameters(getDestinationDiskImage().getStoragePoolId().getValue(), <END> getDestinationDiskImage().getStorageIds().get(0), getDestinationDiskImage().getId(), getDestinationDiskImage().getImageId())); DiskImage imageFromIRS = (DiskImage) ret.getReturnValue(); DiskImageDynamic diskImageDynamic = getDiskImageDynamicDAO().get(getDestinationDiskImage().getImageId()); if (imageFromIRS != null && diskImageDynamic != null) { diskImageDynamic.setactual_size(imageFromIRS.getActualSizeFromDiskImage()); getDiskImageDynamicDAO().update(diskImageDynamic); } }</code><technical_language>happened if fail here? want end command? set actual size if get null VDSM get exception well</technical_language>

Target:
private void updateDiskImageDynamic() { try { VDSReturnValue ret = runVdsCommand( VDSCommandType.GetImageInfo, new GetImageInfoVDSCommandParameters(getDestinationDiskImage().getStoragePoolId().getValue(), getDestinationDiskImage().getStorageIds().get(0), getDestinationDiskImage().getId(), getDestinationDiskImage().getImageId())); DiskImage imageFromIRS = (DiskImage) ret.getReturnValue(); DiskImageDynamic diskImageDynamic = getDiskImageDynamicDAO().get(getDestinationDiskImage().getImageId()); if (imageFromIRS != null && diskImageDynamic != null) { diskImageDynamic.setactual_size(imageFromIRS.getActualSizeFromDiskImage()); getDiskImageDynamicDAO().update(diskImageDynamic); } else { throw new VdcBLLException(VdcBllErrors.ENGINE, "Can't find Snapshot's DiskImage by ID"); } } catch (VdcBLLException e) { log.errorFormat("GetImageInfo on disk {0} did not complete successfully. error {1}", getDestinationDiskImage().getImageId(), e.getMessage()); } }
Source:
<code>public V putIfAbsent(final K key, final V value) throws CacheWriterException { statusTransitioner.checkAvailable(); checkNonNull(key, value); final AtomicBoolean installed = new AtomicBoolean(false); final Function<K, V> mappingFunction = memoize(new Function<K, V>() { @Override public V apply(final K k) { if (cacheWriter != null) { try { cacheWriter.write(k, null, value); } catch (Exception e) { throw newCacheWriterException(e); } } installed.set(true); return value; } }); try { ValueHolder<V> inCache = store.computeIfAbsent(key, mappingFunction); if (installed.get()) { return null; } else { return inCache.value(); } } catch (CacheAccessException e) { V v = null; try { v = mappingFunction.apply(key); if (cacheWriter != null) { store.remove(key); } else { final Store.ValueHolder<V> holder = store.get(key); if (holder == null) { resilienceStrategy.possiblyInconsistent(key, e); } else { return holder.value(); } } } catch (CacheAccessException e1) { resilienceStrategy.possiblyInconsistent(key, e, e1); } <START> return v; <END> } }</code><technical_language>I admit confused in catch block. I change change return of method, I need replace return v; returned installed.get() ? null : v; preserve behaviour</technical_language>

Target:
public V putIfAbsent(final K key, final V value) throws CacheWriterException { statusTransitioner.checkAvailable(); checkNonNull(key, value); final AtomicBoolean installed = new AtomicBoolean(false); final Function<K, V> mappingFunction = memoize(new Function<K, V>() { @Override public V apply(final K k) { if (cacheWriter != null) { try { cacheWriter.write(k, null, value); } catch (Exception e) { throw newCacheWriterException(e); } } installed.set(true); return value; } }); try { ValueHolder<V> inCache = store.computeIfAbsent(key, mappingFunction); if (installed.get()) { return null; } else { return inCache.value(); } } catch (CacheAccessException e) { V computedValue = null; try { computedValue = mappingFunction.apply(key); if (cacheWriter != null) { store.remove(key); } else { final Store.ValueHolder<V> holder = store.get(key); if (holder == null) { resilienceStrategy.possiblyInconsistent(key, e); } else { return holder.value(); } } } catch (CacheAccessException e1) { resilienceStrategy.possiblyInconsistent(key, e, e1); } return installed.get() ? null : computedValue; } }
Source:
<code>public byte[] getAPIData() { ByteArrayOutputStream data = new ByteArrayOutputStream(); try { <START> data.write(sourceAddress64.getValue()); <END> data.write(sourceAddress16.getValue()); data.write(receiveOptions); if (receivedData != null) data.write(receivedData); } catch (IOException e) { e.printStackTrace(); } return data.toByteArray(); }</code><technical_language>Check if addresses null</technical_language>

Target:
public byte[] getAPIData() { ByteArrayOutputStream data = new ByteArrayOutputStream(); try { data.write(sourceAddress64.getValue()); data.write(sourceAddress16.getValue()); data.write(receiveOptions); if (receivedData != null) data.write(receivedData); } catch (IOException e) { } return data.toByteArray(); }
Source:
<code>IgnoreCondition create() { checkConditionType(); try { return createCondition(); <START> } catch (RuntimeException re) { <END> throw re; } catch (Exception e) { throw new RuntimeException(e); } }</code><technical_language>consider catch(Exception e) { throw Throwables.propagate(e) }</technical_language>

Target:
IgnoreCondition create() { checkConditionType(); try { return createCondition(); } catch (RuntimeException re) { throw Throwables.propagate(re); } catch (Exception e) { throw new RuntimeException(e); } }
Source:
<code>public void finalize(Item item) { try (DataDomainManager manager = new DataDomainManager()) { DataDomain domain = manager.getDomainForItem(item); <START> ItemManager items = new ItemManager(); <END> SortedSet<String> domainValues = domain.getDomainValues(); if(item.getItemType() == Item.ItemType.aggregation){ domainValues.clear(); domainValues = DataDomainUtility.getOnlyVisibleDomainValues(item, manager, items); } manager.close(); items.close(); Integer minimum = Integer.parseInt(domainValues.first()); Integer maximum = Integer.parseInt(domainValues.last()); AttributeRange range = new AttributeRange(minimum, maximum); RainbowColorMap colorMap = new RainbowColorMap(range); String[] tmpColors = new String[domainValues.size()]; float[] tmpThresholds = new float[domainValues.size()]; List<Map<String, Object>> tmpBins = new ArrayList<>(); int i = 0; for (String year : domainValues) { Integer intYear = Integer.parseInt(year); Color color = colorMap.valueToColor(intYear); String hex = ColorUtility.toHexLowercase(color); tmpColors[i] = hex; tmpThresholds[i] = (float) intYear; Map<String, Object> binMap = new HashMap<>(); binMap.put("years", intYear); binMap.put("color", hex); tmpBins.add(binMap); i++; } wrapped = new SLDConfig(jspPath, units, style, strokeWidth, SLDGenerator.STROKE_OPACITY_DEFAULT, attrs, tmpThresholds, tmpColors, tmpBins, LegendType.CONTINUOUS); } }</code><technical_language>put this in try-with-resources. need close() statements if do (they handled implicit finally)</technical_language>

Target:
public void finalize(Item item) { try (DataDomainManager manager = new DataDomainManager(); ItemManager items = new ItemManager()) { DataDomain domain = manager.getDomainForItem(item); SortedSet<String> domainValues = domain.getDomainValues(); if(item.getItemType() == Item.ItemType.aggregation){ domainValues.clear(); domainValues = DataDomainUtility.getOnlyVisibleDomainValues(item, manager, items); } Integer minimum = Integer.parseInt(domainValues.first()); Integer maximum = Integer.parseInt(domainValues.last()); AttributeRange range = new AttributeRange(minimum, maximum); RainbowColorMap colorMap = new RainbowColorMap(range); String[] tmpColors = new String[domainValues.size()]; float[] tmpThresholds = new float[domainValues.size()]; List<Map<String, Object>> tmpBins = new ArrayList<>(); int i = 0; for (String year : domainValues) { Integer intYear = Integer.parseInt(year); Color color = colorMap.valueToColor(intYear); String hex = ColorUtility.toHexLowercase(color); tmpColors[i] = hex; tmpThresholds[i] = (float) intYear; Map<String, Object> binMap = new HashMap<>(); binMap.put("years", intYear); binMap.put("color", hex); tmpBins.add(binMap); i++; } wrapped = new SLDConfig(jspPath, units, style, strokeWidth, SLDGenerator.STROKE_OPACITY_DEFAULT, attrs, tmpThresholds, tmpColors, tmpBins, LegendType.CONTINUOUS); } }
Source:
<code>public void testRefreshConfig() throws Exception { GerritConfiguration config = client.refreshConfig(new NullProgressMonitor()); assertNotNull(config); assertNotNull(config.getGerritConfig()); assertNotNull(config.getProjects()); assertNotNull(config.getGerritConfig().getWildProject()); assertEquals("All-Projects", config.getGerritConfig().getWildProject().toString()); <START> assertNotNull(config.getGerritConfig().getSchemes()); <END> }</code><technical_language>this null in some versions</technical_language>

Target:
public void testRefreshConfig() throws Exception { GerritConfiguration config = client.refreshConfig(new NullProgressMonitor()); assertNotNull(config); assertNotNull(config.getGerritConfig()); assertNotNull(config.getProjects()); assertNotNull(config.getGerritConfig().getWildProject()); assertEquals("All-Projects", config.getGerritConfig().getWildProject().toString()); if (client.getVersion().getMinor() >= 12) { assertNotNull(config.getGerritConfig().getSchemes()); } }
Source:
<code>Mono<Response<DetectLanguageResult>> detectLanguageWithResponse(String text, String countryHint, Context context) { List<DetectLanguageInput> languageInputs = new ArrayList<>(); languageInputs.add(new DetectLanguageInput(Integer.toString(0), text, countryHint)); return detectBatchLanguagesWithResponse(languageInputs, null, context).flatMap(response -> { <START> if (response.getValue().iterator().hasNext()) { <END> return Mono.justOrEmpty(new SimpleResponse<>(response, response.getValue().iterator().next())); } return monoError(logger, new RuntimeException("Unable to retrieve language for the provided text.")); }); }</code><technical_language>time .iterator() is called, a new iterator for collection is returned. store variable. justOrEmpty just</technical_language>

Target:
Mono<Response<DetectLanguageResult>> detectLanguageWithResponse(String text, String countryHint, Context context) { Objects.requireNonNull(text, "'text' cannot be null."); List<DetectLanguageInput> languageInputs = Collections.singletonList(new DetectLanguageInput("0", text, countryHint)); return detectBatchLanguagesWithResponse(languageInputs, null, context).map(response -> new SimpleResponse<>(response, response.getValue().iterator().next())); }
Source:
<code><START> public void setMass(float mass) { if (mass <= 0) { logger.error("Invalid value for the mass. It can't be less than or equal to zero."); this.mass = 1; } this.mass = mass; } <END></code><technical_language>set this.mass 1 if parameter is a bad proceed overwrite bad :wink:</technical_language>

Target:
public void setMass(float mass) { if (mass <= 0) { logger.error("Invalid value for the mass. It can't be less than or equal to zero."); this.mass = 1; } else { this.mass = mass; } }
Source:
<code>public List<SDVariable> doDiff(List<SDVariable> i_v1) { SDVariable absIn = sameDiff.abs(arg()); SDVariable maxnorm = outputVariables()[0]; <START> SDVariable maxnormBc = sameDiff.f().reductionBroadcastableWithOrigShape(arg().getShape().length, dimensions, maxnorm); END> maxnormBc = sameDiff.onesLike(arg()).mul(maxnormBc); SDVariable eq = sameDiff.eq(absIn, maxnormBc); SDVariable dAbsXdX = sameDiff.sign(arg()); SDVariable ret = eq.mul(dAbsXdX); return Collections.singletonList(ret); }</code><technical_language>Arg a shape. some place holder logic want put in here. checking for null shapes do this good</technical_language>

Target:
public List<SDVariable> doDiff(List<SDVariable> i_v1) { SDVariable absIn = sameDiff.abs(arg()); SDVariable maxnorm = outputVariables()[0]; int origRank = Shape.rankFromShape(arg().getShape()); SDVariable maxnormBc = sameDiff.f().reductionBroadcastableWithOrigShape(origRank, dimensions, maxnorm); maxnormBc = sameDiff.onesLike(arg()).mul(maxnormBc); SDVariable eq = sameDiff.eq(absIn, maxnormBc); SDVariable dAbsXdX = sameDiff.sign(arg()); SDVariable ret = eq.mul(dAbsXdX); return Collections.singletonList(ret); }
Source:
<code>AbstractDataSetNormalizer(NormalizerStrategy<S> strategy) { <START> super(strategy); <END> }</code><technical_language>a big fan of "friendly" for scope. do explicit protected scope</technical_language>

Target:
protected AbstractDataSetNormalizer(NormalizerStrategy<S> strategy) { super(strategy); }
Source:
<code><START> public static Set<BoltServerAddress> serverSet( String... addresses ) <END> { Set<BoltServerAddress> result = new HashSet<>(); for ( String address : addresses ) { result.add( new BoltServerAddress( address ) ); } return result; }</code><technical_language>this method remain private</technical_language>

Target:
private static Set<BoltServerAddress> serverSet( String... addresses ) { Set<BoltServerAddress> result = new HashSet<>(); for ( String address : addresses ) { result.add( new BoltServerAddress( address ) ); } return result; }
Source:
<code>public static Iterable<Path> searchFiles(final String id, final String bucketName, final BulkJobType type) throws IOException { ensureDirExists(); List<Path> matches = new ArrayList<>(); <START> try (DirectoryStream<Path> stream = Files.newDirectoryStream(getTempdir(), "*" + RECOVERY_FILE_EXTENSION)) { <END> for (final Path file : stream) { final RecoveryJob job = getRecoveryJobByFile(file.toFile()); if(id == null || job.getId().compareTo(UUID.fromString(id)) == 0) { if(Guard.isStringNullOrEmpty(bucketName) || job.getBucketName().equals(bucketName)) { if(type == null || job.getJobType().equals(type)) { matches.add(file); } } } } } catch (final IOException e) { throw new IOException("Could not search files." , e); } return matches; }</code><technical_language>final</technical_language>

Target:
public static Iterable<Path> searchFiles(final String id, final String bucketName, final BulkJobType type) throws IOException { ensureDirExists(); final List<Path> matches = new ArrayList<>(); try (final DirectoryStream<Path> stream = Files.newDirectoryStream(getTempdir(), "*" + RECOVERY_FILE_EXTENSION)) { for (final Path file : stream) { final RecoveryJob job = getRecoveryJobByFile(file.toFile()); if(id == null || job.getId().compareTo(UUID.fromString(id)) == 0) { if(Guard.isStringNullOrEmpty(bucketName) || job.getBucketName().equals(bucketName)) { if(type == null || job.getJobType().equals(type)) { matches.add(file); } } } } } catch (final IOException e) { throw new IOException("Could not search files." , e); } return matches; }
Source:
<code>public void testAppendWithKey() throws Exception { final Appender appender = ctx.getRequiredAppender("KafkaAppenderWithKey"); final LogEvent logEvent = createLogEvent(); appender.append(logEvent); final List<ProducerRecord<byte[], byte[]>> history = kafka.history(); assertEquals(1, history.size()); final ProducerRecord<byte[], byte[]> item = history.get(0); assertNotNull(item); assertEquals(TOPIC_NAME, item.topic()); String msgKey = item.key().toString(); <START> byte[] keyValue = "key".getBytes(); <END> assertArrayEquals(item.key(), keyValue); assertEquals(LOG_MESSAGE, new String(item.value(), StandardCharsets.UTF_8)); }</code><technical_language>charset also</technical_language>

Target:
public void testAppendWithKey() throws Exception { final Appender appender = ctx.getRequiredAppender("KafkaAppenderWithKey"); final LogEvent logEvent = createLogEvent(); appender.append(logEvent); final List<ProducerRecord<byte[], byte[]>> history = kafka.history(); assertEquals(1, history.size()); final ProducerRecord<byte[], byte[]> item = history.get(0); assertNotNull(item); assertEquals(TOPIC_NAME, item.topic()); String msgKey = item.key().toString(); byte[] keyValue = "key".getBytes(StandardCharsets.UTF_8); assertArrayEquals(item.key(), keyValue); assertEquals(LOG_MESSAGE, new String(item.value(), StandardCharsets.UTF_8)); }
Source:
<code>protected void copyVmDevices() { <START> ArrayList<VmDevice> devices = new ArrayList<VmDevice>(vmFromConfiguration.getVmUnamagedDeviceList()); <END> devices.addAll(vmFromConfiguration.getManagedVmDeviceMap().values()); VmDeviceUtils.copyVmDevices(getVmIdFromSnapshot(), getVmId(), vmFromConfiguration, vmFromConfiguration.getStaticData(), true, devices, newDiskImages, _vmInterfaces); }</code><technical_language>devices (and should) defined List<VmDevice> Also, initialize arraylist FromConfiguration.getVmUnamagedDevic eList().size() + vmFromConfiguration.getManagedVmDeviceMap().size(), add items, a matter of taste</technical_language>

Target:
protected void copyVmDevices() { List<VmDevice> devices = new ArrayList<VmDevice>(vmFromConfiguration.getVmUnamagedDeviceList()); devices.addAll(vmFromConfiguration.getManagedVmDeviceMap().values()); VmDeviceUtils.copyVmDevices(getVmIdFromSnapshot(), getVmId(), vmFromConfiguration, vmFromConfiguration.getStaticData(), true, devices, newDiskImages, _vmInterfaces); }
Source:
<code>private Token finishToken(Token token) { _currentToken = token; <START> if (_currentToken == START_OBJECT) <END> { if (_currComplexObjTokenSize > 0) { _complexObjTokenSizeStack.push(_currComplexObjTokenSize); } _currComplexObjTokenSize = _intValue << 1; } else if (_currentToken == START_ARRAY) { if (_currComplexObjTokenSize > 0) { _complexObjTokenSizeStack.push(_currComplexObjTokenSize); } _currComplexObjTokenSize = _intValue; } else if (_currentToken != NOT_AVAILABLE) { _currComplexObjTokenSize--; } return _currentToken; }</code><technical_language>switch instead of if else here</technical_language>

Target:
private Token finishToken(Token token) { _currentToken = token; switch (_currentToken) { case START_OBJECT: if (_currComplexObjTokenSize > 0) { _complexObjTokenSizeStack.push(_currComplexObjTokenSize); } _currComplexObjTokenSize = _intValue << 1; break; case START_ARRAY: if (_currComplexObjTokenSize > 0) { _complexObjTokenSizeStack.push(_currComplexObjTokenSize); } _currComplexObjTokenSize = _intValue; break; case NOT_AVAILABLE: break; default: _currComplexObjTokenSize--; } return _currentToken; }