Browse code

avfilter/vf_histogram: use const for variables that are not supposed to change

Signed-off-by: Paul B Mahol <onemda@gmail.com>

Paul B Mahol authored on 2013/10/05 01:48:19
Showing 1 changed files
... ...
@@ -247,9 +247,9 @@ static int filter_frame(AVFilterLink *inlink, AVFrame *in)
247 247
     out->pts = in->pts;
248 248
 
249 249
     for (k = 0; k < h->ncomp; k++) {
250
-        int is_chroma = (k == 1 || k == 2);
251
-        int dst_h = FF_CEIL_RSHIFT(outlink->h, (is_chroma ? h->desc->log2_chroma_h : 0));
252
-        int dst_w = FF_CEIL_RSHIFT(outlink->w, (is_chroma ? h->desc->log2_chroma_w : 0));
250
+        const int is_chroma = (k == 1 || k == 2);
251
+        const int dst_h = FF_CEIL_RSHIFT(outlink->h, (is_chroma ? h->desc->log2_chroma_h : 0));
252
+        const int dst_w = FF_CEIL_RSHIFT(outlink->w, (is_chroma ? h->desc->log2_chroma_w : 0));
253 253
         for (i = 0; i < dst_h ; i++)
254 254
             memset(out->data[h->desc->comp[k].plane] +
255 255
                    i * out->linesize[h->desc->comp[k].plane],
... ...
@@ -260,7 +260,7 @@ static int filter_frame(AVFilterLink *inlink, AVFrame *in)
260 260
     case MODE_LEVELS:
261 261
         for (k = 0; k < h->ncomp; k++) {
262 262
             const int p = h->desc->comp[k].plane;
263
-            int start = k * (h->level_height + h->scale_height) * h->display_mode;
263
+            const int start = k * (h->level_height + h->scale_height) * h->display_mode;
264 264
             double max_hval_log;
265 265
             unsigned max_hval = 0;
266 266
 
... ...
@@ -299,16 +299,16 @@ static int filter_frame(AVFilterLink *inlink, AVFrame *in)
299 299
         break;
300 300
     case MODE_WAVEFORM:
301 301
         for (k = 0; k < h->ncomp; k++) {
302
-            int offset = k * 256 * h->display_mode;
302
+            const int offset = k * 256 * h->display_mode;
303 303
             gen_waveform(h, in, out, k, h->step, offset, h->waveform_mode);
304 304
         }
305 305
         break;
306 306
     case MODE_COLOR:
307 307
         for (i = 0; i < inlink->h; i++) {
308
-            int iw1 = i * in->linesize[1];
309
-            int iw2 = i * in->linesize[2];
308
+            const int iw1 = i * in->linesize[1];
309
+            const int iw2 = i * in->linesize[2];
310 310
             for (j = 0; j < inlink->w; j++) {
311
-                int pos = in->data[1][iw1 + j] * out->linesize[0] + in->data[2][iw2 + j];
311
+                const int pos = in->data[1][iw1 + j] * out->linesize[0] + in->data[2][iw2 + j];
312 312
                 if (out->data[0][pos] < 255)
313 313
                     out->data[0][pos]++;
314 314
             }
... ...
@@ -325,12 +325,12 @@ static int filter_frame(AVFilterLink *inlink, AVFrame *in)
325 325
         break;
326 326
     case MODE_COLOR2:
327 327
         for (i = 0; i < inlink->h; i++) {
328
-            int iw1 = i * in->linesize[1];
329
-            int iw2 = i * in->linesize[2];
328
+            const int iw1 = i * in->linesize[1];
329
+            const int iw2 = i * in->linesize[2];
330 330
             for (j = 0; j < inlink->w; j++) {
331
-                int u = in->data[1][iw1 + j];
332
-                int v = in->data[2][iw2 + j];
333
-                int pos = u * out->linesize[0] + v;
331
+                const int u = in->data[1][iw1 + j];
332
+                const int v = in->data[2][iw2 + j];
333
+                const int pos = u * out->linesize[0] + v;
334 334
                 if (!out->data[0][pos])
335 335
                     out->data[0][pos] = FFABS(128 - u) + FFABS(128 - v);
336 336
                 out->data[1][pos] = u;