auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public boolean visit(TypeDeclaration elem) { <START> Utils.checkTemplate(collector, elem); <END> if (ElementTypeTeller.isExternal(elem)) { return false; } if (ElementTypeTeller.isSignal(elem)) { Utils.checkModifiers(collector, elem); checkChildren(elem, Messages.ModelVisitor_signal_label, SignalVisitor.ALLOWED_SIGNAL_DECLARATIONS); acceptChildren(elem, new SignalVisitor(collector)); } else if (ElementTypeTeller.isAssociation(elem)) { Utils.checkModifiers(collector, elem); if (ElementTypeTeller.isComposition(elem)) { acceptChildren(elem, new CompositionVisitor(elem, collector)); } else { acceptChildren(elem, new AssociationVisitor(elem, collector)); } checkChildren(elem, Messages.ModelVisitor_association_label, AssociationVisitor.ALLOWED_ASSOCIATION_DECLARATIONS); } else if (ElementTypeTeller.isModelClass(elem)) { checkChildren(elem, Messages.ModelVisitor_class_label, ModelClassVisitor.ALLOWED_MODEL_CLASS_DECLARATIONS); acceptChildren(elem, new ModelClassVisitor(collector)); } else if (ElementTypeTeller.isDataType(elem.resolveBinding())) { checkChildren(elem, "data type", DataTypeVisitor.ALLOWED_DATA_TYPE_DECLARATIONS); acceptChildren(elem, new DataTypeVisitor(collector)); } else if (ElementTypeTeller.isInterface(elem)) { } else if (ElementTypeTeller.isConnector(elem)) { } else { collector.report(new InvalidTypeInModel(collector.getSourceInfo(), elem)); } return false; }</code><technical_language>Is this needed for externals</technical_language>

Target:
public boolean visit(TypeDeclaration elem) { if (ElementTypeTeller.isExternal(elem)) { return false; } Utils.checkTypeParameter(collector, elem); if (ElementTypeTeller.isSignal(elem)) { Utils.checkModifiers(collector, elem); checkChildren(elem, Messages.ModelVisitor_signal_label, SignalVisitor.ALLOWED_SIGNAL_DECLARATIONS); acceptChildren(elem, new SignalVisitor(collector)); } else if (ElementTypeTeller.isAssociation(elem)) { Utils.checkModifiers(collector, elem); if (ElementTypeTeller.isComposition(elem)) { acceptChildren(elem, new CompositionVisitor(elem, collector)); } else { acceptChildren(elem, new AssociationVisitor(elem, collector)); } checkChildren(elem, Messages.ModelVisitor_association_label, AssociationVisitor.ALLOWED_ASSOCIATION_DECLARATIONS); } else if (ElementTypeTeller.isModelClass(elem)) { checkChildren(elem, Messages.ModelVisitor_class_label, ModelClassVisitor.ALLOWED_MODEL_CLASS_DECLARATIONS); acceptChildren(elem, new ModelClassVisitor(collector)); } else if (ElementTypeTeller.isDataType(elem.resolveBinding())) { checkChildren(elem, "data type", DataTypeVisitor.ALLOWED_DATA_TYPE_DECLARATIONS); acceptChildren(elem, new DataTypeVisitor(collector)); } else if (ElementTypeTeller.isInterface(elem)) { } else if (ElementTypeTeller.isConnector(elem)) { } else { collector.report(new InvalidTypeInModel(collector.getSourceInfo(), elem)); } return false; }
Source:
<code>private ASTArgumentList getArgumentList(ASTArguments args) { if (args != null) { <START> super.visit(args, null); <END> return args.getFirstChildOfType(ASTArgumentList.class); } return null; }</code><technical_language>right do this here.... a getter triggers node visitation? visiting nodes this</technical_language>

Target:
private ASTArgumentList getArgumentList(ASTArguments args) { if (args != null) { return args.getFirstChildOfType(ASTArgumentList.class); } return null; }
Source:
<code>private static void buildPointsBlock(Geometry geometry, BlockBuilder blockBuilder) { <START> GeometryType type = GeometryType.getForEsriGeometryType(geometry.getGeometryType()); <END> if (type == GeometryType.POINT) { GEOMETRY.writeSlice(blockBuilder, serialize(geometry)); } else if (type == GeometryType.GEOMETRY_COLLECTION) { GeometryCollection collection = (GeometryCollection) geometry; for (int i = 0; i < collection.getNumGeometries(); i++) { Geometry entry = collection.getGeometryN(i); buildPointsBlock(entry, blockBuilder); } } else { GeometryFactory gFactory = geometry.getFactory(); Coordinate[] vertices = geometry.getCoordinates(); for (Coordinate coordinate : vertices) { GEOMETRY.writeSlice(blockBuilder, serialize(gFactory.createPoint(coordinate))); } } }</code><technical_language>If this is a JTS geometry now, this getForJtsGeometryType</technical_language>

Target:
private static void buildPointsBlock(Geometry geometry, BlockBuilder blockBuilder) { GeometryType type = GeometryType.getForJtsGeometryType(geometry.getGeometryType()); if (type == GeometryType.POINT) { GEOMETRY.writeSlice(blockBuilder, serialize(geometry)); } else if (type == GeometryType.GEOMETRY_COLLECTION) { GeometryCollection collection = (GeometryCollection) geometry; for (int i = 0; i < collection.getNumGeometries(); i++) { Geometry entry = collection.getGeometryN(i); buildPointsBlock(entry, blockBuilder); } } else { GeometryFactory geometryFactory = geometry.getFactory(); Coordinate[] vertices = geometry.getCoordinates(); for (Coordinate coordinate : vertices) { GEOMETRY.writeSlice(blockBuilder, serialize(geometryFactory.createPoint(coordinate))); } } }
Source:
<code>public List<String> getUntrackedFolders() { <START> return (indexDiffFilter == null) ? (List<String>) Collections.EMPTY_LIST <END> : indexDiffFilter.getUntrackedFolders(); }</code><technical_language>Collections.emptyList() remove need for @SuppressWarnings annotation</technical_language>

Target:
public List<String> getUntrackedFolders() { return ((indexDiffFilter == null) ? Collections.<String>emptyList() : indexDiffFilter.getUntrackedFolders()); }
Source:
<code><START> String getNewZipDestination(){ <END> return getNewZipDestination(getProperties().getProperty(UNZIP_BASE_PROPERTY_NAME)); }</code><technical_language>consider returning a File a String here? result in context of a file, cleaner File this point</technical_language>

Target:
File getNewZipDestination(){ return getNewZipDestination(getProperties().getProperty(UNZIP_BASE_PROPERTY_NAME)); }
Source:
<code>public int hashCode() { <START> return Objects.hash(VIEW) + query.hashCode(); <END> }</code><technical_language>Objects.hash(VIEW, query)</technical_language>

Target:
public int hashCode() { return Objects.hash(TYPE, query); }
Source:
<code>private void log(String accessLog, AccessLogData accessLogData) { <START> Set<AccessLogData> logSet = logQueue.get(accessLog); <END> if (logSet == null) { logQueue.putIfAbsent(accessLog, new ConcurrentHashSet<AccessLogData>()); logSet = logQueue.get(accessLog); } if (logSet.size() < LOG_MAX_BUFFER) { logSet.add(accessLogData); } }</code><technical_language>computeIfAbsent optimize this</technical_language>

Target:
private void log(String accessLog, AccessLogData accessLogData) { Set<AccessLogData> logSet = logQueue.computeIfAbsent(accessLog, k -> new ConcurrentHashSet<>()); if (logSet.size() < LOG_MAX_BUFFER) { logSet.add(accessLogData); } else { logger.warn("AccessLog buffer is full skipping buffer "); } }
Source:
<code>public void testLocalhostSingle() throws Exception { SingleMachineProvisioningLocation<SshMachineLocation> l = (SingleMachineProvisioningLocation<SshMachineLocation>) managementContext.getLocationRegistry().resolve("single:(localhost)"); l.setManagementContext(managementContext); <START> SshMachineLocation m1 = l.obtain(); <END> log.info("GOT "+m1); l.release(m1); }</code><technical_language>assert more this, null</technical_language>

Target:
public void testLocalhostSingle() throws Exception { SingleMachineProvisioningLocation<SshMachineLocation> l = (SingleMachineProvisioningLocation<SshMachineLocation>) managementContext.getLocationRegistry().resolve("single:(target='localhost')"); l.setManagementContext(managementContext); SshMachineLocation m1 = l.obtain(); assertNotNull(m1); log.info("GOT "+m1); l.release(m1); }
Source:
<code>private static void enablePartialGetFields(GsonBuilder gb, Multimap<String, String> config) { final Set<String> want = Sets.newHashSet(); for (String p : config.get("fields")) { <START> Iterables.addAll(want, OptionUtil.splitOptionValue(p)); <END> } if (!want.isEmpty()) { gb.addSerializationExclusionStrategy(new ExclusionStrategy() { private final Map<String, String> names = Maps.newHashMap(); @Override public boolean shouldSkipField(FieldAttributes field) { String name = names.get(field.getName()); if (name == null) { try { name = FieldNamingPolicy.LOWER_CASE_WITH_UNDERSCORES.translateName( field.getDeclaringClass().getDeclaredField(field.getName())); names.put(field.getName(), name); } catch (SecurityException e) { return true; } catch (NoSuchFieldException e) { return true; } } return !want.contains(name); } @Override public boolean shouldSkipClass(Class<?> clazz) { return false; } }); } }</code><technical_language>added a toLowerCase before</technical_language>

Target:
private static void enablePartialGetFields(GsonBuilder gb, Multimap<String, String> config) { final Set<String> want = Sets.newHashSet(); for (String p : config.get("fields")) { Iterables.addAll(want, OptionUtil.splitOptionValue(p)); } if (!want.isEmpty()) { gb.addSerializationExclusionStrategy(new ExclusionStrategy() { private final Map<String, String> names = Maps.newHashMap(); @Override public boolean shouldSkipField(FieldAttributes field) { String name = names.get(field.getName()); if (name == null) { try { name = FieldNamingPolicy.LOWER_CASE_WITH_UNDERSCORES.translateName( field.getDeclaringClass().getDeclaredField(field.getName())); names.put(field.getName(), name); } catch (SecurityException e) { return true; } catch (NoSuchFieldException e) { return true; } } return !want.contains(name.toLowerCase()); } @Override public boolean shouldSkipClass(Class<?> clazz) { return false; } }); } }
Source:
<code>public int schedule() { <START> schedulingMainLoopTimingLogger = new SchedulingMainLoopTimingLogger(logger); <END> Policy currentPolicy = schedulingService.getPolicy(); activeObjectCreationRetryTimeNumber = ACTIVEOBJECT_CREATION_RETRY_TIME_NUMBER; Map<JobId, JobDescriptor> jobMap = schedulingService.lockJobsToSchedule(); Map<JobId, JobDescriptor> toUnlock = jobMap; logSelectedJobs(jobMap); if (jobMap.isEmpty()) { return 0; } int tasksStarted = startTasks(currentPolicy, jobMap, toUnlock); schedulingMainLoopTimingLogger.printTimingsINFOLevel(); return tasksStarted; }</code><technical_language>logger changed SchedulingMethodImpl.class</technical_language>

Target:
public int schedule() { schedulingMainLoopTimingLogger = new SchedulingMainLoopTimingLogger(Logger.getLogger(SchedulingMethodImpl.class)); Policy currentPolicy = schedulingService.getPolicy(); activeObjectCreationRetryTimeNumber = ACTIVEOBJECT_CREATION_RETRY_TIME_NUMBER; Map<JobId, JobDescriptor> jobMap = schedulingService.lockJobsToSchedule(); Map<JobId, JobDescriptor> toUnlock = jobMap; logSelectedJobs(jobMap); if (jobMap.isEmpty()) { return 0; } int tasksStarted = startTasks(currentPolicy, jobMap, toUnlock); schedulingMainLoopTimingLogger.printTimingsINFOLevel(); return tasksStarted; }
Source:
<code>public static void toJSON(OutputWriter outputWriter, ArtifactStore store) { outputWriter <START> .add("id", store.getId()) <END> .add("plugin_id", store.getPluginId()) .addChildList("properties", listWriter -> store.forEach(property -> listWriter.addChild(propertyWriter -> ConfigurationPropertyRepresenter.toJSON(propertyWriter, property)))); if (store.hasErrors()) { Map<String, String> fieldMapping = Collections.singletonMap("pluginId", "plugin_id"); outputWriter.addChild("errors", errorWriter -> new ErrorGetter(fieldMapping).toJSON(errorWriter, store)); } }</code><technical_language>Add self, doc find links</technical_language>

