Skip to content

Commit 95ca0a5

Browse files
Small readability improvement
IMHO, just keeping a single insertIdx variable is slightly cleaner.
1 parent fa47c64 commit 95ca0a5

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

app/src/processing/app/Base.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1623,13 +1623,14 @@ private void rebuildRecentBoardsList() throws Exception {
16231623
while (!(boardMenu.getMenuComponent(recentBoardsJMenuIndex) instanceof JSeparator)) {
16241624
boardMenu.remove(recentBoardsJMenuIndex);
16251625
}
1626+
1627+
int insertIdx = recentBoardsJMenuIndex;
16261628
for (String boardId : recentBoardIds) {
16271629
JRadioButtonMenuItem originalItem = boardItems.get(boardId);
16281630
JRadioButtonMenuItem recentItem = new JRadioButtonMenuItem(originalItem.getAction());
1629-
1630-
boardMenu.add(recentItem, recentBoardsJMenuIndex+idxAdv);
16311631
recentItem.setSelected(boardId.equals(currentBoard));
1632-
idxAdv++;
1632+
1633+
boardMenu.add(recentItem, insertIdx++);
16331634
}
16341635
}
16351636

0 commit comments

Comments
 (0)