Address overlooked review comments

Chris Ridd

08.07.2012 8d558f2442f32599065fd25e852a83251c28a313

 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);
  }