diff options
| author | Takeshi Abe <tabe@fixedpoint.jp> | 2012-04-18 00:23:42 +0900 |
|---|---|---|
| committer | Takeshi Abe <tabe@fixedpoint.jp> | 2012-04-18 00:24:17 +0900 |
| commit | b1bee56af9a4cbbaabfe43290d28f53efdebc14d (patch) | |
| tree | 483e9115f98baf1b69b20397852cb50e77028864 /basctl | |
| parent | f00d60997edc56bf0f4653b1bd78c5379cedda6a (diff) | |
calling GetChar() only once suffices
Diffstat (limited to 'basctl')
| -rw-r--r-- | basctl/source/basicide/basobj2.cxx | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/basctl/source/basicide/basobj2.cxx b/basctl/source/basicide/basobj2.cxx index ddbc3a4c63bc..cd9986ccff87 100644 --- a/basctl/source/basicide/basobj2.cxx +++ b/basctl/source/basicide/basobj2.cxx @@ -108,10 +108,11 @@ sal_Bool IsValidSbxName( const String& rName ) { for ( sal_uInt16 nChar = 0; nChar < rName.Len(); nChar++ ) { - sal_Bool bValid = ( ( rName.GetChar(nChar) >= 'A' && rName.GetChar(nChar) <= 'Z' ) || - ( rName.GetChar(nChar) >= 'a' && rName.GetChar(nChar) <= 'z' ) || - ( rName.GetChar(nChar) >= '0' && rName.GetChar(nChar) <= '9' && nChar ) || - ( rName.GetChar(nChar) == '_' ) ); + sal_Unicode c = rName.GetChar(nChar); + sal_Bool bValid = ( ( c >= 'A' && c <= 'Z' ) || + ( c >= 'a' && c <= 'z' ) || + ( c >= '0' && c <= '9' && nChar ) || + ( c == '_' ) ); if ( !bValid ) return sal_False; } |