Target:
public static void toJSON(OutputWriter outputWriter, ArtifactStore store) { outputWriter .addLinks(linksWriter -> linksWriter .addLink("self", Routes.ArtifactStoreConfig.id(store.getId())) .addAbsoluteLink("doc", Routes.ArtifactStoreConfig.DOC) .addLink("find", Routes.ArtifactStoreConfig.find())) .add("id", store.getId()) .add("plugin_id", store.getPluginId()) .addChildList("properties", listWriter -> store.forEach(property -> listWriter.addChild(propertyWriter -> ConfigurationPropertyRepresenter.toJSON(propertyWriter, property)))); if (store.hasErrors()) { Map<String, String> fieldMapping = Collections.singletonMap("pluginId", "plugin_id"); outputWriter.addChild("errors", errorWriter -> new ErrorGetter(fieldMapping).toJSON(errorWriter, store)); } }
Source:
<code>public static AmqpMessageBody fromData(byte[] data) { Objects.requireNonNull(data, "'data' cannot be null."); AmqpMessageBody body = new AmqpMessageBody(); body.bodyType = AmqpMessageBodyType.DATA; <START> body.data = Collections.singletonList(Arrays.copyOf(data, data.length)); <END> return body; }</code><technical_language>I copy is required here. if this for immutability, do this in getter methods currently doing. So, copying useful</technical_language>

Target:
public static AmqpMessageBody fromData(byte[] data) { Objects.requireNonNull(data, "'data' cannot be null."); AmqpMessageBody body = new AmqpMessageBody(); body.bodyType = AmqpMessageBodyType.DATA; body.data = data; return body; }
Source:
<code>public void Login_006_authModalInGlobalNav_staff() { WikiBasePageObject base = new WikiBasePageObject(driver); base.openWikiPage(wikiURL); <START> NavigationBar signInLink = new NavigationBar(driver); <END> signInLink.clickOnSignIn(); AuthModal authModal = signInLink.getAuthModal(); Assert.assertTrue(authModal.isOpened()); authModal.login(credentials.userNameStaff, credentials.passwordStaff); base.verifyUserLoggedIn(credentials.userNameStaff); }</code><technical_language>Move this WikiBasePageObject base = new WikiBasePageObject(driver);</technical_language>

Target:
public void Login_006_authModalInGlobalNav_staff() { WikiBasePageObject base = new WikiBasePageObject(driver); NavigationBar signInLink = new NavigationBar(driver); base.openWikiPage(wikiURL); signInLink.clickOnSignIn(); AuthModal authModal = signInLink.getAuthModal(); Assert.assertTrue(authModal.isOpened()); authModal.login(credentials.userNameStaff, credentials.passwordStaff); base.verifyUserLoggedIn(credentials.userNameStaff); }
Source:
<code>public ListBoxModel doFillCredentialsIdItems(@AncestorInPath ItemGroup context, @QueryParameter String serverId) { AccessControlled _context = (context instanceof AccessControlled ? (AccessControlled) context : Jenkins.getInstance()); if (_context == null || !_context.hasPermission(Computer.CONFIGURE)) { return new StandardUsernameListBoxModel().includeCurrentValue(serverId); } return new StandardUsernameListBoxModel().includeAs( <START> ACL.SYSTEM, <END> context, StandardUsernameCredentials.class, Collections.<DomainRequirement> singletonList(new MavenServerIdRequirement(serverId)) ) .includeCurrentValue(serverId); }</code><technical_language>context instanceof Queue.Task ? Tasks.getDefaultAuthenticationOf((Queue.Task)context) : ACL.SYSTEM is want a raw ACL.SYSTEM</technical_language>

Target:
public ListBoxModel doFillCredentialsIdItems(@AncestorInPath ItemGroup context, @QueryParameter String serverId) { AccessControlled _context = (context instanceof AccessControlled ? (AccessControlled) context : Jenkins.getInstance()); if (_context == null || !_context.hasPermission(Computer.CONFIGURE)) { return new StandardUsernameListBoxModel().includeCurrentValue(serverId); } List<DomainRequirement> domainRequirements = Collections.emptyList(); if (StringUtils.isNotBlank(serverId)) { Collections.<DomainRequirement> singletonList(new MavenServerIdRequirement(serverId)); } return new StandardUsernameListBoxModel().includeAs( context instanceof Queue.Task ? Tasks.getDefaultAuthenticationOf((Queue.Task)context) : ACL.SYSTEM, context, StandardUsernameCredentials.class, domainRequirements ) .includeCurrentValue(serverId); }
Source:
<code>public void testSetTheme() { ThemePropertyListener listener = new ThemePropertyListener(); fManager.addPropertyChangeListener(listener); ITheme currentTheme = fManager.getCurrentTheme(); fManager.setCurrentTheme(BOGUSID); assertEquals(currentTheme, fManager.getCurrentTheme()); fManager.setCurrentTheme(THEME1); assertNotSame(currentTheme, fManager.getCurrentTheme()); ITheme newCurrentTheme = fManager.getCurrentTheme(); ITheme theme1 = getTheme1(); assertEquals(theme1, newCurrentTheme); List events = listener.getEvents(); <START> assertTrue(events.size() > 0); <END> PropertyChangeEvent event = ((PropertyChangeEvent) events.get(0)); assertEquals(IThemeManager.CHANGE_CURRENT_THEME, event.getProperty()); assertEquals(currentTheme, event.getOldValue()); assertEquals(newCurrentTheme, event.getNewValue()); fManager.removePropertyChangeListener(listener); }</code><technical_language>here? changed generate of extra events</technical_language>

Target:
public void testSetTheme() { ThemePropertyListener listener = new ThemePropertyListener(); fManager.addPropertyChangeListener(listener); ITheme currentTheme = fManager.getCurrentTheme(); fManager.setCurrentTheme(BOGUSID); assertEquals(currentTheme, fManager.getCurrentTheme()); fManager.setCurrentTheme(THEME1); assertNotSame(currentTheme, fManager.getCurrentTheme()); ITheme newCurrentTheme = fManager.getCurrentTheme(); ITheme theme1 = getTheme1(); assertEquals(theme1, newCurrentTheme); List events = listener.getEvents(); assertEquals(1, events.size()); PropertyChangeEvent event = ((PropertyChangeEvent) events.get(0)); assertEquals(IThemeManager.CHANGE_CURRENT_THEME, event.getProperty()); assertEquals(currentTheme, event.getOldValue()); assertEquals(newCurrentTheme, event.getNewValue()); fManager.removePropertyChangeListener(listener); }
Source:
<code>private void glusterHostRemove() { boolean forceAction = false; if (getVdsDAO().getAll().size() > 1) { <START> if (isGlusterEnabled() && !hasVolumeOnServer()) { <END> VDSReturnValue returnValue = runVdsCommand( VDSCommandType.GlusterHostRemove, new GlusterHostRemoveVDSParameters((ClusterUtils.getInstance().getUpServer(getVdsGroupId())).getId(), getVdsName(), forceAction)); setSucceeded(returnValue.getSucceeded()); if (!returnValue.getSucceeded()) { errorType = AuditLogType.GLUSTER_HOST_REMOVE_FAILED; } } } }</code><technical_language>merge 'if' checks one? i check ifGlusterEnabled() save hosts db for gluster hosts</technical_language>

Target:
private void glusterHostRemove() { boolean forceAction = false; if (isGlusterEnabled() && getVdsDAO().getAll().size() > 1 && !hasVolumeOnServer()) { VDSReturnValue returnValue = runVdsCommand( VDSCommandType.GlusterHostRemove, new GlusterHostRemoveVDSParameters((ClusterUtils.getInstance().getUpServer(getVdsGroupId())).getId(), getVdsName(), forceAction)); setSucceeded(returnValue.getSucceeded()); if (!returnValue.getSucceeded()) { errorType = AuditLogType.GLUSTER_HOST_REMOVE_FAILED; } } }
Source:
<code>Optional<Path> downloadAndExtractAllureTo(String home, String version) { return downloadAllure(version).map(zipFilePath -> { try { LOGGER.info("Extracting file " + zipFilePath + " to " + home + "..."); final String extractedDirName = "allure-" + version; final Path allureHome = Paths.get(home); final Path extractDir = zipFilePath.getParent(); new ZipFile(zipFilePath.toFile()).extractAll(extractDir.toString()); if (Files.exists(allureHome)) { LOGGER.info("Directory " + allureHome + " already exists, removing it.."); <START> deleteQuietly(allureHome.toFile()); <END> } moveDirectory(extractDir.resolve(extractedDirName).toFile(), allureHome.toFile()); addExecutablePermissions(allureHome); return allureHome; } catch (ZipException | IOException e) { LOGGER.error("Failed to download and extract Allure of version to dir {}", version, home, e); return null; } finally { deleteQuietly(zipFilePath.toFile()); } }); }</code><technical_language>remove /usr in case of bad home directory</technical_language>

Target:
Optional<Path> downloadAndExtractAllureTo(String home, String version) { return downloadAllure(version).map(zipFilePath -> { try { LOGGER.info("Extracting file to {}...", zipFilePath, home); final String extractedDirName = "allure-" + version; final Path allureHome = Paths.get(home); final Path extractDir = zipFilePath.getParent(); new ZipFile(zipFilePath.toFile()).extractAll(extractDir.toString()); if (Files.exists(allureHome)) { LOGGER.info("Directory already exists, skipping it..", allureHome); return null; } moveDirectory(extractDir.resolve(extractedDirName).toFile(), allureHome.toFile()); addExecutablePermissions(allureHome); return allureHome; } catch (ZipException | IOException e) { LOGGER.error("Failed to download and extract Allure of version to dir {}", version, home, e); return null; } finally { deleteQuietly(zipFilePath.toFile()); } }); }
Source:
<code>boolean isCallPermitted() { <START> return false; <END> }</code><technical_language>Do measure metrics here? interesting calls rejected. OpenState circuitBreakerMetrics.onCallNotPermitted();</technical_language>

Target:
boolean isCallPermitted() { circuitBreakerMetrics.onCallNotPermitted(); return false; }
Source:
<code>public ResourceTraversal[] getTraversals(ResourceMappingContext context, IProgressMonitor monitor) throws CoreException { final SynchronizationModel syncModel; if (context == initialContext || !(resource instanceof IFile)) { syncModel = synchronizationModel; } else { SynchronizationModel temp = null; try { temp = ((EMFModelProvider)getModelProvider()).getOrComputeLogicalModel((IFile)resource, context, monitor); } catch (InterruptedException e) { Thread.currentThread().interrupt(); temp = synchronizationModel; } syncModel = temp; } latestModel = syncModel; if (syncModel.getDiagnostic().getSeverity() >= Diagnostic.ERROR) { EMFCompareIDEUIPlugin.getDefault().getLog().log( BasicDiagnostic.toIStatus(syncModel.getDiagnostic())); <START> List<?> datas = syncModel.getDiagnostic().getData(); <END> boolean isInError = true; for (Object data : datas) { if (data instanceof SynchronizationModel) { isInError = false; SynchronizationModel synch = (SynchronizationModel)data; if (synch.getLeftTraversal().getDiagnostic().getSeverity() >= Diagnostic.ERROR || synch.getRightTraversal().getDiagnostic().getSeverity() >= Diagnostic.ERROR || synch.getOriginTraversal().getDiagnostic().getSeverity() >= Diagnostic.ERROR) { isInError = true; break; } } } if (isInError) { return createSingletonTraversal(resource); } } return convertToTraversal(syncModel); }</code><technical_language>'data' is plural of 'datum', need additional 's'. a common mistake french people ;)</technical_language>

Target:
public ResourceTraversal[] getTraversals(ResourceMappingContext context, IProgressMonitor monitor) throws CoreException { final SynchronizationModel syncModel; if (context == initialContext || !(resource instanceof IFile)) { syncModel = synchronizationModel; } else { SynchronizationModel temp = null; try { temp = ((EMFModelProvider)getModelProvider()).getOrComputeLogicalModel((IFile)resource, context, monitor); } catch (InterruptedException e) { Thread.currentThread().interrupt(); temp = synchronizationModel; } syncModel = temp; } latestModel = syncModel; return convertToTraversal(syncModel); }
Source:
<code>public void setup() { super.setup(); zones = api.getConfiguredZones(); for (String zone : zones){ <START> ServerApi api = this.api.getServerApiForZone(zone); <END> for (Resource server : api.list().concat()){ if (server.getName().equals(hostName)) api.delete(server.getId()); } } }</code><technical_language>More mixing of this.api api</technical_language>

Target:
public void setup() { super.setup(); zones = api.getConfiguredZones(); for (String zone : zones){ ServerApi serverApi = api.getServerApiForZone(zone); for (Resource server : serverApi.list().concat()){ if (server.getName().equals(hostName)) serverApi.delete(server.getId()); } } }
Source:
<code>private void printPlanNodesStats(int indent, PlanNode... nodes) { if (Arrays.stream(nodes).anyMatch(this::isPlanNodeStatsKnown)) { <START> String formatedStats = Joiner.on("/").join(Arrays.stream(nodes) <END> .map(this::formatPlanNodeStats) .collect(toImmutableList())); print(indent, "Cost: %s", formatedStats); } }</code><technical_language>nit: formattedStats (double 't' in formatted)</technical_language>

