Linus Torvalds writes:
> And while at it, please make it much more readable by writing it as
>
> time_adjust_step = time_adjust;
> if (time_adjust_step) {
> ..
>
> which is even less to type (no "!= 0", no extra parenthesis, just a
> ";<nl><tab>", and you've saved a whopping three bytes of source code while
... you've forgotten that now I've had to type "time_adjust_step"
twice. :P Anyway, I don't mind changing it, if you think it's worth
pulling that little bit of code out into its own function.
Paul.
-
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/
- Follow-Ups:
- Re: [PATCH] Provide an interface for getting the current tick length
- From: Linus Torvalds <[email protected]>
- Re: [PATCH] Provide an interface for getting the current tick length
- References:
- [PATCH] Provide an interface for getting the current tick length
- From: Paul Mackerras <[email protected]>
- Re: [PATCH] Provide an interface for getting the current tick length
- From: Andrew Morton <[email protected]>
- Re: [PATCH] Provide an interface for getting the current tick length
- From: Paul Mackerras <[email protected]>
- Re: [PATCH] Provide an interface for getting the current tick length
- From: Andrew Morton <[email protected]>
- Re: [PATCH] Provide an interface for getting the current tick length
- From: Paul Mackerras <[email protected]>
- Re: [PATCH] Provide an interface for getting the current tick length
- From: Linus Torvalds <[email protected]>
- [PATCH] Provide an interface for getting the current tick length
- Prev by Date: [PATCH] Fix over-zealous tag clearing in radix_tree_delete
- Next by Date: error during kernel compilation for xilinx ml300 specific kernel
- Previous by thread: Re: [PATCH] Provide an interface for getting the current tick length
- Next by thread: Re: [PATCH] Provide an interface for getting the current tick length
- Index(es):
 |