On July 1, 2013, 9:29 a.m., Roman Rakus wrote:
> src/hardware/dmidecode.c, line 1972
>
<
http://reviewboard-openlmi.rhcloud.com/r/473/diff/1/?file=2765#file2765li...
>
> What is this magic number? Better use some enum.
These numbers.. they don't matter. Only important thing is that they are negative. I
used different numbers in the beginning when I had to debug often, and I haven't
changed it since then. To "fix" this, changing all these numbers to the -1 would
be enough. Enum could work too, but that would require more work, which is currently not
worth it, especially for dmidecode.c file, as there are no plans for adding more
features.
So, for now, I'd just let it be. Later, I'll think about what to do with it
everywhere in the hardware provider.
- Peter
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-openlmi.rhcloud.com/r/473/#review570
-----------------------------------------------------------
On June 27, 2013, 11:58 a.m., Peter Schiffer wrote:
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-openlmi.rhcloud.com/r/473/
-----------------------------------------------------------
(Updated June 27, 2013, 11:58 a.m.)
Review request for OpenLMI Developers.
Repository: openlmi-providers
Description
-------
Hardware: Added battery providers
New providers:
* LMI_BatteryProvider
* LMI_BatteryPhysicalPackageProvider
* LMI_PhysicalBatteryContainerProvider
* LMI_PhysicalBatteryRealizesProvider
Diffs
-----
mof/60_LMI_Hardware.mof e22da2cb866c6a678efa1547ff896f29896dbc2e
src/hardware/LMI_BatteryPhysicalPackageProvider.c PRE-CREATION
src/hardware/LMI_BatteryProvider.c PRE-CREATION
src/hardware/LMI_Hardware.h f88e6b6bb5074cab3cc9f079898b86bb1f3a56cf
src/hardware/LMI_PhysicalBatteryContainerProvider.c PRE-CREATION
src/hardware/LMI_PhysicalBatteryRealizesProvider.c PRE-CREATION
src/hardware/LMI_PointingDeviceProvider.c f3c89d6284c6b3a33d2a8a7c0456731d3b86c04c
src/hardware/dmidecode.h 34bfd473b9939c3b73c3932f463288b7d85f4ea2
src/hardware/dmidecode.c d989a094e59ffa60db06846624aadc6560887520
Diff:
http://reviewboard-openlmi.rhcloud.com/r/473/diff/
Testing
-------
Thanks,
Peter Schiffer