Change theme
Help
Press space for more information.
Show links for this issue (Shortcut: i, l)
Copy issue ID
Previous Issue (Shortcut: k)
Next Issue (Shortcut: j)
Sign in to use full features.
Vote: I am impacted
Notification menu
Refresh (Shortcut: Shift+r)
Go home (Shortcut: u)
Pending code changes (auto-populated)
View issue level access limits(Press Alt + Right arrow for more information)
Unintended behavior
View staffing
Description
uwb library has two places right now where it uses Log.e (see below). In once case it silently ignores that failure and in second it throws a generic RuntimeException.
Instead of doing that, please have proper error handling (seehttps://g3doc.corp.google.com/company/teams/android-api-council/guidelines/index.md?cl=head#return-error ). We should only throw exceptions when it is the library users' fault in how they use the library.
Additionally, we should also really avoid using Log.i as well, it is logcat spanning without much additional help. If you really need it for local iteration, added
const val DEBUG = false
and flag check all these printouts through out the library.