- Status Closed
- Percent Complete
- Task Type Bugs
- Category Codecs
- Assigned To No-one
- Operating System All players
- Severity Low
- Priority Very Low
- Reported Version
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#5317 - mp3_encoder improperly encodes mono .wav files
when using mp3_encoder plugin on a 44khz mono .wav file, the resulting mp3 plays too fast… <linuxstb> it seems that Stereo input is assumed in lots of places…. The encoder itself seems to support mono, but the Rockbox implementation breaks it.
There’s init_mp3_encoder_engine() which is passed ‘true’ for stereo regardless of actual .wav format, and there’s also some code in the encoding loop which tries to downsample stereo to mono (huh what) if nchannels was set to mono.
Looks like someone got carried away with cut & paste
Closed by Buschel
2011-01-05 07:54
Reason for closing: Fixed
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
2011-01-05 07:54
Reason for closing: Fixed
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
Fixed with r28971
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
is this still an issue ?
It’s been worked out of the adaptation to the encoder framework. If this should be fixed and the transcoder will be kept then I might be able to take care of it.
Mike?
Confirmed with e200 & r18013.
Petur? Isn’t this a plugin issue? I’ve never worked on mp3_encoder. Of course I did say I might be able to take care of it in a post up there I hardly recall making. I must have said it for a good reason though but exactly what it is, my memory gnomes have to go fetch. :)
44.1kHz mono encodes correct with attached patch (also see
FS#10678). Nevertheless there is something going wrong with samplerates != 44100. At least a 24kHz mono file encodes strange. Therefor I will not submit this fix and continue work inFS#10678.