Skip to content

Issue 698 mic inline delimiter logic can be optimized#800

Merged
Ducasse merged 2 commits intopillar-markup:devfrom
moufort:698-MicInlineDelimiter-logic-can-be-optimized
Jul 24, 2024
Merged

Issue 698 mic inline delimiter logic can be optimized#800
Ducasse merged 2 commits intopillar-markup:devfrom
moufort:698-MicInlineDelimiter-logic-can-be-optimized

Conversation

@moufort
Copy link
Contributor

@moufort moufort commented Jul 24, 2024

I use the bench.

Before the code do 'MicInlineDelimiter allRegex' 63 000 time in 5s, now he did it 75 000 time.

@Ducasse
Copy link
Contributor

Ducasse commented Jul 24, 2024

Super.
Je vais integrer et on refait une iteration.
je me dis que on pourrait meme stocker la regex
Car a la fin c'est toujours le meme les valeurs du dictionnaire ne change jamais.
Donc pourquoi reconstruire a chaque fois une nouvelle regex.

on pourrait stocker cela

(RegexDictionary values joinUsing: '|') asRegex

Autre point:
Je n'ai pas vu d'utilisation du dictionaire hormis pour l[expression au dessus
donc peut etre que l'on n'a pas besoin du dictionnaire.

@Ducasse Ducasse merged commit 4d01480 into pillar-markup:dev Jul 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants