auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public LiveData<List<Allergy>> getAllergies(RestApi restApi, String uuid) { MutableLiveData<List<Allergy>> allergyLiveData = new MutableLiveData<>(); allergyEntitiesOffline = allergyRoomDAO.getAllAllergiesByPatientID(patientID); if (NetworkUtils.isOnline()) { restApi.getAllergies(uuid).enqueue(new Callback<Results<Allergy>>() { @Override public void onResponse(@NotNull Call<Results<Allergy>> call, @NotNull Response<Results<Allergy>> response) { if (response.isSuccessful()) { allergyRoomDAO.deleteAllPatientAllergy(patientID); allergyList = response.body().getResults(); for (Allergy allergy : allergyList) { allergyRoomDAO.saveAllergy(AppDatabaseHelper.allergyToEntity(allergy, patientID)); } allergyLiveData.setValue(allergyList); } else { ToastUtil.error(OpenMRS.getInstance().getString(R.string.unable_to_fetch_allergies)); allergyLiveData.setValue(entityListToAllergy(allergyEntitiesOffline)); } } @Override public void onFailure(@NotNull Call<Results<Allergy>> call, @NotNull Throwable t) { <START> ToastUtil.error(OpenMRS.getInstance().getString(R.string.unable_to_fetch_allergies)); <END> allergyLiveData.setValue(entityListToAllergy(allergyEntitiesOffline)); } }); } else { allergyLiveData.setValue(entityListToAllergy(allergyEntitiesOffline)); } return allergyLiveData; }</code><technical_language>@rishabh-997 prompt user values updated synced device online</technical_language>

Target:
public LiveData<List<Allergy>> getAllergies(RestApi restApi, String uuid) { MutableLiveData<List<Allergy>> allergyLiveData = new MutableLiveData<>(); allergyEntitiesOffline = allergyRoomDAO.getAllAllergiesByPatientID(patientID); allergyLiveData.setValue(allergyEntityListToAllergyList(allergyEntitiesOffline)); if (NetworkUtils.isOnline()) { restApi.getAllergies(uuid).enqueue(new Callback<Results<Allergy>>() { @Override public void onResponse(@NotNull Call<Results<Allergy>> call, @NotNull Response<Results<Allergy>> response) { if (response.isSuccessful()) { allergyRoomDAO.deleteAllPatientAllergy(patientID); allergyList = response.body().getResults(); for (Allergy allergy : allergyList) { allergyRoomDAO.saveAllergy(AppDatabaseHelper.allergyToEntity(allergy, patientID)); } allergyLiveData.setValue(allergyList); } else { ToastUtil.error(OpenMRS.getInstance().getString(R.string.unable_to_fetch_allergies)); } } @Override public void onFailure(@NotNull Call<Results<Allergy>> call, @NotNull Throwable t) { ToastUtil.error(OpenMRS.getInstance().getString(R.string.unable_to_fetch_allergies)); } }); } return allergyLiveData; }
Source:
<code>private static void cacheUser(User user) { <START> if (!map.containsKey(user.getUsername())) map.put(user.getUsername(), user); <END> }</code><technical_language>changed map.putIfAbsent</technical_language>

Target:
private static void cacheUser(User user) { map.putIfAbsent(user.getUsername(), user); }
Source:
<code>public ImageUploadTinyMCEPage() { TinyMCESettings settings = new TinyMCESettings(TinyMCESettings.Theme.modern); settings.addToolbar(new Toolbar( "toolbar", "insertfile undo redo | styleselect | bold italic | image | alignleft aligncenter alignright alignjustify | bullist numlist outdent indent | imageupload")); settings.addPlugins("imageupload"); settings.addPlugins("image"); <START> TextArea<String> textArea = new TextArea("ta", new Model<String>(TEXT)); <END> textArea.add(new TinyMceBehavior(settings)); add(textArea); add(new ImageUploadPanel("uploadPanel")); }</code><technical_language>Dyamond operator again..</technical_language>

Target:
public ImageUploadTinyMCEPage() { TinyMCESettings settings = new TinyMCESettings(TinyMCESettings.Theme.modern); settings.addToolbar(new Toolbar( "toolbar", "insertfile undo redo | styleselect | bold italic | image | alignleft aligncenter alignright alignjustify | bullist numlist outdent indent | imageupload")); settings.addPlugins("imageupload"); settings.addPlugins("image"); TextArea<String> textArea = new TextArea<>("ta", new Model<String>(TEXT)); textArea.add(new TinyMceBehavior(settings)); add(textArea); add(new ImageUploadPanel("uploadPanel")); }
Source:
<code>protected List<MessageDescriptor> getWarnings(final ResponseEntity<?> authenticationResponse) { <START> List<MessageDescriptor> messageDescriptors = new ArrayList<>(); <END> var passwordExpirationDate = authenticationResponse.getHeaders().getFirstZonedDateTime("X-Cas-PasswordExpirationDate"); if (passwordExpirationDate != null) { var days = Duration.between(Instant.now(), passwordExpirationDate).toDays(); messageDescriptors.add(new PasswordExpiringWarningMessageDescriptor(null, days)); } List<String> warnings = authenticationResponse.getHeaders().get("X-Cas-Warning"); if (warnings != null) { warnings.stream() .map(warning -> new DefaultMessageDescriptor(warning, warning, null)) .forEach(messageDescriptors::add); } return messageDescriptors; }</code><technical_language>val messageDescriptors = new ArrayList<>();</technical_language>

Target:
protected List<MessageDescriptor> getWarnings(final ResponseEntity<?> authenticationResponse) { val messageDescriptors = new ArrayList<MessageDescriptor>(); val passwordExpirationDate = authenticationResponse.getHeaders().getFirstZonedDateTime("X-CAS-PasswordExpirationDate"); if (passwordExpirationDate != null) { val days = Duration.between(Instant.now(), passwordExpirationDate).toDays(); messageDescriptors.add(new PasswordExpiringWarningMessageDescriptor(null, days)); } val warnings = authenticationResponse.getHeaders().get("X-CAS-Warning"); if (warnings != null) { warnings.stream() .map(warning -> new DefaultMessageDescriptor(warning, warning, null)) .forEach(messageDescriptors::add); } return messageDescriptors; }
Source:
<code>private void analyse(SensorContext sensorContext, InputFile inputFile, ProductDependentExecutor executor, List<TreeVisitor> visitors) { ActionParser<Tree> currentParser = this.parser; if (inputFile.filename().endsWith(".vue")) { currentParser = this.vueParser; } ScriptTree scriptTree; try { scriptTree = (ScriptTree) currentParser.parse(inputFile.contents()); scanFile(sensorContext, inputFile, executor, visitors, scriptTree); } catch (RecognitionException e) { checkInterrupted(e); <START> LOG.error("Unable to parse file: " + inputFile.toString()); <END> LOG.error(e.getMessage()); processRecognitionException(e, sensorContext, inputFile); } catch (Exception e) { checkInterrupted(e); processException(e, sensorContext, inputFile); LOG.error("Unable to analyse file: " + inputFile.toString(), e); } }</code><technical_language>@vilchik-elena I inputFile.uri(). more stable toString is absolute, before, current implementation of DefaultInputFile.toString() is project-relative path. goal project-relative path, in case InputFile.getProjectRelativePath()</technical_language>

