Address overlooked review comments

Chris Ridd

08.07.2012 54cea43e8970c0c8175d9ccceea7e26453855be2

Address overlooked review comments

Access to the "strictness" is now done using the configuration
variable, ie it is now modelled on the Certificate syntax
rather than the TelephoneNumber syntax.

 opendj-sdk/opends/src/server/org/opends/server/schema/JPEGSyntax.java

@@ -64,9 +64,6 @@
       extends AttributeSyntax<JPEGAttributeSyntaxCfg>
       implements ConfigurationChangeListener<JPEGAttributeSyntaxCfg>
{
  // Indicates whether this syntax should operate in strict mode.
  private boolean strictMode;

  // The default equality matching rule for this syntax.
  private EqualityMatchingRule defaultEqualityMatchingRule;

@@ -77,7 +74,7 @@
  private SubstringMatchingRule defaultSubstringMatchingRule;

  // The current configuration for this JPEG syntax.
  private JPEGAttributeSyntaxCfg currentConfig;
  private volatile JPEGAttributeSyntaxCfg config;



@@ -124,16 +121,8 @@
          SMR_OCTET_STRING_OID, SYNTAX_JPEG_NAME));
    }

    // We may or may not have access to the config entry.  If we do, then see if
    // we should use the strict compliance mode.  If not, just assume that we
    // won't.
    strictMode = false;
    if (configuration != null)
    {
      currentConfig = configuration;
      currentConfig.addJPEGChangeListener(this);
      strictMode = currentConfig.isStrictFormat();
    }
    this.config = configuration;
    config.addJPEGChangeListener(this);
  }


@@ -251,7 +240,7 @@
                                   MessageBuilder invalidReason)
  {
    // anything is acceptable if we're not strict.
    if (strictMode == false)
    if (!config.isStrictFormat())
        return true;

    /* JFIF files start:
@@ -304,9 +293,7 @@
  public ConfigChangeResult applyConfigurationChange(
              JPEGAttributeSyntaxCfg configuration)
  {
    currentConfig = configuration;
    strictMode = configuration.isStrictFormat();

    this.config = configuration;
    return new ConfigChangeResult(ResultCode.SUCCESS, false);
  }