id	summary	reporter	owner	description	type	status	priority	component	severity	resolution	keywords	cc
61	Raise ValueError on bad time stamps	miki@…	xi	"Currently if a timestamp is badly formatted, syck raises:
{{{
#!python
Traceback (most recent call last):
  File ""<pyshell#2>"", line 1, in <module>
    yaml = syck.load(open(""/var/spool/articles/20070917/9a45fab335a148a5999d8d000a3c23fe/article.yaml"").read())
  File ""/usr/local/lib/python2.5/site-packages/syck/loaders.py"", line 408, in load
    return loader.load()
  File ""/usr/local/lib/python2.5/site-packages/syck/loaders.py"", line 50, in load
    return self._convert(node, {})
  File ""/usr/local/lib/python2.5/site-packages/syck/loaders.py"", line 67, in _convert
    node_to_object)
  File ""/usr/local/lib/python2.5/site-packages/syck/loaders.py"", line 84, in _convert
    object = self.construct(node)
  File ""/usr/local/lib/python2.5/site-packages/syck/loaders.py"", line 174, in construct
    return constructor(node)
  File ""/usr/local/lib/python2.5/site-packages/syck/loaders.py"", line 242, in construct_timestamp
    values = match.groupdict()
AttributeError: 'NoneType' object has no attribute 'groupdict'
}}}
which is cryptic.

`construct_timestamp` should check the if the value was matched:
{{{
#!python
    def construct_timestamp(self, node):
        match = self.timestamp_expr.match(node.value)

        if not match:
            raise ValueError, ""bad time stamp: %s"" % node.value

}}}
"	defect	new	normal	pysyck	normal