Messages in this thread | | | Date | Thu, 19 Mar 2015 14:10:47 +0100 | From | Borislav Petkov <> | Subject | Re: [PATCH] trace, RAS: remove unnecessary const |
| |
On Thu, Mar 19, 2015 at 08:57:56AM -0400, Steven Rostedt wrote: > That's a bit of a stretch. But sure, there's no real downside to having > it, except that it makes one take a double take when seeing it, and > thinking about why it would even be needed (like I did).
Why? It simply declares the contract that this argument won't be changed. And if there's no downside, there's no need to do anything here.
> I think you had the crazy fantasy with the above patch!
Wasn't me.
-- Regards/Gruss, Boris.
ECO tip #101: Trim your mails when you reply. --
| |