You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Migrations have to include api.h to be implemented at the moment. api.h is a rather complex header file which polutes every migration header file with unnecessery command and structure classes at the moment.
Specify a tightly defined header file to define a new migration (HPP). Specify a header file for new migration implemantions (CPP).
The text was updated successfully, but these errors were encountered:
The compile time impact of the current structure should be quite low. Migration Headers are included at most twice.
The migration implementation, which require api.h definitions.
The migration registration (as long as auto-registration does not work) includes all the migrations. The api.h is only included from the first, so impact should be low.
Additionally the QSqlMigrator is an external dependency for projects that use it. Therefore it's a good idea to include the api.h in the precompiled header. Which should eliminate almost all of it's compile time costs.
On the other hand is the ease of us. Using the "api.h" makes the migrations really easy to write or generate with a macro.
Migrations have to include api.h to be implemented at the moment. api.h is a rather complex header file which polutes every migration header file with unnecessery command and structure classes at the moment.
Specify a tightly defined header file to define a new migration (HPP). Specify a header file for new migration implemantions (CPP).
The text was updated successfully, but these errors were encountered: