----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/795/#review1043 -----------------------------------------------------------
src/account/lock.c http://reviewboard-openlmi.rhcloud.com/r/795/#comment577
Whitespace be-gone.
src/account/lock.c http://reviewboard-openlmi.rhcloud.com/r/795/#comment574
There seems to be a missing piece of code here.
src/account/lock.c http://reviewboard-openlmi.rhcloud.com/r/795/#comment578
Code missing.
src/account/lock.c http://reviewboard-openlmi.rhcloud.com/r/795/#comment575
I wouldn't call releasing a lock 'unleashing'. Why not release_lock?
src/account/lock.c http://reviewboard-openlmi.rhcloud.com/r/795/#comment576
Whitespace be-gone.
Also, please use whitespaces consistently. That means write strcmp(something, here) instead of strcmp ( something, here ).
- Jan Synacek
On Sept. 2, 2013, 11:50 a.m., Robin Hack wrote:
This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/795/
(Updated Sept. 2, 2013, 11:50 a.m.)
Review request for OpenLMI Developers.
Repository: openlmi-providers
Description
Account: Added associative thread locking.
Missing files included.
Diffs
src/account/CMakeLists.txt 495d7c6e3e1e083541d3ed004518dc4539f61d48 src/account/LMI_AccountProvider.c 5abad7261331c429de0503b8bfb9f8fc180141cf src/account/lock.h PRE-CREATION src/account/lock.c PRE-CREATION
Diff: http://reviewboard-openlmi.rhcloud.com/r/795/diff/
Testing
Thanks,
Robin Hack