Dropped self assignment when initialising variables in tokenize.c.

The 'int x=x' style statement do not make sense and cause problems with some compilers. Replace it with 'int x=0' to ensure the value have a well known starting point.

Added -Werror=uninitialized -Winit-self to default GCC build to ensure this style of statement do not sneak into the code again.

Fixes #2317 (closed)

lib/tokenize.c

@@ -487,11 +487,11 @@ int oc_enc_tokenize_ac(oc_enc_ctx *_enc,int _pli,ptrdiff_t _fragi,
   zzj=64;
   for(zzi=OC_MINI(_zzi,63);zzi>0;zzi--){
     ogg_uint32_t best_cost;
-    int          best_bits=best_bits;
-    int          best_next=best_next;
-    int          best_token=best_token;
-    int          best_eb=best_eb;
-    int          best_qc=best_qc;
+    int          best_bits=INT_MAX;
+    int          best_next=INT_MAX;
+    int          best_token=INT_MAX;
+    int          best_eb=INT_MAX;
+    int          best_qc=INT_MAX;
     ogg_uint32_t d2;
     int          dq;
     int          qc_m;
@@ -1091,8 +1091,8 @@ void oc_enc_tokenize_dc_frag_list(oc_enc_ctx *_enc,int _pli,
   int                neobs1;
   int                token;
   int                eb;
-  int                token1=token1;
-  int                eb1=eb1;
+  int                token1=OC_DCT_ZRL_TOKEN;
+  int                eb1=INT_MAX;
   /*Return immediately if there are no coded fragments; otherwise we'd flush
      any trailing EOB run into the AC 1 list and never read it back out.*/
   if(_ncoded_fragis<=0)return;
@@ -1328,7 +1328,7 @@ void oc_enc_tokenize_finish(oc_enc_ctx *_enc){
     int       new_eb;
     int       zzj;
     int       plj;
-    ptrdiff_t ti=ti;
+    ptrdiff_t ti=-1;
     int       run_count;
     /*Make sure this coefficient has tokens at all.*/
     if(_enc->ndct_tokens[pli][zzi]<=0)continue;