QuickStart Engine

I was just looking through the code and saw the following code in BaseManager and Inputmanager:

Code: Select all

public bool Enabled
        {
            get { return this.enabled; }
            set
            {
                this.enabled = true;
                this.OnEnabledChanged();
                this.OnPropertyChanged(PropertyName.Enabled);
            }
        }


I think this defines a behavior you dont want. because this.enabled will always be set to true. if you always want enabled to be true removing the setter would be the best option. and just return true in the getter and get rid of the enabled property. Or when you would keep the setter i suggest you replace the this.enabled = true by this.enabled = value.

Hmmm, I'm not sure who set this up, it certainly could be a bug. I don't think we've ever disabled a manager so I don't think it has come up. Thanks for the heads up.