auphong2707/dl4se-code-review-dataset huggingface dataset:
Source:
<code>private List<HomeFeedbackSessionRow> createSessionRows(List<FeedbackSessionDetailsBundle> feedbackSessions, Map<FeedbackSessionAttributes, Boolean> sessionSubmissionStatusMap, int startingSessionIdx) { List<HomeFeedbackSessionRow> rows = new ArrayList<>(); int sessionIdx = startingSessionIdx; for (FeedbackSessionDetailsBundle session : feedbackSessions) { FeedbackSessionAttributes feedbackSession = session.feedbackSession; String sessionName = feedbackSession.getFeedbackSessionName(); boolean hasSubmitted = sessionSubmissionStatusMap.get(feedbackSession); rows.add(new StudentHomeFeedbackSessionRow( PageData.sanitizeForHtml(sessionName), getStudentSubmissionsTooltipForSession(feedbackSession, hasSubmitted), getStudentPublishedTooltipForSession(feedbackSession), getStudentSubmissionStatusForSession(feedbackSession, hasSubmitted), getStudentPublishedStatusForSession(feedbackSession), <START> TimeHelper.formatTime12H(feedbackSession.getEndTimeLocal()), <END> feedbackSession.getEndTimeInIso8601UtcFormat(), getStudentFeedbackSessionActions(feedbackSession, hasSubmitted), sessionIdx)); ++sessionIdx; } return rows; }</code><technical_language>This related, improving student home page adding sort deadline, this chance show time zone info in deadline display? Simply change formatTime12H formatDateTimeForSessions run godmode. Forgot do this in #8687</technical_language>

Target:
private List<HomeFeedbackSessionRow> createSessionRows(List<FeedbackSessionDetailsBundle> feedbackSessions, Map<FeedbackSessionAttributes, Boolean> sessionSubmissionStatusMap, int startingSessionIdx) { List<HomeFeedbackSessionRow> rows = new ArrayList<>(); int sessionIdx = startingSessionIdx; for (FeedbackSessionDetailsBundle session : feedbackSessions) { FeedbackSessionAttributes feedbackSession = session.feedbackSession; String sessionName = feedbackSession.getFeedbackSessionName(); boolean hasSubmitted = sessionSubmissionStatusMap.get(feedbackSession); rows.add(new StudentHomeFeedbackSessionRow( PageData.sanitizeForHtml(sessionName), getStudentSubmissionsTooltipForSession(feedbackSession, hasSubmitted), getStudentPublishedTooltipForSession(feedbackSession), getStudentSubmissionStatusForSession(feedbackSession, hasSubmitted), getStudentPublishedStatusForSession(feedbackSession), TimeHelper.formatDateTimeForSessions(feedbackSession.getEndTime(), feedbackSession.getTimeZone()), feedbackSession.getEndTimeInIso8601UtcFormat(), getStudentFeedbackSessionActions(feedbackSession, hasSubmitted), sessionIdx)); ++sessionIdx; } return rows; }
Source:
<code>public void augmentTerminationURIs(Map<String, String> terminateURIs, URI baseUri) { String baseURI = UriBuilder.fromUri(baseUri) .path(LRAParticipantResource.RESOURCE_PATH) .path(id.getName()) .build().toASCIIString(); <START> terminateURIs.computeIfAbsent(COMPLETE, s -> completeMethod != null ? getURI(baseURI, COMPLETE) : null); <END> terminateURIs.computeIfAbsent(COMPENSATE, s -> compensateMethod != null ? getURI(baseURI, COMPENSATE) : null); terminateURIs.computeIfAbsent(STATUS, s -> statusMethod != null ? getURI(baseURI, STATUS) : null); terminateURIs.computeIfAbsent(FORGET, s -> forgetMethod != null ? getURI(baseURI, FORGET) : null); }</code><technical_language>This code is inefficient: - if key is absent potentially puts a null map - is executing a lambda a simple test is preferable: java if (!terminateURIs.containsKey(COMPLETE) && completeMethod != null) terminateURIs.put(COMPLETE, getURI(baseURI, COMPLETE)); Similarly for other keys</technical_language>

Target:
public void augmentTerminationURIs(Map<String, String> terminateURIs, URI baseUri) { String baseURI = UriBuilder.fromUri(baseUri) .path(LRAParticipantResource.RESOURCE_PATH) .path(javaClass.getName()) .build().toASCIIString(); if (!terminateURIs.containsKey(COMPLETE) && completeMethod != null) { terminateURIs.put(COMPLETE, getURI(baseURI, COMPLETE)); } if (!terminateURIs.containsKey(COMPENSATE) && compensateMethod != null) { terminateURIs.put(COMPENSATE, getURI(baseURI, COMPENSATE)); } if (!terminateURIs.containsKey(STATUS) && statusMethod != null) { terminateURIs.put(STATUS, getURI(baseURI, STATUS)); } if (!terminateURIs.containsKey(FORGET) && forgetMethod != null) { terminateURIs.put(FORGET, getURI(baseURI, FORGET)); } if (!terminateURIs.containsKey(AFTER) && afterLRAMethod != null) { terminateURIs.put(AFTER, getURI(baseURI, AFTER)); } }
Source:
<code>void unsubscribeAChild() { State oldState; State newState; do { <START> oldState = state.get(); <END> newState = oldState.removeChild(); } while (!state.compareAndSet(oldState, newState)); unsubscribeActualIfApplicable(newState); }</code><technical_language>field state read a local variable loop</technical_language>

Target:
void unsubscribeAChild() { State oldState; State newState; final AtomicReference<State> localState = this.state; do { oldState = localState.get(); newState = oldState.removeChild(); } while (!localState.compareAndSet(oldState, newState)); unsubscribeActualIfApplicable(newState); }
Source:
<code>protected void afterInit(SiteRun run) throws Exception { List<Module> modules = Lists.newArrayList(); modules.add(new AbstractModule() { @Override protected void configure() { bind(File.class).annotatedWith(SitePath.class).toInstance(getSitePath()); bind(Browser.class); bind(String.class).annotatedWith(SecureStoreClassName.class) <START> .toProvider(SecureStoreClassNameProvider.class); <END> bind(SecureStore.class).toProvider(SecureStoreProvider.class); } }); modules.add(new GerritServerConfigModule()); Guice.createInjector(modules).injectMembers(this); start(run); }</code><technical_language>Do need this else in SecureStoreProvider? SitePaths injected SecureStoreProvider read class name gerrit.config there. SecureStoreClassName annotation SecureStoreClassNameProvider deleted SecureStoreProvider is needs this information</technical_language>

Target:
protected void afterInit(SiteRun run) throws Exception { List<Module> modules = Lists.newArrayList(); modules.add(new AbstractModule() { @Override protected void configure() { bind(File.class).annotatedWith(SitePath.class).toInstance(getSitePath()); bind(Browser.class); bind(SecureStore.class).toProvider(SecureStoreProvider.class); } }); modules.add(new GerritServerConfigModule()); Guice.createInjector(modules).injectMembers(this); start(run); }
Source:
<code>public void assignRandomTexture() { if(!world.isRemote) { <START> if(getTexturePool().length > 0) { <END> setTexture(getTexturePool()[rand.nextInt(getTexturePool().length)]); } } }</code><technical_language>getTexturePool return null, add a null check here</technical_language>

Target:
public void assignRandomTexture() { if(!world.isRemote && getTexturePool() != null) { if(getTexturePool().length > 0) { setTexture(getTexturePool()[rand.nextInt(getTexturePool().length)]); } } }
Source:
<code>private void startAuthMethodChoice() { <START> List<AuthUI.IdpConfig> idpConfigs = getArguments().providerInfo; <END> if (getArguments().isSingleProviderFlow()) { AuthUI.IdpConfig firstIdpConfig = idpConfigs.get(0); String firstProvider = firstIdpConfig.getProviderId(); switch (firstProvider) { case EmailAuthProvider.PROVIDER_ID: setResult(Resource.<IdpResponse>forFailure(new IntentRequiredException( EmailActivity.createIntent(getApplication(), getArguments()), RequestCodes.EMAIL_FLOW))); break; case PhoneAuthProvider.PROVIDER_ID: setResult(Resource.<IdpResponse>forFailure(new IntentRequiredException( PhoneActivity.createIntent( getApplication(), getArguments(), firstIdpConfig.getParams()), RequestCodes.PHONE_FLOW))); break; default: redirectSignIn(firstProvider, null); break; } } else { setResult(Resource.<IdpResponse>forFailure(new IntentRequiredException( AuthMethodPickerActivity.createIntent(getApplication(), getArguments()), RequestCodes.AUTH_PICKER_FLOW))); } }</code><technical_language>inline getArguments().providerInfo below? (Trick: Ctrl + Shift + N)</technical_language>

Target:
private void startAuthMethodChoice() { if (getArguments().isSingleProviderFlow()) { AuthUI.IdpConfig firstIdpConfig = getArguments().providerInfo.get(0); String firstProvider = firstIdpConfig.getProviderId(); switch (firstProvider) { case EmailAuthProvider.PROVIDER_ID: setResult(Resource.<IdpResponse>forFailure(new IntentRequiredException( EmailActivity.createIntent(getApplication(), getArguments()), RequestCodes.EMAIL_FLOW))); break; case PhoneAuthProvider.PROVIDER_ID: setResult(Resource.<IdpResponse>forFailure(new IntentRequiredException( PhoneActivity.createIntent( getApplication(), getArguments(), firstIdpConfig.getParams()), RequestCodes.PHONE_FLOW))); break; default: redirectSignIn(firstProvider, null); break; } } else { setResult(Resource.<IdpResponse>forFailure(new IntentRequiredException( AuthMethodPickerActivity.createIntent(getApplication(), getArguments()), RequestCodes.AUTH_PICKER_FLOW))); } }
Source:
<code>protected String getTokenFromSession(final WebContext context) { <START> return ((Optional<String>) context.getSessionStore().get(context, Pac4jConstants.CSRF_TOKEN)).orElse(null); <END> }</code><technical_language>this cast dangerous: is optional stored in there</technical_language>

Target:
protected String getTokenFromSession(final WebContext context) { return (String) context.getSessionStore().get(context, Pac4jConstants.CSRF_TOKEN).orElse(null); }
Source:
<code>public void testActionsWithInvert(final IntervalListTools.Action action, final long bases, final int intervals) throws IOException { final IntervalList il = tester(action, true); <START> Assert.assertEquals(il.getBaseCount(), bases, "unexpcted number of bases found."); <END> Assert.assertEquals(il.getIntervals().size(), intervals, "unexpcted number of intervals found."); }</code><technical_language>'unexpected'</technical_language>

Target:
public void testActionsWithInvert(final IntervalListTools.Action action, final long bases, final int intervals) throws IOException { final IntervalList il = tester(action, true); Assert.assertEquals(il.getBaseCount(), bases, "unexpected number of bases found."); Assert.assertEquals(il.getIntervals().size(), intervals, "unexpected number of intervals found."); }
Source:
<code>public GridShmemCommunicationClient( int connIdx, MetricRegistry mreg, int port, long connTimeout, IgniteLogger log, MessageFormatter formatter ) throws IgniteCheckedException { super(connIdx); assert mreg != null; assert port > 0 && port < 0xffff; assert connTimeout >= 0; shmem = new IpcSharedMemoryClientEndpoint(port, (int)connTimeout, log); writeBuf = ByteBuffer.allocate(8 << 10); writeBuf.order(ByteOrder.nativeOrder()); this.formatter = formatter; <START> sentBytesCntMetric = mreg == null ? <END> null : mreg.longMetric(SENT_BYTES_METRIC_NAME, SENT_BYTES_METRIC_DESC); }</code><technical_language>mreg null here</technical_language>

Target:
public GridShmemCommunicationClient( int connIdx, MetricRegistry mreg, int port, long connTimeout, IgniteLogger log, MessageFormatter formatter ) throws IgniteCheckedException { super(connIdx); assert mreg != null; assert port > 0 && port < 0xffff; assert connTimeout >= 0; shmem = new IpcSharedMemoryClientEndpoint(port, (int)connTimeout, log); writeBuf = ByteBuffer.allocate(8 << 10); writeBuf.order(ByteOrder.nativeOrder()); this.formatter = formatter; sentBytesCntMetric = mreg.findMetric(SENT_BYTES_METRIC_NAME); }
Source:
<code>private void initializeFragments() { <START> TaskFragment mTaskFragment = TaskFragment.newInstance(0); <END> mFragmentManager = getSupportFragmentManager(); mFragmentManager.addOnBackStackChangedListener( new FragmentManager.OnBackStackChangedListener() { @Override public void onBackStackChanged() { Fragment f = getSupportFragmentManager().findFragmentById(R.id.container); if (f != null){ updateTitle(f); } } }); updateTitle(mTaskFragment); android.support.v4.app.FragmentTransaction ft = mFragmentManager.beginTransaction(); ft.add(R.id.container, mTaskFragment); ft.commit(); }</code><technical_language>Kind of picky, this taskFragment instead of mTaskFragment</technical_language>

Target:
private void initializeFragments() { TaskFragment taskFragment = TaskFragment.newInstance(0); mFragmentManager = getSupportFragmentManager(); mFragmentManager.addOnBackStackChangedListener( new FragmentManager.OnBackStackChangedListener() { @Override public void onBackStackChanged() { Fragment f = getSupportFragmentManager().findFragmentById(R.id.container); if (f != null){ updateTitle(f); } } }); updateTitle(taskFragment); android.support.v4.app.FragmentTransaction ft = mFragmentManager.beginTransaction(); ft.add(R.id.container, taskFragment); ft.commit(); }
Source:
<code>public void forbidsCloneMethodReturnTypeNotMatchingClassName() throws Exception { <START> final String file = "CloneMethodReturnTypeMustMatchClassName.java"; <END> new PMDAssert( file, Matchers.is(false), Matchers.containsString( String.format( PMDValidatorTest.BRACKETS, "CloneMethodReturnTypeMustMatchClassName" ) ) ).validate(); }</code><technical_language>@Happy-Neko inline this variable</technical_language>

Target:
public void forbidsCloneMethodReturnTypeNotMatchingClassName() throws Exception { new PMDAssert( "CloneMethodReturnTypeMustMatchClassName.java", Matchers.is(false), Matchers.containsString( String.format( PMDValidatorTest.BRACKETS, "CloneMethodReturnTypeMustMatchClassName" ) ) ).validate(); }
Source:
<code>public Class<?> getReturnType(Class<?> firstArgumentType) { <START> return BigInteger.class; <END> }</code><technical_language>Long.class</technical_language>

Target:
public Class<?> getReturnType(Class<?> firstArgumentType) { return Long.class; }
Source:
<code>public void startApp() { quitCommand = new Command("Quit", Command.SCREEN, 1); textBox = new TextBox("Hello World", "Some text", 40, 0); textBox.addCommand(quitCommand); textBox.setCommandListener(this); display.setCurrent(textBox); try { do { Thread.sleep(1000); } while (!textBox.isShown()); <START> } catch (Exception e) { <END> System.out.println("FAIL"); } System.out.println("PAINTED"); }</code><technical_language>Nit: I this catch InterruptedException specifically. sleep for time, 100ms. ideally register for some event occurs TextBox is shown. event, however</technical_language>

Target:
public void startApp() { quitCommand = new Command("Quit", Command.SCREEN, 1); textBox = new TextBox("Hello World", "Some text", 40, 0); textBox.addCommand(quitCommand); textBox.setCommandListener(this); display.setCurrent(textBox); try { do { Thread.sleep(100); } while (!textBox.isShown()); } catch (InterruptedException e) { System.out.println("FAIL"); } System.out.println("PAINTED"); }
Source:
<code>int maybeCommitActiveTasksPerUserRequested() { if (rebalanceInProgress) { return -1; } else { if (eosBetaEnabled && !eosUpgradeModeEnabled) { for (final Task task : activeTaskIterable()) { if (task.commitRequested() && task.commitNeeded()) { <START> return commitAll(); <END> } } return 0; } else { final Map<TaskId, Map<TopicPartition, OffsetAndMetadata>> consumedOffsetsAndMetadataPerTask = new HashMap<>(); for (final Task task : activeTaskIterable()) { if (task.commitRequested() && task.commitNeeded()) { task.prepareCommit(); final Map<TopicPartition, OffsetAndMetadata> offsetAndMetadata = task.committableOffsetsAndMetadata(); if (!offsetAndMetadata.isEmpty()) { consumedOffsetsAndMetadataPerTask.put(task.id(), offsetAndMetadata); } } } commitOffsetsOrTransaction(consumedOffsetsAndMetadataPerTask); for (final Task task : tasks.values()) { if (consumedOffsetsAndMetadataPerTask.containsKey(task.id())) { task.postCommit(); } } return consumedOffsetsAndMetadataPerTask.size(); } } }</code><technical_language>I thought agreed commitAll user requested in task, in 2.6 beyond, matter of eos config? Personally I if/else branching is worth complexity itself.</technical_language>

Target:
int maybeCommitActiveTasksPerUserRequested() { if (rebalanceInProgress) { return -1; } else { for (final Task task : activeTaskIterable()) { if (task.commitRequested() && task.commitNeeded()) { return commitInternal(activeTaskIterable()); } } return 0; } }
Source:
<code>public NotesMergeConflictException(NonNoteEntry base, NonNoteEntry ours, NonNoteEntry theirs) { super(MessageFormat.format( <START> JGitText.get().mergeConflictOnNonNoteEntries, base.name(), <END> ours.name(), theirs.name())); }</code><technical_language>of base, ours, null if NonNoteEntry exist... causing conflict NPE. need similar noteOn here: private static String name(NonNoteEntry e) { return e != null ? e.name() : ""; }</technical_language>

Target:
public NotesMergeConflictException(NonNoteEntry base, NonNoteEntry ours, NonNoteEntry theirs) { super(MessageFormat.format( JGitText.get().mergeConflictOnNonNoteEntries, name(base), name(ours), name(theirs))); }
Source:
<code><START> public Certificate parseCertificate(final String cert) { <END> final PemReader certPem = new PemReader(new StringReader(cert)); try { return readCertificateFromPemObject(certPem.readPemObject()); } catch (final Exception e) { throw new InvalidParameterValueException("Invalid Certificate format. Expected X509 certificate. Failed due to " + e.getMessage()); } finally { IOUtils.closeQuietly(certPem); } }</code><technical_language>Please consider adding a Precondition.checkArgument check cert is blank</technical_language>

Target:
public Certificate parseCertificate(final String cert) { Preconditions.checkArgument(!Strings.isNullOrEmpty(cert)); final PemReader certPem = new PemReader(new StringReader(cert)); try { return readCertificateFromPemObject(certPem.readPemObject()); } catch (final CertificateException | IOException e) { throw new InvalidParameterValueException("Invalid Certificate format. Expected X509 certificate. Failed due to " + e.getMessage()); } finally { IOUtils.closeQuietly(certPem); } }
Source:
<code><START> @Override public Path interceptRemainingPath(Path remainingPath) { <END> return remainingPath; }</code><technical_language>interceptSrc remainingPath follow pattern intercept split src path only? Whats issue if I interceptSource method? Do some concerns? explain if I miss here</technical_language>

Target:
public Path interceptRemainingPath(Path remainingPath) { return remainingPath; }
Source:
<code>private void createTooltipShell(Shell parent) { final Display display = parent.getDisplay(); if (fTipShell != null && !fTipShell.isDisposed()) { fTipShell.dispose(); } elementCount = 0; longestString = 0; fTipShell = new Shell(parent, SWT.ON_TOP | SWT.TOOL); fTipShell.addDisposeListener(e -> e.display.removeFilter(SWT.MouseMove, fListener)); fTipShell.addDisposeListener(e -> e.display.removeFilter(SWT.FocusOut, fFocusLostListener)); fTipShell.addListener(SWT.Deactivate, e -> { if (fTipShell.isDisposed()) { fTipShell.dispose(); } }); fToolTipContent = new StringBuilder(); GridLayout gridLayout = new GridLayout(); <START> gridLayout.numColumns = 2; <END> gridLayout.marginWidth = 2; gridLayout.marginHeight = 2; fTipShell.setLayout(gridLayout); fTipShell.setBackground(display.getSystemColor(SWT.COLOR_INFO_BACKGROUND)); fTipViewer = new Browser(fTipShell, SWT.NO_SCROLL); fTipViewer.setLayout(new GridLayout(3, false)); fTipViewer.addLocationListener(new LocationListener() { @Override public void changing(LocationEvent event) { String timePrefix = "time://"; if (event.location.startsWith(timePrefix)) { String time = event.location.substring(timePrefix.length()); try { long val = Long.parseLong(time); TmfSignalManager.dispatchSignal(new TmfSelectionRangeUpdatedSignal(event.getSource(), TmfTimestamp.fromNanos(val))); } catch (NumberFormatException e) { } event.doit = false; } } @Override public void changed(LocationEvent event) { } }); setupControl(fTipViewer); }</code><technical_language>2 columns, put in it</technical_language>

Target:
private void createTooltipShell(Shell parent) { final Display display = parent.getDisplay(); if (fTipShell != null && !fTipShell.isDisposed()) { fTipShell.dispose(); } fElementCount = 0; fLongestString = 0; fTipShell = new Shell(parent, SWT.ON_TOP | SWT.TOOL | SWT.RESIZE); fTipShell.addDisposeListener(e -> e.display.removeFilter(SWT.MouseMove, fListener)); fTipShell.addDisposeListener(e -> e.display.removeFilter(SWT.FocusOut, fFocusLostListener)); fTipShell.addListener(SWT.Deactivate, e -> { if (!fTipShell.isDisposed()) { fTipShell.dispose(); } }); fToolTipContent = new StringBuilder(); fTipShell.setLayout(new FillLayout()); fTipShell.setBackground(display.getSystemColor(SWT.COLOR_INFO_BACKGROUND)); fTipViewer = new Browser(fTipShell, SWT.NO_SCROLL); fTipViewer.addLocationListener(new LocationListener() { @Override public void changing(LocationEvent event) { String timePrefix = "time://"; if (event.location.startsWith(timePrefix)) { String time = event.location.substring(timePrefix.length()); Long val = Longs.tryParse(time); if (val != null) { TmfSignalManager.dispatchSignal(new TmfSelectionRangeUpdatedSignal(event.getSource(), TmfTimestamp.fromNanos(val))); } event.doit = false; } } @Override public void changed(LocationEvent event) { } }); setupControl(fTipViewer); }
Source:
<code><START> public void removeResultGroupRelations(PerunSession sess, Group resultGroup) { <END> jdbc.update("DELETE FROM groups_groups WHERE result_gid = ?", resultGroup.getId()); jdbc.update("DELETE FROM groups_members WHERE group_id = ? AND membership_type != ?", resultGroup.getId(), MembershipType.DIRECT.getCode()); }</code><technical_language>In redmine 2630</technical_language>

Target:
public void removeResultGroupRelations(PerunSession sess, Group resultGroup) { jdbc.update("DELETE FROM groups_groups WHERE result_gid = ?", resultGroup.getId()); }
Source:
<code><START> AlternateHandle.Id getAlternateId() { <END> return wrapped.getAlternateId(); }</code><technical_language>this method declared private</technical_language>

Target:
private AlternateHandle.Id getAlternateId() { return wrapped.getAlternateId(); }
Source:
<code>private void readTable(String tableName, int startRow, int numRowsToRead) { if (worker != null && !worker.isDone()) { worker.cancel(false); worker = null; } worker = new SwingWorker<ArrayList<Map<String, Object>>, Void>() { @Override protected ArrayList<Map<String, Object>> doInBackground() throws Exception { try { Statement statement = connection.createStatement(); ResultSet resultSet = statement.executeQuery( "SELECT * FROM " + tableName + " LIMIT " + Integer.toString(numRowsToRead) + " OFFSET " + Integer.toString(startRow - 1) ); return resultSetToArrayList(resultSet); } catch (SQLException ex) { <START> LOGGER.log(Level.SEVERE, "Failed to get data for table.", ex); END> } return null; } @Override protected void done() { if (isCancelled()) { return; } super.done(); try { ArrayList<Map<String, Object>> rows = get(); if (Objects.nonNull(rows)) { selectedTableView.setupTable(rows); }else{ selectedTableView.setupTable(Collections.emptyList()); } } catch (InterruptedException | ExecutionException ex) { LOGGER.log(Level.SEVERE, "Unexpected exception while reading table.", ex); } } }; worker.execute(); }</code><technical_language>helpful for debugging purposes if logged more information, i.e., file name table name, other arguments well. concern is is right now, actionable information if received a report of a problem production in open source community</technical_language>

Target:
private void readTable(String tableName, int startRow, int numRowsToRead) { if (worker != null && !worker.isDone()) { worker.cancel(false); worker = null; } worker = new SwingWorker<ArrayList<Map<String, Object>>, Void>() { @Override protected ArrayList<Map<String, Object>> doInBackground() throws Exception { try { Statement statement = connection.createStatement(); ResultSet resultSet = statement.executeQuery( "SELECT * FROM " + tableName + " LIMIT " + Integer.toString(numRowsToRead) + " OFFSET " + Integer.toString(startRow - 1) ); return resultSetToArrayList(resultSet); } catch (SQLException ex) { LOGGER.log(Level.SEVERE, "Failed to get data for table " + tableName, ex); } return null; } @Override protected void done() { if (isCancelled()) { return; } super.done(); try { ArrayList<Map<String, Object>> rows = get(); if (Objects.nonNull(rows)) { selectedTableView.setupTable(rows); }else{ selectedTableView.setupTable(Collections.emptyList()); } } catch (InterruptedException | ExecutionException ex) { LOGGER.log(Level.SEVERE, "Unexpected exception while reading table " + tableName, ex); } } }; worker.execute(); }
Source:
<code>public static void main(String[] args) { if (args.length < 4 || args.length > 8) { <START> System.out.println("Please specify your access key, webhook id, enabled or not, url of webhook and events.\n" + <END> "Events can be a maximum of 4 between conversation.created, conversation.updated, message.created, message.updated\n" + "If third argument is enabled, then status of the webhook will be updated to enabled, otherwise will be disabled\n" + "Usage : java -jar <this jar file> test_accesskey webhook_title webhook-url"); return; } final MessageBirdService wsr = new MessageBirdServiceImpl(args[0]); final MessageBirdClient messageBirdClient = new MessageBirdClient(wsr); try { System.out.println("Updating conversation webhook.."); String[] arrayOfEvents = new String[args.length - 4]; System.arraycopy(args, 4, arrayOfEvents, 0, args.length - 4); ConversationWebhookUpdateRequest request = new ConversationWebhookUpdateRequest( args[2].equals("enabled") ? ConversationWebhookStatus.ENABLED : ConversationWebhookStatus.DISABLED, args[3], parseConversationWebHookEvents(arrayOfEvents) ); final ConversationWebhook conversationWebhookResponse = messageBirdClient.updateConversationWebhook(args[1], request); System.out.println(conversationWebhookResponse.toString()); } catch (GeneralException | UnauthorizedException exception) { exception.printStackTrace(); } }</code><technical_language>I goal of this example is demonstrate users integrate this SDK: provide a CLI client API. idea of command line arguments is nice, I example harder read in this case. I feel more clear obvious hardcode values (other access key)</technical_language>

Target:
public static void main(String[] args) { if (args.length < 1 || args.length > 8) { System.out.println("Please at least specify your access key and a max of 8 arguments.\n" + "You can also specify a webhook id, enabled or not, url of webhook and events. Default value would be used if you don't\n" + "Events can be a maximum of 4 between conversation.created, conversation.updated, message.created, message.updated\n" + "If third argument is enabled, then status of the webhook will be updated to enabled, otherwise will be disabled\n\n" + "Usage : java -jar <this jar file> test_accesskey(Required) webhook_title(Optional) enabled(Optional) webhook-url(Optional) event1(Optional) event2(Optional) event3(Optional) event4(Optional)"); return; } final MessageBirdService wsr = new MessageBirdServiceImpl(args[0]); final MessageBirdClient messageBirdClient = new MessageBirdClient(wsr); try { System.out.println("Updating conversation webhook.."); ConversationWebhookUpdateRequest request = new ConversationWebhookUpdateRequest( getStatus(args) , getWebhookUrl(args), getConversationWebHookEvents(args) ); final ConversationWebhook conversationWebhookResponse = messageBirdClient.updateConversationWebhook(args[1], request); System.out.println(conversationWebhookResponse); } catch (GeneralException | UnauthorizedException exception) { exception.printStackTrace(); } }
Source:
<code><START> public ByteBuffer getExplicitLac() { <END> LOG.debug("fileInfo:GetLac: {}", explicitLac); ByteBuffer retLac = null; synchronized(this) { if (explicitLac != null) { explicitLac.rewind(); retLac = ByteBuffer.wrap(explicitLac.array()); } } return retLac; }</code><technical_language>is lac field in FileInfo. leverage that</technical_language>

Target:
public ByteBuffer getExplicitLac() { LOG.debug("fileInfo:GetLac: {}", explicitLac); ByteBuffer retLac = null; synchronized(this) { if (explicitLac != null) { retLac = ByteBuffer.allocate(explicitLac.capacity()); explicitLac.rewind(); retLac.put(explicitLac); explicitLac.rewind(); retLac.flip(); } } return retLac; }
Source:
<code>private void generateActionSignature(Method method) { generateDoc(method); Name methodName = new Name(method.getBase().getName()); <START> methodName.addWords(method.getName().getWords()); <END> javaBuffer.addLine( "Response %s(Action action) {", jaxrsNames.getMethodName(methodName) ); javaBuffer.addLine("throw new UnsupportedOperationException();"); javaBuffer.addLine("}"); }</code><technical_language>explicitly import classes using, if move code in future incorrect: javaBuffer.addImport(Response.class); javaBuffer.addImport(javaPackages.getXjcPackageName(), "Action");</technical_language>

Target:
private void generateActionSignature(Method method) { generateDoc(method); javaBuffer.addImport(Response.class); javaBuffer.addImport(javaPackages.getXjcPackageName(), "Action"); Name methodName = new Name(method.getBase().getName()); methodName.addWords(method.getName().getWords()); javaBuffer.addLine("Response %s(Action action) {", jaxrsNames.getMethodName(methodName)); javaBuffer.addLine("throw new UnsupportedOperationException();"); javaBuffer.addLine("}"); }
Source:
<code><START> public void addAPIRevisionDeployment(String apiId, String apiRevisionId, List<APIRevisionDeployment> apiRevisionDeployments) throws APIManagementException { <END> APIIdentifier apiIdentifier = APIUtil.getAPIIdentifierFromUUID(apiId); if (apiIdentifier == null) { throw new APIMgtResourceNotFoundException("Couldn't retrieve existing API with API UUID: " + apiId, ExceptionCodes.from(ExceptionCodes.API_NOT_FOUND, apiId)); } APIRevision apiRevision = apiMgtDAO.getRevisionByRevisionUUID(apiRevisionId); if (apiRevision == null) { throw new APIMgtResourceNotFoundException("Couldn't retrieve existing API Revision with Revision UUID: " + apiRevisionId, ExceptionCodes.from(ExceptionCodes.API_NOT_FOUND, apiRevisionId)); } APITemplateBuilder builder = null; APIGatewayManager gatewayManager = APIGatewayManager.getInstance(); APIIdentifier revisionApiIdentifier = getLightweightAPIByUUID(apiRevisionId, tenantDomain).getId(); API api = getRevisionAPI(revisionApiIdentifier, apiRevision); api.setSwaggerDefinition(getOpenAPIDefinition(revisionApiIdentifier)); Set<String> environments = new HashSet<>(); for (APIRevisionDeployment apiRevisionDeployment : apiRevisionDeployments) { environments.add(apiRevisionDeployment.getDeployment()); } api.setEnvironments(environments); try { builder = getAPITemplateBuilder(api); } catch (Exception e) { handleException("Error while publishing to Gateway ", e); } Map<String, String> failedEnvironment = gatewayManager.deployAPIRevisionToGateway(api, builder, tenantDomain); apiMgtDAO.addAPIRevisionDeployment(apiRevisionId, apiRevisionDeployments); }</code><technical_language>Is this line formatted</technical_language>

