Re: [PATCH v4 0/7] MR3506: Improve headers compatibility

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about include/specstrings_strict.h:
> -
> - Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA
> - */
> -
>
> +#ifndef __SPECSTRINGS_STRICT_LEVEL
> +#define __SPECSTRINGS_STRICT_LEVEL 1
> +#endif
> -
>
> +#define __field_bcount(size)
> +#define __field_ecount(size)
> +#define __field_xcount(size)
> -
>
> +#define __reserved
> +#define __success(status)
> +#define __range(lb,ub)
> -
>
> +#if (_MSC_VER >= 1400) && !defined(__midl) && !defined(__WIDL__) && (__SPECSTRINGS_STRICT_LEVEL > 0)

My comment was only about `_MSC_VER` part, the rest was fine. Also we don't need the exact version check, just checking if it's defined should be enough.