Change-Id: I9466931485ed5421be589def16185d5c036bdcb8

Change-Id: I3fdf72b014d2ca0b0bd96fc5eba5606b13b32258

Change-Id: Ic87e8aacb2e8215751422969e78cc58e18fc5a42

Change-Id: Idf72ab1a311fcbbc81b16d9a3c1e78aaa10332c0

Change-Id: I2ef42eca934588b219de52dfc7c373395118cc32

Change-Id: I42ae8b5d1769eba3e5af68b3b183c77f40c35ca8

Appears to no (longer?) be relevant here, as reported by clang-cl.

Change-Id: I339f8b04eb733a9c6a97fcd4b9b4758e6a84d5c3

Change-Id: I944d7bd40a297dac97b954196a0e8e286f0ba712

<https://msdn.microsoft.com/en-us/library/windows/desktop/ms647466%28v=vs.85%29.aspx>
"About Strsafe.h":

  When you include Strsafe.h in your file, the older functions replaced by the
  Strsafe.h functions will be deprecated. Attempts to use these older functions
  will result in a compiler error telling you to use the newer functions. If you
  want to override this behavior, include the following statement before
  including Strsafe.h.

  #define STRSAFE_NO_DEPRECATE

Appears to no (longer?) be relevant here, as reported by clang-cl.

Change-Id: I94a576b71a2a34edd6009059fc5047ba59ec0f12

Change-Id: Ie346bca98a2da91e20ddb533ab19720f193291d4

Change-Id: Ib333ad2880e0a1cbfed2f6a919ca2b92f42dde45

Change-Id: I0ac1ba124ba452deeeeaa473ccbbf865e490c447

Change-Id: I1e26a805ac96f1c57d2f7c795b598c4eaa8d2a3e

Change-Id: I4893f538911449953fadf4cf10f6adb819bc023f

Where wglChoosePixelFormatARB is defined as a macro expanding to
__wglewChoosePixelFormatARB (as WGLEW_GET_FUN is just expanding to its
argument), and __wglewChoosePixelFormatARB is declared in global scope in
workdir/UnpackedTarball/glew/include/GL/wglew.h.
itself) in workdir/UnpackedTarball/glew/include\GL/wglew.h

Change-Id: I0c4d09e9112c2233d25a262ea1f2b35bdf49645c

Change-Id: Ibe968bdb968a0a86723fcc2c9fa958261724c7da

Change-Id: Ie6a35ce1fece5de31f455d88ab3e12010b8047f5

According to
<https://msdn.microsoft.com/en-us/library/windows/desktop/aa383681%28v=vs.85%29.aspx>
("Enabling STRICT"), it is used in combination with including windows.h, so
maybe this #define STRICT (which was there ever since
a934115b "Initial revision," w/o also including
windows.h) was always unnecessary (as reported by clang-cl).

Change-Id: I02cf2584fe2fd52e9513791b18fdf6e7982e89b3

Change-Id: I46f7b68aded50abc0015b458507adbf84d0ffe76

Change-Id: I40f92daa60a08d93ea7bbe309fd16fa7ef27e78b

"VC_EXTRALEAN can only be used in MFC projects."
(<https://support.microsoft.com/en-us/kb/166474> "How to use VC_EXTRALEAN and
WIN32_LEAN_AND_MEAN to enhance the build process in Visual C++")

So presumably indeed not needed here (as reported by clang-cl).

Change-Id: Id98c6e9e862e716cf5e238623c6ac4ca0603fd63

see c665a1bc for __STDC__ not defined under
MSVC/clang-cl.

Change-Id: I65d8a3b33ff40fc80c52b294b0c5bc75d9f5cf2f

Change-Id: I57650fe955a31f433d611d9e74a0762d349bed64

Change-Id: I62fbead59ac83147da6da984f89b64b1d7e4f90d

For MSVC/clang-cl: "__STDC__: Indicates conformance with the ANSI/ISO C99
tandard. Defined as the integer literal constant 1 only if the /Za compiler
option is given and you are not compiling C++ code; otherwise is undefined."
(<https://msdn.microsoft.com/en-us/library/b0084kay.aspx>)

Presumably fine to just replace #if with #ifdef here (as is used in all the
other cases surrounding it), and not care whether it actually expands to true.

Change-Id: Ie58c76d7e72321b06a11a370fd28221e85fcf871

Whatever those _POSIX_ for _MSV_VER were supposed to be good for (have been in
there at least ever since 4f4b7415 "Moved cpp
from cpp to soltools"), but cause -Wunused-macros at least when building with
clang-cl.

Change-Id: I356bf663cd1e86298ba72a179f46bb9d0c10016b

Change-Id: I5402d52ec42fa27c58b276eeb22eeb270c3d7948

Change-Id: Ib20a02a846b745b65ffcf9ec1d45b39a6e0c50e4

Change-Id: Icac0f029461c886ebb03aedcd54189116b66d8a3

Change-Id: Icba6646e183bc283e3c86a223a7f76306123200b

Change-Id: Icc0a4af0e5995116d65d0dea0881e10ca8d92232

Modified Custom Animation ListBox entry to include multiple lines.
Added Effect names in Custom Animation List Control.

Change-Id: Ic974ffc9fca6803ff76135620df1fbe9edcde565
Reviewed-on: https://gerrit.libreoffice.org/22403Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>

Remove L from integer literals if it does not produce warnings
and does not change semantics

Change-Id: I9573a47d3e3401993a1f0fbcf7df4be20f9e6eba
Reviewed-on: https://gerrit.libreoffice.org/22867Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>

and teach the plugin about code that takes the address of a function

Change-Id: Ia9d5afef44520aca236659e8176f1e27135ef4fc
Reviewed-on: https://gerrit.libreoffice.org/22861Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>

Change-Id: I7d0f7e9a1bf9d2d590337cf4bb8cc4860c05444e
Reviewed-on: https://gerrit.libreoffice.org/22864Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: David Tardon <dtardon@redhat.com>
Tested-by: David Tardon <dtardon@redhat.com>

Change-Id: Ibc5844d4e4a89fae7bbc8d2b854384d0c433500f
Reviewed-on: https://gerrit.libreoffice.org/22883Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Xisco Faulí <anistenis@gmail.com>

Iterator category tags carry information that can be used
to select the most efficient algorithms for the specific
requirement set that is implied by the category. OsiCuts
defines bidirectional category tag, but doesn't implement
operator--(). This is illegal: [1].

* [1] http://paste.openstack.org/show/489235

Change-Id: I68a6d297d5c33848c4b8a324e081c5118fd936a4
Reviewed-on: https://gerrit.libreoffice.org/22882Reviewed-by: Michael Stahl <mstahl@redhat.com>
Tested-by: Jenkins <ci@libreoffice.org>

Change-Id: I3e6c84c1f49e9523e881ce12dbb2f7e0d1df956e

maybe we should be using more precision, but we haven't
been in the past

Change-Id: Icf1e1e95099c783435e04f60d8ade755a8d2d4d5

Change-Id: I8820d435e2f26d04ac3eccb710383399d050e020