auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public void removeNode(Node node) { InetAddress host = null; String nodeName = node.getNodeInformation().getName(); if ((host = registeredNodes.remove(nodeName)) != null) { removedNodes.putIfAbsent(host, new AtomicInteger(0)); removedNodes.get(host).incrementAndGet(); if (!registeredNodes.containsValue(host)) { <START> freeHosts.put(host, removedNodes.remove(host).intValue()); <END> try { this.killNodeImpl(node, host); } catch (Exception e) { logger.trace("An exception occurred during node removal", e); } } logger.debug("Node " + nodeName + " removed. #freeHosts:" + freeHosts.size() + " #registered nodes: " + registeredNodes.size()); } else { logger.error("Node " + nodeName + " is not known as a node belonging to this infrastructure manager"); } }</code><technical_language>I moved this line this.killNodeImpl(node, host); a node is deleted forgotten if runtime remote node killed, information Scheduler level wrong. apply for other infrastructures</technical_language>

Target:
public void removeNode(Node node) { InetAddress host = null; String nodeName = node.getNodeInformation().getName(); if ((host = registeredNodes.remove(nodeName)) != null) { logger.debug("Removing node " + node.getNodeInformation().getURL() + " from " + this.getClass().getSimpleName()); removedNodes.putIfAbsent(host, new AtomicInteger(0)); removedNodes.get(host).incrementAndGet(); if (!registeredNodes.containsValue(host)) { try { this.killNodeImpl(node, host); } catch (Exception e) { logger.trace("An exception occurred during node removal", e); } freeHosts.putIfAbsent(host, removedNodes.remove(host).intValue()); } logger.info("Node " + nodeName + " removed. #freeHosts:" + freeHosts.size() + " #registered nodes: " + registeredNodes.size()); } else { logger.error("Node " + nodeName + " is not known as a node belonging to this infrastructure manager"); } }
Source:
<code>protected void executeCancellation() { <START> if (!session.isHost() || session.getRemoteUsers().isEmpty()) <END> sessionManager.stopSession(SessionEndReason.LOCAL_USER_LEFT); }</code><technical_language>suggestion if (session.isHost() && session.getRemoteUsers().isEmpty()) check if host, client disconnects a canceled outgoing negotiation</technical_language>

Target:
protected void executeCancellation() { if (session.isHost() && session.getRemoteUsers().isEmpty()) sessionManager.stopSession(SessionEndReason.LOCAL_USER_LEFT); }
Source:
<code>public AdvancedFilterEditor(DataSetClientServices dataSetClientServices) { initWidget(uiBinder.createAndBindUi(this)); this.dataSetClientServices = dataSetClientServices; okButton.setType(ButtonType.PRIMARY); <START> okButton.addClickHandler(new ClickHandler() { <END> @Override public void onClick(ClickEvent event) { ok(); } }); }</code><technical_language>lambda style in :)</technical_language>

Target:
public AdvancedFilterEditor(DataSetClientServices dataSetClientServices) { initWidget(uiBinder.createAndBindUi(this)); this.dataSetClientServices = dataSetClientServices; okButton.setType(ButtonType.PRIMARY); okButton.addClickHandler(event -> ok()); }
Source:
<code><START> protected Optional() { <END> }</code><technical_language>if this private prevent subclasses other Some, right</technical_language>

Target:
private Optional() { }
Source:
<code>protected void drawStepContent() { super.drawStepContent(); setNextButtonEnabled(false); textField = new TextField(); textField.textProperty().addListener((observable, oldValue, newValue) -> { <START> if ("".equals(newValue.trim())) { <END> setNextButtonEnabled(false); } else { setNextButtonEnabled(true); } }); textField.setText(defaultValue); textField.setLayoutX(10); textField.setLayoutY(40); this.addToContentPane(textField); }</code><technical_language>I newValue.trim().isEmpty()</technical_language>

Target:
protected void drawStepContent() { super.drawStepContent(); setNextButtonEnabled(false); textField = new TextField(); textField.textProperty().addListener((observable, oldValue, newValue) -> { if (StringUtils.isBlank(newValue)) { setNextButtonEnabled(false); } else { setNextButtonEnabled(true); } }); textField.setText(defaultValue); textField.setLayoutX(10); textField.setLayoutY(40); this.addToContentPane(textField); }
Source:
<code>private RelationalSymbolicValue create(Tree.Kind kind, SymbolicValue leftOperand, SymbolicValue rightOperand) { <START> return (RelationalSymbolicValue) RelationalSymbolicValue.create(kind, leftOperand, rightOperand); <END> }</code><technical_language>@inverno imo cast EqualitySymbolicValue, test in this file</technical_language>

Target:
private EqualitySymbolicValue create(Tree.Kind kind, SymbolicValue leftOperand, SymbolicValue rightOperand) { return (EqualitySymbolicValue) RelationalSymbolicValue.create(kind, leftOperand, rightOperand); }
Source:
<code>public static CassandraKeyValueService create( MetricsManager metricsManager, CassandraKeyValueServiceConfig config, CassandraMutationTimestampProvider mutationTimestampProvider, CassandraClientPool clientPool) { return createOrShutdownClientPool(metricsManager, config, CassandraKeyValueServiceRuntimeConfig::getDefault, clientPool, <START> null, <END> mutationTimestampProvider, LoggerFactory.getLogger(CassandraKeyValueService.class), AtlasDbConstants.DEFAULT_INITIALIZE_ASYNC); }</code><technical_language>is this null here?, if Optional parameter, Optional.empty(). In general, if I avoid check for/use nulls I random Null pointer exceptions arise. replace Optional, if crossing file boundaries, generally I want about/I'd double check involves writing null down</technical_language>

Target:
public static CassandraKeyValueService create( MetricsManager metricsManager, CassandraKeyValueServiceConfig config, CassandraMutationTimestampProvider mutationTimestampProvider, CassandraClientPool clientPool) { return createOrShutdownClientPool(metricsManager, config, CassandraKeyValueServiceRuntimeConfig::getDefault, clientPool, Optional.empty(), mutationTimestampProvider, LoggerFactory.getLogger(CassandraKeyValueService.class), AtlasDbConstants.DEFAULT_INITIALIZE_ASYNC); }
Source:
<code>public State getStateAs(Class<? extends State> typeClass) { State newState = null; if (function != null) { <START> newState = function.getStateAs(getMembers(), typeClass); <END> } if (newState == null && baseItem != null) { baseItem.setState(state); newState = baseItem.getStateAs(typeClass); } if (newState == null) { newState = super.getStateAs(typeClass); } return newState; }</code><technical_language>is this change related this PR</technical_language>

Target:
public @Nullable State getStateAs(Class<? extends State> typeClass) { State newState = null; if (function != null) { newState = function.getStateAs(getStateMembers(getMembers()), typeClass); } if (newState == null && baseItem != null) { baseItem.setState(state); newState = baseItem.getStateAs(typeClass); } if (newState == null) { newState = super.getStateAs(typeClass); } return newState; }
Source:
<code>public static Map<String, String> toMap(Properties properties) { <START> Map<String, String> res = new HashMap<String, String>(); <END> for (Property property : properties.getProperty()) { res.put(property.getName(), property.getValue()); } return res; }</code><technical_language>new HashMap<>() do</technical_language>

Target:
public static Map<String, String> toMap(Properties properties) { Map<String, String> res = new HashMap<>(); for (Property property : properties.getProperties()) { res.put(property.getName(), property.getValue()); } return res; }
Source:
<code>protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException { if (!MediaType.TEXT_EVENT_STREAM_VALUE.equalsIgnoreCase(request.getHeader("Accept"))) { response.sendError(HttpStatus.NOT_ACCEPTABLE.value()); } else { response.setStatus(HttpStatus.OK.value()); response.setContentType(MediaType.TEXT_EVENT_STREAM_VALUE); response.setCharacterEncoding("UTF-8"); response.setHeader("Cache-control", "no-cache, no-store"); response.setHeader("Pragma", "no-cache"); response.setHeader("Expires", "-1"); response.setHeader("Allow", "GET, OPTIONS"); response.setHeader("Access-Control-Allow-Origin", "*"); response.setHeader("Access-Control-Allow-Headers", "*"); response.setHeader("Access-Control-Max-Age", "600"); response.flushBuffer(); final AsyncContext asyncContext = request.startAsync(); final Observable<String> observable = BootProgressBeanPostProcessor.getObservable() .observeOn(Schedulers.single()) .subscribeOn(Schedulers.io()) .filter(this::filterBeansByName) .map(this::getBeanDisplayName); final Disposable disposable = observable.subscribe(msg -> streamContent(msg, asyncContext.getResponse()), (e) -> { }, <START> () -> complete(asyncContext)); <END> asyncContext.addListener(new DisposeObservableAsyncListener(disposable)); } }</code><technical_language>(e) -> I log in case of error</technical_language>

Target:
protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException { try { if (!MediaType.TEXT_EVENT_STREAM_VALUE.equalsIgnoreCase(request.getHeader("Accept"))) { response.sendError(HttpStatus.NOT_ACCEPTABLE.value()); return; } response.setStatus(HttpStatus.OK.value()); response.setContentType(MediaType.TEXT_EVENT_STREAM_VALUE); response.setCharacterEncoding("UTF-8"); response.setHeader("Connection", "keep-alive"); response.setHeader("Cache-control", "no-cache, no-store, max-age=0, must-revalidate"); response.setHeader("Pragma", "no-cache"); response.setHeader("Expires", "0"); response.setHeader("Allow", "GET, OPTIONS"); response.setHeader("Access-Control-Allow-Origin", "*"); response.setHeader("Access-Control-Allow-Headers", "*"); response.setHeader("Access-Control-Max-Age", "600"); final AsyncContext asyncContext = request.startAsync(request, response); final Disposable disposable = BootProgressBeanPostProcessor.getProgressObservable() .doOnComplete(() -> { write("booted", "", asyncContext); asyncContext.getResponse().getWriter().flush(); asyncContext.complete(); }) .doOnError((e) -> { logger.error("An error occurred!", e); asyncContext.complete(); }) .subscribe(data -> write("booting", data, asyncContext)); asyncContext.addListener(new BootProgressAsyncListener(disposable)); } catch (Exception e) { logger.error("Failed to process request {}", request.getRequestURI(), e); response.sendError(HttpStatus.INTERNAL_SERVER_ERROR.value()); } }
Source:
<code>IgnoreCondition create() { checkConditionType(); try { return createCondition(); <START> } catch (RuntimeException re) { <END> throw Throwables.propagate(re); } catch (Exception e) { throw new RuntimeException(e); } }</code><technical_language>This catch Exception</technical_language>

Target:
IgnoreCondition create() { checkConditionType(); try { return createCondition(); } catch (Exception re) { throw Throwables.propagate(re); } }
Source:
<code>public void setUp() { ignoreTestDir = JGitTestUtil.getTestResourceFile("excludeTest"); assertTrue("Test resource directory is not a directory",ignoreTestDir.isDirectory()); try { <START> repo = new Repository(ignoreTestDir); <END> } catch (IOException e) { fail("IOException when creating repository at" + ignoreTestDir); } cache = new SimpleIgnoreCache(repo); initCache(); }</code><technical_language>create a Repo call repo.create(). repository exist I assume this sooner lead problems. I LocalDiskRepositoryTestCase a base class copy test files this new location. This ensures tearDown method remove repository</technical_language>

Target:
public void setUp() throws Exception { super.setUp(); ignoreTestDir = JGitTestUtil.getTestResourceFile("excludeTest"); assertTrue("Test resource directory is not a directory",ignoreTestDir.isDirectory()); db = createWorkRepository(); recursiveCopy(ignoreTestDir, db.getDirectory().getParentFile()); cache = new SimpleIgnoreCache(db); initCache(); }
Source:
<code>private void raiseAlert(HttpMessage msg, int id, String evidence) { Alert alert = new Alert(getPluginId(), Alert.RISK_MEDIUM, Alert.CONFIDENCE_MEDIUM, getName()); alert.setDetail( this.getDescription(), msg.getRequestHeader().getURI().toString(), "", "", "", this.getSolution(), this.getReference(), evidence, <START> 0, END> 0, msg); parent.raiseAlert(id, alert); }</code><technical_language>suggestion 749, // CWE-749: Exposed Dangerous Method Function maybe: CWE-676: of Potentially Dangerous Function</technical_language>

