[Lame-dev] [ lame-Feature Requests-1588283 ] Flushing output stream in lame.exe

[SourceForge.net <no...@so...>]

Feature Requests item #1588283, was opened at 2006-10-31 22:58
Message generated for change (Comment added) made by nobody
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=350290&aid=1588283&group_id=290

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Closed
Priority: 3
Private: No
Submitted By: Andrzej Kuczyski (akuczynski)
Assigned to: Nobody/Anonymous (nobody)
Summary: Flushing output stream in lame.exe

Initial Comment:
I suggest flushing output stream in LAME (lame.exe). 
Encoded data would be avaliable faster. That would 
shorten delay for full-duplex playback (when 
recording data, encoding it, writting to file, 
reading from file, decoding and playing). The 
solution is simple - just add fflush(outf) 
to ./frontend/main.c:

...
        if (fwrite(mp3buffer, 1, imp3, outf) != imp3) 
{
            fprintf(stderr, "Error writing mp3 output 
\n");
            return 1;
        }

	fflush(outf);

    } while (iread);
...

Andrzej

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2011-05-12 14:07

Message:
Index.. Nifty :)

----------------------------------------------------------------------

Comment By: SourceForge Robot (sf-robot)
Date: 2006-12-01 03:20

Message:
Logged In: YES 
user_id=1312539
Originator: NO

This Tracker item was closed automatically by the system. It was
previously set to a Pending status, and the original submitter
did not respond within 14 days (the time period specified by
the administrator of this Tracker).

----------------------------------------------------------------------

Comment By: Robert Hegemann (robert)
Date: 2006-11-08 01:56

Message:
Logged In: YES 
user_id=1106

ok, lame has a new option for flushing the output stream: --
flush

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2006-11-07 13:25

Message:
Logged In: NO 

Adding a command line option would be great :). Now I have to recompile
lame.exe to achieve short full-duplex 
playback delay. Adding a command line option is  the wisest solution :).

Andrzej

----------------------------------------------------------------------

Comment By: Robert Hegemann (robert)
Date: 2006-11-05 16:20

Message:
Logged In: YES 
user_id=1106

I'm considering adding a command line option enabling your 
suggested early flushs. But I wouldn't like to call flush 
always.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=350290&aid=1588283&group_id=290