auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public void onMessage(ReadRowsResponse message) { resetStatusBasedBackoff(); lastResponseMs = clock.currentTimeMillis(); timeoutRetryCount = 0; ByteString previouslyProcessedKey = rowMerger.getLastCompletedRowKey(); operationSpan.addAnnotation("Got a response"); rowMerger.onNext(message); operationSpan.addAnnotation("Processed Response", ImmutableMap.of("rowCount", <START> AttributeValue.longAttributeValue(rowMerger.getRowCountInLastMessage()))); <END> ByteString lastProcessedKey = rowMerger.getLastCompletedRowKey(); if (previouslyProcessedKey != lastProcessedKey) { updateLastFoundKey(lastProcessedKey); } else { updateLastFoundKey(message.getLastScannedRowKey()); } if (adapter.autoFlowControlEnabled) { adapter.request(1); } if (resultObserver != null) { resultObserver.onNext(message); } }</code><technical_language>this work multiple responses? Do accumulate overwrite other</technical_language>

Target:
public void onMessage(ReadRowsResponse message) { resetStatusBasedBackoff(); lastResponseMs = clock.currentTimeMillis(); timeoutRetryCount = 0; ByteString previouslyProcessedKey = rowMerger.getLastCompletedRowKey(); operationSpan.addAnnotation("Got a response"); rowMerger.onNext(message); int rowCountInLastMessage = rowMerger.getRowCountInLastMessage(); operationSpan.addAnnotation("Processed Response", ImmutableMap.of("rowCount", AttributeValue.longAttributeValue(rowCountInLastMessage))); totalRowsProcessed += rowCountInLastMessage; ByteString lastProcessedKey = rowMerger.getLastCompletedRowKey(); if (previouslyProcessedKey != lastProcessedKey) { updateLastFoundKey(lastProcessedKey); } else { updateLastFoundKey(message.getLastScannedRowKey()); } if (adapter.autoFlowControlEnabled) { adapter.request(1); } if (resultObserver != null) { resultObserver.onNext(message); } }
Source:
<code>private void updateOutputSpace(Job job, String outputFolder) throws NotConnectedException, PermissionException { List<String> outputSpaces; if (job.getOutputSpace() != null) { outputSpaces = Arrays.asList(Tools.dataSpaceConfigPropertyToUrls(job.getOutputSpace())); } else { outputSpaces = getUserSpaceURIs(); } StringBuilder outputSpace_url_updated = new StringBuilder(); <START> for (String inputSpace : outputSpaces) { <END> if (outputSpace_url_updated.length() > 0) { outputSpace_url_updated.append(" "); } outputSpace_url_updated.append(appendSlashIfNeeded(inputSpace)); outputSpace_url_updated.append(outputFolder); } job.setOutputSpace(outputSpace_url_updated.toString()); log.debug("Output space of job " + job.getName() + " will be " + outputSpace_url_updated); }</code><technical_language>this block is inputfiles....can refactor method</technical_language>

Target:
private void updateOutputSpace(Job job, String outputFolder) throws NotConnectedException, PermissionException { List<String> outputSpaces; if (job.getOutputSpace() != null) { outputSpaces = Arrays.asList(Tools.dataSpaceConfigPropertyToUrls(job.getOutputSpace())); } else { outputSpaces = getUserSpaceURIs(); } StringBuilder outputSpace_url_updated = buildUpdatedSpaceConfiguration(outputFolder, outputSpaces); job.setOutputSpace(outputSpace_url_updated.toString()); log.debug("Output space of job " + job.getName() + " will be " + outputSpace_url_updated); }
Source:
<code>boolean isAttributeToPost(String attributeName) { if (postAttributes.isEmpty()) { trace.log(TraceLevel.INFO, "postAttriburte to use: " + attributeName); return true; } <START> for (Iterator<String> iterator = postAttributes.iterator(); iterator.hasNext();) { <END> String postAttribute = iterator.next(); if (postAttribute.equals(attributeName)) { trace.log(TraceLevel.INFO, "postAttriburte to use: " + attributeName); return true; } } trace.log(TraceLevel.INFO, "postAttriburte do not use: " + attributeName); return false; }</code><technical_language>FYI this written more simply as: for (String postAttribute : postAttributes) { .... } in this case: postAttributes.contains(attributeName) I creating a Set list of attributes is a approach</technical_language>

Target:
boolean isAttributeToPost(String attributeName) { if (includeAttributesSet == null) { return true; } return(includeAttributesSet.contains(attributeName)); }
Source:
<code>protected void setASTInputType(int inputKind) { if (inputKind != fCurrentInputKind) { fCurrentInputKind= inputKind; fDialogSettings.put(SETTINGS_INPUT_KIND, inputKind); for (ASTView.ASTInputKindAction <START> curr : <END> fASTInputKindActions) { curr.setChecked(curr.getInputKind() == inputKind); } refreshASTSettingsActions(); performRefresh(); } }</code><technical_language>rename action</technical_language>

Target:
protected void setASTInputType(int inputKind) { if (inputKind != fCurrentInputKind) { fCurrentInputKind= inputKind; fDialogSettings.put(SETTINGS_INPUT_KIND, inputKind); for (ASTView.ASTInputKindAction action : fASTInputKindActions) { action.setChecked(action.getInputKind() == inputKind); } refreshASTSettingsActions(); performRefresh(); } }
Source:
<code>public void doFilter(ServletRequest request, ServletResponse response, FilterChain chain) throws IOException, ServletException { final HttpServletRequest httpRequest = (HttpServletRequest) request; final String token = tokenExtractor.getToken(httpRequest); if (isNullOrEmpty(token)) { chain.doFilter(request, response); return; } try { final Jws<Claims> jws = Jwts.parser().setSigningKey(keyManager.getKeyPair().getPublic()).parseClaimsJws(token); final Claims claims = jws.getBody(); if (MACHINE_TOKEN_KIND.equals(jws.getHeader().get("kind"))) { <START> final String userId = claims.get(USER_ID_CLAIM, String.class); <END> try { final String userName = userManager.getById(userId).getName(); final Subject authorizedSubject = new AuthorizedSubject( new SubjectImpl(userName, userId, token, false), permissionChecker); EnvironmentContext.getCurrent().setSubject(authorizedSubject); chain.doFilter(addUserInRequest(httpRequest, authorizedSubject), response); } catch (NotFoundException ex) { sendErr( response, SC_UNAUTHORIZED, "Authentication with machine token failed because user for this token no longer exist."); } catch (ServerException ex) { sendErr( response, SC_UNAUTHORIZED, format("Authentication with machine token failed cause: %s", ex.getMessage())); } finally { EnvironmentContext.reset(); } } } catch (RuntimeException ex) { chain.doFilter(request, response); } }</code><technical_language>I happy this method refactored</technical_language>

Target:
public void doFilter(ServletRequest request, ServletResponse response, FilterChain chain) throws IOException, ServletException { final HttpServletRequest httpRequest = (HttpServletRequest) request; final String token = tokenExtractor.getToken(httpRequest); if (isNullOrEmpty(token)) { chain.doFilter(request, response); return; } try { final Jws<Claims> jwt = Jwts.parser().setSigningKey(keyManager.getKeyPair().getPublic()).parseClaimsJws(token); final Claims claims = jwt.getBody(); if (!isMachineToken(jwt)) { chain.doFilter(request, response); return; } try { final String userId = claims.get(USER_ID_CLAIM, String.class); final String userName = userManager.getById(userId).getName(); final Subject authorizedSubject = new AuthorizedSubject( new SubjectImpl(userName, userId, token, false), permissionChecker); EnvironmentContext.getCurrent().setSubject(authorizedSubject); chain.doFilter(addUserInRequest(httpRequest, authorizedSubject), response); } catch (NotFoundException ex) { sendErr( response, SC_UNAUTHORIZED, "Authentication with machine token failed because user for this token no longer exist."); } catch (ServerException ex) { sendErr( response, SC_UNAUTHORIZED, format("Authentication with machine token failed cause: %s", ex.getMessage())); } finally { EnvironmentContext.reset(); } } catch (UnsupportedJwtException | MalformedJwtException | SignatureException | ExpiredJwtException ex) { chain.doFilter(request, response); } }
Source:
<code>public CommandInputLine option(String name, Integer value) { if (value != null) { <START> this.options.put(name, Integer.valueOf(value)); <END> } return this; }</code><technical_language>Guessing wanted argument int? If valueOf is unneeded</technical_language>

Target:
public CommandInputLine option(String name, Integer value) { if (value != null) { this.options.put(name, value); } return this; }
Source:
<code>public StoreConfig(int maxQueryThreads) { <START> this(maxQueryThreads, 10000, 10000, 1); <END> }</code><technical_language>I defaults arbitrary, defaults BatchWriterConfig uses? I for maxLatency, 2 mins (120000 ms)</technical_language>

Target:
public StoreConfig(int maxQueryThreads) { this(maxQueryThreads, 1024 * 1024 * 1000, 120 * 1000 * 60, 1); }
Source:
<code>public void addMembers(final ClassVisitor cv, final int probeCount) { createDataField(cv); <START> createGetMethod(cv, probeCount); <END> if (!seenClinit) { createClinitMethod(cv, probeCount); } }</code><technical_language>For consistency other implementations name this createInitMethod, create method Name is $jacocoInit</technical_language>

Target:
public void addMembers(final ClassVisitor cv, final int probeCount) { createDataField(cv); createInitMethod(cv, probeCount); if (!seenClinit) { createClinitMethod(cv, probeCount); } }
Source:
<code>public boolean isExpired(long timeout) { final long timeLeft = TIME_LEFT_UPDATER.get(this); final long timeEnter = TIME_ENTER_UPDATER.get(this); if (timeEnter != NIL && ((timeLeft != NIL && timeEnter > timeLeft) || timeLeft == NIL)) { <START> final long timeoutNanos = TimeUnit.MILLISECONDS.toNanos(timeout); <END> return System.nanoTime() - timeEnter > timeoutNanos; } return false; }</code><technical_language>this is passed in another place, is config this timeout, computing this time, isExpired timeoutInNanos else (aka read config this compute millis nano's once)</technical_language>

Target:
public boolean isExpired(long timeout) { final long timeLeft = TIME_LEFT_UPDATER.get(this); final long timeEnter = TIME_ENTER_UPDATER.get(this); if (timeLeft - timeEnter < 0) { return System.nanoTime() - timeEnter > timeout; } return false; }
Source:
<code>private Optional<VariantContextWriter> getVariantContextWriter(final VCFFileReader truthReader, final VCFFileReader callReader) { if (OUTPUT_VCF) { <START> final File outputVcfFile = new File(OUTPUT + ".genotype_concordance.vcf.gz"); <END> final VariantContextWriterBuilder builder = new VariantContextWriterBuilder() .setOutputFile(outputVcfFile) .setReferenceDictionary(callReader.getFileHeader().getSequenceDictionary()) .setOption(Options.ALLOW_MISSING_FIELDS_IN_HEADER) .setOption(Options.INDEX_ON_THE_FLY); final VariantContextWriter writer = builder.build(); final List<String> sampleNames = Arrays.asList(OUTPUT_VCF_CALL_SAMPLE_NAME, OUTPUT_VCF_TRUTH_SAMPLE_NAME); final Set<VCFHeaderLine> headerLines = new HashSet<>(); headerLines.addAll(callReader.getFileHeader().getMetaDataInInputOrder()); headerLines.addAll(truthReader.getFileHeader().getMetaDataInInputOrder()); headerLines.add(CONTINGENCY_STATE_HEADER_LINE); writer.writeHeader(new VCFHeader(headerLines, sampleNames)); return Optional.of(writer); } else { return Optional.empty(); } }</code><technical_language>extension a static member beside other extensions</technical_language>

Target:
private Optional<VariantContextWriter> getVariantContextWriter(final VCFFileReader truthReader, final VCFFileReader callReader) { if (OUTPUT_VCF) { final File outputVcfFile = new File(OUTPUT + OUTPUT_VCF_FILE_EXTENSION); final VariantContextWriterBuilder builder = new VariantContextWriterBuilder() .setOutputFile(outputVcfFile) .setReferenceDictionary(callReader.getFileHeader().getSequenceDictionary()) .setOption(Options.ALLOW_MISSING_FIELDS_IN_HEADER) .setOption(Options.INDEX_ON_THE_FLY); final VariantContextWriter writer = builder.build(); final List<String> sampleNames = Arrays.asList(OUTPUT_VCF_CALL_SAMPLE_NAME, OUTPUT_VCF_TRUTH_SAMPLE_NAME); final Set<VCFHeaderLine> headerLines = new HashSet<>(); headerLines.addAll(callReader.getFileHeader().getMetaDataInInputOrder()); headerLines.addAll(truthReader.getFileHeader().getMetaDataInInputOrder()); headerLines.add(CONTINGENCY_STATE_HEADER_LINE); writer.writeHeader(new VCFHeader(headerLines, sampleNames)); return Optional.of(writer); } else { return Optional.empty(); } }
Source:
<code>public void debugContextChanged(DebugContextEvent event) { if ((event.getFlags() != DebugContextEvent.ACTIVATED)) { return; } IAdaptable context = DebugUITools.getDebugContext(); if (context != null) { final IDMContext dmc = context.getAdapter(IDMContext.class); <START> if (dmc instanceof IMIExecutionDMContext || dmc instanceof IFrameDMContext) { <END> final IMIExecutionDMContext executionDMC = DMContexts.getAncestorOfType(dmc, IMIExecutionDMContext.class); if (executionDMC == null) { return; } String eventSessionId = executionDMC.getSessionId(); if (!(DsfSession.isSessionActive(eventSessionId))) { return; } DsfSession session = DsfSession.getSession(eventSessionId); session.getExecutor().execute(new Runnable() { @Override public void run() { DsfServicesTracker tracker = new DsfServicesTracker(GdbUIPlugin.getBundleContext(), eventSessionId); IGDBSynchronizer gdbSync = tracker.getService(IGDBSynchronizer.class); tracker.dispose(); if (gdbSync != null) { gdbSync.setFocus(new IDMContext[] {dmc}, new ImmediateRequestMonitor() {}); } } }); } } }</code><technical_language>Ok, wait MI notif handle directions</technical_language>

Target:
public void debugContextChanged(DebugContextEvent event) { if ((event.getFlags() != DebugContextEvent.ACTIVATED)) { return; } IAdaptable context = DebugUITools.getDebugContext(); if (context != null) { final IDMContext dmc = context.getAdapter(IDMContext.class); if (dmc instanceof IMIExecutionDMContext || dmc instanceof IFrameDMContext) { final IMIExecutionDMContext executionDMC = DMContexts.getAncestorOfType(dmc, IMIExecutionDMContext.class); String eventSessionId = executionDMC.getSessionId(); if (!(DsfSession.isSessionActive(eventSessionId))) { return; } DsfSession session = DsfSession.getSession(eventSessionId); session.getExecutor().execute(new Runnable() { @Override public void run() { DsfServicesTracker tracker = new DsfServicesTracker(GdbUIPlugin.getBundleContext(), eventSessionId); IGDBSynchronizer gdbSync = tracker.getService(IGDBSynchronizer.class); tracker.dispose(); if (gdbSync != null) { gdbSync.setFocus(new IDMContext[] {dmc}, new ImmediateRequestMonitor() {}); } } }); } } }
Source:
<code><START> public void onNameListsImported(Map<String, Integer> nameAmounts, String listName) <END></code><technical_language>need refresh ListInfo here. This action mutates list</technical_language>

Target:
public void onNameListsImported(Map<String, Integer> nameAmounts, String listName) { listInfo = dataSource.getListInfo(listName); settings.setNameListSize(listInfo.getNumInstances()); settingsDialog.refreshSetting(); }
Source:
<code><START> private static void sleepFor(int time, TimeUnit units) { <END> try { Thread.sleep(units.toMillis(time)); } catch (InterruptedException e) { } }</code><technical_language>Please public</technical_language>

Target:
public static void sleepFor(int time, TimeUnit units) { try { Thread.sleep(units.toMillis(time)); } catch (InterruptedException e) { } }
Source:
<code>public void loadExisting() { inflightInvalidations = synchronizedList(new ArrayList<>()); if (!isStrong()) { addInflightInvalidationsForEventualCaches(); } stateService.loadStore(storeIdentifier, configuration).setEvictionListener(this::invalidateHashAfterEviction); <START> LOGGER.debug("Preparing for handling Inflight Invalidations and independent Passive Evictions in loadExisting"); <END> reconnectComplete.set(false); management.init(); }</code><technical_language>this debug statement move well. Change message</technical_language>

Target:
public void loadExisting() { inflightInvalidations = synchronizedList(new ArrayList<>()); if (!isStrong()) { LOGGER.debug("Preparing for handling inflight invalidations"); addInflightInvalidationsForEventualCaches(); } stateService.loadStore(storeIdentifier, configuration).setEvictionListener(this::invalidateHashAfterEviction); reconnectComplete.set(false); management.init(); }
Source:
<code>protected FSDataOutputStream openStreamForNonAppendFS(Path filepath) { try { Path appendTmpFile = new Path(filepath + APPEND_TMP_FILE); fs.rename(filepath, appendTmpFile); FSDataInputStream fsIn = fs.open(appendTmpFile); FSDataOutputStream fsOut = fs.create(filepath); IOUtils.copy(fsIn, fsOut); flush(fsOut); <START> fs.delete(appendTmpFile); <END> return fsOut; } catch (IOException e) { throw new RuntimeException(e); } }</code><technical_language>Do need close fsIn delete</technical_language>

Target:
protected FSDataOutputStream openStreamForNonAppendFS(Path filepath) { try { Path appendTmpFile = new Path(filepath + APPEND_TMP_FILE); fs.rename(filepath, appendTmpFile); FSDataInputStream fsIn = fs.open(appendTmpFile); FSDataOutputStream fsOut = fs.create(filepath); IOUtils.copy(fsIn, fsOut); flush(fsOut); fsIn.close(); fs.delete(appendTmpFile); return fsOut; } catch (IOException e) { throw new RuntimeException(e); } }
Source:
<code>public void select(WebDriver webDriver, final SelectCoordinates selectCoordinates) { selectMatching(webDriver, currentWindowInfo -> currentWindowInfo.get(WINDOW_INFO_INDEX_DOCUMENT_TITLE).trim().toLowerCase() <START> .equals(selectCoordinates.criteria.toLowerCase()), "Unable to locate window with title '" + selectCoordinates.criteria + "'"); <END> }</code><technical_language>split arguments more readable, please</technical_language>

Target:
public void select(WebDriver webDriver, final SelectCoordinates selectCoordinates) { String selectionCriteria = selectCoordinates.criteria; selectMatching(webDriver, currentWindowInfo -> currentWindowInfo.get(WINDOW_INFO_INDEX_DOCUMENT_TITLE).trim() .equalsIgnoreCase(selectionCriteria), String.format("Unable to locate window with title '%s'", selectionCriteria)); }
Source:
<code><START> public State (int moves, int bucketOne, int bucketTwo) { <END> this.moves = moves; this.bucketOne = bucketOne; this.bucketTwo = bucketTwo; }</code><technical_language>I this package-private</technical_language>

Target:
State (int moves, int bucketOne, int bucketTwo) { this.moves = moves; this.bucketOne = bucketOne; this.bucketTwo = bucketTwo; }
Source:
<code>private MafRecord readNextEntry() { try { for (;;) { String buffer = lastLine_ != null ? lastLine_ : brMaf_.readLine(); lastLine_ = null; if (buffer == null) return null; buffer = buffer.trim(); if (buffer.length() > 0 && buffer.charAt(0) != '#') { <START> if (buffer.charAt(0) != 'a') throw new RuntimeException(); <END> String header = new String(buffer); ArrayList<String> entries = new ArrayList<String>(); for (;;) { buffer = brMaf_.readLine(); if (buffer == null) return new MafRecord(header,entries); buffer = buffer.trim(); if (buffer.length() > 0) { if (buffer.charAt(0) == 's' ) { entries.add(buffer); } else if(buffer.charAt(0) == 'a') { lastLine_ = buffer; return new MafRecord(header,entries); } else if(buffer.charAt(0) != '#') { throw new RuntimeException(); } } } } } } catch (IOException e) { return null; } }</code><technical_language>Please give this exception a name</technical_language>

Target:
private MafRecord readNextEntry() { try { String buffer = lastLine_ != null ? lastLine_ : brMaf_.readLine(); lastLine_ = null; for( ; buffer != null ; buffer = brMaf_.readLine() ){ buffer = buffer.trim(); if( buffer.length() > 0 && buffer.charAt(0) != '#') break; } if (buffer == null) return null; if (buffer.charAt(0) != 'a') throw new RuntimeException("unexpected MAF line: "+buffer); String header = buffer; ArrayList<String> entries = new ArrayList<String>(2); for (buffer=brMaf_.readLine() ; buffer != null ; buffer=brMaf_.readLine()) { buffer = buffer.trim(); if (buffer.length() > 0) { if (buffer.charAt(0) == 's' ) { entries.add(buffer); } else if(buffer.charAt(0) == 'a') { break; } else if(buffer.charAt(0) != '#') { throw new RuntimeException("unexpected MAF line: "+buffer); } } } lastLine_ = buffer; return new MafRecord(header,entries); } catch (IOException e) { e.printStackTrace(); return null; } }
Source:
<code>protected String getUserName(UserType userInfo) { String userName = null; if (userInfo != null && userInfo.getPersonName() != null) { if (userInfo.getPersonName().getGivenName() != null && userInfo.getPersonName().getGivenName(). <START> length() > 0) { <END> userName = userInfo.getPersonName().getGivenName(); } if (userInfo.getPersonName().getFamilyName() != null && userInfo.getPersonName().getFamilyName().length() > 0) { if (userName != null) { userName += (" " + userInfo.getPersonName().getFamilyName()); } else { userName = userInfo.getPersonName().getFamilyName(); } } } return userName; }</code><technical_language>NullChecker.isNotNullish(userInfo.getPersonName().getGivenName() )</technical_language>

Target:
protected String getUserName(UserType userInfo) { String userName = null; if (userInfo != null && userInfo.getPersonName() != null) { if (NullChecker.isNotNullish(userInfo.getPersonName().getGivenName())) { userName = userInfo.getPersonName().getGivenName(); } if (NullChecker.isNotNullish(userInfo.getPersonName().getFamilyName())) { if (userName != null) { userName += " " + userInfo.getPersonName().getFamilyName(); } else { userName = userInfo.getPersonName().getFamilyName(); } } } return userName; }
Source:
<code>public boolean isAttributesListPresentInRobotsMetaTag(List<String> expectedAttributes) { String[] currentAttributes = robots.getAttribute("content").split("[, ]+"); if (currentAttributes.length != expectedAttributes.size()) { PageObjectLogging.log( "isAttributesListPresentInRobotsMetaTag", "Number of current attributes is different than expected", <START> false); <END> return false; } return Arrays.asList(currentAttributes).containsAll(expectedAttributes); }</code><technical_language>false in PageObjectLogging.log is causing test fail I consider renaming method assert returning true false, logging content passed method content page</technical_language>

Target:
public boolean isAttributesListPresentInRobotsMetaTag(List<String> expectedAttributes) { List<String> currentAttributes = Arrays.asList(robots.getAttribute("content").split("[, ]+")); PageObjectLogging.log( "Robots Meta Tag passed: " + expectedAttributes, "Robots Meta Tag found on page: " + currentAttributes, true ); return currentAttributes.equals(expectedAttributes); }
Source:
<code>public void done() { if (progressBar != null && !progressBar.isDisposed()) { <START> progressBar.dispose(); <END> progressBar = null; } requestLayout(); }</code><technical_language>Is correct progress bar disappears done()? layout.topControl = null affect same</technical_language>

Target:
public void done() { if ((progressBar.getStyle() & SWT.INDETERMINATE) != 0) { progressBar.setMinimum(0); progressBar.setMaximum(0); progressBar.setSelection(0); } requestLayout(); }
Source:
<code>public URI open(final URI address) throws IllegalAddressException { checkNoAuthority(address); checkNoQuery(address); checkNoFragment(address); String scheme = address.getScheme(); <START> if (scheme != null && !NATIVE_SCHEME.equalsIgnoreCase(scheme)) { <END> throw new IllegalAddressException( address, IllegalAddressException.Reason.INVALID_SCHEME, null); } String path = extractRawPath(address.normalize()); StringBuilder sb = new StringBuilder(2 + NATIVE_SCHEME.length() + path.length()); sb.append(NATIVE_SCHEME).append(':'); if (path.charAt(0) != '/') { sb.append('/'); } sb.append(path); URI normAddress = URI.create(sb.toString()); this.address = normAddress; synchronized (memSpaces) { MemorySpace ms = memSpaces.get(context); if (ms == null) { ms = new MemorySpace(); memSpaces.put(context, ms); } memSpace = ms; } return normAddress; }</code><technical_language>add a checkScheme method</technical_language>

Target:
public URI open(final URI address) throws IllegalAddressException { checkNoAuthority(address); checkNoQuery(address); checkNoFragment(address); checkScheme(address); String path = extractRawPath(address.normalize()); StringBuilder sb = new StringBuilder(2 + NATIVE_SCHEME.length() + path.length()); sb.append(NATIVE_SCHEME).append(':'); if (path.charAt(0) != '/') { sb.append('/'); } sb.append(path); URI normAddress = URI.create(sb.toString()); this.address = normAddress; memSpace = sharedMemorySpace; return normAddress; }
Source:
<code>public FieldElement build() { checkNotNull(label, "label"); checkNotNull(type, "type"); checkNotNull(name, "name"); checkNotNull(tag, "tag"); if (!isValidTag(tag)) { <START> throw new IllegalArgumentException("Illegal tag value: " + tag); <END> } return new AutoValue_FieldElement(label, type, name, tag, documentation, immutableCopyOf(options)); }</code><technical_language>Ahh. . . this guava</technical_language>

Target:
public FieldElement build() { checkNotNull(label, "label"); checkNotNull(type, "type"); checkNotNull(name, "name"); checkNotNull(tag, "tag"); checkArgument(isValidTag(tag), "Illegal tag value: %s", tag); return new AutoValue_FieldElement(label, type, name, tag, documentation, immutableCopyOf(options)); }
Source:
<code>private boolean checkActive(Object element) { boolean wasChildChecked = false; boolean isActive = false; for (Object child : fContentProvider.getChildren(element)) { wasChildChecked <START> = checkActive(child) <END> | wasChildChecked; } if (!fTree.getChecked(element)) { isActive = element instanceof ITimeGraphEntry && fActiveProvider.isActive((ITimeGraphEntry) element); if (isActive || wasChildChecked) { fTree.setChecked(element, true); } } return fTree.getChecked(element); }</code><technical_language>|= if wish</technical_language>

Target:
private boolean checkActive(Object element) { boolean wasChildChecked = false; boolean isActive = false; for (Object child : fContentProvider.getChildren(element)) { wasChildChecked |= checkActive(child); } if (!fTree.getChecked(element)) { if (!wasChildChecked) { isActive = element instanceof ITimeGraphEntry && fActiveProvider.isActive((ITimeGraphEntry) element); } if (isActive || wasChildChecked) { fTree.setChecked(element, true); } } return fTree.getChecked(element); }
Source:
<code>private ValueType getResultILType(GenContext ctx) { Arrow a = (Arrow) this.type; <START> if( <END> isGeneric() && a.getResult() instanceof UnresolvedType && ((UnresolvedType) a.getResult()).getName().equals(this.genTypeName) ) { return new NominalType(new Variable(GENERIC_PREFACE), GENERIC_MEMBER); } return ((Arrow)type).getResult().getILType(ctx); }</code><technical_language>need this if statement</technical_language>

Target:
private ValueType getResultILType(GenContext ctx) { Arrow a = (Arrow) this.type; return a.getResult().getILType(ctx); }
Source:
<code>private void processOptions(RepositoryState state) throws NoMessageException { if (name == null) throw new JGitInternalException(JGitText.get().tagNameNotSpecified); <START> if (message == null) <END> throw new NoMessageException(JGitText.get().tagMessageNotSpecified); if (unsigned == false) throw new UnsupportedOperationException(); }</code><technical_language>empty message is OK</technical_language>

Target:
private void processOptions(RepositoryState state) throws JGitInternalException { if (author == null) author = new PersonIdent(repo); if (name == null) throw new JGitInternalException(JGitText.get().tagNameNotSpecified); if (signed == true) throw new UnsupportedOperationException(); }
Source:
<code>public static void writeMetadataFile(Configuration configuration, Path outputPath, List<Footer> footers) throws IOException { <START> Path qualifiedPath = FileSystem.get(configuration).makeQualified(outputPath); <END> ParquetMetadata metadataFooter = mergeFooters(qualifiedPath, footers); FileSystem fs = outputPath.getFileSystem(configuration); outputPath = outputPath.makeQualified(fs); writeMetadataFile(outputPath, metadataFooter, fs, PARQUET_METADATA_FILE); metadataFooter.getBlocks().clear(); writeMetadataFile(outputPath, metadataFooter, fs, PARQUET_COMMON_METADATA_FILE); }</code><technical_language>this throw exception if outputPath is in a FS other default</technical_language>

Target:
public static void writeMetadataFile(Configuration configuration, Path outputPath, List<Footer> footers) throws IOException { FileSystem fs = outputPath.getFileSystem(configuration); outputPath = outputPath.makeQualified(fs); ParquetMetadata metadataFooter = mergeFooters(outputPath, footers); writeMetadataFile(outputPath, metadataFooter, fs, PARQUET_METADATA_FILE); metadataFooter.getBlocks().clear(); writeMetadataFile(outputPath, metadataFooter, fs, PARQUET_COMMON_METADATA_FILE); }
Source:
<code>public void stop() { logger.info("Avro source stopping: {}", getName(), this); if (server != null) { server.close(); try { server.join(); server = null; } catch (InterruptedException e) { <START> logger.info("Avro source " + getName() + ": Interrupted while waiting " + <END> "for Avro server to stop. Exiting. Exception follows.", e); } } if (socketChannelFactory != null) { socketChannelFactory.releaseExternalResources(); socketChannelFactory = null; } sourceCounter.stop(); if (connectionCountUpdater != null) { connectionCountUpdater.shutdownNow(); connectionCountUpdater = null; } super.stop(); logger.info("Avro source stopped. Metrics: {}", getName(), sourceCounter); }</code><technical_language>I theThread.currentThread().interrupt(); called set interrupt flag of thread</technical_language>

Target:
public void stop() { logger.info("Avro source stopping: {}", getName(), this); if (server != null) { server.close(); try { server.join(); server = null; } catch (InterruptedException e) { logger.info("Avro source " + getName() + ": Interrupted while waiting " + "for Avro server to stop. Exiting. Exception follows.", e); Thread.currentThread().interrupt(); } } if (socketChannelFactory != null) { socketChannelFactory.releaseExternalResources(); socketChannelFactory = null; } sourceCounter.stop(); if (connectionCountUpdater != null) { connectionCountUpdater.shutdownNow(); connectionCountUpdater = null; } super.stop(); logger.info("Avro source stopped. Metrics: {}", getName(), sourceCounter); }
Source:
<code>public void veryLargePageListsMustBeFullyAccessible() throws Exception { int pageSize = (int) ByteUnit.kibiBytes( 8 ); long pageCacheSize = ByteUnit.gibiBytes( 513 ) + pageSize; int pages = Math.toIntExact( pageCacheSize / pageSize ); <START> MemoryAllocator mman = MemoryAllocator.createAllocator( "2 GiB", new LocalMemoryTracker() ); <END> SwapperSet swappers = new SwapperSet(); long victimPage = VictimPageReference.getVictimPage( pageSize, GlobalMemoryTracker.INSTANCE ); PageList pageList = new PageList( pages, pageSize, mman, swappers, victimPage, Long.BYTES ); assertThat( pageList.getPageCount(), is( pages ) ); IntStream.range( 0, pages / 32 ).parallel().forEach( id -> { verifyPageMetaDataIsAccessible( pageList, id * 32 ); } ); IntStream.range( pages - 2000, pages ).parallel().forEach( id -> { verifyPageMetaDataIsAccessible( pageList, id ); } ); }</code><technical_language>This is a local tracker while others in this tests global instance. suspicious</technical_language>

Target:
public void veryLargePageListsMustBeFullyAccessible() throws Exception { int pageSize = (int) ByteUnit.kibiBytes( 8 ); long pageCacheSize = ByteUnit.gibiBytes( 513 ) + pageSize; int pages = Math.toIntExact( pageCacheSize / pageSize ); MemoryAllocator mman = MemoryAllocator.createAllocator( "2 GiB", GlobalMemoryTracker.INSTANCE ); SwapperSet swappers = new SwapperSet(); long victimPage = VictimPageReference.getVictimPage( pageSize, GlobalMemoryTracker.INSTANCE ); PageList pageList = new PageList( pages, pageSize, mman, swappers, victimPage, Long.BYTES ); assertThat( pageList.getPageCount(), is( pages ) ); IntStream.range( 0, pages / 32 ).parallel().forEach( id -> { verifyPageMetaDataIsAccessible( pageList, id * 32 ); } ); IntStream.range( pages - 2000, pages ).parallel().forEach( id -> { verifyPageMetaDataIsAccessible( pageList, id ); } ); }
Source:
<code>public String formatDataSize(int size) { String ret = ""; <START> if (size < (1024 * 1024)) { <END> ret = String.format("%dK", size / 1024); } else { ret = String.format("%.1fM", size / (1024 * 1024.0)); } return ret; }</code><technical_language>@droneboost store 1024 * 1024 a constant</technical_language>

Target:
public String formatDataSize(int size) { String ret = ""; if (size < (DATA_SIZE_ONE_KB)) { ret = String.format("%dK", size / DATA_SIZE_UNIT_KB); } else { ret = String.format("%.1fM", size / (DATA_SIZE_UNIT_MB)); } return ret; }
Source:
<code>public void item() { Alert alert = generateAlert(); List<Alert> alerts = new ArrayList<>(); alerts.add(alert); alerts.addAll(generateAlerts()); AlertsAdapter alertsAdapter = new AlertsAdapter(context, new AlertMenuAdapter(), alerts); for(int i=0; i<alerts.size(); i++) <START> Assertions.assertThat(alertsAdapter).hasItem(alerts.get(i), i); <END> }</code><technical_language>Please stick convention of providing braces blocks</technical_language>

Target:
public void item() { Alert alert = generateAlert(); List<Alert> alerts = new ArrayList<>(); alerts.add(alert); alerts.addAll(generateAlerts()); AlertsAdapter alertsAdapter = new AlertsAdapter(context, new AlertMenuAdapter(), alerts); for(int i=0; i<alerts.size(); i++) { Assertions.assertThat(alertsAdapter).hasItem(alerts.get(i), i); } }
Source:
<code>public void publishKeyPressedLong(String id) { for (AKM868BindingProvider provider : providers) { for (String itemName : provider.getItemNames()) { if (provider.getId(itemName).equals(id) && provider.getChannel(itemName).equals("5")) { <START> logger.debug("Publishing state for: "+itemName+" -state: ON"); <END> eventPublisher.postUpdate(itemName, OnOffType.ON ); } } } }</code><technical_language>parameterised logging</technical_language>

Target:
public void publishKeyPressedLong(String id) { for (AKM868BindingProvider provider : providers) { for (String itemName : provider.getItemNames()) { if (provider.getId(itemName).equals(id) && provider.getChannel(itemName).equals("5")) { logger.debug("Publishing state for -state: ON",itemName); eventPublisher.postUpdate(itemName, OnOffType.ON ); } } } }
Source:
<code>public MythicSpawnerTag(MythicSpawner spawner) { if (spawner != null) { this.spawner = spawner; } else { <START> Debug.echoError("Mythic Spawner referenced is null!"); <END> } }</code><technical_language>This error here. internal java-level error if do new MythicSpawnerTag(null), cases *might* null checked prior constructing</technical_language>

Target:
public MythicSpawnerTag(MythicSpawner spawner) { this.spawner = spawner; }
Source:
<code><START> protected void writeJDBCStoreConnection(XMLExtendedStreamWriter writer, SimpleConnectionFactoryConfiguration configuration) throws XMLStreamException { <END> writer.writeStartElement(Element.SIMPLE_CONNECTION); writeOptional(writer, Attribute.CONNECTION_URL, configuration.connectionUrl()); writeOptional(writer, Attribute.DRIVER_CLASS, configuration.driverClass()); writeOptional(writer, Attribute.USERNAME, configuration.username()); writeOptional(writer, Attribute.PASSWORD, configuration.password()); writer.writeEndElement(); }</code><technical_language>connection configuration is serialized</technical_language>

Target:
private void writeJDBCStoreConnection(XMLExtendedStreamWriter writer, SimpleConnectionFactoryConfiguration configuration) throws XMLStreamException { writer.writeStartElement(Element.SIMPLE_CONNECTION); writeOptional(writer, Attribute.CONNECTION_URL, configuration.connectionUrl()); writeOptional(writer, Attribute.DRIVER_CLASS, configuration.driverClass()); writeOptional(writer, Attribute.USERNAME, configuration.username()); writeOptional(writer, Attribute.PASSWORD, configuration.password()); writer.writeEndElement(); }
Source:
<code>public boolean delete(Path f, boolean recursive) throws IOException { <START> if (f.isRoot()) { <END> if (!recursive) { return false; } LOG.debug("Deleting root content recursively"); } try { entryPoint(INVOCATION_DELETE); boolean outcome = innerDelete(innerGetFileStatus(f, true), recursive); if (outcome) { try { maybeCreateFakeParentDirectory(f); } catch (AccessDeniedException e) { LOG.warn("Cannot create directory marker at {}: {}", f.getParent(), e.toString()); LOG.debug("Failed to create fake dir above {}", f, e); } } return outcome; } catch (FileNotFoundException e) { LOG.debug("Couldn't delete - does not exist: {}", f, e.toString()); instrumentation.errorIgnored(); return false; } catch (AmazonClientException e) { throw translateException("delete", f, e); } }</code><technical_language>this entrypoint, metrics incremented FS open/closed state checked</technical_language>

Target:
public boolean delete(Path f, boolean recursive) throws IOException { try { entryPoint(INVOCATION_DELETE); if (f.isRoot()) { return rejectRootDirectoryDelete(f, recursive); } boolean outcome = innerDelete(innerGetFileStatus(f, true), recursive); if (outcome) { try { maybeCreateFakeParentDirectory(f); } catch (AccessDeniedException e) { LOG.warn("Cannot create directory marker at {}: {}", f.getParent(), e.toString()); LOG.debug("Failed to create fake dir above {}", f, e); } } return outcome; } catch (FileNotFoundException e) { LOG.debug("Couldn't delete - does not exist: {}", f, e.toString()); instrumentation.errorIgnored(); return false; } catch (AmazonClientException e) { throw translateException("delete", f, e); } }
Source:
<code>@Override public Pair<SqlNode, RelDataType> validateAndGetType(SqlNode sqlNode) throws ValidationException { <START> final SqlNode validatedNode = this.validate(sqlNode); <END> final RelDataType type = m_validator.getValidatedNodeType(validatedNode); return Pair.of(validatedNode, type); }</code><technical_language>superfluous this</technical_language>

Target:
@Override public Pair<SqlNode, RelDataType> validateAndGetType(SqlNode sqlNode) throws ValidationException { final SqlNode validatedNode = validate(sqlNode); final RelDataType type = m_validator.getValidatedNodeType(validatedNode); return Pair.of(validatedNode, type); }
Source:
<code>public void setUserActivationKey_shouldCreateUserActivationKey() { User u = new User(); u.setPerson(new Person()); u.addName(new PersonName("Benjamin", "A", "Wolfe")); u.setUsername("bwolfe"); u.getPerson().setGender("M"); User createdUser = userService.createUser(u, "Openmr5xy"); userService.setUserActivationKey(createdUser); <START> String activationKey = dao.getLoginCredential(createdUser).getActivationKey(); <END> assertNotNull(activationKey); }</code><technical_language>instead asserting UserService.getUserActivationKey returns user this dao call</technical_language>

Target:
public void setUserActivationKey_shouldCreateUserActivationKey() { User u = new User(); u.setPerson(new Person()); u.addName(new PersonName("Benjamin", "A", "Wolfe")); u.setUsername("bwolfe"); u.getPerson().setGender("M"); User createdUser = userService.createUser(u, "Openmr5xy"); assertNull(dao.getLoginCredential(createdUser).getActivationKey()); assertEquals(createdUser, userService.setUserActivationKey(createdUser)); assertNotNull(dao.getLoginCredential(createdUser).getActivationKey()); }
Source:
<code>public CloneCommand() { super( "clone", "Clones a section of the world.", "/clone <x1> <y1> <z1> <x2> <y2> <z2> <x> <y> <z> [maskMode] [cloneMode] [tileName] [tileData]", Collections.emptyList() ); <START> setPermission("minecraft.command.seed"); <END> }</code><technical_language>Wrong permission :^)</technical_language>

Target:
public CloneCommand() { super( "clone", "Clones a section of the world.", "/clone <x1> <y1> <z1> <x2> <y2> <z2> <x> <y> <z> [maskMode] [cloneMode] [tileName] [tileData]", Collections.emptyList() ); setPermission("minecraft.command.clone"); }
Source:
<code>public synchronized ApiFuture<Void> add(RowMutationEntry rowMutation) { Preconditions.checkNotNull(rowMutation, "mutation details cannot be null"); final ApiFuture<Void> response = bulkMutateBatcher.add(rowMutation); <START> operationAccountant.registerOperation(ApiFutureUtil.adapt(response)); <END> return response; }</code><technical_language>Please avoid operationAccountant. this functionality is for jvm shutdown hook outstanding batches. this functionality in gcj already. move shutdown hook BulkMutationWrapper</technical_language>

Target:
public synchronized ApiFuture<Void> add(RowMutationEntry rowMutation) { Preconditions.checkNotNull(rowMutation, "mutation details cannot be null"); return bulkMutateBatcher.add(rowMutation); }
Source:
<code>public void close() throws IOException { List<ListenableFuture<Void>> closingChannelsFutures = new ArrayList<>(); ListeningExecutorService listenableBatchPool = MoreExecutors.listeningDecorator(batchPool); for (final Closeable clientCloseHandler : clientCloseHandlers) { closingChannelsFutures.add(listenableBatchPool.submit(new Callable<Void>() { @Override public Void call() throws Exception { clientCloseHandler.close(); return null; } })); } try { Futures.allAsList(closingChannelsFutures).get(); } catch (InterruptedException e) { Thread.interrupted(); <START> throw new IOException("Interrupted while waiting for close", e); <END> } catch (ExecutionException e) { throw new IOException("Exception while waiting for close", e); } elg.shutdownGracefully(); scheduledRetries.shutdown(); awaiteTerminated(scheduledRetries); if (terminateBatchPool) { batchPool.shutdown(); awaiteTerminated(batchPool); } }</code><technical_language>nit: Interrupted while waiting for channels closed</technical_language>

