On Tue, 3 Mar 2015, Andrew Cagney wrote:
The enum is:enum phase1_role { O_INITIATOR=1, O_RESPONDER=2 };
That could prob use a rename too :P Maybe origin_role ?
so st->st_o_role (and a renamed) md->o_role wouldn't be out of place.
_o_ looks a little silly? :)
My preference is actually for st_original_role / md->original_role. I chose st_role to match md->role.
How about st_origin_r ? or st_origin_role with md->origin(_role) Mostly st_original_role is a bit long and we have the 80 chars police :P Paul _______________________________________________ Swan-dev mailing list [email protected] https://lists.libreswan.org/mailman/listinfo/swan-dev
