On Wed, Sep 11, 2013 at 12:23:09PM +0200, Alexandra Khirnova wrote: > --- > libavformat/matroskaenc.c | 20 +++++++++----------- > 1 file changed, 9 insertions(+), 11 deletions(-)
LGTM, and even better, LGTK (Looks Good To Kostya) as well. > --- a/libavformat/matroskaenc.c > +++ b/libavformat/matroskaenc.c > @@ -294,21 +294,20 @@ static mkv_seekhead * mkv_start_seekhead(AVIOContext > *pb, int64_t segment_offset > > - entries[seekhead->num_entries ].elementid = elementid; > - entries[seekhead->num_entries++].segmentpos = filepos - > seekhead->segment_offset; > + seekhead->entries[seekhead->num_entries ].elementid = elementid; > + seekhead->entries[seekhead->num_entries++].segmentpos = filepos - > seekhead->segment_offset; You could drop the spaces inside [] and align the = while you're at it. > @@ -378,21 +377,20 @@ static mkv_cues * mkv_start_cues(int64_t segment_offset) > > - entries[cues->num_entries ].pts = ts; > - entries[cues->num_entries ].tracknum = stream + 1; > - entries[cues->num_entries++].cluster_pos = cluster_pos - > cues->segment_offset; > + cues->entries[cues->num_entries ].pts = ts; > + cues->entries[cues->num_entries ].tracknum = stream + 1; > + cues->entries[cues->num_entries++].cluster_pos = cluster_pos - > cues->segment_offset; same Diego _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel