> > +    @property
> > +    def name(self):
> > +        return self.__name
> > +
> > +    @name.setter
> > +    def name(self, name):
> > +        """ Set the name of the test run
> > +
> > +        We don't allow path seperators in the name, they are silently
> > replaced
> > +        with underscores.
> > +
> > +        """
> > +        replace = ['/', '\\']
> > +
> > +        for r in replace:
> > +            name = name.replace(r, '_')
> > +
> > +        self.__name = name
> > +
> >
> 
> Personally, I'd rather we mangle things when we turn them into file names
> rather than mangling the original.  This way the original string still
> shows up in table headers etc.  That said, this also accomplishes the
> objective of not crashing in stupid places.
> --Jason

I'm working on a v2. I'll make that change.

[snip]

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
Piglit mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to