if();
George Staikos
staikos at kde.org
Wed Jun 11 18:47:22 BST 2003
I found these in cvs so far. They're if() statements with empty
consequences. I am hesitant to touch most of them because they of course
change the flow of the code. It would be good if the authors could review
them to see what needs to be done.
./kdebase/kioslave/imap4/imapparser.cc: if (parseOneNumber (inWords, size));
./kdebase/kioslave/imap4/imapparser.cc: if (value);
./kdebase/ksplashml/wndmain.cpp: if(name.isNull());
./koffice/lib/koscript/koscript_func.cc: if ( !KSUtil::checkArgumentsCount(
context, 2, "arg", true ) );
./koffice/filters/kspread/opencalc/opencalcimport.cc: if ( !header.isNull()
);
./koffice/filters/kspread/opencalc/opencalcimport.cc: if ( !footer.isNull()
);
./kdepim/kmail/kmgroupware.cpp: if (cardConv.addresseeToVCard(addressee, s,
VCardConverter::v3_0));
./kdevelop/parts/distpart/specsupport.cpp: if
(getInfo(s,"Name:",&SpecSupport::setAppName));
./kdevelop/parts/distpart/specsupport.cpp: else if
(getInfo(s,"Version:",&SpecSupport::setAppVersion));
./kdevelop/parts/distpart/specsupport.cpp: else if
(getInfo(s,"Release:",&SpecSupport::setAppRevision));
./kdevelop/parts/distpart/specsupport.cpp: else if
(getInfo(s,"Vendor:",&SpecSupport::setAppVendor));
./kdevelop/parts/distpart/specsupport.cpp: else if
(getInfo(s,"Copyright:",&SpecSupport::setAppLicense));
./kdevelop/parts/distpart/specsupport.cpp: else if
(getInfo(s,"Summary:",&SpecSupport::setAppSummary));
./kdevelop/parts/distpart/specsupport.cpp: else if
(getInfo(s,"Group:",&SpecSupport::setAppGroup));
./kdevelop/parts/distpart/specsupport.cpp: else if
(getInfo(s,"Packager:",&SpecSupport::setAppPackager));
--
George Staikos
More information about the kde-core-devel
mailing list