On Sun, Nov 15, 2009 at 6:31 AM, Matthias Fuchs <[email protected]> wrote: > So far the Pieces length, MetaURL priority and URL priority use xsd:integer. > > Yet none of these defines an applicable range. What that means is that > negative values would be valid as well according to the draft even if they > made no sense at all. In fact for the Pieces length this is probably not the > case as the description in the draft should be enough to provide a positive > range. > > So in my opinion we should either add a range or change them to one of the > unsigned PODS. In that case we still have to define that 0 is not to be used > for pirority. > > > In terms of priority I would opt for a range as imo a priority of 268435456 > would rather be confusing if it was also shown to the user not just used > internally. We could use xsd:unsignedByte for example, when we would have only > positive values and automatically a range (if we exclude 0 in fact) from 1 to > 255. > > > What do you think on that?
good catch, Matthias! pieces length, I think defining a range might be hard. I think the default torrent chunk size is 256k. max range, who knows? limiting it to positive integers should be good, right? priority for metaurl and url, a range wouldn't be bad. does anyone else want 1 to 255? On Sun, Nov 15, 2009 at 4:45 PM, Nicolas Alvarez <[email protected]> wrote: > > Matthias Fuchs wrote: >> So in my opinion we should either add a range or change them to one of the >> unsigned PODS. In that case we still have to define that 0 is not to be >> used for pirority. > > IIRC, XSD has different types for "positive" and "nonnegative". The former > doesn't include 0. positiveInteger, nonNegativeInteger? do I just need to replace "integer" in the schema with "positiveInteger"? metalinkPieces = element metalink:pieces { attribute length { xsd:integer }, attribute type { text }, metalinkHash+ metalinkMetaURL = element metalink:metaurl { metalinkCommonAttributes, attribute priority { xsd:integer }?, attribute type { text }, attribute name { text }?, metalinkUri metalinkURL = element metalink:url { metalinkCommonAttributes, attribute location { xsd:string { minLength = "2" maxLength="2"} }?, attribute priority { xsd:integer }?, metalinkUri -- (( Anthony Bryan ... Metalink [ http://www.metalinker.org ] )) Easier, More Reliable, Self Healing Downloads -- You received this message because you are subscribed to the Google Groups "Metalink Discussion" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/metalink-discussion?hl=.
