Closed Bug 1105308 Opened 5 years ago Closed 5 years ago

[Bluetooth] Cleanup BluetoothUtils.{cpp,h}

Categories

(Firefox OS Graveyard :: Bluetooth, defect)

All
Gonk (Firefox OS)
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: tzimmermann, Assigned: tzimmermann)

References

Details

Attachments

(3 files)

There are two files named BluetoothUtils.{cpp,h}; for BlueZ and Bluedroid backends respectively. The content of these files is mostly the same.

We should merge the backend-independent code into a single file and move it out of the backend directories.
Comment on attachment 8529114 [details] [diff] [review]
[01] Bug 1105308: Cleanup non-shared functions in BlueZ' BluetoothUtils.{cpp,h}

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

LGTM.
Attachment #8529114 - Flags: review?(btian) → review+
Comment on attachment 8529115 [details] [diff] [review]
[02] Bug 1105308: Cleanup non-shared functions in Bluedroid's BluetoothUtils.{cpp,h}

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

LGTM.
Attachment #8529115 - Flags: review?(btian) → review+
Comment on attachment 8529116 [details] [diff] [review]
[03] Bug 1105308: Merge BluetoothUtils.{cpp,h}

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

r=me with nit addressed.

::: dom/bluetooth/BluetoothUtils.h
@@ +4,5 @@
>   * License, v. 2.0. If a copy of the MPL was not distributed with this file,
>   * You can obtain one at http://mozilla.org/MPL/2.0/. */
>  
> +#ifndef mozilla_dom_bluetooth_bluetoothutils_h
> +#define mozilla_dom_bluetooth_bluetoothutils_h

nit: add back '__' at the end.
Attachment #8529116 - Flags: review?(btian) → review+
(In reply to Ben Tian [:btian] from comment #6)
> Comment on attachment 8529116 [details] [diff] [review]
> [03] Bug 1105308: Merge BluetoothUtils.{cpp,h}
> 
> Review of attachment 8529116 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> r=me with nit addressed.
> 
> ::: dom/bluetooth/BluetoothUtils.h
> @@ +4,5 @@
> >   * License, v. 2.0. If a copy of the MPL was not distributed with this file,
> >   * You can obtain one at http://mozilla.org/MPL/2.0/. */
> >  
> > +#ifndef mozilla_dom_bluetooth_bluetoothutils_h
> > +#define mozilla_dom_bluetooth_bluetoothutils_h
> 
> nit: add back '__' at the end.

May I ask why? The underscores violate Mozilla's style guide.
(In reply to Thomas Zimmermann [:tzimmermann] [:tdz] from comment #7)
> > ::: dom/bluetooth/BluetoothUtils.h
> > @@ +4,5 @@
> > >   * License, v. 2.0. If a copy of the MPL was not distributed with this file,
> > >   * You can obtain one at http://mozilla.org/MPL/2.0/. */
> > >  
> > > +#ifndef mozilla_dom_bluetooth_bluetoothutils_h
> > > +#define mozilla_dom_bluetooth_bluetoothutils_h
> > 
> > nit: add back '__' at the end.
> May I ask why? The underscores violate Mozilla's style guide.

You are right. I just realize all other headers should be fixed as well. Please keep the change and open another follow-up bug to revise all headers under bluetooth/ and bluetooth2/ folders. Thanks.
OK, done so: bug 1106007.
You need to log in before you can comment on or make changes to this bug.