Removal of separators is buggy

[GoogleCodeExporter]

1. Remove separator after Options, or after New Folder
2. Note that a new separator marked as a hidden system command and greyed
out now appears near the bottom of the list.
3. Remove the new separator. Fails.
4. Uncheck the 'separator' checkbox.
5. Note that a blank line appears in the list, and greyed out icon on toolbar.
6. Remove the blank line.
7. New separator is finally gone.

If I can't remove that separator the remove box should be greyed out.

1. Add a new separator.
2. Move it up one place so it comes between Rename and Notepad.
3. Edit the separator and uncheck the 'separator' checkbox.
4. OK --> Explorer crash
5. Each time explorer starts it crashes again. Only recovery is to locate
the commands.ini file and remove the rogue entry.

Should not be able to edit a separator. But in any case the ini file reader
needs to be more robust in case the content is invalid.

Using 1.44 on XP-SP3

Original issue reported on code.google.com by simon.to...@gmail.com on 3 Jun 2008 at 11:29

[GoogleCodeExporter]

starting

Original comment by tortoisesvn on 7 Jul 2008 at 5:28

• Changed state: Started

[GoogleCodeExporter]

Fixed in r149.

Original comment by tortoisesvn on 7 Jul 2008 at 7:46

• Changed state: Fixed