auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>protected void configure() { <START> synchronized (lock) { <END> if (alreadyBound) { if (!boundType.equals(getClass())) { throw new RuntimeException("You are trying to use more than one DispatchAsync implementation. " + boundType.getName() + " was already installed."); } } else { alreadyBound = true; boundType = getClass(); bind(ExceptionHandler.class).to(exceptionHandlerType); bind(SecurityCookieAccessor.class).to(sessionAccessorType); bind(ClientActionHandlerRegistry.class).to(clientActionHandlerRegistryType).asEagerSingleton(); } } }</code><technical_language>work client side? this is needed b/c threaded</technical_language>

Target:
protected void configure() { if (alreadyBound) { if (!boundType.equals(getClass())) { throw new RuntimeException("You are trying to use more than one DispatchAsync implementation. " + boundType.getName() + " was already installed."); } } else { alreadyBound = true; boundType = getClass(); bind(ExceptionHandler.class).to(exceptionHandlerType); bind(SecurityCookieAccessor.class).to(sessionAccessorType); bind(ClientActionHandlerRegistry.class).to(clientActionHandlerRegistryType).asEagerSingleton(); } }
Source:
<code>public void testGetCustomData(boolean createRepresentationInSeparateResource) throws Exception { TestsUtil.setCreateRepresentationInSeparateResource(createRepresentationInSeparateResource); DDiagram diagram2 = (DDiagram) createRepresentation(ENTITIES_DESC_NAME); final DFeatureExtension eClass = createExtension(); Command cmd = new RecordingCommand(session.getTransactionalEditingDomain()) { @Override protected void doExecute() { session.getServices().putCustomData(CustomDataConstants.DFEATUREEXTENSION, diagram2, eClass); session.save(defaultProgress); } }; executeCommand(cmd); Collection<EObject> collected; collected = session.getServices().getCustomData(CustomDataConstants.DFEATUREEXTENSION, null); assertEquals(1, collected.size()); assertTrue(collected.contains(eClass)); <START> if (TestsUtil.isCreateRepresentationInSeparateResource()) { <END> collected = session.getServices().getCustomData(CustomDataConstants.DFEATUREEXTENSION, diagram2); assertEquals(1, collected.size()); assertTrue(collected.contains(eClass)); } }</code><technical_language>remove. work in cases</technical_language>

Target:
public void testGetCustomData(boolean createRepresentationInSeparateResource) throws Exception { TestsUtil.setCreateRepresentationInSeparateResource(createRepresentationInSeparateResource); DDiagram diagram2 = (DDiagram) createRepresentation(ENTITIES_DESC_NAME); final DFeatureExtension eClass = createExtension(); Command cmd = new RecordingCommand(session.getTransactionalEditingDomain()) { @Override protected void doExecute() { session.getServices().putCustomData(CustomDataConstants.DFEATUREEXTENSION, diagram2, eClass); session.save(defaultProgress); } }; executeCommand(cmd); Collection<EObject> collected; collected = session.getServices().getCustomData(CustomDataConstants.DFEATUREEXTENSION, null); assertEquals(1, collected.size()); assertTrue(collected.contains(eClass)); collected = session.getServices().getCustomData(CustomDataConstants.DFEATUREEXTENSION, diagram2); assertEquals(1, collected.size()); assertTrue(collected.contains(eClass)); }
Source:
<code>public void removeBreakpoints(IBreakpoint[] breakpoints, final boolean delete) throws CoreException { final List<IBreakpoint> remove = new ArrayList<>(breakpoints.length); List<IBreakpoint> bps = getBreakpoints0(); for (IBreakpoint breakpoint : breakpoints) { if (bps.contains(breakpoint)) { remove.add(breakpoint); } } if (!remove.isEmpty()) { for (IBreakpoint breakpoint : remove) { bps.remove(breakpoint); fMarkersToBreakpoints.remove(breakpoint.getMarker()); } fireUpdate(remove, null, REMOVED); refreshTriggerpointDisplay(); <START> IWorkspaceRunnable r = montitor -> { <END> for (IBreakpoint breakpoint : remove) { if (delete) { breakpoint.delete(); } else { IMarker marker = breakpoint.getMarker(); if (marker.exists()) { IProject project = breakpoint.getMarker().getResource().getProject(); if (project == null || project.isOpen()) { breakpoint.setRegistered(false); } } } } }; getWorkspace().run(r, null, 0, null); } }</code><technical_language>typo: montitor other "pm", "pm" "monitor" in locations</technical_language>

Target:
public void removeBreakpoints(IBreakpoint[] breakpoints, final boolean delete) throws CoreException { final List<IBreakpoint> remove = new ArrayList<>(breakpoints.length); List<IBreakpoint> bps = getBreakpoints0(); for (IBreakpoint breakpoint : breakpoints) { if (bps.contains(breakpoint)) { remove.add(breakpoint); } } if (!remove.isEmpty()) { for (IBreakpoint breakpoint : remove) { bps.remove(breakpoint); fMarkersToBreakpoints.remove(breakpoint.getMarker()); } fireUpdate(remove, null, REMOVED); refreshTriggerpointDisplay(); IWorkspaceRunnable r = monitor -> { for (IBreakpoint breakpoint : remove) { if (delete) { breakpoint.delete(); } else { IMarker marker = breakpoint.getMarker(); if (marker.exists()) { IProject project = breakpoint.getMarker().getResource().getProject(); if (project == null || project.isOpen()) { breakpoint.setRegistered(false); } } } } }; getWorkspace().run(r, null, 0, null); } }
Source:
<code>public void handle(CacheRpcCommand command, Reply reply, DeliverOrder order) { if (command instanceof EndInvalidationCommand) { <START> ExpectingEndInvalidationCommand cmd = new <END> ExpectingEndInvalidationCommand((EndInvalidationCommand) command, latch); delegate.handle(cmd, reply, order); } else { delegate.handle(command, reply, order); } }</code><technical_language>need create a new command all. call java delegate.handle(command, response -> { latch.countDown(); reply.reply(response); }, order);</technical_language>