Target:
private void raiseAlert(HttpMessage msg, int id, String evidence) { Alert alert = new Alert(getPluginId(), Alert.RISK_LOW, Alert.CONFIDENCE_LOW, getName()); alert.setDetail( this.getDescription(), msg.getRequestHeader().getURI().toString(), "", "", "", this.getSolution(), this.getReference(), evidence, 749, 0, msg); parent.raiseAlert(id, alert); }
Source:
<code>public XWikiAttachment remove(int index) { XWikiAttachment removedAttachment = map.remove(this.list.get(index).getFilename()); if (removedAttachment != null) { <START> removedAttachment = super.remove(index); <END> document.setMetaDataDirty(true); } return removedAttachment; }</code><technical_language>manipulate map reset list if map changed, modify list</technical_language>

Target:
public XWikiAttachment remove(int index) { XWikiAttachment removedAttachment = map.remove(this.decorated().get(index).getFilename()); if (removedAttachment != null) { updatedMap(); } return removedAttachment; }
Source:
<code>private void dismissKeyguard() { final ResolvableFuture<Void> future = ResolvableFuture.create(); KeyguardManager keyguardManager = getSystemService(KeyguardManager.class); keyguardManager.requestDismissKeyguard(this, new KeyguardManager.KeyguardDismissCallback() { @Override public void onDismissSucceeded() { future.set(null); } }); try { future.get(500, TimeUnit.MILLISECONDS); } catch (ExecutionException | InterruptedException | TimeoutException e) { <START> Log.w("WebViewTestActivity", "Failed to dismiss keyguard"); <END> } }</code><technical_language>add exception param</technical_language>

Target:
private void dismissKeyguard() { final ResolvableFuture<Void> future = ResolvableFuture.create(); KeyguardManager keyguardManager = getSystemService(KeyguardManager.class); keyguardManager.requestDismissKeyguard(this, new KeyguardManager.KeyguardDismissCallback() { @Override public void onDismissSucceeded() { future.set(null); } }); try { future.get(500, TimeUnit.MILLISECONDS); } catch (ExecutionException | InterruptedException | TimeoutException e) { Log.w("WebViewTestActivity", "Failed to dismiss keyguard", e); } }
Source:
<code>private Predicate<Fragment> exportedJarFragment() { return new Predicate<Fragment>() { @Override public boolean apply(final Fragment input) { <START> return input.isExported() && input.getValue().endsWith(".jar"); <END> } }; }</code><technical_language>a Constant</technical_language>

Target:
private Predicate<Fragment> exportedJarFragment() { return new Predicate<Fragment>() { @Override public boolean apply(final Fragment input) { return input.isExported() && input.getValue().endsWith(JAR_EXT); } }; }
Source:
<code>public StatusAdapter getStatusAdapter(JobInfo info) { if (map == null) { return null; } StatusAdapter statusAdapter = (StatusAdapter) map.remove(info); if (statusAdapter != <START> null) <END> statusAdapter.setProperty( IProgressConstants.NO_IMMEDIATE_ERROR_PROMPT_PROPERTY, Boolean.FALSE); return statusAdapter; }</code><technical_language>follow code style braces please</technical_language>

Target:
public StatusAdapter getStatusAdapter(JobInfo info) { if (map == null) { return null; } StatusAdapter statusAdapter = (StatusAdapter) map.remove(info); if (statusAdapter != null) { statusAdapter.setProperty( IProgressConstants.NO_IMMEDIATE_ERROR_PROMPT_PROPERTY, Boolean.FALSE); } return statusAdapter; }
Source:
<code>public <K, V, M> OutputStream<K, V, M> getOutputStream(String streamId, Function<? super M, ? extends K> keyExtractor, Function<? super M, ? extends V> msgExtractor) { if (keyExtractor == null || msgExtractor == null) { <START> throw new IllegalArgumentException("Neither keyExtractor nor msgExtractor can be null for an output stream."); <END> } return outStreams.computeIfAbsent(runner.getStreamSpec(streamId), streamSpec -> new OutputStreamInternalImpl<K, V, M>(this, streamSpec, keyExtractor, msgExtractor)); }</code><technical_language>Thanks for adding these. Issue: is null</technical_language>

Target:
public <K, V, M> OutputStream<K, V, M> getOutputStream(String streamId, Function<? super M, ? extends K> keyExtractor, Function<? super M, ? extends V> msgExtractor) { if (keyExtractor == null) { throw new IllegalArgumentException("keyExtractor can't be null for an output stream."); } if (msgExtractor == null) { throw new IllegalArgumentException("msgExtractor can't be null for an output stream."); } return outStreams.computeIfAbsent(runner.getStreamSpec(streamId), streamSpec -> new OutputStreamInternalImpl<>(this, streamSpec, (Function<M, K>) keyExtractor, (Function<M, V>) msgExtractor)); }
Source:
<code>public MoveDescription( final Collection<Unit> units, final Route route, final Map<Unit, Unit> unitsToTransports, final Map<Unit, Collection<Unit>> dependentUnits) { super(units); <START> this.route = route; <END> if (this.route == null) { throw new NullPointerException(); } this.unitsToTransports = unitsToTransports; if (this.unitsToTransports == null) { throw new NullPointerException(); } if (!dependentUnits.isEmpty()) { this.dependentUnits = new HashMap<>(); for (final Entry<Unit, Collection<Unit>> entry : dependentUnits.entrySet()) { this.dependentUnits.put(entry.getKey(), new HashSet<>(entry.getValue())); } } else { this.dependentUnits = Map.of(); } }</code><technical_language>Nit: simplify NPE check guava, eg: this.route = Precondtions.checkNotNull(route);</technical_language>

Target:
public MoveDescription( final Collection<Unit> units, final Route route, final Map<Unit, Unit> unitsToTransports, final Map<Unit, Collection<Unit>> dependentUnits) { super(Collections.unmodifiableCollection(units)); this.route = Preconditions.checkNotNull(route); this.unitsToTransports = Collections.unmodifiableMap(unitsToTransports); this.dependentUnits = Collections.unmodifiableMap( dependentUnits.entrySet().stream() .collect(Collectors.toMap(Map.Entry::getKey, e -> Set.copyOf(e.getValue())))); }
Source:
<code>private Iterable<Review> reviews(final Publisher contentPublisher, Set<org.atlasapi.media.entity.simple.Review> simpleReviews) { return simpleReviews.stream() .map(simpleReview -> { if (simpleReview.getPublisherDetails() != null && !getPublisher(simpleReview.getPublisherDetails()).equals(contentPublisher)) { throw new IllegalArgumentException("Review publisher must match content publisher"); } return Review.builder() .withLocale(Locale.forLanguageTag(simpleReview.getLanguage())) .withReview(simpleReview.getReview()) .withDate(simpleReview.getDate()) .withAuthor(simpleReview.getAuthor()) .withAuthorInitials(simpleReview.getAuthorInitials()) .withRating(simpleReview.getRating()) .withReviewType(simpleReview.getReviewType()) .build(); }) <START> .collect(Collectors.toList()); <END> }</code><technical_language>do want modify this afterwards? if not, MoreCollectors.toImmutableList()</technical_language>

Target:
private Iterable<Review> reviews( final Publisher contentPublisher, Set<org.atlasapi.media.entity.simple.Review> simpleReviews ) { return simpleReviews.stream() .map(simpleReview -> { if (simpleReview.getPublisherDetails() != null && !getPublisher(simpleReview.getPublisherDetails()).equals(contentPublisher)) { throw new IllegalArgumentException("Review publisher must match content publisher"); } return Review.builder() .withLocale(Locale.forLanguageTag(simpleReview.getLanguage())) .withReview(simpleReview.getReview()) .withDate(simpleReview.getDate()) .withAuthor(simpleReview.getAuthor()) .withAuthorInitials(simpleReview.getAuthorInitials()) .withRating(simpleReview.getRating()) .withReviewTypeKey(simpleReview.getReviewType().toKey()) .build(); }) .collect(MoreCollectors.toImmutableList()); }
Source:
<code>public List<TriggeredNotification> calculate( List<? extends Person> subjects, Notification notification, OrchestrationManager manager) { List<TriggeredNotification> triggered = new ArrayList<>(); if (null != notification.getGoal().getApproved()) { for(Person p : subjects) { List<TriggeredNotification> t = NotificationCalculator.createTriggeredNotifications( notification, 0D, manager, p.getId()); if (null != t) { triggered.addAll(t); } } } <START> if (triggered.isEmpty()) { <END> return null; } else { return triggered; } }</code><technical_language>return triggered.isEmpty() ? null : triggered; contains letters, parenthesis, braces, semicolons additional colon ternary operator</technical_language>

Target:
public List<TriggeredNotification> calculate( List<? extends Person> subjects, Notification notification, OrchestrationManager manager) { List<TriggeredNotification> triggered = new ArrayList<>(); if (null != notification.getGoal().getApproved()) { for(Person p : subjects) { List<TriggeredNotification> t = NotificationCalculator.createTriggeredNotifications( notification, 0D, manager, p.getId()); if (null != t) { triggered.addAll(t); } } } return triggered.isEmpty() ? null : triggered; }
Source:
<code>private MessageElement readMessage(String documentation) { String name = readName(); <START> MessageElement.Builder builder = MessageElement.builder() <END> .setName(name) .setQualifiedName(prefix + name) .setDocumentation(documentation); String previousPrefix = prefix; prefix = prefix + name + "."; if (readChar() != '{') throw unexpected("expected '{'"); while (true) { String nestedDocumentation = readDocumentation(); if (peekChar() == '}') { pos++; break; } Object declared = readDeclaration(nestedDocumentation, Context.MESSAGE); if (declared instanceof FieldElement) { builder.addField((FieldElement) declared); } else if (declared instanceof OneOfElement) { builder.addOneOf((OneOfElement) declared); } else if (declared instanceof TypeElement) { builder.addType((TypeElement) declared); } else if (declared instanceof ExtensionsElement) { builder.addExtensions((ExtensionsElement) declared); } else if (declared instanceof OptionElement) { builder.addOption((OptionElement) declared); } else if (declared instanceof ExtendElement) { fileBuilder.addExtendDeclaration((ExtendElement) declared); } } prefix = previousPrefix; return builder.build(); }</code><technical_language>Sweet</technical_language>

Target:
private MessageElement readMessage(String documentation) { String name = readName(); MessageElement.Builder builder = MessageElement.builder() .name(name) .qualifiedName(prefix + name) .documentation(documentation); String previousPrefix = prefix; prefix = prefix + name + "."; if (readChar() != '{') throw unexpected("expected '{'"); while (true) { String nestedDocumentation = readDocumentation(); if (peekChar() == '}') { pos++; break; } Object declared = readDeclaration(nestedDocumentation, Context.MESSAGE); if (declared instanceof FieldElement) { builder.addField((FieldElement) declared); } else if (declared instanceof OneOfElement) { builder.addOneOf((OneOfElement) declared); } else if (declared instanceof TypeElement) { builder.addType((TypeElement) declared); } else if (declared instanceof ExtensionsElement) { builder.addExtensions((ExtensionsElement) declared); } else if (declared instanceof OptionElement) { builder.addOption((OptionElement) declared); } else if (declared instanceof ExtendElement) { fileBuilder.addExtendDeclaration((ExtendElement) declared); } } prefix = previousPrefix; return builder.build(); }
Source:
<code>public void unwatchWithinMulti() { final String key = "foo"; final String val = "bar"; jedis.set(key, val); jedis.watch(key); <START> List<String> exp = new ArrayList<>(); <END> Transaction t = jedis.multi(); t.get(key); exp.add(val); t.unwatch(); exp.add("OK"); t.get(key); exp.add(val); List<Object> res = t.exec(); assertEquals(exp, res); }</code><technical_language>List<Object></technical_language>

Target:
public void unwatchWithinMulti() { final String key = "foo"; final String val = "bar"; jedis.set(key, val); jedis.watch(key); List<Object> exp = new ArrayList<>(); Transaction t = jedis.multi(); t.get(key); exp.add(val); t.unwatch(); exp.add("OK"); t.get(key); exp.add(val); List<Object> res = t.exec(); assertEquals(exp, res); }
Source:
<code>protected InputStream loadResource(Uri uri, ContentResolver contentResolver) throws FileNotFoundException { if (ContactsContract.Contacts.CONTENT_URI.getHost().equals(uri.getHost()) <START> && URI_MATCHER.match(uri) != UriMatcher.NO_MATCH) { <END> return loadResourceFromContactUri(uri, contentResolver); } return contentResolver.openInputStream(uri); }</code><technical_language>I match() result passed in, enough parse Uri once</technical_language>

