Closed Bug 319709 Opened 19 years ago Closed 19 years ago

Unneeded allocations in mozMySpell::Suggest

Categories

(Core :: Spelling checker, defect)

x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: bzbarsky, Assigned: mscott)

References

Details

The following code:

349       PRUnichar *dest = (PRUnichar *)malloc(sizeof(PRUnichar) * (outLength + 1));
...
357       // XXX ewwww.
358       tmpPtr[i] = ToNewUnicode(nsDependentString(dest));
359       free(dest);

doesn't make sense to me.  Why not just allocate |dest| using nsMemory and assign it to tmpPtr[i] directly?
Depends on: 307052
fixed by 307052
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.