auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>private void assertSameRecords(int majorVersion, SAMRecord record1, SAMRecord record2) { Assert.assertEquals(record2.getFlags(), record1.getFlags()); Assert.assertEquals(record2.getReadName(), record1.getReadName()); Assert.assertEquals(record2.getReferenceName(), record1.getReferenceName()); Assert.assertEquals(record2.getAlignmentStart(), record1.getAlignmentStart()); if (record1.getReadBases() == SAMRecord.NULL_SEQUENCE && majorVersion < CramVersions.CRAM_v3.major) ; else <START> Assert.assertEquals(record2.getReadBases(), record1.getReadBases()); <END> Assert.assertEquals(record2.getBaseQualities(), record1.getBaseQualities()); }</code><technical_language>Refactor remove empty if clause, add curly braces</technical_language>

Target:
private void assertSameRecords(int majorVersion, SAMRecord record1, SAMRecord record2) { Assert.assertEquals(record2.getFlags(), record1.getFlags()); Assert.assertEquals(record2.getReadName(), record1.getReadName()); Assert.assertEquals(record2.getReferenceName(), record1.getReferenceName()); Assert.assertEquals(record2.getAlignmentStart(), record1.getAlignmentStart()); if (record1.getReadBases() != SAMRecord.NULL_SEQUENCE || majorVersion >= CramVersions.CRAM_v3.major) { Assert.assertEquals(record2.getReadBases(), record1.getReadBases()); } Assert.assertEquals(record2.getBaseQualities(), record1.getBaseQualities()); }
Source:
<code>public Data getResult() { <START> if (result == null) { <END> return ss.toData(newValue); } return result; }</code><technical_language>Caller do invalidations end. Returning a null result server hides actual issue (also unexpected serialization costs). Do mind investigate NearCachedMapProxyImpl apply logic NearCachedClientMapProxy</technical_language>

Target:
public Data getResult() { return result; }
Source:
<code>public int readUnsignedByte() throws IOException { <START> return sageFileChannel.readUnsignedByte(); <END> }</code><technical_language>This ensureProperFile(false) beginning...and read methods, non-absolute need in case called current file is EOF</technical_language>

Target:
public int readUnsignedByte() throws IOException { ensureProperFile(false); return sageFileChannel.readUnsignedByte(); }
Source:
<code>private LoadBalancer() { ConnectionStateListener connectionStateListener = null; UnhandledErrorListener unhandledErrorListener = null; List<Closeable> closeAbles = Lists.newArrayList(); try { <START> curaFramework = CuratorFrameworkFactory.newClient(getZkConnectString(), <END> new ExponentialBackoffRetry(1000, 3)); curaFramework.start(); curaFramework.setACL().withACL(CONFIG.getAcls()); connectionStateListener = getConnectionStateListener(); curaFramework.getConnectionStateListenable() .addListener(connectionStateListener); unhandledErrorListener = getUnhandledErrorListener(); curaFramework.getUnhandledErrorListenable() .addListener(unhandledErrorListener); cache = new PathChildrenCache(curaFramework, CONFIG.getParentPath(), true); cache.start(PathChildrenCache.StartMode.BUILD_INITIAL_CACHE); closeAbles.add(cache); closeAbles.add(curaFramework); }catch(Exception ex){ LOG.error("Exception while creating a zookeeper clients and cache",ex); if ((curaFramework != null) && (connectionStateListener != null)){ curaFramework.getConnectionStateListenable() .removeListener(connectionStateListener); } if ((curaFramework != null) && (unhandledErrorListener != null)){ curaFramework.getUnhandledErrorListenable() .removeListener(unhandledErrorListener); } for (Closeable closeable : closeAbles) { CloseableUtils.closeQuietly(closeable); } } }</code><technical_language>Move this a start() method. this in constructor leads problems</technical_language>

Target:
private LoadBalancer() { try { start(); }catch(Exception ex){ LOG.error("Exception while creating a zookeeper clients and cache",ex); if ((curaFramework != null) && (connectionStateListener != null)){ curaFramework.getConnectionStateListenable() .removeListener(connectionStateListener); } if ((curaFramework != null) && (unhandledErrorListener != null)){ curaFramework.getUnhandledErrorListenable() .removeListener(unhandledErrorListener); } for (Closeable closeable : closeAbles) { CloseableUtils.closeQuietly(closeable); } } }
Source:
<code>public FormValidation doCheckBasePath(@AncestorInPath AbstractProject project, @QueryParameter String basePath) throws IOException, InterruptedException { if (project == null) { return FormValidation.ok(); } FilePath ws = project.getSomeWorkspace(); FilePath reference; if (ws == null) { reference = Jenkins.getInstance().getRootPath(); } else { reference = ws; } FilePath filePath = reference.child(basePath); <START> if (!filePath.getRemote().startsWith(reference.getRemote())) { <END> return FormValidation.error(Messages.ArtifactArchiver_BasePathOutsideWorkspace()); } if (ws != null && ws.exists()) { if (!filePath.exists()) { return FormValidation.warning(Messages.ArtifactArchiver_BasePathDoesNotExist()); } if (!filePath.isDirectory()) { return FormValidation.warning(Messages.ArtifactArchiver_BasePathIsFile()); } } return FormValidation.ok(); }</code><technical_language>this canonicalized paths</technical_language>

Target:
public FormValidation doCheckBasePath(@AncestorInPath AbstractProject project, @QueryParameter String basePath) throws IOException, InterruptedException { if (project == null) { return FormValidation.ok(); } FilePath ws = project.getSomeWorkspace(); FilePath reference; if (ws == null) { reference = Jenkins.getInstance().getRootPath(); } else { reference = ws; } FilePath filePath = reference.child(basePath); if (!reference.equals(filePath) && !filePath.isDescendantOf(reference)) { return FormValidation.error(Messages.ArtifactArchiver_BasePathOutsideWorkspace()); } if (ws != null && ws.exists()) { if (!filePath.exists()) { return FormValidation.warning(Messages.ArtifactArchiver_BasePathDoesNotExist()); } if (!filePath.isDirectory()) { return FormValidation.warning(Messages.ArtifactArchiver_BasePathIsFile()); } } return FormValidation.ok(); }
Source:
<code>public synchronized ListenableFuture<Supplier<LookupSource>> getLookupSource() { assertIn(State.SPILLED); unspillingRequested.set(null); setState(State.UNSPILLING); unspilledLookupSource = SettableFuture.create(); <START> return unspilledLookupSource; <END> }</code><technical_language>checkState(unspilledLookupSource == null, "unspilledLookupSource set")</technical_language>

Target:
public synchronized ListenableFuture<Supplier<LookupSource>> getLookupSource() { assertState(State.SPILLED); unspillingRequested.set(null); setState(State.UNSPILLING); checkState(unspilledLookupSource == null, "unspilledLookupSource already set"); unspilledLookupSource = SettableFuture.create(); return unspilledLookupSource; }
Source:
<code>public void postCollectionDefaultItemReadGroupCreateDefaultItemReadGroupForbidden() throws Exception { ObjectMapper mapper = new ObjectMapper(); GroupRest groupRest = new GroupRest(); MetadataRest metadataRest = new MetadataRest(); metadataRest.put("dc.description", new MetadataValueRest("testingDescription")); metadataRest.put("dc.subject", new MetadataValueRest("testSubject")); groupRest.setMetadata(metadataRest); String token = getAuthToken(eperson.getEmail(), password); getClient(token).perform(post("/api/core/collections/" + collection.getID() + "/itemReadGroup") .content(mapper.writeValueAsBytes(groupRest)) .contentType(contentType)) .andExpect(status().isForbidden()); <START> <END> }</code><technical_language>double check group created</technical_language>

Target:
public void postCollectionDefaultItemReadGroupCreateDefaultItemReadGroupForbidden() throws Exception { ObjectMapper mapper = new ObjectMapper(); GroupRest groupRest = new GroupRest(); MetadataRest metadataRest = new MetadataRest(); metadataRest.put("dc.description", new MetadataValueRest("testingDescription")); metadataRest.put("dc.subject", new MetadataValueRest("testSubject")); groupRest.setMetadata(metadataRest); String token = getAuthToken(eperson.getEmail(), password); getClient(token).perform(post("/api/core/collections/" + collection.getID() + "/itemReadGroup") .content(mapper.writeValueAsBytes(groupRest)) .contentType(contentType)) .andExpect(status().isForbidden()); token = getAuthToken(admin.getEmail(), password); Group anon = groupService.findByName(context, Group.ANONYMOUS); getClient(token).perform(get("/api/core/collections/" + collection.getID() + "/itemReadGroup")) .andExpect(status().isOk()) .andExpect( jsonPath("$", GroupMatcher.matchGroupEntry(anon.getID(), anon.getName()))); }
Source:
<code>protected void dumpState(StateDumpCollector dump) { dump.addState("cacheName", cache.getName()); <START> dump.addState("cacheSie", String.valueOf(cache.size())); <END> }</code><technical_language>cacheSie</technical_language>

Target:
protected void dumpState(StateDumpCollector dump) { dump.addState("cacheName", cache.getName()); dump.addState("cacheSize", String.valueOf(cache.size())); }
Source:
<code>public void setWriteBuffer(ByteBuffer buffer) { <START> writeBuffer = null; <END> writeBuffer = buffer; }</code><technical_language>This is unnecessary</technical_language>

Target:
public void setWriteBuffer(ByteBuffer buffer) { writeBuffer = buffer; }
Source:
<code>public void testExportOk() throws IOException { final Repository a = createBareRepository(); final String name = a.getDirectory().getName(); final File base = a.getDirectory().getParentFile(); final File export = new File(a.getDirectory(), "git-daemon-export-ok"); FileResolver resolver; assertFalse("no git-daemon-export-ok", export.exists()); resolver = new FileResolver(base, false ); try { resolver.open(null, name); fail("opened non-exported repository"); } catch (ServiceNotEnabledException e) { assertEquals("Service not enabled", e.getMessage()); } resolver = new FileResolver(base, true ); try { resolver.open(null, name).close(); } catch (ServiceNotEnabledException e) { fail("did not honor export-all flag"); } assertTrue(export.createNewFile()); <START> assertTrue("has git-daemon-export-ok", export.exists()); <END> resolver = new FileResolver(base, false ); try { resolver.open(null, name).close(); } catch (ServiceNotEnabledException e) { fail("did not honor git-daemon-export-ok"); } }</code><technical_language>of lines is many. Replace call FileUtils.createNewFile</technical_language>

Target:
public void testExportOk() throws IOException { final Repository a = createBareRepository(); final String name = a.getDirectory().getName(); final File base = a.getDirectory().getParentFile(); final File export = new File(a.getDirectory(), "git-daemon-export-ok"); FileResolver resolver; assertFalse("no git-daemon-export-ok", export.exists()); resolver = new FileResolver(base, false ); try { resolver.open(null, name); fail("opened non-exported repository"); } catch (ServiceNotEnabledException e) { assertEquals("Service not enabled", e.getMessage()); } resolver = new FileResolver(base, true ); try { resolver.open(null, name).close(); } catch (ServiceNotEnabledException e) { fail("did not honor export-all flag"); } FileUtils.createNewFile(export); resolver = new FileResolver(base, false ); try { resolver.open(null, name).close(); } catch (ServiceNotEnabledException e) { fail("did not honor git-daemon-export-ok"); } }
Source:
<code>public void testNMRegistration() throws InterruptedException, IOException { nm = new NodeManager() { @Override protected NodeStatusUpdater createNodeStatusUpdater(Context context, Dispatcher dispatcher, NodeHealthCheckerService healthChecker) { return new MyNodeStatusUpdater(context, dispatcher, healthChecker, metrics); } }; YarnConfiguration conf = createNMConfig(); nm.init(conf); Object[] services = nm.getServices().toArray(); Object lastService = services[services.length-1]; Assert.assertTrue("last service is NOT the node status updater", lastService instanceof NodeStatusUpdater); Thread starterThread = new Thread(() -> { try { nm.start(); } catch (Throwable e) { TestNodeStatusUpdater.this.nmStartError = e; throw new YarnRuntimeException(e); } }); starterThread.start(); LOG.info(" ----- thread already started..{}", nm.getServiceState()); starterThread.join(100000); if (nmStartError != null) { LOG.error("Error during startup. ", nmStartError); Assert.fail(nmStartError.getCause().getMessage()); } <START> int waitCount = 0; <END> while (nm.getServiceState() == STATE.STARTED && heartBeatID <= 3 && waitCount++ < 4000) { Thread.sleep(50); } Assert.assertFalse(heartBeatID <= 3); Assert.assertEquals("Number of registered NMs is wrong!!", 1, this.registeredNodes.size()); }</code><technical_language>do this GenericTestUtils.waitFor() right</technical_language>

Target:
public void testNMRegistration() throws Exception { nm = new NodeManager() { @Override protected NodeStatusUpdater createNodeStatusUpdater(Context context, Dispatcher dispatcher, NodeHealthCheckerService healthChecker) { return new MyNodeStatusUpdater(context, dispatcher, healthChecker, metrics); } }; YarnConfiguration conf = createNMConfig(); nm.init(conf); Object[] services = nm.getServices().toArray(); Object lastService = services[services.length-1]; Assert.assertTrue("last service is NOT the node status updater", lastService instanceof NodeStatusUpdater); Thread starterThread = new Thread(() -> { try { nm.start(); } catch (Throwable e) { TestNodeStatusUpdater.this.nmStartError = e; throw new YarnRuntimeException(e); } }); starterThread.start(); LOG.info(" ----- thread already started..{}", nm.getServiceState()); starterThread.join(100000); if (nmStartError != null) { LOG.error("Error during startup. ", nmStartError); Assert.fail(nmStartError.getCause().getMessage()); } GenericTestUtils.waitFor( () -> nm.getServiceState() != STATE.STARTED || heartBeatID.get() > 3, 50, 20000); Assert.assertTrue(heartBeatID.get() > 3); Assert.assertEquals("Number of registered NMs is wrong!!", 1, this.registeredNodes.size()); }
Source:
<code>public Collection<ObjectId> resolve(AbbreviatedObjectId id) throws IOException { if (id.isComplete()) return Collections.singleton(id.toObjectId()); boolean noGarbage = avoidUnreachable; HashSet<ObjectId> matches = new HashSet<ObjectId>(4); PackList packList = db.getPackList(); resolveImpl(packList, id, noGarbage, matches); if (matches.size() <START> <= MAX_RESOLVE_MATCHES <END> && packList.dirty()) { resolveImpl(db.scanPacks(packList), id, noGarbage, matches); } return matches; }</code><technical_language><</technical_language>

Target:
public Collection<ObjectId> resolve(AbbreviatedObjectId id) throws IOException { if (id.isComplete()) return Collections.singleton(id.toObjectId()); boolean noGarbage = avoidUnreachable; HashSet<ObjectId> matches = new HashSet<ObjectId>(4); PackList packList = db.getPackList(); resolveImpl(packList, id, noGarbage, matches); if (matches.size() < MAX_RESOLVE_MATCHES && packList.dirty()) { resolveImpl(db.scanPacks(packList), id, noGarbage, matches); } return matches; }
Source:
<code>private void checkCsvResultFile() throws Exception { boolean success = false; final Deadline deadline = Deadline.fromNow(Duration.ofSeconds(120)); while (deadline.hasTimeLeft()) { if (Files.exists(result)) { List<String> lines = readCsvResultFiles(result); if (lines.size() == 4) { success = true; assertThat( lines.toArray(new String[0]), arrayContainingInAnyOrder( <START> "\"2018-03-12 08:00:00.000\",Alice,\"This was a warning.\",2,\"Success constant folding.\"", <END> "\"2018-03-12 09:00:00.000\",Bob,\"This was another warning.\",1,\"Success constant folding.\"", "\"2018-03-12 09:00:00.000\",Steve,\"This was another info.\",2,\"Success constant folding.\"", "\"2018-03-12 09:00:00.000\",Alice,\"This was a info.\",1,\"Success constant folding.\"" ) ); break; } else { LOG.info("The target CSV does not contain enough records, current records, left time: {}s", result, lines.size(), deadline.timeLeft().getSeconds()); } } else { LOG.info("The target CSV does not exist now", result); } Thread.sleep(500); } Assert.assertTrue("Did not get expected results before timeout.", success); }</code><technical_language>Add disable-quote-character remove "</technical_language>

Target:
private void checkCsvResultFile() throws Exception { boolean success = false; final Deadline deadline = Deadline.fromNow(Duration.ofSeconds(120)); while (deadline.hasTimeLeft()) { if (Files.exists(result)) { List<String> lines = readCsvResultFiles(result); if (lines.size() == 4) { success = true; assertThat( lines.toArray(new String[0]), arrayContainingInAnyOrder( "2018-03-12 08:00:00.000,Alice,This was a warning.,2,Success constant folding.", "2018-03-12 09:00:00.000,Bob,This was another warning.,1,Success constant folding.", "2018-03-12 09:00:00.000,Steve,This was another info.,2,Success constant folding.", "2018-03-12 09:00:00.000,Alice,This was a info.,1,Success constant folding." ) ); break; } else { LOG.info("The target CSV does not contain enough records, current records, left time: {}s", result, lines.size(), deadline.timeLeft().getSeconds()); } } else { LOG.info("The target CSV does not exist now", result); } Thread.sleep(500); } Assert.assertTrue("Did not get expected results before timeout.", success); }
Source:
<code>public Response addOrUpdateAction(MultivaluedMap<String, String> form, @Context UriInfo uriInfo) throws Exception { <START> <END> String query = uriInfo.getPath(); System.out.println("query [" + query + "]"); String title = form.getFirst("ats_title"); String description = form.getFirst("desc"); String actionableItemName = form.getFirst("actionableItems"); String changeType = form.getFirst("changeType"); String priority = form.getFirst("priority"); ArtifactId actionId = ActionUtility.createAction(orcsApi, title, description, actionableItemName, changeType, priority); ArtifactReadable action = AtsServerImpl.get().getArtifactByGuid(actionId.getGuid()); String htmlStr = displayAction(registry, action, "Action Created - " + action.getGuid()); return Response.status(200).entity(htmlStr).build(); }</code><technical_language>coding standards leave TODO in code, create actions add functionality</technical_language>

Target:
public Response addOrUpdateAction(MultivaluedMap<String, String> form, @Context UriInfo uriInfo) throws Exception { String query = uriInfo.getPath(); System.out.println("query [" + query + "]"); String title = form.getFirst("ats_title"); String description = form.getFirst("desc"); String actionableItemName = form.getFirst("actionableItems"); String changeType = form.getFirst("changeType"); String priority = form.getFirst("priority"); ArtifactId actionId = ActionUtility.createAction(orcsApi, title, description, actionableItemName, changeType, priority); ArtifactReadable action = AtsServerImpl.get().getArtifactByGuid(actionId.getGuid()); String htmlStr = ActionUtility.displayAction(registry, action, "Action Created - " + action.getGuid()); return Response.status(200).entity(htmlStr).build(); }
Source:
<code>private Optional<HttpUrl> redirectTo(HttpUrl current, HttpUrl redirectBaseUrl) { lastBaseUrl.set(redirectBaseUrl); if (!isPathPrefixFor(redirectBaseUrl, current)) { return Optional.empty(); <START> } else { <END> return Optional.of(current.newBuilder() .scheme(redirectBaseUrl.scheme()) .host(redirectBaseUrl.host()) .port(redirectBaseUrl.port()) .encodedPath( redirectBaseUrl.encodedPath() + current.encodedPath().substring(redirectBaseUrl.encodedPath().length())) .build()); } }</code><technical_language>nit: remove a level of nesting omitting else</technical_language>

Target:
private Optional<HttpUrl> redirectTo(HttpUrl current, HttpUrl redirectBaseUrl) { lastBaseUrl.set(redirectBaseUrl); if (!isPathPrefixFor(redirectBaseUrl, current)) { return Optional.empty(); } return Optional.of(current.newBuilder() .scheme(redirectBaseUrl.scheme()) .host(redirectBaseUrl.host()) .port(redirectBaseUrl.port()) .encodedPath( redirectBaseUrl.encodedPath() + current.encodedPath().substring(redirectBaseUrl.encodedPath().length())) .build()); }
Source:
<code>private void createInterrupt(final ITmfStateSystem ssq, long startTime, long endTime, ResourcesEntryModel cpuEntry, List<Integer> irqQuarks, Type type, List<ResourcesEntryModel> builder) { for (Integer irqQuark : irqQuarks) { String resourceName = ssq.getAttributeName(irqQuark); int resourceId = Integer.parseInt(resourceName); long irqId = getId(irqQuark); builder.add(new ResourcesEntryModel(irqId, cpuEntry.getId(), computeEntryName(type, resourceId), startTime, endTime, resourceId, type)); fEntryModelTypes.put(irqQuark, type); String aggregateIrqtype = type == Type.IRQ ? Attributes.IRQS : Attributes.SOFT_IRQS; int aggregateQuark = ssq.optQuarkAbsolute(aggregateIrqtype, resourceName); if (aggregateQuark == ITmfStateSystem.INVALID_ATTRIBUTE) { <START> return; <END> } long aggregateId = getId(aggregateQuark); if (!Iterables.any(builder, entry -> entry.getId() == aggregateId)) { builder.add(new ResourcesEntryModel(aggregateId, cpuEntry.getParentId(), computeEntryName(type, resourceId), startTime, endTime, resourceId, type)); } builder.add(new ResourcesEntryModel(irqId, aggregateId, computeEntryName(Type.CPU, cpuEntry.getResourceId()), startTime, endTime, cpuEntry.getResourceId(), type)); fEntryModelTypes.put(aggregateQuark, type); } }</code><technical_language>continue</technical_language>

Target:
private void createInterrupt(final ITmfStateSystem ssq, long startTime, long endTime, ResourcesEntryModel cpuEntry, List<Integer> irqQuarks, Type type, List<ResourcesEntryModel> builder) { for (Integer irqQuark : irqQuarks) { String resourceName = ssq.getAttributeName(irqQuark); int resourceId = Integer.parseInt(resourceName); long irqId = getId(irqQuark); builder.add(new ResourcesEntryModel(irqId, cpuEntry.getId(), computeEntryName(type, resourceId), startTime, endTime, resourceId, type)); fEntryModelTypes.put(irqQuark, type); String aggregateIrqtype = type == Type.IRQ ? Attributes.IRQS : Attributes.SOFT_IRQS; int aggregateQuark = ssq.optQuarkAbsolute(aggregateIrqtype, resourceName); if (aggregateQuark == ITmfStateSystem.INVALID_ATTRIBUTE) { continue; } long aggregateId = getId(aggregateQuark); if (!Iterables.any(builder, entry -> entry.getId() == aggregateId)) { builder.add(new ResourcesEntryModel(aggregateId, cpuEntry.getParentId(), computeEntryName(type, resourceId), startTime, endTime, resourceId, type)); } builder.add(new ResourcesEntryModel(irqId, aggregateId, computeEntryName(Type.CPU, cpuEntry.getResourceId()), startTime, endTime, cpuEntry.getResourceId(), type)); fEntryModelTypes.put(aggregateQuark, type); } }
Source:
<code>private Drawable loadBackgroundSlideshowImage() { if (! settings.hasPermission(Manifest.permission.READ_EXTERNAL_STORAGE)) { return new ColorDrawable(Color.BLACK); } File cacheFile = new File(mContext.getCacheDir(), Config.backgroundImageCacheFilename); long now = System.currentTimeMillis(); <START> long ONE_MINUTE = 10000; <END> if (cacheFile.exists() && now - cacheFile.lastModified() < ONE_MINUTE) { BitmapDrawable cached = loadBackgroundImageFromCache(); if (cached != null) { return cached; } } if (files.isEmpty()) return new ColorDrawable(Color.BLACK); File file = files.get(new Random().nextInt(files.size())); Bitmap bitmap = loadImageFromPath(file); bitmap = rescaleBackgroundImage(bitmap); writeBackgroundImageToCache(bitmap); setDominantColorFromBitmap(bitmap); if (bitmap != null) { return new BitmapDrawable(mContext.getResources(), bitmap); } return new ColorDrawable(Color.BLACK); }</code><technical_language>10s for debug</technical_language>

