Closed Bug 58698 Opened 20 years ago Closed 20 years ago

Solaris only: cmd_cut operation fails in optimize build

Categories

(Core :: DOM: Editor, defect, P3)

Sun
Solaris
defect

Tracking

()

VERIFIED FIXED
mozilla0.9

People

(Reporter: masaki.katakai, Assigned: kinmoz)

References

Details

Attachments

(1 file)

I can not delete one line or multiple selection on Mail Composer (TextEdit mode)
on Solaris optimize build. Please try the following operations,

1) Start Mozilla Mail
2) Set TextEdit mode
3) Select a message
4) Reply
5) On Compose window, make a selection for one line by double click
6) Press Back Space

I expect the line is deleted. However, it fails on only Solaris build.
I never seen on Linux and Windows.

It seems that DeleteRangeTxn.cpp in editor/base/src does not
initialize nsresult result. It works in debug build, however,
it fails in optimize build on Solaris.

Due to this problem, "cmd_cut" operation fails on Solaris as well.
It's serious.

Please try the patch. I'll attach the simple patch.
Assigning to kin, who knows about this problem. Thanks!
Assignee: beppe → kin
r=kin@netscape.com  and sr=sfraser@netscape.com on the 10/31/00 21:52 patch.

I'll probably check this into the Trunk for katakai@japan.sun.com tomorrow morning.
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9
Blocks: 59652
Fix from katakai@japan.sun.com, checked into Trunk:

    mozilla/editor/base/DeleteRangeTxn.cpp  revision 1.54

This bug still exists on the Netscape_20000922_BRANCH.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
KAtakai, please verify this bug and mark VERIFIED-FIXED...thanks!
Thank you very much for integration. I'll verify on Monday.
KAtakai please mark verified-fixed...thanks.
Sorry for late. I've verified the fix on 2000111509 Solaris build.
Thank you for fix! Please mark it as VERIFIED.
marking verified
Status: RESOLVED → VERIFIED
Blocks: 60740
The same fix has been applied to the OEM branch:  Netscape_6_0_OEM_BRANCH.
a=jdunn@netscape.com.
You need to log in before you can comment on or make changes to this bug.