auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>private void startProducer(MqttMessageProducer publisher) { logger.trace("Starting message producer for broker '{}'", name); publisher.setSenderChannel(new MqttSenderChannel() { @Override public void publish(String topic, byte[] payload) throws IOException { if (!started) { logger.warn("Broker connection not started. Cannot publish message to topic '{}'", topic); return; } MqttMessage message = new MqttMessage(payload); message.setQos(qos); message.setRetained(retain); MqttTopic mqttTopic = client.getTopic(topic); MqttDeliveryToken deliveryToken; try { deliveryToken = mqttTopic.publish(message); logger.debug("Publishing message to topic '{}'", deliveryToken.getMessageId(), topic); if (!async) { deliveryToken.waitForCompletion(10000); if (!deliveryToken.isComplete()) { logger.error( "Did not receive completion message within timeout limit whilst publishing to topic '{}'", topic); } } } catch (MqttException e) { <START> logger.error("Could not publish message to topic {}", topic); <END> } } }); }</code><technical_language>this comply [logging guidelines](<LINK_0> if ERROR level. if so, want exception message in log if is a system bug, stacktrace well</technical_language>

Target:
private void startProducer(MqttMessageProducer publisher) { logger.trace("Starting message producer for broker '{}'", name); publisher.setSenderChannel(new MqttSenderChannel() { @Override public void publish(String topic, byte[] payload) throws IOException { if (!started) { logger.warn("Broker connection not started. Cannot publish message to topic '{}'", topic); return; } MqttMessage message = new MqttMessage(payload); message.setQos(qos); message.setRetained(retain); MqttTopic mqttTopic = client.getTopic(topic); MqttDeliveryToken deliveryToken; try { deliveryToken = mqttTopic.publish(message); logger.debug("Publishing message to topic '{}'", deliveryToken.getMessageId(), topic); if (!async) { deliveryToken.waitForCompletion(10000); if (!deliveryToken.isComplete()) { logger.error( "Did not receive completion message within timeout limit whilst publishing to topic '{}'", topic); } } } catch (MqttException e) { logger.error("Could not publish message to topic {}", topic, e); } } }); }
Source:
<code>private RelNode possiblyWrapRootWithOuterLimitFromContext( RelRoot root ) { Object outerLimitObj = plannerContext.getQueryContext().get(PlannerContext.CTX_SQL_OUTER_LIMIT); Long outerLimit = DimensionHandlerUtils.convertObjectToLong(outerLimitObj, true); if (outerLimit == null) { return root.rel; } if (root.rel instanceof LogicalSort) { LogicalSort outerSort = (LogicalSort) root.rel; return LogicalSort.create( outerSort.getInput(), outerSort.collation, <START> makeBigIntLiteral(0), <END> makeBigIntLiteral(outerLimit) ); } return LogicalSort.create( root.rel, root.collation, makeBigIntLiteral(0), makeBigIntLiteral(outerLimit) ); }</code><technical_language>This handle offsets properly, handle properly case outer limit is higher limit. Check out SortCollapseRule for some logic handles of cases. want create a helper method in places</technical_language>

Target:
private RelNode possiblyWrapRootWithOuterLimitFromContext( RelRoot root ) { Object outerLimitObj = plannerContext.getQueryContext().get(PlannerContext.CTX_SQL_OUTER_LIMIT); Long outerLimit = DimensionHandlerUtils.convertObjectToLong(outerLimitObj, true); if (outerLimit == null) { return root.rel; } if (root.rel instanceof Sort) { Sort innerSort = (Sort) root.rel; final int offset = Calcites.getOffset(innerSort); final int fetch = Calcites.collapseFetch( Calcites.getFetch(innerSort), Ints.checkedCast(outerLimit), 0 ); return LogicalSort.create( innerSort.getInput(), innerSort.collation, makeBigIntLiteral(offset), makeBigIntLiteral(fetch) ); } return LogicalSort.create( root.rel, root.collation, makeBigIntLiteral(0), makeBigIntLiteral(outerLimit) ); }
Source:
<code>public void testSlice() throws Exception { <START> Block block1 = readFile("test_dictionaries/string-dictionary.orc", VARCHAR); <END> Block block2 = readFile("test_dictionaries/string-row-group-dictionary.dwrf", VARCHAR); compare(VARCHAR, block1, block2); }</code><technical_language>block1 block2 confusing is clear is tested. rename expected actual. - rename compare method assertEqualBlock - inline null1 null2 variables @Test public void testSlice() throws Exception { Block expected = readFile("test_dictionaries/string-dictionary.orc", VARCHAR); Block actual = readFile("test_dictionaries/string-row-group-dictionary.dwrf", VARCHAR); assertEqualBlocks(VARCHAR, expected, actual); } private static void assertEqualBlocks(Type type, Block expected, Block actual) { assertEquals(expected.getPositionCount(), actual.getPositionCount()); for (int i = 0; i < expected.getPositionCount(); i++) { assertEquals(actual.isNull(i), expected.isNull(i)); if (!expected.isNull(i)) { assertEquals(type.compareTo(expected, i, actual, i), 0); } } }</technical_language>

Target:
public void testSlice() throws Exception { Block expected = readFile("test_dictionaries/string-dictionary.orc", VARCHAR); Block actual = readFile("test_dictionaries/string-row-group-dictionary.dwrf", VARCHAR); assertEqualBlocks(VARCHAR, expected, actual); }
Source:
<code>public void run() { while (this.enabled) { try { this.ledManager.writeLed(true); Thread.sleep(CloudConnectionStatusEnum.SLOW_BLINKING_ON_TIME); this.ledManager.writeLed(false); Thread.sleep(CloudConnectionStatusEnum.SLOW_BLINKING_OFF_TIME); } catch (Exception ex) { <START> break; <END> } } }</code><technical_language>Is catch a more specific exception</technical_language>

Target:
public void run() { while (this.enabled) { try { this.ledManager.writeLed(true); Thread.sleep(CloudConnectionStatusEnum.SLOW_BLINKING_ON_TIME); this.ledManager.writeLed(false); Thread.sleep(CloudConnectionStatusEnum.SLOW_BLINKING_OFF_TIME); } catch (InterruptedException e) { Thread.currentThread().interrupt(); this.enabled = false; } catch (KuraException e) { this.enabled = false; } } }
Source:
<code>} catch (final Exception exception) { <START> log.warn("Error while loading kafka-streams-version.properties: {}", exception.getMessage()); <END> }</code><technical_language>nit: log stacktrace here? IMHO for debugging</technical_language>

Target:
} catch (final Exception exception) { log.warn("Error while loading kafka-streams-version.properties", exception); }
Source:
<code>public JsonJavaArray extractImagesFromWikiPage(String pageHTML){ JsonJavaArray imageAttributeObjects = new JsonJavaArray(); <START> Pattern p = Pattern.compile("<img([^>]*[^>]*)>([^<>]*)</img>"); <END> Matcher m = p.matcher(pageHTML); while(m.find()) { String imgAttrs = m.group(1); if(imgAttrs==null){ break; } JsonJavaObject jsonAttrs = new JsonJavaObject(); imgAttrs = imgAttrs.trim(); String[] attrs = imgAttrs.split("\" "); for(int i = 0; i < attrs.length; i++){ String attr = attrs[i]; attr = attr.replaceAll("\"", ""); String[] nameValue = attr.split("="); if(nameValue.length == 2){ jsonAttrs.put(nameValue[0], nameValue[1]); }else if(nameValue.length == 1){ jsonAttrs.put(nameValue[0], ""); } } imageAttributeObjects.add(jsonAttrs); } return imageAttributeObjects; }</code><technical_language>[^>]\* is redundant, need</technical_language>

Target:
public JsonJavaArray extractImagesFromWikiPage(String pageHTML){ JsonJavaArray imageAttributeObjects = new JsonJavaArray(); Pattern p = Pattern.compile("<img([^>]*)>([^<>]*)</img>"); Matcher m = p.matcher(pageHTML); while(m.find()) { String imgAttrs = m.group(1); if(imgAttrs==null){ break; } JsonJavaObject jsonAttrs = new JsonJavaObject(); imgAttrs = imgAttrs.trim(); String[] attrs = imgAttrs.split("\" "); for(int i = 0; i < attrs.length; i++){ String attr = attrs[i]; attr = attr.replaceAll("\"", ""); String[] nameValue = attr.split("="); if(nameValue.length == 2){ jsonAttrs.put(nameValue[0], nameValue[1]); }else if(nameValue.length == 1){ jsonAttrs.put(nameValue[0], ""); } } imageAttributeObjects.add(jsonAttrs); } return imageAttributeObjects; }
Source:
<code>public void skipTree() { if (currVisit != <START> null) <END> currVisit.ptr = currVisit.buf.length; }</code><technical_language>Please add {}</technical_language>

Target:
public void skipTree() { if (currVisit != null) { currVisit.ptr = currVisit.buf.length; } }
Source:
<code>public static String unlink(LinkType sourceLinkType, Artifact source, String content) throws OseeCoreException { LinkType linkType = checkLinkType(sourceLinkType); String modified = content; HashCollection<String, MatchRange> matchMap = parseOseeWordMLLinks(content); if (!matchMap.isEmpty()) { <START> modified = modifiedContent(linkType, source, content, matchMap, true, null, null); <END> } return modified; }</code><technical_language>provide a method require caller pass null for presentationType</technical_language>

Target:
public static String unlink(LinkType sourceLinkType, Artifact source, String content) throws OseeCoreException { LinkType linkType = checkLinkType(sourceLinkType); String modified = content; HashCollection<String, MatchRange> matchMap = parseOseeWordMLLinks(content); if (!matchMap.isEmpty()) { modified = modifiedContent(linkType, source, content, matchMap, true, null); } return modified; }
Source:
<code>private boolean moveItemToSlotsFillClientTempEntities(EntityRef instigator, EntityRef fromInventory, int slotFrom, EntityRef toInventory, List<Integer> toSlots, Collection<EntityRef> clientTempEntities) { <START> Set<Integer> emptySlotsBefore = toSlots.stream().filter(toSlot -> !InventoryUtils.getItemAt(toInventory, toSlot).exists()).collect(Collectors.toSet()); <END> if (!InventoryUtils.moveItemToSlots(instigator, fromInventory, slotFrom, toInventory, toSlots)) { return true; } for (Integer slot : emptySlotsBefore) { EntityRef itemAt = InventoryUtils.getItemAt(toInventory, slot); if (itemAt.exists()) { clientTempEntities.add(itemAt); } } return false; }</code><technical_language>I find original version easier read in this case</technical_language>

Target:
private boolean moveItemToSlotsFillClientTempEntities(EntityRef instigator, EntityRef fromInventory, int slotFrom, EntityRef toInventory, List<Integer> toSlots, Collection<EntityRef> clientTempEntities) { Set<Integer> emptySlotsBefore = new HashSet<>(); for (Integer toSlot : toSlots) { if (!InventoryUtils.getItemAt(toInventory, toSlot).exists()) { emptySlotsBefore.add(toSlot); } } if (!InventoryUtils.moveItemToSlots(instigator, fromInventory, slotFrom, toInventory, toSlots)) { return true; } for (Integer slot : emptySlotsBefore) { EntityRef itemAt = InventoryUtils.getItemAt(toInventory, slot); if (itemAt.exists()) { clientTempEntities.add(itemAt); } } return false; }
Source:
<code>protected void executeCommand() { <START> if (getTagId() != null) { <END> for (Guid templateGuid : getTemplatesList()) { VmTemplate template = DbFacade.getInstance().getVmTemplateDao().get(templateGuid); if (DbFacade.getInstance().getTagDao().getTagTemplateByTagIdAndByTemplateId(getTagId(), templateGuid) == null) { if (template != null) { appendCustomValue("TemplatesNames", template.getName(), ", "); } TagsTemplateMap map = new TagsTemplateMap(getTagId(), templateGuid); DbFacade.getInstance().getTagDao().attachTemplateToTag(map); noActionDone = false; } else { if (template != null) { appendCustomValue("TemplatesNamesExists", template.getName(), ", "); } } } } setSucceeded(true); }</code><technical_language>i this is other tags commands written, pretty old, this is standard write commands, basically command a 'CanDoAction' method perform needed checks (here getTagId() null) fail command a proper translated error. execute phase clean of checks. do think? new commands in standard way? stick this clear tags mess</technical_language>

Target:
protected void executeCommand() { for (Guid templateGuid : getTemplatesList()) { VmTemplate template = DbFacade.getInstance().getVmTemplateDao().get(templateGuid); if (DbFacade.getInstance().getTagDao().getTagTemplateByTagIdAndByTemplateId(getTagId(), templateGuid) == null) { if (template != null) { appendCustomValue("TemplatesNames", template.getName(), ", "); TagsTemplateMap map = new TagsTemplateMap(getTagId(), templateGuid); DbFacade.getInstance().getTagDao().attachTemplateToTag(map); noActionDone = false; } } else { if (template != null) { appendCustomValue("TemplatesNamesExists", template.getName(), ", "); } } } setSucceeded(true); }
Source:
<code>public boolean post(String topic, Object data) { Event event = constructEvent(topic, data); EventAdmin eventAdmin = Activator.getDefault().getEventAdmin(); if (eventAdmin == null && logger != null) { logger.error(NLS.bind(ServiceMessages.NO_EVENT_ADMIN, event.toString())); <START> return false; <END> } eventAdmin.postEvent(event); return true; }</code><technical_language>is incorrect, in case logger eventAdmin is null get a NPE</technical_language>

Target:
public boolean post(String topic, Object data) { Event event = constructEvent(topic, data); EventAdmin eventAdmin = Activator.getDefault().getEventAdmin(); if (eventAdmin == null) { if (logger != null) { logger.error(NLS.bind(ServiceMessages.NO_EVENT_ADMIN, event.toString())); } return false; } eventAdmin.postEvent(event); return true; }
Source:
<code>public static <T extends Enum<?>> void parseEnumValueList(Class<T> c, List<T> list, String value, char... separators) { EwsUtilities.EwsAssert(c.isEnum(), "EwsUtilities.ParseEnumValueList", "T is not an enum type."); <START> StringBuilder regexp = new StringBuilder(""); <END> regexp.append("["); for (char s : separators) { regexp.append("["); regexp.append(Pattern.quote(s + "")); regexp.append("]"); } regexp.append("]"); String[] enumValues = value.split(regexp.toString()); for (String enumValue : enumValues) { for (T o : c.getEnumConstants()) { if (o.toString().equals(enumValue)) { list.add(o); } } } }</code><technical_language>initializing a blank string is option here</technical_language>

Target:
public static <T extends Enum<?>> void parseEnumValueList(Class<T> c, List<T> list, String value, char... separators) { EwsUtilities.EwsAssert(c.isEnum(), "EwsUtilities.ParseEnumValueList", "T is not an enum type."); StringBuilder regexp = new StringBuilder(); regexp.append("["); for (char s : separators) { regexp.append("["); regexp.append(Pattern.quote(s + "")); regexp.append("]"); } regexp.append("]"); String[] enumValues = value.split(regexp.toString()); for (String enumValue : enumValues) { for (T o : c.getEnumConstants()) { if (o.toString().equals(enumValue)) { list.add(o); } } } }
Source:
<code>public String getForkNodeConnectionString() { <START> return "tcp://" + localHost + ":" + localPort + ( sessionId == null ? "" : "?sessionId=" + sessionId ); <END> }</code><technical_language>If this is local host, do need a vaialble</technical_language>

Target:
public String getForkNodeConnectionString() { return "tcp://" + localHost + ":" + localPort + "?sessionId=" + sessionId; }
Source:
<code>public List<String> getResourceFileList() { if (dirFileList == null) { String[] files = null; if (isDir() && getResourcePath().length() > 1) { files = getResourceFile().list(); } if (files == null) { dirFileList = Collections.emptyList(); } else { Arrays.sort(files, String.CASE_INSENSITIVE_ORDER); List<String> list = Arrays.asList(files); if (getPath().isEmpty()) { <START> list = new ArrayList<>(list); <END> list.removeIf((t) -> { return !getProjectHelper().getAllProjects().stream().anyMatch((p) -> { return p.getName().equalsIgnoreCase(t); }); }); } dirFileList = Collections.unmodifiableList(list); } } return dirFileList; }</code><technical_language>list variable named differently original, easier debug</technical_language>

Target:
public List<String> getResourceFileList() { if (dirFileList == null) { String[] files = null; if (isDir() && getResourcePath().length() > 1) { files = getResourceFile().list(); } if (files == null) { dirFileList = Collections.emptyList(); } else { Arrays.sort(files, String.CASE_INSENSITIVE_ORDER); List<String> listOfFiles = Arrays.asList(files); if (env.hasProjects() && getPath().isEmpty()) { List<String> modifiableListOfFiles = new ArrayList<>(listOfFiles); modifiableListOfFiles.removeIf((t) -> { return !getProjectHelper().getAllProjects().stream().anyMatch((p) -> { return p.getName().equalsIgnoreCase(t); }); }); return dirFileList = Collections.unmodifiableList(modifiableListOfFiles); } dirFileList = Collections.unmodifiableList(listOfFiles); } } return dirFileList; }
Source:
<code>public void testUnmarshallTopLevelEventWithEdgesEmptyProperties() throws Exception { Diagram<Graph, Metadata> diagram = unmarshall(marshaller, BPMN_CATCHING_INTERMEDIATE_EVENT_FILE_PATH); assertDiagram(diagram, AMOUNT_OF_NODES_IN_DIAGRAM); IntermediateMessageEventCatching emptyEvent = getCatchingIntermediateNodeById(diagram, EMPTY_WITH_OUTGOING_EDGE_TOP_LEVEL_EVENT_ID, HAS_NO_INCOME_EDGE, HAS_OUTGOING_EDGE); assertGeneralSet(emptyEvent.getGeneral(), EMPTY_VALUE, EMPTY_VALUE); <START> assertMessageEventExecutionSet(emptyEvent.getExecutionSet(), EMPTY_VALUE, NON_CANCELLING, ""); <END> assertDataIOSet(emptyEvent.getDataIOSet(), EMPTY_VALUE); }</code><technical_language>suggestion assertMessageEventExecutionSet(emptyEvent.getExecutionSet(), EMPTY_VALUE, NON_CANCELLING, EMPTY_VALUE);</technical_language>

Target:
public void testUnmarshallTopLevelEventWithEdgesEmptyProperties() throws Exception { Diagram<Graph, Metadata> diagram = unmarshall(marshaller, BPMN_CATCHING_INTERMEDIATE_EVENT_FILE_PATH); assertDiagram(diagram, AMOUNT_OF_NODES_IN_DIAGRAM); IntermediateMessageEventCatching emptyEvent = getCatchingIntermediateNodeById(diagram, EMPTY_WITH_OUTGOING_EDGE_TOP_LEVEL_EVENT_ID, HAS_NO_INCOME_EDGE, HAS_OUTGOING_EDGE); assertGeneralSet(emptyEvent.getGeneral(), EMPTY_VALUE, EMPTY_VALUE); assertMessageEventExecutionSet(emptyEvent.getExecutionSet(), EMPTY_VALUE, NON_CANCELLING, EMPTY_VALUE); assertDataIOSet(emptyEvent.getDataIOSet(), EMPTY_VALUE); }
Source:
<code><START> public void getExtension__OnUnknownType_ReturnsUnknown() { <END> Embedding embedding = new Embedding("js", "", null); String extension = embedding.getExtension(); assertThat(extension).isEqualTo("unknown"); }</code><technical_language>nice - thanks</technical_language>

Target:
public void getExtension__OnUnknownType_ReturnsUnknown() { Embedding embedding = new Embedding("js", ""); String extension = embedding.getExtension(); assertThat(extension).isEqualTo("unknown"); }
Source:
<code>public void testBadRequest() throws IOException { int port = server.getPort(); String msg = "GET /status HTTP/1.1\n\n"; InetSocketAddress sockAddr = new InetSocketAddress("127.0.0.1", port); Socket sock = new Socket(); sock.connect(sockAddr); OutputStream out = sock.getOutputStream(); out.write(msg.getBytes(StandardCharsets.UTF_8)); out.flush(); byte[] buf = new byte[2048]; int bytesRead = sock.getInputStream().read(buf); Assert.assertTrue("Connection should have been closed", bytesRead == -1); <START> sock.close(); <END> }</code><technical_language>a try-with-resource pattern here</technical_language>

Target:
public void testBadRequest() throws IOException { int port = server.getPort(); String msg = "GET /status HTTP/1.1\n\n"; InetSocketAddress sockAddr = new InetSocketAddress("127.0.0.1", port); try (Socket sock = new Socket()) { sock.connect(sockAddr); OutputStream out = sock.getOutputStream(); out.write(msg.getBytes(StandardCharsets.UTF_8)); out.flush(); byte[] buf = new byte[2048]; int bytesRead = sock.getInputStream().read(buf); Assert.assertTrue("Connection should have been closed", bytesRead == -1); } }
Source:
<code>private static int toPublic(int access) { <START> return access & ~(Opcodes.ACC_PRIVATE + Opcodes.ACC_PROTECTED) | Opcodes.ACC_PUBLIC; <END> }</code><technical_language>| is +</technical_language>

Target:
private static int toPublic(int access) { return access & ~(Opcodes.ACC_PRIVATE | Opcodes.ACC_PROTECTED) | Opcodes.ACC_PUBLIC; }
Source:
<code>public void beforeTestMethod(TestContext testContext) throws Exception { <START> Authentication authentication = TestSecurityContextHolder.getContext().getAuthentication(); <END> Hooks.onLastOperator(Operators.lift((s, sub) -> new SecuritySubContext<>(sub, authentication))); }</code><technical_language>Instead of obtaining Authentication, please SecurityContext original SecurityContext is (this ensures if a custom SecurityContext established is well)</technical_language>

Target:
public void beforeTestMethod(TestContext testContext) throws Exception { SecurityContext securityContext = TestSecurityContextHolder.getContext(); Hooks.onLastOperator(Operators.lift((s, sub) -> new SecuritySubContext<>(sub, securityContext))); }
Source:
<code><START> public ResponseEntity putDictionary(@PathVariable("tenant") final String tenant, <END> @PathVariable("name") final String name, @RequestBody final DictionaryModel dictionaryModel) { final TenantContext context = _contextFactory.getContext(tenant); final FileResource resource = new FileResource(getResourceFile(context, name)); resource.write( stream -> stream.write((Joiner.on("\n").join(dictionaryModel.getEntries()) + "\n").getBytes("UTF-8"))); final TextFileDictionary dictionary = new TextFileDictionary(name, resource.getFile().getAbsolutePath(), "UTF-8", dictionaryModel.isCaseSensitive()); _referenceDataDao.removeDictionary(context, name); _referenceDataDao.addDictionary(context, dictionary); return ResponseEntity.created(ServletUriComponentsBuilder.fromCurrentRequestUri().build().toUri()).build(); }</code><technical_language>ResponseEntity is a raw type. References generic type ResponseEntity<T> parameterized</technical_language>

Target:
public ResponseEntity<Void> putDictionary(@PathVariable("tenant") final String tenant, @PathVariable("name") final String name, @RequestBody final DictionaryModel dictionaryModel) { final TenantContext context = _contextFactory.getContext(tenant); final FileResource resource = new FileResource(getResourceFile(context, name)); resource.write( stream -> stream.write((Joiner.on("\n").join(dictionaryModel.getEntries()) + "\n").getBytes("UTF-8"))); final TextFileDictionary dictionary = new TextFileDictionary(name, resource.getFile().getAbsolutePath(), "UTF-8", dictionaryModel.isCaseSensitive()); _referenceDataDao.removeDictionary(context, name); _referenceDataDao.addDictionary(context, dictionary); return ResponseEntity.created(ServletUriComponentsBuilder.fromCurrentRequestUri().build().toUri()).build(); }
Source:
<code>public MembershipExpirationInDays(Member member, int daysToExpiration, Vo vo) { this.member = member; this.daysToExpiration = daysToExpiration; this.vo = vo; <START> this.message = formatMessage("%s will expire in %s days in %d.", member, vo, daysToExpiration); <END> }</code><technical_language>order be. Please check message</technical_language>

Target:
public MembershipExpirationInDays(Member member, int daysToExpiration, Vo vo) { this.member = member; this.daysToExpiration = daysToExpiration; this.vo = vo; this.message = formatMessage("%s will expire in %d days in %s.", member, daysToExpiration, vo); }
Source:
<code>public void testIscsiLogin() throws Exception { setUriInfo(setUpActionExpectations(VdcActionType.ConnectStorageToVds, StorageServerConnectionParametersBase.class, new String[] { "VdsId", "StorageServerConnection.connection", "StorageServerConnection.portal", "StorageServerConnection.iqn", "StorageServerConnection.port", "StorageServerConnection.storage_type", "StorageServerConnection.user_name", "StorageServerConnection.password" }, new Object[] { GUIDS[0], ISCSI_SERVER_ADDRESS, <START> "1", END> ISCSI_IQN, ISCSI_PORT_STRING, ISCSI_STORAGE_TYPE, ISCSI_USER_NAME, ISCSI_USER_PASS })); Action action = new Action(); IscsiDetails iscsiDetails = new IscsiDetails(); iscsiDetails.setAddress(ISCSI_SERVER_ADDRESS); iscsiDetails.setPort(ISCSI_PORT_INT); iscsiDetails.setTarget(ISCSI_IQN); iscsiDetails.setUsername(ISCSI_USER_NAME); iscsiDetails.setPassword(ISCSI_USER_PASS); action.setIscsi(iscsiDetails); verifyActionResponse(resource.iscsiLogin(action)); }</code><technical_language>constant reused here</technical_language>

