[LU-4739] Code inconsistencies when referring to config_llog_instance Created: 08/Mar/14 Updated: 13/Jul/16 Resolved: 13/Jul/16 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.9.0 |
| Type: | Bug | Priority: | Minor |
| Reporter: | Bobbie Lind (Inactive) | Assignee: | Nathaniel Clark |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | patch | ||
| Severity: | 3 |
| Rank (Obsolete): | 13021 |
| Description |
|
While trying to search for other similar occurrences of an issue I was working to keep consistent with current coding conventions I found that there are two instances where the local variable pointer name of the config_llog_instance was inconsistent with the way it is done in other areas of the code. Most of the files do the following: However, obd_config.c and obd_mount_server.c both have functions that use the following method: |
| Comments |
| Comment by Andreas Dilger [ 14/Mar/14 ] |
|
Bobbi, it would be great if you could submit a patch to clean this up, along with fixing up the coding style in the affected functions (so checkpatch.pl passes cleanly). |
| Comment by Bobbie Lind (Inactive) [ 22/May/14 ] |
|
Patch submitted. http://review.whamcloud.com/10420 |
| Comment by Gerrit Updater [ 11/Jul/16 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/10420/ |
| Comment by Joseph Gmitter (Inactive) [ 13/Jul/16 ] |
|
Patch landed to master for 2.9.0 |