[MacOS Catalina] NPE: Couldn't execute theme name query with the Os

Question

[MacOS Catalina] NPE: Couldn't execute theme name query with the Os

DrDaleks opened this issue 3 years ago · comments

Hi,
Just noticed this NPE, wondered if this is normal or an OS-related issue somehow.
My code calls OsThemeDetector.getDetector().isDark() at some point, and the full stack trace reads

[AWT-EventQueue-0] ERROR com.jthemedetecor.MacOSThemeDetector - Couldn't execute theme name query with the Os
java.lang.NullPointerException
	at java.base/java.util.regex.Matcher.getTextLength(Matcher.java:1770)
	at java.base/java.util.regex.Matcher.reset(Matcher.java:416)
	at java.base/java.util.regex.Matcher.<init>(Matcher.java:253)
	at java.base/java.util.regex.Pattern.matcher(Pattern.java:1133)
	at com.jthemedetecor.MacOSThemeDetector.isDarkTheme(MacOSThemeDetector.java:96)
	at com.jthemedetecor.MacOSThemeDetector.isDark(MacOSThemeDetector.java:86)

More specifically, the NPE is due to the following call (within the isDark() method) returning the '0' byte:

Foundation.invoke(userDefaults, "objectForKey:", Foundation.nsString("AppleInterfaceStyle"))

I should mention that this happens during daytime (and luckily isDark() returns false), so no harm done (besides an avoidable(?) pollution of the logger)

Any clue where that might come from?

In case this helps: using version 3.7 @ openjdk11 @ macOS 10.15.7

rosamarco · Answer 1 · Wed Dec 15 2021 08:53:39 GMT+0800 (China Standard Time)

Got the same error on Monterey 12.1

Chris Turner · Answer 2 · Wed Dec 22 2021 03:25:49 GMT+0800 (China Standard Time)

Also facing same error on Monterey 12.1

ctipper · Answer 3 · Fri Jan 07 2022 22:48:58 GMT+0800 (China Standard Time)

+1 Big Sur, JDK 17
I wonder if the matcher pattern catches the case where system Auto theme is set

ctipper · Answer 4 · Sat Jan 08 2022 05:33:02 GMT+0800 (China Standard Time)

Developer seems to have explored this issue:

For working with the new macOS Catalina you need to combine AppleInterfaceStyle with this new value introduced AppleInterfaceStyleSwitchesAutomatically

https://stackoverflow.com/a/57429660

with a reference to https://github.com/ruiaureliano/macOS-Appearance

with apologies for possible impertinence

ctipper · Answer 5 · Sat Apr 09 2022 17:33:08 GMT+0800 (China Standard Time)

This library would be useful if this issue were fixed. With reference to the example project referenced above the developer quietly updated his project with the correct property for macOS > 10.15 NSApplication.shared.effectiveAppearance.debugDescription A potential PR would need to check platform version.

https://github.com/ruiaureliano/macOS-Appearance/../AppDelegate.swift#L50

Matt Groth · Answer 6 · Sun May 22 2022 07:32:50 GMT+0800 (China Standard Time)

In the meantime, does anyone know how to silence the logger?

I don't fully understand how the logger works. I've tried using

java.util.logging.Logger.getLogger("com.jthemedetecor.MacOSThemeDetector").level = Level.OFF
	java.util.logging.Logger.getLogger("com.jthemedetecor").level = Level.OFF

in my code, but that does nothing.

Whenever I search "how to disable a org.slf4j.Logger" people are talking about things like log4j. It's very confusing. I don't think I have log4j installed, but just in case I tried adding a "log4j.properties" to my classpath with log4j.logger.com.jthemedetecor=OFF but that is not doing anything either.

ctipper · Answer 7 · Sun May 22 2022 16:48:42 GMT+0800 (China Standard Time)

It sounds like you don't want logging at all, in which case you should add sl4j-nop (no-op) logger to your dependencies. I only know log4j with that if you have a logging configuration (log4j2.xml) on your classpath it should ignore libraries that are not configured. This library uses logback with sl4j I have no knowledge of that but it shouldn't matter if you are not using it.

Matt Groth · Answer 8 · Mon May 23 2022 13:10:43 GMT+0800 (China Standard Time)

But what if I just want to disable logging for jthemedetecor without disabling logging for other libraries?

ctipper · Answer 9 · Mon May 23 2022 20:03:32 GMT+0800 (China Standard Time)

You should try to configure logging and then you can selectively enable libraries, what you were doing is unlikely to work.

…

-- Sent with Spark

On 23 May 2022, 06:10 +0100, Matt Groth ***@***.***>, wrote: But what if I just want to disable logging for jthemedetecor without disabling logging for other libraries? — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

David Zhang · Answer 10 · Sun Jul 03 2022 19:35:39 GMT+0800 (China Standard Time)

12.4 JDK18 same thing

David Zhang · Answer 11 · Mon Jul 04 2022 12:11:36 GMT+0800 (China Standard Time)

In the meantime, does anyone know how to silence the logger?

I don't fully understand how the logger works. I've tried using
java.util.logging.Logger.getLogger("com.jthemedetecor.MacOSThemeDetector").level = Level.OFF
	java.util.logging.Logger.getLogger("com.jthemedetecor").level = Level.OFF
in my code, but that does nothing.

Whenever I search "how to disable a org.slf4j.Logger" people are talking about things like log4j. It's very confusing. I don't think I have log4j installed, but just in case I tried adding a "log4j.properties" to my classpath with log4j.logger.com.jthemedetecor=OFF but that is not doing anything either.

use the following two lines in log4j2.properties

logger.jthemedetecor.name = com.jthemedetecor
logger.jthemedetecor.level = off

reference

Victor Ewert · Answer 12 · Thu Jan 19 2023 06:30:21 GMT+0800 (China Standard Time)

In the meantime, does anyone know how to silence the logger?
I don't fully understand how the logger works. I've tried using
java.util.logging.Logger.getLogger("com.jthemedetecor.MacOSThemeDetector").level = Level.OFF
	java.util.logging.Logger.getLogger("com.jthemedetecor").level = Level.OFF
in my code, but that does nothing.
Whenever I search "how to disable a org.slf4j.Logger" people are talking about things like log4j. It's very confusing. I don't think I have log4j installed, but just in case I tried adding a "log4j.properties" to my classpath with log4j.logger.com.jthemedetecor=OFF but that is not doing anything either.
use the following two lines in log4j2.properties
logger.jthemedetecor.name = com.jthemedetecor
logger.jthemedetecor.level = off
reference

In case anyone is using log4j2.xml style properties you can use this:

<Loggers>
    <Logger name="com.jthemedetecor" level="error">
    </Logger>
    <Root level="info">
      ...
    </Root>
</Loggers>