--- mbcoding.c	2002/06/30 10:46:29	1.20
+++ mbcoding.c	2002/09/23 20:36:01	1.25.2.1
@@ -280,7 +280,7 @@
 }
 
 
-static void
+static __inline void
 CodeBlockIntra(const FRAMEINFO * frame,
 			   const MACROBLOCK * pMB,
 			   int16_t qcoeff[6 * 64],
@@ -370,8 +370,10 @@
 
 	// interlacing
 	if (frame->global_flags & XVID_INTERLACING) {
-		BitstreamPutBit(bs, pMB->field_dct);
-		DEBUG1("codep: field_dct: ", pMB->field_dct);
+		if (pMB->cbp) {
+			BitstreamPutBit(bs, pMB->field_dct);
+			DEBUG1("codep: field_dct: ", pMB->field_dct);
+		}
 
 		// if inter block, write field ME flag
 		if (pMB->mode == MODE_INTER || pMB->mode == MODE_INTER_Q) {
@@ -412,24 +414,26 @@
 		 Statistics * pStat)
 {
 
-	int intra = (pMB->mode == MODE_INTRA || pMB->mode == MODE_INTRA_Q);
-
 	if (frame->coding_type == P_VOP) {
-		if (pMB->cbp == 0 && pMB->mode == MODE_INTER && pMB->mvs[0].x == 0 &&
-			pMB->mvs[0].y == 0) {
-			BitstreamPutBit(bs, 1);	// not_coded
-			return;
-		} else
 			BitstreamPutBit(bs, 0);	// coded
 	}
 
-	if (intra)
+	if (pMB->mode == MODE_INTRA || pMB->mode == MODE_INTRA_Q)
 		CodeBlockIntra(frame, pMB, qcoeff, bs, pStat);
 	else
 		CodeBlockInter(frame, pMB, qcoeff, bs, pStat);
 
 }
 
+/*
+// moved to mbcoding.h so that in can be 'static __inline'
+void
+MBSkip(Bitstream * bs)
+{
+	BitstreamPutBit(bs, 1);	// not coded
+}
+*/
+
 /***************************************************************
  * bframe encoding start
  ***************************************************************/
@@ -442,37 +446,22 @@
 	3	0001b	forward mc+q		dbquant, mvdf
 */
 
-void
+static __inline void
 put_bvop_mbtype(Bitstream * bs,
 				int value)
 {
 	switch (value) {
-	case 0:
-		BitstreamPutBit(bs, 1);
-		return;
-
-	case 1:
-		BitstreamPutBit(bs, 0);
-		BitstreamPutBit(bs, 1);
-		return;
-
-	case 2:
-		BitstreamPutBit(bs, 0);
-		BitstreamPutBit(bs, 0);
-		BitstreamPutBit(bs, 1);
-		return;
-
-	case 3:
-		BitstreamPutBit(bs, 0);
-		BitstreamPutBit(bs, 0);
-		BitstreamPutBit(bs, 0);
-		BitstreamPutBit(bs, 1);
-		return;
-
-	default:;					// invalid!
-
+		case MODE_FORWARD:
+			BitstreamPutBit(bs, 0);
+		case MODE_BACKWARD:
+			BitstreamPutBit(bs, 0);
+		case MODE_INTERPOLATE:
+			BitstreamPutBit(bs, 0);
+		case MODE_DIRECT:
+			BitstreamPutBit(bs, 1);
+		default:
+			break;
 	}
-
 }
 
 /*
@@ -482,7 +471,7 @@
 	+2	11b
 */
 
-void
+static __inline void
 put_bvop_dbquant(Bitstream * bs,
 				 int value)
 {
@@ -515,14 +504,15 @@
 			 Bitstream * bs,
 			 Statistics * pStat)
 {
-	int i;
+	int vcode = fcode;
+	unsigned int i;
 
 /*	------------------------------------------------------------------
-		when a block is skipped it is decoded DIRECT(0,)
-		hence are interpolated from forward & backward frames
+		when a block is skipped it is decoded DIRECT(0,0)
+		hence is interpolated from forward & backward frames
 	------------------------------------------------------------------ */
 
-	if (mb->mode == 5) {
+	if (mb->mode == MODE_DIRECT_NONE_MV) {
 		BitstreamPutBit(bs, 1);	// skipped
 		return;
 	}
@@ -545,18 +535,20 @@
 		put_bvop_dbquant(bs, 0);	// todo: mb->dquant = 0
 	}
 
-	if (mb->mode == MODE_INTERPOLATE || mb->mode == MODE_FORWARD) {
-		CodeVector(bs, mb->pmvs[0].x, fcode, pStat);
-		CodeVector(bs, mb->pmvs[0].y, fcode, pStat);
-	}
-
-	if (mb->mode == MODE_INTERPOLATE || mb->mode == MODE_BACKWARD) {
-		CodeVector(bs, mb->b_pmvs[0].x, bcode, pStat);
-		CodeVector(bs, mb->b_pmvs[0].y, bcode, pStat);
-	}
-
-	if (mb->mode == MODE_DIRECT) {
-		// TODO: direct
+	switch (mb->mode) {
+		case MODE_INTERPOLATE:
+			CodeVector(bs, mb->pmvs[1].x, vcode, pStat); //forward vector of interpolate mode
+			CodeVector(bs, mb->pmvs[1].y, vcode, pStat);
+		case MODE_BACKWARD:
+			vcode = bcode;
+		case MODE_FORWARD:
+			CodeVector(bs, mb->pmvs[0].x, vcode, pStat);
+			CodeVector(bs, mb->pmvs[0].y, vcode, pStat);
+			break;
+		case MODE_DIRECT:
+			CodeVector(bs, mb->pmvs[3].x, 1, pStat);	// fcode is always 1 for delta vector
+			CodeVector(bs, mb->pmvs[3].y, 1, pStat);	// prediction is always (0,0)
+		default: break;
 	}
 
 	for (i = 0; i < 6; i++) {
@@ -644,7 +636,7 @@
 
 }
 
-int
+static __inline int
 get_mv_data(Bitstream * bs)
 {
 
@@ -761,9 +753,7 @@
 {
 
 	const uint16_t *scan = scan_tables[direction];
-	int level;
-	int run;
-	int last;
+	int level, run, last;
 
 	do {
 		level = get_coeff(bs, &run, &last, 1, 0);