Use `memcpy` instead of `*(int)*&` - Githubissues

opengl-tutorials / ogl

http://www.opengl-tutorial.org

2.69k stars 925 forks source link

Use `memcpy` instead of `(int)&` #115

Open Garfield1002 opened 3 years ago

Garfield1002 commented 3 years ago

The issues with *(int)*& are that

It is quite confusing (especially for C++ beginners)
In this case, it is a strict aliasing violation

A solution can therefore be to replace it with memcpy

I am only making a single PR, but this change (if approved) probably needs to be made in other locations.

This PR comes after a question I asked on StackOverflow