Skip to content
This repository has been archived by the owner on Nov 3, 2021. It is now read-only.

Bug 860696: add bt test commands. r=echou #57

Merged
merged 4 commits into from
Mar 25, 2014

Conversation

vicamo
Copy link

@vicamo vicamo commented Feb 18, 2014

No description provided.

"'bt remote add [<bd_addr>]':\r\n"
"Add a remote device to the scatternet where the device specified by <bd_addr>\r\n"
"lives and return the address of the newly created device. If <bd_addr> is\r\n"
"omitted, Bluetooth LOCAL address ff:ff:ff:00:00:00 is assumed.\r\n",
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Allowing to specify a <bd_addr> here is kind of misleading because all operations should be limited in the same scatternet now. No matter what address we have here, except those invalid ones, we add a new remote device in current scatternet anyway. So why bother? I'm going to remove it in the next revision.

@vicamo
Copy link
Author

vicamo commented Feb 18, 2014

  • bt remote add should not take any parameter, at least for now.
  • correct commit summary of part 4/4

@vicamo
Copy link
Author

vicamo commented Feb 20, 2014

Somehow I can't see your comments. Could you comment directly on https://github.com/mozilla-b2g/platform_external_qemu/pull/57/files ?

@vicamo
Copy link
Author

vicamo commented Feb 20, 2014

@eric30, Somehow I can't see your comments. Could you comment directly on https://github.com/mozilla-b2g/platform_external_qemu/pull/57/files ?


static int
do_bt_remote_remove_device( ControlClient client, struct bt_device_s *dev,
int fatal )
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A bool would be enough?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bool is not welcome in *.c.

@vicamo
Copy link
Author

vicamo commented Feb 24, 2014

Rebase and address above review comments.

vicamo added a commit that referenced this pull request Mar 25, 2014
Bug 860696: add bt test commands. r=echou
@vicamo vicamo merged commit 12408eb into mozilla-b2g:master Mar 25, 2014
@vicamo vicamo deleted the bugzilla/860696/bt-test-commands branch March 25, 2014 10:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants