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

Merge pull request #42 from GreyMerlin/master

Compile & Link without Level=3 warnings

Cleanup: Last minute tweaks...

 * change a few more functions from long to size_t
 * add a few ASSERT statements where appropriate.  These make sure that
casting a size_t to an int or long will not truncate large values.
 * refactor a few places where the size_t changes could be better
 * fix one usage of _tsplitpath_s() that was wrong.

Cleanup: UnitTests compiles w/o warnings

 * Project properties changed to use /EHa (Async) for all configurations
 * Debug|X64 changed to use Edit-and-Continue
 * Changes made to both VS2017 and VS2015

Fix: Don't mark Tree Mode as checked if disabled

 * Symptom: in the View menu, the Tree Mode entry could be marked with
the check mark, even if the menu item has been disabled (by
un-selecting the Include Folders item on the File Open dialog).
This is confusing.

 * Fix: test m_bRecursive (i.e. Include Subfolders) to determine if
SetCheck(FALSE) or SetCheck(m_bTreeMode) is needed.

Cleanup: copy vs2017 *.filters changes to vs2015

 * a recent small change to Merge.vs2017.vcxproj.filters is copied to
the corresponding vs2015 file.

Bugfix: passing NULL to strlen() causes fault

 * Symptom: a _DEBUG build could fault when calling
strlen(switch_string) in procedure begin_output().

 * Analysis: switch_string (an external char*) might never be give a
value if there are no command-line parameters.

 * Quick Solution: declare local char* mySwitch initialized with either
a copy of switch_string pointer or a pointer to the empty literal ""

Cleanup: remove all level-3 warnings

 * Eliminate 4996 deprecation warnings by appropriate function name and
parameter changes...
StrDup() -> _tcsdup()
strdup() -> _strdup()
_tfopen() -> _tfopen_s()
_topen() -> tsopen_s()
read() -> _read()
close() -> _close()
_itot() -> _itot_s()
_tcsncpy() -> _tcsncpy_s()
_tcscpy() -> _tcscpy_s()
_i64tot() -> _i64tot_s()
_stscanf() -> _stscanf_s()
localtime() -> localtime_s()
_tcslwr() -> _tcslwr_s()

 * remove unnecessary #pragma warning(disable:4355)

 * All changes tested for both vs2015 and vs2017

Cleanup poco: Remove level-3 and linker warnings

 Foundation ...
 * Delete all #pragma warning specifications
 * Repaired all instances of 4996 deprecation warnings...
std::sprintf() -> ::sprintf_s()
controlfp() -> ::controlfp_s()
localtime() -> ::localtime_s()
std::sscanf() -> ::sscanf_s()
 * Remove debug-only warning in UUID.cpp by replacing swap_ranges()
with explicit loop on swap()
 * Remove linking to ws2_32.lib; it provided no features but caused
warnings with iphlpapi.lib
 * Removed ByteOrder.cpp, String.cpp, SignalHandler.cpp from
Foundation compilation as they provided no accessible code.

XML ...
  * Remove XMLString.cpp from XML compilation; it provides no accessible
code.

Util ...
 * Change std::sprintf() to ::sprintf_s()

WinMerge...
 * All changes tested for both vs2015 and vs2017

Cleanup `size_t`: Fewer `#pragma warning` directives

 * Remove various `#pragma warning` directives that relate to
signed/unsigned

4018 - signed/unsigned mismatch
4100 - unreferenced formal parameter
4189 - local variable initialized but unreferenced
4244 - argument conversion, loss of data
4355 - `this` used in base member list
4512 - assignment operator could not be generated
4520 - non-existing error !!
4711 - inlining notice (off by default)

 * use `size_t` as appropriate to eliminate warnings exposed above.
 * use `static_cast<int>` as appropriate

Additional, unrelated ...
 * `_AFX_NO_MFC_CONTROLS_IN_DIALOGS`was used twice for 'X64 Release'
configuration.

Cleanup: Eliminate _CRT_xxx deprecation (3)

 * lost in recent merge with Sdottaka's latest commit:
3f0ab7f9cf3e84ca81af1090d9d0c1dcb8efef9c
 *This again eliminates these symbols in the *.vcxproj files ...
