Reread configuration

Here you can propose new features, make suggestions etc.

Moderators: Hacker, petermad, Stefan2, white

Post Reply
User avatar
MVV
Power Member
Power Member
Posts: 8711
Joined: 2008-08-03, 12:51 UTC
Location: Russian Federation

Reread configuration

Post by *MVV »

Sorry if this already discussed (I can't remember such topics).

We have a menu item Configuration - Save settings.
We need a menu item Configuration - Load settings.

So on Load settings TC will reread all its options from INI files as if it does it after restarting. So, we won't need to restart TC in order to apply IFI file changes.
User avatar
Clo
Moderator
Moderator
Posts: 5731
Joined: 2003-12-02, 19:01 UTC
Location: Bordeaux, France
Contact:

For ages---

Post by *Clo »

2MVV

:) Hello!

• Yes, that was proposed several times for ages (including by myself)… and not implemented.
Support +++, of course !

:mrgreen: KR
Claude
Clo
#31505 Traducteur Français de TC French translator Aide en Français Tutoriels Français English Tutorials
User avatar
Balderstrom
Power Member
Power Member
Posts: 2148
Joined: 2005-10-11, 10:10 UTC

Post by *Balderstrom »

I'm not sure why it even needs to be a manual command. Any half-decent editor can tell when a file it has open has been modified by another process and offers to reread the file from disk.

TC could automatically pop-up a notification Settings file has changed, do you wish to reload configuration?

Support+.
*BLINK* TC9 Added WM_COPYDATA and WM_USER queries for scripting.
User avatar
JohnFredC
Power Member
Power Member
Posts: 886
Joined: 2003-03-14, 13:37 UTC
Location: Sarasota Florida

Post by *JohnFredC »

Support++
Licensed, Mouse-Centric, moving (slowly) toward Touch-centric
User avatar
petermad
Power Member
Power Member
Posts: 16032
Joined: 2003-02-05, 20:24 UTC
Location: Denmark
Contact:

Post by *petermad »

Support ++++
License #524 (1994)
Danish Total Commander Translator
TC 11.51 32+64bit on Win XP 32bit & Win 7, 8.1 & 10 (22H2) 64bit, 'Everything' 1.5.0.1391a
TC 3.60b4 on Android 6, 13, 14
TC Extended Menus | TC Languagebar | TC Dark Help | PHSM-Calendar
Postkutscher
Power Member
Power Member
Posts: 556
Joined: 2006-04-01, 00:11 UTC

Re: For ages---

Post by *Postkutscher »

Clo wrote:• Yes, that was proposed several times for ages (including by myself)… and not implemented.
Support +++, of course !
Exactly.
Support++
User avatar
Clo
Moderator
Moderator
Posts: 5731
Joined: 2003-12-02, 19:01 UTC
Location: Bordeaux, France
Contact:

Simplistic---

Post by *Clo »

2Postkutscher

:) Hello !

• Actually, I've the simplistic following idea :
- I assume that the internal command exists (as number) for the “Apply” buttons of the config. pages,
so we need only this number and a usable common “text” command like i.e. :
cm_ReloadINIs able to do this for all INIs, or maybe several ones more specialized ?
- What do you think ?

:mrgreen: KR
Claude
Clo
#31505 Traducteur Français de TC French translator Aide en Français Tutoriels Français English Tutorials
User avatar
Balderstrom
Power Member
Power Member
Posts: 2148
Joined: 2005-10-11, 10:10 UTC

Post by *Balderstrom »

Why the insistence on a new command? When launched TC should track the TimeStamps on the .ini files (and any possible redirected .ini's). When such files change - and said change is not due to TC writing to the file... then a popup: "Configuration Files have changed, Apply Changes?\n [Apply][Cancel]"
*BLINK* TC9 Added WM_COPYDATA and WM_USER queries for scripting.
User avatar
MVV
Power Member
Power Member
Posts: 8711
Joined: 2008-08-03, 12:51 UTC
Location: Russian Federation

Post by *MVV »

Balderstrom wrote:Why the insistence on a new command? When launched TC should track the TimeStamps on the .ini files (and any possible redirected .ini's). When such files change - and said change is not due to TC writing to the file... then a popup: "Configuration Files have changed, Apply Changes?\n [Apply][Cancel]"
This requires additional thread that will constantly check INI timestamp. I do not think it is a good idea, for me just command like cm_ConfigLoadSettings is better - it doesn't require additional resources.
User avatar
Balderstrom
Power Member
Power Member
Posts: 2148
Joined: 2005-10-11, 10:10 UTC

Post by *Balderstrom »

Really? The only way to edit the Ini file is to open an editor to do so. Which puts TC in the background.
When TC is activated/brought to foreground it would take an insignificant amount of time to check if it's ini files have changed.
*BLINK* TC9 Added WM_COPYDATA and WM_USER queries for scripting.
User avatar
MVV
Power Member
Power Member
Posts: 8711
Joined: 2008-08-03, 12:51 UTC
Location: Russian Federation

Post by *MVV »

Balderstrom wrote:Really? The only way to edit the Ini file is to open an editor to do so. Which puts TC in the background.
When TC is activated/brought to foreground it would take an insignificant amount of time to check if it's ini files have changed.
Hmm, I didn't thought about check on WM_ACTIVATE. Maybe good idea. But I think that menu item would be enough.
User avatar
ghisler(Author)
Site Admin
Site Admin
Posts: 50547
Joined: 2003-02-04, 09:46 UTC
Location: Switzerland
Contact:

Post by *ghisler(Author) »

I tried to add such a command, but unfortunately there are far too many options which can be set, and which have effects on things like plugins, active ftp connections etc. etc., so a restart of TC cannot be avoided.
Author of Total Commander
https://www.ghisler.com
User avatar
ask-rus
Junior Member
Junior Member
Posts: 44
Joined: 2010-06-17, 10:29 UTC
Location: Russian Federation

Post by *ask-rus »

2ghisler(Author)
Maybe try again in TC 8.0?
User avatar
JohnFredC
Power Member
Power Member
Posts: 886
Joined: 2003-03-14, 13:37 UTC
Location: Sarasota Florida

Post by *JohnFredC »

In many cases, "partial" implementations can add value.

If some INI options cannot be "reloaded" without a restart (due to issues internal to TC), then inform the user and proceed (if the user affirms) with the options that CAN be refreshed.

Perhaps the developer's effort to build such a "partial" mechanism outweighs the overall benefit to TC (this is probable, IMO), but that is an entirely separate issue from feasibility of implementation.
Licensed, Mouse-Centric, moving (slowly) toward Touch-centric
Post Reply