Cleaned up CA exceptions. Integrated free-standing handle class.

installer/src/installer-lib/record.cpp

 /**
  * \file record.cpp Implementation of Record class.
  */
 
+#include "installer-lib.h"
 #include "record.h"
 #include "msiquery.h"
 
 //-----------------------------------------------------------------------------------------
 // Record
 //-----------------------------------------------------------------------------------------
 Record::Record( unsigned int n_fields )
 {
   _handle = MsiCreateRecord( n_fields ) ;
   if ( ! _handle )
   {
-    throw std::runtime_error( "Failed to create record" ) ;
+    throw windows_api_error( "MsiCreateRecord", 0 ) ;
   }
 }
 
 Record::~Record()
 {
   if ( _handle != 0 )
   {
     MsiCloseHandle( _handle ) ;
   }
 }
@@ skipping 17 matching lines @@
   only_non_null() ;
   MsiRecordSetStringW( _handle, field_index, value ) ;
 }
 
 /**
  * \par Implementation
  *    - MSDN [MsiRecordGetString](http://msdn.microsoft.com/en-us/library/aa370368%28v=vs.85%29.aspx)
  */
 std::wstring Record::value_string( unsigned int field_index )
 {
-  static wchar_t initial_buffer[ 256 ] = L"" ;
-  DWORD length = 255 ; // one less than the buffer length to hold a terminating null character
+  static wchar_t initial_buffer[ 1024 ] = L"" ;
+  DWORD length = 1023 ; // one less than the buffer length to hold a terminating null character
   UINT x = MsiRecordGetStringW( _handle, field_index, initial_buffer, & length ) ;
   if ( x == ERROR_SUCCESS )
   {
     return std::wstring( initial_buffer ) ;
   }
   if ( x == ERROR_MORE_DATA )
   {
     // Future: handle longer strings.
-    throw std::runtime_error( "Record strings longer than 255 not supported yet" ) ;
+    /*
+     * The present custom action only uses this function for strings that appear in dialog boxes.
+     * A thousand characters is about a dozen lines of text, which is far more than enough.
+     */
+    throw not_yet_supported( "retrieving string values longer than 1023 from a record" ) ;

[Oleksandr, 2014/03/31 07:35:49]

Still, why not just dynamically allocate the buffer here? Doesn't seem to
complicate things much.

[Eric, 2014/03/31 11:23:09]

It wouldn't. The main reason is that we have no need of it for this CA, and code
not written is code you don't need to test. If this were more of a general
publication library, it would be the right thing to do.

Like most get-string functions in Windows API, this one would require a second
call to MsiRecordGetStringW.

   }
-  throw std::runtime_error( "Error retrieving string from record" ) ;
+  throw windows_api_error( "MsiRecordGetStringW", x ) ;
 }
 
 size_t Record::n_fields() const
 {
    unsigned int x = MsiRecordGetFieldCount( _handle ) ;
    if ( x == 0xFFFFFFFF )
    {
-     throw std::runtime_error( "Invalid handle" ) ;
+     throw windows_api_error( "MsiRecordGetFieldCount", x, "invalid handle" ) ;
    }
    return x ;
 }