[Boost-bugs] [Boost C++ Libraries] #7788: locale_data::parse could be simplified

Subject: [Boost-bugs] [Boost C++ Libraries] #7788: locale_data::parse could be simplified
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2012-12-11 16:07:27

#7788: locale_data::parse could be simplified
-------------------------------------+--------------------------------------
 Reporter:  166291@…                 |       Owner:  artyom      
     Type:  Patches                  |      Status:  new         
Milestone:  To Be Determined         |   Component:  locale      
  Version:  Boost Development Trunk  |    Severity:  Optimization
 Keywords:                           |  
-------------------------------------+--------------------------------------
 (If there's another way to submit patches, please do tell. :))

 So I've written this patch that simplifies locale_data::parse greatly.

 Pitfalls:

 '''It does allow invalid locale names''' like (en.UTF-8_at_eruo_US) to be
 parsed in to its correct components, but since it's an internal API that
 doesn't seem to matter.

 It's now case-sensitive. '''Which required a change in the unit tests to
 pass.''' It'd be trivial to fix this (one line of code when appending the
 character), but I personally desire this behaviour as it means I can get
 out what I put in to Boost.Locale via my code. It doesn't seem to break
 anything and the unit tests and/or documentation for the most part don't
 indicate it's important.

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