The !!! have been reviewed in the lib. The critical have been changed to #warning.
--- a/src/jpeg-7a/jdrapi.c Sun Nov 26 16:47:40 2006 +0100
+++ b/src/jpeg-7a/jdrapi.c Sun Nov 26 16:53:17 2006 +0100
@@ -52,7 +52,7 @@
if (myrest->target == target && plane_applied == FALSE)
{
/* Don't bother to IDCT an uninteresting component. */
- /* !!! The "needness" must be updated */
+#warning TODO: The "needness" must be updated depending on the methods used
if (! compptr->component_needed)
continue;
@@ -76,7 +76,7 @@
rest_process_restorators(j_decompress_ptr cinfo,
JSAMPIMAGE output_buf, const jvirt_barray_ptr whole_image[])
{
- /* !!! Check colorspace !!! */
+#warning TODO: Check colorspace
/* Process the plane unpacking */
apply_plane_idct(cinfo, JREST_TARGET_IDCT_Y, output_buf, whole_image);
if (cinfo->num_components > 1)
--- a/src/jpeg-7a/jdrdct.c Sun Nov 26 16:47:40 2006 +0100
+++ b/src/jpeg-7a/jdrdct.c Sun Nov 26 16:53:17 2006 +0100
@@ -599,7 +599,7 @@
ptr=out_gray;
for (y=0;y<DCTSIZE;y++)
for (x=0;x<DCTSIZE;x++)
- out[adr+x+y*w]=*(ptr++); /* !!! Cap !!! */
+ out[adr+x+y*w]=*(ptr++);
}
}
--- a/src/jpeg-7a/jdrfrom.c Sun Nov 26 16:47:40 2006 +0100
+++ b/src/jpeg-7a/jdrfrom.c Sun Nov 26 16:53:17 2006 +0100
@@ -382,6 +382,4 @@
} /* End of main loop */
- /* !!! Free call */
- /*free(dE);*/
}
--- a/src/jpeg-7a/jdrrober.c Sun Nov 26 16:47:40 2006 +0100
+++ b/src/jpeg-7a/jdrrober.c Sun Nov 26 16:53:17 2006 +0100
@@ -307,7 +307,7 @@
last_obj = calc_objective(rest, myrest->bitmap);
/* Save a copy of the first bitmap. z_q in the article. */
- /* !!! It should be in some API */
+ /* !! It should be in some API */
memcpy(myrest->first_bitmap, myrest->bitmap,
sizeof(RSAMP)*myrest->h*myrest->w);