Description, because “alt text” can’t show it well:

			{
				emit differentFiles (ckFile.absoluteFilePath(),
					otherFile.absoluteFilePath(),
					FileCompareWorker::FileComparisonParams{FileComparisonParams::FileNameMatch,
						(ckFile.size() > otherFile.size()) ? FileComparisonParams::File1IsLarger
							: FileComparisonParams::File2IsLarger});
			}

After Alignment

			{
				emit differentFiles (ckFile.absoluteFilePath(),
					otherFile.absoluteFilePath(),
					FileCompareWorker::FileComparisonParams{FileComparisonParams::FileNameMatch,
						(ckFile.size() > otherFile.size()) ? FileComparisonParams::File1IsLarger
														   : FileComparisonParams::File2IsLarger});
			}
  • ulterno@programming.devOP
    link
    fedilink
    English
    arrow-up
    2
    ·
    edit-2
    16 hours ago

    That looks nice and easy to understand.
    I would go one step further and according to my preferences:
    (starting from line5)

    FileCompareWorker::FileComparisonParams
    {
    	FileComparisonParams::FileNameMatch,
    	(ckFile.size() > otherFile.size())
    		? FileComparisonParams::File1IsLarger
    		: FileComparisonParams::File2IsLarger
    }
    

    Break before the curly as I do for most.