2011/1/28 Vitja Makarov <vitja.maka...@gmail.com>: > 2011/1/28 Stefan Behnel <stefan...@behnel.de>: >> Vitja Makarov, 28.01.2011 07:19: >>> 2011/1/27 Stefan Behnel: >>>> Vitja Makarov, 27.01.2011 11:19: >>>>> https://github.com/vitek/cython/blob/master/Cython/Compiler/ExprNodes.py#L4995 >>>>> >>>>> .... >>>>> def analyse_types(self, env): >>>>> if not self.label_num: >>>>> error(self.pos, "'yield' not supported here") >>>>> .... >>>>> >>>>> This error message should be replaced with assertion on self.label_num >>>>> or internal error. >>>> >>>> Yes, if handled by the transform already. >>>> >>> >>> I tried to handle IfStatNode terminator here: >>> >>> https://github.com/vitek/cython/commits/uninitialized >>> >> >> >>> About tests I the easiest way is to add compiler directive -Werror >> >> We already have an "errors are fatal" option, but I like this one. >> > > This one means treat warnings as errors. > >> >>> And add it in cython header comment >>> >>> # cython: werror=True >> >> Or rather "warnings_are_errors=True". > >> Cython/Options.py: >> 'warn': None, >> 'warn.undeclared': False, > > Or better warn.errors > > But I'm wondering how to get access to > env.directives > > inside Erros.warning >
So maybe it would be better to have -Werror command line switch? -- vitja. _______________________________________________ Cython-dev mailing list Cython-dev@codespeak.net http://codespeak.net/mailman/listinfo/cython-dev
