diff options
author | David Robillard <d@drobilla.net> | 2008-09-19 18:16:50 +0000 |
---|---|---|
committer | David Robillard <d@drobilla.net> | 2008-09-19 18:16:50 +0000 |
commit | f201dc3e3c06f888f097ed6d378a597e99e244fd (patch) | |
tree | 01502833cdf1e36eb164e2114574871387f0150b /gtk2_ardour | |
parent | 48a8363080c0b9c79ace0b0b70943ed4ce074d26 (diff) |
Fix a bunch of stupid warnings.
Q: What's a lot harder than initializing a variable?
A: Fixing future non-deterministic bugs that would have been deterministic if you initialized the damn variable :)
git-svn-id: svn://localhost/ardour2/branches/2.0-ongoing@3767 d708f5d6-7413-0410-9779-e7cbd77b26cf
Diffstat (limited to 'gtk2_ardour')
-rw-r--r-- | gtk2_ardour/editor.cc | 4 | ||||
-rw-r--r-- | gtk2_ardour/editor_mouse.cc | 2 | ||||
-rw-r--r-- | gtk2_ardour/editor_ops.cc | 9 |
3 files changed, 6 insertions, 9 deletions
diff --git a/gtk2_ardour/editor.cc b/gtk2_ardour/editor.cc index d97867bb69..b841efb542 100644 --- a/gtk2_ardour/editor.cc +++ b/gtk2_ardour/editor.cc @@ -2278,7 +2278,7 @@ Editor::set_edit_point_preference (EditPoint ep, bool force) break; } - const char* action; + const char* action=NULL; switch (_edit_point) { case EditAtPlayhead: @@ -4382,7 +4382,7 @@ Editor::idle_visual_changer () if (p & VisualChange::TimeOrigin) { - nframes64_t csf, cef; + nframes64_t csf=0, cef=0; nframes64_t current_time_origin = (nframes64_t) floor (horizontal_adjustment.get_value() * frames_per_unit); if (session) { diff --git a/gtk2_ardour/editor_mouse.cc b/gtk2_ardour/editor_mouse.cc index 6cef53a685..ab1f4792eb 100644 --- a/gtk2_ardour/editor_mouse.cc +++ b/gtk2_ardour/editor_mouse.cc @@ -2344,7 +2344,7 @@ Editor::start_marker_grab (ArdourCanvas::Item* item, GdkEvent* event) void Editor::marker_drag_motion_callback (ArdourCanvas::Item* item, GdkEvent* event) { - nframes64_t f_delta; + nframes64_t f_delta = 0; nframes64_t newframe; bool is_start; bool move_both = false; diff --git a/gtk2_ardour/editor_ops.cc b/gtk2_ardour/editor_ops.cc index e3d2f88a20..139e07a366 100644 --- a/gtk2_ardour/editor_ops.cc +++ b/gtk2_ardour/editor_ops.cc @@ -3257,9 +3257,9 @@ Editor::align_selection_relative (RegionPoint point, nframes64_t position, const return; } - nframes64_t distance; + nframes64_t distance = 0; nframes64_t pos = 0; - int dir; + int dir = 1; list<RegionView*> sorted; rs.by_position (sorted); @@ -3271,7 +3271,6 @@ Editor::align_selection_relative (RegionPoint point, nframes64_t position, const pos = position; if (position > r->position()) { distance = position - r->position(); - dir = 1; } else { distance = r->position() - position; dir = -1; @@ -3282,7 +3281,6 @@ Editor::align_selection_relative (RegionPoint point, nframes64_t position, const if (position > r->last_frame()) { distance = position - r->last_frame(); pos = r->position() + distance; - dir = 1; } else { distance = r->last_frame() - position; pos = r->position() - distance; @@ -3294,7 +3292,6 @@ Editor::align_selection_relative (RegionPoint point, nframes64_t position, const pos = r->adjust_to_sync (position); if (pos > r->position()) { distance = pos - r->position(); - dir = 1; } else { distance = r->position() - pos; dir = -1; @@ -4740,7 +4737,7 @@ Editor::toggle_fade_active (bool in) const char* cmd = (in ? _("toggle fade in active") : _("toggle fade out active")); bool have_switch = false; - bool yn; + bool yn = false; begin_reversible_command (cmd); |