<html>
    <head>
      <base href="https://bugs.webkit.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [GTK] Add settings to set font size in points, and deprecate the pixel size settings"
   href="https://bugs.webkit.org/show_bug.cgi?id=142673#c26">Comment # 26</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [GTK] Add settings to set font size in points, and deprecate the pixel size settings"
   href="https://bugs.webkit.org/show_bug.cgi?id=142673">bug 142673</a>
              from <span class="vcard"><a class="email" href="mailto:givascu@igalia.com" title="Gabriel Ivașcu <givascu@igalia.com>"> <span class="fn">Gabriel Ivașcu</span></a>
</span></b>
        <pre>(In reply to Michael Catanzaro from <a href="show_bug.cgi?id=142673#c25">comment #25</a>)
<span class="quote">> (In reply to Gabriel Ivașcu from <a href="show_bug.cgi?id=142673#c24">comment #24</a>)
> > Since there's no overload of std::hypot that takes two ints, I wasn't sure
> > which version of std::hypot C++ will call. We'd prefer the one that has
> > doubles for parameters for reasons of precision, so that's why I added the
> > casts.

> cppreference is your friend:

> <a href="http://en.cppreference.com/w/cpp/numeric/math/hypot">http://en.cppreference.com/w/cpp/numeric/math/hypot</a>

> I think, without the casts, you'll get the fourth overload, since it's being
> called with int parameters. And that says:

> "4) A set of overloads or a function template for all combinations of
> arguments of arithmetic type not covered by (1-3). If any argument has
> integral type, it is cast to double. If any other argument is long double,
> then the return type is long double, otherwise it is double."

> so it will be double even without the casts, and the casts aren't needed.
> Could you check?</span >

Yes, you're right, I was going to comment on this myself. The integral types are converted to double by default, so there's no need for explicit conversion.</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>