--- branches/dev-api-4/xvidcore/src/encoder.c	2003/08/07 15:42:50	1115
+++ branches/dev-api-4/xvidcore/src/encoder.c	2003/09/29 00:30:31	1157
@@ -21,7 +21,7 @@
  *  along with this program ; if not, write to the Free Software
  *  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA
  *
- * $Id: encoder.c,v 1.95.2.38 2003-08-07 15:41:33 chl Exp $
+ * $Id: encoder.c,v 1.95.2.41 2003-09-29 00:30:31 edgomez Exp $
  *
  ****************************************************************************/
 
@@ -389,16 +389,13 @@
 		image_null(&pEnc->queue[n].image);
 
 		
-	for (n = 0; n < pEnc->mbParam.max_bframes+1; n++)
-	{
+	for (n = 0; n < pEnc->mbParam.max_bframes+1; n++) {
 		if (image_create
 			(&pEnc->queue[n].image, pEnc->mbParam.edged_width,
 			 pEnc->mbParam.edged_height) < 0)
 			goto xvid_err_memory5;
-
 	}
 
-
 	/* timestamp stuff */
 
 	pEnc->mbParam.m_stamp = 0;
@@ -423,15 +420,12 @@
 
   xvid_err_memory5:
 
-	if (pEnc->mbParam.max_bframes > 0) {
-        int i;
-
-		for (i = 0; i < pEnc->mbParam.max_bframes+1; i++) {
-			image_destroy(&pEnc->queue[i].image, pEnc->mbParam.edged_width,
+	for (n = 0; n < pEnc->mbParam.max_bframes+1; n++) {
+			image_destroy(&pEnc->queue[n].image, pEnc->mbParam.edged_width,
 						  pEnc->mbParam.edged_height);
 		}
-		xvid_free(pEnc->queue);
-	}
+
+	xvid_free(pEnc->queue);
 
   xvid_err_memory4:
 
@@ -445,11 +439,8 @@
 
 			image_destroy(&pEnc->bframes[i]->image, pEnc->mbParam.edged_width,
 						  pEnc->mbParam.edged_height);
-	
 			xvid_free(pEnc->bframes[i]->mbs);
-	
 			xvid_free(pEnc->bframes[i]);
-
 		}	
 
 		xvid_free(pEnc->bframes);
@@ -538,16 +529,12 @@
 	int i;
 	
 	/* B Frames specific */
-	if (pEnc->mbParam.max_bframes > 0) {
-
-		for (i = 0; i < pEnc->mbParam.max_bframes+1; i++) {
-		
-			image_destroy(&pEnc->queue[i].image, pEnc->mbParam.edged_width,
+	for (i = 0; i < pEnc->mbParam.max_bframes+1; i++) {
+		image_destroy(&pEnc->queue[i].image, pEnc->mbParam.edged_width,
 					  pEnc->mbParam.edged_height);
-		}
-		xvid_free(pEnc->queue);
 	}
 
+	xvid_free(pEnc->queue);
 
 	if (pEnc->mbParam.max_bframes > 0) {
 
@@ -558,9 +545,7 @@
 
 			image_destroy(&pEnc->bframes[i]->image, pEnc->mbParam.edged_width,
 					  pEnc->mbParam.edged_height);
-
 			xvid_free(pEnc->bframes[i]->mbs);
-
 			xvid_free(pEnc->bframes[i]);
 		}
 
@@ -1199,6 +1184,21 @@
 
 		/* ---- update vol flags at IVOP ----------- */
 		pEnc->current->vol_flags = pEnc->mbParam.vol_flags = frame->vol_flags;
+		switch(frame->par) {
+		case XVID_PAR_11_VGA:
+		case XVID_PAR_43_PAL:
+		case XVID_PAR_43_NTSC:
+		case XVID_PAR_169_PAL:
+		case XVID_PAR_169_NTSC:
+		case XVID_PAR_EXT:
+			pEnc->mbParam.par = frame->par;
+			break;
+		default:
+			pEnc->mbParam.par = XVID_PAR_EXT;
+			break;
+		}
+		pEnc->mbParam.par_width = (frame->par_width)?frame->par_width:1;
+		pEnc->mbParam.par_height = (frame->par_height)?frame->par_height:1;
 
         if ((pEnc->mbParam.vol_flags & XVID_VOL_MPEGQUANT)) {
 			if (frame->quant_intra_matrix != NULL)
@@ -1568,6 +1568,11 @@
 
 				current->sStat.kblks++;
 
+				if (pEnc->current->vop_flags & XVID_VOP_GREYSCALE)
+				{	pMB->cbp &= 0x3C;		/* keep only bits 5-2 */
+					qcoeff[4*64+0]=0;		/* zero, because for INTRA MBs DC value is saved */
+					qcoeff[5*64+0]=0;
+				}
 				MBCoding(current, pMB, qcoeff, bs, &current->sStat);
 				stop_coding_timer();
 				continue;
@@ -1894,9 +1899,12 @@
 				}
 			}
 
-#ifdef BFRAMES_DEC_DEBUG
-	BFRAME_DEBUG
-#endif
+			/* keep only bits 5-2 -- Chroma blocks will just be skipped by the
+			 * coding function for BFrames, that's why we don't zero teh DC
+			 * coeffs */
+			if ((frame->vop_flags & XVID_VOP_GREYSCALE))
+				mb->cbp &= 0x3C;
+
 			start_timer();
 			MBCodingBVOP(mb, qcoeff, frame->fcode, frame->bcode, bs,
 						 &frame->sStat, direction);