gltut icon indicating copy to clipboard operation
gltut copied to clipboard

Using memcpy to copy a vector

Open paroj opened this issue 12 years ago • 1 comments

Originally reported by: Anonymous


In http://www.arcsynthesis.org/gltut/Positioning/Tutorial%2003.html, Example 3.2, this: #!c++ memcpy(&fNewData[0], vertexPositions, sizeof(vertexPositions));

should really be this: #!c++ std::copy(vertexPositions, vertexPositions+sizeof(vertexPositions), fNewData.begin());

or even better: #!c++ std::vector fNewData(vertexPositions, vertexPositions+sizeof(vertexPositions));

(std::begin / std::end could be used, but I doubt the tutorials should require C++11 compliance)


  • Bitbucket: https://bitbucket.org/alfonse/gltut/issue/87

paroj avatar Oct 23 '12 08:10 paroj

Original comment by Jason McKesson (Bitbucket: alfonse, GitHub: alfonse):


I'll consider it.

paroj avatar Oct 24 '12 00:10 paroj