auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public void testAuthentication() throws Exception { deployContrib("org.nuxeo.ecm.platform.login.cas2.test", "OSGI-INF/login-yes-contrib.xml"); <START> applyInlineDeployments(); <END> initRequest(); setLoginPasswordInHeader("Administrator", "Administrator", request); naf.doFilter(request, response, chain); String loginError = (String) request.getAttribute(NXAuthConstants.LOGIN_ERROR); LoginContext loginContext = (LoginContext) request.getAttribute("org.nuxeo.ecm.login.context"); assertNull(loginError); assertNotNull(loginContext); assertEquals("Administrator", ((Principal) loginContext.getSubject().getPrincipals().toArray()[0]).getName()); }</code><technical_language>pushInlineDeployments ? - of deployContrib a test contribution</technical_language>

Target:
public void testAuthentication() throws Exception { pushInlineDeployments("org.nuxeo.ecm.platform.login.cas2.test:OSGI-INF/login-yes-contrib.xml"); initRequest(); setLoginPasswordInHeader("Administrator", "Administrator", request); naf.doFilter(request, response, chain); String loginError = (String) request.getAttribute(NXAuthConstants.LOGIN_ERROR); LoginContext loginContext = (LoginContext) request.getAttribute("org.nuxeo.ecm.login.context"); assertNull(loginError); assertNotNull(loginContext); assertEquals("Administrator", ((Principal) loginContext.getSubject().getPrincipals().toArray()[0]).getName()); }
Source:
<code>private QueryParams getParams(GridSettings settings) { QueryParams result; <START> if (settings.getSidx().equals("")) { <END> result = new QueryParams(settings.getPage(), settings.getRows(), new Order("date", Order.Direction.DESC)); } else { if (settings.getSord().equals("asc")) { result = new QueryParams(settings.getPage(), settings.getRows(), new Order(settings.getSidx(), Order.Direction.ASC)); } else { result = new QueryParams(settings.getPage(), settings.getRows(), new Order(settings.getSidx(), Order.Direction.DESC)); } } return result; }</code><technical_language>StringUtils.isEmpty</technical_language>

Target:
private QueryParams getParams(GridSettings settings) { QueryParams result; if (StringUtils.isEmpty(settings.getSidx())) { result = new QueryParams(settings.getPage(), settings.getRows(), new Order("date", Order.Direction.DESC)); } else { if ("asc".equals(settings.getSord())) { result = new QueryParams(settings.getPage(), settings.getRows(), new Order(settings.getSidx(), Order.Direction.ASC)); } else { result = new QueryParams(settings.getPage(), settings.getRows(), new Order(settings.getSidx(), Order.Direction.DESC)); } } return result; }
Source:
<code>private static Path getCmRoot(Path path) throws IOException { Path cmroot = null; HdfsEncryptionShim pathEncryptionShim = hadoopShims.createHdfsEncryptionShim(path.getFileSystem(conf), conf); if (!pathEncryptionShim.isPathEncrypted(path)) { cmroot = cmRootMapping.get(NO_ENCRYPTION); } else { EncryptionZone encryptionZone = pathEncryptionShim.getEncryptionZoneForPath(path); <START> cmroot = cmRootMapping.get(encryptionZone.getPath()); <END> if (cmroot == null) { synchronized (instance) { cmroot = new Path(path.getFileSystem(conf).getUri() + encryptionZone.getPath() + MetastoreConf.getVar(conf, ConfVars.REPLCMENCRYPTEDDIR)); FileSystem cmFs = cmroot.getFileSystem(conf); if (!cmFs.exists(cmroot)) { cmFs.mkdirs(cmroot); cmFs.setPermission(cmroot, new FsPermission("700")); } cmRootMapping.put(encryptionZone.getPath(), cmroot); } } } return cmroot; }</code><technical_language>work fine if path is encrypted</technical_language>

Target:
static Path getCmRoot(Path path) throws IOException { Path cmroot = null; String cmrootDir = fallbackNonEncryptedCmRootDir; String encryptionZonePath = NO_ENCRYPTION; if (enabled) { HdfsEncryptionShim pathEncryptionShim = hadoopShims.createHdfsEncryptionShim(path.getFileSystem(conf), conf); if (pathEncryptionShim.isPathEncrypted(path)) { encryptionZonePath = path.getFileSystem(conf).getUri() + pathEncryptionShim.getEncryptionZoneForPath(path).getPath(); cmrootDir = encryptionZonePath + Path.SEPARATOR + encryptedCmRootDir; } if (encryptionZones.containsKey(encryptionZonePath)) { cmroot = new Path(encryptionZones.get(encryptionZonePath)); } else { cmroot = new Path(cmrootDir); synchronized (instance) { if (!encryptionZones.containsKey(encryptionZonePath)) { createCmRoot(cmroot); encryptionZones.put(encryptionZonePath, cmrootDir); } } } } return cmroot; }
Source:
<code>public static ResourceNode getRootResourceNodeFromResource(ResourceCapabilityTree tree, IResource resource) { log.debug("Looking for IRootResource associated with this resource: " + resource); log.trace("Looking for ResourceNode in the tree containing this resource: " + resource); ResourceNode resourceNode = getResourceNodeWithContent(tree.getRootResourceNode(), resource); if (resourceNode == null) { log.warn("Resource not located in the tree! Resource: " + resource); return null; } while (true) { <START> if (resourceNode.getContent() != null && resourceNode.getContent() instanceof IRootResource) { <END> log.trace("IRootResource Resource Node found for resource: " + resourceNode.getContent()); return resourceNode; } CapabilityNode capabilityNode = ((CapabilityNode) resourceNode.getParent()); if (capabilityNode == null) { log.warn("No IRootResource found in the tree for resource: " + resource); return null; } resourceNode = capabilityNode.getParent(); } }</code><technical_language>resourceNode.getContent() instanceof IRootResource. This expression true, if resource.getContent() != null</technical_language>

Target:
public static ResourceNode getRootResourceNodeFromResource(ResourceCapabilityTree tree, IResource resource) { log.debug("Looking for IRootResource associated with this resource: " + resource); log.trace("Looking for ResourceNode in the tree containing this resource: " + resource); ResourceNode resourceNode = getResourceNodeWithContent(tree.getRootResourceNode(), resource); if (resourceNode == null) { log.warn("Resource not located in the tree! Resource: " + resource); return null; } while (true) { if (resourceNode.getContent() instanceof IRootResource) { log.trace("IRootResource Resource Node found for resource: " + resourceNode.getContent()); return resourceNode; } CapabilityNode capabilityNode = ((CapabilityNode) resourceNode.getParent()); if (capabilityNode == null) { log.warn("No IRootResource found in the tree for resource: " + resource); return null; } resourceNode = capabilityNode.getParent(); } }
Source:
<code>public SecretAsyncClient buildAsyncClient() { Configuration buildConfiguration = (configuration == null) ? Configuration.getGlobalConfiguration().clone() : configuration; URL buildEndpoint = getBuildEndpoint(buildConfiguration); if (buildEndpoint == null) { throw logger.logExceptionAsError( new IllegalStateException( KeyVaultErrorCodeStrings.getErrorString(KeyVaultErrorCodeStrings.VAULT_END_POINT_REQUIRED))); } SecretServiceVersion serviceVersion = version != null ? version : SecretServiceVersion.getLatest(); if (pipeline != null) { return new SecretAsyncClient(vaultUrl, pipeline, serviceVersion); } if (credential == null) { throw logger.logExceptionAsError( new IllegalStateException( KeyVaultErrorCodeStrings.getErrorString(KeyVaultErrorCodeStrings.CREDENTIAL_REQUIRED))); } final List<HttpPipelinePolicy> policies = new ArrayList<>(); <START> policies.add(new UserAgentPolicy(httpLogOptions.getApplicationId(), properties.get(SDK_NAME), <END> properties.get(SDK_VERSION), buildConfiguration)); HttpPolicyProviders.addBeforeRetryPolicies(policies); policies.add(retryPolicy); policies.add(new KeyVaultCredentialPolicy(credential)); policies.addAll(this.policies); HttpPolicyProviders.addAfterRetryPolicies(policies); policies.add(new HttpLoggingPolicy(httpLogOptions)); HttpPipeline pipeline = new HttpPipelineBuilder() .policies(policies.toArray(new HttpPipelinePolicy[0])) .httpClient(httpClient) .build(); return new SecretAsyncClient(vaultUrl, pipeline, serviceVersion); }</code><technical_language>Needs defaults</technical_language>

Target:
public SecretAsyncClient buildAsyncClient() { Configuration buildConfiguration = (configuration == null) ? Configuration.getGlobalConfiguration().clone() : configuration; URL buildEndpoint = getBuildEndpoint(buildConfiguration); if (buildEndpoint == null) { throw logger.logExceptionAsError( new IllegalStateException( KeyVaultErrorCodeStrings.getErrorString(KeyVaultErrorCodeStrings.VAULT_END_POINT_REQUIRED))); } SecretServiceVersion serviceVersion = version != null ? version : SecretServiceVersion.getLatest(); if (pipeline != null) { return new SecretAsyncClient(vaultUrl, pipeline, serviceVersion); } if (credential == null) { throw logger.logExceptionAsError( new IllegalStateException( KeyVaultErrorCodeStrings.getErrorString(KeyVaultErrorCodeStrings.CREDENTIAL_REQUIRED))); } final List<HttpPipelinePolicy> policies = new ArrayList<>(); String clientName = properties.getOrDefault(SDK_NAME, "UnknownName"); String clientVersion = properties.getOrDefault(SDK_VERSION, "UnknownVersion"); policies.add(new UserAgentPolicy(httpLogOptions.getApplicationId(), clientName, clientVersion, buildConfiguration)); HttpPolicyProviders.addBeforeRetryPolicies(policies); policies.add(retryPolicy); policies.add(new KeyVaultCredentialPolicy(credential)); policies.addAll(this.policies); HttpPolicyProviders.addAfterRetryPolicies(policies); policies.add(new HttpLoggingPolicy(httpLogOptions)); HttpPipeline pipeline = new HttpPipelineBuilder() .policies(policies.toArray(new HttpPipelinePolicy[0])) .httpClient(httpClient) .build(); return new SecretAsyncClient(vaultUrl, pipeline, serviceVersion); }
Source:
<code>public static Bundle getBundleByMap(Map<String, Object> row){ Bundle b = new Bundle(); b.setId(row.get("id").toString()); b.setName(row.get("name").toString()); b.setPublishDate((Date)row.get("publish_date")); b.setExpireDate((Date)row.get("expire_date")); <START> b.setOwner(row.get("owner")!=null?row.get("owner").toString():""); <END> b.setForcePush(row.get("force_push") == null ? false :DbConnectionFactory.isDBTrue(row.get("force_push").toString())); return b; }</code><technical_language>UtilMethods.isSet</technical_language>

Target:
public static Bundle getBundleByMap(Map<String, Object> row){ Bundle b = new Bundle(); b.setId(row.get("id").toString()); b.setName(row.get("name").toString()); b.setPublishDate((Date)row.get("publish_date")); b.setExpireDate((Date)row.get("expire_date")); b.setOwner(UtilMethods.isSet(row.get("owner"))?row.get("owner").toString():""); b.setForcePush(UtilMethods.isSet(row.get("force_push")) && DbConnectionFactory .isDBTrue(row.get("force_push").toString())); return b; }
Source:
<code>void verifyMetadata(MessageMetadata metadata) throws Exception { if (metadata.getMessageMetadataType() != MessageType.LOG_ENTRY_MESSAGE || metadata.getSnapshotTimestamp() != srcGlobalSnapshot) { log.error("Wrong message metadata {}, expecting type snapshot {}", metadata, MessageType.LOG_ENTRY_MESSAGE, srcGlobalSnapshot); <START> throw new Exception("wrong type of message"); <END> } }</code><technical_language>need a specific Exception Type</technical_language>

Target:
void verifyMetadata(MessageMetadata metadata) throws Exception { if (metadata.getMessageMetadataType() != MessageType.LOG_ENTRY_MESSAGE) { log.error("Wrong message metadata {}, expecting type snapshot {}", metadata, MessageType.LOG_ENTRY_MESSAGE, srcGlobalSnapshot); throw new Exception("wrong type of message"); } }
Source:
<code>public Optional<Image> getErrorImage() { <START> return Optional.of(errorImage); <END> }</code><technical_language>This ofNullable, right</technical_language>

Target:
public Optional<Image> getErrorImage() { return Optional.ofNullable(errorImage); }
Source:
<code>private boolean isOneToManyAndAdd(ReferenceChange diff, boolean mergeRightToLeft) { if (diff.getReference().getEOpposite() == null) { <START> return false; <END> } boolean isOneToMany = diff.getReference().isMany() && !diff.getReference().getEOpposite().isMany(); boolean isManyToOne = !diff.getReference().isMany() && diff.getReference().getEOpposite().isMany(); return (isOneToMany || isManyToOne) && isAdd(diff, mergeRightToLeft); }</code><technical_language>I good avoid multiple returns enclosing negated if or, easier read, extract a boolean variable: final boolean hasOpposite = diff.getReference().getEOpposite() != null; final boolean isOneToMany = hasOpposite && diff.getReference().isMany ...; final boolean isManyToOne = hasOpposite && diff.getReference().isMany ...; return (isOneToMany || isManyToOne) && ...; I convention in EMF Compare is try enforce final possible</technical_language>

Target:
private boolean isOneToManyAndAdd(ReferenceChange diff, boolean mergeRightToLeft) { final boolean hasOpposite = diff.getReference().getEOpposite() != null; final boolean isOneToMany = hasOpposite && diff.getReference().isMany() && !diff.getReference().getEOpposite().isMany(); final boolean isManyToOne = hasOpposite && !diff.getReference().isMany() && diff.getReference().getEOpposite().isMany(); return (isOneToMany || isManyToOne) && isAdd(diff, mergeRightToLeft); }
Source:
<code>private Long getTargetTxnId(String replPolicy, long sourceTxnId, Statement stmt) throws SQLException { ResultSet rs; String s = "select RTM_TARGET_TXN_ID from REPL_TXN_MAP where RTM_SRC_TXN_ID = " + sourceTxnId + " and RTM_REPL_POLICY = " + quoteString(replPolicy); LOG.debug("Going to execute query <" + s + ">"); rs = stmt.executeQuery(s); if (!rs.next()) { LOG.info("Target txn is missing for the input source txn for ReplPolicy: " + quoteString(replPolicy) + " , srcTxnId: " + sourceTxnId); return -1L; } LOG.debug("targetTxnid for srcTxnId " + sourceTxnId + " is " + rs.getLong(1)); <START> return rs.getLong(1); <END> }</code><technical_language>close(rs) missing</technical_language>

Target:
private Long getTargetTxnId(String replPolicy, long sourceTxnId, Statement stmt) throws SQLException { ResultSet rs = null; try { String s = "select RTM_TARGET_TXN_ID from REPL_TXN_MAP where RTM_SRC_TXN_ID = " + sourceTxnId + " and RTM_REPL_POLICY = " + quoteString(replPolicy); LOG.debug("Going to execute query <" + s + ">"); rs = stmt.executeQuery(s); if (!rs.next()) { LOG.info("Target txn is missing for the input source txn for ReplPolicy: " + quoteString(replPolicy) + " , srcTxnId: " + sourceTxnId); return -1L; } LOG.debug("targetTxnid for srcTxnId " + sourceTxnId + " is " + rs.getLong(1)); return rs.getLong(1); } catch (SQLException e) { LOG.warn("failed to get target txn ids " + e.getMessage()); throw e; } finally { close(rs); } }
Source:
<code>public void editLinesFromStart() { int state = state_; state_ = (state_ + 1) % 4; if (cursorChangedHandler_ == null) cursorChangedHandler_ = editor_.addCursorChangedHandler(this); if (range_ == null) range_ = getSelectionExtent(); isPerformingEditLinesAction_ = true; <START> switch (state) <END> { case 0: editLinesFromStart(false, false); break; case 1: editLinesFromStart(false, true); break; case 2: editLinesFromStart(true, false); break; case 3: editLinesFromStart(true, true); break; } isPerformingEditLinesAction_ = false; }</code><technical_language>Nit: a try/catch this exception stick isPerformingEditLinesAction_ in wrong state</technical_language>

Target:
public void editLinesFromStart() { int state = state_; state_ = (state_ + 1) % 4; if (cursorChangedHandler_ == null) cursorChangedHandler_ = editor_.addCursorChangedHandler(this); if (range_ == null) range_ = getSelectionExtent(); try { isPerformingEditLinesAction_ = true; switch (state) { case 0: editLinesFromStart(false, false); break; case 1: editLinesFromStart(false, true); break; case 2: editLinesFromStart(true, false); break; case 3: editLinesFromStart(true, true); break; } } catch (Exception e) { Debug.logException(e); } finally { isPerformingEditLinesAction_ = false; } }
Source:
<code><START> public BoundedSource<Result> createSourceWithKeys(byte[] startKey, byte[] stopKey, long size) { <END> return new SourceWithKeys(startKey, stopKey, size); }</code><technical_language>package default, for testing only</technical_language>

Target:
BoundedSource<Result> createSourceWithKeys(byte[] startKey, byte[] stopKey, long size) { return new SourceWithKeys(startKey, stopKey, size); }
Source:
<code><START> public static void fireNow(@NonNull final SCMHeadEvent<?> event) { <END> executorService().execute(new Dispatcher(event)); }</code><technical_language>A misleading name; fired (synchronously), soon</technical_language>

Target:
public static void fireNow(@NonNull final SCMHeadEvent<?> event) { executorService().execute(new DispatcherImpl(event)); }
Source:
<code>public void test() { <START> connect(); <END> waitResponse(5000); click(jq("@button")); waitResponse(); Assert.assertEquals("true", getZKLog()); }</code><technical_language>connect, try getStatusCode</technical_language>

Target:
public void test() { Assert.assertEquals(404, getStatusCode(getAddress() + "/zkau/web/something.js.map")); }
Source:
<code>public ServiceConfigurator configure(OperationContext context, ModelNode model) throws OperationFailedException { this.mode = ModelNodes.asEnum(MODE.resolveModelAttribute(context, model), TransactionMode.class); this.locking = ModelNodes.asEnum(LOCKING.resolveModelAttribute(context, model), LockingMode.class); this.timeout = STOP_TIMEOUT.resolveModelAttribute(context, model).asLong(); this.transactionDependency = !EnumSet.of(TransactionMode.NONE, TransactionMode.BATCH).contains(this.mode) ? new ServiceDependency(context.getCapabilityServiceName(LOCAL_TRANSACTION_PROVIDER.getName(), null)) : null; <START> this.tsrServiceName = this.mode == TransactionMode.NON_XA ? context.getCapabilityServiceName(TRANSACTION_SYNCHRONIZATION_REGISTRY.getName(), null) : null; <END> return this; }</code><technical_language>This becomes: this.tsrDependency = (this.mode == TransactionMode.NON_XA) ? new ServiceSupplierDependency<>(context.getCapabilityServiceName(TRANSACTION_SYNCHRONIZATION_REGISTRY.getName(), null)) : null;</technical_language>

Target:
public ServiceConfigurator configure(OperationContext context, ModelNode model) throws OperationFailedException { this.mode = ModelNodes.asEnum(MODE.resolveModelAttribute(context, model), TransactionMode.class); this.locking = ModelNodes.asEnum(LOCKING.resolveModelAttribute(context, model), LockingMode.class); this.timeout = STOP_TIMEOUT.resolveModelAttribute(context, model).asLong(); this.transactionDependency = !EnumSet.of(TransactionMode.NONE, TransactionMode.BATCH).contains(this.mode) ? new ServiceDependency(context.getCapabilityServiceName(LOCAL_TRANSACTION_PROVIDER.getName(), null)) : null; this.tsrDependency = this.mode == TransactionMode.NON_XA ? new ServiceSupplierDependency<>(context.getCapabilityServiceName(TRANSACTION_SYNCHRONIZATION_REGISTRY.getName(), null)) : null; return this; }
Source:
<code>public Cursor selectAll() { <START> return select("", new String[] {}, null); <END> }</code><technical_language>params null, null, null</technical_language>

Target:
public Cursor selectAll() { return select(null, null, null); }
Source:
<code>public void testDistExecScript() throws InterruptedException, ExecutionException, IOException { ScriptingManager scriptingManager = getScriptingManager(manager(0)); Cache<String, String> cache = cache(0); loadData(cache, "/macbeth.txt"); loadScript(scriptingManager, "/wordCountStream.js"); CompletableFuture<Map<String, Long>> resultsFuture = scriptingManager.runScript("wordCountStream.js", new TaskContext().cache(cache(0))); Map<String, Long> results = resultsFuture.get(); assertEquals(3209, results.size()); <START> assertTrue(results.get("macbeth").equals(Long.valueOf(287))); <END> }</code><technical_language>assertEquals well</technical_language>

Target:
public void testDistExecScript() throws InterruptedException, ExecutionException, IOException { ScriptingManager scriptingManager = getScriptingManager(manager(0)); Cache<String, String> cache = cache(0); loadData(cache, "/macbeth.txt"); loadScript(scriptingManager, "/wordCountStream.js"); CompletableFuture<Map<String, Long>> resultsFuture = scriptingManager.runScript("wordCountStream.js", new TaskContext().cache(cache(0))); Map<String, Long> results = resultsFuture.get(); assertEquals(3209, results.size()); assertEquals(results.get("macbeth"), Long.valueOf(287)); }
Source:
<code>public void visitFunctionCall(FunctionCallTree tree) { String functionName = CheckUtils.getFunctionName(tree); if (STREAM_CONTEXT_CREATE.equals(functionName)) { <START> SeparatedList<ExpressionTree> arguments = tree.arguments(); <END> if (!arguments.isEmpty()) { checkSSLConfig(arguments.get(0)); } } if (STREAM_SOCKET_ENABLE_CRYPTO.equals(functionName)) { SeparatedList<ExpressionTree> arguments = tree.arguments(); if (arguments.size() > 2) { checkWeakProtocol(getAssignedValue(arguments.get(2)), STREAM_SOCKET_ENABLE_CRYPTO); } } super.visitFunctionCall(tree); }</code><technical_language>this a List<ExpressionTree>, I a feature of SeparatedList</technical_language>

Target:
public void visitFunctionCall(FunctionCallTree tree) { String functionName = CheckUtils.getFunctionName(tree); if (STREAM_CONTEXT_CREATE.equals(functionName)) { List<ExpressionTree> arguments = tree.arguments(); if (!arguments.isEmpty()) { checkSSLConfig(arguments.get(0)); } } if (STREAM_SOCKET_ENABLE_CRYPTO.equals(functionName)) { SeparatedList<ExpressionTree> arguments = tree.arguments(); if (arguments.size() > 2) { checkWeakProtocol(getAssignedValue(arguments.get(2)), STREAM_SOCKET_ENABLE_CRYPTO); } } super.visitFunctionCall(tree); }
Source:
<code><START> public EventGridServiceVersion getServiceVersion() { return EventGridServiceVersion.V2018_01_01; } <END></code><technical_language>Instead of hardcoding version here, builder (since builder a setter for service version) pass this constructor client instance variable returned here</technical_language>

Target:
public EventGridServiceVersion getServiceVersion() { return this.serviceVersion; }
Source:
<code>public boolean prepare() { if (!prepareCalled) { prepareCalled = true; List<Subtitle> subtitles = playlist.subtitles; for(Subtitle subtitle: subtitles) { Format format = new Format(subtitle.name, MimeTypes.TEXT_VTT, MediaFormat.NO_VALUE, MediaFormat.NO_VALUE, MediaFormat.NO_VALUE, MediaFormat.NO_VALUE, MediaFormat.NO_VALUE, MediaFormat.NO_VALUE, subtitle.language); <START> MediaFormat trackFormat = MediaFormat.createTextFormat(tracks.size(), format.mimeType, <END> format.bitrate, C.MATCH_LONGEST_US, format.language, 0); if (subtitle.isDefault) { tracks.add(TRACK_DEFAULT, new ExposedTrack(trackFormat, format, subtitle.uri)); } else { tracks.add(new ExposedTrack(trackFormat, format, subtitle.uri)); } } } return true; }</code><technical_language>argument createTextFormat format.id (nb - I guess arg supposed int this change, takes a string :))</technical_language>

Target:
public boolean prepare() { if (!prepareCalled) { prepareCalled = true; List<Subtitle> subtitles = playlist.subtitles; for(Subtitle subtitle: subtitles) { Format format = new Format(subtitle.name, MimeTypes.TEXT_VTT, MediaFormat.NO_VALUE, MediaFormat.NO_VALUE, MediaFormat.NO_VALUE, MediaFormat.NO_VALUE, MediaFormat.NO_VALUE, MediaFormat.NO_VALUE, subtitle.language); MediaFormat trackFormat = MediaFormat.createTextFormat(format.id, format.mimeType, format.bitrate, C.MATCH_LONGEST_US, format.language, 0); if (subtitle.isDefault) { tracks.add(TRACK_DEFAULT, new ExposedTrack(trackFormat, format, subtitle.uri)); } else { tracks.add(new ExposedTrack(trackFormat, format, subtitle.uri)); } } } return true; }
Source:
<code><START> private void sendLayoutError(ViewVisitor.CantVisitException exception) { <END> if (mEditorConnection == null || !mEditorConnection.isValid()) { return; } final OutputStream out = mEditorConnection.getBufferedOutputStream(); final OutputStreamWriter writer = new OutputStreamWriter(out); final JsonWriter j = new JsonWriter(writer); try { j.beginObject(); j.name("type").value("layout_error"); j.name("exception_type").value(exception.getExceptionType()); j.name("cid").value(exception.getName()); j.endObject(); j.flush(); } catch (final IOException e) { Log.e(LOGTAG, "Can't write track_message to server", e); } finally { try { j.close(); } catch (final IOException e) { Log.e(LOGTAG, "Can't close writer.", e); } } }</code><technical_language>Great</technical_language>

Target:
private void sendLayoutError(ViewVisitor.CantVisitException exception) { if (mEditorConnection == null || !mEditorConnection.isValid()) { return; } final OutputStream out = mEditorConnection.getBufferedOutputStream(); final OutputStreamWriter writer = new OutputStreamWriter(out); final JsonWriter j = new JsonWriter(writer); try { j.beginObject(); j.name("type").value("layout_error"); j.name("exception_type").value(exception.getExceptionType()); j.name("cid").value(exception.getName()); j.endObject(); } catch (final IOException e) { Log.e(LOGTAG, "Can't write track_message to server", e); } finally { try { j.close(); } catch (final IOException e) { Log.e(LOGTAG, "Can't close writer.", e); } } }
Source:
<code>public List<Task<? extends Serializable>> handle(Context context) throws SemanticException { DropConstraintMessage msg = deserializer.getDropConstraintMessage(context.dmd.getPayload()); String actualDbName = context.isDbNameEmpty() ? msg.getDB() : context.dbName; String actualTblName = context.isTableNameEmpty() ? msg.getTable() : context.tableName; String constraintName = msg.getConstraint(); AlterTableDesc dropConstraintsDesc = new AlterTableDesc(actualDbName + "." + actualTblName, constraintName, context.eventOnlyReplicationSpec()); Task<DDLWork> dropConstraintsTask = TaskFactory.get(new DDLWork(readEntitySet, writeEntitySet, dropConstraintsDesc), context.hiveConf); List<Task<? extends Serializable>> tasks = new ArrayList<Task<? extends Serializable>>(); tasks.add(dropConstraintsTask); context.log.debug("Added drop constrain task : {}:{}", dropConstraintsTask.getId(), actualTblName); databasesUpdated.put(actualDbName, context.dmd.getEventTo()); return Collections.singletonList(dropConstraintsTask); <START> } <END></code><technical_language>Need update tablesUpdated for DropConstraint well</technical_language>

Target:
public List<Task<? extends Serializable>> handle(Context context) throws SemanticException { DropConstraintMessage msg = deserializer.getDropConstraintMessage(context.dmd.getPayload()); String actualDbName = context.isDbNameEmpty() ? msg.getDB() : context.dbName; String actualTblName = context.isTableNameEmpty() ? msg.getTable() : context.tableName; String constraintName = msg.getConstraint(); AlterTableDesc dropConstraintsDesc = new AlterTableDesc(actualDbName + "." + actualTblName, constraintName, context.eventOnlyReplicationSpec()); Task<DDLWork> dropConstraintsTask = TaskFactory.get(new DDLWork(readEntitySet, writeEntitySet, dropConstraintsDesc), context.hiveConf); List<Task<? extends Serializable>> tasks = new ArrayList<Task<? extends Serializable>>(); tasks.add(dropConstraintsTask); context.log.debug("Added drop constrain task : {}:{}", dropConstraintsTask.getId(), actualTblName); databasesUpdated.put(actualDbName, context.dmd.getEventTo()); tablesUpdated.put(actualDbName + "." + actualTblName, context.dmd.getEventTo()); return Collections.singletonList(dropConstraintsTask); }
Source:
<code>public void actionPerformed(ActionEvent event) { <START> MainApplication.worker.submit(() -> this.submitChangeset()); <END> }</code><technical_language>MainApplication.worker.execute(() -> this.submitChangeset()) if return -- this care if errors out, current thread notified of that</technical_language>

Target:
public void actionPerformed(ActionEvent event) { MainApplication.worker.execute(() -> this.submitChangeset()); }
Source:
<code>public List<PredefinedNewsDefinition> getHiddenPredefinedNewsDefinitions(Long setId, Set<String> roles) { try { String query = "from PredefinedNewsDefinition def " <START> + "where not exists (from def.userConfigurations config where config.newsSet.id = :setId) "; <END> for (int i = 0; i < roles.size(); i++) { query = query.concat( "and :role" + i + " not in elements(def.defaultRoles) "); } Query q = this.getSession().createQuery(query); q.setLong("setId", setId); int count = 0; for (String role : roles) { q.setString("role" + count, role); count++; } return (List<PredefinedNewsDefinition>) q.list(); } catch (HibernateException ex) { throw convertHibernateAccessException(ex); } }</code><technical_language>lean mildly uppercase for overlapping keywords. leaving is</technical_language>

Target:
public List<PredefinedNewsDefinition> getHiddenPredefinedNewsDefinitions(Long setId, Set<String> roles) { try { String query = "FROM PredefinedNewsDefinition def " + "WHERE NOT EXISTS (FROM def.userConfigurations config WHERE config.newsSet.id = :setId) "; for (int i = 0; i < roles.size(); i++) { query = query.concat( "AND :role" + i + " NOT IN elements(def.defaultRoles) "); } Query q = this.getSession().createQuery(query); q.setLong("setId", setId); int count = 0; for (String role : roles) { q.setString("role" + count, role); count++; } return (List<PredefinedNewsDefinition>) q.list(); } catch (HibernateException ex) { throw convertHibernateAccessException(ex); } }
Source:
<code><START> protected SimpleFluentIterable(Iterable<TYPE> iterable) { <END> List<TYPE> copy = FluentIterable.copyToList(iterable); this.iterable = copy; }</code><technical_language>Doubt: - I figure out reason for constructors protected. - rely incoming iterable's iterator for removal? I understand incoming iterable support remove(). Is this reason created a defensive copy in constructor</technical_language>

