Re: anon unions are cool

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Jan 22, 2006, at 19:36, Albert Cahalan wrote:
For example, suppose we wanted to rename a badly-named struct member. If the struct is used all over the kernel, this becomes a giant project with huge potential for patch conflicts.

Actually, pure struct-member renames are not that common, however it _is_ common to add an accessor method due to upcoming locking/ virtualization changes or similar. For that case (Using a random recent example from the list. This was decided to not be the best route for pid virtualization, but it's just an example):

struct task_struct {
	[...]
	
	pid_t pid;
	
	[...]
};

This would become:

struct task_struct {
	[...]
	
	union {
		pid_t pid __deprecated;
		pid_t __internal_pid;
	};
	
	[...]
};

static inline pid_t task_pid(struct task_struct *t) {
	return t->__internal_pid;
}

Code that used task->pid would get a deprecation warning, however new code that used the task_pid(task) would work fine; and the task_pid() function itself would generate no warnings.

Cheers,
Kyle Moffett

--
Debugging is twice as hard as writing the code in the first place. Therefore, if you write the code as cleverly as possible, you are, by definition, not smart enough to debug it.
  -- Brian Kernighan


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux