On 03/23/2007 01:24, Morten Nilsen wrote: > Ravenlock wrote: >> Attached is a patch that will enable the net module to run on FreeBSD (I >> hope). I am using it on 6.2-STABLE. I have no idea what other versions >> it will be compatible with. >> >> I am hoping a few people (running 6.2 or otherwise) might step up and >> give it a whirl, and provide their feedback. > > Not running freebsd, but I have got a comment about the code.. > This forward definition of get_ifmib_general is superflous; > > > +#ifdef __FreeBSD__ > > +static int get_ifmib_general(int row, struct ifmibdata *ifmd); > > + > > +static int > > +get_ifmib_general(int row, struct ifmibdata *ifmd) > > +{ > > That's all. >
It most certainly is. As are many in the E code. I was told it was the "E-way"... and was just trying to keep up. If, when I eventually submit the patch, I'm told its not desired, I'll remove it. :) -- Regards, Ravenlock ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys-and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ enlightenment-users mailing list enlightenment-users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-users