auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public StatementContext(PhoenixStatement statement, ColumnResolver resolver, Scan scan, SequenceManager seqManager) { this.statement = statement; this.resolver = resolver; this.scan = scan; this.sequences = seqManager; this.binds = new BindManager(statement.getParameters()); this.aggregates = new AggregationManager(); this.expressions = new ExpressionManager(); PhoenixConnection connection = statement.getConnection(); <START> if (connection!=null) { <END> this.dateFormat = connection.getQueryServices().getProps().get(QueryServices.DATE_FORMAT_ATTRIB, DateUtil.DEFAULT_DATE_FORMAT); this.dateFormatTimeZone = TimeZone.getTimeZone( connection.getQueryServices().getProps().get(QueryServices.DATE_FORMAT_TIMEZONE_ATTRIB, DateUtil.DEFAULT_TIME_ZONE_ID)); this.numberFormat = connection.getQueryServices().getProps().get(QueryServices.NUMBER_FORMAT_ATTRIB, NumberUtil.DEFAULT_NUMBER_FORMAT); } else { this.dateFormat = DateUtil.DEFAULT_DATE_FORMAT; this.dateFormatTimeZone = TimeZone.getTimeZone(DateUtil.DEFAULT_TIME_ZONE_ID); this.numberFormat = NumberUtil.DEFAULT_NUMBER_FORMAT; } this.dateFormatter = DateUtil.getDateFormatter(dateFormat); this.tempPtr = new ImmutableBytesWritable(); this.currentTable = resolver != null && !resolver.getTables().isEmpty() ? resolver.getTables().get(0) : null; this.whereConditionColumns = new ArrayList<Pair<byte[],byte[]>>(); this.dataColumns = this.currentTable == null ? Collections.<PColumn, Integer>emptyMap() : Maps.<PColumn, Integer>newLinkedHashMap(); this.subqueryResults = Maps.<SelectStatement, Object>newHashMap(); }</code><technical_language>crazy do this null check cascade a lot of places. able create a PhoenixConnection IndexMaintainer if need be</technical_language>

Target:
public StatementContext(PhoenixStatement statement, ColumnResolver resolver, Scan scan, SequenceManager seqManager) { this.statement = statement; this.resolver = resolver; this.scan = scan; this.sequences = seqManager; this.binds = new BindManager(statement.getParameters()); this.aggregates = new AggregationManager(); this.expressions = new ExpressionManager(); PhoenixConnection connection = statement.getConnection(); this.dateFormat = connection.getQueryServices().getProps().get(QueryServices.DATE_FORMAT_ATTRIB, DateUtil.DEFAULT_DATE_FORMAT); this.dateFormatTimeZone = TimeZone.getTimeZone( connection.getQueryServices().getProps().get(QueryServices.DATE_FORMAT_TIMEZONE_ATTRIB, DateUtil.DEFAULT_TIME_ZONE_ID)); this.dateFormatter = DateUtil.getDateFormatter(dateFormat); this.numberFormat = connection.getQueryServices().getProps().get(QueryServices.NUMBER_FORMAT_ATTRIB, NumberUtil.DEFAULT_NUMBER_FORMAT); this.tempPtr = new ImmutableBytesWritable(); this.currentTable = resolver != null && !resolver.getTables().isEmpty() ? resolver.getTables().get(0) : null; this.whereConditionColumns = new ArrayList<Pair<byte[],byte[]>>(); this.dataColumns = this.currentTable == null ? Collections.<PColumn, Integer>emptyMap() : Maps.<PColumn, Integer>newLinkedHashMap(); this.subqueryResults = Maps.<SelectStatement, Object>newHashMap(); }
Source:
<code>private Object getMapValue(Class<?> keyType, Class<?> valueType, String propertyName, EDBObject object) { Map<Object, Object> temp = new HashMap<Object, Object>(); for (int i = 0;; i++) { <START> String keyProperty = propertyName + "." + i + ".key"; <END> String valueProperty = propertyName + "." + i + ".value"; if (!object.containsKey(keyProperty)) { break; } Object key = object.getObject(keyProperty); Object value = object.getObject(valueProperty); if (OpenEngSBModel.class.isAssignableFrom(keyType)) { key = convertEDBObjectToUncheckedModel(keyType, edbService.getObject(key.toString())); } if (OpenEngSBModel.class.isAssignableFrom(valueType)) { value = convertEDBObjectToUncheckedModel(valueType, edbService.getObject(value.toString())); } temp.put(key, value); object.remove(keyProperty); object.remove(valueProperty); } return temp; }</code><technical_language>put magical constants real constants pls :-)</technical_language>

Target:
private Object getMapValue(Class<?> keyType, Class<?> valueType, String propertyName, EDBObject object) { Map<Object, Object> temp = new HashMap<Object, Object>(); for (int i = 0;; i++) { String keyProperty = EDBConverterUtils.getEntryNameForMapKey(propertyName, i); String valueProperty = EDBConverterUtils.getEntryNameForMapValue(propertyName, i); if (!object.containsKey(keyProperty)) { break; } Object key = object.getObject(keyProperty); Object value = object.getObject(valueProperty); if (OpenEngSBModel.class.isAssignableFrom(keyType)) { key = convertEDBObjectToUncheckedModel(keyType, edbService.getObject(key.toString())); } if (OpenEngSBModel.class.isAssignableFrom(valueType)) { value = convertEDBObjectToUncheckedModel(valueType, edbService.getObject(value.toString())); } temp.put(key, value); object.remove(keyProperty); object.remove(valueProperty); } return temp; }
Source:
<code>protected BeanDefinitionBuilder parseHandler(Element gatewayElement, ParserContext parserContext) { final BeanDefinitionBuilder jpaOutboundGatewayBuilder = super.parseHandler(gatewayElement, parserContext); final BeanDefinitionBuilder jpaExecutorBuilder = JpaParserUtils.getOutboundGatewayJpaExecutorBuilder(gatewayElement, parserContext); IntegrationNamespaceUtils.setValueIfAttributeDefined(jpaExecutorBuilder, gatewayElement, "persist-mode"); IntegrationNamespaceUtils.setValueIfAttributeDefined(jpaExecutorBuilder, gatewayElement, "flush"); IntegrationNamespaceUtils.setValueIfAttributeDefined(jpaExecutorBuilder, gatewayElement, "flush-size"); <START> IntegrationNamespaceUtils.setValueIfAttributeDefined(jpaExecutorBuilder, gatewayElement, "clean-on-flush"); <END> final BeanDefinition jpaExecutorBuilderBeanDefinition = jpaExecutorBuilder.getBeanDefinition(); final String gatewayId = this.resolveId(gatewayElement, jpaOutboundGatewayBuilder.getRawBeanDefinition(), parserContext); final String jpaExecutorBeanName = gatewayId + ".jpaExecutor"; parserContext.registerBeanComponent(new BeanComponentDefinition(jpaExecutorBuilderBeanDefinition, jpaExecutorBeanName)); jpaOutboundGatewayBuilder.addConstructorArgValue(jpaExecutorBuilderBeanDefinition); jpaOutboundGatewayBuilder.addPropertyValue("gatewayType", OutboundGatewayType.UPDATING); return jpaOutboundGatewayBuilder; }</code><technical_language>more explicit, throw error if "clean-on-flush" is "flush" "flush-size" present</technical_language>

Target:
protected BeanDefinitionBuilder parseHandler(Element gatewayElement, ParserContext parserContext) { final BeanDefinitionBuilder jpaOutboundGatewayBuilder = super.parseHandler(gatewayElement, parserContext); final BeanDefinitionBuilder jpaExecutorBuilder = JpaParserUtils.getOutboundGatewayJpaExecutorBuilder(gatewayElement, parserContext); IntegrationNamespaceUtils.setValueIfAttributeDefined(jpaExecutorBuilder, gatewayElement, "persist-mode"); IntegrationNamespaceUtils.setValueIfAttributeDefined(jpaExecutorBuilder, gatewayElement, "flush"); IntegrationNamespaceUtils.setValueIfAttributeDefined(jpaExecutorBuilder, gatewayElement, "flush-size"); IntegrationNamespaceUtils.setValueIfAttributeDefined(jpaExecutorBuilder, gatewayElement, "clear-on-flush"); final BeanDefinition jpaExecutorBuilderBeanDefinition = jpaExecutorBuilder.getBeanDefinition(); final String gatewayId = this.resolveId(gatewayElement, jpaOutboundGatewayBuilder.getRawBeanDefinition(), parserContext); final String jpaExecutorBeanName = gatewayId + ".jpaExecutor"; parserContext.registerBeanComponent(new BeanComponentDefinition(jpaExecutorBuilderBeanDefinition, jpaExecutorBeanName)); jpaOutboundGatewayBuilder.addConstructorArgReference(jpaExecutorBeanName); jpaOutboundGatewayBuilder.addPropertyValue("gatewayType", OutboundGatewayType.UPDATING); return jpaOutboundGatewayBuilder; }
Source:
<code><START> public NotesToJson(Json json) { <END> this.json = checkNotNull(json, "json"); }</code><technical_language>Remove modifier constructor package private, remove null check</technical_language>

Target:
NotesToJson(Json json) { this.json = json; }
Source:
<code>private int addMappingUnderLock(ObjectStreamClass desc, SerializableDataKey probe) { lock.lock(); try { if (potentiallyInconsistent) { refreshMappingsFromServer(readLookup.entrySet()); potentiallyInconsistent = false; } ObjectStreamClass disconnected = disconnect(desc); SerializableDataKey key = new SerializableDataKey(disconnected, true); while (true) { Integer rep = writeLookup.get(probe); <START> if (rep != null) { <END> return rep; } rep = nextStreamIndex++; try { ObjectStreamClass existingOsc = readLookup.putIfAbsent(rep, disconnected); if (existingOsc == null) { writeLookup.put(key, rep); readLookupLocalCache.put(rep, disconnected); return rep; } else { ObjectStreamClass discOsc = disconnect(existingOsc); writeLookup.put(new SerializableDataKey(discOsc, true), rep); readLookupLocalCache.put(rep, discOsc); } } catch (Throwable t) { potentiallyInconsistent = true; throw t; } } } finally { lock.unlock(); } }</code><technical_language>single-step probing in this method for updating local association cache pause. I envision situations (for example, some other client added a batch of complex cache objects composed of Java objects new cache) failed puts required local copy of class associations is synchronized server nextStreamIndex accurately reflects server state</technical_language>

Target:
private int addMappingUnderLock(ObjectStreamClass desc, SerializableDataKey probe) { lock.lock(); try { if (potentiallyInconsistent) { refreshMappingsFromStateRepository(); potentiallyInconsistent = false; } while (true) { Integer rep = writeLookupCache.get(probe); if (rep != null) { return rep; } rep = nextStreamIndex++; try { ObjectStreamClass disconnected = disconnect(desc); ObjectStreamClass existingOsc = persistentState.putIfAbsent(rep, disconnected); if (existingOsc == null) { cacheMapping(rep, disconnected); return rep; } else { cacheMapping(rep, disconnect(existingOsc)); } } catch (Throwable t) { potentiallyInconsistent = true; throw t; } } } finally { lock.unlock(); } }
Source:
<code>private JSONObject getSlaveAttributeForLabel(String labelName) { LOGGER.info(String.format("Getting slave attributes for label: %s", labelName)); List<MesosSlaveInfo> matchedSlaves = getMesosCloud().getSlaveInfos().stream() <START> .filter(slaveInfo -> slaveInfo.getLabelString() != null && slaveInfo.getLabelString().equals(labelName)) <END> .collect(Collectors.toList()); if (matchedSlaves.size() > 1) { LOGGER.info(String.format("Multiple matches for label %s found, matches: %o", labelName, matchedSlaves.size())); LOGGER.info("For multiple matches, checking for Slave Info marked as default"); return matchedSlaves.stream() .filter(MesosSlaveInfo::isDefaultSlave).collect(Collectors.toList()) .stream() .findFirst() .map(MesosSlaveInfo::getSlaveAttributes) .orElse(null); } return matchedSlaves.stream().findFirst().map(MesosSlaveInfo::getSlaveAttributes).orElse(null); }</code><technical_language>suggestion .filter(slaveInfo -> labelName.equals(slaveInfo.getLabelString())) avoid null pointer exception</technical_language>

