<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
  </head>
  <body>
    <br>
    <br>
    <div class="moz-cite-prefix">On 10/31/21 7:24 PM, David Kahurani
      wrote:<br>
    </div>
    <blockquote type="cite"
cite="mid:CAAZOf27hjTqauPsGaL_y4OvnA=nqjWSgX30RsAAqmEyFqROKSA@mail.gmail.com">
      <meta http-equiv="content-type" content="text/html; charset=UTF-8">
      <div dir="ltr">
        <div dir="ltr"><br>
        </div>
        <br>
        <div class="gmail_quote">
          <div dir="ltr" class="gmail_attr">On Sun, Oct 31, 2021 at 7:17
            PM Nikolay Sivov <<a href="mailto:nsivov@codeweavers.com"
              moz-do-not-send="true">nsivov@codeweavers.com</a>>
            wrote:<br>
          </div>
          <blockquote class="gmail_quote" style="margin:0px 0px 0px
            0.8ex;border-left:1px solid
            rgb(204,204,204);padding-left:1ex">
            <div> <br>
              <br>
              <div>On 10/31/21 7:14 PM, David Kahurani wrote:<br>
              </div>
              <blockquote type="cite">
                <div dir="ltr">
                  <div dir="ltr"><br>
                  </div>
                  <br>
                  <div class="gmail_quote">
                    <div dir="ltr" class="gmail_attr">On Sun, Oct 31,
                      2021 at 7:11 PM Nikolay Sivov <<a
                        href="mailto:nsivov@codeweavers.com"
                        target="_blank" moz-do-not-send="true">nsivov@codeweavers.com</a>>
                      wrote:<br>
                    </div>
                    <blockquote class="gmail_quote" style="margin:0px
                      0px 0px 0.8ex;border-left:1px solid
                      rgb(204,204,204);padding-left:1ex"><br>
                      <br>
                      On 10/31/21 6:34 PM, David Kahurani wrote:<br>
                      > Signed-off-by: David Kahurani <<a
                        href="mailto:k.kahurani@gmail.com"
                        target="_blank" moz-do-not-send="true">k.kahurani@gmail.com</a>><br>
                      > ---<br>
                      >  include/commctrl.h | 4 ++--<br>
                      >  1 file changed, 2 insertions(+), 2
                      deletions(-)<br>
                      ><br>
                      > diff --git a/include/commctrl.h
                      b/include/commctrl.h<br>
                      > index 41abceb..d67f5f7 100644<br>
                      > --- a/include/commctrl.h<br>
                      > +++ b/include/commctrl.h<br>
                      > @@ -3939,8 +3939,8 @@ typedef struct
                      tagLVITEMINDEX<br>
                      >     
                      (INT)SNDMSGW((hwnd),LVM_SETITEMW,0,(LPARAM)(const
                      LVITEMW *)(pitem))<br>
                      >  #define ListView_SetItem
                      WINELIB_NAME_AW(ListView_SetItem)<br>
                      >  #define
                      ListView_SetItemState(hwnd,i,data,dataMask) \<br>
                      > -{ LVITEM _LVi; _LVi.state = data;
                      _LVi.stateMask = dataMask;\<br>
                      > -  SNDMSG(hwnd, LVM_SETITEMSTATE,
                      (WPARAM)(UINT)i, (LPARAM) (LPLVITEM)&_LVi);}<br>
                      > +{ LVITEMW _LVi; _LVi.state = data;
                      _LVi.stateMask = dataMask;\<br>
                      > +  SNDMSGW(hwnd, LVM_SETITEMSTATE,
                      (WPARAM)(UINT)i, (LPARAM) (LPLVITEMW)&_LVi);}<br>
                      >  #define ListView_GetItemState(hwnd,i,mask) \<br>
                      >     
(UINT)SNDMSG((hwnd),LVM_GETITEMSTATE,(WPARAM)(UINT)(i),(LPARAM)(UINT)(mask))<br>
                      >  #define ListView_SetCheckState(hwndLV, i,
                      bCheck) \<br>
                      <br>
                      This doesn't match what SDK does.<br>
                      <br>
                    </blockquote>
                    <div>Are you saying there is some other magic beyond
                      what is already in this macro or a difference in
                      width(s) used? <br>
                    </div>
                  </div>
                </div>
              </blockquote>
              <br>
              SNDMSG resolves to either SendMessageA or SendMessageW, I
              don't see anything special for _SetItemState() in SDK.<br>
            </div>
          </blockquote>
          <div><br>
          </div>
          <div>SNDMSG will resolve to an error if you try to use this
            macro. LVMITEM should also be either LVMITEMA or LVMITEMW.
            It looked to me like this code preceeds these conventions.<br>
          </div>
          <div><br>
          </div>
          <blockquote class="gmail_quote" style="margin:0px 0px 0px
            0.8ex;border-left:1px solid
            rgb(204,204,204);padding-left:1ex">
            <div> <br>
              It's not clear what motivated this change, could you
              clarify?<br>
            </div>
          </blockquote>
          <div><br>
          </div>
          <div>I was trying to use ListView_SetItemState in some other
            code.<br>
          </div>
        </div>
      </div>
    </blockquote>
    <br>
    If you tried to use it in Wine, yes, it won't work, because you
    can't use SendMessage macro when in __WINESRC__. Instead of
    ListView_* macros you'll need to make explicit SendMessage calls.<br>
  </body>
</html>