Feature #11938: [PATCH] cleanup: stdlib: avoid extra calls to eliminate "\n" from Base64

• Status: Closed
• Assignee: -
• Target version: -

[normalperson (Eric Wong)]

stdlib: avoid extra calls to eliminate "\n" from Base64

We may use the '0' (zero) to avoid adding the line feed.
Furthermore, the '*' (asterisk) modifier is not needed for
a single-element arrays.

• Status changed from Open to Closed

Applied in changeset r53488.

---

stdlib: avoid extra calls to eliminate "\n" from Base64

We may use the '0' (zero) to avoid adding the line feed.
Furthermore, the '*' (asterisk) modifier is not needed for
a single-element arrays.

• ext/psych/lib/psych/visitors/yaml_tree.rb (visit_String):
  eliminate chomp
• lib/net/http.rb (connect): eliminate delete
• lib/net/http/header.rb (basic_encode): ditto
• lib/net/imap.rb (authenticate): eliminate gsub
  (self.encode_utf7): shorten delete arg
• lib/net/smtp.rb (base64_encode): eliminate gsub
• lib/open-uri.rb (OpenURI.open_http): eliminate delete
• lib/rss/rss.rb: ditto
• lib/securerandom.rb (base64): ditto
  (urlsafe_base64): eliminate delete!
• lib/webrick/httpauth/digestauth.rb (split_param_value):
  eliminate chop
• lib/webrick/httpproxy.rb (do_CONNECT): eliminate delete
  (setup_upstream_proxy_authentication): ditto
  [ruby-core:72666] [Feature #11938]