auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public boolean isAdVisible(final WebElement element, final String selector, final WebDriver driver) { hideSlot(selector, driver); final BufferedImage backgroundImg = shooter.takeScreenshot(element, driver); PageObjectLogging.log("ScreenshotsComparison", "Background image in " + selector, true, driver); showSlot(selector, driver); try { WebDriverWait wait = new WebDriverWait(driver, AD_TIMEOUT_SEC); wait.until(new ExpectedCondition<Object>() { @Override public Object apply(WebDriver driver) { BufferedImage adImg = shooter.takeScreenshot(element, driver); PageObjectLogging.log("ScreenshotsComparison", "Ad image in " + selector, true); if (adImg.getHeight() == 1) { return false; } return imageComparison.areImagesDifferent(backgroundImg, adImg, IMAGES_THRESHOLD_PERCENT); } }); <START> } catch (TimeoutException e) { <END> return false; } return true; }</code><technical_language>resolve this Sonar issue re-use @drets's new method: <LINK_0> :)</technical_language>

Target:
public boolean isAdVisible(final WebElement element, final String selector, final WebDriver driver) { hideSlot(selector, driver); final BufferedImage backgroundImg = shooter.takeScreenshot(element, driver); PageObjectLogging.log("ScreenshotsComparison", "Background image in " + selector, true, driver); showSlot(selector, driver); try { WebDriverWait wait = new WebDriverWait(driver, AD_TIMEOUT_SEC); wait.until(new ExpectedCondition<Object>() { @Override public Object apply(WebDriver driver) { BufferedImage adImg = shooter.takeScreenshot(element, driver); PageObjectLogging.log("ScreenshotsComparison", "Ad image in " + selector, true); if (adImg.getHeight() == 1) { return false; } return imageComparison.areImagesDifferent(backgroundImg, adImg, IMAGES_THRESHOLD_PERCENT); } }); } catch (TimeoutException e) { PageObjectLogging.logWarning("ScreenshotsComparison", e); return false; } return true; }
Source:
<code>public void resetInstalledLocation() throws CoreException { boolean locationReset = false; if (this.getMarker().getAttribute(REQUESTED_LINE) != null) { int line = this.getMarker().getAttribute(REQUESTED_LINE, -1); setAttribute(IMarker.LINE_NUMBER, line); locationReset = true; } if (this.getMarker().getAttribute(REQUESTED_CHAR_START) != null) { int charStart = this.getMarker().getAttribute(REQUESTED_CHAR_START, -1); setAttribute(IMarker.CHAR_START, charStart); locationReset = true; } if (this.getMarker().getAttribute(REQUESTED_CHAR_END) != null) { int charEnd = this.getMarker().getAttribute(REQUESTED_CHAR_END, -1); setAttribute(IMarker.CHAR_END, charEnd); locationReset = true; } if (this.getMarker().getAttribute(REQUESTED_SOURCE_HANDLE) != null) { <START> String file = this.getMarker().getAttribute(REQUESTED_SOURCE_HANDLE, ""); END> setAttribute(ICBreakpoint.SOURCE_HANDLE, file); locationReset = true; } if (locationReset) { setAttribute( IMarker.MESSAGE, getMarkerMessage() ); } }</code><technical_language>setInstalledLineNumber, changing installed source handle RESET_INSTALLED_LOCATION</technical_language>

Target:
public void resetInstalledLocation() throws CoreException { boolean locationReset = false; if (this.getMarker().getAttribute(REQUESTED_LINE) != null) { int line = this.getMarker().getAttribute(REQUESTED_LINE, -1); setAttribute(IMarker.LINE_NUMBER, line); locationReset = true; } if (this.getMarker().getAttribute(REQUESTED_CHAR_START) != null) { int charStart = this.getMarker().getAttribute(REQUESTED_CHAR_START, -1); setAttribute(IMarker.CHAR_START, charStart); locationReset = true; } if (this.getMarker().getAttribute(REQUESTED_CHAR_END) != null) { int charEnd = this.getMarker().getAttribute(REQUESTED_CHAR_END, -1); setAttribute(IMarker.CHAR_END, charEnd); locationReset = true; } if (this.getMarker().getAttribute(REQUESTED_SOURCE_HANDLE) != null) { String file = this.getMarker().getAttribute(REQUESTED_SOURCE_HANDLE, ""); setAttribute(ICBreakpoint.SOURCE_HANDLE, file); setAttribute(RESET_INSTALLED_LOCATION, Boolean.TRUE); locationReset = true; } if (locationReset) { setAttribute( IMarker.MESSAGE, getMarkerMessage() ); } }
Source:
<code>public void process() { PerformanceMonitor.startActivity("rendering/outline"); glClear(GL_COLOR_BUFFER_BIT | GL_DEPTH_BUFFER_BIT); renderFullscreenQuad(); <START> bindDisplay(); END> PerformanceMonitor.endActivity(); }</code><technical_language>pretty remove this now. If necessary default state change task do it. verify enabling/disabling outline works ok</technical_language>

Target:
public void process() { PerformanceMonitor.startActivity("rendering/outline"); glClear(GL_COLOR_BUFFER_BIT | GL_DEPTH_BUFFER_BIT); renderFullscreenQuad(); PerformanceMonitor.endActivity(); }
Source:
<code><START> void scheduleSnapshot(DomainEventMessage<?> eventMessage) { <END> snapshotter.scheduleSnapshot(aggregateType, eventMessage.getAggregateIdentifier()); }</code><technical_language>Is this intended package-private? private perhaps</technical_language>

Target:
private void scheduleSnapshot(DomainEventMessage<?> eventMessage) { snapshotter.scheduleSnapshot(aggregateType, eventMessage.getAggregateIdentifier()); }
Source:
<code>public InternalNode(List<AbstractNode> children) { this.children = children; this.outgoingEdges = new ArrayList<>(); <START> bindMargins(); <END> DoubleBinding rangeBegin = marginProperty().divide(2).negate(); for (AbstractNode child : children) { child.translateYProperty().bind(rangeBegin.add(child.marginProperty().divide(2))); child.translateXProperty().set(LEVELWIDTH); rangeBegin = rangeBegin.add(child.marginProperty()); Edge e = new Edge(child); this.outgoingEdges.add(e); } this.getChildren().addAll(0, outgoingEdges); this.getChildren().addAll(children); }</code><technical_language>redundant tab</technical_language>

Target:
public InternalNode(List<AbstractNode> children) { this.children = children; this.outgoingEdges = new ArrayList<>(); bindMargins(); bindLeafCount(); DoubleBinding rangeBegin = marginProperty().divide(2).negate(); for (AbstractNode child : children) { child.translateYProperty().bind(rangeBegin.add(child.marginProperty().divide(2))); child.translateXProperty().set(LEVELWIDTH); rangeBegin = rangeBegin.add(child.marginProperty()); Edge e = new Edge(child); this.outgoingEdges.add(e); } this.getChildren().addAll(0, outgoingEdges); this.getChildren().addAll(children); }
Source:
<code>public void testNullPointerExceptionUponDataStreamerClosing() throws Exception { cnt = 0; startGrids(5); boolean failed = false; final CyclicBarrier barrier = new CyclicBarrier(2); multithreadedAsync(new Callable<Object>() { @Override public Object call() throws Exception { U.awaitQuiet(barrier); G.stopAll(true); return null; } }, 1); Ignite g4 = grid(4); IgniteDataStreamer<Object, Object> dataLdr = g4.dataStreamer(DEFAULT_CACHE_NAME); dataLdr.perNodeBufferSize(32); for (int i = 0; i < 100000; i += 2) { dataLdr.addData(i, i); dataLdr.removeData(i + 1); } U.awaitQuiet(barrier); info("Closing data streamer."); try { dataLdr.close(true); } catch (CacheException | IllegalStateException ignore) { <START> failed = true; <END> } finally { assertTrue(failed); } }</code><technical_language>create a separate test, this test is closing streamer stop grids exception caused stopping grid</technical_language>

Target:
public void testNullPointerExceptionUponDataStreamerClosing() throws Exception { cnt = 0; startGrids(5); final CyclicBarrier barrier = new CyclicBarrier(2); multithreadedAsync(new Callable<Object>() { @Override public Object call() throws Exception { U.awaitQuiet(barrier); G.stopAll(true); return null; } }, 1); Ignite g4 = grid(4); IgniteDataStreamer<Object, Object> dataLdr = g4.dataStreamer(DEFAULT_CACHE_NAME); dataLdr.perNodeBufferSize(32); for (int i = 0; i < 100000; i += 2) { dataLdr.addData(i, i); dataLdr.removeData(i + 1); } U.awaitQuiet(barrier); info("Closing data streamer."); try { dataLdr.close(true); } catch (CacheException | IllegalStateException ignore) { } }
Source:
<code>private int getUpdatedCaretPosition(StringBuilder currentCalculatedMessage, int currentCaretPosition, CommitMessageWithCaretPosition messageWithPosition, ICommitMessageProvider2 provider) { int pos = currentCaretPosition; if (currentCaretPosition == CommitMessageWithCaretPosition.NO_POSITION) { if <START> (messageWithPosition .getDesiredCaretPosition() == <END> CommitMessageWithCaretPosition.NO_POSITION) { return pos; } if (messageWithPosition .getDesiredCaretPosition() > messageWithPosition .getMessage().length() || messageWithPosition .getDesiredCaretPosition() < CommitMessageWithCaretPosition.NO_POSITION) { Activator.logWarning( MessageFormat.format( UIText.CommitDialog_CaretPositionOutOfBounds, provider.getClass().getName(), messageWithPosition.getDesiredCaretPosition()), null); return CommitMessageWithCaretPosition.NO_POSITION; } else { pos = currentCalculatedMessage.length(); if (currentCalculatedMessage.length() > 0) { pos += MESSAGE_SEPARATOR.length(); } pos += messageWithPosition.getDesiredCaretPosition(); } } else { Activator .logWarning( MessageFormat.format( UIText.CommitDialog_MultipleCommitMessageProvider2Implementations, provider.getClass().getName()), null); } return pos; }</code><technical_language>I store this in a local variable, this code easier read</technical_language>

Target:
private int getUpdatedCaretPosition(StringBuilder currentCalculatedMessage, int currentCaretPosition, CommitMessageWithCaretPosition messageWithPosition, ICommitMessageProvider2 provider) { int pos = currentCaretPosition; if (currentCaretPosition == CommitMessageWithCaretPosition.NO_POSITION) { String providedMessage = messageWithPosition.getMessage(); if (providedMessage == null || providedMessage.trim().isEmpty()) { return pos; } int providedCaretPosition = messageWithPosition .getDesiredCaretPosition(); if (providedCaretPosition == CommitMessageWithCaretPosition.NO_POSITION) { return pos; } if (providedCaretPosition > providedMessage.length() || providedCaretPosition < 0) { Activator.logWarning( MessageFormat.format( UIText.CommitDialog_CaretPositionOutOfBounds, provider.getClass().getName(), providedCaretPosition), null); return CommitMessageWithCaretPosition.NO_POSITION; } else { pos = currentCalculatedMessage.length(); if (currentCalculatedMessage.length() > 0) { pos += MESSAGE_SEPARATOR.length(); } pos += providedCaretPosition; } } else { Activator .logWarning( MessageFormat.format( UIText.CommitDialog_MultipleCommitMessageProvider2Implementations, provider.getClass().getName()), null); } return pos; }
Source:
<code>public byte[] <START> loadSectionData() <END> throws IOException { byte[] data = new byte[(int)sh_size]; efile.seek(sh_offset); efile.read(data); return data; }</code><technical_language>put check too</technical_language>

Target:
public byte[] loadSectionData() throws IOException { makeSureNotCompressed(); byte[] data = new byte[(int)sh_size]; efile.seek(sh_offset); efile.read(data); return data; }
Source:
<code>public static void endTaskStep(NGuid stepId, ExecutionStatus exitStatus) { try { if (stepId != null) { Step step = JobRepositoryFactory.getJobRepository().getStep(stepId.getValue()); if (step != null) { step.markStepEnded(exitStatus); } <START> JobRepositoryFactory.getJobRepository().updateStep(step); <END> } } catch (Exception e) { log.errorFormat("Failed to terminate step {0} with status {1}", stepId, exitStatus, e); } }</code><technical_language>put this code line part of condition scope (step != null) adding error logs null values, for stepId step entity, helpful</technical_language>

Target:
public static void endTaskStep(NGuid stepId, ExecutionStatus exitStatus) { try { if (stepId != null) { Step step = JobRepositoryFactory.getJobRepository().getStep(stepId.getValue()); if (step != null) { step.markStepEnded(exitStatus); JobRepositoryFactory.getJobRepository().updateStep(step); } } } catch (Exception e) { log.errorFormat("Failed to terminate step {0} with status {1}", stepId, exitStatus, e); } }
Source:
<code>public Mono<AccessToken> getToken(TokenRequestContext request) { <START> String clientId = configuration.get(Configuration.PROPERTY_AZURE_CLIENT_ID); String tenantId = configuration.get(Configuration.PROPERTY_AZURE_TENANT_ID); String clientSecret = configuration.get(Configuration.PROPERTY_AZURE_CLIENT_SECRET); String certPath = configuration.get(Configuration.PROPERTY_AZURE_CLIENT_CERTIFICATE_PATH); String username = configuration.get(Configuration.PROPERTY_AZURE_USERNAME); String password = configuration.get(Configuration.PROPERTY_AZURE_PASSWORD); <END> return Mono.fromSupplier(() -> { if (verifyNotNull(clientId)) { if (verifyNotNull(tenantId, clientSecret)) { return new ClientSecretCredential(tenantId, clientId, clientSecret, identityClientOptions); } else if (verifyNotNull(tenantId, certPath)) { return new ClientCertificateCredential(tenantId, clientId, certPath, null, identityClientOptions); } else if (verifyNotNull(username, password)) { return new UsernamePasswordCredential(clientId, tenantId, username, password, identityClientOptions); } } throw logger.logExceptionAsError(new ClientAuthenticationException( "Cannot create any credentials with the current environment variables", null)); }).flatMap(cred -> cred.getToken(request)); }</code><technical_language>This supplier get configuration asynchronously getToken() is subscribed. This fetches configuration synchronously immediately</technical_language>

Target:
public Mono<AccessToken> getToken(TokenRequestContext request) { return Mono.fromSupplier(() -> { String clientId = configuration.get(Configuration.PROPERTY_AZURE_CLIENT_ID); String tenantId = configuration.get(Configuration.PROPERTY_AZURE_TENANT_ID); String clientSecret = configuration.get(Configuration.PROPERTY_AZURE_CLIENT_SECRET); String certPath = configuration.get(Configuration.PROPERTY_AZURE_CLIENT_CERTIFICATE_PATH); String username = configuration.get(Configuration.PROPERTY_AZURE_USERNAME); String password = configuration.get(Configuration.PROPERTY_AZURE_PASSWORD); if (verifyNotNull(clientId)) { if (verifyNotNull(tenantId, clientSecret)) { return new ClientSecretCredential(tenantId, clientId, clientSecret, identityClientOptions); } else if (verifyNotNull(tenantId, certPath)) { return new ClientCertificateCredential(tenantId, clientId, certPath, null, identityClientOptions); } else if (verifyNotNull(username, password)) { return new UsernamePasswordCredential(clientId, tenantId, username, password, identityClientOptions); } } throw logger.logExceptionAsError(new ClientAuthenticationException( "Cannot create any credentials with the current environment variables", null)); }).flatMap(cred -> cred.getToken(request)); }
Source:
<code>public void update(long value) { histogram.update(value); <START> Snapshot values = histogram.getSnapshot(); <END> percentiles.forEach(x -> updateGaugeValues(x)); }</code><technical_language>this line is unnecessary? "values" variable is used</technical_language>

Target:
public void update(long value) { histogram.update(value); }
Source:
<code><START> public Object getEventObject() { <END> return serializationService.toObject(key); }</code><technical_language>optimize this part? deserialize key for call</technical_language>

Target:
public Object getEventObject() { if (key == null) { key = serializationService.toObject(dataKey); } return key; }
Source:
<code>public Map<AccessCategory.Id, AccessCategory> get() { Map<AccessCategory.Id, AccessCategory> accessCategories = new HashMap<AccessCategory.Id, AccessCategory>(); try { final ReviewDb db = schema.open(); try { for (AccessCategory ac : db.accessCategories().all()) { accessCategories.put(ac.getId(), ac); } } finally { db.close(); } } catch (OrmException e) { throw new ProvisionException("Cannot query approval categories", e); } <START> return accessCategories; <END> }</code><technical_language>This needs unmodifiableMap prevent mutating it</technical_language>

Target:
public Map<AccessCategory.Id, AccessCategory> get() { Map<AccessCategory.Id, AccessCategory> accessCategories = new HashMap<AccessCategory.Id, AccessCategory>(); try { final ReviewDb db = schema.open(); try { for (AccessCategory ac : db.accessCategories().all()) { accessCategories.put(ac.getId(), ac); } } finally { db.close(); } } catch (OrmException e) { throw new ProvisionException("Cannot query approval access categories", e); } return Collections.unmodifiableMap(accessCategories); }
Source:
<code><START> public void endAddressScan() throws Exception { for (MirrorController partition : partitions) { partition.startAddressScan(); <END> } }</code><technical_language>calls partition questionable</technical_language>

Target:
public void endAddressScan() throws Exception { for (MirrorController partition : partitions) { partition.endAddressScan(); } }
Source:
<code>public InterceptorChain resolveParameterValue(Message<?> message) { <START> InterceptorChain tl = CURRENT.get(); <END> if (tl != null) { return tl; } return CurrentUnitOfWork.map(uow -> (InterceptorChain) uow.getResource(INTERCEPTOR_CHAIN_EMITTER_KEY)) .orElseThrow(() -> new IllegalStateException( "InterceptorChain should have been injected")); }</code><technical_language>"tl" stand for? Thread Local chance</technical_language>

Target:
public InterceptorChain resolveParameterValue(Message<?> message) { InterceptorChain interceptorChain = CURRENT.get(); if (interceptorChain != null) { return interceptorChain; } return CurrentUnitOfWork.map(uow -> (InterceptorChain) uow.getResource(INTERCEPTOR_CHAIN_EMITTER_KEY)) .orElseThrow(() -> new IllegalStateException( "InterceptorChain should have been injected")); }
Source:
<code>public BigInteger decodeInteger() throws ASN1Exception { <START> return new BigInteger(drainElementValue()); <END> }</code><technical_language>I need a check next tag is INTEGER_TYPE</technical_language>

Target:
public BigInteger decodeInteger() throws ASN1Exception { if (INTEGER_TYPE != peekType()) { throw log.asnUnexpectedTag(); } return new BigInteger(drainElementValue()); }
Source:
<code>public boolean supports(final RepositoryPath repositoryPath) { if (repositoryPath == null) { return false; } <START> boolean isNotMetadata = ThrowingFunction.unchecked(RepositoryFiles::isMetadata).apply(repositoryPath); <END> if (!isNotMetadata) { return false; } Repository repository = repositoryPath.getRepository(); return ((ImmutableRepository)repository).getRemoteRepository() != null; }</code><technical_language>boolean isNotMetadata returns isMetadata</technical_language>

Target:
public boolean supports(final RepositoryPath repositoryPath) throws IOException { if (repositoryPath == null) { return false; } if (!RepositoryFiles.isMetadata(repositoryPath)) { return false; } Repository repository = repositoryPath.getRepository(); return ((RepositoryData)repository).getRemoteRepository() != null; }
Source:
<code>public ConfigurationClientBuilder endpoint(String endpoint) { try { this.endpoint = new URL(endpoint); } catch (MalformedURLException ex) { <START> throw new IllegalArgumentException("'endpoint' must be a valid URL"); <END> } return this; }</code><technical_language>Better, logger.logAndThrow(new ...)</technical_language>

Target:
public ConfigurationClientBuilder endpoint(String endpoint) { try { this.endpoint = new URL(endpoint); } catch (MalformedURLException ex) { logger.logAndThrow(new IllegalArgumentException("'endpoint' must be a valid URL")); } return this; }
Source:
<code>private void processWaitEvents(boolean finalize) { Iterator<Map.Entry<JoinEvent<K,T>, Future<List>>> waitIterator = waitingEvents.entrySet().iterator(); while (waitIterator.hasNext()) { Map.Entry<JoinEvent<K,T>, Future<List>> waitingEvent = waitIterator.next(); Future<List> future = waitingEvent.getValue(); if (!finalize) { <START> break; <END> } if (future.isDone() || finalize) { try { JoinEvent<K,T> event = waitingEvent.getKey(); joinStream(event.value,event.isStream1Data,future.get()); } catch (InterruptedException | ExecutionException e) { throw new RuntimeException("end window", e); } waitIterator.remove(); } } }</code><technical_language>if (! finalize) Check if future is done. If process break. If not, break. if (finalize) process irrespective of future is done not. In this case this is breaking out of loop anything..</technical_language>

Target:
private void processWaitEvents(boolean finalize) { Iterator<Map.Entry<JoinEvent<K,T>, Future<List>>> waitIterator = waitingEvents.entrySet().iterator(); while (waitIterator.hasNext()) { Map.Entry<JoinEvent<K,T>, Future<List>> waitingEvent = waitIterator.next(); Future<List> future = waitingEvent.getValue(); if (future.isDone() || finalize) { try { JoinEvent<K,T> event = waitingEvent.getKey(); joinStream(event.value,event.isStream1Data,future.get()); } catch (InterruptedException | ExecutionException e) { throw new RuntimeException("end window", e); } waitIterator.remove(); if (!finalize) { break; } } } }
Source:
<code>public void process(ObjectType type, ObjectField field, Sortable annotation) { ToolUi ui = field.as(ToolUi.class); ui.setSortable(annotation.value()); <START> Collections.addAll(ui.getSortOperators(), annotation.sortOperators()); <END> }</code><technical_language>Prefer setter relying result of getter modifiable</technical_language>

Target:
public void process(ObjectType type, ObjectField field, Sortable annotation) { ToolUi ui = field.as(ToolUi.class); ui.setSortable(annotation.value()); ui.setSortOperators(new LinkedHashSet<>(Arrays.asList(annotation.sortOperators()))); }
Source:
<code>public HttpRequestEndPoint getEndPointCourseStructure(final String courseId){ return new HttpRequestEndPoint() { public String getUrl() { try { PrefManager pref = new PrefManager(MainApplication.instance(), PrefManager.Pref.LOGIN); String username = URLEncoder.encode(pref.getCurrentUserProfile().username, "UTF-8"); String block_count = URLEncoder.encode("video", "UTF-8"); String block_fields = URLEncoder.encode("graded,format,student_view_multi_device", "UTF-8"); <START> String block_json = URLEncoder.encode("video", "UTF-8"); <END> String cId = URLEncoder.encode(courseId, "UTF-8"); String url = config.getApiHostURL() + "/api/courses/v1/blocks/?course_id=" + cId + "&user=" + username + "&depth=all&requested_fields=" + block_fields + "&student_view_data=" + block_json + "&block_counts=" + block_count+ "&nav_depth=3"; logger.debug("GET url for enrolling in a Course: " + url); return url; } catch (UnsupportedEncodingException e) { logger.error(e); } return ""; } public String getCacheKey() { return config.getApiHostURL() + "/api/courses/v1/blocks/?course_id=" + courseId; } public Map<String, String> getParameters() { return null; } }; }</code><technical_language>I variable names changed field names now</technical_language>

Target:
public HttpRequestEndPoint getEndPointCourseStructure(final String courseId){ return new HttpRequestEndPoint() { public String getUrl() { try { PrefManager pref = new PrefManager(MainApplication.instance(), PrefManager.Pref.LOGIN); String username = URLEncoder.encode(pref.getCurrentUserProfile().username, "UTF-8"); String block_counts = URLEncoder.encode("video", "UTF-8"); String requested_fields = URLEncoder.encode("graded,format,student_view_multi_device", "UTF-8"); String student_view_data = URLEncoder.encode("video", "UTF-8"); String cId = URLEncoder.encode(courseId, "UTF-8"); String url = config.getApiHostURL() + "/api/courses/v1/blocks/?course_id=" + cId + "&user=" + username + "&depth=all&requested_fields=" + requested_fields + "&student_view_data=" + student_view_data + "&block_counts=" + block_counts+ "&nav_depth=3"; logger.debug("GET url for enrolling in a Course: " + url); return url; } catch (UnsupportedEncodingException e) { logger.error(e); } return ""; } public String getCacheKey() { return config.getApiHostURL() + "/api/courses/v1/blocks/?course_id=" + courseId; } public Map<String, String> getParameters() { return null; } }; }
Source:
<code>private void handleExposeMinMaxValues(Context context,DiscoverySearchFilterFacet field, SearchFacetEntryRest facetEntry) { try { String minValue = searchService.calculateExtremeValue(context, field.getIndexFieldName() + "_min", field.getIndexFieldName() + "_min_sort", DiscoverQuery.SORT_ORDER.asc); String maxValue = searchService.calculateExtremeValue(context, field.getIndexFieldName() + "_max", field.getIndexFieldName() + "_max_sort", DiscoverQuery.SORT_ORDER.desc); if (StringUtils.isNotBlank(minValue) && StringUtils.isNotBlank(maxValue)) { facetEntry.setMinValue(minValue); facetEntry.setMaxValue(maxValue); } } catch (SearchServiceException e) { <START> e.printStackTrace(); <END> } }</code><technical_language>out of curiosity, a printStackTrace() calls in this PR...is a reason logging errors (e.g. log.error();)</technical_language>

