Opening myLeoSettings.leo can clutter the console unworthwhile messages

Bug #951921 reported by SegundoBob
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
leo-editor
Fix Released
Low
Edward K. Ream

Bug Description

Example of a cluttered console:

16:12:19 ~$ leomy

** isPython3: False
Leo 4.9.1 devel, build 5113, 2012-03-10 15:43:07
Python 2.7.2, qt version 4.7.3
linux2
reading settings in /home/bob/bzr/LeoLatest/leo/config/leoSettings.leo
reading settings in /home/bob/.leo/myLeoSettings.leo
reading settings in /home/bob/.leo/workbook.leo
reading settings in /home/bob/.leo/myLeoSettings.leo
reading: /home/bob/.leo/myLeoSettings.leo
removing previous definition for <KeyStroke: Alt+equal> in myLeoSettings.leo
removing previous definition for <KeyStroke: Alt+equal> in myLeoSettings.leo
previous: @command-goto-last-child new: @command-goto-last-child
previous: @command-goto-last-child new: @command-goto-last-child
removing previous definition for <KeyStroke: Alt+equal> in myLeoSettings.leo
removing previous definition for <KeyStroke: Alt+equal> in myLeoSettings.leo
previous: @command-goto-last-child new: @command-goto-last-child
previous: @command-goto-last-child new: @command-goto-last-child
removing previous definition for <KeyStroke: Alt+equal> in myLeoSettings.leo
removing previous definition for <KeyStroke: Alt+equal> in myLeoSettings.leo
previous: @command-goto-last-child new: @command-goto-last-child
previous: @command-goto-last-child new: @command-goto-last-child
--- End Console contents ---

All the messages above beginning with "removing previous..." occur when myLeoSettings.leo is opened.
The messages occur only when plugin mod_scripting.py is enabled.
I have 4 @command nodes without @key. These do not cause messages to the console.
I have 1 @command node with an @key in the headline:
@command goto-last-child @key=alt-=
This one node seems to cause all 12 of the messages above.
If these messages are not useful to the user, they should not be output.
This is a minor problem.

Test System:
Ubuntu 11.10 with Fluxbox window manager
Leo Log Window
Leo 4.9.1 devel, build 5113, 2012-03-10 15:43:07
Python 2.7.2, qt version 4.7.3
linux2
setting leoID from os.getenv('USER'): 'bob'
load dir: /home/bob/bzr/LeoLatest/leo/core
global config dir: /home/bob/bzr/LeoLatest/leo/config
home dir: /home/bob
reading settings in /home/bob/bzr/LeoLatest/leo/config/leoSettings.leo
reading settings in /home/bob/.leo/myLeoSettings.leo
reading settings in /home/bob/.leo/workbook.leo
reading: /home/bob/.leo/workbook.leo

Revision history for this message
Edward K. Ream (edreamleo) wrote : Re: [Bug 951921] [NEW] Opening myLeoSettings.leo can clutter the console unworthwhile messages

On Sat, Mar 10, 2012 at 6:30 PM, SegundoBob <email address hidden> wrote:

> Example of a cluttered console:

Thanks for this report. These messages are new, and I was wondering
if they would cause problems. Now I know they do.

This might be a bit tricky to fix. We want to alert the user to true
conflicts *within the same set of settings*, but clearly the way I
chose doesn't work.

As I write this I see there might be an easier way earlier in the
settings-creation logic. I'll attempt a fix today.

Thanks again for this report.

Edward

Changed in leo-editor:
milestone: none → 4.10-b1
assignee: nobody → Edward K. Ream (edreamleo)
importance: Undecided → Low
Revision history for this message
Edward K. Ream (edreamleo) wrote :

Fixed on the trunk at rev 5118.

This was an excellent fix. It uses the "inverted" key dictionary created by LM.mergeShortcutsDicts.

There could be minor problems with the tests: some "panes" are really aggregates. However, this is a nit, and such problems have always existed.

Changed in leo-editor:
status: New → Fix Released
Revision history for this message
SegundoBob (bhossley) wrote :

Agreed. My console is now clutter free.

Test System:
Ubuntu 11.10 with Fluxbox window manager
Leo Log Window
Leo 4.9.1 devel, build 5121, 2012-03-11 15:35:03
Python 2.7.2, qt version 4.7.3
linux2
setting leoID from os.getenv('USER'): 'bob'
load dir: /home/bob/bzr/LeoLatest/leo/core
global config dir: /home/bob/bzr/LeoLatest/leo/config
home dir: /home/bob
reading settings in /home/bob/bzr/LeoLatest/leo/config/leoSettings.leo
reading settings in /home/bob/.leo/myLeoSettings.leo
reading settings in /home/bob/.leo/workbook.leo
reading: /home/bob/.leo/workbook.leo

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.