[FFmpeg-devel] [PATCH 2/3] libavcodec/ccaption_dec: clean up and standardize white space
Anshul Maheshwari
er.anshul.maheshwari at gmail.com
Wed Jan 6 07:34:12 CET 2016
On Tue, Jan 5, 2016 at 8:58 AM, Aman Gupta <ffmpeg at tmm1.net> wrote:
> From: Aman Gupta <aman at tmm1.net>
>
> ---
> libavcodec/ccaption_dec.c | 98
> ++++++++++++++++++++++-------------------------
> 1 file changed, 45 insertions(+), 53 deletions(-)
>
> diff --git a/libavcodec/ccaption_dec.c b/libavcodec/ccaption_dec.c
> index 4e478e0..96f0ccf 100644
> --- a/libavcodec/ccaption_dec.c
> +++ b/libavcodec/ccaption_dec.c
> @@ -145,10 +145,9 @@ struct Screen {
> * for setting row 1 use row | (1 << 0)
> * for setting row 15 use row | (1 << 14)
> */
> - int16_t row_used;
> + int16_t row_used;
> };
>
> -
> typedef struct CCaptionSubContext {
> AVClass *class;
> struct Screen screen[2];
> @@ -168,7 +167,7 @@ typedef struct CCaptionSubContext {
> char prev_cmd[2];
> /* buffer to store pkt data */
> AVBufferRef *pktbuf;
> -}CCaptionSubContext;
> +} CCaptionSubContext;
>
>
> static av_cold int init_decoder(AVCodecContext *avctx)
> @@ -181,12 +180,12 @@ static av_cold int init_decoder(AVCodecContext
> *avctx)
> ctx->mode = CCMODE_ROLLUP_2;
> ctx->rollup = 2;
> ret = ff_ass_subtitle_header_default(avctx);
> - if(ret < 0) {
> + if (ret < 0) {
> return ret;
> }
> /* allocate pkt buffer */
> ctx->pktbuf = av_buffer_alloc(128);
> - if( !ctx->pktbuf) {
> + if (!ctx->pktbuf) {
> ret = AVERROR(ENOMEM);
> }
> return ret;
> @@ -195,7 +194,7 @@ static av_cold int init_decoder(AVCodecContext *avctx)
> static av_cold int close_decoder(AVCodecContext *avctx)
> {
> CCaptionSubContext *ctx = avctx->priv_data;
> - av_bprint_finalize( &ctx->buffer, NULL);
> + av_bprint_finalize(&ctx->buffer, NULL);
> av_buffer_unref(&ctx->pktbuf);
> return 0;
> }
> @@ -203,14 +202,14 @@ static av_cold int close_decoder(AVCodecContext
> *avctx)
> /**
> * @param ctx closed caption context just to print log
> */
> -static int write_char (CCaptionSubContext *ctx, char *row,uint8_t col,
> char ch)
> +static int write_char(CCaptionSubContext *ctx, char *row, uint8_t col,
> char ch)
> {
> - if(col < SCREEN_COLUMNS) {
> + if (col < SCREEN_COLUMNS) {
> row[col] = ch;
> return 0;
> }
> /* We have extra space at end only for null character */
> - else if ( col == SCREEN_COLUMNS && ch == 0) {
> + else if (col == SCREEN_COLUMNS && ch == 0) {
> row[col] = ch;
> return 0;
> }
> @@ -227,7 +226,7 @@ static int write_char (CCaptionSubContext *ctx, char
> *row,uint8_t col, char ch)
> * If the second byte doesn't pass parity, it returns INVALIDDATA
> * user can ignore the whole pair and pass the other pair.
> */
> -static int validate_cc_data_pair (uint8_t *cc_data_pair)
> +static int validate_cc_data_pair(uint8_t *cc_data_pair)
> {
> uint8_t cc_valid = (*cc_data_pair & 4) >>2;
> uint8_t cc_type = *cc_data_pair & 3;
> @@ -246,21 +245,19 @@ static int validate_cc_data_pair (uint8_t
> *cc_data_pair)
> }
>
> //Skip non-data
> - if( (cc_data_pair[0] == 0xFA || cc_data_pair[0] == 0xFC ||
> cc_data_pair[0] == 0xFD )
> + if ((cc_data_pair[0] == 0xFA || cc_data_pair[0] == 0xFC ||
> cc_data_pair[0] == 0xFD)
> && (cc_data_pair[1] & 0x7F) == 0 && (cc_data_pair[2] & 0x7F) ==
> 0)
> return AVERROR_PATCHWELCOME;
>
> //skip 708 data
> - if(cc_type == 3 || cc_type == 2 )
> + if (cc_type == 3 || cc_type == 2)
> return AVERROR_PATCHWELCOME;
>
> /* remove parity bit */
> cc_data_pair[1] &= 0x7F;
> cc_data_pair[2] &= 0x7F;
>
> -
> return 0;
> -
> }
>
> static struct Screen *get_writing_screen(CCaptionSubContext *ctx)
> @@ -286,7 +283,7 @@ static void roll_up(CCaptionSubContext *ctx)
> struct Screen *screen;
> int i, keep_lines;
>
> - if(ctx->mode == CCMODE_TEXT)
> + if (ctx->mode == CCMODE_TEXT)
> return;
>
> screen = get_writing_screen(ctx);
> @@ -296,22 +293,21 @@ static void roll_up(CCaptionSubContext *ctx)
> */
> keep_lines = FFMIN(ctx->cursor_row + 1, ctx->rollup);
>
> - for( i = 0; i < ctx->cursor_row - keep_lines; i++ )
> + for (i = 0; i < ctx->cursor_row - keep_lines; i++)
> UNSET_FLAG(screen->row_used, i);
>
>
> - for( i = 0; i < keep_lines && screen->row_used; i++ ) {
> + for (i = 0; i < keep_lines && screen->row_used; i++) {
> const int i_row = ctx->cursor_row - keep_lines + i + 1;
>
> - memcpy( screen->characters[i_row], screen->characters[i_row+1],
> SCREEN_COLUMNS );
> - memcpy( screen->colors[i_row], screen->colors[i_row+1],
> SCREEN_COLUMNS);
> - memcpy( screen->fonts[i_row], screen->fonts[i_row+1],
> SCREEN_COLUMNS);
> - if(CHECK_FLAG(screen->row_used, i_row + 1))
> + memcpy(screen->characters[i_row], screen->characters[i_row+1],
> SCREEN_COLUMNS);
> + memcpy(screen->colors[i_row], screen->colors[i_row+1],
> SCREEN_COLUMNS);
> + memcpy(screen->fonts[i_row], screen->fonts[i_row+1],
> SCREEN_COLUMNS);
> + if (CHECK_FLAG(screen->row_used, i_row + 1))
> SET_FLAG(screen->row_used, i_row);
>
> }
> UNSET_FLAG(screen->row_used, ctx->cursor_row);
> -
> }
>
> static int reap_screen(CCaptionSubContext *ctx, int64_t pts)
> @@ -321,9 +317,9 @@ static int reap_screen(CCaptionSubContext *ctx,
> int64_t pts)
> struct Screen *screen = ctx->screen + ctx->active_screen;
> ctx->start_time = ctx->startv_time;
>
> - for( i = 0; screen->row_used && i < SCREEN_ROWS; i++)
> + for (i = 0; screen->row_used && i < SCREEN_ROWS; i++)
> {
> - if(CHECK_FLAG(screen->row_used,i)) {
> + if (CHECK_FLAG(screen->row_used,i)) {
> char *str = screen->characters[i];
> /* skip space */
> while (*str == ' ')
> @@ -331,14 +327,14 @@ static int reap_screen(CCaptionSubContext *ctx,
> int64_t pts)
>
> av_bprintf(&ctx->buffer, "%s\\N", str);
> ret = av_bprint_is_complete(&ctx->buffer);
> - if( ret == 0) {
> + if (ret == 0) {
> ret = AVERROR(ENOMEM);
> break;
> }
> }
>
> }
> - if(screen->row_used && ctx->buffer.len >= 2 ) {
> + if (screen->row_used && ctx->buffer.len >= 2) {
> ctx->buffer.len -= 2;
> ctx->buffer.str[ctx->buffer.len] = 0;
> }
> @@ -347,26 +343,26 @@ static int reap_screen(CCaptionSubContext *ctx,
> int64_t pts)
> return ret;
> }
>
> -static void handle_textattr( CCaptionSubContext *ctx, uint8_t hi, uint8_t
> lo )
> +static void handle_textattr(CCaptionSubContext *ctx, uint8_t hi, uint8_t
> lo)
> {
> int i = lo - 0x20;
> int ret;
> struct Screen *screen = get_writing_screen(ctx);
> char *row = screen->characters[ctx->cursor_row];
>
> - if( i >= 32)
> + if (i >= 32)
> return;
>
> - ctx->cursor_color = pac2_attribs[i][0];
> + ctx->cursor_color = pac2_attribs[i][0];
> ctx->cursor_font = pac2_attribs[i][1];
>
> SET_FLAG(screen->row_used,ctx->cursor_row);
> ret = write_char(ctx, row, ctx->cursor_column, ' ');
> - if(ret == 0)
> + if (ret == 0)
> ctx->cursor_column++;
> }
>
> -static void handle_pac( CCaptionSubContext *ctx, uint8_t hi, uint8_t lo )
> +static void handle_pac(CCaptionSubContext *ctx, uint8_t hi, uint8_t lo)
> {
> static const int8_t row_map[] = {
> 11, -1, 1, 2, 3, 4, 12, 13, 14, 15, 5, 6, 7, 8, 9, 10
> @@ -374,9 +370,9 @@ static void handle_pac( CCaptionSubContext *ctx,
> uint8_t hi, uint8_t lo )
> const int index = ( (hi<<1) & 0x0e) | ( (lo>>5) & 0x01 );
> struct Screen *screen = get_writing_screen(ctx);
> char *row;
> - int indent,i,ret;
> + int indent, i, ret;
>
> - if( row_map[index] <= 0 ) {
> + if (row_map[index] <= 0) {
> av_log(ctx, AV_LOG_DEBUG,"Invalid pac index encountered\n");
> return;
> }
> @@ -389,18 +385,17 @@ static void handle_pac( CCaptionSubContext *ctx,
> uint8_t hi, uint8_t lo )
> ctx->cursor_column = 0;
> indent = pac2_attribs[lo][2];
> row = screen->characters[ctx->cursor_row];
> - for(i = 0;i < indent; i++) {
> + for (i = 0; i < indent; i++) {
> ret = write_char(ctx, row, ctx->cursor_column, ' ');
> - if( ret == 0 )
> + if (ret == 0)
> ctx->cursor_column++;
> }
> -
> }
>
> /**
> * @param pts it is required to set end time
> */
> -static int handle_edm(CCaptionSubContext *ctx,int64_t pts)
> +static int handle_edm(CCaptionSubContext *ctx, int64_t pts)
> {
> int ret = 0;
> struct Screen *screen = ctx->screen + ctx->active_screen;
> @@ -420,12 +415,11 @@ static int handle_eoc(CCaptionSubContext *ctx,
> int64_t pts)
> return ret;
> }
>
> -static void handle_delete_end_of_row( CCaptionSubContext *ctx, char hi,
> char lo)
> +static void handle_delete_end_of_row(CCaptionSubContext *ctx, char hi,
> char lo)
> {
> struct Screen *screen = get_writing_screen(ctx);
> char *row = screen->characters[ctx->cursor_row];
> write_char(ctx, row, ctx->cursor_column, 0);
> -
> }
>
> static void handle_char(CCaptionSubContext *ctx, char hi, char lo,
> int64_t pts)
> @@ -437,12 +431,12 @@ static void handle_char(CCaptionSubContext *ctx,
> char hi, char lo, int64_t pts)
> SET_FLAG(screen->row_used,ctx->cursor_row);
>
> ret = write_char(ctx, row, ctx->cursor_column, hi);
> - if( ret == 0 )
> + if (ret == 0)
> ctx->cursor_column++;
>
> - if(lo) {
> + if (lo) {
> ret = write_char(ctx, row, ctx->cursor_column, lo);
> - if ( ret == 0 )
> + if (ret == 0)
> ctx->cursor_column++;
> }
> write_char(ctx, row, ctx->cursor_column, 0);
> @@ -451,16 +445,16 @@ static void handle_char(CCaptionSubContext *ctx,
> char hi, char lo, int64_t pts)
> ctx->prev_cmd[0] = 0;
> ctx->prev_cmd[1] = 0;
> if (lo)
> - ff_dlog(ctx, "(%c,%c)\n",hi,lo);
> + ff_dlog(ctx, "(%c,%c)\n", hi, lo);
> else
> - ff_dlog(ctx, "(%c)\n",hi);
> + ff_dlog(ctx, "(%c)\n", hi);
> }
>
> static int process_cc608(CCaptionSubContext *ctx, int64_t pts, uint8_t
> hi, uint8_t lo)
> {
> int ret = 0;
> #define COR3(var, with1, with2, with3) ( (var) == (with1) || (var) ==
> (with2) || (var) == (with3) )
> - if ( hi == ctx->prev_cmd[0] && lo == ctx->prev_cmd[1]) {
> + if (hi == ctx->prev_cmd[0] && lo == ctx->prev_cmd[1]) {
> /* ignore redundant command */
> } else if ( (hi == 0x10 && (lo >= 0x40 || lo <= 0x5f)) ||
> ( (hi >= 0x11 && hi <= 0x17) && (lo >= 0x40 && lo <= 0x7f)
> ) ) {
> @@ -511,12 +505,11 @@ static int process_cc608(CCaptionSubContext *ctx,
> int64_t pts, uint8_t hi, uint8
> }
>
> /* set prev command */
> - ctx->prev_cmd[0] = hi;
> - ctx->prev_cmd[1] = lo;
> + ctx->prev_cmd[0] = hi;
> + ctx->prev_cmd[1] = lo;
>
> #undef COR3
> return ret;
> -
> }
>
> static int decode(AVCodecContext *avctx, void *data, int *got_sub,
> AVPacket *avpkt)
> @@ -528,9 +521,9 @@ static int decode(AVCodecContext *avctx, void *data,
> int *got_sub, AVPacket *avp
> int ret = 0;
> int i;
>
> - if ( ctx->pktbuf->size < len) {
> + if (ctx->pktbuf->size < len) {
> ret = av_buffer_realloc(&ctx->pktbuf, len);
> - if(ret < 0) {
> + if (ret < 0) {
> av_log(ctx, AV_LOG_WARNING, "Insufficient Memory of %d
> truncated to %d\n",len, ctx->pktbuf->size);
> len = ctx->pktbuf->size;
> ret = 0;
> @@ -539,17 +532,16 @@ static int decode(AVCodecContext *avctx, void *data,
> int *got_sub, AVPacket *avp
> memcpy(ctx->pktbuf->data, avpkt->data, len);
> bptr = ctx->pktbuf->data;
>
> -
> for (i = 0; i < len; i += 3) {
> uint8_t cc_type = *(bptr + i) & 3;
> - if (validate_cc_data_pair( bptr + i) )
> + if (validate_cc_data_pair(bptr + i))
> continue;
> /* ignoring data field 1 */
> if(cc_type == 1)
> continue;
> else
> process_cc608(ctx, avpkt->pts, *(bptr + i + 1) & 0x7f, *(bptr
> + i + 2) & 0x7f);
> - if(ctx->screen_changed && *ctx->buffer.str)
> + if (ctx->screen_changed && *ctx->buffer.str)
> {
> int start_time = av_rescale_q(ctx->start_time,
> avctx->time_base, (AVRational){ 1, 100 });
> int end_time = av_rescale_q(ctx->end_time, avctx->time_base,
> (AVRational){ 1, 100 });
> --
> 2.5.3
>
>
LGTM
--
with regards
Anshul
:)
More information about the ffmpeg-devel
mailing list