Messages in this thread | | | Date | Thu, 3 Feb 2005 11:38:58 +0100 | From | Pavel Machek <> | Subject | Re: [linux-pm] Re: driver model u32 -> pm_message_t conversion: help needed |
| |
Hi!
> >> > Sorry for being late responding to this, but I'd say this is a > >> prime > example for typedef's considered evil (see Greg's OLS talk > >> ;). > >> > > >> > It would be a lot cleaner if it was made a struct and then > >> passing a > struct pointer as the argument instead of passing the > >> struct by value > as you do right now. > >> > >> Sorry, can't do that. That would require flag day and change > >> everything at once. That is just not feasible. When things are > >> settled, it is okay to change it to struct passed by value.. It is > >> small anyway and at least we will not have problems with freeing it > >> etc. > > Pavel> Well, we could switch to passing struct by reference > > Pavel> (typedef struct pm_message *pm_message_t) > > Pavel> , but AFAICS it would only bring us problems with lifetime > Pavel> rules etc. Lets not do it. Pavel > > This way you end up hiding what is really going on, the very problem > of using typedefs. If the change is really needed why not get it right > in the first go?
Because it is impossible?
[You can't change all drivers at once in incompatible way. See previous discussion, about half a year ago.] Pavel -- People were complaining that M$ turns users into beta-testers... ...jr ghea gurz vagb qrirybcref, naq gurl frrz gb yvxr vg gung jnl! - 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/
| |