Target:
protected SimpleFluentIterable(Iterable<TYPE> iterable) { this.iterable = iterable; }
Source:
<code><START> private PendingWorkerNums notifyAllocatedWorkerRegistered(ResourceID resourceID) { <END> WorkerResourceSpec workerResourceSpec = allocatedNotRegisteredWorkerResourceSpecs.remove(resourceID); if (workerResourceSpec == null) { return PendingWorkerNums.ZERO; } return new PendingWorkerNums( requestedNotAllocatedWorkerCounter.getNum(workerResourceSpec), requestedNotRegisteredWorkerCounter.decreaseAndGet(workerResourceSpec)); }</code><technical_language>I this methods need return something. get rid of PendingWorkerNums.ZERO a correct return value</technical_language>

Target:
private void notifyAllocatedWorkerRegistered(ResourceID resourceID) { WorkerResourceSpec workerResourceSpec = allocatedNotRegisteredWorkerResourceSpecs.remove(resourceID); if (workerResourceSpec == null) { return; } requestedNotRegisteredWorkerCounter.decreaseAndGet(workerResourceSpec); }
Source:
<code>public BeanArchiveBuilder handle(String path) { BeanArchiveBuilder builder = super.handle(path); if (builder == null) { return null; } for (String className : builder.getClasses()) { addToIndex(servletContext <START> .getResourceAsStream(WebAppBeanArchiveScanner.WEB_INF_CLASSES_FILE_PATH + File.separator + className.replace(DOT, SLASH) + Files.CLASS_FILE_EXTENSION)); <END> } builder.setAttribute(JandexDiscoveryStrategy.INDEX_ATTRIBUTE_NAME, buildIndex()); return builder; }</code><technical_language>I a approach add new protected method ServletContextBeanArchiveHandler, e.g. void add(String path) put "adding" logic there. JandexServletContextBeanArchiveHandler override this method. JandexFileSystemBeanArchiveHandler.add(Entry, BeanArchiveBuilder). do think</technical_language>

Target:
public BeanArchiveBuilder handle(String path) { BeanArchiveBuilder builder = super.handle(path); if (builder == null) { return null; } builder.setAttribute(Jandex.INDEX_ATTRIBUTE_NAME, buildIndex()); return builder; }
Source:
<code>public void testFunctionInvocationOnInterface() { SampleImplementation impl = new SampleImplementation(); Optional<String> description = getDescriptionForFunction(s -> impl.interfaceFunction(s)); assertTrue(description.isPresent()); if (JDK_VERSION.equals(JDK_1_8_0_72)) { <START> assertNameMatch("(impl,s) -> interfaceFunction(_)", "testFunctionInvocationOnInterface", CLASSNAME, description.get().toString()); <END> } else { assertNameMatch("s -> interfaceFunction(_)", "testFunctionInvocationOnInterface", CLASSNAME, description.get().toString()); } }</code><technical_language>This seam right. "s -> interfaceFunction(_)" acceptable "(impl,s) -> interfaceFunction(_)" suggests is a lambda shape takes arguments while takes argument. This confusing if locate in a source code. If is distinguish completely ditch left hand side for now. try improve in future versions. If is a significant number of cases for left hand side is reliable replace left hand side unsure "(?) ->". If is difficult detect unsure remove left hand side completely for revisit in future</technical_language>

Target:
public void testFunctionInvocationOnInterface() { SampleImplementation impl = new SampleImplementation(); Optional<String> description = getDescriptionForFunction(s -> impl.interfaceFunction(s)); assertTrue(description.isPresent()); assertNameMatch("interfaceFunction(_)", "testFunctionInvocationOnInterface", CLASSNAME, description.get().toString()); }
Source:
<code>public void addInfoMessages() { mainPanel.clear(); if (!hasResults) { <START> infoLabel.setText("~ No results found ~"); <END> infoLabel.setStyleName(this.resources.filterDropdownStyle().noResultText()); } mainPanel.add(infoLabel); }</code><technical_language>this a constant something</technical_language>

Target:
public void addInfoMessages() { mainPanel.clear(); if (!hasResults) { infoLabel.setText(NO_RESULT_TEXT); infoLabel.setStyleName(this.resources.filterDropdownStyle().noResultText()); } mainPanel.add(infoLabel); }
Source:
<code>public PasswordPolicy setResetEmailStatus(EmailStatus status) { <START> setProperty(RESET_EMAIL_STATUS, status.name()); <END> return this; }</code><technical_language>this method is missing assert for case "status" object is null</technical_language>

Target:
public PasswordPolicy setResetEmailStatus(EmailStatus status) { Assert.notNull(status, "status cannot be null."); setProperty(RESET_EMAIL_STATUS, status.name()); return this; }
Source:
<code>public void nettyIsDisabled() { final Logger logger = LoggerFactory.getLogger("io.netty"); final Logger logger2 = LoggerFactory.getLogger("io.netty.channel"); <START> Assert.assertFalse(logger.isErrorEnabled()); <END> Assert.assertFalse(logger2.isErrorEnabled()); }</code><technical_language>assertj for assertion, please this instead: import static org.assertj.core.api.Assertions.assertThat; assertThat(logger.isErrorEnabled()).isTrue()</technical_language>

Target:
public void nettyIsDisabled() { final Logger logger = LoggerFactory.getLogger("io.netty"); final Logger logger2 = LoggerFactory.getLogger("io.netty.channel"); assertThat(logger.isErrorEnabled()).isFalse(); assertThat(logger2.isErrorEnabled()).isFalse(); }
Source:
<code>public static String obtainComponentName(NamedComponent component) { String name = component.getComponentName(); <START> if (name.startsWith('_' + IntegrationConfigUtils.BASE_PACKAGE)) { <END> name = name.substring(('_' + IntegrationConfigUtils.BASE_PACKAGE).length() + 1); } return name; }</code><technical_language>This more efficient if (name.charAt(0) == '_' && name.startsWith(.... a big deal JMX building graph more initialization</technical_language>

Target:
public static String obtainComponentName(NamedComponent component) { String name = component.getComponentName(); if (name.charAt(0) == '_' && name.startsWith(INTERNAL_COMPONENT_PREFIX)) { name = name.substring((INTERNAL_COMPONENT_PREFIX).length() + 1); } return name; }
Source:
<code>public boolean addAll(Collection<? extends ValueType> toAdd) { try (final Handle handle = rdbi.open()) { for (final ValueType value : toAdd) { final String valueAsString = serializationHelper.encode(value); <START> handle.jedis().rpush(key, valueAsString); <END> if (size() > maxSize) { handle.jedis().lpop(key); } } } return true; }</code><technical_language>ok a couple of problems need address forward. 1. double-checking-out a jedis connection. opened a rdbi handle here, do in size(). this double-checkout is a great recipe run a connection pool dry and/or deadlocks. 2. a concurrency bug here. other users of this structure pushing _during_ invocation of this method, leading incorrect behavior. resolve checking out some of in RDBI LUA code. check out RedisSemaphoreDAO (and is used,) for example</technical_language>

Target:
public boolean addAll(Collection<? extends ValueType> toAdd) { for (final ValueType value : toAdd) { try (Handle handle = rdbi.open()) { String valueStr = serializationHelper.encode(value); final int newSize = handle.attach(RedisCircularBufferDAO.class).add(key, valueStr, maxSize); boolean success = newSize >= 0 && newSize <= maxSize; if (!success) { return false; } } } return true; }
Source:
<code>public static Community[] findAll(Context context) throws SQLException { TableRowIterator tri = null; try { String query = "SELECT c.* FROM community c " + "LEFT JOIN metadatavalue m on (m.resource_id = c.community_id and m.resource_type_id = ? and m.metadata_field_id = ?) "; <START> if("oracle".equals(ConfigurationManager.getProperty("db.name"))){ <END> query += " ORDER BY cast(m.text_value as varchar2(128))"; }else{ query += " ORDER BY m.text_value"; } tri = DatabaseManager.query(context, query, Constants.COMMUNITY, MetadataField.findByElement(context, MetadataSchema.find(context, MetadataSchema.DC_SCHEMA).getSchemaID(), "title", null).getFieldID() ); } catch (SQLException e) { log.error("Find all Communities - ",e); throw e; } List<Community> communities = new ArrayList<Community>(); try { while (tri.hasNext()) { TableRow row = tri.next(); Community fromCache = (Community) context.fromCache( Community.class, row.getIntColumn("community_id")); if (fromCache != null) { communities.add(fromCache); } else { communities.add(new Community(context, row)); } } } finally { if (tri != null) { tri.close(); } } Community[] communityArray = new Community[communities.size()]; communityArray = (Community[]) communities.toArray(communityArray); return communityArray; }</code><technical_language>DatabaseManager.isOracle()</technical_language>

Target:
public static Community[] findAll(Context context) throws SQLException { TableRowIterator tri = null; try { String query = "SELECT c.* FROM community c " + "LEFT JOIN metadatavalue m on (m.resource_id = c.community_id and m.resource_type_id = ? and m.metadata_field_id = ?) "; if(DatabaseManager.isOracle()){ query += " ORDER BY cast(m.text_value as varchar2(128))"; }else{ query += " ORDER BY m.text_value"; } tri = DatabaseManager.query(context, query, Constants.COMMUNITY, MetadataField.findByElement(context, MetadataSchema.find(context, MetadataSchema.DC_SCHEMA).getSchemaID(), "title", null).getFieldID() ); } catch (SQLException e) { log.error("Find all Communities - ",e); throw e; } List<Community> communities = new ArrayList<Community>(); try { while (tri.hasNext()) { TableRow row = tri.next(); Community fromCache = (Community) context.fromCache( Community.class, row.getIntColumn("community_id")); if (fromCache != null) { communities.add(fromCache); } else { communities.add(new Community(context, row)); } } } finally { if (tri != null) { tri.close(); } } Community[] communityArray = new Community[communities.size()]; communityArray = (Community[]) communities.toArray(communityArray); return communityArray; }
Source:
<code>public void close(){ <START> this.shutdown(); <END> }</code><technical_language>Shutdown method returns Future, respect AutoCloseable call follows: this.shutdown().get();</technical_language>

Target:
public void close() throws ExecutionException, InterruptedException { this.shutdown().get(); }
Source:
<code><START> public synchronized Sha256Hash hashForSignatureWitness( <END> int inputIndex, Script scriptCode, Coin prevValue, SigHash type, boolean anyoneCanPay) { byte[] connectedScript = scriptCode.getProgram(); return hashForSignatureWitness(inputIndex, connectedScript, prevValue, type, anyoneCanPay); }</code><technical_language>This parameter ignored, is overridden in line 1284. Is this a bug? I wrong</technical_language>

Target:
public synchronized Sha256Hash hashForSignatureWitness( int inputIndex, Script scriptCode, Coin prevValue, SigHash type, boolean anyoneCanPay) { return hashForSignatureWitness(inputIndex, scriptCode.getProgram(), prevValue, type, anyoneCanPay); }
Source:
<code>private void addPreferences() { mFooterCategory = (PreferenceCategory) findPreference(PREF_KEY_FOOTER_CATEGORY); mOffMessagePreference = new FooterPreference(mFooterCategory.getContext()); final CharSequence title = getText(R.string.adb_wireless_list_empty_off); mOffMessagePreference.setTitle(title); mFooterCategory.addPreference(mOffMessagePreference); <START> final CharSequence deviceNameTitle = getText(R.string.my_device_info_device_name_preference_title); <END> }</code><technical_language>Remove, need it</technical_language>

Target:
private void addPreferences() { mDeviceNamePreference = (Preference) findPreference(PREF_KEY_ADB_DEVICE_NAME); mPairingMethodsCategory = (PreferenceCategory) findPreference(PREF_KEY_PAIRING_METHODS_CATEGORY); mCodePairingPreference = (Preference) findPreference(PREF_KEY_ADB_CODE_PAIRING); mCodePairingPreference.setOnPreferenceClickListener(preference -> { launchDevicePairingFragment(); return true; }); mQrcodePairingPreference = (Preference) findPreference(PREF_KEY_ADB_QRCODE_PAIRING); mQrcodePairingPreference.setOnPreferenceClickListener(preference -> { launchQrcodeScannerFragment(); return true; }); mPairedDevicesCategory = (PreferenceCategory) findPreference(PREF_KEY_PAIRED_DEVICES_CATEGORY); mFooterCategory = (PreferenceCategory) findPreference(PREF_KEY_FOOTER_CATEGORY); mOffMessagePreference = new FooterPreference(mFooterCategory.getContext()); final CharSequence title = getText(R.string.adb_wireless_list_empty_off); mOffMessagePreference.setTitle(title); mFooterCategory.addPreference(mOffMessagePreference); }
Source:
<code>MissingFieldStreamReader(Type type) { <START> requireNonNull(type, "type is required"); <END> this.outputType = type; }</code><technical_language>combine assignment this.outputType = requireNonNull(type, "type is required");</technical_language>

Target:
MissingFieldStreamReader(Type type) { this.outputType = requireNonNull(type, "type is required"); }
Source:
<code>protected void parseInterfaceCriteria(final XMLExtendedStreamReader reader, final Namespace expectedNs, final ModelNode interfaceModel) throws XMLStreamException { if (reader.nextTag() == END_ELEMENT) { return; } requireNamespace(reader, expectedNs); Element element = Element.forName(reader.getLocalName()); switch (element) { case ANY_ADDRESS: case ANY_IPV4_ADDRESS: case ANY_IPV6_ADDRESS: { <START> interfaceModel.get(Element.ANY_ADDRESS.getLocalName()).set(true); <END> requireNoContent(reader); requireNoContent(reader); return; } } do { requireNamespace(reader, expectedNs); element = Element.forName(reader.getLocalName()); switch (element) { case ANY: parseCompoundInterfaceCriterion(reader, expectedNs, interfaceModel.get(ANY).setEmptyObject()); break; case NOT: parseCompoundInterfaceCriterion(reader, expectedNs, interfaceModel.get(NOT).setEmptyObject()); break; default: { parseSimpleInterfaceCriterion(reader, interfaceModel, false); break; } } } while (reader.nextTag() != END_ELEMENT); }</code><technical_language>I do ANY_IPV4_ADDRESS ANY_IPV6_ADDRESS: 1) If namespace is >= DOMAIN_3_0 throw ParseUtils.unexpectedElement. 2) else, earlier xsd declares legal, support them. throw XmlStreamException a new custom message explaining longer supported any-address</technical_language>

