Bug 1447 - add support for pre-emphasis option for FLAC cue sheets
: add support for pre-emphasis option for FLAC cue sheets
Status: RESOLVED WONTFIX
Product: Logitech Media Server
Classification: Unclassified
Component: Formats
: 6.1.0
: All All
: P4 enhancement with 2 votes (vote)
: 7.4.0
Assigned To: Unassigned bug - please assign me!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2005-04-26 16:51 UTC by Kevin Pearsall
Modified: 2009-10-26 20:59 UTC (History)
2 users (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Kevin Pearsall 2005-04-26 16:51:11 UTC
customer requested--some notes from him:
---
I was playing a whole album FLAC with an external cue sheet
(very nice that this works!) and noticed that
a track with FLAGS PRE (preemphasis) didn't decode with
deemphasis set on my external DAC.
---
Some (rare) CDs are encoded with pre-emphasis.  This,
in effect, runs the sound through an equilization curve (when recorded) that must
be undone when you play the CDs back (or they will sound shrill).
There is a bit on the cd that indicates this.  When you extract a CD
to a cue sheet, this shows up as "FLAGS PRE" on a per track basis.

CD players do the deemphasis automatically as it is part of the red-book standard.

I'm not sure of which audio file formats support storing this preemphasis
bit.  WAV certainly can't.  mp3 might depend on the encoder used
(I think lame does, see -e option).  I couldn't find an option in FLAC...

In any case, since you can play using a cue sheet (a *great* feature)
I think SS should using the FLAGS PRE as specified in the cue sheet.

In grep'ing around the slimserver sources, I didn't find any mention
pre emphasis feature.  So, I'm guessing that you don't handle it

However, it is checked for in the CPAN module that you use for FLAC.
Comment 1 James Richardson 2009-06-09 11:51:15 UTC
This is not feasible
Comment 2 George Hines 2009-06-09 12:53:11 UTC
Why is it not feasible / won't be fixed?

Since sox is already being used to trans code whole album cue/bin files, isn't it as simple as adding the "deemph" argument to the sox command line for tracks that have the "PRE" flag set?
Comment 3 George Hines 2009-10-26 20:59:32 UTC
FYI: how to do this, server side:

http://forums.slimdevices.com/showthread.php?t=70256