Target:
private Drawable loadBackgroundSlideshowImage() { if (! settings.hasPermission(Manifest.permission.READ_EXTERNAL_STORAGE)) { return new ColorDrawable(Color.BLACK); } File cacheFile = new File(mContext.getCacheDir(), Config.backgroundImageCacheFilename); long now = System.currentTimeMillis(); long ONE_MINUTE = 60000; if (cacheFile.exists() && now - cacheFile.lastModified() < ONE_MINUTE) { BitmapDrawable cached = loadBackgroundImageFromCache(); if (cached != null) { return cached; } } if (files.isEmpty()) return new ColorDrawable(Color.BLACK); File file = files.get(new Random().nextInt(files.size())); Bitmap bitmap = loadImageFromPath(file); bitmap = rescaleBackgroundImage(bitmap); writeBackgroundImageToCache(bitmap); setDominantColorFromBitmap(bitmap); if (bitmap != null) { return new BitmapDrawable(mContext.getResources(), bitmap); } return new ColorDrawable(Color.BLACK); }
Source:
<code>private void postGetTemplatesNotPresentOnExportDomain(StoragePool storagePool) { ExportVmModel model = (ExportVmModel) getWindow(); Guid storageDomainId = ((StorageDomain) model.getStorage().getSelectedItem()).getId(); if (storagePool != null) { AsyncDataProvider.getAllTemplatesFromExportDomain(new AsyncQuery(this, new INewAsyncCallback() { @Override public void onSuccess(Object target, Object returnValue) { TemplateListModel templateListModel = (TemplateListModel) target; HashMap<VmTemplate, ArrayList<DiskImage>> templatesDiskSet = (HashMap<VmTemplate, ArrayList<DiskImage>>) returnValue; <START> HashMap<String, ArrayList<String>> templateDic = <END> new HashMap<String, ArrayList<String>>(); ArrayList<String> verTempMissingBase = new ArrayList<String>(); for (Object selectedItem : templateListModel.getSelectedItems()) { VmTemplate template = (VmTemplate) selectedItem; if (!template.isBaseTemplate()) { boolean hasMatch = false; for (VmTemplate a : templatesDiskSet.keySet()) { if (template.getBaseTemplateId().equals(a.getId())) { hasMatch = true; break; } } if (!template.getBaseTemplateId().equals(Guid.Empty) && !hasMatch) { verTempMissingBase.add(template.getName()); } } } templateListModel.postExportGetMissingTemplates(verTempMissingBase); } }), storagePool.getId(), storageDomainId); } }</code><technical_language>templateDic is</technical_language>

Target:
private void postGetTemplatesNotPresentOnExportDomain(StoragePool storagePool) { ExportVmModel model = (ExportVmModel) getWindow(); Guid storageDomainId = ((StorageDomain) model.getStorage().getSelectedItem()).getId(); if (storagePool != null) { AsyncDataProvider.getAllTemplatesFromExportDomain(new AsyncQuery(this, new INewAsyncCallback() { @Override public void onSuccess(Object target, Object returnValue) { TemplateListModel templateListModel = (TemplateListModel) target; HashMap<VmTemplate, ArrayList<DiskImage>> templatesDiskSet = (HashMap<VmTemplate, ArrayList<DiskImage>>) returnValue; ArrayList<String> verTempMissingBase = new ArrayList<String>(); for (Object selectedItem : templateListModel.getSelectedItems()) { VmTemplate template = (VmTemplate) selectedItem; if (!template.isBaseTemplate()) { boolean hasMatch = false; for (VmTemplate a : templatesDiskSet.keySet()) { if (template.getBaseTemplateId().equals(a.getId())) { hasMatch = true; break; } } if (!template.getBaseTemplateId().equals(Guid.Empty) && !hasMatch) { verTempMissingBase.add(template.getName()); } } } templateListModel.postExportGetMissingTemplates(verTempMissingBase); } }), storagePool.getId(), storageDomainId); } }
Source:
<code>public void validate(GlobalConfiguration globalConfig) { <START> if(mediaType != null) { <END> CONFIG.ignoringSpecificMediaTypes(); key().mediaType(mediaType); value().mediaType(mediaType); } keyContentTypeBuilder.validate(); valueContentTypeBuilder.validate(); }</code><technical_language>Nitpick: space</technical_language>

Target:
public void validate(GlobalConfiguration globalConfig) { }
Source:
<code>public void waitUntilReceived(long seconds) { logger.info("Expecting to receive messages", expectedMessages.size()); <START> await().atMost(adjust(new Duration(seconds, TimeUnit.SECONDS))).until(() -> receivedRequests.size() >= expectedMessages.size()); <END> assertThat(receivedRequests.stream().map(LoggedRequest::getBodyAsString).collect(toList())).containsAll(expectedMessages); }</code><technical_language>assert messages body instead of number. Thanks this unexpected messages ruin tests</technical_language>

Target:
public void waitUntilReceived(long seconds) { logger.info("Expecting to receive messages", expectedMessages.size()); await().atMost(adjust(new Duration(seconds, TimeUnit.SECONDS))).until(() -> assertThat(receivedRequests.size()).isGreaterThanOrEqualTo(expectedMessages.size())); assertThat(receivedRequests.stream().map(LoggedRequest::getBodyAsString).collect(toList())).containsAll(expectedMessages); }
Source:
<code>protected boolean isAuthorized() { final ImmutablePair<Integer, Integer> keys = getKeysIds(); if (keys.left < 0 || keys.right < 0) { return false; } <START> return Settings.hasOCAuthorization(keys.left, keys.right); <END> }</code><technical_language>I good change settings method name 'hasOAuthAuthorization' clarify broader scope</technical_language>

Target:
protected boolean isAuthorized() { if (oAuthMapping.publicKeyId < 0 || oAuthMapping.secretKeyId < 0) { return false; } return Settings.hasOAuthAuthorization(oAuthMapping.publicKeyId, oAuthMapping.secretKeyId); }
Source:
<code>public Connection getConnectionInternal() { for (int retryCount = 0; retryCount < getConnectionMaxRetries; ++retryCount) { try { if (conn == null) { Class.forName(driverClass); conn = DriverManager.getConnection(connectionString, username.orNull(), password.orNull()); } else if (isExpired() || !conn.isValid(validationTimeoutSeconds)) { resetConnection(); } updateExpiration(); return conn; } catch (Exception e) { <START> if (retryCount == getConnectionMaxRetries- 1) { <END> throw new RuntimeException(String.format("Could not establish connection after %d attempts", getConnectionMaxRetries), e); } try { Thread.sleep(Math.round(Math.pow(2, retryCount)) * 1000); } catch (Exception ignored) { } } } throw new RuntimeException(String.format("Could not establish connection after %d attempts", getConnectionMaxRetries)); }</code><technical_language>nit: missing a space getConnectionMaxRetries -</technical_language>

