With regard to the issues in ecore_wl_input ... I took a look at the expected callback of the wl_seat_listener, and the function that we have setup for that is proper (wrt function params, etc)...so I am unsure why this is complaining about an incorrect type...
dh On 11/12/2015 10:30 AM, Stefan Schmidt wrote: > Hello. > > Amend for the first message. > > On 12/11/15 16:26, Stefan Schmidt wrote: >> Hello. >> >> I took a moment to run our code through the sparse semantic parser. >> https://sparse.wiki.kernel.org/index.php/Main_Page >> >> You can easily do so as well by installing sparse and run "make CC=cgcc" >> to enable the build wrapper. >> >> But now to some actual findings. Really most of it has been noise and i >> reduced it to some things here manually. >> >> Find some remarks on some of the categories inside. If you feel like >> this is on your turf and you either think its false positive or you are >> going to fix it let me know. >> >> lib/eina/eina_fp.c:14:4: warning: constant 0x0000000100000000 is so big >> it is long >> lib/eina/eina_fp.c:439:17: warning: constant 0x00000003243f6a89 is so >> big it is long >> lib/eina/eina_fp.c:440:17: warning: constant 0x00000003243f6a89 is so >> big it is long >> lib/eina/eina_fp.c:441:18: warning: constant 0x00000003243f6a89 is so >> big it is long >> lib/eina/eina_fp.c:447:28: warning: constant 0x00000003243f6a89 is so >> big it is long >> lib/eina/eina_fp.c:447:46: warning: constant 0x00000003243f6a89 is so >> big it is long >> lib/eina/eina_fp.c:453:31: warning: constant 0x00000003243f6a89 is so >> big it is long >> lib/eina/eina_fp.c:479:17: warning: constant 0x00000003243f6a89 is so >> big it is long >> lib/ecore/ecore_anim.c:583:16: warning: constant 0x00000003243f6a89 is >> so big it is long >> lib/ecore/ecore_anim.c:583:16: warning: constant 0x00000003243f6a89 is >> so big it is long >> lib/ecore/ecore_anim.c:588:16: warning: constant 0x00000003243f6a89 is >> so big it is long >> lib/ecore/ecore_anim.c:593:16: warning: constant 0x00000003243f6a89 is >> so big it is long >> >> ----- >> >> lib/eina/eina_rbtree.c:63:33: error: dubious one-bit signed bitfield >> lib/eina/eina_thread_queue.c:62:42: error: dubious one-bit signed bitfield >> lib/evas/canvas/evas_object_text.c:71:43: error: dubious one-bit signed >> bitfield >> lib/evas/canvas/evas_object_text.c:72:46: error: dubious one-bit signed >> bitfield >> modules/evas/engines/buffer/evas_engine.h:47:51: error: dubious one-bit >> signed bitfield >> modules/evas/engines/buffer/evas_engine.h:48:49: error: dubious one-bit >> signed bitfield >> modules/evas/engines/buffer/Evas_Engine_Buffer.h:24:30: error: dubious >> one-bit signed bitfield >> modules/evas/engines/buffer/evas_engine.h:47:51: error: dubious one-bit >> signed bitfield >> modules/evas/engines/buffer/evas_engine.h:48:49: error: dubious one-bit >> signed bitfield >> ../src/modules/evas/engines/buffer/Evas_Engine_Buffer.h:24:30: error: >> dubious one-bit signed bitfield >> ../src/modules/evas/engines/buffer/Evas_Engine_Buffer.h:24:30: error: >> dubious one-bit signed bitfield >> ../src/modules/evas/engines/buffer/Evas_Engine_Buffer.h:24:30: error: >> dubious one-bit signed bitfield >> ../src/modules/evas/engines/buffer/Evas_Engine_Buffer.h:24:30: error: >> dubious one-bit signed bitfield >> lib/ecore_ipc/ecore_ipc_private.h:84:35: error: dubious one-bit signed >> bitfield >> lib/ecore_ipc/ecore_ipc_private.h:102:35: error: dubious one-bit signed >> bitfield >> modules/ecore_evas/engines/x/ecore_evas_x.c:902:29: error: dubious >> one-bit signed bitfield >> modules/ecore_evas/engines/x/ecore_evas_x.c:903:33: error: dubious >> one-bit signed bitfield >> modules/ecore_evas/engines/x/ecore_evas_x.c:904:30: error: dubious >> one-bit signed bitfield >> modules/ecore_evas/engines/x/ecore_evas_x.c:905:34: error: dubious >> one-bit signed bitfield >> modules/ecore_evas/engines/x/ecore_evas_x.c:906:34: error: dubious >> one-bit signed bitfield >> modules/evas/image_loaders/gif/evas_image_load_gif.c:37:23: error: >> dubious one-bit signed bitfield >> >> Funny one. This bitfields are signed so they can only hold -1 and 0. Not >> a real bummer as we normally use them in if (x) only but if we have >> cases where we really expect an 1 after we might be in trouble. Could be >> a good idea wot move them over to Eina_Bool. >> >> ----- >> >> lib/emile/emile_main.c:149:53: warning: right shift by bigger than >> source value >> lib/emile/emile_main.c:150:53: warning: right shift by bigger than >> source value >> lib/emile/emile_main.c:151:53: warning: right shift by bigger than >> source value >> >> Cedric? >> >> ----- >> >> static_libs/rg_etc/rg_etc1.c:1444:21: warning: crazy programmer >> static_libs/rg_etc/rg_etc1.c:1458:21: warning: crazy programmer >> static_libs/rg_etc/rg_etc1.c:1483:21: warning: crazy programmer >> modules/evas/image_savers/tgv/evas_image_save_tgv.c:49:14: warning: >> crazy programmer >> >> Man, I love this warnings. :) >> ----- >> >> lib/ecore_input/ecore_input_compose.h:9900:19: warning: trying to >> concatenate 63013-character string (8191 bytes max) >> modules/evas/engines/gl_common/shader_3d/evas_gl_3d_shaders.x:975:9: >> warning: trying to concatenate 8394-character string (8191 bytes max) >> modules/evas/engines/gl_common/shader_3d/evas_gl_3d_shaders.x:1441:9: >> warning: trying to concatenate 10118-character string (8191 bytes max) >> modules/evas/engines/gl_common/shader_3d/evas_gl_3d_shaders.x:2113:9: >> warning: trying to concatenate 11086-character string (8191 bytes max) >> >> What are we going to do about this? Do we take our maximal size for >> strings seriously or does this not really matter? >> >> ----- >> >> lib/evas/canvas/evas_map.c:284:27: warning: Variable length array is used. >> lib/ector/software/ector_software_rasterizer.c:215:22: warning: Variable >> length array is used. >> lib/ector/software/ector_software_rasterizer.c:250:22: warning: Variable >> length array is used. >> lib/ecore_x/xlib/ecore_x_randr.c:559:42: warning: Variable length array >> is used. >> modules/evas/engines/gl_common/evas_gl_3d_shader.c:181:20: warning: >> Variable length array is used. >> bin/evas/evas_cserve2_shm_debug.c:42:14: warning: Variable length array >> is used. >> >> ----- >> >> lib/emotion/emotion_webcam.c:120:27: warning: dubious: !x & y >> >> ----- >> >> modules/evas/engines/gl_common/evas_gl_core.c:1624:46: error: too many >> arguments for function glsym_evas_gl_native_context_get >> modules/evas/engines/gl_common/evas_gl_core.c:1638:40: error: too many >> arguments for function glsym_evas_gl_engine_data_get >> >> Seems to be a misunderstanding of the function pointer. >> >> ----- >> >> modules/evas/engines/gl_x11/evas_engine.c:230:39: error: too many >> arguments for function glsym_evas_gl_common_error_set >> modules/evas/engines/gl_x11/evas_engine.c:285:44: error: too many >> arguments for function glsym_evas_gl_common_error_set >> modules/evas/engines/gl_x11/evas_engine.c:310:44: error: too many >> arguments for function glsym_evas_gl_common_error_set >> modules/evas/engines/gl_x11/evas_engine.c:329:39: error: too many >> arguments for function glsym_evas_gl_common_error_set >> modules/evas/engines/gl_x11/evas_engine.c:358:39: error: too many >> arguments for function glsym_evas_gl_common_error_set >> modules/evas/engines/gl_x11/evas_engine.c:374:39: error: too many >> arguments for function glsym_evas_gl_common_error_set >> modules/evas/engines/gl_x11/evas_engine.c:381:39: error: too many >> arguments for function glsym_evas_gl_common_error_set >> modules/evas/engines/gl_x11/evas_engine.c:404:39: error: too many >> arguments for function glsym_evas_gl_common_error_set >> modules/evas/engines/gl_x11/evas_engine.c:446:39: error: too many >> arguments for function glsym_evas_gl_common_error_set >> modules/evas/engines/gl_x11/evas_engine.c:474:39: error: too many >> arguments for function glsym_evas_gl_common_error_set >> modules/evas/engines/gl_x11/evas_engine.c:481:39: error: too many >> arguments for function glsym_evas_gl_common_error_set >> modules/evas/engines/gl_x11/evas_engine.c:557:81: error: too many >> arguments for function glsym_evas_gl_common_error_set >> modules/evas/engines/gl_x11/evas_engine.c:558:66: error: too many >> arguments for function glsym_evas_gl_common_error_set >> modules/evas/engines/gl_x11/evas_engine.c:576:48: error: too many >> arguments for function glsym_evas_gl_common_error_set >> modules/evas/engines/gl_x11/evas_engine.c:577:53: error: too many >> arguments for function glsym_evas_gl_common_error_set >> modules/evas/engines/gl_x11/evas_engine.c:599:39: error: too many >> arguments for function glsym_evas_gl_common_error_set >> modules/evas/engines/gl_x11/evas_engine.c:632:39: error: too many >> arguments for function glsym_evas_gl_common_error_set >> modules/evas/engines/gl_x11/evas_engine.c:641:39: error: too many >> arguments for function glsym_evas_gl_common_error_set >> modules/evas/engines/gl_x11/evas_engine.c:850:39: error: too many >> arguments for function glsym_evas_gl_common_error_set >> modules/evas/engines/gl_x11/evas_engine.c:857:39: error: too many >> arguments for function glsym_evas_gl_common_error_set >> modules/evas/engines/gl_x11/evas_engine.c:895:39: error: too many >> arguments for function glsym_evas_gl_common_error_set >> modules/evas/engines/gl_x11/evas_engine.c:902:39: error: too many >> arguments for function glsym_evas_gl_common_error_set >> modules/evas/engines/gl_x11/evas_engine.c:923:39: error: too many >> arguments for function glsym_evas_gl_common_error_set >> modules/evas/engines/gl_x11/evas_engine.c:1127:39: error: too many >> arguments for function glsym_evas_gl_common_error_set >> modules/evas/engines/gl_x11/evas_engine.c:1145:39: error: too many >> arguments for function glsym_evas_gl_common_error_set >> modules/evas/engines/gl_x11/evas_engine.c:1545:40: error: too many >> arguments for function glsym_evas_gl_common_shaders_flush >> modules/evas/engines/gl_x11/evas_engine.c:1967:45: error: too many >> arguments for function glsym_evas_gl_common_error_get >> modules/evas/engines/gl_x11/evas_engine.c:1982:34: error: too many >> arguments for function glsym_evas_gl_common_error_set >> >> ----- >> >> lib/ecore_wayland/ecore_wl_input.c:128:4: warning: incorrect type in >> initializer (incompatible argument 3 (different signedness)) >> lib/ecore_wayland/ecore_wl_input.c:128:4: expected void ( >> *capabilities )( ... ) >> lib/ecore_wayland/ecore_wl_input.c:128:4: got void ( static >> [toplevel] *<noident> )( ... ) > > Missed a few. Here is the rest: > > /usr/include/sys/param.h:99:9: warning: preprocessor token MIN redefined > ../src/lib/ecore/ecore_private.h:73:10: this was the original definition > /usr/include/sys/param.h:100:9: warning: preprocessor token MAX redefined > ../src/lib/ecore/ecore_private.h:77:10: this was the original definition > > ----- > > lib/ecore_imf/ecore_imf_context.c:755:13: warning: incorrect type in > assignment (different argument counts) > lib/ecore_imf/ecore_imf_context.c:755:13: expected void ( *[usertype] > func )( ... ) > lib/ecore_imf/ecore_imf_context.c:755:13: got void ( *[usertype] func > )( ... ) > lib/ecore_imf/ecore_imf_context.c:781:31: error: incompatible types in > comparison expression (different argument counts) > lib/ecore_imf/ecore_imf_context.c:808:19: error: too many arguments for > function func > lib/ecore_imf/ecore_imf_context.c:1230:31: error: incompatible types in > comparison expression (different argument counts) > lib/ecore_imf/ecore_imf_context.c:1256:22: error: too many arguments for > function func > > ----- > > lib/embryo/embryo_float.c:84:11: warning: cast to non-scalar > > regards > Stefan Schmidt > > ------------------------------------------------------------------------------ > _______________________________________________ > enlightenment-devel mailing list > enlightenment-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > ------------------------------------------------------------------------------ _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
