summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLuca Stefani <luca.stefani.ge1@gmail.com>2019-07-24 17:34:39 +0200
committerMichael Bestas <mkbestas@lineageos.org>2020-05-27 00:06:14 +0300
commit398e6c6466b7185c1b3a7a7b2fd56d9de3284ce3 (patch)
treea625f17627ef316a3098d8c0fdb5632ca64cb7dd
parentc2614ab9a32a4ec0a1dc2700981b2a806d5c8469 (diff)
downloadandroid_packages_apps_Gallery2-398e6c6466b7185c1b3a7a7b2fd56d9de3284ce3.tar.gz
android_packages_apps_Gallery2-398e6c6466b7185c1b3a7a7b2fd56d9de3284ce3.tar.bz2
android_packages_apps_Gallery2-398e6c6466b7185c1b3a7a7b2fd56d9de3284ce3.zip
Gallery2: Fix jni build warnings
* warning: implicit conversion from 'int' to 'char' changes value from 128 to -128 -> Convert all the source/destitionation of filters to uint8_t Change-Id: I019081943a70708c2725c1ae52b1c3fbb6e87f9e
-rw-r--r--jni/filters/edge.c8
-rw-r--r--jni/filters/geometry.c36
-rw-r--r--jni/filters/saturated.c2
-rw-r--r--jni/filters/vibrance.c2
4 files changed, 24 insertions, 24 deletions
diff --git a/jni/filters/edge.c b/jni/filters/edge.c
index 9f5d88f77..5e7be3c4c 100644
--- a/jni/filters/edge.c
+++ b/jni/filters/edge.c
@@ -19,7 +19,7 @@
void JNIFUNCF(ImageFilterEdge, nativeApplyFilter, jobject bitmap, jint width, jint height, jfloat p)
{
- char* destination = 0;
+ uint8_t* destination = 0;
AndroidBitmap_lockPixels(env, bitmap, (void**) &destination);
// using contrast function:
@@ -32,15 +32,15 @@ void JNIFUNCF(ImageFilterEdge, nativeApplyFilter, jobject bitmap, jint width, ji
float const c_max = 500.0f;
// pixels must be 4 bytes
- char * dst = destination;
+ uint8_t* dst = destination;
int j, k;
- char * ptr = destination;
+ uint8_t* ptr = destination;
int row_stride = 4 * width;
// set 2 row buffer (avoids bitmap copy)
int buf_len = 2 * row_stride;
- char buf[buf_len];
+ uint8_t buf[buf_len];
int buf_row_ring = 0;
// set initial buffer to black
diff --git a/jni/filters/geometry.c b/jni/filters/geometry.c
index 8537549d0..c2d80f380 100644
--- a/jni/filters/geometry.c
+++ b/jni/filters/geometry.c
@@ -19,7 +19,7 @@
#include "filters.h"
-static __inline__ void flipVertical(char * source, int srcWidth, int srcHeight, char * destination,
+static __inline__ void flipVertical(uint8_t * source, int srcWidth, int srcHeight, uint8_t * destination,
int dstWidth __unused, int dstHeight __unused) {
//Vertical
size_t cpy_bytes = sizeof(char) * 4;
@@ -34,8 +34,8 @@ static __inline__ void flipVertical(char * source, int srcWidth, int srcHeight,
}
}
-static __inline__ void flipHorizontal(char * source, int srcWidth, int srcHeight,
- char * destination, int dstWidth __unused, int dstHeight __unused) {
+static __inline__ void flipHorizontal(uint8_t * source, int srcWidth, int srcHeight,
+ uint8_t * destination, int dstWidth __unused, int dstHeight __unused) {
//Horizontal
size_t cpy_bytes = sizeof(char) * 4;
int width = cpy_bytes * srcWidth;
@@ -52,12 +52,12 @@ static __inline__ void flipHorizontal(char * source, int srcWidth, int srcHeight
}
}
-static __inline__ void flip_fun(int flip, char * source, int srcWidth, int srcHeight, char * destination, int dstWidth, int dstHeight){
+static __inline__ void flip_fun(int flip, uint8_t * source, int srcWidth, int srcHeight, uint8_t * destination, int dstWidth, int dstHeight){
int horiz = (flip & 1) != 0;
int vert = (flip & 2) != 0;
if (horiz && vert){
int arr_len = dstWidth * dstHeight * sizeof(char) * 4;
- char* temp = (char *) malloc(arr_len);
+ uint8_t* temp = (uint8_t *) malloc(arr_len);
flipHorizontal(source, srcWidth, srcHeight, temp, dstWidth, dstHeight);
flipVertical(temp, dstWidth, dstHeight, destination, dstWidth, dstHeight);
free(temp);
@@ -74,7 +74,7 @@ static __inline__ void flip_fun(int flip, char * source, int srcWidth, int srcHe
}
//90 CCW (opposite of what's used in UI?)
-static __inline__ void rotate90(char * source, int srcWidth, int srcHeight, char * destination,
+static __inline__ void rotate90(uint8_t * source, int srcWidth, int srcHeight, uint8_t * destination,
int dstWidth __unused, int dstHeight __unused) {
size_t cpy_bytes = sizeof(char) * 4;
int width = cpy_bytes * srcWidth;
@@ -88,17 +88,17 @@ static __inline__ void rotate90(char * source, int srcWidth, int srcHeight, char
}
}
-static __inline__ void rotate180(char * source, int srcWidth, int srcHeight, char * destination, int dstWidth, int dstHeight){
+static __inline__ void rotate180(uint8_t * source, int srcWidth, int srcHeight, uint8_t * destination, int dstWidth, int dstHeight){
flip_fun(3, source, srcWidth, srcHeight, destination, dstWidth, dstHeight);
}
-static __inline__ void rotate270(char * source, int srcWidth, int srcHeight, char * destination, int dstWidth, int dstHeight){
+static __inline__ void rotate270(uint8_t * source, int srcWidth, int srcHeight, uint8_t * destination, int dstWidth, int dstHeight){
rotate90(source, srcWidth, srcHeight, destination, dstWidth, dstHeight);
flip_fun(3, destination, dstWidth, dstHeight, destination, dstWidth, dstHeight);
}
// rotate == 1 is 90 degrees, 2 is 180, 3 is 270 (positive is CCW).
-static __inline__ void rotate_fun(int rotate, char * source, int srcWidth, int srcHeight, char * destination, int dstWidth, int dstHeight){
+static __inline__ void rotate_fun(int rotate, uint8_t * source, int srcWidth, int srcHeight, uint8_t * destination, int dstWidth, int dstHeight){
switch( rotate )
{
case 1:
@@ -115,7 +115,7 @@ static __inline__ void rotate_fun(int rotate, char * source, int srcWidth, int s
}
}
-static __inline__ void crop(char * source, int srcWidth, int srcHeight, char * destination, int dstWidth, int dstHeight, int offsetWidth, int offsetHeight){
+static __inline__ void crop(uint8_t * source, int srcWidth, int srcHeight, uint8_t * destination, int dstWidth, int dstHeight, int offsetWidth, int offsetHeight){
size_t cpy_bytes = sizeof(char) * 4;
int row_width = cpy_bytes * srcWidth;
int new_row_width = cpy_bytes * dstWidth;
@@ -129,8 +129,8 @@ static __inline__ void crop(char * source, int srcWidth, int srcHeight, char * d
}
void JNIFUNCF(ImageFilterGeometry, nativeApplyFilterFlip, jobject src, jint srcWidth, jint srcHeight, jobject dst, jint dstWidth, jint dstHeight, jint flip) {
- char* destination = 0;
- char* source = 0;
+ uint8_t* destination = 0;
+ uint8_t* source = 0;
if (srcWidth != dstWidth || srcHeight != dstHeight) {
return;
}
@@ -142,8 +142,8 @@ void JNIFUNCF(ImageFilterGeometry, nativeApplyFilterFlip, jobject src, jint srcW
}
void JNIFUNCF(ImageFilterGeometry, nativeApplyFilterRotate, jobject src, jint srcWidth, jint srcHeight, jobject dst, jint dstWidth, jint dstHeight, jint rotate) {
- char* destination = 0;
- char* source = 0;
+ uint8_t* destination = 0;
+ uint8_t* source = 0;
AndroidBitmap_lockPixels(env, src, (void**) &source);
AndroidBitmap_lockPixels(env, dst, (void**) &destination);
rotate_fun(rotate, source, srcWidth, srcHeight, destination, dstWidth, dstHeight);
@@ -152,8 +152,8 @@ void JNIFUNCF(ImageFilterGeometry, nativeApplyFilterRotate, jobject src, jint sr
}
void JNIFUNCF(ImageFilterGeometry, nativeApplyFilterCrop, jobject src, jint srcWidth, jint srcHeight, jobject dst, jint dstWidth, jint dstHeight, jint offsetWidth, jint offsetHeight) {
- char* destination = 0;
- char* source = 0;
+ uint8_t* destination = 0;
+ uint8_t* source = 0;
AndroidBitmap_lockPixels(env, src, (void**) &source);
AndroidBitmap_lockPixels(env, dst, (void**) &destination);
crop(source, srcWidth, srcHeight, destination, dstWidth, dstHeight, offsetWidth, offsetHeight);
@@ -164,8 +164,8 @@ void JNIFUNCF(ImageFilterGeometry, nativeApplyFilterCrop, jobject src, jint srcW
void JNIFUNCF(ImageFilterGeometry, nativeApplyFilterStraighten, jobject src, jint srcWidth __unused,
jint srcHeight __unused, jobject dst, jint dstWidth, jint dstHeight,
jfloat straightenAngle __unused) {
- char* destination = 0;
- char* source = 0;
+ uint8_t* destination = 0;
+ uint8_t* source = 0;
int len = dstWidth * dstHeight * 4;
AndroidBitmap_lockPixels(env, src, (void**) &source);
AndroidBitmap_lockPixels(env, dst, (void**) &destination);
diff --git a/jni/filters/saturated.c b/jni/filters/saturated.c
index f5cf7677a..a672a4ec6 100644
--- a/jni/filters/saturated.c
+++ b/jni/filters/saturated.c
@@ -18,7 +18,7 @@
void JNIFUNCF(ImageFilterSaturated, nativeApplyFilter, jobject bitmap, jint width, jint height, jfloat saturation)
{
- char* destination = 0;
+ uint8_t* destination = 0;
AndroidBitmap_lockPixels(env, bitmap, (void**) &destination);
int i;
int len = width * height * 4;
diff --git a/jni/filters/vibrance.c b/jni/filters/vibrance.c
index feef7bc97..78228d15b 100644
--- a/jni/filters/vibrance.c
+++ b/jni/filters/vibrance.c
@@ -19,7 +19,7 @@
void JNIFUNCF(ImageFilterVibrance, nativeApplyFilter, jobject bitmap, jint width, jint height, jfloat vibrance)
{
- char* destination = 0;
+ uint8_t* destination = 0;
AndroidBitmap_lockPixels(env, bitmap, (void**) &destination);
int i;
int len = width * height * 4;