auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public Aggregator newServerAggregator(Configuration conf) { final PDataType type = getAggregatorExpression().getDataType(); SortOrder sortOrder = getAggregatorExpression().getSortOrder(); return new MaxAggregator(sortOrder) { @Override public PDataType getDataType() { return type; } @Override public Integer getMaxLength() { <START> return getAggregatorExpression().getMaxLength(); <END> } }; }</code><technical_language>Minor nit, a final local variable for getAggregatorExpression() call a number of times</technical_language>

Target:
public Aggregator newServerAggregator(Configuration conf) { final PDataType type = getAggregatorExpression().getDataType(); SortOrder sortOrder = getAggregatorExpression().getSortOrder(); final Integer maxLength = getAggregator().getMaxLength(); return new MaxAggregator(sortOrder) { @Override public PDataType getDataType() { return type; } @Override public Integer getMaxLength() { return maxLength; } }; }
Source:
<code>public boolean equals(Object obj) { return <START> obj instanceof SecureStoreData && storeName.hashCode() == obj.hashCode(); <END> }</code><technical_language>compare hashCode? compare storeName reference equality</technical_language>

Target:
public boolean equals(Object obj) { if (obj instanceof SecureStoreData) { SecureStoreData o = (SecureStoreData) obj; return Objects.equal(storeName, o.storeName); } return equals(obj); }
Source:
<code>public void testSetAndGetExpirationFromDoc() throws CouchbaseLiteException, LiteCoreException { Date dto30 = new Date(System.currentTimeMillis() + 30000L); Date dto0 = new Date(System.currentTimeMillis()); MutableDocument doc1a = new MutableDocument("doc1"); MutableDocument doc1b = new MutableDocument("doc2"); MutableDocument doc1c = new MutableDocument("doc3"); doc1a.setInt("answer", 12); doc1a.setValue("question", "What is six plus six?"); save(doc1a); doc1b.setInt("answer", 22); doc1b.setValue("question", "What is eleven plus eleven?"); save(doc1b); doc1c.setInt("answer", 32); doc1c.setValue("question", "What is twenty plus twelve?"); save(doc1c); assertTrue(db.setDocumentExpiration("doc1", dto30)); assertTrue(db.setDocumentExpiration("doc3", dto30)); assertTrue(db.setDocumentExpiration("doc3", null)); <START> Date v = db.getDocumentExpiration("doc1"); <END> assertSame(v, dto30); assertNull(db.getDocumentExpiration("doc2")); assertNull(db.getDocumentExpiration("doc3")); }</code><technical_language>change v exp</technical_language>

Target:
public void testSetAndGetExpirationFromDoc() throws CouchbaseLiteException, LiteCoreException { Date dto30 = new Date(System.currentTimeMillis() + 30000L); Date dto0 = new Date(System.currentTimeMillis()); MutableDocument doc1a = new MutableDocument("doc1"); MutableDocument doc1b = new MutableDocument("doc2"); MutableDocument doc1c = new MutableDocument("doc3"); doc1a.setInt("answer", 12); doc1a.setValue("question", "What is six plus six?"); save(doc1a); doc1b.setInt("answer", 22); doc1b.setValue("question", "What is eleven plus eleven?"); save(doc1b); doc1c.setInt("answer", 32); doc1c.setValue("question", "What is twenty plus twelve?"); save(doc1c); db.setDocumentExpiration("doc1", dto30); db.setDocumentExpiration("doc3", dto30); db.setDocumentExpiration("doc3", null); Date exp = db.getDocumentExpiration("doc1"); assertSame(exp, dto30); assertNull(db.getDocumentExpiration("doc2")); assertNull(db.getDocumentExpiration("doc3")); }
Source:
<code>boolean isSignedWithSupportedAlgorithm(List<X509Certificate> chain) { <START> for (int index=0; index < chain.size(); ++index) { <END> X509Certificate cert = chain.get(index); boolean certSignatureAlgorithmSupported = false; for (SignatureAndHashAlgorithm supportedAlgorithm : supportedSignatureAlgorithms) { if (supportedAlgorithm.jcaName().equalsIgnoreCase(cert.getSigAlgName())) { certSignatureAlgorithmSupported = true; break; } } if (!certSignatureAlgorithmSupported) { LOGGER.debug("certificate chain is NOT signed with supported algorithm(s)"); return false; } } LOGGER.debug("certificate chain is signed with supported algorithm(s)"); return true; }</code><technical_language>for</technical_language>

Target:
boolean isSignedWithSupportedAlgorithm(List<X509Certificate> chain) { for (X509Certificate certificate : chain) { if (!isSignedWithSupportedAlgorithm(certificate)) { LOGGER.debug("certificate chain is NOT signed with supported algorithm(s)"); return false; } } LOGGER.debug("certificate chain is signed with supported algorithm(s)"); return true; }
Source:
<code>@Nonnull default Set<User> getCulprits() { if (shouldCalculateCulprits()) { return calculateCulprits(); } return new AbstractSet<User>() { public Iterator<User> iterator() { return new AdaptedIterator<String,User>(getCulpritIds().iterator()) { protected User adapt(String id) { return User.get(id); } }; } public int size() { <START> return getCulpritIds().size(); <END> } }; }</code><technical_language>culpritIds computed returning, no? Potentially change while iterating</technical_language>

Target:
@Nonnull default Set<User> getCulprits() { if (shouldCalculateCulprits()) { return calculateCulprits(); } return new AbstractSet<User>() { private Set<String> culpritIds = ImmutableSet.copyOf(getCulpritIds()); public Iterator<User> iterator() { return new AdaptedIterator<String,User>(culpritIds.iterator()) { protected User adapt(String id) { return User.get(id); } }; } public int size() { return culpritIds.size(); } }; }
Source:
<code>public void deleteText(@Nonnull Mapping field, String lang) { <START> fetchTranslation(field, lang).ifPresent(mongoTranslation -> { <END> mango.delete(mongoTranslation); if (owner instanceof Journaled) { JournalData.addJournalEntry(owner, "Deleted translated text for " + field.getName() + " (" + lang + ")"); } }); }</code><technical_language>coundn' t mongo.delete().eq....eq...().executeFor(...) here</technical_language>

Target:
public void deleteText(@Nonnull Mapping field, String lang) { mongo.delete() .where(Translation.TRANSLATION_DATA.inner(TranslationData.OWNER), owner.getUniqueName()) .where(Translation.TRANSLATION_DATA.inner(TranslationData.FIELD), field.getName()) .where(Translation.TRANSLATION_DATA.inner(TranslationData.LANG), lang) .singleFrom(MongoTranslation.class); if (owner instanceof Journaled) { JournalData.addJournalEntry(owner, "Deleted translated text for " + field.getName() + " (" + lang + ")"); } }
Source:
<code>public void execute(final JobExecutionContext jobContext) throws JobExecutionException { SpringBeanAutowiringSupport.processInjectionBasedOnCurrentContext(this); LOGGER.info("********* Starting the Inheritance Error Report Creator job. *********"); curesRuleEffectiveDate = getCuresRuleEffectiveDate(); if (curesRuleEffectiveDate == null) { return; } inheritanceErrorsReportDAO.deleteAll(); List<CertifiedProductFlatSearchResult> listings = certifiedProductSearchDAO.getAllCertifiedProducts(); List<CertifiedProductFlatSearchResult> certifiedProducts = filterData(listings); <START> certifiedProducts = certifiedProducts.subList(0, 100); <END> ExecutorService executorService = null; try { executorService = Executors.newFixedThreadPool(8); for (CertifiedProductFlatSearchResult result : certifiedProducts) { CompletableFuture.supplyAsync(() -> getCertifiedProductSearchDetails(result.getId()), executorService) .thenApply(cp -> check(cp)) .thenAccept(error -> saveInheritanceErrorsReportSingle(error)); } } finally { executorService.shutdown(); } LOGGER.info("Completed the Inheritance Error Report Creator job. *********"); }</code><technical_language>This for testing removed</technical_language>

Target:
public void execute(final JobExecutionContext jobContext) throws JobExecutionException { SpringBeanAutowiringSupport.processInjectionBasedOnCurrentContext(this); LOGGER.info("********* Starting the Inheritance Error Report Creator job. *********"); curesRuleEffectiveDate = getCuresRuleEffectiveDate(); if (curesRuleEffectiveDate == null) { return; } inheritanceErrorsReportDAO.deleteAll(); List<CertifiedProductFlatSearchResult> listings = certifiedProductSearchDAO.getAllCertifiedProducts(); List<CertifiedProductFlatSearchResult> certifiedProducts = filterData(listings); ExecutorService executorService = null; try { Integer threadPoolSize = getThreadCountForJob(); executorService = Executors.newFixedThreadPool(threadPoolSize); for (CertifiedProductFlatSearchResult result : certifiedProducts) { CompletableFuture.supplyAsync(() -> getCertifiedProductSearchDetails(result.getId()), executorService) .thenApply(cp -> check(cp)) .thenAccept(error -> saveInheritanceErrorsReportSingle(error)); } } catch (Exception e) { LOGGER.error(e.getMessage(), e); } finally { executorService.shutdown(); } LOGGER.info("Completed the Inheritance Error Report Creator job. *********"); }
Source:
<code>public DocumentModel publishRendition(DocumentModel doc, DocumentModel target, String renditionName, boolean override) { CoreSession session = doc.getCoreSession(); if (!session.hasPermission(target.getRef(), ADD_CHILDREN)) { <START> log.debug("Permission '" + ADD_CHILDREN + "' is not granted to '" + session.getPrincipal().getName() <END> + "' on document " + target.getPath()); throw new DocumentSecurityException( "Privilege '" + ADD_CHILDREN + "' is not granted to '" + session.getPrincipal().getName() + "'"); } Rendition rendition = StringUtils.isEmpty(renditionName) ? getDefaultRendition(doc, Constants.DEFAULT_RENDTION_PUBLISH_REASON, true, null) : getRendition(doc, renditionName, true); if (rendition == null) { throw new NuxeoException("Unable to render the document"); } DocumentModel renditionDocument = rendition.getHostDocument(); DocumentRef publishedDocumentRef = CoreInstance.doPrivileged(session, (CoreSession s) -> s.publishDocument(renditionDocument, target, override).getRef()); DocumentModel publishedDocument = session.getDocument(publishedDocumentRef); if (override) { RenditionsRemover remover = new RenditionsRemover(publishedDocument); remover.runUnrestricted(); } return publishedDocument; }</code><technical_language>log4j2</technical_language>

Target:
public DocumentModel publishRendition(DocumentModel doc, DocumentModel target, String renditionName, boolean override) { CoreSession session = doc.getCoreSession(); if (!session.hasPermission(target.getRef(), ADD_CHILDREN)) { log.error("Permission '{}' is not granted to '{}' on document '{}'", ADD_CHILDREN, session.getPrincipal().getName(), target.getPath()); throw new DocumentSecurityException( "Privilege '" + ADD_CHILDREN + "' is not granted to '" + session.getPrincipal().getName() + "'"); } Rendition rendition = StringUtils.isEmpty(renditionName) ? getDefaultRendition(doc, Constants.DEFAULT_RENDTION_PUBLISH_REASON, true, null) : getRendition(doc, renditionName, true); if (rendition == null) { throw new NuxeoException("Unable to render the document"); } DocumentModel renditionDocument = rendition.getHostDocument(); DocumentRef publishedDocumentRef = CoreInstance.doPrivileged(session, (CoreSession s) -> s.publishDocument(renditionDocument, target, override).getRef()); DocumentModel publishedDocument = session.getDocument(publishedDocumentRef); if (override) { RenditionsRemover remover = new RenditionsRemover(publishedDocument); remover.runUnrestricted(); } return publishedDocument; }
Source:
<code>public void removePacketInterceptor(StanzaListener packetInterceptor) { <START> synchronized (interceptors) { <END> interceptors.remove(packetInterceptor); } }</code><technical_language>This simply invoke new method</technical_language>

Target:
public void removePacketInterceptor(StanzaListener packetInterceptor) { removeStanzaInterceptor(packetInterceptor); }
Source:
<code>public boolean hasNext() { <START> boolean hasNext = !queue.isEmpty(); <END> boolean interrupted = false; long targetTime = timeService.expectedEndTime(timeout, unit); nextLock.lock(); try { while (!(hasNext = !queue.isEmpty()) && !completed) { try { if (!nextCondition.await(timeService.remainingTime(targetTime, TimeUnit.NANOSECONDS), TimeUnit.NANOSECONDS)) { if (log.isTraceEnabled()) { log.tracef("Did not retrieve entries in allotted timeout: %s units: unit", timeout, unit); } throw new TimeoutException("Did not retrieve entries in allotted timeout: " + timeout + " units: " + unit); } } catch (InterruptedException e) { interrupted = true; } } } finally { nextLock.unlock(); } if (interrupted) { Thread.currentThread().interrupt(); } return hasNext; }</code><technical_language>This longer needs initial value</technical_language>

Target:
public boolean hasNext() { boolean hasNext = !queue.isEmpty(); if (!hasNext) { boolean interrupted = false; long targetTime = timeService.expectedEndTime(timeout, unit); nextLock.lock(); try { while (!(hasNext = !queue.isEmpty()) && !completed) { try { if (!nextCondition.await(timeService.remainingTime(targetTime, TimeUnit.NANOSECONDS), TimeUnit.NANOSECONDS)) { if (log.isTraceEnabled()) { log.tracef("Did not retrieve entries in allotted timeout: %s units: unit", timeout, unit); } throw new TimeoutException("Did not retrieve entries in allotted timeout: " + timeout + " units: " + unit); } } catch (InterruptedException e) { interrupted = true; } } } finally { nextLock.unlock(); } if (interrupted) { Thread.currentThread().interrupt(); } } return hasNext; }
Source:
<code>public Object execute(ExecutionEvent event) throws ExecutionException { IProject project = null; IEditorPart editorPart = HandlerUtil.getActiveWorkbenchWindow(event).getActivePage().getActiveEditor(); if (editorPart != null) { IFileEditorInput input = (IFileEditorInput) editorPart.getEditorInput(); IFile file = input.getFile(); project = file.getProject(); } else { IWorkbenchWindow window = HandlerUtil.getActiveWorkbenchWindow(event); if (window != null) { IStructuredSelection selection = (IStructuredSelection) window.getSelectionService().getSelection(); if (selection instanceof TreeSelection) { TreeSelection treeSelection = (TreeSelection) selection; TreePath path = treeSelection.getPaths()[0]; Object first = path.getFirstSegment(); if (first instanceof IAdaptable) { project = ((IAdaptable) first).getAdapter(IProject.class); } } else { Object firstElement = selection.getFirstElement(); if (firstElement instanceof IAdaptable) { project = ((IAdaptable) firstElement).getAdapter(IProject.class); } } } } if (project != null) { IFile file = project.getFile("/gruntfile.js"); if (file.exists()) { <START> StructuredSelection fileSelection = new StructuredSelection(file); <END> GruntLaunch launch = new GruntLaunch(); launch.launch(fileSelection, null); } } return null; }</code><technical_language>This is causing GruntLaunch run default task. if is a selection selection is ITask, I pass GruntLaunch</technical_language>

Target:
public Object execute(ExecutionEvent event) throws ExecutionException { ISelection selection = WorkbenchResourceUtil.getNamedFileOrTaskSelection(GruntConstants.GRUNT_FILE_JS); if (selection != null) { new GruntLaunch().launch(selection, GruntConstants.DEFAULT_COMMAND); } return null; }
Source:
<code>private ClassLoader createLoaderForJar(URL jarURL) { <START> LOGGER.debug("Creating Loader For jar: " + jarURL); <END> ClassLoader jarLoader = new URLClassLoader(enumerateJar(jarURL), this); if (jarLoader == null) { LOGGER.warn("No jar found with url: " + jarURL); } return jarLoader; }</code><technical_language>fix string concatenation in this file</technical_language>

Target:
private ClassLoader createLoaderForJar(URL jarURL) { LOGGER.debug("Creating Loader For jar: {}", jarURL); ClassLoader jarLoader = new URLClassLoader(enumerateJar(jarURL), this); if (jarLoader == null) { LOGGER.warn("No jar found with url: {}", jarURL); } return jarLoader; }
Source:
<code><START> static <E,T> Validation<E,T> valid(T value) { <END> return new Valid<>(value); }</code><technical_language>ok, null</technical_language>

Target:
static <E,T> Validation<E,T> valid(T value) { Objects.requireNonNull(value, "value is null"); return new Valid<>(value); }
Source:
<code>public void beforeSlim218(Scenario scenario) { LOGGER.info("Preparing scenario for @SLIM-218"); final ClassPathXmlApplicationContext context = new ClassPathXmlApplicationContext("cucumber.xml"); this.deviceRepository = context.getBean(DeviceRepository.class); this.dlmsDeviceRepository = context.getBean(DlmsDeviceRepository.class); this.deviceAuthorizationRepository = context.getBean(DeviceAuthorizationRepository.class); <START> this.deleteDevicesFromPlatform(Arrays.asList("E0026000059790003", "G00XX561204926019")); <END> LOGGER.info("Ready preparing scenario for @SLIM-218"); context.close(); }</code><technical_language>rewrite scenario: Feature: a grid operator I want able perform SmartMeteringInstallation operations a device Background: organisation OrganisationID "LianderNetManagement" @SLIM-218 Scenario: Add a new device "E0026000059790003" a device DeviceID "E0026000059790003" add device request "E0026000059790003" is received device request response ok device "E0026000059790003" added in core database device "E0026000059790003" added in dlms database @SLIM-218 Scenario: Add a new device "G00XX561204926013" a device DeviceID "G00XX561204926013" add device request "G00XX561204926013" is received device request response ok device "G00XX561204926013" added in core database device "G00XX561204926013" added in dlms database this device id is part of scenario name; retrieve scenario.getName()</technical_language>

Target:
public void beforeSlim218(Scenario scenario) { LOGGER.info("Preparing scenario for @SLIM-218"); LOGGER.info("Scenario name: {}", scenario.getName()); final ClassPathXmlApplicationContext context = new ClassPathXmlApplicationContext("cucumber.xml"); this.deviceRepository = context.getBean(DeviceRepository.class); this.dlmsDeviceRepository = context.getBean(DlmsDeviceRepository.class); this.deviceAuthorizationRepository = context.getBean(DeviceAuthorizationRepository.class); this.deleteDevicesFromPlatform(Arrays.asList("E0026000059790003", "G00XX561204926019")); LOGGER.info("Ready preparing scenario for @SLIM-218"); context.close(); }
Source:
<code>private void showSelection(boolean showHead, boolean showRef, boolean showTag, Repository repo, RevCommit selection, Ref ref, Job initJob) { if (initJob == null) { return; } initJob.addJobChangeListener(new JobChangeAdapter() { @Override public void done(IJobChangeEvent event) { if (event.getResult() == null || !event.getResult().isOK()) { return; } Control control = graph.getControl(); if (control.isDisposed()) { return; } control.getDisplay().asyncExec(() -> { if (control.isDisposed()) { return; } <START> if (showHead) { <END> showHead(repo); } if (showRef) { showRef(ref, repo); } if (showTag) { showTag(ref, repo); } if (selection != null) { graph.selectCommitStored(selection); } }); } }); }</code><technical_language>Previously, this done new job started. unnecessary, new code do this in case</technical_language>

Target:
private void showSelection(boolean showHead, boolean showRef, boolean showTag, Repository repo, RevCommit selection, Ref ref, Job initJob) { if (initJob == null) { return; } initJob.addJobChangeListener(new JobChangeAdapter() { @Override public void done(IJobChangeEvent event) { if (event.getResult() == null || !event.getResult().isOK() || job != event.getJob()) { return; } Control control = graph.getControl(); if (control.isDisposed()) { return; } control.getDisplay().asyncExec(() -> { if (control.isDisposed()) { return; } if (showHead) { showHead(repo); } if (showRef) { showRef(ref, repo); } if (showTag) { showTag(ref, repo); } if (selection != null) { graph.selectCommitStored(selection); } }); } }); }
Source:
<code>private void send(String message, String vmID, boolean removeClient) throws ClientConnectionException{ try { ReactorClient client = JsonRpcServer.getReactorsMap().get(vmID); client.sendMessage(message.getBytes()); log.debug("sending events message {}", message); }catch (ClientConnectionException e) { <START> log.error("Failure in sending response", e); <END> } if (removeClient){ JsonRpcServer.removeReactorsMap(vmID); } }</code><technical_language>debug perspective good add some info host</technical_language>

Target:
private void send(String message, String vmID, boolean removeClient) throws ClientConnectionException{ String hostName = null; try { ReactorClient client = JsonRpcServer.getReactorsMap().get(vmID); hostName = client.getHostname(); client.sendMessage(message.getBytes()); log.debug("sending events message {}", message); }catch (ClientConnectionException e) { log.error("Host {}, failed to send event message {}", hostName, e); } if (removeClient){ JsonRpcServer.removeReactorsMap(vmID); } }
Source:
<code>public Consumer createConsumer(Processor processor) throws Exception { afterPropertiesSet(); setExchangePattern(ExchangePattern.InOnly); <START> return new GooglePubsubConsumer(this, processor); <END> }</code><technical_language>need call configureConsumer</technical_language>

Target:
public Consumer createConsumer(Processor processor) throws Exception { afterPropertiesSet(); setExchangePattern(ExchangePattern.InOnly); GooglePubsubConsumer consumer = new GooglePubsubConsumer(this, processor); configureConsumer(consumer); return consumer; }
Source:
<code>private AmazonPolly getPolly() { if (defaultRegion == null) { defaultRegion = Regions.DEFAULT_REGION; } String key = getKey(AMAZON_POLLY_USER_KEY); String secret = getKey(AMAZON_POLLY_USER_SECRET); if (key == null || secret == null) { <START> error("this service requires 2 keys"); <END> return null; } if (polly == null) { try { if (credentials == null) { credentials = new BasicAWSCredentials(key, secret); } polly = AmazonPollyClientBuilder.standard().withCredentials(new AWSStaticCredentialsProvider(credentials)).withRegion(defaultRegion).build(); } catch (Exception e) { try { log.error("could not get client with keys supplied - trying default chain", e); polly = AmazonPollyClient.builder().withCredentials(new DefaultAWSCredentialsProviderChain()).withClientConfiguration(new ClientConfiguration()).withRegion(defaultRegion) .build(); } catch (Exception e2) { error("could not get Polly client - did you set the keys?"); log.error("giving up", e2); polly = null; credentials = null; } } } if (polly != null) { try { DescribeVoicesRequest describeVoicesRequest = new DescribeVoicesRequest(); polly.describeVoices(describeVoicesRequest); } catch (AmazonPollyException e3) { error("The keys are invalid"); polly = null; credentials = null; } } return polly; }</code><technical_language>technically speaking this service requires a key a secret.</technical_language>

Target:
private AmazonPolly getPolly() { if (defaultRegion == null) { defaultRegion = Regions.DEFAULT_REGION; } String key = getKey(AMAZON_POLLY_USER_KEY); String secret = getKey(AMAZON_POLLY_USER_SECRET); if (key == null || secret == null) { error("this service requires a key and a secret"); return null; } if (polly == null) { try { if (credentials == null) { credentials = new BasicAWSCredentials(key, secret); } polly = AmazonPollyClientBuilder.standard().withCredentials(new AWSStaticCredentialsProvider(credentials)).withRegion(defaultRegion).build(); } catch (Exception e) { try { log.error("could not get client with keys supplied - trying default chain", e); polly = AmazonPollyClient.builder().withCredentials(new DefaultAWSCredentialsProviderChain()).withClientConfiguration(new ClientConfiguration()).withRegion(defaultRegion) .build(); } catch (Exception e2) { error("could not get Polly client - did you set the keys?"); log.error("giving up", e2); polly = null; credentials = null; } } } if (polly != null) { try { DescribeVoicesRequest describeVoicesRequest = new DescribeVoicesRequest(); polly.describeVoices(describeVoicesRequest); } catch (AmazonPollyException e3) { error("The keys are invalid"); polly = null; credentials = null; } } return polly; }
Source:
<code>protected void init() { if (properties == null) { properties = new DescribableList<AbstractFolderProperty<?>,AbstractFolderPropertyDescriptor>(this); } else { <START> properties.setOwner(this); <END> } for (AbstractFolderProperty p : properties) { p.setOwner(this); } if (icon == null) { icon = new StockFolderIcon(); } if (views == null) { views = new CopyOnWriteArrayList<View>(); } if (views.isEmpty()) { try { initViews(views); } catch (IOException e) { LOGGER.log(Level.WARNING, "Failed to set up the initial view", e); } } if (viewsTabBar == null) { viewsTabBar = new DefaultViewsTabBar(); } if (primaryView == null) { primaryView = views.get(0).getViewName(); } viewGroupMixIn = new ViewGroupMixIn(this) { @Override protected List<View> views() { return views; } @Override protected String primaryView() { return primaryView; } @Override protected void primaryView(String name) { primaryView = name; } }; if (healthMetrics == null) { List<FolderHealthMetric> metrics = new ArrayList<FolderHealthMetric>(); for (FolderHealthMetricDescriptor d : FolderHealthMetricDescriptor.all()) { FolderHealthMetric metric = d.createDefault(); if (metric != null) { metrics.add(metric); } } healthMetrics = new DescribableList<FolderHealthMetric, FolderHealthMetricDescriptor>(this, metrics); } }</code><technical_language>is this done init (called onLoad()) in readResolve</technical_language>

