[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Nano-devel] Comment/Uncomment feature patch
From: |
Mike Frysinger |
Subject: |
Re: [Nano-devel] Comment/Uncomment feature patch |
Date: |
Tue, 17 May 2016 22:39:42 -0400 |
On 17 May 2016 18:58, Mike Scalora wrote:
> On Tue, May 17, 2016 at 10:08 AM, Mike Frysinger <address@hidden> wrote:
> > On 17 May 2016 06:28, Mike Scalora wrote:
> >> +if test "x$enable_comment" = xno; then
> >> + AC_DEFINE(DISABLE_COMMENT, 1, [Define this to disable the
> >> comment/uncomment functionality.])
> >> +fi
> >
> > i strongly dislike "negative" knobs because it gets harder to read at a
> > glance when you see things like "if !defined(DISABLE_COMMENT)". it is
> > pretty easy to invert this though into an ENABLE_COMMENT symbol.
>
> Are you suggesting the comment feature flag should start a new
> convention by itself? Right now NANO_TINY is the only affirmative flag
> and an all the other features are negative/disable flags.
>
> It wouldn't take that much to switch to ENABLE_COMMENT but would it
> make more sense to switch all the feature flags at the same time?
if we agree on the direction, i think we can move them over bit by bit.
or at least, not add more badness that we'll just need to undo in the
future.
> PS: I thought I had trademarked the hyphen-Mike signature in the early
> nineties, you seem to be creating some brand confusion with "-mike"
> :-P
if it makes you feel better, yours sorts before mine ;)
-mike
signature.asc
Description: Digital signature
- Re: [Nano-devel] Comment/Uncomment feature patch, (continued)
- Re: [Nano-devel] Comment/Uncomment feature patch, Mike Scalora, 2016/05/10
- Re: [Nano-devel] Comment/Uncomment feature patch, Benno Schulenberg, 2016/05/11
- Re: [Nano-devel] Comment/Uncomment feature patch, Mike Scalora, 2016/05/12
- Re: [Nano-devel] Comment/Uncomment feature patch, Benno Schulenberg, 2016/05/15
- Re: [Nano-devel] Comment/Uncomment feature patch, Mike Scalora, 2016/05/15
- Re: [Nano-devel] Comment/Uncomment feature patch, Benno Schulenberg, 2016/05/15
- Re: [Nano-devel] Comment/Uncomment feature patch, Benno Schulenberg, 2016/05/16
- Re: [Nano-devel] Comment/Uncomment feature patch, Mike Scalora, 2016/05/17
- Re: [Nano-devel] Comment/Uncomment feature patch, Mike Frysinger, 2016/05/17
- Re: [Nano-devel] Comment/Uncomment feature patch, Mike Scalora, 2016/05/17
- Re: [Nano-devel] Comment/Uncomment feature patch,
Mike Frysinger <=
- Re: [Nano-devel] Comment/Uncomment feature patch, Benno Schulenberg, 2016/05/18
- Re: [Nano-devel] Comment/Uncomment feature patch, Mike Scalora, 2016/05/18
- Re: [Nano-devel] Comment/Uncomment feature patch, Benno Schulenberg, 2016/05/19
- Re: [Nano-devel] Comment/Uncomment feature patch, Mike Scalora, 2016/05/19
- Re: [Nano-devel] Comment/Uncomment feature patch, Benno Schulenberg, 2016/05/19
- Re: [Nano-devel] Comment/Uncomment feature patch, Mike Scalora, 2016/05/20
- Re: [Nano-devel] Comment/Uncomment feature patch, Benno Schulenberg, 2016/05/21
- Re: [Nano-devel] Comment/Uncomment feature patch, Mike Scalora, 2016/05/20
- Re: [Nano-devel] Comment/Uncomment feature patch, Benno Schulenberg, 2016/05/21
- Re: [Nano-devel] Comment/Uncomment feature patch, Mike Scalora, 2016/05/21