auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code><START> private static long saturatedMult(long left, long right) { <END> long retVal = left * right; if (left != 0 && (Long.MAX_VALUE / left < right)) { return Long.MAX_VALUE; } return retVal; }</code><technical_language>This method saturate MIN_VALUE overflowing negative side. In case trace dinosaurs</technical_language>

Target:
private static long saturatedMult(long left, long right) { long retVal = left * right; if ((left != 0) && ((retVal / left) != right)) { return (sameSign(left, right) ? Long.MAX_VALUE : Long.MIN_VALUE); } return retVal; }
Source:
<code>public ContractsGrantsCollectionActivityDocument() { super(); invoiceDetails = new ArrayList<ContractsGrantsCollectionActivityInvoiceDetail>(); <START> collectionEvents = new ArrayList<CollectionEvent>(); <END> }</code><technical_language>This collection removed now, right</technical_language>

Target:
public ContractsGrantsCollectionActivityDocument() { super(); invoiceDetails = new ArrayList<ContractsGrantsCollectionActivityInvoiceDetail>(); }
Source:
<code>public void bulkAbort(List<ProcessInstanceSummary> processInstances) { if (processInstances == null || processInstances.isEmpty()) { return; } final Map<String, List<Long>> containerInstances = new HashMap<>(); for (ProcessInstanceSummary selected : processInstances) { if (selected.getState() != ProcessInstance.STATE_ACTIVE) { view.displayNotification(constants.Aborting_Process_Instance_Not_Allowed(selected.getId())); continue; } containerInstances.computeIfAbsent(selected.getDeploymentId(), key -> new ArrayList<>()); <START> containerInstances.get(selected.getDeploymentId()).add(selected.getProcessInstanceId()); <END> view.displayNotification(constants.Aborting_Process_Instance(selected.getId())); } if (containerInstances.size() > 0) { abortProcessInstances(containerInstances); } }</code><technical_language>this: containerInstances.computeIfAbsent(selected.getDeploymentId(), key -> new ArrayList<>()).add(selected.getProcessInstanceId());</technical_language>

Target:
public void bulkAbort(List<ProcessInstanceSummary> processInstances) { if (processInstances == null || processInstances.isEmpty()) { return; } final Map<String, List<Long>> containerInstances = new HashMap<>(); for (ProcessInstanceSummary selected : processInstances) { if (selected.getState() != ProcessInstance.STATE_ACTIVE) { view.displayNotification(constants.Aborting_Process_Instance_Not_Allowed(selected.getId())); continue; } containerInstances.computeIfAbsent(selected.getDeploymentId(), key -> new ArrayList<>()).add(selected.getProcessInstanceId()); view.displayNotification(constants.Aborting_Process_Instance(selected.getId())); } if (containerInstances.size() > 0) { abortProcessInstances(containerInstances); } }
Source:
<code>public boolean visit(@Nonnull JMethodCall call) { Collection<JMethod> targets = call.getMethodId().getMethods(); if (targets.size() != 1) { return false; } JMethod target = targets.iterator().next(); if (target.isAbstract() || target.isNative()) { return false; } boolean foundForceInline = false; <START> for (JAnnotation annotation : target.getAnnotations()) { if (formatter.getName(annotation.getType()).equals(FORCE_INLINE_ANNOTATION)) { foundForceInline = true; break; } } <END> if (foundForceInline) { call.addMarker(new InlineMarker(target)); } return true; }</code><technical_language>nit: a good candidate for a helper method/class (that reused other schedulables)</technical_language>

Target:
public boolean visit(@Nonnull JMethodCall call) { Collection<JMethod> targets = call.getMethodId().getMethods(); if (targets.size() != 1) { return false; } JMethod target = targets.iterator().next(); if (target.isAbstract() || target.isNative()) { return false; } for (JAnnotation annotation : target.getAnnotations()) { if (formatter.getName(annotation.getType()).equals(FORCE_INLINE_ANNOTATION)) { JNameValuePair removeAfterValuePair = annotation.getNameValuePair(REMOVE_AFTER_ANNOTATION); if (removeAfterValuePair != null && removeAfterValuePair.getValue() instanceof JBooleanLiteral && ((JBooleanLiteral) removeAfterValuePair.getValue()).getValue() && target.getMarker(RemoveInlinedMarker.class) == null) { target.addMarker(new RemoveInlinedMarker()); } call.addMarker(new InlineMarker(target)); break; } } return true; }
Source:
<code>public void refresh() { super.refresh(); Object value = this.target.eGet(this.eReference); if (value instanceof EObject) { Adapter adapter = this.composedAdapterFactory.adapt((EObject) value, IItemLabelProvider.class); if (adapter instanceof IItemLabelProvider) { IItemLabelProvider labelProvider = (IItemLabelProvider) adapter; this.text.setText(labelProvider.getText(value)); this.image.setImage(ExtendedImageRegistry.INSTANCE.getImage(labelProvider.getImage(value))); <START> getOnClickExpression().ifPresent(expression -> { this.setStyle(); }); <END> } } else if (value == null) { this.image.setImage(null); this.text.setText(Messages.SingleReference_noValue); } }</code><technical_language>If want a StyledText, want override org.eclipse.eef.ide.ui.api.widgets.AbstractEEFWidgetLifecycleManager.setLabelFontStyle() instead</technical_language>

Target:
public void refresh() { super.refresh(); Object value = this.target.eGet(this.eReference); if (value instanceof EObject) { Adapter adapter = this.composedAdapterFactory.adapt((EObject) value, IItemLabelProvider.class); if (adapter instanceof IItemLabelProvider) { IItemLabelProvider labelProvider = (IItemLabelProvider) adapter; this.text.setText(labelProvider.getText(value)); this.text.setUnderlined(false); this.image.setImage(ExtendedImageRegistry.INSTANCE.getImage(labelProvider.getImage(value))); String onClickExpression = Optional.ofNullable(this.description.getOnClickExpression()).orElse(""); if (!onClickExpression.isEmpty()) { this.text.setUnderlined(true); } } } else if (value == null) { this.image.setImage(null); this.text.setText(Messages.SingleReference_noValue); } }
Source:
<code>public void createPartControl(Composite parent) { String name = getViewSite().getSecondaryId(); if (name != null) { name = TmfViewFactory.getBaseSecId(name); } if (name != null) { fViewInfo.setName(name); } <START> fPresentationProvider = new XmlPresentationProvider(name); <END> super.createPartControl(parent); }</code><technical_language>need call addColorSettingsListener new instance</technical_language>

