Conversation
doc/sql.extensions/README.trim
Outdated
| <trim character> ::= | ||
| <value expression> | ||
|
|
||
| <multi-character trim function> ::= |
There was a problem hiding this comment.
отступ неправильный, как будто это продолжение обычной функции TRIM
doc/sql.extensions/README.trim
Outdated
| select | ||
| ltrim(rdb$relation_name, 'RDB$') | ||
| from rdb$relations | ||
| where rdb$relation_name starting with 'RDB$'; |
There was a problem hiding this comment.
Пример не самый удачный, по-моему.
Выглядит будто эта строка и будет отрезаться целиком. А не отдельные символы из неё.
src/dsql/parse.y
Outdated
| | WINDOW | ||
| | WITHOUT | ||
| | CALL // added in FB 6.0 | ||
| | BTRIM |
There was a problem hiding this comment.
Тут тоже с отступом что-то не то или это github так показывает?
There was a problem hiding this comment.
здесь отступ в коде корректный, наверное github что-то не так показывает
src/dsql/ExprNodes.cpp
Outdated
| break; | ||
| } | ||
| } | ||
| { |
There was a problem hiding this comment.
Не понял смысла изменений. Мы добавили ещё один блок, который никак ни на что не влияет?
There was a problem hiding this comment.
в коде у меня здесь все нормально, я не знаю почему так в коммите изменения были указаны(может когда я в самом начале праил код, то сначала удалил эту скобку, а потом заново поставил и она записалась в изменение).
There was a problem hiding this comment.
ну вот сейчас здесь все то же самое. блок никуда не делся
src/dsql/ExprNodes.cpp
Outdated
| while (offsetLead < valueCanonicalLen) | ||
| { | ||
| bool found = false; | ||
| for (int i = 0; i < charactersCanonicalLen; i+=4) |
There was a problem hiding this comment.
откуда здесь +=4? почему мы так перескакиваем по строке?
There was a problem hiding this comment.
как я понял здесь каждая буква записывается по 4 бита, поэтому и +4 так как если ставить меньше он несколько раз одну букву будит смотреть
There was a problem hiding this comment.
Во-первых, не бита, а байта. Во-вторых, длина символов не 4, а charactersCanonicalLen, как я понимаю из оригинального кода
There was a problem hiding this comment.
здесь все исправил теперь работает корректно
doc/sql.extensions/README.trim
Outdated
| where rdb$system_flag = 1; | ||
|
|
||
| C) | ||
|
|
There was a problem hiding this comment.
только здесь не понимаю, тут не нужна 42 пустая строка?
doc/sql.extensions/README.trim
Outdated
|
|
||
| C) | ||
|
|
||
| select |
doc/sql.extensions/README.trim
Outdated
| select | ||
| ltrim('baobab is a tree', 'aboe') | ||
| from rdb$relations; | ||
| is a tree |
There was a problem hiding this comment.
в других примерах нет результата, тут смотрится непонятно
functions to work with different encodings
3 participants
No description provided.