search

ArduPilot / ardupilot

ArduPlane, ArduCopter, ArduRover, ArduSub source
http://ardupilot.org/
GNU General Public License v3.0
10.96k stars 17.48k forks source link

Revisit if RC_Channel/RC_Channel_VarInfo.h should be a .cpp file #13361

Open rmackay9 opened 4 years ago

rmackay9 commented 4 years ago

It looks (to me at least) like RC_Channel's RC_Channel_VarInfo.h file is actually the implementation of the var table so I think it should be in a .cpp file.

I think the closest equivalent is how we've done the AP_RangeFinder/AP_RangeFinder_Params which includes both a .h and a .cpp file.

peterbarker commented 4 years ago

I'm thinking .inc if anything.