[Bf-committers] [Bf-blender-cvs] SVN commit: /data/svn/bf-blender [33936] trunk/blender/source/blender/ blenkernel/intern: Silencing some compiler warnings (gcc)

[Campbell Barton]

What GCC version are you using?

I dont get any warnings with either of these files:
gcc version 4.5.2

warning flags:
 -Wunused-value -Wclobbered  -Wold-style-declaration -Woverride-init
-Wuninitialized -Winit-self -Wmissing-include-dirs
-Wignored-qualifiers -Woverride-init -Werror -Wall -Wcast-align
-Werror=declaration-after-statement
-Werror=implicit-function-declaration -Werror=return-type
-Werror=strict-prototypes -Wno-char-subscripts -Wno-unknown-pragmas
-Wpointer-arith -Wunused-parameter -Wwrite-strings



On Wed, Dec 29, 2010 at 12:27 PM, Joshua Leung <aligorith at gmail.com> wrote:
> Revision: 33936
>          http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=33936
> Author:   aligorith
> Date:     2010-12-29 13:27:03 +0100 (Wed, 29 Dec 2010)
>
> Log Message:
> -----------
> Silencing some compiler warnings (gcc)
>
> Modified Paths:
> --------------
>    trunk/blender/source/blender/blenkernel/intern/particle.c
>    trunk/blender/source/blender/blenkernel/intern/sequencer.c
>
> Modified: trunk/blender/source/blender/blenkernel/intern/particle.c
> ===================================================================
> --- trunk/blender/source/blender/blenkernel/intern/particle.c   2010-12-29 12:18:59 UTC (rev 33935)
> +++ trunk/blender/source/blender/blenkernel/intern/particle.c   2010-12-29 12:27:03 UTC (rev 33936)
> @@ -1158,7 +1158,7 @@
>                pind->dietime = (key + pa->totkey - 1)->time;
>        }
>        else if(pind->cache) {
> -               float start, end;
> +               float start=0.0f, end=0.0f;
>                get_pointcache_keys_for_time(ob, pind->cache, &pind->pm, -1, 0.0f, NULL, NULL);
>                pind->birthtime = pa ? pa->time : pind->cache->startframe;
>                pind->dietime = pa ? pa->dietime : pind->cache->endframe;
> @@ -2807,7 +2807,7 @@
>        float birthtime = 0.0, dietime = 0.0;
>        float t, time = 0.0, dfra = 1.0, frs_sec = sim->scene->r.frs_sec;
>        float col[4] = {0.5f, 0.5f, 0.5f, 1.0f};
> -       float prev_tangent[3], hairmat[4][4];
> +       float prev_tangent[3] = {0.0f, 0.0f, 0.0f}, hairmat[4][4];
>        float rotmat[3][3];
>        int k;
>        int steps = (int)pow(2.0, (double)(psys->renderdata ? part->ren_step : part->draw_step));
> @@ -3025,7 +3025,7 @@
>
>        float birthtime = 0.0, dietime = 0.0;
>        float t, time = 0.0, keytime = 0.0, frs_sec;
> -       float hairmat[4][4], rotmat[3][3], prev_tangent[3];
> +       float hairmat[4][4], rotmat[3][3], prev_tangent[3] = {0.0f, 0.0f, 0.0f};
>        int k, i;
>        int steps = (int)pow(2.0, (double)pset->draw_step);
>        int totpart = edit->totpoint, recalc_set=0;
>
> Modified: trunk/blender/source/blender/blenkernel/intern/sequencer.c
> ===================================================================
> --- trunk/blender/source/blender/blenkernel/intern/sequencer.c  2010-12-29 12:18:59 UTC (rev 33935)
> +++ trunk/blender/source/blender/blenkernel/intern/sequencer.c  2010-12-29 12:27:03 UTC (rev 33936)
> @@ -642,7 +642,7 @@
>  void reload_sequence_new_file(Scene *scene, Sequence * seq, int lock_range)
>  {
>        char str[FILE_MAXDIR+FILE_MAXFILE];
> -       int prev_startdisp, prev_enddisp;
> +       int prev_startdisp=0, prev_enddisp=0;
>        /* note: dont rename the strip, will break animation curves */
>
>        if (ELEM5(seq->type, SEQ_MOVIE, SEQ_IMAGE, SEQ_SOUND, SEQ_SCENE, SEQ_META)==0) {
>
>
> _______________________________________________
> Bf-blender-cvs mailing list
> Bf-blender-cvs at blender.org
> http://lists.blender.org/mailman/listinfo/bf-blender-cvs
>



-- 
- Campbell