Proposal: Rename GattConnection interface to something else

GattConnection is not the ideal name for 3 reasons:

• Bluetooth or Ble is missing from its name, while it’s exclusively for Bluetooth General ATTribute.
• There’s “Connection” in the name, but when an instance is originally created, the connection is not established yet.
• There’s “Connection” in the name, but there’s a disconnect() function… should Connection be magically dropped from the name of the class when this function is called? Of course, this is impossible, but shows how the name is not an ideal fit.

Consequently, I want this class to be renamed when the first Kotlin 1.3 compatible version is released, with a deprecated typealias on initial pre-release (alpha, beta or rc) for source compatibility.

Here are a few name ideas:

1. BleGatt
2. SuspendBluetoothGatt
3. SuspendBleGatt
4. BluetoothGatt (with named import to reference Android’s as AndroidBluetoothGatt or alike)
5. BluetoothGattIsFun (wait… maybe not actual fun yet 🤔)
6. FunBluetoothGatt (this could open the discussion of a library renaming)

Please, add a comment to say what you think about each proposition (I may append more name ideas, but I will keep the numbers stable), and if you have other name ideas, please write them there too.

[JoeBakalor]

Short-link was in regards to Bluetooth classic though not Bluetooth Low Energy

Sent from my iPhone

On Dec 17, 2018, at 8:50 PM, Eric [e-mail] wrote:

Hi, I’m passing through 😃 Given these options I agree with @bj0’s favor/caution with regard to BleGatt. I also see on Wikipedia that Bluetooth was first called “short-link” radio technology.

How does interface ShortLink sound to you?

internal class ShortLinkImpl( private val bluetoothDevice: BluetoothDevice, private val connectionSettings: ShortLink.ConnectionSettings ) : ShortLink, CoroutineScope — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

[odahcam]

You could create a GattConnectionBuilder that returns the GattConnection, it would give time for ya to think in a library rename.