Target:
private void analyse(SensorContext sensorContext, InputFile inputFile, ProductDependentExecutor executor, List<TreeVisitor> visitors) { ActionParser<Tree> currentParser = this.parser; if (inputFile.filename().endsWith(".vue")) { currentParser = this.vueParser; } ScriptTree scriptTree; try { scriptTree = (ScriptTree) currentParser.parse(inputFile.contents()); scanFile(sensorContext, inputFile, executor, visitors, scriptTree); } catch (RecognitionException e) { checkInterrupted(e); LOG.error("Unable to parse file: " + inputFile.uri()); LOG.error(e.getMessage()); processRecognitionException(e, sensorContext, inputFile); } catch (Exception e) { checkInterrupted(e); processException(e, sensorContext, inputFile); LOG.error("Unable to analyse file: " + inputFile.uri(), e); } }
Source:
<code>public static void openProductEditor(IProductModel productModel) { IResource resource = productModel.getUnderlyingResource(); try { IEditorInput input = null; if (resource != null) { input = new FileEditorInput((IFile) resource); } else { <START> File file = new File(productModel.getInstallLocation(), ICoreConstants.FEATURE_FILENAME_DESCRIPTOR); <END> IFileStore store = EFS.getStore(file.toURI()); input = new FileStoreEditorInput(store); } IDE.openEditor(PDEPlugin.getActivePage(), input, IPDEUIConstants.PRODUCT_EDITOR_ID, true); } catch (PartInitException e) { PDEPlugin.logException(e); } catch (CoreException e) { PDEPlugin.logException(e); } }</code><technical_language>I guess that'a a copy-paste mistake. product editor supports local resources AFAIK</technical_language>

Target:
public static void openProductEditor(IProductModel productModel) { IResource resource = productModel.getUnderlyingResource(); try { IEditorInput input = null; if (resource != null) { input = new FileEditorInput((IFile) resource); IDE.openEditor(PDEPlugin.getActivePage(), input, IPDEUIConstants.PRODUCT_EDITOR_ID, true); } } catch (PartInitException e) { PDEPlugin.logException(e); } }
Source:
<code>public boolean remove(K key) { if (service.isRunning()) { String target = buildUrl(service.cacheName, key); Invocation delete = service.getHttpClient().target(target).request().accept(service.getContentType()) .buildDelete(); Response response = delete.invoke(); int status = response.getStatus(); response.close(); <START> if (status != Status.OK.getStatusCode()) { <END> if (status == Status.NOT_FOUND.getStatusCode()) { log.warn("RESTEasyHTTPOperations.remove::Key: " + key + " does not exist in cache: " + service.cacheName); } else { throw new RuntimeException("RESTEasyHTTPOperations.remove::Unexpected HttpStatus: " + status + " " + response.getStatusInfo().getReasonPhrase() + " for request " + target); } } return true; } return false; }</code><technical_language>do log a warning? method return true 204. do a switch (status) { 200 => true, 204 => false, other => exception }</technical_language>

Target:
public boolean remove(K key) { return doDelete(buildUrl(service.cacheName, key), null); }
Source:
<code>private ParseResult parseFlex20Position(DeviceSession deviceSession, ByteBuf buf) { Position position = new Position(getProtocolName()); position.setDeviceId(deviceSession.getDeviceId()); long index = 0; int length = buf.readUnsignedShort(); if (length <= buf.readableBytes() && buf.readUnsignedByte() == 0x0A) { buf.skipBytes(1); index = buf.readUnsignedIntLE(); position.set(Position.KEY_INDEX, index); position.set(Position.KEY_EVENT, buf.readUnsignedShortLE()); <START> buf.skipBytes(4); END> int navSensorState = buf.readUnsignedByte(); position.setValid(BitUtil.check(navSensorState, 1)); position.set(Position.KEY_SATELLITES, navSensorState >> 2); position.setTime(new DateBuilder(new Date(buf.readUnsignedIntLE() * 1000)).getDate()); position.setLatitude(buf.readIntLE() / 600000.0); position.setLongitude(buf.readIntLE() / 600000.0); position.setAltitude(buf.readIntLE() * 0.1); position.setSpeed(UnitsConverter.knotsFromKph(buf.readFloatLE())); position.setCourse(buf.readUnsignedShortLE()); position.set(Position.KEY_ODOMETER, buf.readFloatLE() * 1000); buf.skipBytes(length - buf.readerIndex() - 1); } return new ParseResult(index, position); }</code><technical_language>For fields skipping readXxx readUnsignedXxx. Please avoid skipBytes array something</technical_language>

Target:
private ParseResult parseFlex20Position(DeviceSession deviceSession, ByteBuf buf) { Position position = new Position(getProtocolName()); position.setDeviceId(deviceSession.getDeviceId()); long index = 0; int length = buf.readUnsignedShort(); if (length <= buf.readableBytes() && buf.readUnsignedByte() == 0x0A) { buf.readUnsignedByte(); index = buf.readUnsignedIntLE(); position.set(Position.KEY_INDEX, index); position.set(Position.KEY_EVENT, buf.readUnsignedShortLE()); buf.readUnsignedInt(); int navSensorState = buf.readUnsignedByte(); position.setValid(BitUtil.check(navSensorState, 1)); position.set(Position.KEY_SATELLITES, BitUtil.from(navSensorState, 2)); position.setTime(new DateBuilder(new Date(buf.readUnsignedIntLE() * 1000)).getDate()); position.setLatitude(buf.readIntLE() / 600000.0); position.setLongitude(buf.readIntLE() / 600000.0); position.setAltitude(buf.readIntLE() * 0.1); position.setSpeed(UnitsConverter.knotsFromKph(buf.readFloatLE())); position.setCourse(buf.readUnsignedShortLE()); position.set(Position.KEY_ODOMETER, buf.readFloatLE() * 1000); buf.skipBytes(length - buf.readerIndex() - 1); } return new ParseResult(index, position); }
Source:
<code>public void setNonNullParameter(PreparedStatement ps, int i, Object parameter, JdbcType jdbcType) throws SQLException { Array array = null; if (parameter instanceof Array) { array = (Array)parameter; } else { if (!parameter.getClass().isArray()) { throw new TypeException("ArrayType Handler requires SQL array or java array parameter and does not support type " + parameter.getClass()); } Object[] values = (Object[])parameter; <START> array = ps.getConnection().createArrayOf(jdbcType.name(), values); <END> } ps.setArray(i, array); }</code><technical_language>'free' this array calling setArray() (the array is passed parameter freed user)</technical_language>

Target:
public void setNonNullParameter(PreparedStatement ps, int i, Object parameter, JdbcType jdbcType) throws SQLException { if (parameter instanceof Array) { ps.setArray(i, (Array) parameter); } else { if (!parameter.getClass().isArray()) { throw new TypeException( "ArrayType Handler requires SQL array or java array parameter and does not support type " + parameter.getClass()); } Class<?> componentType = parameter.getClass().getComponentType(); String arrayTypeName = resolveTypeName(componentType); Array array = ps.getConnection().createArrayOf(arrayTypeName, (Object[]) parameter); ps.setArray(i, array); array.free(); } }
Source:
<code>T getNextHandle() { try { while (jsonParser.nextToken() != null) { JsonToken currentToken = jsonParser.currentToken(); switch (currentToken) { case FIELD_NAME: key.pop(); key.push(jsonParser.getCurrentName()); break; case START_OBJECT: NodeOperation operation = visitor.startObject(key.peek()); switch (operation) { case DESCEND: <START> key.push(currentToken.asString()); <END> break; case PROCESS: T handle = visitor.makeBufferedHandle(new JSONSplitter.JsonContainerParser(jsonParser)); if (handle != null) { return handle; } break; case SKIP: jsonParser.skipChildren(); break; default: throw new IllegalStateException("Unknown state"); } break; case END_OBJECT: visitor.endObject(key.peek()); key.pop(); break; case START_ARRAY: NodeOperation operation_array = visitor.startArray(key.peek()); switch (operation_array) { case DESCEND: break; case PROCESS: T handle = visitor.makeBufferedHandle(new JSONSplitter.JsonContainerParser(jsonParser)); if (handle != null) { return handle; } break; case SKIP: jsonParser.skipChildren(); break; default: throw new IllegalStateException("Unknown state"); } break; case END_ARRAY: visitor.endArray(key.peek()); break; } } return null; } catch (IOException e) { throw new RuntimeException("Failed to traverse document", e); } }</code><technical_language>START_OBJECT case push a string moving line 351 line 348, END_OBJECT case pops a key</technical_language>

Target:
T getNextHandle() { try { while (jsonParser.nextToken() != null) { JsonToken currentToken = jsonParser.currentToken(); switch (currentToken) { case FIELD_NAME: key.pop(); key.push(jsonParser.getCurrentName()); break; case START_OBJECT: NodeOperation operation = visitor.startObject(key.peek()); switch (operation) { case DESCEND: END case only as the parser won't hit END_OBJECT in case PROCESS and case SKIP. key.push(currentToken.asString()); break; case PROCESS: T handle = visitor.makeBufferedHandle(new JSONSplitter.JsonContainerParser(jsonParser)); if (handle != null) { return handle; } break; case SKIP: jsonParser.skipChildren(); break; default: throw new IllegalStateException("Unknown state"); } break; case END_OBJECT: visitor.endObject(key.peek()); key.pop(); break; case START_ARRAY: NodeOperation operation_array = visitor.startArray(key.peek()); switch (operation_array) { case DESCEND: break; case PROCESS: T handle = visitor.makeBufferedHandle(new JSONSplitter.JsonContainerParser(jsonParser)); if (handle != null) { return handle; } break; case SKIP: jsonParser.skipChildren(); break; default: throw new IllegalStateException("Unknown state"); } break; case END_ARRAY: visitor.endArray(key.peek()); break; } } return null; } catch (IOException e) { throw new RuntimeException("Failed to traverse document", e); } }
Source:
<code>public GlowItem dropItemNaturally(Location location, ItemStack item) { ThreadLocalRandom tlr = ThreadLocalRandom.current(); double radius = 0.1; double offsetX = tlr.nextDouble(radius * 2) - radius; double offsetY = 0.15; double offsetZ = Math.sqrt(Math.pow(radius, 2) - Math.pow(offsetX, 2)); <START> if (tlr.nextInt(2) == 0) { <END> offsetZ *= -1; } location.add(0.5, 0.5, 0.5); GlowItem dropItem = dropItem(location, item); dropItem.setVelocity(new Vector(offsetX, offsetY, offsetZ)); return dropItem; }</code><technical_language>nextBoolean instead</technical_language>

Target:
public GlowItem dropItemNaturally(Location location, ItemStack item) { ThreadLocalRandom tlr = ThreadLocalRandom.current(); double radius = 0.1; double offsetX = tlr.nextDouble(radius * 2) - radius; double offsetY = 0.15; double offsetZ = Math.sqrt(Math.pow(radius, 2) - Math.pow(offsetX, 2)); if (tlr.nextBoolean()) { offsetZ *= -1; } location.add(0.5, 0.5, 0.5); GlowItem dropItem = dropItem(location, item); dropItem.setVelocity(new Vector(offsetX, offsetY, offsetZ)); return dropItem; }
Source:
<code>protected void executeCommand() { try { VDSReturnValue vdsReturnValue = Backend .getInstance() .getResourceManager() .RunVdsCommand( VDSCommandType.DeleteImageGroup, new DeleteImageGroupVDSCommandParameters(getParameters().getStoragePoolId(), getParameters() .getStorageDomainId(), getParameters().getImageGroupID(), getParameters() .getWipeAfterDelete(), false, getStoragePool().getcompatibility_version().toString())); if (vdsReturnValue.getSucceeded()) { getReturnValue().getInternalTaskIdList().add( createTask(vdsReturnValue.getCreationInfo(), VdcActionType.RemoveVmTemplate, VdcObjectType.Storage, getParameters().getStorageDomainId())); <START> setSucceeded(true); <END> } } catch (VdcBLLException e) { if (e.getErrorCode() == VdcBllErrors.ImageDoesNotExistInDomainError) { log.errorFormat("Image {0} does not exists in domain {1}. Removing disk from DB only.", getDiskImage().getId(), getParameters().getStorageDomainId()); } else { throw e; } } }</code><technical_language>this set true in case of ImageDoesNotExistInDomainError no? in RemoveImage</technical_language>

Target:
protected void executeCommand() { try { VDSReturnValue vdsReturnValue = Backend .getInstance() .getResourceManager() .RunVdsCommand( VDSCommandType.DeleteImageGroup, new DeleteImageGroupVDSCommandParameters(getParameters().getStoragePoolId(), getParameters() .getStorageDomainId(), getParameters().getImageGroupID(), getParameters() .getWipeAfterDelete(), false, getStoragePool().getcompatibility_version().toString())); if (vdsReturnValue.getSucceeded()) { getReturnValue().getInternalTaskIdList().add( createTask(vdsReturnValue.getCreationInfo(), VdcActionType.RemoveVmTemplate, VdcObjectType.Storage, getParameters().getStorageDomainId())); setSucceeded(true); } } catch (VdcBLLException e) { if (e.getErrorCode() == VdcBllErrors.ImageDoesNotExistInDomainError) { getReturnValue().setFault(new VdcFault(e, e.getErrorCode())); log.errorFormat("Image {0} does not exist in domain {1}.", getDiskImage().getId(), getParameters().getStorageDomainId()); setSucceeded(true); } else { throw e; } } }
Source:
<code>public static @Nullable ITmfEvent getPreviousEventMatching(ITmfTrace trace, long startRank, Predicate<ITmfEvent> predicate) { <START> final int targetStep = trace.getCacheSize(); <END> int step = (startRank < targetStep ? (int) startRank : targetStep); long currentRank = startRank; try { while (currentRank > 0) { currentRank = Math.max(currentRank - step, 0); EventMatchingRequest req = new EventMatchingRequest(currentRank, step, predicate, true); trace.sendRequest(req); req.waitForCompletion(); ITmfEvent event = req.getFoundEvent(); if (event != null) { return event; } step *= 2; } } catch (InterruptedException e) { return null; } return null; }</code><technical_language>mentioned in initial review, initial step tied cache size: if a trace a large cache size, if in cache, want a large number of events if event for is close starting one. Bounding upper limit of initial step cache size sense though</technical_language>

Target:
public static @Nullable ITmfEvent getPreviousEventMatching(ITmfTrace trace, long startRank, Predicate<ITmfEvent> predicate) { int step = trace.getCacheSize(); if (startRank < step) { step = (int) startRank; } long currentRank = startRank; try { while (currentRank > 0) { currentRank = Math.max(currentRank - step, 0); List<ITmfEvent> list = new ArrayList<>(step); ArrayFillingRequest req = new ArrayFillingRequest(currentRank, step, list); trace.sendRequest(req); req.waitForCompletion(); Optional<ITmfEvent> matchingEvent = Lists.reverse(list).stream() .filter(predicate) .findFirst(); if (matchingEvent.isPresent()) { return matchingEvent.get(); } } } catch (InterruptedException e) { return null; } return null; }
Source:
<code>public void testJsonToMapWithList() { Map<String, String> map = JsonUtils.jsonToStringMapWithList(new StringReader(JSON_STRING_WITH_ARRAYS)); <START> assertEquals("string value", "string", map.get("string")); <END> assertEquals("long value", "123", map.get("long")); assertEquals("double value", "123.45", map.get("double")); assertEquals("null value", null, map.get("null")); assertEquals("true value", "true", map.get("true")); assertEquals("false value", "false", map.get("false")); assertEquals("encoding", "Chloë", map.get("encoding")); assertEquals("array of simple types only reads string values", "[\"string\",\"123\",\"123.45\",null,\"true\",\"false\"]", map.get("array")); assertEquals("array of objects with only strings as keys and values is supported", "[{\"key1\":\"string1\"},{\"key2\":\"string2\"}]", map.get("array2")); assertEquals("array of objects with values other than strings is not supported", "[{\"key1\":\"123.45\"},{\"key2\":\"false\"}]", map.get("array3")); assertEquals("nested primitive arrays are not supported", "[\"string\",\"123\",\"123.45\",null,\"true\"]", map.get("array4")); assertEquals("nested object arrays are not supported", "[{\"key1\":\"string1\"},{},{\"key3\":\"string3\"}]", map.get("array5")); assertEquals("nested arrays are not supported", "[]", map.get("array6")); assertEquals("empty array read", "[]", map.get("array7")); assertEquals("array with only null read correctly", "[null]", map.get("array8")); assertEquals("nested arrays are not supported", "[\"A\",{},\"C\",\"D\",null]", map.get("array9")); assertNull("object is ignored", map.get("object")); }</code><technical_language>Instead of of individual assertFoo(...)s, I recommend a strategy like: 1. Prepare a Map input. 2. Prepare a JSON string, below, assert against. 3. a single assert call. assertEquals( new JSONObject() .put("logn value", map.get("...")) // .toString(), map );</technical_language>

Target:
public void testJsonToMapWithList() { Map<String, String> map = JsonUtils.jsonToStringMapWithList(new StringReader(JSON_STRING_WITH_ARRAYS)); Map<String, String> actualMap = new HashMap<>(); actualMap.put("string", "string"); actualMap.put("long", "123"); actualMap.put("double", "123.45"); actualMap.put("null", null); actualMap.put("true", "true"); actualMap.put("false", "false"); actualMap.put("encoding", "Chloë"); actualMap.put("array", "[\"string\",\"123\",\"123.45\",null,\"true\",\"false\"]"); actualMap.put("array2", "[{\"key1\":\"string1\"},{\"key2\":\"string2\"}]"); actualMap.put("array3", "[{\"key1\":\"123.45\"},{\"key2\":\"false\"}]"); actualMap.put("array4", "[\"string\",\"123\",\"123.45\",null,\"true\"]"); actualMap.put("array5", "[{\"key1\":\"string1\"},{},{\"key3\":\"string3\"}]"); actualMap.put("array6", "[]"); actualMap.put("array7", "[]"); actualMap.put("array8", "[null]"); actualMap.put("array9", "[\"A\",{},\"C\",\"D\",null]"); assertEquals(actualMap, map); }
Source:
<code>public CommittedTransactionException() { <START> super("Transaction must not be committed"); <END> }</code><technical_language>Clarify this means transaction committed vs. transaction committed</technical_language>

Target:
public CommittedTransactionException() { super("Transaction must be uncommitted"); }
Source:
<code><START> public String getInstanceURL() { <END> if (this.getCustomer() != null) { return String.format("%s/%s/cards/%s", classURL(Customer.class), this.getCustomer(), this.getId()); } else { return String.format("%s/%s/cards/%s", classURL(Recipient.class), this.getRecipient(), this.getId()); } }</code><technical_language>I realize smart enough do this before, ideally this method return null getCustomer() getRecipient() return null</technical_language>

Target:
public String getInstanceURL() { if (this.getCustomer() != null) { return String.format("%s/%s/cards/%s", classURL(Customer.class), this.getCustomer(), this.getId()); } else if (this.getRecipient() != null) { return String.format("%s/%s/cards/%s", classURL(Recipient.class), this.getRecipient(), this.getId()); } else { return null; } }
Source:
<code>public ASTNode visit(Term trm, Void _) throws ParseFailedException { <START> if (!context.isSubsortedEq(expectedSort, trm.getSort()) && <END> !(trm.getSort().equals(KSorts.K) && context.isSubsortedEq(KSorts.K, expectedSort))) { String msg = "Wrong type in cell '" + cellLabel + "'. Expected sort: " + expectedSort + " but found " + trm.getSort(); throw new ParseFailedException(new KException(ExceptionType.ERROR, KExceptionGroup.CRITICAL, msg, getName(), trm.getFilename(), trm.getLocation())); } return trm; }</code><technical_language>I need check kind well. if user puts in a cell contains a bag? error get</technical_language>

Target:
public ASTNode visit(Term trm, Void _) throws ParseFailedException { if (!context.isSubsortedEq(expectedSort, trm.getSort()) && !(context.isSubsortedEq(Sort.KLIST, expectedSort) && (trm.getSort().equals(Sort.KLIST) || trm.getSort().equals(Sort.K) || trm.getSort().equals(Sort.KITEM)))) { String msg = "Wrong type in cell '" + cellLabel + "'. Expected sort: " + expectedSort + " but found " + trm.getSort(); throw new ParseFailedException(new KException(ExceptionType.ERROR, KExceptionGroup.CRITICAL, msg, getName(), trm.getFilename(), trm.getLocation())); } return trm; }
Source:
<code>public KApp(Term label, Term child) { <START> super(label.getLocation(), label.getFilename(), KSorts.KITEM); <END> setLabel(label); setChild(child); }</code><technical_language>fight this, I time work new kast, location information completely redone</technical_language>

Target:
public KApp(Term label, Term child) { super(label.getLocation(), label.getFilename(), Sort.KITEM); setLabel(label); setChild(child); }
Source:
<code>public Response restart(Action action) { <START> validateParameters(action); <END> return performServiceAction(action, GlusterServiceActionType.restart); }</code><technical_language>rsdl_metadata.yaml validating fields: action.name: 'xs:string', action.service_type: 'xs:string'</technical_language>

Target:
public Response restart(Action action) { validateParameters(action); return performServiceAction(action, GlusterServiceActionType.RESTART); }
Source:
<code><START> protected void readBytes(final byte[] bytes) { <END> mIndexBuffer.readBytes(bytes); }</code><technical_language>Lets of newly exposed methods final</technical_language>

Target:
protected final void readBytes(final byte[] bytes) { mIndexBuffer.readBytes(bytes); }
Source:
<code>public void delete(Scene sceneToDelete) throws IOException { StorageOperations.deleteDir(sceneToDelete.getDirectory()); <START> File projectDirectory = ProjectManager.getInstance().getCurrentProject().getDirectory(); <END> DeviceVariableAccessor accessor = new DeviceVariableAccessor(projectDirectory); for (Sprite sprite: sceneToDelete.getSpriteList()) { try { accessor.deleteAllLocalVariables(sprite); } catch (IOException e) { Log.i(TAG, e.getMessage()); } } }</code><technical_language>instead of adding this removal procedure delete id a cleanup eg. project load</technical_language>

Target:
public void delete(Scene sceneToDelete) throws IOException { StorageOperations.deleteDir(sceneToDelete.getDirectory()); }
Source:
<code>boolean equalsArray(List<ConfigurationSetting> settings1, List<ConfigurationSetting> settings2) { if (settings1 == settings2) { return true; } if (settings1 == null || settings2 == null) { return false; } if (settings1.size() != settings2.size()) { return false; } final int size = settings1.size(); <START> for (int i = 0; i < size; i++) { <END> if (!equals(settings1.get(i), settings2.get(i))) { return false; } } return true; }</code><technical_language>if contents ordered in way? affect this equality</technical_language>

Target:
static boolean equalsArray(List<ConfigurationSetting> settings1, List<ConfigurationSetting> settings2) { if (settings1 == settings2) { return true; } if (settings1 == null || settings2 == null) { return false; } if (settings1.size() != settings2.size()) { return false; } final int size = settings1.size(); for (int i = 0; i < size; i++) { if (!equals(settings1.get(i), settings2.get(i))) { return false; } } return true; }
Source:
<code>public ValidationResult validate(Object value) { assert value == null || value instanceof String : "This validation must be run on a String!"; String mask = (String) value; if (!MaskValidator.getInstance().isMaskFormatValid(mask, isPrefixSupported)) { return failWith(getSubnetBadFormatMessage()); <START> } else if (!MaskValidator.getInstance().isMaskValid(mask, isPrefixSupported)) { <END> return failWith(getInvalidMask()); } return new ValidationResult(); }</code><technical_language>I mentioned in common patch, this constructed differently: * If !isPrefixSupported, check supplied string is in a valid netmask format (e.g. MaskValidator.isValidNetmaskFormat()). If not, fail bad format. * Else, need check check MaskValidator.isValidNetmaskFormat() || MaskValidator.isValidPrefixFormat(). If not, fail bad format. * If format passed, if MaskValidator.isValidNetmaskFormat(), need run validity check</technical_language>

Target:
public ValidationResult validate(Object value) { assert value == null || value instanceof String : "This validation must be run on a String!"; String mask = (String) value; if (isPrefixAllowed ? !MaskValidator.getInstance().isValidPrefixFormat(mask) : !MaskValidator.getInstance().isValidNetmaskFormat(mask)){ return failWith(getSubnetBadFormatMessage()); } else if (isPrefixAllowed ? !MaskValidator.getInstance().isPrefixValid(mask) : !MaskValidator.getInstance().isNetmaskValid(mask)) { return failWith(getInvalidMask()); } return new ValidationResult(); }
Source:
<code>public void canApproveAndReassign() throws IOException { <START> final String test = "test"; <END> final Approval approval = new ReassignTo(test); final PullRequest request = new PullRequest(); final Github github = Mockito.mock(Github.class); final User assignee = new User(); assignee.setLogin(test); final GitHubClient client = Mockito.mock(GitHubClient.class); Mockito.doReturn(client).when(github).client(); Mockito.when(client.get(Mockito.any(GitHubRequest.class))) .thenReturn( new GitHubResponse( Mockito.mock(HttpURLConnection.class), assignee ) ); MatcherAssert.assertThat( approval.has( request , github , new Github.Repo("xembly/xembly") ), Matchers.is(true) ); MatcherAssert.assertThat( request.getAssignee().getLogin(), Matchers.is(test) ); }</code><technical_language>I rename this login user. Name test absolutely semantic in this context</technical_language>

Target:
public void canApproveAndReassign() throws IOException { final String login = "test"; final Approval approval = new ReassignTo(login); final PullRequest request = new PullRequest(); final Github github = Mockito.mock(Github.class); final User assignee = new User(); assignee.setLogin(login); final GitHubClient client = Mockito.mock(GitHubClient.class); Mockito.doReturn(client).when(github).client(); Mockito.when(client.get(Mockito.any(GitHubRequest.class))) .thenReturn( new GitHubResponse( Mockito.mock(HttpURLConnection.class), assignee ) ); MatcherAssert.assertThat( approval.has( request, github, new Github.Repo("xembly/xembly") ), Matchers.is(true) ); MatcherAssert.assertThat( request.getAssignee().getLogin(), Matchers.is(login) ); }
Source:
<code>void cleanSubTypes(final String uuid) { final String selector = "[data-parent-row-uuid=\"" + uuid + "\"]"; final NodeList<Element> subDataTypeRows = getElement().querySelectorAll(selector); for (int i = 0; i < subDataTypeRows.length; i++) { final Element item = subDataTypeRows.getAt(i); <START> cleanSubTypes(item.getAttribute("data-row-uuid")); <END> item.remove(); } }</code><technical_language>before: data-row-uuid constant</technical_language>

Target:
void cleanSubTypes(final String uuid) { final String selector = "[data-parent-row-uuid=\"" + uuid + "\"]"; final NodeList<Element> subDataTypeRows = getElement().querySelectorAll(selector); for (int i = 0; i < subDataTypeRows.length; i++) { final Element item = subDataTypeRows.getAt(i); cleanSubTypes(item.getAttribute(UUID_ATTR)); item.remove(); } }
Source:
<code>public boolean perform(AbstractBuild<?, ?> build, Launcher launcher, BuildListener listener) throws InterruptedException, IOException { <START> throw new AbortException("Throwed AbortException from publisher!"); <END> }</code><technical_language>Trivial, word want is threw</technical_language>

Target:
public boolean perform(AbstractBuild<?, ?> build, Launcher launcher, BuildListener listener) throws InterruptedException, IOException { throw new AbortException("Threw AbortException from publisher!"); }
Source:
<code>private static NewIndexDescriptor indexGetForLabelAndPropertyKey( StateHandlingStatementOperations txContext, KernelStatement state, int labelId, int propertyKey ) { final int labelId3 = labelId; <START> final int[] propertyKeyIds = new int[]{propertyKey}; <END> return txContext.indexGetForLabelAndPropertyKey( state, SchemaDescriptorFactory.forLabel( labelId3, propertyKeyIds ) ); }</code><technical_language>ping</technical_language>

Target:
private static NewIndexDescriptor indexGetForLabelAndPropertyKey( StateHandlingStatementOperations txContext, KernelStatement state, int labelId, int propertyKey ) { LabelSchemaDescriptor schemaDescriptor = SchemaDescriptorFactory.forLabel( labelId, propertyKey ); return txContext.indexGetForLabelAndPropertyKey( state, schemaDescriptor ); }
Source:
<code>public RemoteServerConfigurationImpl(List<RemoteServerData> serverData, TaskRunner taskRunner) { <START> actualStateMap = Collections.synchronizedMap(new HashMap<>()); <END> remoteServerDataList = new ArrayList<>(serverData); for (RemoteServerData remoteServerData : serverData) { actualStateMap.put(remoteServerData.getServerName(), new RemoteServerState(RemoteServerState.State.NOT_CONNECTED, remoteServerData.getUsername(), null)); } if (taskRunner == null || !(taskRunner instanceof ScheduledTaskRunner)) { logger.info("Task runner isn't ScheduledTaskRunner. Remote server status task won't be scheduled."); } else { scheduledTaskRunner = (ScheduledTaskRunner) taskRunner; } }</code><technical_language>Consider a ConcurrentHashMap instead</technical_language>

Target:
public RemoteServerConfigurationImpl(List<RemoteServerData> serverData, TaskRunner taskRunner) { init(serverData, taskRunner); }
Source:
<code>public SkED25519PublicKey decodePublicKey( SessionContext session, String keyType, InputStream keyData, Map<String, String> headers) throws IOException, GeneralSecurityException { if (!KEY_TYPE.equals(keyType)) { throw new InvalidKeySpecException("Invalid keyType: " + keyType); } boolean noTouchRequired = false; <START> if (headers.containsKey(NO_TOUCH_REQUIRED_HEADER)) { <END> noTouchRequired = Boolean.parseBoolean(headers.get(NO_TOUCH_REQUIRED_HEADER)); } EdDSAPublicKey edDSAPublicKey = Ed25519PublicKeyDecoder.INSTANCE.decodePublicKey(session, KeyPairProvider.SSH_ED25519, keyData, headers); String appName = KeyEntryResolver.decodeString(keyData, MAX_APP_NAME_LENGTH); return new SkED25519PublicKey(appName, noTouchRequired, edDSAPublicKey); }</code><technical_language>if (GenericUtils.isNotEmpty(headers) && headers.contains(...)) in case null is passed</technical_language>

Target:
public SkED25519PublicKey decodePublicKey( SessionContext session, String keyType, InputStream keyData, Map<String, String> headers) throws IOException, GeneralSecurityException { if (!KEY_TYPE.equals(keyType)) { throw new InvalidKeySpecException("Invalid keyType: " + keyType); } boolean noTouchRequired = parseBooleanHeader(headers, NO_TOUCH_REQUIRED_HEADER, false); EdDSAPublicKey edDSAPublicKey = Ed25519PublicKeyDecoder.INSTANCE.decodePublicKey(session, KeyPairProvider.SSH_ED25519, keyData, headers); String appName = KeyEntryResolver.decodeString(keyData, MAX_APP_NAME_LENGTH); return new SkED25519PublicKey(appName, noTouchRequired, edDSAPublicKey); }
Source:
<code>public void testServerVariableHandling() { String alias = "somealias"; String port = "1234"; String host = "localhost.local"; String username = "acme"; String password = "password"; String url = "service:jmx:remoting-jms://amce.local:1234"; System.setProperty("myalias", alias); System.setProperty("myport", port); System.setProperty("myhost", host); System.setProperty("myusername",username); System.setProperty("mypassword", password); System.setProperty("myurl", url); Server s1 = Server.builder() .setAlias("${myalias}") .setPort("${myport}") .setHost("${myhost}") .setUsername("${myusername}") .setPassword("${mypassword}") .setUrl("${myurl}") .build(); Server s2 = Server.builder() .setAlias(alias) .setPort(port) .setHost(host) .setUsername(username) .setPassword(password) .setUrl(url) .build(); assertEquals(s1.hashCode(), s2.hashCode()); <START> System.clearProperty("myalias"); <END> System.clearProperty("myport"); System.clearProperty("myhost"); System.clearProperty("myusername"); System.clearProperty("myusername"); System.clearProperty("myurl"); }</code><technical_language>want clear properties in a finally block. if this test fails, environment is clean for next test</technical_language>

Target:
public void testServerVariableHandling() { try{ String alias = "somealias"; String port = "1234"; String host = "localhost.local"; String username = "acme"; String password = "password"; String url = "service:jmx:remoting-jms://amce.local:1234"; System.setProperty("myalias", alias); System.setProperty("myport", port); System.setProperty("myhost", host); System.setProperty("myusername",username); System.setProperty("mypassword", password); System.setProperty("myurl", url); Server serverFromSystemProperties = Server.builder() .setAlias("${myalias}") .setPort("${myport}") .setHost("${myhost}") .setUsername("${myusername}") .setPassword("${mypassword}") .setUrl("${myurl}") .build(); Server serverFromDirectParameters = Server.builder() .setAlias(alias) .setPort(port) .setHost(host) .setUsername(username) .setPassword(password) .setUrl(url) .build(); assertEquals(serverFromSystemProperties.hashCode(), serverFromDirectParameters.hashCode()); }finally{ System.clearProperty("myalias"); System.clearProperty("myport"); System.clearProperty("myhost"); System.clearProperty("myusername"); System.clearProperty("myusername"); System.clearProperty("myurl"); } }
Source:
<code>public void tearDown() throws Exception { executor.shutdownNow(); scheduledExecutor.shutdownNow(); executor.awaitTermination(10, SECONDS); <START> scheduledExecutor.awaitTermination(10, SECONDS); <END> }</code><technical_language>awaitTermination returns a boolean. This needs throw exception if terminate. invoke assertTrue</technical_language>

Target:
public void tearDown() throws Exception { executor.shutdownNow(); scheduledExecutor.shutdownNow(); assertTrue(executor.awaitTermination(10, SECONDS)); assertTrue(scheduledExecutor.awaitTermination(10, SECONDS)); }
Source:
<code>public void run() { StringBuffer traceBuffer = new StringBuffer(); try { Path filePath = Paths.get(filename); <START> RandomAccessFile file = new RandomAccessFile(filePath.toFile(), "r"); <END> traceBuffer.append("[" + filename + "]: threaded put object\n"); client.putObject(bucketName, filename, file,null,null,null,null); traceBuffer.append("[" + filename + "]: delete file\n"); Files.delete(Paths.get(filename)); traceBuffer.append("[" + filename + "]: delete object\n"); client.removeObject(bucketName, filename); } catch (Exception e) { System.err.print(traceBuffer.toString()); e.printStackTrace(); } }</code><technical_language>is needed? support filename upload version of put object</technical_language>

Target:
public void run() { StringBuffer traceBuffer = new StringBuffer(); try { traceBuffer.append("[" + filename + "]: threaded put object\n"); client.putObject(bucketName, filename, filename, null, null, null, null); traceBuffer.append("[" + filename + "]: delete file\n"); Files.delete(Paths.get(filename)); traceBuffer.append("[" + filename + "]: delete object\n"); client.removeObject(bucketName, filename); } catch (Exception e) { System.err.print(traceBuffer.toString()); e.printStackTrace(); } }
Source:
<code>public void setUp() { final EventTypeOptionsValidator eventTypeOptionsValidator = new EventTypeOptionsValidator(TOPIC_RETENTION_MIN_MS, TOPIC_RETENTION_MAX_MS); eventTypeService = new EventTypeService(eventTypeRepository, timelineService, partitionResolver, enrichment, subscriptionDbRepository, schemaEvolutionService, partitionsCalculator, featureToggleService, authorizationValidator, timelineSync, transactionTemplate, nakadiSettings, nakadiKpiPublisher, <START> KPI_ET_LOG_EVENT_TYPE, nakadiAuditLogPublisher, AUDIT_ET_LOG_EVENT_TYPE,eventTypeOptionsValidator, <END> adminService); when(transactionTemplate.execute(any())).thenAnswer(invocation -> { final TransactionCallback callback = (TransactionCallback) invocation.getArguments()[0]; return callback.doInTransaction(null); }); }</code><technical_language>missing space</technical_language>

Target:
public void setUp() { final EventTypeOptionsValidator eventTypeOptionsValidator = new EventTypeOptionsValidator(TOPIC_RETENTION_MIN_MS, TOPIC_RETENTION_MAX_MS); eventTypeService = new EventTypeService(eventTypeRepository, timelineService, partitionResolver, enrichment, subscriptionDbRepository, schemaEvolutionService, partitionsCalculator, featureToggleService, authorizationValidator, timelineSync, transactionTemplate, nakadiSettings, nakadiKpiPublisher, KPI_ET_LOG_EVENT_TYPE, nakadiAuditLogPublisher, eventTypeOptionsValidator, adminService); when(transactionTemplate.execute(any())).thenAnswer(invocation -> { final TransactionCallback callback = (TransactionCallback) invocation.getArguments()[0]; return callback.doInTransaction(null); }); }
Source:
<code>public void makeRequest(ConveyorTile tile, Resource target) throws GeoWebCacheException { WMSLayer layer = (WMSLayer) tile.getLayer(); GridSubset gridSubset = layer.getGridSubset(tile.getGridSetId()); Map<String, String> wmsParams = layer.getWMSRequestTemplate(tile.getMimeType(), WMSLayer.RequestType.MAP); wmsParams.put("FORMAT", tile.getMimeType().getFormat()); wmsParams.put("SRS", layer.backendSRSOverride(gridSubset.getSRS())); wmsParams.put("HEIGHT", String.valueOf(gridSubset.getTileHeight())); wmsParams.put("WIDTH", String.valueOf(gridSubset.getTileWidth())); BoundingBox bbox = gridSubset.boundsFromIndex(tile.getTileIndex()); wmsParams.put("BBOX", bbox.toString()); <START> Map<String, String> fullParameters = tile.getFilteringParameters(); <END> if (fullParameters.isEmpty()) { fullParameters = layer.getDefaultParameterFilters(); } wmsParams.putAll(fullParameters); if (tile.getMimeType() == XMLMime.kml) { wmsParams.put("format_options", "mode:superoverlay;overlaymode:auto"); } MimeType mimeType = tile.getMimeType(); makeRequest(tile, layer, wmsParams, mimeType, target); }</code><technical_language>correct local variable name</technical_language>

Target:
public void makeRequest(ConveyorTile tile, Resource target) throws GeoWebCacheException { WMSLayer layer = (WMSLayer) tile.getLayer(); GridSubset gridSubset = layer.getGridSubset(tile.getGridSetId()); Map<String, String> wmsParams = layer.getWMSRequestTemplate(tile.getMimeType(), WMSLayer.RequestType.MAP); wmsParams.put("FORMAT", tile.getMimeType().getFormat()); wmsParams.put("SRS", layer.backendSRSOverride(gridSubset.getSRS())); wmsParams.put("HEIGHT", String.valueOf(gridSubset.getTileHeight())); wmsParams.put("WIDTH", String.valueOf(gridSubset.getTileWidth())); BoundingBox bbox = gridSubset.boundsFromIndex(tile.getTileIndex()); wmsParams.put("BBOX", bbox.toString()); Map<String, String> filteringParameters = tile.getFilteringParameters(); if (filteringParameters.isEmpty()) { filteringParameters = layer.getDefaultParameterFilters(); } wmsParams.putAll(filteringParameters); if (tile.getMimeType() == XMLMime.kml) { wmsParams.put("format_options", "mode:superoverlay;overlaymode:auto"); } MimeType mimeType = tile.getMimeType(); makeRequest(tile, layer, wmsParams, mimeType, target); }
Source:
<code><START> protected Object getNonNullValueForType(Type type) <END> { if (type.getJavaType() == boolean.class) { return true; } if (type.getJavaType() == long.class) { return 1L; } if (type.getJavaType() == double.class) { return 1.0; } if (type.getJavaType() == Slice.class) { return Slices.utf8Slice("_"); } if (type instanceof ArrayType) { ArrayType arrayType = (ArrayType) type; Type elementType = arrayType.getElementType(); Object elementNonNullValue = getNonNullValueForType(elementType); return arrayBlockOf(elementType, elementNonNullValue); } if (type instanceof MapType) { MapType mapType = (MapType) type; Type keyType = mapType.getKeyType(); Type valueType = mapType.getValueType(); Object keyNonNullValue = getNonNullValueForType(keyType); Object valueNonNullValue = getNonNullValueForType(valueType); Map map = ImmutableMap.of(keyNonNullValue, valueNonNullValue); return mapBlockOf(keyType, valueType, map); } if (type instanceof RowType) { RowType rowType = (RowType) type; List<Type> elementTypes = rowType.getTypeParameters(); Object[] elementNonNullValues = elementTypes.stream().map(this::getNonNullValueForType).toArray(Object[]::new); return toRow(elementTypes, elementNonNullValues); } throw new IllegalStateException("Unsupported Java type " + type.getJavaType() + " (for type " + type + ")"); }</code><technical_language>I need this protected. override getNonNullValue. TestLongDecimalType does</technical_language>

Target:
private static Object getNonNullValueForType(Type type) { if (type.getJavaType() == boolean.class) { return true; } if (type.getJavaType() == long.class) { return 1L; } if (type.getJavaType() == double.class) { return 1.0; } if (type.getJavaType() == Slice.class) { return Slices.utf8Slice("_"); } if (type instanceof ArrayType) { ArrayType arrayType = (ArrayType) type; Type elementType = arrayType.getElementType(); Object elementNonNullValue = getNonNullValueForType(elementType); return arrayBlockOf(elementType, elementNonNullValue); } if (type instanceof MapType) { MapType mapType = (MapType) type; Type keyType = mapType.getKeyType(); Type valueType = mapType.getValueType(); Object keyNonNullValue = getNonNullValueForType(keyType); Object valueNonNullValue = getNonNullValueForType(valueType); Map map = ImmutableMap.of(keyNonNullValue, valueNonNullValue); return mapBlockOf(keyType, valueType, map); } if (type instanceof RowType) { RowType rowType = (RowType) type; List<Type> elementTypes = rowType.getTypeParameters(); Object[] elementNonNullValues = elementTypes.stream().map(AbstractTestType::getNonNullValueForType).toArray(Object[]::new); return toRow(elementTypes, elementNonNullValues); } throw new IllegalStateException("Unsupported Java type " + type.getJavaType() + " (for type " + type + ")"); }
Source:
<code>public AllureReportGenerator(final File... inputDirectories) { this.inputDirectories = inputDirectories; <START> testSuiteFiles = new TestSuiteFiles(validateXML, inputDirectories); <END> }</code><technical_language>mb this.testSuiteFiles? this.inputDir..</technical_language>

Target:
public AllureReportGenerator(final File... inputDirectories) { this.inputDirectories = inputDirectories; this.testRunGenerator = new TestRunGenerator(validateXML, inputDirectories); }