Allow to set DIFFCODE::DIFF flag for unique file

Introduce the flag DIFFCODE::THREEWAY to indicate whether or not to compare three items

Remove unused declarations

FolderCompare: Display whether 2 files in diff row are idential when doing 3-way comparison. related Bitbucket #78

Fix crash when exiting UnitTest x64 version

Merge pull request #52 from GreyMerlin/master

Eliminate worst of Level=4 compilation warnings

Remove warning C4706: assignment in conditional

 * Remove all instances of the level=4 warning `warning C4706:
assignment within conditional expression`

Remove warning C4840: non-portable use of class ...

 * Sample of the 5-line warning message ...

warning C4840: non-portable use of class
'ATL::CStringT<wchar_t,StrTraitMFC<wchar_t,ATL::ChTraitsCRT<wchar_t>>>'
as an argument to a variadic function
note:
'ATL::CStringT<wchar_t,StrTraitMFC<wchar_t,ATL::ChTraitsCRT<wchar_t>>>::CStringT'
is non-trivial
note: see declaration of
'ATL::CStringT<wchar_t,StrTraitMFC<wchar_t,ATL::ChTraitsCRT<wchar_t>>>::CStringT'
(compiling source file MergeEditView.cpp)
note: the constructor and destructor will not be called; a bitwise copy
of the class will be passed as the argument
note: see declaration of
'ATL::CStringT<wchar_t,StrTraitMFC<wchar_t,ATL::ChTraitsCRT<wchar_t>>>'
(compiling source file MergeEditView.cpp)

 * These are warnings that only occur with Warning Level == 4.

Remove warning C4238: nonstandard extension used

 * Eliminate warnings such as ...
 `warning C4238: nonstandard extension used: class rvalue used as
lvalue`

 * These are warnings that only occur with Warning Level == 4.

Remove warning C4239 (2)

 * Eliminate warnings such as ...
`warning C4239: nonstandard extension used: 'initializing': conversion
from 'LangFileInfo' to 'LangFileInfo &'`
`note: A non-const reference may only be bound to an lvalue`
 * And ...
`warning C4239: nonstandard extension used: 'argument': conversion from
'DirItemIterator' to 'DirItemIterator &'`
`note: A non-const reference may only be bound to an lvalue`

 * These are warnings that only occur with Warning Level == 4.

Remove warning C4239: nonstandard extension used

 * Eliminate warnings such as ...
`warning C4239: nonstandard extension used: 'argument': conversion
from 'DirItemIterator' to 'DirItemIterator &'`
`note: A non-const reference may only be bound to an lvalue`
 * These are warnings that only occur with Warning Level == 4.

 * This commit only deals with issues with mixing CString and String
data.
 * Discard the PopString class, a self-admitted hack from
stackoverflow.com, that was simply confusing and not all that "pretty"
anyway.

Merge pull request #51 from GreyMerlin/master

UnitTests can now compile with C++17

UnitTests now finds Plugins correctly

 * The x64 configurations of UnitTests were failing to load the
DisplayXMLFiles.dll library, because the test case was looking in the
old "c:/program files (x86)/winmerge" directory.  This directory only
has a 32-bit implementation of the dll file.

 * This patch allows the test case to now use the "c:/Program
Files/WinMerge" directory.

 * All four configurations x64/win32 and Release/Debug now run
correctly.

UnitTests can now compile with C++17

* The GoogleTest  framework can disable use of the `std::tr1`
class entirely!  This is disabled by specifying
`GTEST_HAS_TR1_TUPLE=0;` in the Preprocessor Definition compilation
property.  The UnitTests project does not use TR1 tuples.

 * The deprecation warnings are to note features scheduled to be
deimplemented in some future C++ standard (possibly as early as
C++20).  They remain valid features in C++17.  These warnings
(and the resulting errors) can be removed by specifying
`_SILENCE_ALL_CXX17_DEPRECATION_WARNINGS;` in the Preprocessor
Definition compilation property.  This allows us to watch and study how
other projects react to the necessary changes (mostly related to
`std::function_name::_Unchecked_Iterators::`).

 * The `GTEST_HAS_TR1_TUPLE=0;` Preprocessor definition is now included
in VS2015 as well, so that both VS2015 and VS2017 are both building
similar software.

Merge pull request #50 from GreyMerlin/master

Old-style procedures and C++17 for more projects

Compile Merge subprojects w/ C++17 language rules.

 * Force the **ShellExtensions**, and the various **Poco** sub-projects
to be compiled with C++17 language semantics.
 * Use `unique_ptr<>` to replace `auto_ptr<>`
 * Compile **Merge** with C++17 when using the **Test* configuration.

 * (unrelated) `coretools.h` is not used by **ShellExtensions**

Eliminate old-style procedure declarators

 * Old-style procedure declarators appeared as ...
`myFunc (a, b, c)`
`   int a, c; char * b;`
`{}`
 * This style is very-very old C language.
 * In all modern C and C++, this procedure would be declared as ...
`myFunc (int a, char * b, int c);`

 * This eliminates level-4 warning C4131: "uses old-style declarator"
 * Eliminate the awkward PARAMS() macro, which was used to simulate the
new style on old compilers that lacked the new style.

 * These patches only effect the DiffEngine

Merge pull request #49 from GreyMerlin/master

Compile Merge with C++17 language rules.

Compile Merge with C++17 language rules.

 * Force the **Merge** project to be compiled with C++17 language
semantics.
 * Ignore future (C++20?) deprecation warnings.
 * Use `unique_ptr<>` to replace `auto_ptr<>`

 * Force the **UnitTests** project to be compiled with the C++14
language semantics.  To move this project forward to C++17 will require
a Google Tests framework that is compatible with C++17.

ccrystaltextmakers.cpp: Fix crash when searching or replacing text after reloading files

Merge pull request #47 from GreyMerlin/master

update to newest VS2017 Release 15.5

VS2017 15.5.0 updated version in .sln file

Fix Boost's checking for new VS2017 Version 15.5.0

* Update 5 (dated Oct 2017) changes _MSC_VER to 1912.  See
https://blogs.msdn.microsoft.com/vcblog/2017/11/15/msvc-conformance-improvements-in-visual-studio-2017-version-15-5/

* This cause lots of compilation messages (neither Errors nor Warnings)
with the text "Unknown compiler version - please run the configure
tests and report the results".  I have *not* reported this new issue to
Boost.org (they will eventually catchup without my help)

* This tiny patch will prevent those compilation messages until a newer
version of Boost becomes available and WinMerge chooses to adopt it.

* Since this has been a hard message to find twice now (previously with
VS2017 15.3.0), I have added a new filter Boost to the Solution
Explorer, containing a very small selection of boost configuration
files.  Boost itself is too large to include either completely, or even
the reasonably small subset of boost files that WinMerge uses.

* Mimic changes also for VS2015 to add the Boost filter to the Solution
Explorer, even though this problem will never occur with VS2015

VS2017 - 15.5.0 kludge for GTest UnitTests

 * Using the latest (and just released) version 15.5.0 of VS2017
introduces the following error into the UnitTests project ...

"error C4996: 'std::tr1': warning STL4002: The non-Standard std::tr1
namespace and TR1-only machinery are deprecated and will be REMOVED.
You can define _SILENCE_TR1_NAMESPACE_DEPRECATION_WARNING to
acknowledge that you have received this warning. "

* MSDN claims that this is a C++17 language standards issue, but I was
not able to solve this problem by specifying the use of the C++14
standard language (the Properties control for this is likewise a new
feature of version 15.5.0) See...
https://docs.microsoft.com/en-us/cpp/cpp-conformance-improvements-2017

