auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public void sendBatch(Batch batch) { final Batch filtered = filter(batch); debug.inspectBatch(DEBUG_ID, filtered); int batchSize = batch.getPoints().size(); hyperLog.offer(batch.hashCode()); statistics.reportMetricsCardinality(batch.hashCode()); <START> if (!Arrays.asList(KEYS_NEVER_TO_DROP).contains(batch.getPoints().get(0).getKey())) { <END> if (batchSize > 0 && !rateLimitAllowed(batchSize)) { log.debug("Dropping metrics due to rate limiting", batchSize); statistics.reportMetricsDroppedByRateLimit(batchSize); return; } if (!cardinalityLimitAllowed(hyperLog.cardinality())) { log.debug( "Dropping metrics due to cardinality limiting; cardinality {}", batchSize, hyperLog.cardinality()); statistics.reportMetricsDroppedByCardinalityLimit(batchSize); swapHyperLogLogPlus(); return; } } sinks.stream() .filter(PluginSink::isReady) .forEach(s -> s.sendBatch(filtered)); statistics.reportSentMetrics(batchSize); }</code><technical_language>this code for batch is identical for a single metric. code deduplicated</technical_language>

Target:
public void sendBatch(Batch batch) { final Batch filtered = filter(batch); debug.inspectBatch(DEBUG_ID, filtered); int batchSize = batch.getPoints().size(); hyperLog.get().offer(batch.hashCode()); statistics.reportMetricsCardinality(hyperLog.get().cardinality()); if (isDroppable(batchSize, batch.getPoints().get(0).getKey())) { return; } sinks.stream() .filter(PluginSink::isReady) .forEach(s -> s.sendBatch(filtered)); statistics.reportSentMetrics(batchSize); }
Source:
<code>private static Response buildServerError(Exception exception) { <START> log.error("Error in kick job: {}", Throwables.getStackTraceAsString(exception)); <END> String message = exception.getMessage(); if (message == null) { message = exception.toString(); } final String response = "ERROR: " + exception.getClass() + " was thrown <br/>" + "MESSAGE: " + StringEscapeUtils.escapeEcmaScript(message); return Response.serverError().entity(response).build(); }</code><technical_language>Do Throwables.getStackTraceAsString parameterize a logging message. correct methods slf4j logger for logging exception are: public void warn(String msg, Throwable t); public void error(String msg, Throwable t);</technical_language>

Target:
private static Response buildServerError(Exception exception) { log.warn("", exception); String message = exception.getMessage(); if (message == null) { message = exception.toString(); } final String response = "{" + "\"error\": \"A java exception was thrown.\", " + "\"message\": \"" + StringEscapeUtils.escapeEcmaScript(message) + "\"" + "}"; return Response.serverError().entity(response).build(); }
Source:
<code>public void testQueryFullProcessImageName() { HANDLE h = Kernel32.INSTANCE.OpenProcess(0, false, Kernel32.INSTANCE.GetCurrentProcessId()); char[] path = new char[1024]; <START> IntByReference lpdwSize = new IntByReference(path.length); <END> boolean b = Kernel32.INSTANCE.QueryFullProcessImageName(h, 0, path, lpdwSize); assertTrue(b); assertTrue(lpdwSize.getValue() > 0); }</code><technical_language>java char[] path = new char[WinDef.MAX_PATH];</technical_language>

Target:
public void testQueryFullProcessImageName() { HANDLE h = Kernel32.INSTANCE.OpenProcess(0, false, Kernel32.INSTANCE.GetCurrentProcessId()); assertNotNull("Failed (" + Kernel32.INSTANCE.GetLastError() + ") to get process handle", h); char[] path = new char[WinDef.MAX_PATH]; IntByReference lpdwSize = new IntByReference(path.length); boolean b = Kernel32.INSTANCE.QueryFullProcessImageName(h, 0, path, lpdwSize); assertTrue("Failed (" + Kernel32.INSTANCE.GetLastError() + ") to query process image name", b); assertTrue("Failed to query process image name, empty path returned", lpdwSize.getValue() > 0); }
Source:
<code>private List<PartitionId> getPartitionsInClass(String partitionClass, boolean minimumReplicaCountRequired) { List<PartitionId> toReturn = new ArrayList<>(); rwLock.readLock().lock(); try { if (partitionClass == null) { toReturn.addAll(allPartitions); } else { SortedMap<Integer, List<PartitionId>> partitionsByReplicaCount = partitionIdsByClassAndLocalReplicaCount.get(partitionClass); if (partitionsByReplicaCount == null) { partitionsByReplicaCount = partitionIdsByClassAndLocalReplicaCount.get(defaultPartitionClass); } if (partitionsByReplicaCount == null) { throw new IllegalArgumentException( <START> "No partitions for default partition class " + partitionClass + " found"); <END> } if (minimumReplicaCountRequired) { for (List<PartitionId> partitionIds : partitionsByReplicaCount.tailMap(minimumLocalReplicaCount).values()) { toReturn.addAll(partitionIds); } } else { for (List<PartitionId> partitionIds : partitionsByReplicaCount.values()) { toReturn.addAll(partitionIds); } } } } finally { rwLock.readLock().unlock(); } return toReturn; }</code><technical_language>minor: "No partitions for partition class = '" + partitionClass + "' default partition class = '" + defaultPartitionClass + "' found"</technical_language>

Target:
private List<PartitionId> getPartitionsInClass(String partitionClass, boolean minimumReplicaCountRequired) { List<PartitionId> toReturn = new ArrayList<>(); rwLock.readLock().lock(); try { if (partitionClass == null) { toReturn.addAll(allPartitions); } else { SortedMap<Integer, List<PartitionId>> partitionsByReplicaCount = partitionIdsByClassAndLocalReplicaCount.get(partitionClass); if (partitionsByReplicaCount == null) { partitionsByReplicaCount = partitionIdsByClassAndLocalReplicaCount.get(defaultPartitionClass); } if (partitionsByReplicaCount == null) { throw new IllegalArgumentException( "No partitions for partition class = '" + partitionClass + "' or default partition class = '" + defaultPartitionClass + "' found"); } if (minimumReplicaCountRequired) { for (List<PartitionId> partitionIds : partitionsByReplicaCount.tailMap(minimumLocalReplicaCount).values()) { toReturn.addAll(partitionIds); } } else { for (List<PartitionId> partitionIds : partitionsByReplicaCount.values()) { toReturn.addAll(partitionIds); } } } } finally { rwLock.readLock().unlock(); } return toReturn; }
Source:
<code>public void testDecisionExpressionLiteral() throws Exception { final String expected = loadResource("decision-expression-literal.xml"); setContent(expected); final String actual = getContent(); assertThat(actual).isNotBlank(); XmlAssert.assertThat(actual) .and(expected) .ignoreComments() .ignoreWhitespace() .areIdentical(); <START> final Map<String, String> prefix2Uri = new Maps.Builder<String, String>() .put(DMN.getPrefix(), DMN.getUri()) .build(); <END> XmlAssert.assertThat(actual) .withNamespaceContext(prefix2Uri) .hasXPath("/dmn:definitions" + "/dmn:decision[@id='_4FEA7589-823B-4880-BCFA-AF2F9B145785']" + "/dmn:literalExpression[@id='_35DB53A6-97E7-4D48-9E5A-59CE0015CEF8']" + "/dmn:text[text()='123']"); }</code><technical_language>Please this private static field of class, is in test</technical_language>