Target:
public void addAPIRevisionDeployment(String apiId, String apiRevisionId, List<APIRevisionDeployment> apiRevisionDeployments) throws APIManagementException { APIIdentifier apiIdentifier = APIUtil.getAPIIdentifierFromUUID(apiId); if (apiIdentifier == null) { throw new APIMgtResourceNotFoundException("Couldn't retrieve existing API with API UUID: " + apiId, ExceptionCodes.from(ExceptionCodes.API_NOT_FOUND, apiId)); } APIRevision apiRevision = apiMgtDAO.getRevisionByRevisionUUID(apiRevisionId); if (apiRevision == null) { throw new APIMgtResourceNotFoundException("Couldn't retrieve existing API Revision with Revision UUID: " + apiRevisionId, ExceptionCodes.from(ExceptionCodes.API_NOT_FOUND, apiRevisionId)); } APITemplateBuilder builder = null; APIGatewayManager gatewayManager = APIGatewayManager.getInstance(); APIIdentifier revisionApiIdentifier = getLightweightAPIByUUID(apiRevisionId, tenantDomain).getId(); API api = getRevisionAPI(revisionApiIdentifier, apiRevision); Set<String> environments = new HashSet<>(); for (APIRevisionDeployment apiRevisionDeployment : apiRevisionDeployments) { environments.add(apiRevisionDeployment.getDeployment()); } api.setEnvironments(environments); try { builder = getAPITemplateBuilder(api); } catch (Exception e) { handleException("Error while publishing to Gateway ", e); } Map<String, String> failedEnvironment = gatewayManager.deployAPIRevisionToGateway(api, builder, tenantDomain); apiMgtDAO.addAPIRevisionDeployment(apiRevisionId, apiRevisionDeployments); }
Source:
<code>public void process(Exchange exchange) { String issueKey = exchange.getIn().getHeader(ISSUE_KEY, String.class); if (issueKey == null) { throw new IllegalArgumentException("Missing exchange input header named " + ISSUE_KEY); } String comment = exchange.getIn().getBody(String.class); if (comment == null) { throw new IllegalArgumentException("Missing exchange body, it should specify the string comment."); } <START> Integer minutesSpent = exchange.getIn().getHeader(MINUTES_SPENT, Integer.class); <END> if (null == minutesSpent) { throw new IllegalArgumentException("Missing exchange input header named " + MINUTES_SPENT); } JiraRestClient client = ((JiraEndpoint) getEndpoint()).getClient(); IssueRestClient issueClient = client.getIssueClient(); Issue issue = issueClient.getIssue(issueKey).claim(); WorklogInput worklogInput = WorklogInput .create(issue.getSelf(), comment, new DateTime(), minutesSpent); issueClient.addWorklog(issue.getWorklogUri(), worklogInput); }</code><technical_language>primitive instead of boxed Integer</technical_language>

Target:
public void process(Exchange exchange) { String issueKey = exchange.getIn().getHeader(ISSUE_KEY, String.class); if (issueKey == null) { throw new IllegalArgumentException("Missing exchange input header named " + ISSUE_KEY); } String comment = exchange.getIn().getBody(String.class); if (comment == null) { throw new IllegalArgumentException("Missing exchange body, it should specify the string comment."); } int minutesSpent = exchange.getIn().getHeader(MINUTES_SPENT, DEFAULT_MINUTES_SPENT, int.class); if (DEFAULT_MINUTES_SPENT == minutesSpent) { throw new IllegalArgumentException("Missing exchange input header named " + MINUTES_SPENT); } JiraRestClient client = ((JiraEndpoint) getEndpoint()).getClient(); IssueRestClient issueClient = client.getIssueClient(); Issue issue = issueClient.getIssue(issueKey).claim(); WorklogInput worklogInput = WorklogInput.create(issue.getSelf(), comment, new DateTime(), minutesSpent); issueClient.addWorklog(issue.getWorklogUri(), worklogInput); }
Source:
<code>protected void reloadCache() { schemaMappingCache.invalidateAll(); Set<String> schemas = getSchemaNames(); for (String key : schemaTableMapping.keySet()) { schemaTableMapping.get(key).invalidateAll(); schemaTableMapping.remove(key); <START> } <END> for (final String schema : schemas) { getTableNames(schema); } }</code><technical_language>reason iterate values? schemaTableMapping.values().foreach(v -> v.invalidateAll()); schemaTableMapping.clear();</technical_language>

Target:
protected void reloadCache() { schemaMappingCache.invalidateAll(); Set<String> schemas = getSchemaNames(); schemaTableMapping.keySet().forEach(key -> { schemaTableMapping.get(key).invalidateAll(); }); schemaTableMapping.clear(); for (final String schema : schemas) { getTableNames(schema); } }
Source:
<code>default void addPlugins( AbstractJavacPluginParams.Builder builder, BuildRuleResolver resolver, BuildTarget owner, AbstractJavacPluginProperties.Type type) { for (BuildTarget pluginTarget : getPlugins()) { BuildRule pluginRule = resolver.getRule(pluginTarget); if (!(pluginRule instanceof JavacPlugin)) { throw new HumanReadableException( String.format( "%s: only java_annotation_processor or javac_plugin rules can be specified " <START> + "as plugins. %s is not a java_annotation_processor nor javac_plugin.", <END> owner, pluginTarget)); } JavacPlugin javacPluginRule = (JavacPlugin) pluginRule; if (javacPluginRule.getUnresolvedProperties().getType() == type) { builder.addPluginProperties(javacPluginRule.getPluginProperties()); } } }</code><technical_language>java_plugin in places</technical_language>