Target:
protected void parseInterfaceCriteria(final XMLExtendedStreamReader reader, final Namespace expectedNs, final ModelNode interfaceModel) throws XMLStreamException { if (reader.nextTag() == END_ELEMENT) { return; } requireNamespace(reader, expectedNs); Element element = Element.forName(reader.getLocalName()); switch (element) { case ANY_IPV4_ADDRESS: case ANY_IPV6_ADDRESS: { if(expectedNs.getMajorVersion() >= 3) { throw ParseUtils.unexpectedElement(reader); } else { throw ParseUtils.unsupportedElement(reader, Element.ANY_ADDRESS.getLocalName()); } } case ANY_ADDRESS: { interfaceModel.get(Element.ANY_ADDRESS.getLocalName()).set(true); requireNoContent(reader); requireNoContent(reader); return; } } do { requireNamespace(reader, expectedNs); element = Element.forName(reader.getLocalName()); switch (element) { case ANY: parseCompoundInterfaceCriterion(reader, expectedNs, interfaceModel.get(ANY).setEmptyObject()); break; case NOT: parseCompoundInterfaceCriterion(reader, expectedNs, interfaceModel.get(NOT).setEmptyObject()); break; default: { parseSimpleInterfaceCriterion(reader, interfaceModel, false); break; } } } while (reader.nextTag() != END_ELEMENT); }
Source:
<code>private Response getConfigurationHelper(String environmentId, String feedId, String resourcePath, DataEntity.Role dataType) { String tenantId = getTenantId(); CanonicalPath resource = composeCanonicalPath(tenantId, environmentId, feedId, resourcePath); Resources.Single resourceSingle = inventory.inspect(resource, Resources.Single.class); DataEntity entity; switch (dataType) { case configuration: <START> entity = resourceSingle.configuration().get(null).entity(); <END> break; case connectionConfiguration: entity = resourceSingle.connectionConfiguration().get(null).entity(); break; default: throw new IllegalArgumentException("Unsupported data type: " + dataType.toString()); } return Response.ok(entity).build(); }</code><technical_language>+1 this is a good start expand later. backend support is kinda a matter of changing REST API</technical_language>

Target:
private Response getConfigurationHelper(String environmentId, String feedId, String resourcePath, DataEntity.Role dataType) { String tenantId = getTenantId(); CanonicalPath resource = composeCanonicalPath(tenantId, environmentId, feedId, resourcePath); DataEntity data = inventory.inspect(resource, Resources.Single.class).data().get(dataType).entity(); return Response.ok(data).build(); }
Source:
<code>public static boolean isExpired(String key){ AttestationValue value = getValueFromCache(key); if (value != null && value.getVtime() != null){ if (System.currentTimeMillis() - value.getVtime().getTime() <= cacheTimeout) <START> return false; <END> } return true; }</code><technical_language>Please add braces</technical_language>

Target:
public boolean isExpired(String key) { AttestationValue value = attestationValues.get(key); if (value != null && value.getVtime() != null) { if (System.currentTimeMillis() - value.getVtime().getTime() <= cacheTimeout) { return false; } } return true; }
Source:
<code>public boolean overlaps(long start, long end) { if (end >= start) <START> return overlaps(start) && overlaps(end); <END> return false; }</code><technical_language>logic of this method return true if part of range is covered part of Block's revision range. Currently returns true if entire range is covered Block's range</technical_language>

Target:
public boolean overlaps(long start, long end) { final Long min = stats.get(Attribute.MIN_REVISION_VERSION); final Long max = stats.get(Attribute.MAX_REVISION_VERSION); if(end >= start && min != null && max != null) { final LongRange range = new LongRange(start, end); for (long i = min; i <= max; i++) { if(range.containsLong(i)) { return true; } } } return false; }
Source:
<code>private NuageVspPluginClientLoader(String nuagePluginClientJarLocation) { try { _loader = URLClassLoader.newInstance(new URL[] {new URL("jar:file:" + nuagePluginClientJarLocation + "!/")}, getClass().getClassLoader()); } catch (MalformedURLException e) { <START> e.printStackTrace(); <END> } }</code><technical_language>printStackTrace() emits stdout result in this stack trace lost due redirection. Log exception instead a message explaining operation performed. Also, exception swallowed IllegalStateException thrown</technical_language>

Target:
private NuageVspPluginClientLoader(String nuagePluginClientJarLocation) { try { _loader = URLClassLoader.newInstance(new URL[] {new URL("jar:file:" + nuagePluginClientJarLocation + "!/")}, getClass().getClassLoader()); } catch (MalformedURLException e) { throw new IllegalArgumentException(e); } }