From owner-p4-projects@FreeBSD.ORG Tue Nov 13 22:32:32 2012 Return-Path: Delivered-To: p4-projects@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 32767) id 11FE2763; Tue, 13 Nov 2012 22:32:32 +0000 (UTC) Delivered-To: perforce@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id BFF8C761 for ; Tue, 13 Nov 2012 22:32:31 +0000 (UTC) (envelope-from brooks@freebsd.org) Received: from skunkworks.freebsd.org (skunkworks.freebsd.org [IPv6:2001:4f8:fff6::2d]) by mx1.freebsd.org (Postfix) with ESMTP id 8D36A8FC08 for ; Tue, 13 Nov 2012 22:32:31 +0000 (UTC) Received: from skunkworks.freebsd.org (localhost [127.0.0.1]) by skunkworks.freebsd.org (8.14.5/8.14.5) with ESMTP id qADMWVpX044072 for ; Tue, 13 Nov 2012 22:32:31 GMT (envelope-from brooks@freebsd.org) Received: (from perforce@localhost) by skunkworks.freebsd.org (8.14.5/8.14.5/Submit) id qADMWVFR044069 for perforce@freebsd.org; Tue, 13 Nov 2012 22:32:31 GMT (envelope-from brooks@freebsd.org) Date: Tue, 13 Nov 2012 22:32:31 GMT Message-Id: <201211132232.qADMWVFR044069@skunkworks.freebsd.org> X-Authentication-Warning: skunkworks.freebsd.org: perforce set sender to brooks@freebsd.org using -f From: Brooks Davis Subject: PERFORCE change 219773 for review To: Perforce Change Reviews Precedence: bulk X-BeenThere: p4-projects@freebsd.org X-Mailman-Version: 2.1.14 List-Id: p4 projects tree changes List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 13 Nov 2012 22:32:32 -0000 http://p4web.freebsd.org/@@219773?ac=10 Change 219773 by brooks@brooks_zenith on 2012/11/13 22:32:25 Add the ability to pass in alternative read and row_callback functions in anticipation of capability specific versions. Affected files ... .. //depot/projects/ctsrd/cheribsd/src/ctsrd-lib/libimagebox/decode_png.c#2 edit .. //depot/projects/ctsrd/cheribsd/src/ctsrd-lib/libimagebox/iboxpriv.h#2 edit .. //depot/projects/ctsrd/cheribsd/src/ctsrd-lib/libimagebox/pngbox.c#4 edit .. //depot/projects/ctsrd/cheribsd/src/ctsrd/libexec/readpng/readpng.c#2 edit Differences ... ==== //depot/projects/ctsrd/cheribsd/src/ctsrd-lib/libimagebox/decode_png.c#2 (text+ko) ==== @@ -41,7 +41,8 @@ static void read_png_from_fd(png_structp, png_bytep, png_size_t); void -decode_png(struct ibox_decode_state *ids) +decode_png(struct ibox_decode_state *ids, + png_rw_ptr user_read_fn, png_read_status_ptr read_row_fn) { int bit_depth, color_type, interlace_type; png_uint_32 r, width, height; @@ -75,7 +76,10 @@ read_chunk_callback); #endif - png_set_read_status_fn(png_ptr, read_row_callback); + if (read_row_fn != NULL) + png_set_read_status_fn(png_ptr, read_row_fn); + else + png_set_read_status_fn(png_ptr, read_row_callback); /* * Reject the image if the parser finds a different size than @@ -85,7 +89,10 @@ png_set_user_limits(png_ptr, width, height); #endif - png_set_read_fn(png_ptr, ids, read_png_from_fd); + if (user_read_fn != NULL) + png_set_read_fn(png_ptr, ids, user_read_fn); + else + png_set_read_fn(png_ptr, ids, read_png_from_fd); png_read_info(png_ptr, info_ptr); ==== //depot/projects/ctsrd/cheribsd/src/ctsrd-lib/libimagebox/iboxpriv.h#2 (text+ko) ==== @@ -38,6 +38,7 @@ uint32_t *buffer; }; -void decode_png(struct ibox_decode_state *); +void decode_png(struct ibox_decode_state *ids, + png_rw_ptr user_read_fn, png_read_status_ptr read_row_fn); #endif ==== //depot/projects/ctsrd/cheribsd/src/ctsrd-lib/libimagebox/pngbox.c#4 (text+ko) ==== @@ -62,7 +62,7 @@ { struct ibox_decode_state *ids = arg; - decode_png(ids); + decode_png(ids, NULL, NULL); free(ids); ==== //depot/projects/ctsrd/cheribsd/src/ctsrd/libexec/readpng/readpng.c#2 (text+ko) ==== @@ -33,6 +33,7 @@ #include #include +#include #include #include "imagebox.h" @@ -58,6 +59,6 @@ PROT_READ | PROT_WRITE, MAP_SHARED, bfd, 0)) == MAP_FAILED) err(1, "mmap buffer"); - decode_png(&ids); + decode_png(&ids, NULL, NULL); return (0); }