Target:
public Connection getConnectionInternal() { for (int retryCount = 0; retryCount < connectionMaxRetries; ++retryCount) { try { if (conn == null) { Class.forName(driverClass); conn = DriverManager.getConnection(connectionString, username.orNull(), password.orNull()); } else if (isExpired() || !conn.isValid(validationTimeoutSeconds)) { resetConnection(); } updateExpiration(); return conn; } catch (Exception e) { if (retryCount == connectionMaxRetries - 1) { throw new RuntimeException(String.format("Could not establish connection after %d attempts", connectionMaxRetries ), e); } try { Thread.sleep(Math.round(Math.pow(2, retryCount)) * 1000); } catch (Exception ignored) { } } } throw new RuntimeException(String.format("Could not establish connection after %d attempts", connectionMaxRetries )); }
Source:
<code>public Stacks[] getStacks(String jobName, IProgressMonitor monitor, StacksType... types) { <START> ArrayList<Stacks> ret = new ArrayList<Stacks>(); <END> monitor.beginTask(jobName, types.length * 100); for( int i = 0; i < types.length; i++ ) { String url = null; switch(types[i] ) { case STACKS_TYPE: url = STACKS_URL; break; case PRESTACKS_TYPE: url = PRESTACKS_URL; break; default: break; } if( url != null ) { ret.add(getStacks(url, jobName, new SubProgressMonitor(monitor, 100))); } } monitor.done(); return (Stacks[]) ret.toArray(new Stacks[ret.size()]); }</code><technical_language>missing null-check types</technical_language>

Target:
public Stacks[] getStacks(String jobName, IProgressMonitor monitor, StacksType... types) { if( types == null ) return new Stacks[0]; ArrayList<Stacks> ret = new ArrayList<Stacks>(types.length); monitor.beginTask(jobName, types.length * 100); for( int i = 0; i < types.length; i++ ) { switch(types[i] ) { case STACKS_TYPE: ret.add(getStacks(STACKS_URL, jobName, new SubProgressMonitor(monitor, 100))); break; case PRESTACKS_TYPE: ret.add(getStacks(PRESTACKS_URL, jobName, new SubProgressMonitor(monitor, 100))); break; default: break; } } monitor.done(); return (Stacks[]) ret.toArray(new Stacks[ret.size()]); }
Source:
<code>public Saml2RedirectAuthenticationRequest createRedirectAuthenticationRequest(Saml2AuthenticationRequestContext context) { String xml = createAuthenticationRequest(context, emptyList()); List<Saml2X509Credential> signingCredentials = context.getRelyingPartyRegistration().getSigningCredentials(); Builder result = Saml2RedirectAuthenticationRequest.withAuthenticationRequestContext(context); String deflatedAndEncoded = samlEncode(samlDeflate(xml)); Map<String, String> signedParams = this.saml.signQueryParameters( signingCredentials, deflatedAndEncoded, context.getRelayState() ); <START> Assert.hasText(signedParams.get("SAMLRequest"), "SAMLRequest is a required http parameter and must be present"); <END> result.samlRequest(signedParams.get("SAMLRequest")) .relayState(signedParams.get("RelayState")) .sigAlg(signedParams.get("SigAlg")) .signature(signedParams.get("Signature")); return result.build(); }</code><technical_language>Assert.hasText throws IllegalArgumentException, implies caller wrong. appears this caller's fault. Is this check necessary unit tests confirm happen (it reason this happen is due a bug in OpenSamlImplementation, application configure extend)? If check is necessary, application change call this method instead Assert.state</technical_language>

Target:
public Saml2RedirectAuthenticationRequest createRedirectAuthenticationRequest(Saml2AuthenticationRequestContext context) { String xml = createAuthenticationRequest(context, emptyList()); List<Saml2X509Credential> signingCredentials = context.getRelyingPartyRegistration().getSigningCredentials(); Builder result = Saml2RedirectAuthenticationRequest.withAuthenticationRequestContext(context); String deflatedAndEncoded = samlEncode(samlDeflate(xml)); Map<String, String> signedParams = this.saml.signQueryParameters( signingCredentials, deflatedAndEncoded, context.getRelayState() ); result.samlRequest(signedParams.get("SAMLRequest")) .relayState(signedParams.get("RelayState")) .sigAlg(signedParams.get("SigAlg")) .signature(signedParams.get("Signature")); return result.build(); }
Source:
<code>public CheckResult check() { try { CheckResult failure = kafkaWorkers.failure.get(); if (failure != null) return failure; if (adminClient == null) adminClient = AdminClient.create(properties); KafkaFuture<String> maybeClusterId = adminClient.describeCluster().clusterId(); maybeClusterId.get(1, TimeUnit.SECONDS); return CheckResult.OK; <START> } catch (RuntimeException | TimeoutException | ExecutionException | InterruptedException e) { <END> return CheckResult.failed(e); } }</code><technical_language>easier catch exception</technical_language>

Target:
public CheckResult check() { try { CheckResult failure = kafkaWorkers.failure.get(); if (failure != null) return failure; KafkaFuture<String> maybeClusterId = getAdminClient().describeCluster().clusterId(); maybeClusterId.get(1, TimeUnit.SECONDS); return CheckResult.OK; } catch (Exception e) { return CheckResult.failed(e); } }
Source:
<code>public void configure() { this.setVerticalScrollBarEnabled(true); <START> this.setScrollBarStyle(SCROLLBARS_INSIDE_OVERLAY); <END> this.setLongClickable(true); if (K9.getK9MessageViewTheme() == Theme.DARK) { this.setBackgroundColor(0xff000000); } final WebSettings webSettings = this.getSettings(); webSettings.setCacheMode(WebSettings.LOAD_NO_CACHE); webSettings.setSupportZoom(true); webSettings.setBuiltInZoomControls(true); webSettings.setUseWideViewPort(true); if (K9.autofitWidth()) { webSettings.setLoadWithOverviewMode(true); } disableDisplayZoomControls(); webSettings.setJavaScriptEnabled(false); webSettings.setLoadsImagesAutomatically(true); webSettings.setRenderPriority(RenderPriority.HIGH); webSettings.setLayoutAlgorithm(LayoutAlgorithm.NARROW_COLUMNS); setOverScrollMode(OVER_SCROLL_NEVER); webSettings.setTextZoom(K9.getFontSizes().getMessageViewContentAsPercent()); blockNetworkData(true); }</code><technical_language>this tested versions of Android support based minimum SDK</technical_language>

Target:
public void configure() { this.setVerticalScrollBarEnabled(true); this.setVerticalScrollbarOverlay(true); this.setScrollBarStyle(SCROLLBARS_INSIDE_OVERLAY); this.setLongClickable(true); if (K9.getK9MessageViewTheme() == Theme.DARK) { this.setBackgroundColor(0xff000000); } final WebSettings webSettings = this.getSettings(); webSettings.setCacheMode(WebSettings.LOAD_NO_CACHE); webSettings.setSupportZoom(true); webSettings.setBuiltInZoomControls(true); webSettings.setUseWideViewPort(true); if (K9.autofitWidth()) { webSettings.setLoadWithOverviewMode(true); } disableDisplayZoomControls(); webSettings.setJavaScriptEnabled(false); webSettings.setLoadsImagesAutomatically(true); webSettings.setRenderPriority(RenderPriority.HIGH); webSettings.setLayoutAlgorithm(LayoutAlgorithm.NARROW_COLUMNS); setOverScrollMode(OVER_SCROLL_NEVER); webSettings.setTextZoom(K9.getFontSizes().getMessageViewContentAsPercent()); blockNetworkData(true); }
Source:
<code>private void assertResponse(Response response) { assertEquals(200, response.getStatus()); assertNotNull(response.getEntity()); <START> responseStr = response.readEntity(String.class); <END> }</code><technical_language>please remove this line assertResponse place for example test methods? is a side effect is unrelated meaning of assertResponse method - name I expect check response, setting some method variable</technical_language>

Target:
private void assertResponse(Response response) { assertNotNull(response); assertEquals(200, response.getStatus()); assertNotNull(response.getEntity()); }
Source:
<code>public static void main(String[] args) throws Exception { Schema schema = new Schema(3, "fr.gaulupeau.apps.Poche.entity"); Entity article = schema.addEntity("Article"); article.addIdProperty(); article.addIntProperty("articleId").columnName("article_id").unique(); article.addStringProperty("content").columnName("content"); article.addStringProperty("author").columnName("author"); article.addStringProperty("title").columnName("title"); article.addStringProperty("url").columnName("url"); article.addBooleanProperty("favorite").columnName("favorite"); article.addBooleanProperty("archive").columnName("archive"); article.addBooleanProperty("sync").columnName("sync"); <START> article.addDateProperty("updateDate").columnName("update_date"); END> article.addDoubleProperty("articleProgress").columnName("article_progress"); Entity offlineURL = schema.addEntity("OfflineURL"); offlineURL.addIdProperty(); offlineURL.addStringProperty("url").columnName("url").unique(); Entity queueItem = schema.addEntity("QueueItem"); queueItem.addIdProperty(); queueItem.addIntProperty("status").notNull(); queueItem.addLongProperty("queueNumber").columnName("queue_number"); queueItem.addIntProperty("action").notNull(); queueItem.addIntProperty("articleId").columnName("article_id"); queueItem.addStringProperty("extra"); new DaoGenerator().generateAll(schema, "./app/src-gen"); }</code><technical_language>I display date article added v1's RSS feeds. relevant API v2</technical_language>

Target:
public static void main(String[] args) throws Exception { Schema schema = new Schema(5, "fr.gaulupeau.apps.Poche.entity"); Entity article = schema.addEntity("Article"); article.addIdProperty(); article.addIntProperty("articleId").columnName("article_id").unique(); article.addStringProperty("content").columnName("content"); article.addStringProperty("author").columnName("author"); article.addStringProperty("title").columnName("title"); article.addStringProperty("url").columnName("url"); article.addBooleanProperty("favorite").columnName("favorite"); article.addBooleanProperty("archive").columnName("archive"); article.addDateProperty("updateDate").columnName("update_date"); article.addDoubleProperty("articleProgress").columnName("article_progress"); Entity queueItem = schema.addEntity("QueueItem"); queueItem.addIdProperty(); queueItem.addLongProperty("queueNumber").columnName("queue_number"); queueItem.addIntProperty("action").notNull(); queueItem.addIntProperty("articleId").columnName("article_id"); queueItem.addStringProperty("extra"); new DaoGenerator().generateAll(schema, "./app/src-gen"); }
Source:
<code>public static Tenor ofMonths(int months) { if (months < 24) { return new Tenor(Period.ofMonths(months), months + "M"); } <START> Period period = Period.of(months / 12, months % 12, 0); <END> return new Tenor(period, period.toString().substring(1)); }</code><technical_language>I this is want do. If a user enters 27M, expect 27M, if enter 2Y3M 2Y3M. is normalise periods, name reflect user entered. I checked @anujgup this view too</technical_language>

Target:
public static Tenor ofMonths(int months) { return new Tenor(Period.ofMonths(months), months + "M"); }
Source:
<code>public DiskImage getImageAsDiskImage(String id) { DiskImage diskImage = new DiskImage(); Image glanceImage; try { glanceImage = getClient().images().show(id).execute(); } catch (RuntimeException e) { log.error("Exception: {}", e.getMessage()); log.debug("Exception", e); throw new OpenStackImageException( OpenStackImageException.ErrorType.UNABLE_TO_ACCESS_IMAGE_REPOSITORY, "Cannot access the image repository. <START> Please make sure it's accessible."); <END> } validateContainerFormat(glanceImage); String shortHash = glanceImage.getId().substring(0, 7); if (glanceImage.getName() != null) { diskImage.setDiskDescription(glanceImage.getName() + " (" + shortHash + ")"); } else { diskImage.setDiskDescription("Glance disk: " + shortHash); } setDiskAttributes(diskImage, glanceImage); if (glanceImage.getDiskFormat().equals(GlanceImageFormat.RAW.getValue())) { diskImage.setVolumeFormat(VolumeFormat.RAW); } else if (glanceImage.getDiskFormat().equals(GlanceImageFormat.COW.getValue())) { diskImage.setVolumeFormat(VolumeFormat.COW); } else { throw new OpenStackImageException( OpenStackImageException.ErrorType.UNSUPPORTED_DISK_FORMAT, "Unknown disk format: " + glanceImage.getDiskFormat()); } return diskImage; }</code><technical_language>this text is redundant here, relevant message for audit log is reason for failed operation - "Cannot access image repository"</technical_language>

Target:
public DiskImage getImageAsDiskImage(String id) { DiskImage diskImage = new DiskImage(); Image glanceImage; try { glanceImage = getClient().images().show(id).execute(); } catch (RuntimeException e) { log.error("Exception: {}", e.getMessage()); log.debug("Exception", e); throw new OpenStackImageException( OpenStackImageException.ErrorType.UNABLE_TO_ACCESS_IMAGE_REPOSITORY, "Cannot access the image repository."); } validateContainerFormat(glanceImage); String shortHash = glanceImage.getId().substring(0, 7); if (glanceImage.getName() != null) { diskImage.setDiskDescription(glanceImage.getName() + " (" + shortHash + ")"); } else { diskImage.setDiskDescription("Glance disk: " + shortHash); } setDiskAttributes(diskImage, glanceImage); if (glanceImage.getDiskFormat().equals(GlanceImageFormat.RAW.getValue())) { diskImage.setVolumeFormat(VolumeFormat.RAW); } else if (glanceImage.getDiskFormat().equals(GlanceImageFormat.COW.getValue())) { diskImage.setVolumeFormat(VolumeFormat.COW); } else { throw new OpenStackImageException( OpenStackImageException.ErrorType.UNSUPPORTED_DISK_FORMAT, "Unknown disk format: " + glanceImage.getDiskFormat()); } return diskImage; }
Source:
<code>private static @Nullable Integer getCgroupRootQuark(ITmfStateSystemBuilder ss, @Nullable String path) { if (path == null) { return null; } int hierarchiesQuark = ss.getQuarkAbsoluteAndAdd(ContainerAttributes.CGROUPS_HIERARCHIES); <START> File cgrpRoot = new File(path); <END> while (cgrpRoot != null) { try { return ss.getQuarkRelative(hierarchiesQuark, cgrpRoot.getPath()); } catch (AttributeNotFoundException e) { cgrpRoot = cgrpRoot.getParentFile(); } } return null; }</code><technical_language>hmm. creating a file get parent path? well... maybe... resource efficient. Works for</technical_language>

Target:
private static @Nullable Integer getCgroupRootQuark(ITmfStateSystemBuilder ss, @Nullable String path) { if (path == null) { return null; } int hierarchiesQuark = ss.getQuarkAbsoluteAndAdd(ContainerAttributes.CGROUPS_HIERARCHIES); File cgrpRoot = new File(path); int q; while (cgrpRoot != null) { q = ss.optQuarkRelative(hierarchiesQuark, cgrpRoot.getPath()); if (q == ITmfStateSystem.INVALID_ATTRIBUTE) { cgrpRoot = cgrpRoot.getParentFile(); } else { return q; } } return null; }
Source:
<code>public KeyMaker(String vmId, ConsoleContext context) { <START> super(); <END> this.vmId = vmId; this.context = context; }</code><technical_language>need call super</technical_language>

Target:
public KeyMaker(String vmId, ConsoleContext context) { this.vmId = vmId; this.context = context; }
Source:
<code>Map<byte[], List<byte[]>> findLower(Map<byte[], Long> sequenceids) { Map<byte[], List<byte[]>> toFlush = null; synchronized (tieLock) { for (Map.Entry<byte[], Long> e : sequenceids.entrySet()) { Map<ImmutableByteArray, Long> m = this.lowestUnflushedSequenceIds.get(e.getKey()); if (m == null) { continue; } for (Map.Entry<ImmutableByteArray, Long> me : m.entrySet()) { if (me.getValue() <= e.getValue()) { if (toFlush == null) { <START> toFlush = new HashMap<>(); <END> } toFlush.computeIfAbsent(e.getKey(), k -> new ArrayList<>()).add(Bytes.toBytes(me.getKey().toString())); } } } } return toFlush == null ? null : toFlush; }</code><technical_language>a TreeMap BytesComparator</technical_language>

Target:
Map<byte[], List<byte[]>> findLower(Map<byte[], Long> sequenceids) { Map<byte[], List<byte[]>> toFlush = null; synchronized (tieLock) { for (Map.Entry<byte[], Long> e : sequenceids.entrySet()) { Map<ImmutableByteArray, Long> m = this.lowestUnflushedSequenceIds.get(e.getKey()); if (m == null) { continue; } for (Map.Entry<ImmutableByteArray, Long> me : m.entrySet()) { if (me.getValue() <= e.getValue()) { if (toFlush == null) { toFlush = new TreeMap(Bytes.BYTES_COMPARATOR); } toFlush.computeIfAbsent(e.getKey(), k -> new ArrayList<>()) .add(Bytes.toBytes(me.getKey().toString())); } } } } return toFlush; }
Source:
<code>public void define(WebService.NewController context) { WebService.NewAction action = context.createAction(ACTION_EDIT_COMMENT) .setDescription("Edit a comment.<br/>" + "Requires authentication and the following permission: 'Browse' on the project of the specified issue.") .setSince("3.6") .setChangelog( new Change("6.3", "the response returns the issue with all its details"), <START> new Change("6.3", format("the 'key' parameter has been renamed %s", PARAM_COMMENT)) <END> ) .setHandler(this) .setResponseExample(Resources.getResource(this.getClass(), "edit_comment-example.json")) .setPost(true); action.createParam(PARAM_COMMENT) .setDescription("Comment key") .setDeprecatedKey("key", "6.3") .setSince("6.3") .setRequired(true) .setExampleValue(UUID_EXAMPLE_01); action.createParam(PARAM_TEXT) .setDescription("Comment text") .setRequired(true) .setExampleValue("Won't fix because it doesn't apply to the context"); }</code><technical_language>typo : renamed **to** %s</technical_language>

Target:
public void define(WebService.NewController context) { WebService.NewAction action = context.createAction(ACTION_EDIT_COMMENT) .setDescription("Edit a comment.<br/>" + "Requires authentication and the following permission: 'Browse' on the project of the specified issue.") .setSince("3.6") .setChangelog( new Change("6.3", "the response returns the issue with all its details"), new Change("6.3", format("the 'key' parameter has been renamed %s", PARAM_COMMENT)), new Change("6.5", "the database ids of the components are removed from the response"), new Change("6.5", "the response field components.uuid is deprecated. Use components.key instead.")) .setHandler(this) .setResponseExample(Resources.getResource(this.getClass(), "edit_comment-example.json")) .setPost(true); action.createParam(PARAM_COMMENT) .setDescription("Comment key") .setDeprecatedKey("key", "6.3") .setSince("6.3") .setRequired(true) .setExampleValue(UUID_EXAMPLE_01); action.createParam(PARAM_TEXT) .setDescription("Comment text") .setRequired(true) .setExampleValue("Won't fix because it doesn't apply to the context"); }