On Mon, 13 Dec 2010, Carsten Haitzler (The Rasterman) wrote:
> On Sat, 11 Dec 2010 07:52:44 +0100 (CET) Vincent Torri <[email protected]> > said: > >> >> >> On Fri, 10 Dec 2010, Enlightenment SVN wrote: >> >>> Log: >>> fix void ptr math. odd that no warnings happened before with all my >>> warning flags on. >> >> just use -Wpointer-arith > > it should have been on. i use -W -Wall -Wextra. odd. i have SEEN these > warnings > before too. sure, as i mentioned them when i compiled with suncc. Otherwise, with gcc, from http://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html#Warning-Options "-Wpointer-arith Warn about anything that depends on the .size of. a function type or of void. GNU C assigns these types a size of 1, for convenience in calculations with void * pointers and pointers to functions. In C++, warn also when an arithmetic operation involves NULL. This warning is also enabled by -pedantic." so not enabled with one of your flags. Vincent > >> what about ading in all the EFL: >> >> m4_ifdef([v_rev], >> [ >> EFL_COMPILER_FLAG([-Wpointer-arith]) >> ]) >> >> to be sure we (that is not only you) don't miss them ? >> >> Vincent >> >>> >>> >>> >>> Author: raster >>> Date: 2010-12-10 19:57:05 -0800 (Fri, 10 Dec 2010) >>> New Revision: 55484 >>> Trac: http://trac.enlightenment.org/e/changeset/55484 >>> >>> Modified: >>> trunk/eina/src/lib/eina_strbuf_common.c >>> >>> Modified: trunk/eina/src/lib/eina_strbuf_common.c >>> =================================================================== >>> --- trunk/eina/src/lib/eina_strbuf_common.c 2010-12-11 03:44:33 UTC >>> (rev 55483) +++ trunk/eina/src/lib/eina_strbuf_common.c 2010-12-11 >>> 03:57:05 UTC (rev 55484) @@ -195,14 +195,15 @@ >>> return EINA_FALSE; >>> >>> /* move the existing text */ >>> - memmove(buf->buf + ((len + pos) * csize), buf->buf + (pos * csize), >>> + memmove(((unsigned char *)(buf->buf)) + ((len + pos) * csize), >>> + ((unsigned char *)(buf->buf)) + (pos * csize), >>> (buf->len - pos) * csize); >>> >>> /* and now insert the given string */ >>> memcpy(buf->buf + (pos * csize), str, len * csize); >>> >>> buf->len += len; >>> - memset(buf->buf + (buf->len * csize), 0, csize); >>> + memset(((unsigned char *)(buf->buf)) + (buf->len * csize), 0, csize); >>> return EINA_TRUE; >>> } >>> >>> @@ -305,7 +306,8 @@ >>> if (EINA_UNLIKELY(!_eina_strbuf_common_grow(csize, buf, buf->len + len))) >>> return EINA_FALSE; >>> >>> - memcpy(buf->buf + (buf->len * csize), str, (len + 1) * csize); >>> + memcpy(((unsigned char *)(buf->buf)) + (buf->len * csize), str, >>> + (len + 1) * csize); >>> buf->len += len; >>> return EINA_TRUE; >>> } >>> @@ -346,9 +348,10 @@ >>> if (EINA_UNLIKELY(!_eina_strbuf_common_grow(csize, buf, buf->len + len))) >>> return EINA_FALSE; >>> >>> - memcpy(buf->buf + (buf->len * csize), str, len * csize); >>> + memcpy(((unsigned char *)(buf->buf)) + (buf->len * csize), str, >>> + len * csize); >>> buf->len += len; >>> - memset(buf->buf + (buf->len * csize), 0, csize); >>> + memset(((unsigned char *)(buf->buf)) + (buf->len * csize), 0, csize); >>> return EINA_TRUE; >>> } >>> >>> @@ -382,9 +385,10 @@ >>> if (EINA_UNLIKELY(!_eina_strbuf_common_grow(csize, buf, buf->len + >>> length))) return EINA_FALSE; >>> >>> - memcpy(buf->buf + (buf->len * csize), str, length * csize); >>> + memcpy(((unsigned char *)(buf->buf)) + (buf->len * csize), str, >>> + length * csize); >>> buf->len += length; >>> - memset(buf->buf + (buf->len * csize), 0, csize); >>> + memset(((unsigned char *)(buf->buf)) + (buf->len * csize), 0, csize); >>> return EINA_TRUE; >>> } >>> >>> @@ -507,8 +511,8 @@ >>> if (EINA_UNLIKELY(!_eina_strbuf_common_grow(csize, buf, buf->len + 1))) >>> return EINA_FALSE; >>> >>> - memcpy(buf->buf + ((buf->len)++ *csize), c, csize); >>> - memset(buf->buf + (buf->len * csize), 0, csize); >>> + memcpy(((unsigned char *)(buf->buf)) + ((buf->len)++ *csize), c, csize); >>> + memset(((unsigned char *)(buf->buf)) + (buf->len * csize), 0, csize); >>> return EINA_TRUE; >>> } >>> >>> @@ -574,8 +578,8 @@ >>> } >>> >>> tail_len = buf->len - end + 1; /* includes '\0' */ >>> - memmove(buf->buf + (start * csize), >>> - buf->buf + (end * csize), >>> + memmove(((unsigned char *)(buf->buf)) + (start * csize), >>> + ((unsigned char *)(buf->buf)) + (end * csize), >>> tail_len * csize); >>> buf->len -= remove_len; >>> return _eina_strbuf_common_resize(csize, buf, buf->len); >>> @@ -747,14 +751,15 @@ >>> >>> } >>> >>> - memmove(buf->buf + pos + len2, buf->buf + pos + len1, >>> + memmove(((unsigned char *)(buf->buf)) + pos + len2, >>> + ((unsigned char *)(buf->buf)) + pos + len1, >>> buf->len - pos - len1); >>> } >>> >>> /* and now insert the given string */ >>> - memcpy(buf->buf + pos, with, len2); >>> + memcpy(((unsigned char *)(buf->buf)) + pos, with, len2); >>> buf->len += len2 - len1; >>> - memset((char *)buf->buf + buf->len, 0, 1); >>> + memset(((unsigned char *)(buf->buf)) + buf->len, 0, 1); >>> >>> return EINA_TRUE; >>> } >>> @@ -831,9 +836,11 @@ >>> } >>> >>> /* copy the untouched text */ >>> - memcpy(buf->buf + start, tmp_buf + start_tmp, pos - start); >>> + memcpy(((unsigned char *)(buf->buf)) + start, >>> + tmp_buf + start_tmp, pos - start); >>> /* copy the new string */ >>> - memcpy(buf->buf + pos, with, len2); >>> + memcpy(((unsigned char *)(buf->buf)) + pos, >>> + with, len2); >>> >>> /* calculate the next positions */ >>> start_tmp = pos_tmp + len1; >>> @@ -846,9 +853,10 @@ >>> pos = start + pos_tmp - start_tmp; >>> } >>> /* and now copy the rest of the text */ >>> - memcpy(buf->buf + start, tmp_buf + start_tmp, len - start); >>> + memcpy(((unsigned char *)(buf->buf)) + start, >>> + tmp_buf + start_tmp, len - start); >>> buf->len = len; >>> - memset((char *)buf->buf + buf->len, 0, 1); >>> + memset(((unsigned char *)(buf->buf)) + buf->len, 0, 1); >>> >>> free(tmp_buf); >>> >>> >>> >>> ------------------------------------------------------------------------------ >>> Oracle to DB2 Conversion Guide: Learn learn about native support for PL/SQL, >>> new data types, scalar functions, improved concurrency, built-in packages, >>> OCI, SQL*Plus, data movement tools, best practices and more. >>> http://p.sf.net/sfu/oracle-sfdev2dev >>> _______________________________________________ >>> enlightenment-svn mailing list >>> [email protected] >>> https://lists.sourceforge.net/lists/listinfo/enlightenment-svn >>> >>> >> >> ------------------------------------------------------------------------------ >> Oracle to DB2 Conversion Guide: Learn learn about native support for PL/SQL, >> new data types, scalar functions, improved concurrency, built-in packages, >> OCI, SQL*Plus, data movement tools, best practices and more. >> http://p.sf.net/sfu/oracle-sfdev2dev >> _______________________________________________ >> enlightenment-devel mailing list >> [email protected] >> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel >> > > > -- > ------------- Codito, ergo sum - "I code, therefore I am" -------------- > The Rasterman (Carsten Haitzler) [email protected] > > ------------------------------------------------------------------------------ Oracle to DB2 Conversion Guide: Learn learn about native support for PL/SQL, new data types, scalar functions, improved concurrency, built-in packages, OCI, SQL*Plus, data movement tools, best practices and more. http://p.sf.net/sfu/oracle-sfdev2dev _______________________________________________ enlightenment-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