Target:
private void handleExposeMinMaxValues(Context context,DiscoverySearchFilterFacet field, SearchFacetEntryRest facetEntry) { try { String minValue = searchService.calculateExtremeValue(context, field.getIndexFieldName() + "_min", field.getIndexFieldName() + "_min_sort", DiscoverQuery.SORT_ORDER.asc); String maxValue = searchService.calculateExtremeValue(context, field.getIndexFieldName() + "_max", field.getIndexFieldName() + "_max_sort", DiscoverQuery.SORT_ORDER.desc); if (StringUtils.isNotBlank(minValue) && StringUtils.isNotBlank(maxValue)) { facetEntry.setMinValue(minValue); facetEntry.setMaxValue(maxValue); } } catch (SearchServiceException e) { log.error(e.getMessage(), e); } }
Source:
<code>protected boolean connectLunDisks(Guid hostId) { if (getVm().getDiskMap().isEmpty()) { vmHandler.updateDisksFromDb(getVm()); } List<LunDisk> lunDisks = DisksFilter.filterLunDisks(getVm().getDiskMap().values()); Set<String> connections = new HashSet<>(); for (LunDisk lunDisk : lunDisks) { LUNs lun = lunDisk.getLun(); lun.setLunConnections(new ArrayList<>(storageServerConnectionDao.getAllForLun(lun.getLUNId()))); if (connections.containsAll(lun.getLunConnections().stream().map(c -> <START> c.getConnection()).collect(Collectors.toList()))) <END> { continue; } if (lun.getLunConnections().isEmpty()) { return false; } if (!connect(hostId, lun)) { log.info("Failed to connect a lun disk to vdsm '{}' skipping it", hostId); return false; } connections.addAll(lun.getLunConnections().stream().map(c -> c.getConnection()).collect(Collectors.toList())); } return true; }</code><technical_language>getId</technical_language>

Target:
protected boolean connectLunDisks(Guid hostId) { if (getVm().getDiskMap().isEmpty()) { vmHandler.updateDisksFromDb(getVm()); } List<LunDisk> lunDisks = DisksFilter.filterLunDisks(getVm().getDiskMap().values()); Set<String> connections = new HashSet<>(); for (LunDisk lunDisk : lunDisks) { LUNs lun = lunDisk.getLun(); lun.setLunConnections(new ArrayList<>(storageServerConnectionDao.getAllForLun(lun.getLUNId()))); if (connections.containsAll( lun.getLunConnections() .stream() .map(c -> c.getId()) .collect(Collectors.toList()))) { continue; } if (lun.getLunConnections().isEmpty()) { return false; } if (!connect(hostId, lun)) { log.info("Failed to connect a lun disk to vdsm '{}' skipping it", hostId); return false; } connections.addAll(lun.getLunConnections() .stream() .map(c -> c.getId()) .collect(Collectors.toList())); } return true; }
Source:
<code>static double[] calculateRowVariances(final RealMatrix m) { Utils.nonNull(m, "Cannot calculate variances for a null matrix."); final StandardDeviation std = new StandardDeviation(); <START> final double[] result = new double[m.getRowDimension()]; <END> for (int i = 0; i < m.getRowDimension(); i++) { final double row = m.getRow(i); result[i] = Math.pow(std.evaluate(row), 2); } return result; }</code><technical_language>a stream</technical_language>

Target:
static double[] calculateRowVariances(final RealMatrix m) { Utils.nonNull(m, "Cannot calculate variances for a null matrix."); return MatrixSummaryUtils.getRowVariances(m); }
Source:
<code>public File getMobileSlotScreenshot(WebElement element, WebDriver driver) { Shooter shooter = new Shooter(); File page = shooter.capturePage(driver); <START> BufferedImage scaledPage = shooter.scaleImage(page, chromeDriverScreenshotScale, chromeDriverScreenshotScale); <END> File slotScreenshot = shooter.cropImage(element.getLocation(), element.getSize(), scaledPage); return slotScreenshot; }</code><technical_language>shorter line</technical_language>

Target:
public File getMobileSlotScreenshot(WebElement element, WebDriver driver) { ImageEditor imageEditor = new ImageEditor(); Shooter shooter = new Shooter(); File page = shooter.capturePage(driver); BufferedImage scaledPage = imageEditor.scaleImage( page, chromeDriverScreenshotScale, chromeDriverScreenshotScale ); File slotScreenshot = imageEditor.cropImage(element.getLocation(), element.getSize(), scaledPage); return slotScreenshot; }
Source:
<code>public Query build() { <START> Checks.checkNotEmpty(table, "Please specify table name"); <END> return new Query( distinct, table, columns, where, whereArgs, groupBy, having, orderBy, limit ); }</code><technical_language>Static import</technical_language>

Target:
public Query build() { checkNotEmpty(table, "Please specify table name"); return new Query( distinct, table, columns, where, whereArgs, groupBy, having, orderBy, limit ); }
Source:
<code>public void postToInflux(String payload) { HttpPost post = new HttpPost( "http://" + influxdbEmitterConfig.getHostname() + ":" + influxdbEmitterConfig.getPort() + "/write?db=" + influxdbEmitterConfig.getDatabaseName() + "&u=" + influxdbEmitterConfig.getInfluxdbUserName() + "&p=" + influxdbEmitterConfig.getInfluxdbPassword() ); post.setEntity(new StringEntity(payload, ContentType.DEFAULT_TEXT)); post.setHeader("Content-Type", "application/x-www-form-urlencoded"); try { influxdbClient.execute(post); } catch (IOException ex) { <START> log.info(ex.toString()); <END> } finally { post.releaseConnection(); } }</code><technical_language>this swallows stacktrace. suggestion log.info(ex, "Failed post events InfluxDB.");</technical_language>

