Re: [FFmpeg-devel] [PATCH V3] lavc/golomb: Fix UE golomb overwrite issue.

2017-06-08 Thread Jun Zhao
On 2017/6/8 18:02, Mark Thompson wrote: > On 08/06/17 01:29, Jun Zhao wrote: >> V3: Clean the code logic base on Michael's review. >> V2: Add Add set_ue_golomb_long() to support 32bits UE golomb and update the >> unit test. >> >> From 4de3e0c30af7bc1901562000eda018a6d6849292 Mon Sep 17 00:00:00

Re: [FFmpeg-devel] [PATCH V3] lavc/golomb: Fix UE golomb overwrite issue.

2017-06-08 Thread Michael Niedermayer
On Thu, Jun 08, 2017 at 01:07:40PM +0100, Mark Thompson wrote: > On 08/06/17 12:35, Michael Niedermayer wrote: > > On Thu, Jun 08, 2017 at 11:02:30AM +0100, Mark Thompson wrote: > >> On 08/06/17 01:29, Jun Zhao wrote: > >>> V3: Clean the code logic base on Michael's review. > >>> V2: Add Add set_ue

Re: [FFmpeg-devel] [PATCH V3] lavc/golomb: Fix UE golomb overwrite issue.

2017-06-08 Thread Mark Thompson
On 08/06/17 12:35, Michael Niedermayer wrote: > On Thu, Jun 08, 2017 at 11:02:30AM +0100, Mark Thompson wrote: >> On 08/06/17 01:29, Jun Zhao wrote: >>> V3: Clean the code logic base on Michael's review. >>> V2: Add Add set_ue_golomb_long() to support 32bits UE golomb and update the >>> unit test.

Re: [FFmpeg-devel] [PATCH V3] lavc/golomb: Fix UE golomb overwrite issue.

2017-06-08 Thread Michael Niedermayer
On Thu, Jun 08, 2017 at 11:02:30AM +0100, Mark Thompson wrote: > On 08/06/17 01:29, Jun Zhao wrote: > > V3: Clean the code logic base on Michael's review. > > V2: Add Add set_ue_golomb_long() to support 32bits UE golomb and update the > > unit test. > > > > From 4de3e0c30af7bc1901562000eda018a6d6

Re: [FFmpeg-devel] [PATCH V3] lavc/golomb: Fix UE golomb overwrite issue.

2017-06-08 Thread Mark Thompson
On 08/06/17 01:29, Jun Zhao wrote: > V3: Clean the code logic base on Michael's review. > V2: Add Add set_ue_golomb_long() to support 32bits UE golomb and update the > unit test. > > From 4de3e0c30af7bc1901562000eda018a6d6849292 Mon Sep 17 00:00:00 2001 > From: Jun Zhao > Date: Fri, 2 Jun 2017 1

[FFmpeg-devel] [PATCH V3] lavc/golomb: Fix UE golomb overwrite issue.

2017-06-07 Thread Jun Zhao
V3: Clean the code logic base on Michael's review. V2: Add Add set_ue_golomb_long() to support 32bits UE golomb and update the unit test. From 4de3e0c30af7bc1901562000eda018a6d6849292 Mon Sep 17 00:00:00 2001 From: Jun Zhao Date: Fri, 2 Jun 2017 15:05:49 +0800 Subject: [PATCH V3] lavc/golomb: Fix