Browse Source

[1783] noted a future extension to RendererHolder so it's more visible.

JINMEI Tatuya 13 years ago
parent
commit
e0cee3754d
1 changed files with 5 additions and 1 deletions
  1. 5 1
      src/bin/auth/auth_srv.cc

+ 5 - 1
src/bin/auth/auth_srv.cc

@@ -87,7 +87,11 @@ namespace {
 // the previously set buffer and then release any internal resource in the
 // the previously set buffer and then release any internal resource in the
 // renderer, no matter what happened during the rendering, especially even
 // renderer, no matter what happened during the rendering, especially even
 // when it resulted in an exception.
 // when it resulted in an exception.
-class  RendererHolder {
+//
+// Note: if we need this helper in many other places we might consider making
+// it visible to other modules.  As of this implementation this is the only
+// user of this class, so we hide it within the implementation.
+class RendererHolder {
 public:
 public:
     RendererHolder(MessageRenderer& renderer, OutputBuffer* buffer) :
     RendererHolder(MessageRenderer& renderer, OutputBuffer* buffer) :
         renderer_(renderer)
         renderer_(renderer)