Target:
public void testDecisionExpressionLiteral() throws Exception { final String expected = loadResource("decision-expression-literal.xml"); setContent(expected); final String actual = getContent(); assertThat(actual).isNotBlank(); XmlAssert.assertThat(actual) .and(expected) .ignoreComments() .ignoreWhitespace() .areIdentical(); XmlAssert.assertThat(actual) .withNamespaceContext(NAMESPACES) .hasXPath("/dmn:definitions" + "/dmn:decision[@id='_4FEA7589-823B-4880-BCFA-AF2F9B145785']" + "/dmn:literalExpression[@id='_35DB53A6-97E7-4D48-9E5A-59CE0015CEF8']" + "/dmn:text[text()='123']"); }
Source:
<code>private static void assertLinesMatchWithFastForward(List<String> expectedLines, List<String> actualLines) { Deque<String> expectedDeque = new LinkedList<>(expectedLines); <START> Deque<String> actualDeque = new LinkedList<>(actualLines); <END> while (!expectedDeque.isEmpty()) { String expectedLine = expectedDeque.pop(); String actualLine = actualDeque.peek(); if (matches(expectedLine, actualLine)) { actualDeque.pop(); continue; } if (isFastForwardLine(expectedLine)) { int fastForwardLimit = parseFastForwardLimit(expectedLine); if (expectedDeque.isEmpty()) { int actualRemaining = actualDeque.size(); if (fastForwardLimit == Integer.MAX_VALUE || fastForwardLimit == actualRemaining) { return; } fail(format("terminal fast-forward(%d) error: fast-forward(%d) expected", fastForwardLimit, actualRemaining)); } expectedLine = expectedDeque.peek(); if (fastForwardLimit != Integer.MAX_VALUE) { for (int i = 0; i < fastForwardLimit; i++) { actualDeque.pop(); } if (actualDeque.isEmpty()) { fail(format("%d more lines expected, actual lines is empty", expectedDeque.size())); } continue; } while (true) { if (actualDeque.isEmpty()) { fail(format("no match for `%s` line fast-forwarding: %n%s", expectedLine, actualLines)); } if (matches(expectedLine, actualDeque.pop())) { break; } } } } if (!actualDeque.isEmpty()) { fail("more actual lines than expected: " + actualDeque.size()); } }</code><technical_language>Is reason LinkedLists instead of ArrayDeques (which I performance)</technical_language>

Target:
private static void assertLinesMatchWithFastForward(List<String> expectedLines, List<String> actualLines) { Deque<String> expectedDeque = new ArrayDeque<>(expectedLines); Deque<String> actualDeque = new ArrayDeque<>(actualLines); main: while (!expectedDeque.isEmpty()) { String expectedLine = expectedDeque.pop(); String actualLine = actualDeque.peek(); if (matches(expectedLine, actualLine)) { actualDeque.pop(); continue; } if (isFastForwardLine(expectedLine)) { int fastForwardLimit = parseFastForwardLimit(expectedLine); if (expectedDeque.isEmpty()) { int actualRemaining = actualDeque.size(); if (fastForwardLimit == Integer.MAX_VALUE || fastForwardLimit == actualRemaining) { return; } fail(expectedLines, actualLines, "terminal fast-forward(%d) error: fast-forward(%d) expected", fastForwardLimit, actualRemaining); } if (fastForwardLimit != Integer.MAX_VALUE) { for (int i = 0; i < fastForwardLimit; i++) { actualDeque.pop(); } continue; } expectedLine = expectedDeque.peek(); while (true) { if (actualDeque.isEmpty()) { fail(expectedLines, actualLines, "fast-forward(∞) didn't find: `%s`", snippet(expectedLine)); } if (matches(expectedLine, actualDeque.peek())) { continue main; } actualDeque.pop(); } } int number = expectedLines.size() - expectedDeque.size(); fail(expectedLines, actualLines, "expected line #%d:`%s` doesn't match", number, snippet(expectedLine)); } if (!actualDeque.isEmpty()) { fail(expectedLines, actualLines, "more actual lines than expected: %d", actualDeque.size()); } }
Source:
<code><START> public void createTable(String name) { <END> }</code><technical_language>this return AzureTable</technical_language>

Target:
public AzureTable createTable(String name) { return null; }
Source:
<code>public void testFilteredNotPaged() { MessageColumnIO columnIO = new ColumnIOFactory(true).getColumnIO(schema); MemPageStore memPageStore = writeTestRecords(columnIO, 8); RecordMaterializer<Group> recordConverter = new GroupRecordConverter(schema); RecordReaderImplementation<Group> recordReader = (RecordReaderImplementation<Group>) columnIO.getRecordReader(memPageStore, recordConverter, <START> Optional.of(Either.<UnboundRecordFilter, FilterPredicate>left(not(column("DocId", equalTo(10l)))))); <END> List<Group> all = readAll(recordReader); assertEquals("expecting 8 records " + all, 8, all.size()); for (int i = 0; i < all.size(); i++) { assertEquals("expecting record2", r2.toString(), all.get(i).toString()); } }</code><technical_language>a big fan of type. define a common interface instead</technical_language>

Target:
public void testFilteredNotPaged() { MessageColumnIO columnIO = new ColumnIOFactory(true).getColumnIO(schema); MemPageStore memPageStore = writeTestRecords(columnIO, 8); RecordMaterializer<Group> recordConverter = new GroupRecordConverter(schema); RecordReaderImplementation<Group> recordReader = (RecordReaderImplementation<Group>) columnIO.getRecordReader(memPageStore, recordConverter, FilterCompat.get(not(column("DocId", equalTo(10l))))); List<Group> all = readAll(recordReader); assertEquals("expecting 8 records " + all, 8, all.size()); for (int i = 0; i < all.size(); i++) { assertEquals("expecting record2", r2.toString(), all.get(i).toString()); } }
Source:
<code>protected LobParamsBuilder beginParams() { <START> return super.beginParams().putMap("mergeVariables", this.mergeVariables); <END> }</code><technical_language>This is is building request payload a request Lob's API. new parameter is "merge_variables". camelCase naming conventions for variables actual key in request "merge_variables". java return super.beginParams().putMap("merge_variables", this.mergeVariables);</technical_language>

Target:
protected LobParamsBuilder beginParams() { return super.beginParams().putMap("merge_variables", this.mergeVariables); }
Source:
<code><START> protected boolean validate() { <END> if (!super.validate()) { return false; } if (!getGlusterVolume().getVolumeType().isReplicatedType()) { addValidationMessage(EngineMessage.ACTION_TYPE_NOT_SUPPORTED_FOR_VOLUME_TYPE); return false; } if (!getGlusterVolume().isOnline()) { addValidationMessage(EngineMessage.ACTION_TYPE_FAILED_GLUSTER_VOLUME_IS_DOWN); return false; } if (getGlusterVolume().getAsyncTask() != null && (getGlusterVolume().getAsyncTask().getStatus() == JobExecutionStatus.STARTED || (getGlusterVolume().getAsyncTask().getType() == GlusterTaskType.REMOVE_BRICK && getGlusterVolume().getAsyncTask().getStatus() == JobExecutionStatus.FINISHED))) { addValidationMessage(EngineMessage.ACTION_TYPE_FAILED_GLUSTER_VOLUME_HAS_RUNNING_TASKS); return false; } if (getParameters().getExistingBrick() == null) { addValidationMessage(EngineMessage.ACTION_TYPE_FAILED_BRICKS_REQUIRED); return false; } if (!isValidVolumeBrick(getParameters().getExistingBrick())) { addValidationMessage(EngineMessage.ACTION_TYPE_FAILED_NOT_A_GLUSTER_VOLUME_BRICK); return false; } return true; }</code><technical_language>resetBrick requires support vdsm/gluster - need add a compatibility check check if feature is supported for cluster level. AddGlusterWebhookInternalCommand for example</technical_language>

Target:
protected boolean validate() { if (!super.validate()) { return false; } if (!supportedInConfig(ConfigValues.ResetBrickSupported, getCluster().getCompatibilityVersion())) { return false; } if (!getGlusterVolume().getVolumeType().isReplicatedType()) { addValidationMessage(EngineMessage.ACTION_TYPE_NOT_SUPPORTED_FOR_VOLUME_TYPE); return false; } if (!getGlusterVolume().isOnline()) { addValidationMessage(EngineMessage.ACTION_TYPE_FAILED_GLUSTER_VOLUME_IS_DOWN); return false; } if (getGlusterVolume().getAsyncTask() != null && (getGlusterVolume().getAsyncTask().getStatus() == JobExecutionStatus.STARTED || (getGlusterVolume().getAsyncTask().getType() == GlusterTaskType.REMOVE_BRICK && getGlusterVolume().getAsyncTask().getStatus() == JobExecutionStatus.FINISHED))) { addValidationMessage(EngineMessage.ACTION_TYPE_FAILED_GLUSTER_VOLUME_HAS_RUNNING_TASKS); return false; } if (getParameters().getExistingBrick() == null) { addValidationMessage(EngineMessage.ACTION_TYPE_FAILED_BRICKS_REQUIRED); return false; } if (!isValidVolumeBrick(getParameters().getExistingBrick())) { addValidationMessage(EngineMessage.ACTION_TYPE_FAILED_NOT_A_GLUSTER_VOLUME_BRICK); return false; } return true; }
Source:
<code>private void addProcedureParameter(MetadataFactory mf, Procedure procedure, Operation operation) { for(Parameter parameter : operation.getParameters()) { String name = parameter.getName(); String dataType = TypeFacility.RUNTIME_NAMES.STRING; String isPathParam = TRUE_STRING; if(parameter instanceof PathParameter) { PathParameter p = (PathParameter) parameter; String type = p.getType(); dataType = DataTypeManager.getDataTypeName(TypeFacility.getDataTypeClass(type)); } else if(parameter instanceof QueryParameter) { QueryParameter p = (QueryParameter) parameter; String type = p.getType(); dataType = DataTypeManager.getDataTypeName(TypeFacility.getDataTypeClass(type)); isPathParam = FALSE_STRING; } ProcedureParameter param = mf.addProcedureParameter(name, dataType, Type.In, procedure); param.setProperty(IS_PATH_PARAMETER, isPathParam); param.setNullType(NullType.No_Nulls); <START> param.setDefaultValue(FALSE_STRING); END> param.setAnnotation(parameter.getDescription()); } }</code><technical_language>? false string? if do define</technical_language>

Target:
private void addProcedureParameter(MetadataFactory mf, Procedure procedure, Operation operation) { for(Parameter parameter : operation.getParameters()) { String name = parameter.getName(); String dataType = TypeFacility.RUNTIME_NAMES.STRING; String isPathParam = TRUE_STRING; if(parameter instanceof PathParameter) { PathParameter p = (PathParameter) parameter; String type = p.getType(); dataType = DataTypeManager.getDataTypeName(TypeFacility.getDataTypeClass(type)); } else if(parameter instanceof QueryParameter) { QueryParameter p = (QueryParameter) parameter; String type = p.getType(); dataType = DataTypeManager.getDataTypeName(TypeFacility.getDataTypeClass(type)); isPathParam = FALSE_STRING; } ProcedureParameter param = mf.addProcedureParameter(name, dataType, Type.In, procedure); param.setProperty(IS_PATH_PARAMETER, isPathParam); param.setNullType(NullType.No_Nulls); param.setAnnotation(parameter.getDescription()); } }
Source:
<code>public void onGiveup(Exception firstException, Exception lastException) throws RetryExecutor.RetryGiveupException { <START> LOGGER.info("Giving up execution on exception", lastException); <END> }</code><technical_language>Another nitpick. firstException automatically re-thrown RetryExecutor, I this log more explicit is logging out lastException, propagated (or log out both)</technical_language>

