check on NULL add in wayland elm_drag_start API
@fix
Details
Details
Call elm_grag_start with data = NULL under wayland
Diff Detail
Diff Detail
- Repository
- rEFL core/efl
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
Just a style wish. Thanks for the patch
src/lib/elementary/elm_cnp.c | ||
---|---|---|
3348–3354 | Its just a matter of style, but i would preffer wl_cnp_selection.selbuf = NULL; wl_cnp_selection.buflen = 0; if (data) { do the strdup / strlen here } just for readability reasons. Otherwise, good catch! :) |
src/lib/elementary/elm_cnp.c | ||
---|---|---|
3348–3354 | or use eina_strdup() |