Target:
public void handle(CacheRpcCommand command, Reply reply, DeliverOrder order) { if (command instanceof EndInvalidationCommand) { delegate.handle(command, response -> { latch.countDown(); log.tracef("Latch after count down %s", latch); reply.reply(response); }, order); } else { delegate.handle(command, reply, order); } }
Source:
<code>public PutJobResult call() throws Exception { try { final ModifyJobRequest request = new ModifyJobRequest(jobId); if (priority != null) { request.withPriority(priority); } getClient().modifyJob(request); } <START> catch (IOException e) { <END> throw new CommandException("Error: Request failed with the following error: " + e.getMessage(), e); } String result = "Success: Modified job with job id '" + jobId.toString() + "'"; if (priority != null) { result = result.concat(" with priority " + priority.toString()); } return new PutJobResult(result + "."); }</code><technical_language>Missing final</technical_language>

Target:
public PutJobResult call() throws Exception { try { final ModifyJobRequest request = new ModifyJobRequest(jobId); if (priority != null) { request.withPriority(priority); } getClient().modifyJob(request); } catch (final IOException e) { throw new CommandException("Error: Request failed with the following error: " + e.getMessage(), e); } String result = "Success: Modified job with job id '" + jobId.toString() + "'"; if (priority != null) { result = result.concat(" with priority " + priority.toString()); } return new PutJobResult(result + "."); }
Source:
<code>private void handleTextWebSocketMessage(TextCarbonMessage textCarbonMessage, PatternPathRouter.RoutableDestination<Object> routableEndpoint, Session session) { Object endpoint = routableEndpoint.getDestination(); Map<String, String> paramValues = routableEndpoint.getGroupNameValues(); Method method = new EndpointDispatcher().getOnStringMessageMethod(endpoint); try { List<Object> parameterList = new LinkedList<>(); boolean isStringSatifsfied = false; Arrays.stream(method.getParameters()).forEach( parameter -> { if (parameter.getType() == String.class) { PathParam pathParam = parameter.getAnnotation(PathParam.class); if (pathParam == null) { parameterList.add(textCarbonMessage.getText()); } else { if (isStringSatifsfied == false) { parameterList.add(paramValues.get(pathParam.value())); } else { <START> parameterList.add(null); <END> throw new IllegalArgumentException("String parameters without" + " @PathParam annotation"); } } } else if (parameter.getType() == Session.class) { parameterList.add(session); } else { parameterList.add(null); } } ); executeMethod(method, endpoint, parameterList, session); } catch (Throwable throwable) { handleError(textCarbonMessage, throwable, routableEndpoint, session); } }</code><technical_language>If throwing, do need add a null list</technical_language>

Target:
private void handleTextWebSocketMessage(TextCarbonMessage textCarbonMessage, PatternPathRouter.RoutableDestination<Object> routableEndpoint, Session session) { Object endpoint = routableEndpoint.getDestination(); Map<String, String> paramValues = routableEndpoint.getGroupNameValues(); Optional<Method> methodOptional = new EndpointDispatcher().getOnStringMessageMethod(endpoint); try { methodOptional.ifPresent( method -> { List<Object> parameterList = new LinkedList<>(); boolean isStringSatifsfied = false; Arrays.stream(method.getParameters()).forEach( parameter -> { if (parameter.getType() == String.class) { PathParam pathParam = parameter.getAnnotation(PathParam.class); if (pathParam == null) { parameterList.add(textCarbonMessage.getText()); } else { if (!isStringSatifsfied) { parameterList.add(paramValues.get(pathParam.value())); } else { throw new IllegalArgumentException("String parameters without" + " @PathParam annotation"); } } } else if (parameter.getType() == Session.class) { parameterList.add(session); } else { parameterList.add(null); } } ); executeMethod(method, endpoint, parameterList, session); } ); } catch (Throwable throwable) { handleError(throwable, routableEndpoint, session); } }
Source:
<code>public List<Feature> getInstallationProfiles() { LOGGER.debug("Looking for installation profile features"); List<Feature> profiles = new ArrayList<>(); try { profiles = <START> Arrays.asList(featuresService.listFeatures()) <END> .stream() .filter(f -> f.getName().contains(INSTALLATION_PROFILE_PREFIX)) .sorted(Comparator.comparingInt(Feature::getStartLevel)) .collect(Collectors.toList()); } catch (Exception e) { LOGGER.warn( "Encountered an error while trying to obtain the installation profile features.", e); } return profiles; }</code><technical_language>change? This creates a temporary collection stream it</technical_language>

Target:
public List<Feature> getInstallationProfiles() { LOGGER.debug("Looking for installation profile features"); List<Feature> profiles = new ArrayList<>(); try { profiles = Arrays.stream(featuresService.listFeatures()) .filter(f -> f.getName().contains(INSTALLATION_PROFILE_PREFIX)) .sorted(Comparator.comparingInt(Feature::getStartLevel)) .collect(Collectors.toList()); } catch (Exception e) { LOGGER.warn( "Encountered an error while trying to obtain the installation profile features.", e); } return profiles; }
Source:
<code>public void checkForUpdates() { visit("advanced"); <START> clickButton("Check now"); <END> updated = true; }</code><technical_language>do longer wait for check happen - instant - smells a :bug:</technical_language>

Target:
public void checkForUpdates() { visit("advanced"); final String current = getCurrentUrl(); clickButton("Check now"); waitFor(find(by.button("Check now"))).withTimeout(30, TimeUnit.SECONDS).until(new Predicate<WebElement>() { @Override public boolean apply(@Nullable WebElement webElement) { try { try { webElement.findElement(by.id("it does not matter")); } catch(StaleElementReferenceException e) { if (current.equals(getCurrentUrl())) { return true; } } } catch(Exception e) { } return true; } }); updated = true; }
Source:
<code>private Timer newNettyTimer(AsyncHttpClientConfig config) { <START> ThreadFactory threadFactory = config.getThreadFactory() != null ? config.getThreadFactory() : new DefaultThreadFactory(config.getThreadPoolName()); <END> HashedWheelTimer timer = new HashedWheelTimer(threadFactory); timer.start(); return timer; }</code><technical_language>Please restore current behavior for fallback, "-timer" prefix</technical_language>

Target:
private Timer newNettyTimer(AsyncHttpClientConfig config) { ThreadFactory threadFactory = config.getThreadFactory() != null ? config.getThreadFactory() : new DefaultThreadFactory(config.getThreadPoolName() + "-timer"); HashedWheelTimer timer = new HashedWheelTimer(threadFactory); timer.start(); return timer; }
Source:
<code>public void testNewPomStructure() throws IOException, XmlPullParserException { final File projectDir = getProjectDir(SPACE_B, PROJECT_B2), pom = new File(projectDir, "pom.xml"); final MavenXpp3Reader reader = new MavenXpp3Reader(); final Model model = reader.read(new FileInputStream(pom)); assertThat(model.getVersion()).isEqualTo("1.2.3"); <START> assertThat(model.getDependencies().size()).isEqualTo(6); <END> assertThat(model.getBuild().getPlugins().size()).isEqualTo(1); assertThat(model.getPackaging()).isEqualTo("kjar"); }</code><technical_language>size()).isEqualTo() -> .hasSize(..) Ctrl+f for "size())" apply in pr, please</technical_language>

