[mapguide-trac] #1013: Make BIDI an option via serverconfig.ini

[MapGuide Open Source]

#1013: Make BIDI an option via serverconfig.ini
-------------------------------+--------------------------------------------
 Reporter:  zspitzer           |         Owner:  waltweltonlair
     Type:  enhancement        |        Status:  assigned      
 Priority:  low                |     Milestone:  2.1           
Component:  Rendering Service  |       Version:  2.1.0         
 Severity:  trivial            |    Resolution:                
 Keywords:                     |   External_id:                
-------------------------------+--------------------------------------------
Comment (by waltweltonlair):

 Fix #1013 (Make BIDI an option via serverconfig.ini)

 The motivation for the ticket was to give users the option of avoiding the
 performance impact of BIDI conversion if they know none of their maps need
 it.

 I solved this by adding a check to the converter which determines if
 conversion should be performed.  It simply checks if any character in the
 string is outside the range [0..255].  A more thorough check could be
 implemented, but the simple check is sufficient for the vast majority of
 cases (and it's faster).  If no characters are outside the range then the
 conversion method simply returns the passed-in string.

 With this update the performance impact of the fix for #271 on maps
 containing English-only labels is negligible.  I reran the test where I
 did 100 iterations of the dense labeled map (test 2 in #271 discussion)
 and could not measure a statistically significant performance change.

-- 
Ticket URL: <https://trac.osgeo.org/mapguide/ticket/1013#comment:4>
MapGuide Open Source <http://mapguide.osgeo.org/>
MapGuide Open Source Internals