auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>private List<WebElement> findSummaryElement(int notificationNumber, int groupIndex) { WebElement notifications = <START> getDriver().findElementsByCssSelector(NOTIFICATION_EVENT_SELECTOR).get(notificationNumber); <END> WebElement element = notifications.findElements(cssSelector(TEXT_SELECTOR)).get(groupIndex); return element.findElements(xpath(SUMMARY_SELECTOR)); }</code><technical_language>This good: page in notification area. cheaper prevent weird behaviour if testing in Dashboard page for example (you notif in dashboard in notification tray in case). Moreover, IMO a findElementsWithoutWaiting(By) avoid spending time waiting for elements expect loaded. allow a fail-fast</technical_language>

Target:
private List<WebElement> findSummaryElement(int notificationNumber, int groupIndex) { WebElement notifications = findNotifications().get(notificationNumber); WebElement element = notifications.findElements(cssSelector(TEXT_SELECTOR)).get(groupIndex); return element.findElements(xpath(SUMMARY_SELECTOR)); }
Source:
<code>protected void addDragAndDropSupport(final NatTable nattable) { final int operations = DND.DROP_MOVE | DND.DROP_COPY | DND.DROP_DEFAULT; final DropTarget target = new DropTarget(nattable, operations); final LocalTransfer localTransfer = LocalTransfer.getInstance(); final LocalSelectionTransfer localSelectionTransfer = LocalSelectionTransfer.getTransfer(); final Transfer[] types = new Transfer[] { localSelectionTransfer, localTransfer }; target.setTransfer(types); final NatTableDropListener dropListener = createDropListener(); target.addDropListener(dropListener); BooleanValueStyle values = <START> (BooleanValueStyle) table.getTableConfiguration().getNamedStyle(NattablestylePackage.eINSTANCE.getBooleanValueStyle(), <END> NamedStyleConstants.DRAG_ENABLED); if (null != values && values.isBooleanValue()) { natTable.addDragSupport(operations, types, new NatTableDragSourceListener(this, natTable, Collections.singletonList(GridRegion.ROW_HEADER))); } }</code><technical_language>check this namedstyle valu ein table before, if in tableconfiguration</technical_language>

Target:
protected void addDragAndDropSupport(final NatTable nattable) { final int operations = DND.DROP_MOVE | DND.DROP_COPY | DND.DROP_DEFAULT; final DropTarget target = new DropTarget(nattable, operations); final LocalTransfer localTransfer = LocalTransfer.getInstance(); final LocalSelectionTransfer localSelectionTransfer = LocalSelectionTransfer.getTransfer(); final Transfer[] types = new Transfer[] { localSelectionTransfer, localTransfer }; target.setTransfer(types); final NatTableDropListener dropListener = createDropListener(); target.addDropListener(dropListener); BooleanValueStyle values = (BooleanValueStyle) table.getNamedStyle(NattablestylePackage.eINSTANCE.getBooleanValueStyle(), NamedStyleConstants.DRAG_ENABLED); if (null == values) { values = (BooleanValueStyle) table.getTableConfiguration().getNamedStyle(NattablestylePackage.eINSTANCE.getBooleanValueStyle(), NamedStyleConstants.DRAG_ENABLED); } if (null != values && values.isBooleanValue()) { natTable.addDragSupport(operations, types, new NatTableDragSourceListener(this, natTable, Collections.singletonList(GridRegion.ROW_HEADER))); } }
Source:
<code>public boolean equals(Object other) { if (!(other instanceof GuestId)) { return false; } GuestId that = (GuestId) other; if (this.getGuestId().equalsIgnoreCase(that.getGuestId())) { <START> EqualsBuilder builder = new EqualsBuilder() <END> .append(this.getAttributes(), that.getAttributes()); return builder.isEquals(); } return false; }</code><technical_language>This is a map of strings, EqualsBuilder is calling .equals replaced code doing. gaining this change</technical_language>

Target:
public boolean equals(Object other) { if (!(other instanceof GuestId)) { return false; } GuestId that = (GuestId) other; if (this.getGuestId().equalsIgnoreCase(that.getGuestId()) && this.getAttributes().equals(that.getAttributes())) { return true; } return false; }
Source:
<code><START> protected static String sanitize(String namespace) <END> { Pattern DOT_OR_WHITESPACE = Pattern.compile("[\\s]+|[.]+"); return DOT_OR_WHITESPACE.matcher(namespace).replaceAll("_"); }</code><technical_language>suggestion private static String sanitize(String namespace)</technical_language>

Target:
private static String sanitize(String namespace) { return DOT_OR_WHITESPACE.matcher(namespace).replaceAll("_"); }
Source:
<code>public void testErrorChannel() throws Exception { final DirectChannel requestChannel = new DirectChannel(); requestChannel.subscribe(new AbstractReplyProducingMessageHandler() { @Override protected Object handleRequestMessage(Message<?> requestMessage) { throw new RuntimeException(); } }); final DirectChannel errorChannel = new DirectChannel(); errorChannel.subscribe(new AbstractReplyProducingMessageHandler() { @Override protected Object handleRequestMessage(Message<?> requestMessage) { return new Target("failed"); } }); final QueueChannel replyChannel = new QueueChannel(); final ContentEnricher enricher = new ContentEnricher(); enricher.setRequestChannel(requestChannel); enricher.setErrorChannel(errorChannel); SpelExpressionParser parser = new SpelExpressionParser(); Map<String, Expression> propertyExpressions = new HashMap<String, Expression>(); propertyExpressions.put("name", parser.parseExpression("payload.name + ' target'")); enricher.setPropertyExpressions(propertyExpressions); enricher.setBeanFactory(mock(BeanFactory.class)); enricher.afterPropertiesSet(); final Target target = new Target("replace me"); Message<?> requestMessage = MessageBuilder.withPayload(target).setReplyChannel(replyChannel).build(); enricher.handleMessage(requestMessage); <START> Message<?> reply = replyChannel.receive(0); <END> Target result = (Target) reply.getPayload(); assertEquals("failed target", result.getName()); }</code><technical_language>This fixed well. timing issue, 10000 is enough</technical_language>