* The only simple, short-term solution that I could come up with was to
simply follow the suggestion of defining the _SILENCE_TR1_xxx token as
given above.

* Even though this problem has been discussed in the GoogleTest github
forum since at least 6 June, the underlying problem remains un-resolved
at this hour ...
See https://github.com/google/googletest/issues/1111
See https://github.com/google/googletest/issues/1320
See https://github.com/google/googletest/pull/1311

* I will assume that Google's resolution of this problem will require
WinMerge to adopt the newest GoogleTest framework (which is likely a
good idea anyway).  I will keep an eye on their progress and resolution
and advise accordingly.  In the meantime, this kludge allows WinMerge's
UnitTests project to compile and execute correctly.

ctextmarkerdlg.cpp: Fix an issue in which the list view item is not selected in the Marker dialog box after activating the search marker.

PropGeneral.h: Fix mixed eol-style

MessageBoxDialog.*, PropRegistry.*: Add tooltips

crystaltextview.cpp: Remove a warning

Update *.po files

Add VHDL syntax highlighter

editlib: Fix syntax and diff hilighting not work properly in wrapped lines(2)

Manual: Update 'Comparing and merging text files' section

Manual: Update 'Comparing and merging folders' section

Manual: Update Plugins section

DirCmpReportDlg.cpp: Use OptionsMgr

Add ReportTest.cmd

ccrystaltextview.cpp: Avoid crash when generating a report if editor view width is 0

- Make /noprefs, /ignorews, /ignoreblanklines, /ignorecase and /ignoreeol command line options re-enabled
- Add /config key=value command line option

Merge.cpp: Exit WinMerge after folder comparison or report creation is complete if enabled non-interactive mode

PropGeneral.*: Load language combobox list asynchronously in order to avoid blocking the UI(2)
Avoid possible dead lock.

editor addin.sct: ExecFilterCommand: Fix variable not found error when clicking the Cancel button

Manual: Updaate Locales section

Manual: The option 'Add Shell menu to context menu' no longer exists

Manual: Update Install section (WIP)

Manual: Add Compare image files section (WIP)

Manual: Update Quick start section

DirView.cpp: Remove trailing garbage from a dragged text

Added tag 2.14.0+-jp-142 for changeset f0ac4689d88f

Manual: Update configuration section (WIP)

PropGeneral.*: Load language combobox list asynchronously in order to avoid blocking the UI

Manual: Update FAQ section

Manual: Update shortcut keys section