Target:
private JSONObject getSlaveAttributeForLabel(String labelName) { LOGGER.info(String.format("Getting slave attributes for label: %s", labelName)); List<MesosSlaveInfo> matchedSlaves = getMesosCloud().getSlaveInfos().stream() .filter(slaveInfo -> Objects.equals(labelName, slaveInfo.getLabelString())) .collect(Collectors.toList()); if (matchedSlaves.size() > 1) { LOGGER.info(String.format("Multiple matches for label %s found, matches: %o", labelName, matchedSlaves.size())); LOGGER.info("For multiple matches, checking for Slave Info marked as default"); return matchedSlaves.stream() .filter(MesosSlaveInfo::isDefaultSlave).collect(Collectors.toList()) .stream() .findFirst() .map(MesosSlaveInfo::getSlaveAttributes) .orElse(new JSONObject()); } return matchedSlaves.stream().findFirst().map(MesosSlaveInfo::getSlaveAttributes).orElse(new JSONObject()); }
Source:
<code><START> public void undelete(){ <END> WebElement undeleteLink = wait.forElementVisible(undeleteLinkBy); undeleteLink.click(); }</code><technical_language>this void method if another clickUndeleteLinkInBannerNotification method returning SpecialRestorePageObject ? name a bit more precise ;)</technical_language>

Target:
public SpecialRestorePageObject undelete() { WebElement undeleteLink = wait.forElementVisible(undeleteLinkBy); undeleteLink.click(); return new SpecialRestorePageObject(driver); }
Source:
<code>public static void beforeClass() throws IOException, CommandFailedException { <START> operations.writeAttribute(WEBSERVICES_ADDRESS, "statistics-enabled", true); <END> if (client.options().isDomain) { address = Address.host(client.options().defaultHost) .and("server", "server-one"); } address = address.and("deployment", DEPLOYMENT_FILE_NAME) .and("subsystem", "webservices") .and("endpoint", "test%3ATestService"); verifier = new ResourceVerifier(address, client); }</code><technical_language>properly remembering original putting tests</technical_language>

Target:
public static void beforeClass() throws IOException, CommandFailedException { originalStatisticsEnabledValue = operations.readAttribute(WEBSERVICES_ADDRESS, STATISTICS_ENABLED, ReadAttributeOption.NOT_INCLUDE_DEFAULTS).value(); operations.writeAttribute(WEBSERVICES_ADDRESS, STATISTICS_ENABLED, true); if (client.options().isDomain) { address = Address.host(client.options().defaultHost) .and("server", "server-one"); } address = address.and("deployment", DEPLOYMENT_FILE_NAME) .and("subsystem", "webservices") .and("endpoint", "test%3ATestService"); verifier = new ResourceVerifier(address, client); }
Source:
<code>public final JsonObject json() throws IOException { <START> return null; <END> }</code><technical_language>instead of returning null please throw UnsupportedOperationException("MkFork#json()")</technical_language>

Target:
public final JsonObject json() throws IOException { throw new UnsupportedOperationException("MkFork#json()"); }
Source:
<code>public Builder adapt(Scan scan, ReadHooks readHooks) { throwIfUnsupportedScan(scan); RowFilter filter = buildFilter(scan, readHooks); RowSet.Builder rowSetBuilder = RowSet.newBuilder(); if (scan.isGetScan()) { rowSetBuilder.addRowKeys(ByteString.copyFrom(scan.getStartRow())); } else { rowSetBuilder.addRowRanges( RowRange.newBuilder().setStartKeyClosed(ByteString.copyFrom(scan.getStartRow())) <START> .setEndKeyOpen(ByteString.copyFrom(scan.getStopRow()))); <END> } return ReadRowsRequest.newBuilder().setFilter(filter).setRows(rowSetBuilder); }</code><technical_language>Indentation a bit inconsistent. Personally fine style (with leading periods) in cases indentation (4 spaces instead of 2)</technical_language>

Target:
public Builder adapt(Scan scan, ReadHooks readHooks) { throwIfUnsupportedScan(scan); RowFilter filter = buildFilter(scan, readHooks); RowSet.Builder rowSetBuilder = RowSet.newBuilder(); ByteString startRow = ByteString.copyFrom(scan.getStartRow()); if (scan.isGetScan()) { rowSetBuilder.addRowKeys(startRow); } else { ByteString stopRow = ByteString.copyFrom(scan.getStopRow()); rowSetBuilder.addRowRangesBuilder().setStartKeyClosed(startRow).setEndKeyOpen(stopRow); } return ReadRowsRequest.newBuilder().setRows(rowSetBuilder).setFilter(filter); }
Source:
<code>void setEnableEditMode(boolean enable) { if (mIsMultiSelectEnabled == enable) { return; } if (enable) { mShowHiddenSites = true; mShowSelfHostedSites = false; } else { mShowHiddenSites = false; mShowSelfHostedSites = true; } mIsMultiSelectEnabled = enable; mSelectedPositions.clear(); <START> notifyDataSetChanged(); <END> loadSites(); }</code><technical_language>A notifyDataSetChanged() call is in LoadSitesTask.onPostExecute loadSites() this line. remove this redundant</technical_language>

Target:
void setEnableEditMode(boolean enable) { if (mIsMultiSelectEnabled == enable) { return; } if (enable) { mShowHiddenSites = true; mShowSelfHostedSites = false; } else { mShowHiddenSites = false; mShowSelfHostedSites = true; } mIsMultiSelectEnabled = enable; mSelectedPositions.clear(); loadSites(); }
Source:
<code><START> public void visitCompilationUnit(CompilationUnitTree tree) { <END> super.visitCompilationUnit(tree); checkDuplications(functions, FUNCTION_TO_NAME); }</code><technical_language>Add cleanup here: functions.clear(); methods.clear(); "functions" forgot clean somewhere. "methods" a corner case, in case of exception, enter in "visitCompilationUnit" in a dirty state</technical_language>

Target:
public void visitCompilationUnit(CompilationUnitTree tree) { functions.clear(); methods.clear(); super.visitCompilationUnit(tree); checkDuplications(functions, FUNCTION_TO_NAME); }
Source:
<code>public ArrayList<LectureSlide> getLectureSlide(final List<String> slideId, final String userId) throws AuthenticationException, DatabaseAccessException { final long currentTime = System.currentTimeMillis(); final ArrayList<LectureSlide> allSlides = new ArrayList<LectureSlide>(); for (int slides = slideId.size() - 1; slides >= 0; slides--) { try { allSlides.add(SlideManager.mongoGetLectureSlide(getInstance().auth, getInstance().database, slideId.get(slides), userId, currentTime)); } catch (DatabaseAccessException e) { LOG.info(LoggingConstants.EXCEPTION_MESSAGE, e); if (!e.isRecoverable()) { throw e; } } catch (AuthenticationException e) { <START> LOG.error("Error: {}", e); <END> if (e.getType() != AuthenticationException.INVALID_DATE) { throw e; } } } return allSlides; }</code><technical_language>constant</technical_language>

Target:
public ArrayList<LectureSlide> getLectureSlide(final List<String> slideId, final String userId) throws AuthenticationException, DatabaseAccessException { final long currentTime = System.currentTimeMillis(); final ArrayList<LectureSlide> allSlides = new ArrayList<LectureSlide>(); for (int slides = slideId.size() - 1; slides >= 0; slides--) { try { allSlides.add(SlideManager.mongoGetLectureSlide(getInstance().auth, getInstance().database, slideId.get(slides), userId, currentTime)); } catch (DatabaseAccessException e) { LOG.error(LoggingConstants.EXCEPTION_MESSAGE, e); if (!e.isRecoverable()) { throw e; } } catch (AuthenticationException e) { LOG.error(LoggingConstants.EXCEPTION_MESSAGE, e); if (e.getType() != AuthenticationException.INVALID_DATE) { throw e; } } } return allSlides; }
Source:
<code>void setInitialBounds () { if ((state & ZERO_WIDTH) != 0 && (state & ZERO_HEIGHT) != 0) { long topHandle = topHandle (); <START> if(GTK.GTK_VERSION > OS.VERSION(3, 18, 0)) { <END> if(mustBeVisibleOnInitBounds()) { GTK.gtk_widget_set_visible(topHandle, true); } return; } GtkAllocation allocation = new GtkAllocation(); if ((parent.style & SWT.MIRRORED) != 0) { allocation.x = parent.getClientWidth (); } else { allocation.x = 0; } allocation.y = 0; GTK.gtk_widget_set_visible(topHandle, true); GTK.gtk_widget_set_allocation(topHandle, allocation); } else { resizeHandle (1, 1); forceResize (); } }</code><technical_language>I wrong before, checking original bug talking Eric, version guard >= 3.20.0</technical_language>

Target:
void setInitialBounds () { if ((state & ZERO_WIDTH) != 0 && (state & ZERO_HEIGHT) != 0) { long topHandle = topHandle (); if(GTK.GTK_VERSION >= OS.VERSION(3, 20, 0)) { if(mustBeVisibleOnInitBounds()) { GTK.gtk_widget_set_visible(topHandle, true); } return; } GtkAllocation allocation = new GtkAllocation(); if ((parent.style & SWT.MIRRORED) != 0) { allocation.x = parent.getClientWidth (); } else { allocation.x = 0; } allocation.y = 0; GTK.gtk_widget_set_visible(topHandle, true); GTK.gtk_widget_set_allocation(topHandle, allocation); } else { resizeHandle (1, 1); forceResize (); } }
Source:
<code>public void onEndProduct(ProductId id) throws Exception { super.onEndProduct(id); Product product = getProduct(); <START> byte[] json = new JsonProduct().getJsonObject(product).toString().getBytes("UTF8"); <END> out.write(json); }</code><technical_language>This actual Charset a string identifier. Similar throughout</technical_language>

Target:
public void onEndProduct(ProductId id) throws Exception { super.onEndProduct(id); Product product = getProduct(); byte[] json = new JsonProduct().getJsonObject(product) .toString().getBytes(StandardCharsets.UTF_8); out.write(json); }
Source:
<code>public static WebElement findByFooter(WebDriver webDriver, String tableLocator, String content) { List<String> locators = parseTableLocator(tableLocator, "footer"); <START> System.out.println(Arrays.toString(locators.toArray())); <END> return searchInLocators(webDriver, locators, content); }</code><technical_language>System.out.println(String.format("Find footer locators: %s", locators));</technical_language>

Target:
public static WebElement findByFooter(WebDriver webDriver, String tableLocator, String content) { List<String> locators = parseTableLocator(tableLocator, "footer"); return searchInLocators(webDriver, locators, content); }
Source:
<code>public void executeCommand() { addAuditLogForPartialVMs(); super.executeCommand(); if (getParameters().isImagesExistOnTargetStorageDomain()) { if (!getImages().isEmpty()) { findAndSaveDiskCopies(); <START> for (DiskImage diskImage : getImages()) { initQcowVersionForDisks(diskImage.getId()); } <END> } unregisteredOVFDataDao.removeEntity(ovfEntityData.getEntityId(), null); unregisteredDisksDao.removeUnregisteredDiskRelatedToVM(ovfEntityData.getEntityId(), null); } setActionReturnValue(getVmTemplate().getId()); setSucceeded(true); }</code><technical_language>forEach</technical_language>

Target:
public void executeCommand() { addAuditLogForPartialVMs(); super.executeCommand(); if (getParameters().isImagesExistOnTargetStorageDomain()) { if (!getImages().isEmpty()) { findAndSaveDiskCopies(); getImages().stream().forEach(diskImage -> { initQcowVersionForDisks(diskImage.getId()); }); } unregisteredOVFDataDao.removeEntity(ovfEntityData.getEntityId(), null); unregisteredDisksDao.removeUnregisteredDiskRelatedToVM(ovfEntityData.getEntityId(), null); } setActionReturnValue(getVmTemplate().getId()); setSucceeded(true); }
Source:
<code>public VariantLoaderStepParametersValidator() { super(new String[]{ JobParametersNames.DB_NAME, JobParametersNames.DB_COLLECTIONS_VARIANTS_NAME, <START> JobParametersNames.DB_COLLECTIONS_FILES_NAME, <END> JobParametersNames.INPUT_STUDY_ID, JobParametersNames.INPUT_VCF_ID, JobParametersNames.INPUT_VCF, }, new String[]{}); }</code><technical_language>This parameter necessary, split variants header loading. This applies instantiation of DbCollectionsFilesNameValidator object below</technical_language>

Target:
public VariantLoaderStepParametersValidator() { super(new String[]{JobParametersNames.DB_NAME, JobParametersNames.DB_COLLECTIONS_VARIANTS_NAME, JobParametersNames.INPUT_STUDY_ID, JobParametersNames.INPUT_VCF_ID, JobParametersNames.INPUT_VCF, JobParametersNames.INPUT_VCF_AGGREGATION}, new String[]{}); }
Source:
<code>private static Optional<ExchangeRepositoryMetadata> getExchangeRepositoryMetadata(DeploymentRepository repository, Settings settings, List<String> customDomains) { ExchangeRepositoryMetadata metadata = null; <START> if (repository != null && new ExchangeUriChecker(customDomains).isExchangeRepo(repository.getUrl())) { <END> Server server = settings.getServer(repository.getId()); if (server != null) { Credentials credentials = new Credentials(server.getUsername(), server.getPassword()); metadata = new ExchangeRepositoryMetadata(credentials, repository.getUrl(), customDomains); } } return Optional.ofNullable(metadata); }</code><technical_language>initialize ExchangeUriChecker instance in a line, for if condition</technical_language>

