shshankjain / webm

Automatically exported from code.google.com/p/webm
0 stars 0 forks source link

--disable-postproc is not respect #153

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What is the expected behavior?
postproc.o should not be in libvpx.a

What do you see instead?
It is, and on 0.9.1 it takes 1.6 MB of memory.

What version are you using?
Official 0.9.1

On what operating system?
Windows 7

Can you reproduce using the ivfdec or ivfenc tools?
yes, but its not really appplicable.  You can use size.exe tool on ivfdec.exe 
and look at the bss size.

What command line are you using?
../libvpx/configure --target=x86-win32-gcc --disable-install-docs 
--disable-codecs --enable-vp8 --disable-ccache --disable-debug --disable-psnr 
--disable-postproc
make

Please provide any additional information below.
Some details are in bug #1, but as there are several smaller issues, I'll break 
them out so they are fixable.
A patch is up for review that removes the following lines from postproc.c
char vp8_an[8][64][3072];
int vp8_cd[8][64];
Which will reduce the size of postproc.
But --disable-postproc should be respected, completely removing the module from 
libvpx.

Original issue reported on code.google.com by fbarch...@chromium.org on 28 Jul 2010 at 10:01

GoogleCodeExporter commented 9 years ago
This is the CL to reduce postproc size 
https://review.webmproject.org/#patch,sidebyside,360,1,vp8/common/postproc.c

Original comment by fbarch...@chromium.org on 28 Jul 2010 at 10:16

GoogleCodeExporter commented 9 years ago
Closed in https://review.webmproject.org/360

Original comment by jkoles...@google.com on 2 Sep 2010 at 3:04

GoogleCodeExporter commented 9 years ago

Original comment by albe...@google.com on 8 Mar 2012 at 12:10