kotatogram/lib_ui
1
0
Fork 0
Code Issues Activity

Slightly optimized spoilers to avoid recreating of same click handlers.

Browse source
This commit is contained in:
23rd 2022-01-08 08:20:07 +03:00
parent b1f71ccab3
commit 20b908ea70
1 changed files with 1 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
ui/text/text.cpp
View file

@ -250,7 +250,6 @@ private:
uint16 _maxLnkIndex = 0;
uint16 _maxShiftedLnkIndex = 0;
uint16 _maxSpoilerIndex = 0;
// current state
int32 _flags = 0;
@ -354,9 +353,6 @@ void Parser::createBlock(int32 skipBack) {
uint16(_lnkIndex - kStringLinkIndexShift),
_maxShiftedLnkIndex);
}
if (_spoilerIndex > _maxSpoilerIndex) {
_maxSpoilerIndex = _spoilerIndex;
}
int32 len = int32(_t->_text.size()) + skipBack - _blockStart;
if (len > 0) {
@ -728,7 +724,6 @@ void Parser::trimSourceRange() {
void Parser::finalize(const TextParseOptions &options) {
_t->_links.resize(_maxLnkIndex + _maxShiftedLnkIndex);
_t->_spoilers.resize(_maxSpoilerIndex);
auto monoLnk = uint16(1);
struct {
uint16 mono = 0;
@ -736,7 +731,7 @@ void Parser::finalize(const TextParseOptions &options) {
} lastHandlerIndex;
for (auto &block : _t->_blocks) {
const auto spoilerIndex = block->spoilerIndex();
if (spoilerIndex) {
if (spoilerIndex && (_t->_spoilers.size() < spoilerIndex)) {
_t->_spoilers.resize(spoilerIndex);
const auto handler = (options.flags & TextParseLinks)
? std::make_shared<SpoilerClickHandler>()