Skip to content

Commit b59df20

Browse files
authored
Convert font processor to use PluginLogger (#8549)
Part of #8369 Now that debug logs are logged in verbose mode, we don't need conditional logger creation in this file.
1 parent 447759a commit b59df20

File tree

1 file changed

+8
-12
lines changed

1 file changed

+8
-12
lines changed

src/io/flutter/font/FontPreviewProcessor.java

Lines changed: 8 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@
3232
import com.jetbrains.lang.dart.util.DartResolveUtil;
3333
import io.flutter.FlutterBundle;
3434
import io.flutter.editor.FlutterIconLineMarkerProvider;
35+
import io.flutter.logging.PluginLogger;
3536
import io.flutter.settings.FlutterSettings;
3637
import io.flutter.utils.OpenApiUtils;
3738
import org.jetbrains.annotations.NotNull;
@@ -61,7 +62,7 @@ public class FontPreviewProcessor {
6162
private static final Pattern IMPORT_STATEMENT_PATTERN = Pattern.compile("^\\s*import\\s+[\"']([-_. $A-Za-z0-9/]+\\.dart)[\"'].*");
6263
private static final Map<String, Set<String>> ANALYZED_PROJECT_FILES = new HashMap<>();
6364
private static final Map<String, WorkItem> WORK_ITEMS = new HashMap<>();
64-
private static Logger LOG = Logger.getInstance(FontPreviewProcessor.class);
65+
private static final @NotNull Logger LOG = PluginLogger.createLogger(FontPreviewProcessor.class);
6566

6667
static {
6768
UNSUPPORTED_PACKAGES.put("flutter_icons", FlutterBundle.message("icon.preview.disallow.flutter_icons"));
@@ -83,7 +84,6 @@ public void generate(@NotNull Project project) {
8384
if (ANALYZED_PROJECT_FILES.containsKey(project.getBasePath())) {
8485
return;
8586
}
86-
LOG = FlutterSettings.getInstance().isVerboseLogging() ? Logger.getInstance(FontPreviewProcessor.class) : null;
8787
log("Analyzing project ", project.getName());
8888
ANALYZED_PROJECT_FILES.put(project.getBasePath(), new HashSet<>());
8989
ProjectManager.getInstance().addProjectManagerListener(project, new ProjectManagerListener() {
@@ -436,20 +436,16 @@ private static boolean isInSdk(String path) {
436436
}
437437

438438
private static void log(String msg, String... msgs) {
439-
if (LOG != null) {
440-
final StringBuilder b = new StringBuilder("ICONS -- ");
441-
b.append(msg);
442-
for (String s : msgs) {
443-
b.append(" ").append(s);
444-
}
445-
LOG.info(b.toString());
439+
final StringBuilder b = new StringBuilder("ICONS -- ");
440+
b.append(msg);
441+
for (String s : msgs) {
442+
b.append(" ").append(s);
446443
}
444+
LOG.debug(b.toString());
447445
}
448446

449447
private static void log(String msg, Exception ex) {
450-
if (LOG != null) {
451-
LOG.info("ICONS--" + msg, ex);
452-
}
448+
LOG.debug("ICONS--" + msg, ex);
453449
}
454450

455451
static class WorkItem {

0 commit comments

Comments
 (0)