Skip to content

Commit

Permalink
Perhaps sensible thresholds?
Browse files Browse the repository at this point in the history
  • Loading branch information
katef committed Jul 25, 2020
1 parent b823bf9 commit 5778d25
Showing 1 changed file with 7 additions and 3 deletions.
10 changes: 7 additions & 3 deletions src/libre/ast_compile.c
Original file line number Diff line number Diff line change
Expand Up @@ -33,10 +33,14 @@
* The parameters here bail out if the number of alts or the length of any
* particular string within an alt are below a threshold, especially because
* [xyz] is so common. The exact values only represent cost approximately.
* These are chosen just by trying it and seeing roughly how it goes.
*
* In practice I found it difficult to pick values for these, because for
* the lower end of the scale it's hard to see a meaningful difference,
* and for the upper end of the scale the time is dominated by
* determinisation anyway.
*/
#define AC_COUNT_THRESHOLD 1
#define AC_LENGTH_THRESHOLD 1
#define AC_COUNT_THRESHOLD 5
#define AC_LENGTH_THRESHOLD 5

#define LOG_LINKAGE 0

Expand Down

0 comments on commit 5778d25

Please sign in to comment.