auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public String generateDeviceSpecFile(ITestDevice device) { Path specFilePath = Paths.get( getBundletoolFile().getParentFile().getAbsolutePath(), device.getSerialNumber() + DEVICE_SPEC_FILE_EXTENSION); if (Files.exists(specFilePath)) { return specFilePath.toString(); } String outputDirArg = DEVICE_SPEC_OUTPUT_FLAG + specFilePath.toString(); <START> String[] generateDeviceSpecCmd = new String[] { "java", "-jar", getBundletoolFile().getAbsolutePath(), GET_DEVICE_SPEC_OPTION, outputDirArg }; <END> if (getAdbPath() != null) { String adbArg = "--adb=" + getAdbPath(); List<String> cmdList = new ArrayList<>(Arrays.asList(generateDeviceSpecCmd)); cmdList.add(adbArg); generateDeviceSpecCmd = cmdList.toArray(new String[cmdList.size()]); } CommandResult res = getRunUtil().runTimedCmd(CMD_TIME_OUT, generateDeviceSpecCmd); if (!CommandStatus.SUCCESS.equals(res.getStatus())) { CLog.e( "Failed to generated device spec file. Cmd is %s. Error: %s.", Arrays.toString(generateDeviceSpecCmd), res.getStderr()); return null; } return specFilePath.toString(); }</code><technical_language>this a List (to avoid conversion bellow). add option adb path it. convert String[] in runTimedCmd once</technical_language>

Target:
public String generateDeviceSpecFile(ITestDevice device) { Path specFilePath = Paths.get( getBundletoolFile().getParentFile().getAbsolutePath(), device.getSerialNumber() + DEVICE_SPEC_FILE_EXTENSION); if (Files.exists(specFilePath)) { return specFilePath.toString(); } String outputDirArg = DEVICE_SPEC_OUTPUT_FLAG + specFilePath.toString(); List<String> generateDeviceSpecCmd = new ArrayList<String>( Arrays.asList( "java", "-jar", getBundletoolFile().getAbsolutePath(), GET_DEVICE_SPEC_OPTION, outputDirArg)); if (getAdbPath() != null) { generateDeviceSpecCmd.add("--adb=" + getAdbPath()); } CommandResult res = getRunUtil() .runTimedCmd( CMD_TIME_OUT, generateDeviceSpecCmd.toArray( new String[generateDeviceSpecCmd.size()])); if (!CommandStatus.SUCCESS.equals(res.getStatus())) { CLog.e( "Failed to generated device spec file. Cmd is %s. Error: %s.", generateDeviceSpecCmd.toString(), res.getStderr()); return null; } return specFilePath.toString(); }
Source:
<code>private static String[] getDependencies(ModelTestControllerVersion version) { switch (version) { case EAP_6_4_0: case EAP_6_4_7: return new String[] { formatEAP6SubsystemArtifact(version), "org.infinispan:infinispan-core:5.2.11.Final-redhat-2", "org.infinispan:infinispan-cachestore-jdbc:5.2.11.Final-redhat-2", }; case EAP_7_0_0: case EAP_7_1_0: <START> case EAP_7_2_0_TEMP: <END> return new String[] { formatEAP7SubsystemArtifact(version), formatArtifact("org.jboss.eap:wildfly-clustering-common:%s", version), formatArtifact("org.jboss.eap:wildfly-clustering-infinispan-spi:%s", version), formatArtifact("org.jboss.eap:wildfly-clustering-jgroups-extension:%s", version), formatArtifact("org.jboss.eap:wildfly-clustering-jgroups-spi:%s", version), formatArtifact("org.jboss.eap:wildfly-clustering-service:%s", version), formatArtifact("org.jboss.eap:wildfly-clustering-singleton-api:%s", version), formatArtifact("org.jboss.eap:wildfly-clustering-spi:%s", version), formatArtifact("org.jboss.eap:wildfly-connector:%s", version), }; default: { throw new IllegalArgumentException(); } } }</code><technical_language>EAP_7_2_0</technical_language>

Target:
private static String[] getDependencies(ModelTestControllerVersion version) { switch (version) { case EAP_6_4_0: case EAP_6_4_7: return new String[] { formatEAP6SubsystemArtifact(version), "org.infinispan:infinispan-core:5.2.11.Final-redhat-2", "org.infinispan:infinispan-cachestore-jdbc:5.2.11.Final-redhat-2", }; case EAP_7_0_0: case EAP_7_1_0: case EAP_7_2_0: return new String[] { formatEAP7SubsystemArtifact(version), formatArtifact("org.jboss.eap:wildfly-clustering-common:%s", version), formatArtifact("org.jboss.eap:wildfly-clustering-infinispan-spi:%s", version), formatArtifact("org.jboss.eap:wildfly-clustering-jgroups-extension:%s", version), formatArtifact("org.jboss.eap:wildfly-clustering-jgroups-spi:%s", version), formatArtifact("org.jboss.eap:wildfly-clustering-service:%s", version), formatArtifact("org.jboss.eap:wildfly-clustering-singleton-api:%s", version), formatArtifact("org.jboss.eap:wildfly-clustering-spi:%s", version), formatArtifact("org.jboss.eap:wildfly-connector:%s", version), }; default: { throw new IllegalArgumentException(); } } }
Source:
<code>private void createCompareModeAction() { final IToolBarManager barManager = getSite().getActionBars() .getToolBarManager(); compareModeAction = new Action(UIText.GitHistoryPage_compareMode, <START> UIIcons.ELCL16_COMPARE_VIEW) { <END> public void run() { compareMode = !compareMode; setChecked(compareMode); } }; compareModeAction.setChecked(compareMode); compareModeAction.setToolTipText(UIText.GitHistoryPage_compareMode); barManager.add(compareModeAction); }</code><technical_language>I explicitly spec this action is a check item IAction.AS_CHECK_BOX</technical_language>

Target:
private void createCompareModeAction() { final IToolBarManager barManager = getSite().getActionBars() .getToolBarManager(); compareModeAction = new Action(UIText.GitHistoryPage_compareMode, IAction.AS_CHECK_BOX) { public void run() { compareMode = !compareMode; setChecked(compareMode); } }; compareModeAction.setImageDescriptor(UIIcons.ELCL16_COMPARE_VIEW); compareModeAction.setChecked(compareMode); compareModeAction.setToolTipText(UIText.GitHistoryPage_compareMode); barManager.add(compareModeAction); }
Source:
<code>public void testThatArtifactPayloadIsCorrect() throws Exception { File basedir = getBasedir("projects/iuWithPayload/"); File platform = new File("src/test/resources/eclipse"); List<MavenProject> projects = getSortedProjects(basedir, platform); MavenProject project = getProject(projects, "iuWithPayload"); MavenSession session = newMavenSession(project, projects); lookupMojoWithDefaultConfiguration(project, session, "build-qualifier").execute(); PackageIUMojo mojo = (PackageIUMojo) <START> lookupMojo("package-iu", project.getFile()); setVariableValueToObject(mojo, "project", project); setVariableValueToObject(mojo, "session", session); <END> mojo.execute(); ZipFile zip = new ZipFile(new File(basedir, "target/iuWithPayload-1.0.0.zip")); try { ZipEntry entry = zip.getEntry("file.txt"); assertNotNull("Missing file in artifact", entry); } finally { zip.close(); } }</code><technical_language>replace this lookupConfiguredMojo(project, "package-iu") This automatically sets default values declared in Mojo. omit basedir outputDirectory configuration in src/test/resources/projects/iuWithPayload/pom.xml</technical_language>

Target:
public void testThatArtifactPayloadIsCorrect() throws Exception { File basedir = getBasedir("projects/iuWithPayload/"); File platform = new File("src/test/resources/eclipse"); List<MavenProject> projects = getSortedProjects(basedir, platform); MavenProject project = getProject(projects, "iuWithPayload"); MavenSession session = newMavenSession(project, projects); lookupMojoWithDefaultConfiguration(project, session, "build-qualifier").execute(); PackageIUMojo mojo = (PackageIUMojo) lookupConfiguredMojo(project, "package-iu"); setVariableValueToObject(mojo, "artifactContentFolder", new File(basedir, "src/main/resources").getAbsolutePath()); mojo.execute(); assertThat(getFilesInZip(new File(basedir, "target/iuWithPayload-1.0.0.zip")), hasItem("file.txt")); }
Source:
<code>public byte[] getCacheKey() { byte[] dimBytes = StringUtils.toUtf8(this.dimension); byte[] valBytes = StringUtils.toUtf8(this.value); byte[] extractionFnBytes = null; if (this.getExtractionFn() != null) { extractionFnBytes = this.getExtractionFn().getCacheKey(); } if (extractionFnBytes != null) { return ByteBuffer.allocate(2 + dimBytes.length + valBytes.length + extractionFnBytes.length) .put(CACHE_KEY) .put(dimBytes) .put(STRING_SEPARATOR) <START> .put(valBytes) <END> .put(extractionFnBytes) .array(); } else { return ByteBuffer.allocate(2 + dimBytes.length + valBytes.length) .put(CACHE_KEY) .put(dimBytes) .put(STRING_SEPARATOR) .put(valBytes) .array(); } }</code><technical_language>need STRING_SEPARATOR</technical_language>

Target:
public byte[] getCacheKey() { byte[] dimBytes = StringUtils.toUtf8(dimension); byte[] valBytes = StringUtils.toUtf8(value); byte extractionFnBytes = this.getExtractionFn().getCacheKey(); return ByteBuffer.allocate(3 + dimBytes.length + valBytes.length + extractionFnBytes.length) .put(CACHE_KEY) .put(dimBytes) .put(STRING_SEPARATOR) .put(valBytes) .put(STRING_SEPARATOR) .put(extractionFnBytes) .array(); }
Source:
<code>public Response stop(Action action) { <START> validateParameters(action); <END> return performServiceAction(action, GlusterServiceActionType.stop); }</code><technical_language>rsdl_metadata.yaml validating fields: action.name: 'xs:string', action.service_type: 'xs:string'</technical_language>

Target:
public Response stop(Action action) { validateParameters(action); return performServiceAction(action, GlusterServiceActionType.STOP); }
Source:
<code>public void testJdbcPOJOPollInputOperatorApplication() throws Exception { try { LocalMode lma = LocalMode.newInstance(); Configuration conf = new Configuration(false); conf.addResource(this.getClass().getResourceAsStream("/JdbcProperties.xml")); lma.prepareDAG(new JdbcPOJOPollInputOperatorApplication(), conf); LocalMode.Controller lc = lma.getController(); lc.setHeartbeatMonitoringEnabled(false); ((StramLocalCluster)lc).setExitCondition(new Callable<Boolean>() { @Override public Boolean call() throws Exception { return TupleCount == 10; } }); lc.run(10000); <START> Assert.assertEquals("rows in db", TupleCount, getNumOfRowsinTable(TABLE_POJO_NAME)); <END> } catch (ConstraintViolationException e) { Assert.fail("constraint violations: " + e.getConstraintViolations()); } }</code><technical_language>This test fail intermittently. Check for num rows exit condition well</technical_language>

Target:
public void testJdbcPOJOPollInputOperatorApplication() throws Exception { testApplication(new JdbcPOJOPollInputOperatorApplication()); }
Source:
<code>public static BoilerpipeExtractor getExtractor(String boilerpipeExtractorName) { if (!extractorRepository.containsKey(boilerpipeExtractorName)) { boilerpipeExtractorName = "de.l3s.boilerpipe.extractors." + boilerpipeExtractorName; try { ClassLoader loader = BoilerpipeExtractor.class.getClassLoader(); Class extractorClass = loader.loadClass(boilerpipeExtractorName); extractorRepository.put(boilerpipeExtractorName, (BoilerpipeExtractor)extractorClass.newInstance()); } catch (ClassNotFoundException e) { <START> LOG.error("BoilerpipeExtractor " + boilerpipeExtractorName + " not found!"); <END> } catch (InstantiationException e) { LOG.error("Could not instantiate " + boilerpipeExtractorName); } catch (Exception e) { LOG.error(e); } } return extractorRepository.get(boilerpipeExtractorName); }</code><technical_language>In slf4j structure catch <LINK_0> e.g. logger.debug("The entry is {}.", entry);</technical_language>

Target:
public static BoilerpipeExtractor getExtractor(String boilerpipeExtractorName) { if (!extractorRepository.containsKey(boilerpipeExtractorName)) { boilerpipeExtractorName = "de.l3s.boilerpipe.extractors." + boilerpipeExtractorName; try { ClassLoader loader = BoilerpipeExtractor.class.getClassLoader(); Class extractorClass = loader.loadClass(boilerpipeExtractorName); extractorRepository.put(boilerpipeExtractorName, (BoilerpipeExtractor)extractorClass.newInstance()); } catch (ClassNotFoundException e) { LOG.error("BoilerpipeExtractor not found!", boilerpipeExtractorName); } catch (InstantiationException e) { LOG.error("Could not instantiate {}!", boilerpipeExtractorName); } catch (Exception e) { LOG.error("Error due to the {}!",boilerpipeExtractorName, e); } } return extractorRepository.get(boilerpipeExtractorName); }
Source:
<code>public long updateBandwidthEstimate(long nowMs) { currentBitrateBps = changeBitrate( currentBitrateBps, currentInput.incomingBitRate, nowMs); DiagnosticContext diagnosticContext = this.diagnosticContextProvider.getDiagnosticContext(); <START> if (isValidEstimate() <END> && logger.isTraceEnabled() && diagnosticContext != null) { logger.trace(diagnosticContext .makeTimeSeriesPoint("new_rate_estimate") .addField("estimate", currentBitrateBps) .addField("incoming", currentInput.incomingBitRate) .setTimestampMs(nowMs)); } if (nowMs - timeOfLastLog > kLogIntervalMs) timeOfLastLog = nowMs; return currentBitrateBps; }</code><technical_language>expensive isValidEstimate is, move isTraceEnabled is more return false</technical_language>

Target:
public long updateBandwidthEstimate(long nowMs) { currentBitrateBps = changeBitrate( currentBitrateBps, currentInput.incomingBitRate, nowMs); if (logger.isTraceEnabled() && isValidEstimate()) { logger.trace(diagnosticContext .makeTimeSeriesPoint("new_rate_estimate", nowMs) .addKey("aimd", hashCode()) .addField("estimate", currentBitrateBps) .addField("incoming", currentInput.incomingBitRate)); } if (nowMs - timeOfLastLog > kLogIntervalMs) timeOfLastLog = nowMs; return currentBitrateBps; }
Source:
<code>public void run() { log.info("Starting writer"); try { writerRegistry.register(this); running = true; runUntilCanceled(); } catch (InterruptedException ie) { log.info("Exiting due to thread interrupt"); Thread.currentThread().interrupt(); <START> } catch (Exception e) { <END> log.error("Caught exception running writer: {}", e.getMessage(), e); } finally { running = false; writerRegistry.unregister(this); } }</code><technical_language>catch bare Exception</technical_language>

Target:
public void run() { log.info("Starting writer"); try { writerRegistry.register(this); running = true; runUntilCanceled(); } catch (InterruptedException ie) { log.info("Exiting due to thread interrupt"); Thread.currentThread().interrupt(); } finally { running = false; writerRegistry.unregister(this); } }
Source:
<code>public final void readExecutionData(File jacocoExecutionData, SensorContext context) throws IOException { ExecutionDataVisitor executionDataVisitor = new ExecutionDataVisitor(); <START> if (jacocoExecutionData == null || !jacocoExecutionData.exists() || !jacocoExecutionData.isFile()) { <END> JaCoCoExtensions.LOG.warn("Project coverage is set to 0% as no JaCoCo execution data has been dumped: {}", jacocoExecutionData); } else { JaCoCoExtensions.LOG.info("Analysing {}", jacocoExecutionData); InputStream inputStream = null; try { inputStream = new BufferedInputStream(new FileInputStream(jacocoExecutionData)); ExecutionDataReader reader = new ExecutionDataReader(inputStream); reader.setSessionInfoVisitor(executionDataVisitor); reader.setExecutionDataVisitor(executionDataVisitor); reader.read(); } finally { Closeables.closeQuietly(inputStream); } } CoverageBuilder coverageBuilder = analyze(executionDataVisitor.getMerged()); int analyzedResources = 0; for (ISourceFileCoverage coverage : coverageBuilder.getSourceFiles()) { Resource resource = getResource(coverage, context); if (resource != null) { CoverageMeasuresBuilder builder = analyzeFile(resource, coverage); saveMeasures(context, resource, builder.createMeasures()); analyzedResources++; } } if (analyzedResources == 0) { JaCoCoExtensions.LOG.warn("Coverage information was not collected. Perhaps you forget to include debug information into compiled classes?"); } else { JaCoCoExtensions.LOG.info("No information about coverage per test."); } }</code><technical_language>isFile() checks for existence</technical_language>

Target:
public final void readExecutionData(File jacocoExecutionData, SensorContext context) throws IOException { ExecutionDataVisitor executionDataVisitor = new ExecutionDataVisitor(); if (jacocoExecutionData == null || !jacocoExecutionData.isFile()) { JaCoCoExtensions.LOG.warn("Project coverage is set to 0% as no JaCoCo execution data has been dumped: {}", jacocoExecutionData); } else { JaCoCoExtensions.LOG.info("Analysing {}", jacocoExecutionData); InputStream inputStream = null; try { inputStream = new BufferedInputStream(new FileInputStream(jacocoExecutionData)); ExecutionDataReader reader = new ExecutionDataReader(inputStream); reader.setSessionInfoVisitor(executionDataVisitor); reader.setExecutionDataVisitor(executionDataVisitor); reader.read(); } finally { Closeables.closeQuietly(inputStream); } } CoverageBuilder coverageBuilder = analyze(executionDataVisitor.getMerged()); int analyzedResources = 0; for (ISourceFileCoverage coverage : coverageBuilder.getSourceFiles()) { InputFile groovyFile = getInputFile(coverage); if (groovyFile != null) { CoverageMeasuresBuilder builder = analyzeFile(coverage); saveMeasures(context, groovyFile, builder.createMeasures()); analyzedResources++; } } if (analyzedResources == 0) { JaCoCoExtensions.LOG.warn("Coverage information was not collected. Perhaps you forget to include debug information into compiled classes?"); } else { JaCoCoExtensions.LOG.info("No information about coverage per test."); } }
Source:
<code>protected Set<PropertyPathConnector> getAllSchemaPropertiesConnectors(boolean isOutputConnection) { if (isOutputConnection) { return Collections.singleton(MAIN_CONNECTOR); } else { <START> return Collections.EMPTY_SET; <END> } }</code><technical_language>call method this remove warning: EMPTY_SET -> emptySet()</technical_language>

Target:
protected Set<PropertyPathConnector> getAllSchemaPropertiesConnectors(boolean isOutputConnection) { if (isOutputConnection) { return Collections.singleton(MAIN_CONNECTOR); } else { return Collections.emptySet(); } }
Source:
<code>public RqBuffered(final Request req) { super( <START> new RqOf( <END> req::head, () -> new BufferedInputStream(req.body()) ) ); }</code><technical_language>@fanifieiev RequestOf is a name RqOf</technical_language>

Target:
public RqBuffered(final Request req) { super( new RequestOf( req::head, () -> new BufferedInputStream(req.body()) ) ); }
Source:
<code>public Merchandise__c() { <START> super(); <END> getAttributes().setType("Merchandise__c"); }</code><technical_language>suggestion</technical_language>

Target:
public Merchandise__c() { getAttributes().setType("Merchandise__c"); }
Source:
<code>public synchronized boolean append(@NonNull ICTFPacketDescriptor entry) { ICTFPacketDescriptor entryToAdd = entry; if (entryToAdd.getTimestampBegin() > entryToAdd.getTimestampEnd()) { Activator.log(IStatus.WARNING, "Packet at offset " + entryToAdd.getOffsetBytes() + " begin timestamp is after end timestamp"); entryToAdd = new StreamInputPacketIndexEntry(entryToAdd, Long.MAX_VALUE); } if (!fEntries.isEmpty()) { ICTFPacketDescriptor lastElement = lastElement(); if ((entryToAdd.getTimestampBegin() < lastElement.getTimestampBegin()) && (lastElement.getContentSizeBits() > lastElement.getPayloadStartBits())) { return false; } <START> } <END> fEntries.add(entryToAdd); return true; }</code><technical_language>If I understand correctly: - entry is added, if empty - new entry is added if timestamp is increasing, if empty - new entry is added if timestamp is out of order, long previous entry empty - new entry is if timestamp is out of order previous entry empty Is this intended behaviour</technical_language>

Target:
public synchronized boolean append(@NonNull ICTFPacketDescriptor entry) { ICTFPacketDescriptor entryToAdd = entry; if (entryToAdd.getTimestampBegin() > entryToAdd.getTimestampEnd()) { Activator.log(IStatus.WARNING, "Packet at offset " + entryToAdd.getOffsetBytes() + " begin timestamp is after end timestamp"); entryToAdd = new StreamInputPacketIndexEntry(entryToAdd, Long.MAX_VALUE); } if (!contains(entryToAdd)) { fEntries.add(entryToAdd); fEntries.sort(OFFSET_COMPARE); return true; } return false; }
Source:
<code>private static int printClassDefinitions( String basename, AClass aClass, PrintWriter printWriter) { String[] classNames = basename.split("\\$"); TypeElement innermostTypeElt = aClass.getTypeElement(); TypeElement[] typeElements = getTypeElementsForClasses(innermostTypeElt, classNames); <START> if (innermostTypeElt == null) { <END> throw new BugInCF("typeElement was unexpectedly null in this aClass: " + aClass); } for (int i = 0; i < classNames.length; i++) { String nameToPrint = classNames[i]; printWriter.print(indents(i)); if (aClass.isEnum(nameToPrint)) { printWriter.print("enum "); } else { printWriter.print("class "); } if (i == classNames.length - 1) { printWriter.print(formatAnnotations(aClass.getAnnotations())); } printWriter.print(nameToPrint); printTypeParameters(typeElements[i], printWriter); printWriter.println(" {"); if (aClass.isEnum(nameToPrint) && i != classNames.length - 1) { printWriter.println(indents(i + 1) + " ;"); } printWriter.println(); } return classNames.length; }</code><technical_language>I test innermostTypeElt it; is, transpose statements</technical_language>

Target:
private static int printClassDefinitions( String basename, AClass aClass, PrintWriter printWriter) { String[] classNames = basename.split("\\$"); TypeElement innermostTypeElt = aClass.getTypeElement(); if (innermostTypeElt == null) { throw new BugInCF("typeElement was unexpectedly null in this aClass: " + aClass); } TypeElement[] typeElements = getTypeElementsForClasses(innermostTypeElt, classNames); for (int i = 0; i < classNames.length; i++) { String nameToPrint = classNames[i]; printWriter.print(indents(i)); if (aClass.isEnum(nameToPrint)) { printWriter.print("enum "); } else { printWriter.print("class "); } if (i == classNames.length - 1) { printWriter.print(formatAnnotations(aClass.getAnnotations())); } printWriter.print(nameToPrint); printTypeParameters(typeElements[i], printWriter); printWriter.println(" {"); if (aClass.isEnum(nameToPrint) && i != classNames.length - 1) { printWriter.println(indents(i + 1) + " ;"); } printWriter.println(); } return classNames.length; }
Source:
<code><START> public JenkinsOrchestrator newFreestyleJobWithMaven(String jobName, File projectPath, String branch) { <END> newFreestyleJobConfig(jobName, projectPath); findElement(By.name("hudson-plugins-sonar-SonarBuildWrapper")).click(); findElement(buttonByText("Add build step")).click(); findElement(By.linkText("Invoke top-level Maven targets")).click(); setTextValue(findElement(By.name("_.targets")), "clean package"); findElement(buttonByText("Add build step")).click(); findElement(By.linkText("Invoke top-level Maven targets")).click(); setTextValue(findElement(By.name("_.targets"), 1), "$SONAR_MAVEN_GOAL -Dsonar.host.url=$SONAR_HOST_URL"); findElement(buttonByText("Save")).click(); return this; }</code><technical_language>renamed newFreestyleJobWithStandardMavenStep I Maven publisher</technical_language>

Target:
public JenkinsOrchestrator newFreestyleJobWithMaven(String jobName, File projectPath, String branch) { newFreestyleJobConfig(jobName, projectPath); findElement(By.name("hudson-plugins-sonar-SonarBuildWrapper")).click(); findElement(buttonByText("Add build step")).click(); findElement(By.linkText("Invoke top-level Maven targets")).click(); setTextValue(findElement(By.name("_.targets")), "clean package"); findElement(buttonByText("Add build step")).click(); findElement(By.linkText("Invoke top-level Maven targets")).click(); setTextValue(findElement(By.name("_.targets"), 1), getMavenParams()); findElement(buttonByText("Save")).click(); return this; }
Source:
<code><START> default Iterable<PipelinedRegion<VID, RID, V, R>> getAllPipelinedRegions() { <END> throw new UnsupportedOperationException(); }</code><technical_language>I ExecutionPipelinedRegion interface inherits PipelinedRegion interface. Similar Topology -> SchedulingTopology. SchedulingStrategy operate ExecutionPipelinedRegion instead. Note a LogicalPipelinedRegion I rework it. this method need default LogicalTopology return logical regions (we this method implementation)</technical_language>

Target:
default Iterable<PR> getAllPipelinedRegions() { throw new UnsupportedOperationException(); }
Source:
<code>public void testAllJdkCipherSuitesMapping() { final MechanismDatabase mechanismDatabase = MechanismDatabase.getInstance(); SSLServerSocketFactory ssf = (SSLServerSocketFactory) SSLServerSocketFactory.getDefault(); String[] supportedCipherSuites = ssf.getSupportedCipherSuites(); <START> List<String> TLS1_3CipherSuites = Arrays.asList("TLS_AES_128_GCM_SHA256", "TLS_AES_256_GCM_SHA384"); <END> String unknownCipherSuites = Arrays.stream(supportedCipherSuites) .filter(cipherSuite -> !TLS1_3CipherSuites.contains(cipherSuite) && !cipherSuite.equals("TLS_EMPTY_RENEGOTIATION_INFO_SCSV") && mechanismDatabase.getCipherSuite(cipherSuite) == null) .collect(Collectors.joining(", ")); Assert.assertTrue("There are JDK cipher suites which are unknown to Elytron MechanismDatabase: " + unknownCipherSuites, unknownCipherSuites.isEmpty()); }</code><technical_language>I safer list of TLS 1.3 cipher suites (the dependent JDK version)</technical_language>

Target:
public void testAllJdkCipherSuitesMapping() { final MechanismDatabase mechanismDatabase = MechanismDatabase.getInstance(); SSLServerSocketFactory ssf = (SSLServerSocketFactory) SSLServerSocketFactory.getDefault(); String[] supportedCipherSuites = ssf.getSupportedCipherSuites(); List<String> TLS1_3CipherSuites = Arrays.asList("TLS_AES_128_GCM_SHA256", "TLS_AES_256_GCM_SHA384", "TLS_CHACHA20_POLY1305_SHA256", "TLS_AES_128_CCM_SHA256", "TLS_AES_128_CCM_8_SHA256"); String unknownCipherSuites = Arrays.stream(supportedCipherSuites) .filter(cipherSuite -> !TLS1_3CipherSuites.contains(cipherSuite) && !cipherSuite.equals("TLS_EMPTY_RENEGOTIATION_INFO_SCSV") && mechanismDatabase.getCipherSuite(cipherSuite) == null) .collect(Collectors.joining(", ")); Assert.assertTrue("There are JDK cipher suites which are unknown to Elytron MechanismDatabase: " + unknownCipherSuites, unknownCipherSuites.isEmpty()); }
Source:
<code>public void testListPathWithValidListMaxResultsValues() throws IOException, ExecutionException, InterruptedException { final int fileCount = 10; final String directory = "testWithValidListMaxResultsValues"; createDirectoryWithNFiles(directory, fileCount); final int[] testData = {fileCount + 100, fileCount + 1, fileCount, fileCount - 1, 1}; for (int i = 0; i < testData.length; i++) { int listMaxResults = testData[i]; setListMaxResults(listMaxResults); int expectedListResultsSize = listMaxResults > fileCount ? fileCount : listMaxResults; <START> assertThat(listPath(directory).size(), <END> is(equalTo(expectedListResultsSize))); } }</code><technical_language>assertJ below. is of errors gives, e.g. assertThat(listPath).describedAs(...).hasSize(...)</technical_language>

Target:
public void testListPathWithValidListMaxResultsValues() throws IOException, ExecutionException, InterruptedException { final int fileCount = 10; final String directory = "testWithValidListMaxResultsValues"; createDirectoryWithNFiles(directory, fileCount); final int[] testData = {fileCount + 100, fileCount + 1, fileCount, fileCount - 1, 1}; for (int i = 0; i < testData.length; i++) { int listMaxResults = testData[i]; setListMaxResults(listMaxResults); int expectedListResultsSize = listMaxResults > fileCount ? fileCount : listMaxResults; Assertions.assertThat(listPath(directory)).describedAs( "AbfsClient.listPath result should contain %d items when " + "listMaxResults is %d and directory contains %d items", expectedListResultsSize, listMaxResults, fileCount) .hasSize(expectedListResultsSize); } }
Source:
<code>public String choose(VolumeChooserEnvironment env, String[] options) throws AccumuloException { if (log.isTraceEnabled()) { log.trace("PerTableVolumeChooser.choose"); } VolumeChooser chooser; if (!env.hasTableId() && !env.hasScope()) { return randomChooser.choose(env, options); } ServerConfigurationFactory localConf = loadConf(); lazilyCreateFallbackChooser(); if (env.hasScope()) { chooser = getVolumeChooserForNonTable(env, localConf); } else { chooser = getVolumeChooserForTable(env, localConf); } <START> return chooser.choose(env, options); <END> }</code><technical_language>ignorance part, env.hasTableId() checked first? env.hasTableId() && env.hasScope()</technical_language>

Target:
public String choose(VolumeChooserEnvironment env, String[] options) throws VolumeChooserException { log.trace("{}.choose", getClass().getSimpleName()); return getDelegateChooser(env).choose(env, options); }
Source:
<code>private static void showDialog(final Context context, String title, String message) { AlertDialog.Builder alertDialog = new AlertDialog.Builder(context); alertDialog.setTitle(title); alertDialog.setMessage(message) <START> .setPositiveButton("OK", new DialogInterface.OnClickListener() { <END> @Override public void onClick(DialogInterface dialog, int which) { dialog.dismiss(); } }); alertDialog.show(); }</code><technical_language>need hardcode "OK"</technical_language>

Target:
private static void showDialog(final Context context, String title, String message) { AlertDialog.Builder alertDialog = new AlertDialog.Builder(context); alertDialog.setTitle(title); alertDialog.setMessage(message) .setPositiveButton(R.string.ok, new DialogInterface.OnClickListener() { @Override public void onClick(DialogInterface dialog, int which) { dialog.dismiss(); } }); alertDialog.show(); }
Source:
<code>public void updatePresentation(@NotNull NodePresentation presentation) { presentation.setPresentableText(getData().getName()); presentation.setPresentableIcon(isValid(getData()) ? nodeManager.getNodesResources().projectFolder() : nodeManager.getNodesResources().notValidProjectFolder()); presentation.setPresentableTextCss("font-weight:bold"); <START> if ("private".equals(getData().getVisibility())) { <END> SpanElement lockElement = Document.get().createSpanElement(); lockElement.appendChild(nodeManager.getNodesResources().lock().getSvg().getElement()); presentation.setUserElement(lockElement); } }</code><technical_language>Do show some icon If visibility equals public? If no, why? I is good behaviour show icon for private project show some icon for public projects</technical_language>

Target:
public void updatePresentation(@NotNull NodePresentation presentation) { presentation.setPresentableText(getData().getName()); presentation.setPresentableIcon(isValid(getData()) ? nodeManager.getNodesResources().projectFolder() : nodeManager.getNodesResources().notValidProjectFolder()); presentation.setPresentableTextCss("font-weight:bold"); }
Source:
<code>private void setupBackupRenamePath() { <START> DateTimeFormatter formatter = configuration.getBackupDirDateTimeFormatter(); <END> String subDirNm = formatter.format(LocalDate.now()); int n = 0; do { renamePath.of(configuration.getBackupRoot()).put(Files.SEPARATOR).concat(subDirNm); if (n > 0) { renamePath.put('.').put(Integer.valueOf(n).toString()); } renamePath.put(Files.SEPARATOR).$(); n++; } while (ff.exists(renamePath)); if (ff.mkdirs(renamePath, configuration.getMkDirMode()) != 0) { throw CairoException.instance(ff.errno()).put("could not create [dir=").put(renamePath).put(']'); } }</code><technical_language>date library functions for</technical_language>

Target:
private void setupBackupRenamePath() { TimestampFormat format = configuration.getBackupDirTimestampFormat(); long epochMicros = configuration.getMicrosecondClock().getTicks(); int n = 0; renamePath.of(configuration.getBackupRoot()).put(Files.SEPARATOR); int plen = renamePath.length(); do { renamePath.trimTo(plen); format.format(epochMicros, configuration.getDefaultTimestampLocale(), null, renamePath); if (n > 0) { renamePath.put('.').put(n); } renamePath.put(Files.SEPARATOR).$(); n++; } while (ff.exists(renamePath)); if (ff.mkdirs(renamePath, configuration.getBackupMkDirMode()) != 0) { throw CairoException.instance(ff.errno()).put("could not create [dir=").put(renamePath).put(']'); } }
Source:
<code>protected void verify() { <START> ofNullable(duration).orElseThrow(() -> <END> new IllegalArgumentException("Duration value should not be a null value")); }</code><technical_language>this verification is redundant, _checkNotNull_ verification _withDuration_ method</technical_language>

Target:
protected void verify() { }
Source:
<code>private String getConstraint(final ItemDefinition itemDefinition) { final UnaryTests allowedValues = itemDefinition.getAllowedValues(); <START> if (allowedValues != null && !allowedValues.getText().getValue().isEmpty()) { <END> return translationService.format(DMNDocumentationFactory_Constraints) + allowedValues.getText().getValue(); } return ""; }</code><technical_language>please StringUtils</technical_language>

Target:
private String getConstraint(final ItemDefinition itemDefinition) { final UnaryTests allowedValues = itemDefinition.getAllowedValues(); if (allowedValues != null && !isEmpty(allowedValues.getText().getValue())) { return translationService.format(DMNDocumentationFactory_Constraints) + " " + allowedValues.getText().getValue(); } return ""; }
Source:
<code>public void onPause() { super.onPause(); prefs.unregisterOnSharedPreferenceChangeListener(this); if(!requireFullRestart) { finish(); } else { <START> Toast.makeText(this, "You've changed some important settings, the app will restart.", Toast.LENGTH_SHORT).show(); <END> SharedPreferences prefs = PreferenceManager.getDefaultSharedPreferences(this); prefs.edit().putBoolean("require-layout-update", true).apply(); Intent intent = new Intent(this, getClass()); intent.addFlags(Intent.FLAG_ACTIVITY_CLEAR_TOP | Intent.FLAG_ACTIVITY_NEW_TASK | Intent.FLAG_ACTIVITY_NO_ANIMATION); finish(); overridePendingTransition(0, 0); startActivity(intent); overridePendingTransition(0, 0); return; } }</code><technical_language>Please this translatable</technical_language>

Target:
public void onPause() { super.onPause(); prefs.unregisterOnSharedPreferenceChangeListener(this); if(!requireFullRestart) { finish(); } else { Toast.makeText(this, R.string.app_wil_restart, Toast.LENGTH_SHORT).show(); SharedPreferences prefs = PreferenceManager.getDefaultSharedPreferences(this); prefs.edit().putBoolean("require-layout-update", true).apply(); Intent intent = new Intent(this, getClass()); intent.addFlags(Intent.FLAG_ACTIVITY_CLEAR_TOP | Intent.FLAG_ACTIVITY_NEW_TASK | Intent.FLAG_ACTIVITY_NO_ANIMATION); finish(); overridePendingTransition(0, 0); startActivity(intent); overridePendingTransition(0, 0); return; } }
Source:
<code>public void initialize(ConfigurableApplicationContext applicationContext) { String adminPort = applicationContext.getEnvironment().resolvePlaceholders(ADMIN_PORT_PLACEHOLDER); ServerSocket socket = null; try { socket = new ServerSocket(Integer.parseInt(adminPort)); } catch (IOException e) { throw new AdminPortNotAvailableException(adminPort); } finally { if (socket != null) { try { socket.close(); } catch (IOException e) { <START> e.printStackTrace(); <END> } } } }</code><technical_language>I bother printing stack trace here. do good it</technical_language>

Target:
public void initialize(ConfigurableApplicationContext applicationContext) { String adminPort = applicationContext.getEnvironment().resolvePlaceholders(ADMIN_PORT_PLACEHOLDER); ServerSocket socket = null; try { socket = new ServerSocket(Integer.parseInt(adminPort)); } catch (IOException e) { throw new AdminPortNotAvailableException(adminPort); } finally { if (socket != null) { try { socket.close(); } catch (IOException e) { } } } }
Source:
<code>public void inventoryTick(ItemStack stack, World worldIn, Entity entityIn, int itemSlot, boolean isSelected) { if(entityIn instanceof ServerPlayerEntity) { <START> if(PlayerSavedData.getData((ServerPlayerEntity) entityIn).getTitle() != null) <END> { if (PlayerSavedData.getData((ServerPlayerEntity) entityIn).getTitle().getHeroAspect() == aspect && isSelected) { ((ServerPlayerEntity) entityIn).addPotionEffect(new EffectInstance(effect, duration, effectTier)); } } } }</code><technical_language>Minor suggestion a local variable for title</technical_language>

Target:
public void inventoryTick(ItemStack stack, World worldIn, Entity entityIn, int itemSlot, boolean isSelected) { if(isSelected && entityIn instanceof ServerPlayerEntity) { Title title = PlayerSavedData.getData((ServerPlayerEntity) entityIn).getTitle(); if(title != null) { if (title.getHeroAspect() == aspect) { ((ServerPlayerEntity) entityIn).addPotionEffect(new EffectInstance(effect.get(), duration, effectTier)); } } } }
Source:
<code>static void toGerrit(final String target, final HttpServletRequest req, final HttpServletResponse rsp) throws IOException { final StringBuilder url = new StringBuilder(); <START> url.append(URLEncoder.encode(req.getContextPath(), "UTF-8")); <END> url.append('/'); url.append('#'); url.append(target); rsp.sendRedirect(url.toString()); }</code><technical_language>I this stops admins installing for example /code/review/ server</technical_language>

Target:
static void toGerrit(final String target, final HttpServletRequest req, final HttpServletResponse rsp) throws IOException { final StringBuilder url = new StringBuilder(); url.append(req.getContextPath()); url.append('/'); url.append('#'); url.append(target); rsp.sendRedirect(url.toString()); }
Source:
<code><START> public static <T> PredicateOperation predicate(Operator operator, ImmutableList<Expression<?>> args) { <END> return new PredicateOperation(operator, args); }</code><technical_language><T>s occur in method signature</technical_language>

Target:
public static PredicateOperation predicate(Operator operator, ImmutableList<Expression<?>> args) { return new PredicateOperation(operator, args); }
Source:
<code>public void onCurrentMediaItemChanged(@NonNull final SessionPlayer player, @NonNull final MediaItem item) { final MediaSessionImplBase session = getSession(); if (session == null || player == null || session.getPlayer() != player) { return; } synchronized (session.mLock) { if (mMediaItem != null) { mMediaItem.removeOnMetadataChangedListener(this); } if (item != null) { item.addOnMetadataChangedListener(session.mCallbackExecutor, this); } mMediaItem = item; } <START> boolean notified = updateCurrentMediaItemMetadataWithDuration(player); if (!notified) { notifyCurrentMediaItemChanged(item); } } <END></code><technical_language>ditto. (see below)</technical_language>

