From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: AS4713 221.184.0.0/13 X-Spam-Status: No, score=-2.8 required=3.0 tests=AWL,BAYES_00, DKIM_ADSP_CUSTOM_MED,FORGED_GMAIL_RCVD,FREEMAIL_FORGED_FROMDOMAIN, FREEMAIL_FROM,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, RCVD_IN_DNSWL_MED,SPF_PASS shortcircuit=no autolearn=no autolearn_force=no version=3.4.2 Received: from neon.ruby-lang.org (neon.ruby-lang.org [221.186.184.75]) by dcvr.yhbt.net (Postfix) with ESMTP id 2C4EC1F45F for ; Wed, 8 May 2019 08:28:30 +0000 (UTC) Received: from neon.ruby-lang.org (localhost [IPv6:::1]) by neon.ruby-lang.org (Postfix) with ESMTP id 660C2120A1B; Wed, 8 May 2019 17:28:24 +0900 (JST) Received: from o1678916x28.outbound-mail.sendgrid.net (o1678916x28.outbound-mail.sendgrid.net [167.89.16.28]) by neon.ruby-lang.org (Postfix) with ESMTPS id B1F421209C1 for ; Wed, 8 May 2019 17:28:21 +0900 (JST) Received: by filter0004p3iad2.sendgrid.net with SMTP id filter0004p3iad2-27034-5CD29326-1F 2019-05-08 08:28:22.744526185 +0000 UTC m=+37524.750524542 Received: from herokuapp.com (unknown [34.228.113.206]) by ismtpd0023p1mdw1.sendgrid.net (SG) with ESMTP id 6grDthtwSIOmtqL4G4-qgA for ; Wed, 08 May 2019 08:28:22.485 +0000 (UTC) Date: Wed, 08 May 2019 08:28:22 +0000 (UTC) From: jean.boussier@gmail.com Message-ID: References: Mime-Version: 1.0 X-Redmine-MailingListIntegration-Message-Ids: 68078 X-Redmine-Project: ruby-trunk X-Redmine-Issue-Id: 15836 X-Redmine-Issue-Author: byroot X-Redmine-Sender: byroot X-Mailer: Redmine X-Redmine-Host: bugs.ruby-lang.org X-Redmine-Site: Ruby Issue Tracking System X-Auto-Response-Suppress: All Auto-Submitted: auto-generated X-SG-EID: =?us-ascii?Q?AchqQMoUBMcQgz7gop0XiYUiatGIY7E61JGsTL4FvjfX0DIIHG3+xx9Rd1WEsK?= =?us-ascii?Q?vBUVNF4gHx1rEggkIJ8uxf8TD81igS+lE1Jv8r2?= =?us-ascii?Q?S5pwpb1LO7wj84nVuRu8hdQwODtWxcI5gTvO24w?= =?us-ascii?Q?QbtRx4ZFXhk708ODPAX+jDtK3q9Y8XAnC7Oj8eK?= =?us-ascii?Q?tevCD57SqDodQpDvssuCnsLM4N1dtVZK+lw=3D=3D?= To: ruby-core@ruby-lang.org X-ML-Name: ruby-core X-Mail-Count: 92595 Subject: [ruby-core:92595] [Ruby trunk Feature#15836] [Proposal] Make Module#name and Symbol#to_s return their internal fstrings X-BeenThere: ruby-core@ruby-lang.org X-Mailman-Version: 2.1.15 Precedence: list Reply-To: Ruby developers List-Id: Ruby developers List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ruby-core-bounces@ruby-lang.org Sender: "ruby-core" Issue #15836 has been updated by byroot (Jean Boussier). > Is HashWithIndifferentAccess the main rationale behind this request? No. It's simply the poster child of how common `Symbol#to_s` is in code bases. I shouldn't have mentioned `HashWithIndifferentAccess` because clearly lots of people have a feud with it, and now it's totally shifting the conversation. ### What the proposal is actually about The question here, is wether `Module#name` and `Symbol#to_s` should return a new string on every call. My own understanding of why it's like this is because historically all strings were mutable, so the way to prevent them to be mutated was to duplicate them. But now that frozen strings are very common in code bases, and that `fstring` are a thing, IMHO getting a new string on every call is what is surprising. ---------------------------------------- Feature #15836: [Proposal] Make Module#name and Symbol#to_s return their internal fstrings https://bugs.ruby-lang.org/issues/15836#change-77955 * Author: byroot (Jean Boussier) * Status: Feedback * Priority: Normal * Assignee: * Target version: ---------------------------------------- # Why ? In many codebases, especially Rails apps, these two methods are the source of quite a lot of object allocations. `Module#name` is often accessed for various introspection features, autoloading etc. `Symbol#to_s` is access a lot by HashWithIndifferentAccess other various APIs accepting both symbols and strings. Returning fstrings for both of these methods could significantly reduce allocations, as well as sligthly reduce retention as it would reduce some duplications. Also, more and more Ruby APIs are now returning fstrings. `frozen_string_literal`AFAIK should become the default some day, string used as hash keys are now automatically interned as well. ### Backward compatibilty Of course this is not fully backward compatible, it's inevitable that some code in the wild is mutating the strings returned by these methods, but I do believe it's a rare occurence, and easy to fix. ### Implementation I implemented it here: https://github.com/ruby/ruby/pull/2175 -- https://bugs.ruby-lang.org/