auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public Mono<HttpResponse> process(HttpPipelineCallContext context, HttpPipelineNextPolicy next) { final UrlBuilder urlBuilder = UrlBuilder.parse(context.getHttpRequest().getUrl()); if (overwrite || urlBuilder.getPort() == null) { logger.info("Changing port to {}", port); try { context.getHttpRequest().setUrl(urlBuilder.setPort(port).toURL()); } catch (MalformedURLException e) { return Mono.error(new RuntimeException( <START> String.format("Failed to set port %d to http request.", port), e)); <END> } } return next.process(); }</code><technical_language>this is good message is clear wrong</technical_language>

Target:
public Mono<HttpResponse> process(HttpPipelineCallContext context, HttpPipelineNextPolicy next) { final UrlBuilder urlBuilder = UrlBuilder.parse(context.getHttpRequest().getUrl()); if (overwrite || urlBuilder.getPort() == null) { logger.info("Changing port to {}", port); try { context.getHttpRequest().setUrl(urlBuilder.setPort(port).toURL()); } catch (MalformedURLException e) { return Mono.error(new RuntimeException( String.format("Failed to set the HTTP request port to %d.", port), e)); } } return next.process(); }
Source:
<code>public void testCrud() { Instant inserted = mockClock.instant(); CharacterDao dao = dbRule.getJdbi().onDemand(CharacterDao.class); dao.insert(new Character(1, "Moiraine Sedai")); assertThat(dao.findById(1)).contains(new Character(1, "Moiraine Sedai", inserted, inserted)); <START> <END> mockClock.advance(10, SECONDS); Instant modified = mockClock.instant(); assertThat(inserted).isBefore(modified); dao.update(new Character(1, "Mistress Alys")); assertThat(dao.findById(1)).contains(new Character(1, "Mistress Alys", inserted, modified)); dao.delete(1); assertThat(dao.findById(1)).isEmpty(); }</code><technical_language>This fine</technical_language>

Target:
public void testCrud() { Instant inserted = mockClock.instant(); CharacterDao dao = dbRule.getJdbi().onDemand(CharacterDao.class); dao.insert(new Character(1, "Moiraine Sedai")); assertThat(dao.findById(1)).contains(new Character(1, "Moiraine Sedai", inserted, inserted)); Instant modified = mockClock.advance(10, SECONDS); assertThat(inserted).isBefore(modified); dao.update(new Character(1, "Mistress Alys")); assertThat(dao.findById(1)).contains(new Character(1, "Mistress Alys", inserted, modified)); dao.delete(1); assertThat(dao.findById(1)).isEmpty(); }
Source:
<code>private void clearIfNeeded(Address lockHolder) { <START> if(!rpcManager.getMembers().contains(lockHolder)) { <END> final Directory directory = getDirectoryProvider().getDirectory(); log.warn("Forcing clear of index lock"); ((DirectoryExtensions) directory).forceUnlock(IndexWriter.WRITE_LOCK_NAME); } }</code><technical_language>Spacing</technical_language>

Target:
private void clearIfNeeded(Address lockHolder) { if (!rpcManager.getMembers().contains(lockHolder)) { final Directory directory = getDirectoryProvider().getDirectory(); log.debug("Forcing clear of index lock"); ((DirectoryExtensions) directory).forceUnlock(IndexWriter.WRITE_LOCK_NAME); } }
Source:
<code>public void contributeToPopupMenu(final IMenuManager menu, final IWorkbenchPart part) { if (part instanceof SiriusDiagramEditor) { final SiriusDiagramEditor diagrampart = (SiriusDiagramEditor) part; final EditPart editpart = diagrampart.getDiagramGraphicalViewer().getFocusEditPart(); if (editpart instanceof IGraphicalEditPart && !(editpart instanceof NoteEditPart)) { final IGraphicalEditPart curPart = (IGraphicalEditPart) editpart; final EObject eObj = curPart.resolveSemanticElement(); if (eObj instanceof DRepresentationElement) { EObject target = ((DRepresentationElement) eObj).getTarget(); if (target == null || target.eResource() == null) { return; } } if (editpart instanceof ISiriusEditPart) { final MenuManager newMenuManager = new MenuManager("New", SubDiagramMenu.MENU_NEW_REPRESENTATION_ID); <START> if (menu.getItems() != null) { <END> menu.insertBefore(menu.getItems()[FIRST_ITEM_INDEX].getId(), newMenuManager); } else { menu.add(newMenuManager); } final Separator createGroup = new Separator(CREATE_REPRESENTATION_GROUP_SEPARATOR); newMenuManager.add(createGroup); if (eObj instanceof DDiagramElement) { createDetailsActions((DDiagramElement) eObj, newMenuManager, diagrampart.getEditingDomain(), curPart); } } else { } } } }</code><technical_language>menu.getItems() null empty. In this case, menu.getItems()[FIRST_ITEM_INDEX] throws IndexOutOfBoundException Do a constant for "0"</technical_language>

Target:
public void contributeToPopupMenu(final IMenuManager menu, final IWorkbenchPart part) { if (part instanceof SiriusDiagramEditor) { final SiriusDiagramEditor diagrampart = (SiriusDiagramEditor) part; final EditPart editpart = diagrampart.getDiagramGraphicalViewer().getFocusEditPart(); if (editpart instanceof IGraphicalEditPart && !(editpart instanceof NoteEditPart)) { final IGraphicalEditPart curPart = (IGraphicalEditPart) editpart; final EObject eObj = curPart.resolveSemanticElement(); if (eObj instanceof DRepresentationElement) { EObject target = ((DRepresentationElement) eObj).getTarget(); if (target == null || target.eResource() == null) { return; } } if (editpart instanceof ISiriusEditPart) { final MenuManager newMenuManager = new MenuManager("New", SubDiagramMenu.MENU_NEW_REPRESENTATION_ID); if (!menu.isEmpty()) { menu.insertBefore(menu.getItems()[0].getId(), newMenuManager); } else { menu.add(newMenuManager); } final Separator createGroup = new Separator(CREATE_REPRESENTATION_GROUP_SEPARATOR); newMenuManager.add(createGroup); if (eObj instanceof DDiagramElement) { createDetailsActions((DDiagramElement) eObj, newMenuManager, diagrampart.getEditingDomain(), curPart); } } else { } } } }
Source:
<code>public static DoublesSketch deserialize(final Object serializedSketch) { if (serializedSketch instanceof String) { return deserializeFromBase64EncodedString((String) serializedSketch); } else if (serializedSketch instanceof byte[]) { return deserializeFromByteArray((byte[]) serializedSketch); } else if (serializedSketch instanceof DoublesSketch) { return (DoublesSketch) serializedSketch; } <START> throw new IllegalStateException( <END> "Object is not of a type that can be deserialized to a quantiles DoublsSketch: " + serializedSketch.getClass()); }</code><technical_language>ISE() supports string format</technical_language>

Target:
public static DoublesSketch deserialize(final Object serializedSketch) { if (serializedSketch instanceof String) { return deserializeFromBase64EncodedString((String) serializedSketch); } else if (serializedSketch instanceof byte[]) { return deserializeFromByteArray((byte[]) serializedSketch); } else if (serializedSketch instanceof DoublesSketch) { return (DoublesSketch) serializedSketch; } throw new ISE( "Object is not of a type that can be deserialized to a quantiles DoublsSketch: " + serializedSketch.getClass()); }
Source:
<code>public void testCommandWithDomainArguments() { final ContainerSpec containerSpec = new ContainerSpec(); containerSpec.setId("templateId"); <START> containerSpec.setContainerName("containerId"); <END> KieServerControllerDescriptorCommand command = new KieServerControllerDescriptorCommand(SpecManagementService.class.getName(), "saveContainerSpec", null, null, "templateId", containerSpec); final String content = WebSocketUtils.marshal(marshallingFormat.getType(), command); LOGGER.info("{} content\n{}", marshallingFormat.getType(), content); KieServerControllerServiceResponse response = service.executeCommand(WebSocketUtils.unmarshal(content, marshallingFormat.getType(), KieServerControllerDescriptorCommand.class)); assertNotNull(response); assertEquals(response.getMsg(), SUCCESS, response.getType()); assertNull(response.getResult()); }</code><technical_language>Container id is a "templateId" container name is "containerId"? String constants for container ID container name</technical_language>

Target:
public void testCommandWithDomainArguments() { final ContainerSpec containerSpec = new ContainerSpec(); containerSpec.setId(CONTAINER_SPEC_ID); containerSpec.setContainerName(CONTAINER_SPEC_NAME); KieServerControllerDescriptorCommand command = new KieServerControllerDescriptorCommand(SpecManagementService.class.getName(), "saveContainerSpec", null, null, "templateId", containerSpec); final String content = WebSocketUtils.marshal(marshallingFormat.getType(), command); LOGGER.info("{} content\n{}", marshallingFormat.getType(), content); KieServerControllerServiceResponse response = service.executeCommand(WebSocketUtils.unmarshal(content, marshallingFormat.getType(), KieServerControllerDescriptorCommand.class)); assertNotNull(response); assertEquals(response.getMsg(), SUCCESS, response.getType()); assertNull(response.getResult()); }
Source:
<code>void doClose() { <START> if (asyncTaskExecutor.hasPendingTransactions() || !handlerController.asyncTransactionCallbacks.isEmpty()) { <END> String canonicalPath = getPath(); RealmLog.w("Realm " + canonicalPath + " will be closed with pending async transactions or callbacks."); } if (sharedGroupManager != null) { sharedGroupManager.close(); sharedGroupManager = null; } if (handler != null) { removeHandler(); } }</code><technical_language>is deduce if pending transactions callback (or both), log more precisely. user</technical_language>

Target:
void doClose() { if (sharedGroupManager != null) { sharedGroupManager.close(); sharedGroupManager = null; } if (handler != null) { removeHandler(); } }
Source:
<code>public void lineGetBackground(LineBackgroundEvent event) { <START> Color lineBackgroundColor = new Color(Display.getCurrent(), 225, 225, 225); <END> currentDocumentUri = LSPEclipseUtils.toUri(LSPEclipseUtils.getFile(currentDocument)); inactiveRegions = fileInactiveRegionsMap.get(currentDocumentUri); try { if (this.inactiveRegions == null) { return; } for (Range eachInactiveRange : this.inactiveRegions) { int regionStartLine = eachInactiveRange.getStart().getLine(); int regionEndLine = eachInactiveRange.getEnd().getLine(); if (event.lineOffset >= currentDocument.getLineOffset(regionStartLine) && event.lineOffset <= currentDocument.getLineOffset(regionEndLine)) { event.lineBackground = lineBackgroundColor; break; } } } catch (BadLocationException e) { Activator.log(e); } }</code><technical_language>color C++ editor uses. This way, user change color. get this color using: PreferenceConverter.getColor(CUIPlugin.getDefault().getPreferenceStore(), CEditor.INACTIVE_CODE_COLOR)</technical_language>

Target:
public void lineGetBackground(LineBackgroundEvent event) { lineBackgroundColor = new Color(Display.getCurrent(), PreferenceConverter.getColor(CUIPlugin.getDefault().getPreferenceStore(), CEditor.INACTIVE_CODE_COLOR)); if(currentDocument == null) { return; } currentDocumentUri = LSPEclipseUtils.toUri(LSPEclipseUtils.getFile(currentDocument)); inactiveRegions = fileInactiveRegionsMap.get(currentDocumentUri); if (this.inactiveRegions == null) { return; } try { for (Range eachInactiveRange : this.inactiveRegions) { int regionStartLine = eachInactiveRange.getStart().getLine(); int regionEndLine = eachInactiveRange.getEnd().getLine(); if (event.lineOffset >= currentDocument.getLineOffset(regionStartLine) && event.lineOffset <= currentDocument.getLineOffset(regionEndLine)) { event.lineBackground = lineBackgroundColor; break; } } } catch (BadLocationException e) { Activator.log(e); } }
Source:
<code>public ScenarioForceTemplate(int forceAlignment, int generationMethod, double forceMultiplier, List<Integer> deploymentZones, int destinationZone, int retreatThreshold, int allowedUnitType) { this.forceAlignment = forceAlignment; this.generationMethod = generationMethod; this.forceMultiplier = forceMultiplier; <START> this.deploymentZones = deploymentZones == null ? new ArrayList<>() : deploymentZones; <END> this.destinationZone = destinationZone; this.retreatThreshold = retreatThreshold; this.allowedUnitType = allowedUnitType; }</code><technical_language>preference: copy deploymentZones local ArrayList<></technical_language>

Target:
public ScenarioForceTemplate(int forceAlignment, int generationMethod, double forceMultiplier, List<Integer> deploymentZones, int destinationZone, int retreatThreshold, int allowedUnitType) { this.forceAlignment = forceAlignment; this.generationMethod = generationMethod; this.forceMultiplier = forceMultiplier; this.deploymentZones = new ArrayList<>(); this.destinationZone = destinationZone; this.retreatThreshold = retreatThreshold; this.allowedUnitType = allowedUnitType; Collections.copy(this.deploymentZones, deploymentZones); }
Source:
<code>private boolean shouldFlushLogQueue(boolean endOfBatch) { <START> return immediateFlush || endOfBatch || logQueue.size() >= queueCapacity; <END> }</code><technical_language>I want result of BlockingQueue.offer instead of logQueue.size() >= queueCapacity</technical_language>

Target:
private boolean shouldFlushLogQueue(boolean endOfBatch) { return immediateFlush || endOfBatch; }
Source:
<code>private int fullTableScan(TableReference tableRef, KeyValueService kvs, int numCellsExpected, boolean thorough) { CandidateCellForSweepingRequest request = ImmutableCandidateCellForSweepingRequest.builder() .startRowInclusive(PtBytes.EMPTY_BYTE_ARRAY) .batchSizeHint(1000) .maxTimestampExclusive(Long.MAX_VALUE) .shouldCheckIfLatestValueIsEmpty(thorough) <START> .ignoreGarbageCollectionSentinels(!thorough) <END> .build(); try (ClosableIterator<List<CandidateCellForSweeping>> iter = kvs.getCandidateCellsForSweeping( tableRef, request)) { int numCandidates = Iterators.size(Iterators.concat(Iterators.transform(iter, List::iterator))); Preconditions.checkState(numCandidates == numCellsExpected, "Number of candidates %s != %s", numCandidates, numCellsExpected); return numCandidates; } }</code><technical_language>change this a name, ignore ignore + delete leave while sweeping. is, I this is intuitive</technical_language>

Target:
private int fullTableScan(TableReference tableRef, KeyValueService kvs, int numCellsExpected, boolean thorough) { CandidateCellForSweepingRequest request = ImmutableCandidateCellForSweepingRequest.builder() .startRowInclusive(PtBytes.EMPTY_BYTE_ARRAY) .batchSizeHint(1000) .maxTimestampExclusive(Long.MAX_VALUE) .shouldCheckIfLatestValueIsEmpty(thorough) .shouldDeleteGarbageCollectionSentinels(thorough) .build(); try (ClosableIterator<List<CandidateCellForSweeping>> iter = kvs.getCandidateCellsForSweeping( tableRef, request)) { int numCandidates = Iterators.size(Iterators.concat(Iterators.transform(iter, List::iterator))); Preconditions.checkState(numCandidates == numCellsExpected, "Number of candidates %s != %s", numCandidates, numCellsExpected); return numCandidates; } }
Source:
<code>public void onCorrectAdminPassword(AdminPasswordDialog.Action action) { switch (action) { case ADMIN_SETTINGS: startActivity(new Intent(this, AdminPreferencesActivity.class)); break; case STORAGE_MIGRATION: <START> StorageMigrationDialog storageMigrationDialog = (StorageMigrationDialog) DialogUtils.getDialogFragment(StorageMigrationDialog.class, getSupportFragmentManager()); <END> if (storageMigrationDialog != null) { storageMigrationDialog.startStorageMigration(); } break; } }</code><technical_language>this showIfNotShowing instead? call startStorageMigration returned dialog</technical_language>

Target:
public void onCorrectAdminPassword(AdminPasswordDialog.Action action) { switch (action) { case ADMIN_SETTINGS: startActivity(new Intent(this, AdminPreferencesActivity.class)); break; case STORAGE_MIGRATION: DialogUtils .showIfNotShowing(StorageMigrationDialog.create(savedCount), getSupportFragmentManager()) .startStorageMigration(); break; } }
Source:
<code><START> public int hashCode() { <END> return Objects.hashCode(locale, review); }</code><technical_language>Similarly Java 7 template</technical_language>

Target:
public int hashCode() { return Objects.hash(locale, review); }
Source:
<code>public void testCreateFixedInstanceServer() { fixedInstanceServer = serverApi().createFixedInstanceServer(Server.CreateFixedInstanceServer.create( "java test fixed instance 2", "testing with jclouds", FixedInstanceHardware.create(currentFlavour.id()), "7C5FA1D21B98DE39D7516333AAB7DA54", <START> null, "Test123!", null, Boolean.TRUE, null, null, null, null)); <END> assertNotNull(fixedInstanceServer); assertNotNull(fixedInstanceServer.id()); assertEquals(currentFlavour.hardware().vcore(), fixedInstanceServer.hardware().vcore()); assertEquals(currentFlavour.hardware().coresPerProcessor(), fixedInstanceServer.hardware().coresPerProcessor()); assertEquals(currentFlavour.hardware().ram(), fixedInstanceServer.hardware().ram()); }</code><technical_language>For constructors values, consider adding auto-value builder model class</technical_language>

Target:
public void testCreateFixedInstanceServer() { Server.CreateFixedInstanceServer request = Server.CreateFixedInstanceServer.builder() .name("java test fixed instance") .description("testing with jclouds") .hardware(FixedInstanceHardware.create(currentFlavour.id())) .applianceId("7C5FA1D21B98DE39D7516333AAB7DA54") .password("Test123!") .powerOn(Boolean.TRUE).build(); fixedInstanceServer = serverApi().createFixedInstanceServer(request); assertNotNull(fixedInstanceServer); assertNotNull(fixedInstanceServer.id()); assertEquals(currentFlavour.hardware().vcore(), fixedInstanceServer.hardware().vcore()); assertEquals(currentFlavour.hardware().coresPerProcessor(), fixedInstanceServer.hardware().coresPerProcessor()); assertEquals(currentFlavour.hardware().ram(), fixedInstanceServer.hardware().ram()); }
Source:
<code>public void onRequestPermissionsResult(int requestCode, @NonNull String permissions[], @NonNull int[] grantResults) { if (requestCode == REQUEST_CODE) { if (grantResults.length > 0 && grantResults[0] == PackageManager.PERMISSION_GRANTED) { permissionGranted = true; showFileChooser(); } else { finish(); } <START> } <END> }</code><technical_language>Excellent! add a Toast done</technical_language>

Target:
public void onRequestPermissionsResult(int requestCode, @NonNull String permissions[], @NonNull int[] grantResults) { if (requestCode == REQUEST_CODE) { if (grantResults.length > 0 && grantResults[0] == PackageManager.PERMISSION_GRANTED) { permissionGranted = true; Toast.makeText(getApplicationContext(), "Permission to Access External Storage Granted !", Toast.LENGTH_SHORT).show(); showFileChooser(); } else { Toast.makeText(getApplicationContext(), "Permission to Access External Storage Denied :(", Toast.LENGTH_SHORT).show(); finish(); } } }
Source:
<code>public void noAutoBuild() throws Exception { <START> final String projectName = "ValidationErrors"; END> final IProgressMonitor monitor = new NullProgressMonitor(); final IProject project = createAndPopulateProject(projectName, monitor); IMarker[] markers = findMarkersOnResource(project); Assert.assertArrayEquals(Collections.<IMarker> emptyList().toArray(), markers); setAutoBuild(false); ProjectNature.toggleNature(project, ValidationNature.NATURE_ID); Job.getJobManager().join(ResourcesPlugin.FAMILY_AUTO_BUILD, monitor); markers = findMarkersOnResource(project); Assert.assertArrayEquals(Collections.<IMarker> emptyList().toArray(), markers); }</code><technical_language>this more called "ValidModel" this is a valid project</technical_language>

Target:
public void noAutoBuild() throws CoreException, IOException { final String projectName = "ValidationErrors"; final IProgressMonitor monitor = new NullProgressMonitor(); final IProject project = createAndPopulateProject(projectName, monitor); IMarker[] markers = findMarkersOnResource(project); Assert.assertArrayEquals(Collections.<IMarker> emptyList().toArray(), markers); setAutoBuild(false); ProjectNature.toggleNature(project, ValidationNature.NATURE_ID); waitForAuroBuild(); markers = findMarkersOnResource(project); Assert.assertArrayEquals(Collections.<IMarker> emptyList().toArray(), markers); }
Source:
<code>public Expression generateIL(GenContext ctx) { CallableExprGenerator cegExpr = value.getCallableExpr(ctx); Expression exprToAssign = cegExpr.genExpr(); ValueType exprType = exprToAssign.getExprType(); CallableExprGenerator cegReceiver = target.getCallableExpr(ctx); Expression exprFieldGet = cegReceiver.genExpr(); if (!(exprFieldGet instanceof FieldGet)) <START> throw new RuntimeException("Should be assigning to object"); <END> FieldGet fieldGet = (FieldGet) exprFieldGet; String fieldName = fieldGet.getName(); Expression objExpr = fieldGet.getObjectExpr(); return new wyvern.target.corewyvernIL.expression.FieldSet(exprType, objExpr, fieldName, exprToAssign); }</code><technical_language>Another good place put a reportError() call</technical_language>

Target:
public Expression generateIL(GenContext ctx) { CallableExprGenerator cegExpr = value.getCallableExpr(ctx); Expression exprToAssign = cegExpr.genExpr(); ValueType exprType = exprToAssign.getExprType(); CallableExprGenerator cegReceiver = target.getCallableExpr(ctx); Expression exprFieldGet = cegReceiver.genExpr(); if (!(exprFieldGet instanceof FieldGet)) ToolError.reportError(ErrorMessage.EXPECTED_RECORD_TYPE, HasLocation.UNKNOWN); FieldGet fieldGet = (FieldGet) exprFieldGet; String fieldName = fieldGet.getName(); Expression objExpr = fieldGet.getObjectExpr(); return new wyvern.target.corewyvernIL.expression.FieldSet(exprType, objExpr, fieldName, exprToAssign); }
Source:
<code>public TimeLockServices createTimeLockServices( String client, Supplier<ManagedTimestampService> rawTimestampServiceSupplier, Supplier<LockService> rawLockServiceSupplier) { log.info("Creating async timelock services for client {}", SafeArg.of("client", client)); AsyncOrLegacyTimelockService asyncOrLegacyTimelockService; LockService lockService = instrumentInLeadershipProxy( LockService.class, asyncLockConfiguration.disableLegacySafetyChecksWarningPotentialDataCorruption() ? rawLockServiceSupplier : JavaSuppliers.compose(NonTransactionalLockService::new, rawLockServiceSupplier), client); <START> if (runtime.get().clientTokens().containsKey(client)) { <END> SecureTimelockService secureTimelockService = instrumentInLeadershipProxy( SecureTimelockService.class, () -> AsyncTimeLockServicesCreator.createRawSecureTimelockService(client, rawTimestampServiceSupplier), client); asyncOrLegacyTimelockService = AsyncOrLegacyTimelockService.createFromSecureTimelock( new SecureTimelockResource(secureTimelockService)); return TimeLockServices.create( secureTimelockService, lockService, asyncOrLegacyTimelockService, secureTimelockService); } AsyncTimelockService asyncTimelockService = instrumentInLeadershipProxy( AsyncTimelockService.class, () -> AsyncTimeLockServicesCreator.createRawAsyncTimelockService(client, rawTimestampServiceSupplier), client); asyncOrLegacyTimelockService = AsyncOrLegacyTimelockService.createFromAsyncTimelock( new AsyncTimelockResource(asyncTimelockService)); return TimeLockServices.create( asyncTimelockService, lockService, asyncOrLegacyTimelockService, asyncTimelockService); }</code><technical_language>this execute? happen call startup</technical_language>