Target:
private void printPlanNodesStats(int indent, PlanNode... nodes) { if (Arrays.stream(nodes).anyMatch(this::isPlanNodeStatsKnown)) { String formattedStats = Joiner.on("/").join(Arrays.stream(nodes) .map(this::formatPlanNodeStats) .collect(toImmutableList())); print(indent, "Cost: %s", formattedStats); } }
Source:
<code>public void handlesRequestInTime() throws IOException { final int length = 100000000; final File tempfile = this.temp.newFile("handlesRequestInTime.tmp"); final BufferedWriter bwr = new BufferedWriter(new FileWriter(tempfile)); bwr.write( Joiner.on(RqMultipartTest.CRLF).join( "--zzz", "Content-Disposition: form-data; name=\"test\"", "", "" ) ); for (int ind = 0; ind < length; ++ind) { bwr.write("X"); } bwr.write(RqMultipartTest.CRLF); bwr.write("--zzz--"); bwr.write(RqMultipartTest.CRLF); bwr.close(); final long start = System.currentTimeMillis(); final Request req = new RqFake( Arrays.asList( "POST /post?u=3 HTTP/1.1", "Host: example.com", "Content-Type: multipart/form-data; boundary=zzz", String.format( "Content-Length:%s", tempfile.length() ) ), new TempInputStream(new FileInputStream(tempfile), tempfile) ); final RqMultipart.Smart smartreq = new RqMultipart.Smart( new RqMultipart.Base(req) <START> ); <END> try { MatcherAssert.assertThat( smartreq.single("test").body().available(), Matchers.equalTo(length) ); MatcherAssert.assertThat( System.currentTimeMillis() - start, Matchers.lessThan(3000L) ); } finally { req.body().close(); smartreq.part("test").iterator().next().body().close(); } }</code><technical_language>@raphaelln please pay attention indentation</technical_language>

Target:
public void handlesRequestInTime() throws IOException { final int length = 100000000; final String part = "test"; final File file = this.temp.newFile("handlesRequestInTime.tmp"); final BufferedWriter bwr = new BufferedWriter(new FileWriter(file)); bwr.write( Joiner.on(RqMultipartTest.CRLF).join( "--zzz", String.format(RqMultipartTest.CONTENT, part), "", "" ) ); for (int ind = 0; ind < length; ++ind) { bwr.write("X"); } bwr.write(RqMultipartTest.CRLF); bwr.write("--zzz--"); bwr.write(RqMultipartTest.CRLF); bwr.close(); final long start = System.currentTimeMillis(); final Request req = new RqFake( Arrays.asList( "POST /post?u=3 HTTP/1.1", "Host: example.com", "Content-Type: multipart/form-data; boundary=zzz", String.format("Content-Length:%s", file.length()) ), new TempInputStream(new FileInputStream(file), file) ); final RqMultipart.Smart smart = new RqMultipart.Smart( new RqMultipart.Base(req) ); try { MatcherAssert.assertThat( smart.single(part).body().available(), Matchers.equalTo(length) ); MatcherAssert.assertThat( System.currentTimeMillis() - start, Matchers.lessThan(3000L) ); } finally { req.body().close(); smart.part(part).iterator().next().body().close(); } }
Source:
<code><START> public long getDocumentNumericId() { <END> return documentNumericId; }</code><technical_language>This needs package private, leaking this identifier PR supposed hide public view</technical_language>

Target:
protected long getDocumentNumericId() { return documentNumericId; }
Source:
<code>public void recoverDeletedSecretCodeSnippets() { SecretClient secretClient = getSecretClient(); Poller<KeyVaultSecret, Void> poller = secretClient.beginRecoverDeletedSecret("deletedSecretName"); poller.getObserver() .map(pollResponse -> { System.out.println("Current status: " + pollResponse.getStatus()); if (pollResponse.getStatus() != PollResponse.OperationStatus.NOT_STARTED) { KeyVaultSecret recoveredSecret = pollResponse.getValue(); System.out.println("Recovered key name: " + recoveredSecret.getName()); System.out.println("Recovered key id: " + recoveredSecret.getId()); <START> } <END> return pollResponse.getStatus(); }) .blockLast(); System.out.println("Final status: " + poller.getStatus()); System.out.println("Final response: " + poller.getLastPollResponse()); END: com.azure.security.keyvault.secretclient.recoverDeletedSecret#string }</code><technical_language>This create conflicts sync poller coming in</technical_language>

Target:
public void recoverDeletedSecretCodeSnippets() throws InterruptedException { SecretClient secretClient = getSecretClient(); Poller<KeyVaultSecret, Void> recoverSecretPoller = secretClient.beginRecoverDeletedSecret("deletedSecretName"); while (recoverSecretPoller.getStatus() != PollResponse.OperationStatus.IN_PROGRESS && !recoverSecretPoller.isComplete()) { System.out.println(recoverSecretPoller.getStatus().toString()); Thread.sleep(2000); } KeyVaultSecret recoveredSecret = recoverSecretPoller.getLastPollResponse().getValue(); System.out.println("Recovered Key Name: " + recoveredSecret.getName()); System.out.printf("Recovered Key's Id %s", recoveredSecret.getId()); while (!recoverSecretPoller.isComplete()) { System.out.println(recoverSecretPoller.getStatus().toString()); Thread.sleep(2000); } END: com.azure.security.keyvault.secretclient.recoverDeletedSecret#string }
Source:
<code>private BigtableOptions resolveLegacyOptions(BigtableOptions options) throws IOException { if (options.getClusterId() != null && options.getZoneId() != null) { String instanceId = BigtableClusterUtilities.lookupInstanceId( options.getProjectId(), options.getClusterId(), options.getZoneId() ); if (options.getInstanceId() != null) { Preconditions.checkArgument( options.getInstanceId().equals(instanceId), <START> "Supplied instanceId: '%s', zoneId: '%s' and clusterId: '%s'. They do not match." <END> + "\nFound instanceId '%s' that corresponds to the zoneId/clusterId"); } return options.toBuilder().setInstanceId(instanceId).build(); } return options; }</code><technical_language>Missing variables for replacement</technical_language>

Target:
private BigtableOptions resolveLegacyOptions(BigtableOptions options) throws IOException { if (options.getClusterId() != null && options.getZoneId() != null) { String instanceId = BigtableClusterUtilities.lookupInstanceId( options.getProjectId(), options.getClusterId(), options.getZoneId() ); if (options.getInstanceId() != null) { Preconditions.checkArgument( options.getInstanceId().equals(instanceId), "Supplied instanceId: '%s', zoneId: '%s' and clusterId: '%s'. They do not match." + "\nFound instanceId '%s' that corresponds to the zoneId/clusterId", options.getInstanceId(), options.getZoneId(), options.getClusterId(), instanceId); } return options.toBuilder().setInstanceId(instanceId).build(); } return options; }
Source:
<code><START> public ServiceProviderImpl(ServiceDiscoveryImpl<T> discovery, String serviceName, ProviderStrategy<T> providerStrategy, ThreadFactory threadFactory, CloseableExecutorService executorService, List<InstanceFilter<T>> filters, DownInstancePolicy downInstancePolicy) <END> { this.discovery = discovery; this.providerStrategy = providerStrategy; downInstanceManager = new DownInstanceManager<T>(downInstancePolicy); final ServiceCacheBuilder builder = discovery.serviceCacheBuilder().name(serviceName); if (executorService != null) { builder.executorService(executorService); } else { builder.threadFactory(threadFactory); } cache = builder.build(); ArrayList<InstanceFilter<T>> localFilters = Lists.newArrayList(filters); localFilters.add(downInstanceManager); localFilters.add(new InstanceFilter<T>() { @Override public boolean apply(ServiceInstance<T> instance) { return instance.isEnabled(); } }); instanceProvider = new FilteredInstanceProvider<T>(cache, localFilters); }</code><technical_language>This constructor is public (can directly, skipping ServiceProviderBuilder), accepts ThreadFactory CloseableExecutorService, confusing. I suggest this constructor package-private add 2 public constructors: accepts arguments this ThreadFactory (no CloseableExecutorService) accepts arguments this CloseableExecutorService (no ThreadFactory). delegate this constructor (with null missing argument)</technical_language>

Target:
protected ServiceProviderImpl(ServiceDiscoveryImpl<T> discovery, String serviceName, ProviderStrategy<T> providerStrategy, ThreadFactory threadFactory, CloseableExecutorService executorService, List<InstanceFilter<T>> filters, DownInstancePolicy downInstancePolicy) { this.discovery = discovery; this.providerStrategy = providerStrategy; downInstanceManager = new DownInstanceManager<T>(downInstancePolicy); final ServiceCacheBuilder builder = discovery.serviceCacheBuilder().name(serviceName); if (executorService != null) { builder.executorService(executorService); } else { builder.threadFactory(threadFactory); } cache = builder.build(); ArrayList<InstanceFilter<T>> localFilters = Lists.newArrayList(filters); localFilters.add(downInstanceManager); localFilters.add(new InstanceFilter<T>() { @Override public boolean apply(ServiceInstance<T> instance) { return instance.isEnabled(); } }); instanceProvider = new FilteredInstanceProvider<T>(cache, localFilters); }
Source:
<code>public void testIsDistinctFrom() { <START> long[] arr1 = {112L, 0L}; END> long[] arr2 = {185L, 0L}; Block left = new Int128ArrayBlock(1, Optional.empty(), arr1); Block right = new Int128ArrayBlock(1, Optional.empty(), arr2); assertFalse(DecimalInequalityOperators.distinctBlockPositionLongLong(left, 0, left, 0)); assertTrue(DecimalInequalityOperators.distinctBlockPositionLongLong(left, 0, right, 0)); }</code><technical_language>a1, a2, etc. variables names; here, inline arr1 arr2 - static import distinctBlockPositionLongLong @Test public void testIsDistinctFrom() { Block left = new Int128ArrayBlock(1, Optional.empty(), new long[] {112L, 0L}); Block right = new Int128ArrayBlock(1, Optional.empty(), new long[] {185L, 0L}); assertFalse(distinctBlockPositionLongLong(left, 0, left, 0)); assertTrue(distinctBlockPositionLongLong(left, 0, right, 0)); }</technical_language>

Target:
public void testIsDistinctFrom() { Block left = new Int128ArrayBlock(1, Optional.empty(), new long[]{112L, 0L}); Block right = new Int128ArrayBlock(1, Optional.empty(), new long[]{185L, 0L}); assertFalse(distinctBlockPositionLongLong(left, 0, left, 0)); assertTrue(distinctBlockPositionLongLong(left, 0, right, 0)); }
Source:
<code><START> public void listPatientsNonDefaultBehaviour() throws WebApplicationException, QueryException { <END> Query query = mock(DefaultQuery.class); doReturn(query).when(this.queries).createQuery(anyString(), anyString()); doReturn(query).when(query).bindValue(anyString(), anyString()); doReturn(new ArrayList<Object[]>()).when(query).execute(); Patients result = this.patientsResource.listPatients(0, 30, "eid", "desc"); verify(this.queries).createQuery("select doc.fullName, p.external_id, doc.creator, doc.creationDate, doc.version, doc.author, doc.date" + " from Document doc, doc.object(PhenoTips.PatientClass) p where doc.name <> :t order by " + "p.external_id" + " desc", "xwql"); }</code><technical_language>WebApplicationException is a RuntimeException need a throws clause for</technical_language>

Target:
public void listPatientsNonDefaultBehaviour() throws QueryException { Query query = mock(DefaultQuery.class); doReturn(query).when(this.queries).createQuery(anyString(), anyString()); doReturn(query).when(query).bindValue(anyString(), anyString()); doReturn(new ArrayList<Object[]>()).when(query).execute(); Patients result = this.patientsResource.listPatients(0, 30, "eid", "desc"); verify(this.queries).createQuery("select doc.fullName, p.external_id, doc.creator, doc.creationDate, doc.version, doc.author, doc.date" + " from Document doc, doc.object(PhenoTips.PatientClass) p where doc.name <> :t order by " + "p.external_id" + " desc", "xwql"); }
Source:
<code>public void run() { if (shutdown) { return; } Exception exceptionToThrow = null; try { initialize(); log.info("Initialization complete. Starting worker loop."); } catch (RuntimeException e) { log.error("Unable to initialize after attempts. Shutting down.", lifecycleConfig.maxInitializationAttempts(), e); shutdown(); exceptionToThrow = e; } while (!shouldShutdown()) { <START> exceptionToThrow = null; <END> runProcessLoop(); } finalShutdown(); log.info("Worker loop is complete. Exiting from worker."); if (exceptionToThrow != null) { coordinatorExceptionHandler.propogate(exceptionToThrow); } }</code><technical_language>Need set this. If exception is caught initialize, shutdown is invoked shouldShutdown return true if Scheduler successfully shutdown. enter ShardConsumers shutdown successfully</technical_language>

Target:
public void run() { if (shutdown) { return; } try { initialize(); log.info("Initialization complete. Starting worker loop."); } catch (RuntimeException e) { log.error("Unable to initialize after attempts. Shutting down.", maxInitializationAttempts, e); workerStateChangeListener.onAllInitializationAttemptsFailed(e); shutdown(); } while (!shouldShutdown()) { runProcessLoop(); } finalShutdown(); log.info("Worker loop is complete. Exiting from worker."); }
Source:
<code>static boolean isNullColumn(Iterable<String> dimValues) { if (dimValues == null) { return true; } for (String val : dimValues) { <START> if (!Strings.isNullOrEmpty(val)) { <END> return false; } } return true; }</code><technical_language>I val a non-empty string null, empty string, Indexed<String> returned getDimValueLookup, is Indexed<String> based a NullValueConverterDimDim a GenericIndexed<String> map empty null</technical_language>

Target:
static boolean isNullColumn(Iterable<String> dimValues) { if (dimValues == null) { return true; } for (String val : dimValues) { if (val != null) { return false; } } return true; }
Source:
<code>public JAXBProcessInstanceList processInstancesDataTable(Properties properties) throws Throwable { <START> logger.debug("processInstancesDataTable"); <END> final String configId = properties.getProperty("configId"); final BpmWidgetInfo bpmWidgetInfo = bpmWidgetInfoManager.getBpmWidgetInfo(Integer.parseInt(configId)); final String information = bpmWidgetInfo.getInformationDraft(); final ApsProperties config = new ApsProperties(); try { if (null != configId) { try { if (StringUtils.isNotEmpty(information)) { try { config.loadFromXml(information); } catch (IOException e) { logger.error("Error load configuration ", e.getMessage()); } final JAXBProcessInstanceList processList = new JAXBProcessInstanceList(); this.setElementDatatableFieldDefinition(config, processList); config.loadFromXml(information); String knowledgetSource = (String) config.get(KieBpmSystemConstants.WIDGET_INFO_PROP_KIE_SOURCE_ID); KieBpmConfig bpmConfig = this.getKieFormManager().getKieServerConfigurations().get(knowledgetSource); this.setElementList(bpmConfig, config, processList); processList.setContainerId(config.getProperty("containerId")); processList.setProcessId(config.getProperty("processId")); return processList; } } catch (ApsSystemException e) { logger.error("Error {}", e); } } return null; } catch (Throwable t) { logger.error("Failed to get process instances data table ",t); throw new ApiException(IApiErrorCodes.API_VALIDATION_ERROR, NO_ERROR_GETTING_THE_LIST_OF_PROCESSES, Response.Status.INTERNAL_SERVER_ERROR); } }</code><technical_language>Add meaningful log info if needed</technical_language>

Target:
public JAXBProcessInstanceList processInstancesDataTable(Properties properties) throws Throwable { final String configId = properties.getProperty("configId"); final BpmWidgetInfo bpmWidgetInfo = bpmWidgetInfoManager.getBpmWidgetInfo(Integer.parseInt(configId)); final String information = bpmWidgetInfo.getInformationDraft(); final ApsProperties config = new ApsProperties(); try { if (null != configId) { try { if (StringUtils.isNotEmpty(information)) { try { config.loadFromXml(information); } catch (IOException e) { logger.error("Error load configuration ", e.getMessage()); } final JAXBProcessInstanceList processList = new JAXBProcessInstanceList(); this.setElementDatatableFieldDefinition(config, processList); config.loadFromXml(information); String knowledgetSource = (String) config.get(KieBpmSystemConstants.WIDGET_INFO_PROP_KIE_SOURCE_ID); KieBpmConfig bpmConfig = this.getKieFormManager().getKieServerConfigurations().get(knowledgetSource); this.setElementList(bpmConfig, config, processList); processList.setContainerId(config.getProperty("containerId")); processList.setProcessId(config.getProperty("processId")); return processList; } } catch (ApsSystemException e) { logger.error("Error {}", e); } } return null; } catch (Throwable t) { logger.error("Failed to get process instances data table ",t); throw new ApiException(IApiErrorCodes.API_VALIDATION_ERROR, NO_ERROR_GETTING_THE_LIST_OF_PROCESSES, Response.Status.INTERNAL_SERVER_ERROR); } }
Source:
<code>public ParseTexDialogView(BibDatabaseContext databaseContext) { this.databaseContext = databaseContext; this.setTitle(Localization.lang("LaTeX integration tools")); ViewLoader.view(this) .load() .setAsDialogPane(this); Button parseButton = (Button) this.getDialogPane().lookupButton(parseButtonType); <START> parseButton.disableProperty().bindBidirectional(viewModel.noFilesFoundProperty()); <END> setResultConverter(button -> { if (button == parseButtonType) { viewModel.parse(); } return null; }); }</code><technical_language>parseButton disabled long items selected</technical_language>

Target:
public ParseTexDialogView(BibDatabaseContext databaseContext) { this.databaseContext = databaseContext; this.validationVisualizer = new ControlsFxVisualizer(); this.setTitle(Localization.lang("LaTeX references search")); ViewLoader.view(this) .load() .setAsDialogPane(this); ControlHelper.setAction(parseButtonType, getDialogPane(), event -> viewModel.parseButtonClicked()); Button parseButton = (Button) getDialogPane().lookupButton(parseButtonType); parseButton.disableProperty().bind(viewModel.noFilesFoundProperty().or( Bindings.isEmpty(viewModel.getCheckedFileList()))); }
Source:
<code>private RelatedArtifactChangedColumn() { super(WorldXViewerFactory.COLUMN_NAMESPACE + ".RelatedArtifactChangedColumn", "Related Artifact Changed", 75, <START> SWT.LEFT, true, SortDataType.String, false, <END> "Commited - baseline/commited branch \nUnmodified - Related artifact has not changed " + "\n<date> - Related artifact has been modified after task at the specified date \nEmpty - There is no related artifact"); }</code><technical_language>Committed (spelling)</technical_language>

Target:
private RelatedArtifactChangedColumn() { super(WorldXViewerFactory.COLUMN_NAMESPACE + ".RelatedArtifactChangedColumn", "Related Artifact Changed", 75, SWT.LEFT, true, SortDataType.String, false, "Committed - baseline/commited branch \nUnmodified - Related artifact has not changed " + "\n<date> - Related artifact has been modified after task at the specified date \nEmpty - There is no related artifact"); }
Source:
<code>private List<AppInfo> getAppsFromPackageAndConfig(AppPackage ap, ConfigPackage cp, String configApps) { if (cp == null || configApps == null || !(configApps.equals(CONFIG_INCLUSIVE) || configApps.equals(CONFIG_EXCLUSIVE))) { return ap.getApplications(); } File src = new File(cp.tempDirectory(), "app"); File dest = new File(ap.tempDirectory(), "app"); if (!src.exists()) { return ap.getApplications(); } if (configApps.equals(CONFIG_EXCLUSIVE)) { for (File app : dest.listFiles()) { <START> if (app.getName().endsWith("json")) { <END> FileUtils.deleteQuietly(new File(dest, app.getName())); } } } else { for (File app : src.listFiles()) { FileUtils.deleteQuietly(new File(dest, app.getName())); } } for (File app : src.listFiles()) { try { FileUtils.moveFileToDirectory(app, dest, true); } catch (IOException e) { LOG.warn("Application from the config file failed while processing.", app.getName()); } } try { FileUtils.deleteDirectory(src); } catch (IOException e) { LOG.warn("Failed to delete the Config Apps folder"); } ap.processAppDirectory(configApps.equals(CONFIG_EXCLUSIVE)); return ap.getApplications(); }</code><technical_language>endsWith(".json")</technical_language>

Target:
private List<AppInfo> getAppsFromPackageAndConfig(AppPackage ap, ConfigPackage cp, String configApps) { if (cp == null || configApps == null || !(configApps.equals(CONFIG_INCLUSIVE) || configApps.equals(CONFIG_EXCLUSIVE))) { return ap.getApplications(); } File src = new File(cp.tempDirectory(), "app"); File dest = new File(ap.tempDirectory(), "app"); if (!src.exists()) { return ap.getApplications(); } if (configApps.equals(CONFIG_EXCLUSIVE)) { for (File file : dest.listFiles()) { if (file.getName().endsWith(".json")) { FileUtils.deleteQuietly(new File(dest, file.getName())); } } } else { for (File file : src.listFiles()) { FileUtils.deleteQuietly(new File(dest, file.getName())); } } for (File file : src.listFiles()) { try { FileUtils.moveFileToDirectory(file, dest, true); } catch (IOException e) { LOG.warn("Application from the config file failed while processing.", file.getName()); } } try { FileUtils.deleteDirectory(src); } catch (IOException e) { LOG.warn("Failed to delete the Config Apps folder"); } ap.processAppDirectory(configApps.equals(CONFIG_EXCLUSIVE)); return ap.getApplications(); }
Source:
<code>public static synchronized void cleanup() throws IOException { <START> System.out.println("deleting dirs"); <END> for(File dir : allDirsCreated) { System.err.println("cleaning up " + dir.getCanonicalPath()); recursivelyDelete(dir); } allDirsCreated.clear(); }</code><technical_language>Left diagnostic output</technical_language>

Target:
public static synchronized void cleanup() throws IOException { for (File dir : allDirsCreated) { recursivelyDelete(dir); } allDirsCreated.clear(); }
Source:
<code>private void initPopupCloseOverlay() { if (DEBUG) Log.d(TAG, "initPopupCloseOverlay() called"); if (closeOverlayView != null) return; closeOverlayView = View.inflate(service, R.layout.player_popup_close_overlay, null); closeOverlayButton = closeOverlayView.findViewById(R.id.closeButton); <START> final int layoutParamType = Build.VERSION.SDK_INT < android.os.Build.VERSION_CODES.O ? <END> WindowManager.LayoutParams.TYPE_PHONE : WindowManager.LayoutParams.TYPE_APPLICATION_OVERLAY; final int flags = WindowManager.LayoutParams.FLAG_NOT_FOCUSABLE | WindowManager.LayoutParams.FLAG_NOT_TOUCHABLE | WindowManager.LayoutParams.FLAG_ALT_FOCUSABLE_IM; WindowManager.LayoutParams closeOverlayLayoutParams = new WindowManager.LayoutParams( ViewGroup.LayoutParams.MATCH_PARENT, ViewGroup.LayoutParams.MATCH_PARENT, layoutParamType, flags, PixelFormat.TRANSLUCENT); closeOverlayLayoutParams.gravity = Gravity.LEFT | Gravity.TOP; closeOverlayLayoutParams.softInputMode = WindowManager.LayoutParams.SOFT_INPUT_ADJUST_RESIZE; closeOverlayButton.setVisibility(View.GONE); windowManager.addView(closeOverlayView, closeOverlayLayoutParams); }</code><technical_language>add some kind of function for this, I code is</technical_language>

Target:
private void initPopupCloseOverlay() { if (DEBUG) { Log.d(TAG, "initPopupCloseOverlay() called"); } if (closeOverlayView != null) { return; } closeOverlayView = View.inflate(service, R.layout.player_popup_close_overlay, null); closeOverlayButton = closeOverlayView.findViewById(R.id.closeButton); final int flags = WindowManager.LayoutParams.FLAG_NOT_FOCUSABLE | WindowManager.LayoutParams.FLAG_NOT_TOUCHABLE | WindowManager.LayoutParams.FLAG_ALT_FOCUSABLE_IM; final WindowManager.LayoutParams closeOverlayLayoutParams = new WindowManager.LayoutParams( ViewGroup.LayoutParams.MATCH_PARENT, ViewGroup.LayoutParams.MATCH_PARENT, popupLayoutParamType(), flags, PixelFormat.TRANSLUCENT); closeOverlayLayoutParams.gravity = Gravity.LEFT | Gravity.TOP; closeOverlayLayoutParams.softInputMode = WindowManager.LayoutParams.SOFT_INPUT_ADJUST_RESIZE; closeOverlayButton.setVisibility(View.GONE); windowManager.addView(closeOverlayView, closeOverlayLayoutParams); }
Source:
<code>public CPPASTLiteralExpression copy(CopyStyle style) { CPPASTLiteralExpression copy = new CPPASTLiteralExpression(fKind, <START> this.getValue(), <END> fNumericCompilerSuffixes); copy.fStringLiteralSize = fStringLiteralSize; copy.fEvaluation = fEvaluation; copy.fUserDefinedLiteralOperator = fUserDefinedLiteralOperator; copy.fImplicitNames = fImplicitNames; return copy(copy, style); }</code><technical_language>Redundant qualifier. below</technical_language>

Target:
public CPPASTLiteralExpression copy(CopyStyle style) { CPPASTLiteralExpression copy = new CPPASTLiteralExpression(fKind, getValue().clone(), fNumericCompilerSuffixes); copy.fStringLiteralSize = fStringLiteralSize; copy.fEvaluation = fEvaluation; copy.fUserDefinedLiteralOperator = fUserDefinedLiteralOperator; copy.fImplicitNames = fImplicitNames; return copy(copy, style); }