Target:
private static Optional<ExchangeRepositoryMetadata> getExchangeRepositoryMetadata(DeploymentRepository repository, Settings settings, List<String> customDomains) { ExchangeRepositoryMetadata metadata = null; ExchangeUriChecker exchangeUriChecker = new ExchangeUriChecker(customDomains); if (repository != null && exchangeUriChecker.isExchangeRepo(repository.getUrl())) { Server server = settings.getServer(repository.getId()); if (server != null) { Credentials credentials = new Credentials(server.getUsername(), server.getPassword()); metadata = new ExchangeRepositoryMetadata(credentials, repository.getUrl(), customDomains); } } return Optional.ofNullable(metadata); }
Source:
<code>protected String getTextTransform(String textToInsert, String oldText, CSSValue value) { if (value.getCssValueType() == CSSValue.CSS_PRIMITIVE_VALUE) { CSSPrimitiveValue primitiveValue = (CSSPrimitiveValue) value; String textTransform = primitiveValue.getStringValue(); if ("capitalize".equals(textTransform)) { String newText = StringUtils.capitalize(oldText + textToInsert); if (newText.length() > 0) { return newText.substring(newText.length() - 1); } } if ("uppercase".equals(textTransform)) { <START> if (textToInsert != null) { <END> return textToInsert.toUpperCase(); } } if ("lowercase".equals(textTransform)) { if (textToInsert != null) { return textToInsert.toLowerCase(); } } if ("inherit".equals(textTransform)) { return textToInsert; } } return textToInsert; }</code><technical_language>combine if clause</technical_language>

Target:
protected String getTextTransform(String textToInsert, String oldText, CSSValue value) { if (value.getCssValueType() != CSSValue.CSS_PRIMITIVE_VALUE) { return textToInsert; } String textTransform = ((CSSPrimitiveValue) value).getStringValue(); if ("capitalize".equals(textTransform)) { String newText = StringUtils.capitalize(oldText + textToInsert); if (newText.length() > 0) { return newText.substring(newText.length() - 1); } } if ("uppercase".equals(textTransform) && textToInsert != null) { return textToInsert.toUpperCase(); } if ("lowercase".equals(textTransform) && textToInsert != null) { return textToInsert.toLowerCase(); } if ("inherit".equals(textTransform)) { return textToInsert; } return textToInsert; }
Source:
<code>public static MoveEntityEvent.Teleport handleDisplaceEntityTeleportEvent(Entity entityIn, Transform<World> fromTransform, Transform<World> toTransform, boolean apiCall) { final PhaseTracker phaseTracker = PhaseTracker.getInstance(); final PhaseData peek = phaseTracker.getCurrentPhaseData(); final IPhaseState state = peek.state; final PhaseContext<?> context = peek.context; <START> Sponge.getCauseStackManager().pushCause(entityIn); <END> MoveEntityEvent.Teleport event = SpongeEventFactory.createMoveEntityEventTeleport(Sponge.getCauseStackManager().getCurrentCause(), fromTransform, toTransform, (org.spongepowered.api.entity.Entity) entityIn); SpongeImpl.postEvent(event); return event; }</code><technical_language>popping pushing. need ensure stack is stack neutral method is completed, popping cause, wrapping a try resources a stack frame</technical_language>

Target:
public static MoveEntityEvent.Teleport handleDisplaceEntityTeleportEvent(Entity entityIn, Transform<World> fromTransform, Transform<World> toTransform, boolean apiCall) { final PhaseTracker phaseTracker = PhaseTracker.getInstance(); final PhaseData peek = phaseTracker.getCurrentPhaseData(); final IPhaseState state = peek.state; final PhaseContext<?> context = peek.context; try (CauseStackManager.StackFrame frame = Sponge.getCauseStackManager().pushCauseFrame()) { frame.pushCause(entityIn); MoveEntityEvent.Teleport event = SpongeEventFactory.createMoveEntityEventTeleport(Sponge.getCauseStackManager().getCurrentCause(), fromTransform, toTransform, (org.spongepowered.api.entity.Entity) entityIn); SpongeImpl.postEvent(event); return event; } }
Source:
<code>@Override public void init(StreamGraph graph, Config config) { Supplier<Integer> initialValue = () -> 0; MessageStream<PageViewEvent> pageViewEvents = graph.getInputStream("pageViewEventStream", (k, m) -> (PageViewEvent) m); pageViewEvents .partitionBy(m -> m.memberId) .window(Windows.keyedTumblingWindow(m -> m.memberId, Duration.ofMinutes(5), initialValue, (m, c) -> c + 1)) .map(MyStreamOutput::new) <START> .sendTo(graph.getOutputStream("steramId", m -> m.memberId, m -> m.memberId)); <END> }</code><technical_language>nit: streamId</technical_language>

Target:
@Override public void init(StreamGraph graph, Config config) { Supplier<Integer> initialValue = () -> 0; MessageStream<PageViewEvent> pageViewEvents = graph.getInputStream("pageViewEventStream", (k, m) -> (PageViewEvent) m); OutputStream<String, MyStreamOutput, MyStreamOutput> pageViewEventPerMemberStream = graph .getOutputStream("pageViewEventPerMemberStream", m -> m.memberId, m -> m); pageViewEvents .partitionBy(m -> m.memberId) .window(Windows.keyedTumblingWindow(m -> m.memberId, Duration.ofMinutes(5), initialValue, (m, c) -> c + 1)) .map(MyStreamOutput::new) .sendTo(pageViewEventPerMemberStream); }
Source:
<code>public DiskImageConditionFieldAutoCompleter() { mVerbs.put("ALIAS", "ALIAS"); mVerbs.put("DESCRIPTION", "DESCRIPTION"); mVerbs.put("PROVISIONED_SIZE", "PROVISIONED_SIZE"); mVerbs.put("SIZE", "SIZE"); mVerbs.put("ACTUAL_SIZE", "ACTUAL_SIZE"); mVerbs.put("CREATION_DATE", "CREATION_DATE"); mVerbs.put("BOOTABLE", "BOOTABLE"); mVerbs.put("FORMAT", "FORMAT"); mVerbs.put("STATUS", "STATUS"); buildCompletions(); getTypeDictionary().put("ALIAS", String.class); getTypeDictionary().put("DESCRIPTION", String.class); getTypeDictionary().put("PROVISIONED_SIZE", Long.class); getTypeDictionary().put("SIZE", Long.class); getTypeDictionary().put("ACTUAL_SIZE", Long.class); getTypeDictionary().put("CREATION_DATE", Date.class); getTypeDictionary().put("BOOTABLE", Boolean.class); getTypeDictionary().put("FORMAT", VolumeFormat.class); getTypeDictionary().put("STATUS", ImageStatus.class); mColumnNameDict.put("ALIAS", "disk_alias"); mColumnNameDict.put("DESCRIPTION", "disk_description"); mColumnNameDict.put("PROVISIONED_SIZE", "size"); <START> mColumnNameDict.put("SIZE", "actual_size"); <END> mColumnNameDict.put("ACTUAL_SIZE", "actual_size"); mColumnNameDict.put("CREATION_DATE", "creation_date"); mColumnNameDict.put("BOOTABLE", "boot"); mColumnNameDict.put("FORMAT", "volume_format"); mColumnNameDict.put("STATUS", "imageStatus"); buildBasicValidationTable(); }</code><technical_language>Note in order sync'd rest-api, "SIZE" text mapped "size" property (instead of "actual_size" property)</technical_language>

Target:
public DiskImageConditionFieldAutoCompleter() { mVerbs.put("ALIAS", "ALIAS"); mVerbs.put("DESCRIPTION", "DESCRIPTION"); mVerbs.put("PROVISIONED_SIZE", "PROVISIONED_SIZE"); mVerbs.put("SIZE", "SIZE"); mVerbs.put("ACTUAL_SIZE", "ACTUAL_SIZE"); mVerbs.put("CREATION_DATE", "CREATION_DATE"); mVerbs.put("BOOTABLE", "BOOTABLE"); mVerbs.put("FORMAT", "FORMAT"); mVerbs.put("STATUS", "STATUS"); buildCompletions(); getTypeDictionary().put("ALIAS", String.class); getTypeDictionary().put("DESCRIPTION", String.class); getTypeDictionary().put("PROVISIONED_SIZE", Long.class); getTypeDictionary().put("SIZE", Long.class); getTypeDictionary().put("ACTUAL_SIZE", Long.class); getTypeDictionary().put("CREATION_DATE", Date.class); getTypeDictionary().put("BOOTABLE", Boolean.class); getTypeDictionary().put("FORMAT", VolumeFormat.class); getTypeDictionary().put("STATUS", ImageStatus.class); mColumnNameDict.put("ALIAS", "disk_alias"); mColumnNameDict.put("DESCRIPTION", "disk_description"); mColumnNameDict.put("PROVISIONED_SIZE", "size"); mColumnNameDict.put("SIZE", "size"); mColumnNameDict.put("ACTUAL_SIZE", "actual_size"); mColumnNameDict.put("CREATION_DATE", "creation_date"); mColumnNameDict.put("BOOTABLE", "boot"); mColumnNameDict.put("FORMAT", "volume_format"); mColumnNameDict.put("STATUS", "imageStatus"); buildBasicValidationTable(); }
Source:
<code>static List<Date> getDays(long endTs, @Nullable Long lookback) { List<Date> result = new ArrayList<>(); <START> for (Long epochMillis : DateUtil.epochDays(endTs, lookback)) { <END> result.add(new Date(epochMillis)); } return result; }</code><technical_language>Accidental box</technical_language>

Target:
static List<Date> getDays(long endTs, @Nullable Long lookback) { List<Date> result = new ArrayList<>(); for (long epochMillis : DateUtil.epochDays(endTs, lookback)) { result.add(new Date(epochMillis)); } return result; }
Source:
<code>protected Snapshot doPopulate(Snapshot model, org.ovirt.engine.core.common.businessentities.Snapshot entity) { <START> if (isPopulate()) { <END> VdcQueryReturnValue queryReturnValue = runQuery(VdcQueryType.GetVmOvfConfigurationBySnapshot, new IdQueryParameters(Guid.createGuidFromString(model.getId()))); if (queryReturnValue.getSucceeded() && queryReturnValue.getReturnValue() != null) { return SnapshotMapper.mapSnapshotConfiguration((String) queryReturnValue.getReturnValue(), ConfigurationType.OVF, model); } } return model; }</code><technical_language>btw true for fetchCreatedEntity() is triggers mapEntity()</technical_language>

Target:
protected Snapshot doPopulate(Snapshot model, org.ovirt.engine.core.common.businessentities.Snapshot entity) { VdcQueryReturnValue queryReturnValue = runQuery(VdcQueryType.GetVmOvfConfigurationBySnapshot, new IdQueryParameters(Guid.createGuidFromString(model.getId()))); if (queryReturnValue.getSucceeded() && queryReturnValue.getReturnValue() != null) { return SnapshotMapper.mapSnapshotConfiguration((String) queryReturnValue.getReturnValue(), ConfigurationType.OVF, model); } return model; }
Source:
<code>public void testGetDrugByMapping_shouldReturnADrugThatMatchesTheSpecifiedSourceAndBestMapType() throws Exception { Drug drugs = conceptService.getDrugByMapping("WGT234", conceptService.getConceptSource(1), Arrays.asList(conceptService.getConceptMapType(2)), false); assertNull(drugs); <START> <END> }</code><technical_language>Please remove this extra blank lines</technical_language>

Target:
public void testGetDrugByMapping_shouldReturnADrugThatMatchesTheSpecifiedSourceAndBestMapType() throws Exception { executeDataSet(GET_DRUG_MAPPINGS); List<ConceptMapType> conceptMapTypeList = conceptService.getConceptMapTypes(false,true); Drug drug = conceptService.getDrugByMapping("CD41003", conceptService.getConceptSource(1), conceptMapTypeList, false); assertNull(drug); }
Source:
<code>private Context(final List<G> objects, final List<M> attributes, final BinaryRelation relation) { <START> this.objects = objects; <END> this.attributes = attributes; this.relation = relation; }</code><technical_language>defensive copy here, this.objects = ImmutableList.copyOf(objects); this.objects = new ArrayList<G>(objects);</technical_language>

Target:
private Context(final List<G> objects, final List<M> attributes, final BinaryRelation relation) { this.objects = ImmutableList.copyOf(objects); this.attributes = ImmutableList.copyOf(attributes); this.relation = relation; }
Source:
<code><START> public static ByteBuffer handleBufferUnderflow(final SSLEngine engine, ByteBuffer buffer) { <END> if (buffer.position() < buffer.limit()) { return buffer; } ByteBuffer replaceBuffer = enlargeBuffer(buffer, engine.getSession().getPacketBufferSize()); buffer.flip(); replaceBuffer.put(buffer); return replaceBuffer; }</code><technical_language>Add Preconditions.checkArgument checks verify passed engine buffer null</technical_language>

Target:
public static ByteBuffer handleBufferUnderflow(final SSLEngine engine, ByteBuffer buffer) { if (engine == null || buffer == null) { return buffer; } if (buffer.position() < buffer.limit()) { return buffer; } ByteBuffer replaceBuffer = enlargeBuffer(buffer, engine.getSession().getPacketBufferSize()); buffer.flip(); replaceBuffer.put(buffer); return replaceBuffer; }
Source:
<code>public void setError(@Nullable Throwable caught) { Resources resources = getContext().getResources(); ErrorType errorType = getErrorType(caught); icon.setImageDrawable(ContextCompat.getDrawable(getContext(), errorType.icon())); errorText.setText(resources.getString(errorType.text())); button.setText(resources.getString(errorType.buttonText())); button.setOnClickListener(errorType.buttonClickListener(this)); if (errorType.hasFooterText()) { <START> footerLayout.setVisibility(VISIBLE); <END> footerText.setText(resources.getString(errorType.footerText())); } }</code><technical_language>curious. set visibility of footerLayout "VISIBLE" if set "GONE" in view_wiki_error.xml</technical_language>

Target:
public void setError(@Nullable Throwable caught) { Resources resources = getContext().getResources(); ErrorType errorType = getErrorType(caught); icon.setImageDrawable(ContextCompat.getDrawable(getContext(), errorType.icon())); errorText.setText(resources.getString(errorType.text())); button.setText(resources.getString(errorType.buttonText())); button.setOnClickListener(errorType.buttonClickListener(this)); if (errorType.hasFooterText()) { footerLayout.setVisibility(VISIBLE); footerText.setText(resources.getString(errorType.footerText())); } else { footerLayout.setVisibility(GONE); } }
Source:
<code>public ApiClient(ApiClientConfiguration config) { PoolingHttpClientConnectionManager connectionManager = new PoolingHttpClientConnectionManager(); connectionManager.closeExpiredConnections(); <START> connectionManager.closeIdleConnections(5, TimeUnit.SECONDS); END> StandardHttpRequestRetryHandler requestRetryHandler = new StandardHttpRequestRetryHandler(3, true); HttpClient client = HttpClientBuilder.create().setConnectionManager(connectionManager).setRetryHandler(requestRetryHandler) .setUserAgent("OktaApiClient").disableCookieManagement().build(); this.httpClient = client; this.baseUrl = config.getBaseUrl(); this.apiVersion = config.getApiVersion(); this.configuration = config; this.token = config.getApiToken(); initMarshaller(); }</code><technical_language>Nit: this a constant</technical_language>

Target:
public ApiClient(ApiClientConfiguration config) { StandardHttpRequestRetryHandler requestRetryHandler = new StandardHttpRequestRetryHandler(RETRY_COUNT, true); HttpClient client = HttpClientBuilder.create().setRetryHandler(requestRetryHandler) .setUserAgent("OktaSDKJava_v" + Utils.getSdkVersion()).disableCookieManagement().build(); this.httpClient = client; this.baseUrl = config.getBaseUrl(); this.apiVersion = config.getApiVersion(); this.configuration = config; this.token = config.getApiToken(); initMarshaller(); }
Source:
<code>public CompletableFuture<RspList<Response>> invokeRemoteCommands(List<Address> recipients, ReplicableCommand command, ResponseMode mode, long timeout, RspFilter filter, DeliverOrder deliverOrder) { CompletableFuture<RspList<Response>> future; try { if (recipients != null && mode == ResponseMode.GET_FIRST && STAGGER_DELAY_NANOS > 0) { future = new CompletableFuture<>(); RspList<Response> rsps = new RspList<>(); for (Address recipient : recipients) { rsps.put(recipient, new Rsp<>(recipient)); } long nanoTimeout = timeout > 0 ? TimeUnit.MILLISECONDS.toNanos(timeout) : Long.MAX_VALUE; <START> long deadline = System.nanoTime() + nanoTimeout; <END> processCallsStaggered(command, filter, recipients, mode, deliverOrder, req_marshaller, future, 0, deadline, rsps); } else { future = processCalls(command, recipients == null, timeout, filter, recipients, mode, deliverOrder, req_marshaller); } return future; } catch (Exception e) { return rethrowAsCacheException(e); } }</code><technical_language>TimeService</technical_language>