Target:
public void createPartControl(Composite parent) { String name = getViewSite().getSecondaryId(); if (name != null) { name = TmfViewFactory.getBaseSecId(name); } if (name != null) { fViewInfo.setName(name); } fPresentationProvider = new XmlPresentationProvider(name); TimeGraphViewer timeGraphViewer = getTimeGraphViewer(); timeGraphViewer.getTimeGraphControl().colorSettingsChanged(timeGraphViewer.getTimeGraphProvider().getStateTable()); super.createPartControl(parent); }
Source:
<code>public CalDavConfig(String key, String username, String password, String url, int reloadMinutes, int historicLoadMinutes) { <START> super(); <END> this.key = key; this.username = username; this.password = password; this.url = url; this.reloadMinutes = reloadMinutes; this.historicLoadMinutes = historicLoadMinutes; }</code><technical_language>CalDavConfig super class (except of java.lang.Object). Do need a super() here</technical_language>

Target:
public CalDavConfig(String key, String username, String password, String url, int reloadMinutes, int historicLoadMinutes) { this.key = key; this.username = username; this.password = password; this.url = url; this.reloadMinutes = reloadMinutes; this.historicLoadMinutes = historicLoadMinutes; }
Source:
<code>public void search( @RequestParam("q") String query, @RequestParam(value = "type", required = false) Set<String> types, HttpServletRequest request, HttpServletResponse response) throws IOException { final Map<String, List<?>> searchResults = new TreeMap<>(); if (logger.isDebugEnabled()) { logger.debug( <START> "Searching with q={}, type={}", query, ArrayUtils.toString(types.toArray())); <END> } for (ISearchStrategy strategy : searchStrategies) { if (types == null || types.isEmpty() || types.contains("") || types.contains(strategy.getResultTypeName())) { searchResults.put(strategy.getResultTypeName(), strategy.search(query, request)); } } if (searchResults.isEmpty()) { logger.debug("Nothing found for query string: {}", query); response.setStatus(HttpServletResponse.SC_NOT_FOUND); } else { response.setHeader(HttpHeaders.CONTENT_TYPE, MediaType.APPLICATION_JSON_VALUE); jsonMapper.writeValue(response.getOutputStream(), searchResults); } }</code><technical_language>some of this logging code is unnecessary. I work nicely... logger.debug("Searching q={}, type={}", query, types); it, if ArrayUtils.toString(types.toArray()) part is necessary</technical_language>

Target:
public void search( @RequestParam("q") String query, @RequestParam(value = "type", required = false) Set<String> types, HttpServletRequest request, HttpServletResponse response) throws IOException { logger.debug("Searching with q={}, type={}", query, types); final Map<String, List<?>> searchResults = new TreeMap<>(); for (ISearchStrategy strategy : searchStrategies) { if (types == null || types.isEmpty() || types.contains("") || types.contains(strategy.getResultTypeName())) { searchResults.put(strategy.getResultTypeName(), strategy.search(query, request)); } } if (searchResults.isEmpty()) { logger.debug("Nothing found for query string: {}", query); response.setStatus(HttpServletResponse.SC_NOT_FOUND); } else { response.setHeader(HttpHeaders.CONTENT_TYPE, MediaType.APPLICATION_JSON_VALUE); jsonMapper.writeValue(response.getOutputStream(), searchResults); } }
Source:
<code>public int setCloseTime(int childSequenceNumber, long endTime) { for (int i = 0; i < nbChildren; i++) { <START> if (childSequenceNumber == this.children[i]) { <END> childEnd[i] = endTime; return i; } } return -1; }</code><technical_language>need for "this"</technical_language>

Target:
public int setCloseTime(int childSequenceNumber, long endTime) { for (int i = 0; i < nbChildren; i++) { if (childSequenceNumber == children[i]) { childEnd[i] = endTime; return i; } } return -1; }
Source:
<code>private void join(EntityBareJid mucJid) throws SmackException.NotConnectedException, SmackException.NoResponseException, InterruptedException, XMPPException.XMPPErrorException, MultiUserChatException.MucAlreadyJoinedException, MultiUserChatException.NotAMucServiceException { <START> lastPresenceSent = null; <END> if (muc != null) { muc.removePresenceInterceptor(presenceInterceptor); logger.info("Leaving a MUC we already occupy."); muc.leave(); } MultiUserChatManager mucManager = MultiUserChatManager.getInstanceFor(xmppConnection); muc = mucManager.getMultiUserChat(mucJid); if (presenceInterceptor != null) { muc.addPresenceInterceptor(presenceInterceptor); } muc.createOrJoin(mucNickname); logger.info("Joined MUC: " + mucJid); setPresenceExtensions(mucClientManager.getPresenceExtensions()); }</code><technical_language>do want reset this if a rejoin</technical_language>

