Amend comment to 'delete this'
authorColin Fletcher <colin.m.fletcher@googlemail.com>
Mon, 17 Jun 2013 15:46:18 +0000 (16:46 +0100)
committerColin Fletcher <colin.m.fletcher@googlemail.com>
Fri, 21 Jun 2013 20:33:56 +0000 (21:33 +0100)
Amend comment to 'delete this' to describe why I think it's OK.

gtk2_ardour/sfdb_freesound_mootcher.cc

index e6861cd1f4b842d19ed115cdd18c237f7ce63ad4..8e57d9a0c44bab7b7702047202485b89ef5086a4 100644 (file)
@@ -340,7 +340,7 @@ Mootcher::doneWithMootcher()
        // update the sound info pane if the selection in the list box is still us \r
        sfb->refresh_display(ID, audioFileName);\r
 \r
-       delete this; // XXX is this a good idea?\r
+       delete this; // this should be OK to do as long as Progress and Finished signals are always received in the order in which they are emitted\r
 }\r
 \r
 static void *\r