[PATCH:xdm 5/5] Simplify FailedLogin code
Alan Coopersmith
alan.coopersmith at oracle.com
Fri Jun 3 11:08:11 PDT 2011
On 06/ 3/11 03:37 AM, walter harms wrote:
> mmmh, you are changing
> FailedLogin (d, struct greet_info) into FailedLogin (d, char *)
>
> maybe it is better to keep greet_info as future version
> may need additional info. I would stick with struct greet_info
> and use greet->name instead username.
>
> It is not a real problem just my experience.
This isn't a widely called API, just a local helper function called
exactly twice - as the patch shows it's trivial to update all the callers
if and when we find a need for some other arguments.
Until then, this gets rid of the need to shove username into greet->name
temporarily and then rip it back out just to satisfy the previous design.
I'll take better and more maintainable code now against a small chance of
having to do 2 minutes more work at some future point if we find a need
for some other bit of the greet struct.
--
-Alan Coopersmith- alan.coopersmith at oracle.com
Oracle Solaris Platform Engineering: X Window System
More information about the xorg-devel
mailing list