For file list (dirview) use of drag images was disabled already in Bug 1274. Now disabling it for tree view (driveview) too and consequently removing all related support code.
That gets rid of global GlobalDragImageList, which the 64-bit linker has problems with.
https://winscp.net/tracker/1274https://winscp.net/tracker/618
Source commit: 10f13c6de6259711dbd0688402d9a11423fcc8b0
UNICODE define added everywhere (except Putty, which in Ansi code) to make sure A/W suffixes are added consistently across the code
CreateDirectory renamed to avoid problems with being interpreted as CreateDirectoryW in C++ code and CreateDirectory in Pascal code (being also a function in WinAPI)
Source commit: b8d27b5097829c42f853b629e57aebaaab066b9e
Mostly
– explicit static global variables
– const, sign and down casts safety
– same-named variables in nested scopes
– incorrectly identified uninitialized variables
– semicolon after function implementation
– unused variables, fields, arguments and macros
– exact include filename case
– field initialization order
– typename use
– struct vs class declarations
– incorrectly identified uninitialized variables
– refactoring to solve Clang internal compiler error when using new variable in __finally block
Source commit: 3da995ff5169eefcfe3b3b7cc0d019c660096c07
Partially rolls back/reimplements Bug 1057 from a128fd8e.
Also fixes not compilable DScpComp and broken /Browse switch caused by the same commit
Source commit: f7c5a4d72cd7f63ebdbf67255492a38cb33af169
Originally, I resolved this using new TMulticastEvent, but once I've realized that TUnixDirView::DoChangeDirectory implementation is noop, it was not needed anymore.
Keeping the multicast implementation, as it may be useful in future.
Source commit: 0d53f122585d0cc9d44f22767f03a3ddef49e10d