Fix exposure and fixed warnings

bug:7234321

7 files changed, 17 insertions, 15 deletions

diff --git a/jni/Android.mk b/jni/Android.mk
index 8453f76d1..38fa453b9 100644
--- a/jni/Android.mk
+++ b/jni/Android.mk
@@ -27,7 +27,7 @@ LOCAL_MODULE    := libjni_filtershow_filters
 LOCAL_SRC_FILES := filters/bw.c \
 		   filters/gradient.c \
 	           filters/saturated.c \
-                   filters/brightness.c \
+                   filters/exposure.c \
                    filters/contrast.c \
                    filters/hue.c \
 		   filters/vignette.c
diff --git a/jni/filters/contrast.c b/jni/filters/contrast.c
index 45209c1fb..6c1b976cf 100644
--- a/jni/filters/contrast.c
+++ b/jni/filters/contrast.c
@@ -14,6 +14,7 @@
  * limitations under the License.
  */
 
+#include <math.h>
 #include "filters.h"
 
 unsigned char clamp(int c)
diff --git a/jni/filters/brightness.c b/jni/filters/exposure.c
index 88044a28c..6b32798c8 100644
--- a/jni/filters/brightness.c
+++ b/jni/filters/exposure.c
@@ -16,21 +16,21 @@
 
 #include "filters.h"
 
-void JNIFUNCF(ImageFilterBrightness, nativeApplyFilter, jobject bitmap, jint width, jint height, jfloat bright)
+void JNIFUNCF(ImageFilterExposure, nativeApplyFilter, jobject bitmap, jint width, jint height, jfloat bright)
 {
     char* destination = 0;
     AndroidBitmap_lockPixels(env, bitmap, (void**) &destination);
     unsigned char * rgb = (unsigned char * )destination;
     int i;
     int len = width * height * 4;
-    int c = (int)(bright);
-    int m =  (c>0)?(255+c):255;
+
+    int m =   (255-bright);
 
     for (i = 0; i < len; i+=4)
     {
-        rgb[RED]   = clamp((255*(rgb[RED]))/m+c);
-        rgb[GREEN] = clamp((255*(rgb[GREEN]))/m+c);
-        rgb[BLUE]  = clamp((255*(rgb[BLUE]))/m+c);
+        rgb[RED]   = clamp((255*(rgb[RED]))/m);
+        rgb[GREEN] = clamp((255*(rgb[GREEN]))/m);
+        rgb[BLUE]  = clamp((255*(rgb[BLUE]))/m);
     }
     AndroidBitmap_unlockPixels(env, bitmap);
 }
diff --git a/jni/filters/filters.h b/jni/filters/filters.h
index 5fe595b9e..44a442290 100644
--- a/jni/filters/filters.h
+++ b/jni/filters/filters.h
@@ -43,6 +43,6 @@ typedef unsigned int Color;
 #define ALPHA i+3
 #define CLAMP(c) (MAX(0, MIN(255, c)))
 
-unsigned char __inline__ clamp(int c);
+__inline__ unsigned char  clamp(int c);
 
 #endif // FILTERS_H
diff --git a/jni/filters/vignette.c b/jni/filters/vignette.c
index 7cff517b6..2799ff001 100644
--- a/jni/filters/vignette.c
+++ b/jni/filters/vignette.c
@@ -20,7 +20,7 @@ static int* gVignetteMap = 0;
 static int gVignetteWidth = 0;
 static int gVignetteHeight = 0;
 
-void __inline__ createVignetteMap(int w, int h)
+__inline__ void createVignetteMap(int w, int h)
 {
     if (gVignetteMap && (gVignetteWidth != w || gVignetteHeight != h))
     {
diff --git a/src/com/android/gallery3d/filtershow/FilterShowActivity.java b/src/com/android/gallery3d/filtershow/FilterShowActivity.java
index eb95d7452..4570eb017 100644
--- a/src/com/android/gallery3d/filtershow/FilterShowActivity.java
+++ b/src/com/android/gallery3d/filtershow/FilterShowActivity.java
@@ -753,14 +753,14 @@ public class FilterShowActivity extends Activity implements OnItemClickListener
                 mImageShow.setVisibility(View.VISIBLE);
                 mImageShow.setShowControls(true);
                 ImagePreset preset = mImageShow.getImagePreset();
-                ImageFilter filter = preset.getFilter("Brightness");
+                ImageFilter filter = preset.getFilter("Exposure");
                 if (filter == null) {
-                    ImageFilterBrightness bright = new ImageFilterBrightness();
+                    ImageFilterExposure bright = new ImageFilterExposure();
                     ImagePreset copy = new ImagePreset(preset);
                     copy.add(bright);
                     copy.setHistoryName(bright.getName());
                     copy.setIsFx(false);
-                    filter = copy.getFilter("Brightness");
+                    filter = copy.getFilter("Exposure");
                     mImageShow.setImagePreset(copy);
                 }
                 mImageShow.setCurrentFilter(filter);
diff --git a/src/com/android/gallery3d/filtershow/filters/ImageFilterBrightness.java b/src/com/android/gallery3d/filtershow/filters/ImageFilterExposure.java
index ab61a00c2..85b6e4f6b 100644
--- a/src/com/android/gallery3d/filtershow/filters/ImageFilterBrightness.java
+++ b/src/com/android/gallery3d/filtershow/filters/ImageFilterExposure.java
@@ -3,14 +3,15 @@ package com.android.gallery3d.filtershow.filters;
 
 import android.graphics.Bitmap;
 
-public class ImageFilterBrightness extends ImageFilter {
+public class ImageFilterExposure extends ImageFilter {
 
-    public ImageFilterBrightness() {
-        mName = "Brightness";
+    public ImageFilterExposure() {
+        mName = "Exposure";
     }
 
     native protected void nativeApplyFilter(Bitmap bitmap, int w, int h, float bright);
 
+    @Override
     public void apply(Bitmap bitmap) {
         int w = bitmap.getWidth();
         int h = bitmap.getHeight();