From: Aaryaman Vasishta Subject: Re: Add a GUI for dxdiag program Message-Id: Date: Sat, 27 Aug 2016 21:43:29 +0530 In-Reply-To: References: <57c1550a.11092e0a.17169.26e8@mx.google.com> On Sat, Aug 27, 2016 at 9:12 PM, Ruslan Kabatsayev wrote: > On Sat, Aug 27, 2016 at 12:31 PM, Aaryaman Vasishta > 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


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