Target:
public void testNewPomStructure() throws IOException, XmlPullParserException { final File projectDir = getProjectDir(SPACE_B, PROJECT_B2), pom = new File(projectDir, "pom.xml"); final MavenXpp3Reader reader = new MavenXpp3Reader(); final Model model = reader.read(new FileInputStream(pom)); assertThat(model.getVersion()).isEqualTo("1.2.3"); assertThat(model.getDependencies()).hasSize(6); assertThat(model.getBuild().getPlugins()).hasSize(1); assertThat(model.getPackaging()).isEqualTo("kjar"); }
Source:
<code>private void emitReset(StringBuilder builder) { builder.append(" public static void reset(").append(targetClass) .append(" target) {\n"); if (parentInjector != null) { builder.append(" ").append(parentInjector) .append(".reset(target);\n\n"); } for (ViewInjection injection : viewIdMap.values()) { for (ViewBinding viewBinding : injection.getViewBindings()) { builder.append(" target.").append(viewBinding.getName()) .append(" = null;\n"); } } <START> for (ResourceInjection injection : resourceIdMap.values()) { <END> for (ResourceBinding binding : injection.getResourceBindings()) { builder.append(" target.").append(binding.getName()) .append(" = null;\n"); } } for (CollectionBinding collectionBinding : collectionBindings.keySet()) { builder.append(" target.").append(collectionBinding.getName()) .append(" = null;\n"); } builder.append(" }\n"); }</code><technical_language>This dropped. need clean non-View resources</technical_language>

Target:
private void emitReset(StringBuilder builder) { builder.append(" public static void reset(").append(targetClass) .append(" target) {\n"); if (parentInjector != null) { builder.append(" ").append(parentInjector) .append(".reset(target);\n\n"); } for (ViewInjection injection : viewIdMap.values()) { for (ViewBinding viewBinding : injection.getViewBindings()) { builder.append(" target.").append(viewBinding.getName()) .append(" = null;\n"); } } for (CollectionBinding collectionBinding : collectionBindings.keySet()) { builder.append(" target.").append(collectionBinding.getName()) .append(" = null;\n"); } builder.append(" }\n"); }
Source:
<code>public static boolean isFlashEdgeCase( byte[] request , int requestsize ){ <START> byte[] req = "<policy-file-request/>".getBytes( UTF8_CHARSET ); <END> for( int i = 0 ; i < requestsize && i < req.length ; i++ ){ if( req[i] != request[i] ){ return false; } } return requestsize >= req.length; }</code><technical_language>This cached for speed (as a private static final I suppose)</technical_language>

Target:
public static boolean isFlashEdgeCase( byte[] request , int requestsize ){ for( int i = 0 ; i < requestsize && i < FLASH_POLICY_REQUEST.length ; i++ ){ if( FLASH_POLICY_REQUEST[i] != request[i] ){ return false; } } return requestsize >= FLASH_POLICY_REQUEST.length; }
Source:
<code>private static void injectSoundFileInStream( MediaStream stream, String fileName) throws Throwable { OpusFile of = new OpusFile(new OggPacketReader( Util.class.getClassLoader().getResourceAsStream(fileName))); OpusAudioData opusAudioData; int seq = new Random().nextInt(0xFFFF); long ts = new Random().nextInt(0xFFFF); long ssrc = new Random().nextInt(0xFFFF); byte pt = stream.getDynamicRTPPayloadType(Constants.OPUS); long timePushPreviousData = 0; while ((opusAudioData = of.getNextAudioPacket()) != null) { if (seq > AbstractCodec2.SEQUENCE_MAX) { seq = 0; } int nSamples = opusAudioData.getNumberOfSamples(); ts += nSamples; if (ts > TimestampUtils.MAX_TIMESTAMP_VALUE) { ts = ts - TimestampUtils.MAX_TIMESTAMP_VALUE; } byte[] data = opusAudioData.getData(); RawPacket rtp = Util.makeRTP( ssrc, pt, seq++, ts, data.length + RawPacket.FIXED_HEADER_SIZE ); System.arraycopy( data, 0, rtp.getBuffer(), rtp.getPayloadOffset(), data.length); <START> long sleepTime = nSamples/48 - (System.currentTimeMillis() - timePushPreviousData); <END> if (sleepTime < 0) sleepTime = 0; Thread.sleep(sleepTime); stream.injectPacket(rtp, true, null); timePushPreviousData = System.currentTimeMillis(); } }</code><technical_language>timePushPreviousData set previous iteration *after* sleep() call, account long sleep took. I missing something</technical_language>

Target:
private static void injectSoundFileInStream( MediaStream stream, String fileName) throws Throwable { OpusFile of = new OpusFile(new OggPacketReader( Util.class.getClassLoader().getResourceAsStream(fileName))); OpusAudioData opusAudioData; int seq = new Random().nextInt(0xFFFF); long ts = new Random().nextInt(0xFFFF); long ssrc = new Random().nextInt(0xFFFF); byte pt = stream.getDynamicRTPPayloadType(Constants.OPUS); long timeForNextPacket = System.currentTimeMillis(); long sentDuration = 0; while ((opusAudioData = of.getNextAudioPacket()) != null) { if (seq > AbstractCodec2.SEQUENCE_MAX) { seq = 0; } int nSamples = opusAudioData.getNumberOfSamples(); ts += nSamples; if (ts > TimestampUtils.MAX_TIMESTAMP_VALUE) { ts = ts - TimestampUtils.MAX_TIMESTAMP_VALUE; } byte[] data = opusAudioData.getData(); RawPacket rtp = Util.makeRTP( ssrc, pt, seq++, ts, data.length + RawPacket.FIXED_HEADER_SIZE ); System.arraycopy( data, 0, rtp.getBuffer(), rtp.getPayloadOffset(), data.length); int duration = nSamples/48; timeForNextPacket += duration; sentDuration += duration; stream.injectPacket(rtp, true, null); long sleep = timeForNextPacket - System.currentTimeMillis(); if (sleep > 0 && sentDuration > 200) { Thread.sleep(sleep); } } }
Source:
<code>private List<Patch> runNopolProcessor(List<TestResult> tests, SourceLocation sourceLocation, SpoonedClass spoonCl, NopolProcessor nopolProcessor) { AngelicValue angelicValue; List<Patch> patches = new ArrayList<>(); try { angelicValue = buildConstraintsModelBuilder(nopolProcessor, sourceLocation, spooner); } catch (UnsupportedOperationException | DynamicCompilationException ignored) { return patches; } if (angelicValue != null) { this.nopolResult.incrementNbAngelicValues(); } else { <START> this.nopolResult.setNopolStatus(NopolStatus.NO_ANGELIC_VALUE); <END> } Synthesizer synth = SynthesizerFactory.build(sourceFiles, spooner, nopolContext, sourceLocation, nopolProcessor, angelicValue, spoonCl); if (synth == Synthesizer.NO_OP_SYNTHESIZER) { return patches; } Collection<TestCase> failingTest = reRunFailingTestCases(getFailingTestCase(tests), new URLClassLoader(classpath)); if (failingTest.isEmpty()) { return patches; } List<Patch> tmpPatches = synth.buildPatch(classpath, tests, failingTest, nopolContext.getMaxTimeBuildPatch()); if (tmpPatches.size() > 0) { for (int i = 0; i < tmpPatches.size(); i++) { Patch patch = tmpPatches.get(i); if (nopolContext.isSkipRegressionStep() || isOk(patch, tests, synth.getProcessor())) { patches.add(patch); if (nopolContext.isOnlyOneSynthesisResult()) { this.nopolResult.setNopolStatus(NopolStatus.PATCH); return patches; } } else { logger.debug("Could not find a patch in {}", sourceLocation); } } } return patches; }</code><technical_language>Unnecessary statement</technical_language>

Target:
private List<Patch> runNopolProcessor(List<TestResult> tests, SourceLocation sourceLocation, SpoonedClass spoonCl, NopolProcessor nopolProcessor) { AngelicValue angelicValue; List<Patch> patches = new ArrayList<>(); try { angelicValue = buildConstraintsModelBuilder(nopolProcessor, sourceLocation, spooner); } catch (UnsupportedOperationException | DynamicCompilationException ignored) { return patches; } if (angelicValue != null) { this.nopolResult.incrementNbAngelicValues(); } Synthesizer synth = SynthesizerFactory.build(sourceFiles, spooner, nopolContext, sourceLocation, nopolProcessor, angelicValue, spoonCl); if (synth == Synthesizer.NO_OP_SYNTHESIZER) { return patches; } Collection<TestCase> failingTest = reRunFailingTestCases(getFailingTestCase(tests), new URLClassLoader(classpath)); if (failingTest.isEmpty()) { return patches; } List<Patch> tmpPatches = synth.buildPatch(classpath, tests, failingTest, nopolContext.getMaxTimeBuildPatch()); if (tmpPatches.size() > 0) { for (int i = 0; i < tmpPatches.size(); i++) { Patch patch = tmpPatches.get(i); if (nopolContext.isSkipRegressionStep() || isOk(patch, tests, synth.getProcessor())) { patches.add(patch); if (nopolContext.isOnlyOneSynthesisResult()) { return patches; } } else { logger.debug("Could not find a patch in {}", sourceLocation); } } } return patches; }
Source:
<code>public void loadMessages(MessageRepository messageRepository) { List<BackupMessage> messages = messageRepository.findAll(); logger.info("Loading messages from backup storage.", messages.size()); int retry = 0; toResent.set(new ConcurrentLinkedQueue<>()); sendMessages(messages); do { if (retry > 0) { List<Pair<Message, Topic>> retryMessages = new ArrayList(toResent.getAndSet(new ConcurrentLinkedQueue<>())); resendMessages(retryMessages, retry); } try { Thread.sleep(resentSleep); } catch (InterruptedException e) { logger.warn("Sleep interrupted", e); } retry++; <START> } while (toResent.get().size() > 0 && retry <= maxResent); <END> logger.info("Finished resending messages from backup storage after retry #{} with #{} not sent messages.", retry, toResent.get().size()); }</code><technical_language>this retry < maxResent</technical_language>

Target:
public void loadMessages(MessageRepository messageRepository) { List<BackupMessage> messages = messageRepository.findAll(); logger.info("Loading messages from backup storage.", messages.size()); int retry = 0; toResend.set(new ConcurrentLinkedQueue<>()); sendMessages(messages); do { if (retry > 0) { List<Pair<Message, Topic>> retryMessages = new ArrayList(toResend.getAndSet(new ConcurrentLinkedQueue<>())); resendMessages(retryMessages, retry); } try { Thread.sleep(resendSleep); } catch (InterruptedException e) { logger.warn("Sleep interrupted", e); } retry++; } while (toResend.get().size() > 0 && retry <= maxResendRetries); logger.info("Finished resending messages from backup storage after retry #{} with #{} not sent messages.", retry, toResend.get().size()); }
Source:
<code>public void tighten() { RealInterval res; switch (op) { <START> case NEG: case ABS: case EXP: case LN: case SQRT: <END> case COS: res = RealUtils.cos(e); break; case SIN: res = RealUtils.sin(e); break; case TAN: case ACOS: case ASIN: case ATAN: case COSH: case SINH: case TANH: case ACOSH: case ASINH: case ATANH: default: throw new UnsupportedOperationException("Equation does not support " + op.name()); } l.set(res.getLB()); u.set(res.getUB()); }</code><technical_language>moved other treated cases</technical_language>

