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