Closed Bug 1931710 Opened 3 months ago Closed 2 months ago

Use LocalFile directly rather via helper fopen in cZ

Categories

(SeaMonkey :: Chat, task)

Tracking

(seamonkey2.53+ fixed)

RESOLVED FIXED
Tracking Status
seamonkey2.53 + fixed

People

(Reporter: iannbugzilla, Assigned: iannbugzilla)

Details

(Whiteboard: SM2.53.20)

Attachments

(1 file)

At moment there are a mix of callers to the help fopen and others that just do "new LocalFile" directly.
Let's switch everything to use new LocalFile and remove fopen helper.
Whilst here remove the unused extra arguments for LocalFile.

[Approval Request Comment]
Regression caused by (bug #): n/a
User impact if declined: none
Testing completed (on m-c, etc.): 2.53.20pre
Risk to taking this patch (and alternatives if risky): low
String changes made by this patch: none

Attachment #9438154 - Flags: review?(frgrahl)
Attachment #9438154 - Flags: approval-comm-release?
Whiteboard: SM2.53.20

Comment on attachment 9438154 [details] [diff] [review]
1931710-irc-file-utils-fopen-25320.patch

lgtm r/a+

Attachment #9438154 - Flags: review?(frgrahl)
Attachment #9438154 - Flags: review+
Attachment #9438154 - Flags: approval-comm-release?
Attachment #9438154 - Flags: approval-comm-release+

Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/7ae5365f6e5d
Use LocalFile directly rather via helper fopen in cZ. r=frg

Status: ASSIGNED → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: