D9690: Add contacts-reading plugin (Android side)

Simon Redman noreply at phabricator.kde.org
Thu May 31 00:08:10 UTC 2018


sredman marked 2 inline comments as done.
sredman added inline comments.

INLINE COMMENTS

> philipc wrote in ContactsHelper.java:117
> Couldn't we create these two streams in a try-with-resources block? I believe Android Studio has a quick fix for that

It looks like we could, but try-with-resources requires API level 19 (current min is 9)

REPOSITORY
  R225 KDE Connect - Android application

REVISION DETAIL
  https://phabricator.kde.org/D9690

To: sredman, #kde_connect, mtijink
Cc: philipc, kdeconnect, nicolasfella, andyholmes, mtijink, yannux, Danial0_0, johnq, Pitel, adeen-s, SemperPeritus, ndavis, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, menasshock, ach, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20180531/04d987ff/attachment.html>


More information about the KDEConnect mailing list