Make cm_ commands "class based"

Here you can propose new features, make suggestions etc.

Moderators: Hacker, petermad, Stefan2, white

Post Reply
User avatar
ZoSTeR
Power Member
Power Member
Posts: 1052
Joined: 2004-07-29, 11:00 UTC

Make cm_ commands "class based"

Post by *ZoSTeR »

When looking at the INC file the commands aren't very logical.
E.g.:
cm_SelectFiles
cm_ExchangeSelection
cm_ClearAll
cm_SaveSelection

It might be more logical to use:
cm_Selection_Files
cm_Selection_Exchange
cm_Selection_ClearAll
cm_Selection_Save

I guess this might be hard to change but maybe some aliases won't hurt.
Last edited by ZoSTeR on 2013-10-16, 21:41 UTC, edited 2 times in total.
User avatar
Balderstrom
Power Member
Power Member
Posts: 2148
Joined: 2005-10-11, 10:10 UTC

Post by *Balderstrom »

Why not use: cm_CommandBrowser ?
User avatar
ZoSTeR
Power Member
Power Member
Posts: 1052
Joined: 2004-07-29, 11:00 UTC

Post by *ZoSTeR »

When you select a command from the browser it makes sense but reading those commands elsewhere not so much (for non TC afiados).
User avatar
Balderstrom
Power Member
Power Member
Posts: 2148
Joined: 2005-10-11, 10:10 UTC

Post by *Balderstrom »

You know the command browser also has a filter at the bottom right? so say you want to see "selection", put that in the filter box.
*BLINK* TC9 Added WM_COPYDATA and WM_USER queries for scripting.
User avatar
ZoSTeR
Power Member
Power Member
Posts: 1052
Joined: 2004-07-29, 11:00 UTC

Post by *ZoSTeR »

Yeah I know i can find everything in the browser. This is about naming or "classing" the commands for their purpose.

E.g: cm_ClearAll
Does it clear my lister, my windows, my car or my c: drive?

*edit: the "easier to find" -> "more logical" in the first post
Last edited by ZoSTeR on 2013-10-16, 21:40 UTC, edited 1 time in total.
User avatar
Balderstrom
Power Member
Power Member
Posts: 2148
Joined: 2005-10-11, 10:10 UTC

Post by *Balderstrom »

Ah, well that would break existing installs/menus/scripts/etc. And TC/Mr.Ghisler is pretty fanatical about backwards compatibility -- possibly too much so, as I think many suggestions that in the end would improve the usability of TC are not considered.
*BLINK* TC9 Added WM_COPYDATA and WM_USER queries for scripting.
User avatar
ZoSTeR
Power Member
Power Member
Posts: 1052
Joined: 2004-07-29, 11:00 UTC

Post by *ZoSTeR »

That's why i suggested an alias system. I guess the "to find" was misplaced. See edit.
Post Reply