Merge branch 'master' of https://github.com/danielepanozzo/libigl

Former-commit-id: c2cf2dbd70d46fea64a05ec12973c86b99087f17

include/igl/viewer/ViewerCore.cpp

@@ -319,10 +319,10 @@ IGL_INLINE void igl::ViewerCore::draw(ViewerData& data, OpenGL_state& opengl, bo
 IGL_INLINE void igl::ViewerCore::draw_buffer(ViewerData& data,
                             OpenGL_state& opengl,
                             bool update_matrices,
-                            Eigen::Matrix<char,Eigen::Dynamic,Eigen::Dynamic>& R,
-                            Eigen::Matrix<char,Eigen::Dynamic,Eigen::Dynamic>& G,
-                            Eigen::Matrix<char,Eigen::Dynamic,Eigen::Dynamic>& B,
-                            Eigen::Matrix<char,Eigen::Dynamic,Eigen::Dynamic>& A)
+                            Eigen::Matrix<unsigned char,Eigen::Dynamic,Eigen::Dynamic>& R,
+                            Eigen::Matrix<unsigned char,Eigen::Dynamic,Eigen::Dynamic>& G,
+                            Eigen::Matrix<unsigned char,Eigen::Dynamic,Eigen::Dynamic>& B,
+                            Eigen::Matrix<unsigned char,Eigen::Dynamic,Eigen::Dynamic>& A)
 {
   assert(R.rows() == G.rows() && G.rows() == B.rows() && B.rows() == A.rows());
   assert(R.cols() == G.cols() && G.cols() == B.cols() && B.cols() == A.cols());
@@ -359,7 +359,7 @@ IGL_INLINE void igl::ViewerCore::draw_buffer(ViewerData& data,
   glBindFramebuffer(GL_FRAMEBUFFER, frameBuffer);
 
   // Clear the buffer
-  glClearColor(0.f, 0.f, 0.f, 0.f);
+  glClearColor(background_color(0), background_color(1), background_color(2), 0.f);
   glClear(GL_COLOR_BUFFER_BIT | GL_DEPTH_BUFFER_BIT);
 
   // Save old viewport

include/igl/viewer/ViewerCore.h

@@ -61,10 +61,10 @@ public:
   IGL_INLINE void draw_buffer(ViewerData& data,
                               OpenGL_state& opengl,
                               bool update_matrices,
-                              Eigen::Matrix<char,Eigen::Dynamic,Eigen::Dynamic>& R,
-                              Eigen::Matrix<char,Eigen::Dynamic,Eigen::Dynamic>& G,
-                              Eigen::Matrix<char,Eigen::Dynamic,Eigen::Dynamic>& B,
-                              Eigen::Matrix<char,Eigen::Dynamic,Eigen::Dynamic>& A);
+                              Eigen::Matrix<unsigned char,Eigen::Dynamic,Eigen::Dynamic>& R,
+                              Eigen::Matrix<unsigned char,Eigen::Dynamic,Eigen::Dynamic>& G,
+                              Eigen::Matrix<unsigned char,Eigen::Dynamic,Eigen::Dynamic>& B,
+                              Eigen::Matrix<unsigned char,Eigen::Dynamic,Eigen::Dynamic>& A);
 
   // ------------------- Properties
 

include/igl/viewer/ViewerData.cpp

@@ -219,9 +219,9 @@ IGL_INLINE void igl::ViewerData::set_uv(const Eigen::MatrixXd& UV_V, const Eigen
 
 
 IGL_INLINE void igl::ViewerData::set_texture(
-  const Eigen::Matrix<char,Eigen::Dynamic,Eigen::Dynamic>& R,
-  const Eigen::Matrix<char,Eigen::Dynamic,Eigen::Dynamic>& G,
-  const Eigen::Matrix<char,Eigen::Dynamic,Eigen::Dynamic>& B)
+  const Eigen::Matrix<unsigned char,Eigen::Dynamic,Eigen::Dynamic>& R,
+  const Eigen::Matrix<unsigned char,Eigen::Dynamic,Eigen::Dynamic>& G,
+  const Eigen::Matrix<unsigned char,Eigen::Dynamic,Eigen::Dynamic>& B)
 {
   texture_R = R;
   texture_G = G;

include/igl/viewer/ViewerData.h

@@ -62,9 +62,9 @@ public:
   IGL_INLINE void set_uv(const Eigen::MatrixXd& UV);
   IGL_INLINE void set_uv(const Eigen::MatrixXd& UV_V, const Eigen::MatrixXi& UV_F);
   IGL_INLINE void set_texture(
-                    const Eigen::Matrix<char,Eigen::Dynamic,Eigen::Dynamic>& R,
-                    const Eigen::Matrix<char,Eigen::Dynamic,Eigen::Dynamic>& G,
-                    const Eigen::Matrix<char,Eigen::Dynamic,Eigen::Dynamic>& B);
+                    const Eigen::Matrix<unsigned char,Eigen::Dynamic,Eigen::Dynamic>& R,
+                    const Eigen::Matrix<unsigned char,Eigen::Dynamic,Eigen::Dynamic>& G,
+                    const Eigen::Matrix<unsigned char,Eigen::Dynamic,Eigen::Dynamic>& B);
 
   // Sets points given a list of point vertices. In constrast to `set_points`
   // this will (purposefully) clober existing points.
@@ -118,9 +118,9 @@ public:
   Eigen::MatrixXi F_uv; // optional faces for UVs
 
   // Texture
-  Eigen::Matrix<char,Eigen::Dynamic,Eigen::Dynamic> texture_R;
-  Eigen::Matrix<char,Eigen::Dynamic,Eigen::Dynamic> texture_G;
-  Eigen::Matrix<char,Eigen::Dynamic,Eigen::Dynamic> texture_B;
+  Eigen::Matrix<unsigned char,Eigen::Dynamic,Eigen::Dynamic> texture_R;
+  Eigen::Matrix<unsigned char,Eigen::Dynamic,Eigen::Dynamic> texture_G;
+  Eigen::Matrix<unsigned char,Eigen::Dynamic,Eigen::Dynamic> texture_B;
 
   // Overlays