Fix several warnings in cocoa frontend

1 files changed, 95 insertions, 106 deletions

diff --git a/cocoa/apple_image.m b/cocoa/apple_image.m
index c6f9df324..91dfcb4e6 100644
--- a/cocoa/apple_image.m
+++ b/cocoa/apple_image.m
@@ -9,7 +9,7 @@
  *
  * NetSurf is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.	 See the
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
@@ -26,29 +26,19 @@
 #include "desktop/plotters.h"
 #include "utils/utils.h"
 
-#include "cocoa/schedule.h"
+#import "cocoa/schedule.h"
+#import "cocoa/bitmap.h"
 
 typedef struct apple_image_content {
 	struct content base;
 
 	struct bitmap *bitmap;	/**< Created NetSurf bitmap */
 
-    NSUInteger frames;
-    NSUInteger currentFrame;
-    int *frameTimes;
+	NSUInteger frames;
+	NSUInteger currentFrame;
+	int *frameTimes;
 } apple_image_content;
 
-static nserror apple_image_create(const content_handler *handler,
-		lwc_string *imime_type, const http_parameter *params,
-		llcache_handle *llcache, const char *fallback_charset,
-		bool quirks, struct content **c);
-static bool apple_image_convert(struct content *c);
-static void apple_image_destroy(struct content *c);
-static bool apple_image_redraw(struct content *c, struct content_redraw_data *data,
-		const struct rect *clip, const struct redraw_context *ctx);
-static nserror apple_image_clone(const struct content *old, 
-		struct content **newc);
-static content_type apple_image_content_type(void);
 
 static void *apple_image_get_internal(const struct content *c, void *context)
 {
@@ -57,55 +47,7 @@ static void *apple_image_get_internal(const struct content *c, void *context)
 	return ai_c->bitmap;
 }
 
-static const content_handler apple_image_content_handler = {
-	.create = apple_image_create,
-	.data_complete = apple_image_convert,
-	.destroy = apple_image_destroy,
-	.redraw = apple_image_redraw,
-	.clone = apple_image_clone,
-	.get_internal = apple_image_get_internal,
-	.type = apple_image_content_type,
-	.no_share = false
-};
-
-static nserror register_for_type( NSString *mime )
-{
-	const char *type = [mime UTF8String];
-	/* nsgif has priority since it supports animated GIF */
-#ifdef WITH_GIF
-	if (strcmp(type, "image/gif") == 0)
-		return NSERROR_OK;
-#endif
-
-	nserror error = content_factory_register_handler( type, &apple_image_content_handler );
-	if (error != NSERROR_OK) return error;
-	
-	return NSERROR_OK;
-}
-
-nserror apple_image_init(void)
-{
-	NSArray *utis = [NSBitmapImageRep imageTypes];
-	for (NSString *uti in utis) {
-		NSDictionary *declaration = [(NSDictionary *)UTTypeCopyDeclaration( (CFStringRef)uti ) autorelease];
-		id mimeTypes = [[declaration objectForKey: (NSString *)kUTTypeTagSpecificationKey] objectForKey: (NSString *)kUTTagClassMIMEType];
-		
-		if (mimeTypes == nil) continue;
-		
-		if (![mimeTypes isKindOfClass: [NSArray class]]) {
-			mimeTypes = [NSArray arrayWithObject: mimeTypes];
-		}
-		
-		for (NSString *mime in mimeTypes) {
-			nserror error = register_for_type( mime );
-			if (error != NSERROR_OK) return error;
-		} 
-	}
-	
-	return NSERROR_OK;
-}
-
-nserror apple_image_create(const content_handler *handler,
+static nserror apple_image_create(const content_handler *handler,
 		lwc_string *imime_type, const http_parameter *params,
 		llcache_handle *llcache, const char *fallback_charset,
 		bool quirks, struct content **c)
@@ -132,30 +74,29 @@ nserror apple_image_create(const content_handler *handler,
 
 static void animate_image_cb( void *ptr )
 {
-    struct apple_image_content *ai = ptr;
-    ++ai->currentFrame;
-    if (ai->currentFrame >= ai->frames) ai->currentFrame = 0;
-    
-    [(NSBitmapImageRep *)ai->bitmap setProperty: NSImageCurrentFrame withValue: [NSNumber numberWithUnsignedInteger: ai->currentFrame]];
-    bitmap_modified( ai->bitmap );
-    
-    union content_msg_data data;
-    data.redraw.full_redraw = true;
-    data.redraw.x = data.redraw.object_x = 0;
-    data.redraw.y = data.redraw.object_y = 0;
-    data.redraw.width = data.redraw.object_width = ai->base.width;
-    data.redraw.height = data.redraw.object_height = ai->base.height;
+	struct apple_image_content *ai = ptr;
+	++ai->currentFrame;
+	if (ai->currentFrame >= ai->frames) ai->currentFrame = 0;
+
+	[(NSBitmapImageRep *)ai->bitmap setProperty: NSImageCurrentFrame withValue: [NSNumber numberWithUnsignedInteger: ai->currentFrame]];
+	cocoa_bitmap_modified( ai->bitmap );
+
+	union content_msg_data data;
+	data.redraw.full_redraw = true;
+	data.redraw.x = data.redraw.object_x = 0;
+	data.redraw.y = data.redraw.object_y = 0;
+	data.redraw.width = data.redraw.object_width = ai->base.width;
+	data.redraw.height = data.redraw.object_height = ai->base.height;
 	data.redraw.object = &ai->base;
-    content_broadcast( &ai->base, CONTENT_MSG_REDRAW, data );
+	content_broadcast( &ai->base, CONTENT_MSG_REDRAW, data );
 
-    cocoa_schedule(ai->frameTimes[ai->currentFrame], animate_image_cb, ai );
+	cocoa_schedule(ai->frameTimes[ai->currentFrame], animate_image_cb, ai );
 }
 
 /**
  * Convert a CONTENT_APPLE_IMAGE for display.
  */
-
-bool apple_image_convert(struct content *c)
+static bool apple_image_convert(struct content *c)
 {
 	apple_image_content *ai_c = (apple_image_content *)c;
 	unsigned long size;
@@ -170,7 +111,7 @@ bool apple_image_convert(struct content *c)
 		content_broadcast(c, CONTENT_MSG_ERROR, msg_data);
 		return false;
 	}
-	
+
 	c->width = [image pixelsWide];
 	c->height = [image pixelsHigh];
 	ai_c->bitmap = (void *)image;
@@ -178,40 +119,40 @@ bool apple_image_convert(struct content *c)
 	NSString *url = [NSString stringWithUTF8String: nsurl_access(llcache_handle_get_url( content_get_llcache_handle( c )) )];
 	NSString *title = [NSString stringWithFormat: @"%@ (%dx%d)", [url lastPathComponent], c->width, c->height];
 	content__set_title(c, [title UTF8String] );
-	
+
 	content_set_ready(c);
 	content_set_done(c);
 	content_set_status(c, "");
-    
-    struct apple_image_content *ai = (struct apple_image_content *)c;
-    NSUInteger frames = [[image valueForProperty: NSImageFrameCount] unsignedIntegerValue];
-    if (frames > 1) {
-        ai->frames = frames;
-        ai->currentFrame = 0;
-        ai->frameTimes = calloc( ai->frames , sizeof(int));
-        for (NSUInteger i = 0; i < frames; i++) {
-            [image setProperty: NSImageCurrentFrame withValue: [NSNumber numberWithUnsignedInteger: i]];
-            ai->frameTimes[i] = 1000 * [[image valueForProperty: NSImageCurrentFrameDuration] floatValue];
-        }
-        [image setProperty: NSImageCurrentFrame withValue: [NSNumber numberWithUnsignedInteger: 0]];
-        cocoa_schedule( ai->frameTimes[0], animate_image_cb, ai );
-    }
-	
+
+	struct apple_image_content *ai = (struct apple_image_content *)c;
+	NSUInteger frames = [[image valueForProperty: NSImageFrameCount] unsignedIntegerValue];
+	if (frames > 1) {
+		ai->frames = frames;
+		ai->currentFrame = 0;
+		ai->frameTimes = calloc( ai->frames , sizeof(int));
+		for (NSUInteger i = 0; i < frames; i++) {
+			[image setProperty: NSImageCurrentFrame withValue: [NSNumber numberWithUnsignedInteger: i]];
+			ai->frameTimes[i] = 1000 * [[image valueForProperty: NSImageCurrentFrameDuration] floatValue];
+		}
+		[image setProperty: NSImageCurrentFrame withValue: [NSNumber numberWithUnsignedInteger: 0]];
+		cocoa_schedule( ai->frameTimes[0], animate_image_cb, ai );
+	}
+
 	return true;
 }
 
 
-void apple_image_destroy(struct content *c)
+static void apple_image_destroy(struct content *c)
 {
 	apple_image_content *ai_c = (apple_image_content *)c;
 
 	[(id)ai_c->bitmap release];
 	ai_c->bitmap = NULL;
-        cocoa_schedule(-1, animate_image_cb, c );
+	cocoa_schedule(-1, animate_image_cb, c );
 }
 
 
-nserror apple_image_clone(const struct content *old, struct content **newc)
+static nserror apple_image_clone(const struct content *old, struct content **newc)
 {
 	apple_image_content *ai;
 	apple_image_content *ai_old = (apple_image_content *)old;
@@ -235,11 +176,11 @@ nserror apple_image_clone(const struct content *old, struct content **newc)
 	}
 
 	*newc = (struct content *) ai;
-	
+
 	return NSERROR_OK;
 }
 
