diff options
author | Campbell Barton <ideasman42@gmail.com> | 2015-11-05 21:09:14 +0300 |
---|---|---|
committer | Campbell Barton <ideasman42@gmail.com> | 2015-11-05 21:34:05 +0300 |
commit | 7a09d15ade9ba065f4b4cb997d0ebe612b0d129c (patch) | |
tree | d6fb344892160d2f0c6b94c11462159821b38646 /source/blender | |
parent | 1adc196723e058d11f8a9cb7fa4635f0e4d08b40 (diff) |
Cleanup: comments/style
Diffstat (limited to 'source/blender')
-rw-r--r-- | source/blender/blenkernel/intern/softbody.c | 2 | ||||
-rw-r--r-- | source/blender/blenlib/intern/easing.c | 2 | ||||
-rw-r--r-- | source/blender/blenlib/intern/task.c | 6 | ||||
-rw-r--r-- | source/blender/bmesh/intern/bmesh_polygon.c | 1 | ||||
-rw-r--r-- | source/blender/editors/space_file/file_ops.c | 2 | ||||
-rw-r--r-- | source/blender/editors/space_file/filelist.c | 4 | ||||
-rw-r--r-- | source/blender/editors/space_image/image_draw.c | 2 | ||||
-rw-r--r-- | source/blender/editors/space_outliner/outliner_tools.c | 2 | ||||
-rw-r--r-- | source/blender/editors/space_sequencer/sequencer_edit.c | 3 | ||||
-rw-r--r-- | source/blender/editors/transform/transform.c | 2 | ||||
-rw-r--r-- | source/blender/freestyle/intern/scene_graph/SceneHash.cpp | 6 |
11 files changed, 15 insertions, 17 deletions
diff --git a/source/blender/blenkernel/intern/softbody.c b/source/blender/blenkernel/intern/softbody.c index 880a47b3256..9c4074275f0 100644 --- a/source/blender/blenkernel/intern/softbody.c +++ b/source/blender/blenkernel/intern/softbody.c @@ -2229,7 +2229,7 @@ static void softbody_calc_forcesEx(Scene *scene, Object *ob, float forcetime, fl /* bproot= sb->bpoint; */ /* need this for proper spring addressing */ /* UNUSED */ if (do_springcollision || do_aero) - sb_sfesf_threads_run(scene, ob, timenow, sb->totspring, NULL); + sb_sfesf_threads_run(scene, ob, timenow, sb->totspring, NULL); /* after spring scan because it uses Effoctors too */ do_effector= pdInitEffectors(scene, ob, NULL, sb->effector_weights, true); diff --git a/source/blender/blenlib/intern/easing.c b/source/blender/blenlib/intern/easing.c index 90c8528338e..93c3cad904c 100644 --- a/source/blender/blenlib/intern/easing.c +++ b/source/blender/blenlib/intern/easing.c @@ -330,7 +330,7 @@ float BLI_easing_quint_ease_out(float time, float begin, float change, float dur float BLI_easing_quint_ease_in_out(float time, float begin, float change, float duration) { if ((time /= duration / 2) < 1.0f) - return change / 2 * time * time * time * time * time + begin; + return change / 2 * time * time * time * time * time + begin; time -= 2.0f; return change / 2 * (time * time * time * time * time + 2) + begin; } diff --git a/source/blender/blenlib/intern/task.c b/source/blender/blenlib/intern/task.c index a5325aa1cc5..5803448a4c0 100644 --- a/source/blender/blenlib/intern/task.c +++ b/source/blender/blenlib/intern/task.c @@ -140,10 +140,10 @@ static bool task_scheduler_thread_wait_pop(TaskScheduler *scheduler, Task **task /* Assuming we can only have a void queue in 'exit' case here seems logical (we should only be here after * our worker thread has been woken up from a condition_wait(), which only happens after a new task was * added to the queue), but it is wrong. - * Waiting on condition may wake up the thread even if condition is not signaled (spurious wakeups), and some + * Waiting on condition may wake up the thread even if condition is not signaled (spurious wake-ups), and some * race condition may also empty the queue **after** condition has been signaled, but **before** awoken thread * reaches this point... - * See http://stackoverflow.com/questions/8594591/why-does-pthread-cond-wait-have-spurious-wakeups + * See http://stackoverflow.com/questions/8594591 * * So we only abort here if do_exit is set. */ @@ -399,7 +399,7 @@ TaskPool *BLI_task_pool_create(TaskScheduler *scheduler, void *userdata) * \a BLI_task_pool_work_and_wait() on it to be sure it will be processed). * * \note Background pools are non-recursive (that is, you should not create other background pools in tasks assigned - * to a brackground pool, they could end never being executed, since the 'fallback' background thread is already + * to a background pool, they could end never being executed, since the 'fallback' background thread is already * busy with parent task in single-threaded context). */ TaskPool *BLI_task_pool_create_background(TaskScheduler *scheduler, void *userdata) diff --git a/source/blender/bmesh/intern/bmesh_polygon.c b/source/blender/bmesh/intern/bmesh_polygon.c index 56d109fbea1..0465148fd88 100644 --- a/source/blender/bmesh/intern/bmesh_polygon.c +++ b/source/blender/bmesh/intern/bmesh_polygon.c @@ -921,7 +921,6 @@ void BM_face_triangulate( /* loop over calculated triangles and create new geometry */ for (i = 0; i < totfilltri; i++) { - /* the order is reverse, otherwise the normal is flipped */ BMLoop *l_tri[3] = { loops[tris[i][0]], loops[tris[i][1]], diff --git a/source/blender/editors/space_file/file_ops.c b/source/blender/editors/space_file/file_ops.c index feea40443fa..ccd1b7cb029 100644 --- a/source/blender/editors/space_file/file_ops.c +++ b/source/blender/editors/space_file/file_ops.c @@ -1307,7 +1307,7 @@ void file_sfile_filepath_set(SpaceFile *sfile, const char *filepath) BLI_split_dirfile(filepath, sfile->params->dir, sfile->params->file, sizeof(sfile->params->dir), sizeof(sfile->params->file)); } - else{ + else { BLI_split_dir_part(filepath, sfile->params->dir, sizeof(sfile->params->dir)); } } diff --git a/source/blender/editors/space_file/filelist.c b/source/blender/editors/space_file/filelist.c index 0357692fb4f..7f405e8ad16 100644 --- a/source/blender/editors/space_file/filelist.c +++ b/source/blender/editors/space_file/filelist.c @@ -394,7 +394,7 @@ static int compare_direntry_generic(const FileListInternEntry *entry1, const Fil } } else if (entry2->typeflag & FILE_TYPE_DIR) { - return 1; + return 1; } /* make sure "." and ".." are always first */ @@ -1069,7 +1069,7 @@ static void filelist_cache_preview_runf(TaskPool *pool, void *taskdata, int UNUS // printf("%s: %d - %s - %p\n", __func__, preview->index, preview->path, preview->img); BLI_assert(preview->flags & (FILE_TYPE_IMAGE | FILE_TYPE_MOVIE | FILE_TYPE_FTFONT | - FILE_TYPE_BLENDER | FILE_TYPE_BLENDER_BACKUP | FILE_TYPE_BLENDERLIB)); + FILE_TYPE_BLENDER | FILE_TYPE_BLENDER_BACKUP | FILE_TYPE_BLENDERLIB)); if (preview->flags & FILE_TYPE_IMAGE) { source = THB_SOURCE_IMAGE; diff --git a/source/blender/editors/space_image/image_draw.c b/source/blender/editors/space_image/image_draw.c index 81711df355c..35a53afa550 100644 --- a/source/blender/editors/space_image/image_draw.c +++ b/source/blender/editors/space_image/image_draw.c @@ -622,7 +622,7 @@ static void draw_image_buffer_tiled(SpaceImage *sima, ARegion *ar, Scene *scene, } else { glaDrawPixelsSafe(x, y, dx, dy, dx, GL_LUMINANCE, GL_UNSIGNED_INT, - (unsigned char*)rect + channel_offset); + (unsigned char *)rect + channel_offset); } } } diff --git a/source/blender/editors/space_outliner/outliner_tools.c b/source/blender/editors/space_outliner/outliner_tools.c index 253430a4b13..a209681e18c 100644 --- a/source/blender/editors/space_outliner/outliner_tools.c +++ b/source/blender/editors/space_outliner/outliner_tools.c @@ -800,7 +800,7 @@ static Base *outline_delete_hierarchy(bContext *C, Scene *scene, Base *base) Object *parent; if (!base) { - return NULL; + return NULL; } for (child_base = scene->base.first; child_base; child_base = base_next) { diff --git a/source/blender/editors/space_sequencer/sequencer_edit.c b/source/blender/editors/space_sequencer/sequencer_edit.c index 8025994d16c..9d8db67b3ee 100644 --- a/source/blender/editors/space_sequencer/sequencer_edit.c +++ b/source/blender/editors/space_sequencer/sequencer_edit.c @@ -3858,8 +3858,7 @@ static int sequencer_export_subtitles_invoke(bContext *C, wmOperator *op, const static int sequencer_export_subtitles_exec(bContext *C, wmOperator *op) { Scene *scene = CTX_data_scene(C); - Sequence *seq = BKE_sequencer_active_get(scene); - Sequence *seq_next; + Sequence *seq, *seq_next; Editing *ed = BKE_sequencer_editing_get(scene, false); ListBase text_seq = {0}; int iter = 0; diff --git a/source/blender/editors/transform/transform.c b/source/blender/editors/transform/transform.c index b79c550241b..49634e558a1 100644 --- a/source/blender/editors/transform/transform.c +++ b/source/blender/editors/transform/transform.c @@ -5574,7 +5574,7 @@ static void slide_origdata_interp_data_vert( float (*faces_center)[3] = BLI_array_alloca(faces_center, l_num); BMLoop *l; - BM_ITER_ELEM_INDEX (l, &liter, sv->v, BM_LOOPS_OF_VERT, j) {; + BM_ITER_ELEM_INDEX (l, &liter, sv->v, BM_LOOPS_OF_VERT, j) { BM_face_calc_center_mean(l->f, faces_center[j]); } diff --git a/source/blender/freestyle/intern/scene_graph/SceneHash.cpp b/source/blender/freestyle/intern/scene_graph/SceneHash.cpp index 9d595c235c2..2af02ab5764 100644 --- a/source/blender/freestyle/intern/scene_graph/SceneHash.cpp +++ b/source/blender/freestyle/intern/scene_graph/SceneHash.cpp @@ -30,9 +30,9 @@ namespace Freestyle { string SceneHash::toString() { - stringstream ss; - ss << hex << _sum; - return ss.str(); + stringstream ss; + ss << hex << _sum; + return ss.str(); } void SceneHash::visitNodeSceneRenderLayer(NodeSceneRenderLayer& node) |