Read Access Violation in Window::RegisterCallbacks()

[safi-huss]

Hi,

I am getting a read access violation when I reach this line;

`void Window::RegisterCallbacks(std::shared_ptr arg_pGame)
{
m_pGame = arg_pGame; //Read Access violation here

return;

}
`
I am new to this, but my hunch is that since we are passing it "shared_from_this", it may not be move assignable. I could be completely wrong too.

How would I fix it?

[jpvanoosten]

It looks like you've made some changes to the code. For example, std::shared_ptr is a template class but you are not specifying any template arguments in the code example. It should be std::shared_ptr<Game> as can be seen here DX12Lib/src/Window.cpp#L188.

Also, when using shared_from_this, your class must publicly inherit from std::enable_shared_from_this<T> (where T is the derived class) as can be seen here: DX12Lib/inc/Game.h#L42.

[safi-huss]

It looks like you've made some changes to the code. For example, std::shared_ptr is a template class but you are not specifying any template arguments in the code example. It should be std::shared_ptr as can be seen here DX12Lib/src/Window.cpp#L188.

Sorry, I guess that was my mistake when trying to copy-paste the code and trying to get the code snippet to work. My code does have the template assigned.

`void Window::RegisterCallbacks(std::shared_ptr<Game arg_pGame)
{
m_pGame = arg_pGame;

return;

}
`
EDIT: I had to remove the ending angle brace for some reason, the code snippet tool omits that. The same is so in my original post.

Same goes for the class declarations as follows:

class Game : public std::enable_shared_from_this<Game> {

I tried changing the weak_ptr m_pGame to shared_ptr m_pGame, and commenting out the locks. but still, I get a similar error for Read Access violation.

Some of the ComPtr Objects are uninitialized at this point. Is that to be expected or have I bungled the code?

[safi-huss]

I believe I made a grave error in the code. Apparently, my window initialization was incorrect and I was using invalid handles to the Window to initialize the rest of the code.

Thank you so much for bearing my ineptitude. And a million thanks for writing these tutorials!