Target:
public void close() throws IOException { List<ListenableFuture<Void>> closingChannelsFutures = new ArrayList<>(); ListeningExecutorService listenableBatchPool = MoreExecutors.listeningDecorator(batchPool); for (final Closeable clientCloseHandler : clientCloseHandlers) { closingChannelsFutures.add(listenableBatchPool.submit(new Callable<Void>() { @Override public Void call() throws Exception { clientCloseHandler.close(); return null; } })); } try { Futures.allAsList(closingChannelsFutures).get(); } catch (InterruptedException e) { Thread.interrupted(); throw new IOException("Interrupted while waiting for channels to be closed", e); } catch (ExecutionException e) { throw new IOException("Exception while waiting for channels to be closed", e); } elg.shutdownGracefully(); scheduledRetries.shutdown(); awaiteTerminated(scheduledRetries); if (terminateBatchPool) { batchPool.shutdown(); awaiteTerminated(batchPool); } }
Source:
<code>private CodecConfigurationException getError(final Exception cause) { return new CodecConfigurationException(format("Unable to get value for property '%s' in %s", propertyMetadata.getName(), <START> propertyMetadata.getTypeData().getType()), cause); <END> }</code><technical_language>intention is below. message reads intent is string is name of enclosing class, property type. end like: "Unable get for property 'someInt' in java.lang.Integer"</technical_language>

Target:
private CodecConfigurationException getError(final Exception cause) { return new CodecConfigurationException(format("Unable to get value for property '%s' in %s", propertyMetadata.getName(), propertyMetadata.getDeclaringClassName()), cause); }
Source:
<code>public static List<String> getList(RenderingContext ctx, String key) { Object value = ctx.getParameter(key); if (value instanceof String) { return Arrays.asList((String) value); <START> } else if (value instanceof List) { <END> List<?> values = (List<?>) value; List<String> list = new ArrayList<>(values.size()); for (Object val : values) { if (val != null) { list.add(val.toString()); } } return list; } return Collections.emptyList(); }</code><technical_language>ensure set a List in context do a cast get</technical_language>

Target:
public static List<String> getList(RenderingContext ctx, String key) { Object value = ctx.getParameter(key); if (value == null) { return Collections.emptyList(); } return ((List<String>) value).stream().filter(Objects::nonNull).collect(Collectors.toList()); }
Source:
<code>public RitualCreate() { for (Ritual i : BloodMagic.RITUAL_MANAGER.getRituals()) { <START> addSubcommand(new CommandTreeBase() { <END> public Ritual ritual = i; @Override public String getName() { return this.ritual.getName(); } @Override public String getUsage(ICommandSender sender) { return new TextComponentTranslation(this.ritual.getUnlocalizedName() + ".info").getFormattedText(); } @Override public void execute(MinecraftServer server, ICommandSender sender, String... args) throws CommandException { EntityPlayerMP player = args.length < 2 ? getCommandSenderAsPlayer(sender) : getPlayer(server, sender, args[0]); boolean safe = false; if (args.length > 0 && args.length < 3) { int k = args.length - 1; if (args[k].equals("true") || args[k].equals("false")) { safe = Boolean.parseBoolean(args[k]); } else if (args[0].equals("safe")) safe = true; } BlockPos pos = player.getPosition().down(); World world = player.getEntityWorld(); EnumFacing direction = EnumFacing.NORTH; if (RitualHelper.createRitual(world, pos, direction, this.ritual, safe)) sender.sendMessage(new TextComponentTranslation("commands.bloodmagic.success")); else sender.sendMessage(new TextComponentTranslation("commands.bloodmagic.ritual.create.error.outOfWorldBoundaries")); } }); addSubcommand(new CommandTreeHelp(this)); } }</code><technical_language>haha a command for single ritual</technical_language>