Manual/screenshots/*.png: Update screenshots(3)

Manual/screenshots/*.png: Update screenshots(2)

Fine-tune the widths of panels in status bar

Manual/screenshots/*.png: Update screenshots(1)

DirView.cpp: Restore lost feature: PATCH: [ 1881454 ] Create pair for unique folder and open them

Merge.rc: &1 -> &1st &2 -> &2nd, &3 -> &3rd

MergeEditView.cpp: Show Copy to Left menu item rather than Copy from Right menu item on Context menu

editor addin.sct: Fix untranslated text

CreateRelease.cmd: Copy releases after destination folder created

Fix some methods in subclass being no longer called

Make internal editor undoable again

- Upload releases to virustotal.com using CreateRelease.cmd rather than BuildArc.cmd
- Use rmdir /s /q command instead of del /s /q to delete subfolders

Remove Check For Update menu item(2)

BuildArc.cmd: Fix vcomp140.dll not being copied to x64-zip-version foolder

Fix BuildArc.cmd and CreateRelease.cmd

Update .hgignore

BuildBin.vs2015.cmd: Revert unintentional changes

Add CreateRelease.cmd

Remove Check For Update menu item

BuildArc.cmd: Use downloaded files

BuidAll.*.cmd: Run unit tests

Korean.po: Change file encoding UTF-8 to CP949

Installer: Remove Japanese help

DownloadDeps.cmd: setlocal

BuildBin.*.cmd: Remove unneeded call of del command

Installer: - Use downloaded files
- Fix url

hg_clone.cmd: pull and update if already cloned

Add WinMerge32BitPluginProxy.exe

Add DownloadDeps.cmd

Merge pull request #45 from sheppaul/patch-1

Update Korean.po

Update Korean.po

Translation is updated.

Merge pull request #44 from GreyMerlin/master

Level 4 Warning Cleanup, first attempt

Cleanup2: C4456 "Decl hides previous local decl"

 * Remove a number of C4456 (level=4) warnings.  Declarations hiding
previous local declarations is generally sloppy/dangerous practice.
 * This commit now eliminates these warnings from WinMerge (but not
from the 'poco' sub-projects.

Cleanup2: C4456 "Decl hides previous local decl"

 * Remove a number of C4456 (level=4) warnings.  Declarations hiding
previous local declarations is generally sloppy/dangerous practice.
 * and a few other minor cleanup changes

Cleanup2: use named item for empty DirItem.size

 * Clarifies code and eliminates warning "C4245: conversion from
'int' to 'Poco::File::FileSize', signed/unsigned mismatch"

Cleanup2: Cleanup 'Output Style' declarations

 * Output Style values (selecting how to format patches that are
generated by the `Tools|Generate Patch` menu dialog) were defined in two
unrelated places (with slightly different names), but need to have the
identical values.
 * Furthermore, five of the eight values are not used by WinMerge.
 * And- the values may get stored into the Registry, so they cannot be
accidentally changed in the future.

 * These changes simplify this situation by basing both definitions on
the `diff.h` values, while leaving the two name groups both existing.
 * All declarations and references to the five unused values are
commented-out

Cleanup2: Uninitialized Local Variables

I've set warning level = 4 for the WinMerge project (but not for the
three 'poco' projects).  It generates LOTS of warnings; many categories
are not interesting and I certainly do not recommend that the Level=4
become an official setting.  But some warning categories do seem
important, either functionally or aesthetically.   So, I'll slowly be
working on the ones that I think are interesting...

 * L4 warning C4701: "potentially uninitialized local variable used".
This commit eliminates all occurrences from WinMerge and UnitTests
projects (but not from the 'poco' projects).

Cleanup2: More Build security checking options

 * Add SDLCheck=true to all compilations for additional "Security
Checking".  See
https://docs.microsoft.com/en-us/cpp/build/reference/sdl-enable-additional-security-checks
 * Make sure that <BufferSecurityCheck>=true.  It had been set to false
for the three poco projects.
 * Both VS2015 and VS2017 project files are changed.

Merge pull request #43 from GreyMerlin/master

Some `size_t` should be `ptrdiff_t`,

Cleanup2: gtest/* files into Solution Explorer

 * Add the ./External/gtest/*.h and *.cc files into the UnitTests
Solution Explorer, with appropriate sub-folders
 * The various *.cc files (except for gtest-all.cc) are marked with
  the 'ExcludedFromBuild' = True  because they are #included
and not compiled directly.
 * Changes are for both VS2015 and VS2017.

Cleanup: Fixup 1: some size_t should be ptrdiff_t

 * Some recent type changes to size_t should have been to ptrdiff_t
(which is signed).
 * In particular, str_pos(), get_coding() and fget_coding() in
  cs2cs.cpp use the value -2 as a return flag.  This was being
improperly handled via size_t.
 * Finding this change led to areas with similar problems, in particular
* next and prev in DirrRangeInfo{} struct
* Open and Close in RxMatchRes{} struct.

Update frhed revision to 43fbdca

WinMerge.*.sln: Add Test solution configuration

ShellExtension: Bump revision to 1.17.0.0

ShellExtension: Make ShellExtension buildable without the source code of WinMerge

Merge7z: Add Merge7.vs2017.*

Merge7z: Rename Merge7z.* to Merge.vs2015.*

Use template version of secure functions