Target:
public CompletableFuture<RspList<Response>> invokeRemoteCommands(List<Address> recipients, ReplicableCommand command, ResponseMode mode, long timeout, RspFilter filter, DeliverOrder deliverOrder) { CompletableFuture<RspList<Response>> future; try { if (recipients != null && mode == ResponseMode.GET_FIRST && STAGGER_DELAY_NANOS > 0) { future = new CompletableFuture<>(); RspList<Response> rsps = new RspList<>(); for (Address recipient : recipients) { rsps.put(recipient, new Rsp<>(recipient)); } long nanoTimeout = timeout > 0 ? TimeUnit.MILLISECONDS.toNanos(timeout) : Long.MAX_VALUE; long deadline = timeService.expectedEndTime(nanoTimeout, TimeUnit.NANOSECONDS); processCallsStaggered(command, filter, recipients, mode, deliverOrder, req_marshaller, future, 0, deadline, rsps); } else { future = processCalls(command, recipients == null, timeout, filter, recipients, mode, deliverOrder, req_marshaller); } return future; } catch (Exception e) { return rethrowAsCacheException(e); } }
Source:
<code>private List<GlusterVolumeSnapshotEntity> prepareVolumeSnapshotsList(GlusterVolumeEntity volume, Object[] snapshotInfoDetails) { List<GlusterVolumeSnapshotEntity> newSnapshotsList = new ArrayList<GlusterVolumeSnapshotEntity>(); for (Object obj : snapshotInfoDetails) { Map<String, Object> snapshotInfo = (Map<String, Object>) obj; Object[] snapVolumes = (Object[]) (snapshotInfo.get(SNAPVOLUMES)); Map<String, Object> snapVolume = (Map<String, Object>) snapVolumes[0]; GlusterVolumeSnapshotEntity newSnapshot = new GlusterVolumeSnapshotEntity(); newSnapshot.setClusterId(volume.getClusterId()); newSnapshot.setVolumeId(volume.getId()); try { DateFormat df = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss"); newSnapshot.setCreatedAt(df.parse((String) snapshotInfo.get(CREATETIME))); } catch (Exception e) { log.error("Could not populate snapshots of volume '{}' on cluster '{}': {}", volume.getName(), volume.getClusterId(), <START> e.getMessage()); <END> log.debug("Exception", e); } newSnapshot.setSnapshotId(Guid.createGuidFromString((String) snapshotInfo.get(UUID))); newSnapshot.setSnapshotName((String) snapshotInfo.get(NAME)); newSnapshot.setDescription((String) snapshotInfo.get(DESCRIPTION)); newSnapshot.setStatus(GlusterSnapshotStatus.from((String) snapVolume.get(STATUS))); newSnapshotsList.add(newSnapshot); } return newSnapshotsList; }</code><technical_language>This is a failure in parsing timestamp affect much. Message modified accordingly</technical_language>

Target:
private List<GlusterVolumeSnapshotEntity> prepareVolumeSnapshotsList(GlusterVolumeEntity volume, Object[] snapshotInfoDetails) { List<GlusterVolumeSnapshotEntity> newSnapshotsList = new ArrayList<GlusterVolumeSnapshotEntity>(); for (Object obj : snapshotInfoDetails) { Map<String, Object> snapshotInfo = (Map<String, Object>) obj; Object[] snapVolumes = (Object[]) (snapshotInfo.get(SNAPVOLUMES)); Map<String, Object> snapVolume = (Map<String, Object>) snapVolumes[0]; GlusterVolumeSnapshotEntity newSnapshot = new GlusterVolumeSnapshotEntity(); newSnapshot.setClusterId(volume.getClusterId()); newSnapshot.setVolumeId(volume.getId()); try { DateFormat df = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss"); newSnapshot.setCreatedAt(df.parse((String) snapshotInfo.get(CREATETIME))); } catch (Exception e) { log.info("Could not populate creation time for snapshot '{}' of volume '{}' on cluster '{}': {}", (String) snapshotInfo.get(NAME), volume.getName(), volume.getClusterId(), e.getMessage()); log.debug("Exception", e); } newSnapshot.setSnapshotId(Guid.createGuidFromString((String) snapshotInfo.get(UUID))); newSnapshot.setSnapshotName((String) snapshotInfo.get(NAME)); newSnapshot.setDescription((String) snapshotInfo.get(DESCRIPTION)); newSnapshot.setStatus(GlusterSnapshotStatus.from((String) snapVolume.get(STATUS))); newSnapshotsList.add(newSnapshot); } return newSnapshotsList; }
Source:
<code>public SystemProducer getProducer(String systemName, Config config, MetricsRegistry registry) { jobToManager.putIfAbsent(parseJobId(config), new InMemoryManager()); <START> return new InMemorySystemProducer(systemName, jobToManager.get(parseJobId(config))); <END> }</code><technical_language>curious. jobId work for high level API job? Besides functionality JobId here. odd JobId in a factory for producer/consumer(In memory producer consumer of a samza job a part of a samza job)</technical_language>

Target:
public SystemProducer getProducer(String systemName, Config config, MetricsRegistry registry) { return new InMemorySystemProducer(systemName, getOrDefaultInMemoryManagerByTestId(config)); }
Source:
<code>public Account find(final String nameOrEmail) throws OrmException { Set<Account.Id> r = findAll(nameOrEmail); if (r.size() == 1) { <START> byId.get(r.iterator().next()).getAccount(); <END> } else { final int lt = nameOrEmail.indexOf('<'); final int gt = nameOrEmail.indexOf('>'); if (lt >= 0 && gt > lt && nameOrEmail.contains("@")) { for (Account.Id id : r) { Account a = byId.get(id).getAccount(); if (a.getFullName().equals(nameOrEmail.substring(0, lt - 1))) { return a; } } } } return null; }</code><technical_language>this database query executes, nothing. return result</technical_language>

Target:
public Account find(final String nameOrEmail) throws OrmException { Set<Account.Id> r = findAll(nameOrEmail); return r.size() == 1 ? byId.get(r.iterator().next()).getAccount() : null; }
Source:
<code>public boolean removeAttribute(PerunSession sess, User user, AttributeDefinition attribute) throws InternalErrorException { try { if (0 < jdbc.update("DELETE FROM user_attr_values WHERE attr_id=? AND user_id=?", attribute.getId(), user.getId())) { <START> log.debug("Attribute (its value) was removed from user. Attribute={}, user={}", attribute, user); <END> return true; } return false; } catch (RuntimeException ex) { throw new InternalErrorException(ex); } }</code><technical_language>"Attribute removed" is more concise "Attribute (its value) removed"</technical_language>

Target:
public boolean removeAttribute(PerunSession sess, User user, AttributeDefinition attribute) throws InternalErrorException { try { if (0 < jdbc.update("DELETE FROM user_attr_values WHERE attr_id=? AND user_id=?", attribute.getId(), user.getId())) { log.debug("Attribute value for was removed from user {}.", attribute.getName(), user); return true; } return false; } catch (RuntimeException ex) { throw new InternalErrorException(ex); } }
Source:
<code>protected void setSyncConfig(SyncToolConfig syncConfig) { this.syncConfig = syncConfig; this.syncConfig.setVersion(version); File exclusionListFile = this.syncConfig.getExcludeList(); if(exclusionListFile != null){ this.fileExclusionManager = new FileExclusionManager(exclusionListFile); } else{ <START> fileExclusionManager = new FileExclusionManager(); <END> } ChangedList.getInstance() .setFileExclusionManager(this.fileExclusionManager); }</code><technical_language>more consistent this.fileExclusionManager</technical_language>

Target:
protected void setSyncConfig(SyncToolConfig syncConfig) { this.syncConfig = syncConfig; this.syncConfig.setVersion(version); File exclusionListFile = this.syncConfig.getExcludeList(); if(exclusionListFile != null){ this.fileExclusionManager = new FileExclusionManager(exclusionListFile); } else{ this.fileExclusionManager = new FileExclusionManager(); } ChangedList.getInstance() .setFileExclusionManager(this.fileExclusionManager); }