Target:
default void addPlugins( AbstractJavacPluginParams.Builder builder, BuildRuleResolver resolver, BuildTarget owner, AbstractJavacPluginProperties.Type type) { for (BuildTarget pluginTarget : getPlugins()) { BuildRule pluginRule = resolver.getRule(pluginTarget); if (!(pluginRule instanceof JavacPlugin)) { throw new HumanReadableException( String.format( "%s: only java_annotation_processor or java_plugin rules can be specified " + "as plugins. %s is not a java_annotation_processor nor java_plugin.", owner, pluginTarget)); } JavacPlugin javacPluginRule = (JavacPlugin) pluginRule; if (javacPluginRule.getUnresolvedProperties().getType() == type) { builder.addPluginProperties(javacPluginRule.getPluginProperties()); } } }
Source:
<code><START> public HomePagePresenter(EventBus eventBus, MyView view, MyProxy proxy) { <END> super(eventBus, view, proxy, ApplicationPresenter.SLOT_SetMainContent); }</code><technical_language>Clean "public"</technical_language>

Target:
HomePagePresenter(EventBus eventBus, MyView view, MyProxy proxy) { super(eventBus, view, proxy, ApplicationPresenter.SLOT_SetMainContent); }
Source:
<code>private ArtifactoryHttpClient(String artifactoryUrl, String username, String password, String accessToken, Log log) { this.artifactoryUrl = StringUtils.stripEnd(artifactoryUrl, "/"); this.username = username; this.password = password; this.accessToken = accessToken; this.log = log; <START> this.insecureTls = false; <END> }</code><technical_language>set false declaration</technical_language>

Target:
private ArtifactoryHttpClient(String artifactoryUrl, String username, String password, String accessToken, Log log) { this.artifactoryUrl = StringUtils.stripEnd(artifactoryUrl, "/"); this.username = username; this.password = password; this.accessToken = accessToken; this.log = log; }
Source:
<code>public void getAssociationLnObjects(final DeviceMessageMetadata deviceMessageMetadata, <START> final com.alliander.osgp.domain.core.valueobjects.smartmetering.GetAssociationLnObjectsRequest request) throws FunctionalException { <END> LOGGER.debug("getAssociationLnObjects for organisationIdentification: for deviceIdentification: {}", deviceMessageMetadata.getOrganisationIdentification(), deviceMessageMetadata.getDeviceIdentification()); final SmartMeter smartMeteringDevice = this.domainHelperService.findSmartMeter(deviceMessageMetadata .getDeviceIdentification()); final GetAssociationLnObjectsRequest requestDto = new GetAssociationLnObjectsRequest(); this.osgpCoreRequestMessageSender.send(new RequestMessage(deviceMessageMetadata.getCorrelationUid(), deviceMessageMetadata.getOrganisationIdentification(), deviceMessageMetadata.getDeviceIdentification(), smartMeteringDevice.getIpAddress(), requestDto), deviceMessageMetadata.getMessageType(), deviceMessageMetadata.getMessagePriority(), deviceMessageMetadata.getScheduleTime()); }</code><technical_language>![MAJOR](<LINK_0> Remove this unused method parameter "request". [![rule](<LINK_1>](<LINK_2></technical_language>

Target:
public void getAssociationLnObjects(final DeviceMessageMetadata deviceMessageMetadata, final com.alliander.osgp.domain.core.valueobjects.smartmetering.GetAssociationLnObjectsRequest request) throws FunctionalException { LOGGER.debug("getAssociationLnObjects for organisationIdentification: for deviceIdentification: {}", deviceMessageMetadata.getOrganisationIdentification(), deviceMessageMetadata.getDeviceIdentification()); final SmartMeter smartMeteringDevice = this.domainHelperService.findSmartMeter(deviceMessageMetadata .getDeviceIdentification()); final GetAssociationLnObjectsRequestDto requestDto = new GetAssociationLnObjectsRequestDto(); this.osgpCoreRequestMessageSender.send(new RequestMessage(deviceMessageMetadata.getCorrelationUid(), deviceMessageMetadata.getOrganisationIdentification(), deviceMessageMetadata.getDeviceIdentification(), smartMeteringDevice.getIpAddress(), requestDto), deviceMessageMetadata.getMessageType(), deviceMessageMetadata.getMessagePriority(), deviceMessageMetadata.getScheduleTime()); }
Source:
<code>public void saveCurrentPropertyForAll() { boolean error = false; List<Property> ppList = getProperties(); for (Property pp : ppList) { this.property = pp; Property processProperty = new Property(); processProperty.setTitle(this.property.getTitle()); processProperty.setValue(this.property.getValue()); for (Task task : this.steps) { Process process = task.getProcess(); if (!task.equals(this.currentStep)) { process = prepareProcessWithProperty(process, processProperty); } else { if (!process.getProperties().contains(this.property)) { process.getProperties().add(this.property); } } List<Property> props = process.getProperties(); for (Property nextProcessProperty : props) { if (nextProcessProperty.getTitle() == null) { process.getProperties().remove(nextProcessProperty); } } try { this.serviceManager.getProcessService().save(process); } catch (DataException e) { error = true; <START> logger.error(e.getMessage(), e); <END> Helper.setFehlerMeldung("Properties for process " + process.getTitle() + " could not be saved"); } } } if (!error) { Helper.setMeldung("propertySaved"); } }</code><technical_language>setErrorMessage</technical_language>

Target:
public void saveCurrentPropertyForAll() { List<Property> ppList = getProperties(); for (Property pp : ppList) { this.property = pp; Property processProperty = new Property(); processProperty.setTitle(this.property.getTitle()); processProperty.setValue(this.property.getValue()); for (Task task : this.steps) { Process process = task.getProcess(); if (!task.equals(this.currentStep)) { process = prepareProcessWithProperty(process, processProperty); } else { if (!process.getProperties().contains(this.property)) { process.getProperties().add(this.property); } } List<Property> props = process.getProperties(); for (Property nextProcessProperty : props) { if (nextProcessProperty.getTitle() == null) { process.getProperties().remove(nextProcessProperty); } } try { this.serviceManager.getProcessService().save(process); Helper.setMeldung("propertySaved"); } catch (DataException e) { Helper.setErrorMessage("Properties for process " + process.getTitle() + " could not be saved", logger, e); } } } }
Source:
<code>private void ensureSize() { while (true) { long addressToRemove; Lock entryWriteLock; lruLock.lock(); try { if (currentSize <= maxSize) { break; } int hashCode = UNSAFE.getInt(firstAddress + 24); entryWriteLock = locks.getLockFromHashCode(hashCode).writeLock(); if (!entryWriteLock.tryLock()) { addressToRemove = 0; } else { addressToRemove = UNSAFE.getLong(firstAddress); } } finally { lruLock.unlock(); } if (addressToRemove == 0) { entryWriteLock.lock(); try { lruLock.lock(); try { if (currentSize <= maxSize) { break; } int hashCode = UNSAFE.getInt(firstAddress + 24); Lock innerLock = locks.getLockFromHashCode(hashCode).writeLock(); if (innerLock == entryWriteLock) { addressToRemove = UNSAFE.getLong(firstAddress); } else { addressToRemove = 0; } } finally { lruLock.unlock(); } } finally { if (addressToRemove == 0) { entryWriteLock.unlock(); } } } if (addressToRemove != 0) { if (trace) { log.tracef("Removing entry: %d due to eviction due to size %d being larger than maximum of %d", addressToRemove, currentSize, maxSize); } try { InternalCacheEntry<WrappedBytes, WrappedBytes> ice = offHeapEntryFactory.fromMemory(addressToRemove); <START> passivator.passivate(ice); <END> performRemove(addressToRemove, offHeapEntryFactory.getKey(addressToRemove)); evictionManager.onEntryEviction(Collections.singletonMap(ice.getKey(), ice)); } finally { entryWriteLock.unlock(); } } } }</code><technical_language>I if move passivate call unlock entryWriteLock better. This block other heap reads writes for lock while performing passivation, a while. hold ISPN lock for key fine</technical_language>

Target:
private void ensureSize() { while (true) { long addressToRemove; Lock entryWriteLock; lruLock.lock(); try { if (currentSize <= maxSize) { break; } int hashCode = UNSAFE.getInt(firstAddress + 24); entryWriteLock = locks.getLockFromHashCode(hashCode).writeLock(); if (!entryWriteLock.tryLock()) { addressToRemove = 0; } else { addressToRemove = UNSAFE.getLong(firstAddress); } } finally { lruLock.unlock(); } if (addressToRemove == 0) { entryWriteLock.lock(); try { lruLock.lock(); try { if (currentSize <= maxSize) { break; } int hashCode = UNSAFE.getInt(firstAddress + 24); Lock innerLock = locks.getLockFromHashCode(hashCode).writeLock(); if (innerLock == entryWriteLock) { addressToRemove = UNSAFE.getLong(firstAddress); } else { addressToRemove = 0; } } finally { lruLock.unlock(); } } finally { if (addressToRemove == 0) { entryWriteLock.unlock(); } } } if (addressToRemove != 0) { if (trace) { log.tracef("Removing entry: %d due to eviction due to size %d being larger than maximum of %d", addressToRemove, currentSize, maxSize); } InternalCacheEntry<WrappedBytes, WrappedBytes> ice = null; try { ice = offHeapEntryFactory.fromMemory(addressToRemove); performRemove(addressToRemove, offHeapEntryFactory.getKey(addressToRemove)); evictionManager.onEntryEviction(Collections.singletonMap(ice.getKey(), ice)); } finally { entryWriteLock.unlock(); if (ice != null) { passivator.passivate(ice); } } } } }
Source:
<code>public boolean signalContainer(ContainerSignalContext ctx) throws IOException { String pid = ctx.getPid(); Signal signal = ctx.getSignal(); <START> LOG.debug("Sending signal to pid as user {}", <END> signal.getValue(), pid, ctx.getUser()); if (!containerIsAlive(pid)) { return false; } try { killContainer(pid, signal); } catch (IOException e) { if (!containerIsAlive(pid)) { return false; } throw e; } return true; }</code><technical_language>debug is guarded, cost</technical_language>

Target:
public boolean signalContainer(ContainerSignalContext ctx) throws IOException { String user = ctx.getUser(); String pid = ctx.getPid(); Signal signal = ctx.getSignal(); LOG.debug("Sending signal to pid as user {}", signal.getValue(), pid, user); if (!containerIsAlive(pid)) { return false; } try { killContainer(pid, signal); } catch (IOException e) { if (!containerIsAlive(pid)) { return false; } throw e; } return true; }
Source:
<code><START> private String runSshCommand(URIish sshUri, FS fs, String command) <END> throws IOException { RemoteSession session = null; Process process = null; StreamCopyThread errorThread = null; try (MessageWriter stderr = new MessageWriter()) { session = SshSessionFactory.getInstance().getSession(sshUri, null, fs, 5_000); process = session.exec(command, 0); errorThread = new StreamCopyThread(process.getErrorStream(), stderr.getRawStream()); errorThread.start(); try (BufferedReader reader = new BufferedReader( new InputStreamReader(process.getInputStream(), Constants.CHARSET))) { return reader.readLine(); } } finally { if (errorThread != null) { try { errorThread.halt(); } catch (InterruptedException e) { } finally { errorThread = null; } } if (process != null) { process.destroy(); } if (session != null) { SshSessionFactory.getInstance().releaseSession(session); } } }</code><technical_language>OK. sounds good. step put code parameter, I happily do work :)</technical_language>

Target:
private String runSshCommand(URIish sshUri, FS fs, String command) throws IOException { RemoteSession session = null; Process process = null; StreamCopyThread errorThread = null; try (MessageWriter stderr = new MessageWriter()) { session = SshSessionFactory.getInstance().getSession(sshUri, null, fs, 5_000); process = session.exec(command, 0); errorThread = new StreamCopyThread(process.getErrorStream(), stderr.getRawStream()); errorThread.start(); try (BufferedReader reader = new BufferedReader( new InputStreamReader(process.getInputStream(), Constants.CHARSET))) { return reader.readLine(); } } finally { if (process != null) { process.destroy(); } if (errorThread != null) { try { errorThread.halt(); } catch (InterruptedException e) { } finally { errorThread = null; } } if (session != null) { SshSessionFactory.getInstance().releaseSession(session); } } }
Source:
<code><START> public Map<String, CacheEntry> getLayerBuildMap() { <END> return _layerBuilds.getMap(); }</code><technical_language>Restore visibility of fields methods in this class package level unit tests moved jaggr-core</technical_language>

Target:
Map<String, CacheEntry> getLayerBuildMap() { return _layerBuilds.getMap(); }
Source:
<code>public Collection<AccountGroup.Id> missing(final AccountGroup.Id key) { <START> return new ArrayList<AccountGroup.Id> (); <END> }</code><technical_language>Collections.emptyList();</technical_language>

Target:
public Collection<AccountGroup.Id> missing(final AccountGroup.Id key) { return Collections.emptyList(); }