Target:
public void testIscsiLogin() throws Exception { setUriInfo(setUpActionExpectations(VdcActionType.ConnectStorageToVds, StorageServerConnectionParametersBase.class, new String[] { "VdsId", "StorageServerConnection.connection", "StorageServerConnection.portal", "StorageServerConnection.iqn", "StorageServerConnection.port", "StorageServerConnection.storage_type", "StorageServerConnection.user_name", "StorageServerConnection.password" }, new Object[] { GUIDS[0], ISCSI_SERVER_ADDRESS, StorageServerConnections.DEFAULT_TPGT, ISCSI_IQN, ISCSI_PORT_STRING, ISCSI_STORAGE_TYPE, ISCSI_USER_NAME, ISCSI_USER_PASS })); Action action = new Action(); IscsiDetails iscsiDetails = new IscsiDetails(); iscsiDetails.setAddress(ISCSI_SERVER_ADDRESS); iscsiDetails.setPort(ISCSI_PORT_INT); iscsiDetails.setTarget(ISCSI_IQN); iscsiDetails.setUsername(ISCSI_USER_NAME); iscsiDetails.setPassword(ISCSI_USER_PASS); action.setIscsi(iscsiDetails); verifyActionResponse(resource.iscsiLogin(action)); }
Source:
<code>public View getView(int position, View convertView, ViewGroup parent) { if (convertView == null) { <START> convertView = View.inflate(getContext(), R.layout.list_contributor_item, null); <END> ViewHolder viewHolder = new ViewHolder(convertView); convertView.setTag(viewHolder); } Contributor contributor = getItem(position); ViewHolder viewHolder = (ViewHolder) convertView.getTag(); viewHolder.contributorName.setText(contributor.getLogin()); if (!TextUtils.isEmpty(contributor.getAvatarUrl())) { App.getInstance().getPicasso() .load(contributor.getAvatarUrl()) .into(viewHolder.contributorAvatar); } return convertView; }</code><technical_language>this pr s scope this parent, false pattern</technical_language>

Target:
public View getView(int position, View convertView, ViewGroup parent) { if (convertView == null) { convertView = mLayoutInflator.inflate(R.layout.list_contributor_item, parent, false); ViewHolder viewHolder = new ViewHolder(convertView); convertView.setTag(viewHolder); } Contributor contributor = getItem(position); ViewHolder viewHolder = (ViewHolder) convertView.getTag(); viewHolder.contributorName.setText(contributor.getLogin()); if (!TextUtils.isEmpty(contributor.getAvatarUrl())) { App.getInstance().getPicasso() .load(contributor.getAvatarUrl()) .into(viewHolder.contributorAvatar); } return convertView; }
Source:
<code>protected void applyCenterInside(@NonNull TextureView view) { <START> if(intrinsicVideoSize.y <= view.getWidth() && intrinsicVideoSize.x <= view.getHeight()) { <END> applyCenter(view); } else { applyFitCenter(view); } }</code><technical_language>Merge conflict. This comparing width x height y</technical_language>

Target:
protected void applyCenterInside(@NonNull TextureView view) { if(intrinsicVideoSize.x <= view.getWidth() && intrinsicVideoSize.y <= view.getHeight()) { applyCenter(view); } else { applyFitCenter(view); } }
Source:
<code>public AuthEntry(Info info, Type... types) { Set<Type> typeSet = Sets.newHashSet(); <START> for (Type type : types) { <END> typeSet.add(type); } this.info = info; this.typeSet = ImmutableSet.copyOf(typeSet); }</code><technical_language>? this logic is necessary</technical_language>

Target:
public AuthEntry(Info info, Type... types) { List<String> temp = Lists.newArrayList(); for (Type type : types) { temp.add(type.name()); } this.info = info; this.types = ImmutableSet.copyOf(types); this.stringTypes = ImmutableSet.copyOf(temp); }
Source:
<code>private ServerPing getPingInfo( String motd, int protocol ) { <START> new ServerPing( <END> new ServerPing.Protocol( bungee.getName() + " " + bungee.getGameVersion(), protocol ), new ServerPing.Players( listener.getMaxPlayers(), bungee.getOnlineCount(), null ), motd, BungeeCord.getInstance().config.getFaviconObject() ); }</code><technical_language>kind of missing a return here..................</technical_language>

Target:
private ServerPing getPingInfo(String motd, int protocol) { return new ServerPing( new ServerPing.Protocol( bungee.getName() + " " + bungee.getGameVersion(), protocol ), new ServerPing.Players( listener.getMaxPlayers(), bungee.getOnlineCount(), null ), motd, BungeeCord.getInstance().config.getFaviconObject() ); }
Source:
<code>public DefaultThriftService(PrestoThriftService prestoThriftService, HostLocationHandle hostLocationHandle) { <START> requireNonNull(hostLocationHandle, "hostLocationHandle is null"); <END> requireNonNull(prestoThriftService, "prestoThriftService is null"); this.hostLocationHandle = hostLocationHandle; this.prestoThriftService = prestoThriftService; }</code><technical_language>inline assignments</technical_language>

Target:
public DefaultThriftService(PrestoThriftService prestoThriftService, HostLocationHandle hostLocationHandle) { this.prestoThriftService = requireNonNull(prestoThriftService, "prestoThriftService is null"); this.hostLocationHandle = requireNonNull(hostLocationHandle, "hostLocationHandle is null"); }
Source:
<code>public boolean equals(Object o) { if (this == o) { return true; } <START> if (o == null || getClass() != o.getClass()) { <END> return false; } AdHocActivationCondition that = (AdHocActivationCondition) o; return Objects.equals(value, that.value); }</code><technical_language>like: java if (o instanceof AdHocActivationCondition) { AdHocActivationCondition = (AdHocActivationCondition) o; return Objects.equals(value, that.value); } return false; instanceof operator check null well</technical_language>