Target:
private void join(EntityBareJid mucJid) throws SmackException.NotConnectedException, SmackException.NoResponseException, InterruptedException, XMPPException.XMPPErrorException, MultiUserChatException.MucAlreadyJoinedException, MultiUserChatException.NotAMucServiceException { resetLastPresenceSent(); if (muc != null) { muc.removePresenceInterceptor(presenceInterceptor); logger.info("Leaving a MUC we already occupy."); muc.leave(); } MultiUserChatManager mucManager = MultiUserChatManager.getInstanceFor(xmppConnection); muc = mucManager.getMultiUserChat(mucJid); if (presenceInterceptor != null) { muc.addPresenceInterceptor(presenceInterceptor); } muc.createOrJoin(mucNickname); logger.info("Joined MUC: " + mucJid); setPresenceExtensions(mucClientManager.getPresenceExtensions()); }
Source:
<code>private static List<AnnotationRange> prepareAnnotationRanges( int start, int end, @Nullable Editor editor, @NotNull User user, @NotNull IFile file) { List<AnnotationRange> annotationRanges = new ArrayList<>(); TextAttributes contributionTextAttributes; if (editor != null) { contributionTextAttributes = getContributionTextAttributes(editor, user); } else { contributionTextAttributes = null; } <START> for (int i = 0; i < end - start; i++) { int currentStart = start + i; int currentEnd = start + i + 1; <END> AnnotationRange annotationRange; if (editor != null) { RangeHighlighter rangeHighlighter = addRangeHighlighter(currentStart, currentEnd, editor, contributionTextAttributes, file); if (rangeHighlighter == null) { throw new IllegalStateException( "Failed to create range highlighter for range (" + currentStart + "," + currentEnd + ") for file " + file); } annotationRange = new AnnotationRange(currentStart, currentEnd, rangeHighlighter); } else { annotationRange = new AnnotationRange(currentStart, currentEnd); } annotationRanges.add(annotationRange); } return annotationRanges; }</code><technical_language>iterating start -> end more intuitive. suggestion for (int currentStart = start; currentStart < end; currentStart++) { int currentEnd = currentStart + 1;</technical_language>

Target:
private static List<AnnotationRange> prepareAnnotationRanges( int start, int end, @Nullable Editor editor, @NotNull User user, @NotNull IFile file) { checkRange(start, end); List<AnnotationRange> annotationRanges = new ArrayList<>(); TextAttributes contributionTextAttributes; if (editor != null) { contributionTextAttributes = getContributionTextAttributes(editor, user); } else { contributionTextAttributes = null; } for (int currentStart = start; currentStart < end; currentStart++) { int currentEnd = currentStart + 1; AnnotationRange annotationRange; if (editor != null) { RangeHighlighter rangeHighlighter = addRangeHighlighter(currentStart, currentEnd, editor, contributionTextAttributes, file); if (rangeHighlighter == null) { throw new IllegalStateException( "Failed to create range highlighter for range (" + currentStart + "," + currentEnd + ") for file " + file); } annotationRange = new AnnotationRange(currentStart, currentEnd, rangeHighlighter); } else { annotationRange = new AnnotationRange(currentStart, currentEnd); } annotationRanges.add(annotationRange); } return annotationRanges; }
Source:
<code>public long count(String name, String namePattern, Optional<HivePrincipal> principal) { CriteriaBuilder cb = criteriaBuilder(); CriteriaQuery<DeviceType> criteria = cb.createQuery(DeviceType.class); Root<DeviceType> from = criteria.from(DeviceType.class); Predicate[] nameAndPrincipalPredicates = CriteriaHelper.deviceTypeListPredicates(cb, from, ofNullable(name), ofNullable(namePattern), principal); criteria.where(nameAndPrincipalPredicates); TypedQuery<DeviceType> query = createQuery(criteria); <START> return query.getResultList().size(); <END> }</code><technical_language>Change getResultList().size() predicate count</technical_language>

Target:
public long count(String name, String namePattern, Optional<HivePrincipal> principal) { CriteriaBuilder cb = criteriaBuilder(); CriteriaQuery<Long> criteria = cb.createQuery(Long.class); Root<DeviceType> from = criteria.from(DeviceType.class); Predicate[] nameAndPrincipalPredicates = CriteriaHelper.deviceTypeListPredicates(cb, from, ofNullable(name), ofNullable(namePattern), principal); criteria.where(nameAndPrincipalPredicates); criteria.select(cb.count(from)); return count(criteria); }
Source:
<code>public void setUp() throws Exception { Calcites.setSystemProperties(); new ApproximateHistogramDruidModule().configure(null); final QueryableIndex index = IndexBuilder.create() .tmpDir(temporaryFolder.newFolder()) .indexMerger(TestHelper.getTestIndexMergerV9()) .schema( new IncrementalIndexSchema.Builder() .withMetrics( new CountAggregatorFactory("cnt"), new DoubleSumAggregatorFactory("m1", "m1"), new ApproximateHistogramAggregatorFactory( "hist_m1", "m1", null, null, null, null ) ) .withRollup(false) .build() ) .rows(CalciteTests.ROWS1) .buildMMappedIndex(); walker = new SpecificSegmentsQuerySegmentWalker(CalciteTests.queryRunnerFactoryConglomerate()).add( DataSegment.builder() .dataSource(DATA_SOURCE) .interval(index.getDataInterval()) .version("1") .shardSpec(new LinearShardSpec(0)) .build(), index ); final PlannerConfig plannerConfig = new PlannerConfig(); final DruidSchema druidSchema = CalciteTests.createMockSchema(walker, plannerConfig); final DruidOperatorTable operatorTable = new DruidOperatorTable( ImmutableSet.of(new QuantileSqlAggregator()), ImmutableSet.of() ); final Map<String, Authenticator> defaultMap = Maps.newHashMap(); defaultMap.put("allowAll", new AllowAllAuthenticator()); plannerFactory = new PlannerFactory( druidSchema, CalciteTests.createMockQueryLifecycleFactory(walker), operatorTable, CalciteTests.createExprMacroTable(), plannerConfig, new AuthConfig(), <START> new AuthenticatorMapper(defaultMap, "allowAll"), <END> new AuthorizerMapper(null) { @Override public Authorizer getAuthorizer(String name) { return new AllowAllAuthorizer(); } }, CalciteTests.getJsonMapper() ); }</code><technical_language>This AuthenticatorMapper AuthorizerMapper in TestHelper some similar test-utility class</technical_language>

Target:
public void setUp() throws Exception { Calcites.setSystemProperties(); new ApproximateHistogramDruidModule().configure(null); final QueryableIndex index = IndexBuilder.create() .tmpDir(temporaryFolder.newFolder()) .indexMerger(TestHelper.getTestIndexMergerV9()) .schema( new IncrementalIndexSchema.Builder() .withMetrics( new CountAggregatorFactory("cnt"), new DoubleSumAggregatorFactory("m1", "m1"), new ApproximateHistogramAggregatorFactory( "hist_m1", "m1", null, null, null, null ) ) .withRollup(false) .build() ) .rows(CalciteTests.ROWS1) .buildMMappedIndex(); walker = new SpecificSegmentsQuerySegmentWalker(CalciteTests.queryRunnerFactoryConglomerate()).add( DataSegment.builder() .dataSource(DATA_SOURCE) .interval(index.getDataInterval()) .version("1") .shardSpec(new LinearShardSpec(0)) .build(), index ); final PlannerConfig plannerConfig = new PlannerConfig(); final DruidSchema druidSchema = CalciteTests.createMockSchema(walker, plannerConfig); final DruidOperatorTable operatorTable = new DruidOperatorTable( ImmutableSet.of(new QuantileSqlAggregator()), ImmutableSet.of() ); plannerFactory = new PlannerFactory( druidSchema, CalciteTests.createMockQueryLifecycleFactory(walker), operatorTable, CalciteTests.createExprMacroTable(), plannerConfig, new AuthConfig(), AuthTestUtils.TEST_AUTHENTICATOR_MAPPER, AuthTestUtils.TEST_AUTHORIZER_MAPPER, CalciteTests.getJsonMapper() ); }
Source:
<code>public void setConstraint(String constr) { <START> if (!isValidConstraint(constr)) <END> throw new WrongValueException("Invalid constraint: " + constr + " (the constraint should be formatted in yyyyMMdd)"); setConstraint(constr != null ? new SimpleDateConstraint(constr) : null); }</code><technical_language>put in SimpleDateConstraint</technical_language>

Target:
public void setConstraint(String constr) { setConstraint(constr != null ? new SimpleDateConstraint(constr) : null); }
Source:
<code>public static void main(String[] args) throws Exception { ConfigurableApplicationContext context = SpringApplication.run(Application.class, args); <START> MessageChannel channelRestoutput = context.getBean("toSplunk", MessageChannel.class); <END> sendWithRest(channelRestoutput); System.out.println("Consumer"); }</code><technical_language>Minor, of course, good names for variables. I proper camel style - channelRestOutput</technical_language>

Target:
public static void main(String[] args) throws Exception { ConfigurableApplicationContext context = SpringApplication.run(Application.class, args); MessageChannel channelRestOutput = context.getBean("toSplunk", MessageChannel.class); sendWithRest(channelRestOutput); System.out.println("Consumer"); }
Source:
<code>public void setup(OperatorContext context) { try { if (jsonSchema != null) { JsonSchemaFactory factory = JsonSchemaFactory.byDefault(); JsonNode schemaNode = JsonLoader.fromString(jsonSchema); schema = factory.getJsonSchema(schemaNode); } <START> objMapper = new ObjectMapper(); <END> } catch (ProcessingException | IOException e) { DTThrowable.wrapIfChecked(e); } }</code><technical_language>default object mapper check if fields in json in class, if find mapping fail parsing. need relax this condition setting mapper.configure(DeserializationConfig.Feature.FAIL_ON_UNKNOWN_PROPERTIES, false); This allow users pojo number of fields present in json. This user extract interested fields json</technical_language>

Target:
public void setup(OperatorContext context) { try { if (jsonSchema != null) { JsonSchemaFactory factory = JsonSchemaFactory.byDefault(); JsonNode schemaNode = JsonLoader.fromString(jsonSchema); schema = factory.getJsonSchema(schemaNode); } objMapper = new ObjectMapper(); objMapper.disable(DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES); } catch (ProcessingException | IOException e) { DTThrowable.wrapIfChecked(e); } }
Source:
<code>private Chunk allocateChunk() { <START> SoftReference<Chunk> chunkRef = chunksCache.pollLast(); <END> while (chunkRef != null && chunkRef.get() == null) { chunkRef = chunksCache.pollLast(); } Chunk chunk = chunkRef == null ? null : chunkRef.get(); if (chunk == null) { chunk = new Chunk(this.defaultChunkSize); } else { chunk.cursor = 0; chunk.startIndex = 0; chunk.chunkSize = chunk.keys.length; } return chunk; }</code><technical_language>do simplifying this method to: java while (true) { final SoftReference<Chunk> chunkRef = chunksCache.pollLast(); if (chunkRef == null) { return new Chunk(defaultChunkSize); } final Chunk chunk = chunkRef.get(); if (chunk != null) { chunk.cursor = 0; chunk.startIndex = 0; chunk.chunkSize = chunk.keys.length; return chunk; } }</technical_language>

Target:
private Chunk allocateChunk() { while (true) { final SoftReference<Chunk> chunkRef = chunksCache.pollLast(); if (chunkRef == null) { return new Chunk(defaultChunkSize); } final Chunk chunk = chunkRef.get(); if (chunk != null) { chunk.cursor = 0; chunk.startIndex = 0; chunk.chunkSize = chunk.keys.length; return chunk; } } }
Source:
<code>public void testMask() { BlockBuilder blockBuilder = BIGINT.createBlockBuilder(new BlockBuilderStatus(), 10); blockBuilder.appendNull(); blockBuilder.writeLong(42L); blockBuilder.appendNull(); blockBuilder.writeLong(43L); blockBuilder.appendNull(); <START> Block result = blockBuilder.mask(new int[] {0, 1, 1, 1, 4}); <END> assertEquals(result.getPositionCount(), 5); assertTrue(result.isNull(0)); assertEquals(BIGINT.getLong(result, 1), 42L); assertEquals(BIGINT.getLong(result, 2), 42L); assertEquals(BIGINT.getLong(result, 3), 42L); assertTrue(result.isNull(4)); for (int position : ImmutableList.of(-1, 6)) { try { blockBuilder.mask(new int[] {position}).getLong(0, 0); fail("Expected to fail"); } catch (IllegalArgumentException e) { assertTrue(e.getMessage().startsWith("position is not valid")); } } Block block = blockBuilder.build(); result = block.mask(new int[] {0, 1, 1, 1, 4}); assertEquals(result.getPositionCount(), 5); assertTrue(result.isNull(0)); assertEquals(BIGINT.getLong(result, 1), 42L); assertEquals(BIGINT.getLong(result, 2), 42L); assertEquals(BIGINT.getLong(result, 3), 42L); assertTrue(result.isNull(4)); for (int position : ImmutableList.of(-1, 6)) { try { block.mask(new int[] {position}).getLong(0, 0); fail("Expected to fail"); } catch (IllegalArgumentException e) { assertTrue(e.getMessage().startsWith("position is not valid")); } } }</code><technical_language>extract this a method share code</technical_language>

Target:
public void testMask() { BlockBuilder blockBuilder = BIGINT.createFixedSizeBlockBuilder(5).appendNull().writeLong(42L).appendNull().writeLong(43L).appendNull(); int[] positions = new int[] {0, 1, 1, 1, 4}; Block expected = BIGINT.createFixedSizeBlockBuilder(5).appendNull().writeLong(42).writeLong(42).writeLong(42).appendNull().build(); assertBlockEquals(BIGINT, blockBuilder.getPositions(positions), expected); assertInvalidMask(blockBuilder, new int[] {-1}); assertInvalidMask(blockBuilder, new int[] {6}); Block block = blockBuilder.build(); assertBlockEquals(BIGINT, block.getPositions(positions), expected); assertInvalidMask(block, new int[] {-1}); assertInvalidMask(block, new int[] {6}); }
Source:
<code>void showList(JsArray<BranchInfo> result) { <START> if (open && (result.length() != 0)) { Gerrit.display(PageLinks.toProjectBranches(getProjectKey())); return; } <END> setToken(getTokenForScreen(qMatch, qStart)); ProjectBranchesScreen.this.match = qMatch; ProjectBranchesScreen.this.start = qStart; if (result.length() <= pageSize) { branchTable.display(Natives.asList(result)); next.setVisible(false); } else { branchTable.displaySubset(Natives.asList(result), 0, result.length() - 1); setupNavigationLink(next, qMatch, qStart + pageSize); } if (qStart > 0) { setupNavigationLink(prev, qMatch, qStart - pageSize); } else { prev.setVisible(false); } delBranch.setVisible(branchTable.hasBranchCanDelete()); Set<String> checkedRefs = branchTable.getCheckedRefs(); branchTable.setChecked(checkedRefs); updateForm(); if (!isCurrentView()) { display(); } }</code><technical_language>This sense here. In project list projects links navigate project screen. In project list idea filter reacting ENTER key is type filter find project (first in list), allow open pressing ENTER, switching mouse clicking result. situation is different. branches in list clickable, is open. This code reloads project branches screen match skip info is lost. This is confusing. branches clickable, I remove open logic this class</technical_language>

Target:
void showList(JsArray<BranchInfo> result) { setToken(getTokenForScreen(qMatch, qStart)); ProjectBranchesScreen.this.match = qMatch; ProjectBranchesScreen.this.start = qStart; if (result.length() <= pageSize) { branchTable.display(Natives.asList(result)); next.setVisible(false); } else { branchTable.displaySubset(Natives.asList(result), 0, result.length() - 1); setupNavigationLink(next, qMatch, qStart + pageSize); } if (qStart > 0) { setupNavigationLink(prev, qMatch, qStart - pageSize); } else { prev.setVisible(false); } delBranch.setVisible(branchTable.hasBranchCanDelete()); Set<String> checkedRefs = branchTable.getCheckedRefs(); branchTable.setChecked(checkedRefs); updateForm(); if (!isCurrentView()) { display(); } }
Source:
<code>protected IProject[] build(int kind, Map<String, String> args, final IProgressMonitor monitor) throws CoreException { IResourceDelta delta = getDelta(getProject()); if (delta == null) return null; delta.accept(delta1 -> { IPath ipath = delta1.getFullPath(); if (!"java".equals(ipath.getFileExtension())) return true; String path = ipath.toPortableString(); for (int i = 0; i < classes.length; i++) { if (classes[i].build) continue; if (path.startsWith(classes[i].sourceDir)) { classes[i].build = true; } } return true; }); final IWorkspaceRoot root = getProject().getWorkspace().getRoot(); for (MainClass <START> classe : <END> classes) { MainClass mainClass = classe; if (mainClass.build) { mainClass.build = false; IResource library = root.findMember(mainClass.outputDir); JNIGeneratorApp gen = new JNIGeneratorApp(); gen.setMainClassName(mainClass.mainClassName, library.getLocation().toPortableString()); gen.generate(); library.refreshLocal(IResource.DEPTH_INFINITE, null); } } return null; }</code><technical_language>rename clazz</technical_language>

Target:
protected IProject[] build(int kind, Map<String, String> args, final IProgressMonitor monitor) throws CoreException { IResourceDelta delta = getDelta(getProject()); if (delta == null) return null; delta.accept(delta1 -> { IPath ipath = delta1.getFullPath(); if (!"java".equals(ipath.getFileExtension())) return true; String path = ipath.toPortableString(); for (int i = 0; i < classes.length; i++) { if (classes[i].build) continue; if (path.startsWith(classes[i].sourceDir)) { classes[i].build = true; } } return true; }); final IWorkspaceRoot root = getProject().getWorkspace().getRoot(); for (MainClass mainClass : classes) { if (mainClass.build) { mainClass.build = false; IResource library = root.findMember(mainClass.outputDir); JNIGeneratorApp gen = new JNIGeneratorApp(); gen.setMainClassName(mainClass.mainClassName, library.getLocation().toPortableString()); gen.generate(); library.refreshLocal(IResource.DEPTH_INFINITE, null); } } return null; }
Source:
<code>public void roundtrip() throws IOException { when( storageFileItem.getInputStream() ).thenReturn( new ByteArrayInputStream( prefixFile1( true ).getBytes( UTF8 ) ) ); final List<String> entries = m.read( storageFileItem ).entries(); assertThat( entries, is( notNullValue() ) ); assertThat( entries.size(), is( 3 ) ); final ByteArrayOutputStream outputStream = new ByteArrayOutputStream(); m.write( entries, outputStream ); assertThat( outputStream.size(), greaterThan( 15 ) ); final String output = new String( outputStream.toByteArray(), UTF8 ); <START> assertThat( output.replace("\r", ""), equalTo( withStandardHeaders( prefixFile1( false ) ).replace("\r", "") ) ); <END> }</code><technical_language>special reason strip carriage returns now</technical_language>

Target:
public void roundtrip() throws IOException { when( storageFileItem.getInputStream() ).thenReturn( new ByteArrayInputStream( prefixFile1( true ).getBytes( UTF8 ) ) ); final List<String> entries = m.read( storageFileItem ).entries(); assertThat( entries, is( notNullValue() ) ); assertThat( entries.size(), is( 3 ) ); final ByteArrayOutputStream outputStream = new ByteArrayOutputStream(); m.write( entries, outputStream ); assertThat( outputStream.size(), greaterThan( 15 ) ); final String output = new String( outputStream.toByteArray(), UTF8 ); assertThat( output, equalTo( withStandardHeaders( prefixFile1( false ) ) ) ); }
Source:
<code>protected void selectionUpdatedHandler(SelectionUpdatedEvent event) { ListGridRecord[] selectedRecords = this.getSelectedRecords(); ArrayList<String> selectedTasksIds = new ArrayList<>(); for (ListGridRecord selectedRecord : selectedRecords) { Task task = TaskRecord.getTask(selectedRecord); selectedTasksIds.add(task.getJobId() + "_" + task.getId()); <START> } <END> controller.getModel().setSelectedTasksIds(selectedTasksIds); }</code><technical_language>nice a method returns: task.getJobId() + "_" + task.getId()</technical_language>

Target:
protected void selectionUpdatedHandler(SelectionUpdatedEvent event) { ListGridRecord[] selectedRecords = this.getSelectedRecords(); List<String> selectedTasksIds = new ArrayList<>(selectedRecords.length); for (ListGridRecord selectedRecord : selectedRecords) { Task task = TaskRecord.getTask(selectedRecord); selectedTasksIds.add(task.getJobId() + "_" + task.getId()); } controller.getModel().setSelectedTasksIds(selectedTasksIds); }
Source:
<code>public <T, CC, C extends IClientProvider<T, CC>> C getClientProvider(Class<C> clientProviderClass) { log.info("ClientProvider request received for class: " + clientProviderClass.getCanonicalName()); <START> for (IInternalClientProvider<?, ?> internalClientProvider : internalClientProviders) { <END> Class<?> internalClientProviderClass = internalClientProvider.getClass(); if (doTypeArgumentsMatch(VALID_CLIENT_PROVIDERS, clientProviderClass, internalClientProviderClass, 2)) { C c = (C) Proxy.newProxyInstance(clientProviderClass.getClassLoader(), new Class[] { clientProviderClass }, new ClientProviderAdapter(internalClientProvider)); log.debug("Providing ClientProvider."); return c; } } log.debug("Not able to provide ClientProvider!"); return null; }</code><technical_language>keys of map do match</technical_language>

Target:
public <T, CC, C extends IClientProvider<T, CC>> C getClientProvider(Class<C> clientProviderClass) { log.info("ClientProvider request received for class: " + clientProviderClass.getCanonicalName()); for (Class<?> internalClientProviderClass : internalClientProviders.keySet()) { IInternalClientProvider<?, ?> internalClientProvider = internalClientProviders.get(internalClientProviderClass); if (doTypeArgumentsMatch(VALID_CLIENT_PROVIDERS, clientProviderClass, internalClientProviderClass, 2)) { @SuppressWarnings("unchecked") C c = (C) Proxy.newProxyInstance(clientProviderClass.getClassLoader(), new Class[] { clientProviderClass }, new ClientProviderAdapter<T, CC>((IInternalClientProvider<T, CC>) internalClientProvider, coreModelCapability)); log.debug("Providing ClientProvider."); return c; } } log.debug("Not able to provide ClientProvider!"); return null; }
Source:
<code>public Constraint[] getConstraints() { <START> return Arrays.copyOf(constraints, constraints.length); <END> }</code><technical_language>ConstraintStreamScoreDirectorFactory.getConstraints is non-public API. avoid defensive copies in non-public API, a probably-one-day-public-API, avoid performance surprises reduce code size. Note array is defensively copied, constraint instances not. watertight. This method part of a public API argue</technical_language>

Target:
public Constraint[] getConstraints() { return constraints; }
Source:
<code>private BpmnNode compensationEvent(CatchEvent event, CompensateEventDefinition e) { String nodeId = event.getId(); Node<View<IntermediateCompensationEvent>, Edge> node = factoryManager.newNode(nodeId, IntermediateCompensationEvent.class); IntermediateCompensationEvent definition = node.getContent().getDefinition(); CatchEventPropertyReader p = propertyReaderFactory.of(event); definition.setGeneral(new BPMNGeneralSet(new Name(p.getName()), new Documentation(p.getDocumentation()))); definition.setBackgroundSet(p.getBackgroundSet()); definition.setFontSet(p.getFontSet()); definition.setDimensionsSet(p.getCircleDimensionSet()); SLADueDate slaDueDate = new SLADueDate(p.getSlaDueDate()); BaseCancellingEventExecutionSet executionSet = <START> new BaseCancellingEventExecutionSet(null, slaDueDate); <END> definition.setExecutionSet(executionSet); node.getContent().setBounds(p.getBounds()); return BpmnNode.of(node, p); }</code><technical_language>if this null is good here, in general I create instances for properties. sense set a new CancelActivity(false)</technical_language>

Target:
private BpmnNode compensationEvent(CatchEvent event, CompensateEventDefinition e) { String nodeId = event.getId(); Node<View<IntermediateCompensationEvent>, Edge> node = factoryManager.newNode(nodeId, IntermediateCompensationEvent.class); IntermediateCompensationEvent definition = node.getContent().getDefinition(); CatchEventPropertyReader p = propertyReaderFactory.of(event); definition.setGeneral( new BPMNGeneralSet( new Name(p.getName()), new Documentation(p.getDocumentation()) ) ); definition.setBackgroundSet(p.getBackgroundSet()); definition.setFontSet(p.getFontSet()); definition.setDimensionsSet(p.getCircleDimensionSet()); CancelActivity cancelActivity = new CancelActivity(false); SLADueDate slaDueDate = new SLADueDate(p.getSlaDueDate()); BaseCancellingEventExecutionSet executionSet = new BaseCancellingEventExecutionSet(cancelActivity, slaDueDate); definition.setExecutionSet(executionSet); node.getContent().setBounds(p.getBounds()); return BpmnNode.of(node, p); }
Source:
<code><START> public boolean pollWithTimeout(Guid hostId, long timeoutInMillis) { <END> return pollingWithTimeoutAndCustomCondition(hostId, timeoutInMillis, () -> true); }</code><technical_language>this a generic poller meant engine parts - class name is HostSetupNetworkPoller meant setup networks flow only</technical_language>

Target:
public boolean pollWithTimeout(Guid hostId, long timeoutInMillis) { return pollingWithTimeoutAndCustomCondition(hostId, timeoutInMillis, Function.identity()); }
Source:
<code>private static void closeQuietly(LifeCycle service) { try { if (service != null) { service.stop(); } } <START> catch (InterruptedException ie) { <END> Thread.currentThread().interrupt(); } catch (Exception ignored) { } }</code><technical_language>name this e, do elsewhere. I thought needed prevent a warning, apparently a problem if catch block is empty</technical_language>

Target:
private static void closeQuietly(LifeCycle service) { try { if (service != null) { service.stop(); } } catch (InterruptedException e) { Thread.currentThread().interrupt(); } catch (Exception ignored) { } }
Source:
<code>private void writeResponse(Request request, Response response, OrganizationDto organization, GroupDto group) { <START> WsUserGroups.CreateWsResponse.Builder respBuilder = WsUserGroups.CreateWsResponse.newBuilder(); <END> respBuilder.setGroup(toProtobuf(organization, group, 0)); writeProtobuf(respBuilder.build(), request, response); }</code><technical_language>remark: I feedback call this classes CreateResponse Ws. I a strong opinion this</technical_language>

Target:
private void writeResponse(Request request, Response response, OrganizationDto organization, GroupDto group) { WsUserGroups.CreateWsResponse.Builder respBuilder = WsUserGroups.CreateWsResponse.newBuilder(); respBuilder.setGroup(toProtobuf(organization, group, 0, false)); writeProtobuf(respBuilder.build(), request, response); }
Source:
<code>public List<IdNamePair> getLabels() { List<IdNamePair> list = new ArrayList<>(values.size()); <START> if(values == null) { <END> return list; } for(int i = 0; i < values.size(); i++) { String id = values.get(i); IdNamePair pair = new IdNamePair(id, getValueText(i, id)); list.add(pair); } return list; }</code><technical_language>If this is true a NullPointerException thrown</technical_language>

Target:
public List<IdNamePair> getLabels() { if(values == null) { return Collections.emptyList(); } List<IdNamePair> list = new ArrayList<>(values.size()); for(int i = 0; i < values.size(); i++) { String id = values.get(i); IdNamePair pair = new IdNamePair(id, getValueText(i, id)); list.add(pair); } return list; }
Source:
<code>public void cartesianShouldEqualItself() { assertEqual( pointValue( CoordinateReferenceSystem.Cartesian, 1.0, 2.0 ), pointValue( CoordinateReferenceSystem.Cartesian, 1.0, 2.0 ) ); assertEqual( pointValue( CoordinateReferenceSystem.Cartesian, -1.0, 2.0 ), pointValue( CoordinateReferenceSystem.Cartesian, -1.0, 2.0 ) ); assertEqual( pointValue( CoordinateReferenceSystem.Cartesian, -1.0, -2.0 ), pointValue( CoordinateReferenceSystem.Cartesian, -1.0, -2.0 ) ); <START> assertEqual( pointValue( CoordinateReferenceSystem.Cartesian, 0.0, 0.0 ), pointValue( CoordinateReferenceSystem.Cartesian, 0.0, 0.0 ) ); <END> }</code><technical_language>This a nice spot for static import</technical_language>

Target:
public void cartesianShouldEqualItself() { assertEqual( pointValue( Cartesian, 1.0, 2.0 ), pointValue( Cartesian, 1.0, 2.0 ) ); assertEqual( pointValue( Cartesian, -1.0, 2.0 ), pointValue( Cartesian, -1.0, 2.0 ) ); assertEqual( pointValue( Cartesian, -1.0, -2.0 ), pointValue( Cartesian, -1.0, -2.0 ) ); assertEqual( pointValue( Cartesian, 0.0, 0.0 ), pointValue( Cartesian, 0.0, 0.0 ) ); }
Source:
<code>public InternetDataConnection.InputStreamProvider forInput() throws IOException { <START> if (input != null) <END> throw new RuntimeException("Input has already been enabled"); if (connected) throw new RuntimeException("Cannot enable input after having already connected"); connection.setDoInput(true); input = new InternetDataConnection.InputStreamProvider(); return input; }</code><technical_language>is a good practice leave brackets if is line them. If else adds another line produce a bug</technical_language>

Target:
public InternetDataConnection.InputStreamProvider forInput() throws IOException { if (input != null) { throw new RuntimeException("Input has already been enabled"); } if (connected) { throw new RuntimeException("Cannot enable input after having already connected"); } connection.setDoInput(true); input = new InternetDataConnection.InputStreamProvider(); return input; }
Source:
<code>public void clearTokensForUser(String userId) { List<TokenCacheItem> results = this.getTokensForUser(userId); for (TokenCacheItem item : results) { if (item.getUserInfo() != null <START> && item.getUserInfo().getUserId().equalsIgnoreCase(userId)) { <END> try { this.removeItem(CacheKey.createCacheKey(item)); } catch (final AuthenticationException exception) { Logger.e(TAG, "Fail to create cachekey", "", exception.getCode(), exception); } } } }</code><technical_language>NPE if getUserId returns null</technical_language>

Target:
public void clearTokensForUser(String userId) { final List<TokenCacheItem> results = this.getTokensForUser(userId); for (TokenCacheItem item : results) { if (item.getUserInfo() != null && item.getUserInfo().getUserId() != null && item.getUserInfo().getUserId().equalsIgnoreCase(userId)) { try { this.removeItem(CacheKey.createCacheKey(item)); } catch (final AuthenticationException exception) { Logger.e(TAG, "Fail to create cachekey", "", exception.getCode(), exception); } } } }
Source:
<code>public void join(MutationState newMutation) { if (this == newMutation) { return; } this.sizeOffset += newMutation.sizeOffset; for (Map.Entry<TableRef, Map<ImmutableBytesPtr,RowMutationState>> entry : newMutation.mutations.entrySet()) { TableRef tableRef = entry.getKey(); PTable table = tableRef.getTable(); boolean isIndex = table.getType() == PTableType.INDEX; Map<ImmutableBytesPtr,RowMutationState> existingRows = this.mutations.put(tableRef, entry.getValue()); if (existingRows != null) { for (Map.Entry<ImmutableBytesPtr,RowMutationState> rowEntry : entry.getValue().entrySet()) { RowMutationState existingRowMutationState = existingRows.put(rowEntry.getKey(), rowEntry.getValue()); if (existingRowMutationState != null) { Map<PColumn,byte[]> existingValues = existingRowMutationState.getColumnValues(); if (existingValues != PRow.DELETE_MARKER) { Map<PColumn,byte[]> newRow = rowEntry.getValue().getColumnValues(); if (newRow != PRow.DELETE_MARKER) { for (Map.Entry<PColumn,byte[]> valueEntry : newRow.entrySet()) { existingRowMutationState.getColumnValues().put(valueEntry.getKey(), valueEntry.getValue()); } <START> existingRowMutationState.getOrderOfStatementsInConnection().addAll(rowEntry.getValue().getOrderOfStatementsInConnection()); <END> existingRows.put(rowEntry.getKey(), existingRowMutationState); } } } else { if (!isIndex) { numRows++; } } } this.mutations.put(entry.getKey(), existingRows); } else { if (!isIndex) { numRows += entry.getValue().size(); } } } throwIfTooBig(); }</code><technical_language>nice if logic for combining column values set statement indexes moved a method RowMutationState, a join method. Also, work replace for loop following: existingRowMutationState.getColumnValues().putAll(newRow);</technical_language>

Target:
public void join(MutationState newMutation) { if (this == newMutation) { return; } this.sizeOffset += newMutation.sizeOffset; for (Map.Entry<TableRef, Map<ImmutableBytesPtr,RowMutationState>> entry : newMutation.mutations.entrySet()) { TableRef tableRef = entry.getKey(); PTable table = tableRef.getTable(); boolean isIndex = table.getType() == PTableType.INDEX; Map<ImmutableBytesPtr,RowMutationState> existingRows = this.mutations.put(tableRef, entry.getValue()); if (existingRows != null) { for (Map.Entry<ImmutableBytesPtr,RowMutationState> rowEntry : entry.getValue().entrySet()) { RowMutationState existingRowMutationState = existingRows.put(rowEntry.getKey(), rowEntry.getValue()); if (existingRowMutationState != null) { Map<PColumn,byte[]> existingValues = existingRowMutationState.getColumnValues(); if (existingValues != PRow.DELETE_MARKER) { Map<PColumn,byte[]> newRow = rowEntry.getValue().getColumnValues(); if (newRow != PRow.DELETE_MARKER) { existingRowMutationState.join(rowEntry.getValue()); existingRows.put(rowEntry.getKey(), existingRowMutationState); } } } else { if (!isIndex) { numRows++; } } } this.mutations.put(entry.getKey(), existingRows); } else { if (!isIndex) { numRows += entry.getValue().size(); } } } throwIfTooBig(); }
Source:
<code><START> public void reset() { <END> shutdown(); backedIterator.reset(); shouldWork.set(true); this.thread = new AsyncPrefetchThread(buffer, backedIterator, terminator, workspace); Integer deviceId = Nd4j.getAffinityManager().getDeviceForCurrentThread(); Nd4j.getAffinityManager().attachThreadToDevice(thread, deviceId); thread.setDaemon(true); thread.start(); nextElement = null; }</code><technical_language>this need synchronized still</technical_language>

Target:
public void reset() { buffer.clear(); if (thread != null) thread.interrupt(); try { if (thread != null) thread.join(); } catch (InterruptedException e) { throw new RuntimeException(e); } buffer.clear(); backedIterator.reset(); shouldWork.set(true); this.thread = new AsyncPrefetchThread(buffer, backedIterator, terminator, null); Nd4j.getAffinityManager().attachThreadToDevice(thread, deviceId); thread.setDaemon(true); thread.start(); nextElement = null; }
Source:
<code>public void scanDirectories(String[] directories) { if (DEBUG) Log.d(TAG, "scanDirectories: " + Arrays.toString(directories)); try { <START> long start = System.nanoTime() / 1000000; <END> prescan(null, true); long prescan = System.nanoTime() / 1000000; if (ENABLE_BULK_INSERTS) { mMediaInserter = new MediaInserter(mMediaProvider, 500); } for (int i = 0; i < directories.length; i++) { processDirectory(directories[i], mClient); } if (ENABLE_BULK_INSERTS) { mMediaInserter.flushAll(); mMediaInserter = null; } long scan = System.nanoTime() / 1000000; postscan(directories); long end = System.nanoTime() / 1000000; if (true) { Log.i(TAG, "< SCAN > : " + Arrays.toString(directories)); Log.i(TAG, " prescan time: " + (prescan - start) + "ms"); Log.i(TAG, " scan time: " + (scan - prescan) + "ms"); Log.i(TAG, "postscan time: " + (end - scan) + "ms"); Log.i(TAG, " total time: " + (end - start) + "ms"); } } catch (SQLException e) { Log.e(TAG, "SQLException in MediaScanner.scan()", e); } catch (UnsupportedOperationException e) { Log.e(TAG, "UnsupportedOperationException in MediaScanner.scan()", e); } catch (RemoteException e) { Log.e(TAG, "RemoteException in MediaScanner.scan()", e); } finally { releaseResources(); } }</code><technical_language>leaves timings unchanged currentTimeMillis()</technical_language>

Target:
public void scanDirectories(String[] directories) { if (DEBUG) Log.d(TAG, "scanDirectories: " + Arrays.toString(directories)); try { long start = System.currentTimeMillis(); prescan(null, true); long prescan = System.currentTimeMillis(); if (ENABLE_BULK_INSERTS) { mMediaInserter = new MediaInserter(mMediaProvider, 500); } for (int i = 0; i < directories.length; i++) { processDirectory(directories[i], mClient); } if (ENABLE_BULK_INSERTS) { mMediaInserter.flushAll(); mMediaInserter = null; } long scan = System.currentTimeMillis(); postscan(directories); long end = System.currentTimeMillis(); if (DEBUG) { Log.i(TAG, "< SCAN > : " + Arrays.toString(directories)); Log.i(TAG, " prescan time: " + (prescan - start) + "ms"); Log.i(TAG, " scan time: " + (scan - prescan) + "ms"); Log.i(TAG, "postscan time: " + (end - scan) + "ms"); Log.i(TAG, " total time: " + (end - start) + "ms"); } } catch (SQLException e) { Log.e(TAG, "SQLException in MediaScanner.scan()", e); } catch (UnsupportedOperationException e) { Log.e(TAG, "UnsupportedOperationException in MediaScanner.scan()", e); } catch (RemoteException e) { Log.e(TAG, "RemoteException in MediaScanner.scan()", e); } finally { releaseResources(); } }
Source:
<code>public void getDescription_OldAnnotation() throws NoSuchMethodException { <START> String actualResult = getDescriptionWithMethodNameAndParamIndex(TEST_METHOD_1, FIRST_INDEX); <END> assertEquals(SECOND_DESCRIPTION_METHOD_ANNOTATION, actualResult); }</code><technical_language>Abbreviation for variable name: Result -> res. abbreviation rules</technical_language>

Target:
public void getDescription_OldAnnotation() throws NoSuchMethodException { String actualRes = getDescriptionWithMethodNameAndParamIndex(TEST_METHOD_1, FIRST_INDEX); assertEquals(SECOND_DESCRIPTION_METHOD_ANNOTATION, actualRes); }