On Wed, Feb 16, 2022 at 10:32:43AM +0000, Ferruh Yigit wrote: > On 2/15/2022 5:30 PM, Bruce Richardson wrote: > > C++ does not allow implicit conversion to/from void*, so we need an > > explicit cast to allow the driver sdk header to be included from C++ > > code. > > > > I remember patches removing explicit "void *" cast, in the past, > > to document, is the rule as following: > - public and sdk headers should support c++, hence these files > must have explicit "void *" cast > - .c files should NOT have explicit "void *" cast > That would be my understanding yes. It's annoying they conflict, so I would suggest softening the second rule to allow casts to be present. Thankfully with chkincs support we should be able to detect violations of rule #1, which is the more important of the two.
/Bruce
