close Warning: Can't synchronize with repository "(default)" (/var/svn/tolp does not appear to be a Subversion repository.). Look in the Trac log for more information.

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#1476 closed defect (fixed)

Implementación de TextMatch

Reported by: Humberto Carralero Owned by: Jorge
Priority: highest Milestone: Mantainance
Component: Text Version: 3.1
Severity: blocker Keywords:
Cc:

Description

Cuando se usa TOLMachine (componente de TOLGrid que encapsula el intérprete de TOL) se genera la advertencia: "Using internal version of TextMatch could cause invalid results. Please use a TCL version of TOL like tolsh or tolbase.", la cual no es viable en este caso.
Del mensaje se infiere que la implementación de TextMatch que se ha utilizado en este caso podría no dar resultados correctos, o sea, su implementación no es correcta y por eso se ha optado por una nueva implementación que usa TCL.
No creo que haya ningún inconveniente en hacer una implementación correcta de TextMatch usando una de las muchísimas bibliotecas sobre expresiones regulares disponibles; por ejemplo, ya que en otras partes de TOL se usan bibliotecas de Boost, podría usarse "Boost.Regex". Con esta nueva implementación se evitaría crear una dependencia innecesaria de TCL para este caso.
Otro aspecto preocupante del warning es que si el resultado que se está obteniendo no es correcto, entonces podemos asegurar que algunos resultados que muestra BWS no son correctos (por ejemplo, los informes de DECO y DUETO) ya que utilizan en su código llamadas a TextMatch.
La versión de TOL usada en BWS es la 2.01 que no será actualizada hasta que se hagan todos los Test de funcionalidad requeridos para garantizar que todo funcione bien en TOLGrid y BWS.

Change History (4)

comment:1 Changed 13 years ago by Víctor de Buen Remiro

Component: KernelText
Owner: changed from Víctor de Buen Remiro to Jorge
Status: newassigned
Version: 2.0.13.1

Reasigno a Jorge porque yo no tengo ni idea de esas cosas

comment:2 Changed 13 years ago by Jorge

(In [4326]) refs #1460, #1476, se implementa TextMatch en C (tomado de la implementación interna de tcl), ya no se recurre a Tcl_Eval, el argumento pattern se pasa por referencia evitando una copia temporal, se eliminan algunos warnings de compilación

comment:3 Changed 13 years ago by Jorge

Resolution: fixed
Status: assignedclosed

La próxima versión de TOL incorpora un TextMatch implementado en C que corrige los errores antes asociados a TextMatch.

comment:4 Changed 13 years ago by Jorge

(In [4394]) refs #1476, portamos la solución que se subió en [4326]

Note: See TracTickets for help on using tickets.