auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>public String cleanValue(String value) { value = value.replaceAll("\\s", ""); <START> if (value.startsWith("\"") || value.startsWith("\'")) { <END> return value.substring(1, value.length() - 2); } ; return value; }</code><technical_language>if string end a " '? strip character</technical_language>

Target:
public String cleanValue(String value) { value = value.replaceAll("\\s", ""); if ((value.startsWith("\"") && value.endsWith("\""))|| (value.startsWith("\'")) && value.endsWith("\'")) { return value.substring(1, value.length() - 2); } ; return value; }
Source:
<code>public void createControls(Composite parent, TabbedPropertySheetPage tabbedPropertySheetPage) { super.createControls(parent, tabbedPropertySheetPage); eObject = layoutOption; FormData layoutData = (FormData) text.getLayoutData(); layoutData.right = new FormAttachment(96, 0); help = getWidgetFactory().createCLabel(composite, ""); FormData data = new FormData(); data.top = new FormAttachment(text, 0, SWT.TOP); data.left = new FormAttachment(nameLabel); help.setLayoutData(data); help.setImage(getHelpIcon()); help.setToolTipText(getToolTipText()); <START> removeOverrideButton = LayoutOptionPropertiesUtils.createRemoveOptionButton(text, composite, (ViewpointPropertySheetPage) tabbedPropertySheetPage, getWidgetFactory(), layoutOption); <END> }</code><technical_language>a pixels main widget "Remove" button</technical_language>

Target:
public void createControls(Composite parent, TabbedPropertySheetPage tabbedPropertySheetPage) { super.createControls(parent, tabbedPropertySheetPage); eObject = layoutOption; FormData layoutData = (FormData) text.getLayoutData(); layoutData.right = new FormAttachment(95, 10); help = getWidgetFactory().createCLabel(composite, ""); FormData data = new FormData(); data.top = new FormAttachment(text, 0, SWT.TOP); data.left = new FormAttachment(nameLabel); help.setLayoutData(data); help.setImage(getHelpIcon()); help.setToolTipText(getToolTipText()); removeOverrideButton = LayoutOptionPropertiesUtils.createRemoveOptionButton(text, composite, (ViewpointPropertySheetPage) tabbedPropertySheetPage, getWidgetFactory(), layoutOption); }
Source:
<code>public void onClick(Notification notification) { if (!notification.getOptions().isOngoing()) { String event = notification.isRepeating() ? "clear" : "cancel"; if (actionIdentifier != null) { LocalNotification.fireEvent("action", notification, actionIdentifier); <START> if (event == "clear") { <END> notification.clear(); } else { notification.cancel(); } } else { LocalNotification.fireEvent("click", notification); } LocalNotification.fireEvent(event, notification); } super.onClick(notification); }</code><technical_language>event.equals("clear")</technical_language>

Target:
public void onClick(Notification notification) { LocalNotification.fireEvent("click", notification, actionIdentifier); super.onClick(notification); if (notification.getOptions().isOngoing()) return; String event = notification.isRepeating() ? "clear" : "cancel"; LocalNotification.fireEvent(event, notification); }
Source:
<code>public void doFilter(ServletRequest req, ServletResponse resp, FilterChain filterChain) throws IOException, ServletException { <START> log.info("Entering MaxInactiveFilter"); <END> final HttpServletRequest request = (HttpServletRequest) req; checkMaxInactive(request); log.info("Continuing on to other filters from MaxInactiveFilter"); filterChain.doFilter(req, resp); }</code><technical_language>this line 69 log.debug</technical_language>

Target:
public void doFilter(ServletRequest req, ServletResponse resp, FilterChain filterChain) throws IOException, ServletException { log.debug("Entering MaxInactiveFilter"); final HttpServletRequest request = (HttpServletRequest) req; checkMaxInactive(request); log.debug("Continuing on to other filters from MaxInactiveFilter"); filterChain.doFilter(req, resp); }
Source:
<code>public SpinnerAdapter(final Context context, final int textViewResourceId, final String[] objects, int textUnit, float textSize) { super(context, textViewResourceId, objects); this.context = context; this.textUnit = textUnit; this.textSize = textSize; items = new String[objects.length+1]; items[0] = ""; for(int i = 1; i < items.length; i ++){ items[i] = objects[i-1]; <START> } <END> }</code><technical_language>Linter do System.arraycopy(objects, 0, items, 1, items.length - 1);. built-in copy methods is faster a manual copying loops</technical_language>

Target:
public SpinnerAdapter(final Context context, final int textViewResourceId, final String[] objects, int textUnit, float textSize) { super(context, textViewResourceId, objects); this.context = context; this.textUnit = textUnit; this.textSize = textSize; items = new String[objects.length+1]; items[0] = ""; System.arraycopy(objects, 0, items, 1, items.length - 1); }
Source:
<code>protected void autoFillNameTextField() { <START> setAccountNameWithString(mobileNrInputTextField.getText().trim()); <END> }</code><technical_language>is necessary anymore other PR merged already</technical_language>

Target:
protected void autoFillNameTextField() { setAccountNameWithString(mobileNrInputTextField.getText()); }
Source:
<code>public Attribute.AttributeMap getAttributes() { <START> return null; <END> }</code><technical_language>is this return null return attributes</technical_language>

Target:
public Attribute.AttributeMap getAttributes() { return attributes; }
Source:
<code>public void render (Batch batch, Texture currentScreenTexture, Texture nextScreenTexture, float percent) { float width = currentScreenTexture.getWidth(); float height = currentScreenTexture.getHeight(); float x = 0; float y = 0; if (interpolation != null) percent = interpolation.apply(percent); Gdx.gl.glClearColor(0.0f, 0.0f, 0.0f, 1.0f); Gdx.gl.glClear(GL20.GL_COLOR_BUFFER_BIT); batch.begin(); float fade = percent * 2; if (fade > 1.0f) { fade = 1.0f - (percent * 2 - 1.0f); Color fadeIn = new Color(Color.WHITE); fadeIn.a = 1.0f - fade; batch.setColor(fadeIn); batch.draw(nextScreenTexture, 0, 0, width / 2, height / 2, nextScreenTexture.getWidth(), nextScreenTexture.getHeight(), 1, 1, 0, 0, 0, nextScreenTexture.getWidth(), nextScreenTexture.getHeight(), false, true); } else { Color fadeOut = new Color(Color.WHITE); fadeOut.a = 1.0f - fade; batch.setColor(fadeOut); batch.draw(currentScreenTexture, 0, 0, width / 2, height / 2, width, height, 1, 1, 0, 0, 0, (int)width, (int)height, false, true); } <START> Color c = new Color(Color.WHITE); <END> c.a = fade; batch.setColor(c); batch.draw(texture, 0, 0, width, height); batch.end(); batch.setColor(Color.WHITE); }</code><technical_language>a temp variable new instances. for rest of this PR</technical_language>

Target:
public void render (Batch batch, Texture currentScreenTexture, Texture nextScreenTexture, float percent) { float width = currentScreenTexture.getWidth(); float height = currentScreenTexture.getHeight(); float x = 0; float y = 0; if (interpolation != null) percent = interpolation.apply(percent); Gdx.gl.glClearColor(0.0f, 0.0f, 0.0f, 1.0f); Gdx.gl.glClear(GL20.GL_COLOR_BUFFER_BIT); batch.begin(); float fade = percent * 2; if (fade > 1.0f) { fade = 1.0f - (percent * 2 - 1.0f); color.a = 1.0f - fade; batch.setColor(color); batch.draw(nextScreenTexture, 0, 0, width / 2, height / 2, nextScreenTexture.getWidth(), nextScreenTexture.getHeight(), 1, 1, 0, 0, 0, nextScreenTexture.getWidth(), nextScreenTexture.getHeight(), false, true); } else { color.a = 1.0f - fade; batch.setColor(color); batch.draw(currentScreenTexture, 0, 0, width / 2, height / 2, width, height, 1, 1, 0, 0, 0, (int)width, (int)height, false, true); } color.a = fade; batch.setColor(color); batch.draw(texture, 0, 0, width, height); batch.end(); batch.setColor(Color.WHITE); }
Source:
<code>private List<String> getSortedListOfTeams() { List<String> teams = new ArrayList<String>(rosterTeamNameEmailTable.keySet()); teams.remove(Const.USER_TEAM_FOR_INSTRUCTOR); Collections.sort(teams); return new ArrayList<String>(teams); <START> } <END></code><technical_language>Code duplication in functions</technical_language>

Target:
private List<String> getSortedListOfTeams() { List<String> teams = new ArrayList<String>(rosterTeamNameMembersTable.keySet()); teams.remove(Const.USER_TEAM_FOR_INSTRUCTOR); Collections.sort(teams); return teams; }
Source:
<code>public PerformanceReportEntity aggregateNumberAndValueOfPayments() { <START> long[] results = (long[]) entityManager.get() <END> .createNativeQuery("SELECT COUNT(c.amount) as total_volume, SUM(c.amount) as total_amount, AVG(c.amount) as average_amount FROM ChargeEntity c WHERE c.status = :status") .setParameter("status", CAPTURED) .setMaxResults(1) .getSingleResult(); long totalVolume = results[0]; long totalAmount = results[1]; long averageAmount = results[2]; return new PerformanceReportEntity( totalVolume, totalAmount, averageAmount ); }</code><technical_language>Unfortunately, this cast work. if did, returned columns longs (the AVG ends a double). rudimentary knowledge of JPA, mapping results of a query a POJO involving entity a bunch of XML is a bit of a pain. a native query more so. However, query is capabilities of JPQL, plain createQuery. this allow get a PerformanceReportEntity object out: SELECT NEW uk.gov.pay.connector.model.domain.report.PerformanceReportEntity(COUNT(c.amount), SUM(c.amount), AVG(c.amount)) ChargeEntity c c.status = :status a grasp of JPA able offer a suggestion</technical_language>

Target:
public PerformanceReportEntity aggregateNumberAndValueOfPayments() { return (PerformanceReportEntity) entityManager.get() .createQuery("SELECT NEW uk.gov.pay.connector.model.domain.report.PerformanceReportEntity(COUNT(c.amount), SUM(c.amount), AVG(c.amount)) FROM ChargeEntity c WHERE c.status = :status") .setParameter("status", CAPTURED.toString()) .setMaxResults(1) .getSingleResult(); }
Source:
<code>public void replaceInvalidEmptyStringMacAddressesWithNull(List<VmNetworkInterface> vmNetworkInterfaces) { <START> for (VmNetworkInterface vmNetworkInterface : vmNetworkInterfaces) { String macAddress = vmNetworkInterface.getMacAddress(); if (macAddress != null && macAddress.isEmpty()) { vmNetworkInterface.setMacAddress(null); } } } <END></code><technical_language>prefer streams</technical_language>

Target:
public void replaceInvalidEmptyStringMacAddressesWithNull(List<VmNetworkInterface> vmNetworkInterfaces) { vmNetworkInterfaces.stream() .filter(this::hasEmptyStringMacAddress) .forEach(e->e.setMacAddress(null)); }
Source:
<code>public void finish() { closeCurrentFile(); String fileName = getOutputFilePath(); String temporaryFileName = fileName + temporaryFileSuffix; sftpUtils.uploadFile(tempFile, temporaryFileName); Map<String, String> executedFiles = new HashMap<>(); <START> executedFiles.put("temporary_filename", fileName + temporaryFileSuffix); <END> executedFiles.put("real_filename", fileName); fileList.add(executedFiles); fileIndex++; }</code><technical_language>Minor: temporaryFileName :)</technical_language>

Target:
public void finish() { closeCurrentFile(); String fileName = getOutputFilePath(); String temporaryFileName = fileName + temporaryFileSuffix; sftpUtils.uploadFile(tempFile, temporaryFileName); Map<String, String> executedFiles = new HashMap<>(); executedFiles.put("temporary_filename", temporaryFileName); executedFiles.put("real_filename", fileName); fileList.add(executedFiles); fileIndex++; }
Source:
<code>public void put(long key, Chain chain) { try { segmentFor(key).put(key, chain); } catch (OversizeMappingException e) { try { handleOversizeMappingException(key, (long k) -> segmentFor(k).put(k, chain)); } catch (OversizeMappingException e1) { segmentFor(key).remove(key); <START> throw e; <END> } } }</code><technical_language>easier do: java try { try { segmentFor(key).put(key, chain); } catch (OversizeMappingException e) { handleOversizeMappingException(key, (long k) -> segmentFor(k).put(k, chain)); } } catch (Throwable t) { segmentFor(key).remove(key); throw t; } general idea right... fact trap OSME harmless out is independent thing. general idea is "something wrong putting" happened, safest do is invalidate</technical_language>

Target:
public void put(long key, Chain chain) { try { try { segmentFor(key).put(key, chain); } catch (OversizeMappingException e) { handleOversizeMappingException(key, (long k) -> segmentFor(k).put(k, chain)); } } catch (Throwable t) { segmentFor(key).remove(key); throw t; } }
Source:
<code>public void afterRefreshTreatment(boolean succeeded) { if (!succeeded) { lockManager.releaseLock(monitoringLock); return; } try { Guid storagePoolId = null; <START> ArrayList<VDSDomainsData> <END> domainsList = null; synchronized (this) { unrespondedAttempts.set(0); setLastUpdate(); try { hostMonitoring.afterRefreshTreatment(); if (cachedVds != null && cachedVds.getStatus() != VDSStatus.Maintenance) { storagePoolId = cachedVds.getStoragePoolId(); domainsList = cachedVds.getDomains(); } hostMonitoring = null; } catch (IRSErrorException ex) { logAfterRefreshFailureMessage(ex); if (log.isDebugEnabled()) { logException(ex); } } catch (RuntimeException ex) { logAfterRefreshFailureMessage(ex); logException(ex); } } if (domainsList != null) { updateVdsDomainsData(cachedVds, storagePoolId, domainsList); setDomains(cachedVds.getDomains()); } } catch (Exception e) { log.error("Timer update runtime info failed. Exception:", ExceptionUtils.getRootCauseMessage(e)); log.debug("Exception:", e); } finally { lockManager.releaseLock(monitoringLock); } }</code><technical_language>reason List</technical_language>

Target:
public void afterRefreshTreatment(boolean succeeded) { if (!succeeded) { lockManager.releaseLock(monitoringLock); return; } try { Guid storagePoolId = null; ArrayList<VDSDomainsData> domainsList = null; synchronized (this) { unrespondedAttempts.set(0); setLastUpdate(); try { hostMonitoring.afterRefreshTreatment(); if (cachedVds != null && cachedVds.getStatus() != VDSStatus.Maintenance) { storagePoolId = cachedVds.getStoragePoolId(); domainsList = cachedVds.getDomains(); } hostMonitoring = null; } catch (IRSErrorException ex) { logAfterRefreshFailureMessage(ex); if (log.isDebugEnabled()) { logException(ex); } } catch (RuntimeException ex) { logAfterRefreshFailureMessage(ex); logException(ex); } } if (domainsList != null) { updateVdsDomainsData(cachedVds, storagePoolId, domainsList); setDomains(domainsList); } } catch (Exception e) { log.error("Timer update runtime info failed. Exception:", ExceptionUtils.getRootCauseMessage(e)); log.debug("Exception:", e); } finally { lockManager.releaseLock(monitoringLock); } }
Source:
<code>public long saveRun(Run run) throws IOException { if (runStore != null) { long id; if (run.getId() == AbstractRun.UNSAVED_ID) { runStore.save(run); try { String name = namingScheme.generateNameFor(run); run.setName(name); validateNameOrThrow(run, namingScheme); Number newId = (long) runStore.save(run); <START> if (newId.longValue() != run.getId()) { <END> log.error("Expected Run ID doesn't match returned value from database insert: rolling back..."); runStore.remove(run); throw new IOException("Something bad happened. Expected Run ID doesn't match returned value from DB insert"); } else { id = run.getId(); } } catch (MisoNamingException e) { throw new IOException("Cannot save Run - issue with generating name"); } } else { runStore.save(run); id = run.getId(); } return id; } else { throw new IOException("No runStore available. Check that it has been declared in the Spring config."); } }</code><technical_language>equal run.id changed. check is unnecessary though. A similar check previously generated name based a predicted ID Run saved. is saved first, ID is solid. (L1535) changed run.setId(runStore.save(run));</technical_language>

Target:
public long saveRun(Run run) throws IOException { if (runStore != null) { long id; if (run.getId() == AbstractRun.UNSAVED_ID) { run.setName(generateTemporaryName()); run.setId(runStore.save(run)); try { String name = namingScheme.generateNameFor(run); run.setName(name); validateNameOrThrow(run, namingScheme); id = runStore.save(run); } catch (MisoNamingException e) { throw new IOException("Cannot save Run - issue with generating name"); } } else { runStore.save(run); id = run.getId(); } return id; } else { throw new IOException("No runStore available. Check that it has been declared in the Spring config."); } }
Source:
<code>public boolean equals(Object obj) { if (this == obj) return true; if (obj == null) return false; if (getClass() != obj.getClass()) <START> return false; <END> CommitMessageWithCaretPosition other = (CommitMessageWithCaretPosition) obj; if (desiredCaretPosition != other.desiredCaretPosition) return false; if (message == null) { if (other.message != null) return false; } else if (!message.equals(other.message)) return false; return true; }</code><technical_language>Please supply missing curly braces</technical_language>

Target:
public boolean equals(Object obj) { if (this == obj) { return true; } if (obj == null) { return false; } if (getClass() != obj.getClass()) { return false; } CommitMessageWithCaretPosition other = (CommitMessageWithCaretPosition) obj; return desiredCaretPosition == other.desiredCaretPosition && Objects.equals(message, other.message); }
Source:
<code>public void testUploadProject() throws ProjectManagerException { System.out.println("projectManagerTest: testUploadProject"); ProjectManager manager = createProjectManager(); String projectName = "myTestUploadProject"; String projectDescription = "This is to test uploading a project"; User user = new User("testUser"); Project project = createProjectUtil(manager, projectName, projectDescription, user); <START> File testDir = new File("src/test/resources/project/testjob/testjob.zip"); <END> System.out.println("Uploading zip file: " + testDir.getAbsolutePath()); Props props = new Props(); manager.uploadProject(project, testDir, "zip", user, props); System.out.println("Uploaded project successfully"); }</code><technical_language>Alternatively, consider this method: <LINK_0></technical_language>

Target:
public void testUploadProject() throws Exception { System.out.println("TestUploadProject"); Project project = manager.createProject(PROJECT_NAME, PROJECT_DESCRIPTION, user); File testDir = new File(this.getClass().getClassLoader().getResource("project/testjob/testjob.zip").getFile()); System.out.println("Uploading zip file: " + testDir.getAbsolutePath()); Props props = new Props(); manager.uploadProject(project, testDir, FILE_TYPE, user, props); verify(loader).uploadProjectFile(project, PROJECT_VERSION + 1, FILE_TYPE, testDir.getName(), testDir, user.getUserId()); verify(loader).uploadFlows(eq(project), eq(PROJECT_VERSION + 1), anyCollection()); verify(loader).changeProjectVersion(project, PROJECT_VERSION + 1, user.getUserId()); verify(loader, times(2)).uploadProjectProperties(eq(project), anyList()); verify(loader).postEvent(project, ProjectLogEvent.EventType.UPLOADED, user.getUserId(), "Uploaded project files zip " + testDir.getName()); verify(loader).cleanOlderProjectVersion(project.getId(), PROJECT_VERSION + 1 - PROJECT_VERSION_RETENTIION); }
Source:
<code>public static Matcher<Watermark> watermark(long timestamp) { <START> return new WatermarkMatcher(timestamp); <END> }</code><technical_language>nit: I in this case easier FeatureMatcher public static Matcher<Watermark> watermark(long timestamp) { return new FeatureMatcher<Watermark, Long>( equalTo(timestamp), "a watermark value", "value of watermark" ) { @Override protected Long featureValueOf(Watermark actual) { return actual.getTimestamp(); } }; }</technical_language>

Target:
public static Matcher<Watermark> watermark(long timestamp) { return new FeatureMatcher<Watermark, Long>( equalTo(timestamp), "a watermark with value", "value of watermark" ) { @Override protected Long featureValueOf(Watermark actual) { return actual.getTimestamp(); } }; }
Source:
<code>public void cancelRefresh() { if(refresh == null) { return; } else { refresh.cancel(); <START> } <END> }</code><technical_language>This simplified if(refresh != null) { refresh.cancel(); }</technical_language>

Target:
public void cancelRefresh() { if(refresh != null) { refresh.cancel(); } }
Source:
<code>private List<CustomAction> customActions() { List<CustomAction> actions = new LinkedList<>(); CustomAction pos = new CustomAction("org.finra.datagenerator", "transform", Transform.class); actions.add(pos); pos = new CustomAction("org.finra.datagenerator", "positive", Assign.class); actions.add(pos); return actions; <START> } <END></code><technical_language>do touch positive engine</technical_language>

Target:
private List<CustomAction> customActions() { List<CustomAction> actions = new LinkedList<>(); CustomAction pos = new CustomAction("org.finra.datagenerator", "transform", Transform.class); actions.add(pos); return actions; }
Source:
<code>public synchronized Swagger read(Set<Class<?>> classes) { <START> listeners.clear(); <END> for(Class<?> cls: classes) { if( ReaderListener.class.isAssignableFrom( ReaderListener.class ) && listeners.containsKey( cls )){ try { listeners.put( cls, (ReaderListener) cls.newInstance()); } catch (Exception e) { LOGGER.error("Failed to create ReaderListener", e); } } } for( ReaderListener listener : listeners.values()){ listener.beforeScan( this, swagger ); } for(Class<?> cls: classes) read(cls); for( ReaderListener listener : listeners.values()){ listener.afterScan( this, swagger); } return swagger; }</code><technical_language>I suggest convert listeners local variable reinitialize read</technical_language>

Target:
public Swagger read(Set<Class<?>> classes) { Map<Class<?>,ReaderListener> listeners = new HashMap<Class<?>, ReaderListener>(); for(Class<?> cls: classes) { if( ReaderListener.class.isAssignableFrom( cls ) && !listeners.containsKey( cls )){ try { listeners.put( cls, (ReaderListener) cls.newInstance()); } catch (Exception e) { LOGGER.error("Failed to create ReaderListener", e); } } } for( ReaderListener listener : listeners.values()){ try { listener.beforeScan(this, swagger); } catch( Exception e ){ LOGGER.error( "Unexpected error invoking beforeScan listener [" + listener.getClass().getName() + "]", e ); } } for(Class<?> cls: classes) read(cls); for( ReaderListener listener : listeners.values()){ try { listener.afterScan( this, swagger); } catch( Exception e ){ LOGGER.error( "Unexpected error invoking afterScan listener [" + listener.getClass().getName() + "]", e ); } } return swagger; }
Source:
<code>public void deleteTopicFromRegistry(String topicName, String subscriptionId) throws BrokerManagerAdminException { try { QueueManagerService queueManagerService = AndesBrokerManagerAdminServiceDSHolder.getInstance().getQueueManagerService(); queueManagerService.deleteTopicFromRegistry(topicName, subscriptionId); } catch (QueueManagerException e) { String message = e.getMessage(); throw new BrokerManagerAdminException("Error in deleting topic from registry. " + <START> "" + message, e); <END> } }</code><technical_language>need append empty string :)</technical_language>

Target:
public void deleteTopicFromRegistry(String topicName, String subscriptionId) throws BrokerManagerAdminException { try { QueueManagerService queueManagerService = AndesBrokerManagerAdminServiceDSHolder.getInstance().getQueueManagerService(); queueManagerService.deleteTopicFromRegistry(topicName, subscriptionId); } catch (QueueManagerException e) { String message = e.getMessage(); log.error("Error in deleting topic from registry.", e); throw new BrokerManagerAdminException("Error in deleting topic from registry. " , e); } }
Source:
<code>public String toString() { String classPath = subPaths.get(0).toString(); for (int i = 1; i < subPaths.size(); i++) { classPath += File.pathSeparatorChar + subPaths.get(i).toString(); <START> if (subPaths.get(i).toString().contains("testrunner")) { <END> classPath += File.pathSeparatorChar + subPaths.get(i).toString() + File.separatorChar + "tmc-junit-runner.jar"; } } return classPath; }</code><technical_language>a special case here. Instead extra method add "special" stuff this if necesary</technical_language>

Target:
public String toString() { if (subPaths.isEmpty()) { return ""; } String classPath = subPaths.get(0).toString(); for (int i = 1; i < subPaths.size(); i++) { classPath += File.pathSeparatorChar + subPaths.get(i).toString(); } return classPath; }
Source:
<code>private T getClaimValue(final String name) { final Bean<?> bean = bm.resolve(bm.getBeans(JsonWebToken.class)); if (RequestScoped.class.equals(bean.getScope())) { jsonWebToken = JsonWebToken.class.cast(bm.getReference(bean, JsonWebToken.class, null)); } if (jsonWebToken == null || !bean.getScope().equals(RequestScoped.class)) { logger.warning(String.format("Can't retrieve claim %s. No active principal.", name)); return null; } <START> final Optional<T> claimValue = jsonWebToken.claim(name); <END> logger.finest(String.format("Found ClaimValue=%s for name=%s", claimValue, name)); return claimValue.orElse(null); }</code><technical_language>Java 8</technical_language>

Target:
private T getClaimValue(final String name) { final Bean<?> bean = bm.resolve(bm.getBeans(JsonWebToken.class)); JsonWebToken jsonWebToken = null; if (RequestScoped.class.equals(bean.getScope())) { jsonWebToken = JsonWebToken.class.cast(bm.getReference(bean, JsonWebToken.class, null)); } if (jsonWebToken == null || !bean.getScope().equals(RequestScoped.class)) { logger.warning(String.format("Can't retrieve claim %s. No active principal.", name)); return null; } final Optional<T> claimValue = jsonWebToken.claim(name); logger.finest(String.format("Found ClaimValue=%s for name=%s", claimValue, name)); return claimValue.orElse(null); }
Source:
<code>private IASTTranslationUnit getAST() { if (fTranslationUnit != null) return fTranslationUnit; IWorkbenchWindow window = PlatformUI.getWorkbench().getActiveWorkbenchWindow(); if (window == null) return null; IWorkbenchPage page = window.getActivePage(); if (page == null) return null; IEditorPart editor = page.getActiveEditor(); if (editor == null) return null; IWorkingCopyManager manager = CUIPlugin.getDefault().getWorkingCopyManager(); ITranslationUnit unit = manager.getWorkingCopy(editor.getEditorInput()); if (unit == null) return null; try { fTranslationUnit = <START> unit.getAST(CCorePlugin.getIndexManager().getIndex(unit.getCProject()), <END> ITranslationUnit.AST_SKIP_ALL_HEADERS); return fTranslationUnit; } catch (CoreException e) { CUIPlugin.log(e); } return null; }</code><technical_language>safely index-based AST, index read lock held for lifetime of AST</technical_language>

Target:
private IASTTranslationUnit getAST() { if (fTranslationUnit != null) return fTranslationUnit; IWorkbenchWindow window = PlatformUI.getWorkbench().getActiveWorkbenchWindow(); if (window == null) return null; IWorkbenchPage page = window.getActivePage(); if (page == null) return null; IEditorPart editor = page.getActiveEditor(); if (editor == null) return null; IWorkingCopyManager manager = CUIPlugin.getDefault().getWorkingCopyManager(); ITranslationUnit unit = manager.getWorkingCopy(editor.getEditorInput()); if (unit == null) return null; try { fTranslationUnit = unit.getAST(CCorePlugin.getIndexManager().getIndex(unit.getCProject(), IIndexManager.ADD_DEPENDENCIES), ITranslationUnit.AST_SKIP_ALL_HEADERS); return fTranslationUnit; } catch (CoreException e) { CUIPlugin.log(e); } return null; }
Source:
<code>public static int getValue(String text) { if (text == null || text.isEmpty() || text.equals("build")) { return BUILD; } if (text.equals("codan")) { return CODAN; } <START> if (text.equals("build,codan")) { END> return BUILD_AND_CODAN; } throw new IllegalArgumentException("Unknown context value: " + text); }</code><technical_language>This is apparently out of date</technical_language>

Target:
public static int getValue(String text) { if (text == null || text.isEmpty() || text.equals("build")) { return BUILD; } if (text.equals("codan")) { return CODAN; } throw new IllegalArgumentException("Unknown context value: " + text); }
Source:
<code>private boolean response429ButExceptionIsNotRetryable(Response response, Exception exception) { return (response.status() == 429) && !isExceptionIsRetryable(exception); <START> } <END></code><technical_language>sense parameterise error code hoping match? feel strongly this, in case</technical_language>

Target:
private boolean response429ButExceptionIsNotRetryable(Response response, Exception exception) { return (response.status() == 429) && !isExceptionRetryable(exception); }
Source:
<code>protected static JSIBackgroundDataType getBackgroundData(BackgroundData source) { JSIBackgroundDataType toReturn = new JSIBackgroundDataType(); JSIFactMappingValuesType factMappingValuesType = new JSIFactMappingValuesType(); toReturn.setFactMappingValues(factMappingValuesType); final List<FactMappingValue> unmodifiableFactMappingValues = source.getUnmodifiableFactMappingValues(); List<JSIFactMappingValueType> toSet = new ArrayList<>(); <START> populateJSIFactMappingValuesType(toSet, unmodifiableFactMappingValues); <END> factMappingValuesType.setFactMappingValue(toSet); return toReturn; }</code><technical_language>similar question populate</technical_language>

Target:
protected static JSIBackgroundDataType getBackgroundData(BackgroundData source) { JSIBackgroundDataType toReturn = new JSIBackgroundDataType(); JSIFactMappingValuesType factMappingValuesType = new JSIFactMappingValuesType(); toReturn.setFactMappingValues(factMappingValuesType); final List<FactMappingValue> unmodifiableFactMappingValues = source.getUnmodifiableFactMappingValues(); List<JSIFactMappingValueType> toSet = populateJSIFactMappingValuesType(unmodifiableFactMappingValues); factMappingValuesType.setFactMappingValue(toSet); return toReturn; }
Source:
<code>void scheduleMessageWithTransaction() { final ServiceBusMessage testData = new ServiceBusMessage(TEST_CONTENTS.getBytes(UTF_8)); final Instant scheduledEnqueueTime = Instant.now(); final long sequenceNumber = 1; when(asyncSender.scheduleMessage(testData, scheduledEnqueueTime, transactionContext)).thenReturn(Mono.just(sequenceNumber)); sender.scheduleMessage(testData, scheduledEnqueueTime, transactionContext); verify(asyncSender, times(1)).scheduleMessage(testData, scheduledEnqueueTime, transactionContext); <START> verify(asyncSender).scheduleMessage(singleMessageCaptor.capture(), scheduleMessageCaptor.capture(), transactionCaptor.capture()); <END> final ServiceBusMessage message = singleMessageCaptor.getValue(); Assertions.assertArrayEquals(testData.getBody(), message.getBody()); final Instant scheduledEnqueueTimeActual = scheduleMessageCaptor.getValue(); Assertions.assertEquals(scheduledEnqueueTime, scheduledEnqueueTimeActual); Assertions.assertEquals(transactionContext, transactionCaptor.getValue()); }</code><technical_language>Do need capture it? 336 returns parameters equal</technical_language>

Target:
void scheduleMessageWithTransaction() { final ServiceBusMessage testData = new ServiceBusMessage(TEST_CONTENTS.getBytes(UTF_8)); final Instant scheduledEnqueueTime = Instant.now(); final long expected = 1; when(asyncSender.scheduleMessage(testData, scheduledEnqueueTime, transactionContext)).thenReturn(Mono.just(expected)); long actual = sender.scheduleMessage(testData, scheduledEnqueueTime, transactionContext); Assertions.assertEquals(expected, actual); verify(asyncSender).scheduleMessage(testData, scheduledEnqueueTime, transactionContext); }
Source:
<code>public void testCatchErrorOnEmbeddedSubprocessWithoutErrorCode() { try { testCatchErrorOnEmbeddedSubprocess(); <START> } catch (ActivitiException re) { <END> } }</code><technical_language>If suppose throw exception need call fail method migrate catchThrowable</technical_language>

Target:
public void testCatchErrorOnEmbeddedSubprocessWithoutErrorCode() { Throwable throwable = catchThrowable(() -> repositoryService .createDeployment() .addClasspathResource("org/activiti/engine/test/bpmn/event/error/BoundaryErrorEventTest.testCatchErrorOnEmbeddedSubprocessWithoutErrorCode.bpmn20.xml") .deploy()); assertThat(throwable).isInstanceOf(ActivitiException.class) .hasMessageContaining("activiti-error-missing-error-code"); }
Source:
<code>public void build() { class TestItem { } final PutResolver<TestItem> putResolver = mock(PutResolver.class); final GetResolver<TestItem> getResolver = mock(GetResolver.class); final DeleteResolver<TestItem> deleteResolver = mock(DeleteResolver.class); final SQLiteTypeMapping<TestItem> typeMapping = SQLiteTypeMapping.<TestItem>builder() .putResolver(putResolver) .getResolver(getResolver) .deleteResolver(deleteResolver) .build(); <START> assertThat(typeMapping.putResolver()).isEqualTo(putResolver); <END> assertThat(typeMapping.getResolver()).isEqualTo(getResolver); assertThat(typeMapping.deleteResolver()).isEqualTo(deleteResolver); }</code><technical_language>isSame below, mistake</technical_language>

Target:
public void build() { class TestItem { } final PutResolver<TestItem> putResolver = mock(PutResolver.class); final GetResolver<TestItem> getResolver = mock(GetResolver.class); final DeleteResolver<TestItem> deleteResolver = mock(DeleteResolver.class); final SQLiteTypeMapping<TestItem> typeMapping = SQLiteTypeMapping.<TestItem>builder() .putResolver(putResolver) .getResolver(getResolver) .deleteResolver(deleteResolver) .build(); assertThat(typeMapping.putResolver()).isSameAs(putResolver); assertThat(typeMapping.getResolver()).isSameAs(getResolver); assertThat(typeMapping.deleteResolver()).isSameAs(deleteResolver); }
Source:
<code>public BatchSizeIncreasingIterator(BatchProvider<T> batchProvider, int originalBatchSize, @Nullable ClosableIterator<T> currentResults) { Preconditions.checkArgument(originalBatchSize > 0); this.batchProvider = batchProvider; this.originalBatchSize = originalBatchSize; <START> this.currentResults = new ExhaustibleClosableIterator<>(currentResults); <END> if (currentResults != null) { this.lastBatchSize = originalBatchSize; } }</code><technical_language>pretty want set this if currentResults is null (i.e. this line moved if currentResults != null block). If currentResults is null, want stay null block of updateResultsIfNeeded() fire</technical_language>

Target:
public BatchSizeIncreasingIterator(BatchProvider<T> batchProvider, int originalBatchSize, @Nullable ClosableIterator<T> currentResults) { Preconditions.checkArgument(originalBatchSize > 0); this.batchProvider = batchProvider; this.originalBatchSize = originalBatchSize; if (currentResults != null) { this.currentResults = new ExhaustibleClosableIterator<>(currentResults); this.lastBatchSize = originalBatchSize; } }
Source:
<code>public double calcTurnWeight(int edgeFrom, int nodeVia, int edgeTo) { <START> if (edgeFrom == EdgeIterator.NO_EDGE || edgeTo == EdgeIterator.NO_EDGE) { <END> return 0; } if (turnCostExt.isUTurn(edgeFrom, edgeTo)) { return uTurnCost; } long turnFlags = turnCostExt.getTurnCostFlags(edgeFrom, nodeVia, edgeTo); if (turnCostEncoder.isTurnRestricted(turnFlags)) return Double.POSITIVE_INFINITY; return turnCostEncoder.getTurnCost(turnFlags); }</code><technical_language>Do EdgeIterator.isValid is more clear (and in calcMillis)</technical_language>

Target:
public double calcTurnWeight(int edgeFrom, int nodeVia, int edgeTo) { if (!EdgeIterator.Edge.isValid(edgeFrom) || !EdgeIterator.Edge.isValid(edgeTo)) { return 0; } if (turnCostExt.isUTurn(edgeFrom, edgeTo)) { return uTurnCost; } long turnFlags = turnCostExt.getTurnCostFlags(edgeFrom, nodeVia, edgeTo); if (turnCostEncoder.isTurnRestricted(turnFlags)) return Double.POSITIVE_INFINITY; return turnCostEncoder.getTurnCost(turnFlags); }
Source:
<code>void processFiles() { File[] files = storage.listFiles(new FilenameFilter() { public boolean accept(File dir, String name) { return name.endsWith(RaygunOnFailedSendOfflineStorageHandler.fileExtension); } }); if (files == null) { return; } for (File file : files) { InputStream inputStream = null; ByteArrayOutputStream outputStream = null; try { inputStream = getInputStream(file); outputStream = new ByteArrayOutputStream(); byte[] buffer = new byte[(int) file.length()]; int length; while ((length = inputStream.read(buffer)) != -1) { outputStream.write(buffer, 0, length); } inputStream.close(); int reponseCode = client.send(outputStream.toString("UTF-8")); <START> if (reponseCode == 202 || reponseCode == 429 ) { <END> file.delete(); } } catch (IOException e) { Logger.getLogger("Raygun4Java").warning("exception processing offline payload: " + e.getMessage()); return; } finally { if (outputStream != null) { try { outputStream.close(); } catch (IOException e) { Logger.getLogger("Raygun4Java").warning("exception closing outputStream: " + e.getMessage()); } } if (inputStream != null) { try { inputStream.close(); } catch (IOException e) { Logger.getLogger("Raygun4Java").warning("exception closing inputStream: " + e.getMessage()); } } } } }</code><technical_language>I 4xx response treated same. API return a 400, 403, 413 429 code</technical_language>

