Martin Schwenke has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/536257 ) Change subject: FSAL_VFS: Put debug ACL code under a single #ifdef ...................................................................... FSAL_VFS: Put debug ACL code under a single #ifdef More code is coming here. Having more #ifdef's sprinkled through the function bodies is going to make this unreadable and unmaintainable. So, create a single section of code to be used when ENABLE_VFS_DEBUG_ACL is enabled. Factor out the bits that are reusable. Signed-off-by: Martin Schwenke <mschwenke(a)ddn.com&gt; Change-Id: Ic9044e66cd034e1aeb9ec8aea59e17fb89a45b63 --- M src/FSAL/FSAL_VFS/vfs/attrs.c 1 file changed, 79 insertions(+), 38 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/57/536257/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/536257 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: Ic9044e66cd034e1aeb9ec8aea59e17fb89a45b63 Gerrit-Change-Number: 536257 Gerrit-PatchSet: 1 Gerrit-Owner: Martin Schwenke <martin(a)meltin.net&gt; Gerrit-MessageType: newchange