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
Avoid partial construction by implementing copy/move assignment operators/constructors.
e.g. look at Board, Renderer, FPScounter, etc.
instead of constructing in mainwindow.h and then calling Attach(), Start() etc, which means the object isn't correctly initialized until those are called, see if you can implement this pattern.
MainWindow.h
fpscounter fps{nullptr_t}
MainWindow.cpp
fps = fps(foo, bar, blah);
For example Renderer.h has
Microsoft::Graphics::Canvas::CanvasRenderTarget _spritesheet{ nullptr };
Which isn't truly initialized until
Renderer.cpp
_spritesheet = Microsoft::Graphics::Canvas::CanvasRenderTarget(_canvasDevice, _spriteDipsPerRow, _spriteDipsPerRow, _dpi);
The text was updated successfully, but these errors were encountered:
Avoid partial construction by implementing copy/move assignment operators/constructors.
e.g. look at Board, Renderer, FPScounter, etc.
instead of constructing in mainwindow.h and then calling Attach(), Start() etc, which means the object isn't correctly initialized until those are called, see if you can implement this pattern.
MainWindow.h
fpscounter fps{nullptr_t}
MainWindow.cpp
fps = fps(foo, bar, blah);
For example Renderer.h has
Microsoft::Graphics::Canvas::CanvasRenderTarget _spritesheet{ nullptr };
Which isn't truly initialized until
Renderer.cpp
_spritesheet = Microsoft::Graphics::Canvas::CanvasRenderTarget(_canvasDevice, _spriteDipsPerRow, _spriteDipsPerRow, _dpi);
The text was updated successfully, but these errors were encountered: