strnlen not available on GCC 4.4.2 (MacPorts)

[GoogleCodeExporter]

On GCC 4.4.2 on Mac OS X (Snow Leopard) installed via MacPorts, there
doesn't seem to be the strnlen function available. On MinGW it's the same
thing, so perhaps the nlen function in StringUtils.h should look something
like this (unfortunately I don't know, how the Compiler Macros a really
called, so I used some pseudo macros): 

FOG_INLINE sysuint_t nlen(const char* s, sysuint_t maxlen) 
{
#ifdef FOG_MINGW || (FOG_MAC && GCC_44)
  for(int i=0; i<maxlen; ++i) 
    if(s[i] == '\0') 
      return i;
  return -1;
#else
  return ::strnlen(s, maxlen);
#endif
}

Original issue reported on code.google.com by niels.pf...@gmail.com on 23 Nov 2009 at 12:04

[GoogleCodeExporter]

Accepted

Original comment by kobalicek.petr on 23 Nov 2009 at 10:55

• Changed state: Accepted
• Added labels: Priority-Critical
• Removed labels: Priority-Medium

[GoogleCodeExporter]

Fixed in my local copy, will be commited later

Original comment by kobalicek.petr on 27 Nov 2009 at 8:35

[GoogleCodeExporter]

Should be fixed, please verify

Original comment by kobalicek.petr on 12 Dec 2009 at 2:10

• Changed state: Fixed

[GoogleCodeExporter]

Ok, no problem anymore

Original comment by niels.pf...@gmail.com on 12 Dec 2009 at 2:44

[GoogleCodeExporter]

According to 
http://pubs.opengroup.org/onlinepubs/9699919799/functions/strlen.html the 
implementation of nlen should return maxlen rather -1 if null is not 
encountered.

Original comment by partow on 19 Dec 2010 at 7:10

[GoogleCodeExporter]

Since r259 nlen looks like that:
FOG_INLINE sysuint_t nlen(const char* str, sysuint_t maxlen)
{
  const char* p = str;
  if (!p) return 0;
  const char* end = str + maxlen;

  while (p < end && *p) p++;
  return (sysuint_t)(p - str);
}

That's ok, right?

Original comment by niels.pf...@gmail.com on 19 Dec 2010 at 8:13

• Added labels: Priority-Low
• Removed labels: Priority-Critical

[GoogleCodeExporter]

That's okay.

I think we should export these functions and make optimized non-inlined 
variants. After my commit I'd like to solve this.

I temporary reopened this to make this visible

Original comment by kobalicek.petr on 5 Feb 2011 at 1:59

• Changed state: Reopened

[GoogleCodeExporter]

This is definitely fixed, because Fog contains now own strnlen functionality. 

Original comment by kobalicek.petr on 10 Oct 2011 at 7:36

• Changed state: Fixed