Target:
protected InputStream loadResource(Uri uri, ContentResolver contentResolver) throws FileNotFoundException { final int matchedUri = URI_MATCHER.match(uri); return loadResourceFromUri(uri, contentResolver, matchedUri); }
Source:
<code>void verifySpace(int destination, String path, long length) throws StopRequestException { resetBytesDownloadedSinceLastCheckOnSpace(); File dir = null; if (path == null) { throw new IllegalArgumentException("path can't be null"); } switch (destination) { case DownloadsDestination.DESTINATION_CACHE_PARTITION: case DownloadsDestination.DESTINATION_CACHE_PARTITION_NOROAMING: case DownloadsDestination.DESTINATION_CACHE_PARTITION_PURGEABLE: dir = downloadDataDir; break; case DownloadsDestination.DESTINATION_EXTERNAL: dir = externalStorageDir; break; case DownloadsDestination.DESTINATION_SYSTEMCACHE_PARTITION: dir = systemCacheDir; break; case DownloadsDestination.DESTINATION_FILE_URI: if (path.startsWith(externalStorageDir.getPath())) { dir = externalStorageDir; } else if (path.startsWith(downloadDataDir.getPath())) { dir = downloadDataDir; } else if (path.startsWith(systemCacheDir.getPath())) { dir = systemCacheDir; } else if (path.startsWith(internalStorageDir.getPath())) { dir = internalStorageDir; } else { for (File aExternalStorageDir : externalStorageDirs) { <START> if (aExternalStorageDir != null) { <END> if (path.startsWith(aExternalStorageDir.getPath())) { dir = aExternalStorageDir; break; } } } } break; } if (dir == null) { throw new IllegalStateException("invalid combination of destination: " + destination + ", path: " + path); } findSpace(dir, length, destination); }</code><technical_language>collapse ifs one</technical_language>

