auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public V apply(EntryView.ReadWriteEntryView<K, V> entry) { try { V merged = value; if (entry.find().isPresent()) { V t = entry.get(); if (remappingFunction instanceof BiFunctionMapper) { BiFunctionMapper mapper = (BiFunctionMapper) this.remappingFunction; Object toStorage = mapper.getValueDataConversion().toStorage(t); merged = remappingFunction.apply((V) toStorage, value); } else { merged = remappingFunction.apply(t, value); } } if (merged == null) { entry.set(merged); } else { <START> if (remappingFunction instanceof BiFunctionMapper) { <END> BiFunctionMapper mapper = (BiFunctionMapper) this.remappingFunction; Object fromStorage = mapper.getValueDataConversion().fromStorage(merged); entry.set((V) fromStorage, metadata); } else { entry.set(merged, metadata); } } return merged; } catch (Exception ex) { throw new UserRaisedFunctionalException(ex); } }</code><technical_language>else if</technical_language>

Target:
public V apply(EntryView.ReadWriteEntryView<K, V> entry) { try { V merged = value; if (entry.find().isPresent()) { V t = entry.get(); if (remappingFunction instanceof BiFunctionMapper) { BiFunctionMapper mapper = (BiFunctionMapper) this.remappingFunction; Object toStorage = mapper.getValueDataConversion().toStorage(t); merged = remappingFunction.apply((V) toStorage, value); } else { merged = remappingFunction.apply(t, value); } } if (merged == null) { entry.set(merged); } else if (remappingFunction instanceof BiFunctionMapper) { BiFunctionMapper mapper = (BiFunctionMapper) this.remappingFunction; Object fromStorage = mapper.getValueDataConversion().fromStorage(merged); entry.set((V) fromStorage, metadata); } else { entry.set(merged, metadata); } return merged; } catch (Exception ex) { throw new UserRaisedFunctionalException(ex); } }
Source:
<code>private void validateTimeRange() { Long currentTime = EnvironmentEdgeManager.currentTimeMillis(); if (startTime != null && (startTime.compareTo(currentTime) > 0 || <START> (endTime != null && startTime.compareTo(endTime) >= 0))) { <END> throw new RuntimeException(INVALID_TIME_RANGE_EXCEPTION_MESSAGE); } }</code><technical_language>add a check end time is in future comparing current time</technical_language>

Target:
private void validateTimeRange() { Long currentTime = EnvironmentEdgeManager.currentTimeMillis(); Long st = (startTime == null) ? 0 : startTime; Long et = (endTime == null) ? currentTime : endTime; if (st.compareTo(currentTime) > 0 || et.compareTo(currentTime) > 0 || st.compareTo(et) >= 0) { throw new RuntimeException(INVALID_TIME_RANGE_EXCEPTION_MESSAGE); } }
Source:
<code>public Object execute(final ExecutionEvent event) throws ExecutionException { final INattableModelManager manager = getCurrentNattableModelManager(); final ExportAsTableConfigurationWizard wizard = new ExportAsTableConfigurationWizard(); IStructuredSelection selection = new StructuredSelection(manager.getTable()); <START> wizard.init(null, selection); <END> WizardDialog dialog = new WizardDialog(Display.getDefault().getActiveShell(), wizard); dialog.open(); return null; }</code><technical_language>check empty selection (to gain some performances)</technical_language>

Target:
public Object execute(final ExecutionEvent event) throws ExecutionException { final INattableModelManager manager = getCurrentNattableModelManager(); final Table table = manager.getTable(); if (table != null) { final ExportAsTableConfigurationWizard wizard = new ExportAsTableConfigurationWizard(); IStructuredSelection selection = new StructuredSelection(table); wizard.init(null, selection); WizardDialog dialog = new WizardDialog(Display.getDefault().getActiveShell(), wizard); dialog.open(); } return null; }
Source:
<code>private void initWaypoints() { SwingWorker<List<Waypoint>, Waypoint> worker = new SwingWorker<List<Waypoint>, Waypoint>() { @Override protected List<Waypoint> doInBackground() throws Exception { Case currentCase = Case.getCurrentCaseThrows(); return MapWaypoint.getWaypoints(currentCase.getSleuthkitCase()); } @Override protected void done() { if (isDone() && !isCancelled()) { try { List<Waypoint> waypoints = get(); if (waypoints == null || waypoints.isEmpty()) { return; } for (Waypoint point : waypoints) { mapPanel.addWaypoint(point); } mapPanel.setCenterLocation(waypoints.get(0)); } catch (ExecutionException ex) { <START> logger.log(Level.WARNING, "An exception occured while initalizing waypoints for geolocation window.", ex); <END> MessageNotifyUtil.Message.error(Bundle.GLTopComponent_initilzation_error()); } catch (InterruptedException ex) { logger.log(Level.WARNING, "The initilization thread for geolocation window was interrupted.", ex); } } } }; worker.execute(); }</code><technical_language>typo - "initializing"</technical_language>

Target:
private void initWaypoints() { SwingWorker<List<Waypoint>, Waypoint> worker = new SwingWorker<List<Waypoint>, Waypoint>() { @Override protected List<Waypoint> doInBackground() throws Exception { Case currentCase = Case.getCurrentCaseThrows(); return MapWaypoint.getWaypoints(currentCase.getSleuthkitCase()); } @Override protected void done() { if (isDone() && !isCancelled()) { try { List<Waypoint> waypoints = get(); if (waypoints == null || waypoints.isEmpty()) { return; } for (Waypoint point : waypoints) { mapPanel.addWaypoint(point); } mapPanel.setCenterLocation(waypoints.get(0)); } catch (ExecutionException ex) { logger.log(Level.WARNING, "An exception occured while initializing waypoints for geolocation window.", ex); MessageNotifyUtil.Message.error(Bundle.GLTopComponent_initilzation_error()); } catch (InterruptedException ex) { logger.log(Level.WARNING, "The initializing thread for geolocation window was interrupted.", ex); } } } }; worker.execute(); }
Source:
<code>private void appendUserData(Account user) { <START> sb.append(user.getFullName()).append(" <").append(user.getPreferredEmail()) <END> .append(">"); }</code><technical_language>full name preferred email optional for a user. This complicates format user: boolean needSpace = false; boolean wroteData = false; if (user.getFullName() != null && !user.getFullName().isEmpty()) { sb.append(user.getFullName()); needSpace = true; wroteData = true; } if (user.getPreferredEmail() != null && !user.getPreferredEmail().isEmpty()) { if (needSpace) { sb.append(' '); } sb.append('<').append(user.getPreferredEmail()).append('>'); wroteData = true; } if (!wroteData) { sb.append("Anonymous Coward #"); sb.append(user.getId()); }</technical_language>

Target:
private void appendUserData(Account user) { if (user.getFullName() != null && ! user.getFullName().isEmpty()) { sb.append(user.getFullName()); } if (user.getPreferredEmail() != null && ! user.getPreferredEmail().isEmpty()) { if (sb.length() > 0) { sb.append(" "); } sb.append(user.getPreferredEmail()); } if (sb.length() == 0) { sb.append("Anonymout Coward #").append(user.getId()); } }
Source:
<code>private boolean isValidNpmUser(NpmUser npmUser) { <START> return npmUser != null && npmUser.getName() != null && npmUser.getPassword() != null && npmUser.getDate() != null && npmUser.getRoles() != null && npmUser.getType() != null && npmUser.getId() != null; <END> }</code><technical_language>Cosmetic: Indentation: return npmUser != null && npmUser.getName() != null && npmUser.getPassword() != null && npmUser.getDate() != null && npmUser.getRoles() != null && npmUser.getType() != null && npmUser.getId() != null;</technical_language>

Target:
private boolean isValidNpmUser(NpmUser npmUser) { return npmUser != null && npmUser.getName() != null && npmUser.getPassword() != null; }
Source:
<code>public void afterPropertiesSet() throws Exception { <START> Assert.isTrue(!(jsrDecider == null), "A decider implementation is required"); <END> Assert.notNull(name, "A name is required for a decision state"); }</code><technical_language>jsrDecider != null Assert.notNull clearer</technical_language>

Target:
public void afterPropertiesSet() throws Exception { Assert.isTrue(jsrDecider != null, "A decider implementation is required"); Assert.notNull(name, "A name is required for a decision state"); }
Source:
<code>public void downloadAsyncLanguagesFromServer() throws Exception { Log.i(TAG, "Starting to download Languages From Server"); AsyncExecutor asyncExecutor = new AsyncExecutor(); CredentialsReader credentialsReader = CredentialsReader.getInstance(); IConnectivityManager connectivity = NetworkManagerFactory.getConnectivityManager( PreferencesState.getInstance().getContext()); ISettingsRepository settingsRepository = new SettingsDataSource( PreferencesState.getInstance().getContext()); DownloadLanguageTranslationUseCase downloader = <START> new DownloadLanguageTranslationUseCase(credentialsReader, connectivity, <END> settingsRepository); downloader.downloadAsync(asyncExecutor); }</code><technical_language>Pass settingsRepository argument DownloadLanguageTranslationUseCase is necessary</technical_language>

Target:
public void downloadAsyncLanguagesFromServer() throws Exception { Log.i(TAG, "Starting to download Languages From Server"); AsyncExecutor asyncExecutor = new AsyncExecutor(); CredentialsReader credentialsReader = CredentialsReader.getInstance(); IConnectivityManager connectivity = NetworkManagerFactory.getConnectivityManager( PreferencesState.getInstance().getContext()); DownloadLanguageTranslationUseCase downloader = new DownloadLanguageTranslationUseCase(credentialsReader, connectivity); downloader.downloadAsync(asyncExecutor); }
Source:
<code>public boolean execute(final IDebugCommandRequest request) { final Optional<ICommandControlDMContext> context = getContext(request); if (!context.isPresent()) { request.done(); return false; } fExecutor.execute(new DsfRunnable() { @Override public void run() { final IGDBExtendedFunctions funcService = fTracker.getService(IGDBExtendedFunctions.class); if (funcService == null) { request.done(); } else { funcService.getVersion(context.get(), new DataRequestMonitor<String>(fExecutor, null) { @Override protected void handleCompleted() { String str; if (isSuccess()) { str = "======= GDB version: " + getData() + " ======="; } else { str = "Could not obtain GDB version. Error: " + getStatus(); } funcService.notify(context.get(), str, new RequestMonitor(fExecutor, null) { @Override protected void handleCompleted() { request.done(); } }); } }); } } }); <START> return false; <END> }</code><technical_language>Sorry, I forgot re-enablement done automatically. Please set this true. Technically, double-clicking button _could_ trigger dialogs up, in this case, command is fast enough happen. If want happen, set a breakpoint in code executes command, double-click command; of delay of bp, dialogs. FYI, handled this is extending RefreshableDebugCommand (written for case) mimic logic in GdbConnectCommand. is overly complicated in situation, need anyway</technical_language>

Target:
public boolean execute(final IDebugCommandRequest request) { final Optional<ICommandControlDMContext> context = getContext(request); if (!context.isPresent()) { request.done(); return false; } fExecutor.execute(new DsfRunnable() { @Override public void run() { final IGDBExtendedFunctions funcService = fTracker.getService(IGDBExtendedFunctions.class); if (funcService == null) { request.done(); } else { funcService.getVersion(context.get(), new DataRequestMonitor<String>(fExecutor, null) { @Override protected void handleCompleted() { String str; if (isSuccess()) { str = "======= GDB version: " + getData() + " ======="; } else { str = "Could not obtain GDB version. Error: " + getStatus(); } funcService.notify(context.get(), str, new RequestMonitor(fExecutor, null) { @Override protected void handleCompleted() { request.done(); } }); } }); } } }); return true; }
Source:
<code>private boolean matchAllConfigs(String[] configs) { if (configs != null) { for (String config : configs) { switch (config) { case CONFIG_VOICE_CAPABLE: TelephonyManager telephonyManager = mContext.getSystemService( TelephonyManager.class); if (!telephonyManager.isVoiceCapable()) { return false; } break; case CONFIG_HAS_RECENTS: final Resources systemRes = mContext.getResources().getSystem(); final int id = systemRes.getIdentifier("config_hasRecents", "bool", "android"); if (id == Resources.ID_NULL || !systemRes.getBoolean(id)) { return false; } break; case CONFIG_HDMI_SOURCE: final String DEVICE_TYPE_HDMI_SOURCE = "4"; if <START> (!getSystemProperty("ro.hdmi.device_type") <END> .equals(DEVICE_TYPE_HDMI_SOURCE)) { return false; } break; default: break; } } } return true; }</code><technical_language>Note this returns a CSV of ints aka . 4,5 is a valid value</technical_language>

Target:
private boolean matchAllConfigs(String[] configs) { if (configs != null) { for (String config : configs) { switch (config) { case CONFIG_VOICE_CAPABLE: TelephonyManager telephonyManager = mContext.getSystemService( TelephonyManager.class); if (!telephonyManager.isVoiceCapable()) { return false; } break; case CONFIG_HAS_RECENTS: final Resources systemRes = mContext.getResources().getSystem(); final int id = systemRes.getIdentifier("config_hasRecents", "bool", "android"); if (id == Resources.ID_NULL || !systemRes.getBoolean(id)) { return false; } break; case CONFIG_HDMI_SOURCE: final int DEVICE_TYPE_HDMI_SOURCE = 4; try { if (!getDeviceType().contains(DEVICE_TYPE_HDMI_SOURCE)) { return false; } } catch (Exception exception) { Log.e( LOG_TAG, "Exception while looking up HDMI device type.", exception); } break; default: break; } } } return true; }
Source:
<code><START> private static void generateGetOutputChannelsMethod(ClassDefinition classDefinition, FieldDefinition probeOutputChannelsField) <END> { MethodDefinition method = classDefinition.declareMethod( a(PUBLIC), "getOutputChannels", type(int[].class)); Variable thisVariable = method.getThis(); method.getBody() .append(thisVariable.getField(probeOutputChannelsField)) .ret(int[].class); }</code><technical_language>I guess appendTo of primary reasons generated JoinProbe class. change, is generated class necessary? aware probeHashChannel.isPresent() check inlined, * matter? generated class outperform SimpleJoinProbe? * a simpler solution (two classes SimpleJoinProbe, expecting probe hash channel not) yield performance generated code</technical_language>

Target:
private static void generateGetOutputChannelsMethod(ClassDefinition classDefinition, FieldDefinition probeOutputChannelsField) { MethodDefinition method = classDefinition.declareMethod( a(PUBLIC), "getOutputChannels", type(int[].class)); method.getBody() .append(method.getThis().getField(probeOutputChannelsField)) .ret(int[].class); }
Source:
<code>public PiecewisePolynomialResult interpolate(double[] xValues, double[] yValues) { ArgumentChecker.notNull(xValues, "xValues"); ArgumentChecker.notNull(yValues, "yValues"); ArgumentChecker.isTrue(xValues.length == yValues.length, "xValues length = yValues length"); PiecewisePolynomialResult result; <START> if (isClamped()) { <END> double[][] xyValuesAll = getDataTotal(xValues, yValues); result = _baseMethod.interpolate(xyValuesAll[0], xyValuesAll[1]); } else { double[] xyValues = getProduct(xValues, yValues); result = _baseMethod.interpolate(xValues, xyValues); } int nIntervalsAll = result.getNumberOfIntervals(); double[] nodes = result.getKnots().getData(); if (Math.abs(xValues[xValues.length - 1] - nodes[nIntervalsAll]) < EPS) { double lastNodeX = nodes[nIntervalsAll]; double lastNodeY = FUNC.evaluate(result, lastNodeX).getEntry(0); double extraNode = 2.0 * nodes[nIntervalsAll] - nodes[nIntervalsAll - 2]; double extraDerivative = FUNC.differentiate(result, lastNodeX).getEntry(0); double[] newKnots = new double[nIntervalsAll + 2]; System.arraycopy(result.getKnots().getData(), 0, newKnots, 0, nIntervalsAll + 1); newKnots[nIntervalsAll + 1] = extraNode; double[][] newCoefMatrix = new double[nIntervalsAll + 1][]; for (int i = 0; i < nIntervalsAll; ++i) { newCoefMatrix[i] = Arrays.copyOf(result.getCoefMatrix().getRowVector(i).getData(), result.getOrder()); } newCoefMatrix[nIntervalsAll] = new double[result.getOrder()]; newCoefMatrix[nIntervalsAll][result.getOrder() - 1] = lastNodeY; newCoefMatrix[nIntervalsAll][result.getOrder() - 2] = extraDerivative; result = new PiecewisePolynomialResult(new DoubleMatrix1D(newKnots), new DoubleMatrix2D(newCoefMatrix), result.getOrder(), 1); } return result; }</code><technical_language>This set in a class member construction time data clamping is provided construction is immutable. function call test is eliminated</technical_language>

Target:
public PiecewisePolynomialResult interpolate(double[] xValues, double[] yValues) { ArgumentChecker.notNull(xValues, "xValues"); ArgumentChecker.notNull(yValues, "yValues"); ArgumentChecker.isTrue(xValues.length == yValues.length, "xValues length = yValues length"); PiecewisePolynomialResult result; if (_isClamped) { double[][] xyValuesAll = getDataTotal(xValues, yValues); result = _baseMethod.interpolate(xyValuesAll[0], xyValuesAll[1]); } else { double[] xyValues = getProduct(xValues, yValues); result = _baseMethod.interpolate(xValues, xyValues); } return extrapolateByLinearFunction(result, xValues); }
Source:
<code>private MarshallableEntry<K, V> load(Object key, boolean fetchValue, boolean fetchMetadata) { RestResponse response = null; try { response = CompletionStages.join(cacheClient.get(encodeKey(key))); if (isSuccessful(response.getStatus())) { String contentType = getHeader("Content-Type", response); Metadata metadata; long created, lastUsed; if (fetchMetadata) { long ttl = timeHeaderToLong(getHeader(TIME_TO_LIVE_SECONDS, response)); long maxidle = timeHeaderToLong(getHeader(MAX_IDLE_TIME_SECONDS, response)); metadata = metadataHelper.buildMetadata(contentType, ttl, TimeUnit.SECONDS, maxidle, TimeUnit.SECONDS); created = timeHeaderToLong(getHeader(CREATED, response)); lastUsed = timeHeaderToLong(getHeader(LAST_USED, response)); } else { metadata = null; created = -1; lastUsed = -1; } Object value; if (fetchValue) { byte[] bytes = response.getBodyAsByteArray(); value = unmarshall(contentType, bytes); } else { value = null; } return entryFactory.create(key, value, metadata, created, lastUsed); } else if (response.getStatus() == 404) { return null; } else { throw log.httpError(String.valueOf(response.getStatus())); } } catch (IOException e) { throw log.httpError(e); } catch (Exception e) { throw new PersistenceException(e); } finally { <START> if (response != null) { <END> response.close(); } } }</code><technical_language>a RestResponse is AutoCloseable, convert this a try-with-resources</technical_language>

