[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:                
-------------------------------+--------------------------------------------
Changes (by waltweltonlair):

  * status:  new => assigned
  * owner:  => waltweltonlair

Comment:

 Well I'm still awake, and I've put together an initial fix for this.  I've
 updated the BIDIConverter class to first check if the input string
 contains right-to-left characters, and only if it does will the conversion
 algorithm be called.

 Initial results for 100 iterations of the dense labeled map (test 2 in
 #217 discussion)

  * time without any BIDI checks: 58.8 seconds
  * time with unoptimized BIDI checks: 63.0 seconds
  * time with optimized BIDI checks: 59.0 seconds

 So with the optimized check it's now only a 0.3% performance impact for
 this map.

 I will do more testing tomorrow and then get this submitted.

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