auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public boolean equals(Object o) { if (this == o) { return true; } if (o == null || getClass() != o.getClass()) { return false; } LogisticRegressionSpamMessageModel that = (LogisticRegressionSpamMessageModel) o; <START> if (!text.equals(that.text)) { <END> return false; } if (!read.equals(that.read)) { return false; } return spamPrediction.equals(that.spamPrediction); }</code><technical_language>for consistency other classes, protect nulls, Objects.equals() return Objects.equals(text, that.text) && Objects.equals(read, that.read) && ..</technical_language>

Target:
public boolean equals(Object o) { if (this == o) { return true; } if (o == null || getClass() != o.getClass()) { return false; } LogisticRegressionSpamMessageModel that = (LogisticRegressionSpamMessageModel) o; return Objects.equals(text, that.text) && Objects.equals(read, that.read) && Objects.equals(spamPrediction, that.spamPrediction); }
Source:
<code>public void scanCurrentRowStates(Set<ImmutableBytesPtr> rows, Collection<? extends ColumnReference> columns, long ts) throws IOException { Scan s = IndexManagementUtil.newLocalStateScan(Collections.singletonList(columns)); FilterList filterList = new FilterList(FilterList.Operator.MUST_PASS_ONE); for (ImmutableBytesPtr row : rows) { filterList.addFilter(new RowFilter(CompareOperator.EQUAL, new BinaryComparator(row.get()))); } s.setFilter(filterList); s.setTimeRange(0, ts); Region region = this.env.getRegion(); <START> try (RegionScanner scanner = region.getScanner(s)) { <END> boolean more; do { List<Cell> kvs = new ArrayList<Cell>(1); more = scanner.next(kvs); if (kvs.isEmpty()) { return; } Result r = Result.create(kvs); Cell cell = kvs.get(0); byte[] rowKey = new byte[cell.getRowLength()]; System.arraycopy(cell.getRowArray(), cell.getRowOffset(), rowKey, 0, cell.getRowLength()); results.put(new ImmutableBytesPtr(rowKey), r); } while (more); } }</code><technical_language>fyi, In perf issues FilterLists enormous numbers of filters (> 5000)</technical_language>

Target:
public void scanCurrentRowStates(Set<ImmutableBytesPtr> rows, Collection<? extends ColumnReference> columns, long ts) throws IOException { if (results == null) { results = new ConcurrentHashMap<>(); } Scan s = IndexManagementUtil.newLocalStateScan(Collections.singletonList(columns)); List<MultiRowRangeFilter.RowRange> ranges = new ArrayList<>(); for (ImmutableBytesPtr row : rows) { ranges.add(new MultiRowRangeFilter.RowRange(row.get(), true, row.get(), true)); } s.setFilter(new MultiRowRangeFilter(ranges)); s.setTimeRange(0, ts); Region region = this.env.getRegion(); try (RegionScanner scanner = region.getScanner(s)) { boolean more; do { List<Cell> kvs = new ArrayList<Cell>(1); more = scanner.next(kvs); if (kvs.isEmpty()) { return; } Result r = Result.create(kvs); Cell cell = kvs.get(0); byte[] rowKey = new byte[cell.getRowLength()]; System.arraycopy(cell.getRowArray(), cell.getRowOffset(), rowKey, 0, cell.getRowLength()); results.put(new ImmutableBytesPtr(rowKey), r); } while (more); } }
Source:
<code>public void initialize(ResourceManager rm) { try { Datastore ds = new Datastore(new ResourceConfigurator().getDefaultConfig()); String modelFile; String lexFile; if (type.equals("ACE")) { File modelDir = ds.getDirectory("org.cogcomp.re", "ACE_GOLD_BI", 1.0, false); modelFile = modelDir.getPath() + File.separator + "ACE_GOLD_BI" + File.separator + "ACE_GOLD_BI.lc"; lexFile = modelDir.getPath() + File.separator + "ACE_GOLD_BI" + File.separator + "ACE_GOLD_BI.lex"; } else { File modelDir = ds.getDirectory("org.cogcomp.re", "SEMEVAL", 1.1, false); modelFile = modelDir.getPath() + File.separator + "SEMEVAL" + File.separator + "SEMEVAL.lc"; lexFile = modelDir.getPath() + File.separator + "SEMEVAL" + File.separator + "SEMEVAL.lex"; <START> } <END> relationClassifier = new relation_classifier(); relationClassifier.readModel(modelFile); relationClassifier.readLexicon(lexFile); constrainedClassifier = new ACERelationConstrainedClassifier(relationClassifier); File gazetteersResource = ds.getDirectory("org.cogcomp.gazetteers", "gazetteers", 1.6, false); GazetteersFactory.init(5, gazetteersResource.getPath() + File.separator + "gazetteers", true); WordNetManager.loadConfigAsClasspathResource(true); wordNet = WordNetManager.getInstance(); gazetteers = GazetteersFactory.get(); } catch (Exception e) { e.printStackTrace(); } }</code><technical_language>bad, I forgot a classifier. classifier for SemEval is semeval_relation_classifier. need constrainedClassifier anymore, semeval_relation_classifier load model lexicon</technical_language>

Target:
public void initialize(ResourceManager rm) { try { Datastore ds = new Datastore(new ResourceConfigurator().getDefaultConfig()); File modelDir = ds.getDirectory("org.cogcomp.re", "ACE_GOLD_BI", 1.0, false); String modelFile = modelDir.getPath() + File.separator + "ACE_GOLD_BI" + File.separator + "ACE_GOLD_BI.lc"; String lexFile = modelDir.getPath() + File.separator + "ACE_GOLD_BI" + File.separator + "ACE_GOLD_BI.lex"; relationClassifier = new relation_classifier(); relationClassifier.readModel(modelFile); relationClassifier.readLexicon(lexFile); constrainedClassifier = new ACERelationConstrainedClassifier(relationClassifier); File gazetteersResource = ds.getDirectory("org.cogcomp.gazetteers", "gazetteers", 1.6, false); GazetteersFactory.init(5, gazetteersResource.getPath() + File.separator + "gazetteers", true); WordNetManager.loadConfigAsClasspathResource(true); wordNet = WordNetManager.getInstance(); gazetteers = GazetteersFactory.get(); } catch (Exception e) { e.printStackTrace(); } }
Source:
<code>public void fillTo(int size) { if (size <= 0) return; <START> if (!pool.getConfiguration().isPrefill()) <END> return; if (!credential.equals(pool.getPrefillCredential())) return; while (!pool.isFull()) { try { if (pool.getRequestSemaphore() .tryAcquire(pool.getConfiguration().getBlockingTimeout(), TimeUnit.MILLISECONDS)) { try { if (pool.isShutdownInProgress()) { return; } if (listeners.size() >= size) { return; } try { ConnectionListener cl = pool.createConnectionListener(credential); boolean added = false; synchronized (listeners) { if (listeners.size() < size) { listeners.add(cl); added = true; } } if (!added) { pool.destroyConnectionListener(cl); return; } } catch (ResourceException re) { return; } } finally { pool.getRequestSemaphore().release(); } } } catch (InterruptedException ignored) { Thread.interrupted(); } } }</code><technical_language>This</technical_language>

Target:
public void fillTo(int size) { if (size <= 0) return; while (!pool.isFull()) { if (pool.isShutdown()) { return; } if (listeners.size() >= size) { return; } try { ConnectionListener cl = pool.createConnectionListener(credential); boolean added = false; if (listeners.size() < size) { listeners.add(cl); added = true; } if (!added) { pool.destroyConnectionListener(cl); return; } } catch (ResourceException re) { return; } } }
Source:
<code>public void testAdd() throws CoreException { TaskReviewsMappingsStore taskReviewsMappingStore = getEmptyTaskReviewStore(); TaskData reviewData1 = addReviewData(review1, descriptionWithTaskUrl1); TaskContainerDelta delta = new TaskContainerDelta(review1, Kind.ADDED); taskReviewsMappingStore.addTaskAssocation(review1, reviewData1); taskReviewsMappingStore.containersChanged(ImmutableSet.of(delta)); assertTrue(taskReviewsMappingStore.getReviewUrls(taskUrl1).contains(reviewUrl1)); <START> when(review1.getAttribute(TaskReviewsMappingsStore.ATTR_ASSOCIATED_TASK)).thenReturn(taskUrl1); <END> assertTrue(taskReviewsMappingStore.getTaskUrl(review1).equals(taskUrl1)); }</code><technical_language>Fixed addressed in setup</technical_language>

Target:
public void testAdd() throws CoreException { TaskReviewsMappingsStore taskReviewsMappingStore = getEmptyTaskReviewStore(); TaskData reviewData1 = addReviewData(review1, descriptionWithTaskUrl1); TaskContainerDelta delta = new TaskContainerDelta(review1, Kind.ADDED); taskReviewsMappingStore.addTaskAssocation(review1, reviewData1); taskReviewsMappingStore.containersChanged(ImmutableSet.of(delta)); assertTrue(taskReviewsMappingStore.getReviewUrls(taskUrl1).contains(reviewUrl1)); assertTrue(taskReviewsMappingStore.getTaskUrl(review1).equals(taskUrl1)); }
Source:
<code>public String getSystemPort() { <START> return (String) getCapability(AndroidMobileCapabilityType.SYSTEM_PORT); <END> }</code><technical_language>int</technical_language>

Target:
public Integer getSystemPort() { return (Integer) getCapability(AndroidMobileCapabilityType.SYSTEM_PORT); }
Source:
<code>public Resolver<Boolean> getRegisterEnabledResolver() { String key = "stormpath.web.register.enabled.resolver"; if (containsKey(key)) { try { return getInstance(key); } catch (ServletException e) { throw new RuntimeException("Couldn't instantiate stormpath.web.register.enabled.resolver", e); } } else { boolean enabled = CFG.getBoolean("stormpath.web.register.enabled"); RegisterEnabledResolver resolver = <START> new RegisterEnabledResolver(enabled, ApplicationResolver.INSTANCE, getRegisterEnabledPredicate()); <END> SINGLETONS.put(key, resolver); return resolver; } }</code><technical_language>replace ApplicationResolver.INSTANCE getApplicationResolver() if want isolate static usage place</technical_language>

Target:
public Resolver<Boolean> getRegisterEnabledResolver() { String key = "stormpath.web.register.enabled.resolver"; if (containsKey(key)) { try { return getInstance(key); } catch (ServletException e) { throw new RuntimeException("Couldn't instantiate stormpath.web.register.enabled.resolver", e); } } else { boolean enabled = CFG.getBoolean("stormpath.web.register.enabled"); ApplicationResolver appResolver = getApplicationResolver(); BiPredicate<Boolean,Application> regEnabledPredicate = getRegisterEnabledPredicate(); RegisterEnabledResolver resolver = new RegisterEnabledResolver(enabled, appResolver, regEnabledPredicate); SINGLETONS.put(key, resolver); return resolver; } }
Source:
<code>private void map(final RepositoryMapping m) { final IResource r; final File git; final IResource dotGit; IContainer c = null; m.clear(); r = getProject().findMember(m.getContainerPath()); if (r instanceof IContainer) { c = (IContainer) r; } else if (r != null) { c = (IContainer) r.getAdapter(IContainer.class); } if (c == null) { Activator.logError(CoreText.GitProjectData_mappedResourceGone, new FileNotFoundException(m.getContainerPath().toString())); m.clear(); return; } m.setContainer(c); git = c.getLocation().append(m.getGitDirPath()).toFile(); if (!git.isDirectory() || !new File(git, "config").isFile()) { String path = m.getContainerPath().toString(); Activator.logError(MessageFormat.format( CoreText.GitProjectData_mappedResourceGone, path), new FileNotFoundException(path)); m.clear(); return; } try { m.setRepository(Activator.getDefault().getRepositoryCache() .lookupRepository(git)); } catch (IOException ioe) { <START> Activator.logError(CoreText.GitProjectData_mappedResourceGone, <END> new FileNotFoundException(m.getContainerPath().toString())); m.clear(); return; } m.fireRepositoryChanged(); trace("map " + c + " -> " + m.getRepository()); try { c.setSessionProperty(MAPPING_KEY, m); } catch (CoreException err) { Activator.logError( CoreText.GitProjectData_failedToCacheRepoMapping, err); } dotGit = c.findMember(Constants.DOT_GIT); if (dotGit != null && dotGit.getLocation().toFile().equals(git)) { protect(dotGit); } }</code><technical_language>this one</technical_language>

Target:
private void map(final RepositoryMapping m) { final IResource r; final File git; final IResource dotGit; IContainer c = null; m.clear(); r = getProject().findMember(m.getContainerPath()); if (r instanceof IContainer) { c = (IContainer) r; } else if (r != null) { c = (IContainer) r.getAdapter(IContainer.class); } if (c == null) { logGoneMappedResource(m); m.clear(); return; } m.setContainer(c); git = c.getLocation().append(m.getGitDirPath()).toFile(); if (!git.isDirectory() || !new File(git, "config").isFile()) { logGoneMappedResource(m); m.clear(); return; } try { m.setRepository(Activator.getDefault().getRepositoryCache() .lookupRepository(git)); } catch (IOException ioe) { logGoneMappedResource(m); m.clear(); return; } m.fireRepositoryChanged(); trace("map " + c + " -> " + m.getRepository()); try { c.setSessionProperty(MAPPING_KEY, m); } catch (CoreException err) { Activator.logError( CoreText.GitProjectData_failedToCacheRepoMapping, err); } dotGit = c.findMember(Constants.DOT_GIT); if (dotGit != null && dotGit.getLocation().toFile().equals(git)) { protect(dotGit); } }
Source:
<code><START> public PlanNode visitValues(ValuesNode node, RewriteContext<Void> context) <END> { ValuesNode intermediate = new ValuesNode( planNodeIdAllocationStrategy.getId(node), node.getOutputSymbols(), node.getRows().stream() .map(SymbolMapper.this::map) .collect(toImmutableList())); Set<Symbol> added = new HashSet<>(); return pruneValuesNode(intermediate, planNodeIdAllocationStrategy.getId(intermediate), added::add); }</code><technical_language>I good assert newly colliding columns values</technical_language>

Target:
public PlanNode visitValues(ValuesNode node, RewriteContext<Void> context) { ValuesNode intermediate = new ValuesNode( node.getId(), node.getOutputSymbols().stream() .map(SymbolMapper.this::map) .collect(toImmutableList()), node.getRows().stream() .map(SymbolMapper.this::map) .collect(toImmutableList())); Set<Symbol> added = new HashSet<>(); return pruneValuesNode(intermediate, planNodeIdGenerationStrategy.getPlanNodeId(node), added::add); }
Source:
<code>public void setUp() { em = new BasicExecutionManager("mycontext"); ec = new BasicExecutionContext(em); ex = Executors.newCachedThreadPool(); data = Collections.synchronizedMap(new LinkedHashMap<Object,Object>()); <START> data.clear(); <END> cancelledWhileSleeping = new Semaphore(0); }</code><technical_language>data.clear is unnecessary; created map</technical_language>

Target:
public void setUp() { em = new BasicExecutionManager("mycontext"); ec = new BasicExecutionContext(em); ex = Executors.newCachedThreadPool(); data = Collections.synchronizedMap(new LinkedHashMap<Object,Object>()); started = new Semaphore(0); waitInTask = new Semaphore(0); cancelledWhileSleeping = new Semaphore(0); }
Source:
<code>public String toString() { return "PushResponse {" + "active='" + active + '\'' + ", iden='" + iden + '\'' + ", type='" + type + '\'' + ", dismissed=" + dismissed + ", direction='" + direction + '\'' + ", senderIdentifier='" + senderIdentifier + '\'' + ", senderEmail='" + senderEmail + '\'' + ", senderEmailNormalized='" + senderEmailNormalized + '\'' + ", senderName='" + senderName + '\'' + <START> ", receiverIdentifer='" + receiverIdentifer + '\'' + <END> ", receiverEmail='" + receiverEmail + '\'' + ", receiverEmailNormalized='" + receiverEmailNormalized + '\'' + ", title='" + title + '\'' + ", body='" + body + '\'' + ", errorCode='" + errorCode + '\'' + ", pushError=" + pushError + '}'; }</code><technical_language>"receiverIdentifier" nice refactor variable name spelled correctly</technical_language>

Target:
public String toString() { return "PushResponse {" + "active='" + active + '\'' + ", iden='" + iden + '\'' + ", type='" + type + '\'' + ", dismissed=" + dismissed + ", direction='" + direction + '\'' + ", senderIdentifier='" + senderIdentifier + '\'' + ", senderEmail='" + senderEmail + '\'' + ", senderEmailNormalized='" + senderEmailNormalized + '\'' + ", senderName='" + senderName + '\'' + ", receiverIdentifier='" + receiverIdentifier + '\'' + ", receiverEmail='" + receiverEmail + '\'' + ", receiverEmailNormalized='" + receiverEmailNormalized + '\'' + ", title='" + title + '\'' + ", body='" + body + '\'' + ", errorCode='" + errorCode + '\'' + ", pushError=" + pushError + '}'; }
Source:
<code>public void testGetEnumValues() { Ds3EnumConstant alpha = new Ds3EnumConstant("ALPHA", null); <START> Ds3EnumConstant bravo = new Ds3EnumConstant("BRAVO", null); <END> final ImmutableList.Builder<Ds3EnumConstant> builder = ImmutableList.builder(); builder.add(alpha); builder.add(bravo); final ImmutableList<Ds3EnumConstant> enumConstants = builder.build(); final String expectedResult = " ALPHA,\n" + " BRAVO"; final String result = TypeHelper.getEnumValues(enumConstants); assertThat(result, is(expectedResult)); }</code><technical_language>Missing finals for Ds3Elements in this file</technical_language>

Target:
public void testGetEnumValues() { final Ds3EnumConstant alpha = new Ds3EnumConstant("ALPHA", null); final Ds3EnumConstant bravo = new Ds3EnumConstant("BRAVO", null); final ImmutableList.Builder<Ds3EnumConstant> builder = ImmutableList.builder(); builder.add(alpha); builder.add(bravo); final ImmutableList<Ds3EnumConstant> enumConstants = builder.build(); final String expectedResult = " ALPHA,\n" + " BRAVO"; final String result = TypeHelper.getEnumValues(enumConstants); assertThat(result, is(expectedResult)); }
Source:
<code>private String getCoursePaginationList(List<String> coursePaginationList) throws EntityDoesNotExistException { String courseName = ""; List<CourseAttributes> courses = logic.getCoursesForStudentAccount(account.googleId); java.util.Collections.sort(courses); for (int i = 0; i < courses.size(); i++) { CourseAttributes course = courses.get(i); coursePaginationList.add(course.id); <START> if (courseId == "") { <END> courseId = course.id; } if (course.id.equals(courseId)) { courseName = course.id + " : " + course.name; } } if (courseName.equals("")) { throw new EntityDoesNotExistException("Trying to access a course that does not exist."); } return courseName; }</code><technical_language>This is changing previous logic, correct this mistake anyway</technical_language>

Target:
private String getCoursePaginationList(List<String> coursePaginationList) throws EntityDoesNotExistException { String courseName = ""; List<CourseAttributes> courses = logic.getCoursesForStudentAccount(account.googleId); java.util.Collections.sort(courses); for (int i = 0; i < courses.size(); i++) { CourseAttributes course = courses.get(i); coursePaginationList.add(course.id); if (courseId.isEmpty()) { courseId = course.id; } if (course.id.equals(courseId)) { courseName = course.id + " : " + course.name; } } if (courseName.equals("")) { throw new EntityDoesNotExistException("Trying to access a course that does not exist."); } return courseName; }
Source:
<code>public double getVoltageSafely() throws InvalidSensorException { readStatus3(); <START> if (System.currentTimeMillis() - lastRead > PDP.MAX_AGE) { <END> throw new InvalidSensorException("Can not read voltage from PDP"); } return cachedVoltage; }</code><technical_language>this code is duplicated, break out function implement readStatus methods</technical_language>

Target:
public double getVoltageSafely() throws InvalidSensorException { readStatus(3); if (System.currentTimeMillis() - lastRead > PDP.MAX_AGE) { throw new InvalidSensorException("Can not read voltage from PDP"); } return cachedVoltage; }
Source:
<code>public byte[] signData(byte[] inputData) throws EncryptionException { try { PrivateKey privateKey = this.getPrivate(); Signature signer = Signature.getInstance("SHA1withDSA"); signer.initSign(privateKey); signer.update(inputData); return (signer.sign()); <START> } catch (Exception e) { <END> throw new EncryptionException("DSA signing failed", e); } }</code><technical_language>Instead of catching exception please list exceptions catching. do this java7 overloaded catching mechanism</technical_language>

Target:
public byte[] signData(byte[] inputData) throws EncryptionException { try { PrivateKey privateKey = this.getPrivate(); Signature signer = Signature.getInstance("SHA1withDSA"); signer.initSign(privateKey); signer.update(inputData); return (signer.sign()); } catch (NoSuchAlgorithmException | InvalidKeyException | SignatureException e) { throw new EncryptionException("DSA signing failed", e); } }
Source:
<code>public UUID getCoordinator() { return rtState.top.nodesByOrder.values().stream() <START> .filter(n -> !n.isClient()) <END> .map(ZookeeperClusterNode::id) .findFirst() .orElse(null); }</code><technical_language>n -> !n.isClient() && !n.isDaemon()</technical_language>

Target:
public UUID getCoordinator() { return rtState.top.nodesByOrder.values().stream() .filter(n -> !n.isClient() && !n.isDaemon()) .map(ZookeeperClusterNode::id) .findFirst() .orElse(null); }
Source:
<code>public void shouldFilterNonEmptyList() { <START> final List<Integer> list = Collections.unmodifiableList( <END> IntStream.range(0, 4).boxed().collect(Collectors.toList()) ); final List<Integer> actual = ListOps.filter(list, x -> x % 2 > 0); final List<Integer> expected = Arrays.asList(1, 3); assertNotNull(actual); assertFalse(actual.isEmpty()); assertEquals(expected, actual); }</code><technical_language>I is important opportunity show-off Java Streams API can, here; IntStream.range() idiom of those. in this specific case, written, feels a awkward Arrays.asList() for a two-element list upshift IntStream create a four-element list. *could* create a large input list a highly-selective function in filter() yields a short list. For example, a list [1, 21) values evenly divisible 5</technical_language>

Target:
public void shouldFilterNonEmptyList() { Predicate<Integer> predicate = x -> x % 2 > 0; final List<Integer> list = Collections.unmodifiableList( IntStream.range(0, 100).boxed().collect(Collectors.toList()) ); final List<Integer> actual = ListOps.filter(list, predicate); final List<Integer> expected = list.stream() .filter(predicate) .collect(Collectors.toList()); assertNotNull(actual); assertFalse(actual.isEmpty()); assertEquals(expected, actual); }
Source:
<code>public String getComponentType() { <START> return "r2dbc:reactive-inbound-channel-adapter"; <END> }</code><technical_language>Huh? r2dbc is "reactive". need extra prefix this channel adapter is reactive. R2DBC is non-reactive. Please, check R2dbcMessageHandler for similar issue</technical_language>

Target:
public String getComponentType() { return "r2dbc:inbound-channel-adapter"; }
Source:
<code>default <E> DataTransactionResult transform(int x, int y, int z, Key<? extends BaseValue<E>> key, Function<E, E> function) { if (supports(x, y, z, key)) { <START> Optional<E> optional = get(x, y, z, key); <END> if (optional.isPresent()) { return offer(x, y, z, key, function.apply(optional.get())); } } return DataTransactionBuilder.failNoData(); }</code><technical_language>remove final qualifiers</technical_language>

Target:
default <E> DataTransactionResult transform(int x, int y, int z, Key<? extends BaseValue<E>> key, Function<E, E> function) { if (supports(x, y, z, key)) { final Optional<E> optional = get(x, y, z, key); if (optional.isPresent()) { return offer(x, y, z, key, function.apply(optional.get())); } } return DataTransactionBuilder.failNoData(); }
Source:
<code>public void jobStartedNotification(String scriptId, String assignedJobId) { if (jobGraph == null) { jobGraph = PigStats.get().getJobGraph(); } log.info("jobStartedNotification - jobId " + assignedJobId + ", jobGraph:\n" + jobGraph); for (JobStats jobStats : jobGraph) { if (assignedJobId.equals(jobStats.getJobId())) { log.info("jobStartedNotification - scope " + jobStats.getName() + " is jobId " + assignedJobId); DAGNode<PigJob> node = this.dagNodeNameMap.get(jobStats.getName()); if (node == null) { log.warn("jobStartedNotification - unrecognized operator name found (" + jobStats.getName() + ") for jobId " + assignedJobId); } else { node.getJob().setId(assignedJobId); <START> if (jobClient == null) { <END> addMapReduceJobState(node.getJob(), null); } else { addMapReduceJobState(node.getJob(), jobClient); } dagNodeJobIdMap.put(node.getJob().getId(), node); pushEvent(scriptId, new Event.JobStartedEvent(node)); } } } }</code><technical_language>remove if/then null check call addMapReduceJobState(node.getJob(), jobClient); below. method checks for null delegates addMapReduceJobState(node.getJob()); necessary</technical_language>

Target:
public void jobStartedNotification(String scriptId, String assignedJobId) { log.info("jobStartedNotification - jobId " + assignedJobId + ", jobGraph:\n" + getJobGraph()); for (JobStats jobStats : getJobGraph()) { if (assignedJobId.equals(jobStats.getJobId())) { log.info("jobStartedNotification - scope " + jobStats.getName() + " is jobId " + assignedJobId); DAGNode<PigJob> node = this.dagNodeNameMap.get(jobStats.getName()); if (node == null) { log.warn("jobStartedNotification - unrecognized operator name found (" + jobStats.getName() + ") for jobId " + assignedJobId); } else { node.getJob().setId(assignedJobId); addMapReduceJobState(node.getJob()); dagNodeJobIdMap.put(node.getJob().getId(), node); pushEvent(scriptId, new Event.JobStartedEvent(node)); } } } }
Source:
<code>public void initializeOnMaster(ClassLoader loader) throws Exception { super.initializeOnMaster(loader); <START> while (true) { synchronized (lock) { if (!blocking) { return; } lock.wait(10); } } <END> }</code><technical_language>suggest a OneShotLatch here</technical_language>

Target:
public void initializeOnMaster(ClassLoader loader) throws Exception { super.initializeOnMaster(loader); oneShotLatch.await(); }
Source:
<code>static Object[] grouped(Object[] array, int groupSize) { assert array.length > groupSize; final Object[] results = copy(empty(), 1 + ((array.length - 1) / groupSize)); results[0] = copyRange(array, 0, groupSize); for (int start = groupSize, i = 1; start < array.length; i++) { <START> final int nextLength = Math.min(groupSize, array.length - (i * groupSize)); <END> results[i] = copyRange(array, start, start + nextLength); start += nextLength; } return results; }</code><technical_language>Traversable.grouped() behaves this: java [1,2,3,4,5].grouped(2) = [[1,2],[3,4],[5]] [[1,2],[3,4],[5,?]] instead, right? is content of element of subtree? null</technical_language>

Target:
static Object[] grouped(Object[] array, int groupSize) { assert array.length > groupSize; final Object[] results = newInstance(1 + ((array.length - 1) / groupSize)); results[0] = copyRange(array, 0, groupSize); for (int start = groupSize, i = 1; start < array.length; i++) { final int nextLength = Math.min(groupSize, array.length - (i * groupSize)); results[i] = copyRange(array, start, start + nextLength); start += nextLength; } return results; }
Source:
<code>public void passivateAll() throws PersistenceException { if (enabled && !skipOnStop) { long start = timeService.time(); log.passivatingAllEntries(); <START> List<MarshalledEntry> entries = StreamSupport <END> .stream(container.spliterator(), false) .map(e -> marshalledEntryFactory.newMarshalledEntry(e.getKey(), e.getValue(), internalMetadata(e))) .collect(Collectors.toList()); persistenceManager.writeBatchToAllNonTxStores(entries, BOTH, 0); log.passivatedEntries(entries.size(), Util.prettyPrintTime(timeService.timeDuration(start, TimeUnit.MILLISECONDS))); } }</code><technical_language>want do in a batch possibly. If for example heap surely run OOM issues. I if people a cache loader heap. this issue :D</technical_language>

Target:
public void passivateAll() throws PersistenceException { if (enabled && !skipOnStop) { long start = timeService.time(); log.passivatingAllEntries(); int count = container.sizeIncludingExpired(); Iterable<MarshalledEntry> iterable = () -> new IteratorMapper<>(container.iterator(), e -> marshalledEntryFactory.newMarshalledEntry(e.getKey(), e.getValue(), internalMetadata(e))); persistenceManager.writeBatchToAllNonTxStores(iterable, BOTH, 0); log.passivatedEntries(count, Util.prettyPrintTime(timeService.timeDuration(start, TimeUnit.MILLISECONDS))); } }
Source:
<code>public void onError(Throwable e) { <START> if (hasTooManyElements) { <END> return; } child.onError(e); }</code><technical_language>Report plugin</technical_language>

Target:
public void onError(Throwable e) { if (hasTooManyElements) { RxJavaPluginUtils.handleException(e); return; } child.onError(e); }
Source:
<code><START> private FunctionTranslator(Map<FunctionMetadata, MethodHandle> functionMapping) <END> { this.functionMapping = functionMapping; }</code><technical_language>move private functions public</technical_language>

Target:
private FunctionTranslator(Map<FunctionMetadata, MethodHandle> functionMapping) { this.functionMapping = requireNonNull(functionMapping, "functionMapping is null"); }
Source:
<code>public View getView(int position, View convertView, ViewGroup parent) { View row = convertView; MiniSiteHolder holder; if (row == null) { LayoutInflater inflater = mActivity.getLayoutInflater(); row = inflater.inflate(R.layout.mini_site_list_row, parent, false); holder = new MiniSiteHolder(); holder.photosView = (CoverPhotoPagerView) row.findViewById(R.id.cover_photo_pager_view); holder.coverPhotoLabel = (TextView) row.findViewById(R.id.cover_photo_label); holder.topLabel = (TextView) row.findViewById(R.id.top_label); row.setTag(holder); } else { holder = (MiniSiteHolder)row.getTag(); } final MiniSite miniSite = mMiniSites.get(position); holder.photosView.configureWithPhotos(miniSite.getPhotos()); holder.coverPhotoLabel.setText(String.format("%s Field Reports", miniSite.getFieldReportsCount())); <START> holder.topLabel.setText(miniSite.getName()); <END> final TapGestureListener listener = new TapGestureListener(mActivity, mSite, miniSite); final GestureDetectorCompat gesture = new GestureDetectorCompat(mActivity, listener); row.setOnClickListener(new View.OnClickListener() { @Override public void onClick(View v) { mActivity.replaceFragment(MiniSiteFragment.newInstance(mSite, miniSite)); } }); holder.photosView.setOnTouchListener(new View.OnTouchListener() { @Override public boolean onTouch(View v, MotionEvent event) { gesture.onTouchEvent(event); return false; } }); return row; }</code><technical_language>Do need this null check anymore</technical_language>

