yes, maxOccurs is inclusive.

How about treating"None" as UNBOUNDED for some time? If I remember correctly previously this was the only way to specify "unlimited" and some people might have set it explicitely instead of relying on the default.

On the other hand probably my fork is only used by me+you so maybe we don't need to worry about backwards compatibility? (but we should start writing that down in a change log)