tasn pushed a commit to branch master.
http://git.enlightenment.org/core/efl.git/commit/?id=d85029a5c0a8605c0654b517310639a3a1b04a46
commit d85029a5c0a8605c0654b517310639a3a1b04a46
Author: Tom Hacohen <t...@stosb.com>
Date: Thu Jun 18 13:44:51 2015 +0100
Eo: Fix a potentially dangerous lack of {}.
You should always use curly brackets. Especially when the inside statement
has its own curlys. This can be confusing and has already lead to bugs in
many projects.
---
src/lib/eo/eo.c | 14 ++++++++------
1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/src/lib/eo/eo.c b/src/lib/eo/eo.c
index c143a4a..1a18287 100644
--- a/src/lib/eo/eo.c
+++ b/src/lib/eo/eo.c
@@ -722,12 +722,14 @@ _eo_api_desc_get(const void *api_func, const _Eo_Class
*klass, const _Eo_Class *
* function name itself. Slow, but this should rarely happen.
*/
for (unsigned int i = 0; i < cur_klass->desc->ops.count; i++)
- if (api_name && !strcmp(api_name, op_descs[i].api_name))
- {
- if (op_descs[i].api_func == NULL || op_descs[i].api_func
== ((void (*)())-1))
- break;
- return &op_descs[i];
- }
+ {
+ if (api_name && !strcmp(api_name, op_descs[i].api_name))
+ {
+ if (op_descs[i].api_func == NULL ||
op_descs[i].api_func == ((void (*)())-1))
+ break;
+ return &op_descs[i];
+ }
+ }
#endif
}
}
--
