diff mbox

[FFmpeg-devel] avcodec/hevcdec: make ff_hevc_frame_nb_refs take a const pointer

Message ID 20180510102301.10028-1-timo@rothenpieler.org
State Accepted
Commit 46c1ee19171c4704ad7dec65b1c716a727238486
Headers show

Commit Message

Timo Rothenpieler May 10, 2018, 10:23 a.m. UTC 
---
 libavcodec/hevc_refs.c | 4 ++--
 libavcodec/hevcdec.h   | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

Comments

Philip Langdale May 10, 2018, 2:02 p.m. UTC | #1 
On Thu, 10 May 2018 12:23:01 +0200
Timo Rothenpieler <timo@rothenpieler.org> wrote:

> ---
>  libavcodec/hevc_refs.c | 4 ++--
>  libavcodec/hevcdec.h   | 2 +-
>  2 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/libavcodec/hevc_refs.c b/libavcodec/hevc_refs.c
> index 68c730edcc..7cf3a55725 100644
> --- a/libavcodec/hevc_refs.c
> +++ b/libavcodec/hevc_refs.c
> @@ -508,12 +508,12 @@ fail:
>      return ret;
>  }
>  
> -int ff_hevc_frame_nb_refs(HEVCContext *s)
> +int ff_hevc_frame_nb_refs(const HEVCContext *s)
>  {
>      int ret = 0;
>      int i;
>      const ShortTermRPS *rps = s->sh.short_term_rps;
> -    LongTermRPS *long_rps   = &s->sh.long_term_rps;
> +    const LongTermRPS *long_rps = &s->sh.long_term_rps;
>  
>      if (rps) {
>          for (i = 0; i < rps->num_negative_pics; i++)
> diff --git a/libavcodec/hevcdec.h b/libavcodec/hevcdec.h
> index b311edc8ae..f0f588f2b8 100644
> --- a/libavcodec/hevcdec.h
> +++ b/libavcodec/hevcdec.h
> @@ -544,7 +544,7 @@ int ff_hevc_res_scale_sign_flag(HEVCContext *s,
> int idx); /**
>   * Get the number of candidate references for the current frame.
>   */
> -int ff_hevc_frame_nb_refs(HEVCContext *s);
> +int ff_hevc_frame_nb_refs(const HEVCContext *s);
>  
>  int ff_hevc_set_new_ref(HEVCContext *s, AVFrame **frame, int poc);
>  

Looks good. Thanks for doing this.

--phil
Timo Rothenpieler May 10, 2018, 4:58 p.m. UTC | #2 
applied and backported to 4.0
diff mbox

Patch

diff --git a/libavcodec/hevc_refs.c b/libavcodec/hevc_refs.c
index 68c730edcc..7cf3a55725 100644
--- a/libavcodec/hevc_refs.c
+++ b/libavcodec/hevc_refs.c
@@ -508,12 +508,12 @@  fail:
     return ret;
 }
 
-int ff_hevc_frame_nb_refs(HEVCContext *s)
+int ff_hevc_frame_nb_refs(const HEVCContext *s)
 {
     int ret = 0;
     int i;
     const ShortTermRPS *rps = s->sh.short_term_rps;
-    LongTermRPS *long_rps   = &s->sh.long_term_rps;
+    const LongTermRPS *long_rps = &s->sh.long_term_rps;
 
     if (rps) {
         for (i = 0; i < rps->num_negative_pics; i++)
diff --git a/libavcodec/hevcdec.h b/libavcodec/hevcdec.h
index b311edc8ae..f0f588f2b8 100644
--- a/libavcodec/hevcdec.h
+++ b/libavcodec/hevcdec.h
@@ -544,7 +544,7 @@  int ff_hevc_res_scale_sign_flag(HEVCContext *s, int idx);
 /**
  * Get the number of candidate references for the current frame.
  */
-int ff_hevc_frame_nb_refs(HEVCContext *s);
+int ff_hevc_frame_nb_refs(const HEVCContext *s);
 
 int ff_hevc_set_new_ref(HEVCContext *s, AVFrame **frame, int poc);