On 10/21/2011 07:37 PM, Dodji Seketeli wrote:
It also makes linemap_expand_location_full to return the location it
resolved to.

I think I'd prefer to have expand_location call linemap_resolve_location and then linemap_expand_location, and perhaps remove linemap_expand_location_full.

Incidentally, I notice that there's no assert to enforce the requirement that linemap_expand_location only takes spelling locations.

Jason

Reply via email to