auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>static BattleGroup createFirstStrike( final Collection<UnitType> unitTypes, final GameProperties properties) { <START> final List<FiringSquadron> firingSquadrons = <END> FiringSquadron.createWithTargetInformation(unitTypes).stream() .map( firingSquadron -> { return createFiringSquadron( firingSquadron, unitIsFirstStrike(properties), noDestroyerPresent()) .name(FIRST_STRIKE_UNITS) .build(); }) .collect(Collectors.toList()); final BattleGroup.BattleGroupBuilder battleGroupBuilder = BattleGroup.builder() .offenseSquadrons(firingSquadrons) .fireType(BattleState.FireType.NORMAL) .casualtiesOnDefenseReturnFire(false); if (Properties.getDefendingSubsSneakAttack(properties)) { battleGroupBuilder.defenseSquadrons(firingSquadrons).casualtiesOnOffenseReturnFire(false); } return battleGroupBuilder.build(); }</code><technical_language>nit, var name more descriptive, this is 'first strike' firing squadron? Is this right group units? I if instead unit-type assign a phase per-phase. complicated negate units add later, leads logic seemingly in multiple places (ie: if this predicate is false here, needs true in another place)</technical_language>

Target:
static BattleGroup createFirstStrike( final Collection<UnitType> unitTypes, final GameProperties properties) { final List<FiringSquadron> firingSquadrons = FiringSquadron.createWithTargetInformation(unitTypes).stream() .map( firingSquadron -> firingSquadron.toBuilder() .firingUnitsAnd(unitIsFirstStrike(properties)) .enemyUnitRequirements(noDestroyerPresent()) .name(FIRST_STRIKE_UNITS) .build()) .collect(Collectors.toList()); return BattleGroup.builder() .fireType(BattleState.FireType.NORMAL) .offenseSquadrons(firingSquadrons) .casualtiesOnDefenseReturnFire(false) .defenseSquadrons( Properties.getDefendingSubsSneakAttack(properties) ? firingSquadrons : List.of()) .casualtiesOnOffenseReturnFire(!Properties.getDefendingSubsSneakAttack(properties)) .build(); }
Source:
<code>private String getSubmissionFormTableHeaderFragmentHtml(String questionNumberString, String responseNumberString) { StringBuilder tableHeaderFragmentHtml = new StringBuilder(); String tableHeaderFragmentTemplate = FeedbackQuestionFormTemplates.RUBRIC_SUBMISSION_FORM_HEADER_FRAGMENT; if(getChoiceAssginedWeight(rubricChoices) == null){ for(int i = 0 ; i < numOfRubricChoices ; i++) { String tableHeaderCell = FeedbackQuestionFormTemplates.populateTemplate(tableHeaderFragmentTemplate, "${qnIndex}", questionNumberString, "${respIndex}", responseNumberString, "${col}", Integer.toString(i), "${rubricChoiceValue}", (rubricChoices.get(i) + "(" + (numOfRubricChoices-i) + ")")); tableHeaderFragmentHtml.append(tableHeaderCell + Const.EOL); } }else{ for(int i = 0 ; i < numOfRubricChoices ; i++) { String tableHeaderCell = FeedbackQuestionFormTemplates.populateTemplate(tableHeaderFragmentTemplate, "${qnIndex}", questionNumberString, "${respIndex}", responseNumberString, "${col}", Integer.toString(i), "${rubricChoiceValue}", rubricChoices.get(i)); tableHeaderFragmentHtml.append(tableHeaderCell + Const.EOL); <START> } <END> } return tableHeaderFragmentHtml.toString(); }</code><technical_language>coding style is in a aspects here: - if (condition) { for (loop) { notice space before/after brackets - } else { ditto - (a - b) ditto, before/after - int i = 0; i < ...; i++ whitespace semicolon - closing } indentation level starting for keyword. Add: here, in a other places too, concern is same</technical_language>

Target:
private String getSubmissionFormTableHeaderFragmentHtml(String questionNumberString, String responseNumberString) { StringBuilder tableHeaderFragmentHtml = new StringBuilder(); String tableHeaderFragmentTemplate = FeedbackQuestionFormTemplates.RUBRIC_SUBMISSION_FORM_HEADER_FRAGMENT; for(int i = 0; i < numOfRubricChoices; i++) { String tableHeaderCell = FeedbackQuestionFormTemplates.populateTemplate(tableHeaderFragmentTemplate, "${qnIndex}", questionNumberString, "${respIndex}", responseNumberString, "${col}", Integer.toString(i), "${rubricChoiceValue}", (rubricChoices.get(i) + " (" + (i+1) + ")")); tableHeaderFragmentHtml.append(tableHeaderCell + Const.EOL); } return tableHeaderFragmentHtml.toString(); }
Source:
<code>Loader( final TaskScheduler taskScheduler, final Environment environment, final GenieFileTransferService fileTransferService, <START> final MeterRegistry registry <END> ) { this.environment = environment; this.registry = registry; this.fileTransferService = fileTransferService; final long refreshRate = environment.getProperty( ScriptLoadBalancerProperties.REFRESH_RATE_PROPERTY, Long.class, 300_000L ); taskScheduler.scheduleWithFixedDelay(this::refresh, refreshRate); }</code><technical_language>increase scope creep want get rid of this service soon if inclined flip this ResourceLoader I object</technical_language>

Target:
protected Loader( final TaskScheduler taskScheduler, final Environment environment, final GenieFileTransferService fileTransferService, final MeterRegistry registry ) { this.environment = environment; this.registry = registry; this.fileTransferService = fileTransferService; final long refreshRate = environment.getProperty( ScriptLoadBalancerProperties.REFRESH_RATE_PROPERTY, Long.class, 300_000L ); taskScheduler.scheduleWithFixedDelay(this::refresh, refreshRate); }
Source:
<code>protected synchronized void forceStop() { <START> System.out.println("Stopping in forcestop"); END> if( isProcessRunning()) { try { process.terminate(); addForceStopEvent(); } catch( DebugException e ) { addForceStopFailedEvent(e); } } process = null; getActualBehavior().setServerStopped(); nextStopRequiresForce = false; }</code><technical_language>system out please - tracing</technical_language>

Target:
protected synchronized void forceStop() { if( isProcessRunning()) { try { process.terminate(); addForceStopEvent(); } catch( DebugException e ) { addForceStopFailedEvent(e); } } process = null; getActualBehavior().setServerStopped(); nextStopRequiresForce = false; }
Source:
<code>public void testGetIntersectingElementsRange() { Iterable<ISegment> intersectingElements; intersectingElements = fSegmentStore.getIntersectingElements(16, 20); assertEquals(0, Iterables.size(intersectingElements)); intersectingElements = fSegmentStore.getIntersectingElements(1, 15); assertEquals(5, Iterables.size(intersectingElements)); intersectingElements = fSegmentStore.getIntersectingElements(2, 14); assertEquals(5, Iterables.size(intersectingElements)); intersectingElements = fSegmentStore.getIntersectingElements(11, 13); assertEquals(1, Iterables.size(intersectingElements)); assert (SEGMENT_10_14.equals(intersectingElements)); intersectingElements = fSegmentStore.getIntersectingElements(10, 14); assertEquals(1, Iterables.size(intersectingElements)); <START> assert (SEGMENT_10_14.equals(intersectingElements)); <END> intersectingElements = fSegmentStore.getIntersectingElements(14, 18); assertEquals(1, Iterables.size(intersectingElements)); assert (SEGMENT_10_14.equals(intersectingElements)); intersectingElements = fSegmentStore.getIntersectingElements(1, 2); assertEquals(1, Iterables.size(intersectingElements)); assert (SEGMENT_2_6.equals(intersectingElements)); }</code><technical_language>this added, assert() do compile a specific flag. Also, segement equals() iterable of segments? guessing this wants do: assertEquals(SEGMENT_10_14, Iterables.getOnlyElement(intersectingElements));</technical_language>

Target:
public void testGetIntersectingElementsRange() { Iterable<ISegment> intersectingElements; intersectingElements = fSegmentStore.getIntersectingElements(16, 20); assertEquals(0, Iterables.size(intersectingElements)); intersectingElements = fSegmentStore.getIntersectingElements(1, 15); assertEquals(5, Iterables.size(intersectingElements)); intersectingElements = fSegmentStore.getIntersectingElements(2, 14); assertEquals(5, Iterables.size(intersectingElements)); intersectingElements = fSegmentStore.getIntersectingElements(11, 13); assertEquals(1, Iterables.size(intersectingElements)); assertEquals(SEGMENT_10_14, Iterables.getOnlyElement(intersectingElements)); intersectingElements = fSegmentStore.getIntersectingElements(10, 14); assertEquals(1, Iterables.size(intersectingElements)); assertEquals(SEGMENT_10_14, Iterables.getOnlyElement(intersectingElements)); intersectingElements = fSegmentStore.getIntersectingElements(14, 18); assertEquals(1, Iterables.size(intersectingElements)); assertEquals(SEGMENT_10_14, Iterables.getOnlyElement(intersectingElements)); intersectingElements = fSegmentStore.getIntersectingElements(1, 2); assertEquals(1, Iterables.size(intersectingElements)); assertEquals(SEGMENT_2_6, Iterables.getOnlyElement(intersectingElements)); }
Source:
<code>private void drawMapMatched(List<Point> coordinates) { try { MapboxMapMatching client = MapboxMapMatching.builder() .accessToken(Mapbox.getAccessToken()) .profile(PROFILE_DRIVING) .coordinates(coordinates) .build(); client.enqueueCall(new Callback<MapMatchingResponse>() { @Override public void onResponse(Call<MapMatchingResponse> call, Response<MapMatchingResponse> response) { List<LatLng> mapMatchedPoints = new ArrayList<>(); if (response.code() == 200) { String geometry = response.body().matchings().get(0).geometry(); <START> List<Position> positions = PolylineUtils.decode(geometry, Constants.PRECISION_6); <END> if (positions == null) { return; } for (int i = 0; i < positions.size(); i++) { mapMatchedPoints.add(new LatLng( positions.get(i).getLatitude(), positions.get(i).getLongitude())); } if (mapMatchedRoute != null) { map.removeAnnotation(mapMatchedRoute); } mapMatchedRoute = map.addPolyline(new PolylineOptions() .addAll(mapMatchedPoints) .color(Color.parseColor("#3bb2d0")) .width(4)); } else { Log.e(TAG, "Too many coordinates, profile not found, invalid input, or no match."); } } @Override public void onFailure(Call<MapMatchingResponse> call, Throwable throwable) { Log.e(TAG, "MapboxMapMatching error: " + throwable.getMessage()); } }); } catch (ServicesException servicesException) { Log.e(TAG, "MapboxMapMatching error: " + servicesException.getMessage()); servicesException.printStackTrace(); } }</code><technical_language>This needs Point right</technical_language>

Target:
private void drawMapMatched(List<Point> coordinates) { try { MapboxMapMatching client = MapboxMapMatching.builder() .accessToken(Mapbox.getAccessToken()) .profile(PROFILE_DRIVING) .coordinates(coordinates) .build(); client.enqueueCall(new Callback<MapMatchingResponse>() { @Override public void onResponse(Call<MapMatchingResponse> call, Response<MapMatchingResponse> response) { List<LatLng> mapMatchedPoints = new ArrayList<>(); if (response.code() == 200) { String geometry = response.body().matchings().get(0).geometry(); List<Point> pointList = PolylineUtils.decode(geometry, Constants.PRECISION_6); if (pointList == null) { return; } for (Point singlePosition : pointList) { mapMatchedPoints.add(new LatLng(singlePosition.latitude(), singlePosition.longitude())); } if (mapMatchedRoute != null) { map.removeAnnotation(mapMatchedRoute); } mapMatchedRoute = map.addPolyline(new PolylineOptions() .addAll(mapMatchedPoints) .color(Color.parseColor("#3bb2d0")) .width(4)); } else { Log.e(TAG, "Too many coordinates, profile not found, invalid input, or no match."); } } @Override public void onFailure(Call<MapMatchingResponse> call, Throwable throwable) { Log.e(TAG, "MapboxMapMatching error: " + throwable.getMessage()); } }); } catch (ServicesException servicesException) { Log.e(TAG, "MapboxMapMatching error: " + servicesException.getMessage()); servicesException.printStackTrace(); } }
Source:
<code>protected void removeTranslations() { <START> mango.select(MongoTranslation.class) <END> .eq(Translation.TRANSLATION_DATA.inner(TranslationData.OWNER), owner.getUniqueName()) .delete(); }</code><technical_language>mongo.delete^^</technical_language>

Target:
protected void removeTranslations() { mongo.delete() .where(Translation.TRANSLATION_DATA.inner(TranslationData.OWNER), owner.getUniqueName()) .manyFrom(MongoTranslation.class); }
Source:
<code>public void activateEditor(IWorkbenchPage aPage, IStructuredSelection aSelection) { if (aSelection == null || aSelection.isEmpty()) { return; } IFile file = null; if ((aSelection.getFirstElement() instanceof TmfTraceElement)) { <START> TmfTraceElement fTrace = ((TmfTraceElement)aSelection.getFirstElement()); <END> final TmfTraceElement traceElement = fTrace.getElementUnderTraceFolder(); final IResource resource = traceElement.getResource(); if (resource instanceof IFile) { file = (IFile) resource; } else if (resource instanceof IFolder) { file = ((IFolder) resource).getFile(traceElement.getName() + '_'); } } else if ((aSelection.getFirstElement() instanceof TmfExperimentElement)) { TmfExperimentElement experimentElement = (TmfExperimentElement) aSelection.getFirstElement(); file = experimentElement.getResource().getFile(experimentElement.getName() + '_'); } if (file != null) { IEditorInput tmpInput = new FileEditorInput(file); IEditorPart localEditor = aPage.findEditor(tmpInput); if (localEditor != null) { aPage.activate(localEditor); aPage.bringToTop(localEditor); } else { IEditorReference[] refs = aPage.getEditorReferences(); for (IEditorReference editorReference : refs) { try { if (editorReference.getEditorInput().equals(tmpInput)) { editorReference.getEditor(true); aPage.bringToTop(editorReference.getEditor(false)); } } catch (PartInitException e) { } } } } }</code><technical_language>rename fTrace (not a class field variable)</technical_language>

Target:
public void activateEditor(IWorkbenchPage aPage, IStructuredSelection aSelection) { if (aSelection == null || aSelection.isEmpty()) { return; } IFile file = null; if ((aSelection.getFirstElement() instanceof TmfTraceElement)) { TmfTraceElement traceElement = ((TmfTraceElement)aSelection.getFirstElement()); traceElement = traceElement.getElementUnderTraceFolder(); final IResource resource = traceElement.getResource(); if (resource instanceof IFile) { file = (IFile) resource; } else if (resource instanceof IFolder) { file = ((IFolder) resource).getFile(traceElement.getName() + '_'); } } else if ((aSelection.getFirstElement() instanceof TmfExperimentElement)) { TmfExperimentElement experimentElement = (TmfExperimentElement) aSelection.getFirstElement(); file = experimentElement.getResource().getFile(experimentElement.getName() + '_'); } if (file != null) { IEditorInput tmpInput = new FileEditorInput(file); IEditorPart localEditor = aPage.findEditor(tmpInput); if (localEditor != null) { aPage.activate(localEditor); aPage.bringToTop(localEditor); } else { IEditorReference[] refs = aPage.getEditorReferences(); for (IEditorReference editorReference : refs) { try { if (editorReference.getEditorInput().equals(tmpInput)) { localEditor = editorReference.getEditor(true); if (localEditor != null) { aPage.bringToTop(localEditor); } } } catch (PartInitException e) { } } } } }
Source:
<code>MarkerBackTrackingAlgorithm(List<MarkerHolder> oldMarkers, List<MarkerHolder> currentMarkers) { this.oldMarkers = oldMarkers; this.currentMarkers = currentMarkers; this.oldSize = oldMarkers.size(); this.currentSize = currentMarkers.size(); <START> memoization = new int[oldSize + 1][currentSize + 1]; <END> }</code><technical_language>this. Relevant: <LINK_0></technical_language>

Target:
MarkerBackTrackingAlgorithm(List<MarkerHolder> oldMarkers, List<MarkerHolder> currentMarkers) { this.oldMarkers = oldMarkers; this.currentMarkers = currentMarkers; this.oldSize = oldMarkers.size(); this.currentSize = currentMarkers.size(); this.memoization = new int[oldSize + 1][currentSize + 1]; this.createdWarningTypes = new ArrayList<>(); this.removedWarningTypes = new ArrayList<>(); }
Source:
<code>public Object invoke(PageContext pc, Object[] args) throws PageException { if (args.length < 2) { throw new FunctionException(pc, "StringSome", 2, 3, args.length); } <START> return call(pc, Caster.toString(args[0]), Caster.toFunction(args[1]), args[2]); <END> }</code><technical_language>fail 2 arguments in request, line 3 is expecting 3 arguments line 17 handles 2. i do other arround, if(args.length==2) return call(pc, Caster.toString(args[0]), Caster.toFunction(args[1])); else if(args.length==3) return call(pc, Caster.toString(args[0]), Caster.toFunction(args[1]), args[2]); else throw new FunctionException(pc, "StringSome", 2, 3, args.length);</technical_language>

Target:
public Object invoke(PageContext pc, Object[] args) throws PageException { if(args.length==2) { return call(pc, Caster.toString(args[0]), Caster.toFunction(args[1]), null); } else if(args.length==3) { return call(pc, Caster.toString(args[0]), Caster.toFunction(args[1]), args[2]); } else { throw new FunctionException(pc, "StringReduce", 2, 3, args.length); } }
Source:
<code>public Collection<EntityProjection> make(String query) { Parser parser = new Parser(); Document parsedDocument = parser.parseDocument(query); fragmentMap.clear(); List<FragmentDefinition> fragmentDefinitions = parsedDocument.getDefinitions().stream() .filter(definition -> definition instanceof FragmentDefinition) .map(definition -> (FragmentDefinition) definition) .collect(Collectors.toList()); fragmentMap.putAll(FragmentResolver.resolve(fragmentDefinitions)); parsedDocument.getDefinitions().forEach(definition -> { if (definition instanceof OperationDefinition) { OperationDefinition operationDefinition = (OperationDefinition) definition; <START> if (operationDefinition.getOperation() == OperationDefinition.Operation.MUTATION) { <END> return; } addRootProjection(operationDefinition.getSelectionSet()); } else if (!(definition instanceof FragmentDefinition)) { throw new InvalidEntityBodyException( String.format("Unsupported definition type {%s}.", definition.getClass())); } }); return rootProjections; }</code><technical_language>I want skip mutations. want skip subscriptions</technical_language>

Target:
public GraphQLProjectionInfo make(String query) { Parser parser = new Parser(); Document parsedDocument = parser.parseDocument(query); fragmentResolver.addFragments(parsedDocument); parsedDocument.getDefinitions().forEach(definition -> { if (definition instanceof OperationDefinition) { OperationDefinition operationDefinition = (OperationDefinition) definition; if (operationDefinition.getOperation() == OperationDefinition.Operation.SUBSCRIPTION) { return; } variableResolver.newScope(operationDefinition); addRootProjection(operationDefinition.getSelectionSet()); } else if (!(definition instanceof FragmentDefinition)) { throw new InvalidEntityBodyException( String.format("Unsupported definition type {%s}.", definition.getClass())); } }); return new GraphQLProjectionInfo(rootLocation, rootProjections, relationshipMap); }
Source:
<code>protected void processMonoValuedEReference(EReference eReference, SIRIUS processedDescription, SIRIUS currentDescription, TransformationCache cache) { if (!processedDescription.eIsSet(eReference)) { Object currentValue = currentDescription.eGet(eReference); if (currentValue instanceof EObject) { <START> IDescriptionPreprocessor preprocessor = SiriusPropertiesCorePlugin.getPlugin().getDescriptionPreprocessor((EObject) currentValue); if (preprocessor != null) { <END> Object processedValue = preprocessor.convert((EObject) currentValue, cache); processedDescription.eSet(eReference, processedValue); } } } }</code><technical_language>java8-isation [optional] method returning a description xxx (preprocessor, converter, etc) return instead optional</technical_language>

Target:
protected void processMonoValuedEReference(EReference eReference, SIRIUS processedDescription, SIRIUS currentDescription, TransformationCache cache) { if (!processedDescription.eIsSet(eReference)) { Object currentValue = currentDescription.eGet(eReference); if (currentValue instanceof EObject) { Optional<IDescriptionPreprocessor> preprocessor = SiriusPropertiesCorePlugin.getPlugin().getDescriptionPreprocessor((EObject) currentValue); if (preprocessor.isPresent()) { Object processedValue = preprocessor.get().convert((EObject) currentValue, cache); processedDescription.eSet(eReference, processedValue); } } } }
Source:
<code>public static RootDSE get(LdapProviderType ldapProviderType, Attributes rootDseRecords) throws NumberFormatException, NamingException { switch (ldapProviderType) { case activeDirectory: return new ADRootDSE(rootDseRecords); case ipa: return new IPARootDSE(rootDseRecords); case rhds: return new RHDSRootDSE(rootDseRecords); case itds: return new ITDSRootDSE(rootDseRecords); case openLdap: return new DefaultRootDSE(rootDseRecords); default: return new <START> DefaultRootDSE(rootDseRecords); <END> } }</code><technical_language>throw exception return OpenLDAP DSE</technical_language>

Target:
public static RootDSE get(LdapProviderType ldapProviderType, Attributes rootDseRecords) throws NumberFormatException, NamingException { switch (ldapProviderType) { case activeDirectory: return new ADRootDSE(rootDseRecords); case ipa: return new IPARootDSE(rootDseRecords); case rhds: return new RHDSRootDSE(rootDseRecords); case itds: return new ITDSRootDSE(rootDseRecords); case openLdap: return new DefaultRootDSE(rootDseRecords); default: throw new IllegalArgumentException("Invalid LDAP provider type."); } }
Source:
<code>private static RelNode trimUnusedfields(RelNode relNode) { final List<RelOptTable> relOptTables = RelOptUtil.findAllTables(relNode); <START> final RelOptSchema relOptSchema = relOptTables.get(0).getRelOptSchema(); <END> final RelBuilder relBuilder = RelFactories.LOGICAL_BUILDER.create( relNode.getCluster(), relOptSchema); final RelFieldTrimmer relFieldTrimmer = new RelFieldTrimmer(null, relBuilder); final RelNode rel = relFieldTrimmer.trim(relNode); return rel; }</code><technical_language>if TableScan in tree of relNode</technical_language>

Target:
private static RelNode trimUnusedfields(RelNode relNode) { final List<RelOptTable> relOptTables = RelOptUtil.findAllTables(relNode); RelOptSchema relOptSchema = null; if (relOptTables.size() != 0) { relOptSchema = relOptTables.get(0).getRelOptSchema(); } final RelBuilder relBuilder = RelFactories.LOGICAL_BUILDER.create( relNode.getCluster(), relOptSchema); final RelFieldTrimmer relFieldTrimmer = new RelFieldTrimmer(null, relBuilder); final RelNode rel = relFieldTrimmer.trim(relNode); return rel; }
Source:
<code>private void waitForClusterReady() throws InterruptedException { <START> await().atMost(50, TimeUnit.SECONDS).until(clusterIsReady()); <END> }</code><technical_language>50 seconds??</technical_language>

Target:
private void waitForClusterReady() throws InterruptedException { boolean clusterOk = false; while (!clusterOk) { if (node1.clusterInfo().split("\n")[0].contains("ok") && node2.clusterInfo().split("\n")[0].contains("ok") && node3.clusterInfo().split("\n")[0].contains("ok")) { clusterOk = true; } AwaitilityUtils.waitFor(50); } }
Source:
<code><START> public TmfConstantTransform(ITmfTimestamp offset) { <END> fOffset = new TmfNanoTimestamp(offset); }</code><technical_language>If (?!) offset is larger Long.MAX_VALUE nanoseconds (using a scale), this overflow..</technical_language>

Target:
public TmfConstantTransform(ITmfTimestamp offset) { fOffset = offset; }
Source:
<code>public static void updateVmLock(final VM vm) { vm.setLockInfo(getLockManager().getLockInfo(String.format("%s%s", vm.getId(), <START> LockingGroup.VM))); <END> }</code><technical_language>LockingGroup.VM.name() worry of some unexpected LockingGroup.toString() future implementation</technical_language>

Target:
public static void updateVmLock(final VM vm) { vm.setLockInfo(getLockManager().getLockInfo(String.format("%s%s", vm.getId(), LockingGroup.VM.name()))); }
Source:
<code>public SeekableStreamIndexTaskTuningConfig convertToTaskTuningConfig() { return new KafkaIndexTaskTuningConfig( getMaxRowsInMemory(), getMaxBytesInMemory(), getMaxRowsPerSegment(), getMaxTotalRows(), getIntermediatePersistPeriod(), getBasePersistDirectory(), <START> 0, <END> getIndexSpec(), true, isReportParseExceptions(), getHandoffConditionTimeout(), isResetOffsetAutomatically(), getSegmentWriteOutMediumFactory(), getIntermediateHandoffPeriod(), isLogParseExceptions(), getMaxParseExceptions(), getMaxSavedParseExceptions() ); }</code><technical_language>This is 0 in toString. Please fix KinesisSupervisorTuningConfig well</technical_language>

Target:
public SeekableStreamIndexTaskTuningConfig convertToTaskTuningConfig() { return new KafkaIndexTaskTuningConfig( getMaxRowsInMemory(), getMaxBytesInMemory(), getMaxRowsPerSegment(), getMaxTotalRows(), getIntermediatePersistPeriod(), getBasePersistDirectory(), getMaxPendingPersists(), getIndexSpec(), true, isReportParseExceptions(), getHandoffConditionTimeout(), isResetOffsetAutomatically(), getSegmentWriteOutMediumFactory(), getIntermediateHandoffPeriod(), isLogParseExceptions(), getMaxParseExceptions(), getMaxSavedParseExceptions() ); }
Source:
<code>protected boolean scrollSelectedCellIntoView(final GridWidget gridWidget) { if (!isSelectionOriginSet(gridWidget) && !isHeaderSelectionOriginSet(gridWidget)) { return false; } boolean isHeaderCellSelected = false; if (!isSelectionOriginSet(gridWidget) && isHeaderSelectionOriginSet(gridWidget)) { isHeaderCellSelected = true; } <START> final BaseGridRendererHelper.RenderingInformation renderingInformation = computeRenderingInformation(gridWidget); <END> if (Objects.isNull(renderingInformation)) { return false; } if (!isGridColumnCandidateForScroll(gridWidget, renderingInformation, isHeaderCellSelected)) { return false; } final double dx = getCellScrollDeltaX(gridWidget, isHeaderCellSelected); final double dy = getCellScrollDeltaY(gridWidget, isHeaderCellSelected); if (dx != 0 || dy != 0) { adjustViewportTransform(gridLayer.getViewport(), new Point2D(dx, dy)); } return true; }</code><technical_language>a static import</technical_language>

Target:
protected boolean scrollSelectedCellIntoView(final GridWidget gridWidget) { if (!isSelectionOriginSet(gridWidget) && !isHeaderSelectionOriginSet(gridWidget)) { return false; } boolean isHeaderCellSelected = false; if (!isSelectionOriginSet(gridWidget) && isHeaderSelectionOriginSet(gridWidget)) { isHeaderCellSelected = true; } if (!isGridColumnCandidateForScroll(gridWidget, isHeaderCellSelected)) { return false; } final double dx = getCellScrollDeltaX(gridWidget, isHeaderCellSelected); final double dy = getCellScrollDeltaY(gridWidget, isHeaderCellSelected); if (dx != 0 || dy != 0) { adjustViewportTransform(gridLayer.getViewport(), new Point2D(dx, dy)); } return true; }
Source:
<code>private void setSnackbarMessage(String status,boolean showBar) { String internetStatus=""; if(status.equalsIgnoreCase("Wifi enabled")||status.equalsIgnoreCase("Mobile data enabled")){ internetStatus="Network Status: Connected"; }else { <START> internetStatus="Network Status: Disconnected"; <END> } if(internetStatus.equalsIgnoreCase("Network Status: Disconnected")) { snackbar = Snackbar .make(linearLayout, internetStatus, Snackbar.LENGTH_LONG) .setAction("Retry", new View.OnClickListener() { @Override public void onClick(View view) { startActivity(new Intent(android.provider.Settings.ACTION_WIRELESS_SETTINGS)); } }); } else { snackbar = Snackbar .make(linearLayout, internetStatus, Snackbar.LENGTH_SHORT); } snackbar.setActionTextColor(Color.WHITE); if(internetStatus.equalsIgnoreCase("Network Status: Disconnected")){ if(internetConnected){ snackbar.show(); internetConnected=false; } }else{ if(!internetConnected){ internetConnected=true; snackbar.show(); } } }</code><technical_language>strings resources? This translation</technical_language>

Target:
private void setSnackbarMessage(String status,boolean showBar) { String internetStatus=""; if(status.equalsIgnoreCase("Wifi enabled")||status.equalsIgnoreCase("Mobile data enabled")){ internetStatus=getResources().getString(R.string.offline_network_connected); }else { internetStatus=getResources().getString(R.string.offline_network_disconnected); } if(internetStatus.equalsIgnoreCase(getResources().getString(R.string.offline_network_disconnected))) { snackbar = Snackbar .make(linearLayout, internetStatus, Snackbar.LENGTH_LONG) .setAction(getResources().getString(R.string.offline_network_retry), new View.OnClickListener() { @Override public void onClick(View view) { startActivity(new Intent(android.provider.Settings.ACTION_WIRELESS_SETTINGS)); } }); } else { snackbar = Snackbar .make(linearLayout, internetStatus, Snackbar.LENGTH_SHORT); } snackbar.setActionTextColor(Color.WHITE); if(internetStatus.equalsIgnoreCase(getResources().getString(R.string.offline_network_disconnected))){ if(internetConnected){ snackbar.show(); internetConnected=false; } }else{ if(!internetConnected){ internetConnected=true; snackbar.show(); } } }
Source:
<code><START> protected PacketDecoder createPacketDecoder() { <END> return ClientPacketDecoder.INSTANCE; }</code><technical_language>need for this change, technically doesnt create, get method is more aptly named</technical_language>

Target:
protected PacketDecoder createPacketDecoder() { return new ClientPacketDecoder(); }
Source:
<code>public void latchAwaitSeconds(int sec) throws InterruptedException { <START> System.out.println("AWAIT " + latch); <END> if (!latch.await(sec, TimeUnit.SECONDS)) { throw new RuntimeException("Await failed"); } }</code><technical_language>This removed</technical_language>

Target:
public void latchAwaitSeconds(int sec) throws InterruptedException { if (!latch.await(sec, TimeUnit.SECONDS)) { throw new RuntimeException("Await failed"); } }
Source:
<code>private static void populateDataToHiveTable(String destination, String source, String partition) { String queryStatement = format("INSERT INTO TABLE %s PARTITION (%s) SELECT * FROM %s", destination, partition, source); onHive().executeQuery("SET mapreduce.map.memory.mb=256"); onHive().executeQuery("SET mapreduce.map.java.opts=-Xmx196m"); onHive().executeQuery("SET mapreduce.reduce.memory.mb=256"); <START> onHive().executeQuery("SET mapreduce.reduce.java.opts=-Xmx196m"); <END> onHive().executeQuery("SET hive.enforce.bucketing = true"); onHive().executeQuery("SET hive.enforce.sorting = true"); onHive().executeQuery(queryStatement); }</code><technical_language>Xmx is smaller memory.mb. Is this heap size query-max-memory-per-node</technical_language>

Target:
private static void populateDataToHiveTable(String destination, String source, String partition) { String queryStatement = format("INSERT INTO TABLE %s PARTITION (%s) SELECT * FROM %s", destination, partition, source); onHive().executeQuery("set hive.enforce.bucketing = true"); onHive().executeQuery("set hive.enforce.sorting = true"); onHive().executeQuery(queryStatement); }
Source:
<code>private static Map<String,String> createMap(URL accumuloPropsLocation, Map<String,String> overrides) { CompositeConfiguration config = new CompositeConfiguration(); config.setThrowExceptionOnMissing(false); config.setDelimiterParsingDisabled(true); PropertiesConfiguration propsConfig = new PropertiesConfiguration(); propsConfig.setDelimiterParsingDisabled(true); if (accumuloPropsLocation != null) { try { propsConfig.load(accumuloPropsLocation.openStream()); } catch (IOException | ConfigurationException e) { throw new IllegalArgumentException(e); } } config.addConfiguration(propsConfig); <START> Map<String,String> result = new HashMap<>(); <END> config.getKeys().forEachRemaining(key -> result.put(key, config.getString(key))); overrides.keySet().forEach(key -> result.put(key, overrides.get(key))); String credProvider = result.get(Property.GENERAL_SECURITY_CREDENTIAL_PROVIDER_PATHS.getKey()); if (credProvider != null) { org.apache.hadoop.conf.Configuration hadoopConf = new org.apache.hadoop.conf.Configuration( CachedConfiguration.getInstance()); hadoopConf.set(CredentialProviderFactoryShim.CREDENTIAL_PROVIDER_PATH, credProvider); for (Property property : Property.values()) { if (property.isSensitive()) { char[] value = CredentialProviderFactoryShim.getValueFromCredentialProvider(hadoopConf, property.getKey()); if (value != null) { result.put(property.getKey(), new String(value)); } } } } return result; }</code><technical_language>guava immutable map builder</technical_language>

Target:
private static ImmutableMap<String,String> createMap(URL accumuloPropsLocation, Map<String,String> overrides) { CompositeConfiguration config = new CompositeConfiguration(); config.setThrowExceptionOnMissing(false); config.setDelimiterParsingDisabled(true); PropertiesConfiguration propsConfig = new PropertiesConfiguration(); propsConfig.setDelimiterParsingDisabled(true); if (accumuloPropsLocation != null) { try { propsConfig.load(accumuloPropsLocation.openStream()); } catch (IOException | ConfigurationException e) { throw new IllegalArgumentException(e); } } config.addConfiguration(propsConfig); Map<String,String> result = new HashMap<>(); config.getKeys().forEachRemaining(key -> result.put(key, config.getString(key))); overrides.forEach(result::put); String credProvider = result.get(Property.GENERAL_SECURITY_CREDENTIAL_PROVIDER_PATHS.getKey()); if (credProvider != null) { org.apache.hadoop.conf.Configuration hadoopConf = new org.apache.hadoop.conf.Configuration( CachedConfiguration.getInstance()); hadoopConf.set(CredentialProviderFactoryShim.CREDENTIAL_PROVIDER_PATH, credProvider); for (Property property : Property.values()) { if (property.isSensitive()) { char[] value = CredentialProviderFactoryShim.getValueFromCredentialProvider(hadoopConf, property.getKey()); if (value != null) { result.put(property.getKey(), new String(value)); } } } } return ImmutableMap.copyOf(result); }
Source:
<code>private long initializeCipher() { String ivString = getHeaderAsString(MantaHttpHeaders.ENCRYPTION_IV); if (ivString == null || ivString.isEmpty()) { String msg = "Initialization Vector (IV) was not set for the object. Unable to decrypt."; MantaClientEncryptionException e = new MantaClientEncryptionException(msg); annotateException(e); throw e; } byte[] iv = Base64.getDecoder().decode(ivString); if (LOGGER.isTraceEnabled()) { LOGGER.trace("IV: {}", Hex.encodeHexString(iv)); } final int mode = Cipher.DECRYPT_MODE; try { <START> if (startPosition != null && startPosition > 0) { <END> this.cipher.init(mode, secretKey, cipherDetails.getEncryptionParameterSpec(iv, startPosition)); return startPosition % cipherDetails.getBlockSizeInBytes(); } else { this.cipher.init(mode, secretKey, cipherDetails.getEncryptionParameterSpec(iv)); return 0L; } } catch (InvalidKeyException | InvalidAlgorithmParameterException e) { String msg = "Error initializing cipher"; MantaClientEncryptionException mce = new MantaClientEncryptionException(msg, e); annotateException(mce); throw mce; } }</code><technical_language>Is null check necessary for startPosition</technical_language>

Target:
private long initializeCipher() { String ivString = getHeaderAsString(MantaHttpHeaders.ENCRYPTION_IV); if (ivString == null || ivString.isEmpty()) { String msg = "Initialization Vector (IV) was not set for the object. Unable to decrypt."; MantaClientEncryptionException e = new MantaClientEncryptionException(msg); annotateException(e); throw e; } byte[] iv = Base64.getDecoder().decode(ivString); if (LOGGER.isTraceEnabled()) { LOGGER.trace("IV: {}", Hex.encodeHexString(iv)); } final int mode = Cipher.DECRYPT_MODE; try { if (startPosition > 0) { this.cipher.init(mode, secretKey, cipherDetails.getEncryptionParameterSpec(iv, startPosition)); return startPosition % cipherDetails.getBlockSizeInBytes(); } else { this.cipher.init(mode, secretKey, cipherDetails.getEncryptionParameterSpec(iv)); return 0L; } } catch (InvalidKeyException | InvalidAlgorithmParameterException e) { String msg = "Error initializing cipher"; MantaClientEncryptionException mce = new MantaClientEncryptionException(msg, e); annotateException(mce); throw mce; } }
Source:
<code>public void run() { <START> Logger log = Logger.getLogger(this.getClass().getCanonicalName()); <END> long startTime = System.currentTimeMillis(); long finishTime = startTime + WORK_TIMEOUT; log.info("Starting work " + name + " on node " + System.getProperty("jboss.node.name")); while (!quit) { try { Thread.sleep(SLEEP_STEP); } catch (InterruptedException e) { log.error("Was interrupted while waiting for work to finish", e); } if (System.currentTimeMillis() >= finishTime) quit = true; } log.info("Finishing work " + name + " after " + TimeUnit.MILLISECONDS.toSeconds(System.currentTimeMillis() - startTime) + " seconds"); }</code><technical_language>log creation run() ? is redundant</technical_language>

Target:
public void run() { long startTime = System.currentTimeMillis(); long finishTime = startTime + WORK_TIMEOUT; while (!quit) { try { Thread.sleep(SLEEP_STEP); } catch (InterruptedException ignored) { } if (System.currentTimeMillis() >= finishTime) quit = true; } }
Source:
<code>public void refresh() { logger.info( "Refreshing Created new Index Name [{}]", alias); final RetryOperation retryOperation = new RetryOperation() { @Override public boolean doOp() { try { String[] indexes = getIndexes(AliasType.Read); <START> Observable.from(indexes).subscribeOn(Schedulers.io()).map(new Func1<String, String>() { <END> @Override public String call(String index) { esProvider.getClient().admin().indices().prepareRefresh( index ).execute().actionGet(); return index; } }).toBlocking().last(); logger.debug( "Refreshed index: " + alias); return true; } catch ( IndexMissingException e ) { logger.error( "Unable to refresh index after create. Waiting before sleeping.", e ); throw e; } } }; doInRetry( retryOperation ); logger.debug( "Refreshed index: " + alias); }</code><technical_language>If this in parallel batches, want subscribe anything. run in caller's thread</technical_language>

Target:
public void refresh() { logger.info( "Refreshing Created new Index Name [{}]", alias); final RetryOperation retryOperation = new RetryOperation() { @Override public boolean doOp() { try { String[] indexes = getIndexes(AliasType.Read); Observable.from(indexes).map(new Func1<String, String>() { @Override public String call(String index) { esProvider.getClient().admin().indices().prepareRefresh( index ).execute().actionGet(); return index; } }).toBlocking().last(); logger.debug( "Refreshed index: " + alias); return true; } catch ( IndexMissingException e ) { logger.error( "Unable to refresh index after create. Waiting before sleeping.", e ); throw e; } } }; doInRetry( retryOperation ); logger.debug( "Refreshed index: " + alias); }
Source:
<code>private ListStatusResult parseToListStatusResult(@Nonnull final IMAPResponse[] r) throws ParsingException, ImapAsyncClientException { if (r.length < 1) { throw new ImapAsyncClientException(FailureType.INVALID_INPUT); } final Response taggedResponse = r[r.length - 1]; if (!taggedResponse.isOK()) { throw new ImapAsyncClientException(FailureType.INVALID_INPUT); } final List<ListInfo> v = new ArrayList<ListInfo>(); final Map<String, Status> m = new HashMap<>(); for (int i = 0; i < r.length; i++) { final IMAPResponse ir = r[i]; if (ir.keyEquals("LIST")) { <START> v.add(new ListInfo(ir)); <END> } else if (ir.keyEquals("STATUS")) { final Status status = new Status(ir); m.put(status.mbox, status); } } return new ListStatusResult(v, m); }</code><technical_language>more issue I realized sun code is handle \NonExistent. ListInfo.canOpen set true Status class. extend ListInfo for \NonExistent attribute</technical_language>

Target:
private ListStatusResult parseToListStatusResult(@Nonnull final IMAPResponse[] r) throws ParsingException, ImapAsyncClientException { if (r.length < 1) { throw new ImapAsyncClientException(FailureType.INVALID_INPUT); } final Response taggedResponse = r[r.length - 1]; if (!taggedResponse.isOK()) { throw new ImapAsyncClientException(FailureType.INVALID_INPUT); } final List<ExtensionListInfo> v = new ArrayList<ExtensionListInfo>(); final Map<String, Status> m = new HashMap<String, Status>(); for (int i = 0; i < r.length; i++) { final IMAPResponse ir = r[i]; if (ir.keyEquals("LIST")) { v.add(new ExtensionListInfo(ir)); } else if (ir.keyEquals("STATUS")) { final Status status = new Status(ir); m.put(status.mbox, status); } } return new ListStatusResult(v, m); }
Source:
<code>private List<AqlSearchResult.SearchEntry> filterBuildAqlSearchResults(List<AqlSearchResult.SearchEntry> itemsToFilter, Map<String,Boolean> buildArtifactsSha1){ List<AqlSearchResult.SearchEntry> filteredResults = new ArrayList<>(); Map<String,List<AqlSearchResult.SearchEntry>> firstPriority = new HashMap<>(); Map<String,List<AqlSearchResult.SearchEntry>> secondPriority = new HashMap<>(); Map<String,List<AqlSearchResult.SearchEntry>> thirdPriority = new HashMap<>(); boolean isBuildNameMatch; boolean isBuildNumberMatch; for (AqlSearchResult.SearchEntry item:itemsToFilter) { <START> if(!buildArtifactsSha1.containsKey(item.getActualSha1())){ <END> continue; } isBuildNameMatch = item.getBuildName().equals(this.buildName); isBuildNumberMatch = item.getBuildNumber().equals(this.buildNumber); if (isBuildNameMatch && isBuildNumberMatch) { firstPriority = addToListInMap(firstPriority,item); continue; } if (isBuildNameMatch) { secondPriority = addToListInMap(secondPriority,item); continue; } thirdPriority = addToListInMap(thirdPriority,item); } for (Map.Entry<String,Boolean> entry : buildArtifactsSha1.entrySet()) { String shaToMatch = entry.getKey(); if (firstPriority.containsKey(shaToMatch)) { filteredResults.addAll(firstPriority.get(shaToMatch)); } else if (secondPriority.containsKey(shaToMatch)) { filteredResults.addAll(secondPriority.get(shaToMatch)); } else if (thirdPriority.containsKey(shaToMatch)) { filteredResults.addAll(thirdPriority.get(shaToMatch)); } } return filteredResults; }</code><technical_language>Missing white-spaces in line</technical_language>

Target:
private List<AqlSearchResult.SearchEntry> filterBuildAqlSearchResults(List<AqlSearchResult.SearchEntry> itemsToFilter, Map<String,Boolean> buildArtifactsSha1){ Map<String, List<AqlSearchResult.SearchEntry>> firstPriority = new HashMap<>(); Map<String, List<AqlSearchResult.SearchEntry>> secondPriority = new HashMap<>(); Map<String, List<AqlSearchResult.SearchEntry>> thirdPriority = new HashMap<>(); List<AqlSearchResult.SearchEntry> filteredResults = new ArrayList<>(); for (AqlSearchResult.SearchEntry item : itemsToFilter) { if (!buildArtifactsSha1.containsKey(item.getActualSha1())){ continue; } boolean isBuildNameMatch = item.getBuildName().equals(this.buildName); boolean isBuildNumberMatch = item.getBuildNumber().equals(this.buildNumber); if (isBuildNameMatch) { if (isBuildNumberMatch) { addToListInMap(firstPriority,item); continue; } addToListInMap(secondPriority,item); continue; } addToListInMap(thirdPriority,item); } for (Map.Entry<String, Boolean> entry : buildArtifactsSha1.entrySet()) { String shaToMatch = entry.getKey(); if (firstPriority.containsKey(shaToMatch)) { filteredResults.addAll(firstPriority.get(shaToMatch)); } else if (secondPriority.containsKey(shaToMatch)) { filteredResults.addAll(secondPriority.get(shaToMatch)); } else if (thirdPriority.containsKey(shaToMatch)) { filteredResults.addAll(thirdPriority.get(shaToMatch)); } } return filteredResults; }
Source:
<code>private static InputStream postPrivate(final String urlPath, final Map<String, String> parameters) { final HttpsURLConnection connection = createConnection(urlPath, false); final OutputStream stream = getOutputStream(connection); if (stream == null) { return null; } final PrintWriter out = new PrintWriter(stream); try { out.print(createQueryStringForParameters(parameters)); <START> out.close(); <END> if (connection.getResponseCode() != HttpURLConnection.HTTP_OK) { Lg.e("server response error(", connection.getResponseCode(), "): ", connection.getResponseMessage()); return null; } Lg.i("used CipherSuite: ", connection.getCipherSuite()); return connection.getInputStream(); } catch (final IOException e) { Lg.ex(e, "IOException while posting"); return null; } finally { out.close(); } }</code><technical_language>out closed a time in finally block</technical_language>

Target:
private static InputStream postPrivate(final String urlPath, final Map<String, String> parameters) { final HttpsURLConnection connection = createConnection(urlPath, false); final OutputStream stream = getOutputStream(connection); if (stream == null) { return null; } final PrintWriter out = new PrintWriter(stream); try { out.print(createQueryStringForParameters(parameters)); out.flush(); if (connection.getResponseCode() != HttpURLConnection.HTTP_OK) { Lg.e("server response error(", connection.getResponseCode(), "): ", connection.getResponseMessage()); return null; } Lg.i("used CipherSuite: ", connection.getCipherSuite()); return connection.getInputStream(); } catch (final IOException e) { Lg.ex(e, "IOException while posting"); return null; } finally { out.close(); } }
Source:
<code>public boolean include(TreeWalk walker) throws MissingObjectException, IncorrectObjectTypeException, IOException { pointer = null; if (walker.isSubtree()) { return walker.isRecursive(); } ObjectId objectId = walker.getObjectId(0); ObjectLoader object = walker.getObjectReader().open(objectId); if (object.isLarge()) { return false; } <START> ObjectStream stream = object.openStream(); byte[] buffer = new byte[VERSION_LINE_LEGACY.length]; int read = stream.read(buffer); if (!validVersionLine(read, buffer)) { return false; } <END> int line = 2; int foundKeys = 0; byte[][] values = new byte[REQUIRED_KEYS.length][]; buffer = object.getCachedBytes(); IntList lineMap = RawParseUtils.lineMap(buffer, 0, buffer.length); int lineMapSize = lineMap.size() - 1; for (int r = 0; r < REQUIRED_KEYS.length; r++) { byte[] key = REQUIRED_KEYS[r]; for (; line < lineMapSize; line++) { int lineStart = lineMap.get(line); int match = RawParseUtils.match(buffer, lineStart, key); if (match > 0) { foundKeys++; int len = lineMap.get(line + 1) - match - 1; values[r] = new byte[len]; System.arraycopy(buffer, match, values[r], 0, len); break; } } } if (foundKeys == REQUIRED_KEYS.length) { LongObjectId oid = LongObjectId.fromString(values[OID_INDEX], 0); int size = RawParseUtils.parseBase10(values[SIZE_INDEX], 0, new MutableInteger()); pointer = new LargeObjectPointer(oid, size); return true; } return false; }</code><technical_language>I if this is work - imagine file contains current version string - is character longer version string IOW match buffer content buffer content is small... I read line bytes..</technical_language>

Target:
public boolean include(TreeWalk walker) throws MissingObjectException, IncorrectObjectTypeException, IOException { pointer = null; if (walker.isSubtree()) { return walker.isRecursive(); } ObjectId objectId = walker.getObjectId(0); ObjectLoader object = walker.getObjectReader().open(objectId); if (object.isLarge()) { return false; } ObjectStream stream = object.openStream(); byte[] buffer = new byte[VERSION_LINES[0].length]; int read = stream.read(buffer); if (!validVersionLine(read, buffer)) { return false; } int line = 2; int foundKeys = 0; byte[][] values = new byte[REQUIRED_KEYS.length][]; buffer = object.getCachedBytes(); IntList lineMap = RawParseUtils.lineMap(buffer, 0, buffer.length); int lineMapSize = lineMap.size() - 1; for (int r = 0; r < REQUIRED_KEYS.length; r++) { byte[] key = REQUIRED_KEYS[r]; for (; line < lineMapSize; line++) { int lineStart = lineMap.get(line); int match = RawParseUtils.match(buffer, lineStart, key); if (match > 0) { foundKeys++; int len = lineMap.get(line + 1) - match - 1; values[r] = new byte[len]; System.arraycopy(buffer, match, values[r], 0, len); break; } } } if (foundKeys == REQUIRED_KEYS.length) { LongObjectId oid = LongObjectId.fromString(values[OID_INDEX], 0); int size = RawParseUtils.parseBase10(values[SIZE_INDEX], 0, new MutableInteger()); pointer = new LargeObjectPointer(oid, size); return true; } return false; }
Source:
<code>public IRemoteProcess getCommandShell(int flags) throws IOException { <START> return new JSchProcessBuilder(getRemoteConnection(), "shell").start(flags); <END> }</code><technical_language>This argument constructor, argument constructor removed is used. If this is kept, is missing //$NON-NLS-$</technical_language>

Target:
public IRemoteProcess getCommandShell(int flags) throws IOException { return new JSchProcessBuilder(getRemoteConnection()).start(flags); }