CodeSnip Old Code (SVN)

Made further changes to TMainForm.SnippetsChangeHandler:
* Added handler for Snippets' evBeforeDeleteCategory event that clears history list and overview pane.
* Removed clearing of history list from other handlers since this is now taken care of when handling evBeforeDeleteCategory and evBeforeDeleteRoutine.
* Groped together identical event handlers in case statement.

* Added two new event kinds: evBeforeCategoryDelete, called just before a category is deleted (freed) along with a reference to the category being freed and evAfterCategoryDelete which is triggered immediately after the category is freed.
* Modified TSnippets.InternalDeleteCategory to trigger the two new events before and after the category is freed.

Removed unused IsEqual methods from the ISearch interface and all the classes that implement it.

Corrected bug in TRoutineList.IsEqual that was causing it to return opposite of required value in most cases.

* Removed unused nested function from TOverviewFrame.Redisplay.
* Added some explanatory comments.

Fixed bug where TSnipKindGrouping and TAlphaGrouping were always including whole database in groups when a selection had been made via one of the searches. Populate methods of both classes were iterating across Snippets.Routines instead of using the provided SnippetList.

Version numbers updated to 3.4.0.159, ready for next release.

Changed TUserDBBackup.Restore to ensure the file stream used to read the backup file is closed when the file has been read. This was causing the file to be locked open, preventing subsequent write accesses.

* Added two new event kinds: evBeforeRoutineDelete, called just before a snippet is deleted (freed) along with a reference to the snippet being freed and evAfterRoutineDelete which is triggered immediately after the snippet is freed.
* Modified TSnippets.InternalDeleteRoutine to trigger the two new events before and after the snippet is freed.

Added handler for Snippets' evBeforeRoutineDelete event to TMainForm.SnippetsChangeHandler that clears history list and overview pane. This is to try to overcome the "invalid cast" error due to dangling references to deleted routines. This event is called before the actual snippet object is freed, so references to the snippet are valid here.