_CRT_SECURE_NO_WARNINGS
_CRT_NONSTDC_NO_WARNINGS
_CRT_SECURE_NO_DEPRECATE

Cleanup: Misc `size_t` related issues

 * eliminate conversion warnings for `size_t` related expressions.
 * convert various `int` and `long` to `size_t`
 * add `static_cast<int>` or add `static_cast<long>` as appropriate
 * classical `(int)`-style casting is used in *.c files, of course

While this commit changes various parameters and data items to `size_t`,
the focus of these changes was to completely eliminate the various
warnings. This code is still fundamentally restricted to 2GB of text.

There is still much work to do to generalize the fundamentals of
WinMerge to be fully `size_t` capable (for the X64 platform).

Cleanup: Eliminate 'C4091' warnings

The "warning C4091: 'typedef ': ignored on left of 'tagGPFIDL_FLAGS'
when no variable is declared" is generated within the Microsoft
<shlobj.h> include file, but only for "toolsets" that support XP.  I
have no idea why this is true?!?!

 * A `#pragma warning` now is pushed and popped around all
`#include <shlobj.h>` statements to prevent this meaningless
and obnoxious warning message.

Cleanup CrystalEdit: `size_t` related issues

 * eliminate conversion warnings for `size_t` related expressions.
 * convert various `int` and `long` to `size_t`
 * add `static_cast<int>` or add `static_cast<long>` as appropriate
 * classical `(int)` casting is used in ASSERT, for simple convenience.
 * add a few ASSERT() statements to assure that various `size_t` values
remain within the positive `int` value range.
 * Since `-1` is often used as a "size flag", these changes now
explicitly test for `x != -1` rather than `x >= 0`

While this commit changes various parameters and data items to `size_t`,
the focus of these changes was to completely eliminate the various
warnings. This code is still fundamentally restricted to 2GB of text.

There is still much work to do to generalize the fundamentals of
CrystalEdit to be fully `size_t` capable (for the X64 platform).  Two
items of important note:
  1. MFC's CPOINT structure is limited to `LONG` values.
   The `.x` member should become `size_t`.
2. MFC's CSTRING data is limited to `int` maximum length

Cleanup poco/XML: `size_t` related warnings

 * properly cast return type for `XML_GetCurrentByteIndex`

Cleanup: Eliminate _CRT_xxx deprecation (2)

 * lost in Merge with -jp-141

Cleanup poco/Foundation: 'size_t' related warnings

 * eliminate conversion warnings for `size_t` related expressions.
 * convert a few `int` to `size_t`
 * SharedMemory files `_size` is now LONGLONG and mappings are correctly
created for files larger than 65kb.

Cleanup: Eliminate _CRT_xxx deprecation situations

* The VC compiler began emitting Warnings for various deprecated CRT and
POSIX function names beginning with VS2005.  It was convenient to
ignore these warnings by defining the following preprocessor symbols in
the *.vcxproj files ...
_CRT_SECURE_NO_WARNINGS
_CRT_NONSTDC_NO_WARNINGS
_CRT_SECURE_NO_DEPRECATE

 * This change eliminates those symbol definitions, and appropriately
upgrades the code to properly use the modern, security enhanced,
functions.
 * Most of the existing code properly uses TCHAR, however some areas
explicitly use 'wchar_t' and some areas still use the old 'char' type.
 * The areas that still use 'char' need further investigation and
possible repair as the rest of the program completely uses Unicode.

Cleanup: Remove useless AFX_INSERT_LOCATION

 * Most *.h files do not have MFC's AFX_INSERT_LOCATION indicator
 * And its generally useless
 * Now its gone, with general cleanup at the end of *.h files

Cleanup: Eliminate silly *.inl files in CCrystalEdit

 * Elimination #defines for INLINE that always ended up blank
 * Merged code from *.inl files into appropriate *.h files
 * Eliminate ancient CE_DLL_BUILD stuff
 * Appropriate changes to *.vcxproj and *.vcxproj.filters files to
account for removal of the five *.inl files, for both
VS2015 and VS2017