No description provided.
Cleanup: Replace "URL" with "URI" where we aren't actually working with URLs #987
pull luke-jr wants to merge 1 commits into bitcoin:master from luke-jr:ipc_name_fix changing 15 files +87 −84-
luke-jr commented at 9:27 PM on March 25, 2012: member
-
TheBlueMatt commented at 12:33 AM on March 26, 2012: member
You really need to stop calling things bugfixes that arent ;). This is a code cleanliness change, no functional or user-visible changes are made.
-
luke-jr commented at 12:36 AM on March 26, 2012: member
I consider it a bug to say "apple" when it refers to a grapefruit :p
-
TheBlueMatt commented at 3:35 AM on March 26, 2012: member
I dont if its not user-facing and doesnt affect functionality.
-
luke-jr commented at 2:26 PM on April 3, 2012: member
Rebased, this should probably go into 0.6.1.
-
Diapolo commented at 4:45 PM on April 6, 2012: none
NACK,
I think while we are working on the URI handling and the IPC stuff, there should be no cosmetic changes to the code, that only renames stuff. After the main work is finished I'm fine with a rename to URI, but that's for sure a core dev vote :).
-
Bugfix: Replace "URL" with "URI" where we aren't actually working with URLs fa2544e79f
-
gmaxwell commented at 6:22 PM on April 6, 2012: contributor
Since we don't have any merge ready IPC fixes, I'm inclined to take this largely cosmetic fix now— we are in cleanup mode afterall, Sorry Diapolo.
- gmaxwell referenced this in commit 138d08c531 on Apr 6, 2012
- gmaxwell merged this on Apr 6, 2012
- gmaxwell closed this on Apr 6, 2012
-
Diapolo commented at 8:07 PM on April 6, 2012: none
No prob, as long as I'm allowed to say what I think ;).
- coblee referenced this in commit 40bbc1b916 on Jul 17, 2012
- suprnurd referenced this in commit 84ac8de59d on Dec 5, 2017
- ptschip referenced this in commit 924c27a8bc on Mar 8, 2018
- Bushstar referenced this in commit 2885163e52 on Sep 1, 2019
- lateminer referenced this in commit fa1dbab247 on Oct 30, 2019
- DrahtBot locked this on Sep 8, 2021