<div dir="ltr">I guess I was worried about the string containing a non-digit. Like "1.5", which sounds like something might actually be a valid value.<div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Dec 15, 2014 at 7:19 PM, Justin Bogner <span dir="ltr"><<a href="mailto:mail@justinbogner.com" target="_blank">mail@justinbogner.com</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">Kuba Brecka <<a href="mailto:kuba.brecka@gmail.com">kuba.brecka@gmail.com</a>> writes:<br>
> We recently had a broken version check because an newer OS X version<br>
> is treated as MACOS_VERSION_UNKNOWN which is less than all the defined<br>
> values. Let's have a separate enum value for unknown but newer<br>
> versions, so the ">=" and "<=" version checks still work even in<br>
> upcoming OS X releases.<br>
><br>
> <a href="http://reviews.llvm.org/D6137" target="_blank">http://reviews.llvm.org/D6137</a><br>
><br>
> Files:<br>
>   projects/compiler-rt/lib/sanitizer_common/sanitizer_mac.cc<br>
>   projects/compiler-rt/lib/sanitizer_common/sanitizer_mac.h<br>
><br>
</div></div>> Index: projects/compiler-rt/lib/sanitizer_common/sanitizer_mac.cc<br>
> ===================================================================<br>
> --- projects/compiler-rt/lib/sanitizer_common/sanitizer_mac.cc<br>
> +++ projects/compiler-rt/lib/sanitizer_common/sanitizer_mac.cc<br>
> @@ -298,6 +298,11 @@<br>
>          case '2': return MACOS_VERSION_MOUNTAIN_LION;<br>
>          case '3': return MACOS_VERSION_MAVERICKS;<br>
>          case '4': return MACOS_VERSION_YOSEMITE;<br>
> +        case '5':<br>
> +        case '6':<br>
> +        case '7':<br>
> +        case '8':<br>
> +        case '9': return MACOS_VERSION_UNKNOWN_NEWER;<br>
>          default: return MACOS_VERSION_UNKNOWN;<br>
<br>
Why not just change the default case to NEWER, since this switch covers<br>
every digit now?<br>
<br>
>        }<br>
>      }<br>
> Index: projects/compiler-rt/lib/sanitizer_common/sanitizer_mac.h<br>
> ===================================================================<br>
> --- projects/compiler-rt/lib/sanitizer_common/sanitizer_mac.h<br>
> +++ projects/compiler-rt/lib/sanitizer_common/sanitizer_mac.h<br>
> @@ -27,6 +27,7 @@<br>
>    MACOS_VERSION_MOUNTAIN_LION,<br>
>    MACOS_VERSION_MAVERICKS,<br>
>    MACOS_VERSION_YOSEMITE,<br>
> +  MACOS_VERSION_UNKNOWN_NEWER<br>
>  };<br>
><br>
>  MacosVersion GetMacosVersion();<br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div></div>