Re: RealDeal patch to Matchbox 0.50

Good Morning,

> > [...] which makes Matchbox usable with
> > my RealDeal thingy when using -DUSE_REALDEAL.

> I'm sorry but you you explain what your patch is actually for ?

Well, it contains the modifications I made to Matchbox in order to
use it with the program you saw screenshots of. That is, take four
placement values as commandline arguments and put the window _there_
instead of making it 'full'screen.

Since I doubt it would be actually useful for anybody but me, I was
just hoping you'd include it anyway (by means of the #ifdef's it 
should not disturb anybody...)
The other things were intended to be the bribe-money =).

> How does one configure it then ;-) ?
One adds "-DUSE_REALDEAL" to src/Makefile by hand ;-).

Matthew, if you'd be willing to apply this patch per principle, but
don't want to do it because it's too dirty programmed I'd be happy
to improve on that. It would require a brief explanation of your
commandline argument parser though. 
Also, I realize the usage of a global variable is no good style.
Just, how would one handle the params all the way through to the
existing Matchbox parser and then from there to
main_client_move_resize()?

After all, commandline options might be the wrong way, and this
should be done through hints?

Sorry for all the newbie questions; just kinda hard stuff to me,
since at school we use Borland and the teacher is confident that
using indentation was "bad, American" style.

> Like you say, including <linux/*> headers is bad form.  It would
> probably be better just to use glibc's <fcntl.h>, which will prototype
> those functions as well as declaring the dnotify constants (under
> -D_GNU_SOURCE).
It wasn't clear if Matthew needed some stuff from linux/fcntl.h,
specifically. But that way it's all good then, I guess.

Faithfully yours, Raphael