On Sat, Dec 30, 2006 at 12:57:20PM +0100, Dominik Vogt wrote: > On Sat, Dec 30, 2006 at 02:13:29AM +0000, seventh guardian wrote: > [snip] > > Ok, here goes the patch module_interface.c. The first goal of having a > > functional system is hopefully done. > > I'm going to create a branch in cvs where we can apply, test and > modify the patches. Don't bother to to the changes I posted > earlier; I'll do it myself. > > The branch name will be "fvwm-module-struct-branch". To pull it > from cvs into a directory "modstruct-branch", issue > > $ cvs co -Pd modstruct-branch -r fvwm-module-struct-branch fvwm
I'm done proof-reading and cleaning up the code. Casting the module pointer to int did not even work on 32-bit machines because there is a limit to the numbers that can be used in an fdset (it's a limited size bit field). Instead, I added a member "slot" to the module struct that has no specific meaning other that that the number is unique and is between 0 and MAX_NUM_MODULES-1 (=256, from libs/defaults.h). It's a bit expensive to maintain the highest slot used when a module is destroyed, but for now it will do. I hope thig ugly bit of code goes away soon. Ciao Dominik ^_^ ^_^ -- Dominik Vogt, [EMAIL PROTECTED]
signature.asc
Description: Digital signature