Target:
void processFiles() { File[] files = storage.listFiles(new FilenameFilter() { public boolean accept(File dir, String name) { return name.endsWith(RaygunOnFailedSendOfflineStorageHandler.fileExtension); } }); if (files == null) { return; } for (File file : files) { InputStream inputStream = null; ByteArrayOutputStream outputStream = null; try { inputStream = getInputStream(file); outputStream = new ByteArrayOutputStream(); byte[] buffer = new byte[(int) file.length()]; int length; while ((length = inputStream.read(buffer)) != -1) { outputStream.write(buffer, 0, length); } inputStream.close(); int reponseCode = client.send(outputStream.toString("UTF-8")); if (deleteOnStatusCodes.contains(reponseCode)) { file.delete(); } } catch (IOException e) { Logger.getLogger("Raygun4Java").warning("exception processing offline payload: " + e.getMessage()); return; } finally { if (outputStream != null) { try { outputStream.close(); } catch (IOException e) { Logger.getLogger("Raygun4Java").warning("exception closing outputStream: " + e.getMessage()); } } if (inputStream != null) { try { inputStream.close(); } catch (IOException e) { Logger.getLogger("Raygun4Java").warning("exception closing inputStream: " + e.getMessage()); } } } } }
Source:
<code>public static Predicate<GHEventsSubscriber> isApplicableFor(final AbstractProject<?, ?> project) { return new Predicate<GHEventsSubscriber>() { @Override public boolean apply(GHEventsSubscriber subscriber) { Validate.notNull(subscriber); <START> return subscriber.isApplicable(project); <END> } }; }</code><technical_language>write notNull(subscriber).isApplicable(project); (Same other places, static imports)</technical_language>

Target:
public static Predicate<GHEventsSubscriber> isApplicableFor(final AbstractProject<?, ?> project) { return new Predicate<GHEventsSubscriber>() { @Override public boolean apply(GHEventsSubscriber subscriber) { return notNull(subscriber).isApplicable(project); } }; }
Source:
<code>public void close() { if (!shuttingDown.getAndSet(true)) { logger.info("CoordinatorBackedRouter closing"); <START> if (!operationPool.isShutdown()) { <END> try { operationPool.shutdown(); operationPool.awaitTermination(1, TimeUnit.MINUTES); coordinator.close(); logger.info("CoordinatorBackedRouter closed"); } catch (IOException e) { logger.error("Error closing Coordinator in CoordinatorBackedRouter", e); } catch (Exception e) { logger.error("Error shutting down operationPool in CoordinatorBackedRouter", e); } } } }</code><technical_language>awaiting termination otherwise, this isTerminated() check do awaitTermination() in else. if understanding ExecutorService interface is wrong</technical_language>

Target:
public void close() throws IOException { try { if (routerOpen.compareAndSet(true, false)) { logger.info("CoordinatorBackedRouter closing"); operationPool.shutdown(); operationPool.awaitTermination(1, TimeUnit.MINUTES); coordinator.close(); logger.info("CoordinatorBackedRouter closed"); } else { operationPool.awaitTermination(1, TimeUnit.MINUTES); } } catch (InterruptedException e) { logger.error("Error shutting down operationPool in CoordinatorBackedRouter", e); } }
Source:
<code>private <START> String renderInput(Options <END> options, String inputFile) { Asciidoctor asciidoctor = JRubyAsciidoctor.create(); return asciidoctor.renderFile(new File(inputFile), options); }</code><technical_language>return void result is</technical_language>

Target:
private void renderInput(Options options, String inputFile) { Asciidoctor asciidoctor = JRubyAsciidoctor.create(); asciidoctor.renderFile(new File(inputFile), options); }
Source:
<code>public StorageFactory storageFactory() { if (environment().isTests()) { <START> if (storageFactoryForTests != null) { <END> return storageFactoryForTests; } this.storageFactoryForTests = InMemoryStorageFactory.newInstance(); return storageFactoryForTests; } checkNotNull(productionStorageFactory, "Production `%s` is not configured." + " Please call `configureStorage()`.", StorageFactory.class.getSimpleName() ); return productionStorageFactory; }</code><technical_language>I java if (storageFactoryForTests == null) { this.storageFactoryForTests = InMemoryStorageFactory.newInstance(); } return storageFactoryForTests; line shorter</technical_language>

Target:
public StorageFactory storageFactory() { if (environment().isTests()) { if (storageFactoryForTests == null) { this.storageFactoryForTests = InMemoryStorageFactory.newInstance(); } return storageFactoryForTests; } checkNotNull(productionStorageFactory, "Production `%s` is not configured." + " Please call `configureStorage()`.", StorageFactory.class.getSimpleName() ); return productionStorageFactory; }
Source:
<code>public void testCommentAndExtraInSameOrder() throws Exception { String comment = makeString(17, "z"); byte[] extra = makeString(11, "a").getBytes(); File f = createTemporaryZipFile(); ZipOutputStream out = createZipOutputStream(f); ZipEntry ze = new ZipEntry("x"); ze.setSize(0); ze.setTime(TEST_TIME); ze.setExtra(extra); ze.setComment(comment); out.putNextEntry(ze); out.closeEntry(); ze = new ZipEntry("y"); <START> ze.setTime(TEST_TIME); <END> ze.setExtra(extra); ze.setComment(comment); out.putNextEntry(ze); out.closeEntry(); out.close(); ZipFile zipFile = new ZipFile(f); try { assertEquals(comment, zipFile.getEntry("x").getComment()); assertTrue(Arrays.equals(extra, zipFile.getEntry("x").getExtra())); assertEquals(comment, zipFile.getEntry("y").getComment()); assertTrue(Arrays.equals(extra, zipFile.getEntry("y").getExtra())); } finally { zipFile.close(); } }</code><technical_language>If removed tests implicitly system clock, a test confirms defaulting behavior if one? removing some coverage. here, in tests for code responsible for handling absence of explicit time fields</technical_language>

Target:
public void testCommentAndExtraInSameOrder() throws Exception { String comment = makeString(17, "z"); byte[] extra = makeString(11, "a").getBytes(); File f = createTemporaryZipFile(); ZipOutputStream out = createZipOutputStream(f); ZipEntry ze = new ZipEntry("x"); ze.setSize(0); ze.setTime(ENTRY_TIME); ze.setExtra(extra); ze.setComment(comment); out.putNextEntry(ze); out.closeEntry(); ze = new ZipEntry("y"); ze.setTime(ENTRY_TIME); ze.setExtra(extra); ze.setComment(comment); out.putNextEntry(ze); out.closeEntry(); out.close(); ZipFile zipFile = new ZipFile(f); try { assertEquals(comment, zipFile.getEntry("x").getComment()); assertTrue(Arrays.equals(extra, zipFile.getEntry("x").getExtra())); assertEquals(comment, zipFile.getEntry("y").getComment()); assertTrue(Arrays.equals(extra, zipFile.getEntry("y").getExtra())); } finally { zipFile.close(); } }
Source:
<code>public boolean isRange() { <START> int ordinal = this.ordinal(); <END> return (ordinal == GREATER_THAN.ordinal() || ordinal == GREATER_THAN_OR_EQUAL.ordinal() || ordinal == LESS_THAN .ordinal() || ordinal == LESS_THAN_OR_EQUAL.ordinal() || ordinal == BETWEEN.ordinal()); }</code><technical_language>ordinal? suggestion return this == GREATER_THAN || this == GREATER_THEN_OR_EQUAL || this == LESS_THEN || this == LESS_THAN_OR_EQUAL || this == BETWEEN;</technical_language>

Target:
public boolean isRange() { return (this == GREATER_THAN || this == GREATER_THAN_OR_EQUAL || this == LESS_THAN || this == LESS_THAN_OR_EQUAL || this == BETWEEN); }
Source:
<code>public void pollMessages() throws Exception { <START> int count = 0; <END> if (instances != getContext().getInstanceCount()) { instances = getContext().getInstanceCount(); DefaultKafkaConfigurer kafkaConfigurer = new DefaultKafkaConfigurer(); handleInstancesChanged(kafkaConfigurer); updateConsumerInfos(kafkaConfigurer.getTopicPartitions(), consumerInfos); } for (KafkaConsumerInfo<OFFSET> info : consumerInfos.values()) { Iterator<KafkaMessage<OFFSET>> iterator = readMessages(info); while (iterator.hasNext()) { KafkaMessage<OFFSET> message = iterator.next(); processMessage(message); info.setReadOffset(message.getNextOffset()); ++count; } } if (count > 0) { saveReadOffsets(Maps.transformValues(consumerInfos, consumerToOffset)); } }</code><technical_language>Move closer is (i.e. for)</technical_language>

Target:
public void pollMessages() throws Exception { if (instances != getContext().getInstanceCount()) { instances = getContext().getInstanceCount(); DefaultKafkaConfigurer kafkaConfigurer = new DefaultKafkaConfigurer(); handleInstancesChanged(kafkaConfigurer); updateConsumerInfos(kafkaConfigurer.getTopicPartitions(), consumerInfos); } int count = 0; for (KafkaConsumerInfo<OFFSET> info : consumerInfos.values()) { Iterator<KafkaMessage<OFFSET>> iterator = readMessages(info); while (iterator.hasNext()) { KafkaMessage<OFFSET> message = iterator.next(); processMessage(message); info.setReadOffset(message.getNextOffset()); ++count; } } if (count > 0) { saveReadOffsets(Maps.transformValues(consumerInfos, consumerToOffset)); } }
Source:
<code>public void layOutGalleryDescription() { GalleryItemFragment item = getCurrentItem(); if (item == null || item.getImageTitle() == null || item.getMediaInfo() == null || item.getMediaInfo().getMetadata() == null) { infoContainer.setVisibility(View.GONE); return; } updateProgressBar(true); disposeImageCaptionDisposable(); imageCaptionDisposable = Observable.zip(MediaHelper.INSTANCE.getImageCaptions(item.getImageTitle().getPrefixedText()), ServiceFactory.get(new WikiSite(Service.COMMONS_URL)).getProtectionInfo(item.getImageTitle().getPrefixedText()), <START> ImageTagsProvider.getImageTagsObservable(getCurrentItem().getMediaPage().pageId(), WikipediaApp.getInstance().getAppOrSystemLanguageCode()), new Function3<Map<String, String>, MwQueryResponse, Map<String, List<String>>, Boolean>() { <END> @Override public Boolean apply(Map<String, String> captions, MwQueryResponse protectionInfoRsp, Map<String, List<String>> imageTags) throws Throwable { item.getMediaInfo().setCaptions(captions); imageTagsCount = imageTags.size(); return protectionInfoRsp.query().isEditProtected(); } }) .subscribeOn(Schedulers.io()) .observeOn(AndroidSchedulers.mainThread()) .subscribe(this::updateGalleryDescription, L::e); }</code><technical_language>Replace lambda</technical_language>

Target:
public void layOutGalleryDescription() { GalleryItemFragment item = getCurrentItem(); if (item == null || item.getImageTitle() == null || item.getMediaInfo() == null || item.getMediaInfo().getMetadata() == null) { infoContainer.setVisibility(View.GONE); return; } updateProgressBar(true); disposeImageCaptionDisposable(); imageCaptionDisposable = Observable.zip(MediaHelper.INSTANCE.getImageCaptions(item.getImageTitle().getPrefixedText()), ServiceFactory.get(new WikiSite(Service.COMMONS_URL)).getProtectionInfo(item.getImageTitle().getPrefixedText()), ImageTagsProvider.getImageTagsObservable(getCurrentItem().getMediaPage().pageId(), WikipediaApp.getInstance().getAppOrSystemLanguageCode()), (captions, protectionInfoRsp, imageTags) -> { item.getMediaInfo().setCaptions(captions); imageTagsCount = imageTags.size(); return protectionInfoRsp.query().isEditProtected(); }) .subscribeOn(Schedulers.io()) .observeOn(AndroidSchedulers.mainThread()) .subscribe(this::updateGalleryDescription, L::e); }
Source:
<code>public String getSqlStatement() { <START> return (logic != null ? logic.toString() + " " : "") + column.getSqlKeyword() + " " + operator.getSqlStatement(); <END> }</code><technical_language>backticks necessary prevent keyword collisions (PR #138). E.g. if is a column named group, clause for this column needs wrap group in backticks prevent MySQLSyntaxErrorException</technical_language>

Target:
public String getSqlStatement() { return (logic != null ? logic.toString() + " " : "") + "`" + column.getSqlKeyword() + "` " + operator.getSqlStatement(); }