---
 libavcodec/cinepakenc.c | 148 ++----------------------------------------------
 1 file changed, 5 insertions(+), 143 deletions(-)

diff --git a/libavcodec/cinepakenc.c b/libavcodec/cinepakenc.c
index 619ee83..951d851 100644
--- a/libavcodec/cinepakenc.c
+++ b/libavcodec/cinepakenc.c
@@ -154,11 +154,6 @@ typedef struct {
     mb_info *mb;                                //MB RD state
     int min_strips;          //the current limit
     int max_strips;          //the current limit
-#ifdef CINEPAKENC_DEBUG
-    mb_info *best_mb;                           //TODO: remove. only used for 
printing stats
-    int num_v1_mode, num_v4_mode, num_mc_mode;
-    int num_v1_encs, num_v4_encs, num_skips;
-#endif
 // options
     int max_extra_cb_iterations;
     int skip_empty_cb;
@@ -241,11 +236,6 @@ static av_cold int cinepak_encode_init(AVCodecContext 
*avctx)
     if (!(s->mb = av_malloc(mb_count*sizeof(mb_info))))
         goto enomem;
 
-#ifdef CINEPAKENC_DEBUG
-    if (!(s->best_mb = av_malloc(mb_count*sizeof(mb_info))))
-        goto enomem;
-#endif
-
     av_lfg_init(&s->randctx, 1);
     s->avctx = avctx;
     s->w = avctx->width;
@@ -286,10 +276,6 @@ static av_cold int cinepak_encode_init(AVCodecContext 
*avctx)
     s->min_strips = s->min_min_strips;
     s->max_strips = s->max_max_strips;
 
-#ifdef CINEPAKENC_DEBUG
-    s->num_v1_mode = s->num_v4_mode = s->num_mc_mode = s->num_v1_encs = 
s->num_v4_encs = s->num_skips = 0;
-#endif
-
     return 0;
 
 enomem:
@@ -303,9 +289,6 @@ enomem:
     av_freep(&s->strip_buf);
     av_freep(&s->frame_buf);
     av_freep(&s->mb);
-#ifdef CINEPAKENC_DEBUG
-    av_freep(&s->best_mb);
-#endif
 
     for(x = 0; x < (avctx->pix_fmt == AV_PIX_FMT_RGB24 ? 4 : 3); x++)
         av_freep(&s->pict_bufs[x]);
@@ -313,11 +296,7 @@ enomem:
     return AVERROR(ENOMEM);
 }
 
-static int64_t calculate_mode_score(CinepakEncContext *s, int h, strip_info 
*info, int report, int *training_set_v1_shrunk, int *training_set_v4_shrunk
-#ifdef CINEPAK_REPORT_SERR
-, int64_t *serr
-#endif
-)
+static int64_t calculate_mode_score(CinepakEncContext *s, int h, strip_info 
*info, int report, int *training_set_v1_shrunk, int *training_set_v4_shrunk)
 {
     //score = FF_LAMBDA_SCALE * error + lambda * bits
     int x;
@@ -331,10 +310,6 @@ static int64_t calculate_mode_score(CinepakEncContext *s, 
int h, strip_info *inf
 
     //av_log(s->avctx, AV_LOG_INFO, "sizes %3i %3i -> %9lli score mb_count 
%i", info->v1_size, info->v4_size, (long long int)ret, mb_count);
 
-#ifdef CINEPAK_REPORT_SERR
-    *serr = 0;
-#endif
-
     switch(info->mode) {
     case MODE_V1_ONLY:
         //one byte per MB
@@ -344,9 +319,6 @@ static int64_t calculate_mode_score(CinepakEncContext *s, 
int h, strip_info *inf
         for(x = 0; x < mb_count; x++) {
             mb = &s->mb[x];
             ret += FF_LAMBDA_SCALE * mb->v1_error;
-#ifdef CINEPAK_REPORT_SERR
-            *serr += mb->v1_error;
-#endif
 // this function is never called for report in MODE_V1_ONLY
 //            if(!report)
             mb->best_encoding = ENC_V1;
@@ -366,9 +338,6 @@ static int64_t calculate_mode_score(CinepakEncContext *s, 
int h, strip_info *inf
                 else
                     score1 = s->lambda * 33 + FF_LAMBDA_SCALE * 
(mberr=mb->v4_error);
                 ret += score1;
-#ifdef CINEPAK_REPORT_SERR
-                *serr += mberr;
-#endif
             }
         } else { // find best mode per block
             for(x = 0; x < mb_count; x++) {
@@ -378,15 +347,9 @@ static int64_t calculate_mode_score(CinepakEncContext *s, 
int h, strip_info *inf
 
                 if(score1 <= score2) {
                     ret += score1;
-#ifdef CINEPAK_REPORT_SERR
-                    *serr += mb->v1_error;
-#endif
                     mb->best_encoding = ENC_V1;
                 } else {
                     ret += score2;
-#ifdef CINEPAK_REPORT_SERR
-                    *serr += mb->v4_error;
-#endif
                     mb->best_encoding = ENC_V4;
                 }
             }
@@ -404,36 +367,21 @@ static int64_t calculate_mode_score(CinepakEncContext *s, 
int h, strip_info *inf
                 score1 = s->lambda * 1  + FF_LAMBDA_SCALE * mb->skip_error;
                 if(mb->best_encoding == ENC_SKIP) {
                     ret += score1;
-#ifdef CINEPAK_REPORT_SERR
-                    *serr += mb->skip_error;
-#endif
                 } else if(mb->best_encoding == ENC_V1) {
                     if((score2=s->lambda * 10 + FF_LAMBDA_SCALE * 
mb->v1_error) >= score1) {
                         mb->best_encoding = ENC_SKIP;
                         ++v1_shrunk;
                         ret += score1;
-#ifdef CINEPAK_REPORT_SERR
-                        *serr += mb->skip_error;
-#endif
                     } else {
                         ret += score2;
-#ifdef CINEPAK_REPORT_SERR
-                        *serr += mb->v1_error;
-#endif
                     }
                 } else {
                     if((score3=s->lambda * 34 + FF_LAMBDA_SCALE * 
mb->v4_error) >= score1) {
                         mb->best_encoding = ENC_SKIP;
                         ++v4_shrunk;
                         ret += score1;
-#ifdef CINEPAK_REPORT_SERR
-                        *serr += mb->skip_error;
-#endif
                     } else {
                         ret += score3;
-#ifdef CINEPAK_REPORT_SERR
-                        *serr += mb->v4_error;
-#endif
                     }
                 }
             }
@@ -448,21 +396,12 @@ static int64_t calculate_mode_score(CinepakEncContext *s, 
int h, strip_info *inf
 
                 if(score1 <= score2 && score1 <= score3) {
                     ret += score1;
-#ifdef CINEPAK_REPORT_SERR
-                    *serr += mb->skip_error;
-#endif
                     mb->best_encoding = ENC_SKIP;
                 } else if(score2 <= score3) {
                     ret += score2;
-#ifdef CINEPAK_REPORT_SERR
-                    *serr += mb->v1_error;
-#endif
                     mb->best_encoding = ENC_V1;
                 } else {
                     ret += score3;
-#ifdef CINEPAK_REPORT_SERR
-                    *serr += mb->v4_error;
-#endif
                     mb->best_encoding = ENC_V4;
                 }
             }
@@ -937,16 +876,9 @@ static void write_strip_header(CinepakEncContext *s, int 
y, int h, int keyframe,
     //av_log(s->avctx, AV_LOG_INFO, "write_strip_header() %x keyframe=%d\n", 
buf[0], keyframe);
 }
 
-static int rd_strip(CinepakEncContext *s, int y, int h, int keyframe, 
AVPicture *last_pict, AVPicture *pict, AVPicture *scratch_pict, unsigned char 
*buf, int64_t *best_score
-#ifdef CINEPAK_REPORT_SERR
-, int64_t *best_serr
-#endif
-)
+static int rd_strip(CinepakEncContext *s, int y, int h, int keyframe, 
AVPicture *last_pict, AVPicture *pict, AVPicture *scratch_pict, unsigned char 
*buf, int64_t *best_score)
 {
     int64_t score = 0;
-#ifdef CINEPAK_REPORT_SERR
-    int64_t serr;
-#endif
     int best_size = 0;
     strip_info info;
 // for codebook optimization:
@@ -996,11 +928,7 @@ static int rd_strip(CinepakEncContext *s, int y, int h, 
int keyframe, AVPicture
                 info.mode = mode;
 // choose the best encoding per block, based on current experience
                 score = calculate_mode_score(s, h, &info, 0,
-                                             &v1shrunk, &v4shrunk
-#ifdef CINEPAK_REPORT_SERR
-, &serr
-#endif
-);
+                                             &v1shrunk, &v4shrunk);
 
                 if(mode != MODE_V1_ONLY){
                     int extra_iterations_limit = s->max_extra_cb_iterations;
@@ -1025,11 +953,7 @@ static int rd_strip(CinepakEncContext *s, int y, int h, 
int keyframe, AVPicture
 // ever grow the training sets back)
                     for(;;) {
                         score = calculate_mode_score(s, h, &info, 1,
-                                                     &v1shrunk, &v4shrunk
-#ifdef CINEPAK_REPORT_SERR
-, &serr
-#endif
-);
+                                                     &v1shrunk, &v4shrunk);
 // do we have a reason to reiterate? if so, have we reached the limit?
                         if((!v1shrunk && !v4shrunk) || 
!extra_iterations_limit--) break;
 // recompute the codebooks, omitting the extra blocks
@@ -1057,23 +981,11 @@ static int rd_strip(CinepakEncContext *s, int y, int h, 
int keyframe, AVPicture
                 if(best_size == 0 || score < *best_score) {
 
                     *best_score = score;
-#ifdef CINEPAK_REPORT_SERR
-                    *best_serr = serr;
-#endif
                     best_size = encode_mode(s, h, scratch_pict, last_pict, 
&info, s->strip_buf + STRIP_HEADER_SIZE);
 
                     //av_log(s->avctx, AV_LOG_INFO, "mode %i, %3i, %3i: %18lli 
%i B", mode, info.v1_size, info.v4_size, (long long int)score, best_size);
                     //av_log(s->avctx, AV_LOG_INFO, "\n");
-#ifdef CINEPAK_REPORT_SERR
-                    av_log(s->avctx, AV_LOG_INFO, "mode %i, %3i, %3i: %18lli 
%i B\n", mode, v1_size, v4_size, (long long int)serr, best_size);
-#endif
 
-#ifdef CINEPAKENC_DEBUG
-                    //save MB encoding choices
-                    memcpy(s->best_mb, s->mb, mb_count*sizeof(mb_info));
-#endif
-
-                    //memcpy(strip_temp + STRIP_HEADER_SIZE, strip_temp, 
best_size);
                     write_strip_header(s, y, h, keyframe, s->strip_buf, 
best_size);
 
                 }
@@ -1081,28 +993,6 @@ static int rd_strip(CinepakEncContext *s, int y, int h, 
int keyframe, AVPicture
         }
     }
 
-#ifdef CINEPAKENC_DEBUG
-    //gather stats. this will only work properly of MAX_STRIPS == 1
-    if(best_info.mode == MODE_V1_ONLY) {
-        s->num_v1_mode++;
-        s->num_v1_encs += s->w*h/MB_AREA;
-    } else {
-        if(best_info.mode == MODE_V1_V4)
-            s->num_v4_mode++;
-        else
-            s->num_mc_mode++;
-
-        int x;
-        for(x = 0; x < s->w*h/MB_AREA; x++)
-            if(s->best_mb[x].best_encoding == ENC_V1)
-                s->num_v1_encs++;
-            else if(s->best_mb[x].best_encoding == ENC_V4)
-                s->num_v4_encs++;
-            else
-                s->num_skips++;
-    }
-#endif
-
     best_size += STRIP_HEADER_SIZE;
     memcpy(buf, s->strip_buf, best_size);
 
@@ -1125,10 +1015,6 @@ static int rd_frame(CinepakEncContext *s, const AVFrame 
*frame, int isakeyframe,
     int num_strips, strip, i, y, nexty, size, temp_size, best_size;
     AVPicture last_pict, pict, scratch_pict;
     int64_t best_score = 0, score, score_temp;
-#ifdef CINEPAK_REPORT_SERR
-    int64_t best_serr = 0, serr, serr_temp;
-#endif
-
     int best_nstrips;
 
     if(s->pix_fmt == AV_PIX_FMT_RGB24) {
@@ -1179,9 +1065,6 @@ static int rd_frame(CinepakEncContext *s, const AVFrame 
*frame, int isakeyframe,
     for(num_strips = s->min_strips; num_strips <= s->max_strips && num_strips 
<= s->h / MB_SIZE; num_strips++) {
         score = 0;
         size = 0;
-#ifdef CINEPAK_REPORT_SERR
-        serr = 0;
-#endif
 
         for(y = 0, strip = 1; y < s->h; strip++, y = nexty) {
             int strip_height;
@@ -1204,17 +1087,10 @@ static int rd_frame(CinepakEncContext *s, const AVFrame 
*frame, int isakeyframe,
             get_sub_picture(s, 0, y, (AVPicture*)s->last_frame,    &last_pict);
             get_sub_picture(s, 0, y, (AVPicture*)s->scratch_frame, 
&scratch_pict);
 
-            if((temp_size = rd_strip(s, y, strip_height, isakeyframe, 
&last_pict, &pict, &scratch_pict, s->frame_buf + size + CVID_HEADER_SIZE, 
&score_temp
-#ifdef CINEPAK_REPORT_SERR
-, &serr_temp
-#endif
-)) < 0)
+            if((temp_size = rd_strip(s, y, strip_height, isakeyframe, 
&last_pict, &pict, &scratch_pict, s->frame_buf + size + CVID_HEADER_SIZE, 
&score_temp)) < 0)
                 return temp_size;
 
             score += score_temp;
-#ifdef CINEPAK_REPORT_SERR
-            serr += serr_temp;
-#endif
             size += temp_size;
             //av_log(s->avctx, AV_LOG_INFO, "strip %d, isakeyframe=%d", strip, 
isakeyframe);
             //av_log(s->avctx, AV_LOG_INFO, "\n");
@@ -1222,14 +1098,8 @@ static int rd_frame(CinepakEncContext *s, const AVFrame 
*frame, int isakeyframe,
 
         if(best_score == 0 || score < best_score) {
             best_score = score;
-#ifdef CINEPAK_REPORT_SERR
-            best_serr = serr;
-#endif
             best_size = size + write_cvid_header(s, s->frame_buf, num_strips, 
size, isakeyframe);
             //av_log(s->avctx, AV_LOG_INFO, "best number of strips so far: 
%2i, %12lli, %i B\n", num_strips, (long long int)score, best_size);
-#ifdef CINEPAK_REPORT_SERR
-            av_log(s->avctx, AV_LOG_INFO, "best number of strips so far: %2i, 
%12lli, %i B\n", num_strips, (long long int)serr, best_size);
-#endif
 
             FFSWAP(AVFrame *, s->best_frame, s->scratch_frame);
             memcpy(buf, s->frame_buf, best_size);
@@ -1306,18 +1176,10 @@ static av_cold int cinepak_encode_end(AVCodecContext 
*avctx)
     av_freep(&s->strip_buf);
     av_freep(&s->frame_buf);
     av_freep(&s->mb);
-#ifdef CINEPAKENC_DEBUG
-    av_freep(&s->best_mb);
-#endif
 
     for(x = 0; x < (avctx->pix_fmt == AV_PIX_FMT_RGB24 ? 4 : 3); x++)
         av_freep(&s->pict_bufs[x]);
 
-#ifdef CINEPAKENC_DEBUG
-    av_log(avctx, AV_LOG_INFO, "strip coding stats: %i V1 mode, %i V4 mode, %i 
MC mode (%i V1 encs, %i V4 encs, %i skips)\n",
-        s->num_v1_mode, s->num_v4_mode, s->num_mc_mode, s->num_v1_encs, 
s->num_v4_encs, s->num_skips);
-#endif
-
     return 0;
 }
 
-- 
2.1.4

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to