Ticket #32 (closed defect: fixed)

Opened 6 years ago

Last modified 6 years ago

Error activating classic buttons

Reported by: work.eric@… Owned by: dsmith
Priority: critical Milestone: Version 0.6.00
Component: wminput Version: 0.5.02
Keywords: configuration Cc:

Description

This is likely a copy and paste error in root/trunk/wminput/conf.c. I'm surprised no one reported this before since you can't use L, R, X, and Y without fixing it. It's a simple one line fix. I didn't think a patch would be necessary for something so simple.

121 case CONF_CC: 122 conf->rpt_mode_flags |= CWIID_RPT_CLASSIC; 123 - conf->nunchuk_bmap[button].active = 1; 123 + conf->classic_bmap[button].active = 1; 124 conf->classic_bmap[button].action = action; 125 break; 126 }

Change History

in reply to: ↑ description   Changed 6 years ago by anonymous

Replying to work.eric@gmail.com:

This is likely a copy and paste error in root/trunk/wminput/conf.c. I'm surprised no one reported this before since you can't use L, R, X, and Y without fixing it. It's a simple one line fix. I didn't think a patch would be necessary for something so simple. 121 case CONF_CC: 122 conf->rpt_mode_flags |= CWIID_RPT_CLASSIC; 123 - conf->nunchuk_bmap[button].active = 1; 123 + conf->classic_bmap[button].active = 1; 124 conf->classic_bmap[button].action = action; 125 break; 126 }

It looks like the formatting got messed up.

File: root/trunk/wminput/conf.c
Line: 123
Old: conf->nunchuk_bmap[button].active = 1;
New: conf->classic_bmap[button].active = 1;

  Changed 6 years ago by dsmith

  • status changed from new to closed
  • resolution set to fixed

done: r91 good call - thanks

Note: See TracTickets for help on using tickets.