Merge pull request #357 from johnhaddon/glImprovements

IECoreGL Improvements

Author: davidsminor

include/IECore/RefCounted.h

@@ -169,6 +169,16 @@ inline void intrusive_ptr_release( const IECore::RefCounted *r )
 	r->removeRef();
 }
 
+/// Implementation of tbb_hasher to allow intrusive_ptrs to be used
+/// with tbb_concurrent_* containers.
+template<typename T>
+inline size_t tbb_hasher( const boost::intrusive_ptr<T> &ptr )
+{
+	// This is the same as what tbb uses for raw pointers
+	const size_t h = reinterpret_cast<size_t>( ptr.get() );
+	return (h >> 3) ^ h;
+}
+
 } // namespace IECore
 
 

include/IECoreGL/HitRecord.h

@@ -53,18 +53,15 @@ class HitRecord
 		/// for the OpenGL select buffer. Raises an exception if
 		/// more than one name is specified in the record.
 		HitRecord( const GLuint *hitRecord );
-		HitRecord( float dMin, float dMax, const IECore::InternedString &primName );
+		HitRecord( float dMin, float dMax, GLuint name );
 
 		/// The minimum and maximum depths of the hit, normalised
 		/// in the 0-1 range between the near and far clipping planes.
 		float depthMin;
 		float depthMax;
 
-		/// Unlike the gl hit record, the HitRecord stores
-		/// only one name - this is because the NameStateComponent
-		/// and the Renderer "name" attribute specify only a single
-		/// name for each primitive rendered.
-		IECore::InternedString name;
+		/// Identifier for the hit object.
+		GLuint name;
 
 		/// Performs comparison based on the depth.min member.
 		bool operator < ( const HitRecord &other ) const;

include/IECoreGL/Selector.h

@@ -107,6 +107,10 @@ class Selector : boost::noncopyable
 		/// If rendering a Scene, this will be called automatically
 		/// by the NameStateComponents within the Scene.
 		void loadName( GLuint name );
+		/// Generates a new name (by incrementing an internal counter) and
+		/// loads and returns it. No guarantee is made that generated names
+		/// will not clash with names loaded explicitly with the method above.
+		GLuint loadName();
 
 		/// A State that should be used as the base state for
 		/// selection drawing.

include/IECoreGL/State.h

@@ -62,12 +62,16 @@ class State : public Bindable
 				/// new bindings. It is the caller's responsibility to keep both arguments
 				/// alive until after destruction of the ScopedBinding.
 				ScopedBinding( const State &s, State &currentState );
+				/// As above, but does nothing if bind is false.
+				ScopedBinding( const State &s, State &currentState, bool bind );
 				/// Reverts the state changes and modifications to currentState
 				/// made by the constructor.
 				~ScopedBinding();
 
 			private :
 
+				void init( const State &s, bool bind = true );
+
 				State &m_currentState;
 				std::vector<StateComponentPtr> m_savedComponents;
 

include/IECoreGL/ToGLStateConverter.h

@@ -0,0 +1,84 @@
+//////////////////////////////////////////////////////////////////////////
+//
+//  Copyright (c) 2014, Image Engine Design Inc. All rights reserved.
+//
+//  Redistribution and use in source and binary forms, with or without
+//  modification, are permitted provided that the following conditions are
+//  met:
+//
+//     * Redistributions of source code must retain the above copyright
+//       notice, this list of conditions and the following disclaimer.
+//
+//     * Redistributions in binary form must reproduce the above copyright
+//       notice, this list of conditions and the following disclaimer in the
+//       documentation and/or other materials provided with the distribution.
+//
+//     * Neither the name of Image Engine Design nor the names of any
+//       other contributors to this software may be used to endorse or
+//       promote products derived from this software without specific prior
+//       written permission.
+//
+//  THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS
+//  IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
+//  THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
+//  PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR
+//  CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
+//  EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
+//  PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
+//  PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF
+//  LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
+//  NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
+//  SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+//
+//////////////////////////////////////////////////////////////////////////
+
+#ifndef IECOREGL_TOGLSTATECONVERTER_H
+#define IECOREGL_TOGLSTATECONVERTER_H
+
+#include "IECoreGL/ToGLConverter.h"
+
+namespace IECore
+{
+
+IE_CORE_FORWARDDECLARE( CompoundObject )
+IE_CORE_FORWARDDECLARE( Data )
+
+} // namespace IECore
+
+namespace IECoreGL
+{
+
+IE_CORE_FORWARDDECLARE( State )
+IE_CORE_FORWARDDECLARE( StateComponent )
+
+/// Converts IECore::CompoundObject objects containing shaders and attributes
+/// into IECoreGL::State objects.
+/// \ingroup conversionGroup
+class ToGLStateConverter : public ToGLConverter
+{
+
+	public :
+
+		typedef IECore::CompoundObject InputType;
+		typedef IECoreGL::State ResultType;
+
+		IE_CORE_DECLARERUNTIMETYPEDEXTENSION( IECoreGL::ToGLStateConverter, ToGLStateConverterTypeId, ToGLConverter );
+
+		ToGLStateConverter( IECore::ConstCompoundObjectPtr toConvert = NULL );
+		virtual ~ToGLStateConverter();
+
+	protected :
+
+		virtual IECore::RunTimeTypedPtr doConversion( IECore::ConstObjectPtr src, IECore::ConstCompoundObjectPtr operands ) const;
+
+	private :
+
+		static ConverterDescription<ToGLStateConverter> g_description;
+
+};
+
+IE_CORE_DECLAREPTR( ToGLStateConverter );
+
+} // namespace IECoreGL
+
+#endif // IECOREGL_TOGLSTATECONVERTER_H

include/IECoreGL/TypeIds.h

@@ -121,6 +121,7 @@ enum TypeId
 	ToGLBufferConverterTypeId = 105078,
 	UIntTextureTypeId = 105079,
 	PrimitiveSelectableTypeId = 105080,
+	ToGLStateConverterTypeId = 105081,
 	LastCoreGLTypeId = 105999,
 };
 

include/IECoreGL/TextureUnits.h renamed to include/IECoreGL/bindings/CurvesPrimitiveBinding.h

@@ -1,6 +1,6 @@
 //////////////////////////////////////////////////////////////////////////
 //
-//  Copyright (c) 2007, Image Engine Design Inc. All rights reserved.
+//  Copyright (c) 2014, Image Engine Design Inc. All rights reserved.
 //
 //  Redistribution and use in source and binary forms, with or without
 //  modification, are permitted provided that the following conditions are
@@ -32,23 +32,14 @@
 //
 //////////////////////////////////////////////////////////////////////////
 
-#ifndef IECOREGL_TEXTUREUNITS_H
-#define IECOREGL_TEXTUREUNITS_H
-
-#include "IECoreGL/GL.h"
-
-#include <vector>
+#ifndef IECOREGL_CURVESPRIMITIVEBINDING_H
+#define IECOREGL_CURVESPRIMITIVEBINDING_H
 
 namespace IECoreGL
 {
 
-/// Returns a vector containing GL_TEXTURE0, GL_TEXTURE1, ...
-/// GL_TEXTUREN for all texture unit enums up to
-/// GL_MAX_COMBINED_TEXTURE_IMAGE_UNITS - 1. That way you can
-/// actually program texture code like a grown up,
-/// using like loops and stuff.
-const std::vector<GLenum> &textureUnits();
+void bindCurvesPrimitive();
 
 } // namespace IECoreGL
 
-#endif // IECOREGL_TEXTUREUNITS_H
+#endif // IECOREGL_CURVESPRIMITIVEBINDING_H

include/IECoreGL/bindings/ToGLStateConverterBinding.h

@@ -0,0 +1,45 @@
+//////////////////////////////////////////////////////////////////////////
+//
+//  Copyright (c) 2014, Image Engine Design Inc. All rights reserved.
+//
+//  Redistribution and use in source and binary forms, with or without
+//  modification, are permitted provided that the following conditions are
+//  met:
+//
+//     * Redistributions of source code must retain the above copyright
+//       notice, this list of conditions and the following disclaimer.
+//
+//     * Redistributions in binary form must reproduce the above copyright
+//       notice, this list of conditions and the following disclaimer in the
+//       documentation and/or other materials provided with the distribution.
+//
+//     * Neither the name of Image Engine Design nor the names of any
+//       other contributors to this software may be used to endorse or
+//       promote products derived from this software without specific prior
+//       written permission.
+//
+//  THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS
+//  IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO,
+//  THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
+//  PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR
+//  CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
+//  EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
+//  PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
+//  PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF
+//  LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
+//  NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
+//  SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+//
+//////////////////////////////////////////////////////////////////////////
+
+#ifndef IECOREGL_TOGLSTATECONVERTERBINDING_H
+#define IECOREGL_TOGLSTATECONVERTERBINDING_H
+
+namespace IECoreGL
+{
+
+void bindToGLStateConverter();
+
+} // namespace IECoreGL
+
+#endif // IECOREGL_TOGLSTATECONVERTERBINDING_H

src/IECoreGL/HitRecord.cpp

@@ -46,7 +46,7 @@ using namespace IECoreGL;
 HitRecord::HitRecord( const GLuint *hitRecord )
 	:	depthMin( (float)hitRecord[1]/(float)Imath::limits<GLuint>::max() ),
 		depthMax( (float)hitRecord[2]/(float)Imath::limits<GLuint>::max() ),
-		name( NameStateComponent::nameFromGLName( hitRecord[3] ) )
+		name( hitRecord[3] )
 
 {
 	if( hitRecord[0] != 1 )
@@ -55,8 +55,8 @@ HitRecord::HitRecord( const GLuint *hitRecord )
 	}
 }
 
-HitRecord::HitRecord( float dMin, float dMax, const IECore::InternedString &primName )
-	:	depthMin( dMin ), depthMax( dMax ), name( primName )
+HitRecord::HitRecord( float dMin, float dMax, GLuint name )
+	:	depthMin( dMin ), depthMax( dMax ), name( name )
 {
 }
 

src/IECoreGL/Primitive.cpp

@@ -45,7 +45,6 @@
 #include "IECoreGL/TypedStateComponent.h"
 #include "IECoreGL/ShaderStateComponent.h"
 #include "IECoreGL/Shader.h"
-#include "IECoreGL/TextureUnits.h"
 #include "IECoreGL/NumericTraits.h"
 #include "IECoreGL/UniformFunctions.h"
 #include "IECoreGL/CachedConverter.h"