Closed Bug 375878 Opened 17 years ago Closed 17 years ago

Remove users of ::Recycle

Categories

(Core :: XPCOM, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: Biesinger, Assigned: taras.mozilla)

References

()

Details

Attachments

(1 file)

225  * Deprecated: don't use |Recycle|, just call |nsMemory::Free| directly
226  *
227  * Return the given buffer to the heap manager. Calls allocator::Free()
228  */
229 inline void Recycle( char* aBuffer) { nsMemory::Free(aBuffer); }
230 inline void Recycle( PRUnichar* aBuffer) { nsMemory::Free(aBuffer); }

seems like we should remove them. (nsMemory::Free has since been deprecated by NS_Free)
Here is my first stab at it.
Attachment #260075 - Flags: superreview+
Attachment #260075 - Flags: review+
Assignee: nobody → tglek
the search parts were fixed by Bug 374862; I checked the rest in:

Checking in editor/libeditor/html/nsHTMLEditor.cpp;
/cvsroot/mozilla/editor/libeditor/html/nsHTMLEditor.cpp,v  <--  nsHTMLEditor.cpp
new revision: 1.548; previous revision: 1.547
done
Checking in netwerk/streamconv/converters/nsDirIndexParser.cpp;
/cvsroot/mozilla/netwerk/streamconv/converters/nsDirIndexParser.cpp,v  <--  nsDirIndexParser.cpp
new revision: 1.21; previous revision: 1.20
done
Checking in rdf/datasource/src/nsFileSystemDataSource.cpp;
/cvsroot/mozilla/rdf/datasource/src/nsFileSystemDataSource.cpp,v  <--  nsFileSystemDataSource.cpp
new revision: 1.147; previous revision: 1.146
done
Checking in security/manager/ssl/src/nsNSSCallbacks.cpp;
/cvsroot/mozilla/security/manager/ssl/src/nsNSSCallbacks.cpp,v  <--  nsNSSCallbacks.cpp
new revision: 1.48; previous revision: 1.47
done
Checking in xpinstall/src/nsInstallExecute.cpp;
/cvsroot/mozilla/xpinstall/src/nsInstallExecute.cpp,v  <--  nsInstallExecute.cpp
new revision: 1.33; previous revision: 1.32
done
Checking in xpinstall/src/nsInstallFile.cpp;
/cvsroot/mozilla/xpinstall/src/nsInstallFile.cpp,v  <--  nsInstallFile.cpp
new revision: 1.71; previous revision: 1.70
done
Checking in xpinstall/src/nsInstallUninstall.cpp;
/cvsroot/mozilla/xpinstall/src/nsInstallUninstall.cpp,v  <--  nsInstallUninstall.cpp
new revision: 1.27; previous revision: 1.26
done
Checking in xpinstall/src/nsJSInstall.cpp;
/cvsroot/mozilla/xpinstall/src/nsJSInstall.cpp,v  <--  nsJSInstall.cpp
new revision: 1.125; previous revision: 1.124
done
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Component: String → XPCOM
You need to log in before you can comment on or make changes to this bug.