Merge pull request #4308 from RosettaCommons/roccomoretti/reorganize_rosettascript_handling
Reorganize RosettaScriptsParser tag handling.
RosettaScriptsParser::generate_mover_for_protocol() was a bit of a muddle. Break out the detailed handling into sub-functions. Also, iterate through the subtags once, in order, rather than in multiple stages.
* This may cause issues for some existing RosettaScripts, which (incorrectly) rely on having sections like RESIDUE_SELECTORS or SCOREFUNCTIONS _after_ the mover/filter section which use them. These should be rare, and can be fixed by reordering the tags. (Remembering you can repeat sections.)