shithub: libvpx

Info  •  Files  •  Log  •  Branches

Convert filter kernel choice to lookup

Also removed unused declaration related 6 tap filter

Change-Id: Ic17f516141d885157918505f4204081e4c951fad

--- a/vp9/common/vp9_filter.c

+++ b/vp9/common/vp9_filter.c

@@ -97,19 +97,15 @@

   { 0, -3,  1,  38, 64, 32, -1, -3}

 };

+

+static const subpel_kernel* vp9_filter_kernels[4] = {

+  vp9_sub_pel_filters_8,

+  vp9_sub_pel_filters_8lp,

+  vp9_sub_pel_filters_8s,

+  vp9_bilinear_filters

+};

+

 const subpel_kernel *vp9_get_filter_kernel(INTERPOLATION_TYPE type) {

-  switch (type) {

-    case EIGHTTAP:

-      return vp9_sub_pel_filters_8;

-    case EIGHTTAP_SMOOTH:

-      return vp9_sub_pel_filters_8lp;

-    case EIGHTTAP_SHARP:

-      return vp9_sub_pel_filters_8s;

-    case BILINEAR:

-      return vp9_bilinear_filters;

-    default:

-      assert(!"Invalid interpolation type.");

-      return NULL;

-  }

+  return vp9_filter_kernels[type];

 }

--- a/vp9/common/vp9_filter.h

+++ b/vp9/common/vp9_filter.h

@@ -39,7 +39,6 @@

 const subpel_kernel *vp9_get_filter_kernel(INTERPOLATION_TYPE type);

 extern const subpel_kernel vp9_bilinear_filters[SUBPEL_SHIFTS];

-extern const subpel_kernel vp9_sub_pel_filters_6[SUBPEL_SHIFTS];

 extern const subpel_kernel vp9_sub_pel_filters_8[SUBPEL_SHIFTS];

 extern const subpel_kernel vp9_sub_pel_filters_8s[SUBPEL_SHIFTS];

 extern const subpel_kernel vp9_sub_pel_filters_8lp[SUBPEL_SHIFTS];