Target:
public TimeLockServices createTimeLockServices( String client, Supplier<ManagedTimestampService> rawTimestampServiceSupplier, Supplier<LockService> rawLockServiceSupplier) { log.info("Creating async timelock services for client {}", SafeArg.of("client", client)); AsyncOrLegacyTimelockService asyncOrLegacyTimelockService; AsyncTimelockService asyncTimelockService = instrumentInLeadershipProxy( metricsManager.getTaggedRegistry(), AsyncTimelockService.class, () -> createRawAsyncTimelockService(client, rawTimestampServiceSupplier), client); asyncOrLegacyTimelockService = AsyncOrLegacyTimelockService.createFromAsyncTimelock( new AsyncTimelockResource(lockLog, asyncTimelockService)); LockService lockService = instrumentInLeadershipProxy( metricsManager.getTaggedRegistry(), LockService.class, asyncLockConfiguration.disableLegacySafetyChecksWarningPotentialDataCorruption() ? rawLockServiceSupplier : JavaSuppliers.compose(NonTransactionalLockService::new, rawLockServiceSupplier), client); return TimeLockServices.create( asyncTimelockService, lockService, asyncOrLegacyTimelockService, asyncTimelockService); }
Source:
<code>public static List<Program> preparePrograms() { List<Program> programs = new ArrayList<>(); List<Stage> stages = new ArrayList<>(); List<TrackedEntityAttribute> attributes = new ArrayList<>(); TrackedEntityAttribute attribute1 = new TrackedEntityAttribute("attribute1", "uuid"); TrackedEntityAttribute attribute2 = new TrackedEntityAttribute("attribute2", "uuid"); attributes.add(attribute1); attributes.add(attribute2); TrackedEntity trackedEntity = new TrackedEntity("trackedEntityName","trackedEntityID"); Program program1 = new Program(); program1.setName("Program1"); program1.setUuid("program1UUID"); program1.setRegistration(true); program1.setSingleEvent(false); program1.setTrackedEntity(trackedEntity); program1.setStages(stages); <START> program1.setAttributes(attributes); <END> Program program2 = new Program(); program2.setName("Program2"); program2.setUuid("program2UUID"); program2.setRegistration(true); program2.setSingleEvent(false); program2.setTrackedEntity(trackedEntity); program2.setStages(stages); program2.setAttributes(attributes); programs.add(program1); programs.add(program2); return programs; }</code><technical_language>constructor</technical_language>

Target:
public static List<Program> preparePrograms() { List<Program> programs = new ArrayList<>(); List<Stage> stages = new ArrayList<>(); List<TrackedEntityAttribute> attributes = new ArrayList<>(); attributes.add(new TrackedEntityAttribute("attribute1", "uuid")); attributes.add(new TrackedEntityAttribute("attribute2", "uuid")); TrackedEntity trackedEntity = new TrackedEntity("trackedEntityName","trackedEntityID"); programs.add(new Program("program1UUID", "Program1", trackedEntity, stages, attributes, false, true, null)); programs.add(new Program("program2UUID", "Program2", trackedEntity, stages, attributes, false, true, null)); return programs; }
Source:
<code>public void inspectRuntimeShape(RuntimeShapeInspector inspector) { <START> inspector.visit("delegate", delegate); <END> }</code><technical_language>add lookupCacheSize runtimeShape</technical_language>

Target:
public void inspectRuntimeShape(RuntimeShapeInspector inspector) { inspector.visit("cachedValues", cachedValues != null); inspector.visit("delegate", delegate); }
Source:
<code>public static File getProjectExamplesFile(URL url, String prefix, String suffix, IProgressMonitor monitor) { <START> File offline = OfflineUtil.getOfflineFile(url); <END> File file = null; if (PROTOCOL_FILE.equals(url.getProtocol()) || PROTOCOL_PLATFORM.equalsIgnoreCase(url.getProtocol())) { try { file = new File(new URI(url.toExternalForm())); } catch (Exception e) { file = new File(url.getFile()); } if (!file.exists()) return null; } else { if (OfflineUtil.isOfflineEnabled()) { return OfflineUtil.getOfflineFile(url); } if (monitor.isCanceled()) { return null; } file = getCacheFile(url); long urlModified = getRemoteTimestamp(url); if( canReuseCache(file, url, urlModified)) { return file; } file.getParentFile().mkdirs(); if (monitor.isCanceled()) { return null; } file = downloadExamplesFile(file, prefix, url, urlModified, monitor); } return file; }</code><technical_language>unused variable</technical_language>

Target:
public static File getProjectExamplesFile(URL url, String prefix, String suffix, IProgressMonitor monitor) { File file = null; if (PROTOCOL_FILE.equals(url.getProtocol()) || PROTOCOL_PLATFORM.equalsIgnoreCase(url.getProtocol())) { try { file = new File(new URI(url.toExternalForm())); } catch (Exception e) { file = new File(url.getFile()); } if (!file.exists()) return null; } else { if (OfflineUtil.isOfflineEnabled()) { return OfflineUtil.getOfflineFile(url); } if (monitor.isCanceled()) { return null; } file = getCacheFile(url); long urlModified = getRemoteTimestamp(url); if( canReuseCache(file, url, urlModified)) { return file; } file.getParentFile().mkdirs(); if (monitor.isCanceled()) { return null; } file = downloadExamplesFile(file, prefix, url, urlModified, monitor); } return file; }
Source:
<code><START> private IMethod[] <END> toArray(List<IMethod> methods) { return methods.toArray(new IMethod[methods.size()]); }</code><technical_language>static</technical_language>

Target:
private static IMethod[] toArray(List<IMethod> methods) { return methods.toArray(new IMethod[methods.size()]); }
Source:
<code>public void update() { synchronized (mutex) { seedTracker.refreshSeeds(); Optional<Entity> upNode = Iterables.tryFind(getMembers(), EntityPredicates.attributeEqualTo(SERVICE_UP, Boolean.TRUE)); setAttribute(SERVICE_UP, upNode.isPresent()); if (upNode.isPresent()) { <START> setAttribute(HOSTNAME, upNode.get().getAttribute(HOSTNAME)); <END> setAttribute(THRIFT_PORT, upNode.get().getAttribute(THRIFT_PORT)); List<String> currentNodes = getAttribute(CASSANDRA_CLUSTER_NODES); Set<String> oldNodes = (currentNodes != null) ? ImmutableSet.copyOf(currentNodes) : ImmutableSet.<String>of(); Set<String> newNodes = MutableSet.<String>of(); for (Entity member : getMembers()) { if (member instanceof CassandraNode && member.getAttribute(SERVICE_UP) == Boolean.TRUE) { HostAndPort address = HostAndPort.fromParts(member.getAttribute(HOSTNAME), member.getAttribute(THRIFT_PORT)); newNodes.add(address.toString()); } } if (Sets.difference(oldNodes, newNodes).size() > 0) { setAttribute(CASSANDRA_CLUSTER_NODES, MutableList.copyOf(newNodes)); } } else { setAttribute(HOSTNAME, null); setAttribute(THRIFT_PORT, null); setAttribute(CASSANDRA_CLUSTER_NODES, Collections.<String>emptyList()); } } }</code><technical_language>wrong - want CassandraNode.HOSTNAME. HOSTNAME refers cluster's hostname config fully qualified name "cassandra.cluster.hostname" is a thing. for THRIFT_PORT</technical_language>

Target:
public void update() { synchronized (mutex) { seedTracker.refreshSeeds(); Optional<Entity> upNode = Iterables.tryFind(getMembers(), EntityPredicates.attributeEqualTo(SERVICE_UP, Boolean.TRUE)); setAttribute(SERVICE_UP, upNode.isPresent()); if (upNode.isPresent()) { setAttribute(HOSTNAME, upNode.get().getAttribute(Attributes.HOSTNAME)); setAttribute(THRIFT_PORT, upNode.get().getAttribute(CassandraNode.THRIFT_PORT)); List<String> currentNodes = getAttribute(CASSANDRA_CLUSTER_NODES); Set<String> oldNodes = (currentNodes != null) ? ImmutableSet.copyOf(currentNodes) : ImmutableSet.<String>of(); Set<String> newNodes = MutableSet.<String>of(); for (Entity member : getMembers()) { if (member instanceof CassandraNode && member.getAttribute(SERVICE_UP) == Boolean.TRUE) { HostAndPort address = HostAndPort.fromParts(member.getAttribute(HOSTNAME), member.getAttribute(THRIFT_PORT)); newNodes.add(address.toString()); } } if (Sets.symmetricDifference(oldNodes, newNodes).size() > 0) { setAttribute(CASSANDRA_CLUSTER_NODES, MutableList.copyOf(newNodes)); } } else { setAttribute(HOSTNAME, null); setAttribute(THRIFT_PORT, null); setAttribute(CASSANDRA_CLUSTER_NODES, Collections.<String>emptyList()); } } }
Source:
<code>private void doClear() { <START> if (true || formRenderer.isValid()) { <END> formRenderer.synchronize(); } formRenderer.unBind(); }</code><technical_language>this true</technical_language>

Target:
private void doClear() { if (formRenderer.isValid()) { formRenderer.forceModelSynchronization(); } formRenderer.unBind(); }
Source:
<code>public UserDTO addUser(UserDTO user) { validateUserFields(user); user.getPrivateData().getVpnProfile().setVpnPassword(generatePassword()); User userEntity = user.createUserEntity(); <START> userEntity.setConfirmationCode(CryptoUtil.getRandomDigits(5)); <END> userEntity = User.getRepository().save(userEntity); ldapUserService.createVPNAccount(userEntity.getVPNLoginID().toString(), userEntity.getVPNPassword()); UserDTO userDTO = UserDTO.createInstanceWithPrivateData(userEntity); if (properties.getIsRunningInTestMode()) { userDTO.setConfirmationCode(userEntity.getConfirmationCode()); } else { sendMobileNumberConfirmationMessage(userEntity); } return userDTO; }</code><technical_language>number of 5 a setting</technical_language>

Target:
public UserDTO addUser(UserDTO user) { validateUserFields(user); user.getPrivateData().getVpnProfile().setVpnPassword(generatePassword()); User userEntity = user.createUserEntity(); userEntity.setConfirmationCode(CryptoUtil.getRandomDigits(mobileNumberConfirmationCodeDigits)); userEntity = User.getRepository().save(userEntity); ldapUserService.createVPNAccount(userEntity.getVPNLoginID().toString(), userEntity.getVPNPassword()); UserDTO userDTO = UserDTO.createInstanceWithPrivateData(userEntity); if (properties.getIsRunningInTestMode()) { userDTO.setConfirmationCode(userEntity.getConfirmationCode()); } else { sendMobileNumberConfirmationMessage(userEntity); } return userDTO; }
Source:
<code>public void channelRead0(ChannelHandlerContext ctx, HttpObject obj) throws RestServiceException { logger.trace("Reading on channel " + ctx.channel() + " from " + ctx.channel().remoteAddress()); if (obj != null) { if (vetRequest(obj)) { RestObject convertedObj = convertObjToGeneric(obj); handleMessage(convertedObj); } else { logger.error("Malformed request received - " + obj); nettyMetrics.malformedRequestErrorCount.inc(); <START> throw new RestServiceException("Malformed request received - " + obj, RestServiceErrorCode.BadRequest); <END> } } }</code><technical_language>in cases throw exception, do close connection</technical_language>

Target:
public void channelRead0(ChannelHandlerContext ctx, HttpObject obj) throws RestServiceException { logger.trace("Reading on channel " + ctx.channel() + " from " + ctx.channel().remoteAddress()); if (obj != null && obj instanceof HttpRequest) { if (obj.getDecoderResult().isSuccess()) { handleRequest((HttpRequest) obj); } else { logger.error("Malformed request received - " + obj); nettyMetrics.malformedRequestErrorCount.inc(); throw new RestServiceException("Malformed request received - " + obj, RestServiceErrorCode.MalformedRequest); } } else if (obj != null && obj instanceof HttpContent) { handleContent((HttpContent) obj); } else { nettyMetrics.unknownHttpObjectErrorCount.inc(); throw new RestServiceException("Content received is null or not of a known type", RestServiceErrorCode.UnknownHttpObject); } }
Source:
<code>public RetryingPrestoThriftServiceProvider( ConnectedThriftServiceProvider original, @ForRetryDriver ListeningScheduledExecutorService retryExecutor, ThriftConnectorConfig config) { this.original = requireNonNull(original, "original is null"); <START> retry = RetryDriver.retry(requireNonNull(retryExecutor, "retryExecutor is null")) <END> .maxAttempts(config.getMaxRetryAttempts()) .stopRetryingWhen(e -> e instanceof PrestoThriftServiceException && !((PrestoThriftServiceException) e).isRetryable()) .withClassifier(RetryingPrestoThriftServiceProvider::classifyException) .exponentialBackoff( config.getMinRetrySleepTime(), config.getMaxRetrySleepTime(), config.getMaxRetryDuration(), config.getRetryScaleFactor()); }</code><technical_language>remove null check for config? Also, retryExecutor null check separately</technical_language>

Target:
public RetryingPrestoThriftServiceProvider( ConnectedThriftServiceProvider original, @ForRetryDriver ListeningScheduledExecutorService retryExecutor, ThriftConnectorConfig config) { this.original = requireNonNull(original, "original is null"); requireNonNull(retryExecutor, "retryExecutor is null"); requireNonNull(config, "config is null"); retry = RetryDriver.retry(retryExecutor) .maxAttempts(config.getMaxRetryAttempts()) .stopRetryingWhen(e -> e instanceof PrestoThriftServiceException && !((PrestoThriftServiceException) e).isRetryable()) .withClassifier(RetryingPrestoThriftServiceProvider::classifyException) .exponentialBackoff( config.getMinRetrySleepTime(), config.getMaxRetrySleepTime(), config.getMaxRetryDuration(), config.getRetryScaleFactor()); }
Source:
<code>public ListenableFuture<PlayerResult> movePlaylistItem( @IntRange(from = 0) int fromIndex, @IntRange(from = 0) int toIndex) { ResolvableFuture<PlayerResult> future = ResolvableFuture.<PlayerResult>create(); future.set(new <START> PlayerResult(RESULT_ERROR_UNKNOWN, getCurrentMediaItem())); <END> return future; }</code><technical_language>questions Question 1) default implementation returns error instead of remove + add? Question 2) RESULT_ERROR_NOT_SUPPORTED</technical_language>

Target:
public ListenableFuture<PlayerResult> movePlaylistItem( @IntRange(from = 0) int fromIndex, @IntRange(from = 0) int toIndex) { ResolvableFuture<PlayerResult> future = ResolvableFuture.<PlayerResult>create(); future.set(new PlayerResult(RESULT_ERROR_NOT_SUPPORTED, getCurrentMediaItem())); return future; }
Source:
<code>public void startUp() { metricsProcessor.startAsync(); logTailer.startAsync(); <START> sink.startAsync(); <END> }</code><technical_language>need wait for start completion of of services? this service is a meta service starts other services</technical_language>

Target:
public void startUp() { serviceManager.startAsync().awaitHealthy(); }
Source:
<code>public void addData(String guid, String classification, PageOrientation orientation) { DataRightEntry toAdd = new DataRightEntry(); toAdd.setClassification(classification); toAdd.setGuid(guid); toAdd.setOrientation(orientation); if (data == null) { <START> data = new ArrayList<DataRightEntry>(); <END> } data.add(toAdd); }</code><technical_language>replace this if statement a call getData(). I cleaner</technical_language>

Target:
public void addData(String guid, String classification, PageOrientation orientation) { DataRightEntry toAdd = new DataRightEntry(); toAdd.setClassification(classification); toAdd.setGuid(guid); toAdd.setOrientation(orientation); getData(); data.add(toAdd); }
Source:
<code>public static void main(String[] args) { final RestServer restServer; try { final InvocationOptions options = new InvocationOptions(args); PropertyConfigurator.configure(options.logPropsFile); final Properties properties = Utils.loadProps(options.serverPropsFilePath); final VerifiableProperties verifiableProperties = new VerifiableProperties(properties); final MetricRegistry metricRegistry = new MetricRegistry(); final ClusterMap clusterMap = new ClusterMapManager(options.hardwareLayoutFilePath, options.partitionLayoutFilePath, new ClusterMapConfig(verifiableProperties)); logger.info("Bootstrapping RestServer"); restServer = new RestServer(verifiableProperties, metricRegistry, clusterMap); Runtime.getRuntime().addShutdownHook(new Thread() { public void run() { logger.info("Received shutdown signal. Requesting RestServer shutdown"); restServer.shutdown(); } }); restServer.start(); restServer.awaitShutdown(); } catch (Exception e) { logger.error("Exception during bootstrap of RestServer", e); } finally { <START> logger.info("Exiting"); <END> System.exit(0); } }</code><technical_language>Exiting what? more descriptive</technical_language>

Target:
public static void main(String[] args) { final RestServer restServer; try { final InvocationOptions options = new InvocationOptions(args); PropertyConfigurator.configure(options.logPropsFile); final Properties properties = Utils.loadProps(options.serverPropsFilePath); final VerifiableProperties verifiableProperties = new VerifiableProperties(properties); final MetricRegistry metricRegistry = new MetricRegistry(); final ClusterMap clusterMap = new ClusterMapManager(options.hardwareLayoutFilePath, options.partitionLayoutFilePath, new ClusterMapConfig(verifiableProperties)); logger.info("Bootstrapping RestServer"); restServer = new RestServer(verifiableProperties, metricRegistry, clusterMap); Runtime.getRuntime().addShutdownHook(new Thread() { public void run() { logger.info("Received shutdown signal. Requesting RestServer shutdown"); restServer.shutdown(); } }); restServer.start(); restServer.awaitShutdown(); } catch (Exception e) { logger.error("Exception during bootstrap of RestServer", e); } finally { logger.info("Exiting RestServerMain"); System.exit(0); } }
Source:
<code>public void testChangeUserPasswordH2DB() throws Exception { String loginStatusString = authenticatorClient.login <START> ("testu1", String.valueOf(userNewPassword), context.getInstance().getHosts().get("default")); <END> assertTrue(loginStatusString.contains("JSESSIONID"), "Unsuccessful login"); }</code><technical_language>remove hardcoded password</technical_language>

Target:
public void testChangeUserPasswordH2DB() throws Exception { String loginStatusString = authenticatorClient.login (userName, String.valueOf(userNewPassword), automationContext.getInstance().getHosts().get("default")); assertTrue(loginStatusString.contains("JSESSIONID"), "Unsuccessful login"); }
Source:
<code>private boolean isNotSpendingTxnsInConfidenceType(Transaction tx, ConfidenceType confidenceType) { <START> boolean isNotSpendingTxnsInConfidenceType = true; <END> for (TransactionInput txInput : tx.getInputs()) { Transaction connectedTx = this.getTransaction(txInput.getOutpoint().getHash()); if (connectedTx!=null && connectedTx.getConfidence().getConfidenceType().equals(confidenceType)) { isNotSpendingTxnsInConfidenceType = false; } } return isNotSpendingTxnsInConfidenceType; }</code><technical_language>this boolean result</technical_language>

Target:
private boolean isNotSpendingTxnsInConfidenceType(Transaction tx, ConfidenceType confidenceType) { for (TransactionInput txInput : tx.getInputs()) { Transaction connectedTx = this.getTransaction(txInput.getOutpoint().getHash()); if (connectedTx != null && connectedTx.getConfidence().getConfidenceType().equals(confidenceType)) { return false; } } return true; }
Source:
<code>public void writeToNBT(NBTTagCompound nbt) { <START> nbt.setInteger("dimension", this.dimension); <END> nbt.setTag("ForgeCaps", serializable.serializeNBT()); }</code><technical_language>versed in WorldSavedData, is this required? I thought WorldSavedData is per-world making this redundant</technical_language>

Target:
public void writeToNBT(NBTTagCompound nbt) { nbt.setTag("ForgeCaps", serializable.serializeNBT()); }
Source:
<code>public TmfStatisticsTree getStatisticData() { <START> return fModel.getStatisticData(); <END> }</code><technical_language>revert change</technical_language>

Target:
public TmfStatisticsTree getStatisticData() { if (fStatisticsData == null) { fStatisticsData = new TmfStatisticsTree(); } return fStatisticsData; }
Source:
<code>public static Project parseProject(JSONObject content) throws JSONException { final Project result = new Project(); result.setId(JsonInput.getInt(content, "id")); result.setIdentifier(JsonInput.getStringOrNull(content, "identifier")); result.setName(JsonInput.getStringNotNull(content, "name")); result.setDescription(JsonInput .getStringOrEmpty(content, "description")); result.setHomepage(JsonInput.getStringOrEmpty(content, "homepage")); result.setCreatedOn(getDateOrNull(content, "created_on")); result.setUpdatedOn(getDateOrNull(content, "updated_on")); final JSONObject parentProject = JsonInput.getObjectOrNull(content, "parent"); if (parentProject != null) result.setParentId(JsonInput.getInt(parentProject, "id")); result.setTrackers(JsonInput.getListOrNull(content, "trackers", TRACKER_PARSER)); <START> result.setCustomFields(JsonInput.getListOrNull(content, "custom_fields", <END> CUSTOM_FIELD_PARSER)); return result; }</code><technical_language>guessing this set NULL if element is found. is some convenient method set empty collection in this case</technical_language>

Target:
public static Project parseProject(JSONObject content) throws JSONException { final Project result = new Project(); result.setId(JsonInput.getInt(content, "id")); result.setIdentifier(JsonInput.getStringOrNull(content, "identifier")); result.setName(JsonInput.getStringNotNull(content, "name")); result.setDescription(JsonInput .getStringOrEmpty(content, "description")); result.setHomepage(JsonInput.getStringOrEmpty(content, "homepage")); result.setCreatedOn(getDateOrNull(content, "created_on")); result.setUpdatedOn(getDateOrNull(content, "updated_on")); final JSONObject parentProject = JsonInput.getObjectOrNull(content, "parent"); if (parentProject != null) result.setParentId(JsonInput.getInt(parentProject, "id")); result.setTrackers(JsonInput.getListOrNull(content, "trackers", TRACKER_PARSER)); result.setCustomFields(JsonInput.getListOrEmpty(content, "custom_fields", CUSTOM_FIELD_PARSER)); return result; }
Source:
<code>public BlobIndexValue(long size, long offset) { <START> this(size, offset, (byte)0, Utils.Infinite_TTL); <END> }</code><technical_language>This infinite time</technical_language>

Target:
public BlobIndexValue(long size, long offset) { this(size, offset, (byte)0, Utils.Infinite_Time); }
Source:
<code>private void closeOpenEditorsForProject(IProject project) { final List<IEditorReference> editorsToBeClosed = new ArrayList<>(); Map<IFile, IEditorReference> fileEditors = findAllEditorReferences(); try { List<IFile> projectfiles = <START> findAllProjectFiles(project); <END> for (IResource resource : projectfiles) { if (fileEditors.containsKey(resource)) { editorsToBeClosed.add(fileEditors.get(resource)); } } } catch (CoreException e) { Activator.logError("Problems during auto-close of related editors", e); } if (!editorsToBeClosed.isEmpty()) { PlatformUI.getWorkbench().getDisplay() .syncExec(new Runnable() { @Override public void run() { IEditorReference[] editorToBeClosed = new IEditorReference[editorsToBeClosed .size()]; editorToBeClosed = editorsToBeClosed .toArray(editorToBeClosed); activePage.closeEditors(editorToBeClosed, true); } }); } }</code><technical_language>I big concern algorithm. for people < 100 editors open (Eclipse is simply unusable higher number of editors), projects > 100000 files. HUGE projects. collect lists files? need is iterate opened editor files ask if file.getProject() equals project</technical_language>

Target:
private void closeOpenEditorsForProject(IProject project) { final List<IEditorReference> editorsToBeClosed = new ArrayList<>(); Map<IFile, IEditorReference> fileEditors = findAllEditorReferences(); Set<IFile> keySet = fileEditors.keySet(); for (IFile file : keySet) { if (file.getProject().equals(project)) { editorsToBeClosed.add(fileEditors.get(file)); } } if (!editorsToBeClosed.isEmpty()) { PlatformUI.getWorkbench().getDisplay() .syncExec(new Runnable() { @Override public void run() { IEditorReference[] editorToBeClosed = new IEditorReference[editorsToBeClosed .size()]; editorToBeClosed = editorsToBeClosed .toArray(editorToBeClosed); activePage.closeEditors(editorToBeClosed, true); } }); } }
Source:
<code>public PerfMmapStateProvider(ITmfTrace trace) { <START> super(trace, "perfMmap"); END> }</code><technical_language>a standard ID</technical_language>

Target:
public PerfMmapStateProvider(ITmfTrace trace) { super(trace, "Perf MMap Symbol Resolution"); }
Source:
<code><START> public StreamReadsRequest getStreamReadsRequest(String readGroupSetId) { <END> return StreamReadsRequest.newBuilder() .setReadGroupSetId(readGroupSetId) .setReferenceName(referenceName) .setStart(start) .setEnd(end) .build(); }</code><technical_language>for those, I understand if want do in a separate PR</technical_language>

Target:
StreamReadsRequest getStreamReadsRequest(String readGroupSetId) { return StreamReadsRequest.newBuilder() .setReadGroupSetId(readGroupSetId) .setReferenceName(referenceName) .setStart(start) .setEnd(end) .build(); }
Source:
<code>public Slf4JLogger() { <START> log = LoggerFactory.getLogger(Slf4JLogger.class); <END> }</code><technical_language>Making this change category configure loggin need change. Previously, work: <category name="p6spy"> <priority value="DEBUG" /> </category> this change <category name="com.p6spy.engine.spy.appender.Slf4JLogger"> <priority value="DEBUG" /> </category> a parent package com.p6spy. While this is a terribly bad thing, give users impression configure logging for p6spy. Unfortunately, this case a single logger</technical_language>

Target:
public Slf4JLogger() { log = LoggerFactory.getLogger("p6spy"); }