search

TACC / Galaxy

Galaxy is an asynchronous parallel visualization ray tracer for performant rendering in distributed computing environments. Galaxy builds upon Intel OSPRay and Intel Embree, including ray queueing and sending logic inspired by TACC GraviT.
Apache License 2.0
19 stars 9 forks source link

Add cstring header to ClientWindow for memset #3

Closed kks32 closed 5 years ago

kks32 commented 5 years ago

When I tried to compile Galaxy on Fedora 29 with GCC 8.3, I got the following error of memset not being declared in the scope. I noticed ClientWindow.cpp doesn't include cstring header so I have added it in this PR. https://en.cppreference.com/w/cpp/string/byte/memset

galaxy/src/multiserver/ClientWindow.cpp: In member function ‘void gxy::ClientWindow::Resize(int, int)’:
galaxy/src/multiserver/ClientWindow.cpp:122:3: error: ‘memset’ was not declared in this scope
   memset(frameids, 0, width*height*sizeof(int));
   ^~~~~~
galaxy/src/multiserver/ClientWindow.cpp:122:3: note: ‘memset’ is defined in header ‘<cstring>’; did you forget to ‘#include <cstring>’?
galaxy/src/multiserver/ClientWindow.cpp:23:1:
+#include <cstring>
 #include <string>
galaxy/src/multiserver/ClientWindow.cpp:122:3:
   memset(frameids, 0, width*height*sizeof(int));
   ^~~~~~