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
today fails due to the QCoreApplication having the constructor: QCoreApplication::QCoreApplication(int &argc, char **argv)
notice the ampersant there.
I would argue the same as the person that reported bug 592 that the argc and argv are there for backwards compatibility and it would be nice if they follow the (granted ugly) unsafe behavior we inherited from the time when we were still walking in bearskins.
Which is to say, would you consider removing the const on args.argc in order to make this library class's constructor work.
The text was updated successfully, but these errors were encountered:
This is a followup of sorts to bug #592, but now for a differnent issue that likely is a regression for the exact same usecase as that bug.
The simple code of:
today fails due to the QCoreApplication having the constructor:
QCoreApplication::QCoreApplication(int &argc, char **argv)
notice the ampersant there.
I would argue the same as the person that reported bug 592 that the
argc
andargv
are there for backwards compatibility and it would be nice if they follow the (granted ugly) unsafe behavior we inherited from the time when we were still walking in bearskins.Which is to say, would you consider removing the const on
args.argc
in order to make this library class's constructor work.The text was updated successfully, but these errors were encountered: