DO NOT REPLY [Bug 24266] New: - AxisConfig constructor doesn't error check

[bu...@apache.org]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=24266>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=24266

AxisConfig constructor doesn't error check

           Summary: AxisConfig constructor doesn't error check
           Product: Axis-C++
           Version: current (nightly)
          Platform: All
        OS/Version: Other
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: Basic Architecture
        AssignedTo: axis-dev@ws.apache.org
        ReportedBy: slang@mcs.anl.gov


If the ReadConfFile() call fails (AXIS_HOME env isn't set for example), the
error that gets returned is ignored in AxisConfig::AxisConfig().  Given that the
constructor doesn't allow for a return parameter, maybe the constructor should
just be empty, and the user should be required to call ReadConfFile() directly.
 That way they can at least determine if it fails.  Since a bad user
configuration is usually a common source of error, it would be nice if
ReadConfFile returned more informative error information, rather than just FAIL.  

Also, it would be nice if the filename and path of the axis config file could be
overridden from the default ($AXIS_HOME/axiscpp.conf).  Maybe overload the
ReadConfFile method with one that takes a parameter?