mostly whitespace changes

stepmania/src/SDL_SaveJPEG.cpp

@@ -24,7 +24,8 @@ namespace jpeg
 
 
 #define OUTPUT_BUFFER_SIZE	4096
-typedef struct {
+typedef struct
+{
 	struct jpeg::jpeg_destination_mgr pub;
 
 	SDL_RWops *ctx;
@@ -38,13 +39,11 @@ typedef struct {
  */
 static void init_destination( jpeg::j_compress_ptr cinfo )
 {
-	/* We don't actually need to do anything */
+	/* nop */
 	return;
 }
 
-/*
+/* Empty the output buffer; called whenever buffer is full. */
- * Empty the output buffer --- called whenever buffer is full.
- */
 static jpeg::boolean empty_output_buffer( jpeg::j_compress_ptr cinfo )
 {
 	my_destination_mgr * dest = (my_destination_mgr *) cinfo->dest;
@@ -66,13 +65,9 @@ static void term_destination (jpeg::j_compress_ptr cinfo)
 {
 	/* Write data remaining in the buffer */
 	my_destination_mgr *dest = (my_destination_mgr *) cinfo->dest;
-	int nbytes;
+	SDL_RWwrite(dest->ctx, dest->buffer, 1, OUTPUT_BUFFER_SIZE - dest->pub.free_in_buffer);
-
-	nbytes = SDL_RWwrite(dest->ctx, dest->buffer, 1, OUTPUT_BUFFER_SIZE - dest->pub.free_in_buffer);
 	dest->pub.next_output_byte = dest->buffer;
 	dest->pub.free_in_buffer = OUTPUT_BUFFER_SIZE;
-
-	return;
 }
 
 /*
@@ -89,8 +84,7 @@ static void jpeg_SDL_RW_dest(jpeg::j_compress_ptr cinfo, SDL_RWops *ctx)
 	 * only before the first one.  (If we discarded the buffer at the end of
 	 * one image, we'd likely lose the start of the next one.)
 	 * This makes it unsafe to use this manager and a different source
-   * manager serially with the same JPEG object.  Caveat programmer.
+	 * manager serially with the same JPEG object.  Caveat programmer. */
-   */
 	if( cinfo->dest == NULL )
 	{
 		/* first time for this JPEG object? */
@@ -119,13 +113,7 @@ void IMG_SaveJPG_RW( SDL_Surface *surface, SDL_RWops *dest, bool bHighQual )
 
 	struct jpeg::jpeg_compress_struct cinfo;
 
-  /* Step 1: allocate and initialize JPEG compression object */
+	/* Set up the error handler. */
-
-  /* We have to set up the error handler first, in case the initialization
-   * step fails.  (Unlikely, but it could happen if you are out of memory.)
-   * This routine fills in the contents of struct jerr, and returns jerr's
-   * address which we place into the link field in cinfo.
-   */
 	struct jpeg::jpeg_error_mgr jerr;
 	cinfo.err = jpeg::jpeg_std_error( &jerr );
 
@@ -133,73 +121,44 @@ void IMG_SaveJPG_RW( SDL_Surface *surface, SDL_RWops *dest, bool bHighQual )
 	jpeg::jpeg_CreateCompress(&cinfo, JPEG_LIB_VERSION, \
 		(size_t) sizeof(struct jpeg::jpeg_compress_struct));
 
-  /* Step 2: specify data destination (eg, a file) */
-  /* Note: steps 2 and 3 can be done in either order. */
-
-  /* Here we use the library-supplied code to send compressed data to a
-   * stdio stream.  You can also write your own code to do something else.
-   * VERY IMPORTANT: use "b" option to fopen() if you are on a machine that
-   * requires it in order to write binary files.
-   */
-  jpeg_SDL_RW_dest(&cinfo, dest);
-
-  /* Step 3: set parameters for compression */
-
-  /* First we supply a description of the input image.
-   * Four fields of the cinfo struct must be filled in:
-   */
 	cinfo.image_width = surface->w; 	/* image width and height, in pixels */
 	cinfo.image_height = surface->h;
 	cinfo.input_components = 3;		/* # of color components per pixel */
 	cinfo.in_color_space = jpeg::JCS_RGB; 	/* colorspace of input image */
-  /* Now use the library's routine to set default compression parameters.
+
-   * (You must set at least cinfo.in_color_space before calling this,
+	/* Set compression parameters.  You must set at least cinfo.in_color_space before
-   * since the defaults depend on the source color space.)
+	 * calling this.*/
-   */
 	jpeg::jpeg_set_defaults(&cinfo);
+
 	if( bHighQual )
 		jpeg::jpeg_set_quality( &cinfo, 150, TRUE );
 	else
 		jpeg::jpeg_set_quality( &cinfo, 40, TRUE );
 
-  /* Step 4: Start compressor */
+	jpeg_SDL_RW_dest( &cinfo, dest );
 
-  /* TRUE ensures that we will write a complete interchange-JPEG file.
+	/* Start the compressor. */
-   * Pass TRUE unless you are very sure of what you're doing.
-   */
 	jpeg::jpeg_start_compress( &cinfo, TRUE );
 
-  /* Step 5: while (scan lines remain to be written) */
-  /*           jpeg_write_scanlines(...); */
-
 	/* Here we use the library's state variable cinfo.next_scanline as the
 	 * loop counter, so that we don't have to keep track ourselves.
 	 * To keep things simple, we pass one scanline per call; you can pass
-   * more if you wish, though.
+	 * more if you wish, though. */
-   */
 	const int row_stride = surface->pitch;	/* JSAMPLEs per row in image_buffer */
 
 	while( cinfo.next_scanline < cinfo.image_height )
 	{
 		/* jpeg_write_scanlines expects an array of pointers to scanlines.
 		 * Here the array is only one element long, but you could pass
-     * more than one scanline at a time if that's more convenient.
+		 * more than one scanline at a time if that's more convenient. */
-     */
 		jpeg::JSAMPROW row_pointer = & ((jpeg::JSAMPLE*)surface->pixels)[cinfo.next_scanline * row_stride];
 		jpeg::jpeg_write_scanlines( &cinfo, &row_pointer, 1 );
 	}
 
-  /* Step 6: Finish compression */
+	/* Finish compression. */
-
 	jpeg::jpeg_finish_compress( &cinfo );
-  /* After finish_compress, we can close the output file. */
-
-  /* Step 7: release JPEG compression object */
-
-  /* This is an important step since it will release a good deal of memory. */
 	jpeg::jpeg_destroy_compress( &cinfo );
 
-  /* And we're done! */
 	if( dst_surface )
 		SDL_FreeSurface( dst_surface );
 }