Skip to content

Add support for FFmpeg's draw_horiz_band feature #508

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
JDarnley wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Add support for FFmpeg's draw_horiz_band feature #508

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
e9ca2a0
avcdec: add initial callback for draw_horiz_band
JDarnley Jul 4, 2018
3b695b6
avcdec: duplicate code to set output properties and flowdef
JDarnley Jul 5, 2018
d898257
avcdec: use draw_horiz_band to output chunks
JDarnley Jul 5, 2018
43c35e0
avcdec: remove logging and dead code in draw_horiz_band
JDarnley Jul 5, 2018
95795d8
avcdec: don't lose duplicated uref in set_output_pic_properties
JDarnley Jul 6, 2018
70032fc
avcdec: don't duplicate things again in draw_horiz_band
JDarnley Jul 6, 2018
0cd3ebd
avcdec: adjust timestamps on draw_horiz_band urefs
JDarnley Jul 6, 2018
35e77c4
avcdec: use the type argument in draw_horiz_band to set progressive a…
JDarnley Jul 9, 2018
e26ecb3
upipe-av: don't output mixed chunks and frames
JDarnley Aug 1, 2018
224e0a2
avcdec: improve setting progressive, tf, and bf attributes in draw_ho…
Aug 2, 2018
2a59092
avcdec: signal separated fields in flow_def draw_horiz_band urefs
JDarnley Aug 6, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions lib/upipe-av/upipe_avcodec_decode.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -297,6 +297,9 @@ static int upipe_avcdec_get_buffer_pic(struct AVCodecContext *context,
return -1;
}

if (frame->interlaced_frame)
UBASE_FATAL(upipe, uref_attr_set_void(flow_def_attr, NULL, UDICT_TYPE_VOID, "sepfields"));
Copy link
Collaborator

@cmassiot cmassiot Sep 3, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use a macro to define a new attribute.


UBASE_FATAL(upipe, uref_pic_flow_set_align(flow_def_attr, align))
UBASE_FATAL(upipe, uref_pic_flow_set_hsize(flow_def_attr, context->width))
UBASE_FATAL(upipe, uref_pic_flow_set_vsize(flow_def_attr, context->height))
Expand Down Expand Up @@ -1099,10 +1102,12 @@ static void draw_horiz_band(AVCodecContext *avctx, const AVFrame *frame,
else if (type == 1) {
uref_pic_delete_progressive(uref);
uref_pic_set_tf(uref);
uref_attr_set_void(uref, NULL, UDICT_TYPE_VOID, "sepfields");
}
else if (type == 2) {
uref_pic_delete_progressive(uref);
uref_pic_set_bf(uref);
uref_attr_set_void(uref, NULL, UDICT_TYPE_VOID, "sepfields");
}
/* Other values are undocumented/invalid. */

Expand Down