U++ forum: Welcome to the forum

Search on this site

Search in forums

Home » U++ Library support » U++ Core » IsUpper bug?

Show: Today's Messages :: Show Polls :: Message Navigator
E-mail to friend

IsUpper bug? [message #9543]

Wed, 16 May 2007 09:00

unodgs
Messages: 1366
Registered: November 2005
Location: Poland

Ultimate Contributor

I discovered that IsUpper for '_' return false. Because of that my function:

bool IsUpperString(const char *q)
{
	while(*q)
		if(!IsUpper(*q++))
		   return false;
	return true;
}

did not work well for strings like NUMBER_10_AND_20 and I had to modify it to:

bool IsUpperString(const char *q)
{
	while(*q)
	{
		if(*q != '_' && !IsUpper(*q))
		   return false;
		q++;
	}
	return true;
}

Report message to a moderator

Re: IsUpper bug? [message #9549 is a reply to message #9543]

Wed, 16 May 2007 11:55

mirek
Messages: 13975
Registered: November 2005

Ultimate Member

unodgs wrote on Wed, 16 May 2007 03:00

I discovered that IsUpper for '_' return false. Because of that my function:

bool IsUpperString(const char *q)
{
	while(*q)
		if(!IsUpper(*q++))
		   return false;
	return true;
}

did not work well for strings like NUMBER_10_AND_20 and I had to modify it to:

bool IsUpperString(const char *q)
{
	while(*q)
	{
		if(*q != '_' && !IsUpper(*q))
		   return false;
		q++;
	}
	return true;
}

Of what letter is '_' uppercase variant?

I think the check should look like this

if(IsLetter(*q) && IsUpper(*q))

Mirek

Report message to a moderator

Re: IsUpper bug? [message #9550 is a reply to message #9549]

Wed, 16 May 2007 12:30

unodgs
Messages: 1366
Registered: November 2005
Location: Poland

Ultimate Contributor

I just thought that it would be better if IsUpper returned true for non-letter chars. That could simplify algorithms like this one for IsUpperString. On the other hand it is true that underscore or numebers have not upper case variants so IsUpper returns false in such case. Ok let's leave it as it is.

Report message to a moderator

Previous Topic:	Compile error "thread-local storage not supported ..."
Next Topic:	Adding subclass objects to base class array using AppendPick()

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

] [

]

Current Time: Sat May 04 22:12:22 CEST 2024

Total time taken to generate the page: 0.02788 seconds