Bommels05/Mozilla
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Don't use memcpy for single-character Replace/Assign since it performs worse than simple assignment. Bug 312681, r+sr=darin.

Browse Source 
git-svn-id: svn://10.0.0.236/trunk@182378 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
bryner%brianryner.com
2005-10-17 16:28:21 +00:00
parent 01f4b5e853
commit 4027bc91f8
2 changed files with 22 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
mozilla/xpcom/string/src/nsTSubstring.cpp
View File

@@ -283,6 +283,15 @@ nsTSubstring_CharT::EnsureMutable()
// ---------------------------------------------------------------------------
// This version of Assign is optimized for single-character assignment.
void
nsTSubstring_CharT::Assign( char_type c )
{
if (ReplacePrep(0, mLength, 1))
*mData = c;
}
void
nsTSubstring_CharT::Assign( const char_type* data, size_type length )
{
@@ -421,6 +430,17 @@ nsTSubstring_CharT::Adopt( char_type* data, size_type length )
}
// This version of Replace is optimized for single-character replacement.
void
nsTSubstring_CharT::Replace( index_type cutStart, size_type cutLength, char_type c )
{
cutStart = PR_MIN(cutStart, Length());
if (ReplacePrep(cutStart, cutLength, 1))
mData[cutStart] = c;
}
void
nsTSubstring_CharT::Replace( index_type cutStart, size_type cutLength, const char_type* data, size_type length )
{