Target:
public void tighten() { RealInterval res; switch (op) { case COS: res = RealUtils.cos(e); break; case SIN: res = RealUtils.sin(e); break; case NEG: res = new RealIntervalConstant(-e.getUB(), -e.getLB()); break; case SQR: res = RealUtils.iPower(e, 2); break; case SQRT: res = RealUtils.iRoot(e, 2); break; case CUB: res = RealUtils.iPower(e, 3); break; case CBRT: res = RealUtils.iRoot(e, 3); break; case ABS: double linf = e.getLB(); double lsup = e.getUB(); if(lsup < 0.){ res = new RealIntervalConstant(-lsup, -linf); }else if (linf < 0.){ res = new RealIntervalConstant(0., Math.max(-linf, lsup)); }else{ res = new RealIntervalConstant(linf, lsup); } break; case EXP: case LN: case TAN: case ACOS: case ASIN: case ATAN: case COSH: case SINH: case TANH: case ACOSH: case ASINH: case ATANH: default: throw new UnsupportedOperationException("Equation does not support " + op.name()+". Consider using Ibex instead."); } l.set(res.getLB()); u.set(res.getUB()); }
Source:
<code><START> public Account removeGroup(String hrefOrName) { <END> GroupMembership groupMembership = null; for (GroupMembership aGroupMembership : getGroupMemberships()) { if (aGroupMembership.getGroup().getName().equalsIgnoreCase(hrefOrName) || aGroupMembership.getGroup().getHref().equalsIgnoreCase(hrefOrName)) { groupMembership = aGroupMembership; break; } } if (groupMembership != null){ groupMembership.delete(); } return this; }</code><technical_language>@since 1.0.RC5</technical_language>

Target:
public Account removeGroup(String hrefOrName) { GroupMembership groupMembership = null; for (GroupMembership aGroupMembership : getGroupMemberships()) { if (aGroupMembership.getGroup().getName().equals(hrefOrName) || aGroupMembership.getGroup().getHref().equals(hrefOrName)) { groupMembership = aGroupMembership; break; } } if (groupMembership != null){ groupMembership.delete(); } else { throw new IllegalStateException("This account does not belong to the specified group."); } return this; }
Source:
<code>public JobProperty<?> reconfigure(StaplerRequest req, @Nonnull JSONObject form) throws Descriptor.FormException { DatadogJobProperty prop = (DatadogJobProperty) super.reconfigure(req, form); <START> if (prop == null){ <END> return null; } boolean isEnableFile = form.getBoolean("enableFile"); boolean isEnableTagProperties = form.getBoolean("enableProperty"); if (!isEnableFile) { prop.tagFile = null; prop.emitOnCheckout = false; } if (!isEnableTagProperties) { prop.tagProperties = null; } return prop; }</code><technical_language>Is this a safety check? Whats case this null</technical_language>

Target:
public JobProperty<?> reconfigure(StaplerRequest req, @Nonnull JSONObject form) throws Descriptor.FormException { DatadogJobProperty prop = (DatadogJobProperty) super.reconfigure(req, form); boolean isEnableFile = form.getBoolean("enableFile"); boolean isEnableTagProperties = form.getBoolean("enableProperty"); if (!isEnableFile) { prop.tagFile = null; prop.emitOnCheckout = false; } if (!isEnableTagProperties) { prop.tagProperties = null; } return prop; }
Source:
<code>protected String cleanExpression(String rawExpression) { if (rawExpression != null && rawExpression.trim().startsWith(MVEL_ESCAPE_SYMBOL)) { return rawExpression.replaceFirst(MVEL_ESCAPE_SYMBOL, ""); } Optional<JsonNode> optionalJSONNode = JsonUtils.convertFromStringToJSONNode(rawExpression); if (optionalJSONNode.isPresent()) { JsonNode jsonNode = optionalJSONNode.get(); if (jsonNode.isTextual() && jsonNode.asText() != null && jsonNode.asText().trim().startsWith(MVEL_ESCAPE_SYMBOL)) { String expression = jsonNode.asText(); return expression.replaceFirst(MVEL_ESCAPE_SYMBOL, ""); } } <START> throw new IllegalArgumentException("Malformed MVEL expression '" + rawExpression + "'"); <END> }</code><technical_language>Do want put message ContantHolder class</technical_language>

Target:
protected String cleanExpression(String rawExpression) { if (rawExpression != null && rawExpression.trim().startsWith(MVEL_ESCAPE_SYMBOL)) { return rawExpression.replaceFirst(MVEL_ESCAPE_SYMBOL, ""); } Optional<JsonNode> optionalJSONNode = JsonUtils.convertFromStringToJSONNode(rawExpression); if (optionalJSONNode.isPresent()) { JsonNode jsonNode = optionalJSONNode.get(); if (jsonNode.isTextual() && jsonNode.asText() != null && jsonNode.asText().trim().startsWith(MVEL_ESCAPE_SYMBOL)) { String expression = jsonNode.asText(); return expression.replaceFirst(MVEL_ESCAPE_SYMBOL, ""); } } throw new IllegalArgumentException(MALFORMED_MVEL_EXPRESSION + "'" + rawExpression + "'"); }
Source:
<code>public void open() throws Exception { super.open(); <START> timestampAssigner = watermarkStrategy.createTimestampAssigner(this::getMetricGroup); <END> watermarkGenerator = watermarkStrategy.createWatermarkGenerator(); wmOutput = new WatermarkEmitter(output, getContainingTask().getStreamStatusMaintainer()); watermarkInterval = getExecutionConfig().getAutoWatermarkInterval(); if (watermarkInterval > 0) { final long now = getProcessingTimeService().getCurrentProcessingTime(); getProcessingTimeService().registerTimer(now + watermarkInterval, this); } }</code><technical_language>@kl0u reviewing DeserializationSchema#open() method suggested add "user" metric group user metrics, is easier distinguish system user metrics</technical_language>

Target:
public void open() throws Exception { super.open(); timestampAssigner = watermarkStrategy.createTimestampAssigner(this::getMetricGroup); watermarkGenerator = watermarkStrategy.createWatermarkGenerator(this::getMetricGroup); wmOutput = new WatermarkEmitter(output, getContainingTask().getStreamStatusMaintainer()); watermarkInterval = getExecutionConfig().getAutoWatermarkInterval(); if (watermarkInterval > 0) { final long now = getProcessingTimeService().getCurrentProcessingTime(); getProcessingTimeService().registerTimer(now + watermarkInterval, this); } }
Source:
<code>public Map<String, String> getStartupOptions() { <START> return startupOptionsBuilderRef.get().build(); <END> }</code><technical_language>options rebuilt call. want leave door open in case need option change for new connection</technical_language>

Target:
public Map<String, String> getStartupOptions() { return startupOptionsRef.get(); }
Source:
<code>public void destroy() { final ExecutionMonitor execmon = _wapp != null ? _wapp.getConfiguration().getExecutionMonitor() : null; _rque = null; final ServerPush sp = _spush; if (sp != null) { _spush = null; try { sp.stop(); } catch (Throwable ex) { log.warn("Failed to stop server-push, " + sp, ex); } } try { final List<Page> pages = new ArrayList<Page>(_pages); for (Page page : pages) { try { for (Component root: page.getRoots()) { root.detach(); <START> } <END> ((PageCtrl) page).destroy(); } catch (Throwable ex) { log.warn("Failed to destroy " + page, ex); } } _pages.clear(); } catch (Throwable ex) { log.warn("Failed to clean up pages of " + this, ex); } if (execmon != null) execmon.desktopDestroy(this); _attrs.getAttributes().clear(); _comps = new HashMap<String, Component>(2); _meds = null; final Object lock = getActivationLock(); if (lock != null) { synchronized (lock) { lock.notifyAll(); } } }</code><technical_language>move page destroy function</technical_language>

Target:
public void destroy() { final ExecutionMonitor execmon = _wapp != null ? _wapp.getConfiguration().getExecutionMonitor() : null; _rque = null; final ServerPush sp = _spush; if (sp != null) { _spush = null; try { sp.stop(); } catch (Throwable ex) { log.warn("Failed to stop server-push, " + sp, ex); } } try { final List<Page> pages = new ArrayList<Page>(_pages); for (Page page : pages) { try { ((PageCtrl) page).destroy(); } catch (Throwable ex) { log.warn("Failed to destroy " + page, ex); } } _pages.clear(); } catch (Throwable ex) { log.warn("Failed to clean up pages of " + this, ex); } if (execmon != null) execmon.desktopDestroy(this); _attrs.getAttributes().clear(); _comps = new HashMap<String, Component>(2); _meds = null; final Object lock = getActivationLock(); if (lock != null) { synchronized (lock) { lock.notifyAll(); } } }
Source:
<code>protected void syncSearch() { super.syncSearch(); VdcQueryParametersBase queryParameters = new VdcQueryParametersBase(); queryParameters.setRefresh(getIsQueryFirstTime()); Frontend.getInstance().runQuery(VdcQueryType.GetAllVmsAndVmPools, queryParameters, new AsyncQuery(this, new INewAsyncCallback() { @Override public void onSuccess(Object model, Object returnValue) { UserPortalListModel userPortalListModel <START> = <END> (UserPortalListModel) model; ArrayList<VM> vms = new ArrayList<VM>(); ArrayList<VmPool> pools = new ArrayList<VmPool>(); VdcQueryReturnValue retValue = (VdcQueryReturnValue) returnValue; if (retValue != null && retValue.getSucceeded()) { List<Object> list = (ArrayList<Object>) retValue.getReturnValue(); if (list != null) { for (Object object : list) { if (object instanceof VM) { vms.add((VM) object); } else if (object instanceof VmPool) { pools.add((VmPool) object); } } } } onVmAndPoolLoad(vms, pools); } })); }</code><technical_language>Findbugs is right, this is a DLS indeed, please remove this line</technical_language>

Target:
protected void syncSearch() { super.syncSearch(); VdcQueryParametersBase queryParameters = new VdcQueryParametersBase(); queryParameters.setRefresh(getIsQueryFirstTime()); Frontend.getInstance().runQuery(VdcQueryType.GetAllVmsAndVmPools, queryParameters, new AsyncQuery(this, new INewAsyncCallback() { @Override public void onSuccess(Object model, Object returnValue) { ArrayList<VM> vms = new ArrayList<VM>(); ArrayList<VmPool> pools = new ArrayList<VmPool>(); VdcQueryReturnValue retValue = (VdcQueryReturnValue) returnValue; if (retValue != null && retValue.getSucceeded()) { List<Object> list = (ArrayList<Object>) retValue.getReturnValue(); if (list != null) { for (Object object : list) { if (object instanceof VM) { vms.add((VM) object); } else if (object instanceof VmPool) { pools.add((VmPool) object); } } } } onVmAndPoolLoad(vms, pools); } })); }
Source:
<code>public BaseChangeEvent<T> getNextEvent() throws IOException { String line; while ((((line = response.readBodyLine())) != null)) { watchStream.feedLine(line); <START> if (watchStream.getState().equals(OsWatchStream.HAVE_EVENT)) <END> return ChangeEvent.fromBsonDocument(watchStream.getNextEvent(), documentClass, codecRegistry); if (watchStream.getState().equals(OsWatchStream.HAVE_ERROR)) { response.close(); throw new IOException("Watch stream has error"); } } throw new IOException("Stream closed"); }</code><technical_language>Multiple sequential calls watchStream.getState(). If is changing in a local variable</technical_language>

Target:
public BaseChangeEvent<T> getNextEvent() throws AppException, IOException { while (true) { watchStream.feedLine(response.readBodyLine()); String watchStreamState = watchStream.getState(); if (watchStreamState.equals(OsWatchStream.HAVE_EVENT)) { return ChangeEvent.fromBsonDocument(watchStream.getNextEvent(), documentClass, codecRegistry); } if (watchStreamState.equals(OsWatchStream.HAVE_ERROR)) { response.close(); throw watchStream.getError(); } } }
Source:
<code>public void onResume(ClientContext context) throws FetchException { if(logMINOR) Logger.minor(this, "Restarting SplitFileFetcher from storage..."); boolean resumed = parent instanceof ClientGetter && ((ClientGetter)parent).resumedFetcher(); this.context = context; try { KeySalter salter = getSalter(); raf.onResume(context); this.storage = new SplitFileFetcherStorage(raf, realTimeFlag, this, blockFetchContext, context.random, context.jobRunner, context.getChkFetchScheduler(realTimeFlag).fetchingKeys(), context.ticker, context.memoryLimitedJobRunner, new CRCChecksumChecker(), context.jobRunner.newSalt(), salter, resumed, callbackCompleteViaTruncation != null); } catch (ResumeFailedException e) { raf.free(); Logger.error(this, "Failed to resume storage file: "+e+" for "+raf, e); throw new FetchException(FetchExceptionMode.BUCKET_ERROR, e); } catch (IOException e) { raf.free(); Logger.error(this, "Failed to resume due to I/O error: "+e+" raf = "+raf, e); throw new FetchException(FetchExceptionMode.BUCKET_ERROR, e); } catch (StorageFormatException e) { raf.free(); Logger.error(this, "Failed to resume due to storage error: "+e+" raf = "+raf, e); throw new FetchException(FetchExceptionMode.INTERNAL_ERROR, "Resume failed: "+e, e); } catch (FetchException e) { raf.free(); throw e; } synchronized(this) { lastNotifiedStoreFetch = System.currentTimeMillis(); } getter = new SplitFileFetcherGet(this, storage); if(storage.start(resumed)) <START> getter.schedule(context, storage.hasCheckedStore()); <END> }</code><technical_language>While touching this line, want add braces</technical_language>

Target:
public void onResume(ClientContext context) throws FetchException { if(logMINOR) Logger.minor(this, "Restarting SplitFileFetcher from storage..."); boolean resumed = parent instanceof ClientGetter && ((ClientGetter)parent).resumedFetcher(); this.context = context; try { KeySalter salter = getSalter(); raf.onResume(context); this.storage = new SplitFileFetcherStorage(raf, realTimeFlag, this, blockFetchContext, context.random, context.jobRunner, context.getChkFetchScheduler(realTimeFlag).fetchingKeys(), context.ticker, context.memoryLimitedJobRunner, new CRCChecksumChecker(), context.jobRunner.newSalt(), salter, resumed, callbackCompleteViaTruncation != null); } catch (ResumeFailedException e) { raf.free(); Logger.error(this, "Failed to resume storage file: "+e+" for "+raf, e); throw new FetchException(FetchExceptionMode.BUCKET_ERROR, e); } catch (IOException e) { raf.free(); Logger.error(this, "Failed to resume due to I/O error: "+e+" raf = "+raf, e); throw new FetchException(FetchExceptionMode.BUCKET_ERROR, e); } catch (StorageFormatException e) { raf.free(); Logger.error(this, "Failed to resume due to storage error: "+e+" raf = "+raf, e); throw new FetchException(FetchExceptionMode.INTERNAL_ERROR, "Resume failed: "+e, e); } catch (FetchException e) { raf.free(); throw e; } synchronized(this) { lastNotifiedStoreFetch = System.currentTimeMillis(); } getter = new SplitFileFetcherGet(this, storage); if (storage.start(resumed)) { getter.schedule(context, storage.hasCheckedStore()); } }
Source:
<code>private void buildChildrenEntries(AggregatedCalledFunction threadNode) { FlamegraphDepthEntry threadEntry = new FlamegraphDepthEntry("", 0, fActiveTrace.getEndTime().toNanos() - fActiveTrace.getStartTime().toNanos(), fFlameGraphEntries.size()); <START> fChildrenEntries = new ArrayList<>(); <END> fThreadDuration = 0L; threadNode.getChildren().stream() .sorted(Comparator.comparingLong(AggregatedCalledFunction::getDuration)) .forEach(rootFunction -> { fTimestampStack.push(fThreadDuration); setData(rootFunction); fThreadDuration += rootFunction.getDuration(); fTimestampStack.pop(); }); fChildrenEntries.forEach(child -> { if (child != null) { threadEntry.addChild(child); } }); threadEntry.setName(threadNode.getSymbol().toString()); fFlameGraphEntries.add(threadEntry); }</code><technical_language>this too</technical_language>

Target:
private void buildChildrenEntries(AggregatedCalledFunction threadNode) { FlamegraphDepthEntry threadEntry = new FlamegraphDepthEntry("", 0, fActiveTrace.getEndTime().toNanos() - fActiveTrace.getStartTime().toNanos(), fFlameGraphEntries.size()); List<FlamegraphDepthEntry> childrenEntries = new ArrayList<>(); fThreadDuration = 0L; threadNode.getChildren().stream() .sorted(Comparator.comparingLong(AggregatedCalledFunction::getDuration)) .forEach(rootFunction -> { Deque<Long> timestampStack = new ArrayDeque<>(); timestampStack.push(fThreadDuration); setData(rootFunction, childrenEntries, timestampStack); fThreadDuration += rootFunction.getDuration(); timestampStack.pop(); }); childrenEntries.forEach(child -> { if (child != null) { threadEntry.addChild(child); } }); threadEntry.setName(threadNode.getSymbol().toString()); fFlameGraphEntries.add(threadEntry); }
Source:
<code>public void testRLEOnly() throws Exception { <START> RunLengthBitPackingHybridEncoder encoder = new RunLengthBitPackingHybridEncoder(3, 5, 10, new DirectByteBufferAllocator()); <END> for (int i = 0; i < 100; i++) { encoder.writeInt(4); } for (int i = 0; i < 100; i++) { encoder.writeInt(5); } ByteArrayInputStream is = new ByteArrayInputStream(encoder.toBytes().toByteArray()); assertEquals(200, BytesUtils.readUnsignedVarInt(is)); assertEquals(4, BytesUtils.readIntLittleEndianOnOneByte(is)); assertEquals(200, BytesUtils.readUnsignedVarInt(is)); assertEquals(5, BytesUtils.readIntLittleEndianOnOneByte(is)); assertEquals(-1, is.read()); }</code><technical_language>factor out this line</technical_language>

Target:
public void testRLEOnly() throws Exception { RunLengthBitPackingHybridEncoder encoder = getRunLengthBitPackingHybridEncoder(); for (int i = 0; i < 100; i++) { encoder.writeInt(4); } for (int i = 0; i < 100; i++) { encoder.writeInt(5); } ByteArrayInputStream is = new ByteArrayInputStream(encoder.toBytes().toByteArray()); assertEquals(200, BytesUtils.readUnsignedVarInt(is)); assertEquals(4, BytesUtils.readIntLittleEndianOnOneByte(is)); assertEquals(200, BytesUtils.readUnsignedVarInt(is)); assertEquals(5, BytesUtils.readIntLittleEndianOnOneByte(is)); assertEquals(-1, is.read()); }
Source:
<code>public void getApplicationByCandidateIdShouldReturnAMaximumNumberOfApplicationsGivenByTheSizeParameter() { List<ApplicationEntity> applicationEntities = Arrays.asList(applicationEntity, applicationEntity); List<CandidateApplicationDTO> candidateApplicationDTOs = Arrays.asList(dummyCandidateApplicationDTO, dummyCandidateApplicationDTO); given(candidateRepository.findOne(CANDIDATE_ID)).willReturn(candidateEntity); given(applicationsRepository.findByCandidateEntity(candidateEntity, ZERO_TWO_PAGE_REQUEST)) .willReturn(new PageImpl<ApplicationEntity>(applicationEntities)); given(converterService.convert(applicationEntities, CandidateApplicationDTO.class)) .willReturn(candidateApplicationDTOs); PagingResponse<CandidateApplicationDTO> <START> result = <END> applicationsService .getApplicationsByCandidateId(CANDIDATE_ID, PAGE_NUMBER_ZERO, PAGE_SIZE_TWO); assertThat(result.getRows().size(), is(2)); assertThat(result.getRows(), equalTo(candidateApplicationDTOs)); }</code><technical_language>This variable final</technical_language>

Target:
public void getApplicationByCandidateIdShouldReturnAMaximumNumberOfApplicationsGivenByTheSizeParameter() { final List<ApplicationEntity> applicationEntities = Arrays.asList(applicationEntity, applicationEntity); final List<CandidateApplicationDTO> candidateApplicationDTOs = Arrays.asList(dummyCandidateApplicationDTO, dummyCandidateApplicationDTO); given(candidateRepository.findOne(CANDIDATE_ID)).willReturn(candidateEntity); given(applicationsRepository.findByCandidateEntity(candidateEntity, ZERO_TWO_PAGE_REQUEST)) .willReturn(new PageImpl<ApplicationEntity>(applicationEntities)); given(converterService.convert(applicationEntities, CandidateApplicationDTO.class)) .willReturn(candidateApplicationDTOs); final PagingResponse<CandidateApplicationDTO> result = applicationsService .getApplicationsByCandidateId(CANDIDATE_ID, PAGE_NUMBER_ZERO, PAGE_SIZE_TWO); assertThat(result.getRows().size(), is(2)); assertThat(result.getRows(), equalTo(candidateApplicationDTOs)); }
Source:
<code>private void updatePositionLeaderNoValidation(Metadata.LeaderAndEpoch currentLeaderAndEpoch) { if (position != null && !position.currentLeader.equals(currentLeaderAndEpoch)) { <START> FetchPosition newPosition = new FetchPosition(position.offset, position.offsetEpoch, currentLeaderAndEpoch); <END> transitionState(FetchStates.FETCHING, () -> { this.position = newPosition; this.nextRetryTimeMs = null; }); } }</code><technical_language>nit: newPosition created lazy</technical_language>

Target:
private void updatePositionLeaderNoValidation(Metadata.LeaderAndEpoch currentLeaderAndEpoch) { if (position != null) { FetchPosition newPosition = new FetchPosition(position.offset, position.offsetEpoch, currentLeaderAndEpoch); transitionState(FetchStates.FETCHING, () -> { this.position = newPosition; this.nextRetryTimeMs = null; }); } }
Source:
<code>public void setup(final IDocument document, IPath location, LocationKind locationKind) { if (document == null) { return; } Job job = new Job("Initialize Language Servers for " + location.toFile().getName()) { @Override protected IStatus run(IProgressMonitor monitor) { LanguageServiceAccessor.getLanguageServers(document, capabilities -> true); <START> return Status.OK_STATUS; <END> } }; job.setUser(true); job.setPriority(Job.INTERACTIVE); job.schedule(100); locationMap.put(location, job); }</code><technical_language>returning, stop monitoring buffer locationMap.remove(location);</technical_language>

Target:
public void setup(final IDocument document, IPath location, LocationKind locationKind) { if (document == null) { return; } Job job = new Job("Initialize Language Servers for " + location.toFile().getName()) { @Override protected IStatus run(IProgressMonitor monitor) { ITextFileBuffer buffer = ITextFileBufferManager.DEFAULT.getTextFileBuffer(document); if (buffer == null || buffer.getLocation() == null) { return Status.OK_STATUS; } if (monitor.isCanceled()) { return Status.CANCEL_STATUS; } LanguageServiceAccessor.getLanguageServers(document, capabilities -> true); locationMap.remove(location); return Status.OK_STATUS; } }; job.setUser(true); job.setPriority(Job.INTERACTIVE); locationMap.put(location, job); job.schedule(100); }
Source:
<code>private void appendValuesToBuffer() { <START> verify(decodedBlock != null); <END> ensureValueBufferSize(); if (decodedBlock.mayHaveNull()) { appendLongsWithNullsToBuffer(); } else { appendLongsToBuffer(); } }</code><technical_language>this call is a duplicate of in copyValues; remove</technical_language>

Target:
private void appendValuesToBuffer() { ensureValuesBufferCapacity(); int[] positions = getPositions(); if (decodedBlock.mayHaveNull()) { for (int i = positionsOffset; i < positionsOffset + batchSize; i++) { int position = positions[i]; int newIndex = setLong(valuesBuffer, valuesBufferIndex, decodedBlock.getLong(position)); if (!decodedBlock.isNull(position)) { valuesBufferIndex = newIndex; } } } else { for (int i = positionsOffset; i < positionsOffset + batchSize; i++) { valuesBufferIndex = setLong(valuesBuffer, valuesBufferIndex, decodedBlock.getLong(positions[i])); } } }
Source:
<code><START> boolean hasMappingErrors(String vreName) { <END> return getRawCollectionsTraversal(vreName).outE(HAS_NEXT_ERROR).hasNext(); }</code><technical_language>This public</technical_language>

Target:
public boolean hasMappingErrors(String vreName) { return getRawCollectionsTraversal(vreName).outE(HAS_NEXT_ERROR).hasNext(); }
Source:
<code>public void define(NewController context) { NewAction action = context.createAction("groups") .setDescription("Lists the groups a user belongs to. Requires Administer System permission.") .setHandler(this) .setResponseExample(getClass().getResource("groups-example.json")) .addSelectionModeParam() .addSearchQuery("users", "group names") .addPagingParams(25) <START> .setChangelog(new Change("6.4", "Paging response fields as been moved under a 'paging' node")) <END> .setSince("5.2"); action.createParam(PARAM_LOGIN) .setDescription("A user login") .setExampleValue("admin") .setRequired(true); }</code><technical_language>Paging response fields moved a Paging object Paging response fields moved a Paging object</technical_language>

Target:
public void define(NewController context) { NewAction action = context.createAction("groups") .setDescription("Lists the groups a user belongs to. Requires Administer System permission.") .setHandler(this) .setResponseExample(getClass().getResource("groups-example.json")) .addSelectionModeParam() .addSearchQuery("users", "group names") .addPagingParams(25) .setChangelog(new Change("6.4", "Paging response fields moved to a Paging object")) .setSince("5.2"); action.createParam(PARAM_LOGIN) .setDescription("A user login") .setExampleValue("admin") .setRequired(true); action.createParam(PARAM_ORGANIZATION) .setDescription("Organization key") .setExampleValue("my-org") .setInternal(true) .setSince("6.4"); }