re: CVS commit: src/sys/dev/videomode

2014-11-17 Thread Jared McNeill
On Mon, 17 Nov 2014, matthew green wrote: hmm, does this change the module ABI? ie, kernel version bump time? I don't think anything in sys/modules uses edidvar.h

re: CVS commit: src/sys/dev/videomode

2014-11-16 Thread matthew green
"Jared D. McNeill" writes: > Module Name: src > Committed By: jmcneill > Date: Mon Nov 17 00:46:04 UTC 2014 > > Modified Files: > src/sys/dev/videomode: edid.c edidreg.h edidvar.h > > Log Message: > Parse the extension block count field, and make it available in struct > edid_inf

Re: CVS commit: src/sys/dev/videomode

2011-04-10 Thread Christos Zoulas
In article <20110410161417.ge4...@nef.pbox.org>, Alistair Crooks wrote: >hi christos, > >On Sat, Apr 09, 2011 at 04:53:39PM -0400, Christos Zoulas wrote: >> - use struct copy instead of memcpy > >i haven't looked into this specific instance, but won't gcc (at least) >produce the same code for mem

Re: CVS commit: src/sys/dev/videomode

2011-04-10 Thread Alistair Crooks
hi christos, On Sat, Apr 09, 2011 at 04:53:39PM -0400, Christos Zoulas wrote: > - use struct copy instead of memcpy i haven't looked into this specific instance, but won't gcc (at least) produce the same code for memcpy as struct copy, and the memcpy call has the (admittedly arguable) distinction