Target:
public RitualCreate() { for (Ritual ritual : BloodMagic.RITUAL_MANAGER.getRituals()) { ritualList.add(BloodMagic.RITUAL_MANAGER.getId(ritual)); } }
Source:
<code>private void getMocksForAnnotatedFields(Class<?> testClass) { for (Field field : testClass.getDeclaredFields()) { if (field.isAnnotationPresent(ReplaceWithMock.class)) { overrideModules.add(new MockitoOverrideModule(field.getAnnotation(ReplaceWithMock.class), field.getType())); } <START> if (field.isAnnotationPresent(WrapWithSpy.class)) { <END> spyTargets.add(Key.get(field.getType())); } } }</code><technical_language>is this key unique enough if multiple fields type, in class in class parents</technical_language>

Target:
private void getMocksForAnnotatedFields(Class<?> testClass) { for (Field field : testClass.getDeclaredFields()) { if (field.isAnnotationPresent(ReplaceWithMock.class)) { overrideModules.add(new MockitoOverrideModule<>(field.getAnnotation(ReplaceWithMock.class), field.getType())); } if (field.isAnnotationPresent(WrapWithSpy.class)) { WrapWithSpy spyAnnotation = field.getAnnotation(WrapWithSpy.class); if (spyAnnotation.name().isEmpty()) { spyTargets.add(Key.get(field.getType())); } else { spyTargets.add(Key.get(field.getType(), Names.named(spyAnnotation.name()))); } } } }
Source:
<code>public long getEtag(final Organization organization) { <START> final SqlQuery sqlQuery = Ebean.createSqlQuery("SELECT etag from portal.alerts_etags where organization = ?;"); <END> final models.ebean.Organization ebeanOrganization = models.ebean.Organization.findByOrganization(organization); if (ebeanOrganization != null) { sqlQuery.setParameter(1, ebeanOrganization.getId()); final SqlRow row = sqlQuery.findUnique(); if (row != null) { return row.getLong("etag"); } } return 0; }</code><technical_language>I love calling this etags in database. tracking version of set of data (e.g. organization's alerts). more in organization table a column</technical_language>

Target:
public long getEtag(final Organization organization) { return AlertEtags.getEtagByOrganization(organization); }
Source:
<code>public static int maxMemorySizeWithHotplugInMb(int osId, Version compatibilityVersion) { final ConfigValues configValue = getMaxMemConfigValueByOsId(osId); return Config.<Integer>getValue( configValue, <START> (compatibilityVersion != null) ? compatibilityVersion.getValue() : Version.getLast().getValue() <END> ); }</code><technical_language>Please remove unnecessary brackets: return Config.<Integer>getValue( configValue, compatibilityVersion != null ? compatibilityVersion.getValue() : Version.getLast().getValue()</technical_language>

Target:
public static int maxMemorySizeWithHotplugInMb(int osId, Version compatibilityVersion) { final ConfigValues configValue = getMaxMemConfigValueByOsId(osId); return Config.<Integer>getValue( configValue, compatibilityVersion != null ? compatibilityVersion.getValue() : Version.getLast().getValue() ); }