Remove redundant calls

sort_internal and thread_internal have an else statement that can be
removed because normalize_searching_criteria(search_keys) is called
anyways after the case.

Author: gobijan

lib/net/imap.rb

@@ -2880,8 +2880,6 @@ def copy_internal(cmd, set, mailbox)
     def sort_internal(cmd, sort_keys, search_keys, charset)
       if search_keys.instance_of?(String)
         search_keys = [RawData.new(search_keys)]
-      else
-        normalize_searching_criteria(search_keys)
       end
       normalize_searching_criteria(search_keys)
       synchronize do
@@ -2893,8 +2891,6 @@ def sort_internal(cmd, sort_keys, search_keys, charset)
     def thread_internal(cmd, algorithm, search_keys, charset)
       if search_keys.instance_of?(String)
         search_keys = [RawData.new(search_keys)]
-      else
-        normalize_searching_criteria(search_keys)
       end
       normalize_searching_criteria(search_keys)
       synchronize do