Target:
private MarshallableEntry<K, V> load(Object key, boolean fetchValue, boolean fetchMetadata) { try (RestResponse response = CompletionStages.join(cacheClient.get(encodeKey(key)))) { if (isSuccessful(response.getStatus())) { String contentType = getHeader("Content-Type", response); Metadata metadata; long created, lastUsed; if (fetchMetadata) { long ttl = timeHeaderToLong(getHeader(TIME_TO_LIVE_SECONDS, response)); long maxidle = timeHeaderToLong(getHeader(MAX_IDLE_TIME_SECONDS, response)); metadata = metadataHelper.buildMetadata(contentType, ttl, TimeUnit.SECONDS, maxidle, TimeUnit.SECONDS); created = timeHeaderToLong(getHeader(CREATED, response)); lastUsed = timeHeaderToLong(getHeader(LAST_USED, response)); } else { metadata = null; created = -1; lastUsed = -1; } Object value; if (fetchValue) { byte[] bytes = response.getBodyAsByteArray(); value = unmarshall(contentType, bytes); } else { value = null; } return entryFactory.create(key, value, metadata, created, lastUsed); } else if (response.getStatus() == 404) { return null; } else { throw log.httpError(String.valueOf(response.getStatus())); } } catch (IOException e) { throw log.httpError(e); } catch (Exception e) { throw new PersistenceException(e); } }
Source:
<code>public void send(Send request) throws IOException { System.out.println("send"); ByteBufferChannel byteBufferChannel = new ByteBufferChannel(ByteBuffer.allocate((int) request.sizeInBytes())); while (!request.isSendComplete()) { request.writeTo(byteBufferChannel); } byteBufferChannel.getBuffer().position(0); ByteBuf byteBuf = Unpooled.wrappedBuffer(byteBufferChannel.getBuffer()); Http2StreamChannel childChannel = http2StreamChannelBootstrap.open().syncUninterruptibly().getNow(); Http2Headers http2Headers = new DefaultHttp2Headers().method(HttpMethod.POST.asciiName()).scheme("https").path("/"); http2Headers.set(RestUtils.Headers.HTTP2_FRONTEND_REQUEST, "true"); <START> channelPromise = childChannel.newPromise(); <END> childChannel.attr(Http2ResponseHandler.RESPONSE_CALLBACK).set(new Callback<ByteBuf>() { @Override public void onCompletion(ByteBuf result, Exception exception) { responseByteBuf = result; channelPromise.setSuccess(); } }); DefaultHttp2HeadersFrame headersFrame = new DefaultHttp2HeadersFrame(http2Headers, false); DefaultHttp2DataFrame dataFrame = new DefaultHttp2DataFrame(byteBuf, true); childChannel.write(headersFrame); childChannel.write(dataFrame); childChannel.flush(); }</code><technical_language>Instead of combining ambry callbacks netty promises here, setting attribute a CompletableFuture ambry's FutureResult? wait future requiring lines 135-137</technical_language>

Target:
public void send(Send request) throws IOException { ByteBufferChannel byteBufferChannel = new ByteBufferChannel(ByteBuffer.allocate((int) request.sizeInBytes())); while (!request.isSendComplete()) { request.writeTo(byteBufferChannel); } byteBufferChannel.getBuffer().position(0); ByteBuf byteBuf = Unpooled.wrappedBuffer(byteBufferChannel.getBuffer()); Http2StreamChannel childChannel = http2StreamChannelBootstrap.open().syncUninterruptibly().getNow(); Http2Headers http2Headers = new DefaultHttp2Headers().method(HttpMethod.POST.asciiName()).scheme("https").path("/"); responsePromise = childChannel.eventLoop().newPromise(); childChannel.attr(Http2ResponseHandler.RESPONSE_PROMISE).set(responsePromise); DefaultHttp2HeadersFrame headersFrame = new DefaultHttp2HeadersFrame(http2Headers, false); DefaultHttp2DataFrame dataFrame = new DefaultHttp2DataFrame(byteBuf, true); childChannel.write(headersFrame); childChannel.write(dataFrame); childChannel.flush(); }
Source:
<code><START> private String getLabel(final EObject eObject) { <END> String label = "Element whithout name"; if (eObject instanceof IdentifiedElement) { label = new IdentifiedElementQuery((IdentifiedElement) eObject).getLabel(); } else if (eObject instanceof RepresentationExtensionDescription) { label = ((RepresentationExtensionDescription) eObject).getName(); } else { label = eObject.eClass().getName(); } return label; }</code><technical_language>protected reusable child classes</technical_language>

Target:
protected String getLabel(final EObject eObject) { String label = "Element whithout name"; if (eObject instanceof IdentifiedElement) { label = new IdentifiedElementQuery((IdentifiedElement) eObject).getLabel(); } else if (eObject instanceof RepresentationExtensionDescription) { label = ((RepresentationExtensionDescription) eObject).getName(); } else { label = eObject.eClass().getName(); } return label; }
Source:
<code>public void setPlayersInTeam(String team, List<Player> playerList, Function<Player, Rating> ratingProvider) { int totalRating = 0; for (Player player : playerList) { if (player == null) { continue; } PlayerCardTooltipController playerCardTooltipController = uiService.loadFxml("theme/player_card_tooltip.fxml"); playerCardTooltipController.setPlayer(player); <START> RatingChangedLabelController ratingChangedLabelController = uiService.loadFxml("theme/rating_changed_label.fxml"); <END> ratingChange.put(player.getId(), ratingChangedLabelController); HBox container = new HBox(playerCardTooltipController.getRoot(), ratingChangedLabelController.getRoot()); teamPane.getChildren().add(container); totalRating += RatingUtil.getRating(ratingProvider.apply(player)); } String teamTitle; if ("1".equals(team) || "-1".equals(team)) { teamTitle = i18n.get("game.tooltip.teamTitleNoTeam"); } else if ("null".equals(team)) { teamTitle = i18n.get("game.tooltip.observers"); } else { teamTitle = i18n.get("game.tooltip.teamTitle", Integer.valueOf(team) - 1, totalRating); } teamNameLabel.setText(teamTitle); }</code><technical_language>call rating_change_label.fxml</technical_language>

Target:
public void setPlayersInTeam(String team, List<Player> playerList, Function<Player, Rating> ratingProvider) { int totalRating = 0; for (Player player : playerList) { if (player == null) { continue; } PlayerCardTooltipController playerCardTooltipController = uiService.loadFxml("theme/player_card_tooltip.fxml"); playerCardTooltipController.setPlayer(player); RatingChangeLabelController ratingChangeLabelController = uiService.loadFxml("theme/rating_change_label.fxml"); ratingChangeControllersByPlayerId.put(player.getId(), ratingChangeLabelController); HBox container = new HBox(playerCardTooltipController.getRoot(), ratingChangeLabelController.getRoot()); teamPane.getChildren().add(container); totalRating += RatingUtil.getRating(ratingProvider.apply(player)); } String teamTitle; if ("1".equals(team) || "-1".equals(team)) { teamTitle = i18n.get("game.tooltip.teamTitleNoTeam"); } else if ("null".equals(team)) { teamTitle = i18n.get("game.tooltip.observers"); } else { teamTitle = i18n.get("game.tooltip.teamTitle", Integer.valueOf(team) - 1, totalRating); } teamNameLabel.setText(teamTitle); }
Source:
<code>public final Iterable<N> toDepthFirstIterable() { <START> return () -> AbstractTree.this.depthFirstIterator(); <END> }</code><technical_language>Please replace occurrences of AbstractTree.this. Indeed, code style recommend add this a prefix of a function call function is in current object</technical_language>

Target:
public final Iterable<N> toDepthFirstIterable() { return () -> this.depthFirstIterator(); }
Source:
<code>public boolean check(final VDS host) { final int checks = VDSM_RESPONSIVENESS_PERIOD_IN_SECONDS / Config.<Integer> getValue(ConfigValues.SetupNetworksPollingTimeout); HostSetupNetworkPoller poller = new HostSetupNetworkPoller(); for (int i = 0; i < checks; i++) { if (poller.poll(host.getId())) { log.info("Engine managed to communicate with VDSM agent on host '{}' ('{}')", <START> host.getName(), <END> host.getId()); return true; } } return false; }</code><technical_language>prefer getHostName() ease correlation other issues logs</technical_language>

Target:
public boolean check(final VDS host) { final int checks = VDSM_RESPONSIVENESS_PERIOD_IN_SECONDS / Config.<Integer> getValue(ConfigValues.SetupNetworksPollingTimeout); HostSetupNetworkPoller poller = new HostSetupNetworkPoller(); for (int i = 0; i < checks; i++) { if (poller.poll(host.getId())) { log.info("Engine managed to communicate with VDSM agent on host '{}' with address '{}' ('{}')", host.getName(), host.getHostName(), host.getId()); return true; } } return false; }
Source:
<code>private void removeQuestionsAndResponsesIfNotAllowed(FeedbackSessionResultsBundle bundle) { Iterator<FeedbackResponseAttributes> iter = bundle.responses.iterator(); while (iter.hasNext()) { FeedbackResponseAttributes fdr = iter.next(); if (!(instructor != null && instructor.isAllowedForPrivilege(fdr.giverSection, fdr.feedbackSessionName, Const.ParamsNames.INSTRUCTOR_PERMISSION_VIEW_SESSION_IN_SECTIONS) && instructor.isAllowedForPrivilege(fdr.recipientSection, fdr.feedbackSessionName, <START> Const.ParamsNames.INSTRUCTOR_PERMISSION_VIEW_SESSION_IN_SECTIONS))) { <END> iter.remove(); } } }</code><technical_language>Introduce explaining variables</technical_language>

Target:
private void removeQuestionsAndResponsesIfNotAllowed(FeedbackSessionResultsBundle bundle) { Iterator<FeedbackResponseAttributes> iter = bundle.responses.iterator(); while (iter.hasNext()) { FeedbackResponseAttributes fdr = iter.next(); boolean instructorCanViewSessionInGiverSection = instructor.isAllowedForPrivilege(fdr.giverSection, fdr.feedbackSessionName, Const.ParamsNames.INSTRUCTOR_PERMISSION_VIEW_SESSION_IN_SECTIONS); boolean instructorCanViewSessionInRecipientSection = instructor.isAllowedForPrivilege(fdr.recipientSection, fdr.feedbackSessionName, Const.ParamsNames.INSTRUCTOR_PERMISSION_VIEW_SESSION_IN_SECTIONS); boolean instructorDoesNotHaveSessionViewingPrivileges = !(instructorCanViewSessionInGiverSection && instructorCanViewSessionInRecipientSection); if (instructorDoesNotHaveSessionViewingPrivileges) { iter.remove(); } } }
Source:
<code>public void update() throws BankException, LoginException, BankChoiceException, IOException { super.update(); if (username == null || password == null || username.length() == 0 || password.length() == 0) { throw new LoginException(res.getText(R.string.invalid_username_password).toString()); } urlopen = login(); String response = null; Matcher matcher; try { response = urlopen.open("<LINK_0>"); matcher = reAccounts.matcher(response); while (matcher.find()) { Account acc = new Account(Html.fromHtml(matcher.group(1)).toString().trim(), Helpers.parseBalance(matcher.group(2)), ("fund".equals(matcher.group(4)) ? "f:" : "")+matcher.group(3).trim()); if ("fund".equals(matcher.group(4))) { acc.setType(Account.FUNDS); } else { balance = balance.add(Helpers.parseBalance(matcher.group(3))); } accounts.add(acc); } if (accounts.isEmpty()) { throw new BankException(res.getText(R.string.no_accounts_found).toString()); } <START> } finally { <END> super.updateComplete(); } }</code><technical_language>remove finally {</technical_language>

Target:
public void update() throws BankException, LoginException, BankChoiceException, IOException { super.update(); if (username == null || password == null || username.length() == 0 || password.length() == 0) { throw new LoginException(res.getText(R.string.invalid_username_password).toString()); } urlopen = login(); String response = urlopen.open("<LINK_0>"); Matcher matcher = reAccounts.matcher(response); while (matcher.find()) { Account acc = new Account(Html.fromHtml(matcher.group(1)).toString().trim(), Helpers.parseBalance(matcher.group(2)), ("fund".equals(matcher.group(4)) ? "f:" : "")+matcher.group(3).trim()); if ("fund".equals(matcher.group(4))) { acc.setType(Account.FUNDS); } else { balance = balance.add(Helpers.parseBalance(matcher.group(3))); } accounts.add(acc); } if (accounts.isEmpty()) { throw new BankException(res.getText(R.string.no_accounts_found).toString()); } super.updateComplete(); }
Source:
<code>public void start() throws Exception { if (this.jettyConfiguration != null && this.jettyConfiguration.length() > 0) { String[] configs = this.jettyConfiguration.split(","); for (String config : configs) { server = createServerOrApplyConfiguration(config, server); } } else { server = new Server(); ServerConnector http = new ServerConnector(server); http.setPort(port); if (host != null) { http.setHost(host); } server.addConnector(http); } context = new ServletContextHandler(server, ninjaContextPath); ninjaProperties = new NinjaPropertiesImpl(ninjaMode); String serverName = (this.serverUri != null ? this.serverUri.toString() : createServerName()); ninjaProperties.setProperty(NinjaConstant.serverName, serverName); ninjaServletListener.setNinjaProperties(ninjaProperties); ninjaServletListener.setLogInjectorException(false); context.addEventListener(ninjaServletListener); context.addFilter(GuiceFilter.class, "/*", null); context.addServlet(DefaultServlet.class, "/"); try { server.start(); } catch (Exception cause) { if (ninjaServletListener.getNinjaBootstrap() != null && ninjaServletListener.getNinjaBootstrap().getInjectorException() != null) { throw ninjaServletListener.getNinjaBootstrap().getInjectorException(); <START> } <END> else { throw cause; } } }</code><technical_language>Ninja's codebases else line "}" } else { sound picky... uniformity is important in long run codebase grows :) Thanks</technical_language>

Target:
public void start() throws Exception { if (this.jettyConfiguration != null && this.jettyConfiguration.length() > 0) { String[] configs = this.jettyConfiguration.split(","); for (String config : configs) { server = createServerOrApplyConfiguration(config, server); } } else { server = new Server(); ServerConnector http = new ServerConnector(server); http.setPort(port); if (host != null) { http.setHost(host); } server.addConnector(http); } context = new ServletContextHandler(server, ninjaContextPath); ninjaProperties = new NinjaPropertiesImpl(ninjaMode); String serverName = (this.serverUri != null ? this.serverUri.toString() : createServerName()); ninjaProperties.setProperty(NinjaConstant.serverName, serverName); ninjaServletListener.setNinjaProperties(ninjaProperties); context.addEventListener(ninjaServletListener); context.addFilter(GuiceFilter.class, "/*", null); context.addServlet(DefaultServlet.class, "/"); try { server.start(); } catch (Exception e) { if (e.getCause() != null && e.getCause() instanceof CreationException) { throw (CreationException)e.getCause(); } else { throw e; } } }
Source:
<code>public void run() { String sessionID = null; try { LOG.info("To read message from adf..."); ReceiveQueueMessageResult resultQM = service.receiveQueueMessage(requestQueueName, opts); BrokeredMessage message = resultQM.getValue(); if (message != null && message.getMessageId() != null) { sessionID = message.getReplyToSessionId(); BufferedReader rd = new BufferedReader( new InputStreamReader(message.getBody())); StringBuilder sb = new StringBuilder(); String line; while ((line = rd.readLine()) != null) { sb.append(line); } rd.close(); String msg = sb.toString(); LOG.info("ADF message: " + msg); service.deleteMessage(message); ADFJob.JobType jobType = ADFJob.getJobType(msg); switch (jobType) { case REPLICATION: <START> ADFReplicationJob job = new ADFReplicationJob(msg, sessionID); <END> job.startJob(); break; case HIVE: ADFHiveJob hiveJob = new ADFHiveJob(msg, sessionID); hiveJob.startJob(); break; case PIG: ADFPigJob pigJob = new ADFPigJob(msg, sessionID); pigJob.startJob(); break; default: } } else { LOG.info("No message from adf"); } } catch (FalconException e) { if (sessionID != null) { sendErrorMessage(sessionID, e.toString()); } LOG.info(e.toString()); } catch (ServiceException | IOException e) { LOG.info(e.toString()); } }</code><technical_language>Instead of creating concrete classes for supported ADFJob type(REPLICATION, HIVE, PIG) user Factory pattern</technical_language>

Target:
public void run() { String sessionID = null; try { LOG.info("To read message from adf..."); ReceiveQueueMessageResult resultQM = service.receiveQueueMessage(requestQueueName, opts); BrokeredMessage message = resultQM.getValue(); if (message != null && message.getMessageId() != null) { sessionID = message.getReplyToSessionId(); BufferedReader rd = new BufferedReader( new InputStreamReader(message.getBody())); StringBuilder sb = new StringBuilder(); String line; while ((line = rd.readLine()) != null) { sb.append(line); } rd.close(); String msg = sb.toString(); LOG.info("ADF message: " + msg); service.deleteMessage(message); ADFJob job = ADFJobFactory.buildADFJob(msg, sessionID); job.startJob(); } else { LOG.info("No message from adf"); } } catch (FalconException e) { if (sessionID != null) { sendErrorMessage(sessionID, e.toString()); } LOG.info(e.toString()); } catch (ServiceException | IOException e) { LOG.info(e.toString()); } }
Source:
<code>public List<FeedbackQuestionAttributes> getCopiableFeedbackQuestionsForInstructor(String googleId) throws EntityDoesNotExistException { List<FeedbackQuestionAttributes> copiableQuestions = new ArrayList<>(); List<CourseAttributes> courses = coursesLogic.getCoursesForInstructor(googleId); for (CourseAttributes course : courses) { List<FeedbackSessionAttributes> sessions = fsLogic.getFeedbackSessionsForCourse(course.getId()); for (FeedbackSessionAttributes session : sessions) { List<FeedbackQuestionAttributes> questions = getFeedbackQuestionsForSession(session.getFeedbackSessionName(), course.getId()); copiableQuestions.addAll(questions); } } copiableQuestions.sort(Comparator.comparing((FeedbackQuestionAttributes copiableQuestion) -> copiableQuestion.courseId) <START> .thenComparing(copiableQuestion -> copiableQuestion.getQuestionDetails().getQuestionTypeDisplayName())); <END> return copiableQuestions; }</code><technical_language>careful, comparators missed here</technical_language>

Target:
public List<FeedbackQuestionAttributes> getCopiableFeedbackQuestionsForInstructor(String googleId) throws EntityDoesNotExistException { List<FeedbackQuestionAttributes> copiableQuestions = new ArrayList<>(); List<CourseAttributes> courses = coursesLogic.getCoursesForInstructor(googleId); for (CourseAttributes course : courses) { List<FeedbackSessionAttributes> sessions = fsLogic.getFeedbackSessionsForCourse(course.getId()); for (FeedbackSessionAttributes session : sessions) { List<FeedbackQuestionAttributes> questions = getFeedbackQuestionsForSession(session.getFeedbackSessionName(), course.getId()); copiableQuestions.addAll(questions); } } copiableQuestions.sort(Comparator.comparing((FeedbackQuestionAttributes question) -> question.courseId) .thenComparing(question -> question.feedbackSessionName) .thenComparing(question -> question.getQuestionDetails().getQuestionTypeDisplayName()) .thenComparing(question -> question.getQuestionDetails().getQuestionText())); return copiableQuestions; }
Source:
<code>public boolean isBucketEmpty(String volume, String bucket) throws IOException { String keyPrefix = getBucketKey(volume, bucket); Iterator<Map.Entry<CacheKey<String>, CacheValue<OmKeyInfo>>> iterator = ((TypedTable< String, OmKeyInfo>) keyTable).cacheIterator(); while (iterator.hasNext()) { Map.Entry< CacheKey<String>, CacheValue<OmKeyInfo>> entry = iterator.next(); String key = entry.getKey().getCacheKey(); OmKeyInfo omKeyInfo = entry.getValue().getCacheValue(); if (key.startsWith(keyPrefix) && omKeyInfo != null) { return false; } } try (TableIterator<String, ? extends KeyValue<String, OmKeyInfo>> keyIter = keyTable.iterator()) { KeyValue<String, OmKeyInfo> kv = keyIter.seek(keyPrefix); if (kv != null && kv.getKey().startsWith(keyPrefix) && <START> keyTable.get(kv.getKey()) != null) { <END> return false; } } return true; }</code><technical_language>keyTable.get(.. ) checks cache if entry is in cache, DB. avoid reading DB check cache alone. This is applicable for volumeEmpty check if maintain a full bucket table cache matter</technical_language>

Target:
public boolean isBucketEmpty(String volume, String bucket) throws IOException { String keyPrefix = getBucketKey(volume, bucket); Iterator<Map.Entry<CacheKey<String>, CacheValue<OmKeyInfo>>> iterator = ((TypedTable< String, OmKeyInfo>) keyTable).cacheIterator(); while (iterator.hasNext()) { Map.Entry< CacheKey<String>, CacheValue<OmKeyInfo>> entry = iterator.next(); String key = entry.getKey().getCacheKey(); OmKeyInfo omKeyInfo = entry.getValue().getCacheValue(); if (key.startsWith(keyPrefix) && omKeyInfo != null) { return false; } } try (TableIterator<String, ? extends KeyValue<String, OmKeyInfo>> keyIter = keyTable.iterator()) { KeyValue<String, OmKeyInfo> kv = keyIter.seek(keyPrefix); if (kv != null) { CacheValue<OmKeyInfo> cacheValue = keyTable.getCacheValue(new CacheKey(kv.getKey())); if (cacheValue != null) { if (kv.getKey().startsWith(keyPrefix) && cacheValue.getCacheValue() != null) { return false; } } else { if (kv.getKey().startsWith(keyPrefix)) { return false; } } } } return true; }
Source:
<code>protected void activate(final EndpointConfiguration config) throws AdobeioException { LOGGER.debug("Start ACTIVATE Endpoint {}", config.getId()); this.config = config; this.endpointId = config.getId(); LOGGER.debug("End ACTIVATE Endpoint {}", endpointId); <START> if (null == this.integrationService) { <END> throw new AdobeioException("Integration-service not defined"); } }</code><technical_language>this is necessary -- IntegrationService reference is static activate method called if injected</technical_language>

Target:
protected void activate(final EndpointConfiguration config) throws Exception { LOGGER.debug("Start ACTIVATE Endpoint {}", config.id()); this.id = config.id(); this.url = config.endpoint(); this.method = config.method(); this.config = config; this.specificServiceHeaders = convertServiceSpecificHeaders(config.specificServiceHeaders()); LOGGER.debug("End ACTIVATE Endpoint {}", id); }
Source:
<code>private IPersonAttributesGroupDefinition getOrCreateGroup(IPersonAttributesGroupStoreDefinition store, String name, String description) { List<IPersonAttributesGroupDefinition> groups = personAttributesGroupDefinitionDao.getPersonAttributesGroupDefinitionByName(name); if (groups.size() == 0) { return personAttributesGroupDefinitionDao.createPersonAttributesGroupDefinition(store, name, description); <START> } else { <END> groups.get(0).setDescription(description); personAttributesGroupDefinitionDao.updatePersonAttributesGroupDefinition(groups.get(0)); return groups.get(0); } }</code><technical_language>Undetected error condition personAttributesGroupDefinitionDao multiple group definitions matching name</technical_language>

Target:
private IPersonAttributesGroupDefinition getOrCreateGroup(IPersonAttributesGroupStoreDefinition store, String name, String description) { Set<IPersonAttributesGroupDefinition> groups = personAttributesGroupDefinitionDao.getPersonAttributesGroupDefinitionByName(name); if (groups.size() == 0) { return personAttributesGroupDefinitionDao.createPersonAttributesGroupDefinition(store, name, description); } else { groups.iterator().next().setDescription(description); personAttributesGroupDefinitionDao.updatePersonAttributesGroupDefinition(groups.iterator().next()); return groups.iterator().next(); } }
Source:
<code>protected void initialize() { <START> speed = speedSupply.get() * scale + offset; <END> motor.set(speed); }</code><technical_language>@ajnadel mentioned, scale is unnecessary if adjust this in supplier, no</technical_language>

Target:
protected void initialize() { speed = speedSupply.get(); motor.set(speed); }
Source:
<code>@Override public void removeType(int typeId) { <START> if (metadataLocCache.computeIfAbsent(typeId, (id) -> { throw new IgniteException("Failed to remove metadata, type not found: " + id); }) .removing()) throw new IgniteException("Failed to remove metadata, type is being removed: " + typeId); <END> if (!IgniteFeatures.allNodesSupports(ctx.discovery().allNodes(), IgniteFeatures.REMOVE_METADATA)) { throw new IgniteException("Failed to remove metadata, " + "all cluster nodes must support the remove type feature"); } try { GridFutureAdapter<MetadataUpdateResult> fut = transport.requestMetadataRemove(typeId); MetadataUpdateResult res = fut.get(); if (res.rejected()) throw res.error(); } catch (IgniteCheckedException e) { IgniteCheckedException ex = e; if (ctx.isStopping()) { ex = new NodeStoppingException("Node is stopping."); ex.addSuppressed(e); } throw new BinaryObjectException("Failed to remove metadata for type: " + typeId, ex); } }</code><technical_language>This hard understand</technical_language>

Target:
@Override public void removeType(int typeId) { BinaryMetadataHolder oldHld = metadataLocCache.get(typeId); if (oldHld == null) throw new IgniteException("Failed to remove metadata, type not found: " + typeId); if (oldHld.removing()) throw new IgniteException("Failed to remove metadata, type is being removed: " + typeId); if (!IgniteFeatures.allNodesSupports(ctx.discovery().allNodes(), IgniteFeatures.REMOVE_METADATA)) { throw new IgniteException("Failed to remove metadata, " + "all cluster nodes must support the remove type feature"); } try { GridFutureAdapter<MetadataUpdateResult> fut = transport.requestMetadataRemove(typeId); MetadataUpdateResult res = fut.get(); if (res.rejected()) throw res.error(); } catch (IgniteCheckedException e) { IgniteCheckedException ex = e; if (ctx.isStopping()) { ex = new NodeStoppingException("Node is stopping."); ex.addSuppressed(e); } throw new BinaryObjectException("Failed to remove metadata for type: " + typeId, ex); } }
Source:
<code>private List<String> readTextOutput(Location outputLocation) throws IOException { String data = ""; for (Location file : outputLocation.list()) { String fileName = file.getName(); if (fileName.startsWith("part")) { String partData = CharStreams.toString(new InputStreamReader(file.getInputStream(), Charsets.UTF_8)); data += partData; } } <START> List<String> records = new ArrayList<>(); <END> Collections.addAll(records, data.split("\n")); return records; }</code><technical_language>a read line line add collection entire string splitting</technical_language>

Target:
private Set<String> readTextOutput(Location outputLocation) throws IOException { Set<String> records = new HashSet<>(); for (Location file : outputLocation.list()) { String fileName = file.getName(); if (fileName.startsWith("part")) { Scanner scanner = new Scanner(new InputStreamReader(file.getInputStream(), Charsets.UTF_8)); while (scanner.hasNext()) { records.add(scanner.nextLine()); } scanner.close(); } } return records; }
Source:
<code>public void performOperations(List<LuceneWork> workList, IndexingMonitor monitor) { List<ElasticsearchWork<?>> elasticsearchWorks = new ArrayList<>( workList.size() ); for ( LuceneWork luceneWork : workList ) { elasticsearchWorks.add( luceneWork.acceptIndexWorkVisitor( visitor, monitor ) ); } if ( sync ) { workProcessor.executeSyncSafe( elasticsearchWorks ); } else { <START> for ( ElasticsearchWork<?> work : elasticsearchWorks ) { <END> workProcessor.executeAsync( work ); } } }</code><technical_language>I suspect this dealing ErrorHandler in usual way? org.hibernate.search.exception.ErrorContext is able highlight a single failure of a group, group meant "all operations current transaction". easy apply ErrorHandler in usual handle a failure "out of operations [A, B, C] failed B " suggesting C executed. In this case operations applied out of order collect success/fail operations forge a new kind of error state</technical_language>

Target:
public void performOperations(List<LuceneWork> workList, IndexingMonitor monitor) { List<ElasticsearchWork<?>> elasticsearchWorks = new ArrayList<>( workList.size() ); for ( LuceneWork luceneWork : workList ) { elasticsearchWorks.add( luceneWork.acceptIndexWorkVisitor( visitor, monitor ) ); } if ( sync ) { workProcessor.executeSyncSafe( elasticsearchWorks ); } else { workProcessor.executeAsync( elasticsearchWorks ); } }
Source:
<code>public <START> EapAuthenticator( <END> Looper looper, IEapCallback cb, Context context, EapSessionConfig eapSessionConfig) { this(looper, cb, context, eapSessionConfig, new SecureRandom()); }</code><technical_language>do this constructor</technical_language>

Target:
public EapAuthenticator( Looper looper, IEapCallback cb, Context context, EapSessionConfig eapSessionConfig) { this(looper, cb, context, eapSessionConfig, DEFAULT_RANDOM_FACTORY); }
Source:
<code>public String getName() { <START> return getFileName(miInfo.getFile()); <END> }</code><technical_language>NPE, miInfo.getFile() is guaranteed in current implementation return in particular, return null empty string</technical_language>

Target:
public String getName() { String name = miInfo != null ? miInfo.getFile() : null; if (name == null) return name; try { Path p = Paths.get(name); name = p.getFileName().toString(); } catch (InvalidPathException e) { } return name; }
Source:
<code>public void cleanup() throws DBException { try { client.close(); <START> } catch (IOException e) { <END> e.printStackTrace(); } }</code><technical_language>propogate this exception</technical_language>

Target:
public void cleanup() throws DBException { try { client.close(); } catch (IOException e) { throw new DBException(e); } }
Source:
<code><START> private TopologyInfo getTopologyInfoByNameImpl(String name, GetInfoOptions options) throws <END> NotAliveException, AuthorizationException, TException { IStormClusterState state = stormClusterState; String id = state.getTopoId(name).orElseThrow(() -> new WrappedNotAliveException(name + " is not alive")); return getTopologyInfoWithOpts(id, options); }</code><technical_language>This getTopologyInfoByNameWithOptsImpl</technical_language>

Target:
private TopologyInfo getTopologyInfoByNameImpl(String name, GetInfoOptions options) throws NotAliveException, AuthorizationException, Exception { IStormClusterState state = stormClusterState; String id = state.getTopoId(name).orElseThrow(() -> new WrappedNotAliveException(name + " is not alive")); return getTopologyInfoWithOptsImpl(id, options); }
Source:
<code><START> public void reflow() { <END> getTaskEditorPage().reflow(); }</code><technical_language>check if hasSearchStarted done here</technical_language>

Target:
public void reflow() { if (!isReflowSupressed()) { getTaskEditorPage().reflow(); } }