Target:
public boolean equals(Object o) { if (this == o) { return true; } if (o instanceof AdHocActivationCondition) { AdHocActivationCondition that = (AdHocActivationCondition) o; return Objects.equals(value, that.value); } return false; }
Source:
<code><START> public int processLocator(int rollCount, RollupExecutionContext executionContext, <END> RollupBatchWriter rollupBatchWriter, Locator locator) { if (log.isTraceEnabled()) log.trace("Rolling up (check,metric,dimension) for (gran,slot,shard) {}", locator, parentSlotKey); try { executeRollupForLocator(executionContext, rollupBatchWriter, locator); rollCount += 1; } catch (Throwable any) { executionContext.markUnsuccessful(any); executionContext.decrementReadCounter(); log.error(any.getMessage(), any); log.error("BasicRollup failed for at {}", parentSlotKey, serverTime); } return rollCount; }</code><technical_language>Do want expose this out caller? I.e: allowing skip Instrumentation? thinking want to. protected (test classes call)</technical_language>

Target:
protected int processLocator(int rollCount, RollupExecutionContext executionContext, RollupBatchWriter rollupBatchWriter, Locator locator) { if (log.isTraceEnabled()) log.trace("Rolling up (check,metric,dimension) for (gran,slot,shard) {}", locator, parentSlotKey); try { executeRollupForLocator(executionContext, rollupBatchWriter, locator); rollCount += 1; } catch (Throwable any) { executionContext.markUnsuccessful(any); executionContext.decrementReadCounter(); log.error(any.getMessage(), any); log.error("BasicRollup failed for at {}", parentSlotKey, serverTime); } return rollCount; }
Source:
<code>public SpeechToTextManager(Context context, Listener listener) { this.listener = listener; speechRecognizer = SpeechRecognizer.createSpeechRecognizer(context); speechRecognizerIntent = new Intent(RecognizerIntent.ACTION_RECOGNIZE_SPEECH); <START> speechRecognizerIntent.putExtra(RecognizerIntent.EXTRA_LANGUAGE_MODEL, RecognizerIntent.LANGUAGE_MODEL_WEB_SEARCH); <END> speechRecognizerIntent.putExtra(RecognizerIntent.EXTRA_LANGUAGE, "en"); speechRecognizerIntent.putExtra(RecognizerIntent.EXTRA_PARTIAL_RESULTS, true); speechRecognizerIntent.putExtra(RecognizerIntent.EXTRA_CALLING_PACKAGE, context.getPackageName()); speechRecognizer.setRecognitionListener(this); dialog = new MaterialDialog.Builder(context) .customView(R.layout.voice_recognizble_layout, false) .build(); ButterKnife.bind(this, dialog); }</code><technical_language>a web search, this is better? suggestion speechRecognizerIntent.putExtra(RecognizerIntent.EXTRA_LANGUAGE_MODEL, RecognizerIntent.LANGUAGE_MODEL_FREE_FORM);</technical_language>

Target:
public SpeechToTextManager(Context context, Listener listener) { this.listener = listener; speechRecognizer = SpeechRecognizer.createSpeechRecognizer(context); speechRecognizerIntent = new Intent(RecognizerIntent.ACTION_RECOGNIZE_SPEECH); speechRecognizerIntent.putExtra(RecognizerIntent.EXTRA_LANGUAGE_MODEL, RecognizerIntent.LANGUAGE_MODEL_FREE_FORM); speechRecognizerIntent.putExtra(RecognizerIntent.EXTRA_LANGUAGE, Locale.getDefault().getLanguage()); speechRecognizer.setRecognitionListener(this); dialog = new MaterialDialog.Builder(context) .customView(R.layout.voice_recognizble_layout, false) .build(); dialog.getWindow().getAttributes().windowAnimations = R.style.speech_dialog_animation; ButterKnife.bind(this, dialog); }
Source:
<code>public final ImageWorker ignoreColorMap() throws IOException { if (image == null) { throw new IOException("image is null"); } final ColorModel cm = image.getColorModel(); if (cm == null) { throw new IOException("image has null color model"); } SampleModel sm = image.getSampleModel(); if (sm == null) { throw new IOException("image has null sample model"); } if (cm instanceof IndexColorModel) { RenderingHints newHints = getRenderingHints(); ImageLayout layout = (ImageLayout) newHints.get(JAI.KEY_IMAGE_LAYOUT); if (layout == null) layout = new ImageLayout2(); final ColorModel newCm = new ComponentColorModel( ColorSpace.getInstance(ColorSpace.CS_GRAY), cm.hasAlpha(), <START> cm.isAlphaPremultiplied(), cm.getTransparency(), cm.getTransferType()); <END> layout.setColorModel(newCm); newHints.put(JAI.KEY_IMAGE_LAYOUT, layout); image = FormatDescriptor.create(image, sm.getDataType(), newHints); } return this; }</code><technical_language>I more careful this initialization. final image result in 1 band transparecy. This code preserve transparency of input image is wrong if colormap add transparency mask translucent this reflected in final image</technical_language>

Target:
public final ImageWorker ignoreColorMap() throws IOException { if (image == null) { throw new IOException("image is null"); } final ColorModel cm = image.getColorModel(); if (cm == null) { throw new IOException("image has null color model"); } final SampleModel sm = image.getSampleModel(); if (sm == null) { throw new IOException("image has null sample model"); } if (cm instanceof IndexColorModel) { final RenderingHints newHints = getRenderingHints(); final ImageLayout layout = new ImageLayout2(); final ColorModel newCm = new ComponentColorModel( ColorSpace.getInstance(ColorSpace.CS_GRAY), false, false, Transparency.OPAQUE, cm.getTransferType()); layout.setColorModel(newCm); newHints.put(JAI.KEY_IMAGE_LAYOUT, layout); image = FormatDescriptor.create(image, sm.getDataType(), newHints); } return this; }
Source:
<code>PermissionManager(final SyncUser user) { this.user = user; threadId = Thread.currentThread().getId(); managementRealmConfig = new SyncConfiguration.Builder( user, getRealmUrl(RealmType.MANAGEMENT_REALM, user.getAuthenticationUrl())) .errorHandler(new SyncSession.ErrorHandler() { @Override public void onError(SyncSession session, ObjectServerError error) { synchronized (errorLock) { <START> managementRealmError = error; <END> } } }) .modules(new ManagementModule()) .build(); permissionRealmConfig = new SyncConfiguration.Builder( user, getRealmUrl(RealmType.PERMISSION_REALM, user.getAuthenticationUrl())) .errorHandler(new SyncSession.ErrorHandler() { @Override public void onError(SyncSession session, ObjectServerError error) { synchronized (errorLock) { permissionRealmError = error; } } }) .modules(new PermissionModule()) .waitForInitialRemoteData() .readOnly() .build(); }</code><technical_language>user do errors</technical_language>

Target:
private PermissionManager(final SyncUser user) { this.user = user; threadId = Thread.currentThread().getId(); managementRealmConfig = new SyncConfiguration.Builder( user, getRealmUrl(RealmType.MANAGEMENT_REALM, user.getAuthenticationUrl())) .errorHandler(new SyncSession.ErrorHandler() { @Override public void onError(SyncSession session, ObjectServerError error) { synchronized (errorLock) { managementRealmError = error; } } }) .modules(new ManagementModule()) .build(); permissionRealmConfig = new SyncConfiguration.Builder( user, getRealmUrl(RealmType.PERMISSION_REALM, user.getAuthenticationUrl())) .errorHandler(new SyncSession.ErrorHandler() { @Override public void onError(SyncSession session, ObjectServerError error) { synchronized (errorLock) { permissionRealmError = error; } } }) .modules(new PermissionModule()) .waitForInitialRemoteData() .build(); }
Source:
<code>public boolean setPreferredNetworkType(int subId, int networkType) { enforceModifyPermissionOrCarrierPrivilege(subId); if (DBG) log("setPreferredNetworkType: subId " + subId + " type " + networkType); Boolean success = (Boolean) sendRequest(CMD_SET_PREFERRED_NETWORK_TYPE, networkType, subId); if (DBG) log("setPreferredNetworkType: " + (success ? "ok" : "fail")); if (success) { Settings.Global.putInt(mPhone.getContext().getContentResolver(), Settings.Global.PREFERRED_NETWORK_MODE + subId, networkType); <START> TelephonyManager.putIntAtIndex(mPhone.getContext().getContentResolver(), android.provider.Settings.Global.PREFERRED_NETWORK_MODE, mSubscriptionController.getPhoneId(subId), networkType); <END> } return success; }</code><technical_language>Agree, remove this</technical_language>

Target:
public boolean setPreferredNetworkType(int subId, int networkType) { TelephonyPermissions.enforceCallingOrSelfModifyPermissionOrCarrierPrivilege( mApp, subId, "setPreferredNetworkType"); if (DBG) log("setPreferredNetworkType: subId " + subId + " type " + networkType); Boolean success = (Boolean) sendRequest(CMD_SET_PREFERRED_NETWORK_TYPE, networkType, subId); if (DBG) log("setPreferredNetworkType: " + (success ? "ok" : "fail")); if (success) { Settings.Global.putInt(mPhone.getContext().getContentResolver(), Settings.Global.PREFERRED_NETWORK_MODE + subId, networkType); } return success; }
Source:
<code>public String toString() { StringBuilder sb = new StringBuilder(); sb.append("{"); if (getSpec() != null) { <START> sb.append("Specs: ").append(getSpec()).append(","); <END> } if (mLayout.getData().getValuesList() != null) { sb.append("Values: ").append(mLayout.getData().getValuesList()).append(","); } if (mLayout.getData().getPartitionName() != null) { sb.append("PartitionName: ").append(mLayout.getData().getPartitionName()).append(","); } if (mLayout.getData().getDbName() != null) { sb.append("DatabaseName: ").append(mLayout.getData().getDbName()).append(","); } if (mLayout.getData().getTableName() != null) { sb.append("TableName: ").append(mLayout.getData().getTableName()).append(","); } if (mLayout.getData().getParametersMap() != null) { sb.append("Parameters: ").append(mLayout.getData().getParametersMap()); } sb.append("}"); return sb.toString(); }</code><technical_language>suggestion sb.append("Spec: ").append(getSpec()).append(",");</technical_language>

Target:
public String toString() { return MoreObjects.toStringHelper(this) .add("Spec", getSpec()) .add("Values", mLayout.getData().getValuesList()) .add("PartitionName", mLayout.getData().getPartitionName()) .add("DatabaseName", mLayout.getData().getDbName()) .add("TableName", mLayout.getData().getTableName()) .add("Parameters", mLayout.getData().getParametersMap()) .toString(); }
Source:
<code>void writeOffsetFiles() { storeToSSps.entrySet().stream() .filter(entry -> isPersistedStore(entry.getKey())) .forEach((entry) -> { String storeName = entry.getKey(); Map<SystemStreamPartition, String> offsets = entry.getValue().stream() .filter(lastProcessedOffsets::containsKey) .collect(Collectors.toMap(Function.identity(), lastProcessedOffsets::get)); try { <START> String fileContents = OBJECT_MAPPER.writerWithType( <END> TypeFactory.defaultInstance().constructMapType( HashMap.class, SystemStreamPartition.class, String.class)).writeValueAsString(offsets); File offsetFile = new File(getStoreLocation(storeName), OFFSET_FILE); FileUtil.writeWithChecksum(offsetFile, fileContents); } catch (Exception e) { throw new SamzaException("Failed to write offset file for side input store: " + storeName, e); } }); }</code><technical_language>1. Is writer required if SamzaObjectMapper? if SamzaObjectMapper directly? 2. save ObjectWriter a static field</technical_language>

Target:
void writeOffsetFiles() { storeToSSps.entrySet().stream() .filter(entry -> isPersistedStore(entry.getKey())) .forEach((entry) -> { String storeName = entry.getKey(); Map<SystemStreamPartition, String> offsets = entry.getValue().stream() .filter(lastProcessedOffsets::containsKey) .collect(Collectors.toMap(Function.identity(), lastProcessedOffsets::get)); try { String fileContents = OBJECT_WRITER.writeValueAsString(offsets); File offsetFile = new File(getStoreLocation(storeName), OFFSET_FILE); FileUtil.writeWithChecksum(offsetFile, fileContents); } catch (Exception e) { throw new SamzaException("Failed to write offset file for side input store: " + storeName, e); } }); }
Source:
<code>private void logReturnedToken(final AuthenticationResult result) { if (result != null && result.getAccessToken() != null) { String accessTokenHash = getTokenHash(result.getAccessToken()); String refreshTokenHash = getTokenHash(result.getRefreshToken()); <START> Logger.i(TAG, "Access tokenID and refresh tokenID returned. ", String.format( <END> "Access TokenID: %s and Refresh TokenID: %s .", accessTokenHash, refreshTokenHash)); } }</code><technical_language>delete this function, log tokens</technical_language>

