Re: [Maria-developers] Rev 3480: Post review changes (total (really))
![](https://secure.gravatar.com/avatar/3c2329333dd5898f2e72818af773243d.jpg?s=120&d=mm&r=g)
Hi, Oleksandr! On Dec 23, Oleksandr Byelkin wrote:
22.12.2012 21:28, Sergei Golubchik пишет:
@@ -100,6 +101,8 @@ struct st_dynamic_column_value
typedef struct st_dynamic_column_value DYNAMIC_COLUMN_VALUE;
+/* old functions (deprecated) */ I usually prefer code to comments. Kind of a comment that a compiler can make use of. That means an assert(aaa == 0) instead of /* aaa should be 0 here */ and instead of comment above, I'd rather have __attribute__((deprecated))
__attribute__((deprecated)) is GCC, we also use MS compiler. So ifdef is better until we get appropriate multi-platform macro.
1. __attribute__ is defined to be empty for other compilers (somewhere in my_global.h, I guess), so it won't break the compilation. 2. I did not mean to use it instead of the #ifdef, I mean to use it instead of a /* old functions (deprecated) */ comment. Because a compiler cannot read the comment. 3. And anyway, it was just a suggestion, you can keep your current code and not change anything. As you prefer.
everything else will be fixed.
Thanks! Regards, Sergei
participants (1)
-
Sergei Golubchik