Unoffical empeg BBS

Quick Links: Empeg FAQ | RioCar.Org | Hijack | BigDisk Builder | jEmplode | emphatic
Repairs: Repairs

Topic Options
#130637 - 12/12/2002 11:11 Bringing up lame --nogap again
wfaulk
carpal tunnel

Registered: 25/12/2000
Posts: 16706
Loc: Raleigh, NC US
I remember from a few months ago that one of the empeg guys (Peter?) had determined that lame's VBR nogap option worked fine if you did the following things:
  1. Hack the lame source to get nogap to work with VBR and to generate VBR headers
  2. Use a utility to fix the VBR headers because the ones that lame writes are incorrect or invalid or corrupt.
Is that all still correct? If so, has someone submitted a bug report to the lame developers that their VBR headers are ... well ... lame?
_________________________
Bitt Faulk

Top
#130638 - 12/12/2002 13:09 Re: Bringing up lame --nogap again [Re: wfaulk]
tfabris
carpal tunnel

Registered: 20/12/1999
Posts: 31597
Loc: Seattle, WA
That's not exactly the way I remember it. Here's how I remember it:

- Lame doesn't put VBR headers on Nogap files.

- We never determined why it behaved this way.

- We guessed it was maybe because VBR headers screwed up gapless playback on some players. Not the empeg with the latest software, but just some players. Maybe. So perhaps that's why they did it. But that's just a guess.

- We found that we could run a VBR-header-adder tool on the files and they would still play right on the car player... PROVIDED that the VBR-header-adder tool didn't have a bug that stripped the last frame off of the file.

I don't think LAME needs any bug fixing...
_________________________
Tony Fabris

Top
#130639 - 12/12/2002 13:28 Re: Bringing up lame --nogap again [Re: tfabris]
wfaulk
carpal tunnel

Registered: 25/12/2000
Posts: 16706
Loc: Raleigh, NC US
The way I remember it is:
  • lame didn't put headers on vbr nogap files because they screwed up nogap playback. The reason that they did that is because they were invalid.
  • A tool was used to add headers to the lame --nogap vbr files, but failed because no header existed.
  • Lame was hacked to include the headers, despite their brokenness.
  • Tool was used to fix broken headers.
_________________________
Bitt Faulk

Top
#130640 - 12/12/2002 13:45 Re: Bringing up lame --nogap again [Re: wfaulk]
wfaulk
carpal tunnel

Registered: 25/12/2000
Posts: 16706
Loc: Raleigh, NC US
Here it is.

Looks like we were both right.

So -- is there any update on this? Is this still the most appropriate way to create accurate nogapped mp3s?
_________________________
Bitt Faulk

Top
#130641 - 12/12/2002 13:47 Re: Bringing up lame --nogap again [Re: wfaulk]
tfabris
carpal tunnel

Registered: 20/12/1999
Posts: 31597
Loc: Seattle, WA
To me, the best way is to just use lame with CBR instead of VBR, then all the problems go away. No fiddling around after the files were created.
_________________________
Tony Fabris

Top
#130642 - 12/12/2002 15:01 Re: Bringing up lame --nogap again [Re: tfabris]
wfaulk
carpal tunnel

Registered: 25/12/2000
Posts: 16706
Loc: Raleigh, NC US
To me, the best way is to fix the broken software.

To that extent, it looks like the problem with lame's vbr header always exists and looks like it might be an off-by-one problem. Probably just not counting in the header as a frame, or something like that.

Looking at it now.
_________________________
Bitt Faulk

Top
#130643 - 12/12/2002 15:59 Re: Bringing up lame --nogap again [Re: wfaulk]
wfaulk
carpal tunnel

Registered: 25/12/2000
Posts: 16706
Loc: Raleigh, NC US
I just sent some email to a lame developer. We'll see what comes of it.
_________________________
Bitt Faulk

Top
#130644 - 13/12/2002 05:11 Re: Bringing up lame --nogap again [Re: wfaulk]
tms13
old hand

Registered: 30/07/2001
Posts: 1115
Loc: Lochcarron and Edinburgh
Here's my patch.

I'm not sure that the generated headers are still wrong in the current code, but I run mp3tool over them anyway, just in case.


Attachments
129477-lame-frontend.diff (59 downloads)

_________________________
Toby Speight
030103016 (80GB Mk2a, blue)
030102806 (0GB Mk2a, blue)

Top
#130645 - 13/12/2002 07:19 Re: Bringing up lame --nogap again [Re: tms13]
wfaulk
carpal tunnel

Registered: 25/12/2000
Posts: 16706
Loc: Raleigh, NC US
The Xing tags are still wrong whether or not nogap is specified, and that's the real problem I'm trying to get fixed in the lame code. I've not gotten a response yet. I'm trying to get on the lame mailing list, but it's being a pain, and I haven't gotten a response from the developer I emailed directly, but I wouldn't be surprised if it got binned in a spam folder for him, as I sent it to an @users.sourceforge.net account.
_________________________
Bitt Faulk

Top
#130646 - 13/12/2002 10:18 Re: Bringing up lame --nogap again [Re: wfaulk]
Terminator
old hand

Registered: 12/01/2000
Posts: 1079
Loc: Dallas, TX
If you are trying to get ahold of Mark T, I think he has little interest in nogap at this point. I can't remember who was doing all the work on -nogap.

Top
#130647 - 13/12/2002 14:54 Re: Bringing up lame --nogap again [Re: Terminator]
wfaulk
carpal tunnel

Registered: 25/12/2000
Posts: 16706
Loc: Raleigh, NC US
Well, it's not really a nogap issue. It's a Xing header issue at its core. Once that is solved, it should be possible to reenable Xing headers on nogap, but that's a bridge to be crossed later.

And I am trying to contact Robert Hegemann.
_________________________
Bitt Faulk

Top