tp: make sure to delete tables in reverse order of creation
This CL changes the dropping of tables to be in reverse order of
creation. This is important as, if we have "reusable" statements in
table functions referencing an underlying table/table function, dropping
the tables in the wrong order causes SQLite to get very confused leading
to crashes.
Bug: 311229092
Change-Id: Icfc9dffefe57231032855c4442452cbc29188a80
diff --git a/src/trace_processor/sqlite/sqlite_table.h b/src/trace_processor/sqlite/sqlite_table.h
index 3f7c3bd..d6d40b3 100644
--- a/src/trace_processor/sqlite/sqlite_table.h
+++ b/src/trace_processor/sqlite/sqlite_table.h
@@ -324,7 +324,7 @@
module.xDisconnect = &xDestroy;
break;
case TableType::kExplicitCreate:
- // xConnect and xDestroy will be called when the table is CREATE-ed and
+ // xCreate and xDestroy will be called when the table is CREATE-ed and
// DROP-ed respectively.
module.xCreate = &xCreate;
module.xDestroy = &xDestroy;
