Skip to content

Tildes are now part of normal tokens

Michael Aaron Murphy requested to merge huntergoldstein:panic into master

Created by: huntergoldstein

Fixes: Closes #406 (closed).

Summary: Instead of parsing tildes separately from normal tokes they are now included inside normal tokens, but the tilde flag is set so when they are expanded the ExpansionFunction::tilde function is called on the string.

Drawbacks:

  • Very untested (but all the tests pass, and the examples)
  • I'm not sure this is the optimal solution, but it works
  • This is dependent on expand_func.tilde expecting a full path that is prefixed with a tilde; I'm not sure if we're ok with that.

Merge request reports