Bug# 1687: resValue not always updated when making a transcoder,

                   patch from mark_russell@csgsystems.com


git-svn-id: https://svn.apache.org/repos/asf/xerces/c/trunk@173571 13f79535-47bb-0310-9956-ffa450edef68

src/xercesc/util/TransService.cpp

@@ -200,7 +200,11 @@ XMLTransService::makeNewTranscoderFor(  const   XMLCh* const            encoding
 
     // If we found it, then call the factory method for it
     if (ourMapping)
-        return ourMapping->makeNew(blockSize);
+	{		
+       XMLTranscoder* temp = ourMapping->makeNew(blockSize);
+       resValue = temp ? XMLTransService::Ok : XMLTransService::InternalFailure;
+       return temp;
+     }
 
     //
     //  For now, we have a little list of encodings that we disallow
@@ -214,7 +218,10 @@ XMLTransService::makeNewTranscoderFor(  const   XMLCh* const            encoding
         {
             // If its one of our guys, then pretend we don't understand it
             if (!XMLString::compareString(upBuf, gDisallowList[index]))
+			{
+                resValue = XMLTransService::UnsupportedEncoding;               
                 return 0;
+			}
         }
     }