Differential D1612

Fixing curly braces and indentation in misleading parts of code
ClosedPublic
Actions

Authored by martinkostolny on May 14 2016, 5:21 PM.

Details

Reviewers
asensi
yurchor
Group Reviewers
Krusader
Summary

As schwarzer pointed out (here D1471) we have some misleading if statement indentations. This diff should fix the situation.

Test Plan

Tested firing up Disk Usage window on folder with subfolders and files.

Diff Detail

Repository
R167 Krusader
Lint
Lint Skipped
Unit
Unit Tests Skipped
martinkostolny updated this revision to Diff 3817.May 14 2016, 5:21 PM
martinkostolny retitled this revision from to Fixing curly braces and indentation in misleading parts of code.
martinkostolny updated this object.
martinkostolny edited the test plan for this revision. (Show Details)
martinkostolny added a reviewer: Krusader.
martinkostolny set the repository for this revision to R167 Krusader.
yurchor accepted this revision.May 14 2016, 6:54 PM
yurchor added a reviewer: yurchor.
yurchor added a subscriber: yurchor.

Tested to work as expected.

This revision is now accepted and ready to land.May 14 2016, 6:54 PM
asensi accepted this revision.May 15 2016, 11:35 AM
asensi added a reviewer: asensi.
asensi added a subscriber: asensi.

Thanks!

martinkostolny closed this revision.May 15 2016, 9:37 PM

Thanks for testing! Pushed.

Revision Contents
Changeset List

Diff 3817

View Options

krusader/DiskUsage/diskusage.cpp

Loading...
View Options

krusader/VFS/virtualcopyjob.cpp

Loading...