Bug 1029 - Server should send STREAMINFO header with FLAC stream
: Server should send STREAMINFO header with FLAC stream
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Formats
: 6.0.0
: PC Windows XP
: P2 normal (vote)
: ---
Assigned To: Richard Titmuss
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2005-03-10 14:55 UTC by Vidur Apparao
Modified: 2008-08-18 10:53 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 Vidur Apparao 2005-03-10 14:55:24 UTC
Richard, it seems like there's little harm in sending the STREAMINFO to the
player. I know you've tried this to get FLAC working with Softsqueeze. Care to
commit your patch?
Comment 1 Blackketter Dean 2005-03-10 14:57:05 UTC
Richard, can you try changing FLAC.pm line 152 to:

	$tags->{'OFFSET'}  = 0;  # the header info is an important part of the file.  don't skip it. # $flac-
>{'startAudioData'};

and see if this resolves your issue.  Since the flac info headers are a valid part of the flac bitstream (and 
all decoders can interpret them) we should stream them out.
Comment 2 Richard Titmuss 2005-03-10 15:05:16 UTC
Dean, you've already found my patch! This does not really solve the problem with
Softsqueeze. This will work if you play the track normally, but not if you use
cue sheets, or fwd/rew through the stream.

If the STREAMINFO was append to all the start of all streams, even if you play
from the middle of a flac that would work for me, but I am not sure it is the
right thing to do. And I don't have a patch for that yet. Comments?
Comment 3 Blackketter Dean 2005-03-11 17:50:47 UTC
Feel free to commit that patch.  Would like vidur to comment on sending streaminfo on flac stream 
fragments.  I wonder if we should send this metadata as part of the play command or if it should be 
inband.  I'm concerned about whether we should be slicing and dicing FLAC streams....  Why not?