sync

TODO

@@ -18,6 +18,11 @@ sys	0m0.136s
 
   is the real time difference just due to smalltile vs. thinstrip?
 
+  can we trick vipsthumbnail into using fatstrip?
+
+- vipsthumbnail a tif with an embedded profile ... don't seem to get a profile
+  in the jpg?
+
 
 
 blocking bugs

libvips/conversion/embed.c

@@ -401,8 +401,13 @@ vips_embed_build( VipsObject *object )
 	case VIPS_EXTEND_COPY:
 		if( vips_image_copy_fields( conversion->out, embed->in ) )
 			return( -1 );
+
+		/* embed is used in many places. SMALLTILE would force most
+		 * pipelines into SMALLTILE mode, so stick with THINSTRIP,
+		 * even though SMALLTILE might be a little faster for us.
+		 */
 		vips_demand_hint( conversion->out, 
-			VIPS_DEMAND_STYLE_SMALLTILE, embed->in, NULL );
+			VIPS_DEMAND_STYLE_FATSTRIP, embed->in, NULL );
 
 		conversion->out->Xsize = embed->width;
 		conversion->out->Ysize = embed->height;

libvips/conversion/sequential.c

@@ -34,8 +34,8 @@
  */
 
 /*
-#define VIPS_DEBUG
  */
+#define VIPS_DEBUG
 
 #ifdef HAVE_CONFIG_H
 #include <config.h>
@@ -79,7 +79,7 @@ vips_sequential_generate( VipsRegion *or,
 	 */
 	if( r->top != sequential->y_pos ) {
 		printf( "vips_sequential_generate: error -- "
-			"at position %d in file, but position %d requested",
+			"at position %d in file, but position %d requested\n",
 			sequential->y_pos, r->top );
 
 		vips_error( "VipsSequential", 

libvips/iofuncs/image.c

@@ -1621,6 +1621,10 @@ vips_image_write_gen( VipsRegion *or,
 	VipsRegion *ir = (VipsRegion *) seq;
 	VipsRect *r = &or->valid;
 
+	printf( "vips_image_write_gen: "
+		"left = %d, top = %d, width = %d, height = %d\n",
+		r->left, r->top, r->width, r->height ); 
+
 	/* Copy with pointers.
 	 */
 	if( vips_region_prepare( ir, r ) ||