auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public void toggleFullscreen() { if (DEBUG) Log.d(TAG, "toggleFullscreen() called"); if (simpleExoPlayer == null || getCurrentMetadata() == null) return; if (popupPlayerSelected()) { setRecovery(); service.removeViewFromParent(); <START> Intent intent = NavigationHelper.getPlayerIntent( <END> service, MainActivity.class, this.getPlayQueue(), this.getRepeatMode(), this.getPlaybackSpeed(), this.getPlaybackPitch(), this.getPlaybackSkipSilence(), null, true ); intent.addFlags(Intent.FLAG_ACTIVITY_NEW_TASK); intent.putExtra(Constants.KEY_SERVICE_ID, getCurrentMetadata().getMetadata().getServiceId()); intent.putExtra(Constants.KEY_LINK_TYPE, StreamingService.LinkType.STREAM); intent.putExtra(Constants.KEY_URL, getVideoUrl()); intent.putExtra(Constants.KEY_TITLE, getVideoTitle()); intent.putExtra(VideoDetailFragment.AUTO_PLAY, true); service.onDestroy(); context.startActivity(intent); return; } else { if (fragmentListener == null) return; isFullscreen = !isFullscreen; setControlsSize(); fragmentListener.onFullscreenStateChanged(isInFullscreen()); } if (!isInFullscreen()) { titleTextView.setVisibility(View.GONE); channelTextView.setVisibility(View.GONE); playerCloseButton.setVisibility(videoPlayerSelected() ? View.VISIBLE : View.GONE); } else { titleTextView.setVisibility(View.VISIBLE); channelTextView.setVisibility(View.VISIBLE); playerCloseButton.setVisibility(View.GONE); } setupScreenRotationButton(); }</code><technical_language>I this kind of switching in places, a common function created</technical_language>

Target:
public void toggleFullscreen() { if (DEBUG) { Log.d(TAG, "toggleFullscreen() called"); } if (simpleExoPlayer == null || getCurrentMetadata() == null) { return; } if (popupPlayerSelected()) { setRecovery(); service.removeViewFromParent(); Intent intent = NavigationHelper.getPlayerIntent( service, MainActivity.class, this.getPlayQueue(), this.getRepeatMode(), this.getPlaybackSpeed(), this.getPlaybackPitch(), this.getPlaybackSkipSilence(), null, true, !isPlaying(), isMuted() ); intent.addFlags(Intent.FLAG_ACTIVITY_NEW_TASK); intent.putExtra(Constants.KEY_SERVICE_ID, getCurrentMetadata().getMetadata().getServiceId()); intent.putExtra(Constants.KEY_LINK_TYPE, StreamingService.LinkType.STREAM); intent.putExtra(Constants.KEY_URL, getVideoUrl()); intent.putExtra(Constants.KEY_TITLE, getVideoTitle()); intent.putExtra(VideoDetailFragment.AUTO_PLAY, true); service.onDestroy(); context.startActivity(intent); return; } else { if (fragmentListener == null) { return; } isFullscreen = !isFullscreen; setControlsSize(); fragmentListener.onFullscreenStateChanged(isFullscreen()); } if (!isFullscreen()) { titleTextView.setVisibility(View.GONE); channelTextView.setVisibility(View.GONE); playerCloseButton.setVisibility(videoPlayerSelected() ? View.VISIBLE : View.GONE); } else { titleTextView.setVisibility(View.VISIBLE); channelTextView.setVisibility(View.VISIBLE); playerCloseButton.setVisibility(View.GONE); } setupScreenRotationButton(); }
Source:
<code>CircuitCreationTask(TorConfig config, Directory directory, ConnectionCache connectionCache, CircuitPathChooser pathChooser, CircuitManagerImpl circuitManager, TorInitializationTracker initializationTracker) { this.config = config; this.directory = directory; this.connectionCache = connectionCache; this.circuitManager = circuitManager; this.initializationTracker = initializationTracker; this.pathChooser = pathChooser; <START> this.executor = Executors.newCachedThreadPool(new ThreadFactory() { <END> @Override public Thread newThread(Runnable r) { Thread t = new Thread(r); t.setName("CircuitCreationTask worker"); t.setDaemon(true); return t; } }); this.buildHandler = createCircuitBuildHandler(); this.internalBuildHandler = createInternalCircuitBuildHandler(); this.predictor = new CircuitPredictor(); this.lastNewCircuit = new AtomicLong(); }</code><technical_language>Guava ThreadFactoryBuilder for this: Executes.newCachedThreadPool(new ThreadBuilderFactory().setDaemon(true).setName("CircuitCreationTask worker").build()); written out unconvinced this is (damn java's verbosity) I leave change do in future</technical_language>

Target:
CircuitCreationTask(TorConfig config, Directory directory, ConnectionCache connectionCache, CircuitPathChooser pathChooser, CircuitManagerImpl circuitManager, TorInitializationTracker initializationTracker) { this.config = config; this.directory = directory; this.connectionCache = connectionCache; this.circuitManager = circuitManager; this.initializationTracker = initializationTracker; this.pathChooser = pathChooser; this.executor = Threading.newPool("CircuitCreationTask worker"); this.buildHandler = createCircuitBuildHandler(); this.internalBuildHandler = createInternalCircuitBuildHandler(); this.predictor = new CircuitPredictor(); this.lastNewCircuit = new AtomicLong(); }
Source:
<code>private void handleAuthenticationRequest(HttpServletRequest request, HttpServletResponse response) throws IOException, ServletException { Map paramMap = request.getParameterMap(); SessionDTO sessionDTO = new SessionDTO(); sessionDTO.setAction(getAttribute(paramMap, PassiveRequestorConstants.ACTION)); sessionDTO.setAttributes(getAttribute(paramMap, PassiveRequestorConstants.ATTRIBUTE)); sessionDTO.setContext(getAttribute(paramMap, PassiveRequestorConstants.CONTEXT)); sessionDTO.setReplyTo(getAttribute(paramMap, PassiveRequestorConstants.REPLY_TO)); sessionDTO.setPseudo(getAttribute(paramMap, PassiveRequestorConstants.PSEUDO)); sessionDTO.setRealm(getAttribute(paramMap, PassiveRequestorConstants.REALM)); sessionDTO.setRequest(getAttribute(paramMap, PassiveRequestorConstants.REQUEST)); sessionDTO.setRequestPointer(getAttribute(paramMap, PassiveRequestorConstants.REQUEST_POINTER)); sessionDTO.setPolicy(getAttribute(paramMap, PassiveRequestorConstants.POLCY)); <START> sessionDTO.setTenantDomain(getAttribute(paramMap, "tenantDomain")); <END> sessionDTO.setReqQueryString(request.getQueryString()); String sessionDataKey = UUIDGenerator.generateUUID(); addSessionDataToCache(sessionDataKey, sessionDTO, IdPManagementUtil.getIdleSessionTimeOut (CarbonContext.getThreadLocalCarbonContext().getTenantDomain())); sendToAuthenticationFramework(request, response, sessionDataKey, sessionDTO); }</code><technical_language>constant MultitenantConstants.TENANT_DOMAIN</technical_language>

Target:
private void handleAuthenticationRequest(HttpServletRequest request, HttpServletResponse response) throws IOException, ServletException { Map paramMap = request.getParameterMap(); SessionDTO sessionDTO = new SessionDTO(); sessionDTO.setAction(getAttribute(paramMap, PassiveRequestorConstants.ACTION)); sessionDTO.setAttributes(getAttribute(paramMap, PassiveRequestorConstants.ATTRIBUTE)); sessionDTO.setContext(getAttribute(paramMap, PassiveRequestorConstants.CONTEXT)); sessionDTO.setReplyTo(getAttribute(paramMap, PassiveRequestorConstants.REPLY_TO)); sessionDTO.setPseudo(getAttribute(paramMap, PassiveRequestorConstants.PSEUDO)); sessionDTO.setRealm(getAttribute(paramMap, PassiveRequestorConstants.REALM)); sessionDTO.setRequest(getAttribute(paramMap, PassiveRequestorConstants.REQUEST)); sessionDTO.setRequestPointer(getAttribute(paramMap, PassiveRequestorConstants.REQUEST_POINTER)); sessionDTO.setPolicy(getAttribute(paramMap, PassiveRequestorConstants.POLCY)); sessionDTO.setTenantDomain(getAttribute(paramMap, MultitenantConstants.TENANT_DOMAIN)); sessionDTO.setReqQueryString(request.getQueryString()); String sessionDataKey = UUIDGenerator.generateUUID(); addSessionDataToCache(sessionDataKey, sessionDTO, IdPManagementUtil.getIdleSessionTimeOut (CarbonContext.getThreadLocalCarbonContext().getTenantDomain())); sendToAuthenticationFramework(request, response, sessionDataKey, sessionDTO); }
Source:
<code>protected void configure() { try { ConfigurationManager.loadCascadedPropertiesFromResources("application"); } catch (Exception ex) { <START> <END> } bind(AbstractConfiguration.class).toInstance(ConfigurationManager.getConfigInstance()); bind(BaseServerStartup.class).to(SampleServerStartup.class); bind(OriginManager.class).to(BasicNettyOriginManager.class); install(new ZuulFiltersModule()); bind(FilterFileManager.class).asEagerSingleton(); bind(ServerStatusManager.class); bind(SessionContextDecorator.class).to(ZuulSessionContextDecorator.class); bind(Registry.class).to(DefaultRegistry.class); bind(RequestCompleteHandler.class).to(BasicRequestCompleteHandler.class); bind(AbstractDiscoveryClientOptionalArgs.class).to(DiscoveryClient.DiscoveryClientOptionalArgs.class); bind(RequestMetricsPublisher.class).to(BasicRequestMetricsPublisher.class); bind(AccessLogPublisher.class).toInstance(new AccessLogPublisher("ACCESS", (channel, httpRequest) -> ClientRequestReceiver.getRequestFromChannel(channel).getContext().getUUID())); }</code><technical_language>this throw a RuntimeException</technical_language>

Target:
protected void configure() { try { ConfigurationManager.loadCascadedPropertiesFromResources("application"); } catch (Exception ex) { throw new RuntimeException("Error loading configuration: " + ex.getMessage(), ex); } bind(AbstractConfiguration.class).toInstance(ConfigurationManager.getConfigInstance()); bind(BaseServerStartup.class).to(SampleServerStartup.class); bind(OriginManager.class).to(BasicNettyOriginManager.class); install(new ZuulFiltersModule()); bind(FilterFileManager.class).asEagerSingleton(); bind(ServerStatusManager.class); bind(SessionContextDecorator.class).to(ZuulSessionContextDecorator.class); bind(Registry.class).to(DefaultRegistry.class); bind(RequestCompleteHandler.class).to(BasicRequestCompleteHandler.class); bind(AbstractDiscoveryClientOptionalArgs.class).to(DiscoveryClient.DiscoveryClientOptionalArgs.class); bind(RequestMetricsPublisher.class).to(BasicRequestMetricsPublisher.class); bind(AccessLogPublisher.class).toInstance(new AccessLogPublisher("ACCESS", (channel, httpRequest) -> ClientRequestReceiver.getRequestFromChannel(channel).getContext().getUUID())); }
Source:
<code>public List<Object[]> parse(File file, HCatSchema schema, List<String> names) { try { List<String> lines = Files.readAllLines(file.toPath(), charset); <START> <END> if (this.header) { lines.remove(0); } List<Object[]> records = new ArrayList<>(lines.size()); for (String line : lines) { records.add(parseRow(line, names.size())); } return records; } catch (IOException e) { throw new RuntimeException("Error while reading file", e); } }</code><technical_language>safer construct a new List subList assumption Files.readAllLines return a concrete List supports optional remove operation</technical_language>

Target:
public List<Object[]> parse(File file, HCatSchema schema, List<String> names) { try { List<String> lines = Files.readAllLines(file.toPath(), charset); if (this.hasHeader) { lines = lines.subList(1, lines.size()); } List<Object[]> records = new ArrayList<>(lines.size()); for (String line : lines) { records.add(parseRow(line, names.size())); } return records; } catch (IOException e) { throw new RuntimeException("Error while reading file", e); } }
Source:
<code>public AsyncPoolStats getStats() { <START> AsyncPoolStats stats = null; <END> synchronized (_lock) { stats = new AsyncPoolStats( _totalCreated, _totalDestroyed, _createErrors, _destroyErrors, _checkedOut, _timedOut, _badDestroyed, _maxSize, _poolSize, _sampleMaxCheckedOut, _sampleMaxPoolSize ); _sampleMaxCheckedOut = _checkedOut; _sampleMaxPoolSize = _poolSize; } return stats; }</code><technical_language>need declaration synchronized. write synchronized (_lock) { return new AsyncPoolStats(....); }</technical_language>

Target:
public AsyncPoolStats getStats() { synchronized (_lock) { AsyncPoolStats stats = new AsyncPoolStats( _totalCreated, _totalDestroyed, _totalCreateErrors, _totalDestroyErrors, _totalBadDestroyed, _totalTimedOut, _checkedOut, _maxSize, _poolSize, _sampleMaxCheckedOut, _sampleMaxPoolSize ); _sampleMaxCheckedOut = _checkedOut; _sampleMaxPoolSize = _poolSize; return stats; } }
Source:
<code><START> private DefaultTestInstances(List<Object> instances) { <END> this.instances = instances; }</code><technical_language>of Preconditions safe side</technical_language>

Target:
private DefaultTestInstances(List<Object> instances) { this.instances = Preconditions.notEmpty(instances, "instances must not be empty"); }
Source:
<code>protected void checkFile(@NotNull GoFile file, @NotNull final ProblemsHolder problemsHolder) { final MultiMap<String, GoMethodDeclaration> map = new MultiMap<String, GoMethodDeclaration>(); List<GoFile> files = GoPsiImplUtil.getAllPackageFiles(file); <START> for (GoFile goFile : files) { <END> for (GoMethodDeclaration method : goFile.getMethods()) { if (method.getReceiver().getType() == null) continue; map.putValue(method.getName(), method); } } file.accept(new GoRecursiveVisitor() { @Override public void visitMethodDeclaration(@NotNull final GoMethodDeclaration m) { final GoType type = m.getReceiver().getType(); if (type == null) return; final String name = m.getName(); if (name == null) return; Collection<GoMethodDeclaration> byKey = map.get(name); if (byKey.size() <= 1) return; byKey = ContainerUtil.filter(byKey, new Condition<GoMethodDeclaration>() { @Override public boolean value(GoMethodDeclaration d) { return Comparing.equal(d.getReceiver().getType().getText(), type.getText()); } }); if (byKey.size() <= 1) return; PsiElement identifier = m.getNameIdentifier(); problemsHolder.registerProblem(identifier == null ? m : identifier, "Duplicate method name"); } }); }</code><technical_language>Please GoMethodIndex instead of constructing map</technical_language>

Target:
protected void checkFile(@NotNull GoFile file, @NotNull final ProblemsHolder problemsHolder) { final PsiDirectory parent = file.getParent(); if (parent == null) return; final Project project = file.getProject(); final String packageName = file.getPackageName(); final GlobalSearchScope scope = GlobalSearchScopesCore.directoryScope(parent, false); for (GoMethodDeclaration m : file.getMethods()) { if (m.isBlank()) { continue; } GoType type = m.getReceiver().getType(); if (type == null) continue; final String methodName = m.getName(); if (methodName == null) continue; String key = packageName + "." + type.getText(); Collection<GoMethodDeclaration> declarations = GoMethodIndex.find(key, project, scope); declarations = ContainerUtil.filter(declarations, new Condition<GoMethodDeclaration>() { @Override public boolean value(GoMethodDeclaration d) { return Comparing.equal(d.getName(), methodName); } }); if (declarations.size() <= 1) continue; PsiElement identifier = m.getNameIdentifier(); problemsHolder.registerProblem(identifier == null ? m : identifier, "Duplicate method name"); } }
Source:
<code>public void saveOrUpdateShouldSaveAProperApplicationDTO() { given(converterService.convert(applicationDTO, ApplicationEntity.class)) .willReturn(applicationEntity); given(applicationsRepository.saveAndFlush(applicationEntity)) .willReturn(this.applicationEntity); given(converterService.convert(applicationEntity, ApplicationDTO.class)) .willReturn(applicationDTO); Long result = applicationsService.saveOrUpdate(applicationDTO); assertThat(result, notNullValue()); assertThat(result, equalTo(applicationDTO.getId())); <START> then(converterService).should().convert(applicationDTO, ApplicationEntity.class); <END> then(applicationsRepository).should().saveAndFlush(applicationEntity); }</code><technical_language>Remove matching method call well</technical_language>

Target:
public void saveOrUpdateShouldSaveAProperApplicationDTO() { given(converterService.convert(applicationDTO, ApplicationEntity.class)) .willReturn(applicationEntity); given(applicationsRepository.saveAndFlush(applicationEntity)) .willReturn(this.applicationEntity); Long result = applicationsService.saveOrUpdate(applicationDTO); assertThat(result, notNullValue()); assertThat(result, equalTo(applicationDTO.getId())); then(converterService).should().convert(applicationDTO, ApplicationEntity.class); then(applicationsRepository).should().saveAndFlush(applicationEntity); }
Source:
<code>protected ResourceSet loadResource(IEditorInput editorInput) { <START> final URI resourceURI = EditUIUtil.getURI(getEditorInput(), null); <END> return ResourceSetHelpers.loadResourceSetWithProxies(resourceURI, getCommandStack()); }</code><technical_language>passed in editorInput</technical_language>

Target:
protected ResourceSet loadResource(IEditorInput editorInput) { final URI resourceURI = EditUIUtil.getURI(editorInput, null); return ResourceSetHelpers.loadResourceSetWithProxies(resourceURI, getCommandStack()); }
Source:
<code><START> void runMetricsSenderThread() { <END> metricsSenderThread.execute(new Runnable() { @Override public void run() { while (true) { while (!metricsRequestQueue.isEmpty()) { sendMetricsToDashboard(metricsRequestQueue.poll()); } } } }); }</code><technical_language>Is reason remove 'private' access modifier</technical_language>

Target:
private void runMetricsSenderThread() { metricsSenderExecutor.execute(new Runnable() { @Override public void run() { while (true) { try { final String request = metricsRequestQueue.take(); sendMetricsToDashboard(request); } catch (InterruptedException e) { LOG.log(Level.WARNING, "Dashboard: Interrupted while sending metrics to the dashboard server.", e); } } } }); }
Source:
<code><START> public boolean equals(Object other) { <END> return other == this || other instanceof Form && this.displayName.equals(((Form) other).displayName) && this.jrFormId.equals(((Form) other).jrFormId) && this.md5Hash.equals(((Form) other).md5Hash); }</code><technical_language>For equals hashCode, I md5Hash. feels odd a hash of a hash. display name form ID implicitly part of hash, right</technical_language>

Target:
public boolean equals(Object other) { return other == this || other instanceof Form && this.md5Hash.equals(((Form) other).md5Hash); }
Source:
<code>public List<ExtensionEntry> getExtensionsByService(String serviceName) { <START> return serviceEntries.get(serviceName); <END> }</code><technical_language>this check null return empty list no</technical_language>

Target:
public List<ExtensionEntry> getExtensionsByService(String serviceName) { return serviceEntries.containsKey(serviceName) ? serviceEntries.get(serviceName) : Collections.<ExtensionEntry> emptyList(); }
Source:
<code>public void visitDeclarationExpression(final DeclarationExpression exp) { if (exp.isMultipleAssignmentDeclaration()) { makeNode("sequence", new Runnable() { @Override public void run() { for (Expression e : exp.getTupleExpression().getExpressions()) { final VariableExpression v = (VariableExpression) e; makeNode("declareVariable", new Runnable() { @Override public void run() { literal(v.getType()); literal(v.getName()); } }); } if (exp.getRightExpression() instanceof ListExpression) { multipleAssignment(exp, exp.getTupleExpression(), (ListExpression)exp.getRightExpression()); <START> } <END> } }); } else { makeNode("declareVariable", new Runnable() { @Override public void run() { VariableExpression v = exp.getVariableExpression(); loc(exp); literal(v.getType()); literal(v.getName()); visit(exp.getRightExpression()); } }); } }</code><technical_language>if not</technical_language>

Target:
public void visitDeclarationExpression(final DeclarationExpression exp) { if (exp.isMultipleAssignmentDeclaration()) { makeNode("sequence", new Runnable() { @Override public void run() { for (Expression e : exp.getTupleExpression().getExpressions()) { final VariableExpression v = (VariableExpression) e; makeNode("declareVariable", new Runnable() { @Override public void run() { literal(v.getType()); literal(v.getName()); } }); } multipleAssignment(exp, exp.getTupleExpression(), exp.getRightExpression()); } }); } else { makeNode("declareVariable", new Runnable() { @Override public void run() { VariableExpression v = exp.getVariableExpression(); loc(exp); literal(v.getType()); literal(v.getName()); visit(exp.getRightExpression()); } }); } }
Source:
<code>protected List<String> getChildren(String root) { <START> LOG.debug("Preparing to call getChildren() on {}", root); <END> final long start = System.currentTimeMillis(); int numChildren = 0; try { final List<String> children = curator.getChildren().forPath(root); numChildren = children.size(); return children; } catch (NoNodeException nne) { return Collections.emptyList(); } catch (Throwable t) { throw Throwables.propagate(t); } finally { log(OperationType.GET_CHILDREN, Optional.of(numChildren), Optional.absent(), start, root); } }</code><technical_language>this information useful for us? If yes: suggest making trace() instead (it's pretty verbose). If not: please delete</technical_language>

Target:
protected List<String> getChildren(String root) { LOG.trace("Preparing to call getChildren() on {}", root); final long start = System.currentTimeMillis(); int numChildren = 0; try { final List<String> children = curator.getChildren().forPath(root); numChildren = children.size(); return children; } catch (NoNodeException nne) { return Collections.emptyList(); } catch (Throwable t) { throw Throwables.propagate(t); } finally { log(OperationType.GET_CHILDREN, Optional.of(numChildren), Optional.absent(), start, root); } }
Source:
<code>private void loadActions() { JSONArray actionList = apiConnection.execute("lists/actionmenus/", CallType.GET, new JsonArrayResultHandler(), session); for (int i = 0; i < actionList.length(); i++) { JSONObject action = actionList.getJSONObject(i).getJSONObject("ActionMenu"); JSONArray functions = action.getJSONArray("functionTypes"); JSONObject stakeholders = action.getJSONObject("activeForStakeholder"); JSONArray keys = stakeholders.names(); Map<Integer, JSONArray> functionsMap = new HashMap<Integer, JSONArray>(); for (int j = 0; j < keys.length(); j++) { if (stakeholders.getBoolean(keys.getString(j))) { <START> functionsMap.put(Integer.parseInt(keys.getString(j)), functions); <END> } } setFunctions(functionsMap); } }</code><technical_language>ParseInt of a getString changed keys.getInt(j)</technical_language>

Target:
private void loadActions() { JSONArray actionList = apiConnection.execute("lists/actionmenus/", CallType.GET, new JsonArrayResultHandler(), session); for (int i = 0; i < actionList.length(); i++) { JSONObject action = actionList.getJSONObject(i).getJSONObject("ActionMenu"); JSONArray functions = action.getJSONArray("functionTypes"); JSONObject stakeholders = action.getJSONObject("activeForStakeholder"); JSONArray keys = stakeholders.names(); Map<Integer, JSONArray> functionMap = new HashMap<Integer, JSONArray>(); for (int j = 0; j < keys.length(); j++) { if (stakeholders.getBoolean(keys.getString(j))) { functionMap.put(keys.getInt(j), functions); } } setFunctions(functionMap); } }
Source:
<code>public static ImageDescriptor <START> getImageDescriptor(String <END> id) { if (id.endsWith(".gif")){ id = id.replace(".gif", ".png"); } return TeamUIPlugin.getImageDescriptor(id); }</code><technical_language>I a mistake in place offer a generic method. API work for constants in ISharedTeamImages. convert concrete images Team public</technical_language>

Target:
public static ImageDescriptor getImageDescriptor(String id) { String key = null; switch (id) { case ISharedImages.IMG_DIRTY_OVR: key = TeamUIPlugin.FILE_DIRTY_OVR; break; case ISharedImages.IMG_CHECKEDIN_OVR: key = TeamUIPlugin.FILE_CHECKEDIN_OVR; break; case ISharedImages.IMG_CHECKEDOUT_OVR: key = TeamUIPlugin.FILE_CHECKEDOUT_OVR; break; case ISharedImages.IMG_CONFLICT_OVR: key = TeamUIPlugin.FILE_CONFLICT_OVR; break; case ISharedImages.IMG_ERROR_OVR: key = TeamUIPlugin.FILE_ERROR_OVR; break; case ISharedImages.IMG_WARNING_OVR: key = TeamUIPlugin.FILE_WARNING_OVR; break; case ISharedImages.IMG_HOURGLASS_OVR: key = TeamUIPlugin.FILE_HOURGLASS_OVR; break; default: key = id; } return TeamUIPlugin.getImageDescriptor(key); }
Source:
<code>public javax.cache.CacheManager ehcache3TicketCacheManager( @Qualifier ("ehcache3CacheManagerConfiguration") final ServiceCreationConfiguration ehcache3CacheManagerConfiguration) { val ehcacheProperties = casProperties.getTicket().getRegistry().getEhcache3(); val cachingProvider = Caching.getCachingProvider(EhcacheCachingProvider.class.getName()); <START> val ehcacheProvider = (EhcacheCachingProvider) cachingProvider; <END> val statisticsAllEnabled = ehcacheProperties.isEnableStatistics() ? ConfigurationElementState.ENABLED : ConfigurationElementState.DISABLED; val managementAllAllEnabled = ehcacheProperties.isEnableManagement() ? ConfigurationElementState.ENABLED : ConfigurationElementState.DISABLED; val jsr107Config = new Jsr107Configuration(null, new HashMap<>(), false, managementAllAllEnabled, statisticsAllEnabled); val configuration = new DefaultConfiguration(ehcacheProvider.getDefaultClassLoader(), ehcache3CacheManagerConfiguration, jsr107Config); return ehcacheProvider.getCacheManager(ehcacheProvider.getDefaultURI(), configuration); }</code><technical_language>Merge line above</technical_language>

Target:
public javax.cache.CacheManager ehcache3TicketCacheManager( @Qualifier ("ehcache3CacheManagerConfiguration") final ServiceCreationConfiguration ehcache3CacheManagerConfiguration) { val ehcacheProperties = casProperties.getTicket().getRegistry().getEhcache3(); val ehcacheProvider = (EhcacheCachingProvider) Caching.getCachingProvider(EhcacheCachingProvider.class.getName()); val statisticsAllEnabled = ehcacheProperties.isEnableStatistics() ? ConfigurationElementState.ENABLED : ConfigurationElementState.DISABLED; val managementAllAllEnabled = ehcacheProperties.isEnableManagement() ? ConfigurationElementState.ENABLED : ConfigurationElementState.DISABLED; val jsr107Config = new Jsr107Configuration(null, new HashMap<>(), false, managementAllAllEnabled, statisticsAllEnabled); val configuration = new DefaultConfiguration(ehcacheProvider.getDefaultClassLoader(), ehcache3CacheManagerConfiguration, jsr107Config); return ehcacheProvider.getCacheManager(ehcacheProvider.getDefaultURI(), configuration); }
Source:
<code>if (o != null) { if (o instanceof String) { return $((String) o); } if (o instanceof SafeHtml) { return $(((SafeHtml) o).asString()); } if (o instanceof GQuery) { return (GQuery) o; } if (o instanceof Function) { return new GQuery(((Function) o).getElement()); } if (o instanceof JsonBuilder) { return new GQuery(((JsonBuilder) o).<Element>getDataImpl()); } if (o instanceof IsWidget) { return $(Arrays.asList(o)); } if (o instanceof JavaScriptObject) { JavaScriptObject jso = (JavaScriptObject) o; if (JsUtils.isFunction(jso)) { new JsUtils.JsFunction(jso).fe(); return $(); } if (!JsUtils.isWindow(jso) && !JsUtils.isElement(jso) && JsUtils.isArray(jso)) { JsArrayMixed c = jso.cast(); JsNodeArray elms = JsNodeArray.create(); for (int i = 0; i < c.length(); i++) { Object obj = c.getObject(i); if (obj instanceof Node) { elms.addNode((Node) obj); } } return new GQuery(elms); } if (JsUtils.isNodeList(jso)) { return new GQuery(jso.<NodeList<Element>> cast()); } if (JsUtils.isEvent(jso)) { jso = jso.<Event>cast().getCurrentEventTarget(); } return new GQuery(jso.<Element> cast()); } <START> console <END> .log("Error: GQuery.$(Object o) could not wrap the type : ", o.getClass().getName(), o); }</code><technical_language>in favor of this general constructor... java errors (people want wrap object is this method) detected compile time runtime...</technical_language>

Target:
if (o != null) { if (o instanceof String) { return $((String) o); } if (o instanceof SafeHtml) { return $(((SafeHtml) o).asString()); } if (o instanceof GQuery) { return (GQuery) o; } if (o instanceof Function) { return new GQuery(((Function) o).getElement()); } if (o instanceof JsonBuilder) { return new GQuery(((JsonBuilder) o).<Element>getDataImpl()); } if (o instanceof IsWidget) { return $(Arrays.asList(o)); } if (o instanceof JavaScriptObject) { JavaScriptObject jso = (JavaScriptObject) o; if (JsUtils.isFunction(jso)) { new JsUtils.JsFunction(jso).fe(); return $(); } if (!JsUtils.isWindow(jso) && !JsUtils.isElement(jso) && JsUtils.isArray(jso)) { JsArrayMixed c = jso.cast(); JsNodeArray elms = JsNodeArray.create(); for (int i = 0; i < c.length(); i++) { Object obj = c.getObject(i); if (obj instanceof Node) { elms.addNode((Node) obj); } } return new GQuery(elms); } if (JsUtils.isNodeList(jso)) { return new GQuery(jso.<NodeList<Element>> cast()); } if (JsUtils.isEvent(jso)) { jso = jso.<Event>cast().getCurrentEventTarget(); } return new GQuery(jso.<Element> cast()); } throw new RuntimeException("Error: GQuery.$(Object o) could not wrap the type : " + o.getClass().getName() + " " + o); }
Source:
<code>public static MambuAPIServiceFactory getFactory(String domain, String username, String password) { <START> Injector injector = Guice.createInjector(new MambuAPIModule(Protocol.HTTPS, domain, username, password, "Mambu SDKv.10")); <END> return new MambuAPIServiceFactory(injector); }</code><technical_language>invalid - v 10</technical_language>

Target:
public static MambuAPIServiceFactory getFactory(String domain, String username, String password) { Injector injector = Guice.createInjector(new MambuAPIModule(Protocol.HTTPS, domain, username, password, DEFAULT_USER_AGENT_HEADER_VALUE)); return new MambuAPIServiceFactory(injector); }
Source:
<code>public void applyTo(BlockHound.Builder builder) { allowTestsToBlock(builder); builder.allowBlockingCallsInside(CoreTestBlockHoundIntegration.class.getName(), "writeJUnitReport"); builder.blockingMethodCallback(bm -> <START> writeJUnitReport(new AssertionError(String.format("Blocking call! %s on thread %s", bm, Thread.currentThread())), "Blocking")); <END> Thread.setDefaultUncaughtExceptionHandler((thread, t) -> { LogFactory.getLogger("Infinispan-TEST").fatal("Throwable was not caught in thread " + thread + " - exception is: " + t); if (!(t instanceof UndeliverableException)) { writeJUnitReport(t, "Uncaught"); } }); }</code><technical_language>Please do a TestSuiteProgress.fakeTestFailure(testName, exception), I a script for Test failed messages in output I run tests locally :)</technical_language>

Target:
public void applyTo(BlockHound.Builder builder) { allowTestsToBlock(builder); builder.allowBlockingCallsInside(CoreTestBlockHoundIntegration.class.getName(), "writeJUnitReport"); builder.blockingMethodCallback(bm -> { String testName = TestResourceTracker.getCurrentTestName(); AssertionError assertionError = new AssertionError(String.format("Blocking call! %s on thread %s", bm, Thread.currentThread())); TestSuiteProgress.fakeTestFailure(testName + ".BlockingChecker", assertionError); writeJUnitReport(testName, assertionError, "Blocking"); throw assertionError; }); Thread.setDefaultUncaughtExceptionHandler((thread, t) -> { LogFactory.getLogger("Infinispan-TEST").fatal("Throwable was not caught in thread " + thread + " - exception is: " + t); if (!(t instanceof UndeliverableException)) { writeJUnitReport(TestResourceTracker.getCurrentTestName(), t, "Uncaught"); } }); }
Source:
<code>public ClearSiteDataServerHttpHeadersWriter(Directive... directives) { <START> Assert.notEmpty(directives, "Directives cannot be empty or null."); <END> this.headerWriterDelegate = StaticServerHttpHeadersWriter.builder() .header(CLEAR_SITE_DATA_HEADER, transformToHeaderValue(directives)) .build(); }</code><technical_language>Please lowercase "Directives" here, a bit easier map error message parameter name</technical_language>

Target:
public ClearSiteDataServerHttpHeadersWriter(Directive... directives) { Assert.notEmpty(directives, "directives cannot be empty or null."); this.headerWriterDelegate = StaticServerHttpHeadersWriter.builder() .header(CLEAR_SITE_DATA_HEADER, transformToHeaderValue(directives)) .build(); }
Source:
<code>public void setUiScale(float scale) { <START> canvas.setUiScale(1.0f / scale); <END> canvasRenderer.setUiScale(scale); }</code><technical_language>sense bigger elements scale is bigger. Is reason choose invert this</technical_language>

Target:
public void setUiScale(float scale) { canvas.setUiScale(scale); canvasRenderer.setUiScale(1.0f / scale); }
Source:
<code>public void onPreAdvertiseRefs(UploadPack up) throws UploadPackMayNotContinueException { <START> for (PreUploadHook hook : hooks) { <END> hook.onPreAdvertiseRefs(up); } }</code><technical_language>curly braces</technical_language>

Target:
public void onPreAdvertiseRefs(UploadPack up) throws UploadPackMayNotContinueException { for (int i = 0; i < count; i++) hooks[i].onPreAdvertiseRefs(up); }
Source:
<code>public Widget getWidget(String itemName) { for (ItemUIProvider provider : itemUIProviders) { Widget currentWidget = provider.getWidget(itemName); if (currentWidget != null) { <START> if (currentWidget instanceof Default) { <END> currentWidget = resolveDefault(currentWidget, null); } return currentWidget; } } return null; }</code><technical_language>removed</technical_language>

Target:
public Widget getWidget(String itemName) { for (ItemUIProvider provider : itemUIProviders) { Widget currentWidget = provider.getWidget(itemName); if (currentWidget != null) { return resolveDefault(currentWidget); } } return null; }
Source:
<code>private static Schema generateSchema() { Schema schema = new Schema("Sweep", SweepSchema.class.getPackage().getName() + ".generated", NAMESPACE, <START> OptionalType.JAVA8); <END> schema.addTableDefinition("progress", new TableDefinition() {{ javaTableName("SweepProgress"); tableNameIsSafeLoggable(); namedComponentsSafeByDefault(); rowName(); rowComponent("dummy", ValueType.VAR_LONG); columns(); column("full_table_name", "n", ValueType.STRING); column("minimum_swept_timestamp", "m", ValueType.VAR_SIGNED_LONG); column("start_row", "s", ValueType.BLOB); column("cells_deleted", "d", ValueType.VAR_LONG); column("cells_examined", "e", ValueType.VAR_LONG); conflictHandler(ConflictHandler.IGNORE_ALL); ignoreHotspottingChecks(); }}); schema.addTableDefinition("priority", new TableDefinition() {{ javaTableName("SweepPriority"); tableNameIsSafeLoggable(); namedComponentsSafeByDefault(); rowName(); rowComponent("full_table_name", ValueType.STRING); columns(); column("write_count", "w", ValueType.VAR_LONG); column("last_sweep_time", "t", ValueType.VAR_LONG); column("minimum_swept_timestamp", "m", ValueType.VAR_SIGNED_LONG); column("cells_deleted", "d", ValueType.VAR_LONG); column("cells_examined", "e", ValueType.VAR_LONG); conflictHandler(ConflictHandler.IGNORE_ALL); }}); schema.validate(); return schema; }</code><technical_language>Is this a drive-by conversion of sweep schema Java8 optionals? ok that</technical_language>

Target:
private static Schema generateSchema() { Schema schema = new Schema("Sweep", SweepSchema.class.getPackage().getName() + ".generated", NAMESPACE, OptionalType.JAVA8); schema.addTableDefinition("progress", new TableDefinition() {{ javaTableName("SweepProgress"); allSafeForLoggingByDefault(); rowName(); rowComponent("dummy", ValueType.VAR_LONG); columns(); column("full_table_name", "n", ValueType.STRING); column("minimum_swept_timestamp", "m", ValueType.VAR_SIGNED_LONG); column("start_row", "s", ValueType.BLOB); column("cells_deleted", "d", ValueType.VAR_LONG); column("cells_examined", "e", ValueType.VAR_LONG); conflictHandler(ConflictHandler.IGNORE_ALL); ignoreHotspottingChecks(); }}); schema.addTableDefinition("priority", new TableDefinition() {{ javaTableName("SweepPriority"); allSafeForLoggingByDefault(); rowName(); rowComponent("full_table_name", ValueType.STRING); columns(); column("write_count", "w", ValueType.VAR_LONG); column("last_sweep_time", "t", ValueType.VAR_LONG); column("minimum_swept_timestamp", "m", ValueType.VAR_SIGNED_LONG); column("cells_deleted", "d", ValueType.VAR_LONG); column("cells_examined", "e", ValueType.VAR_LONG); conflictHandler(ConflictHandler.IGNORE_ALL); }}); schema.validate(); return schema; }
Source:
<code>public static AccountObject toAccount(String accountJSONString) { AccountObject account = null; try { JSONObject accountJSON = new JSONObject(accountJSONString); String userName = accountJSON.getString(AccountJSONEnum.USERNAME); String userNickName = accountJSON.getString(AccountJSONEnum.NICK_NAME); String userPassword = accountJSON.getString(AccountJSONEnum.PASSWORD); String userEmail = accountJSON.getString(AccountJSONEnum.EMAIL); <START> long userEnable = accountJSON.getInt(AccountJSONEnum.ENABLE); <END> account = new AccountObject(userName); account.setNickName(userNickName) .setPassword(userPassword) .setEmail(userEmail) .setEnable(userEnable == 1 ? true : false); } catch (JSONException e) { account = null; } return account; }</code><technical_language>int userEnable = accountJSON.getInt(AccountJSONEnum.ENABLE);</technical_language>

Target:
public static AccountObject toAccount(String accountJSONString) { AccountObject account = null; try { JSONObject accountJSON = new JSONObject(accountJSONString); String userName = accountJSON.getString(AccountJSONEnum.USERNAME); String userNickName = accountJSON.getString(AccountJSONEnum.NICK_NAME); String userPassword = accountJSON.getString(AccountJSONEnum.PASSWORD); String userEmail = accountJSON.getString(AccountJSONEnum.EMAIL); int userEnable = accountJSON.getInt(AccountJSONEnum.ENABLE); account = new AccountObject(userName); account.setNickName(userNickName) .setPassword(userPassword) .setEmail(userEmail) .setEnable(userEnable == 1 ? true : false); } catch (JSONException e) { account = null; } return account; }
Source:
<code>private long stopTimer() { <START> return System.nanoTime() - startNanos.get(); <END> }</code><technical_language>concerned stopTimer method. thoughts: - get() method of optional generally avoided. This is a pretty good article optional usage: <LINK_0>, sums a number of similar points. I this is mark startNanos @Nullable a null do a classic if( null ) type of check. Though, a lot of internal state route. if allow for disabled timers, assigning startNanos System.nanoTime hurt (we talking a ns operation, is a place performance improvements)</technical_language>

Target:
private long stopTimer() { return System.nanoTime() - startNanos; }
Source:
<code>public void processSoftwareCapabilities(VDS vds) { VDSReturnValue returnValue = ResourceManager.getInstance().runVdsCommand(VDSCommandType.GlusterServersList, new VdsIdVDSCommandParametersBase(vds.getId())); if (!returnValue.getSucceeded()) { vds.setStatus(VDSStatus.NonOperational); <START> vds.setNonOperationalReason(NonOperationalReason.GLUSTER_COMMAND_FAILED); <END> } }</code><technical_language>please call vdsNonOperational event (you this is done in virtMonitoringStrategy</technical_language>

Target:
public void processSoftwareCapabilities(VDS vds) { VDSReturnValue returnValue = ResourceManager.getInstance().runVdsCommand(VDSCommandType.GlusterServersList, new VdsIdVDSCommandParametersBase(vds.getId())); if (!returnValue.getSucceeded()) { vds.setStatus(VDSStatus.NonOperational); vds.setNonOperationalReason(NonOperationalReason.GLUSTER_COMMAND_FAILED); vdsNonOperational(vds, NonOperationalReason.GLUSTER_COMMAND_FAILED); } }
Source:
<code>public void onActivate(ActivateEvent event, EntityRef entity) { BlockItemFactory blockFactory = new BlockItemFactory(entityManager); BlockComponent blockComponent = entity.getComponent(BlockComponent.class); EntityRef flagTaker = event.getInstigator(); if (blockComponent.getBlock().getBlockFamily().getURI().toString().equals("LightAndShadowResources:blackFlag")) { <START> flagTaker.send(new GiveItemAction(flagTaker, blockFactory.newInstance(blockManager.getBlockFamily("LightAndShadowResources:blackFlag")))); <END> } if (blockComponent.getBlock().getBlockFamily().getURI().toString().equals("LightAndShadowResources:redFlag")) { flagTaker.send(new GiveItemAction(flagTaker, blockFactory.newInstance(blockManager.getBlockFamily("LightAndShadowResources:redFlag")))); } worldProvider.setBlock(blockComponent.getPosition(), blockManager.getBlock(BlockManager.AIR_ID)); entity.destroy(); }</code><technical_language>GiveItemAction is deprecated. Shift InventoryManager's giveItem method instead</technical_language>

Target:
public void onActivate(ActivateEvent event, EntityRef entity) { BlockItemFactory blockFactory = new BlockItemFactory(entityManager); inventoryManager = new InventoryAuthoritySystem(); BlockComponent blockComponent = entity.getComponent(BlockComponent.class); LASTeam teamComponent = entity.getComponent(LASTeam.class); EntityRef flagTaker = event.getInstigator(); if (teamComponent.team.equals("red")) { inventoryManager.giveItem(flagTaker, EntityRef.NULL, blockFactory.newInstance(blockManager.getBlockFamily("LightAndShadowResources:redFlag"))); } if (teamComponent.team.equals("black")) { inventoryManager.giveItem(flagTaker, EntityRef.NULL, blockFactory.newInstance(blockManager.getBlockFamily("LightAndShadowResources:blackFlag"))); } worldProvider.setBlock(blockComponent.getPosition(), blockManager.getBlock(BlockManager.AIR_ID)); entity.destroy(); }
Source:
<code>IdInputPanel(Composite parent, Consumer<Boolean> callback) { <START> super(parent, SWT.NONE); <END> this.setLayout(new RowLayout(SWT.VERTICAL)); Composite fieldContainer = new Composite(this, SWT.NONE); fieldContainer.setLayout(new GridLayout(2, true)); this.textfield = UIUtils.createLinkedFieldInput(this.getIdLabelText(), this.getIdTooltipText(), fieldContainer); textfield.setTextLimit(ID_LENGTH); textfield.setLayoutData(new GridData(SWT.FILL, SWT.BEGINNING, true, false)); Button verify = new Button(this, SWT.FLAT); verify.setText(VERIFICATION_BUTTON_TEXT); verify.setEnabled(false); textfield.addModifyListener(new ModifyListener() { @Override public void modifyText(ModifyEvent e) { verify.setEnabled(textfield.getText().length() == ID_LENGTH); } }); statusContainer = new Composite(this, SWT.NONE); statusContainer.setLayout(new RowLayout(SWT.VERTICAL)); verify.addSelectionListener(new SelectionAdapter() { @Override public void widgetSelected(SelectionEvent e) { for (Control control : statusContainer.getChildren()) { control.dispose(); } callback.accept(verifyUserIdRegistration()); statusContainer.layout(true, true); parent.layout(true, true); parent.redraw(); parent.update(); } }); }</code><technical_language>this complicated for a constructor -- break flow methods</technical_language>

Target:
IdInputPanel(Composite parent, Consumer<Boolean> callback) { super(parent, SWT.NONE); this.setLayout(RegistrationStep.createRowLayout(SWT.VERTICAL)); Composite fieldContainer = new Composite(this, SWT.NONE); fieldContainer.setLayout(new GridLayout(2, false)); this.textfield = UIUtils.createLinkedFieldInput(this.getIdLabelText(), this.getIdTooltipText(), fieldContainer); textfield.setSize(250, SWT.DEFAULT); textfield.setTextLimit(ID_LENGTH); textfield.setLayoutData(new GridData(350, SWT.DEFAULT)); Button verify = new Button(fieldContainer, SWT.FLAT); verify.setText(VERIFICATION_BUTTON_TEXT); verify.setEnabled(false); verify.setLayoutData(new GridData(SWT.FILL, SWT.CENTER, true, false, 2, 1)); textfield.addModifyListener(new ModifyListener() { @Override public void modifyText(ModifyEvent e) { verify.setEnabled(textfield.getText().length() == ID_LENGTH); } }); statusContainer = new Composite(this, SWT.NONE); statusContainer.setLayout(RegistrationStep.createRowLayout(SWT.VERTICAL)); verify.addSelectionListener(new SelectionAdapter() { @Override public void widgetSelected(SelectionEvent e) { for (Control control : statusContainer.getChildren()) { control.dispose(); } callback.accept(verifyUserIdRegistration()); statusContainer.layout(true, true); parent.layout(true, true); parent.redraw(); parent.update(); } }); }
Source:
<code><START> @NonNull @Override public String getEditorName() { <END> return AZTEC_EDITOR_NAME; }</code><technical_language>NIT: move separate lines</technical_language>

Target:
public String getEditorName() { return AZTEC_EDITOR_NAME; }
Source:
<code>public static Address fromBase58(NetworkParameters params, String base58) throws AddressFormatException { NetworkParameters networkParameters = (params != null) ? params : Address.getParametersFromAddress(base58); return Address.fromBase58(networkParameters, base58); <START> } <END></code><technical_language>This method appears do Address.toBase58 dashj. this change solve issues</technical_language>

Target:
public static Address fromBase58(NetworkParameters params, String base58) throws AddressFormatException { NetworkParameters networkParameters = (params != null) ? params : getParametersFromAddress(base58); return Address.fromBase58(networkParameters, base58); }
Source:
<code><START> public void sendCommand(Command command) throws RecoverablePduException, UnrecoverablePduException, <END> SmppTimeoutException, SmppChannelException, InterruptedException { if (command.getSms()) { Position lastPosition = getLastPosition(command.getDeviceId()); if (lastPosition != null) { Context.getServerManager().getProtocol(lastPosition.getProtocol()) .sendSmsCommand(devicesById.get(command.getDeviceId()).getPhone(), command); } else if (command.getType().equals(Command.TYPE_CUSTOM)) { Context.getSmppManager().sendMessageSync(devicesById.get(command.getDeviceId()).getPhone(), command.getString(Command.KEY_DATA), true); } else { throw new RuntimeException("Command " + command.getType() + " is not supported"); } } else { ActiveDevice activeDevice = Context.getConnectionManager().getActiveDevice(command.getDeviceId()); if (activeDevice != null) { activeDevice.sendCommand(command); } else { throw new RuntimeException("Device is not online"); } } }</code><technical_language>Same. I throw generic exception</technical_language>

Target:
public void sendCommand(Command command) throws Exception { long deviceId = command.getDeviceId(); if (command.getSms()) { Position lastPosition = getLastPosition(deviceId); if (lastPosition != null) { BaseProtocol protocol = Context.getServerManager().getProtocol(lastPosition.getProtocol()); protocol.sendSmsCommand(devicesById.get(deviceId).getPhone(), command); } else if (command.getType().equals(Command.TYPE_CUSTOM)) { Context.getSmppManager().sendMessageSync(devicesById.get(deviceId).getPhone(), command.getString(Command.KEY_DATA), true); } else { throw new RuntimeException("Command " + command.getType() + " is not supported"); } } else { ActiveDevice activeDevice = Context.getConnectionManager().getActiveDevice(deviceId); if (activeDevice != null) { activeDevice.sendCommand(command); } else { if (fallbackToSms) { command.setSms(true); sendCommand(command); } else { throw new RuntimeException("Device is not online"); } } } }
Source:
<code>public @NonNull TmfModelResponse<TmfTreeModel<TimeGraphEntryModel>> fetchTree(Map<String, Object> fetchParameters, @Nullable IProgressMonitor monitor) { <START> List<TimeGraphEntryModel> EntryList = new ArrayList<>(fEntryBuilder.size()); <END> moduleEntryModelList = new ArrayList<>(); Boolean fetchTreeIsComplete = true; synchronized (fEntryBuilder) { fetchTreeIsComplete &= addTrace(monitor, EntryList); if (monitor != null) { if (monitor.isCanceled()) { return new TmfModelResponse<>(null, Status.CANCELLED, CommonStatusMessage.TASK_CANCELLED); } } } Collections.reverse(EntryList); Status status = fetchTreeIsComplete ? Status.COMPLETED : Status.RUNNING; String msg = fetchTreeIsComplete ? CommonStatusMessage.COMPLETED : CommonStatusMessage.RUNNING; return new TmfModelResponse<>(new TmfTreeModel<>(Collections.emptyList(), EntryList), status, msg); }</code><technical_language>entryList lowercase e</technical_language>

Target:
public @NonNull TmfModelResponse<TmfTreeModel<TimeGraphEntryModel>> fetchTree(Map<String, Object> fetchParameters, @Nullable IProgressMonitor monitor) { List<TimeGraphEntryModel> entryList = new ArrayList<>(); moduleEntryModelList = new ArrayList<>(); boolean fetchTreeIsComplete; synchronized (fEntryBuilder) { fetchTreeIsComplete = addTrace(monitor); if (monitor != null) { if (monitor.isCanceled()) { return new TmfModelResponse<>(null, Status.CANCELLED, CommonStatusMessage.TASK_CANCELLED); } } } entryList = buildEntryList(monitor); Status status = fetchTreeIsComplete ? Status.COMPLETED : Status.RUNNING; String msg = fetchTreeIsComplete ? CommonStatusMessage.COMPLETED : CommonStatusMessage.RUNNING; return new TmfModelResponse<>(new TmfTreeModel<>(Collections.emptyList(), entryList), status, msg); }