Re: Directional switching

Mikael Magnusson <[email protected]>
Newsgroups gmane.comp.window-managers.openbox
Message-ID <CAHYJk3SGM6q+fv=WFufiun0JQAJhqxcy81MxrnZq6fDDf0S_rA@mail.gmail.com>
On Thu, Jul 9, 2015 at 7:12 AM, Ian Zimmerman <[email protected]> wrote:
> On 2015-07-09 05:34 +0200, Mikael Magnusson wrote:
>
>> You'll want to initialize these integers to something in
>> config_startup(), then you don't need the elses during parsing.
>
> Is config_startup called during live reconfiguration?

Yes, your config values aren't the only ones that don't break during
reconfiguration :).

>> Would I be correct in thinking that setting one to two is equivalent
>> to setting the other to one half? (If they were not integers).
>
> Yes, it is equivalent mathematically; depending on how it's implemented,
> there may be some loss of precision with a single parameter (if it's
> used to multiply and then divide).  That is probably negligible.  But I
> also believe two parameters are clearer in this case.  What would you
> call the single relative parameter?
>
> Thanks for your comments, I really appreciate the opportunity to improve
> the code!

I'm not sure either, it just seemed more reasonable to configure a
single ratio, than two values that are really just one value that
cancel each other out if you increase both. AngleToDistanceRatio? :)

-- 
Mikael Magnusson
_______________________________________________
openbox mailing list
[email protected]
http://icculus.org/mailman/listinfo/openbox
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.