Closed Bug 225994 Opened 21 years ago Closed 21 years ago

DRefTool analysis for universalchardet files

Categories

(Core :: Internationalization, defect)

x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: timeless, Assigned: timeless)

Details

Attachments

(1 file, 1 obsolete file)

void nsUniversalDetector::HandleData(const char* aBuf, PRUint32 aLen)

What should i do when it fails?
Status: NEW → ASSIGNED
Attachment #144646 - Flags: review?(smontagu)
Comment on attachment 144646 [details] [diff] [review]
make handledata return rv's. -uwp

do you want to fix this call site too?
http://lxr.mozilla.org/seamonkey/source/extensions/universalchardet/src/nsUnive
rsalDetector.cpp#383
Attachment #144646 - Flags: review?(smontagu)
Attachment #144646 - Attachment is obsolete: true
Attachment #146662 - Flags: review?(smontagu)
Comment on attachment 146662 [details] [diff] [review]
make handledata return rv's -uwp, and propagate them

>@@ -203,8 +203,11 @@ void nsUniversalDetector::HandleData(con
>   switch (mInputState)
>   {
>   case eEscAscii:
>-    if (nsnull == mEscCharSetProber)
>+    if (nsnull == mEscCharSetProber) {
>       mEscCharSetProber = new nsEscCharSetProber;
>+      if (!mEscCharSetProber)
>+        return NS_ERROR_OUT_OF_MEMORY;

Nit: change |if (!mEscCharSetProber)| to if (nsnull == mEscCharSetProber)| (or
vice versa, I suppose)

r=smontagu.
Attachment #146662 - Flags: review?(smontagu) → review+
Attachment #146662 - Flags: superreview?(roc)
Attachment #146662 - Flags: superreview?(roc) → superreview+
mozilla/extensions/universalchardet/src/nsUniversalDetector.cpp 	1.18
mozilla/extensions/universalchardet/src/nsUniversalDetector.h 	1.9
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: