diff --git a/source/src/protocols/denovo_design/components/FoldGraph.cc b/source/src/protocols/denovo_design/components/FoldGraph.cc index 9b3ac5c635e..0f894a0c78f 100644 --- a/source/src/protocols/denovo_design/components/FoldGraph.cc +++ b/source/src/protocols/denovo_design/components/FoldGraph.cc @@ -852,7 +852,7 @@ FoldGraph::compute_best_solution( SegmentNames const & staple_loops ) const solutions[sol].push_back( new_visited ); } else { Solution tmpset; - for ( core::Size t=1; t dis_sc_sc ) { temp_shortest_dis = dis_sc_sc; diff --git a/source/src/protocols/fldsgn/MatchResidues.cc b/source/src/protocols/fldsgn/MatchResidues.cc index 6afccf812dc..e6b8b95c120 100644 --- a/source/src/protocols/fldsgn/MatchResidues.cc +++ b/source/src/protocols/fldsgn/MatchResidues.cc @@ -62,7 +62,7 @@ core::Real MatchResidues::compute_comb( core::pose::Pose const & pose, VecSize const & comb ) const { std::map< core::id::AtomID, core::id::AtomID > atom_id_map; - for ( core::Size i = 1; i < comb.size(); i++ ) { + for ( core::Size i = 1; i <= comb.size(); i++ ) { const core::id::AtomID mod_id(pose.residue_type( comb[i] ).atom_index( "CA" ), comb[i] ); const core::id::AtomID ref_id(pose.residue_type( reference_residues_indexes_[i] ).atom_index( "CA" ), reference_residues_indexes_[i]); atom_id_map.insert( std::make_pair(mod_id, ref_id) ); @@ -75,7 +75,7 @@ MatchResidues::superimpose_comb( core::pose::Pose & pose, VecSize const & comb ) { core::id::AtomID_Map< core::id::AtomID > atom_map; core::pose::initialize_atomid_map( atom_map, pose, core::id::AtomID::BOGUS_ATOM_ID() ); - for ( core::Size i = 1; i < comb.size(); ++i ) { + for ( core::Size i = 1; i <= comb.size(); ++i ) { const core::id::AtomID mod_id(pose.residue_type( comb[i] ).atom_index( "CA" ), comb[i] ); const core::id::AtomID ref_id(pose.residue_type( reference_residues_indexes_[i] ).atom_index( "CA" ), reference_residues_indexes_[i]); atom_map.set( mod_id, ref_id); diff --git a/source/src/protocols/legacy_sewing/conformation/Assembly.cc b/source/src/protocols/legacy_sewing/conformation/Assembly.cc index 58398aa8f23..b0e3228499e 100644 --- a/source/src/protocols/legacy_sewing/conformation/Assembly.cc +++ b/source/src/protocols/legacy_sewing/conformation/Assembly.cc @@ -1259,7 +1259,7 @@ Assembly::natives_select( utility::vector1 Assembly::pose_loop_anchors() const { utility::vector1 loop_anchors; - for ( core::Size i=1; i Assembly::disconnected_segments() const { utility::vector1 disconnected_segments; - for ( core::Size i=1; i helix_pair; - for ( core::Size upstream_res = 1; upstream_res < pose.size(); ++upstream_res ) { + for ( core::Size upstream_res = 1; upstream_res <= pose.size(); ++upstream_res ) { if ( pose.secstruct(upstream_res) == 'H' ) { for ( core::Size downstream_res = upstream_res+1; downstream_res <= pose.size(); ++downstream_res ) { if ( pose.secstruct(downstream_res) == 'H' ) { - if ( element_blocks[upstream_res] != element_blocks[downstream_res] && pose.residue(upstream_res).xyz(2).distance(pose.residue(upstream_res).xyz(2)) <= crit_dist_ ) { + if ( element_blocks[upstream_res] != element_blocks[downstream_res] && pose.residue(upstream_res).xyz(2).distance(pose.residue(downstream_res).xyz(2)) <= crit_dist_ ) { helix_pair.first = element_blocks[upstream_res]; helix_pair.second = element_blocks[downstream_res]; helix_pairs.insert(helix_pair); @@ -117,7 +117,7 @@ BlockwiseAnalysisMover::apply( core::pose::Pose& pose){ //core::Size label_res = 1; for ( auto current_pair : helix_pairs ) { scc.Reset(); // this may not be needed anymore, but I'm leaving it here for safety - for ( core::Size current_res = 1; current_res < pose.size(); ++current_res ) { + for ( core::Size current_res = 1; current_res <= pose.size(); ++current_res ) { if ( element_blocks[current_res] == current_pair.first ) { scc.AddResidue( 0, pose.residue(current_res) ); } @@ -134,7 +134,7 @@ BlockwiseAnalysisMover::apply( core::pose::Pose& pose){ d_median = r.distance; has_disulfide = false; core::Size last_upstream_res = 1; - for ( core::Size upstream_res = 1; upstream_res add_reslabel(1,label); - for ( core::Size upstream_res = 1; upstream_res helix_pair; - for ( core::Size upstream_res = 1; upstream_res < pose.size(); ++upstream_res ) { + for ( core::Size upstream_res = 1; upstream_res <= pose.size(); ++upstream_res ) { if ( pose.secstruct(upstream_res) == 'H' ) { for ( core::Size downstream_res = upstream_res+2; downstream_res <= pose.size(); ++downstream_res ) { if ( pose.secstruct(downstream_res) == 'H' ) { - if ( element_blocks[upstream_res] != element_blocks[downstream_res] && pose.residue(upstream_res).xyz(2).distance(pose.residue(upstream_res).xyz(2)) <= crit_dist_ ) { + if ( element_blocks[upstream_res] != element_blocks[downstream_res] && pose.residue(upstream_res).xyz(2).distance(pose.residue(downstream_res).xyz(2)) <= crit_dist_ ) { helix_pair.first = element_blocks[upstream_res]; helix_pair.second = element_blocks[downstream_res]; helix_pairs.insert(helix_pair); @@ -195,7 +195,7 @@ OmnibusDisulfideAnalysisLabelerMover::apply( core::pose::Pose& pose){ utility::vector1< core::Size > selection1; utility::vector1< core::Size > selection2; - for ( core::Size current_res = 1; current_res < pose.size(); ++current_res ) { + for ( core::Size current_res = 1; current_res <= pose.size(); ++current_res ) { if ( element_blocks[current_res] == current_pair.first ) { selection1.push_back(current_res); } diff --git a/source/src/protocols/rna/movers/RNAIdealizeMover.cc b/source/src/protocols/rna/movers/RNAIdealizeMover.cc index ee9a6a5ec17..4de3dadac5e 100644 --- a/source/src/protocols/rna/movers/RNAIdealizeMover.cc +++ b/source/src/protocols/rna/movers/RNAIdealizeMover.cc @@ -237,7 +237,7 @@ RNAIdealizeMover::apply( pose::Pose & pose ) suite_mm->set_jump( true ); protocols::minimization_packing::MinMoverOP minm = utility::pointer::make_shared< protocols::minimization_packing::MinMover >( suite_mm, scorefxn, "lbfgs_armijo_nonmonotone", 0.001, true ); - for ( Size ii = 1; ii < ideal_pose.size(); ++ii ) { + for ( Size ii = 1; ii <= ideal_pose.size(); ++ii ) { for ( Size jj = 1; jj <= ideal_pose.residue_type( ii ).natoms(); ++jj ) { ConstraintOP constraint = utility::pointer::make_shared< CoordinateConstraint >( core::id::AtomID( jj, ii ), core::id::AtomID( 1, my_anchor ), diff --git a/source/src/protocols/splice/SampleRotamersFromPDB.cc b/source/src/protocols/splice/SampleRotamersFromPDB.cc index 6989a6224bd..1ce9bd78534 100644 --- a/source/src/protocols/splice/SampleRotamersFromPDB.cc +++ b/source/src/protocols/splice/SampleRotamersFromPDB.cc @@ -229,7 +229,7 @@ void SampleRotamersFromPDB_RotamerSetOperation::fill_rotamer_matrix_from_db_file } //new_res->set_all_chi(Rots_real); bool res_exsits_in_db = false; - for ( core::Size i=1; isize(); row++ ) { //go over all the PSSM sements provided by the user + for ( core::Size row = 1; row <= seqprof->size(); row++ ) { //go over all the PSSM sements provided by the user utility::vector1< core::Size > cur_prof_row = seqprof->prof_row(row); diff --git a/source/src/protocols/stepwise/legacy/modeler/rna/StepWiseRNA_WorkingParametersSetup.cc b/source/src/protocols/stepwise/legacy/modeler/rna/StepWiseRNA_WorkingParametersSetup.cc index 259c8c04164..200985a32ed 100644 --- a/source/src/protocols/stepwise/legacy/modeler/rna/StepWiseRNA_WorkingParametersSetup.cc +++ b/source/src/protocols/stepwise/legacy/modeler/rna/StepWiseRNA_WorkingParametersSetup.cc @@ -233,7 +233,7 @@ StepWiseWorkingParametersSetup::get_user_input_alignment_res_list( core::Size co ObjexxFCL::FArray1D < bool > const & partition_definition = working_parameters_->partition_definition(); bool contain_non_root_partition_seq_num = false; - for ( core::Size ii = 1; ii < working_alignment.size(); ii++ ) { + for ( core::Size ii = 1; ii <= working_alignment.size(); ii++ ) { if ( partition_definition( working_alignment[ii] ) != partition_definition( root_res ) ) contain_non_root_partition_seq_num = true; }