Bump revision to 1.17.2.0

update frhed to 0.10904.2017.1

- Fix warning C6054: String '...' might not be zero-terminated.
- Fix warning C28183: '...' could be '0', and is a copy of the value found in '...'
- Fix warning C6387: warning C6387: <argument> may be <value>: this does not adhere to the specification for the function <function name>: Lines: x, y

Merge

Merge pull request #89 from GreyMerlin/master

Various, esp. Improve compare+display of last lines in files

Improve Options > Color > Differences dialog layout

- Allow more room for the various lines of caption test by sliding the
various controls slightly to the right.

Improve compare+display of last lines in files (2)

Symptom ...

An F5 Refresh could produce slightly different comparison results
(near the end of the file) when input files did not have explicit EOL
marking after the last line of the file.

Discussion ...

The original file compare window is generated directly from the
original files.  However, subsequent F5 Refresh displays are
generated from temporary copies of the files that are made after they
are read for the first time; these temporary files would **never** have
EOL markings following the last line of the file - thus causing slightly
different output.

Solutions ...

When making internal temporary copies of the original input files,
a final EOL marking will be written to the temp file **if** the original
file had a final EOL.

Improve compare+display of last lines in files (1)

Symptoms ...

ASSERT could occur (with Debug build), and possibly a fault could
occur with Release builds, if longest file was the Left File and that
file did not have an explicit EOL marking after the last line of the
file, and there were visible differences at the very end of the file.

Discussion ...

The internal `m_ptBuf[]->m_aLines[]` buffers could end up with
different numbers of line entries in situations where the last line of
a file had no explicit EOL marking.  Normally, the two (or three)
`m_aLines[]` buffers would have "ghost" line entries to represent
"missing" lines, so that all `m_aLines[]` buffers would be of the same
length.  The lack of a final EOL could confuse that logic.  At some
points in the comparison logic, the length used for processing the two
(or three) buffers is determined simply by the length of `m_aLines[0]`

Solution ...

By the end of the `PrimeTextBuffers()` procedure, the two (or
three) `m_ptBuf[]->m_aLines[]` buffers will now **always** have the same
length.  Furthermore, there will no longer be <null> entries at the end
of the `m_aLines[]` arrays.  These will now be deleted (if **all**
`m_aLines[]` buffers have them), or they will be replaced with
appropriate "ghost" lines (if only **some** of the buffers have final
<null> entries)

Also, a number of comments are improved.

Tweak "Fix alignment of the comboboxes in OpenDlg"

- adjusted by 1 pixel for even better visual alignment

Adopt performance improvement from diffutils

- include [commit ...414dc606](
http://git.savannah.gnu.org/cgit/diffutils.git/commit/?id=68b82f6f8419a815cfcf962b3061352d414dc606)
from **diffutils** 2016-10-25 (ver 3.5)

- this commit improves the performance of the TOO_EXPENSIVE
optimization on "modern" processors (ca. 2016) by upgrading the minimum
threshold of this optimization from 255 to 4096.

Improve interrelation with GNU diffutils source

- Remove four *.c files never used by diffutils or WinMerge

- Change `Src/CompareEngines/DiffUtils{.c|.h}` to be
`./Wrap_DiffUtils{.c|.h}` to remove confusion about the **diffutils**
name.  These files are part of WinMerge, not GNU diffutils.

- Change Solution Explorer to have the entry **GNU diffutils** rather
than **DiffEngine**

- Copy solution changes from VS2017 to VS2015

Fix compilation errors on the Test configuration

Fix alignment of the comboboxes in OpenDlg. (github#88)

Add Window/Split menu item(2)

Add Window/Split menu item

Merge pull request #87 from GreyMerlin/master

Very Long Path and File Names (and more)

Very Long Path and File Names (5)

Very Long Path and File Names (4)

* Adapt changes from Poco/Foundation (1.9.1) related to this issue

* this is a "cleaner" solution than I had implemented, although only
for the Poco/Foundation parts of the code

Very Long Path and File Names (3)

* Use the `\\?\`file name prefix for directory recursion and various
file open operations.  This is available for almost all Win32 API
functions when using the "xxxW" Unicode procedures.  This works for
both Win32 and X64 compilations.  Microsoft documentation leads me to
believe that this works for WinXP and beyond (although I can only test
the Win10 X64 environment).

*The prefix is used even for path names that "appear" to be short,
because a long file name could be appended to any short path, making
the result become long.

* At this point, the basics (directory recursion, file comparison,
and opening&editing text files) have been tested and are properly
functional.

* Not Tested: Copy, Move, Rename, Remove of files and directories.

* Not Tested: projects, filters and plugins, XML, Excel.

* Not Implemented: Opening binary files (`frhed.dll`) or image files
(`WinIMerge.dll`)

* Not Implemented: Internal 7zip operations.

* Not Implemented: interactions with UNC (networking) file names.

* Not Implemented: interaction with `ShellExtension.dll`

Very Long Path and File Names (2)

* First attempt to add "\\?\" as a prefix for all file names as they
are being presented to Windows API for file manipulation purposes

Very Long Path and File Names (1)

* define MAX_PATH_FULL as 32767.  Use this to replace many instances of
MAX_PATH and _MAX_PATH

Three Trivial Changes

- Remove the **poco** `_CHANGELOG` file.  It is only for Apple
environments, and has been removed from **poco** version 1.9.1 (sometime
after version 1.4).

- Tweak two comments in `WinMergeShell.cpp`

- Remove an unused procedure declaration from `WinMergeShell.h`.  There
is no corresponding procedure definition.

Prevent faults when mousing over Plugin Settings

* With a Debug build, an assert() would occur when mousing over the
Plugin Settings context menu item if Plugins are disabled.

* With a Release build, a fault could occur in the same conditions.

Remove new compilation error with VS2015

* VS2015 must have tightened up a rule for templates.  This always used
to compile.

* now compiles correctly with both VS2015 and VS2017

Hide unused Poco/Foundation files from Intellisense

* Use `#if 0` ... `#endif` to force various UNIX, VMS, WINCE and
non-Unicode WIN32 files to show as non-compiled code.

 * there are more files that could be marked this way, but these are in
the File System implementation

Fix problems with _countf() in newest VS 15.7.2

The `vcruntime.h` version of `_countf()` gives syntax errors starting
with VS 15.7.2, but only with `CCrystalTextView::m_SourceDefs` (which is
local to this .cpp file), and only for X64 compilations (Win32 is
ok, probably because no alignment issues are involved).  I think
that this could be related to C++17 compliance issues.  This patch
reverts to a 'traditional' definition of `_countf()`, a pre-existing
part of the CCrystalTextView package.

Adapt boost to latest VS compiler version

Merge

DiffTextBuffer.cpp: Fix crash when pressing Alt+Down key while loading large files (Revert part of GitHub #82)

DirDoc.cpp: Fix crash when Settings/ActivePane == -1

MergeDoc.*: Add ForEachView() and Remove GetActiveMergeViewIndexType()

MergeDoc.cpp: Fix crash when saving file if the active pane is diff detail pane

Improvements Japanese

FileOrFolderSelect.*: Move the position of frequently used optional arguments forward

- OpenView.cpp: Display 'Open' instead of 'Save As' in the title of Open dialog when selecting Load Project menu item
- Make the pane active when clicking on the header bar
- Merge with rev.9409

Merge pull request #86 from Luigi-C/patch-1

Update Italian.po

Update Italian.po

The correct italian word for the english "Yes" is "Si", where the "i" is not accented.

FilepathEdit.cpp: Fix incorrect function declaration

OptionsMgr.h: Fix mixed-eol

Added tag 2.15.4 for changeset 7d1863b8c686

Bump revision to 2.15.4

Update release notes and changelog for 2.15.4

Allow short key name in -config command line option

Merge.cpp: Remove unused struct definition

Remove unused forward declaration class

DiffTextBuffer.h: Fix typo

Merge2.rc: Make Class Wizard work again

Merge.cpp: cleanup

DirView.*: Remove unused functions

resource.h: Remove unused resource symbols

MergeDoc.*: Mark ignored lines by Prediffer plugin as Trivial

Installer: Add PrediffLineFilter.sct plugin

PrediffLineFilter.sct: Add Ignore Case option to settings dialog

GhostTextBuffer.cpp: Fixed a problem that line without EOL exists even though it is not the last line

GhostTextBuffer.cpp: Avoid assertion failure when deleting ghost lines

Merge pull request #85 from Lippe35/master

Add missing translation Portuguese

Add missing translation Portuguese

Merge pull request #84 from Lippe35/master

Improvements Portuguese

Improvements Portuguese

Fix crash when removing ghost lines. I hope this fixes the issue #80 (GitHub)

Add PrediffLineFilter.sct

Merge pull request #82 from GreyMerlin/master

Two patches related to reading large files (i.e. > 2GB)

Improve reading of files > 2GB (3)

* This is just the "next" in a series of patches to allow WinMerge to
properly handle very large ( i.e. > 2GB) files.

* This is a "kludge" patch allow for a responsive UI while reading
large files.  After every input line is read and processed,
`::PeekMessage()` and `PumpMessage()` are called to allow the UI to
process messages.
* A much more robust solution would involve running
`CMergeDoc::LoadOneFile()` to run on threads that are independant of
the UI.  This patch could then be easily removed.
* This patch also removes a useless label and `goto`,

* **Important Note** ... The reading and display of large
files remains seriously slow (with a completely non-responsive
WinMerge window for much of the time).  Simple vertical scrolling
seems reasonably responsive.  However ... any manipulation of the
"shape" or "z-order" of panes within the WinMerge window consumes
**seriously** more elapsed time while the Vertical Scroll Bar is being
tediously re-calculated.  For this and other reasons ... I don't
consider large files to be very useful yet.

Improve reading of files > 2GB (2)

* This is just the "next" in a series of patches to allow WinMerge to
properly handle very large ( i.e. > 2GB) files; although all of the
problems noted and fixed here can also be exhibited with files
somewhat smaller than that 2GB size.
* The main improvement here is to notice "lines" longer than 2^16 chars
and forcefully truncate them.  For binary-like text files, long runs of
'\x00' and '\xff' characters are tolerated for lines containing up to
2^18 characters.
* This greatly improves the performance of the display and scrolling of
"binary-like" files (files whose extension does not match the **Binary
File Pattern**).
* Note too that the function `MultiByteToWideChar()` can only handle
output lines up to INT_MAX characters (thus, input lines up to
INT_MAX/2 characters).
* I've dropped in a few `// TODO` comments to looping situations that
are particularly slow and should be re-designed with very long files in
mind.

* **Important Note** ... The reading and display of large
files remains seriously slow (with a completely non-responsive
WinMerge window for much of the time).  Simple vertical scrolling
seems reasonably responsive.  However ... any manipulation of the
"shape" or "z-order" of panes within the WinMerge window consumes
**seriously** more elapsed time while the Vertical Scroll Bar is being
tediously re-calculated.  For this and other reasons ... I don't
consider large files to be very useful yet.

Fix the problem that dot icon in a drop-down menu is not drawn correctly

OpenView.*: Merge with ProjectFilePathDlg and remove ProjectFilePathDlg. Add Options button to OpenView

Merge pull request #81 from GreyMerlin/master

Improve reading of files > 2GB (1)

Improve reading of files > 2GB

* Now that commit #4e4e46c has allowed files larger than 2GB, there are
number of places in WinMerge that are not prepared to handle files that
large.  These will have to be discovered by "trial and error".  These
are the first two problems encountered ...

* This patch repairs two places that `assert`with long file lengths.
* It should be noted that Windows can only read/write in chunks of up to
2GB.  The code around lines 227 to 232 is part of a loop that is
capable of reading very large files, and now has the correct setup code
to limit the size of each successive read.
* The `assert` at line 911 is completely unnecessary for both 32- and
64-bit builds.

SharedMemory_WIN32.cpp: Avoid access violation when mapping a file on Google Drive File Stream to memory(2). (sf.net#2206)
the previous fix did not work on release build

UniFile.cpp: Remove 2GB file size limit for 64-bit build

SharedMemory_WIN32.cpp: Avoid access violation when mapping a file on Google Drive File Stream to memory. (sf.net#2206)

DirCmpReport.cpp: Fix an issue where garbled text was copied to the clipboard when generating a report on the clipboard. (sf.net#2200)

Merge pull request #79 from GreyMerlin/master

Version issues plus Adapt to VS2017 15.6.1 changes

Simplify version.h file name confusions

Problem ...

* there were two, very different, `version.h` files ...
   `.\version.h` which simply contains current version defines
   `.\Src\Common\Version.h` which declares the `CVersionInfo` class
      implemented in the `\Src\Common\Version.cpp` file.
* The files `.\version.h` and `.\Src\res\Merge2.rc` are not visible in
   the Solution Explorer tree.

Solution ...

* I have changed `.\Src\Common\Version.*` to be
   `.\Src\Common\VersionInfo.*`.  This also follows more typical file
    naming norms.
* I have moved these new `VersionInfo.*` files from the **Header Files**
   section of the Solution Explorer, into the **MFCGui/Common** section
* I have added the `.\version.h`file into the **Header Files** section,
   so that it becomes a visible part of the project.  I have **not**
   modified the name or purpose of this file because it is generated by
   `BuildBin.vs2017.cmd` during the release build process.
* I have added the `.\Src\res\Merge2.rc` to the **Resource Files**
   section of the Solution Explorer (so that it becomes visible to the
   project).  I have marked that file as "Excluded From Build" (in all
   build configurations) because it is directly included from the
   `.\Src\res\Merge.rc2`file.

* I have made these changes for both VS2015 and VS2017 projects.

Adapt to VS2017 version 15.6.1

* These problems were introduced with the upgrade to VS2017 15.6.1
* Once again, the boost library needs to be aware of the newest VS2017
compiler version, which is now _MSC_VER == 1913
* two functions in `DirView.cpp` and one function in `MovedBlocks.cpp`
were causing a new level=3 warning ::
   "warning C4834: discarding return value of function with 'nodiscard'
attribute"
   This was caused by increased checking on the `std::accumulate()`
function.

* Note:: the `isPresent()` function of `MovedBlocks.cpp` is never
actually called, which masked the obvious bug in the code
(now repaired).
* These changes also work correctly with VS2015 (although the new
checking and warning message are not in VS2015)

Update version as in SetVersion.cmd = 2.15.3

DirView.cpp: Re-detect the file encoding when opening files from DirView. (sf.net#2131)

Add Pause button to Folder Compare Progress dialog (sf.net#828,#1222,#1237)

Merge.rc: Add Ctrl++/Ctrl+-(not numpad key) shortcut key to zoom in/out. (sf.net#1727)

Update Slovak translation (sf.net#2902)

PatchDlg.*: Fix the issue that File1/File2 fields don't remember manual input (sf.net#1949)

Added tag 2.15.3 for changeset 82b36418c467

Merge

Update release notes and changelog for 2.15.3

DownloadDeps.cmd: Update ShellExtension version

Merge pull request #78 from Lippe35/master

Improvements Portuguese

Improvements Portuguese

Merge pull request #77 from Lippe35/master

Add missing translation and corrections Portuguese

Add missing translation and corrections Portuguese

DirView.cpp: Remove trailing garbage from a dragged text (2) fixes github#75

Manual: Fix the issue "sf.net#2165 Manual "Using Filters" Line Filter Example"

Polish.po: Update Polish translation (sf.net#2177)

Merge pull request #76 from Lippe35/master

Corrections and improvements Portuguese

Corrections and improvements Portuguese

Contributors.txt: Change the file encoding to UTF-8

Translations/WinMerge/*.po: Change the file encoding to UTF-8

Merge

Merge pull request #74 from Lippe35/master

Corrections Portuguese

Corrections Portuguese

Fix to show text that contains & on message box, correctly. sf.net#2067 (2)