On Mon, Jan 9, 2012 at 11:58, Iustin Pop <ius...@google.com> wrote:
> This is a trivial conversion.
> ---
>  autotools/convert-constants |   10 ++++++++++
>  1 files changed, 10 insertions(+), 0 deletions(-)
>
> diff --git a/autotools/convert-constants b/autotools/convert-constants
> index 89f9f00..ae36774 100755
> --- a/autotools/convert-constants
> +++ b/autotools/convert-constants
> @@ -27,8 +27,12 @@ import re
>  from ganeti import constants
>  from ganeti import compat
>
> +#: Constant name regex
>  CONSTANT_RE = re.compile("^[A-Z][A-Z0-9_-]+$")
>
> +#: The type of regex objects
> +RE_TYPE = type(CONSTANT_RE)
> +
>
>  def NameRules(name):
>   """Converts the upper-cased Python name to Haskell camelCase.
> @@ -129,6 +133,12 @@ def ConvertVariable(name, value):
>         lines.append("-- | Skipped list/set %s, is not homogeneous" % name)
>     else:
>       lines.append("-- | Skipped list/set %s, cannot convert all elems" % 
> name)
> +  elif isinstance(value, RE_TYPE):
> +    tvs = HaskellTypeVal(value.pattern)

According to the doc, this is always a string. Any reason to anyway
going over HaskellTypeVal?

Rest LGTM

René

Reply via email to