Target:
public void testErrorChannel() throws Exception { final DirectChannel requestChannel = new DirectChannel(); requestChannel.subscribe(new AbstractReplyProducingMessageHandler() { @Override protected Object handleRequestMessage(Message<?> requestMessage) { throw new RuntimeException(); } }); final DirectChannel errorChannel = new DirectChannel(); errorChannel.subscribe(new AbstractReplyProducingMessageHandler() { @Override protected Object handleRequestMessage(Message<?> requestMessage) { return new Target("failed"); } }); final QueueChannel replyChannel = new QueueChannel(); final ContentEnricher enricher = new ContentEnricher(); enricher.setRequestChannel(requestChannel); enricher.setErrorChannel(errorChannel); SpelExpressionParser parser = new SpelExpressionParser(); Map<String, Expression> propertyExpressions = new HashMap<String, Expression>(); propertyExpressions.put("name", parser.parseExpression("payload.name + ' target'")); enricher.setPropertyExpressions(propertyExpressions); enricher.setBeanFactory(mock(BeanFactory.class)); enricher.afterPropertiesSet(); final Target target = new Target("replace me"); Message<?> requestMessage = MessageBuilder.withPayload(target).setReplyChannel(replyChannel).build(); enricher.handleMessage(requestMessage); Message<?> reply = replyChannel.receive(10000); Target result = (Target) reply.getPayload(); assertEquals("failed target", result.getName()); }
Source:
<code><START> private <S extends Message, A extends AggregatePart<I, S, ?>> AggregatePartRepository<I, A> getRepository(Class<S> stateClass) { <END> @SuppressWarnings("unchecked") final AggregatePartRepository<I, A> cached = (AggregatePartRepository<I, A>) partsAccess.get(stateClass); if (cached != null) { return cached; } final AggregatePartRepository<I, A> repo = lookup(stateClass); partsAccess.put(stateClass, repo); return repo; }</code><technical_language>line long. Is split two? Perhaps, a carriage return <S extends Message, do</technical_language>

Target:
AggregatePartRepository<I, A> getRepository(Class<S> stateClass) { @SuppressWarnings("unchecked") final AggregatePartRepository<I, A> cached = (AggregatePartRepository<I, A>) partsAccess.get(stateClass); if (cached != null) { return cached; } final AggregatePartRepository<I, A> repo = lookup(stateClass); partsAccess.put(stateClass, repo); return repo; }
Source:
<code>public void createCheckRun() throws Exception { GHCheckRun checkRun = gitHub.getRepository("jglick/github-api-test") .createCheckRun("foo", "4a929d464a2fae7ee899ce603250f7dab304bc4b") .withStatus(GHCheckRun.Status.COMPLETED) .withConclusion(GHCheckRun.Conclusion.SUCCESS) .withDetailsURL("<LINK_0>") .withExternalID("whatever") .withStartedAt(new Date(999_999_000)) .withCompletedAt(new Date(999_999_999)) .withOutput("Some Title", "what happened…") .withAnnotation("stuff.txt", 1, GHCheckRun.AnnotationLevel.NOTICE, "hello to you too") .withTitle("Look here") .done() <START> .withImage("Unikitty", "<LINK_1>") <END> .withCaption("Princess Unikitty") .done() .done() .withAction("Help", "what I need help with", "doit") .create(); assertEquals("completed", checkRun.getStatus()); assertEquals(1, checkRun.getOutput().getAnnotationsCount()); assertEquals(535049329, checkRun.id); }</code><technical_language>Image optional field need a builder? images readable api? I them..</technical_language>

Target:
public void createCheckRun() throws Exception { GHCheckRun checkRun = gitHub.getRepository("jglick/github-api-test") .createCheckRun("foo", "4a929d464a2fae7ee899ce603250f7dab304bc4b") .withStatus(GHCheckRun.Status.COMPLETED) .withConclusion(GHCheckRun.Conclusion.SUCCESS) .withDetailsURL("<LINK_0>") .withExternalID("whatever") .withStartedAt(new Date(999_999_000)) .withCompletedAt(new Date(999_999_999)) .add(new GHCheckRunBuilder.Output("Some Title", "what happened…") .add(new GHCheckRunBuilder.Annotation("stuff.txt", 1, GHCheckRun.AnnotationLevel.NOTICE, "hello to you too").withTitle("Look here")) .add(new GHCheckRunBuilder.Image("Unikitty", "<LINK_1>") .withCaption("Princess Unikitty"))) .add(new GHCheckRunBuilder.Action("Help", "what I need help with", "doit")) .create(); assertEquals("completed", checkRun.getStatus()); assertEquals(1, checkRun.getOutput().getAnnotationsCount()); assertEquals(546384586, checkRun.id); }
Source:
<code>public Mono<Response<Secret>> restoreSecretWithResponse(byte[] backup) { <START> return withContext(context -> restoreSecret(backup, context)); <END> }</code><technical_language>private methods named withResponse suffix, if is return</technical_language>

Target:
public Mono<Response<Secret>> restoreSecretWithResponse(byte[] backup) { return withContext(context -> restoreSecretWithResponse(backup, context)); }
Source:
<code>public void verifyBmpProcessing() { final InetSocketAddress bgpTelemetryAddress = stack.opennms().getNetworkProtocolAddress(NetworkProtocol.BMP); final String node = "<node type=\"A\" label=\"TestNode\" foreignSource=\"SmokeTests\" foreignId=\"TestNode\">" + "<labelSource>H</labelSource>" + "<sysContact>Me</sysContact>" + <START> "<sysDescription>PDP-8</sysDescription>" + <END> "<sysLocation>German DevJam 2020</sysLocation>" + "<sysName>TestNode</sysName>" + "<sysObjectId>.1.3.6.1.4.1.8072.3.2.255</sysObjectId>" + "<createTime>2020-02-25T13:20:00.123-04:00</createTime>" + "<lastCapsdPoll>2020-02-25T13:21:00.456-04:00</lastCapsdPoll>" + "</node>"; given().body(node) .basePath("/opennms/rest/nodes") .contentType(ContentType.XML).post() .then().assertThat() .statusCode(201); final String ipInterface = "<ipInterface isManaged=\"M\" snmpPrimary=\"P\">" + "<ipAddress>" + getLocalAddress() + "</ipAddress>" + "<hostName>test-machine1.local</hostName>" + "</ipInterface>"; given().body(ipInterface) .basePath("/opennms/rest/nodes/SmokeTests:TestNode/ipinterfaces") .contentType(ContentType.XML).post() .then().assertThat() .statusCode(201); await().atMost(1, MINUTES).pollDelay(0, SECONDS).pollInterval(5, SECONDS) .until(() -> { sendBmpTelemetryMessage(bgpTelemetryAddress); return matchRrdFileFromNodeResource("SmokeTests:TestNode"); }); }</code><technical_language>suggestion "<sysDescription>Black Ops 4</sysDescription>" +</technical_language>

Target:
public void verifyBmpProcessing() { final InetSocketAddress bgpTelemetryAddress = stack.opennms().getNetworkProtocolAddress(NetworkProtocol.BMP); final String node = "<node type=\"A\" label=\"TestNode\" foreignSource=\"SmokeTests\" foreignId=\"TestNode\">" + "<labelSource>H</labelSource>" + "<sysContact>Me</sysContact>" + "<sysDescription>Black Ops 4</sysDescription>" + "<sysLocation>German DevJam 2020</sysLocation>" + "<sysName>TestNode</sysName>" + "<sysObjectId>.1.3.6.1.4.1.8072.3.2.255</sysObjectId>" + "<createTime>2020-02-25T13:20:00.123-04:00</createTime>" + "<lastCapsdPoll>2020-02-25T13:21:00.456-04:00</lastCapsdPoll>" + "</node>"; given().body(node) .basePath("/opennms/rest/nodes") .contentType(ContentType.XML).post() .then().assertThat() .statusCode(201); final String ipInterface = "<ipInterface isManaged=\"M\" snmpPrimary=\"P\">" + "<ipAddress>" + getLocalAddress() + "</ipAddress>" + "<hostName>test-machine1.local</hostName>" + "</ipInterface>"; given().body(ipInterface) .basePath("/opennms/rest/nodes/SmokeTests:TestNode/ipinterfaces") .contentType(ContentType.XML).post() .then().assertThat() .statusCode(201); await().atMost(1, MINUTES).pollDelay(0, SECONDS).pollInterval(5, SECONDS) .until(() -> { sendBmpTelemetryMessage(bgpTelemetryAddress); return matchRrdFileFromNodeResource("SmokeTests:TestNode"); }); }
Source:
<code>private boolean vmUsesScsiReservation() { <START> List<VmDevice> vmManagedDevices = DbFacade.getInstance().getVmDeviceDao().getVmDeviceByVmId(getVmId()); <END> if (vmManagedDevices != null && !vmManagedDevices.isEmpty()) { for (VmDevice device : vmManagedDevices) { if (device.isScsiReservationUsed()) { return true; } } } return false; }</code><technical_language>I this list of devices managed vm, list of devices vm? "devices" good enough in context of this method? Do get get null getVmDeviceByVmId() devices</technical_language>

Target:
private boolean vmUsesScsiReservation() { List<VmDevice> devices = DbFacade.getInstance().getVmDeviceDao().getVmDeviceByVmId(getVmId()); if (devices != null) { for (VmDevice device : devices) { if (device.isScsiReservationUsed()) { return true; } } } return false; }
Source:
<code>private void setupBackPressureCheckTimer(final Map<String, Object> topologyConf) { Set<Integer> nonLocalTasks = Sets.difference(workerState.getTaskToComponent().keySet(), new HashSet<>(workerState.getLocalTaskIds())); if (nonLocalTasks.isEmpty()) { LOG.info("BackPressure change checking is disabled as there is only one worker"); return; <START> } <END> final Long bpCheckIntervalMs = ObjectReader.getLong(topologyConf.get(Config.TOPOLOGY_BACKPRESSURE_CHECK_MILLIS)); workerState.backPressureCheckTimer.scheduleRecurringMs(bpCheckIntervalMs, bpCheckIntervalMs, () -> workerState.refreshBackPressureStatus()); LOG.info("BackPressure status change checking will be performed every millis", bpCheckIntervalMs); }</code><technical_language>remove unused topologyConf</technical_language>

Target:
private void setupBackPressureCheckTimer(final Map<String, Object> topologyConf) { if (workerState.isSingleWorker()) { LOG.info("BackPressure change checking is disabled as there is only one worker"); return; } final Long bpCheckIntervalMs = ObjectReader.getLong(topologyConf.get(Config.TOPOLOGY_BACKPRESSURE_CHECK_MILLIS)); workerState.backPressureCheckTimer.scheduleRecurringMs(bpCheckIntervalMs, bpCheckIntervalMs, () -> workerState.refreshBackPressureStatus()); LOG.info("BackPressure status change checking will be performed every millis", bpCheckIntervalMs); }
Source:
<code>private void performRename(String newName) { RenameSettings settings = dtoFactory.createDto(RenameSettings.class); RenameParams renameParams = dtoFactory.createDto(RenameParams.class); renameParams.setNewName(newName); VirtualFile file = textEditor.getEditorInput().getFile(); TextDocumentIdentifier textDocumentIdentifier = dtoBuildHelper.createTDI(file); renameParams.setTextDocument(textDocumentIdentifier); org.eclipse.lsp4j.Position position = dtoFactory.createDto(org.eclipse.lsp4j.Position.class); position.setCharacter(cursorPosition.getCharacter()); position.setLine(cursorPosition.getLine()); renameParams.setPosition(position); settings.setUpdateReferences(true); settings.setRenameParams(renameParams); settings.setRenameKind(RenameKind.JAVA_ELEMENT); extensionServiceClient .rename(settings) .then( result -> { enableAutoSave(); undoChanges(); RefactoringStatus refactoringStatus = result.getRefactoringStatus(); <START> if (!FATAL.equals(refactoringStatus.getRefactoringSeverity())) { <END> applyWorkspaceEditAction.applyWorkspaceEdit(result.getCheWorkspaceEdit()); } clientServerEventService.sendFileTrackingResumeEvent(); sendOpenEvent(); }) .catchError( error -> { undoChanges(); enableAutoSave(); clientServerEventService.sendFileTrackingResumeEvent(); notificationManager.notify( locale.failedToRename(), error.getMessage(), FAIL, FLOAT_MODE); }); }</code><technical_language>do inform user wrong in this case</technical_language>

Target:
private void performRename(String newName) { RenameSettings settings = dtoFactory.createDto(RenameSettings.class); RenameParams renameParams = dtoFactory.createDto(RenameParams.class); renameParams.setNewName(newName); VirtualFile file = textEditor.getEditorInput().getFile(); TextDocumentIdentifier textDocumentIdentifier = dtoBuildHelper.createTDI(file); renameParams.setTextDocument(textDocumentIdentifier); org.eclipse.lsp4j.Position position = dtoFactory.createDto(org.eclipse.lsp4j.Position.class); position.setCharacter(cursorPosition.getCharacter()); position.setLine(cursorPosition.getLine()); renameParams.setPosition(position); settings.setUpdateReferences(true); settings.setRenameParams(renameParams); settings.setRenameKind(RenameKind.JAVA_ELEMENT); extensionServiceClient .rename(settings) .then( result -> { enableAutoSave(); undoChanges(); RefactoringStatus refactoringStatus = result.getRefactoringStatus(); if (!FATAL.equals(refactoringStatus.getRefactoringSeverity())) { applyWorkspaceEditAction.applyWorkspaceEdit(result.getCheWorkspaceEdit()); clientServerEventService.sendFileTrackingResumeEvent(); sendOpenEvent(); } else { notificationManager.notify( locale.failedToRename(), getErrorMessage(refactoringStatus.getRefactoringStatusEntries()), FAIL, FLOAT_MODE); } }) .catchError( error -> { undoChanges(); enableAutoSave(); clientServerEventService.sendFileTrackingResumeEvent(); notificationManager.notify( locale.failedToRename(), error.getMessage(), FAIL, FLOAT_MODE); }); }
Source:
<code>public Integer getNumChanges() { if (numChanges == null) { Integer numCount = null; for (final Leg leg : legs){ <START> if (!(leg instanceof Individual)) { <END> if (numCount == null) { numCount = 0; } else { numCount++; } } } return numCount; } else { return numChanges; } }</code><technical_language>explicitly test for leg instanceof Public here</technical_language>

Target:
public Integer getNumChanges() { if (numChanges == null) { Integer numCount = null; for (final Leg leg : legs){ if (leg instanceof Public) { if (numCount == null) { numCount = 0; } else { numCount++; } } } return numCount; } else { return numChanges; } }
Source:
<code>public static <T> T open(DBI dbi, Class<T> sqlObjectType) { try { check(OPEN); return (T) OPEN.invoke(dbi, sqlObjectType); } catch (Throwable t) { <START> throw new RuntimeException("Error during `SqlObjectBuilder.open` invocation", t); <END> } }</code><technical_language>this wrap e.g. UnableToOpenConnectionException other DbiException subclasses</technical_language>

Target:
public static <T> T open(DBI dbi, Class<T> sqlObjectType) { try { check(OPEN); return (T) OPEN.invoke(dbi, sqlObjectType); } catch (RuntimeException e) { throw e; } catch (Throwable t) { throw new RuntimeException("Error during `SqlObjectBuilder.open` invocation", t); } }
Source:
<code>public Optional<Timeline> getTimeline(final UUID id) { final List<Timeline> timelines = jdbcTemplate.query( <START> BASE_TIMELINE_QUERY + " WHERE t.t_id=?", <END> new Object[]{id}, timelineRowMapper); return Optional.ofNullable(timelines.isEmpty() ? null : timelines.get(0)); }</code><technical_language>add LIMIT 1? necessary primary key</technical_language>

Target:
public Optional<Timeline> getTimeline(final UUID id) { final List<Timeline> timelines = jdbcTemplate.query( BASE_TIMELINE_QUERY + " WHERE t.tl_id=?", timelineRowMapper, id); return Optional.ofNullable(timelines.isEmpty() ? null : timelines.get(0)); }
Source:
<code>void insert(GroupInfo info) { Comparator<GroupInfo> c = new Comparator<GroupInfo>() { @Override public int compare(GroupInfo a, GroupInfo b) { int cmp = <START> Strings.nullToEmpty(a.name()).compareTo( <END> Strings.nullToEmpty(b.name())); if (cmp != 0) { return cmp; } return a.getGroupUUID().compareTo(b.getGroupUUID()); } }; int insertPosition = table.getRowCount(); int left = 1; int right = table.getRowCount() - 1; while (left <= right) { int middle = (left + right) >>> 1; GroupInfo i = getRowItem(middle); int cmp = c.compare(i, info); if (cmp < 0) { left = middle + 1; } else if (cmp > 0) { right = middle - 1; } else { return; } } insertPosition = left; table.insertRow(insertPosition); applyDataRowStyle(insertPosition); populate(insertPosition, info); }</code><technical_language>This build fail. com.google.common.base.Strings client-side :-( extra efforts client side</technical_language>

Target:
void insert(GroupInfo info) { Comparator<GroupInfo> c = new Comparator<GroupInfo>() { @Override public int compare(GroupInfo a, GroupInfo b) { int cmp = nullToEmpty(a.name()).compareTo(nullToEmpty(b.name())); if (cmp != 0) { return cmp; } return a.getGroupUUID().compareTo(b.getGroupUUID()); } private String nullToEmpty(@Nullable String str) { return (str == null) ? "" : str; } }; int insertPosition = table.getRowCount(); int left = 1; int right = table.getRowCount() - 1; while (left <= right) { int middle = (left + right) >>> 1; GroupInfo i = getRowItem(middle); int cmp = c.compare(i, info); if (cmp < 0) { left = middle + 1; } else if (cmp > 0) { right = middle - 1; } else { return; } } insertPosition = left; table.insertRow(insertPosition); applyDataRowStyle(insertPosition); populate(insertPosition, info); }
Source:
<code>public void logRetry(Logger log, String message) { <START> long now = System.currentTimeMillis(); <END> if (lastRetryLog < 0) { log.debug(getMessage(message)); lastRetryLog = now; } else if ((now - lastRetryLog) > logInterval) { log.warn(getMessage(message)); lastRetryLog = now; } }</code><technical_language>I strongly recommend System.nanoTime() for elapsed time. System.currentTimeMillis() is for clock time, result in a surprising negative difference values</technical_language>

Target:
public void logRetry(Logger log, String message) { long now = System.nanoTime(); if (lastRetryLog < 0) { if (log.isDebugEnabled()) { log.debug(getMessage(message)); } lastRetryLog = now; } else if ((now - lastRetryLog) > logIntervalNanoSec) { log.warn(getMessage(message)); lastRetryLog = now; } else { if (log.isTraceEnabled()) { log.trace(getMessage(message)); } } }
Source:
<code>public static void eToast(Context context, int resId) { <START> Log.e(Thread.currentThread().getStackTrace()[methodDepth].toString(), context.getString(resId)); <END> toastWorker(context, context.getString(resId)); }</code><technical_language>This call eToast method takes a string, pass in context.getString(resId) string. iToast</technical_language>

Target:
public static void eToast(Context context, int resId) { eToast(context, context.getString(resId)); }
Source:
<code>public void testEmptyFeatures() { <START> Map<String, Map<String, Long>> emptyMap = new HashMap<>(); <END> Features<VersionLevelRange> emptyFinalizedFeatures = Features.emptyFinalizedFeatures(); assertEquals(new HashMap<>(), emptyFinalizedFeatures.all()); assertEquals(emptyMap, emptyFinalizedFeatures.serialize()); assertEquals(emptyFinalizedFeatures, Features.deserializeFinalizedFeatures(emptyMap)); Features<VersionRange> emptySupportedFeatures = Features.emptySupportedFeatures(); assertEquals(new HashMap<>(), emptySupportedFeatures.all()); assertEquals( new HashMap<String, HashMap<String, Long>>(), emptySupportedFeatures.serialize()); assertEquals(emptySupportedFeatures, Features.deserializeSupportedFeatures(emptyMap)); }</code><technical_language>need apache license title</technical_language>

Target:
public void testEmptyFeatures() { Map<String, Map<String, Short>> emptyMap = new HashMap<>(); Features<FinalizedVersionRange> emptyFinalizedFeatures = Features.emptyFinalizedFeatures(); assertTrue(emptyFinalizedFeatures.features().isEmpty()); assertTrue(emptyFinalizedFeatures.toMap().isEmpty()); assertEquals(emptyFinalizedFeatures, Features.fromFinalizedFeaturesMap(emptyMap)); Features<SupportedVersionRange> emptySupportedFeatures = Features.emptySupportedFeatures(); assertTrue(emptySupportedFeatures.features().isEmpty()); assertTrue(emptySupportedFeatures.toMap().isEmpty()); assertEquals(emptySupportedFeatures, Features.fromSupportedFeaturesMap(emptyMap)); }
Source:
<code>private void setLabelScores(LabelType type, LabelInfo label, short score, Account.Id accountId) throws OrmException { <START> if (label.approved != null || label.rejected != null) { return; } <END> if (type.getMin() == null || type.getMax() == null) { return; } if (score != 0) { if (score == type.getMin().getValue() && type.isBlock()) { label.rejected = accountLoader.get(accountId); } else if (score == type.getMax().getValue()) { label.approved = accountLoader.get(accountId); } else if (score < 0) { label.disliked = accountLoader.get(accountId); label.value = score; } else if (score > 0 && label.disliked == null) { label.recommended = accountLoader.get(accountId); label.value = score; } } return; }</code><technical_language>This condition is part is incorrect. If submit rules run (which initLabels() for open change) set approved/rejected fields of LabelInfo. If is set this method setting label.rejected, label.approved</technical_language>

Target:
private void setLabelScores(LabelType type, LabelInfo label, short score, Account.Id accountId) throws OrmException { if (label.approved != null || label.rejected != null) { return; } if (type.getMin() == null || type.getMax() == null) { return; } if (score < 0) { label.disliked = accountLoader.get(accountId); label.value = score; } else if (score > 0 && label.disliked == null) { label.recommended = accountLoader.get(accountId); label.value = score; } }
Source:
<code>private void initializeSnapshotsLeasesParams() { if (isRestoreLease()) { Guid activeSnapshotLeaseDomainId = getVm().getStaticData().getLeaseStorageDomainId(); Guid selectedLeaseDomainId = getParameters().getDstLeaseDomainId(); if (selectedLeaseDomainId == null && getDstSnapshot().getVmConfiguration() != null) { Guid dstLeaseDomainId = OvfUtils.fetchLeaseDomainId(getDstSnapshot().getVmConfiguration()); if (dstLeaseDomainId != null && activeSnapshotLeaseDomainId != null && !dstLeaseDomainId.equals( activeSnapshotLeaseDomainId)) { getParameters().setDstLeaseDomainId(activeSnapshotLeaseDomainId); getParameters().setLeaseAction(LeaseAction.UPDATE_LEASE_INFO_AND_LEASE_DOMAIN_ID); <START> return; <END> } else { getParameters().setDstLeaseDomainId(dstLeaseDomainId); } } getParameters().setLeaseAction(determineLeaseAction(activeSnapshotLeaseDomainId, getParameters().getDstLeaseDomainId())); } }</code><technical_language>instead of early return, I clearer save action in a local variable</technical_language>

Target:
private void initializeSnapshotsLeasesParams() { if (isRestoreLease()) { Guid activeSnapshotLeaseDomainId = getVm().getStaticData().getLeaseStorageDomainId(); Guid selectedLeaseDomainId = getParameters().getDstLeaseDomainId(); LeaseAction leaseAction = null; if (selectedLeaseDomainId == null && getDstSnapshot().getVmConfiguration() != null) { Guid dstLeaseDomainId = OvfUtils.fetchLeaseDomainId(getDstSnapshot().getVmConfiguration()); if (dstLeaseDomainId != null && activeSnapshotLeaseDomainId != null && !dstLeaseDomainId.equals( activeSnapshotLeaseDomainId)) { getParameters().setDstLeaseDomainId(activeSnapshotLeaseDomainId); leaseAction = LeaseAction.UPDATE_LEASE_INFO_AND_LEASE_DOMAIN_ID; } else { getParameters().setDstLeaseDomainId(dstLeaseDomainId); } } if (leaseAction == null) { leaseAction = determineLeaseAction(activeSnapshotLeaseDomainId, getParameters().getDstLeaseDomainId()); } getParameters().setLeaseAction(leaseAction); } }
Source:
<code>public void testDatasourceRemove() throws Exception { final RemoveDsBeanRemote bean = lookupEJB(MODULE_NAME, RemoveDsBean.class, RemoveDsBeanRemote.class); bean.testDatasource(); <START> remove(datasourceAddress()); <END> }</code><technical_language>This changes test removing L66</technical_language>

Target:
public void testDatasourceRemove() throws Exception { final RemoveDsBeanRemote bean = lookupEJB(MODULE_NAME, RemoveDsBean.class, RemoveDsBeanRemote.class); bean.testDatasource(); try { remove(datasourceAddress()); fail("Removal of the datasource should have failed."); } catch (MgmtOperationException expected) { } bean.testDatasource(); }
Source:
<code>private void updateSslVerifyUser(boolean value) { StoredConfig userConfig = GlobalConfigCache.getInstance() .getUserConfig(); try { <START> userConfig.load(); <END> updateSslVerify(userConfig, value); } catch (IOException | ConfigInvalidException e) { LOG.error(MessageFormat.format(JGitText.get().userConfigFileInvalid, userConfig, e)); } }</code><technical_language>If I understand this change correctly, this needed anymore</technical_language>

Target:
private void updateSslVerifyUser(boolean value) { StoredConfig userConfig = null; try { userConfig = GlobalConfigCache.getInstance().getUserConfig(); updateSslVerify(userConfig, value); } catch (IOException | ConfigInvalidException e) { LOG.error(MessageFormat.format(JGitText.get().userConfigFileInvalid, userConfig, e)); } }
Source:
<code>public InterpreterResult internalInterpret(String cmd, InterpreterContext context) { this.intpContext = context; TerminalManager.getInstance().setInterpreterContext(context); if (null == terminalThread) { try { terminalPort = RemoteInterpreterUtils.findRandomAvailablePortOnAllLocalInterfaces(); terminalThread = new TerminalThread(terminalPort); terminalThread.start(); } catch (IOException e) { LOGGER.error(e.getMessage(), e); return new InterpreterResult(Code.ERROR, e.getMessage()); } <START> for (int i = 0; i < 5 && !terminalThread.isRunning(); i++) { <END> try { LOGGER.info("loop = " + i); Thread.sleep(500); } catch (InterruptedException e) { LOGGER.error(e.getMessage(), e); } } } setParagraphConfig(); createTerminalDashboard(context.getNoteId(), context.getParagraphId(), terminalPort); return new InterpreterResult(Code.SUCCESS); }</code><technical_language>if is running 5 loops ? consider add timeout mechanism allow timeout threshold configurable</technical_language>

Target:
public InterpreterResult internalInterpret(String cmd, InterpreterContext context) { this.intpContext = context; TerminalManager.getInstance().setInterpreterContext(context); if (null == terminalThread) { try { terminalPort = RemoteInterpreterUtils.findRandomAvailablePortOnAllLocalInterfaces(); terminalThread = new TerminalThread(terminalPort); terminalThread.start(); } catch (IOException e) { LOGGER.error(e.getMessage(), e); return new InterpreterResult(Code.ERROR, e.getMessage()); } for (int i = 0; i < 10 && !terminalThread.isRunning(); i++) { try { LOGGER.info("loop = " + i); Thread.sleep(500); } catch (InterruptedException e) { LOGGER.error(e.getMessage(), e); } } if (!terminalThread.isRunning()) { LOGGER.error("Terminal interpreter can not running!"); } } setParagraphConfig(); createTerminalDashboard(context.getNoteId(), context.getParagraphId(), terminalPort); return new InterpreterResult(Code.SUCCESS); }
Source:
<code>private void initLayout() { setProgressBarVisible(false); <START> pushToApplicationsManager = new PushToApplicationsManager(dialogService, stateManager); <END> BorderPane head = new BorderPane(); head.setTop(createMenu()); head.setCenter(createToolbar()); setTop(head); SplitPane.setResizableWithParent(sidePane, Boolean.FALSE); splitPane.getItems().addAll(sidePane, tabbedPane); mainStage.showingProperty().addListener(new ChangeListener<Boolean>() { @Override public void changed(ObservableValue<? extends Boolean> observable, Boolean oldValue, Boolean showing) { if (showing) { setDividerPosition(); EasyBind.subscribe(sidePane.visibleProperty(), visible -> { if (visible) { if (!splitPane.getItems().contains(sidePane)) { splitPane.getItems().add(0, sidePane); setDividerPosition(); } } else { splitPane.getItems().remove(sidePane); } }); mainStage.showingProperty().removeListener(this); observable.removeListener(this); } } }); setCenter(splitPane); }</code><technical_language>Move this constructor pushToApplicationsManager final</technical_language>

Target:
private void initLayout() { setProgressBarVisible(false); BorderPane head = new BorderPane(); head.setTop(createMenu()); head.setCenter(createToolbar()); setTop(head); SplitPane.setResizableWithParent(sidePane, Boolean.FALSE); splitPane.getItems().addAll(sidePane, tabbedPane); mainStage.showingProperty().addListener(new ChangeListener<Boolean>() { @Override public void changed(ObservableValue<? extends Boolean> observable, Boolean oldValue, Boolean showing) { if (showing) { setDividerPosition(); EasyBind.subscribe(sidePane.visibleProperty(), visible -> { if (visible) { if (!splitPane.getItems().contains(sidePane)) { splitPane.getItems().add(0, sidePane); setDividerPosition(); } } else { splitPane.getItems().remove(sidePane); } }); mainStage.showingProperty().removeListener(this); observable.removeListener(this); } } }); setCenter(splitPane); }
Source:
<code>public AbstractBulkLightblueRequest() { <START> this(Collections.<E> emptyList()); <END> }</code><technical_language>simply ...=new ArrayList</technical_language>

Target:
public AbstractBulkLightblueRequest() { this.requests = new ArrayList<E>(); }
Source:
<code>private void assertSorted(int[] input) { <START> if (input.length < 2) { <END> return; } for (int i = 1; i < input.length; i++) { if (input[i-1] > input[i]) { throw new IllegalStateException("Input must be sorted"); } } }</code><technical_language><= 1 clearer < 2</technical_language>

Target:
private void assertSorted(int[] input) { if (input.length <= 1) { return; } for (int i = 1; i < input.length; i++) { if (input[i-1] > input[i]) { throw new IllegalStateException("Input must be sorted"); } } }
Source:
<code>protected void showDialog(Bundle state) { Context context = getContext(); Resources res = context.getResources(); AlertDialog.Builder builder = new AlertDialog.Builder(context, R.style.Calypso_AlertDialog); View titleView = View.inflate(getContext(), R.layout.detail_list_preference_title, null); mWhichButtonClicked = DialogInterface.BUTTON_NEGATIVE; builder.setPositiveButton(android.R.string.ok, this); <START> builder.setNegativeButton(res.getString(R.string.cancel).toUpperCase(Locale.getDefault()), this); <END> if (titleView != null) { TextView titleText = (TextView) titleView.findViewById(R.id.title); if (titleText != null) { titleText.setText(getTitle()); } builder.setCustomTitle(titleView); } else { builder.setTitle(getTitle()); } if (mDialogMessage != null) { builder.setMessage(mDialogMessage); } View view = View.inflate(getContext(), getDialogLayoutResource(), null); if (view != null) { onBindDialogView(view); builder.setView(view); } if ((mDialog = builder.create()) == null) return; if (state != null) { mDialog.onRestoreInstanceState(state); } mDialog.setOnDismissListener(this); mDialog.getWindow().setSoftInputMode(WindowManager.LayoutParams.SOFT_INPUT_STATE_VISIBLE); mDialog.show(); Button positive = mDialog.getButton(DialogInterface.BUTTON_POSITIVE); Button negative = mDialog.getButton(DialogInterface.BUTTON_NEGATIVE); if (positive != null) WPPrefUtils.layoutAsFlatButton(positive); if (negative != null) WPPrefUtils.layoutAsFlatButton(negative); }</code><technical_language>another chance android.R.string.cancel here</technical_language>

Target:
protected void showDialog(Bundle state) { Context context = getContext(); Resources res = context.getResources(); AlertDialog.Builder builder = new AlertDialog.Builder(context, R.style.Calypso_AlertDialog); View titleView = View.inflate(getContext(), R.layout.detail_list_preference_title, null); mWhichButtonClicked = DialogInterface.BUTTON_NEGATIVE; builder.setPositiveButton(android.R.string.ok, this); builder.setNegativeButton(res.getString(android.R.string.cancel).toUpperCase(Locale.getDefault()), this); if (titleView != null) { TextView titleText = (TextView) titleView.findViewById(R.id.title); if (titleText != null) { titleText.setText(getTitle()); } builder.setCustomTitle(titleView); } else { builder.setTitle(getTitle()); } if (mDialogMessage != null) { builder.setMessage(mDialogMessage); } View view = View.inflate(getContext(), getDialogLayoutResource(), null); if (view != null) { onBindDialogView(view); builder.setView(view); } if ((mDialog = builder.create()) == null) return; if (state != null) { mDialog.onRestoreInstanceState(state); } mDialog.setOnDismissListener(this); mDialog.getWindow().setSoftInputMode(WindowManager.LayoutParams.SOFT_INPUT_STATE_VISIBLE); mDialog.show(); Button positive = mDialog.getButton(DialogInterface.BUTTON_POSITIVE); Button negative = mDialog.getButton(DialogInterface.BUTTON_NEGATIVE); if (positive != null) WPPrefUtils.layoutAsFlatButton(positive); if (negative != null) WPPrefUtils.layoutAsFlatButton(negative); }
Source:
<code>public void stop() throws Throwable { <START> boltSchedulers.values().forEach( s -> s.stop() ); <END> boltSchedulers.clear(); forkJoinThreadPool.shutdown(); forkJoinThreadPool = null; }</code><technical_language>nice handle s.stop() errors if BoltScheduler fails stop try shutdown others</technical_language>

Target:
public void stop() { boltSchedulers.values().forEach( this::stopScheduler ); boltSchedulers.clear(); forkJoinThreadPool.shutdown(); forkJoinThreadPool = null; }
Source:
<code>public DataSet<ExecRow> getDataSet(DataSetProcessor dsp) throws StandardException { if (!isOpen) throw new IllegalStateException("Operation is not open"); if (bypass) { return source.getDataSet(dsp); } final long fetchLimit = getFetchLimit(); long offset = getTotalOffset(); OperationContext operationContext = dsp.createOperationContext(this); dsp.incrementOpDepth(); DataSet<ExecRow> sourceDS = source.getDataSet(dsp); dsp.decrementOpDepth(); DataSet<ExecRow> sourceSet = sourceDS.map(new CloneFunction<>(operationContext)); if (dsp.isSparkExplain()) { <START> DataSet<ExecRow> ds = dsp.createDataSet(Iterators.transform(Collections.emptyIterator(), <END> new Function<String, ExecRow>() { @Nullable @Override public ExecRow apply(@Nullable String n) { try { return getExecRowDefinition().getClone(); } catch (Exception e) { throw new RuntimeException(e); } } } ), this.explainPlan ); handleSparkExplain(ds, sourceDS, dsp); return ds; } else return sourceSet.zipWithIndex(operationContext).mapPartitions(new OffsetFunction<SpliceOperation, ExecRow>(operationContext, offset, fetchLimit)); }</code><technical_language>dsp.getEmpty()</technical_language>

Target:
public DataSet<ExecRow> getDataSet(DataSetProcessor dsp) throws StandardException { if (!isOpen) throw new IllegalStateException("Operation is not open"); if (bypass) { return source.getDataSet(dsp); } final long fetchLimit = getFetchLimit(); long offset = getTotalOffset(); OperationContext operationContext = dsp.createOperationContext(this); dsp.incrementOpDepth(); DataSet<ExecRow> sourceDS = source.getDataSet(dsp); dsp.decrementOpDepth(); DataSet<ExecRow> sourceSet = sourceDS.map(new CloneFunction<>(operationContext)); if (dsp.isSparkExplain()) { DataSet<ExecRow> ds = dsp.getEmpty(); handleSparkExplain(ds, sourceDS, dsp); return ds; } else return sourceSet.zipWithIndex(operationContext).mapPartitions(new OffsetFunction<SpliceOperation, ExecRow>(operationContext, offset, fetchLimit)); }
Source:
<code>public StatementExecution buildPreparedStatement( DbParameterAccessor[] accessors) throws SQLException { DbParameterAccessors paramAccessors = new DbParameterAccessors(accessors); PreparedStatement ps = environment.getConnection().prepareCall(toSqlString(paramAccessors)); StatementExecution cs; if (paramAccessors.containsReturnValue()) { cs = environment.createFunctionStatementExecution(ps); } else { cs = environment.createStatementExecution(ps); } <START> bindParametersTo(paramAccessors, cs); <END> return cs; }</code><technical_language>do accessors.bindParameters(cs) here</technical_language>

Target:
public StatementExecution buildPreparedStatement( DbParameterAccessor[] accessors) throws SQLException { DbParameterAccessors paramAccessors = new DbParameterAccessors(accessors); PreparedStatement ps = environment.getConnection().prepareCall(toSqlString(paramAccessors)); StatementExecution cs; if (paramAccessors.containsReturnValue()) { cs = environment.createFunctionStatementExecution(ps); } else { cs = environment.createStatementExecution(ps); } paramAccessors.bindParameters(cs); return cs; }
Source:
<code>public VaultContainer(String dockerImageName) { super(dockerImageName); <START> setWaitStrategy(Wait.forHttp("/v1/secret/is_alive").forStatusCode(400)); <END> }</code><technical_language>sense for this be: setWaitStrategy(Wait.forHttp("/v1/sys/health").forStatusCode(200)); (looking <LINK_0>)</technical_language>

Target:
public VaultContainer(String dockerImageName) { super(dockerImageName); setWaitStrategy(Wait.forHttp("/v1/sys/health").forStatusCode(200)); withCreateContainerCmdModifier(cmd -> cmd.withCapAdd(IPC_LOCK)); withEnv("VAULT_ADDR", "<LINK_1>" + port); withExposedPorts(port); }
Source:
<code>public static String getRandomString(int length) { Random r = new Random(); StringBuilder sb = new StringBuilder(length); for (int i = 0; i < length; i++) { <START> sb.append((char) r.nextInt(25) + 'A'); END> } return sb.toString(); }</code><technical_language>r.nextInt(25) r.nextInt(26) r.nextInt('Z' + 1 - 'A'). r.nextInt(x) returns a 0 x - 1</technical_language>

Target:
public static String getRandomString(int length) { Random r = new Random(); StringBuilder sb = new StringBuilder(length); for (int i = 0; i < length; i++) { sb.append((char) r.nextInt(26) + 'A'); } return sb.toString(); }
Source:
<code>public void testRemoveWithinBounds(){ <START> ArrayListOfIntsWritable a = new ArrayListOfIntsWritable(); <END> a.add(one).add(three).add(five).add(seven); assertTrue(one == a.remove(0)); assertTrue(three == a.get(0)); assertTrue(five == a.get(1)); assertTrue(five == a.remove(1)); assertTrue(seven == a.get(2)); }</code><technical_language>Hmm... testing Writable version here? If so, a bug before---please fix</technical_language>

Target:
public void testRemoveWithinBounds(){ ArrayListOfInts a = new ArrayListOfInts(); a.add(one).add(three).add(five).add(seven); assertTrue(one == a.remove(0)); assertTrue(three == a.get(0)); assertTrue(five == a.get(1)); assertTrue(five == a.remove(1)); assertTrue(seven == a.get(2)); }
Source:
<code>public void testSimpleBehavior() { PolymerClassDefinition def = parseAndExtractClassDefFromCall( LINE_JOINER.join( " ", "var FunBehavior = {", " properties: {", " ", " isFun: {", " type: Boolean,", " value: true,", " }", " },", " listeners: {", " click: 'doSomethingFun',", " },", " ", " foo: 'hooray',", "", " ", " doSomethingFun: function(funAmount) { alert('Something ' + funAmount + ' fun!'); },", " ", " created: function() {}", "};", "var A = Polymer({", " is: 'x-element',", " properties: {", " pets: {", " type: Array,", " notify: true,", " },", " name: String,", " },", " behaviors: [ FunBehavior ],", "});")); assertNotNull(def); assertEquals(1, def.version); <START> assertTrue(def.target.isName()); <END> assertEquals("A", def.target.getString()); assertNull(def.nativeBaseElement); assertThat(def.behaviors).hasSize(1); assertThat(def.props).hasSize(3); }</code><technical_language>While here... this updated assertNode(def.target).hasType(Token.NAME)</technical_language>

Target:
public void testSimpleBehavior() { PolymerClassDefinition def = parseAndExtractClassDefFromCall( LINE_JOINER.join( " ", "var FunBehavior = {", " properties: {", " ", " isFun: {", " type: Boolean,", " value: true,", " }", " },", " listeners: {", " click: 'doSomethingFun',", " },", " ", " foo: 'hooray',", "", " ", " doSomethingFun: function(funAmount) { alert('Something ' + funAmount + ' fun!'); },", " ", " created: function() {}", "};", "var A = Polymer({", " is: 'x-element',", " properties: {", " pets: {", " type: Array,", " notify: true,", " },", " name: String,", " },", " behaviors: [ FunBehavior ],", "});")); assertNotNull(def); assertEquals(PolymerClassDefinition.DefinitionType.ObjectLiteral, def.defType); NodeSubject.assertNode(def.target).hasType(Token.NAME); assertEquals("A", def.target.getString()); assertNull(def.nativeBaseElement); assertThat(def.behaviors).hasSize(1); assertThat(def.props).hasSize(3); }
Source:
<code>public void manageMods() { ModManagerController modManagerController = uiService.loadFxml("theme/mod_manager.fxml"); List<Action> actions = Arrays.asList(new Action(i18n.get("modVault.modManager.cancel"), Type.OK_DONE, Event::consume), new Action(i18n.get("modVault.modManager.save"), event -> modManagerController.apply())); <START> notificationService.addNotification(new ImmediateNotification(i18n.get("modVault.modManager"), i18n.get("modVault.modManager.description"), Severity.INFO, actions, modManagerController.getRoot() )); <END> }</code><technical_language>Is this avoid JFXDialog in UIService.showInDialog? I this UIService</technical_language>

Target:
public void manageMods() { ModManagerController modManagerController = uiService.loadFxml("theme/mod_manager.fxml"); JFXDialog jfxDialog = uiService.showInDialog(modVaultRoot, modManagerController.getRoot(), i18n.get("modVault.modManager")); jfxDialog.setOnDialogClosed(event -> modManagerController.apply()); }
Source:
<code>public static void main(String[] args) { CliWrapper cliWrapper = new CliWrapper(); CommandLine commandLine = null; try { commandLine = cliWrapper.parse(args); } catch (ParseException e) { printUsage(cliWrapper); System.out.println(e.getMessage()); System.exit(1); } Configuration.instance().setConfigurationFilename(commandLine.getOptionValue(CliWrapper.CONF)); Configuration.instance().init(); Configuration.instance().updateWithCliOptions(commandLine); ConnectorFacade facade = ConnectorFacadeFactory.INSTANCE.build(Configuration.instance().getProperty("cli.connector")); <START> new Engine(facade).run(); <END> }</code><technical_language>Nice move</technical_language>

Target:
public static void main(String[] args) { CliWrapper cliWrapper = new CliWrapper(); CommandLine commandLine = null; try { commandLine = cliWrapper.parse(args); } catch (ParseException e) { printUsage(cliWrapper); System.out.println(e.getMessage()); System.exit(1); } ConfigurationHolder.initFromFile(commandLine.getOptionValue(CliWrapper.CONF)); ConfigurationHolder.instance().updateWithCliOptions(commandLine); ConnectorFacade facade = ConnectorFacadeFactory.INSTANCE.build(ConfigurationHolder.instance().getProperty("cli.connector")); new Engine(facade).run(); }
Source:
<code>public void configure(Binder binder) { <START> if (binder != null) { <END> SqlBindings.addOperatorConversion(binder, BloomFilterOperatorConversion.class); } }</code><technical_language>this? binder annotated @Nullable. a similar change codebase</technical_language>

Target:
public void configure(Binder binder) { SqlBindings.addOperatorConversion(binder, BloomFilterOperatorConversion.class); ExpressionModule.addExprMacro(binder, BloomFilterExprMacro.class); }