[prev in list] [next in list] [prev in thread] [next in thread] 

List:       wine-devel
Subject:    Re: Add a GUI for dxdiag program
From:       Aaryaman Vasishta <jem456.vasishta () gmail ! com>
Date:       2016-08-27 16:25:29
Message-ID: CABVHfRutYtrMsPgW8d-0KOhaoi=inWBk8XOhyzaj9vG_gHgRGg () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


On Sat, Aug 27, 2016 at 9:12 PM, Ruslan Kabatsayev <b7.10110111@gmail.com>
wrote:

> On Sat, Aug 27, 2016 at 12:31 PM, Aaryaman Vasishta
> <jem456.vasishta@gmail.com> wrote:
> >>
> >> +INT_PTR CALLBACK system_tab_proc(HWND hsystab, UINT msg, WPARAM wparam,
> >> LPARAM lparam)
> >> +{
> >> +    switch(msg)
> >> +    {
> >> +    case WM_COMMAND:
> >> +        switch(HIWORD(wparam))
> >> +        {
> >> +        case BN_CLICKED:
> >> +            switch(wparam)
> >> +            {
> >> +            case IDC_WHQL_CHECKBOX:
> >> +            {
> >> +                update_dxdiag_info();
> >> +                return TRUE;
> >> +            }
> >> +            }
> >> +            break;
> >
> > There's an indentation issue here.
> Is there? Looking at original main.c I see that "case" labels are also
> unindented there. Or do you mean something other?
>
Ah, my bad. I didn't really look at the whole file. If it's consistent with
the rest of the file then it's fine :)

Cheers,
Aaryaman

[Attachment #5 (text/html)]

<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Aug \
27, 2016 at 9:12 PM, Ruslan Kabatsayev <span dir="ltr">&lt;<a \
href="mailto:b7.10110111@gmail.com" \
target="_blank">b7.10110111@gmail.com</a>&gt;</span> wrote:<br><blockquote \
class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex"><span class="">On Sat, Aug 27, 2016 at 12:31 PM, Aaryaman \
Vasishta<br> &lt;<a href="mailto:jem456.vasishta@gmail.com">jem456.vasishta@gmail.com</a>&gt; \
wrote:<br> &gt;&gt;<br>
&gt;&gt; +INT_PTR CALLBACK system_tab_proc(HWND hsystab, UINT msg, WPARAM wparam,<br>
&gt;&gt; LPARAM lparam)<br>
&gt;&gt; +{<br>
&gt;&gt; +      switch(msg)<br>
&gt;&gt; +      {<br>
&gt;&gt; +      case WM_COMMAND:<br>
&gt;&gt; +            switch(HIWORD(wparam))<br>
&gt;&gt; +            {<br>
&gt;&gt; +            case BN_CLICKED:<br>
&gt;&gt; +                  switch(wparam)<br>
&gt;&gt; +                  {<br>
&gt;&gt; +                  case IDC_WHQL_CHECKBOX:<br>
&gt;&gt; +                  {<br>
&gt;&gt; +                        update_dxdiag_info();<br>
&gt;&gt; +                        return TRUE;<br>
&gt;&gt; +                  }<br>
&gt;&gt; +                  }<br>
&gt;&gt; +                  break;<br>
&gt;<br>
&gt; There&#39;s an indentation issue here.<br>
</span>Is there? Looking at original main.c I see that &quot;case&quot; labels are \
also<br> unindented there. Or do you mean something other?<br></blockquote><div>Ah, \
my bad. I didn&#39;t really look at the whole file. If it&#39;s consistent with the \
rest of the file then it&#39;s fine \
:)<br><br></div><div>Cheers,<br></div><div>Aaryaman<br></div><div><br>  \
<br></div></div><br></div></div>


[Attachment #6 (text/plain)]




[prev in list] [next in list] [prev in thread] [next in thread]