Closed Bug 783676 Opened 12 years ago Closed 12 years ago

Export OS.Path

Categories

(Toolkit Graveyard :: OS.File, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla17

People

(Reporter: Yoric, Assigned: Yoric)

References

Details

(Keywords: dev-doc-needed)

Attachments

(3 files, 4 obsolete files)

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) — Splinter Review
Assignee: nobody → dteller
Attachment #652897 - Flags: review?(taras.mozilla)
Attached patch Companion testsuite (obsolete) — Splinter Review
Let's add some tests to the mix
Attachment #652900 - Flags: review?(taras.mozilla)
Attached patch Exporting OS.Path itself (obsolete) — 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+
Attached patch Companion testsuite (obsolete) — Splinter Review
Attachment #652900 - Attachment is obsolete: true
Attachment #653796 - Flags: review+
Attachment #652903 - Attachment is obsolete: true
Attachment #653799 - Flags: 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

Created:
Updated:
Size: