__in, __out, and a variety of related macros (like __in_opt and __in_out) are Microsoft macros that assist in static analysis (they don't do anything besides help static analysis). MinGW defines these to no-ops, but two (__in and __out) cannot be defined in C++ code because they conflict with libstdc++ argument names. (See line 222 of https://sourceforge.net/p/mingw-w64/mingw-w64/ci/cef07470773f3c56e5189125f6b82bf278265e69/tree/mingw-w64-headers/include/sal.h)

Comment on attachment 8910909 [details] Bug 1402065 Add the pdfium patch to the repo and update.sh https://reviewboard.mozilla.org/r/182384/#review188432

Comment on attachment 8910908 [details] Bug 1402065 Remove __in and __out from our in-tree copy of pdfium https://reviewboard.mozilla.org/r/182382/#review188434

Pushed by ryanvm@gmail.com: https://hg.mozilla.org/integration/autoland/rev/ec31e5b2ba19 Remove __in and __out from our in-tree copy of pdfium r=jwatt https://hg.mozilla.org/integration/autoland/rev/0b1d706bd1e5 Add the pdfium patch to the repo and update.sh r=jwatt

https://hg.mozilla.org/mozilla-central/rev/ec31e5b2ba19 https://hg.mozilla.org/mozilla-central/rev/0b1d706bd1e5