potential bug in isPossibleSimpleContentRevalImpl?

[Markos Zaharioudakis]

The following code in method SchemaValidatorImpl::isPossibleSimpleContentRevalImpl

      if ( schemaTypeCode == store::XS_ID ||
          schemaTypeCode == store::XS_IDREF )
        return false;
      else
        return false; //true;

return false always. It should return true in the "else" case, no?

[Chris Hillery (ceejatec)]

Lowering priority - this is probably at worst a missed opportunity for optimization, not an error.