The /o in Ruby regex stands for "oh the humanity "

(jpcamara.com)

174 points | by todsacerdoti a day ago ago

39 comments

  • fanf2 a day ago

    This is one of the features that Ruby cribbed directly from Perl. The Ruby documentation seems really bad, in particular “interpolation mode” is grievously misleading.

    Perl’s documentation is far more clear about the consequences:

    (https://perldoc.perl.org/perlop#Regexp-Quote-Like-Operators)

       o   Compile pattern only once.
    
      […]
    
      PATTERN may contain variables, which will be
      interpolated every time the pattern search is
      evaluated, except for when the delimiter is a
      single quote. […] Perl will not recompile the
      pattern unless an interpolated variable that
      it contains changes. You can force Perl to skip
      the test and never recompile by adding a /o
      (which stands for "once") after the trailing
      delimiter. Once upon a time, Perl would recompile
      regular expressions unnecessarily, and this
      modifier was useful to tell it not to do so,
      in the interests of speed. But now, the only
      reasons to use /o are one of:
    
      [reasons]
    
      The bottom line is that using /o is almost
      never a good idea.
    
    In the days before Perl automatically memoized the compilation of regexes with interpolation, even back in the 1990s, it said,

      However, mentioning /o constitutes a promise
      that you won't change the variables in the
      pattern. If you change them, Perl won't even
      notice.
    
    Perl 4’s documentation is briefer. It says,

    (https://github.com/Perl/perl5/blob/perl-4.0.00/perl.man#L272...)

      PATTERN may contain references to scalar
      variables, which will be interpolated
      (and the pattern recompiled) every time the
      pattern search is evaluated. […] If you want
      such a pattern to be compiled only once, add
      an “o” after the trailing delimiter. This
      avoids expensive run-time recompilations, and
      is useful when the value you are interpolating
      won't change over the life of the script.
  • jononor a day ago

    It looks like an emoji for someone getting bashed in the head with a long stick. So that makes sense?

  • tialaramex 21 hours ago

    This is a footgun. A language should strive not to add footguns. Every footgun you provide, somebody is going to blow their foot off with it, so that's a high price. If your language is popular it might be a lot of somebodies.

    The opposite behaviour (we have a constant regular expression, we re-use it often but the tooling doesn't realise and so it's created each time we mention it) is not a footgun, it results in poor performance, and so you might want (especially in some managed languages) to just magically optimise this case, but if not you won't cause mysterious bugs. An expert, asked "Why is this slow?" can just fix it - you have to supply basic tools for that, but this flag is not a sensible tool.

    • roughly an hour ago

      One of my favorites was Python’s datetime.time() object evaluating to True for every value except exact midnight, which is the sort of thing that makes fine sense when you think about the underlying implementation but is absolutely going to take a toe off of someone.

      My favorite part about that one was it got to go through the full feature deprecation cycle before removal because several people argued in the bug thread about it that they were relying on that behavior in their systems.

    • elif 17 hours ago

      Is it really though? There are tons of characters you can add to a regex that have difficult if not impossible to mentally comprehend impacts on the potential matches. That's why you need 100 test cases for every 10 characters you write in a regex. Regex itself could all be a footgun by this standard. No one in the history of no one has ever thought "why dont I just add a random character to my regex I don't need or understand" that's just boogie man level irrational fear if you think this has any bearing on the ease of use of ruby.

      • stouset 15 hours ago

        Regexes are not fundamentally hard. People make regexes hard by trying to parse things by sight rather than finding a spec. If you have a spec, and it can be parsed by a regular expression, they are pretty damn rote to implement.

        If you aren’t working from a specified input grammar, the task is going to be borderline impossible no matter the tool and you’re going to have a bad time. If you aren’t working with a regular grammar, this is the wrong tool for the job and again you’re going to have a bad time.

        A hint; if you find yourself using `.`, you are probably shooting yourself in the foot.

      • pitched 16 hours ago

        Ruby is a well-sharpened knife. Not everyone should be given a sharp knife though, especially children. And not all jobs need a sharp knife, like buttering toast. So I think it’s good for dull knives to exist as part of your tool belt. If we can only choose one language though, I’d rather it be a nimble, sharp one.

    • emmelaich 12 hours ago

      Sometimes you want to blow your foot off.

  • riffraff a day ago

    Unsurprisingly, `END {}` is also inherited from perl, tho I think it originally comes from awk.

  • cbsmith a day ago

    As an old Perl programmer, I knew immediately what the /o would do. ;-)

  • kazinator a day ago

    > Modifier o means that the first time a literal regexp with interpolations is encountered, the generated Regexp object is saved and used for all future evaluations of that literal regexp.

    That is crystal clear to me. It means that on the next execution, the new values of the interpolation will be ignored; the regexp is now "baked" with the first ones.

    Like this in C++:

      void fun(int arg)
      {
         static int once = arg;
      }
    
    if we call this as f(42) the first time, once gets initialized to 42. If we then call it f(73), once stays 42.

    There is a function in POSIX for once-only initializations: pthread_once. C++ compilers for multithreaded environments emit thread-safe code to do something similar to pthread_once to ensure that even if there are several concurrent first invocations of the function, the initialization happens once.

  • rco8786 a day ago

    Love these sorts of deep dives, thanks!

  • alfiedotwtf 12 hours ago

    If you don’t like /o, you’re going to hate Perl’s /e

  • lupire a day ago

    This is the same problem people have with closures, where it's unclear to the user whether the argument is captured by name or by value.

    • layer8 a day ago

      This isn't the same problem, because this is about whether the regex is instantiated each time the code around the regex is executed, or only the first time and cached for subsequent executions. The same could in theory happen with closures, but I haven't ever seen programming-language semantics where, for example, a function containing the definition of a closure that depends on an argument of that outer function, would use the argument value of the first invocation of the function for all subsequent invocations of the function.

      For example, when you have

          fn f x = (y -> x + y)
      
      then a sequence of invocations of f

          f 1 3
          f 2 6
      
      will yield 4 and 8 respectively, but never will the second invocation yield 7 due to reusing the value of x from the first invocation. However, that is precisely what happens in the article's regex example, because the equivalent is for the closure value (y -> x + y) to be cached between invocations, so that the x retains the value of the first invocation of f — regardless of whether x is a reference by name or by value.
    • ethan_smith 21 hours ago

      The parallel is apt, but regex /o is more like a closure that captures by value at declaration time rather than an ambiguity between capture strategies.

  • phoronixrly a day ago

    It's kind of a cool feature. I like it.

    • thayne a day ago

      Is it? I can't think of a non-contrived case where this would actually be useful.

      And in any case where it would be useful, it seems like a better way to optimize would just be to refactor the regex out into a constant.

      • naniwaduni a day ago

        The context is that this is a feature cribbed straight from perl, where where it's passed down from perl 4/pre-5.6, where compiled regexen weren't first-class values. Pretty much every use of it this century is a mistake.

      • kayodelycaon a day ago

        Actually, I have a way this would work well. If you’re interpolating a value that comes from configuration and wouldn’t change.

        Example: /admin@#{Rails.config.x.domain}/io

        But you’re right that a constant would be a lot more clear. “o” is a footgun.

      • baobun 15 hours ago

        An HTTP application server matching routes based on runtime configiuration (domains and whatnot) is not really that niche or contrived? Loads of other situations where input not changing during the thread/process lifecycle is part of a set of hot regexes large enough that explicitly compiling each is not a great experience.

        I, for one, appreciate /o.

  • jwlake 18 hours ago

    this is similar to the g modifier in javascript?

    • Lio 10 hours ago

      No, g is the global modifier so gives you multiple mateches rather than stopping on the first match encountered.

  • zer00eyz a day ago

    Im sorry but the classics never go out of style:

    "Some people, when confronted with a problem, think 'I know, I'll use regular expressions.' Now they have two problems."

    • stavros a day ago

      Yeah but it's kind of tired when it's being used every time someone makes a mistake with regex. I've used them extensively in my career and never once regretted it.

      • apgwoz a day ago

        The problem with regexps is that “Sometimes a smart person, who has done the work, and knows how to leverage regular expressions correctly, decides they are appropriate for solving a problem where there is shared maintenance. Now, you have people who haven’t put in the work, and have been told repeatedly through ‘witty quips’ to not bother.”

    • jodrellblank a day ago

      The second problem being how to deal with all the extra time they just freed up?

  • IshKebab a day ago

    Seems par for the course for Ruby.

  • Joker_vD a day ago

    > I didn’t recognize /o. It didn’t seem critically important to lookup yet.

    > With nothing else to investigate, I finally looked up the docs for what the /o regex modifier does.

    I'll probably never understand this mode of thinkning. But then again, Ruby programmers are, after all, people who chose to write Ruby.

    > /o is referred to as “Interpolation mode”, which sounded pretty harmless.

    Really? Those words sound quite alarming to me, due to personal reminiscences of eval.

    Also, this whole "/o" feaure seems insane. If I have an interpolation in my regex, obviously I have to re-interpolate it every time a new value is submitted, or I'd hit this very bug. And if the value is expected to the same every time, then I can just compile it once and save the result myself, right? In which case, I probably could even do without interpolation in the first place.

    • gpvos 21 hours ago

      It's a feature dating from the 1990s, when Perl (and I guess Ruby?) didn't have a way for the user to store a compiled regex, and this was a useful shortcut for a very specific optimization, which Ruby documented badly. Perl (and I guess Ruby?) later evolved in a way that made /o unnecessary, but the (now mis)feature remained.

    • apgwoz a day ago

      “Compilation”, I think, is exactly right. This feature is less about interpolation than it is about compilation of a single regexp to be used many times. It’s just shrouded in confusing documentation that should say: “/o tells ruby to rewrite this code such that it refers to a new statically allocated regexp object.” And when you write it that way, you see how insane it is for a function call to be hoisted automatically like this, without an explicit, obvious, syntactic annotation.

      • gpvos 21 hours ago

        The implications of "statically allocated" are less clear than if you'd just write "compiled only once".