update comments

bmalrat · bmalrat · commit eb6d7d2d4d5d · 2022-05-19T20:02:46.000-04:00
diff --git a/src/RunSql.cpp b/src/RunSql.cpp
@@ -190,13 +190,14 @@ bool RunSql::executeNextStatement()
         case SQLITE_ROW:
         {
             // If we get here, the SQL statement returns some sort of data. So hand it over to the model for display. Don't set the modified flag
-            // because statements that display data don't change data as well.
+            // because statements that display data don't change data as well, except if the statement are one of INSERT/UPDATE/DELETE that could
+            // return datas with the RETURNING keyword.
 
             releaseDbAccess();
 
             lk.lock();
 
-            // in case of usage of RETURNING keyword with INSERT/UPDATE/DELETE
+            // Set the modified flag to true if the statement was one of INSERT/UPDATE/DELETE triggered by a possible RETURNING keyword.
             if (query_part_type == StatementType::InsertStatement || query_part_type == StatementType::UpdateStatement || 
                 query_part_type == StatementType::DeleteStatement)
                 modified = true;
