From ff771e770506b76b951e06dea91617575da1f03a Mon Sep 17 00:00:00 2001
From: James Zern <jzern@google.com>
Date: Fri, 13 Jul 2012 20:36:14 -0700
Subject: [PATCH] don't install webp/decode_vp8.h

the VP8 decode functions do not need to be public; only
GetInfo/CheckSignature need to be marked extern as they're used by
libwebpmux.

Change-Id: Id9ab4d6166b0271cf5d04563c6dac1fcc84adbdc
---
 src/Makefile.am       |  4 +++-
 src/dec/Makefile.am   |  5 +++--
 src/webp/decode_vp8.h | 26 +++++++++++++-------------
 3 files changed, 19 insertions(+), 16 deletions(-)

diff --git a/src/Makefile.am b/src/Makefile.am
index ef5e281c..ff4638a9 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -9,9 +9,11 @@ lib_LTLIBRARIES = libwebp.la
 libwebp_la_SOURCES =
 libwebpinclude_HEADERS =
 libwebpinclude_HEADERS += webp/decode.h
-libwebpinclude_HEADERS += webp/decode_vp8.h
 libwebpinclude_HEADERS += webp/encode.h
 libwebpinclude_HEADERS += webp/types.h
+noinst_HEADERS =
+noinst_HEADERS += webp/decode_vp8.h
+noinst_HEADERS += webp/format_constants.h
 
 libwebp_la_LIBADD =
 libwebp_la_LIBADD += dec/libwebpdecode.la
diff --git a/src/dec/Makefile.am b/src/dec/Makefile.am
index fceeda8f..43516b62 100644
--- a/src/dec/Makefile.am
+++ b/src/dec/Makefile.am
@@ -19,9 +19,10 @@ libwebpdecode_la_SOURCES += webpi.h
 
 libwebpdecodeinclude_HEADERS =
 libwebpdecodeinclude_HEADERS += ../webp/decode.h
-libwebpdecodeinclude_HEADERS += ../webp/decode_vp8.h
 libwebpdecodeinclude_HEADERS += ../webp/types.h
-noinst_HEADERS = ../webp/format_constants.h
+noinst_HEADERS =
+noinst_HEADERS += ../webp/decode_vp8.h
+noinst_HEADERS += ../webp/format_constants.h
 
 libwebpdecode_la_CPPFLAGS = $(USE_EXPERIMENTAL_CODE)
 libwebpdecodeincludedir = $(includedir)/webp
diff --git a/src/webp/decode_vp8.h b/src/webp/decode_vp8.h
index 93e86566..63cb95a3 100644
--- a/src/webp/decode_vp8.h
+++ b/src/webp/decode_vp8.h
@@ -106,22 +106,22 @@ struct VP8Io {
 };
 
 // Internal, version-checked, entry point
-WEBP_EXTERN(int) VP8InitIoInternal(VP8Io* const, int);
+int VP8InitIoInternal(VP8Io* const, int);
 
 // Set the custom IO function pointers and user-data. The setter for IO hooks
 // should be called before initiating incremental decoding. Returns true if
 // WebPIDecoder object is successfully modified, false otherwise.
-WEBP_EXTERN(int) WebPISetIOHooks(WebPIDecoder* const idec,
-                                 VP8IoPutHook put,
-                                 VP8IoSetupHook setup,
-                                 VP8IoTeardownHook teardown,
-                                 void* user_data);
+int WebPISetIOHooks(WebPIDecoder* const idec,
+                    VP8IoPutHook put,
+                    VP8IoSetupHook setup,
+                    VP8IoTeardownHook teardown,
+                    void* user_data);
 
 // Main decoding object. This is an opaque structure.
 typedef struct VP8Decoder VP8Decoder;
 
 // Create a new decoder object.
-WEBP_EXTERN(VP8Decoder*) VP8New(void);
+VP8Decoder* VP8New(void);
 
 // Must be called to make sure 'io' is initialized properly.
 // Returns false in case of version mismatch. Upon such failure, no other
@@ -131,24 +131,24 @@ static WEBP_INLINE int VP8InitIo(VP8Io* const io) {
 }
 
 // Start decoding a new picture. Returns true if ok.
-WEBP_EXTERN(int) VP8GetHeaders(VP8Decoder* const dec, VP8Io* const io);
+int VP8GetHeaders(VP8Decoder* const dec, VP8Io* const io);
 
 // Decode a picture. Will call VP8GetHeaders() if it wasn't done already.
 // Returns false in case of error.
-WEBP_EXTERN(int) VP8Decode(VP8Decoder* const dec, VP8Io* const io);
+int VP8Decode(VP8Decoder* const dec, VP8Io* const io);
 
 // Return current status of the decoder:
-WEBP_EXTERN(VP8StatusCode) VP8Status(VP8Decoder* const dec);
+VP8StatusCode VP8Status(VP8Decoder* const dec);
 
 // return readable string corresponding to the last status.
-WEBP_EXTERN(const char*) VP8StatusMessage(VP8Decoder* const dec);
+const char* VP8StatusMessage(VP8Decoder* const dec);
 
 // Resets the decoder in its initial state, reclaiming memory.
 // Not a mandatory call between calls to VP8Decode().
-WEBP_EXTERN(void) VP8Clear(VP8Decoder* const dec);
+void VP8Clear(VP8Decoder* const dec);
 
 // Destroy the decoder object.
-WEBP_EXTERN(void) VP8Delete(VP8Decoder* const dec);
+void VP8Delete(VP8Decoder* const dec);
 
 //------------------------------------------------------------------------------
 // Miscellaneous VP8/VP8L bitstream probing functions.