Author: gs Date: Wed Mar 11 12:01:58 2009 New Revision: 56068 URL: http://svn.opensuse.org/viewcvs/yast?rev=56068&view=rev Log: updateInfo(): add missing break; (patch related info), add checks for valid pointer, omit default Modified: trunk/ncurses-pkg/src/NCPkgTable.cc Modified: trunk/ncurses-pkg/src/NCPkgTable.cc URL: http://svn.opensuse.org/viewcvs/yast/trunk/ncurses-pkg/src/NCPkgTable.cc?rev=56068&r1=56067&r2=56068&view=diff ============================================================================== --- trunk/ncurses-pkg/src/NCPkgTable.cc (original) +++ trunk/ncurses-pkg/src/NCPkgTable.cc Wed Mar 11 12:01:58 2009 @@ -1141,27 +1141,38 @@ switch (mode) { case I_Descr: - packager->InfoText()->longDescription( pkgPtr ); + if ( packager->InfoText() ) + packager->InfoText()->longDescription( pkgPtr ); break; case I_Technical: - packager->InfoText()->technicalData( pkgPtr, slbPtr ); + if ( packager->InfoText() ) + packager->InfoText()->technicalData( pkgPtr, slbPtr ); break; case I_Files: - packager->InfoText()->fileList( slbPtr ); + if ( packager->InfoText() ) + packager->InfoText()->fileList( slbPtr ); break; case I_Deps: - packager->InfoText()->dependencyList( pkgPtr, slbPtr ); + if ( packager->InfoText() ) + packager->InfoText()->dependencyList( pkgPtr, slbPtr ); break; case I_Versions: - packager->VersionsList()->fillAvailableList( slbPtr ); + if ( packager->VersionsList() ) + packager->VersionsList()->fillAvailableList( slbPtr ); break; case I_PatchDescr: - packager->InfoText()->patchDescription( pkgPtr, slbPtr ); + if ( packager->InfoText() ) + packager->InfoText()->patchDescription( pkgPtr, slbPtr ); + break; case I_PatchPkgs: - packager->fillPatchPackages( packager->PatchPkgs(), pkgPtr ); + if ( packager->PatchPkgs() ) + packager->fillPatchPackages( packager->PatchPkgs(), pkgPtr ); + break; case I_PatchPkgsVersions: - packager->fillPatchPackages( packager->PatchPkgsVersions(), pkgPtr, true ); - default: + if ( packager->PatchPkgsVersions() ) + packager->fillPatchPackages( packager->PatchPkgsVersions(), pkgPtr, true ); break; + // Intentionally omitting 'default' branch so the compiler can + // catch unhandled enum states } } -- To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org For additional commands, e-mail: yast-commit+help@opensuse.org