Re: [PATCH 12/13] revert: Introduce skip-all to cleanup sequencer data

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

 



Hi Miles,

Miles Bader writes:
> Ramkumar Ramachandra <artagnon@xxxxxxxxx> writes:
>> Interesting side note: I'd initially wanted to use "skip_all" and
>> "continue", but "continue" is a C keyword.  That's why I'd reluctantly
>> suffixed "_oper" to both for consistency.
>
> It seems a good idea to restrict such uglification to only those cases
> where it's necessary, not make _everything_ ugly just for the sake of
> consistency.....
>
> [The traditional thing to do with C-keyword conflicts, when there's no
> obvious and natural alternative, seems to just be intentional mispelling
> -- "continu", "kontinue", "cont", "_continue", whatever.  Yes, they're
> ugly, but people will know why you did it, and they'll forgive you.]

Interesting.  Are there such examples in the Git codebase as well?

-- Ram
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]