Messages in this thread | | | Date | Thu, 24 Oct 2013 07:28:28 +0200 | From | Michael Opdenacker <> | Subject | Re: [PATCH] reiserfs: remove duplicate define |
| |
Hi Jan,
Thank you for your review!
On 10/23/2013 11:50 AM, Jan Kara wrote: > On Mon 21-10-13 09:54:57, Michael Opdenacker wrote: >> This patch removes a duplicate define in fs/reiserfs/reiserfs.h > Hum, so the duplicate define certainly isn't nice but it's really a > result of a namespace collision between return codes of two different (sets > of) functions. So deleting the duplicate isn't really solving the problem, > just hiding it. So I'd prefer more one of the following two solutions: > 1) Just ignore the problem. Reiserfs is mostly dead and this isn't likely to > cause any subtle issues. > 2) Prefix the return codes somehow so that those two error namespaces don't > clash. As a bonus you can convert defines to enums but I'm not sure that's > worth the bother (prefixing is a simple search & replace so that should be > trivial, well except for the CARRY_ON case). I like your second solution, adding a prefix to avoid collisions between two error namespaces, all the more as this looks like a good solution for similar issues that I found.
Thanks again,
Cheers,
Michael.
-- Michael Opdenacker, CEO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com +33 484 258 098
| |