[FFmpeg-devel] [PATCH] oggpagesize option added

[Justin Ruggles]

On 01/25/2011 02:16 PM, Andres Gonzalez wrote:

> Hi,
> 
> Here is the patch. I should point out that the feature is very simple, 
> and the preferred size is just that, "preferred". This means that 
> several pages could be far from the value given. But the patch 
> acomplishes my main purpose, which was allowing ffmpeg produce videos 
> that could be streamed through oggfwd and Icecast ("-oggpagesize 4096" 
> works fine for this).
> 
> Please let me know if there is something wrong or missing.
> Regards,
> Andr?s
> 

> ---
>  libavformat/oggenc.c |   31 +++++++++++++++++++++++++++++--
>  1 files changed, 29 insertions(+), 2 deletions(-)
> 
> diff --git a/libavformat/oggenc.c b/libavformat/oggenc.c
> index b72efc8..b07385c 100644
> --- a/libavformat/oggenc.c
> +++ b/libavformat/oggenc.c
> @@ -19,7 +19,9 @@
>   * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
>   */
>  
> +#include "libavutil/avassert.h"


I don't see anything in your patch using anything from avassert.h.

>  #include "libavutil/crc.h"
> +#include "libavutil/opt.h"
>  #include "libavutil/random_seed.h"
>  #include "libavcodec/xiph.h"
>  #include "libavcodec/bytestream.h"


-Justin