Merge pull request #31 from GreyMerlin/master

Three unrelated simple changes

Fix properties for WinXP targeting

Add VS Filters to various projects

* Copy various *_vs100.vcxproj.filters as *.vs2015.vcxproj.filters
to capture project file organization within Solution Explorer
* Add two relevant folders to MergeLang project for improved
project organization
* Add two relevant *.vbs files to MergeLang project (used only in
Post-Build Event step)
* Add relevant *.vbs file to Batch Files solution folder
* Also make sure that the *.vs2015.sln files are encoded as UTF-8-BOM
for proper Visual Studio selection.

Typo corrections to new Go and Rust Copyrights

Bump revision to 2.14.0+-jp-137

editlib/cplusplus.cpp: Add C++11 keywords.

Add Go and Rust syntax highlighter

MergeDoc.cpp: Fix incorrect in-line diff coloring after swapping panes

Remove unused declarations

DirCmpReport.*: Create HTML report with UTF-8

DirCompProgressBar.*: Use Dynamic Dialog Layout for MFC

Bump revision to 2.14.0+-jp-136

Merge.cpp: To decrease startup time, call CleanupWMtemp() function in CMergeApp::ExitInstance() instead in CMergeApp:InitInstance().

Format resource file

Merge.*vcxproj.filters: Fix filename case

multiformatText.cpp: Fix Unpacker plugin not working properly when the selected custom codepage is 65001(UTF-8)

MergeEditView.cpp: Fix crash when selecting 'Copy to Right/Left And Advance' menu item while editing the last difference file opened from the folder comparison window

StdAfx.*: Move GetClipTcharTextFormat() function from StdAfx.* to CCrystalTextView class

Upgrade project files to VS2017

Add the alternative functions of _fstat() and _wstat() for the bug https://connect.microsoft.com/VisualStudio/feedback/details/1600505/stat-not-working-on-windows-xp-using-v14-xp-platform-toolset-vs2015

FileOrFolderSelect.cpp: Use paths::DoesPathExist() instead of _tstati64()

Add 'Test' configuration to test WinMerge

Update boost to 1.64.0

Add GetDescription() function

OptionsMgr.h: Add COptionsMgr::GetNameList() function.

CCrystalTextView.*: Add CCrystalTextView::GetTextBocks() function.

UpdatePoFilesFromPotFile.vbs: Fix the last translated message in *.po file being lost if the po file is not terminated with CR+LF or LF

MessageBoxDialog.*: Add CMessageBoxDialog::SetFormerResult() method

MainFrm.cpp: Fix crash when the argument 'flags' of the function CMainFrm::DoFileOpen() is NULL.

Merge pull request #29 from GreyMerlin/master

Standardize Various Visual Studio File Names

Standardize .cmd, .sln and .vcprojx file names (3)

* Change *.vcxproj.filters file names to *.vs2015.vcxproj.filters

Standardize .cmd, .sln and .vcprojx file names (2)

* Adapt WinMerge.vs2015.sln to new *.vs2015.vcxproj file names
* Adapt new BuildBin.vs2015.cmd to proper .sln name
* Adapt new BuildAll.vs2015.cmd to new BuildBin.vs2015 name
* Adapt BuildAll.vs2015.cmd to use VS140 (i.e. VS2015)
* Add new BuildBin.vs2015.cmd and BuildAll.vs2015.cmd added
to WinMerge.vs2015.sln file

Standardize .cmd, .sln and .vcprojx file names (1)

* copy BuildBin.cmd as BuildBin.vs2015.cmd
* copy BuildAll2.cmd as BuildAll.vs2015.cmd
* change all *_vs2015.vcxproj to *.vs2015.vcxproj
* change WinMerge_vs2015.sln to WinMerge.vs2015.sln
* using *.vs2015.* file names from related projects and solutions.

* This commit simply handles file name changes, detailed content
changes will be applied in the next commit

Generalize .gitIgnore patterns

Added tag 2.14.0+-jp-135 for changeset 9d4e7604a3e3

Bump revision to 2.14.0+-jp-135

MergeDocLineDiffs.cpp: Avoid out of bounds access(2). (github#28)

Bump revision to 2.14.0+-jp-134

MergeDocLineDiffs.cpp: Avoid out of bounds access

GhostTextBuffer.cpp: Fix crash when redoing to delete text included ghost lines

ShellExtension: Bump revision to 1.16.5.11

ShellExtension: Fix stupid mistake in IsWindows8OrGreater() function

ShellExtension: Register.bat didn't work on Windows 10

Add sheppaul to Contributors.txt

Korean translation update submitted by sheppaul

Bump revision to 2.14.0+-jp-133

Fix warning C6001: Using uninitialized memory '...'

Cppcheck: The scope of the variable '...' can be reduced.

Add BuildInstaller.cmd, UploadToGithub.cmd, UploadToVirusTotal.cmd

Cppcheck: Checking if unsigned variable '...' is less than zero.

Cppcheck: Variable '...' is assigned a value that is never used.

WinMerge.sln: Fix build error on UnitTest project

Use std::array

Cppcheck: Variable '...' is reassigned a value before the old one has been used.

Cppcheck: %lX in format string (no. 1) requires 'unsigned long' but the argument type is 'void *'.

Cppcheck: %d in format string (no. 1) requires 'int' but the argument type is 'UINT {aka unsigned int}'.

MergeDoc.cpp: Reduce file access times on checking whether a file is changed

MessageBoxDialog.cpp: Fix invisible text in high-contrast mode

ShellExtension: Bump revision to 1.16.5.10

ShellExtension: Use icons instead of bitmaps

ShellExtension: Attempt to fix crash on Win7 when doing the operation below:
1. Press Windows key
2. Enter "cmd"
3. Press Ctrl+Shift+Enter key

stringdiffs.cpp: Reduce compiler warnings

Bump revision to 2.14.0+-jp-132

DirCompProgresBar.cpp: Fix: The range of Progress bar was not set. (CProgressBar::SetRange32() was accidentally removed in r8505 )

Fix issue#64 where files and folders that exist only on one side in the differing folders are not displayed when unchecking the 'Show Differences Items' menu item in the tree mode

MainFrm.cpp: Fix crash when changing the font of DirView

DirViewColItems.cpp: Fix indentation

Fix assertion on WindowsXP

ImgMergeFrm.cpp: Fix mixed eol-style

CompareMSExcelFiles.sct: Applied the patch submitted by tori932
- Imaging of excel sheet did not work on Excel2010
- Imaging of very wide sheet also did not work

Bump revision to 2.14.0+-jp-131

Add shortcut(Ctrl+Alt+E) to open with external editor

*.vcxproj: Apply GreyMerlin's changes

ccrystaleditview.cpp: Use IDropTarget::Release() instead of delete operator.

Merge

Chinese_Simplified.isl: Update Chinese_Simplified.isl file. bitbucket#63

Merge pull request #26 from GreyMerlin/bugfix/DropTargetMemLeak

Fix Memory Leak with Drop Targets

Merge pull request #25 from GreyMerlin/feature/TimeStars

Feature/time stars

Merge pull request #24 from GreyMerlin/bugfix/DefaultFonts

Use proper font for View>Use Default Font menu item

Fix Memory Leak with Drop Targets

Symptom:
* There will be 4 memory leaks (each 80 bytes long) reported at
program termination for each File-View window that is opened.

Discussion:
 * This is only noticeable with a _DEBUG compilation.
 * This will happen when either Text or Binary files are being
viewed/edited.
 * Each File-View window establishes four Drop Targets and registers
  them with Windows.  Each also establishes an 'alternate' Drop
Target for use with various internal Drag-And-Drop operations.
 * When the edit windows are closed, the Drop Targets are properly
Revoke()'ed and deleted.
 * However: the 'alternate' Drop Targets are not deleted ... MemLeak!

Implementation:
* procedure OnDestroy() in file
Externals/crystaledit/editlib/ccrystaleditview.cpp now deletes
the 'alternate' Drop Target before deleting the master one.
* Additionally, there is now an ASSERT() in procedure
SetAlternateDropTarget() guarding the assignment of the
'alternate' to make sure that it was still NULL

Cleanup Date column alignment

Symptom: Date columns use a leading "* " to indicate which date
is newest.  This looks ragged and awkward.

Implementation: Add an appropriate "  " as leading text for date
columns that do not have the "* " appended.

Comment: Clearly the two characters "* " occupy slightly more
horizontal space than two "  " characters, unless a fixed-
font is used.  Nonetheless, the columns almost appear
aligned with the default Segoe UI font.

Historical Comment: I've been using an equivalent to this patch
in WinMergeU since 11/2009.  I've never been visually
aware of the slight column mis-alignment caused by
proportional-width fonts.

Use proper font for 'View > Use Default Font'

Symptom:
Use of the 'View > Use Default Font' menu selection always
sets the Directory Tree window to the 12-point Courier New font,
which is not the default font for that window.

Discussion:
WinMerge has an Options::Font structure that maintains an
in-memory copy of 'actual' and 'default' values for the font of
each of the Directory Tree and File View windows.  This structure
is populated at program initialization by procedure SetDefaults()
in file Src/OptionFont.cpp.  The 'default' values are generated by
this procedure, the 'actual' values are loaded from the Registry.

The existing code only queries the system for the default fixed-
size font (via the default MIME codepage) and then uses this font
as the 'default' value for the Options::Font structure for both
Directory and File windows.  This value is typically Courier New.

However, WinMerge actually uses the Menu font as the actual Directory
window font if no other font is marked as 'Specified' in the
Registry.  The default Menu font (for English, since Windows 7) is
Segoe UI.  But this use of Segoe UI is never reflected in the
Options::Font structure, nor in the Registry.

Implementation:
The procedure SetDefaults() is modified to additionally
capture the existing Menu font information for the Directory Tree
window, while continuing to use the codepage MIME font for the
File View window.

A "helper" function is added: InitializeLogFont().  It should only
be used within Src/OptionFont.cpp.  It is documented by comment in
the Src/OptionFont.h file.

The information in the Registry for the Options::Font structure
now always reflects the values of the in-memory structure, and
the values being used by the windows themselves.

Incidental changes:
* Numerous added or modified comments in Src/OptionFont.cpp
* Reordering of all Registry related code to be in the canonical
order implied by the layout of the LOGFONT structure itself.
* Explicitly cast the three boolean values (Italic, Underline,
Strikeout) to boolean to invoke the properly typed Options::Font
procedures (InitOption, SaveOption)
* Remove two unnecessary String() function references relating to
font in files Src/DirView.cpp and Src/MergeEditView.cpp

Support for diff3-style conflict file. bitbucket#61

Merge branch 'master' of https://github.com/sdottaka/winmerge-v2

Merge pull request #23 from GreyMerlin/bugfix/3WayBinary

Properly handle 3-way comparison of Binary files.

Merge pull request #22 from GreyMerlin/feature/TweakAbout

Show "(Debug)" in About dialog

Merge pull request #21 from GreyMerlin/feature/VS2015.2

VS2015.2 - Delete another obsolete Project attribute

Merge branch 'bugfix/3WayBinary'

Merge branch 'feature/TweakAbout'

Merge branch 'feature/VS2015.2'

Properly handle 3-way comparison of Binary files.

Symptom: Binary files always mis-compare with 3-way comparison,
although the same files compare correctly with 2-way compares.

Background: 3-way comparison of files F0, F1 and F2 is handled by two
2-way comparisons of F1/F0 and F1/F2.  However, the two 2-way
comparisons for binary are slightly different from the standard
2-way text comparison.

Root Problems:
 * See procedure read_files() in Src/diffutils/src/io.c
This area of code was allowing binary files to be treated as text,
looking for endline situations and modifying the contents of the
first filevec[i].buffer[] array by placing sentinel markers into
that buffer.  But this was only done for one file, F1 in the example
above.  The contents of F0 and F2 were not being modified.  This
caused the memcmp() in diff_2_files() in Src/diffutils/src/analyze.c
to detect a mis-miscompare.

 * See procedure diff_2_files() in Src/diffutils/src/analyze.c
A binary comparison is flagged by read_files(). Because read_files
is only checking one file (e.g. F1), the buffer size for the other
file (e.g. F0 or F2) is never changed to be identical to the size
for F1.  This makes it impossible for the comparison loop to stay
in sync between the two files.

Solution:
 * Within read_files(), after appears_binary flag has been set, there
is new logic that allocates buffers for each file, based on the
size of existing buffers, the file size, and a reasonable maximum
threshold.  Binary file processing ends with a flag being returned
to the caller.  [It is within find_identical_ends() that buffers
are allocated for text files and the search for text-line boundaries
occurs, and was incorrectly used for binary files.]

 * Within diff_2_files() is the code that performs the 2-way binary
comparison, after read_files() has determined that at least one
file is binary.  Tests are made to determine if inequality can be
known without reading the data.  Buffers of the same size are read
from each file.  The buffers are checked to determine if the buffer
contents are compatible and the data values of the two buffers are
equal.  The outer-loop will repeat to read each subsequent block.
[Note: it is typical to enter this loop [the for (;;) statement]
with some data already read into filevec[0].buffer, but not a
complete buffer's amount; likewise filevec[1].buffer will typically
still be empty.]

Incidental changes...

 * Trivial fix to a comment, discovered while reading code in
Src/DiffWrapper.cpp.

 * Trivial column alignment issue in analyze.c

 * Better checking for non-existent files with more use of S_ISREG()
define (internally, non-existent files are handled with an open
descriptor on the NUL: device).  This allow small optimization of
buffer allocation.

 * Assert that files opened as S_ISCHR() really are the NUL: file

Show "(Debug)" in About dialog

 * Also allow "x86" to be translated in About dialog.
 * Also show WIN32, UNICODE, _DEBUG in Configuration log
when appropriate

VS2015.2 - Delete another obsolete Project attribute

 * This removes the use of the Microsoft.CPP.UpgradeFromVC71.props
file from three of the vcxProj files (it wasn't used in other files).
 * This .props file only sets the preprocessor flag _VC80_UPGRADE which
is never used by this software.
 * _VC80_UPGRADE is referenced in one place in MFC's <afxreg.h> file,
however that code is also never invoked, and only generates a
#define that also is never used in these projects

Merge pull request #20 from GreyMerlin/feature/Readme

Readme.md - fix first line

Merge pull request #19 from GreyMerlin/feature/VS2015

Feature/vs2015

Readme.md - fix first line

 * the first line of readme.md needs a space character between the end of the
first URL and the following parenthesis '('.  Without the space the two
separate URLs are parsed by most browsers as one long (and useless)
URL.

Convert to VS2015 - (Warnings)

 * remove Warnings in Compile, Librarian and Linker options
 * VS2015 warns about /EHsc and changes them to /EHa, so
I changed all compilations to /EHa i.e. ExceptionHandling=Async
to eliminate the warnings
 * ImageHasSafeExceptionHandlers=true is incompatible with EditAndContinue, so
I changed Debug linking to be ImageHasSafeExceptionHandlers=false
 * Make sure DebugInformationFormat=EditAndContinue for all _DEBUG compiles
and default otherwise
 * Set Profile=false for X64 Debug; otherwise incremental linking is disabled.
Use of Profile should be individual and not released.
 * Unittests is now configured to depend on poco

Convert to VS2015 - (SubSystem)

 * For all Linker and Librarian runs ...
        make sure that SubSystem=Windows is set
        make sure that TargetMachine is properly set

Convert to VS2015 - (MultiProcessor Compilations)

 * turn on MultiProcessorCompilation=true
 * remove MinimalRebuild=true (i.e. defaults to false)

Convert to VS2015 - (oops #2)

 * the change in (simplify #2) that changes build directory names was a *very wrong* idea.
 * this patch puts those names back as they always have been.