search

qt4cg / qtspecs

QT4 specifications
https://qt4cg.org/
Other
28 stars 15 forks source link

1387 Another tweak to build-uri #1440

Closed ndw closed 2 months ago

ndw commented 2 months ago

In #1387 @ChristianGruen observes that the + seems to be a special case in the tel: scheme. In fact, I think we concluded that the special case is that you shouldn't encode the path segment of a non-hierarchical URI.

This PR attempts to implement that.

ndw commented 2 months ago

Yes, you've persuaded me. And I've tried to change the rules accordingly. It was not easy to work out how best to describe those rules. Comments welcome.

ndw commented 2 months ago

Good catch on the duplicated definition. I confess, I was zoomed in on those couple of paragraphs trying to make something that wasn't awful and convoluted. I'll look at harmonizing those two definitions tomorrow morning. That seems like a friendly ammendment.

On the jar: URI test, I'm inclined to live with it. jar: URIs are a terrible hack anyway. The alternative would be yet more special cases and the whole thing could become (even more) unweildy.

ChristianGruen commented 2 months ago

I’ve readded Propose Merge without Discussion, maybe we can add it to today’s list.

ndw commented 2 months ago

The CG agreed to merge this issue without discussion at meeting 090.