2010-06-12 Tatsuhiro Tsujikawa <t-tujikawa@users.sourceforge.net>

Made protected member variable private.
	* src/XmlRpcMethod.h
	* src/XmlRpcMethodImpl.cc

ChangeLog

@@ -1,3 +1,9 @@
+2010-06-12  Tatsuhiro Tsujikawa  <t-tujikawa@users.sourceforge.net>
+
+	Made protected member variable private.
+	* src/XmlRpcMethod.h
+	* src/XmlRpcMethodImpl.cc
+
 2010-06-12  Tatsuhiro Tsujikawa  <t-tujikawa@users.sourceforge.net>
 
 	Made protected member variable private.

src/XmlRpcMethod.h

@@ -63,11 +63,11 @@ struct XmlRpcResponse;
 // subclass. If you add new XmlRpcMethod subclass, don't forget to add
 // it to XmlRpcMethodFactory.
 class XmlRpcMethod {
-protected:
+private:
   SharedHandle<OptionParser> _optionParser;
 
   Logger* _logger;
-
+protected:
   // Subclass must implement this function to fulfil XmlRpcRequest
   // req.  The return value of this method is used as a return value
   // of XML-RPC request.
@@ -91,6 +91,11 @@ protected:
   void applyChangeableGlobalOption(Option* dest, Option* src) const;
 
   BDE createErrorResponse(const Exception& e);
+
+  const SharedHandle<OptionParser>& getOptionParser() const
+  {
+    return _optionParser;
+  }
 public:
   XmlRpcMethod();
 

src/XmlRpcMethodImpl.cc

@@ -1016,7 +1016,7 @@ BDE GetGlobalOptionXmlRpcMethod::process
   BDE result = BDE::dict();
   for(std::map<std::string, std::string>::const_iterator i =
         e->getOption()->begin(), eoi = e->getOption()->end(); i != eoi; ++i) {
-    SharedHandle<OptionHandler> h = _optionParser->findByName((*i).first);
+    SharedHandle<OptionHandler> h = getOptionParser()->findByName((*i).first);
     if(!h.isNull() && !h->isHidden()) {
       result[(*i).first] = (*i).second;
     }