Re: [Boost-bugs] [Boost C++ Libraries] #5266: mapped_region::flush improvement

Subject: Re: [Boost-bugs] [Boost C++ Libraries] #5266: mapped_region::flush improvement
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2012-01-09 19:33:45

#5266: mapped_region::flush improvement
--------------------------------------+-------------------------------------
  Reporter:  dmitry_b@…               |       Owner:  igaztanaga  
      Type:  Feature Requests         |      Status:  reopened    
 Milestone:  To Be Determined         |   Component:  interprocess
   Version:  Boost Development Trunk  |    Severity:  Problem     
Resolution:                           |    Keywords:              
--------------------------------------+-------------------------------------
Changes (by dan.eloff@…):

  * status:  closed => reopened
  * version:  Boost 1.46.0 => Boost Development Trunk
  * resolution:  wontfix =>

Comment:

 How about an overload:

 mapped_region::flush(std::size_t mapping_offset, std::size_t numbytes,
 bool sync)

 which uses msycn(MS_SYNC) on posix systems and FlushViewOfFile +
 FlushFileBuffers on windows

 If there are platforms where it cannot be supported, flush with sync=true
 should throw an exception.

 This removes the burden of writing a multiple platform flush(...,
 sync=true) from the user, which this library does a great job of for other
 memory mapped file operations. That should cover all the common use cases.

-- 
Ticket URL: <https://svn.boost.org/trac/boost/ticket/5266#comment:2>
Boost C++ Libraries <http://www.boost.org/>
Boost provides free peer-reviewed portable C++ source libraries.