Target:
public void onCurrentMediaItemChanged(@NonNull final SessionPlayer player, @NonNull final MediaItem item) { final MediaSessionImplBase session = getSession(); if (session == null || player == null || session.getPlayer() != player) { return; } synchronized (session.mLock) { if (mMediaItem != null) { mMediaItem.removeOnMetadataChangedListener(this); } if (item != null) { item.addOnMetadataChangedListener(session.mCallbackExecutor, this); } mMediaItem = item; } boolean notifyingPended = updateCurrentMediaItemMetadataWithDuration(player); if (!notifyingPended) { notifyCurrentMediaItemChanged(item); } }
Source:
<code>private static void checkNtpSync(ConfigurableApplicationContext context) { String host = context.getBean(IntegrasjonspunktProperties.class).getNtpHost(); try { NTPClient client = new NTPClient(host); long offset = client.getOffset(); if (Math.abs(offset) > 5000) { String errorStr = String.format("Startup failed. Offset from NTP host %s was more than 5 seconds (%sms). Adjust local clock and try again.", host, offset); log.error(errorStr); String stars = "\n**************************\n"; <START> System.out.println(stars+errorStr+stars); <END> context.close(); } } catch (IOException e) { log.error("Error while syncing with NTP {}, continuing startup..", host); } }</code><technical_language>![MAJOR](<LINK_0> 'Severity: MAJOR') Replace this of System.out System.err a logger. [![rule](<LINK_1>](<LINK_2></technical_language>

Target:
private static void checkNtpSync(ConfigurableApplicationContext context) { String host = context.getBean(IntegrasjonspunktProperties.class).getNtpHost(); try { NTPClient client = new NTPClient(host); long offset = client.getOffset(); if (Math.abs(offset) > 5000) { String errorStr = String.format("Startup failed. Offset from NTP host %s was more than 5 seconds (%sms). Adjust local clock and try again.", host, offset); log.error(errorStr); String stars = "\n**************************\n"; System.out.println(stars+errorStr+stars); context.close(); } } catch (IOException e) { log.error(String.format("Error while syncing with NTP %s, continuing startup..", host), e); } }
Source:
<code>protected void setupSSL() { try { sslContext = new SslContextBuilder(InetAddress.getLocalHost().getHostName()).build(); } catch (GeneralSecurityException ex) { <START> throw Throwables.propagate(ex); <END> } catch (UnknownHostException ex) { throw Throwables.propagate(ex); } }</code><technical_language>nit. faster write throw new RuntimeException(e); :)</technical_language>

Target:
protected void setupSSL() { try { sslContext = new SslContextBuilder(InetAddress.getLocalHost().getHostName()).build(); } catch (GeneralSecurityException ex) { throw new RuntimeException(ex); } catch (UnknownHostException ex) { throw new RuntimeException(ex); } }
Source:
<code>private void initDatabase() throws Exception { String jndiName = config.getString("database.jndi"); if (jndiName != null) { dataSource = (DataSource) new InitialContext().lookup(jndiName); } else { String driverFile = config.getString("database.driverFile"); if (driverFile != null) { URLClassLoader classLoader = (URLClassLoader) ClassLoader.getSystemClassLoader(); Method method = URLClassLoader.class.getDeclaredMethod("addURL", URL.class); method.setAccessible(true); method.invoke(classLoader, new File(driverFile).toURI().toURL()); } String driver = config.getString("database.driver"); if (driver != null) { Class.forName(driver); } <START> HikariConfig hConfig = new HikariConfig(); <END> hConfig.setDriverClassName(config.getString("database.driver")); hConfig.setJdbcUrl(config.getString("database.url")); hConfig.setUsername(config.getString("database.user")); hConfig.setPassword(config.getString("database.password")); hConfig.setConnectionInitSql("SELECT 1 FROM DUAL"); hConfig.setIdleTimeout(600000); int maxPoolSize = config.getInteger("database.maxPoolSize"); if (maxPoolSize != 0) { hConfig.setMaximumPoolSize(maxPoolSize); } dataSource = new HikariDataSource(hConfig); } }</code><technical_language>please rename poolConfig hikariConfig that</technical_language>

Target:
private void initDatabase() throws Exception { String jndiName = config.getString("database.jndi"); if (jndiName != null) { dataSource = (DataSource) new InitialContext().lookup(jndiName); } else { String driverFile = config.getString("database.driverFile"); if (driverFile != null) { URLClassLoader classLoader = (URLClassLoader) ClassLoader.getSystemClassLoader(); Method method = URLClassLoader.class.getDeclaredMethod("addURL", URL.class); method.setAccessible(true); method.invoke(classLoader, new File(driverFile).toURI().toURL()); } String driver = config.getString("database.driver"); if (driver != null) { Class.forName(driver); } HikariConfig hikariConfig = new HikariConfig(); hikariConfig.setDriverClassName(config.getString("database.driver")); hikariConfig.setJdbcUrl(config.getString("database.url")); hikariConfig.setUsername(config.getString("database.user")); hikariConfig.setPassword(config.getString("database.password")); hikariConfig.setConnectionInitSql("SELECT 1 FROM DUAL"); hikariConfig.setIdleTimeout(600000); int maxPoolSize = config.getInteger("database.maxPoolSize"); if (maxPoolSize != 0) { hikariConfig.setMaximumPoolSize(maxPoolSize); } dataSource = new HikariDataSource(hikariConfig); } }
Source:
<code>public void addResolverListener(String expression, ResolverListener listener) throws CloudnameException { <START> DynamicExpression dynamicExpression = <END> new DynamicExpression(expression, listener, this, zkGetter); synchronized (dynamicAddressMonitor) { DynamicExpression previousExpression = dynamicAddressesByListener.put( listener, dynamicExpression); if (previousExpression != null) { throw new IllegalArgumentException("It is not legal to register a listener twice."); } } dynamicExpression.start(); }</code><technical_language>final</technical_language>

Target:
public void addResolverListener(String expression, ResolverListener listener) throws CloudnameException { final DynamicExpression dynamicExpression = new DynamicExpression(expression, listener, this, zkGetter); synchronized (dynamicAddressMonitor) { DynamicExpression previousExpression = dynamicAddressesByListener.put( listener, dynamicExpression); if (previousExpression != null) { throw new IllegalArgumentException("It is not legal to register a listener twice."); } } dynamicExpression.start(); }
Source:
<code>private void status(int sc) { if (sc == SC_UNAUTHORIZED) { StringBuilder v = new StringBuilder(); <START> v.append("Basic"); <END> v.append(" realm=\"" + REALM_NAME + "\""); setHeader(WWW_AUTHENTICATE, v.toString()); } else if (containsHeader(WWW_AUTHENTICATE)) { setHeader(WWW_AUTHENTICATE, null); } }</code><technical_language>LIT_BASIC put a space realm in next line</technical_language>

Target:
private void status(int sc) { if (sc == SC_UNAUTHORIZED) { StringBuilder v = new StringBuilder(); v.append(LIT_BASIC); v.append(" realm=\"" + REALM_NAME + "\""); setHeader(WWW_AUTHENTICATE, v.toString()); } else if (containsHeader(WWW_AUTHENTICATE)) { setHeader(WWW_AUTHENTICATE, null); } }
Source:
<code>protected Iterator<ServiceRecord> getServiceRecords(MarketoService marketoService, PluginTask task) { Iterable<ObjectNode> nodes = null; <START> if (!task.getQueryBy().isPresent()) { <END> nodes = marketoService.getPrograms(); } else if (task.getQueryBy().isPresent() && task.getQueryBy().get() == QueryBy.TAG_TYPE) { nodes = marketoService.getProgramsByTag(task.getTagType().get(), task.getTagVallue().get()); } else if (task.getQueryBy().isPresent() && task.getQueryBy().get() == QueryBy.DATE_RANGE) { nodes = marketoService.getProgramsByDateRange(task.getEarliestUpdatedAt().get(), task.getLatestUpdatedAt().get(), task.getFilterType().orNull(), task.getFilterValues().orNull()); } return FluentIterable.from(nodes).transform(MarketoUtils.TRANSFORM_OBJECT_TO_JACKSON_SERVICE_RECORD_FUNCTION).iterator(); }</code><technical_language>above, update if branches</technical_language>

Target:
protected Iterator<ServiceRecord> getServiceRecords(MarketoService marketoService, PluginTask task) { Iterable<ObjectNode> nodes = null; if (task.getQueryBy().isPresent()) { switch (task.getQueryBy().get()) { case TAG_TYPE: nodes = marketoService.getProgramsByTag(task.getTagType().get(), task.getTagVallue().get()); break; case DATE_RANGE: nodes = marketoService.getProgramsByDateRange(task.getEarliestUpdatedAt().get(), task.getLatestUpdatedAt().get(), task.getFilterType().orNull(), task.getFilterValues().orNull()); } } else { nodes = marketoService.getPrograms(); } return FluentIterable.from(nodes).transform(MarketoUtils.TRANSFORM_OBJECT_TO_JACKSON_SERVICE_RECORD_FUNCTION).iterator(); }
Source:
<code><START> public int compareTo(byte[] bytes, int offset, int len) { <END> if (this.length == this.data.length && len == bytes.length) { int res = UnsignedBytes.lexicographicalComparator().compare(this.data, bytes); return UnsignedBytes.lexicographicalComparator().compare(this.data, bytes); } else { int minLen = Math.min(this.length, len); for (int i = this.offset, j = offset; i < minLen; i++, j++) { int a = (this.data[i] & 0xff); int b = (bytes[j] & 0xff); if (a != b) { return a - b; } } return this.length - len; } }</code><technical_language>add a sanity check offset >= 0 && len >=0 && offset+len <= bytes.length. Preconditions.checkArgument guava</technical_language>

Target:
public int compareTo(byte[] bytes, int offset, int len) { Preconditions.checkArgument(offset >= 0 && len >= 0 && offset + len <= bytes.length); return compareToUnchecked(bytes, offset, len); }