pccasto / rubyripper

Automatically exported from code.google.com/p/rubyripper
0 stars 0 forks source link

Feature request: ability to pass a 'nice' value to all encoders #252

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Headline says it: It would have been nice to have a box in which to input 
a 'nice' value to pass to all encoders.... 

Original issue reported on code.google.com by Xeno...@gmail.com on 22 Oct 2008 at 4:54

GoogleCodeExporter commented 8 years ago
Should it really be made optional? Can't we use it as default value?

Original comment by rubyripp...@gmail.com on 26 Oct 2008 at 9:20

GoogleCodeExporter commented 8 years ago
Well. People might have different reasons for different values. Some might like 
high
priority and some low. Anyway it wouldn't be your fault if somebody chose a 
stupid
value (the option could also be marked 'advanced' with a warning). 
Personally I would pass a value for piority slightly below normal to all 
encoders,
maybe even lower... a value in the range 6 to 12 maybe.

Original comment by Xeno...@gmail.com on 26 Oct 2008 at 3:36

GoogleCodeExporter commented 8 years ago
I do want to protect some users from making stupid choices ;) Encoding processes
having lower priority should never result in a breaked encoding. On the other 
hand
some people might notice a more responsive desktop at a small encoding time 
cost. I
see no problems.

From a usability perspective it's not always good to offer more options. I'd 
like to
keep rubyripper a lean app and to be usefull with the default values. Too much
options only works confusing for most users.

So prepending "nice -n 6" to the encoding command would do the trick, right?

Original comment by rubyripp...@gmail.com on 26 Oct 2008 at 8:05

GoogleCodeExporter commented 8 years ago
I see your point... :)
Well, if you want to hardcode it I think that (-n 6) would be a sane value. 
And people might be a little more happy as their webbrowsing might be a tad 
more 
fluent while ripping CDs...

Original comment by Xeno...@gmail.com on 26 Oct 2008 at 8:37

GoogleCodeExporter commented 8 years ago
Fixed with commit 30cec02.

Original comment by rubyripp...@gmail.com on 1 Nov 2008 at 12:02