Target:
protected void init() { if (properties == null) { properties = new DescribableList<AbstractFolderProperty<?>,AbstractFolderPropertyDescriptor>(this); } else { properties.setOwner(this); } for (AbstractFolderProperty p : properties) { p.setOwner(this); } if (icon == null) { icon = new StockFolderIcon(); } if (views == null) { views = new CopyOnWriteArrayList<View>(); } if (viewsTabBar == null) { viewsTabBar = new DefaultViewsTabBar(); } viewGroupMixIn = new ViewGroupMixIn(this) { @Override protected List<View> views() { return views; } @Override protected String primaryView() { return primaryView == null ? views.get(0).getViewName() : primaryView; } @Override protected void primaryView(String name) { primaryView = name; } }; if (views.isEmpty()) { try { initViews(views); } catch (IOException e) { LOGGER.log(Level.WARNING, "Failed to set up the initial view", e); } } if (healthMetrics == null) { List<FolderHealthMetric> metrics = new ArrayList<FolderHealthMetric>(); for (FolderHealthMetricDescriptor d : FolderHealthMetricDescriptor.all()) { FolderHealthMetric metric = d.createDefault(); if (metric != null) { metrics.add(metric); } } healthMetrics = new DescribableList<FolderHealthMetric, FolderHealthMetricDescriptor>(this, metrics); } }
Source:
<code>public CloseableIterator<CacheEntry<K, V>> iterator() { Iterator<CacheEntry<K, V>> iterator = new DataContainerRemoveIterator<>(cache); <START> return Closeables.iterator(iterator); <END> }</code><technical_language>happened forwarding entry</technical_language>

Target:
public CloseableIterator<CacheEntry<K, V>> iterator() { Iterator<CacheEntry<K, V>> iterator = new DataContainerRemoveIterator<>(cache); RemovableIterator<K, CacheEntry<K, V>> removableIterator = new RemovableIterator<>(iterator, cache, CacheEntry::getKey); return Closeables.iterator(new IteratorMapper<>(removableIterator, e -> new EntryWrapper<>(cache, e))); }
Source:
<code>private boolean isOptionEnabledOnVolume(GlusterVolumeEntity volume, List<PredicateFilter<GlusterVolumeOptionEntity>> predicates) { List<GlusterVolumeOptionEntity> volumeOptionsEnabled = new ArrayList<GlusterVolumeOptionEntity>(volume.getOptions()); List<GlusterVolumeOptionEntity> filteredOptions = volumeOptionsEnabled; for(PredicateFilter<GlusterVolumeOptionEntity> predicate : predicates) { <START> filteredOptions = ListUtils.filter(volumeOptionsEnabled, predicate); <END> } if(filteredOptions.size() > 0) { return true; } return false; }</code><technical_language>this filteredOptions = ListUtils.filter(filteredOptions, predicate);</technical_language>

Target:
private boolean isOptionEnabledOnVolume(GlusterVolumeEntity volume, List<PredicateFilter<GlusterVolumeOptionEntity>> predicates) { List<GlusterVolumeOptionEntity> volumeOptionsEnabled = new ArrayList<GlusterVolumeOptionEntity>(volume.getOptions()); List<GlusterVolumeOptionEntity> filteredOptions = volumeOptionsEnabled; for(PredicateFilter<GlusterVolumeOptionEntity> predicate : predicates) { filteredOptions = ListUtils.filter(filteredOptions, predicate); } if(filteredOptions.size() > 0) { return true; } return false; }
Source:
<code>protected HttpManagement(InterruptingObserver interruptingObserver, String userAgent) { this.interruptingObserver = interruptingObserver; <START> this.client = generateHttpClientBuilder().setUserAgent(userAgent).build(); <END> }</code><technical_language>userAgent is set now, [Line 195](<LINK_0> removed. is initialized, null anyway</technical_language>

Target:
protected HttpManagement(InterruptingObserver interruptingObserver, String userAgent) { this.interruptingObserver = interruptingObserver; this.userAgent = userAgent; this.client = generateHttpClientBuilder().build(); }
Source:
<code>protected void onCreate(Bundle savedInstanceState) { super.onCreate(savedInstanceState); setContentView(R.layout.activity_result); mTextView = (TextView) findViewById(R.id.resultInfo); mTextView.setMovementMethod(new ScrollingMovementMethod()); String resultText; try { resultText = convertIntentDataToJsonString(); } catch (final JSONException e) { resultText = "{\"error \" : \"Unable to convert to JSON\"}"; } mTextView.setText(resultText); mLogView = (TextView) findViewById(R.id.adalLogs); <START> mLogView.setMovementMethod(new ScrollingMovementMethod()); <END> mLogView.setText(getLogsFromIntent()); final Button doneButton = (Button) findViewById(R.id.resultDone); doneButton.setOnClickListener(new View.OnClickListener() { @Override public void onClick(View v) { ResultActivity.this.finish(); } }); }</code><technical_language>do need log text box scrollable</technical_language>

Target:
protected void onCreate(Bundle savedInstanceState) { super.onCreate(savedInstanceState); setContentView(R.layout.activity_result); mTextView = (TextView) findViewById(R.id.resultInfo); mTextView.setMovementMethod(new ScrollingMovementMethod()); String resultText; try { resultText = convertIntentDataToJsonString(); } catch (final JSONException e) { resultText = "{\"error \" : \"Unable to convert to JSON\"}"; } mTextView.setText(resultText); mLogView = (TextView) findViewById(R.id.adalLogs); mLogView.setText(((AndroidAutomationApp)this.getApplication()).getADALLogs()); final Button doneButton = (Button) findViewById(R.id.resultDone); doneButton.setOnClickListener(new View.OnClickListener() { @Override public void onClick(View v) { ResultActivity.this.finish(); } }); }
Source:
<code>@Override protected void onCreate(@Nullable Bundle savedInstanceState) { super.onCreate(savedInstanceState); setContentView(R.layout.activity_main); mMapView = findViewById(R.id.mapView); mEyesRecyclerView = findViewById(R.id.eyesRecyclerView); mMouthRecyclerView = findViewById(R.id.mouthRecyclerView); mHatRecyclerView = findViewById(R.id.hatRecyclerView); mPreviewView = findViewById(R.id.previewView); ArcGISMap map = new ArcGISMap(Basemap.createTopographic()); mMapView.setMap(map); mGraphicsOverlay = new GraphicsOverlay(); mMapView.getGraphicsOverlays().add(mGraphicsOverlay); mMapView.setOnTouchListener( new DefaultMapViewOnTouchListener(MainActivity.this, mMapView) { @Override public boolean onSingleTapConfirmed(MotionEvent motionEvent) { addGraphic(mGraphicsOverlay, mapPointFrom(mMapView, motionEvent), mCurrentMultilayerSymbol); return true; } }); setupRecyclerViews(); symbolStyleSearchResultObservable.addObserver((o, arg) -> { for (SymbolStyleSearchResult symbol : ((SymbolStyleSearchResultObservable) o).getSymbols()) { switch (symbol.getCategory().toLowerCase(Locale.ROOT)) { case "eyes": mEyesAdapter.addSymbol(symbol); break; case "mouth": mMouthAdapter.addSymbol(symbol); break; case "hat": mHatAdapter.addSymbol(symbol); break; case "face": mFaceSymbolKey = symbol.getKey(); break; } <START> Log.d(TAG, symbol.getCategory()); <END> } animateRecyclerViews(); }); requestPermissions(); }</code><technical_language>This is useful for now, lets remove if need for final sample</technical_language>

Target:
@Override protected void onCreate(@Nullable Bundle savedInstanceState) { super.onCreate(savedInstanceState); setContentView(R.layout.activity_main); mMapView = findViewById(R.id.mapView); mEyesRecyclerView = findViewById(R.id.eyesRecyclerView); mMouthRecyclerView = findViewById(R.id.mouthRecyclerView); mHatRecyclerView = findViewById(R.id.hatRecyclerView); mPreviewView = findViewById(R.id.previewView); ArcGISMap map = new ArcGISMap(Basemap.createTopographic()); mMapView.setMap(map); mGraphicsOverlay = new GraphicsOverlay(); mMapView.getGraphicsOverlays().add(mGraphicsOverlay); mMapView.setOnTouchListener( new DefaultMapViewOnTouchListener(MainActivity.this, mMapView) { @Override public boolean onSingleTapConfirmed(MotionEvent motionEvent) { addGraphic(mGraphicsOverlay, mapPointFrom(mMapView, motionEvent), mCurrentMultilayerSymbol); return true; } }); setupRecyclerViews(); mSymbolStyleSearchResultObservable.addObserver((o, arg) -> { for (SymbolStyleSearchResult symbol : ((SymbolStyleSearchResultObservable) o).getSymbols()) { switch (symbol.getCategory().toLowerCase(Locale.ROOT)) { case "eyes": mEyesAdapter.addSymbol(symbol); break; case "mouth": mMouthAdapter.addSymbol(symbol); break; case "hat": mHatAdapter.addSymbol(symbol); break; case "face": mFaceSymbolKey = symbol.getKey(); break; } } animateRecyclerViews(); }); requestPermissions(); }
Source:
<code>public FindEventsQueryDto makeFindEventsQueryDto() { FindEventsQueryDto result = new FindEventsQueryDto(EventLogCategoryDto.STANDARD_EVENT_LOG, new DateTime(), new DateTime()); <START> return result; <END> }</code><technical_language>return object immediately. Sonar complain. This counts for methods well</technical_language>

Target:
public FindEventsQueryDto makeFindEventsQueryDto() { return new FindEventsQueryDto(EventLogCategoryDto.STANDARD_EVENT_LOG, new DateTime(), new DateTime()); }
Source:
<code>public static boolean isZero(final Number number) { if (Objects.isNull(number)) { return false; } else if (number instanceof Integer) { return number.intValue() == INTEGER_ZERO; } else if (number instanceof Long) { return number.longValue() == LONG_ZERO; } else if (number instanceof Byte) { return number.byteValue() == BYTE_ZERO; } else if (number instanceof Short) { return number.shortValue() == SHORT_ZERO; } else if (number instanceof Float) { return number.floatValue() == FLOAT_ZERO; } else if (number instanceof Double) { return number.doubleValue() == DOUBLE_ZERO; } <START> return false; <END> }</code><technical_language>This is wrong IMO, if I create a subclass of Number, this method return false if Number's intValue() returns 0</technical_language>

Target:
public static boolean isZero(final Number number) { if (Objects.isNull(number)) { return false; } else if (number instanceof Integer) { return number.intValue() == INTEGER_ZERO; } else if (number instanceof Long || number instanceof LongAccumulator || number instanceof LongAdder) { return number.longValue() == LONG_ZERO; } else if (number instanceof Byte) { return number.byteValue() == BYTE_ZERO; } else if (number instanceof Short) { return number.shortValue() == SHORT_ZERO; } else if (number instanceof Float) { return number.floatValue() == FLOAT_ZERO; } else if (number instanceof Double || number instanceof DoubleAccumulator || number instanceof DoubleAdder) { return number.doubleValue() == DOUBLE_ZERO; } else if (number instanceof BigInteger) { return ((BigInteger) number).signum() == INTEGER_ZERO; } else if (number instanceof BigDecimal) { return ((BigDecimal) number).compareTo(BigDecimal.ZERO) == INTEGER_ZERO; } else if (number instanceof AtomicInteger) { return ((AtomicInteger) number).get() == INTEGER_ZERO; } else if (number instanceof AtomicLong) { return ((AtomicLong) number).get() == LONG_ZERO; } else { return number.intValue() == INTEGER_ZERO && number.longValue() == LONG_ZERO && number.byteValue() == BYTE_ZERO && number.shortValue() == SHORT_ZERO && number.floatValue() == FLOAT_ZERO && number.doubleValue() == DOUBLE_ZERO; } }
Source:
<code>void closeSocket() { if (sock != null && !sock.isClosed() && sockBeingClosed.compareAndSet(false, true)) { <START> LOG.error("Unexpected exception causing shutdown while sock is still open"); <END> if (closeSocketAsync) { closeSockAsync(); } else { closeSockSync(); } } }</code><technical_language>if I right, here, logged out exception, if any. get shutdown() function, close socket catching exceptions. instead of LOG.error("Unexpected exception... print out a LOG.info closing socket, indicating in logs if do sync async</technical_language>

Target:
void closeSocket() { if (sock != null && !sock.isClosed() && sockBeingClosed.compareAndSet(false, true)) { if (closeSocketAsync) { LOG.info("Asynchronously closing socket to learner {}.", getSid()); closeSockAsync(); } else { LOG.info("Synchronously closing socket to learner {}.", getSid()); closeSockSync(); } } }
Source:
<code>public Object execute(final ExecutionEvent event) throws ExecutionException { final RepositoryTreeNode node = getSelectedNodes(event).get(0); if (!(node.getObject() instanceof Ref)) return null; final Ref ref = (Ref) node.getObject(); Repository repo = node.getRepository(); String refName = ref.getLeaf().getName(); final BranchOperationUI op; <START> if (refName.startsWith(Constants.R_HEADS)) <END> op = new BranchOperationUI(repo, ref.getName()); else op = new BranchOperationUI(repo, ref.getLeaf().getObjectId()); op.start(); return null; }</code><technical_language>needed (see remark in BranchOperationUI)</technical_language>

Target:
public Object execute(final ExecutionEvent event) throws ExecutionException { final RepositoryTreeNode node = getSelectedNodes(event).get(0); if (!(node.getObject() instanceof Ref)) return null; final Ref ref = (Ref) node.getObject(); Repository repo = node.getRepository(); String refName = ref.getLeaf().getName(); final BranchOperationUI op; if (refName.startsWith(Constants.R_REFS)) op = new BranchOperationUI(repo, ref.getName()); else op = new BranchOperationUI(repo, ref.getLeaf().getObjectId()); op.start(); return null; }
Source:
<code><START> public void twoLaunchesTerminate() throws Throwable { <END> Assume.assumeFalse("Test framework only supports multiple launches for non-remote", remote); doLaunch(EXEC_PATH + EXEC_NAME); GdbLaunch launch1 = getGDBLaunch(); GdbLaunch launch2 = doLaunchInner(); launch1.terminate(); launch2.terminate(); long endTime = System.currentTimeMillis() + TestsPlugin.massageTimeout(2000); boolean allTerminated = false; try { while (System.currentTimeMillis() < endTime) { if (launch1.isTerminated() && launch2.isTerminated()) { allTerminated = true; GdbDebugOptions.trace("Both launches terminated\n"); } assertSessionExecutorResponsive(launch1, "launch1", endTime - System.currentTimeMillis(), TimeUnit.MILLISECONDS); assertSessionExecutorResponsive(launch2, "launch2", endTime - System.currentTimeMillis(), TimeUnit.MILLISECONDS); if (allTerminated) { break; } } } catch (TimeoutException te) { throw new Exception("Timeout while trying to access executor thread", te); } assertTrue("Timeout waiting for launches to terminate", allTerminated); }</code><technical_language>this test fail solution</technical_language>

Target:
public void twoLaunchesTerminate() throws Throwable { Assume.assumeFalse("Test framework only supports multiple launches for non-remote", remote); doLaunch(EXEC_PATH + EXEC_NAME); GdbLaunch launch1 = getGDBLaunch(); GdbLaunch launch2 = doLaunchInner(); launch1.terminate(); launch2.terminate(); long endTime = System.currentTimeMillis() + TestsPlugin.massageTimeout(2000); boolean allTerminated = false; try { while (System.currentTimeMillis() < endTime) { if (launch1.isTerminated() && launch2.isTerminated()) { allTerminated = true; GdbDebugOptions.trace("Both launches terminated\n"); } assertSessionExecutorResponsive(launch1, "launch1", endTime - System.currentTimeMillis(), TimeUnit.MILLISECONDS); assertSessionExecutorResponsive(launch2, "launch2", endTime - System.currentTimeMillis(), TimeUnit.MILLISECONDS); if (allTerminated) { break; } Thread.sleep(100); } } catch (TimeoutException te) { throw new Exception("Timeout while trying to access executor thread", te); } assertTrue("Timeout waiting for launches to terminate", allTerminated); }
Source:
<code>public void setNamedParams(Map<String, String> params) { if (params != null) { List<Map<String, String>> paramsProperty = getPropertyValue(SavedSearchConstants.NAMED_PARAMS_PROPERTY_NAME); if (paramsProperty == null) { paramsProperty = new ArrayList<>(); } Map<String, String> variable; for (String key : params.keySet()) { String value = params.get(key); <START> variable = new HashMap<>(1); <END> variable.put("key", key); variable.put("value", value); paramsProperty.add(variable); } doc.setPropertyValue(SavedSearchConstants.NAMED_PARAMS_PROPERTY_NAME, (Serializable) paramsProperty); } }</code><technical_language>If possible, Collections.singletonMap(key, value) here</technical_language>

Target:
public void setNamedParams(Map<String, String> params) { if (params != null) { List<Map<String, String>> paramsProperty = getPropertyValue(SavedSearchConstants.NAMED_PARAMS_PROPERTY_NAME); if (paramsProperty == null) { paramsProperty = new ArrayList<>(); } Map<String, String> variable; for (String key : params.keySet()) { String value = params.get(key); variable = new HashMap<>(2); variable.put("key", key); variable.put("value", value); paramsProperty.add(variable); } doc.setPropertyValue(SavedSearchConstants.NAMED_PARAMS_PROPERTY_NAME, (Serializable) paramsProperty); } }
Source:
<code>public ComputerLauncher getDelegatedLauncher() { ComputerLauncher l = launcher; while (l instanceof DelegatingComputerLauncher) { <START> l = ((DelegatingComputerLauncher)l).getLauncher(); <END> } return l; }</code><technical_language>if nested launcher is a delegating launcher well? I expect method continue recursion then</technical_language>

Target:
public ComputerLauncher getDelegatedLauncher() { ComputerLauncher l = launcher; while (true) { if (l instanceof DelegatingComputerLauncher) { l = ((DelegatingComputerLauncher) l).getLauncher(); } else if (l instanceof ComputerLauncherFilter) { l = ((ComputerLauncherFilter) l).getCore(); } else { break; } } return l; }
Source:
<code>private void checkServiceInterface(DetailAST interfaceDefToken) { DetailAST serviceInterfaceAnnotationNode = null; String nameValue = null; DetailAST modifiersToken = interfaceDefToken.findFirstToken(TokenTypes.MODIFIERS); for (DetailAST ast = modifiersToken.getFirstChild(); ast != null; ast = ast.getNextSibling()) { if (ast.getType() != TokenTypes.ANNOTATION) { continue; } for (DetailAST annotationChild = ast.getFirstChild(); annotationChild != null; annotationChild = annotationChild.getNextSibling()) { if (annotationChild.getType() == TokenTypes.IDENT) { if (!"ServiceInterface".equals(annotationChild.getText())) { break; } else { serviceInterfaceAnnotationNode = ast; } } if (annotationChild.getType() == TokenTypes.ANNOTATION_MEMBER_VALUE_PAIR) { <START> if ("name".equals(annotationChild.findFirstToken(TokenTypes.IDENT).getText())) { <END> nameValue = getNamePropertyValue(annotationChild.findFirstToken(TokenTypes.EXPR)); } } } } if (serviceInterfaceAnnotationNode == null) { return; } if (nameValue.isEmpty()) { log(serviceInterfaceAnnotationNode, String.format("The ''name'' property of @ServiceInterface, ''%s'' should not be empty.", nameValue)); } if (nameValue.contains(" ")) { log(serviceInterfaceAnnotationNode, String.format("The ''name'' property of @ServiceInterface, ''%s'' should not contain white space.", nameValue)); } if (nameValue.length() > 10) { log(serviceInterfaceAnnotationNode, "[DEBUG] length = " + nameValue.length() + ", name = " + nameValue); log(serviceInterfaceAnnotationNode, String.format("The ''name'' property of @ServiceInterface ''%s'' should not have a length > 10.", nameValue)); } }</code><technical_language>merge if conditions</technical_language>

Target:
private void checkServiceInterface(DetailAST interfaceDefToken) { DetailAST serviceInterfaceAnnotationNode = null; String nameValue = null; DetailAST modifiersToken = interfaceDefToken.findFirstToken(TokenTypes.MODIFIERS); for (DetailAST ast = modifiersToken.getFirstChild(); ast != null; ast = ast.getNextSibling()) { if (ast.getType() != TokenTypes.ANNOTATION) { continue; } if (!"ServiceInterface".equals(ast.findFirstToken(TokenTypes.IDENT).getText())) { continue; } serviceInterfaceAnnotationNode = ast; DetailAST annotationMemberValuePairToken = ast.findFirstToken(TokenTypes.ANNOTATION_MEMBER_VALUE_PAIR); if ("name".equals(annotationMemberValuePairToken.findFirstToken(TokenTypes.IDENT).getText())) { nameValue = getNamePropertyValue(annotationMemberValuePairToken.findFirstToken(TokenTypes.EXPR)); break; } } if (serviceInterfaceAnnotationNode == null) { return; } Pattern serviceNamePattern = Pattern.compile("^[a-zA-Z0-9]{1,20}$"); if (!serviceNamePattern.matcher(nameValue).find()) { log(serviceInterfaceAnnotationNode, String.format( "The ''name'' property of @ServiceInterface, ''%s'' should be non-empty, alphanumeric and not more than 10 characters", nameValue)); } }
Source:
<code>private void getJobXML(HttpServletRequest request, HttpServletResponse response) throws IOException { String jobId = request.getParameter("jobId"); String sessionId = request.getParameter("sessionId"); String fileName = "job_" + jobId + ".xml"; try (ServletOutputStream out = response.getOutputStream(); InputStream is = new ByteArrayInputStream(((SchedulerServiceImpl) Service.get()).getJobXML(sessionId, jobId) .getBytes(StandardCharsets.UTF_8))) { response.setContentType("application/xml"); response.setHeader("Content-disposition", "attachment; filename=" + fileName); response.setHeader("Location", fileName); int buffer; while ((buffer = is.read()) != -1) { out.write(buffer); <START> out.flush(); <END> } LOGGER.debug("Successfully downloaded job XML for job: {}", jobId); } catch (RestServerException e) { LOGGER.debug("Failed to download workflow. User permission not granted: ", e); response.addHeader("proactive_error", "User permission not granted: HTTP status code " + e.getStatus()); response.sendError(HttpServletResponse.SC_FORBIDDEN, e.getMessage()); } catch (IOException | ServiceException e) { LOGGER.warn("Failed to download workflow", e); response.sendError(HttpServletResponse.SC_INTERNAL_SERVER_ERROR, "Failed to download workflow: " + e.getMessage()); } }</code><technical_language>I flush is needed here. flush byte, it? Try resources auto close streams. execute flush</technical_language>

Target:
private void getJobXML(HttpServletRequest request, HttpServletResponse response) throws IOException { String jobId = request.getParameter("jobId"); String sessionId = request.getParameter("sessionId"); String fileName = "job_" + jobId + ".xml"; try (ServletOutputStream out = response.getOutputStream(); InputStream is = new ByteArrayInputStream(((SchedulerServiceImpl) Service.get()).getJobXML(sessionId, jobId) .getBytes(StandardCharsets.UTF_8))) { response.setContentType("application/xml"); response.setHeader("Content-disposition", "attachment; filename=" + fileName); response.setHeader("Location", fileName); int buffer; while ((buffer = is.read()) != -1) { out.write(buffer); } out.flush(); LOGGER.debug("Successfully downloaded job XML for job: {}", jobId); } catch (RestServerException e) { if (e.getStatus() == HttpServletResponse.SC_FORBIDDEN) { LOGGER.debug("Failed to download workflow. User permission not granted: ", e); response.addHeader("proactive_error", "User permission not granted: HTTP status code " + e.getStatus()); } else { LOGGER.warn("Failed to download workflow", e); } response.sendError(e.getStatus(), e.getMessage()); } catch (IOException | ServiceException e) { LOGGER.warn("Failed to download workflow", e); response.sendError(HttpServletResponse.SC_INTERNAL_SERVER_ERROR, "Failed to download workflow: " + e.getMessage()); } }
Source:
<code>protected void initDiskVmElement() { setDiskVmElement(new DiskVmElement(new <START> VmDeviceId(null, null))); <END> }</code><technical_language>IMHO a no-argument C'tor</technical_language>

Target:
protected void initDiskVmElement() { setDiskVmElement(new DiskVmElement()); }