Skip to content

Commit db63c7c

Browse files
committed
Optimize imports and resolve Intellij warnings.
1 parent 240df37 commit db63c7c

1 file changed

Lines changed: 40 additions & 41 deletions

File tree

  • sierra-tools/previewer/src/main/java/org/httprpc/sierra/tools/previewer

sierra-tools/previewer/src/main/java/org/httprpc/sierra/tools/previewer/MainFrame.java

Lines changed: 40 additions & 41 deletions
Original file line numberDiff line numberDiff line change
@@ -13,16 +13,22 @@
1313
*/
1414
package org.httprpc.sierra.tools.previewer;
1515

16-
import java.awt.BorderLayout;
17-
import java.awt.Toolkit;
18-
import java.awt.event.KeyEvent;
19-
import java.io.File;
20-
import java.io.IOException;
21-
import java.nio.file.Files;
22-
import java.nio.file.Path;
23-
import java.nio.file.StandardOpenOption;
24-
import java.util.concurrent.ExecutionException;
25-
import java.util.function.Consumer;
16+
import org.fife.rsta.ui.search.FindDialog;
17+
import org.fife.rsta.ui.search.ReplaceDialog;
18+
import org.fife.rsta.ui.search.SearchEvent;
19+
import org.fife.rsta.ui.search.SearchListener;
20+
import org.fife.ui.autocomplete.AutoCompletion;
21+
import org.fife.ui.autocomplete.CompletionProvider;
22+
import org.fife.ui.rsyntaxtextarea.RSyntaxTextArea;
23+
import org.fife.ui.rsyntaxtextarea.SyntaxConstants;
24+
import org.fife.ui.rtextarea.SearchEngine;
25+
import org.fife.ui.rtextarea.SearchResult;
26+
import org.httprpc.sierra.Outlet;
27+
import org.httprpc.sierra.UILoader;
28+
import org.httprpc.sierra.tools.previewer.engine.RenderingEngine;
29+
import org.httprpc.sierra.tools.previewer.model.RenderError;
30+
import org.httprpc.sierra.tools.previewer.model.RenderResult;
31+
2632
import javax.swing.ImageIcon;
2733
import javax.swing.JFileChooser;
2834
import javax.swing.JFrame;
@@ -41,22 +47,16 @@
4147
import javax.swing.event.DocumentEvent;
4248
import javax.swing.event.DocumentListener;
4349
import javax.swing.filechooser.FileNameExtensionFilter;
44-
import org.fife.rsta.ui.search.FindDialog;
45-
import org.fife.rsta.ui.search.ReplaceDialog;
46-
import org.fife.rsta.ui.search.SearchEvent;
47-
import org.fife.rsta.ui.search.SearchListener;
48-
import org.fife.ui.autocomplete.AutoCompletion;
49-
import org.fife.ui.autocomplete.CompletionProvider;
50-
import org.fife.ui.rsyntaxtextarea.RSyntaxTextArea;
51-
import org.fife.ui.rsyntaxtextarea.SyntaxConstants;
52-
import org.fife.ui.rtextarea.SearchContext;
53-
import org.fife.ui.rtextarea.SearchEngine;
54-
import org.fife.ui.rtextarea.SearchResult;
55-
import org.httprpc.sierra.Outlet;
56-
import org.httprpc.sierra.UILoader;
57-
import org.httprpc.sierra.tools.previewer.engine.RenderingEngine;
58-
import org.httprpc.sierra.tools.previewer.model.RenderError;
59-
import org.httprpc.sierra.tools.previewer.model.RenderResult;
50+
import java.awt.BorderLayout;
51+
import java.awt.Toolkit;
52+
import java.awt.event.KeyEvent;
53+
import java.io.File;
54+
import java.io.IOException;
55+
import java.nio.file.Files;
56+
import java.nio.file.Path;
57+
import java.nio.file.StandardOpenOption;
58+
import java.util.concurrent.ExecutionException;
59+
import java.util.function.Consumer;
6060

6161
/**
6262
* The main application window for the Sierra UI Previewer. UI is defined in
@@ -79,7 +79,6 @@ public class MainFrame extends JFrame implements SearchListener {
7979
private @Outlet JMenuBar menuBar = null;
8080
private @Outlet JMenuItem openItem = null;
8181
private @Outlet JMenuItem saveItem = null;
82-
private @Outlet JMenu searchMenu = null;
8382
private @Outlet JMenuItem findItem = null;
8483
private @Outlet JMenuItem replaceItem = null;
8584
private @Outlet JMenu recentMenu = null;
@@ -138,7 +137,7 @@ public void changedUpdate(DocumentEvent e) {
138137
var iconURL = getClass().getResource("/sierra.png");
139138
var icon = new ImageIcon(iconURL).getImage();
140139
setIconImage(icon);
141-
140+
142141
triggerRender();
143142
}
144143

@@ -215,19 +214,19 @@ private CompletionProvider createCompletionProvider() {
215214

216215
// --- Editor Setup ---
217216
/**
218-
* Creates the custom RSyntaxTextArea/associated functionality and adds it
217+
* Creates the custom RSyntaxTextArea/associated functionality and adds it
219218
* to the placeholder that Sierra injected.
220219
*/
221220
private void setupCustomEditor() {
222221
findDialog = new FindDialog(this, this);
223222
replaceDialog = new ReplaceDialog(this, this);
224-
223+
225224
// This ties the properties of the two dialogs together (match case,
226225
// regex, etc.).
227-
SearchContext context = findDialog.getSearchContext();
226+
var context = findDialog.getSearchContext();
228227
replaceDialog.setSearchContext(context);
229-
230-
int acceleratorKey = Toolkit.getDefaultToolkit().getMenuShortcutKeyMaskEx();
228+
229+
var acceleratorKey = Toolkit.getDefaultToolkit().getMenuShortcutKeyMaskEx();
231230
findItem.setAccelerator(KeyStroke.getKeyStroke(KeyEvent.VK_F, acceleratorKey));
232231
findItem.addActionListener((e) -> {
233232
if (replaceDialog.isVisible()) {
@@ -262,23 +261,23 @@ private void setupCustomEditor() {
262261

263262
editorScrollPane.setViewportView(editorPane);
264263
}
265-
264+
266265
// -- Extra search/replace functionality
267-
266+
268267
@Override
269268
public String getSelectedText() {
270269
return editorPane.getSelectedText();
271270
}
272-
271+
273272
/**
274273
* Listens for events from our search dialogs and actually does the dirty
275274
* work.
276275
*/
277276
@Override
278277
public void searchEvent(SearchEvent e) {
279278

280-
SearchEvent.Type type = e.getType();
281-
SearchContext context = e.getSearchContext();
279+
var type = e.getType();
280+
var context = e.getSearchContext();
282281
SearchResult result = null;
283282

284283
switch (type) {
@@ -306,11 +305,11 @@ public void searchEvent(SearchEvent e) {
306305
statusBar.setText("Unknown search event");
307306
break;
308307
}
309-
308+
310309
if(result == null){
311310
return;
312311
}
313-
312+
314313
String text;
315314
if (result.wasFound()) {
316315
text = "Text found; occurrences marked: " + result.getMarkedCount();
@@ -326,7 +325,7 @@ public void searchEvent(SearchEvent e) {
326325
statusBar.setText(text);
327326

328327
}
329-
328+
330329
// --- Rendering/Control Logic ---
331330
/**
332331
* Implements the debounce mechanism.

0 commit comments

Comments
 (0)