Target:
void verifySpace(int destination, String path, long length) throws StopRequestException { resetBytesDownloadedSinceLastCheckOnSpace(); File dir = null; if (path == null) { throw new IllegalArgumentException("path can't be null"); } switch (destination) { case DownloadsDestination.DESTINATION_CACHE_PARTITION: case DownloadsDestination.DESTINATION_CACHE_PARTITION_NOROAMING: case DownloadsDestination.DESTINATION_CACHE_PARTITION_PURGEABLE: dir = downloadDataDir; break; case DownloadsDestination.DESTINATION_EXTERNAL: dir = externalStorageDir; break; case DownloadsDestination.DESTINATION_SYSTEMCACHE_PARTITION: dir = systemCacheDir; break; case DownloadsDestination.DESTINATION_FILE_URI: if (path.startsWith(externalStorageDir.getPath())) { dir = externalStorageDir; } else if (path.startsWith(downloadDataDir.getPath())) { dir = downloadDataDir; } else if (path.startsWith(systemCacheDir.getPath())) { dir = systemCacheDir; } else if (path.startsWith(internalStorageDir.getPath())) { dir = internalStorageDir; } else { for (File anExternalStorageDir : externalStorageDirs) { if (anExternalStorageDir != null && path.startsWith(anExternalStorageDir.getPath()) { dir = anExternalStorageDir; break; } } } break; } if (dir == null) { throw new IllegalStateException("invalid combination of destination: " + destination + ", path: " + path); } findSpace(dir, length, destination); }
Source:
<code>public void greetTest() { <START> <END> RestAssuredMockMvc.given() .contentType(ContentType.JSON) .param("The param", "Johan") .when() .get("/users/greet") .then() .statusCode(200) .body(containsString("hello, Johan")); }</code><technical_language>remove this blank line</technical_language>

Target:
public void greetTest() { given().contentType(ContentType.JSON) .param("name", "Johan") .when() .get("/users/greet") .then() .statusCode(200) .body(containsString("hello, Johan")); }
Source:
<code>public void canCombatDamageHappen(AreaEffectCloudApplyEvent event) { if ( ! MUtil.isHarmfulPotion(event.getEntity().getBasePotionData().getType().getEffectType())) return; ProjectileSource projectileSource = event.getEntity().getSource(); if ( ! (projectileSource instanceof Entity)) return; Entity thrower = (Entity)projectileSource; List<LivingEntity> affectedList = new ArrayList<LivingEntity>(); for (LivingEntity affectedEntity : event.getAffectedEntities()) { EntityDamageByEntityEvent sub = new EntityDamageByEntityEvent(thrower, affectedEntity, EntityDamageEvent.DamageCause.CUSTOM, 0D); if (EngineCombat.get().canCombatDamageHappen(sub, false)) continue; affectedList.add(affectedEntity); } for (LivingEntity affectedEnt : affectedList) { event.getAffectedEntities().remove(affectedEnt); } <START> } <END></code><technical_language>also, blank line please ^^" (see for description )</technical_language>

Target:
public void canCombatDamageHappen(AreaEffectCloudApplyEvent event) { if ( ! MUtil.isHarmfulPotion(event.getEntity().getBasePotionData().getType().getEffectType())) return; ProjectileSource projectileSource = event.getEntity().getSource(); if ( ! (projectileSource instanceof Entity)) return; Entity thrower = (Entity)projectileSource; List<LivingEntity> affectedList = new ArrayList<LivingEntity>(); for (LivingEntity affectedEntity : event.getAffectedEntities()) { EntityDamageByEntityEvent sub = new EntityDamageByEntityEvent(thrower, affectedEntity, EntityDamageEvent.DamageCause.CUSTOM, 0D); if (EngineCombat.get().canCombatDamageHappen(sub, false)) continue; affectedList.add(affectedEntity); } event.getAffectedEntities().removeAll(affectedList); }
Source:
<code>private static Type best(List<Type> minimalCandidates) { Collections.sort(minimalCandidates, new Comparator<Type>() { @Override public int compare(Type type, Type t1) { return type.name().compareTo(t1.name()); } }); Collections.sort(minimalCandidates, new Comparator<Type>() { @Override <START> public int compare(Type type, Type t1) { <END> Symbol.TypeSymbol typeSymbol = type.symbol(); Symbol.TypeSymbol t1Symbol = t1.symbol(); if (typeSymbol.isInterface() && t1Symbol.isInterface()) { return 0; } else if (typeSymbol.isInterface()) { return 1; } else if (t1Symbol.isInterface()) { return -1; } return 0; } }); return minimalCandidates.get(0); }</code><technical_language>This merge other comparator pass sorting</technical_language>

Target:
private static Type best(List<Type> minimalCandidates) { Collections.sort(minimalCandidates, new Comparator<Type>() { @Override public int compare(Type type, Type t1) { Symbol.TypeSymbol typeSymbol = type.symbol(); Symbol.TypeSymbol t1Symbol = t1.symbol(); if (typeSymbol.isInterface() && t1Symbol.isInterface()) { return type.name().compareTo(t1.name()); } else if (typeSymbol.isInterface()) { return 1; } else if (t1Symbol.isInterface()) { return -1; } return type.name().compareTo(t1.name()); } }); return minimalCandidates.get(0); }
Source:
<code>private void startDataRegions(DataStorageConfiguration cfg) throws IgniteCheckedException { if (dataRegionsStarted) return; assert cfg != null; registerMetricsMBeans(cctx.gridConfig()); startDataRegions(); initPageMemoryDataStructures(cfg); dataRegionsStarted = true; DecimalFormat dblFmt = new DecimalFormat("#.##", DecimalFormatSymbols.getInstance(Locale.US)); if (log.isQuiet()) { U.quiet(false, "Data Regions Started: " + dataRegionMap.size()); U.quietMultipleLines(false, IgniteKernal.dataStorageReport(this, dblFmt, false)); <START> } else if (log.isInfoEnabled()) { <END> log.info("Data Regions Started: " + dataRegionMap.size()); log.info(IgniteKernal.dataStorageReport(this, dblFmt, false)); } }</code><technical_language>code style else if a new line</technical_language>

Target:
private void startDataRegions(DataStorageConfiguration cfg) throws IgniteCheckedException { if (dataRegionsStarted) return; assert cfg != null; registerMetricsMBeans(cctx.gridConfig()); startDataRegions(); initPageMemoryDataStructures(cfg); dataRegionsStarted = true; if (log.isQuiet()) { U.quiet(false, "Data Regions Started: " + dataRegionMap.size()); U.quietMultipleLines(false, IgniteKernal.dataStorageReport(this, false)); } else if (log.isInfoEnabled()) { log.info("Data Regions Started: " + dataRegionMap.size()); log.info(IgniteKernal.dataStorageReport(this, false)); } }
Source:
<code><START> private String getRunContext(final Run run) { <END> if (run instanceof WorkflowRun) { List<? extends Action> actionList = ((WorkflowRun) run).getAllActions(); for (final Action currAction : actionList) { if (currAction instanceof CauseAction) { for (final Cause currCause : ((CauseAction) currAction).getCauses()) { if (currCause instanceof TeamPushCause) { return ((TeamPushCause) currCause).getRunContext(); } } } } } return DEFAULT_RUN_CONTEXT; }</code><technical_language>is this code workflow job specific</technical_language>

Target:
private String getRunContext(final Run run) { List<? extends Action> actionList = run.getAllActions(); for (final Action currAction : actionList) { if (currAction instanceof CauseAction) { for (final Cause currCause : ((CauseAction) currAction).getCauses()) { if (currCause instanceof TeamPushCause) { return ((TeamPushCause) currCause).getRunContext(); } } } } return DEFAULT_RUN_CONTEXT; }
Source:
<code><START> public void testTimeoutWasNotTriggered() throws IOException, ExecutionException, InterruptedException { <END> FreeStyleProject project = j.createFreeStyleProject(); project.setDisplayName("Check timeout"); Builder step = new BuildStepWithTimeout(new FakeBuildStep(0), new QuickBuildTimeOutStrategy(500), null); project.getBuildersList().add(step); Future<FreeStyleBuild> future = project.scheduleBuild2(0, new Cause.UserIdCause()); FreeStyleBuild build = future.get(); assertTrue(IOUtils.toString(build.getLogReader()).contains("Test")); }</code><technical_language>JFYI: easy for throws Exception in test codes</technical_language>

Target:
public void testTimeoutWasNotTriggered() throws Exception { final FreeStyleProject project = createProjectWithBuildStepWithTimeout(TINY_DELAY, null); final FreeStyleBuild build = project.scheduleBuild2(0, new Cause.UserIdCause()).get(); j.assertBuildStatusSuccess(build); j.assertLogContains("Test", build); }
Source:
<code>public WritableComponentView(Composite parent, final WritableComponentProperty property, boolean displayName, final boolean isPreview) { super(parent, SWT.NONE); setLayout(new FillLayout(SWT.VERTICAL)); this.parent = parent; this.property = property; if (displayName) { propertyName = new Label(this, SWT.NONE); propertyName.setAlignment(SWT.CENTER); propertyName.setBackground(SWTResourceManager.getColor(SWT.COLOR_WHITE)); propertyName.setFont(SWTResourceManager.getFont("Arial", 10, SWT.BOLD)); propertyName.setText(property.displayName()); } composite = new Composite(this, SWT.NONE); composite.setBackground(SWTResourceManager.getColor(SWT.COLOR_CYAN)); GridLayout glComposite = new GridLayout(1, false); glComposite.horizontalSpacing = 1; glComposite.marginWidth = 2; glComposite.verticalSpacing = 0; glComposite.marginHeight = 2; composite.setLayout(glComposite); text = new Text(composite, SWT.BORDER | SWT.RIGHT); text.setLayoutData(new GridData(SWT.FILL, SWT.CENTER, true, false, 1, 1)); text.setFont(SWTResourceManager.getFont("Arial", 9, SWT.NORMAL)); bindText(property); text.addFocusListener(new FocusListener() { @Override public void focusLost(FocusEvent e) { text.removeModifyListener(textModifyListener); } @Override public void focusGained(FocusEvent e) { text.addModifyListener(textModifyListener); } }); text.addListener(SWT.Traverse, new Listener() { @Override public void handleEvent(Event event) { if (event.detail == SWT.TRAVERSE_RETURN) { if (!isPreview) { <START> sendValue(); <END> } } } }); }</code><technical_language>&& nested ifs (I this code added clean modifying anyway)</technical_language>

Target:
public WritableComponentView(Composite parent, final WritableComponentProperty property, boolean displayName, final boolean isPreview) { super(parent, SWT.NONE); setLayout(new FillLayout(SWT.VERTICAL)); this.parent = parent; this.property = property; if (displayName) { propertyName = new Label(this, SWT.NONE); propertyName.setAlignment(SWT.CENTER); propertyName.setBackground(SWTResourceManager.getColor(SWT.COLOR_WHITE)); propertyName.setFont(SWTResourceManager.getFont("Arial", 10, SWT.BOLD)); propertyName.setText(property.displayName()); } composite = new Composite(this, SWT.NONE); composite.setBackground(SWTResourceManager.getColor(SWT.COLOR_CYAN)); GridLayout glComposite = new GridLayout(1, false); glComposite.horizontalSpacing = 1; glComposite.marginWidth = 2; glComposite.verticalSpacing = 0; glComposite.marginHeight = 2; composite.setLayout(glComposite); text = new Text(composite, SWT.BORDER | SWT.RIGHT); text.setLayoutData(new GridData(SWT.FILL, SWT.CENTER, true, false, 1, 1)); text.setFont(SWTResourceManager.getFont("Arial", 9, SWT.NORMAL)); bindText(property); text.addListener(SWT.Traverse, new Listener() { @Override public void handleEvent(Event event) { if (event.detail == SWT.TRAVERSE_RETURN && !isPreview) { sendValue(); } } }); }
Source:
<code><START> private long sendRequestTimeoutMillis() <END> { int emitQueueSize = approximateBuffersToEmitCount.get(); if (emitQueueSize < 5) { return (long) (lastFillTimeMillis * config.httpTimeoutAllowanceFactor); } if (emitQueueSize < 10) { return (long) (lastFillTimeMillis * 0.9); } return (long) (lastFillTimeMillis * 0.5); }</code><technical_language>add constants, configs set right</technical_language>

Target:
private long sendRequestTimeoutMillis() { int emitQueueSize = approximateBuffersToEmitCount.get(); if (emitQueueSize < EMIT_QUEUE_THRESHOLD_1) { return (long) (lastFillTimeMillis * config.httpTimeoutAllowanceFactor); } if (emitQueueSize < EMIT_QUEUE_THRESHOLD_2) { return (long) (lastFillTimeMillis * EQUILIBRIUM_ALLOWANCE_FACTOR); } return (long) (lastFillTimeMillis * TIGHT_ALLOWANCE_FACTOR); }
Source:
<code>private DevicePropertiesUtils mockDevicePropertiesUtils(String customDevPropSpec) { DevicePropertiesUtils mockedUtils = spy(new DevicePropertiesUtils()); doReturn(customDevPropSpec). when(mockedUtils) .getCustomDeviceProperties(eq(Version.v3_3)); doReturn(""). when(mockedUtils) .getCustomDeviceProperties(eq(Version.v3_2)); doReturn(new HashSet<Version>(Arrays.asList(Version.v3_2, Version.v3_3))). when(mockedUtils) .getSupportedClusterLevels(); try { mockedUtils.init(); } catch (InitializationException e) { <START> e.printStackTrace(); <END> } return mockedUtils; }</code><technical_language>please a proper logger instead of standard output remove this line completely</technical_language>

Target:
private DevicePropertiesUtils mockDevicePropertiesUtils(String customDevPropSpec) { DevicePropertiesUtils mockedUtils = spy(new DevicePropertiesUtils()); doReturn(customDevPropSpec). when(mockedUtils) .getCustomDeviceProperties(eq(Version.v3_3)); doReturn(""). when(mockedUtils) .getCustomDeviceProperties(eq(Version.v3_2)); doReturn(new HashSet<Version>(Arrays.asList(Version.v3_2, Version.v3_3))). when(mockedUtils) .getSupportedClusterLevels(); try { mockedUtils.init(); } catch (InitializationException ex) { log.error("Error initializing DevicePropertiesUtils instance", ex); } return mockedUtils; }
Source:
<code>public static boolean onFarmlandTrample(World world, BlockPos pos, IBlockState state, float fallDistance, Entity entity) { boolean trampleResult = entity.canTrample(world, state.getBlock(), pos, fallDistance); <START> BlockEvent.FarmlandTrampleEvent event = new BlockEvent.FarmlandTrampleEvent(world, pos, state, fallDistance, trampleResult, entity); <END> MinecraftForge.EVENT_BUS.post(event); return event.getResult() == Event.Result.ALLOW || (event.getResult() == Event.Result.DEFAULT && trampleResult); }</code><technical_language>fire event if entity wouldnt trample ground? Is real case for making entities more destructive are</technical_language>

Target:
public static boolean onFarmlandTrample(World world, BlockPos pos, IBlockState state, float fallDistance, Entity entity) { if (entity.canTrample(world, state.getBlock(), pos, fallDistance)) { BlockEvent.FarmlandTrampleEvent event = new BlockEvent.FarmlandTrampleEvent(world, pos, state, fallDistance, entity); MinecraftForge.EVENT_BUS.post(event); return !event.isCanceled(); } return false; }
Source:
<code>public void testSwitchStatement() { enableTypeInfoValidation = false; String switchStatement = lines( "function foo(x) {", " switch(x) {", " case 1: ", " var y = 5;", <START> " default:", <END> " var y = 5;", " case 2: ", " var y = 5;", " }", "}"); valid(switchStatement); Node switchNode = parseValidScript(switchStatement); Node defaultBlock = IR.block(); Node nodeToAppendChildTo = stream(NodeUtil.preOrderIterable(switchNode)) .filter(node -> node.isCase()) .findFirst() .get(); nodeToAppendChildTo.addChildToFront(defaultBlock); expectInvalid(switchNode, Check.SCRIPT); }</code><technical_language>Is default this expression invalid? add defaultBlock node tree invalid? is point of defaultBlock below</technical_language>

Target:
public void testSwitchStatement() { enableTypeInfoValidation = false; String switchStatement = lines( "function foo(x) {", " switch(x) {", " case 1: ", " var y = 5;", " case 2: ", " var y = 5;", " }", "}"); valid(switchStatement); Node rootScriptNode = parseValidScript(switchStatement); Node blockNode = IR.block(); Node nodeToAppendChildTo = stream(NodeUtil.preOrderIterable(rootScriptNode)) .filter(node -> node.isCase()) .findFirst() .get(); nodeToAppendChildTo.addChildToFront(blockNode); expectInvalid(rootScriptNode, Check.SCRIPT); }
Source:
<code>private void initCreateButton() { createButton = new Button(Util.C.buttonCreateProject()); createButton.setEnabled(false); createButton.addClickHandler(new ClickHandler() { @Override public void onClick(final ClickEvent event) { <START> enableForm(false); <END> doCreateProject(); } }); }</code><technical_language>I is confusing put this instead of in doCreateProject (right RPC call). If ends another trigger create (via keyboard...), missed, duplicated</technical_language>

Target:
private void initCreateButton() { createButton = new Button(Util.C.buttonCreateProject()); createButton.setEnabled(false); createButton.addClickHandler(new ClickHandler() { @Override public void onClick(final ClickEvent event) { doCreateProject(); } }); }
Source:
<code>private static void waitForSymbolNames(String... symbolNames) { final SWTBotView viewBot = <START> fBot.viewById(CallStackView.ID); <END> List<String> symbolNameList = Lists.newArrayList(symbolNames); WaitUtils.waitUntil(vBot -> symbolNameList.equals(getVisibleStackFrames(vBot)), viewBot, "Wrong symbol names, expected:" + symbolNameList + ", got: " + getVisibleStackFrames(viewBot)); }</code><technical_language>field name in a static method... rename sfBot</technical_language>

Target:
private static void waitForSymbolNames(String... symbolNames) { final SWTBotView viewBot = sfBot.viewById(CallStackView.ID); List<String> symbolNameList = Lists.newArrayList(symbolNames); WaitUtils.waitUntil(vBot -> symbolNameList.equals(getVisibleStackFrames(vBot)), viewBot, "Wrong symbol names, expected:" + symbolNameList + ", got: " + getVisibleStackFrames(viewBot)); }
Source:
<code>public static List<TheoreticalSensitivityMetrics> calculateSensitivities(final int simulationSize, final Histogram<Integer> depthHistogram, final Histogram<Integer> baseQHistogram, final List<Double> alleleFractions) { final List<TheoreticalSensitivityMetrics> metricsOverVariousAlleleFractions = new ArrayList<>(); final double logOddsThreshold = 6.2; final Histogram<Double> sensitivityHistogram = new Histogram<>(); sensitivityHistogram.setBinLabel("allele_fraction"); sensitivityHistogram.setValueLabel("theoretical_sensitivity"); <START> for (Double alleleFraction : alleleFractions) { <END> final TheoreticalSensitivityMetrics theoreticalSensitivityMetrics = new TheoreticalSensitivityMetrics(); log.info("Calculating theoretical sensitivity for AF = " + alleleFraction + "."); theoreticalSensitivityMetrics.ALLELE_FRACTION = alleleFraction; theoreticalSensitivityMetrics.THEORETICAL_SENSITIVITY = TheoreticalSensitivity.theoreticalSensitivity(depthHistogram, baseQHistogram, simulationSize, logOddsThreshold, alleleFraction); theoreticalSensitivityMetrics.SIMULATION_SIZE = simulationSize; theoreticalSensitivityMetrics.LOG_ODDS_THRESHOLD = logOddsThreshold; theoreticalSensitivityMetrics.LARGE_NUMBER_OF_DRAWS = LARGE_NUMBER_OF_DRAWS; theoreticalSensitivityMetrics.DEPTH_BIN_WIDTH = DEPTH_BIN_WIDTH; theoreticalSensitivityMetrics.RANDOM_SEED = RANDOM_SEED; metricsOverVariousAlleleFractions.add(theoreticalSensitivityMetrics); } return metricsOverVariousAlleleFractions; }</code><technical_language>alleleFraction final</technical_language>

Target:
public static List<TheoreticalSensitivityMetrics> calculateSensitivities(final int simulationSize, final Histogram<Integer> depthHistogram, final Histogram<Integer> baseQHistogram, final List<Double> alleleFractions) { final List<TheoreticalSensitivityMetrics> metricsOverVariousAlleleFractions = new ArrayList<>(); final double logOddsThreshold = 6.2; for (final double alleleFraction : alleleFractions) { final TheoreticalSensitivityMetrics theoreticalSensitivityMetrics = new TheoreticalSensitivityMetrics(); theoreticalSensitivityMetrics.ALLELE_FRACTION = alleleFraction; theoreticalSensitivityMetrics.THEORETICAL_SENSITIVITY = TheoreticalSensitivity.theoreticalSensitivity(depthHistogram, baseQHistogram, simulationSize, logOddsThreshold, alleleFraction); theoreticalSensitivityMetrics.THEORETICAL_SENSITIVITY_Q = QualityUtil.getPhredScoreFromErrorProbability((1 - theoreticalSensitivityMetrics.THEORETICAL_SENSITIVITY)); theoreticalSensitivityMetrics.SAMPLE_SIZE = simulationSize; theoreticalSensitivityMetrics.LOG_ODDS_THRESHOLD = logOddsThreshold; metricsOverVariousAlleleFractions.add(theoreticalSensitivityMetrics); } return metricsOverVariousAlleleFractions; }
Source:
<code>static Set<String> getDisabledTests(Path testMappingsRootPath, String testGroup) { Set<String> disabledTests = new HashSet<>(); if (!testGroup.equals(PRESUBMIT)) { return disabledTests; } File <START> disabledPresubmitTestsFile = <END> new File(testMappingsRootPath.toString(), DISABLED_PRESUBMIT_TESTS); try { disabledTests.addAll( Arrays.asList( FileUtil.readStringFromFile(disabledPresubmitTestsFile) .split("\\r?\\n"))); } catch (IOException e) { RuntimeException runtimeException = new RuntimeException( String.format( "IO error (%s) when reading disabled tests from file (%s)", e.getMessage(), disabledPresubmitTestsFile.getAbsolutePath()), e); throw runtimeException; } return disabledTests; }</code><technical_language>check file exists</technical_language>

Target:
static Set<String> getDisabledTests(Path testMappingsRootPath, String testGroup) { Set<String> disabledTests = new HashSet<>(); File disabledPresubmitTestsFile = new File(testMappingsRootPath.toString(), DISABLED_PRESUBMIT_TESTS_FILE); if (!(testGroup.equals(PRESUBMIT) && disabledPresubmitTestsFile.exists())) { return disabledTests; } try { disabledTests.addAll( Arrays.asList( FileUtil.readStringFromFile(disabledPresubmitTestsFile) .split("\\r?\\n"))); } catch (IOException e) { throw new RuntimeException( String.format( "IO exception (%s) when reading disabled tests from file (%s)", e.getMessage(), disabledPresubmitTestsFile.getAbsolutePath()), e); } return disabledTests; }