Target:
public View getView(int position, View convertView, ViewGroup parent) { View row = convertView; MiniSiteHolder holder; if (row == null) { LayoutInflater inflater = mActivity.getLayoutInflater(); row = inflater.inflate(R.layout.mini_site_list_row, parent, false); holder = new MiniSiteHolder(); holder.photosView = (CoverPhotoPagerView) row.findViewById(R.id.cover_photo_pager_view); holder.topLabel = (TextView) row.findViewById(R.id.top_label); row.setTag(holder); } else { holder = (MiniSiteHolder)row.getTag(); } final MiniSite miniSite = mMiniSites.get(position); holder.photosView.configureWithPhotos(miniSite.getPhotos()); holder.topLabel.setText(miniSite.getName()); if (mSite != null) { final TapGestureListener listener = new TapGestureListener(mActivity, mSite, miniSite); final GestureDetectorCompat gesture = new GestureDetectorCompat(mActivity, listener); row.setOnClickListener(new View.OnClickListener() { @Override public void onClick(View v) { mActivity.replaceFragment(MiniSiteFragment.newInstance(mSite, miniSite)); } }); holder.photosView.setOnTouchListener(new View.OnTouchListener() { @Override public boolean onTouch(View v, MotionEvent event) { gesture.onTouchEvent(event); return false; } }); } return row; }
Source:
<code>public void testRobotIsCreatedWithCorrectInitialPositionAndOrientation() { final GridPosition initialGridPosition = new GridPosition(0, 0); final Robot robot = new Robot(initialGridPosition, Orientation.NORTH); <START> assertEquals(robot.getOrientation(), initialOrientation); <END> assertEquals(robot.getGridPosition(), initialGridPosition); }</code><technical_language>need initialOrientation in intitialGridPosition :)</technical_language>

Target:
public void testRobotIsCreatedWithCorrectInitialPositionAndOrientation() { final Orientation initialOrientation = Orientation.NORTH; final GridPosition initialGridPosition = new GridPosition(0, 0); final Robot robot = new Robot(initialGridPosition, initialOrientation); assertEquals(robot.getOrientation(), initialOrientation); assertEquals(robot.getGridPosition(), initialGridPosition); }
Source:
<code>public boolean deleteImage(String id) { String[] fields = VMImageToImage.decodeFieldsFromUniqueId(id); boolean custom = fields.length == 6 ? true : false; if (custom) { String group = fields[1]; String storage = fields[2]; StorageServiceKeys keys = api.getStorageAccountApi(group).getKeys(storage); <START> BlobHelper.deleteContainerIfExists(storage, keys.key1(), "system"); <END> return true; } return false; }</code><technical_language>Is deleteContainerIfExists return some type of status verify container deleted successfully</technical_language>

Target:
public boolean deleteImage(String id) { VMImage image = VMImageToImage.decodeFieldsFromUniqueId(id); if (image.custom()) { StorageServiceKeys keys = api.getStorageAccountApi(image.group()).getKeys(image.storage()); BlobHelper.deleteContainerIfExists(image.storage(), keys.key1(), "system"); return !BlobHelper.customImageExists(image.storage(), keys.key1()); } return false; }
Source:
<code>public static void writeStyle(final InputStream in, final File styleFile) throws IOException { File temporaryFile = File.createTempFile(styleFile.getName(), null); BufferedOutputStream out = new BufferedOutputStream( new FileOutputStream(temporaryFile)); try { IOUtils.copy(in, out); out.flush(); } finally { out.close(); } <START> org.geoserver.data.util.IOUtils.rename(temporaryFile, styleFile); <END> }</code><technical_language>If, for reason, this fails, temporary file left disk, check if worked, if not, delete temp file (and do in other rename in IOUtils I guess)</technical_language>

Target:
public static void writeStyle(final InputStream in, final File styleFile) throws IOException { final File temporaryFile = File.createTempFile(styleFile.getName(), null, styleFile.getParentFile()); BufferedOutputStream out = null; try { out = new BufferedOutputStream(new FileOutputStream(temporaryFile)); IOUtils.copy(in, out); out.flush(); } finally { out.close(); } try { org.geoserver.data.util.IOUtils.rename(temporaryFile, styleFile); } finally { if (temporaryFile.exists()) { temporaryFile.delete(); } } }
Source:
<code>private XStream createXStream() { <START> XStream xStream = new XStream(new DomDriver(CHARSET)); <END> xStream.alias("colorIDSet", ColorIDSet.class); return xStream; }</code><technical_language>Is this behavior ? more specific, is need for a charset ? producing a string</technical_language>

Target:
private XStream createXStream() { XStream xStream = new XStream(); xStream.alias("colorIDSet", ColorIDSet.class); return xStream; }
Source:
<code>private ResolveStepIndexResult resolveStepInPageOccurrence(final PageSteps pageWithSteps, final StepIdentifier stepIdentifier) { <START> if (stepIdentifier.getStepInPageOccurrence() < pageWithSteps.getSteps().size()) { <END> StepDescription stepDescription = pageWithSteps.getSteps().get(stepIdentifier.getStepInPageOccurrence()); String screenshotFileName = stepDescription.getScreenshotFileName(); int index = stepDescription.getIndex(); return ResolveStepIndexResult.requestedIndexFound(index, screenshotFileName); } else { int redirectStepInPageOccurrence = getStepInPageOccurrenceWithMostMatchingLabels(pageWithSteps.getSteps(), stepIdentifier); StepDescription stepDescription = pageWithSteps.getSteps().get(redirectStepInPageOccurrence); int index = stepDescription.getIndex(); StepIdentifier redirectStepIdentifier = StepIdentifier.withDifferentStepInPageOccurrence(stepIdentifier, redirectStepInPageOccurrence); LOGGER.warn("stepInPageOccurrence " + stepIdentifier.getStepInPageOccurrence() + " does not exist in " + stepIdentifier + ". Redirecting to " + redirectStepIdentifier); return ResolveStepIndexResult.otherStepInPageOccurrenceFound(index, redirectStepIdentifier, stepDescription.getScreenshotFileName()); } }</code><technical_language>This if-condition extracted a private method understand kind of case this branch covers</technical_language>

Target:
private ResolveStepIndexResult resolveStepInPageOccurrence(final PageSteps pageWithSteps, final StepIdentifier stepIdentifier) { if (isStepInRangeOfPageWithSteps(pageWithSteps, stepIdentifier)) { StepDescription stepDescription = pageWithSteps.getSteps().get(stepIdentifier.getStepInPageOccurrence()); String screenshotFileName = stepDescription.getScreenshotFileName(); int index = stepDescription.getIndex(); return ResolveStepIndexResult.requestedIndexFound(index, screenshotFileName); } else { int redirectStepInPageOccurrence = getStepInPageOccurrenceWithMostMatchingLabels(pageWithSteps.getSteps(), stepIdentifier); StepDescription stepDescription = pageWithSteps.getSteps().get(redirectStepInPageOccurrence); int index = stepDescription.getIndex(); StepIdentifier redirectStepIdentifier = StepIdentifier.withDifferentStepInPageOccurrence(stepIdentifier, redirectStepInPageOccurrence); LOGGER.warn("stepInPageOccurrence " + stepIdentifier.getStepInPageOccurrence() + " does not exist in " + stepIdentifier + ". Redirecting to " + redirectStepIdentifier); return ResolveStepIndexResult.otherStepInPageOccurrenceFound(index, redirectStepIdentifier, stepDescription.getScreenshotFileName()); } }
Source:
<code>public ShellSpout(String... command) { _command = command; <START> _process = new ShellProcess(new JsonSerializer(), _command); <END> }</code><technical_language>hardcoded JsonSerializer here, a default serializer in defaults.yaml</technical_language>

Target:
public ShellSpout(String... command) { _command = command; }
Source:
<code>public String toString() { <START> return "| |"; <END> }</code><technical_language>sense mention name here</technical_language>

Target:
public String toString() { return getName(); }
Source:
<code>private static FileDialog createNewFileDialog(Shell parent, int style) { String[] desiredPath = fOverridePath; fOverride = false; <START> return new FileDialog(parent, style) { <END> @Override public String open() { return desiredPath != null ? desiredPath[0] : null; } @Override protected void checkSubclass() { } @Override public String getFileName() { return desiredPath != null ? desiredPath[0].substring(desiredPath[0].lastIndexOf(FILE_SEPARATOR) + 1) : ""; } @Override public String[] getFileNames() { return desiredPath != null? desiredPath : new String[0]; } }; }</code><technical_language>some testing: SWT.MULTI is set, selected files in folder. If select /home/user/a.txt /home/user/b.txt then: open()=/home/user/a.txt getFilterPath()=/home/user getFileName()=a.txt getFileNames()=[a.txt, b.txt]</technical_language>

Target:
private static FileDialog createNewFileDialog(Shell parent, int style) { String[] desiredPath = fOverridePath; fOverride = false; return new FileDialog(parent, style) { @Override public String open() { return desiredPath != null ? desiredPath[0] : null; } @Override protected void checkSubclass() { } @Override public String getFileName() { return getFileName(desiredPath[0]); } @Override public String[] getFileNames() { List<String> inStrings = Arrays.asList(desiredPath); List<String> outStrings = new ArrayList<>(); for (String entry : inStrings) { outStrings.add(getFileName(entry)); } return outStrings.toArray(new String[outStrings.size()]); } @Override public String getFilterPath() { return (desiredPath != null) ? new Path(desiredPath[0]).removeLastSegments(1).toString() : ""; } private String getFileName(String path) { return new Path(path).lastSegment(); } }; }
Source:
<code>public Vector<T> slice(int beginIndex, int endIndex) { if ((beginIndex >= endIndex) || (beginIndex >= size()) || isEmpty()) { return empty(); } else if ((beginIndex <= 0) && (endIndex >= length())) { return this; } else { <START> return take(endIndex).drop(beginIndex); <END> } }</code><technical_language>I this is bad idea for {length=1_000_000; beginIndex=300_000; endIndex=600_000} changes big loops executed (but in previous implementation)</technical_language>

Target:
public Vector<T> slice(int beginIndex, int endIndex) { if ((beginIndex >= endIndex) || (beginIndex >= size()) || isEmpty()) { return empty(); } else if ((beginIndex <= 0) && (endIndex >= length())) { return this; } else { final long index = Math.max(beginIndex, 0); final long length = Math.min(endIndex, length()); HashArrayMappedTrie<Integer, T> trie = HashArrayMappedTrie.empty(); for (int i = (int) index; i < length; i++) { trie = trie.put(trie.size(), get(i)); } return wrap(trie); } }
Source:
<code>public Map<ItemStack, ISmeltingRecipe> getRecipes() { if (recipes != null) return recipes; <START> recipes = Maps.newHashMap(); <END> staticView = Maps.transformValues(recipes, ISmeltingRecipe::getGenericOutput); return recipes; }</code><technical_language>I suggest array-backed maps instead map is iterated (itemstack class override hashcode equals)</technical_language>

Target:
public List<ISmeltingRecipe> getRecipes() { if (recipes != null) return recipes; recipes = Lists.newArrayList(); staticView = new VanillaView(); return recipes; }
Source:
<code>public void handleAction(ActionParameters params) throws ActionException { params.requireLoggedInUser(); int datasourceId = params.getRequiredParamInt(StatisticsHelper.PARAM_DATASOURCE_ID); int id = params.getRequiredParamInt(ActionConstants.PARAM_ID); StatisticalIndicator ind = indicatorService.findById(id, params.getUser().getId()); if(ind == null) { throw new ActionDeniedException("Unknown indicator/not the owner: " + id ); } JSONObject selectors = params.getHttpParamAsJSON(StatisticsHelper.PARAM_SELECTORS); if(selectors != null) { int regionset = params.getRequiredParamInt("regionset"); try { indicatorService.deleteIndicatorData(id, regionset, selectors.getInt("year")); } catch (JSONException ex) { <START> throw new ActionParamsException("Year was not part of the selectors parameter"); <END> } StatisticsHelper.flushDataFromCache(datasourceId, Integer.toString(id), regionset, selectors); } else if (!indicatorService.delete(id, params.getUser().getId())) { throw new ActionParamsException("Indicator wasn't removed: " + + id); } StatisticalDatasourcePlugin datasource = StatisticalDatasourcePluginManager.getInstance().getPlugin(datasourceId); if (datasource != null && datasource.canCache()) { } LOG.info("Deleted indicator", id); try { ResponseHelper.writeResponse(params, StatisticsHelper.toJSON(ind)); } catch (JSONException ex) { ResponseHelper.writeResponse(params, JSONHelper.createJSONObject("deleted", id)); } }</code><technical_language>This caused a moment of confusion. I thought myself: StatisticalIndicatorService#deleteIndicatorData(...) throw a JSONException determine error due selectors a year parameter. I understood is selectors.getInt("year") try'ed here</technical_language>

Target:
public void handleAction(ActionParameters params) throws ActionException { params.requireLoggedInUser(); int datasourceId = params.getRequiredParamInt(StatisticsHelper.PARAM_DATASOURCE_ID); int id = params.getRequiredParamInt(ActionConstants.PARAM_ID); StatisticalIndicator ind = indicatorService.findById(id, params.getUser().getId()); if(ind == null) { throw new ActionDeniedException("Unknown indicator/not the owner: " + id ); } JSONObject selectors = params.getHttpParamAsJSON(StatisticsHelper.PARAM_SELECTORS); if(selectors != null) { int regionset = params.getRequiredParamInt("regionset"); int year = parseYearSelector(selectors); indicatorService.deleteIndicatorData(id, regionset, year); LOG.info("Deleted indicator data for indicator:", id, "year:", year, "regionset:", regionset); StatisticsHelper.flushDataFromCache(datasourceId, Integer.toString(id), regionset, selectors); } else { removeIndicator(id, params.getUser().getId()); LOG.info("Deleted indicator", id); } StatisticalDatasourcePlugin datasource = StatisticalDatasourcePluginManager.getInstance().getPlugin(datasourceId); if (datasource != null && datasource.canCache()) { } try { ResponseHelper.writeResponse(params, StatisticsHelper.toJSON(ind)); } catch (JSONException ex) { ResponseHelper.writeResponse(params, JSONHelper.createJSONObject("id", id)); } }
Source:
<code>private void stopActivityTransitionTimer() { if (this.mActivityTransitionTimerTask != null) { this.mActivityTransitionTimerTask.cancel(); } if (this.mActivityTransitionTimer != null) { this.mActivityTransitionTimer.cancel(); } mIsInBackground = false; <START> sAppIsInTheBackground = false; <END> }</code><technical_language>Do need variables hold information</technical_language>

Target:
private void stopActivityTransitionTimer() { if (this.mActivityTransitionTimerTask != null) { this.mActivityTransitionTimerTask.cancel(); } if (this.mActivityTransitionTimer != null) { this.mActivityTransitionTimer.cancel(); } sAppIsInTheBackground = false; }
Source:
<code>public void dispose() { super.dispose(); synchronized (fBuildJobMap) { fBuildJobMap.values().forEach(Job::cancel); } if (fZoomThread != null) { fZoomThread.cancel(); } ResourcesPlugin.getWorkspace().removeResourceChangeListener(this); <START> getSite().getPage().removePartListener(fPartListener); <END> getSite().getPage().removePartListener(fPartListener2); }</code><technical_language>I this due other exception. a null check. java.lang.IllegalArgumentException org.eclipse.core.runtime.ListenerList.remove(ListenerList.java:217) org.eclipse.ui.internal.WorkbenchPage.removePartListener(WorkbenchPage.java:3355) org.eclipse.tracecompass.tmf.ui.views.timegraph.AbstractTimeGraphView.dispose(AbstractTimeGraphView.java:1458)</technical_language>

Target:
public void dispose() { super.dispose(); synchronized (fBuildJobMap) { fBuildJobMap.values().forEach(Job::cancel); } if (fZoomThread != null) { fZoomThread.cancel(); } deactivateContextService(); ResourcesPlugin.getWorkspace().removeResourceChangeListener(this); IWorkbenchPartSite site = getSite(); if (site != null) { IWorkbenchPage page = site.getPage(); if (page != null) { page.removePartListener(fPartListener); page.removePartListener(fPartListener2); } } }
Source:
<code>public static Long searchHistogram(String name, MetricRegistry mreg) { Scanner sc = new Scanner(name).useDelimiter("_"); if (!sc.hasNext()) return null; Metric m = mreg.findMetric(sc.next()); if (!(m instanceof HistogramMetric)) return null; HistogramMetric h = (HistogramMetric)m; long[] bounds = h.bounds(); long[] values = h.value(); if (!sc.hasNextLong()) return null; long lowBound = sc.nextLong(); if (!sc.hasNextLong()) { if (sc.hasNext() && INF.equals(sc.next()) && bounds[bounds.length-1] == lowBound) return values[values.length-1]; return null; } long highBound = sc.nextLong(); <START> for (int i=0; i<bounds.length; i++) { <END> if (bounds[i] == highBound) { if ((i == 0 && lowBound != 0) || bounds[i-1] != lowBound) return null; return values[i]; } } return null; }</code><technical_language>missed spaces</technical_language>

Target:
public static Long searchHistogram(String name, MetricRegistry mreg) { Scanner sc = new Scanner(name).useDelimiter("_"); if (!sc.hasNext()) return null; Metric m = mreg.findMetric(sc.next()); if (!(m instanceof HistogramMetric)) return null; HistogramMetric h = (HistogramMetric)m; long[] bounds = h.bounds(); long[] values = h.value(); if (!sc.hasNextLong()) return null; long lowBound = sc.nextLong(); if (!sc.hasNextLong()) { if (sc.hasNext() && INF.equals(sc.next()) && bounds[bounds.length - 1] == lowBound) return values[values.length - 1]; return null; } long highBound = sc.nextLong(); int idx = binarySearch(bounds, highBound); if (idx < 0) return null; if ((idx == 0 && lowBound != 0) || (idx != 0 && bounds[idx - 1] != lowBound)) return null; return values[idx]; }
Source:
<code>void produced(long c) { arbiter.produced(c); AtomicLong requestedField = requested; if (requestedField.get() != Long.MAX_VALUE) { <START> requestedField.addAndGet(-c); <END> } }</code><technical_language>Call getAndAddRequest here? In case requested is changed Long.MAX_VALUE time</technical_language>

Target:
void produced(long c) { if (c != 0L) { arbiter.produced(c); BackpressureUtils.produced(requested, c); } }
Source:
<code>public void inputChanged(Viewer viewer, Object oldInput, Object newInput) { if (viewer instanceof AbstractTreeViewer) { manager = new DeferredTreeContentManager( <END> (AbstractTreeViewer) viewer); } super.inputChanged(viewer, oldInput, newInput); }</code><technical_language>is this change for</technical_language>

Target:
public void inputChanged(Viewer viewer, Object oldInput, Object newInput) { if (viewer instanceof AbstractTreeViewer) { manager = new DeferredTreeContentManager(this, (AbstractTreeViewer) viewer); } super.inputChanged(viewer, oldInput, newInput); }