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

Use '#pragma once' in more .h files

UnicodeString.cpp: Fix crash when generating HTML report.

Remove git_clone.cmd

Fix the revisions of external sources

BuildArc.cmd: No need to zip installer. (it was due to a restriction of geocities.co.jp)

Do not create installer in BuildBin.*.cmd

Remove BuildAll2.cmd, BuildBin.cmd, WinMerge.sln and Merge.vcxproj*
Use *.vs2015.* or *.vs2017.* instead of above

Restore htmlhelp build to use saxon instead of xsltproc

Fix search marker not working if 'Match case' checkbox is unchecked

Forgot to add ctextmarkerdlg.{h,cpp} Github#40

Fix untranslated strings

temporarily set revision to 2.15.2.0

Remove Japanese help

Restore Help menu

Restore URL

Merge

Added tag 2.14.0+-jp-141 for changeset cee70b23e6ae

Merged in jdunne525/winmerge-v2 (pull request #1)

More on Issue #73 Fixed the -dl -dm and -dr description parameters for conflict files (for both 2 way and 3 way conflict files).

Approved-by: Takashi Sawanaka <sdottaka@users.sourceforge.net>

Bump revision to 2.14.0+-jp-141

Add some tests

editlib: Implement Marker feature(2)

More on Issue #73 Fixed the -dl -dm and -dr description parameters for conflict files (for both 2 way and 3 way conflict files).

FolderCompare: Fix crash when recomparing selected items

Fix issue #77: Drag'n'Drop files with result 'identical'

Fix issue #73: Command line parameter /dr doesn't work with conflict file

DisplayXMLFiles plugin: Upgrade project file to VS2017 and fix #71

crystaledit/Sample: Add x64 platform configuration

Merge pull request #38 from GreyMerlin/master

ShellExtension: migrate to VS2017, etc.

ShellExtension VS2017(5) - Remove old files

 * Remove the old .sln, .vcxproj, and .filters files that were used as
the base for original *.2015.* files, in commit (1) of this group.

ShellExtension VS2017(4) - Eliminate Warnings

 * Eliminate various compilation and linking warnings
 * Debug builds use Incremental linking
 * Debug builds don't use 'Safe Exception Handlers'
 * DLL exports no longer have assigned numbers
 * _vsntprintf() changed to _vsntprintf_s()
 * Make sure WINVER set to 0x501 (for XP compilation)
 * .rc file locally defines _USING_V110_SDK71_ to prevent duplicate
declaration warnings.  Note: this _USING_V110_SDK71_
definition is implicitly provided for all .cpp compilations by the
use of an Win-XP compatible Toolset, such as v141_xp)
 * All .h files now use #pragma once
 * Eliminate deprecation errors for GetVersionEx() by simulating Version
Helper code that works on all versions, Win2000 and beyond
(including WinXP)

ShellExtension VS2017(3) - Updates

 * updates to .vs2017.sln and .vs2017.vcxproj files
 * Debug compilations also use v141_xp toolset.
 * also to .vs2015.vcxproj file

ShellExtension VS2017(2) - Updates

* Eliminate Debug <CustomBuildStep> for registering the shell
extension; it is leftover from a long-ago VS and no longer
functions correctly.
 * Also for VS2015

ShellExtension VS2017(1) - Initial copy of files

 * Also for VS2015

Add version info test files

Merge

Merge pull request #37 from GreyMerlin/master

Tweak vs2017.vcxproj.filters

Tweak vs2017.vcxproj.filters

 * various Filters (i.e. compilation instructions) were not completely
copied from vs2015 into vs2017 for various recent changes.

BuildBin*.cmd: Use vswhere.exe

UploadToGithub.cmd: Add virualtotal.com's scan result URL to release description

Bump revision to 2.14.0+-jp-140

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

editlib: marking text with double click is a litle bit annoying?

Merge.cpp: avoid an assertion failure

FolderCompare: Fix sorting of version, eol and encoding columns not working properly

Bump revision to 2.14.0+-jp-139

Installer: add ignoreversion and comparetimestamp flags to dlls in [Files] section

PropMarkerColors.*: Forgot to add the files. github#35

Merge pull request #36 from GreyMerlin/fix/jp-138

Missing line in .filters file

Missing line in .filters file

 * There should be an additional </ClInclude> in the file
Merge.vs2017.vcxproj.filters at line (1397) to properly terminate the
entry for MergeStatusBar.h, prior to the introduction on
PropMarkerColors.h

Fix broken Japanese.po

Bump revision to 2.14.0+-jp-138

editlib: Implement Marker feature (WIP)

ccrystaltextiview.cpp: remove duplicate code

crystaledit/Sample/StdAfx.cpp: Enable visual style

crystaledit/Sample/SampleDoc.cpp: Fix memory leak

crystaledit/Sample: Upgrade SampleStatic project from vs9 to vs2017

Use a vector instead of an array.

HexMergeDoc.cpp: Use customized colors for Hex Editor Window.

Merge pull request #33 from GreyMerlin/fix-Boost

Update Boost's checking for new VS2017 Update 3

Update Boost's checking for new VS2017 Update 3 (2)

 * tweak comment

Update Boost's checking for new VS2017 Update 3

* Update 3 (dated 8/14/2017) changes _MSC_VER to 1911.  See
https://blogs.msdn.microsoft.com/chuckw/2017/08/14/visual-studio-2017-15-3-update/

* 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

* This tiny patch will prevent those messages until a newer version of
Boost (1.65.0 - now in beta) becomes available.

Merge pull request #32 from GreyMerlin/master

Setup for VS2017 Compilation

VS2017 - update .sln, .vcxproj and .cmd files

* update to *.vs2017.vcxproj names
* update .sln header lines for VS2017
* update .vcxproj ToolsVersion in header
* update PlatformToolset to v141_xp (was v140)
* add new *.vs2017.cmd files to VS solution explorer
* update BuildBin.vs2017.cmd to use VS141COMNTOOLS
* Update VisualStudioVersion to 15.0.26430.16

VS2017 - Copy all *.vs2015 files as *.vs2017