fix typos

dotnet/interopcollections/ReadOnlyInteropCollection.cs

@@ -4,7 +4,7 @@ using System.Collections.Generic;
 
 namespace WinSCP
 {
-    // Originally, the interop classes were each implemented separatelly. Probably because at the time they were
+    // Originally, the interop classes were each implemented separately. Probably because at the time they were
     // originally implemented, it was not possible to implement them via generic parent class.
     // Now it seems to work (something was improved in in .NET interop meanwhile?)
     //

source/core/Common.cpp

@@ -977,7 +977,7 @@ int __fastcall CompareLogicalText(
   {
     Result = lstrcmpi(S1.c_str(), S2.c_str());
   }
-  // For deterministics results
+  // For deterministic results
   if (Result == 0)
   {
     Result = lstrcmp(S1.c_str(), S2.c_str());
@@ -4204,7 +4204,7 @@ UnicodeString __fastcall AssemblyNewClassInstanceStart(
       // Historically we use Dim .. With instead of object initilizer.
       // But for inline use, we have to use object initialize.
       // We should consistently always use object initilizers.
-      // Unfortunatelly VB.NET object initializer (contrary to C#) does not allow trailing comma.
+      // Unfortunately VB.NET object initializer (contrary to C#) does not allow trailing comma.
       Result += SpaceOrPara + RtfKeyword(L"With");
       if (Inline)
       {

source/core/Configuration.cpp

@@ -1655,7 +1655,7 @@ TStorage __fastcall TConfiguration::GetStorage()
   TGuard Guard(FCriticalSection);
   if (FStorage == stDetect)
   {
-    DebugFail(); // This is never called, as the detection is completelly overriden by TWinConfiguration
+    DebugFail(); // This is never called, as the detection is completely overriden by TWinConfiguration
     if (FileExists(ApiPath(IniFileStorageNameForReading)))
     {
       FStorage = stIniFile;

source/core/CopyParam.cpp

@@ -829,7 +829,7 @@ bool __fastcall TCopyParamType::SkipTransfer(
   UnicodeString FileName, bool Directory) const
 {
   bool Result = false;
-  // we deliberatelly do not filter directories, as path is added to resume list
+  // we deliberately do not filter directories, as path is added to resume list
   // when a transfer of file or directory is started,
   // so for directories we need to recurse and check every single file
   if (!Directory && (FTransferSkipList.get() != NULL))

source/core/CoreMain.cpp

@@ -143,7 +143,7 @@ void CoreLoad()
     ShowExtendedException(&E);
   }
 
-  // should be noop, unless exception occured above
+  // should be noop, unless exception occurred above
   ConfigStorage->CloseAll();
 
   StoredSessions = new TStoredSessionList();

source/core/FileMasks.cpp

@@ -220,9 +220,9 @@ UnicodeString __fastcall TFileMasks::ComposeMaskStr(
     }
   }
 
-  // For directories, the above will add slash ay the end of masks,
+  // For directories, the above will add slash at the end of masks,
   // breaking size and time masks and thus circumverting their validation.
-  // This performes as hoc validation to cover the scenario.
+  // This performes ad hoc validation to cover the scenario.
   // For files this makes no difference, but no harm either
   TFileMasks Temp(Directory ? 1 : 0);
   Temp = ResultNoDirMask;

source/core/FileOperationProgress.cpp

@@ -669,7 +669,7 @@ void __fastcall TFileOperationProgressType::SetSkipToAll()
 //---------------------------------------------------------------------------
 void __fastcall TFileOperationProgressType::ChangeTransferSize(__int64 ASize)
 {
-  // reflect change on file size (due to text transfer mode conversion particulary)
+  // reflect change on file size (due to text transfer mode conversion particularly)
   // on total transfer size
   if (TotalSizeSet)
   {

source/core/FtpFileSystem.cpp

@@ -1556,7 +1556,7 @@ void __fastcall TFTPFileSystem::CopyToLocal(TStrings * FilesToCopy,
 UnicodeString TFTPFileSystem::RemoteExtractFilePath(const UnicodeString & Path)
 {
   UnicodeString Result;
-  // If the path ends with a slash, FZAPI CServerPath contructor does not identify the path as VMS.
+  // If the path ends with a slash, FZAPI CServerPath constructor does not identify the path as VMS.
   // It is probably ok to use UnixExtractFileDir for all paths passed to FZAPI,
   // but for now, we limit the impact of the change to VMS.
   if (FVMS)
@@ -1710,7 +1710,7 @@ void __fastcall TFTPFileSystem::Source(
   // Support for MDTM does not necessarily mean that the server supports
   // non-standard hack of setting timestamp using
   // MFMT-like (two argument) call to MDTM.
-  // IIS definitelly does.
+  // IIS definitely does.
   if (FFileTransferPreserveTime &&
       ((FServerCapabilities->GetCapability(mfmt_command) == yes) ||
        ((FServerCapabilities->GetCapability(mdtm_command) == yes))))
@@ -3528,7 +3528,7 @@ void __fastcall TFTPFileSystem::HandleReplyStatus(UnicodeString Response)
         // (the ... can be "z/OS")
         // https://www.ibm.com/docs/en/zos/latest?topic=2rc-215-mvs-is-operating-system-this-server-ftp-server-is-running-name
         // FZPI has a different incompatible detection.
-        // MVS FTP servers have two separate MVS and Unix file systems cooexisting in the same session.
+        // MVS FTP servers have two separate MVS and Unix file systems coexisting in the same session.
         FMVS = (FSystem.SubString(1, 3) == L"MVS");
         if (FMVS)
         {
@@ -3617,7 +3617,7 @@ void TFTPFileSystem::ProcessFeatures()
     {
       // Serv-U lists all SITE commands in one line like:
       //  SITE PSWD;SET;ZONE;CHMOD;MSG;EXEC;HELP
-      // But ProFTPD lists them separatelly:
+      // But ProFTPD lists them separately:
       //  SITE UTIME
       //  SITE RMDIR
       //  SITE COPY
@@ -4211,7 +4211,7 @@ bool __fastcall TFTPFileSystem::HandleAsynchRequestVerifyCertificate(
 
       // TryWindowsSystemCertificateStore is set for the same set of failures
       // as trigger NE_SSL_UNTRUSTED flag in ne_openssl.c's verify_callback().
-      // Use WindowsValidateCertificate only as a last resort (after checking the cached fiungerprint)
+      // Use WindowsValidateCertificate only as a last resort (after checking the cached fingerprint)
       // as it can take a very long time (up to 1 minute).
       if (!VerificationResult && TryWindowsSystemCertificateStore)
       {

source/core/HierarchicalStorage.cpp

@@ -383,7 +383,7 @@ void __fastcall THierarchicalStorage::CloseSubKeyPath()
   }
 
   int Levels = FKeyHistory.back().Levels;
-  FKeyHistory.back().Levels = 1; // to satify the assertion in CloseSubKey()
+  FKeyHistory.back().Levels = 1; // to satisfy the assertion in CloseSubKey()
   while (Levels > 0)
   {
     CloseSubKey();
@@ -1208,7 +1208,7 @@ bool __fastcall TCustomIniFileStorage::OpenSubKey(const UnicodeString & Key, boo
 {
   bool Result;
 
-  // To cache root access in advance, otherwise we end up calling outselves, what TAutoFlag does not like
+  // To cache root access in advance, otherwise we end up calling ourselves, what TAutoFlag does not like
   GetCurrentAccess();
 
   {
@@ -1671,7 +1671,7 @@ bool __fastcall TCustomIniFileStorage::HasAccess(unsigned int Access)
 //===========================================================================
 TIniFileStorage * __fastcall TIniFileStorage::CreateFromPath(const UnicodeString & AStorage)
 {
-  // The code was originally inline in the parent contructor call in the TIniFileStorage::TIniFileStorage [public originally].
+  // The code was originally inline in the parent constructor call in the TIniFileStorage::TIniFileStorage [public originally].
   // But if the TMemIniFile constructor throws (e.g. because the INI file is locked), the exception causes access violation.
   // Moving the code to a factory solves this.
   TMemIniFile * IniFile = new TMemIniFile(AStorage);
@@ -1911,7 +1911,7 @@ UnicodeString __fastcall TOptionsIniFile::FormatKey(const UnicodeString & Sectio
   {
     Result += PathDelim;
   }
-  Result += Ident; // Can be empty, when called from a contructor, AllowSection or ReadSection
+  Result += Ident; // Can be empty, when called from a constructor, AllowSection or ReadSection
   if (DebugAlwaysTrue(AllowSection(Section)))
   {
     Result.Delete(1, FRootKey.Length());

source/core/PuttyIntf.cpp

@@ -779,7 +779,7 @@ TPrivateKey * LoadKey(TKeyType KeyType, const UnicodeString & FileName, const Un
   {
     // While theoretically we may get "unable to open key file" and
     // so we should check system error code,
-    // we actully never get here unless we call KeyType previously
+    // we actually never get here unless we call KeyType previously
     // and handle ktUnopenable accordingly.
     Error = AnsiString(ErrorStr);
   }

source/core/Queue.cpp

@@ -1323,8 +1323,8 @@ void __fastcall TTerminalItem::ProcessEvent()
     UnicodeString Message;
     if (ExceptionMessageFormatted(&E, Message))
     {
-      // do not show error messages, if task was cancelled anyway
-      // (for example if transfer is cancelled during reconnection attempts)
+      // do not show error messages, if task was canceled anyway
+      // (for example if transfer is canceled during reconnection attempts)
       if (!FCancel &&
           (FTerminal->QueryUserException(L"", &E, qaOK | qaCancel, NULL, qtError) == qaCancel))
       {
@@ -2705,7 +2705,7 @@ void __fastcall TTerminalThread::FatalAbort()
 {
   if (FAbandoned)
   {
-    // We cannot use TTerminal::FatalError as the terminal still runs on a backgroud thread,
+    // We cannot use TTerminal::FatalError as the terminal still runs on a background thread,
     // may have its TCallbackGuard armed right now.
     throw ESshFatal(NULL, L"");
   }

source/core/RemoteFiles.cpp

@@ -1423,7 +1423,7 @@ void __fastcall TRemoteFile::FindLinkedFile()
     {
       if (LinkedBy->LinkTo == LinkTo)
       {
-        // this is currenly redundant information, because it is used only to
+        // this is currently redundant information, because it is used only to
         // detect broken symlink, which would be otherwise detected
         // by FLinkedFile == NULL
         FCyclicLink = true;
@@ -3102,7 +3102,7 @@ __int64 TSynchronizeProgress::GetProcessed(const TFileOperationProgressType * Cu
   DebugAssert(!TFileOperationProgressType::IsIndeterminateOperation(CurrentItemOperationProgress->Operation));
 
   // Need to calculate the total size on the first call only,
-  // as at the time the constructor it called, we usually do not have sizes of folders caculated yet.
+  // as at the time the constructor it called, we usually do not have sizes of folders calculated yet.
   if (FTotalSize < 0)
   {
     FTotalSize = 0;

source/core/S3FileSystem.cpp

@@ -105,7 +105,7 @@ static void NeedS3Config()
 TStrings * GetS3Profiles()
 {
   NeedS3Config();
-  // S3 allegedly treats the section case-sensitivelly, but our GetS3ConfigValue (ReadString) does not,
+  // S3 allegedly treats the section case-sensitively, but our GetS3ConfigValue (ReadString) does not,
   // so consistently we return case-insensitive list.
   std::unique_ptr<TStrings> Result(new TStringList());
   if (S3ConfigFile.get() != NULL)
@@ -1237,8 +1237,8 @@ void TS3FileSystem::ReadDirectoryInternal(
       if (Data.IsTruncated)
       {
         // We have report that with max-keys=1, server can return IsTruncated response with no keys,
-        // so we would loop infinitelly. For now, if we do GET request only to check for bucket/folder existence (MaxKeys == -1),
-        // we are happy with a successfull response and never loop, even if IsTruncated.
+        // so we would loop infinitely. For now, if we do GET request only to check for bucket/folder existence (MaxKeys == -1),
+        // we are happy with a successful response and never loop, even if IsTruncated.
         if ((MaxKeys == 0) ||
             ((MaxKeys > 0) && (Data.KeyCount < MaxKeys)))
         {
@@ -1253,7 +1253,7 @@ void TS3FileSystem::ReadDirectoryInternal(
     } while (Continue);
 
     // Listing bucket root directory will report an error if the bucket does not exist.
-    // But there won't be any prefix/ entry, so if the bucket is ampty, the Data.Any is false.
+    // But there won't be any prefix/ entry, so if the bucket is empty, the Data.Any is false.
     // But when listing a prefix, we do not get any error, when the "prefix" does not exist.
     // But when the prefix does exist, there's at least the prefix/ entry. If there's none, it means that the path does not exist.
     // Even an empty-named entry/subprefix (which are ignored for other purposes) still indicate that the prefix exists.
@@ -1662,7 +1662,7 @@ unsigned short TS3FileSystem::AclGrantToPermissions(S3AclGrant & AclGrant, const
     }
     else
     {
-      FTerminal->LogEvent(1, FORMAT(L"Unspported permission for canonical user %s", (StrFromS3(Properties.OwnerId))));
+      FTerminal->LogEvent(1, FORMAT(L"Unsupported permission for canonical user %s", (StrFromS3(Properties.OwnerId))));
     }
   }
   else if (AclGrant.granteeType == S3GranteeTypeAllAwsUsers)

source/core/ScpFileSystem.cpp

@@ -967,7 +967,7 @@ void __fastcall TSCPFileSystem::ChangeDirectory(const UnicodeString ADirectory)
   }
 
   UnicodeString ToDir;
-  // This effectivelly disallows entering subdirectories starting with ~ and containing space
+  // This effectively disallows entering subdirectories starting with ~ and containing space
   if (!Directory.IsEmpty() &&
       ((Directory[1] != L'~') || (Directory.SubString(1, 2) == L"~ ")))
   {
@@ -1991,7 +1991,7 @@ void __fastcall TSCPFileSystem::SCPSource(const UnicodeString FileName,
         // We do ASCII transfer: convert EOL of current block
         // (we don't convert whole buffer, cause it would produce
         // huge memory-transfers while inserting/deleting EOL characters)
-        // Than we add current block to file buffer
+        // Then we add current block to file buffer
         if (OperationProgress->AsciiTransfer)
         {
           int ConvertParams =

source/core/SessionData.cpp

@@ -953,7 +953,7 @@ void __fastcall TSessionData::Load(THierarchicalStorage * Storage, bool PuttyImp
     // In case we are re-loading, reset passwords, to avoid pointless
     // re-cryption, while loading username/hostname. And moreover, when
     // the password is wrongly encrypted (using a different master password),
-    // this breaks sites reload and consequently an overal operation,
+    // this breaks sites reload and consequently an overall operation,
     // such as opening Sites menu
     ClearSessionPasswords();
     FProxyPassword = L"";
@@ -1667,7 +1667,7 @@ void TSessionData::ImportFromOpenssh(TStrings * Lines)
           {
             if (Negated)
             {
-              // Skip even if matched by other possitive patterns
+              // Skip even if matched by other positive patterns
               SkippingSection = true;
               break;
             }

source/core/SftpFileSystem.cpp

@@ -1636,7 +1636,7 @@ protected:
     if (Response->Capacity > 0)
     {
       // particularly when uploading a file that completely fits into send buffer
-      // over slow line, we may end up seemingly completing the transfer immediatelly
+      // over slow line, we may end up seemingly completing the transfer immediately
       // but hanging the application for a long time waiting for responses
       // (common is that the progress window would not even manage to draw itself,
       // showing that upload finished, before the application "hangs")
@@ -3910,7 +3910,7 @@ void __fastcall TSFTPFileSystem::CopyFile(
     try
     {
       SourceRemoteHandle = SFTPOpenRemoteFile(FileNameCanonical, SSH_FXF_READ, Encrypted, Size);
-      // SFTP_EXT_COPY_FILE does not allow overwritting existing files
+      // SFTP_EXT_COPY_FILE does not allow overwriting existing files
       // (the specification does not mandate it, but it is implemented like that both in ProFTPD and Bitvise).
       // So using SSH_FXF_EXCL for consistency.
       DestRemoteHandle = SFTPOpenRemoteFile(NewNameCanonical, SSH_FXF_WRITE | SSH_FXF_CREAT | SSH_FXF_EXCL, Encrypted, Size);
@@ -3987,7 +3987,7 @@ void __fastcall TSFTPFileSystem::CreateLink(const UnicodeString FileName,
         {
           // ProFTPD/mod_sftp followed OpenSSH symlink bug even for link implementation.
           // This will be fixed with the next release with
-          // SSH version string bumbed to "mod_sftp/1.0.0"
+          // SSH version string bumped to "mod_sftp/1.0.0"
           // http://bugs.proftpd.org/show_bug.cgi?id=4080
           UnicodeString ProFTPDVerStr = GetSessionInfo().SshImplementation;
           CutToChar(ProFTPDVerStr, L'/', false);
@@ -4847,7 +4847,7 @@ void __fastcall TSFTPFileSystem::Source(
     __finally
     {
       // Either queue is empty now (noop call then),
-      // or some error occured (in that case, process remaining responses, ignoring other errors)
+      // or some error occurred (in that case, process remaining responses, ignoring other errors)
       Queue.DisposeSafe();
     }
 
@@ -5038,7 +5038,7 @@ RawByteString __fastcall TSFTPFileSystem::SFTPOpenRemoteFile(
   bool SendSize =
     (Size >= 0) &&
     FLAGSET(OpenType, SSH_FXF_CREAT | SSH_FXF_TRUNC) &&
-    // Particuarly VanDyke VShell (4.0.3) does not support SSH_FILEXFER_ATTR_ALLOCATION_SIZE
+    // Particularly VanDyke VShell (4.0.3) does not support SSH_FILEXFER_ATTR_ALLOCATION_SIZE
     // (it fails open request when the attribute is included).
     // It's SFTP-6 attribute, so support structure should be available.
     // It's actually not with VShell. But VShell supports the SSH_FILEXFER_ATTR_ALLOCATION_SIZE.
@@ -5763,7 +5763,7 @@ void __fastcall TSFTPFileSystem::Sink(
     }
 
     // if the transfer was finished, the file is usually closed already
-    // (except for special cases like SFTPv6 EOF indication or partial file downlaod)
+    // (except for special cases like SFTPv6 EOF indication or partial file download)
     if (FTerminal->Active && !RemoteHandle.IsEmpty())
     {
       // do not wait for response

source/core/Terminal.cpp

@@ -814,7 +814,7 @@ void TParallelOperation::Done(
       else
       {
         // This is actually not useful at the moment, as when creating directory fails and "Skip" is pressed,
-        // the current code in CopyToRemote/CreateDirectory will behave as, if it succedded, so Successs will be true here.
+        // the current code in CopyToRemote/CreateDirectory will behave as, if it succeded, so Success will be true here.
         FDirectories.erase(DirectoryIterator);
         if (FFileList->Count > FListIndex)
         {
@@ -1655,7 +1655,7 @@ void __fastcall TTerminal::Open()
     }
     __finally
     {
-      // This does not make it through, if terminal thread is abandonded,
+      // This does not make it through, if terminal thread is abandoned,
       // see also TTerminalManager::DoConnectTerminal
       DoInformation(L"", true, 0);
     }
@@ -1841,16 +1841,16 @@ void __fastcall TTerminal::Reopen(int Params)
   bool PrevAutoReadDirectory = FAutoReadDirectory;
   // here used to be a check for FExceptionOnFail being 0
   // but it can happen, e.g. when we are downloading file to execute it.
-  // however I'm not sure why we mind having excaption-on-fail enabled here
+  // however I'm not sure why we mind having exception-on-fail enabled here
   int PrevExceptionOnFail = FExceptionOnFail;
   try
   {
     FReadCurrentDirectoryPending = false;
     FReadDirectoryPending = false;
-    // Not sure why we are suspeding the transaction in the first place,
-    // but definitelly when set while connecting auto loaded workspace session, it causes loading the directory twice.
+    // Not sure why we are suspending the transaction in the first place,
+    // but definitely when set while connecting auto loaded workspace session, it causes loading the directory twice.
     // (when reconnecting lost connection, it's usually prevented by cached directory)
-    // Preventing that by suspeding transaction only when there is one.
+    // Preventing that by suspending transaction only when there is one.
     FSuspendTransaction = (FInTransaction > 0);
     FExceptionOnFail = 0;
     // typically, we avoid reading directory, when there is operation ongoing,
@@ -1917,7 +1917,7 @@ bool __fastcall TTerminal::PromptUser(TSessionData * Data, TPromptKind Kind,
   // If PromptUser is overridden in descendant class, the overridden version
   // is not called when accessed via TSessionIU interface.
   // So this is workaround.
-  // Actually no longer needed as we do not uverride DoPromptUser
+  // Actually no longer needed as we do not override DoPromptUser
   // anymore in TSecondaryTerminal.
   return DoPromptUser(Data, Kind, Name, Instructions, Prompts, Results);
 }
@@ -3152,7 +3152,7 @@ unsigned int __fastcall TTerminal::ConfirmFileOverwrite(
     }
     else
     {
-      // particularly with parallel transfers, the overal operation can be already cancelled by other parallel operation
+      // particularly with parallel transfers, the overall operation can be already cancelled by other parallel operation
       if (OperationProgress->Cancel > csContinue)
       {
         LogEvent(1, L"Transfer cancelled in parallel operation");
@@ -3220,19 +3220,19 @@ unsigned int __fastcall TTerminal::ConfirmFileOverwrite(
     switch (BatchOverwrite)
     {
       case boAll:
-        LogEvent(1, L"Overwritting all files");
+        LogEvent(1, L"Overwriting all files");
         Result = qaYes;
         break;
 
       case boNone:
-        LogEvent(1, L"Not overwritting any file");
+        LogEvent(1, L"Not overwriting any file");
         Result = qaNo;
         break;
 
       case boOlder:
         if (FileParams == NULL)
         {
-          LogEvent(1, L"Not overwritting due to lack of file information");
+          LogEvent(1, L"Not overwriting due to lack of file information");
           Result = qaNo;
         }
         else
@@ -4172,7 +4172,7 @@ TUsableCopyParamAttrs __fastcall TTerminal::UsableCopyParamAttrs(int Params)
     FLAGMASK(FLAGSET(Params, cpDelete), cpaNoClearArchive) |
     FLAGMASK(!IsCapable[fcIgnorePermErrors], cpaNoIgnorePermErrors) |
     // the following three are never supported for download,
-    // so when they are not suppored for upload too,
+    // so when they are not supported for upload too,
     // set them in General flags, so that they do not get enabled on
     // Synchronize dialog.
     FLAGMASK(!IsCapable[fcModeChangingUpload], cpaNoRights) |
@@ -7624,7 +7624,7 @@ void __fastcall TTerminal::DirectorySource(
     CreateTargetDirectory(DestFullName, Attrs, CopyParam);
   }
 
-  // Paralell transfers (cpNoRecurse) won't be allowed if any of these are set anyway (see CanParallel).
+  // Parallel transfers (cpNoRecurse) won't be allowed if any of these are set anyway (see CanParallel).
   // Exception is ClearArchive, which is does not prevent parallel transfer, but is silently ignored for directories.
   if (DoRecurse && !OperationProgress->Cancel)
   {
@@ -7665,7 +7665,7 @@ bool TTerminal::UseAsciiTransfer(
   return
     IsCapable[fcTextMode] &&
     CopyParam->UseAsciiTransfer(BaseFileName, Side, MaskParams) &&
-    // Used either before PartSize is set (to check that file is to be transfered in binary mode, hence allows parallel file transfer),
+    // Used either before PartSize is set (to check that file is to be transferred in binary mode, hence allows parallel file transfer),
     // or later during parallel transfer (which never happens for ascii mode).
     DebugAlwaysTrue(CopyParam->PartSize < 0);
 }
@@ -8223,7 +8223,7 @@ void __fastcall TTerminal::Sink(
 
         FFileSystem->DirectorySunk(DestFullName, File, CopyParam);
 
-        // Do not delete directory if some of its files were skip.
+        // Do not delete directory if some of its files were skipped.
         // Throw "skip file" for the directory to avoid attempt to deletion
         // of any parent directory
         if (FLAGSET(Params, cpDelete) && SinkFileParams.Skipped)

source/core/WebDAVFileSystem.cpp

@@ -1196,7 +1196,7 @@ void __fastcall TWebDAVFileSystem::ConfirmOverwrite(
   switch (Answer)
   {
     case qaYes:
-    // Can happen when moving to background (and the server manages to commit the interrupeted foreground transfer).
+    // Can happen when moving to background (and the server manages to commit the interrupted foreground transfer).
     // WebDAV does not support resumable uploads.
     // Resumable downloads are not implemented.
     case qaRetry:
@@ -1926,7 +1926,7 @@ int TWebDAVFileSystem::NeonRequestAuth(
     {
       if (FileSystem->FIgnoreAuthenticationFailure == iafPasswordFailed)
       {
-        // Fail PROPFIND /nonexising request...
+        // Fail PROPFIND /nonexisting request...
         Result = false;
       }
       else

source/dragext/DragExt.cpp

@@ -468,7 +468,7 @@ static bool UnregisterServer(bool AllUsers)
     // Previously we were setting the value explicitly to 0,
     // but doing that for both HKLM and HKCU.
     // While on register, we set it to 1 for HKLM only,
-    // what makes the extension disabled effectivelly,
+    // what makes the extension disabled effectively,
     // as KHCU value 0, is kept even after re-registration
     RegDeleteValue(HKey, L"Enable");
 

source/dragext/DragExt.h

@@ -18,7 +18,7 @@ DEFINE_GUID(CLSID_ShellExtension, 0xe15e1d68, 0x0d1c, 0x49f7,
 #pragma pack(push, 4)
 //---------------------------------------------------------------------------
 // Note that the change between 0 and 1 was incompatible in both directions.
-// So unfortunatelly version 0 extension (4.x and older) will accept incompatible request
+// So unfortunately version 0 extension (4.x and older) will accept incompatible request
 // from version 1 application (5.x and newer).
 // Luckily the extension will gracefully fail, when using the Unicode path in
 // GetShortPathName(CommStruct->DropDest, ...) and will ignore the request.

source/filezilla/FtpControlSocket.cpp

@@ -4530,7 +4530,7 @@ void CFtpControlSocket::TransferFinished(bool preserveFileTimeForUploads)
         // that the server supports non-standard hack
         // of setting timestamp using
         // MFMT-like (two argument) call to MDTM.
-        // IIS definitelly does.
+        // IIS definitely does.
         command = L"MDTM";
       }
       if (Send( command + L" " + timestr + L" " + filename))

source/forms/CopyLocal.cpp

@@ -86,7 +86,7 @@ void __fastcall TCopyLocalDialog::ShortCutHintLabelClick(TObject *)
 void __fastcall TCopyLocalDialog::FormShow(TObject *)
 {
   InstallPathWordBreakProc(DirectoryEdit);
-  // Does not work when set from a contructor
+  // Does not work when set from a constructor
   ShortCutHintPanel->Color = Application->HintColor;
   UpdateControls();
 }

source/forms/CustomScpExplorer.cpp

@@ -1955,7 +1955,7 @@ TTBXPopupMenu * __fastcall TCustomScpExplorerForm::HistoryMenu(
     // workaround
     // In Pascal the size of TTBXPopupMenu is 132, in C++ 136,
     // operator new allocates memory in Pascal code, but calls inline
-    // contructor in C++, leading in problems, the function does
+    // constructor in C++, leading in problems, the function does
     // both in Pascal code
     FHistoryMenu[Side == osLocal][Back] = CreateTBXPopupMenu(this);
     UpdateHistoryMenu(Side, Back);
@@ -5367,7 +5367,7 @@ void __fastcall TCustomScpExplorerForm::DuplicateTab()
     TTerminalManager * Manager = TTerminalManager::Instance();
     TManagedTerminal * ASession = Manager->NewManagedTerminal(SessionData.get());
     DebugAssert(ASession->LocalBrowser == ManagedSession->LocalBrowser);
-    // We definitelly want these
+    // We definitely want these
     ASession->Disconnected = ManagedSession->Disconnected;
     // Not sure about these two
     ASession->Permanent = ManagedSession->Permanent;

source/forms/Glyphs.cpp

@@ -22,7 +22,7 @@ __fastcall TGlyphsModule::TGlyphsModule(TComponent* Owner)
   SetPixelsPerInch(Screen->PixelsPerInch);
 }
 //---------------------------------------------------------------------------
-// Contructor without scaling
+// Constructor without scaling
 __fastcall TGlyphsModule::TGlyphsModule()
   : TDataModule(Application)
 {

source/forms/Login.cpp

@@ -130,7 +130,7 @@ void __fastcall TLoginDialog::InitControls()
   int FtpsNoneIndex = FtpsToIndex(ftpsNone);
   int FtpsImplicitIndex = FtpsToIndex(ftpsImplicit);
   // Items item setter is implemented as deleting and re-adding the item. If we do it for the last item
-  // (explicit for FTP, implicit for WebDAV/S3), the ItemIndex is effectivelly reset to -1.
+  // (explicit for FTP, implicit for WebDAV/S3), the ItemIndex is effectively reset to -1.
   // This happens when TLS is set in the default session settings.
   // Also as TransferProtocolComboChange is not triggered it results in currupted state in respect to protocol/tls to port number sync.
   int Index = FtpsCombo->ItemIndex;

source/packages/jcl/jedi.inc

@@ -464,7 +464,7 @@
   SUPPORTS_CLASS_FIELDS          Compiler supports class fields (D9.NET, D10+)
   SUPPORTS_CLASS_HELPERS         Compiler supports class helpers (D9.NET, D10+)
   SUPPORTS_CLASS_OPERATORS       Compiler supports class operators (D9.NET, D10+)
-  SUPPORTS_CLASS_CTORDTORS       Compiler supports class contructors/destructors (D14+)
+  SUPPORTS_CLASS_CTORDTORS       Compiler supports class constructors/destructors (D14+)
   SUPPORTS_STRICT                Compiler supports strict keyword (D9.NET, D10+)
   SUPPORTS_STATIC                Compiler supports static keyword (D9.NET, D10+)
   SUPPORTS_FINAL                 Compiler supports final keyword (D9.NET, D10+)

source/windows/GUITools.cpp

@@ -2050,7 +2050,7 @@ void __fastcall TFrameAnimation::Repaint()
   FPaintBox->Repaint();
   if (!FPainted)
   {
-    // Paint later, alternativelly we may keep trying Repaint() in Animate().
+    // Paint later, alternatively we may keep trying Repaint() in Animate().
     // See also a hack in TAuthenticateForm::Log.
     FPaintBox->Invalidate();
   }

source/windows/WinInterface.cpp

@@ -1200,11 +1200,11 @@ void __fastcall CenterButtonImage(TButton * Button)
     // The margins seem to extend the area over which the image is centered,
     // so we have to set it to a double of desired padding.
     // The original formula is - 2 * ((CaptionWidth / 2) - (ImageWidth / 2) + ScaleByTextHeight(Button, 2))
-    // the one below is equivalent, but with reduced rouding.
+    // the one below is equivalent, but with reduced rounding.
     // Without the change, the rounding caused the space between icon and caption too
     // small on 200% zoom.
     // Note that (CaptionWidth / 2) - (ImageWidth / 2)
-    // is approximatelly same as half of caption width before padding.
+    // is approximately same as half of caption width before padding.
     Button->ImageMargins->Left = -(CaptionWidth - ImageWidth + ScaleByTextHeight(Button, 4));
   }
   else
@@ -1492,7 +1492,7 @@ static void __fastcall AppGetMainFormHandle(void * /*Data*/, HWND & Handle)
   TForm * MainForm = GetMainForm();
   // This, among other, causes minimizing of the top-level non-MainForm minimize other child windows.
   // Like clicking "Minimize" on Progress window over Synchronization progress window over Synchronization checklist window.
-  // Would also have a lot of other effects (hopefully possitive) and may render lot of existing MainFormLike code obsolete.
+  // Would also have a lot of other effects (hopefully positive) and may render lot of existing MainFormLike code obsolete.
   if ((MainForm != NULL) && IsMainFormLike(MainForm) && MainForm->HandleAllocated())
   {
     Handle = MainForm->Handle;