[Webkit-unassigned] [Bug 182479] [Win] Fix MSVC's treating __attribute__((warn_unused_result))

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=182479

--- Comment #6 from Yousuke Kimoto <Yousuke.Kimoto at sony.com> ---
(In reply to Alex Christensen from comment #3)
> maybe we should check _MSC_VER instead for people using clang on windows.
Table about _MSC_VER  and Visual Studio version:
https://blogs.msdn.microsoft.com/vcblog/2015/12/04/clang-with-microsoft-codegen-in-vs-2015-update-1/
https://cpprefjp.github.io/implementation.html#visual_cpp

I have two questions about _MSC_VER.

1. Is VisualStudio 2017 a recommended IDE on windows ?

2. When clang is used on windows, which template should be chosen, a) or b)?
  a) template<typename T> inline WKRetainPtr<T> adoptWK(T) _Check_return_;
  b) template<typename T> inline WKRetainPtr<T> adoptWK(T) __attribute__((warn_unused_result));

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20180206/a88d698c/attachment-0001.html>