[svn-r4724]

Purpose:
    Eliminated warnings

Description:
    1. H5CommonFG.cpp:
	On IRIX and Windows 98, if a function, that is declared to
	return a value, throws an exception in "else" statement, the
	compiler still complains that the function doesn't return a value.

    2. Others:
	H5IdComponent::operator= shouldn't be virtual because the subclass'
	operator= have different type for the rhs argument.

Solution:
    1.
    Replaced:
        if (something valid)
            return a value
        else
            throw an exception

    with:
        if (something invalid)
            throw an exception
        return a value

    2. Removed 'virtual'

Platforms tested:
    FreeBSD 4.4 (hawkwind)
    SunOS 5.7 (arabica)
    Linux 6.2 (eirene)

1 files changed, 1 insertions, 1 deletions

diff --git a/c++/src/H5PropList.h b/c++/src/H5PropList.h
index 037b0d8..325f7f5 100644
--- a/c++/src/H5PropList.h
+++ b/c++/src/H5PropList.h
@@ -25,7 +25,7 @@ class PropList : public IdComponent {
 	void copy( const PropList& like_plist );
 
 	// Make a copy of the given property list using assignment statement
-	virtual PropList& operator=( const PropList& rhs );
+	PropList& operator=( const PropList& rhs );
 
 	// Gets the class of this property list, i.e. H5P_FILE_CREATE,
 	// H5P_FILE_ACCESS, ...