View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0004820 | ardour | bugs | public | 2012-04-05 00:23 | 2020-04-19 20:16 |
| Reporter | ahurst | Assigned To | cth103 | ||
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | ||
| Product Version | 3.0-beta3 | ||||
| Target Version | 3.0 | ||||
| Summary | 0004820: Definitely unintentional switch statement fall-through, possible UI weirdness | ||||
| Description | Medium impact static analysis bug: Missing break in switch at gtk2_ardour/editor_mouse.cc:874. | ||||
| Additional Information | *** Missing break in switch. This case (value 3) is not terminated by a 'break' statement. 874 case MouseDraw: 875 switch (item_type) { 876 case NoteItem: 877 if (internal_editing()) { 878 /* trim notes if we're in internal edit mode and near the ends of the note */ 879 ArdourCanvas::CanvasNote* cn = dynamic_cast<ArdourCanvas::CanvasNote*> (item); 880 if (cn && cn->big_enough_to_trim() && cn->mouse_near_ends()) { 881 _drags->set (new NoteResizeDrag (this, item), event, current_canvas_cursor); 882 } else { 883 _drags->set (new NoteDrag (this, item), event); 884 } 885 return true; 886 } 887 break; 888 889 default: 890 break; 891 } 892 *** The above case falls through to this one. 893 case MouseObject: | ||||
| Tags | No tags attached. | ||||
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2012-04-05 00:23 | ahurst | New Issue | |
| 2012-04-05 01:06 | cth103 | cost | => 0.00 |
| 2012-04-05 01:06 | cth103 | Target Version | => 3.0 beta4 |
| 2012-04-07 01:57 | cth103 | Note Added: 0013088 | |
| 2012-04-07 01:57 | cth103 | Status | new => resolved |
| 2012-04-07 01:57 | cth103 | Resolution | open => fixed |
| 2012-04-07 01:57 | cth103 | Assigned To | => cth103 |
| 2012-05-23 15:08 | cth103 | Target Version | 3.0 beta4 => 3.0 |
| 2020-04-19 20:16 | system | Note Added: 0023029 | |
| 2020-04-19 20:16 | system | Status | resolved => closed |
