From 6580f37357e9c9c85404d81fb0fca8eb3e28b3c3 Mon Sep 17 00:00:00 2001 From: Jeffrey Altman Date: Tue, 9 Apr 2013 10:48:12 -0400 Subject: [PATCH] Windows: Comment CommonWrite VDL Change Add a comment reminding the reader that CcSetFileSizes only needs to be called on a ValidDataLength change if the VDL value has decreased. A write operation cannot result in a decrease therefore CcSetFileSizes does not need to be called from within AFSCommonWrite(). Change-Id: Iaf867ec876a6265dc2c8a7ba2319fdf67503a467 Reviewed-on: http://gerrit.openafs.org/9757 Reviewed-by: Rod Widdowson Tested-by: BuildBot Reviewed-by: Peter Scott Reviewed-by: Jeffrey Altman --- src/WINNT/afsrdr/kernel/lib/AFSWrite.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/WINNT/afsrdr/kernel/lib/AFSWrite.cpp b/src/WINNT/afsrdr/kernel/lib/AFSWrite.cpp index 744ef0aa0..2d5f44b70 100644 --- a/src/WINNT/afsrdr/kernel/lib/AFSWrite.cpp +++ b/src/WINNT/afsrdr/kernel/lib/AFSWrite.cpp @@ -709,7 +709,8 @@ try_exit: } // - // If this extended the Vdl, then update it accordingly + // If this extended the VDL, then update it accordingly. + // Increasing the VDL does not require a call to CcSetFileSizes. // if( liStartingByte.QuadPart + ulByteCount > pFcb->Header.ValidDataLength.QuadPart) -- 2.39.5