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

Fixed bug in nsString::Cut code where the sizeof(chartype) was multipled a bit aggresively; I also tuned the routine somewhat as well

Browse Source 
git-svn-id: svn://10.0.0.236/trunk@2069 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
kipp
1998-05-20 23:18:25 +00:00
parent e80bba2116
commit a9dcc1b0ed
4 changed files with 36 additions and 28 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
mozilla/string/obsolete/nsString.cpp
View File

@@ -1013,16 +1013,18 @@ PRInt32 nsString::Insert(PRUnichar aChar,PRInt32 anOffset){
* @param aCount -- number of chars to be cut
* @return *this
*------------------------------------------------------*/
nsString& nsString::Cut(PRInt32 anOffset,PRInt32 aCount) {
if((anOffset>=0) && (anOffset<mLength)) {
nsString&
nsString::Cut(PRInt32 anOffset, PRInt32 aCount)
{
if (PRUint32(anOffset) < PRUint32(mLength)) {
PRInt32 spos=anOffset+aCount;
PRInt32 delcnt=(spos<mLength) ? aCount : mLength-anOffset;
if(spos<mLength) {
nsCRT::memmove(&mStr[anOffset],&mStr[spos],sizeof(chartype)*mLength-spos);
mStr[mLength-aCount]=0;
if (spos < mLength) {
nsCRT::memmove(&mStr[anOffset], &mStr[spos],
sizeof(chartype) * (mLength - spos));
}
else mStr[anOffset]=0;
mLength-=delcnt;
mLength -= delcnt;
mStr[mLength] = 0; // restore zero terminator
}
return *this;
}