windows: Fix a few things clang complains about.

base/file_util_win.cc

Index: base/file_util_win.cc
diff --git a/base/file_util_win.cc b/base/file_util_win.cc
index d1337a29184f944bcd676321260ed6ad2763a8b6..eb72afbea4dd17355f8e1c0ed896e90029c7d223 100644
--- a/base/file_util_win.cc
+++ b/base/file_util_win.cc
@@ -988,8 +988,9 @@ void MemoryMappedFile::CloseHandles() {
 bool HasFileBeenModifiedSince(const FileEnumerator::FindInfo& find_info,
                               const base::Time& cutoff_time) {
   base::ThreadRestrictions::AssertIOAllowed();
+  FILETIME file_time = cutoff_time.ToFileTime();
   long result = CompareFileTime(&find_info.ftLastWriteTime,  // NOLINT

[Evan Martin, 2012/01/23 18:19:49]

Do you think the NOLINT bit here is related to your change?

[Nico, 2012/01/23 18:36:03]

The CompareFileTime() call further up (line 67) in this file also has a NOLINT
annotation (and doesn't take the address of a temporary), so I thought it's not
related. Then again, maybe that line up there was copypasted from here.

Do we use any tools that look at this NOLINT these days?

-                                &cutoff_time.ToFileTime());
+                                &file_time);
   return result == 1 || result == 0;
 }