Actually, SQLite AUTOINCREMENT ids start at 1, so 0 is definitely an …

…error
apsun · Aug 22, 2021 · bc5f045 · bc5f045
1 parent a6dadca
commit bc5f045
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/app/src/main/java/com/crossbowffs/nekosms/loader/InboxSmsLoader.java b/app/src/main/java/com/crossbowffs/nekosms/loader/InboxSmsLoader.java
@@ -40,10 +40,8 @@ public static Uri writeMessage(Context context, SmsMessageData messageData) {
             id = ContentUris.parseId(uri);
         }
 
-        // An ID of 0 when writing to the SMS inbox could mean we don't have the
+        // An ID of 0 when writing to the SMS inbox means we don't have the
         // OP_WRITE_SMS permission. See ContentProvider#rejectInsert(Uri, ContentValues).
-        // Another explanation would be that this is actually the first message written.
-        // Since it's unlikely to be the latter, assume the write failed.
         if (id <= 0) {
             Xlog.e("Writing to SMS inbox failed");
             throw new DatabaseException("Failed to write message to SMS inbox");