search

nim-lang / RFCs

A repository for your Nim proposals.
135 stars 26 forks source link

Rename xmltree.escape to escapeXml #333

Closed planetis-m closed 3 years ago

planetis-m commented 3 years ago

It usually conflicts with strutils.escape. Comparably json has escapeJson.

Araq commented 3 years ago

So what. import strutils except escape works well. There are lots of other solutions like typing xmltree.escape.

planetis-m commented 3 years ago

After some testing, I think that escapeJson is not a good name to begin with. I mean for my use-case it turned out to be a better match than even strutils.escape

Araq commented 3 years ago

So ... close?