mh0rst / turionpowercontrol

Automatically exported from code.google.com/p/turionpowercontrol
9 stars 4 forks source link

Disallow certain combinations of command line parameters #17

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
There's some (perceived) redundancy in command line parameters space, e.g.

-node ... vs -set node ...
-core ... vs -set core ...

This causes confusion among users, for instance:

TurionPowerControl -set node 0 vcore 1.40 -nbvid 0 40

Manipulates vcore of node 0 but nbvid of _all_ nodes.

Short-term remedy is disallowing combinations of -set with any other command 
line parameter.

Original issue reported on code.google.com by kszy...@gmail.com on 17 Jan 2013 at 6:42

GoogleCodeExporter commented 9 years ago
Alternative approach would be dropping 'node' and 'core' -set suboptions and 
advocating use of -node/-core instead.

For instance, we'd have:
TurionPowerControl -node 0 -set vcore 1.40

Original comment by kszy...@gmail.com on 17 Jan 2013 at 7:08

GoogleCodeExporter commented 9 years ago
With r154, -set may only be used exclusively thus taking care of this issue.

Original comment by kszy...@gmail.com on 20 Jan 2013 at 4:07