Hi Marcel, >> + void Disable() >> + >> + Shudown SAP server and remove the SDP record. >> + >> + Possible errors: org.bluez.Error.Failed > >I don't like this. If you have properties then just changing >the property should be enough. So a SetProperty is more appropriate. I see another option to get rid of 'Enabled' property and leave the methods. What would you say on that? >> + >> + void Disconnect(boolean type) >> + >> + Disconnect SAP client from the server. >The 'type' >> + parameter indicates disconnection type. >> + >> + True - gracefull disconnection >> + False - immediate disconnection >> + >> + Possible errors: org.bluez.Error.Failed > >I don't like this style of method names at all. Using method >names like GracefulDisconnect and ImmediateDisconnect would be better. That's fine. >However I am not sure we should differentiate here at all. We >should always to the graceful disconnect. What will the >immediate disconnect bring us? That's actually intended for testing only. One of PTS test cases expects the tester to trigger immediate disconnect. In practce, it is only used when connection to sim card is lost, but this is obviously done internally. Thanks, /Waldek-- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
