Agreed

Sent from my iPhone

> On 10 nov 2013, at 00:46, Simon Cropp <notifications@github.com> wrote:
> 
> In src/NServiceBus.Core/Licensing/LicenseDeserializer.cs:
> 
> > +                license.MaxThroughputPerSecond = int.Parse(maxMessageThroughputPerSecond);
> > +            }
> > +
> > +            var workerThreads = doc.SelectSingleNode("/license/@WorkerThreads").Value;
> > +            //TODO: if null should this be 1?
> > +            if (workerThreads == "Max")
> > +            {
> > +                license.AllowedNumberOfThreads = MaxOfWorkerThreads;
> > +            }
> > +            else
> > +            {
> > +                license.AllowedNumberOfThreads = int.Parse(workerThreads);
> > +            }
> > +
> > +            var allowedNumberOfWorkerNodes = doc.SelectSingleNode("/license/@AllowedNumberOfWorkerNodes").Value;
> > +            //TODO: if null should this be MinNumberOfWorkerNodes?
> not sure this is required. i checked all the current licenses and all have AllowedNumberOfWorkerNodes. so in the interest of dont code for the unexpected will delete this todo
> 
> —
> Reply to this email directly or view it on GitHub.