<html>
    <head>
      <base href="https://bugs.webkit.org/" />
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Fixed compilation with !ENABLE(SVG_FONTS)."
   href="https://bugs.webkit.org/show_bug.cgi?id=156850#c5">Comment # 5</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Fixed compilation with !ENABLE(SVG_FONTS)."
   href="https://bugs.webkit.org/show_bug.cgi?id=156850">bug 156850</a>
              from <span class="vcard"><a class="email" href="mailto:annulen&#64;yandex.ru" title="Konstantin Tokarev &lt;annulen&#64;yandex.ru&gt;"> <span class="fn">Konstantin Tokarev</span></a>
</span></b>
        <pre>(In reply to <a href="show_bug.cgi?id=156850#c4">comment #4</a>)
<span class="quote">&gt; (In reply to <a href="show_bug.cgi?id=156850#c3">comment #3</a>)
&gt; &gt; (In reply to <a href="show_bug.cgi?id=156850#c2">comment #2</a>)
&gt; &gt; &gt; We prefer guarding source files to guarding build sytem when it is possible.
&gt; &gt; 
&gt; &gt; It's the opposite of what we've been doing recently. Maybe we should discuss
&gt; &gt; this on webkit-dev?
&gt; 
&gt; What do you mean? I haven't seen any patch removing if guards from source
&gt; file.</span >

<a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - Remove USE(TEXTURE_MAPPER) guards inside TextureMapper sources."
   href="show_bug.cgi?id=155944">https://bugs.webkit.org/show_bug.cgi?id=155944</a>

Though in case of TEXTURE_MAPPER there is no direct condition in build system, instead choice is made by inclusion (or not inclusion) of TextureMapper.cmake in PlatformXXX.cmake.

<span class="quote">&gt; Source files are almost always guarded properly, I can't see a good reason
&gt; to break this good practice now. I think most of the build system guard
&gt; is redundant.</span >

What is good about guards in .cpp files? They add no information in most cases, and also slightly increase build time, not only because of having to compile empty files, but also extra time is needed to generate large ninja manifest, also ninja has to process more edges.</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>