Returning DBL_MAX doesn't make sense. The traditional C++ implementation and
other spreadsheet products return an error.

Change-Id: I8189c3ecf4aa7b1df00ec2fd8b91030085848bf4

Surely it doesn't make sense, but we should return zero instead?

At the two call sites, there are tests against a zero having been returned,
but not against a negative value. And the return value is even passed as the
nArrayLength parameter to a formula::DoubleVectorRefToken constructor, which
is of type size_t, thus unsigned. Passing for instance -4 to it ends up being
interpreted as 18446744073709551612, which has fun consequences.

Change-Id: Ifca97cd4bae8e921345bcf9d0ca6bb040e3c3b79

Change-Id: I252987d6c5e5da56a83742c96b4e86abbb7108c6

Having these statistical functions perform well is essential in many cases,
and their implementations seem to be correct.

Change-Id: I30afa096295cc163f8c0146e916a77aa411dd07b

Change-Id: Ib71d74d4b66b02a6a680b5ab0c905b987d59cf7f

Change-Id: I43b59b4359a010e7b64df5825ae3ce845e47fc4b

The initialism “TDF” is rather obscure still.

Change-Id: I987e96d869a5126f5d137d188a34b4ae7d4888f8

Change-Id: I11c7c94a52e4be7286a919e61818d08d5b9ad623

Change-Id: I3aacb177d209db74ffadf5284550cdd4972554de

Change-Id: Ie063071cc57d756fd5da20a15533a6857940f676

Change-Id: I9d7d1e16c34eac75652a1275b6e4ac3f7973cb56

Change-Id: I33a740af4ccbcaa9ed4e3ffd79c4f8cda9b37dc4

Change-Id: I7569a20877411a5ceb4a7475d82cb634eb6cb114

In general it should work like this..

Change-Id: Ic45ca6138c1b252ce9d216ce61c7129c055342ef

... similar to ocArrayOpen,ocArrayClose

Change-Id: I10b4cd2f74167cc57ef7a2ed96e0e98f0cd86805

Change-Id: Icfbf088ee8d91d90779dc804b7dff299df1170a4

Change-Id: Ic82ccfcba08ce276eac08609de215ba8edf8e813

Change-Id: Ia9d54fd582f8e195edaa05937ee0f0290e56875f

Change-Id: I62cce7321dfe038d1723c73ba9deb0450a03f0e5

Change-Id: I63b41ea4619098390824f335859d62b4fa8fcc66

Change-Id: I48c716d87ce3e46f61393ae140bf2e05d9fd6241

in order to get the font colors for menus right.

includes revert "set parent styles to fall back to"
This reverts commit 710d0344.

Change-Id: Ica73a30a44a5dd61a8310f2bee74068c406f470a

Change-Id: I187c4d1bef306ec05985cd6e8072d044e12a993f

... makes it hard to clean up these problems.

But the warnings on almost every std::copy and std::transform are
obnoxious, so disable them with _SCL_SECURE_NO_WARNINGS

Change-Id: Ieea611175fb16b78ff972d194767d3d192992de5

Change-Id: Id70289b0c88dd203cac6301303fe7452088f60f9

Change-Id: I07e6757f447ad1c4514f2ddcdf26589190445870

inet_ntoa, gethostbyname, gethostbyaddr are all deprecated in the
Windows SDK 8.1; use the replacement functions unless targetting WinXP.

Change-Id: Ic290ce925835c97f0ca0c611d1fd2675d0045ec7

... i.e. decide based on target instead of SDK version.

Change-Id: I225588d34f961c19e223ed9c1b9a934cd5196cf7

The value set here does not correspond to any valid value in
sdkddkver.h; it is a mystery what setting an invalid value is supposed
to achieve.

Change-Id: I76b1da148806e249870154efda3e6c4fa20010bf

Since with the 8.x SDK it is not possible to target Windows XP anyway
according to MSDN, just set _WIN32_WINNT to _WIN32_WINNT_VISTA so that
we can at least work on cleaning up API calls deprecated in SDK 8.1.

Change-Id: Ic433caf454f57187b91ef6fdec3f3bcde3f1af79

... and remove the definitions in makefiles, except in fpicker, which
defines a different value for the Vista file picker.

The WINVER value is derived automatically from _WIN32_WINNT by SDK
header sdkddkver.h.

Change-Id: I73358a9bf5d070f71821654cae2a29335a754a21

... so that -D_WIN32_WINNT=0x0502 does not break the build.

Change-Id: I423ba9c194dc4928d59e1713255bce9c3150bcc2

Change-Id: Ib6d9233e8c322fd7165fce8763bb2a4eee9a7850

Change-Id: Ie354e0237b97c872936bd0c87ad014ffc2ba56da

Change-Id: I3558ca7fa3d2a2c47d1066e3bd24aeb7619452e0

makes menubar font get correct color for me under f21

Change-Id: I5ef842cc151e07d13a0dca7a982f0e2d95e5c6d6

Change-Id: I64fcac75b96747d7ea7ac857e8a316c73ad82265

Change-Id: I523bdcccc7ad84e9eb9bcfb7c9b436b4614bc97b

Change-Id: Id2f9073969babe7ad5984f87949ed870f9d5a8f7

(better than making it implicitly declared, as defining it defaulted is
deprecated for that case because of the user-declared copy assignment op)

Change-Id: I0c2c4c063e19e3a15b06e75d0c080911acf26ca3