[Nut-upsdev] Patching CyberPower UPS drivers
Ben Kamen
ben at benkamen.net
Thu Jan 28 17:03:22 UTC 2016
On 2016-01-28 8:18 AM, Charles Lepple wrote:
> I definitely like your approach. Thanks for moving some of the "magic numbers" to named constants, as well.
Thanks. Always nice to have real coders sanity check anything I've written with a "thumbs up" response. (I'm more a hardware guy)
And sscanf was working like poo -- so I figured I'd parse it differently with something that didn't seem so sensitive.
> A few items:
>
> * Since v2.6.5, some of the string functions were renamed. rtrim() -> str_rtrim(), for instance.
I'll take a look at the latest.... as mentioned, 2.6.5 is what's on my CentOS system with nothing later (yet) so I just downloaded the SRPMS to work on.
but I also have 2.7.3 downloaded -- so I'll see what I can do with that.
> * If you are taking out a block of code, "#if 0" is better than comment characters (which don't nest), and removing it completely is even better (fewer false positives when searching). If nothing else, it's in version control.
Yea - the comments were just temporary to keep the old stuff in my view while I was re-writing around it.
Then once it worked and I planned on removing it.
I'll send along something more finished once I'm done.
Thanks again!
-Ben
More information about the Nut-upsdev
mailing list