labath added a comment.

While we wait for the verdict on the template stuff, it occurred to me that you 
don't need the PerformCleanup bool member as std::function has an "empty" 
state. So the destructor can just do `if(Cleanup) Cleanup()` and the disable 
function can reset the cleanup object. (If you want to make sure the object is 
always created with a non-empty std::function initially, you can add an assert 
in the constructor).


https://reviews.llvm.org/D43662



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to