auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public void testParseMetadata() throws Exception { <START> String json = IOUtils.toString(getClass().getResourceAsStream("vectorLayers.json")); <END> VectorLayersMetadata layersMetadata = VectorLayersMetadata.parseMetadata(json); List<VectorLayerMetadata> layers = layersMetadata.getLayers(); assertEquals(15, layers.size()); VectorLayerMetadata water = layers.get(1); assertEquals("waterway", water.getId()); assertEquals(Integer.valueOf(0), water.getMinZoom()); assertEquals(Integer.valueOf(14), water.getMaxZoom()); assertEquals("String", water.getFields().get("class")); VectorLayerMetadata mountainPeak = layers.get(4); assertEquals("mountain_peak", mountainPeak.getId()); assertEquals(Integer.valueOf(0), mountainPeak.getMinZoom()); assertEquals(Integer.valueOf(14), mountainPeak.getMaxZoom()); assertEquals("Number", mountainPeak.getFields().get("rank")); }</code><technical_language>toString(java.io.InputStream) usage is deprecated. IOUtils.toString(inputStream, StandardCharsets.UTF_8.name()) is recommended. This is critical, is causing a build issue some travis build</technical_language>

Target:
public void testParseMetadata() throws Exception { String json = IOUtils.toString( getClass().getResourceAsStream("vectorLayers.json"), Charset.forName("UTF-8")); VectorLayersMetadata layersMetadata = VectorLayersMetadata.parseMetadata(json); List<VectorLayerMetadata> layers = layersMetadata.getLayers(); assertEquals(15, layers.size()); VectorLayerMetadata water = layers.get(1); assertEquals("waterway", water.getId()); assertEquals(Integer.valueOf(0), water.getMinZoom()); assertEquals(Integer.valueOf(14), water.getMaxZoom()); assertEquals("String", water.getFields().get("class")); VectorLayerMetadata mountainPeak = layers.get(4); assertEquals("mountain_peak", mountainPeak.getId()); assertEquals(Integer.valueOf(0), mountainPeak.getMinZoom()); assertEquals(Integer.valueOf(14), mountainPeak.getMaxZoom()); assertEquals("Number", mountainPeak.getFields().get("rank")); }
Source:
<code>public boolean checkCommitInfo() { updateStateFromUI(); if (commitMessage.trim().length() == 0) { MessageDialog.openWarning(getShell(), UIText.CommitDialog_ErrorNoMessage, UIText.CommitDialog_ErrorMustEnterCommitMessage); return false; } boolean authorValid = false; if (author.length() > 0) { authorValid = RawParseUtils.parsePersonIdent(author) != null; <START> } <END> if (!authorValid) { MessageDialog.openWarning(getShell(), UIText.CommitDialog_ErrorInvalidAuthor, UIText.CommitDialog_ErrorInvalidAuthorSpecified); return false; } boolean committerValid = false; if (committer.length() > 0) { committerValid = RawParseUtils.parsePersonIdent(committer) != null; } if (!committerValid) { MessageDialog.openWarning(getShell(), UIText.CommitDialog_ErrorInvalidAuthor, UIText.CommitDialog_ErrorInvalidCommitterSpecified); return false; } authorHandler.updateProposals(); committerHandler.updateProposals(); return true; }</code><technical_language>Style nit: unneeded curlies</technical_language>

Target:
public boolean checkCommitInfo() { updateStateFromUI(); if (commitMessage.trim().length() == 0) { MessageDialog.openWarning(getShell(), UIText.CommitDialog_ErrorNoMessage, UIText.CommitDialog_ErrorMustEnterCommitMessage); return false; } boolean authorValid = false; if (author.length() > 0) authorValid = RawParseUtils.parsePersonIdent(author) != null; if (!authorValid) { MessageDialog.openWarning(getShell(), UIText.CommitDialog_ErrorInvalidAuthor, UIText.CommitDialog_ErrorInvalidAuthorSpecified); return false; } boolean committerValid = false; if (committer.length() > 0) committerValid = RawParseUtils.parsePersonIdent(committer) != null; if (!committerValid) { MessageDialog.openWarning(getShell(), UIText.CommitDialog_ErrorInvalidAuthor, UIText.CommitDialog_ErrorInvalidCommitterSpecified); return false; } authorHandler.updateProposals(); committerHandler.updateProposals(); return true; }
Source:
<code>public void basicTest() throws InstantiationException { StoreKeyConverterFactory storeKeyConverterFactory = new StoreKeyConverterFactoryImpl(); <START> StoreKeyConverter storeKeyConverter = storeKeyConverterFactory.getStoreKeyConverter(); <END> assertTrue(storeKeyConverter instanceof StoreKeyConverterImplNoOp); }</code><technical_language>Instead of explicit assert, suffice? java StoreKeyConverterImplNoOp storeKeyConverter = (StoreKeyConverterImplNoOp) storeKeyConverterFactory.getStoreKeyConverter();</technical_language>

Target:
public void basicTest() throws InstantiationException { StoreKeyConverterFactory storeKeyConverterFactory = new StoreKeyConverterFactoryImpl(); StoreKeyConverter storeKeyConverter = storeKeyConverterFactory.getStoreKeyConverter(); assertTrue("StoreKeyConverter should have been StoreKeyConverterImplNoOp impl", storeKeyConverter instanceof StoreKeyConverterImplNoOp); }
Source:
<code>private void commitCreate(AsyncFileEntry entry, boolean success) { LOGGER.debug("commitCreate({},{}): Starting...", entry.getName(), success); if (success) { entry.commit(); <START> entry.getParent().ifPresent(e -> e.addChild(entry)); <END> } removeFromProcessors(entry); }</code><technical_language>need careful adding a node parent node of synchronization block, problems</technical_language>

Target:
private void commitCreate(AsyncFileEntry entry, boolean success) { LOGGER.debug("commitCreate({},{}): Starting...", entry.getName(), success); if (success) { entry.commit(); entry.getParent().ifPresent(e -> e.addChild(entry)); } onFinish(); }
Source:
<code>public List<List<WebElement>> getRows() { return getWrappedElement() <START> .findElements(By.cssSelector("tr")) <END> .stream() .map(rowElement -> rowElement.findElements(By.cssSelector("td"))) .filter(row -> row.size() > 0) .collect(toList()); }</code><technical_language>changing selector type produce unwanted side effects in some legacy browsers (which a lot for some enterprise web apps). please move change xpath css separate PR</technical_language>

Target:
public List<List<WebElement>> getRows() { return getWrappedElement() .findElements(By.xpath(". .stream() .map(rowElement -> rowElement.findElements(By.xpath(". .filter(row -> row.size() > 0) .collect(toList()); }
Source:
<code>public KafkaPoller(Consumer<String, ChangeEvent> consumer, Uris uris, Instant firstStartTime, int batchSize, Collection<String> topics, RdfRepository rdfRepo) { this.consumer = consumer; this.uris = uris; this.firstStartTime = firstStartTime; this.batchSize = batchSize; this.topics = topics; this.rdfRepo = rdfRepo; try { this.root = uris.builder().build(); } catch (URISyntaxException e) { throw new RuntimeException(e); } this.topicPartitions = <START> setUpTopicPartitions(); <END> }</code><technical_language>I a rename of method sense (topicsToPartitions()?). setup partitions anymore, transforms a list of topics a list of topic partitions. Also, extracting topics a parameter explicit method is operating. Minor enough, feel free ignore</technical_language>

Target:
public KafkaPoller(Consumer<String, ChangeEvent> consumer, Uris uris, Instant firstStartTime, int batchSize, Collection<String> topics, RdfRepository rdfRepo) { this.consumer = consumer; this.uris = uris; this.firstStartTime = firstStartTime; this.batchSize = batchSize; this.topics = topics; this.rdfRepo = rdfRepo; try { this.root = uris.builder().build(); } catch (URISyntaxException e) { throw new RuntimeException(e); } this.topicPartitions = topicsToPartitions(); }
Source:
<code>private <T> T executeInTimerContext(Supplier<T> supplier) { <START> try (Timer.Context timer = this.conjureTimelockServiceBlockingMetrics.startTransactions().time()) { <END> return supplier.get(); } }</code><technical_language>nit: this referring fields (outside of assigning in a constructor, if needed e.g. referencing a parent class of class)</technical_language>

Target:
private <T> T executeInTimerContext(Supplier<T> supplier) { try (Timer.Context timer = conjureTimelockServiceBlockingMetrics.startTransactions().time()) { return supplier.get(); } }
Source:
<code>Optional<CollectorRegistry> register(List<Gauge> gauges, Supplier<CollectorRegistry> collectorRegistryCreator) { Supplier<CollectorRegistry> collectorRegistrySupplier = () -> { try { CollectorRegistry collectorRegistry = collectorRegistryCreator.get(); gauges.forEach(collectorRegistry::register); return collectorRegistry; } catch (Exception e) { logger.error("Gauges registration failed."); <START> e.printStackTrace(); <END> return null; } }; return register(collectorRegistrySupplier); }</code><technical_language>Logging</technical_language>

Target:
Optional<CollectorRegistry> register(List<Gauge> gauges, Supplier<CollectorRegistry> collectorRegistryCreator) { Supplier<CollectorRegistry> collectorRegistrySupplier = () -> { try { CollectorRegistry collectorRegistry = collectorRegistryCreator.get(); gauges.forEach(collectorRegistry::register); return collectorRegistry; } catch (Exception e) { logger.error("Gauges registration failed.", e); return null; } }; return register(collectorRegistrySupplier); }
Source:
<code>public void testGetOutOfBandManagementOptionsInvalid() { try { oobmService.getOptions(null); Assert.fail("CloudRuntimeException was expected for finding options of host with out-of-band management configuration"); } catch (CloudRuntimeException e) { <START> } <END> }</code><technical_language>This try block fail construct is unnecessary. Changing @Test @Test(expected = CloudRuntimeException.class) accomplishes thing</technical_language>

Target:
public void testGetOutOfBandManagementOptionsInvalid() { oobmService.getOptions(null); Assert.fail("CloudRuntimeException was expected for finding options of host with out-of-band management configuration"); }
Source:
<code>public Set<OperatingSystem> getVirtualGuestOperatingSystems() { return Sets.newHashSet(FluentIterable.from(operatingSystems) .transform(new Function<ContainerVirtualGuestConfigurationOption, OperatingSystem>() { @Override public OperatingSystem apply(ContainerVirtualGuestConfigurationOption input) { <START> if (input.getTemplate().getOperatingSystemReferenceCode() != null) { <END> return OperatingSystem.builder() .id(input.getTemplate().getOperatingSystemReferenceCode()) .operatingSystemReferenceCode(input.getTemplate().getOperatingSystemReferenceCode()) .build(); } else { logger.warn("Skipping entry because of missing operatingSystemReferenceCode for %s", input); return null; } } }).filter(Predicates.notNull())); }</code><technical_language>I prefer fail-fastso == null, strong feelings</technical_language>

Target:
public Set<OperatingSystem> getVirtualGuestOperatingSystems() { return Sets.newHashSet(FluentIterable.from(operatingSystems) .transform(new Function<ContainerVirtualGuestConfigurationOption, OperatingSystem>() { @Override public OperatingSystem apply(ContainerVirtualGuestConfigurationOption input) { String operatingSystemReferenceCode = input.getTemplate().getOperatingSystemReferenceCode(); if (operatingSystemReferenceCode == null) { return null; } else { return OperatingSystem.builder() .id(operatingSystemReferenceCode) .operatingSystemReferenceCode(operatingSystemReferenceCode) .build(); } } }).filter(Predicates.notNull())); }
Source:
<code>private void fetchGitHubSearch(@NonNull final String searchString) { checkNotNull(searchString); Log.d(TAG, "fetchGitHubSearch(" + searchString + ")"); if (isOngoingRequest(searchString.hashCode())) { Log.d(TAG, "Found an ongoing request for repository " + searchString); return; } final String uri = getUniqueId(searchString); Subscription subscription = createNetworkObservable(searchString) .subscribeOn(Schedulers.computation()) <START> .map((repositories) -> { <END> final List<Integer> repositoryIds = new ArrayList<>(); for (GitHubRepository repository : repositories) { gitHubRepositoryStore.put(repository); repositoryIds.add(repository.getId()); } return new GitHubRepositorySearch(searchString, repositoryIds); }) .doOnSubscribe(() -> startRequest(uri)) .doOnCompleted(() -> completeRequest(uri)) .doOnError(doOnError(uri)) .subscribe(gitHubRepositorySearchStore::put, e -> Log.e(TAG, "Error fetching GitHub repository search for '" + searchString + "'", e)); addRequest(searchString.hashCode(), subscription); }</code><technical_language>in next line add repositories.size() default size</technical_language>

Target:
private void fetchGitHubSearch(@NonNull final String searchString) { checkNotNull(searchString); Log.d(TAG, "fetchGitHubSearch(" + searchString + ")"); if (isOngoingRequest(searchString.hashCode())) { Log.d(TAG, "Found an ongoing request for repository " + searchString); return; } final String uri = getUniqueId(searchString); Subscription subscription = createNetworkObservable(searchString) .subscribeOn(Schedulers.computation()) .map((repositories) -> { final List<Integer> repositoryIds = new ArrayList<>(repositories.size()); for (GitHubRepository repository : repositories) { gitHubRepositoryStore.put(repository); repositoryIds.add(repository.getId()); } return new GitHubRepositorySearch(searchString, repositoryIds); }) .doOnSubscribe(() -> startRequest(uri)) .doOnCompleted(() -> completeRequest(uri)) .doOnError(doOnError(uri)) .subscribe(gitHubRepositorySearchStore::put, e -> Log.e(TAG, "Error fetching GitHub repository search for '" + searchString + "'", e)); addRequest(searchString.hashCode(), subscription); }
Source:
<code>public Object execute() throws Exception { if (!force) { String answer = session .readLine( "Are you sure you want to remove all gazetteer entries inside of the offline solr gazetteer core?(y/n)", ' ') .toLowerCase(); if (!("y".equals(answer) || "yes".equals(answer))) { session.getConsole().println("Aborting."); return null; } } try { SolrClient solrClient = clientFactory.newClient(OfflineGazetteerPlugin.STANDALONE_GAZETTEER_CORE_NAME); <START> Failsafe.with(retryPolicy).get(() -> solrClient.deleteByQuery("*:*")); <END> } catch (Exception e) { LOGGER.info("Error while executing", e); session.getConsole().println("Error while submitting remove all, exiting."); throw e; } session.getConsole().println("Removeall submitted successfully."); return null; }</code><technical_language>alternative save trouble. <LINK_0></technical_language>

Target:
public Object execute() throws Exception { if (!force) { String answer = session .readLine( "Are you sure you want to remove all gazetteer entries inside of the solr gazetteer collection?(y/n)", ' ') .toLowerCase(); if (!("y".equals(answer) || "yes".equals(answer))) { session.getConsole().println("Aborting."); return null; } } SolrClient solrClient = clientFactory.newClient(COLLECTION_NAME); Boolean response = Failsafe.with( new RetryPolicy() .retryWhen(false) .withMaxDuration(5, TimeUnit.SECONDS) .withBackoff(25, 1_000, TimeUnit.MILLISECONDS)) .get(() -> solrClient.isAvailable()); if (response == null || !response) { LOGGER.error("Could not contact solr to remove all"); session.getConsole().println("Could not contact solr to remove all, exiting."); return null; } try { solrClient.deleteByQuery("*:*"); } catch (Exception e) { LOGGER.info("Error while executing", e); session.getConsole().println("Error while submitting remove all, exiting."); throw e; } session.getConsole().println("Removeall submitted successfully."); return null; }
Source:
<code>public void onNext(Object request) { exchange = endpoint.createExchange(); exchange.getIn().setBody(request); exchange.getIn().setHeaders(headers); <START> consumer.process(exchange, doneSync -> { <END> }); if (exchange.hasOut()) { responseObserver.onNext(exchange.getOut().getBody()); } else { responseObserver.onNext(exchange.getIn().getBody()); } responseObserver.onCompleted(); }</code><technical_language>I evaluate response exchange exited route, callback function is called. If call soon pushing message, exchange completed</technical_language>

Target:
public void onNext(Object request) { CountDownLatch latch = new CountDownLatch(1); Object responseBody = null; exchange = endpoint.createExchange(); exchange.getIn().setBody(request); exchange.getIn().setHeaders(headers); consumer.process(exchange, doneSync -> { latch.countDown(); }); try { latch.await(); if (exchange.hasOut()) { responseBody = exchange.getOut().getBody(); } else { responseBody = exchange.getIn().getBody(); } if (responseBody instanceof List) { List<?> responseList = (List<?>)responseBody; responseList.forEach((responseItem) -> { responseObserver.onNext(responseItem); }); } else { responseObserver.onNext(responseBody); } responseObserver.onCompleted(); } catch (InterruptedException e) { responseObserver.onError(e); } }
Source:
<code>public void testDclassWrapper() { <START> String userAgentSample = "Mozilla/5.0 (Linux; U; Android 2.2; en; HTC Aria A6380 Build/ERE27) AppleWebKit/540.13+ (KHTML, like Gecko) Version/3.1 Mobile Safari/524.15.0"; <END> dClass dw = new dClass(dtree_path); dw.init(); HashMap c = (HashMap) dw.classify(userAgentSample); assertEquals("Vendor as expected", "HTC" , c.get("vendor")); assertEquals("Model as expected" , "A6366", c.get("model")); assertEquals("device_os as expected","Android",c.get("device_os")); assertEquals("is_tablet as expected","false",c.get("is_tablet")); }</code><technical_language>Nit: Inconsistent spacing</technical_language>

Target:
public void testDclassWrapper() { String userAgentSample = "Mozilla/5.0 (Linux; U; Android 2.2; en; HTC Aria A6380 Build/ERE27) AppleWebKit/540.13+ (KHTML, like Gecko) Version/3.1 Mobile Safari/524.15.0"; dClass dw = new dClass(dtree_path); dw.init(); Map<String, String> c = dw.classify(userAgentSample); assertEquals("Vendor as expected", "HTC" , c.get("vendor")); assertEquals("Model as expected" , "A6366", c.get("model")); assertEquals("device_os as expected","Android",c.get("device_os")); assertEquals("is_tablet as expected","false",c.get("is_tablet")); }
Source:
<code>public void close() { shutdownExecutorService(); for (String executorKey : executors.keySet()) { DefaultExecutor executor = executors.remove(executorKey); if (executor != null) { <START> executor.getWatchdog().destroyProcess(); <END> } } }</code><technical_language>btw, do need try catch destroyProcess</technical_language>

Target:
public void close() { shutdownExecutorService(); for (String executorKey : executors.keySet()) { DefaultExecutor executor = executors.remove(executorKey); if (executor != null) { try { executor.getWatchdog().destroyProcess(); } catch (Exception e){ LOGGER.error("error destroying executor for paragraphId: " + executorKey, e); } } } }
Source:
<code>public static IntervalList concatenate(final Collection<IntervalList> lists) { return lists.stream() .reduce(IntervalList::concatenate) .orElseThrow(() -> new SAMException("Cannot concatenate an empty list of IntervalLists.")); <START> } <END></code><technical_language>This a step performance-wise; existing implementation a single mutable IntervalList accumulator, new creates populates a new IntervalList (and header clone) reduction (lists.size() times, step larger). This negatively impact of other operations too, indirectly overlaps, subtract, difference, union</technical_language>

Target:
public static IntervalList concatenate(final Collection<IntervalList> lists) { final SAMFileHeader header = lists.stream() .findFirst() .map(IntervalList::getHeader) .orElseThrow( () -> new IllegalArgumentException("Cannot combine empty collection of IntervalLists")); return lists.stream() .reduce(new IntervalList(header), IntervalList::addOther, IntervalList::concatenate); }
Source:
<code>private boolean slaveAttributesMatch(Offer offer) { boolean slaveTypeMatch = true; JSONObject cloudAttributes = getMesosCloud().getCloudAttributes(); if (cloudAttributes != null && cloudAttributes.size() > 0) { <START> Iterator<Attribute> iter = offer.getAttributesList().iterator(); <END> while (iter.hasNext()) { Attribute attribute = iter.next(); if (!(cloudAttributes.containsKey(attribute.getName()) && attribute.getText().getValue().equals(cloudAttributes.getString(attribute.getName())))) { slaveTypeMatch = false; break; } } } return slaveTypeMatch; }</code><technical_language>of iterator? about? for (Attribute attribute : offer.getAddtributesList()) { }</technical_language>

Target:
private boolean slaveAttributesMatch(Offer offer) { boolean slaveTypeMatch = true; JSONObject slaveAttributes = getMesosCloud().getSlaveAttributes(); Map<String, String> attributesMap = new HashMap<String, String>(); for (Attribute attribute : offer.getAttributesList()) { attributesMap.put(attribute.getName(), attribute.getText().getValue()); } if (slaveAttributes != null && slaveAttributes.size() > 0) { Iterator iterator = slaveAttributes.keys(); while (iterator.hasNext()) { String key = (String) iterator.next(); if (!(attributesMap.containsKey(key) && attributesMap.get(key).toString().equals(slaveAttributes.getString(key)))) { slaveTypeMatch = false; break; } } } return slaveTypeMatch; }
Source:
<code>public final void putChar(char value) { checkLimits(2); Unsafe.getUnsafe().putChar(appendAddress, value); <START> appendAddress += 2; <END> }</code><technical_language>constants Character.BYTES</technical_language>

Target:
public final void putChar(char value) { checkLimits(Character.BYTES); Unsafe.getUnsafe().putChar(appendAddress, value); appendAddress += Character.BYTES; }
Source:
<code>public boolean equals(Object obj) { if (obj == this) return true; if (obj == null) return false; StreamConfig config = (StreamConfig) obj; if (!(config instanceof StreamConfig)) { return false; } if (config.getStreamType() == null) { if (this.getStreamType() != null) return false; } else { if (!config.getStreamType().equals(this.getStreamType())) return false; } if (config.getLocation() == null) { if (this.getLocation() != null) return false; } else { if (!config.getLocation().equals(this.getLocation())) return false; } if (config.getURL() == null) { if (this.getURL() != null) return false; } else { if (!config.getURL().equals(this.getURL())) return false; } return true; <START> } <END></code><technical_language>equals hashcode implementations more concise. I recommend IDEs code generators create for you</technical_language>

Target:
public boolean equals(Object obj) { if (this == obj) return true; if (obj == null) return false; if (getClass() != obj.getClass()) return false; StreamConfig other = (StreamConfig) obj; if (location == null) { if (other.location != null) return false; } else if (!location.equals(other.location)) return false; if (streamType != other.streamType) return false; if (url == null) { if (other.url != null) return false; } else if (!url.equals(other.url)) return false; return true; }
Source:
<code>public void userEventTriggered(ChannelHandlerContext ctx, Object evt) throws Exception { if (evt instanceof IdleStateEvent) { MqttFixedHeader pingreqFixedHeader = new MqttFixedHeader(MqttMessageType.PINGREQ, false, MqttQoS.AT_MOST_ONCE, false, 0); MqttMessage pingreqMessage = new MqttMessage(pingreqFixedHeader); <START> ctx.channel().writeAndFlush(pingreqMessage); <END> System.out.println("Sent PINGREQ"); } else { super.userEventTriggered(ctx, evt); } }</code><technical_language>ctx.writeAndFlush(...)</technical_language>

Target:
public void userEventTriggered(ChannelHandlerContext ctx, Object evt) throws Exception { if (evt instanceof IdleStateEvent) { MqttFixedHeader pingreqFixedHeader = new MqttFixedHeader(MqttMessageType.PINGREQ, false, MqttQoS.AT_MOST_ONCE, false, 0); MqttMessage pingreqMessage = new MqttMessage(pingreqFixedHeader); ctx.writeAndFlush(pingreqMessage); System.out.println("Sent PINGREQ"); } else { super.userEventTriggered(ctx, evt); } }
Source:
<code>protected void deleteAsyncQuery() { String cleanupDateFormatted = evaluateFormattedFilterDate(Calendar.DATE, queryCleanupDays); String filterExpression = "createdOn=le='" + cleanupDateFormatted + "'"; Collection<AsyncQuery> asyncQueryList = asyncQueryDao.deleteAsyncQueryAndResultCollection(filterExpression); Iterator itr = asyncQueryList.iterator(); while (itr.hasNext()) { AsyncQuery query = (AsyncQuery) itr.next(); <START> resultStorageEngine.deleteResultsByID(query.getId()); <END> } }</code><technical_language>I rename function deleteResultsByID deleteResultsCollection. Collection<AsyncQuery> argument. of iterator logic moved function implementation. reason in database interpretation delete row for a filesystem-based, delete multiple time</technical_language>

Target:
protected void deleteAsyncQuery() { try { Date cleanupDate = DateUtil.calculateFilterDate(Calendar.DATE, queryCleanupDays); PathElement createdOnPathElement = new PathElement(AsyncQuery.class, Long.class, "createdOn"); FilterExpression fltDeleteExp = new LEPredicate(createdOnPathElement, cleanupDate); Collection<AsyncQuery> asyncQueryList = asyncQueryDao.deleteAsyncQueryAndResultCollection(fltDeleteExp); resultStorageEngine.deleteResultsCollection(asyncQueryList); } catch (Exception e) { log.error("Exception in scheduled cleanup: {}", e); } }
Source:
<code>private static ZoneInfo lookupZoneInfo(String timezoneId) { try { ZoneInfo zoneInfo = ZoneInfoDB.getInstance().makeTimeZone(timezoneId); if (zoneInfo == null) { zoneInfo = ZoneInfoDB.getInstance().makeTimeZone("GMT"); } if (zoneInfo == null) { <START> throw new IllegalStateException("GMT not found: \"" + timezoneId + "\""); <END> } return zoneInfo; } catch (IOException e) { throw new IllegalStateException( "Error loading timezone: \"" + timezoneId + "\"", e); } }</code><technical_language>AssertionError</technical_language>

Target:
private static ZoneInfo lookupZoneInfo(String timezoneId) { try { ZoneInfo zoneInfo = ZoneInfoDB.getInstance().makeTimeZone(timezoneId); if (zoneInfo == null) { zoneInfo = ZoneInfoDB.getInstance().makeTimeZone("GMT"); } if (zoneInfo == null) { throw new AssertionError("GMT not found: \"" + timezoneId + "\""); } return zoneInfo; } catch (IOException e) { throw new AssertionError("Error loading timezone: \"" + timezoneId + "\"", e); } }
Source:
<code>public void notify(EclipseContext context, EventType type, Object data) { if (!Policy.DEBUG_CONTEXTS) { return; } String msg = context.toString(); <START> if (msg.startsWith("Anonymous Context")) { <END> return; } msg = type + " : " + describe(context); WorkbenchSWTActivator.trace(Policy.DEBUG_CONTEXTS_FLAG, msg, null); }</code><technical_language>msg null? do need consider Anonymous Context is this some constant value</technical_language>

Target:
public void notify(EclipseContext context, EventType type, Object data) { if (!Policy.DEBUG_CONTEXTS) { return; } String msg = context.toString(); if (EclipseContext.ANONYMOUS_CONTEXT_NAME.equals(msg)) { return; } msg = type + " : " + describe(context); WorkbenchSWTActivator.trace(Policy.DEBUG_CONTEXTS_FLAG, msg, null); }
Source:
<code>public void setSelectedTestArtifactName(@NotNull String selectedTestArtifactName) { <START> ImmutableSet<String> knownArtifacts = <END> ImmutableSet.of(AndroidProject.ARTIFACT_ANDROID_TEST, AndroidProject.ARTIFACT_UNIT_TEST); checkArgument(knownArtifacts.contains(selectedTestArtifactName), "Unknown test artifact %s.", selectedTestArtifactName); mySelectedTestArtifactName = selectedTestArtifactName; }</code><technical_language>A set 2 elements is overkill. check if artifact name is of 2 constants</technical_language>

Target:
public void setSelectedTestArtifactName(@NotNull String selectedTestArtifactName) { checkArgument(selectedTestArtifactName == AndroidProject.ARTIFACT_ANDROID_TEST || selectedTestArtifactName == AndroidProject.ARTIFACT_UNIT_TEST); mySelectedTestArtifactName = selectedTestArtifactName; }
Source:
<code>public Person[] getRecipients(ZulipApp app) { Person[] recipientsCache; String[] ids = TextUtils.split(this.getDisplayRecipent(), ","); recipientsCache = new Person[ids.length]; for (int i = 0; i < ids.length; i++) { recipientsCache[i] = Person.getById(app, <START> Integer.parseInt(ids[i])); <END> } return recipientsCache; }</code><technical_language>Integer.parseIntthrows NumberFormatException. unlikely, a good idea catch it</technical_language>

Target:
public Person[] getRecipients(ZulipApp app) { Person[] recipientsCache; String[] ids = TextUtils.split(this.getDisplayRecipent(), ","); recipientsCache = new Person[ids.length]; for (int i = 0; i < ids.length; i++) { try { recipientsCache[i] = Person.getById(app, Integer.parseInt(ids[i])); } catch (NumberFormatException e) { ZLog.logException(e); } } return recipientsCache; }
Source:
<code>private boolean fetchUsersList(String dotComBlogId, final int localTableBlogId, final int offset) { if (mUsersEndOfListReached || mUsersFetchRequestInProgress || !NetworkUtils.checkConnection(this)) { return false; } mUsersFetchRequestInProgress = true; PeopleUtils.fetchUsers(dotComBlogId, localTableBlogId, offset, new PeopleUtils.FetchUsersCallback() { @Override public void onSuccess(List<Person> peopleList, boolean isEndOfList) { boolean isFreshList = (offset == 0); <START> mCanRefreshUsers = false; <END> mUsersEndOfListReached = isEndOfList; PeopleTable.saveUsers(peopleList, localTableBlogId, isFreshList); PeopleListFragment peopleListFragment = getListFragment(); if (peopleListFragment != null) { peopleListFragment.fetchingRequestFinished(PeopleListFilter.TEAM, isFreshList, true); } refreshOnScreenFragmentDetails(); mUsersFetchRequestInProgress = false; } @Override public void onError() { PeopleListFragment peopleListFragment = getListFragment(); if (peopleListFragment != null) { boolean isFirstPage = offset == 0; peopleListFragment.fetchingRequestFinished(PeopleListFilter.TEAM, isFirstPage, false); } mUsersFetchRequestInProgress = false; ToastUtils.showToast(PeopleManagementActivity.this, R.string.error_fetch_users_list, ToastUtils.Duration.SHORT); } }); return true; }</code><technical_language>If I understand correctly, this flag is a limit relevant data fetch happen once. do capabilities, roles permissions so, I recommend renaming mHasRefreshedUsers (and adjust semantics accordingly)? for rest of mCanRefresh* vars. Thanks</technical_language>

Target:
private boolean fetchUsersList(String dotComBlogId, final int localTableBlogId, final int offset) { if (mUsersEndOfListReached || mUsersFetchRequestInProgress || !NetworkUtils.checkConnection(this)) { return false; } mUsersFetchRequestInProgress = true; PeopleUtils.fetchUsers(dotComBlogId, localTableBlogId, offset, new PeopleUtils.FetchUsersCallback() { @Override public void onSuccess(List<Person> peopleList, boolean isEndOfList) { boolean isFreshList = (offset == 0); mHasRefreshedUsers = true; mUsersEndOfListReached = isEndOfList; PeopleTable.saveUsers(peopleList, localTableBlogId, isFreshList); PeopleListFragment peopleListFragment = getListFragment(); if (peopleListFragment != null) { peopleListFragment.fetchingRequestFinished(PeopleListFilter.TEAM, isFreshList, true); } refreshOnScreenFragmentDetails(); mUsersFetchRequestInProgress = false; } @Override public void onError() { PeopleListFragment peopleListFragment = getListFragment(); if (peopleListFragment != null) { boolean isFirstPage = offset == 0; peopleListFragment.fetchingRequestFinished(PeopleListFilter.TEAM, isFirstPage, false); } mUsersFetchRequestInProgress = false; ToastUtils.showToast(PeopleManagementActivity.this, R.string.error_fetch_users_list, ToastUtils.Duration.SHORT); } }); return true; }
Source:
<code>private String insanelyHugeCommitMessage() { StringBuilder b = new StringBuilder(HUGE_SIZE); b.append("An insanely huge commit message\n\n"); String oneLine = "012345678901234567890123456789012345678901234567890123456789\n"; while <START> (b.length() < <END> HUGE_SIZE) { b.append(oneLine); } return b.toString(); }</code><technical_language>This is calculating character count in iteration. calculate required size in advance</technical_language>

Target:
private String insanelyHugeCommitMessage() { final String oneLine = "012345678901234567890123456789012345678901234567890123456789\n"; StringBuilder b = new StringBuilder(HUGE_SIZE + oneLine.length()); b.append("An insanely huge commit message\n\n"); while (b.length() < HUGE_SIZE) { b.append(oneLine); } return b.toString(); }
Source:
<code>public boolean checkHaReservationStatusForCluster(VDSGroup cluster, List<VDS> failedHosts) { List<VDS> hosts = DbFacade.getInstance().getVdsDao().getAllForVdsGroup(cluster.getId()); <START> if (hosts == null) { <END> return true; } if (hosts.size() < 2) { return false; } List<Pair<Guid, Pair<Integer, Integer>>> hostsUnutilizedResources = getUnutilizedResources(hosts); List<VM> vmInCluster = DbFacade.getInstance().getVmDao().getAllForVdsGroup(cluster.getId()); if (vmInCluster == null) { return true; } vmInCluster = LinqUtils.filter(vmInCluster, new Predicate<VM>() { @Override public boolean eval(VM v) { return v.isAutoStartup(); } }); Map<Guid, List<VM>> hostToHaVmsMapping = mapVmToHost(vmInCluster); for (VDS host : hosts) { boolean isHaSafe = findReplacementForHost(cluster, host, hostToHaVmsMapping.get(host.getId()), hostsUnutilizedResources); if (!isHaSafe) { failedHosts.add(host); } } if (failedHosts.size() == 0) { return true; } else { return false; } }</code><technical_language>null empty</technical_language>

Target:
public boolean checkHaReservationStatusForCluster(VDSGroup cluster, List<VDS> failedHosts) { List<VDS> hosts = DbFacade.getInstance().getVdsDao().getAllForVdsGroup(cluster.getId()); if (hosts == null || hosts.isEmpty()) { return true; } if (hosts.size() < 2) { failedHosts.addAll(hosts); return false; } List<Pair<Guid, Pair<Integer, Integer>>> hostsUnutilizedResources = getUnutilizedResources(hosts); Map<Guid, List<VM>> hostToHaVmsMapping = mapHaVmToHostByCluster(cluster.getId()); for (VDS host : hosts) { if (hostToHaVmsMapping.get(host.getId()) != null) { boolean isHaSafe = findReplacementForHost(cluster, host, hostToHaVmsMapping.get(host.getId()), hostsUnutilizedResources); if (!isHaSafe) { failedHosts.add(host); } } } return failedHosts.isEmpty(); }
Source:
<code>public void addItemToNonListPropertyTest() throws IOException, OperationException { <START> String fieldsDataAsJSon = readPropertiesFromFile("newFields.json"); <END> OperationContext ctx = new OperationContext(coreSession); ctx.setInput(doc); OperationChain chain = new OperationChain("testChain"); chain.add(AddItemToListProperty.ID).set("xpath", "dc:title").set("ComplexJsonProperties", fieldsDataAsJSon); DocumentModel resultDoc = (DocumentModel) service.run(ctx, chain); }</code><technical_language>fieldsDataAsJ**s**on instead of fieldsDataAsJ**S**on</technical_language>

Target:
public void addItemToNonListPropertyTest() throws IOException, OperationException { String fieldsDataAsJson = readPropertiesFromFile("newFields.json"); OperationContext ctx = new OperationContext(coreSession); ctx.setInput(doc); OperationChain chain = new OperationChain("testChain"); chain.add(AddItemToListProperty.ID).set("xpath", "dc:title").set("complexJsonProperties", fieldsDataAsJson); DocumentModel resultDoc = (DocumentModel) service.run(ctx, chain); }
Source:
<code>public static void signout(Context context) { try { signOut(); <START> } catch (IllegalStateException e) { <END> initialize(context); signOut(); } }</code><technical_language>rewrite this method more sense so: java try { Twitter.getInstance(); // Check for initialization } catch (IllegalStateException e) { initialize(context); } signOut();</technical_language>

Target:
public static void signout(Context context) { try { Twitter.getInstance(); } catch (IllegalStateException e) { initialize(context); } signOut(); }
Source:
<code>public CompletionTime processOneToOne(String tenantId, Span item) throws RetryAttemptException { if (item.topLevelSpan()) { CompletionTime ct = new CompletionTime(); ct.setId(item.getId()); URL url = item.url(); if (url != null) { if (item.clientSpan()) { ct.setUri(Constants.URI_CLIENT_PREFIX + url.getPath()); } else { ct.setUri(url.getPath()); } ct.setEndpointType(url.getProtocol() == null ? null : url.getProtocol().toUpperCase()); } else { ct.setEndpointType("Unknown"); } ct.setDuration(TimeUnit.MILLISECONDS.convert(item.getDuration(), TimeUnit.NANOSECONDS)); ct.setTimestamp(item.getTimestamp()/1000); ct.setOperation(SpanDeriverUtil.deriveOperation(item)); ct.setFault(SpanDeriverUtil.deriveFault(item)); List<Property> spanProperties = item.properties(); ct.getProperties().addAll(spanProperties); ct.setHostAddress(Span.ipv4Address(spanProperties)); if (log.isLoggable(Level.FINEST)) { <START> log.finest("TraceCompletionTimeDeriver span=" + item + " ret=" + ct); <END> } return ct; } return null; }</code><technical_language>ret CompletionTime better</technical_language>

Target:
public CompletionTime processOneToOne(String tenantId, Span item) throws RetryAttemptException { if (item.topLevelSpan()) { CompletionTime ct = new CompletionTime(); ct.setId(item.getId()); URL url = item.url(); if (url != null) { if (item.clientSpan()) { ct.setUri(Constants.URI_CLIENT_PREFIX + url.getPath()); } else { ct.setUri(url.getPath()); } ct.setEndpointType(url.getProtocol() == null ? null : url.getProtocol().toUpperCase()); } else { ct.setEndpointType("Unknown"); } ct.setDuration(TimeUnit.MILLISECONDS.convert(item.getDuration(), TimeUnit.NANOSECONDS)); ct.setTimestamp(item.getTimestamp()/1000); ct.setOperation(SpanDeriverUtil.deriveOperation(item)); ct.setFault(SpanDeriverUtil.deriveFault(item)); List<Property> spanProperties = item.properties(); ct.getProperties().addAll(spanProperties); ct.setHostAddress(Span.ipv4Address(spanProperties)); if (log.isLoggable(Level.FINEST)) { log.finest("TraceCompletionTimeDeriver span=" + item + " completion time=" + ct); } return ct; } return null; }
Source:
<code>public void testHeaderRowCountDoesNotMatch() { <START> ConcurRequestExtractFile testFile = buildConcurRequestExtractFile(); <END> testFile.setRecordCount(new Integer(3)); if (concurRequestExtractFileValidationService.fileRowCountMatchesHeaderRowCount(testFile)) { assertTrue("Invalid: Header row count matched file row count. They should NOT have matched.", false); } else { assertTrue("Valid: Header row count did NOT match file row count.", true); } }</code><technical_language>I need separate tests here, for a test case amounts match for a test case amounts match</technical_language>

Target:
public void testHeaderRowCountDoesNotMatch() { List<ConcurRequestExtractFile> testFiles = ConcurRequestExtractFileFixture.BAD_FILE_COUNT_FILE.createConcurRequestExtractFiles(); assertFalse("Expected Result: Header row count should NOT match file row count.", concurRequestExtractFileValidationService.requestExtractHeaderRowValidatesToFileContents(testFiles)); }
Source:
<code>@Test public void implicitError() throws Exception { Controller c = new PowershellScript("$ErrorActionPreference = 'Stop'; Write-Error \"Bogus error\"").launch(new EnvVars(), ws, launcher, listener); while (c.exitStatus(ws, launcher, listener) == null) { Thread.sleep(100); } ByteArrayOutputStream baos = new ByteArrayOutputStream(); c.writeLog(ws, baos); assertTrue(c.exitStatus(ws, launcher).intValue() != 0); <START> assertTrue(baos.toString().contains("Bogus error")); <END> c.cleanup(ws); }</code><technical_language>reason for this change? previous assertion assertThat benefit of showing expected actual strings if assertion failed, new assertion "expected true, false", is a lot useful</technical_language>

Target:
@Test public void implicitError() throws Exception { Controller c = new PowershellScript("$ErrorActionPreference = 'Stop'; Write-Error \"Bogus error\"").launch(new EnvVars(), ws, launcher, listener); while (c.exitStatus(ws, launcher, listener) == null) { Thread.sleep(100); } ByteArrayOutputStream baos = new ByteArrayOutputStream(); c.writeLog(ws, baos); assertTrue(c.exitStatus(ws, launcher).intValue() != 0); assertThat(baos.toString(), containsString("Bogus error")); c.cleanup(ws); }
Source:
<code>private static void format(List<FileResource<?>> fileResourceList, Properties formatProfile) { for (FileResource<?> fileResource : fileResourceList) { if (fileResource.isDirectory()) { List<FileResource<?>> newFileResourceList = new ArrayList<>(); List<Resource<?>> resourceList = fileResource.listResources(); for (Resource<?> resource : resourceList) { newFileResourceList.add(resource.reify(FileResource.class)); } format(newFileResourceList, formatProfile); } <START> else if (fileResource.getName().endsWith(".java")) <END> { fileResource.setContents(Roaster.format(formatProfile, fileResource.getContents())); } } }</code><technical_language>check if fileResource instanceof JavaResource instead</technical_language>

Target:
private static void format(List<FileResource<?>> fileResourceList, Properties formatProfile) { for (FileResource<?> fileResource : fileResourceList) { if (fileResource.isDirectory()) { List<FileResource<?>> newFileResourceList = new ArrayList<>(); List<Resource<?>> resourceList = fileResource.listResources(); for (Resource<?> resource : resourceList) newFileResourceList.add(resource.reify(FileResource.class)); format(newFileResourceList, formatProfile); } else if (fileResource instanceof JavaResource) { if (formatProfile == null) fileResource.setContents(Roaster.format(fileResource.getContents())); else fileResource.setContents(Roaster.format(formatProfile, fileResource.getContents())); } } }
Source:
<code>public void createSavepoint() { String savepointName = "eee" + this.hashCode(); if (savepointName.length() > 10) savepointName = savepointName.substring(1, 9); savepoint = null; try { savepoint = statement.getConnection().setSavepoint(savepointName); <START> } catch (SQLException e) { <END> e.printStackTrace(); } }</code><technical_language>Similar exceptions actually. safe print ignore them</technical_language>

Target:
public void createSavepoint() { String savepointName = "eee" + this.hashCode(); if (savepointName.length() > 10) savepointName = savepointName.substring(1, 9); savepoint = null; try { savepoint = statement.getConnection().setSavepoint(savepointName); } catch (SQLException e) { throw new RuntimeException("Exception while setting savepoint", e); } }
Source:
<code>public void launch(List<ModuleLaunchRequest> moduleLaunchRequests, boolean aggregate, String parentArgs[]) { List<ModuleLaunchRequest> reversed = new ArrayList<>(moduleLaunchRequests); Collections.reverse(reversed); if (moduleLaunchRequests.size() == 1 || !aggregate) { <START> launchModulesIndividually(moduleLaunchRequests); <END> } else { aggregateAndLaunchModules(moduleLaunchRequests, parentArgs); } }</code><technical_language>nice if methods named consistently, e.g. launchIndividualModules launchAggregatedModules</technical_language>

Target:
public void launch(List<ModuleLaunchRequest> moduleLaunchRequests, boolean aggregate, String parentArgs[]) { List<ModuleLaunchRequest> reversed = new ArrayList<>(moduleLaunchRequests); Collections.reverse(reversed); if (moduleLaunchRequests.size() == 1 || !aggregate) { launchIndividualModules(moduleLaunchRequests); } else { launchAggregatedModules(moduleLaunchRequests, parentArgs); } }
Source:
<code>private int newOrderIndex(SAMRecord read, int oldIndex, Map<Integer, Integer> newOrder) { <START> if (oldIndex == NO_ALIGNMENT_REFERENCE_INDEX) <END> return NO_ALIGNMENT_REFERENCE_INDEX; else { final Integer n = newOrder.get(oldIndex); if (n == null) throw new PicardException("BUG: no mapping found for read " + read.getSAMString()); else return n; } }</code><technical_language>add { } if body</technical_language>

Target:
private int newOrderIndex(SAMRecord read, int oldIndex, Map<Integer, Integer> newOrder) { if (oldIndex == NO_ALIGNMENT_REFERENCE_INDEX) { return NO_ALIGNMENT_REFERENCE_INDEX; } else { final Integer n = newOrder.get(oldIndex); if (n == null) { throw new PicardException("BUG: no mapping found for read " + read.getSAMString()); } else { return n; } } }
Source:
<code>public APIMethodCaller<T> fetch(String id) { <START> return this.fetch().addPathComponent("id", id); <END> }</code><technical_language>Is this for suggestion fix? If so, a separate PR? Thanks :)</technical_language>

Target:
public APIMethodCaller<T> fetch(String id) { return this.fetch().addParam("id", id); }
Source:
<code>public void init() throws Exception { try { final DefaultConfigurationBuilder objectConfiguration = new DefaultConfigurationBuilder(); objectConfiguration.addProperty("[@destinationURL]", destination); objectRepository = new FilePersistentObjectRepository(); objectRepository.setFileSystem(fileSystem); objectRepository.configure(objectConfiguration); objectRepository.init(); LOGGER.debug("{} created in {}", getClass().getName(), destination); } catch (Exception e) { <START> LOGGER.error("Failed to initialize repository: {}", e.getMessage(), e); <END> throw e; } }</code><technical_language>get ride of getMessage</technical_language>

Target:
public void init() throws Exception { try { final DefaultConfigurationBuilder objectConfiguration = new DefaultConfigurationBuilder(); objectConfiguration.addProperty("[@destinationURL]", destination); objectRepository = new FilePersistentObjectRepository(); objectRepository.setFileSystem(fileSystem); objectRepository.configure(objectConfiguration); objectRepository.init(); LOGGER.debug("{} created in {}", getClass().getName(), destination); } catch (Exception e) { LOGGER.error("Failed to initialize repository", e); throw e; } }
Source:
<code>public void visitFunctionInvocation(FunctionInvocation functionInvocation) { String name = functionInvocation.getName().replaceAll("\\.", "#"); String typeDef = goloFunctionSignature(functionInvocation.getArity()); Handle handle = FUNCTION_INVOCATION_HANDLE; List<Object> bootstrapArgs = new ArrayList<>(); <START> bootstrapArgs.add(functionInvocation.isConstant() ? 1 : 0); <END> if (functionInvocation.isOnReference()) { ReferenceTable table = context.referenceTableStack.peek(); methodVisitor.visitVarInsn(ALOAD, table.get(functionInvocation.getName()).getIndex()); } if (functionInvocation.isOnModuleState()) { visitReferenceLookup(new ReferenceLookup(functionInvocation.getName())); } if (functionInvocation.isAnonymous() || functionInvocation.isOnReference() || functionInvocation.isOnModuleState()) { methodVisitor.visitTypeInsn(CHECKCAST, "java/lang/invoke/MethodHandle"); MethodType type = genericMethodType(functionInvocation.getArity() + 1).changeParameterType(0, MethodHandle.class); typeDef = type.toMethodDescriptorString(); handle = CLOSURE_INVOCATION_HANDLE; } List<String> argumentNames = visitInvocationArguments(functionInvocation); bootstrapArgs.addAll(argumentNames); methodVisitor.visitInvokeDynamicInsn(name, typeDef, handle, bootstrapArgs.toArray()); for (FunctionInvocation invocation : functionInvocation.getAnonymousFunctionInvocations()) { invocation.accept(this); } }</code><technical_language>Ok, answer for point :smile:</technical_language>

Target:
public void visitFunctionInvocation(FunctionInvocation functionInvocation) { String name = functionInvocation.getName().replaceAll("\\.", "#"); String typeDef = goloFunctionSignature(functionInvocation.getArity()); Handle handle = FUNCTION_INVOCATION_HANDLE; List<Object> bootstrapArgs = new ArrayList<>(); bootstrapArgs.add(functionInvocation.isConstant() ? 1 : 0); if (functionInvocation.isOnReference()) { ReferenceTable table = context.referenceTableStack.peek(); methodVisitor.visitVarInsn(ALOAD, table.get(functionInvocation.getName()).getIndex()); } if (functionInvocation.isOnModuleState()) { visitReferenceLookup(new ReferenceLookup(functionInvocation.getName())); } if (functionInvocation.isAnonymous() || functionInvocation.isOnReference() || functionInvocation.isOnModuleState()) { methodVisitor.visitTypeInsn(CHECKCAST, "gololang/FunctionReference"); MethodType type = genericMethodType(functionInvocation.getArity() + 1).changeParameterType(0, FunctionReference.class); typeDef = type.toMethodDescriptorString(); handle = CLOSURE_INVOCATION_HANDLE; } List<String> argumentNames = visitInvocationArguments(functionInvocation); bootstrapArgs.addAll(argumentNames); methodVisitor.visitInvokeDynamicInsn(name, typeDef, handle, bootstrapArgs.toArray()); for (FunctionInvocation invocation : functionInvocation.getAnonymousFunctionInvocations()) { invocation.accept(this); } }