Rename update_extentions to Normalize_extentions as it better conveys the intent;
Remove the unnecessary copy of the extensions vector;
Remove the unnecessary copy of the RustusConf object;
By the way, is there any reason for the extensions to be sorted? I couldn't find any mention of that in the specification. I'd argue that it's better to drop this requirement if there's no reason for it and switch to HashSet instead of Vec. If it's required for any other reason, we can still switch to BTreeSet.
update_extentionstoNormalize_extentionsas it better conveys the intent;RustusConfobject;By the way, is there any reason for the extensions to be sorted? I couldn't find any mention of that in the specification. I'd argue that it's better to drop this requirement if there's no reason for it and switch to
HashSetinstead ofVec. If it's required for any other reason, we can still switch toBTreeSet.