Target:
private void logReturnedToken(final AuthenticationResult result) { if (result != null && result.getAccessToken() != null) { Logger.i(TAG, "Access tokenID and refresh tokenID returned. ", null); } }
Source:
<code>private Map<String, String> getFormattingProperties(String rawFormat, List<String> positionalParameters) { <START> Map<String, String> properties = new HashMap<String, String>(); <END> String[] valpairs = rawFormat.split(","); for (String pair : valpairs) { Matcher matcher = keyValuePattern.matcher(pair.trim()); String key, value; if (matcher.find()) { key = matcher.group(1); value = matcher.group(2); properties.put(key, value); } else { if (positionalParameters.isEmpty()) { } else { properties.put(positionalParameters.remove(0), pair.trim()); } } } return properties; }</code><technical_language>diamond operator</technical_language>

Target:
private Map<String, String> getFormattingProperties(String rawFormat, List<String> positionalParameters) { Map<String, String> properties = new HashMap<>(); String[] valpairs = rawFormat.split(","); for (String pair : valpairs) { Matcher matcher = keyValuePattern.matcher(pair.trim()); String key, value; if (matcher.find()) { key = matcher.group(1); value = matcher.group(2); properties.put(key, value); } else { if (positionalParameters.isEmpty()) { } else { properties.put(positionalParameters.remove(0), pair.trim()); } } } return properties; }
Source:
<code>private static Expression.FieldAccess buildFieldAccess(IdentifierTree identifierTree) { if (!identifierTree.symbol().isVariableSymbol() || identifierTree.symbol().owner().isMethodSymbol() || identifierTree.name().equals("this")) { return null; } <START> Expression.Variable rightSide = variableWithId(identifierTree.name()); <END> Expression.FieldAccess fieldAccess; if (identifierTree.symbol().isStatic()) { fieldAccess = new Expression.FieldAccess( new Expression.ClassName(identifierTree.symbol().owner().type().fullyQualifiedName()), rightSide); } else { fieldAccess = new Expression.FieldAccess(rightSide); } return fieldAccess; }</code><technical_language>remark Optional correct nullness annotation</technical_language>

Target:
private static Optional<Expression.FieldAccess> buildFieldAccess(IdentifierTree identifierTree) { Symbol identifierTreeSymbol = identifierTree.symbol(); if (!identifierTreeSymbol.isVariableSymbol() || identifierTreeSymbol.owner().isMethodSymbol() || identifierTree.name().equals("this") || identifierTree.name().equals("super")) { return Optional.empty(); } Expression.Variable rightSide = variableWithId(identifierTree.name()); if (identifierTreeSymbol.isStatic()) { return Optional.of(new Expression.FieldAccess( new Expression.ClassName(identifierTreeSymbol.owner().type().fullyQualifiedName()), rightSide)); } return Optional.of(new Expression.FieldAccess(rightSide)); }
Source:
<code><START> public static void assertNeverCalled() { <END> throw new AssertException("Should never been called"); }</code><technical_language>right ... change</technical_language>

Target:
public static <T> T assertNeverCalled() { throw new AssertException("Should never been called"); }
Source:
<code>public PremisEventObject getPremisEventObject(PID pid) { <START> return new PremisEventObject(pid, this, dataLoader, repoObjFactory).validateType(); <END> }</code><technical_language>Hmm, else cacheloader, feel convert this working if possible</technical_language>

Target:
public PremisEventObject getPremisEventObject(PID pid) { RepositoryObject repoObj = getRepositoryObject(pid); if (!(repoObj instanceof PremisEventObject)) { throw new FedoraException("Object with pid " + pid + " is not a premis event"); } return (PremisEventObject) repoObj; }
Source:
<code>private void addRootProjection(SelectionSet selectionSet) { List<Selection> selections = selectionSet.getSelections(); <START> Map<String, Field> rootSelectionField = new HashMap<>(); <END> selections.stream().forEach(rootSelection -> { if (!(rootSelection instanceof Field)) { throw new InvalidEntityBodyException("Entity selection must be a graphQL field."); } String entityName = ((Field) rootSelection).getName(); if (SCHEMA.equals(entityName) || TYPE.equals(entityName)) { return; } if (rootSelectionField.containsKey(entityName)) { Field fields = rootSelectionField.get(entityName); fields.getSelectionSet().getSelections().addAll( ((Field) rootSelection).getSelectionSet().getSelections()); } else { rootSelectionField.put(entityName, (Field) rootSelection); } }); rootSelectionField.entrySet() .stream() .forEach(selectionField -> { String entityName = selectionField.getKey(); Class<?> entityType = entityDictionary.getEntityClass(selectionField.getKey()); if (entityType == null) { throw new InvalidEntityBodyException(String.format("Unknown entity {%s}.", selectionField.getKey())); } rootProjections.put(entityName, createProjection(entityType, selectionField.getValue())); }); }</code><technical_language>rename rootSelectionFields</technical_language>

Target:
private void addRootProjection(SelectionSet selectionSet) { List<Selection> selections = selectionSet.getSelections(); Map<String, Field> rootSelectionFields = new HashMap<>(); selections.stream().forEach(rootSelection -> { if (!(rootSelection instanceof Field)) { throw new InvalidEntityBodyException("Entity selection must be a graphQL field."); } String entityName = ((Field) rootSelection).getName(); if (SCHEMA.equals(entityName) || TYPE.equals(entityName)) { return; } if (rootSelectionFields.containsKey(entityName)) { Field fields = rootSelectionFields.get(entityName); fields.getSelectionSet().getSelections().addAll( ((Field) rootSelection).getSelectionSet().getSelections()); } else { rootSelectionFields.put(entityName, (Field) rootSelection); } }); rootSelectionFields.entrySet() .stream() .forEach(selectionField -> { String entityName = selectionField.getKey(); Class<?> entityType = entityDictionary.getEntityClass(selectionField.getKey()); if (entityType == null) { throw new InvalidEntityBodyException(String.format("Unknown entity {%s}.", selectionField.getKey())); } rootProjections.put(entityName, createProjection(entityType, selectionField.getValue())); }); }
Source:
<code>public void executeMojo() <START> throws MojoExecutionException, MojoExecutionException { <END> if (!canCheckCoverage()) { return; } executeCheck(); }</code><technical_language>Duplicate in throws clause - [fixed in master](<LINK_0></technical_language>

Target:
public void executeMojo() throws MojoExecutionException { if (!canCheckCoverage()) { return; } executeCheck(); }