View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0037903 | pas2js | rtl | public | 2020-10-09 20:58 | 2020-10-23 13:15 |
Reporter | henrique | Assigned To | Michael Van Canneyt | ||
Priority | normal | Severity | minor | Reproducibility | always |
Status | resolved | Resolution | fixed | ||
Platform | Pas2Js | OS | Windows | ||
Fixed in Version | trunk | ||||
Summary | 0037903: Long string in DFM | ||||
Description | When a very large string is filled within DFM, Delphi places the sign ("+") to split the string into smaller parts, but this is giving an invalid token error. I attached a correction, creating a new token type for the plus sign. | ||||
Tags | No tags attached. | ||||
Fixed in Revision | 901 | ||||
Attached Files |
|
|
Classes.patch (448 bytes)
diff -r packages/rtl/classes.pas (88ca854d) packages/rtl/classes.pas (Working Tree) 1252c1252,1253 < toColon // : --- > toColon, // : > toPlus // + 9846c9847,9848 < ':' --- > ':', > '+' 10253a10256 > '+' : SetToken(toPlus); 10382c10385 < while (parser.NextToken = classes.toString) and (Parser.TokenString='+') do --- > while parser.NextToken = toPlus do |
|
this will be applied? |
|
I'm not entirely satisfied with the new + token, since it introduces an incompatibility but I have no better alternative. Thank you for the patch ! |
Date Modified | Username | Field | Change |
---|---|---|---|
2020-10-09 20:58 | henrique | New Issue | |
2020-10-09 20:58 | henrique | File Added: Classes.patch | |
2020-10-14 13:09 | henrique | Note Added: 0126293 | |
2020-10-23 12:09 | Mattias Gaertner | Assigned To | => Michael Van Canneyt |
2020-10-23 12:09 | Mattias Gaertner | Status | new => assigned |
2020-10-23 12:35 | Michael Van Canneyt | Status | assigned => resolved |
2020-10-23 12:35 | Michael Van Canneyt | Resolution | open => fixed |
2020-10-23 12:35 | Michael Van Canneyt | Fixed in Version | => trunk |
2020-10-23 12:35 | Michael Van Canneyt | Fixed in Revision | => 901 |
2020-10-23 12:35 | Michael Van Canneyt | Note Added: 0126486 |