Target:
public void postToInflux(String payload) { HttpPost post = new HttpPost( "http://" + influxdbEmitterConfig.getHostname() + ":" + influxdbEmitterConfig.getPort() + "/write?db=" + influxdbEmitterConfig.getDatabaseName() + "&u=" + influxdbEmitterConfig.getInfluxdbUserName() + "&p=" + influxdbEmitterConfig.getInfluxdbPassword() ); post.setEntity(new StringEntity(payload, ContentType.DEFAULT_TEXT)); post.setHeader("Content-Type", "application/x-www-form-urlencoded"); try { influxdbClient.execute(post); } catch (IOException ex) { log.info(ex, "Failed to post events to InfluxDB."); } finally { post.releaseConnection(); } }
Source:
<code>public void testDoubleSequence() throws Exception { <START> tester.testRoundTrip(DOUBLE, doubleSequence(0, 0.1, 30_000, false)); <END> tester.testRoundTrip(DOUBLE, doubleSequence(0, 0.1, 30_000, true)); List<Map<Integer, TupleDomainFilter>> filters = ImmutableList.of( ImmutableMap.of(0, DoubleRange.of(0, false, false, Double.MAX_VALUE / 2000, false, false, false)), ImmutableMap.of(0, DoubleRange.of(-1, true, false, 1, false, false, false)), ImmutableMap.of(0, DoubleRange.of(-1, true, true, 2, false, false, false)), ImmutableMap.of(0, DoubleRange.of(1, true, true, 2, true, false, false)), ImmutableMap.of(0, DoubleRange.of(1, false, false, 4, false, true, true)), ImmutableMap.of(0, DoubleRange.of(0, false, false, 40, false, false, true))); tester.testRoundTrip(DOUBLE, doubleSequence(0, 0.1, 30_000, false), filters); tester.testRoundTrip(DOUBLE, doubleSequence(0, 0.1, 30_000, true), filters); }</code><technical_language>this test is redundant; is included in test L232 call test filter filter cases: if (useSelectiveOrcReader) { assertFileContentsPresto(types, tempFile, expectedValues, orcEncoding, orcPredicate, Optional.empty()); for (Map<Integer, TupleDomainFilter> columnFilters : filters) { assertFileContentsPresto(types, tempFile, filterRows(types, expectedValues, columnFilters), orcEncoding, orcPredicate, Optional.of(columnFilters)); } return; }</technical_language>

Target:
public void testDoubleSequence() throws Exception { List<Map<Integer, TupleDomainFilter>> filters = ImmutableList.of( ImmutableMap.of(0, DoubleRange.of(0, false, false, 1_000, false, false, false)), ImmutableMap.of(0, IS_NULL), ImmutableMap.of(0, IS_NOT_NULL)); tester.testRoundTrip(DOUBLE, doubleSequence(0, 0.1, 30_000), filters); tester.testRoundTripTypes( ImmutableList.of(DOUBLE, DOUBLE), ImmutableList.of( doubleSequence(0, 0.1, 10_000), doubleSequence(0, 0.1, 10_000)), ImmutableList.of( ImmutableMap.of( 0, DoubleRange.of(1.0, false, true, 7.0, false, true, true), 1, DoubleRange.of(3.0, false, true, 9.0, false, true, false)))); }
Source:
<code><START> public static boolean waitProcess(int pid, long poll, long timeout) { <END> boolean exists = false; try { final long startTime = System.currentTimeMillis(); long now; do { Thread.sleep(poll); exists = (getPid(pid) >= 0) ? true : false; now = System.currentTimeMillis(); } while (exists && (now - startTime) < timeout); } catch (Exception e) { Thread.currentThread().interrupt(); s_logger.warn("Failed waiting for pid to exit - {}", pid, e); } return exists; }</code><technical_language>@ibinshtok is difference this waitProcess in PppLinux</technical_language>

Target:
private static boolean waitProcess(int pid, long poll, long timeout) { boolean exists = false; try { final long startTime = System.currentTimeMillis(); long now; do { Thread.sleep(poll); exists = isProcessRunning(pid); now = System.currentTimeMillis(); } while (exists && now - startTime < timeout); } catch (Exception e) { Thread.currentThread().interrupt(); logger.warn("Failed waiting for pid to exit - {}", pid, e); } return exists; }
Source:
<code>private boolean syncFlags(LocalMessage localMessage, Message remoteMessage) throws MessagingException { boolean messageChanged = false; <START> if (localMessage == null || localMessage.isSet(Flag.DELETED)) { <END> return false; } if (remoteMessage.isSet(Flag.DELETED)) { if (localMessage.getFolder().syncRemoteDeletions()) { localMessage.setFlag(Flag.DELETED, true); messageChanged = true; } } else { for (Flag flag : SYNC_FLAGS) { if (remoteMessage.isSet(flag) != localMessage.isSet(flag)) { localMessage.setFlag(flag, remoteMessage.isSet(flag)); messageChanged = true; } } } return messageChanged; }</code><technical_language>this condition</technical_language>

Target:
private boolean syncFlags(LocalMessage localMessage, Message remoteMessage) throws MessagingException { boolean messageChanged = false; if (localMessage == null) { return false; } if (remoteMessage.isSet(Flag.DELETED)) { if (localMessage.getFolder().syncRemoteDeletions()) { localMessage.setFlag(Flag.DELETED, true); messageChanged = true; } } else { for (Flag flag : SYNC_FLAGS) { if (remoteMessage.isSet(flag) != localMessage.isSet(flag)) { localMessage.setFlag(flag, remoteMessage.isSet(flag)); messageChanged = true; } } } return messageChanged; }
Source:
<code>public void init() { if (fileBackup != null) { for (FileBackup.SavedBuffer savedBuffer : fileBackup.getSavedFiles()) { savedBuffer.open(new FileBackup.SavedBuffer.Callback() { @Override public void process(List<String> params, FileChannel channel) { <START> LOG.debug("Loading buffer: params={}, buffer={}", params, channel); <END> loadBufferFromFile(params, channel); } }); } } }</code><technical_language>This log message is a bit important appears file backup mode is enables, I want leave info</technical_language>

Target:
public void init() { if (fileBackup != null) { for (FileBackup.SavedBuffer savedBuffer : fileBackup.getSavedFiles()) { savedBuffer.open(new FileBackup.SavedBuffer.Callback() { @Override public void process(List<String> params, FileChannel channel) { LOG.info("Loading buffer: params={}, buffer={}", params, channel); loadBufferFromFile(params, channel); } }); } } }
Source:
<code>public static @Nullable ITmfStateInterval queryUntilNonNullValue(ITmfStateSystem ss, int attributeQuark, long t1, long t2) { if (t2 < t1) { throw new TimeRangeException(ss.getSSID() + " Start:" + t1 + ", End:" + t2); } Iterator<ITmfStateInterval> iter = new QuarkIterator(ss, attributeQuark, t1); while (iter.hasNext()) { ITmfStateInterval interval = iter.next(); if (interval.getStartTime() > t2) { return null; } if (!interval.getStateValue().isNull()) { return interval; <START> } <END> } return null; }</code><technical_language>Iterators.tryFind(interval -> (!interval.getStateValue().isNull() && interval.getStartTime() <= t2)).or(null) do that</technical_language>

Target:
public static @Nullable ITmfStateInterval queryUntilNonNullValue(ITmfStateSystem ss, int attributeQuark, long t1, long t2) { if (t2 < t1) { throw new TimeRangeException(ss.getSSID() + " Start:" + t1 + ", End:" + t2); } Iterator<ITmfStateInterval> iter = new QuarkIterator(ss, attributeQuark, t1); return Iterators .tryFind(iter, interval -> (!interval.getStateValue().isNull() && interval.getStartTime() <= t2)) .orNull(); }
Source:
<code>private void addToolbarActions(IToolBarManager toolbarManager) { Action collapseAction = new Action(UIText.UIUtils_CollapseAll, PlatformUI.getWorkbench().getSharedImages().getImageDescriptor( ISharedImages.IMG_ELCL_COLLAPSEALL)) { @Override public void run() { getTreeViewer().collapseAll(); } }; <START> toolbarManager.add(collapseAction); <END> }</code><technical_language>Add collapseHandler = new CollapseAllHandler(getTreeViewer()); IHandlerService handlerService = getSite().getService(IHandlerService.class); handlerService.activateHandler(CollapseAllHandler.COMMAND_ID, collapseHandler); handle standard key binding for collapse command</technical_language>

Target:
private void addToolbarActions(IToolBarManager toolbarManager) { Action collapseAction = new Action(UIText.UIUtils_CollapseAll, PlatformUI.getWorkbench().getSharedImages().getImageDescriptor( ISharedImages.IMG_ELCL_COLLAPSEALL)) { @Override public void run() { getTreeViewer().collapseAll(); } }; collapseAction.setActionDefinitionId(CollapseAllHandler.COMMAND_ID); collapseHandler = new CollapseAllHandler(getTreeViewer()); IHandlerService handlerService = getSite() .getService(IHandlerService.class); handlerService.activateHandler(CollapseAllHandler.COMMAND_ID, collapseHandler); toolbarManager.add(collapseAction); }
Source:
<code>public FormValidation doCheckRemoteFS(@QueryParameter String value) throws IOException, ServletException { if(Util.fixEmptyAndTrim(value)==null) return FormValidation.error(Messages.Slave_Remote_Director_Mandatory()); if(value.startsWith("\\\\") || value.startsWith("/net/")) return FormValidation.warning(Messages.Slave_Network_Mounted_File_System_Warning()); if (!value.contains("\\") && !value.startsWith("/")) { <START> return FormValidation.error(Messages.Slave_Remote_Relative_Path_Warning()); <END> } return FormValidation.ok(); }</code><technical_language>warning</technical_language>

Target:
public FormValidation doCheckRemoteFS(@QueryParameter String value) throws IOException, ServletException { if(Util.fixEmptyAndTrim(value)==null) return FormValidation.error(Messages.Slave_Remote_Director_Mandatory()); if(value.startsWith("\\\\") || value.startsWith("/net/")) return FormValidation.warning(Messages.Slave_Network_Mounted_File_System_Warning()); if (Util.isRelativePath(value)) { return FormValidation.warning(Messages.Slave_Remote_Relative_Path_Warning()); } return FormValidation.ok(); }
Source:
<code>public static void writeToDisk(boolean generatePackagesStructure, String outputPath, String relativeFileName, ClassFile classFile) throws IOException { FileOutputStream file = getFileOutputStream(generatePackagesStructure, outputPath, relativeFileName); try (BufferedOutputStream output = new BufferedOutputStream(file, DEFAULT_WRITING_SIZE)) { output.write(classFile.header, 0, classFile.headerOffset); output.write(classFile.contents, 0, classFile.contentsOffset); output.flush(); <START> } catch(IOException e) { <END> throw e; } }</code><technical_language>it: catch(e) throw e is unnecessary here</technical_language>

Target:
public static void writeToDisk(boolean generatePackagesStructure, String outputPath, String relativeFileName, ClassFile classFile) throws IOException { FileOutputStream file = getFileOutputStream(generatePackagesStructure, outputPath, relativeFileName); try (BufferedOutputStream output = new BufferedOutputStream(file, DEFAULT_WRITING_SIZE)) { output.write(classFile.header, 0, classFile.headerOffset); output.write(classFile.contents, 0, classFile.contentsOffset); output.flush(); } }
Source:
<code>@Test public void dragNDropByCoordinatesAndDurationTest() throws Exception { Activity activity = new Activity("io.appium.android.apis", ".view.DragAndDropDemo"); driver.startActivity(activity); AndroidElement dragDot1 = driver.findElement(By.id("io.appium.android.apis:id/drag_dot_1")); AndroidElement dragDot3 = driver.findElement(By.id("io.appium.android.apis:id/drag_dot_3")); WebElement dragText = driver.findElement(By.id("io.appium.android.apis:id/drag_text")); assertEquals("Drag text not empty", "", dragText.getText()); Point center1 = dragDot1.getCenter(); Point center2 = dragDot3.getCenter(); TouchAction dragNDrop = new TouchAction(driver) .longPress(longPressOptions() .withPosition(coordinates(center1.x, center1.y)) .withDuration(ofSeconds(2))) .moveTo(position() <START> .withPosition(coordinates(center2.x, center2.y))) <END> .release(); dragNDrop.perform(); assertNotEquals("Drag text empty", "", dragText.getText()); }</code><technical_language>this expression visually helps understand, this position is calculated previous element previous is absolute</technical_language>

Target:
@Test public void dragNDropByCoordinatesAndDurationTest() throws Exception { Activity activity = new Activity("io.appium.android.apis", ".view.DragAndDropDemo"); driver.startActivity(activity); AndroidElement dragDot1 = driver.findElement(By.id("io.appium.android.apis:id/drag_dot_1")); AndroidElement dragDot3 = driver.findElement(By.id("io.appium.android.apis:id/drag_dot_3")); WebElement dragText = driver.findElement(By.id("io.appium.android.apis:id/drag_text")); assertEquals("Drag text not empty", "", dragText.getText()); Point center1 = dragDot1.getCenter(); Point center2 = dragDot3.getCenter(); TouchAction dragNDrop = new TouchAction(driver) .longPress(longPressOptions() .withPosition(point(center1.x, center1.y)) .withDuration(ofSeconds(2))) .moveTo(point(center2.x, center2.y)) .release(); dragNDrop.perform(); assertNotEquals("Drag text empty", "", dragText.getText()); }
Source:
<code>public void onCancel() { <START> errorCallback.onError(new RequestCanceledException("Canceled request")); <END> }</code><technical_language>need a message parameter for RequestCanceledException ? (same for sender server side)</technical_language>

Target:
public void onCancel() { errorCallback.onError(new RequestCanceledException()); }
Source:
<code>public void startConflictResolution(ConsistentHash conflictCH, List<Address> mergeMembers) { topology = new CacheTopology(topology.getTopologyId() + 1, topology.getRebalanceId() + 1, <START> conflictCH, conflictCH, conflictCH, <END> CacheTopology.Phase.CONFLICT_RESOLUTION, mergeMembers, persistentUUIDs(mergeMembers)); }</code><technical_language>Ok, removing union CH work, I conflict CH for reading, set currentCH change CacheTopology for CONFLICT_RESOLUTION</technical_language>

Target:
public void startConflictResolution(ConsistentHash conflictCH, List<Address> mergeMembers) { topology = new CacheTopology(topology.getTopologyId() + 1, topology.getRebalanceId() + 1, conflictCH, null, CacheTopology.Phase.CONFLICT_RESOLUTION, mergeMembers, persistentUUIDs(mergeMembers)); }
Source:
<code>public <T2 extends ModelClass, U2 extends ModelClass> boolean matches(Message<T2, U2> expected) { if (expected == this) { return true; } if (expected == null) { return false; } if (fromActor.isPresent() && !fromActor.equals(expected.fromActor)) { return false; } if (sender != null && expected.sender != null && expected.sender.hasParticipant() && sender.getParticipant().get() != expected.sender.getParticipant().get()) { return false; } if (target != null && expected.target != null && expected.target.hasParticipant() && target.getParticipant().get() != expected.target.getParticipant().get()) { return false; } boolean areSignalsEqual; try { areSignalsEqual = BasedOnFields.equal(getWrapped(), expected.getWrapped()); } catch (IllegalAccessException e) { return false; } if (areSignalsEqual) { if (sender != null && expected.sender != null && !expected.sender.hasParticipant()) { <START> expected.sender.bindTo(sender.getParticipant().get()); <END> } if (!expected.target.hasParticipant()) { expected.target.bindTo(target.getParticipant().get()); } } return areSignalsEqual; }</code><technical_language>I check classes of senders binding, in this case error</technical_language>

Target:
public <T2 extends ModelClass, U2 extends ModelClass> boolean matches(Message<T2, U2> expected) { if (expected == this) { return true; } if (expected == null) { return false; } if (fromActor.isPresent() && !fromActor.equals(expected.fromActor)) { return false; } if (sender != null && expected.sender != null && expected.sender.hasParticipant() && sender.getParticipant().get() != expected.sender.getParticipant().get()) { return false; } if (target != null && expected.target != null && expected.target.hasParticipant() && target.getParticipant().get() != expected.target.getParticipant().get()) { return false; } boolean areSignalsEqual; try { areSignalsEqual = BasedOnFields.equal(getWrapped(), expected.getWrapped()); } catch (IllegalAccessException e) { return false; } if (areSignalsEqual) { if (sender != null && expected.sender != null && !expected.sender.hasParticipant() && sender.getParticipantType().equals(expected.sender.getParticipantType())) { expected.sender.bindTo(sender.getParticipant().get()); } if (!expected.target.hasParticipant() && target.getParticipantType().equals(expected.target.getParticipantType())) { expected.target.bindTo(target.getParticipant().get()); } } return areSignalsEqual; }
Source:
<code>public void <START> getSupportedOutputSizes_ImageCapture_withDefaultConfig() <END> { setupCamera(CameraCharacteristics.INFO_SUPPORTED_HARDWARE_LEVEL_LEGACY); SupportedSurfaceCombination supportedSurfaceCombination = new SupportedSurfaceCombination( mContext, CAMERA_ID, mMockCamcorderProfileHelper); ImageCapture imageCapture = new ImageCapture.Builder().build(); List<Size> resultList = supportedSurfaceCombination.getSupportedOutputSizes(imageCapture); List<Size> expectedList = Arrays.asList(new Size[]{ new Size(4032, 3024), new Size(1920, 1440), new Size(1280, 960), new Size(640, 480), new Size(3840, 2160), new Size(1920, 1080), new Size(1280, 720), new Size(1280, 720), new Size(960, 544), new Size(800, 450) }); assertEquals(expectedList, resultList); }</code><technical_language>do test ImageCapture other cases? Do do specific ImageCapture/ImageAnalysis/Preview other targetRotation/targetResolution configurations</technical_language>

Target:
public void getSupportedOutputSizes_ImageCapture_withDefaultConfig() { setupCamera(CameraCharacteristics.INFO_SUPPORTED_HARDWARE_LEVEL_LEGACY); SupportedSurfaceCombination supportedSurfaceCombination = new SupportedSurfaceCombination( mContext, CAMERA_ID, mMockCamcorderProfileHelper); ImageCapture imageCapture = new ImageCapture.Builder().build(); FakeLifecycleOwner fakeLifecycle = new FakeLifecycleOwner(); CameraX.bindToLifecycle(fakeLifecycle, CameraSelector.DEFAULT_BACK_CAMERA, imageCapture); List<Size> resultList = supportedSurfaceCombination.getSupportedOutputSizes(imageCapture); List<Size> expectedList = Arrays.asList(new Size[]{ new Size(4032, 3024), new Size(1920, 1440), new Size(1280, 960), new Size(640, 480), new Size(3840, 2160), new Size(1920, 1080), new Size(1280, 720), new Size(960, 544), new Size(800, 450) }); assertEquals(expectedList, resultList); }
Source:
<code>public EEFSelectLifecycleManager(EEFSelectDescription description, IVariableManager variableManager, IInterpreter interpreter, TransactionalEditingDomain editingDomain) { <START> super(description, variableManager, <END> interpreter, editingDomain); this.description = description; }</code><technical_language>done</technical_language>

Target:
public EEFSelectLifecycleManager(EEFSelectDescription description, IVariableManager variableManager, IInterpreter interpreter, TransactionalEditingDomain editingDomain) { super(variableManager, interpreter, editingDomain); this.description = description; }