Closed
Bug 43897
Opened 25 years ago
Closed 25 years ago
xpti does not correctly handle .xpt file rename
Categories
(Core :: XPConnect, defect, P3)
Tracking
()
RESOLVED
FIXED
People
(Reporter: jband_mozilla, Assigned: jband_mozilla)
Details
Attachments
(1 file)
|
7.82 KB,
patch
|
Details | Diff | Splinter Review |
My bad. Pointed out by Bill Law:
news://news.mozilla.org/395198AB.2E2263B4%40netscape.com
|
Assignee | |
Comment 1•25 years ago
|
||
|
|
Assignee | |
Comment 2•25 years ago
|
||
Added a patch to fix the bug. It has two parts: 1) It was failing to detect the
renamed file at all because the "if(k == countOfFilesInWorkingSet)" was in the
wrong block :( ...and... 2) the workingset merge function was not setup to deal
with infos which had previously been read from the manifest as being in one file
actually appearing in another.
McCabe, can you review?
|
||
Comment 3•25 years ago
|
||
r=mccabe. The problem was that 'else' ?
|
|
Assignee | |
Updated•25 years ago
|
Whiteboard: [fix in hand]
|
|
Assignee | |
Updated•25 years ago
|
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 4•25 years ago
|
||
fix checked in
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Whiteboard: [fix in hand]
You need to log in
before you can comment on or make changes to this bug.
Description
•