diff options
author | Campbell Barton <ideasman42@gmail.com> | 2013-04-13 18:57:08 +0400 |
---|---|---|
committer | Campbell Barton <ideasman42@gmail.com> | 2013-04-13 18:57:08 +0400 |
commit | 2baa59e36d4994a3dd58c254964a0caf5bc081a3 (patch) | |
tree | 666c2f5ccd12c0cce1375098e1d64205c48f3198 /source | |
parent | 53bb59f209bae874d7d73c5fb882452c1fee9cd8 (diff) |
code cleanup: remove unused string formatting in bli_adddirstrings(), also remove unused initializations and comment unused vars.
Diffstat (limited to 'source')
-rw-r--r-- | source/blender/blenkernel/intern/context.c | 2 | ||||
-rw-r--r-- | source/blender/blenlib/intern/storage.c | 42 | ||||
-rw-r--r-- | source/blender/blenloader/intern/readfile.c | 4 | ||||
-rw-r--r-- | source/blender/editors/space_file/file_draw.c | 1 | ||||
-rw-r--r-- | source/blender/editors/space_view3d/view3d_edit.c | 2 | ||||
-rw-r--r-- | source/blender/editors/space_view3d/view3d_ruler.c | 2 |
6 files changed, 17 insertions, 36 deletions
diff --git a/source/blender/blenkernel/intern/context.c b/source/blender/blenkernel/intern/context.c index d899990a66a..428e9d70b96 100644 --- a/source/blender/blenkernel/intern/context.c +++ b/source/blender/blenkernel/intern/context.c @@ -85,9 +85,11 @@ struct bContext { } data; /* data evaluation */ +#if 0 struct { int render; } eval; +#endif }; /* context */ diff --git a/source/blender/blenlib/intern/storage.c b/source/blender/blenlib/intern/storage.c index 24d4f7cdfe3..e5c0d91fd29 100644 --- a/source/blender/blenlib/intern/storage.c +++ b/source/blender/blenlib/intern/storage.c @@ -311,16 +311,14 @@ static void bli_builddir(struct BuildDirCtx *dir_ctx, const char *dirname) */ static void bli_adddirstrings(struct BuildDirCtx *dir_ctx) { - char datum[100]; -// char buf[512]; // UNUSED - char size[250]; const char *types[8] = {"---", "--x", "-w-", "-wx", "r--", "r-x", "rw-", "rwx"}; /* symbolic display, indexed by mode field value */ - int num, mode; + int num; #ifdef WIN32 __int64 st_size; #else off_t st_size; + int mode; #endif struct direntry *file; @@ -328,8 +326,10 @@ static void bli_adddirstrings(struct BuildDirCtx *dir_ctx) time_t zero = 0; for (num = 0, file = dir_ctx->files; num < dir_ctx->nrfiles; num++, file++) { + + + /* Mode */ #ifdef WIN32 - mode = 0; BLI_strncpy(file->mode1, types[0], sizeof(file->mode1)); BLI_strncpy(file->mode2, types[0], sizeof(file->mode2)); BLI_strncpy(file->mode3, types[0], sizeof(file->mode3)); @@ -355,6 +355,8 @@ static void bli_adddirstrings(struct BuildDirCtx *dir_ctx) } #endif + + /* User */ #ifdef WIN32 strcpy(file->owner, "user"); #else @@ -370,12 +372,16 @@ static void bli_adddirstrings(struct BuildDirCtx *dir_ctx) } #endif + + /* Time */ tm = localtime(&file->s.st_mtime); // prevent impossible dates in windows if (tm == NULL) tm = localtime(&zero); strftime(file->time, sizeof(file->time), "%H:%M", tm); strftime(file->date, sizeof(file->date), "%d-%b-%y", tm); + + /* Size */ /* * Seems st_size is signed 32-bit value in *nix and Windows. This * will buy us some time until files get bigger than 4GB or until @@ -398,32 +404,6 @@ static void bli_adddirstrings(struct BuildDirCtx *dir_ctx) else { BLI_snprintf(file->size, sizeof(file->size), "%d B", (int)st_size); } - - strftime(datum, 32, "%d-%b-%y %H:%M", tm); /* XXX, is this used? - campbell */ - - if (st_size < 1000) { - BLI_snprintf(size, sizeof(size), "%10d", - (int) st_size); - } - else if (st_size < 1000 * 1000) { - BLI_snprintf(size, sizeof(size), "%6d %03d", - (int) (st_size / 1000), (int) (st_size % 1000)); - } - else if (st_size < 100 * 1000 * 1000) { - BLI_snprintf(size, sizeof(size), "%2d %03d %03d", - (int) (st_size / (1000 * 1000)), (int) ((st_size / 1000) % 1000), (int) (st_size % 1000)); - } - else { - /* XXX, whats going on here?. 2x calls - campbell */ - BLI_snprintf(size, sizeof(size), "> %4.1f M", (double) (st_size / (1024.0 * 1024.0))); - BLI_snprintf(size, sizeof(size), "%10d", (int) st_size); - } - -#if 0 - BLI_snprintf(buf, sizeof(buf), "%s %s %s %7s %s %s %10s %s", - file->mode1, file->mode2, file->mode3, file->owner, - file->date, file->time, size, file->relname); -#endif } } diff --git a/source/blender/blenloader/intern/readfile.c b/source/blender/blenloader/intern/readfile.c index 98c37e85e53..2c821229fb4 100644 --- a/source/blender/blenloader/intern/readfile.c +++ b/source/blender/blenloader/intern/readfile.c @@ -7365,7 +7365,6 @@ static void do_versions_nodetree_multi_file_output_format_2_62_1(Scene *sce, bNo bNodeSocket *old_image = BLI_findlink(&node->inputs, 0); bNodeSocket *old_z = BLI_findlink(&node->inputs, 1); bNodeSocket *sock; - char basepath[FILE_MAXDIR]; char filename[FILE_MAXFILE]; /* ugly, need to remove the old inputs list to avoid bad pointer checks when adding new sockets. @@ -7377,6 +7376,8 @@ static void do_versions_nodetree_multi_file_output_format_2_62_1(Scene *sce, bNo /* looks like storage data can be messed up somehow, stupid check here */ if (old_data) { + char basepath[FILE_MAXDIR]; + /* split off filename from the old path, to be used as socket sub-path */ BLI_split_dirfile(old_data->name, basepath, filename, sizeof(basepath), sizeof(filename)); @@ -7384,7 +7385,6 @@ static void do_versions_nodetree_multi_file_output_format_2_62_1(Scene *sce, bNo nimf->format = old_data->im_format; } else { - basepath[0] = '\0'; BLI_strncpy(filename, old_image->name, sizeof(filename)); } diff --git a/source/blender/editors/space_file/file_draw.c b/source/blender/editors/space_file/file_draw.c index 4a0665d663e..e144fdd290e 100644 --- a/source/blender/editors/space_file/file_draw.c +++ b/source/blender/editors/space_file/file_draw.c @@ -389,7 +389,6 @@ static void file_draw_preview(uiBlock *block, struct direntry *file, int sx, int uiButSetDragImage(but, file->path, get_file_icon(file), imb, scale); glDisable(GL_BLEND); - imb = NULL; } } diff --git a/source/blender/editors/space_view3d/view3d_edit.c b/source/blender/editors/space_view3d/view3d_edit.c index 522fd8b777c..17d467e593e 100644 --- a/source/blender/editors/space_view3d/view3d_edit.c +++ b/source/blender/editors/space_view3d/view3d_edit.c @@ -3969,7 +3969,7 @@ static float view_autodist_depth_margin(ARegion *ar, const int mval[2], int marg bool ED_view3d_autodist(Scene *scene, ARegion *ar, View3D *v3d, const int mval[2], float mouse_worldloc[3], bool alphaoverride) { bglMats mats; /* ZBuffer depth vars */ - float depth_close = FLT_MAX; + float depth_close; double cent[2], p[3]; /* Get Z Depths, needed for perspective, nice for ortho */ diff --git a/source/blender/editors/space_view3d/view3d_ruler.c b/source/blender/editors/space_view3d/view3d_ruler.c index 15d5aee9020..c71617bab6a 100644 --- a/source/blender/editors/space_view3d/view3d_ruler.c +++ b/source/blender/editors/space_view3d/view3d_ruler.c @@ -95,7 +95,7 @@ static bool ED_view3d_snap_ray(bContext *C, float r_co[3], { float dist_px = 12; /* snap dist */ float r_no_dummy[3]; - bool ret = false; + bool ret; Scene *scene = CTX_data_scene(C); View3D *v3d = CTX_wm_view3d(C); |