lkml.org 
[lkml]   [2002]   [Feb]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] HDLC patch for 2.5.5 (0/3)
Francois Romieu wrote:
>
> Krzysztof Halasa <khc@pm.waw.pl> :
> [...]
> > Now... You just want to introduce an artificial struct which contains
> > only the union... Why?
>
> Copy-paste abuse.
>
> > We could use just the union instead (?).
>
> Yes. I'll try that tonite.
>
> [...]
> > Yes, the compiler would compile that. Anyway, don't you think it's
> > a little messy? Void * pointers are IMHO not that evil.
>
> Once the union in a struct disappears it should be minimal.
>
> Regarding void * against union/union * I feel like minimal type checking is
> better than none.

Agreed... the more type checking you can provide, the better.

Note that if it is possible to avoid the union in a manner that follows
the VFS inode cleanup, feel free to that too... Basically each private
info struct looks like

struct foo_inode {
/* ... driver-private stuff here ... */
struct inode vfs_inode; /* last member of struct */
};

Making it the last member of the structure ensures that people are not
tempted to do gross typecasting based on assumptions that the
"superclass" type is always located at the beginning of the subclass
substructure. (I don't know how well that applies to this case, just a
suggestion...)

Jeff


--
Jeff Garzik | "Why is it that attractive girls like you
Building 1024 | always seem to have a boyfriend?"
MandrakeSoft | "Because I'm a nympho that owns a brewery?"
| - BBC TV show "Coupling"
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 13:18    [W:0.072 / U:0.108 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site