Closed Bug 783676 Opened 7 years ago Closed 7 years ago

Export OS.Path

Categories

(Toolkit :: OS.File, enhancement)

enhancement
Not set

Tracking

()

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+
Attachment #652897 - Attachment is obsolete: true
Attachment #653795 - Flags: 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+
You need to log in before you can comment on or make changes to this bug.