On Tuesday, November 14, 2017 at 10:57:11 AM UTC-6, Edward K. Ream wrote:

For full details, see 417 
> <https://github.com/leo-editor/leo-editor/issues/417>. The issue starts 
> with an updated summary. Devs, please take a look.
>

Recent revs use the reloadSettings pattern in most plugins. The pattern is 
straightforward, but any change to code could be buggy. Please report any 
problem immediately. 

Terry recently suggested simply replacing ivars inited to settings by the 
calls to c.config.get*.  This does indeed make all settings automatically 
reloadable via the reload-settings command.  However, I think the 
reloadSettings pattern is even better because it encapsulates and 
summarizes the settings each class uses.  You could call it a form of 
documentation.

Finally, I am leery about supporting reload-settings in the mod_scripting 
plugin because of concerns about inadvertently enabling @script nodes.  Not 
sure those fears are justified.

Edward

-- 
You received this message because you are subscribed to the Google Groups 
"leo-editor" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/leo-editor.
For more options, visit https://groups.google.com/d/optout.

Reply via email to