Closed Bug 1225340 Opened 6 years ago Closed 6 years ago

Followup of bug 1223806: Refine nullity check macros in BluetoothServiceBluedroid

Categories

(Firefox OS Graveyard :: Bluetooth, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(firefox45 fixed)

RESOLVED FIXED
2.6 S2 - 12/4
Tracking Status
firefox45 --- fixed

People

(Reporter: ben.tian, Assigned: ben.tian)

References

Details

Attachments

(1 file, 3 obsolete files)

Per bug 1223806 comment 12 change, macros in [1] should be revised to match each method's requirement.

[1] https://dxr.mozilla.org/mozilla-central/source/dom/bluetooth/bluedroid/BluetoothServiceBluedroid.cpp#39
Depends on: 1223806
Summary: Refine nullity check macros in BluetoothServiceBluedroid → Followup of bug 1223806: Refine nullity check macros in BluetoothServiceBluedroid
Assignee: nobody → btian
Rebase on the latest m-c.
Attachment #8688215 - Attachment is obsolete: true
Restore GATT macro part.
Attachment #8689872 - Attachment is obsolete: true
Attachment #8689875 - Flags: review?(tzimmermann)
Comment on attachment 8689875 [details] [diff] [review]
Patch 1 (v3): Refine nullity check macros in BluetoothServiceBluedroid

Review of attachment 8689875 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/bluetooth/bluedroid/BluetoothServiceBluedroid.cpp
@@ +35,5 @@
>  #include "mozilla/StaticMutex.h"
>  #include "mozilla/StaticPtr.h"
>  #include "mozilla/unused.h"
>  
> +#define ENSURE_BLUETOOTH_ENABLED(runnable, result)                     \

Maybe |ENSURE_BLUETOOTH_IS_ENABLED| (?) That sounds more natural to me.
Attachment #8689875 - Flags: review?(tzimmermann) → review+
Revise per reviewer's comment.
Attachment #8689875 - Attachment is obsolete: true
https://hg.mozilla.org/mozilla-central/rev/f92df94073a3
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.6 S2 - 12/4
You need to log in before you can comment on or make changes to this bug.