Target:
public void onGiveup(Exception firstException, Exception lastException) throws RetryExecutor.RetryGiveupException { LOGGER.error("Giving up retry, first exception is [{}], last exception is [{}]", firstException.getMessage(), lastException.getMessage()); }
Source:
<code>public void createDeployment() throws Exception { WebArchive testDeployment = ShrinkWrap.create(WebArchive.class, ARCHIVE_FILE_NAME); testDeployment.addClasses(TaskSchedulerServletContextListener.class); tmpDir = new File("target/deployments/" + this.getClass().getSimpleName()); new File(tmpDir, "archives").mkdirs(); testDeployment.as(ZipExporter.class).exportTo(new File(tmpDir, "archives/" + ARCHIVE_FILE_NAME), true); final WildFlyManagedConfiguration config = createConfiguration("domain.xml", "host.xml", getClass().getSimpleName()); <START> config.setAdminOnly(true); <END> config.setReadOnlyDomain(true); config.setReadOnlyHost(true); String hostProps = config.getHostCommandLineProperties(); hostProps = hostProps == null ? "" : hostProps; config.setHostCommandLineProperties(hostProps + "\n-Djboss.unsupported.fail-boot-on-runtime-failure=true"); this.addTestResources(); }</code><technical_language>this point, domain created started. WildFlyManagedConfiguration object creation configuration removed</technical_language>

Target:
public void createDeployment() { WebArchive testDeployment = ShrinkWrap.create(WebArchive.class, ARCHIVE_FILE_NAME); testDeployment.addClasses(TaskSchedulerServletContextListener.class); tmpDir = new File("target/deployments/" + this.getClass().getSimpleName()); new File(tmpDir, "archives").mkdirs(); testDeployment.as(ZipExporter.class).exportTo(new File(tmpDir, "archives/" + ARCHIVE_FILE_NAME), true); }
Source:
<code><START> public void sendPasswordResetLinkEmail(PerunSession sess, Member member, String namespace, String url, String mailAttributeUrn, String language) throws InternalErrorException, PrivilegeException, MemberNotExistsException { <END> Utils.checkPerunSession(sess); getMembersManagerBl().checkMemberExists(sess, member); if (!AuthzResolver.isAuthorized(sess, Role.VOADMIN, member)) { throw new PrivilegeException(sess, "sendPasswordResetLinkEmail"); } getMembersManagerBl().sendPasswordResetLinkEmail(sess, member, namespace, url, mailAttributeUrn, language); }</code><technical_language>I add check attribute for urn exists</technical_language>

Target:
public void sendPasswordResetLinkEmail(PerunSession sess, Member member, String namespace, String url, String mailAttributeUrn, String language) throws InternalErrorException, PrivilegeException, MemberNotExistsException, UserNotExistsException, AttributeNotExistsException { Utils.checkPerunSession(sess); getMembersManagerBl().checkMemberExists(sess, member); if (!AuthzResolver.isAuthorized(sess, Role.VOADMIN, member)) { throw new PrivilegeException(sess, "sendPasswordResetLinkEmail"); } Attribute mailAttribute = null; AttributeDefinition ad = getPerunBl().getAttributesManager().getAttributeDefinition(sess, mailAttributeUrn); try { if (ad.getEntity().equals("user")) { User user = perunBl.getUsersManagerBl().getUserByMember(sess, member); mailAttribute = getPerunBl().getAttributesManager().getAttribute(sess, user, mailAttributeUrn); } if (ad.getEntity().equals("member")) { mailAttribute = getPerunBl().getAttributesManager().getAttribute(sess, member, mailAttributeUrn); } } catch (WrongAttributeAssignmentException ex) { throw new InternalErrorException(ex); } String mailAddress = (String) mailAttribute.getValue(); getMembersManagerBl().sendPasswordResetLinkEmail(sess, member, namespace, url, mailAddress, language); }
Source:
<code>private void sendMuteCommand(String itemName, String command, DataOutputStream outToServer, BufferedReader i) throws IOException { char[] cbuf = new char[50]; outToServer.write(hexStringToByteArray(command)); i.read(cbuf, 0, 50); String result = new String(cbuf); <START> if (result.contains("Mute=on") || result.contains("MuteOn")) <END> eventPublisher.postUpdate(itemName, OnOffType.OFF); else if (result.contains("Mute=off") || result.contains("MuteOff")) eventPublisher.postUpdate(itemName, OnOffType.ON); else logger.error("Error sending command, received this: " + new String(cbuf)); }</code><technical_language>curly braces</technical_language>

Target:
private void sendMuteCommand(String itemName, String command, DataOutputStream outToServer, BufferedReader i) throws IOException { char[] cbuf = new char[50]; logger.debug("Sending mute command {}", command); outToServer.write(hexStringToByteArray(command)); i.read(cbuf, 0, 50); String result = new String(cbuf); logger.trace("Received this result: {}", result); if (result.contains("Mute=on") || result.contains("MuteOn")) { eventPublisher.postUpdate(itemName, OnOffType.OFF); logger.debug("Setting mute item on OFF", itemName); } else if (result.contains("Mute=off") || result.contains("MuteOff")) { eventPublisher.postUpdate(itemName, OnOffType.ON); logger.debug("Setting mute item on ON", itemName); } else logger.error("Error sending mute command {}, received this: {}", command, result); }
Source:
<code>public void onCreate(Bundle savedInstanceState) { super.onCreate(savedInstanceState); ((SyncthingActivity) getActivity()).registerOnServiceConnectedListener(this); mIsCreate = ((SettingsActivity) getActivity()).getIsCreate(); setHasOptionsMenu(true); if (mIsCreate) { addPreferencesFromResource(R.xml.device_settings_create); } else { addPreferencesFromResource(R.xml.device_settings_edit); } mDeviceId = findPreference("device_id"); mDeviceId.setOnPreferenceChangeListener(this); mName = (EditTextPreference) findPreference("name"); mName.setOnPreferenceChangeListener(this); mAddresses = (EditTextPreference) findPreference("addresses"); mAddresses.setOnPreferenceChangeListener(this); mCompression = (ListPreference) findPreference("compression"); mCompression.setOnPreferenceChangeListener(this); mIntroducer = (CheckBoxPreference) findPreference("introducer"); mIntroducer.setOnPreferenceChangeListener(this); if (!mIsCreate) { mVersion = findPreference("version"); mVersion.setSummary("?"); mCurrentAddress = findPreference("current_address"); mCurrentAddress.setSummary("?"); } if (mIsCreate) { <START> mDevice = null; <END> if (savedInstanceState != null) { mDevice = (RestApi.Device) savedInstanceState.getSerializable("device"); } if (mDevice == null) { mDevice = new RestApi.Device(); mDevice.Name = ""; mDevice.DeviceID = ""; mDevice.Addresses = "dynamic"; mDevice.Compression = "always"; ((EditTextPreference) mDeviceId).setText(mDevice.DeviceID); } } }</code><technical_language>This is pointless, mDevice is null (not initialized)</technical_language>

Target:
public void onCreate(Bundle savedInstanceState) { super.onCreate(savedInstanceState); ((SyncthingActivity) getActivity()).registerOnServiceConnectedListener(this); mIsCreate = ((SettingsActivity) getActivity()).getIsCreate(); setHasOptionsMenu(true); if (mIsCreate) { addPreferencesFromResource(R.xml.device_settings_create); } else { addPreferencesFromResource(R.xml.device_settings_edit); } mDeviceId = findPreference("device_id"); mDeviceId.setOnPreferenceChangeListener(this); mName = (EditTextPreference) findPreference("name"); mName.setOnPreferenceChangeListener(this); mAddresses = (EditTextPreference) findPreference("addresses"); mAddresses.setOnPreferenceChangeListener(this); mCompression = (ListPreference) findPreference("compression"); mCompression.setOnPreferenceChangeListener(this); mIntroducer = (CheckBoxPreference) findPreference("introducer"); mIntroducer.setOnPreferenceChangeListener(this); if (!mIsCreate) { mVersion = findPreference("version"); mVersion.setSummary("?"); mCurrentAddress = findPreference("current_address"); mCurrentAddress.setSummary("?"); } if (mIsCreate) { if (savedInstanceState != null) { mDevice = (RestApi.Device) savedInstanceState.getSerializable("device"); } if (mDevice == null) { mDevice = new RestApi.Device(); mDevice.Name = ""; mDevice.DeviceID = ""; mDevice.Addresses = "dynamic"; mDevice.Compression = "always"; ((EditTextPreference) mDeviceId).setText(mDevice.DeviceID); } } }
Source:
<code>private RollingUpdateOp rollingUpdateTimedoutError(final RollingUpdateOpFactory opFactory, final String host, final JobId jobId, final TaskStatus taskStatus) { final List<TaskStatus.State> previousJobStates = getPreviousJobStates(jobId, host, 10); final String baseError = "timed out waiting for job to reach state RUNNING "; final String stateInfo = String.format( "(terminal job state %s, previous states: %s)", taskStatus.getState(), Joiner.on("->").join(previousJobStates)); final Map<String, Object> metadata = Maps.newHashMap(); metadata.put("jobState", taskStatus.getState()); metadata.put("previousJobStates", previousJobStates); metadata.put("throttleState", taskStatus.getThrottled()); if (taskStatus.getThrottled().equals(ThrottleState.IMAGE_MISSING)) { return opFactory.error( baseError + "due to missing Docker image " + stateInfo, host, RollingUpdateError.IMAGE_MISSING, metadata); } if (taskStatus.getThrottled().equals(ThrottleState.IMAGE_PULL_FAILED)) { return opFactory.error( baseError + "due to failure pulling Docker image " + stateInfo, host, RollingUpdateError.IMAGE_PULL_FAILED, metadata); } <START> if (!taskStatus.getContainerError().equals("")) { <END> return opFactory.error( baseError + stateInfo + " container error: " + taskStatus.getContainerError(), host, RollingUpdateError.TIMED_OUT_WAITING_FOR_JOB_TO_REACH_RUNNING, metadata); } return opFactory.error( baseError + stateInfo, host, RollingUpdateError.TIMED_OUT_WAITING_FOR_JOB_TO_REACH_RUNNING, metadata); }</code><technical_language>This is a matter of personal preference, guava a isNullOrEmpty() this code base a lot</technical_language>

Target:
private RollingUpdateOp rollingUpdateTimedoutError(final RollingUpdateOpFactory opFactory, final String host, final JobId jobId, final TaskStatus taskStatus) { final List<TaskStatus.State> previousJobStates = getPreviousJobStates(jobId, host, 10); final String baseError = "timed out waiting for job to reach state RUNNING "; final String stateInfo = String.format( "(terminal job state %s, previous states: %s)", taskStatus.getState(), Joiner.on("->").join(previousJobStates)); final Map<String, Object> metadata = Maps.newHashMap(); metadata.put("jobState", taskStatus.getState()); metadata.put("previousJobStates", previousJobStates); metadata.put("throttleState", taskStatus.getThrottled()); if (taskStatus.getThrottled().equals(ThrottleState.IMAGE_MISSING)) { return opFactory.error( baseError + "due to missing Docker image " + stateInfo, host, RollingUpdateError.IMAGE_MISSING, metadata); } if (taskStatus.getThrottled().equals(ThrottleState.IMAGE_PULL_FAILED)) { return opFactory.error( baseError + "due to failure pulling Docker image " + stateInfo, host, RollingUpdateError.IMAGE_PULL_FAILED, metadata); } if (!Strings.isNullOrEmpty(taskStatus.getContainerError())) { return opFactory.error( baseError + stateInfo + " container error: " + taskStatus.getContainerError(), host, RollingUpdateError.TIMED_OUT_WAITING_FOR_JOB_TO_REACH_RUNNING, metadata); } return opFactory.error( baseError + stateInfo, host, RollingUpdateError.TIMED_OUT_WAITING_FOR_JOB_TO_REACH_RUNNING, metadata); }
Source:
<code>public void addArgumentsToInstallFeatures(String repositoryLocation, String installIUs, String destination, String profile) { launcher.addArguments( "-metadataRepository", repositoryLocation, "-artifactRepository", repositoryLocation, "-profileProperties", "org.eclipse.update.install.features=true", "-installIU", installIUs, <START> "-bundlepool", destination + File.separator + "lib", <END> "-shared", destination + File.separator + "lib" + File.separator + "p2", "-destination", destination + File.separator + profile, "-profile", profile, "-roaming"); }</code><technical_language>replaced Paths</technical_language>

Target:
public void addArgumentsToInstallFeatures(String repositoryLocation, String installIUs, String destination, String profile) { launcher.addArguments( "-metadataRepository", repositoryLocation, "-artifactRepository", repositoryLocation, "-profileProperties", "org.eclipse.update.install.features=true", "-installIU", installIUs, "-bundlepool", Paths.get(destination + File.separator + "lib").toString(), "-shared", Paths.get(destination + File.separator + "lib" + File.separator + "p2").toString(), "-destination", Paths.get(destination + File.separator + profile).toString(), "-profile", profile, "-roaming"); }
Source:
<code>public JdbcClientBuilder fromType(JdbcDriverType type, String db, String host, int port) { dbDriver(type.getDriver()); String dbHost = host; if (System.getProperty("osee.application.server") == null) { dbHost = JDBC_SERVER__LOCAL_CONNECTIONS; <START> } else { <END> dbHost = System.getProperty("osee.application.server"); } dbUri(type.getUriFormat(), type.getPrefix(), dbHost, port, db); switch (type) { case hsql: dbParamsInUri(true); if (!Strings.isValid(getDbUsername())) { dbUsername("public"); } break; case oracle_thin: if (!getDbProps().containsKey("SetBigStringTryClob")) { dbParam("SetBigStringTryClob", "true"); } if (!getDbProps().containsKey("includeSynonyms")) { dbParam("includeSynonyms", "true"); } break; default: break; } return this; }</code><technical_language>OseeClient.OSEE_APPLICATION_SERVER</technical_language>

Target:
public JdbcClientBuilder fromType(JdbcDriverType type, String db, String host, int port) { dbDriver(type.getDriver()); String dbHost = host; if (System.getProperty(OseeClient.OSEE_APPLICATION_SERVER) == null) { dbHost = JDBC_SERVER__LOCAL_CONNECTIONS; } else { dbHost = System.getProperty(OseeClient.OSEE_APPLICATION_SERVER); } dbUri(type.getUriFormat(), type.getPrefix(), dbHost, port, db); switch (type) { case hsql: dbParamsInUri(true); if (!Strings.isValid(getDbUsername())) { dbUsername("public"); } break; case oracle_thin: if (!getDbProps().containsKey("SetBigStringTryClob")) { dbParam("SetBigStringTryClob", "true"); } if (!getDbProps().containsKey("includeSynonyms")) { dbParam("includeSynonyms", "true"); } break; default: break; } return this; }
Source:
<code>public void testGoodSubjectAmqp() { Message message = new AMQPMessage(1); <START> message.setAddress("test.address"); <END> securityServerPlugin.handleMessage(mockServerSession, null, message, false, false); assertThat(message.getStringProperty("subject"), is("Hello World")); }</code><technical_language>pencil2: plugin adds some extra stuff message AMQP message, a good idea check for here</technical_language>

Target:
public void testGoodSubjectAmqp() { Message message = new AMQPMessage(1); message.setAddress("test.address"); securityServerPlugin.handleMessage(mockServerSession, null, message, false, false); assertThat(message.getStringProperty("subject"), is("Hello World")); assertThat( ((AMQPMessage) message) .getProtonMessage() .getApplicationProperties() .getValue() .get("subject"), is("Hello World")); }
Source:
<code>public static Observation shallowClone(Observation observation) { if (null == observation) { return null; } final Request request = observation.getRequest(); if (null == request) { throw new IllegalArgumentException("missing request for observation!"); } Request clonedRequest = new Request(request.getCode()); clonedRequest.setDestinationContext(request.getDestinationContext()); clonedRequest.setType(request.getType()); clonedRequest.setMID(request.getMID()); clonedRequest.setToken(request.getToken()); clonedRequest.setOptions(request.getOptions()); clonedRequest.setPayload(request.getPayload()); clonedRequest.setUserContext(request.getUserContext()); clonedRequest.addMessageObserver(new MessageObserverAdapter() { @Override public void onResponse(Response response) { request.setResponse(response); } <START> }); <END> return new Observation(clonedRequest, observation.getContext()); }</code><technical_language>do need this ? This behavior a persistent store InMemoryObservationStore</technical_language>

Target:
public static Observation shallowClone(Observation observation) { if (null == observation) { return null; } Request request = observation.getRequest(); if (null == request) { throw new IllegalArgumentException("missing request for observation!"); } Request clonedRequest = new Request(request.getCode()); clonedRequest.setDestinationContext(request.getDestinationContext()); clonedRequest.setType(request.getType()); clonedRequest.setMID(request.getMID()); clonedRequest.setToken(request.getToken()); clonedRequest.setOptions(request.getOptions()); clonedRequest.setPayload(request.getPayload()); clonedRequest.setUserContext(request.getUserContext()); return new Observation(clonedRequest, observation.getContext()); }
Source:
<code>public void validateChartData() { super.validateChartData(); if (getXDescriptorsInfo().areNumericals()) { throw new IllegalArgumentException("Bar chart X axis cannot be numerical."); } if (!getYDescriptorsInfo().areNumericals()) { throw new IllegalArgumentException("Bar chart Y axis must be numerical."); } <START> if (countDiffDescriptors(getXDescriptors()) > 1) { <END> throw new IllegalArgumentException("Bar chart can only have one X axis."); } }</code><technical_language>countDistinct</technical_language>

Target:
public void validateChartData() { super.validateChartData(); if (getXDescriptorsInfo().areNumericals()) { throw new IllegalArgumentException("Bar chart X axis cannot be numerical."); } if (!getYDescriptorsInfo().areNumericals()) { throw new IllegalArgumentException("Bar chart Y axis must be numerical."); } if (countDistinctDescriptors(getXDescriptors()) > 1) { throw new IllegalArgumentException("Bar chart can only have one X axis."); } }
Source:
<code>public void cleanUp() throws Exception { if (executor != null) { executor.shutdownNow(); } <START> httpClient.close(); <END> query(format("DROP TABLE IF EXISTS %s", TABLE_NAME)); }</code><technical_language>check for null too</technical_language>

Target:
public void cleanUp() throws Exception { if (executor != null) { executor.shutdownNow(); } if (queryCanceller != null) { queryCanceller.close(); } }
Source:
<code>public void goalSelected(Goal goal) { if(application.getGoals().contains(goal)) { deleteGoal(goal); } else { addGoal(goal); if (goal.getBehaviorCount() > 0){ Intent intent = new Intent(getApplicationContext(), GoalTryActivity.class); intent.putExtra("goal", goal); intent.putExtra("category", mCategory); startActivity(intent); <START> } <END> } }</code><technical_language>add else block a Toast includes "Goal Selected". do in other places, consistent</technical_language>

Target:
public void goalSelected(Goal goal) { if(application.getGoals().contains(goal)) { deleteGoal(goal); } else { addGoal(goal); if (goal.getBehaviorCount() > 0){ Intent intent = new Intent(getApplicationContext(), GoalTryActivity.class); intent.putExtra("goal", goal); intent.putExtra("category", mCategory); startActivity(intent); } else{ Toast.makeText(this, R.string.goal_selected, Toast.LENGTH_SHORT).show(); } } }
Source:
<code>public void moveAliens() { Iterator iteralien = ConcreteAggregate.createIterator(aliens); while (iteralien.hasNext()) { Alien alien = (Alien) iteralien.next(); alien.hmovement(); } if (Alien.getupdateLogic()) { <START> Iterator iteralien2 = ConcreteAggregate.createIterator(aliens); <END> while (iteralien2.hasNext()) { Alien alien = (Alien) iteralien2.next(); alien.vmovement(); } Game.logfile.writeString("Aliens reached a border and moved down"); } }</code><technical_language>iteralien2 -> iterAliens2</technical_language>

Target:
public void moveAliens() { Iterator iterAliens = ConcreteAggregate.createIterator(aliens); while (iterAliens.hasNext()) { Alien alien = (Alien) iterAliens.next(); alien.hmovement(); } if (Alien.getupdateLogic()) { Iterator iterAliens2 = ConcreteAggregate.createIterator(aliens); while (iterAliens2.hasNext()) { Alien alien = (Alien) iterAliens2.next(); alien.vmovement(); } Game.logfile.writeString("Aliens reached a border and moved down"); } }
Source:
<code>public void visitMethodInvocation(MethodInvocationTree mit) { <START> if (isInvocationOnVariable(mit, variable) && SET_SSL_CHECK_SERVER_ID.matches(mit) && isNotFalse(mit.arguments().get(0))) { <END> this.isSecured = true; } super.visitMethodInvocation(mit); }</code><technical_language>This test condition leads FP if bad resolving variable symbol. I prefer replace isInvocationOnVariable isInvocationOnTheGivenVariableButReturnTrueIfYourAreUnsureAboutOnWhatThisMethodIsInvoked :wink:</technical_language>

Target:
public void visitMethodInvocation(MethodInvocationTree mit) { if (isInvocationOnVariable(mit, variable, true) && SET_SSL_CHECK_SERVER_ID.matches(mit) && isNotFalse(mit.arguments().get(0))) { this.isSecured = true; } super.visitMethodInvocation(mit); }
Source:
<code>public Map<String, LocalMapStats> getStats() { <START> LocalMapStatsProvider localMapStatsProvider = mapServiceContext.getLocalMapStatsProvider(); <END> return localMapStatsProvider.createAllLocalMapStats(); }</code><technical_language>statisticsAwareService anymore</technical_language>

Target:
public Map<String, LocalMapStats> getStats() { return statisticsAwareService.getStats(); }
Source:
<code>void download() { if (downloadBatchStatus.status() == PAUSED) { return; } if (downloadBatchStatus.status() == DELETION) { return; } <START> if (downloadBatchStatus.status() == DOWNLOADED) { <END> return; } downloadBatchStatus.markAsDownloading(downloadsBatchPersistence); notifyCallback(downloadBatchStatus); totalBatchSizeBytes = getTotalSize(downloadFiles); if (totalBatchSizeBytes <= ZERO_BYTES) { DownloadError downloadError = new DownloadError(DownloadError.Error.NETWORK_ERROR_CANNOT_DOWNLOAD_FILE); downloadBatchStatus.markAsError(downloadError, downloadsBatchPersistence); notifyCallback(downloadBatchStatus); return; } DownloadFile.Callback fileDownloadCallback = downloadFileStatus -> { fileBytesDownloadedMap.put(downloadFileStatus.downloadFileId(), downloadFileStatus.bytesDownloaded()); long currentBytesDownloaded = getBytesDownloadedFrom(fileBytesDownloadedMap); downloadBatchStatus.update(currentBytesDownloaded, totalBatchSizeBytes); if (downloadFileStatus.isMarkedAsError()) { downloadBatchStatus.markAsError(downloadFileStatus.error(), downloadsBatchPersistence); } callbackThrottle.update(downloadBatchStatus); }; for (DownloadFile downloadFile : downloadFiles) { downloadFile.download(fileDownloadCallback); if (batchCannotContinue()) { break; } } if (networkError()) { DownloadsNetworkRecoveryCreator.getInstance().scheduleRecovery(); } if (downloadBatchStatus.status() == DOWNLOADED) { downloadBatchStatus.markAsDownloaded(downloadsBatchPersistence); } callbackThrottle.stopUpdates(); }</code><technical_language>of them? :)</technical_language>

Target:
void download() { DownloadBatchStatus.Status status = downloadBatchStatus.status(); if (status == PAUSED) { return; } if (status == DELETION) { return; } if (status == DOWNLOADED) { return; } downloadBatchStatus.markAsDownloading(downloadsBatchPersistence); notifyCallback(downloadBatchStatus); totalBatchSizeBytes = getTotalSize(downloadFiles); if (totalBatchSizeBytes <= ZERO_BYTES) { DownloadError downloadError = new DownloadError(DownloadError.Error.NETWORK_ERROR_CANNOT_DOWNLOAD_FILE); downloadBatchStatus.markAsError(downloadError, downloadsBatchPersistence); notifyCallback(downloadBatchStatus); return; } DownloadFile.Callback fileDownloadCallback = downloadFileStatus -> { fileBytesDownloadedMap.put(downloadFileStatus.downloadFileId(), downloadFileStatus.bytesDownloaded()); long currentBytesDownloaded = getBytesDownloadedFrom(fileBytesDownloadedMap); downloadBatchStatus.update(currentBytesDownloaded, totalBatchSizeBytes); if (downloadFileStatus.isMarkedAsError()) { downloadBatchStatus.markAsError(downloadFileStatus.error(), downloadsBatchPersistence); } callbackThrottle.update(downloadBatchStatus); }; for (DownloadFile downloadFile : downloadFiles) { downloadFile.download(fileDownloadCallback); if (batchCannotContinue()) { break; } } if (networkError()) { DownloadsNetworkRecoveryCreator.getInstance().scheduleRecovery(); } if (status == DOWNLOADED) { downloadBatchStatus.markAsDownloaded(downloadsBatchPersistence); } callbackThrottle.stopUpdates(); }
Source:
<code>public void checkPaths() throws ConfigurationException { File badInstanceClid = new File(generator.getNuxeoHome(), getDefaultDataDir() + File.separator + "instance.clid"); if (badInstanceClid.exists() && !getDataDir().equals(badInstanceClid.getParentFile())) { log.warn("Moving to {}.", () -> badInstanceClid, this::getDataDir); try { FileUtils.moveFileToDirectory(badInstanceClid, getDataDir(), true); } catch (IOException e) { throw new ConfigurationException("NXP-6722 move failed: " + e.getMessage(), e); } } File oldPackagesPath = new File(getDataDir(), getDefaultPackagesDir()); if (oldPackagesPath.exists() && !oldPackagesPath.equals(getPackagesDir())) { log.warn("NXP-8014 Packages cache location changed. You can safely delete or move its content to {}", () -> oldPackagesPath, this::getPackagesDir); } File oldPath = new File(getRuntimeHome(), "data" + File.separator + "vcsh2repo"); String message = String.format("NXP-5370, NXP-5460. " + "Please rename 'vcsh2repo' directory from %s to %s", <START> oldPath, new File(generator.getDataDir(), "h2" + File.separator + "nuxeo")); <END> checkPath(oldPath, message); }</code><technical_language>This is obsolete (I remember used). I worth keeping</technical_language>

Target:
public void checkPaths() throws ConfigurationException { File badInstanceClid = new File(generator.getNuxeoHome(), getDefaultDataDir() + File.separator + "instance.clid"); if (badInstanceClid.exists() && !getDataDir().equals(badInstanceClid.getParentFile())) { log.warn("Moving to {}.", () -> badInstanceClid, this::getDataDir); try { FileUtils.moveFileToDirectory(badInstanceClid, getDataDir(), true); } catch (IOException e) { throw new ConfigurationException("NXP-6722 move failed: " + e.getMessage(), e); } } File oldPackagesPath = new File(getDataDir(), getDefaultPackagesDir()); if (oldPackagesPath.exists() && !oldPackagesPath.equals(getPackagesDir())) { log.warn("NXP-8014 Packages cache location changed. You can safely delete or move its content to {}", () -> oldPackagesPath, this::getPackagesDir); } }
Source:
<code>protected boolean isConditionsDefaultValueEnabled() { try { <START> final String ConditionsDefaultValueEnabled = PropertyAccessor.getInstance() <END> .getProperty(NhincConstants.GATEWAY_PROPERTY_FILE, NhincConstants.ENABLE_CONDITIONS_DEFAULT_VALUE); if (ConditionsDefaultValueEnabled != null) { return !ConditionsDefaultValueEnabled.equals(Boolean.FALSE.toString()); } } catch (final PropertyAccessException pae) { LOG.warn("Property not found: {}", NhincConstants.ENABLE_CONDITIONS_DEFAULT_VALUE, pae.getLocalizedMessage()); LOG.trace("Property not found exception: {}", pae.getLocalizedMessage(), pae); } return Boolean.TRUE; }</code><technical_language>java name convention. <LINK_0></technical_language>

Target:
protected boolean isConditionsDefaultValueEnabled() { try { final Boolean conditionsDefaultValueEnabled = PropertyAccessor.getInstance().getPropertyBoolean( NhincConstants.GATEWAY_PROPERTY_FILE, NhincConstants.ENABLE_CONDITIONS_DEFAULT_VALUE); return !conditionsDefaultValueEnabled.equals(Boolean.FALSE); } catch (final PropertyAccessException pae) { LOG.trace("Property not found exception: {}", pae.getLocalizedMessage(), pae); } return Boolean.TRUE; }
Source:
<code>public JWindowPreference(Window window) { super (window.getName()); try { this.width = window.getWidth(); this.height = window.getHeight(); this.screenX = window.getLocationOnScreen().x; this.screenY = window.getLocationOnScreen().y; if (window instanceof JFrame) { this.isMaximized = (((JFrame)window).getState() & Frame.MAXIMIZED_BOTH) == Frame.MAXIMIZED_BOTH; } else { this.isMaximized = false; } <START> } catch (Exception ignored) { <END> } this.weakRef = new WeakReference<>(window); window.addWindowStateListener(this); window.addComponentListener(this); }</code><technical_language>exception ignoring</technical_language>

Target:
public JWindowPreference(Window window) { super (window.getName()); try { this.width = window.getWidth(); this.height = window.getHeight(); this.screenX = window.getLocationOnScreen().x; this.screenY = window.getLocationOnScreen().y; if (window instanceof JFrame) { this.isMaximized = (((JFrame)window).getExtendedState() & Frame.MAXIMIZED_BOTH) == Frame.MAXIMIZED_BOTH; } else { this.isMaximized = false; } } catch (Exception ignored) { } this.weakRef = new WeakReference<>(window); window.addWindowStateListener(this); window.addComponentListener(this); }
Source:
<code>public ImageWorker(final RenderedImage image) { <START> inheritanceStopPoint = this.image = image; <END> setnoData(extractNoDataProperty(image)); }</code><technical_language>sense call setImage here? do semantics in method this one</technical_language>

Target:
public ImageWorker(final RenderedImage image) { setImage(image); }
Source:
<code>private void reComputeDiagnostics(String pomPath) { String pomUri = prefixURI(pomPath); Type type = new TypeToken<PublishDiagnosticsParams>() {}.getType(); PublishDiagnosticsParams result = doGetOne(Commands.RECOMPUTE_POM_DIAGNOSTICS, singletonList(pomUri), type); <START> CheLanguageClient cheLanguageClient = <END> clientFactory.create(findInitializedLanguageServer().get().getId()); cheLanguageClient.publishDiagnostics(result); }</code><technical_language>create CheLanguageClient is enough add a new method: public void publishDiagnostics(PublishDiagnosticsParams diagnostics) { eventService.publish(new ExtendedPublishDiagnosticsParams(serverId, diagnostics)); }</technical_language>

Target:
private void reComputeDiagnostics(String pomPath) { String pomUri = prefixURI(pomPath); executeCommand(RECOMPUTE_POM_DIAGNOSTICS, singletonList(pomUri)); }