Comment 4 for bug 679919

Revision history for this message
rew (r-e-wolff) wrote : Re: [Bug 679919] Re: Swap images in controlpoints window.

Hi,

Your patch contains sometimes differences to the original in tabs/spaces.

     XRCCTRL(*this, "cp_editor_celeste_button", wxButton)->Disable();
     XRCCTRL(*this, "cp_editor_choice_zoom", wxChoice)->Disable();
     XRCCTRL(*this, "cp_editor_previous_img", wxButton)->Disable();
+ XRCCTRL(*this, "cp_editor_swap_img", wxButton)->Disable();
     XRCCTRL(*this, "cp_editor_next_img", wxButton)->Disable();

You probably used a tab, while the original uses spaces (and less than
8). To get your code accepted please go through your patch and copy
the tabs/spaces from the lines above or below the line you patched.

Thanks! Looks good!

 Roger.

--
** <email address hidden> ** http://www.BitWizard.nl/ ** +31-15-2600998 **
** Delftechpark 26 2628 XH Delft, The Netherlands. KVK: 27239233 **
*-- BitWizard writes Linux device drivers for any device you may have! --*
Q: It doesn't work. A: Look buddy, doesn't work is an ambiguous statement.
Does it sit on the couch all day? Is it unemployed? Please be specific!
Define 'it' and what it isn't doing. --------- Adapted from lxrbot FAQ