Bug - Fixed Debug log when Rollover comes

slyz

Developer
I generally leave my computer on during the night, and I often leave Mafia running. Today, I had the following debug log repeated 1000+ times in the 3 hours between the start of rollover and the moment I closed Mafia:
Code:
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
         KoLmafia v15.1 r10925, Windows XP, Java 1.6.0_24
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
 Please note: do not post this log in the KoLmafia thread. If you
 would like the dev team to look at it, please write a bug report
 at kolmafia.us. Include specific information about what you were 
 doing when you made this and include the log as an attachment.
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
 Timestamp: Thu Apr 19 07:58:43 CEST 2012
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=


Unexpected error, debug log printed.
class java.lang.NullPointerException: null
java.lang.NullPointerException
	at java.util.regex.Matcher.getTextLength(Unknown Source)
	at java.util.regex.Matcher.reset(Unknown Source)
	at java.util.regex.Matcher.<init>(Unknown Source)
	at java.util.regex.Pattern.matcher(Unknown Source)
	at net.sourceforge.kolmafia.chat.ChatParser.parsePlayerIds(ChatParser.java:353)
	at net.sourceforge.kolmafia.chat.ChatParser.parseLines(ChatParser.java:192)
	at net.sourceforge.kolmafia.chat.HistoryEntry.<init>(HistoryEntry.java:84)
	at net.sourceforge.kolmafia.chat.ChatPoller.getEntries(ChatPoller.java:178)
	at net.sourceforge.kolmafia.chat.ChatPoller.run(ChatPoller.java:73)

I guess this was introduced with r10922. I'll have a look at it later on, unless someone beats me to it.
 

Veracity

Developer
Staff member
It happened to me a couple of days ago. My guess it came from the fixes to support KoL tabbed chat.
 

Grotfang

Developer
It happened to me a couple of days ago. My guess it came from the fixes to support KoL tabbed chat.

Shouldn't have done. That doesn't use ChatPoller. I think it's more likely to what Slyz says about Hola's id check. Still working on updating chat, but having to fit it around revision. It'll be a while yet before tabbed chat is properly integrated outside of the relay browser.
 
Top