-content_type apple_image_content_type(void)
+static content_type apple_image_content_type(void)
 {
 	return CONTENT_IMAGE;
 }
@@ -247,8 +188,7 @@ content_type apple_image_content_type(void)
 /**
  * Redraw a CONTENT_APPLE_IMAGE with appropriate tiling.
  */
-
-bool apple_image_redraw(struct content *c, struct content_redraw_data *data,
+static bool apple_image_redraw(struct content *c, struct content_redraw_data *data,
 		const struct rect *clip, const struct redraw_context *ctx)
 {
 	apple_image_content *ai_c = (apple_image_content *)c;
@@ -263,4 +203,53 @@ bool apple_image_redraw(struct content *c, struct content_redraw_data *data,
 			ai_c->bitmap, data->background_colour, flags);
 }
 
+static const content_handler apple_image_content_handler = {
+	.create = apple_image_create,
+	.data_complete = apple_image_convert,
+	.destroy = apple_image_destroy,
+	.redraw = apple_image_redraw,
+	.clone = apple_image_clone,
+	.get_internal = apple_image_get_internal,
+	.type = apple_image_content_type,
+	.no_share = false
+};
+
+static nserror register_for_type( NSString *mime )
+{
+	const char *type = [mime UTF8String];
+	/* nsgif has priority since it supports animated GIF */
+#ifdef WITH_GIF
+	if (strcmp(type, "image/gif") == 0)
+		return NSERROR_OK;
+#endif
+
+	nserror error = content_factory_register_handler( type, &apple_image_content_handler );
+	if (error != NSERROR_OK) return error;
+
+	return NSERROR_OK;
+}
+
+/* exported interface documented in cocoa/apple_image.h */
+nserror apple_image_init(void)
+{
+	NSArray *utis = [NSBitmapImageRep imageTypes];
+	for (NSString *uti in utis) {
+		NSDictionary *declaration = [(NSDictionary *)UTTypeCopyDeclaration( (CFStringRef)uti ) autorelease];
+		id mimeTypes = [[declaration objectForKey: (NSString *)kUTTypeTagSpecificationKey] objectForKey: (NSString *)kUTTagClassMIMEType];
+
+		if (mimeTypes == nil) continue;
+
+		if (![mimeTypes isKindOfClass: [NSArray class]]) {
+			mimeTypes = [NSArray arrayWithObject: mimeTypes];
+		}
+
+		for (NSString *mime in mimeTypes) {
+			nserror error = register_for_type( mime );
+			if (error != NSERROR_OK) return error;
+		}
+	}
+
+	return NSERROR_OK;
+}
+
 #endif /* WITH_APPLE_IMAGE */