Anonymous | Login | Signup for a new account | 2021-01-26 14:43 UTC | ![]() |
Main | My View | View Issues | Roadmap | Repositories |
View Issue Details [ Jump to Notes ] | [ Print ] | ||||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||||
0000498 | Firmware | Structure / Style | public | 2012-01-04 13:00 | 2014-02-18 23:05 | ||||||
Reporter | Fred | ||||||||||
Assigned To | Fred | ||||||||||
Priority | normal | Severity | minor | Reproducibility | always | ||||||
Status | assigned | Resolution | open | ||||||||
Platform | OS | OS Version | |||||||||
Product Version | 0.2.0-SNAPSHOT | ||||||||||
Target Version | 0.2.1 | Fixed in Version | |||||||||
Summary | 0000498: Remove all CLEAR_ flags and replace with a macro that clears | ||||||||||
Description | Current stuff is error prone and bad for maintenance with duplicated flag defines everywhere, yuck. Probably move to something like this: SETBITS(variable, MASK); CLEARRBITS(variable, MASK); or ENABLEBITS(variable, MASK); DISABLEBITS(variable, MASK); or similar. | ||||||||||
Tags | No tags attached. | ||||||||||
FirmwareVersion | |||||||||||
Issue Type | Improvement | ||||||||||
Risk of Breakage | low | ||||||||||
Attached Files | |||||||||||
![]() |
|
![]() Fred (administrator) 2012-01-04 14:14 |
OK, to do this properly there need to be three variants of each, 8, 16, 32 bits. Without this the best you can hope for is single bit, and not safely. I'm hoping that it's possible to generate some sort of error if they are misused with the wrong type of input var. Quite possibly. |
Copyright © 2000 - 2011 MantisBT Group |