Skip to main content

Topic: OggEnc v2.87 (aoTuV Beta 6.03) padding switch (Read 4537 times) previous topic - next topic

0 Members and 1 Guest are viewing this topic.
  • Deep_Elem
  • [*][*][*]
OggEnc v2.87 (aoTuV Beta 6.03) padding switch
I noticed some time ago that the --padding switch has disappeared from the various oggenc2.exe builds that I've downloaded from rarewares. I'm currently using OggEnc v2.87 (aoTuV Beta 6.03), still with no padding switch. I've searched and cannot find any thread explaining what happened to this switch.

I liked this switch and would like to have it back. Can anyone explain what happened to it and whether we can expect to see its return in future releases?

  • john33
  • [*][*][*][*][*]
  • Developer
OggEnc v2.87 (aoTuV Beta 6.03) padding switch
Reply #1
The option was removed in November of last year because it caused issues with Windows 7. I've not looked at it since but I'll have another look now.
John
----------------------------------------------------------------
My compiles and utilities are at http://www.rarewares.org/

  • lvqcl
  • [*][*][*][*][*]
  • Developer
OggEnc v2.87 (aoTuV Beta 6.03) padding switch
Reply #2
Found the suggestion about padding: http://www.hydrogenaudio.org/forums/index....ost&p=43768

But there's a problem with that code because of:

vorbis_analysis_headerout(&vd,opt->comments, &header_main,&header_comments,&header_codebooks);

allocates memory for comments and stores it in 2 variables:  header_comments.packet  and vd.backend_state.header1.
The latter variable is defined in codec_internal.h and should be also updated somehow.

  • john33
  • [*][*][*][*][*]
  • Developer
OggEnc v2.87 (aoTuV Beta 6.03) padding switch
Reply #3
And not a trivial exercise, it would seem.
John
----------------------------------------------------------------
My compiles and utilities are at http://www.rarewares.org/

  • Deep_Elem
  • [*][*][*]
OggEnc v2.87 (aoTuV Beta 6.03) padding switch
Reply #4
John,

Thanks for your prompt reply to my question. I really appreciate it and all the work you've put into oggenc2.

  • lvqcl
  • [*][*][*][*][*]
  • Developer
OggEnc v2.87 (aoTuV Beta 6.03) padding switch
Reply #5
http://wiki.xiph.org/Metadata :

Quote
METADATA_BLOCK_PADDING: Nul padding.
...
METADATA_BLOCK_VORBIS_COMMENT: Also known as FLAC tags, the contents of a VorbisComment packet.


Does it mean that padding should have its own block (instead of adding unused space to VorbisComment block)? Or is it true only for FLAC files?

Added: yes it seems to be related to FLAC only
  • Last Edit: 28 September, 2011, 01:35:02 PM by lvqcl