Author terry.reedy
Recipients Saimadhav.Heblikar, jesstess, taleinat, terry.reedy
Date 2014-08-06.05:41:55
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1407303716.51.0.826635051746.issue22083@psf.upfronthosting.co.za>
In-reply-to
Content
The _here methods are the event handlers for the right click context menu options. Set_breakpoint, as a separate method, is only needed in the restore_file_breaks (from breakpoint file) method. I gather that you are thinking of using set/clear_breakpoint when a line number is clicked.

You have elsewhere suggested a different refactoring - replacing set and clear with toggle. That seems plausible as a good idea.  If text will have no breakpoints set after 
   self.text.update() # this enables setting "BREAK" tags to be visible
in restore_file_breaks(), there would be no problem. If not, toggle would need a force_set option. We need to know when the circumstances of when restore_file_breaks is called.

The bell is both undependable and uniformative. If breakpoints are not allowed for unnamed files, the context menu items should be grayed out (if possible, as done, for instance, for 'cut' when there is no selectionj).  If a line numbers is clicked for unnamed, we should display a message box explaining.
History
Date User Action Args
2014-08-06 05:41:56terry.reedysetrecipients: + terry.reedy, taleinat, jesstess, Saimadhav.Heblikar
2014-08-06 05:41:56terry.reedysetmessageid: <1407303716.51.0.826635051746.issue22083@psf.upfronthosting.co.za>
2014-08-06 05:41:56terry.reedylinkissue22083 messages
2014-08-06 05:41:55terry.reedycreate