Update sdch's config.h for Android

Update sdch's config.h for Android
    
Avoid redefining HAVE_MALLOC_H, which is already defined in Android's AndroidConfig.h.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=136333

[Steve Block, 2012-05-09 14:33:52]

For reviewing convenience ...

$ diff sdch/linux/config.h sdch/android/config.h 
1,2c1
< /* src/config.h.  Generated from config.h.in by configure.  */
< /* src/config.h.in.  Generated from configure.ac by autoheader.  */
---
> /* android/config.h.  Based on linux/config.h with manual edits. */
22a22,23
> /* This is defined in AndroidConfig.h. */
> #undef HAVE_MALLOC_H

[wtc, 2012-05-09 19:26:47]

Drive-by review comments on patch set 1:

I think this CL is fine.  But we can also continue to use
linux/config.h for Android.

http://codereview.chromium.org/10332074/diff/1/sdch/android/config.h
File sdch/android/config.h (right):

http://codereview.chromium.org/10332074/diff/1/sdch/android/config.h#newcode111
sdch/android/config.h:111: #if defined(__i386__) || defined(__x86_64__)

Note this code here, which is a manual change to the
config.h generated for Linux.

So linux/config.h has already been manually edited to
suit our purposes.  It seems fine to just add the
#undef HAVE_MALLOC_H line to linux/config.h.

[jar (doing other things), 2012-05-10 06:40:14]

http://codereview.chromium.org/10332074/diff/1/sdch/android/config.h
File sdch/android/config.h (right):

http://codereview.chromium.org/10332074/diff/1/sdch/android/config.h#newcode111
sdch/android/config.h:111: #if defined(__i386__) || defined(__x86_64__)
+1

Can you take that approach?

On 2012/05/09 19:26:47, wtc wrote:
> 
> Note this code here, which is a manual change to the
> config.h generated for Linux.
> 
> So linux/config.h has already been manually edited to
> suit our purposes.  It seems fine to just add the
> #undef HAVE_MALLOC_H line to linux/config.h.

[jar (doing other things), 2012-05-10 16:17:15]

This L G T M..... but I'll let wtc concur, since he is more familiar with this
method.

[wtc, 2012-05-10 16:58:28]

Patch set 2 LGTM.  Thanks.