Project

General

Profile

User story #1527

LexicalAnalyzer: convert to string if token begins with numbers

Added by Giuseppe Ferro over 1 year ago. Updated 7 months ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
19.07.2023
Due date:
21.07.2023
% Done:

0%

Estimated time:
Git branch (link):
Git merge to develop (link):
SVN commit (link/?p=rev):

Description

This is the standard behaviour in C so we have discussed on the Tuedays' meetings to keep the current behaviour, unless there is a very strong use-case not do it.

Non ricordo esattamente l’use case, peró quasi sicuramente é legato alla mia re-implementazione di sdd. Siccome il LexicalAnalyzer é utilizzato da tutti i parser dobbiamo supportare anche ad esempio xml e json.
Se non applichiamo il cambio i seguenti casi molto probabilmente danno errore:

<A>1234ciao</A> { “A”:”1234a” }

Inoltre non capisco perché dobbiamo utilizzare le stesse regole del C, che tra l’altro possiamo bypassare (e lo facciamo nel caso del configuration database compilato) facendo “1234ciao”=xxx

History

#1 Updated by Giuseppe Ferro over 1 year ago

  • Assignee set to Giuseppe Ferro
  • Due date set to 21.07.2023

Currently something a token like 1234a is returning ERROR_TOKEN.
The proposed change is to convert to STRING_TOKEN anyway

#2 Updated by Giuseppe Ferro over 1 year ago

  • Status changed from New to Code: Impl

#3 Updated by Giuseppe Ferro over 1 year ago

  • Status changed from Code: Impl to Unit: Rev

#4 Updated by Giuseppe Ferro over 1 year ago

  • Assignee changed from Giuseppe Ferro to André Neto

Changes on #1527_LexAn_StringConversion

#5 Updated by André Neto 7 months ago

  • Assignee deleted (André Neto)
  • Status changed from Unit: Rev to Closed
  • Description updated (diff)

#6 Updated by André Neto 7 months ago

  • Assignee set to André Neto
  • Status changed from Closed to Unit: Rev
  • Description updated (diff)

#7 Updated by André Neto 7 months ago

  • Assignee deleted (André Neto)
  • Status changed from Unit: Rev to Closed

Merged in 275ca94acaa657f159ad3c2c99c1d4a04a6e3e4a

Also available in: Atom PDF