On 12/1/21 19:20, Paul Eggert wrote:
On 12/1/21 13:02, Robbie Harwood wrote:@@ -1099,7 +1099,7 @@ optimize_utf8 (re_dfa_t *dfa) } break; default: - abort (); + break; }Likewise, it's not clear why this change is needed. The 'abort' should not be reachable.Is the intent to make the code a bit smaller by avoding calls to 'abort'?
A followup idea: would it help to replace 'abort ()' with 'DEBUG_ASSERT (false)', or to replace 'if (!X) abort ();' with 'DEBUG_ASSERT (X);'?