| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
|
|
|
| |
#### Enable misc-unused-using-decls check ✎
- Enabled the `misc-unused-using-decls` clang-tidy check to enforce removal of unused using-declarations
- Removed several unused using-declarations across multiple files to comply with the new check
- Updated clang-tidy configuration to activate the check instead of suppressing it
<a href="https://nda.ya.ru/t/qa0kX64r7DqvtN"><font size="2">Autodescription by Yandex Code Assistant</font></a>
commit_hash:5dd7bcf766b71f2041902fbf314e13f2b81d9a4d
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
https://clang.llvm.org/extra/clang-tidy/checks/performance/move-const-arg.html
This check useful, as it shows, when a `std::move` is useless, so
to expectations of a programmer are not aligned with reality. The
most useful profit is to see `const T&` arguments, that are
potentially can be replaced with `T`.
Also disabled `CheckTriviallyCopyableMove`,
see https://nda.ya.ru/t/Emvgx5Z67fpZY5 why.
commit_hash:c80c264a9774721aabbac85b350ccdbb8b9a39a2
|
| |
|
|
| |
commit_hash:41791bb9a63f1993010fb68562e9a5607927307d
|
| |
|
|
|
| |
в основном они протекают после включения contrib/ydb/core/protos (что на самом деле не публичная часть GRPC сервиса)
commit_hash:1fec0018e2f898417521c5b0b39d6ba64fb97c80
|
| |
|
|
| |
commit_hash:9589937209af8e742c0c6401bd1c232f073856bf
|
| |
|
|
| |
commit_hash:153e546ce3d52153d488b9b36d63c3b44f5a959f
|
| |
|
|
| |
commit_hash:c180c2db4897962cb70063ffdc2ddcd21e5e3418
|
| |
|
|
| |
commit_hash:464a95fd6de1a395f8ffe20092e837df995e623c
|
| |
|
|
|
| |
https://clang.llvm.org/extra/clang-tidy/checks/modernize/use-designated-initializers.html
commit_hash:d73eb463c7bec2bda4c362aab10af49979a7ddd3
|
| |
|
|
|
| |
https://clang.llvm.org/extra/clang-tidy/checks/modernize/raw-string-literal.html
commit_hash:80a511a46b35539fa1f834d4fb5de452ccfbe0ac
|
| |
|
|
| |
commit_hash:a504ad9d540aa714c550320e0ce20f9eb8bd17cf
|
| |
|
|
| |
commit_hash:c1acbb89857acf40764b131b015b4be9bafd0aee
|
| |
|
|
|
| |
- https://clang.llvm.org/extra/clang-tidy/checks/misc/use-anonymous-namespace.html
commit_hash:746be9b9db256111b2fb6ab4d89844746deebcda
|
| |
|
|
| |
commit_hash:4d77ad10fd4db303459ec4e45e139967c7fc8196
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The check `modernize-use-override` is useful as it:
1. Removes redundant `override` when it is used with a `final`.
2. Removes redundant `virtual` or replace it with a `override`.
So it is more clean for a reader, that overriding happens.
It is enabled not out of order, as it is just an alias to
`cppcoreguidelines-explicit-virtual-functions`.
I also decided to switch a strategy of enabling checks. Now
I will enable only a single rule with a single PR and prefer
rules with a non-breaking autofix.
In the new year with new linter checks! 🎄
commit_hash:e6e233baa90b31e5f65e11837546690c47f71ab5
|
| |
|
|
|
| |
Then I will remove an `Antlr4Parser` usages from the YDB.
commit_hash:0183482e46de023f71698e03db9a0c51ef10fe98
|
| |
|
|
|
|
|
|
|
| |
This patch introduces an extension configuration for the Clang Tidy.
It is merged with Arcadia Clang Tidy configuration. To begin with,
more checks for identifiers naming are added.
Documentation: https://nda.ya.ru/t/AhbDZbiF7MKe3M.
commit_hash:3481da4c8df0a4d23a991d4a660ae050d2dc5d33
|
| |
|
|
| |
commit_hash:6c5c638d87eb41018d637f1535af104fac16e51c
|
| |
|
|
| |
commit_hash:a6b640bc576263b62884a0b4187ba79c893e13d9
|
| |
|
|
| |
commit_hash:edf72fd8386c41f858f677152612bd58680506a6
|
| |
|
|
| |
commit_hash:55bc611cdaa0d8a0fc3c4c7708ed9f17cc4976cf
|
| |
|
|
|
|
|
| |
Monarch regex engine is so slow on regexes like
`([A-Z]|[a-z]|_|[0-9])*`, but works well on
`([A-Za-z_0-9])*`. So such optimization is added.
commit_hash:b749e59818a20d7fd1d1ba21b233e1467e9d6ced
|
| |
|
|
| |
commit_hash:b3270397329599800d4e7b1f92b8e0f18e94cfd6
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Support `Before` at core `TRegexPattern` and `IGenericLexer`.
- Added `Name` and `Extension` to core `THighlighting`.
- Added `Tighlighting` for `YQLs` factory method.
- Added `--language` option to `yql_highlight`.
- Added `artifact` targets for `YQLs`.
Yes, using the `NSQLTranslation::THighlighting` for
`YQLs` is not correct, but much simplier than generalize
this infrastructure just for a `YQLs`. So here is a
trade-off between development time and a clean code.
Results:
- JetBrains: https://nda.ya.ru/t/PXkZVE8m7H5wHS.
- Vim: https://nda.ya.ru/t/Am-6ZHQa7H5wJi.
- TextMate: https://nda.ya.ru/t/wH0YggAf7H5wKw.
- yql_highlight: https://nda.ya.ru/t/3FaCm57q7H7QSF.
commit_hash:f0e1abb8e7f1b083df531d761b357330bd514cb0
|
| |
|
|
|
|
|
|
|
|
| |
Introduced a Vim syntax highlighting for YQL.
This is a replacement for an existing almost
manually written conguration. It uses regexes
generated from the original ANTLR4 grammar.
Now only Default lexer mode is supported.
commit_hash:85fa094593bd9d80373754a492b46ede1a50148d
|
| |
|
|
| |
commit_hash:b06f3d2f5fa81e2073966eb172758758bd94fdf2
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fix `TRegexLexer` performance. Now it is just 2 times slower than a reference ANTLR implementation on Release mode, so merged regexes are 3 times better than scan&compare.

---
- Related to `YQL-19616`
- Related to https://github.com/ydb-platform/ydb/issues/15129
- Related to https://github.com/vityaman/ydb/issues/42
---
Pull Request resolved: https://github.com/ytsaurus/ytsaurus/pull/1278
commit_hash:1529f641172fea13f0d33fbfd06a4827c6efde01
|
| |
|
|
| |
commit_hash:3a624a323006078de71f50747f7b2e8cadba7ccd
|
| |
|
|
| |
commit_hash:fb1727dda2b8c7d2ff42d4436c54cb7aa1ce4bc2
|
| |
|
|
| |
commit_hash:b6187f8eba6e8debc23f1928b2e44a396f3511ad
|
| |
|
|
|
|
|
|
|
|
| |
- Related to https://github.com/ydb-platform/ydb/issues/15129
- Related to https://github.com/vityaman/ydb/issues/11
---
Pull Request resolved: https://github.com/ytsaurus/ytsaurus/pull/1201
commit_hash:53ef677a35649a6dc77d8c4269a8aceefcd15026
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fixed regex lexer issues:
- `TSKIP` token recognition
- `HEXGIGITS` number recognition
- `EOF` token content
---
- Related to https://github.com/ydb-platform/ydb/issues/15129
- Related to https://github.com/vityaman/ydb/issues/11
---
Pull Request resolved: https://github.com/ytsaurus/ytsaurus/pull/1190
commit_hash:497c39efcbbe4e387da523b5e2c8abaa6485d93b
|
| |
|
|
| |
commit_hash:6768768ea3a3962231d3fabdffb2ce0db44e9347
|
| |
|
|
| |
commit_hash:28e9e4bd6b02e8914d82b2aafe9f341b5492421f
|
| |
|
|
| |
commit_hash:e57b3e95787cc8037f200f1b6b6073e35403b27e
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- [x] Parse YQL grammar to extract lexer grammar into `TLexerGrammar`.
- [x] Translate `TLexerGrammar` into regexes.
- [x] Implement a lexer via regexes `TRegexLexer` to test generated regexes validity.
- [x] Test on `Default` syntax mode.
- [x] Test on `ANSI` syntax mode.
---
- Related to https://github.com/ydb-platform/ydb/issues/15129
- Requirement for https://github.com/ytsaurus/ytsaurus/pull/1112
---
Pull Request resolved: https://github.com/ytsaurus/ytsaurus/pull/1127
commit_hash:03ffffe81cdafe7f93a4d3fd9a3212fe67f1c72d
|
| |
|
|
|
|
|
|
|
|
|
|
| |
When we run completion engine on multi-statement query, where preceding statements are syntactically incorrect, `antlr4-c3` does not return candidates. Running engine only on a current statement provides a best-effort attempt to provide candidates.
- Related to https://github.com/ydb-platform/ydb/issues/9056
- Depends on https://github.com/ytsaurus/ytsaurus/pull/1127 (`ELexerFlavor`)
---
Pull Request resolved: https://github.com/ytsaurus/ytsaurus/pull/1144
commit_hash:0ced9443a9712191f5420246531f781ca4bc5f42
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
- [x] Added `antlr_ast/antlr4` module and moved `TLexerTokensCollector4` there from `proto_ast/antlr4`.
- [x] Moved stuff around back and forth.
Ready for a review.
---
Co-authored-by: vityaman [[email protected]]
Pull Request resolved: https://github.com/ytsaurus/ytsaurus/pull/1128
commit_hash:e08785c3408ef813505bdc7511560e9536f4ab79
|
| |
|
|
|
| |
refine Antlr deps
commit_hash:71b99424edb283bf56be6e79c9c9a19d805ce2ce
|
| |
|
|
| |
commit_hash:2aa733486a55a04c39ff8bb1c7eaca3a85719647
|
| |
|
|
| |
commit_hash:6be543b7c5bff6ee474ee606c920197fb2569767
|
| |
|
|
| |
commit_hash:fee365c90a176dd33a967cee20994b21d530080c
|
| |
|
|
|
| |
init
commit_hash:a21a71769ad2095e40909c69255a3cf38eabc179
|
| |
|
|
| |
commit_hash:70690bcdabf0bcfe095fb10e9e58717281d11d53
|
| |
|
|
| |
commit_hash:09502f46a7ee665609d2c4ba8d9e0aa421720cdb
|
| |
|
|
| |
commit_hash:d780798556aedbe2be898d69185380f2ecb95f9c
|
|
|
init
commit_hash:7d4c435602078407bbf20dd3c32f9c90d2bbcbc0
|