Regkey functions return error code instead of bool

chrome/installer/util/delete_reg_value_work_item.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_INSTALLER_UTIL_DELETE_REG_VALUE_WORK_ITEM_H_
 #define CHROME_INSTALLER_UTIL_DELETE_REG_VALUE_WORK_ITEM_H_
 #pragma once
 
 #include <windows.h>
 
@@ skipping 22 matching lines @@
     VALUE_DELETED,
     // One possible outcome after Do(). Value is not found.
     VALUE_NOT_FOUND,
     // The status after Do() and Rollback() is called.
     VALUE_ROLLED_BACK,
     // Another possible outcome after Do() (when there is an error).
     VALUE_UNCHANGED
   };
 
   DeleteRegValueWorkItem(HKEY predefined_root, const std::wstring& key_path,
-                         const std::wstring& value_name, DWORD type);
+                         const std::wstring& value_name);
 
   // Root key of the target key under which the value is set. The root key can
   // only be one of the predefined keys on Windows.
   HKEY predefined_root_;
 
   // Path of the target key under which the value is set.
   std::wstring key_path_;
 
   // Name of the value to be set.
   std::wstring value_name_;
 
-  // DWORD that tells whether data value is of type REG_SZ, REG_DWORD, or
-  // REG_QWORD
-  // Ideally we do not need this information from user of this class and can
-  // check the registry for the type. But to simpify implementation we are
-  // going to put the burden on the caller for now to provide us the type.
-  DWORD type_;
-
   DeletionStatus status_;
 
-  // Data of the previous value.
-  std::wstring old_str_;  // if data is of type REG_SZ
-  DWORD old_dw_;  // if data is of type REG_DWORD
-  int64 old_qword_;  // if data is of type REG_QWORD
+  // Previous value.
+  DWORD previous_type_;
+  std::string previous_value_;

[grt (UTC plus 2), 2011/01/14 15:53:43]

It looks like you're using a std::string as a bucket of bytes.  Why not use an
actual bucket of bytes (std::vector<uint8>)?

[amit, 2011/01/15 01:28:11]

Yep, back to vector :)
I started with std::vector then thought using WriteInto would be cool with
std::string so switched to it. Now discovered that WriteInto is not going to
work for non-strings.

 };
 
 #endif  // CHROME_INSTALLER_UTIL_DELETE_REG_VALUE_WORK_ITEM_H_