Is binding ToConstant and calling InSingletonScope redundant?

Question

Well, this question is pretty simply stated by the title.

For a local variable factory:

var factory = Fluently.Configure()
...

Are these two lines equivalent:

Bind<ISessionFactory>().ToConstant(factory).InSingletonScope();

and:

Bind<ISessionFactory>().ToConstant(factory);

Answer 1

In the latest version of ninject, when you create a ToConstant binding it will automatically set the Scope to Singleton. Thus, the InSingletonScope() part in your example is redundant. From ninject code base:

    /// <summary>
    /// Indicates that the service should be bound to the specified constant value.
    /// </summary>
    /// <param name="value">The constant value.</param>
    public IBindingWhenInNamedWithOrOnSyntax<T> ToConstant(T value)
    {
        Binding.ProviderCallback = ctx => new ConstantProvider<T>(value);
        Binding.Target = BindingTarget.Constant;
        Binding.ScopeCallback = StandardScopeCallbacks.Singleton;

        return this;
    }