But just to remind Chris and co.; this new ivtvctl patch is only useful (and in fact, will only compile) if the infrared part
of Jean-Francois's patch is applied (the ivtv_reset_ir_gpio() routine, a couple of places in ivtv-ioctl.c that call it, and a definition
in ivtv.h).



Consider it done! This adds the long-only option --reset-ir to ivtvctl. We're like totally out of letters to use for short options, and I didn't think this would be something you'd use enough to get a short option. I declared static int option_reset_ir, I hope that's ok.


I also removed a whole mess of close(fd) calls which as far as I could tell would only be called with fd = -1 (the device isn't opened until after all the options are set up).

I'd also like to submit a patch to reorder the output of usage() to be in case-insensitive alphabetic order by short option character (keeping the grouping Usage, Expert though). I thought this should be separate from the above changes though, in case you don't want the help redone.

------------------------------------------------------------------------



\


------------------------------------------------------- This SF.Net email is sponsored by Oracle Space Sweepstakes Want to be the first software developer in space? Enter now for the Oracle Space Sweepstakes! http://ads.osdn.com/?ad_id=7412&alloc_id=16344&op=click _______________________________________________ ivtv-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ivtv-devel

Reply via email to