An example case is Array.chunkBySize 3 [|1..4|], the code calculates the
number of chunks to be 2, and then does not copy the final (partial) chunk
as 4 mod 2 is 0. Similarly for Array.chunkBySize 5 [|1..12|], number of
chunks is 3 and 12 % 3 = 0

Changing check to len % chunkSize fixes this.

Fixes #501
Closes #503

This was causing internal execution to fail when directory names
contained various special character.

fixes #486
closes #497

commit a24d85bad0154536fb1ab738e03d1d29e310ea1b
Author: latkin <latkin@microsoft.com>
Date:   Fri Jun 12 15:24:41 2015 -0700

    Revert to all old code

commit 9cf81de567c9dc92f8321b66dca2924b3a8421a3
Author: latkin <latkin@microsoft.com>
Date:   Thu Jun 11 11:52:11 2015 -0700

    Add extra test case for fixed non-inline APIs

commit f552478309ca8cff1edf96822e46d04e4c399bc8
Author: latkin <latkin@microsoft.com>
Date:   Thu Jun 11 10:52:18 2015 -0700

    Use lazy-evaluated resource string for all non-inline APIs

commit 76871e6eb10d889a1812d18ffb8157775315671a
Author: latkin <latkin@microsoft.com>
Date:   Mon Jun 8 19:54:19 2015 -0700

    Add tests for multitargeting scenarios

commit 3d6cfcf0e50dc51a0b10b3e48127fcdd745a5b29
Author: latkin <latkin@microsoft.com>
Date:   Thu Jun 11 10:01:57 2015 -0700

    Roll back 52a72523

On package load, check if any of the custom F# settings are somehow missing
from the settings store.  Explicitly set them to their desired defaults if
they are indeed missing.

Added to the language service package since this is guaranteed
to load any time the project system package is loaded, but the converse is
not true (e.g. open loose F# script file without opening a solution).

fixes #199
closes #491

commit da9f29aed5cf4911f7ec57f0632b7e9ed78fe6bc
Author: latkin <latkin@microsoft.com>
Date:   Wed Jun 10 09:33:28 2015 -0700

    Don't create unused binding in VS 12.0

commit 92c89198a0bdb937a70b0078576df17d854d2f65
Author: latkin <latkin@microsoft.com>
Date:   Tue Jun 9 18:55:54 2015 -0700

    APIs used are new for 14.0, omit them for 12.0 builds

commit a662e18d90fa968b9102c503b63f1dda9c4e9f72
Author: latkin <latkin@microsoft.com>
Date:   Tue Jun 9 17:28:22 2015 -0700

    Code review feedback, and add clickable URLs setting

commit b6697a7c24eceb4413987d0453be9cbea4ebb5d9
Author: latkin <latkin@microsoft.com>
Date:   Mon Jun 8 18:13:35 2015 -0700

    Explicitly set custom F# VS settings if they are missing

- Older NUnit used internally compares collections differently
  - Has trouble with nested collections
  - Enforces that empty collections have the same type (newer NUnit is lax about this)
- Add `NoMT` test tag to various new tests that don't support mixed .NET version testing (i.e. .NET 2/4 multitargeting)
- Make sure all IDE bits build with correct versions of VS SDK dependencies
  - IDE unit tests were all failing with latest VS RC installed
- Various small tweaks to account for versioning and file paths used by VS

closes #482

commit 539225b611d2e2e6a2a00bdd89d9d1fe7911600a
Author: latkin <latkin@microsoft.com>
Date:   Tue Jun 2 12:35:30 2015 -0700

    Add missing project file

commit f8d9fe81c92153fb7fbd6519925b69d4f8a4a827
Author: latkin <latkin@microsoft.com>
Date:   Tue Jun 2 07:27:43 2015 -0700

    Another core unit tests fix

commit f6d830bb4b9672b24b72b7454cdde05955163a4b
Author: latkin <latkin@microsoft.com>
Date:   Thu May 28 13:33:08 2015 -0700

    Use correct, consistent versions of VS binaries

commit 827044ca4daa1f5d1c5e028930e7495ec047e05d
Author: VFSharpTeam <vfsharpteam@microsoft.com>
Date:   Thu May 28 10:01:23 2015 -0700

    More test fixes

commit fe8661f6b2a409b9ddfe4a8e1c857169667d399c
Author: latkin <latkin@microsoft.com>
Date:   Wed May 27 16:36:37 2015 -0700

    First batch

Proper handling of VSUTDCF options in up-to-date check, and safer TryGetActiveConfigurationAndPlatform

 - Use `IVsSolutionBuildManager5.FindActiveProjectCfgName` instead of `IVsSolutionBuildManager2.FindActiveProjectCfg` in `TryGetActiveConfigurationAndPlatform`, as it is not sensitive to sync context
 - Give proper consideration to the VSUTDCF options flags passed to QueryStartUpToDateCheck

fixes #471
closes #479

partially addresses #106
closes #477

commit 5505b7174d4de7c1bf4e7e887886a08db04abb7a
Author: Don Syme <donsyme@fastmail.fm>
Date:   Sun May 31 16:48:05 2015 +1000

    fix tests for invalidation handlers

commit 0871018e0abd9e88a0d165fb57d8240c7d1c74a5
Author: Don Syme <donsyme@fastmail.fm>
Date:   Sun May 31 16:35:47 2015 +1000

    add diagnostic counters to signature

commit e8afa9c2ea538e3433ff2a0dc36065f7404c30af
Author: Don Syme <donsyme@fastmail.fm>
Date:   Sun May 31 15:07:35 2015 +1000

    Fix memory leak: not disposing even subscriptions to invalidation events

This was added a while back because we had the opposite problem: items were
mysteriously appearing directly under the root. See http://visualfsharp.codeplex.com/workitem/118
At some point another fix must have restored the expected 'Visual F# Items' node. Thus
we now need to revert the original fix.

fixes #282

fixes #392
closes #470

commit a75cb385bfe60988eec5080c724fdadf877355e1
Author: latkin <latkin@microsoft.com>
Date:   Tue May 26 13:32:08 2015 -0700

    Update w/ code review feedback

commit d2e159101920966c9a873c832531f1c131e4babc
Author: latkin <latkin@microsoft.com>
Date:   Fri May 22 16:45:19 2015 -0700

    Add back keyboard shortcut and command handling for 'Send line to interactive'

fixes #238
closes #434

commit 3150fb2987e4e7afb921695c2def88d3d59ae90d
Author: Don Syme <donsyme@fastmail.fm>
Date:   Tue May 19 13:06:35 2015 +0100

    update fix to 238

commit 663adf209494e8366976fb6f87ca6064e6b62f99
Author: Don Syme <donsyme@fastmail.fm>
Date:   Sat May 9 19:11:06 2015 +0100

    Fix 238

fixes #453
closes #454

closes #452

fixes #445
closes #447

Implementation of automated cross-F#-version testing mentioned in #386. This is adapted from an existing strategy used in the past for validating cross-CLR-version scenarios.

Approach ("downtarget")
 - Build each test case against vPrevious FSharp.Core (4.3.1.0 for now)
 - Run resulting EXE as-is, it will bind to vPrevious

Approach ("redirect")
 - Build each test case against vPrevious FSharp.Core (4.3.1.0 for now)
 - Run resulting exe inside of process that targets vCurrent, with binding redirects

closes #446

commit 423c10d2550bfbdbb6bedd406e56e3d928dd72cb
Author: latkin <latkin@microsoft.com>
Date:   Mon May 18 15:27:53 2015 -0700

    Removing from CI build

commit e29e727a7b64d2215fca7ee61bd6f5805fe82c3b
Merge: 133a57a db6c198
Author: latkin <latkin@microsoft.com>
Date:   Mon May 18 15:26:27 2015 -0700

    Merge branch 'crosstarget-test' of https://github.com/latkin/visualfsharp into latkin-crosstarget-test

    Conflicts:
    	tests/fsharpqa/Source/CodeGen/EmittedIL/Misc/env.lst

commit db6c19851ee0702961afb8f281dfc0c0b37dde2b
Author: latkin <latkin@microsoft.com>
Date:   Tue May 12 17:06:34 2015 -0700

    More script fixes

commit 105051fb713b467eeca305c3fc79e0482b767d74
Author: latkin <latkin@microsoft.com>
Date:   Tue May 12 16:29:03 2015 -0700

    Fix for RunTests.cmd

commit 65c84533673cca7dfeb0740bdeae78204700c9e8
Author: latkin <latkin@microsoft.com>
Date:   Tue May 12 15:39:30 2015 -0700

    Omit some cases from CI build

commit a86ef97767eb9911e8bc60c3c207d923e9196845
Author: latkin <latkin@microsoft.com>
Date:   Tue May 12 14:17:07 2015 -0700

    Add to CI build (can be removed)

commit ab100d28afb9e82be8928985c090f8ff30277be9
Author: latkin <latkin@microsoft.com>
Date:   Tue May 12 14:16:49 2015 -0700

    Update log paths

commit 4fe04d69ecbb075f8ddfc376cc8388f20dca9d3a
Author: latkin <latkin@microsoft.com>
Date:   Tue May 12 14:12:24 2015 -0700

    Comment, use better name for test suites

commit 907c11d9c927b445e605ad9299f9efec64c195d5
Author: latkin <latkin@microsoft.com>
Date:   Tue May 12 13:58:46 2015 -0700

    Using exe impl of ExecAssembly, much faster, allows for easier platform targeting

commit e5041d4261a062852365279fdd64a448966609a6
Author: latkin <latkin@microsoft.com>
Date:   Mon May 11 17:36:40 2015 -0700

    Support for targeting downlevel, then executing redirected back to vCurrent

commit 3577b5a1a1e979d806cb6d46b92373aaa95ca379
Author: latkin <latkin@microsoft.com>
Date:   Mon May 11 16:38:25 2015 -0700

    Detect obviously incompatible test cases in RunAll, instead of requiring annotation

commit 883e1bcc87a1866b4cedfa145a3b88cda950c4c8
Author: latkin <latkin@microsoft.com>
Date:   Mon May 11 16:21:53 2015 -0700

    Marking more tests that can't be run cross-version

commit 3ed9c094406e905863d8e4dea20a8d9561a32e37
Author: latkin <latkin@microsoft.com>
Date:   Fri May 8 17:53:06 2015 -0700

    Start ignoring certain tests that do not support cross-version

commit ae153f3e6d3ff64e63f333ffbbf643a39f61ac74
Author: latkin <latkin@microsoft.com>
Date:   Fri May 8 17:50:05 2015 -0700

    Add support for auto-tagging a test with 'FSI' based on presence of 'FSIMODE' var

commit 63753f33eb928015e572137e0cb81449e24a9ff4
Author: latkin <latkin@microsoft.com>
Date:   Fri May 8 16:10:39 2015 -0700

    Updates to RunTests for cross-targeting

commit 46c786b145f3eb9092e5c00240d70104573b8e19
Author: latkin <latkin@microsoft.com>
Date:   Fri May 8 15:10:53 2015 -0700

    Fixing specification of compiler flags in tests

    SCFLAGS should be used by individual tests to specify required compiler or fsi flags
    ISCFLAGS and IFSIFLAGS should be reserved only for infrastructure use, they apply globally

    ISCFLAGS was populated with '-g --optimize' (emit debug info and enable
    optimizations), but is now clear.  A few tests now need to specify
    these flags explicitly.  This is a better default - tests should not need
    to rely on or know about globally defined flags.

commit 118c1f08c7c8fa055f088547b65a442417ab565a
Author: latkin <latkin@microsoft.com>
Date:   Tue May 5 17:37:08 2015 -0700

    Start working on cross-targeting automation

fixes #212
closes #444

commit fb8a73e0e0c76f2b12ae2adc676934c73d712375
Author: Don Syme <donsyme@fastmail.fm>
Date:   Tue May 12 18:53:28 2015 +0100

    fix 212 - enable tests properly

commit 2ec23c13844abfae7cb9c7d6e281e6850f6043e1
Author: Don Syme <donsyme@fastmail.fm>
Date:   Tue May 12 18:50:07 2015 +0100

    fix 212 - record initialization order

fixes #34
closes #443

commit 4f6c739e88a7587604ead96e2d61f7254103327d
Author: Don Syme <donsyme@fastmail.fm>
Date:   Tue May 12 17:38:00 2015 +0100

    Fix 34 - extension member duplicate check (5)

commit 17b1cdb6129521df724e6fe9dac7b7c015f44798
Author: Don Syme <donsyme@fastmail.fm>
Date:   Tue May 12 17:37:19 2015 +0100

    Fix 34 - extension member duplicate check (4)

commit e9bfa8191249533d2ddf8168bc664cc0952dfbb0
Author: Don Syme <donsyme@fastmail.fm>
Date:   Tue May 12 17:35:46 2015 +0100

    Fix 34 - extension member duplicate check (3)

commit 8d8533213dcbaf97e510b0629e19f1fea520e682
Author: Don Syme <donsyme@fastmail.fm>
Date:   Tue May 12 17:25:32 2015 +0100

    Fix 34 - extension member duplicate check (2)

fix 147 - consider implicit ctors in type check env when determining what type variables can be generalized

fixes #147
closes #435

commit 82932c63d118da5a224e4563ab7ca215e44c1c70
Author: Don Syme <donsyme@fastmail.fm>
Date:   Tue May 12 15:46:30 2015 +0100

    update baseline

commit f38e5a4cbbddb56c6c277f3bc34a2e768a333576
Author: Don Syme <donsyme@fastmail.fm>
Date:   Sat May 9 20:18:16 2015 +0100

    fix 147 (tests)

commit e18ca740fa14df394f280d811b790c23193bd47f
Author: Don Syme <donsyme@fastmail.fm>
Date:   Sat May 9 20:16:06 2015 +0100

    fix 147

fixes #316
closes #433

commit 6d9584caf54e0655c8c67375bc622dc3e38c2c28
Author: Don Syme <donsyme@fastmail.fm>
Date:   Sat May 9 18:14:42 2015 +0100

    fix 316

commit da567164f7c5d79119c13f5f5c2373b0dedd77d3
Author: Don Syme <donsyme@fastmail.fm>
Date:   Sat May 9 17:45:54 2015 +0100

    Fix 316

fixes #299
closes #432

commit 59fc4a9e50ab77c079d32946828d33793082a22a
Author: Don Syme <donsyme@fastmail.fm>
Date:   Sat May 9 17:30:23 2015 +0100

    fix 299 (3)

commit e9012242db4086f23c5d4e0ea8310e0337f1f3d6
Author: Don Syme <donsyme@fastmail.fm>
Date:   Sat May 9 17:26:52 2015 +0100

    fix 299 (2)

commit 8ddd8014c3b67f3e83f9e0e5f9211b7e3928f0a9
Author: Don Syme <donsyme@fastmail.fm>
Date:   Sat May 9 17:26:32 2015 +0100

    fix 299

fixes #342
closes #431

fixes #362
closes #430

commit 22bf8ebf25de08905342d5984052b683479e5a65
Author: Don Syme <donsyme@fastmail.fm>
Date:   Sat May 9 12:08:27 2015 +0100

    fix 362

commit bc28158073b8605dc03a0816a1679a729595fda0
Author: Don Syme <donsyme@fastmail.fm>
Date:   Sat May 9 12:06:27 2015 +0100

    fix 362 (2)

commit 6bfdeac2c8718032be43c75c6a09ea66866bbbc5
Author: Don Syme <donsyme@fastmail.fm>
Date:   Sat May 9 11:58:48 2015 +0100

    fix 362

fixes #420
closes #423

commit 90868139ed5a07291142a27ef8ff4e7624e8efb5
Author: latkin <latkin@microsoft.com>
Date:   Thu May 7 17:21:49 2015 -0700

    Fix up surface area tests

commit 7b544fff68021f8672ec4ed29ad82237babd77f9
Author: latkin <latkin@microsoft.com>
Date:   Wed May 6 17:12:07 2015 -0700

    Add/adjust tests

commit 5ab7886e3a5f97c3153605e18bf9b5e0fb762023
Author: latkin <latkin@microsoft.com>
Date:   Wed May 6 14:38:34 2015 -0700

    Remove unnecessary access modifier conversions for abstract class ctors

fixes #246
closes #391

commit 2c37b20d129a231a16d2e64cbf2f007aa73e40c4
Author: enricosada <enrico@sada.io>
Date:   Wed Apr 29 18:57:04 2015 +0200

    fix invalid already rendered folder error

    fix invalid already rendered folder error when a project contains
    folders with same name

commit b8d03011db57b3b5df66c31d3a1564352c947cd1
Author: enricosada <enrico@sada.io>
Date:   Wed May 6 23:30:12 2015 +0200

    add regression tests

fixes #11
closes #374

commit f677819e9f4258eeee5577c23ad107132a901b07
Author: enricosada <enrico@sada.io>
Date:   Wed Feb 25 00:02:18 2015 +0100

    fix app.config up-to-date status

commit bc419357eccfd52c6e46103409521052f5321071
Author: enricosada <enrico@sada.io>
Date:   Wed May 6 17:17:25 2015 +0200

    fix test build up-to-date should fail if the .exe.config doesn't exists

commit 8219016c1f3009218a26545c1c96b6dddc26d515
Author: enricosada <enrico@sada.io>
Date:   Wed May 6 16:47:31 2015 +0200

    add assert AssertBuildSuccessful to check if a build is successful

This hijacks standard shortcuts to open toolbar menus, in particular the _D_ebug menu.
Unfortunately there are no decent keybindings available that aren't already mapped. Fallback
is to just remove our keybinding and let users add a custom binding if they want to.

vsixlangpack files were of the wrong format. FOr VS 2015.

fixes #411

fixes #417

closes #414
closes #404

commit 7583a2d60eb968fec2d4ab00a046682eef3f9a19
Author: latkin <latkin@microsoft.com>
Date:   Tue May 5 10:18:12 2015 -0700

    Add comment on use of <gcConcurrent>, and enable for tests

commit d5de019fd8aa96d8b4c90b280163a66804268e1c
Author: latkin <latkin@microsoft.com>
Date:   Mon May 4 11:39:31 2015 -0700

    Setting server GC mode for compiler

closes #390

commit a9c9d606fac0d439d768fc2072258304b2405340
Author: Max Malook <community@malook.de>
Date:   Wed Apr 29 04:43:26 2015 +0200

    handle AllowNullLiteral(false) correctly, fixes #383

commit 574b9b395d6df64a259ae68ca77e7b083c6105f1
Author: Max Malook <community@malook.de>
Date:   Wed Apr 29 01:34:54 2015 +0200

    add tests to verify correct usage of new argument on AllowNullLiteralAttribute

When the assembly reference set for the compilation is incomplete, some
nterfaces to types relevant to compilation may lie in assemblies outside
the assembly reference set. This applies particularly to private ("internals
visible to") interfaces found in .NET assemblies.

This causes very substantial usability bugs in practice as various parts of
type inference and other checking "give up" when you get this condition. The
C# compiler doesn't give up in the same way.

In most cases it is reasonable to simply skip interfaces-that-lie-outside-the-
set-of-referenced-assemblies during F# compilation. Skipping unresolvable
interfaces is pretty much harmless: any substantive analysis on the interface
type (such as implementing it) will require the assembly holding the interface
type.

There are some exceptions: if an interface I1 lies outside the referenceable
set and you try to implement I2 inheriting from I1 then we'd better not skip
I1. Indeed if you even try to find the methods on I2 then we'd better not skip
I1. These are covered by "FoldPrimaryHierarchyOfType" in the code.

fixes #337
closes #356

commit dd5205c769828e2e16e736c126cb62d68e7beb87
Author: Don Syme <donsyme@fastmail.fm>
Date:   Thu Apr 23 23:53:59 2015 +0100

    add test case

commit db28771c75d022247dee6dea60a89b8c29fab4b5
Author: Don Syme <donsyme@fastmail.fm>
Date:   Fri Apr 10 12:18:50 2015 +0200

    skip unloadable interfaces (2)

commit 18a47124480efdf58a75ce2fa7273c6f7550c1c0
Author: Don Syme <donsyme@fastmail.fm>
Date:   Fri Apr 10 11:53:39 2015 +0200

    skip unloadable interfaces

closes #384

reference System.Runtime in scripts to allow ExtensionAttribute to be resolved when referencing PCL assemblies

fixes #381
closes #382

- Eliminate the optimization that removes try/catch blocks around code assumed to be effect-free
- Reading fields is treated as an effect if the implementing type has [<AllowNullLiteral>]

fixes #368
closes #376

commit 6c94628b5c30b28abde11cd3852b4e09a3c1c00e
Author: latkin <latkin@microsoft.com>
Date:   Wed Apr 22 13:14:38 2015 -0700

    Fixes and updates from code review

commit 80ba79ee4c4423a42e0e6698546a0aace75c264d
Author: latkin <latkin@microsoft.com>
Date:   Tue Apr 21 16:58:16 2015 -0700

    Tests for effects and codegen of try/catch

commit 5597aa08bf9474edacba465488e97033740aaa9b
Author: latkin <latkin@microsoft.com>
Date:   Tue Apr 21 13:13:59 2015 -0700

    Reading fields from a nullable type is an effect

commit ff5fdc653f99cbe7257e18dd3f55e0070df8d6cc
Author: latkin <latkin@microsoft.com>
Date:   Tue Apr 21 12:10:49 2015 -0700

    Remove try/catch elimination optimization

fixes #370
closes #373

commit 3404c8ab3857e6408cc1602cb69f3999813f0cb5
Author: latkin <latkin@microsoft.com>
Date:   Tue Apr 21 10:32:12 2015 -0700

    Optimization to remove a branch from comparison

commit a693443a622679ba711539fe11152b134e796dc3
Author: latkin <latkin@microsoft.com>
Date:   Mon Apr 20 21:10:07 2015 -0700

    Implement proper ER-mode comparison for floating point types

    fixes #370

fixes #338
closes #348

commit 9b68fc52e7fe80d8401b139ec4af78ebc3ce73b3
Author: latkin <latkin@microsoft.com>
Date:   Wed Apr 15 10:40:46 2015 -0700

    Use func, not lazy

commit 55b811d8f556fae6c3c4569957a12d12e8e9df00
Author: latkin <latkin@microsoft.com>
Date:   Tue Apr 14 13:11:29 2015 -0700

    Use dedicated type for lex result

commit 49424b54ccaffa75c9be44960204f66d350764aa
Author: latkin <latkin@microsoft.com>
Date:   Tue Apr 7 18:28:21 2015 -0700

    Fix lexing for unicode escape sequences

    fixes #338

    Changes lexing of unicode escape sequences to match the F# spec (which says things should work the same as C#).

    - For short escape sequences, directly encode the hex value into a char
    - For long escape sequences, validate that the total codepoint is <= 0x0010FFFF
      - If it is, follow same logic as before (which was correct)
      - If it isn't, issue an error (same as C#)