Closed Bug 780062 Opened 12 years ago Closed 12 years ago

Alarm API - Follow-up fix for bug 776785 (better to use long in IDL)

Categories

(Core :: DOM: Device Interfaces, defect)

defect
Not set
normal

Tracking

()

RESOLVED WONTFIX

People

(Reporter: airpingu, Assigned: airpingu)

References

Details

Attachments

(1 file, 1 obsolete file)

Blocks: 776758
Component: Hardware Abstraction Layer (HAL) → DOM: Device Interfaces
Attached patch Patch (obsolete) — Splinter Review
Hi Justin again :)

According to Ms2ger's saying, it seems better to use |long|s in IDL files? This follow-up addresses this issue. As your discover, that's also confusing to me that the existing codes are using both |long|s and |PRInt32|, but it seems using |long|s is in the majority in dom/. So let's follow the convention. ;)

Thanks for your review again (and check-in). :) It was my bad to do the extra changes in the IDL.
Assignee: nobody → clian
Attachment #648591 - Flags: review?(justin.lebar+bug)
Flags: in-testsuite-
Blocks: 776785
No longer blocks: 776758
Summary: Alarm API - Follow-up fix for bug 776758 (better to use long in IDL) → Alarm API - Follow-up fix for bug 776785 (better to use long in IDL)
Attached patch PatchSplinter Review
Ah, sorry for typing the wrong title. Please see comment #1 for the notes.
Attachment #648591 - Attachment is obsolete: true
Attachment #648591 - Flags: review?(justin.lebar+bug)
Attachment #648593 - Flags: review?(justin.lebar+bug)
> It was my bad to do the extra changes in the IDL.

No, I should have caught it in the review.

I'm not actually sure what we should do here.  This patch is probably correct, but I've posted to dev-platform for clarification.  Once we figure things out there, we can decide what to do with this patch.

https://groups.google.com/forum/?fromgroups#!topic/mozilla.dev.platform/ORQNJ-z9log
Comment on attachment 648593 [details] [diff] [review]
Patch

Based on the discussion on the mailing list, I think the consensus was against deprecating PRInt32 inside XPIDL in favor of long.  So I'm clearing the r? on this patch.

Gene, if you personally prefer long, r=me on the patch; just let me know and I'll push it for you.
Attachment #648593 - Flags: review?(justin.lebar+bug)
(In reply to Justin Lebar [:jlebar] from comment #4)
> Comment on attachment 648593 [details] [diff] [review]
> Patch
> 
> Based on the discussion on the mailing list, I think the consensus was
> against deprecating PRInt32 inside XPIDL in favor of long.  So I'm clearing
> the r? on this patch.

I'm kind of confused about the phrase here. You mean 

against deprecating (v.) PRInt32
or 
against deprecating (adj.) PRInt32

> 
> Gene, if you personally prefer long, r=me on the patch; just let me know and
> I'll push it for you.

No worries! If the PRInt32 is better to use in theory, I'm definitely OK with it! :) So directly close this issue?
"deprecating" is a gerund and thus a noun.  I'm not sure what you mean.  :)

> No worries! If the PRInt32 is better to use in theory, I'm definitely OK with it! :)

Basically, I prefer PRInt32, Ms2ger prefers long, and the mailing list did not seem to think that PRInt32 was so bad that you can't use it.  So whatever you prefer (closing this bug or checking it in) is fine with me.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
I think you mean WONTFIX?
Resolution: FIXED → WONTFIX
(In reply to Justin Lebar [:jlebar] from comment #7)
> I think you mean WONTFIX?

OK. Thank you! :)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: