dev builds
themes manual
device status forums
mailing lists
IRC bugs
dev guide

Rockbox mail archive

Subject: Re: kugel: r21460 - trunk/apps/metadata

Re: kugel: r21460 - trunk/apps/metadata

From: Magnus Holmgren <>
Date: Mon, 22 Jun 2009 20:16:03 +0200


Some nitpicking about a recent commit:

> FS#10364 - Optional debug output for vorbis.c by Jeffrey Goode.
> Gets rid of the (recently added) annyoing vorbis debug statements.
> Comment out "#define LOGF_ENABLE" in vorbis.c to restore them.

Any particular reason to adding a new define, LOGFQUEUE (a name that
makes sense in playback.c, where it is used for queue-related logging),
and only having it do something in simulator builds?

(I thought DEBUGF too was controlled by LOGF_ENABLE, by the way. :) )

Received on 2009-06-22

Page template was last modified "Tue Sep 7 00:00:02 2021" The Rockbox Crew -- Privacy Policy