Closed Bug 783676 Opened 12 years ago Closed 12 years ago

Export OS.Path

Categories

(Toolkit Graveyard :: OS.File, enhancement)

Product:
Toolkit Graveyard
Component:
OS.File
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
mozilla17

People

(Reporter: Yoric, Assigned: Yoric)

References

Details

(Keywords: dev-doc-needed)

Attachments

(3 files, 4 obsolete files)

Bugfix: normalization of a/../..
(deleted), patch
Yoric
: review+
Details | Diff | Splinter Review
Exporting OS.Path itself
(deleted), patch
Yoric
: review+
Details | Diff | Splinter Review
Companion testsuite
(deleted), patch
Yoric
: review+
Details | Diff | Splinter Review
At the moment, we have a OS.Unix.Path under Unix, a OS.Win.Path under Windows, but not a OS.Path. We should fix this.
Attached patch Bugfix: normalization of a/../.. (obsolete) (deleted) — — Splinter Review
Assignee: nobody → dteller
Attachment #652897 - Flags: review?(taras.mozilla)
Attached patch Companion testsuite (obsolete) (deleted) — — Splinter Review
Let's add some tests to the mix
Attachment #652900 - Flags: review?(taras.mozilla)
Attached patch Exporting OS.Path itself (obsolete) (deleted) — — Splinter Review
Attachment #652903 - Flags: review?(taras.mozilla)
Well, code turned out slightly longer than I expected, but still reasonably trivial.
Comment on attachment 652897 [details] [diff] [review] Bugfix: normalization of a/../.. rubberstamp
Attachment #652897 - Flags: review?(taras.mozilla) → review+
Attachment #652900 - Flags: review?(taras.mozilla) → review+
Attachment #652903 - Flags: review?(taras.mozilla) → review+
Attachment #652897 - Attachment is obsolete: true
Attachment #653795 - Flags: review+
Attached patch Companion testsuite (obsolete) (deleted) — — Splinter Review
Attachment #652900 - Attachment is obsolete: true
Attachment #653796 - Flags: review+
Attached patch Exporting OS.Path itself (deleted) — — Splinter Review
Attachment #652903 - Attachment is obsolete: true
Attachment #653799 - Flags: review+
Attached patch Companion testsuite (deleted) — — Splinter Review
Fixed a typo in the testsuite.
Attachment #653796 - Attachment is obsolete: true
Attachment #654542 - Flags: review+
Product: Toolkit → Toolkit Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: