Fix minor errors in internal docs

This commit is contained in:
mandreyel 2019-03-12 22:58:51 +01:00
parent cafa31360f
commit 18fbf199ec
2 changed files with 13 additions and 10 deletions

View File

@ -400,9 +400,10 @@ void basic_mmap<AccessMode, ByteT>::unmap()
if(data_) { ::munmap(const_cast<pointer>(get_mapping_start()), mapped_length_); } if(data_) { ::munmap(const_cast<pointer>(get_mapping_start()), mapped_length_); }
#endif #endif
// If file_handle_ was obtained by our opening it (when map is called with a path, // If `file_handle_` was obtained by our opening it (when map is called with
// rather than an existing file handle), we need to close it, otherwise it must not // a path, rather than an existing file handle), we need to close it,
// be closed as it may still be used outside this instance. // otherwise it must not be closed as it may still be used outside this
// instance.
if(is_handle_internal_) if(is_handle_internal_)
{ {
#ifdef _WIN32 #ifdef _WIN32

View File

@ -76,15 +76,17 @@ private:
// Points to the first requested byte, and not to the actual start of the mapping. // Points to the first requested byte, and not to the actual start of the mapping.
pointer data_ = nullptr; pointer data_ = nullptr;
// Length, in bytes, requested by user, which may not be the length of the full // Length--in bytes--requested by user (which may not be the length of the
// mapping, and the entire length of the full mapping. // full mapping) and the length of the full mapping.
size_type length_ = 0; size_type length_ = 0;
size_type mapped_length_ = 0; size_type mapped_length_ = 0;
// Letting user map a file using both an existing file handle and a path introcudes // Letting user map a file using both an existing file handle and a path
// On POSIX, we only need a file handle to create a mapping, while on Windows // introcudes some complexity (see `is_handle_internal_`).
// systems the file handle is necessary to retrieve a file mapping handle, but any // On POSIX, we only need a file handle to create a mapping, while on
// subsequent operations on the mapped region must be done through the latter. // Windows systems the file handle is necessary to retrieve a file mapping
// handle, but any subsequent operations on the mapped region must be done
// through the latter.
handle_type file_handle_ = INVALID_HANDLE_VALUE; handle_type file_handle_ = INVALID_HANDLE_VALUE;
#ifdef _WIN32 #ifdef _WIN32
handle_type file_mapping_handle_ = INVALID_HANDLE_VALUE; handle_type file_mapping_handle_ = INVALID_HANDLE_VALUE;
@ -94,7 +96,7 @@ private:
// introcudes some complexity in that we must not close the file handle if // introcudes some complexity in that we must not close the file handle if
// user provided it, but we must close it if we obtained it using the // user provided it, but we must close it if we obtained it using the
// provided path. For this reason, this flag is used to determine when to // provided path. For this reason, this flag is used to determine when to
// close file_handle_. // close `file_handle_`.
bool is_handle_internal_; bool is_handle_internal_;
public: public: