<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>62215</bug_id>
          
          <creation_ts>2011-06-07 09:00:46 -0700</creation_ts>
          <short_desc>Make style/checkers/cpp.py allow comparisons with 0 in LIKELY and UNLIKELY</short_desc>
          <delta_ts>2011-06-07 12:55:19 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Tools / Tests</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Dmitry Lomov">dslomov</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>dslomov</cc>
    
    <cc>levin</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>416539</commentid>
    <comment_count>0</comment_count>
    <who name="Dmitry Lomov">dslomov</who>
    <bug_when>2011-06-07 09:00:46 -0700</bug_when>
    <thetext>cpp_style.py does not currenly allow

if (LIKELY(ptr != 0)) ....</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>416623</commentid>
    <comment_count>1</comment_count>
      <attachid>96259</attachid>
    <who name="Dmitry Lomov">dslomov</who>
    <bug_when>2011-06-07 10:40:10 -0700</bug_when>
    <thetext>Created attachment 96259
Fix + unit-test</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>416743</commentid>
    <comment_count>2</comment_count>
      <attachid>96259</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-06-07 12:55:15 -0700</bug_when>
    <thetext>Comment on attachment 96259
Fix + unit-test

Clearing flags on attachment: 96259

Committed r88256: &lt;http://trac.webkit.org/changeset/88256&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>416745</commentid>
    <comment_count>3</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-06-07 12:55:19 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>96259</attachid>
            <date>2011-06-07 10:40:10 -0700</date>
            <delta_ts>2011-06-07 12:55:15 -0700</delta_ts>
            <desc>Fix + unit-test</desc>
            <filename>62215.patch</filename>
            <type>text/plain</type>
            <size>2527</size>
            <attacher name="Dmitry Lomov">dslomov</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCAzZTBi
MDdmLi4zOWZiOTFhIDEwMDY0NAotLS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hh
bmdlTG9nCkBAIC0xLDMgKzEsMTMgQEAKKzIwMTEtMDYtMDcgIERtaXRyeSBMb21vdiAgPGRzbG9t
b3ZAZ29vZ2xlLmNvbT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKwor
ICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NjIyMTUKKyAg
ICAgICAgQWxsb3cgY29tcGFyaXNvbnMgd2l0aCAwIGluIExJS0VMWSBhbmQgVU5MSUtFTFkgbWFj
cm9zLgorCisgICAgICAgICogU2NyaXB0cy93ZWJraXRweS9zdHlsZS9jaGVja2Vycy9jcHAucHk6
CisgICAgICAgICogU2NyaXB0cy93ZWJraXRweS9zdHlsZS9jaGVja2Vycy9jcHBfdW5pdHRlc3Qu
cHk6CisKIDIwMTEtMDYtMDcgIE5pY28gV2ViZXIgIDx0aGFraXNAY2hyb21pdW0ub3JnPgogCiAg
ICAgICAgIFJldmlld2VkIGJ5IERpbWl0cmkgR2xhemtvdi4KZGlmZiAtLWdpdCBhL1Rvb2xzL1Nj
cmlwdHMvd2Via2l0cHkvc3R5bGUvY2hlY2tlcnMvY3BwLnB5IGIvVG9vbHMvU2NyaXB0cy93ZWJr
aXRweS9zdHlsZS9jaGVja2Vycy9jcHAucHkKaW5kZXggY2E1ZDFkOS4uZWMzYjNlZSAxMDA2NDQK
LS0tIGEvVG9vbHMvU2NyaXB0cy93ZWJraXRweS9zdHlsZS9jaGVja2Vycy9jcHAucHkKKysrIGIv
VG9vbHMvU2NyaXB0cy93ZWJraXRweS9zdHlsZS9jaGVja2Vycy9jcHAucHkKQEAgLTIzNDEsOCAr
MjM0MSw5IEBAIGRlZiBjaGVja19mb3JfY29tcGFyaXNvbnNfdG9femVybyhjbGVhbl9saW5lcywg
bGluZV9udW1iZXIsIGVycm9yKToKIAogICAgICMgSW5jbHVkZSBOVUxMIGhlcmUgc28gdGhhdCB1
c2VycyBkb24ndCBoYXZlIHRvIGNvbnZlcnQgTlVMTCB0byAwIGZpcnN0IGFuZCB0aGVuIGdldCB0
aGlzIGVycm9yLgogICAgIGlmIHNlYXJjaChyJ1s9IV09XHMqKE5VTEx8MHx0cnVlfGZhbHNlKVxX
JywgbGluZSkgb3Igc2VhcmNoKHInXFcoTlVMTHwwfHRydWV8ZmFsc2UpXHMqWz0hXT0nLCBsaW5l
KToKLSAgICAgICAgZXJyb3IobGluZV9udW1iZXIsICdyZWFkYWJpbGl0eS9jb21wYXJpc29uX3Rv
X3plcm8nLCA1LAotICAgICAgICAgICAgICAnVGVzdHMgZm9yIHRydWUvZmFsc2UsIG51bGwvbm9u
LW51bGwsIGFuZCB6ZXJvL25vbi16ZXJvIHNob3VsZCBhbGwgYmUgZG9uZSB3aXRob3V0IGVxdWFs
aXR5IGNvbXBhcmlzb25zLicpCisgICAgICAgIGlmIG5vdCBzZWFyY2goJ0xJS0VMWScsIGxpbmUp
IGFuZCBub3Qgc2VhcmNoKCdVTkxJS0VMWScsIGxpbmUpOgorICAgICAgICAgICAgZXJyb3IobGlu
ZV9udW1iZXIsICdyZWFkYWJpbGl0eS9jb21wYXJpc29uX3RvX3plcm8nLCA1LAorICAgICAgICAg
ICAgICAgICAgJ1Rlc3RzIGZvciB0cnVlL2ZhbHNlLCBudWxsL25vbi1udWxsLCBhbmQgemVyby9u
b24temVybyBzaG91bGQgYWxsIGJlIGRvbmUgd2l0aG91dCBlcXVhbGl0eSBjb21wYXJpc29ucy4n
KQogCiAKIGRlZiBjaGVja19mb3JfbnVsbChjbGVhbl9saW5lcywgbGluZV9udW1iZXIsIGZpbGVf
c3RhdGUsIGVycm9yKToKZGlmZiAtLWdpdCBhL1Rvb2xzL1NjcmlwdHMvd2Via2l0cHkvc3R5bGUv
Y2hlY2tlcnMvY3BwX3VuaXR0ZXN0LnB5IGIvVG9vbHMvU2NyaXB0cy93ZWJraXRweS9zdHlsZS9j
aGVja2Vycy9jcHBfdW5pdHRlc3QucHkKaW5kZXggMWJkOGEwZS4uOGUxYjhjNyAxMDA2NDQKLS0t
IGEvVG9vbHMvU2NyaXB0cy93ZWJraXRweS9zdHlsZS9jaGVja2Vycy9jcHBfdW5pdHRlc3QucHkK
KysrIGIvVG9vbHMvU2NyaXB0cy93ZWJraXRweS9zdHlsZS9jaGVja2Vycy9jcHBfdW5pdHRlc3Qu
cHkKQEAgLTQxMzcsNiArNDEzNywxOSBAQCBjbGFzcyBXZWJLaXRTdHlsZVRlc3QoQ3BwU3R5bGVU
ZXN0QmFzZSk6CiAgICAgICAgIHNlbGYuYXNzZXJ0X2xpbnQoCiAgICAgICAgICAgICAnaWYgKG90
aGVydHJ1ZSA9PSBmb250VHlwZSknLAogICAgICAgICAgICAgJycpCisgICAgICAgIHNlbGYuYXNz
ZXJ0X2xpbnQoCisgICAgICAgICAgICAnaWYgKExJS0VMWShmb28gPT0gMCkpJywKKyAgICAgICAg
ICAgICcnKQorICAgICAgICBzZWxmLmFzc2VydF9saW50KAorICAgICAgICAgICAgJ2lmIChVTkxJ
S0VMWShmb28gPT0gMCkpJywKKyAgICAgICAgICAgICcnKQorICAgICAgICBzZWxmLmFzc2VydF9s
aW50KAorICAgICAgICAgICAgJ2lmIChMSUtFTFkoZm9vID09IE5VTEwpKScsCisgICAgICAgICAg
ICAnVXNlIDAgaW5zdGVhZCBvZiBOVUxMLiAgW3JlYWRhYmlsaXR5L251bGxdIFs1XScpCisgICAg
ICAgIHNlbGYuYXNzZXJ0X2xpbnQoCisgICAgICAgICAgICAnaWYgKFVOTElLRUxZKGZvbyA9PSBO
VUxMKSknLAorICAgICAgICAgICAgJ1VzZSAwIGluc3RlYWQgb2YgTlVMTC4gIFtyZWFkYWJpbGl0
eS9udWxsXSBbNV0nKQorCiAKICAgICBkZWYgdGVzdF91c2luZ19zdGQoc2VsZik6CiAgICAgICAg
IHNlbGYuYXNzZXJ0X2xpbnQoCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>