58f989715a
This PR #1330, addressing #1312 (parseQuery is partial) and #1245 (internal server error). User-visible changes: - hledger-web now handles malformed regular expressions (eg, a query consisting of the single character `?`) gracefully, showing a tidy error message instead "internal server error". API/internal changes: - The Regex type alias has been replaced by the Regexp ADT, which contains both the compiled regular expression (so is guaranteed to be usable at runtime) and the original string (so can be serialised, printed, compared, etc.) A Regexp also knows whether is it case sensitive or case insensitive. The Hledger.Utils.Regex api has changed. - Typeable and Data instances are no longer derived for hledger's data types; they were redundant/no longer needed - NFData instances are no longer derived for hledger's data types. This speeds up a full build by roughly 7%. But it means we can't deep-evaluate hledger values, or time hledger code with Criterion. https://github.com/simonmichael/hledger/pull/1330#issuecomment-684075129 has some ideas on this. - Query no longer has a custom Show instance - Some internal use of regexps was replaced by text replacement or parsers. - Hledger.Utils.String: quoteIfNeeded now actually escapes quotes in strings; dropped escapeQuotes - Hledger.Utils.Tree: dropped some old utilities - dropped some obsolete code for the old --display option Merge branch 'regexp' into master |
||
|---|---|---|
| .. | ||
| AccountTransactionsReport.hs | ||
| BalanceReport.hs | ||
| BudgetReport.hs | ||
| EntriesReport.hs | ||
| MultiBalanceReport.hs | ||
| PostingsReport.hs | ||
| ReportOptions.hs | ||
| ReportTypes.hs | ||
| TransactionsReport.hs | ||