Hi Thomas, thanks for the patch!
We want the tests to "pass" after every commit, i.e. have no failures. That may mean, in this case, introducing the tests marked as todo, and then later removing those todos.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/1825#note_19803
Setting the return type of various wined3d refcounting functions to void
These patches modify:
- wined3d_incref
- wined3d_decref
- wined3d_resource_incref
- wined3d_resource_decref
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/1755
On Sat Dec 17 10:41:32 2022 +0000, Etaash Mathamsetty wrote:
> I also think you should squash c4e27bd0483c0a1fb872111b6c03b9ad240d85da
> (mark tests as todo) and d02be967f9cca0805b5eeaef6a74b3ec51a67619 (add
> tests) into one commit
I can do so, but before semi stub implementation even more tests are failing. See description. Should i squash mark all tests as todo and remove the markers with semi stub commit? Thanks for your help.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/1825#note_19792
Revision 24b26f8bd6a49c855eec417a6585d44c9458e3ec changed FreeBSD
(and DragonFly) not to use /proc any longer. Hence we also do not
need the exe_link variable on those two platforms, either. Avoid
declaring it there.
(This avoids a compiler warning about an unused variable with GCC 12.)
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/1833
find_device_from_devnode was unnecessarily guarded by HAVE_SYS_INOTIFY_H,
alas its use in process_monitor_event was not, so linking failed. Simply
make find_device_from_devnode generally available.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/1831
On Thu Dec 15 20:00:54 2022 +0000, Etaash Mathamsetty wrote:
> you should mark them as todo in the code for the tests then
I can do so, but before semi stub implementation even more tests are failing. See description. Should i squash mark all tests as todo and remove the markers with semi stub commit? Thanks for your help.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/1825#note_19782