Documentation|Client: Todo note about renaming ModelRenderer

skyjake · Sep 2, 2014 · e613224 · e613224
1 parent 63ff0f2
commit e613224
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/doomsday/client/include/render/modelrenderer.h b/doomsday/client/include/render/modelrenderer.h
@@ -30,6 +30,11 @@
  * The model renderer prepares available model assets for drawing (using ModelDrawable),
  * and keeps the set of needed ModelDrawable instances in memory.
  *
+ * @todo Consider renaming the class: the term "renderer" has the connotation of actually
+ * performing rendering, while in practice the ModelDrawables will be drawing themselves.
+ * This is the top-level class responsible for model assets and all their associated
+ * data. Perhaps the class should be instead portrayed more as a specialized Bank. -jk
+ *
  * @ingroup render
  */
 class ModelRenderer