[haiku-commits] Change in ...haiku[master]: [WIP]:Burl-remove Urlencode BUrl rejects the strings with delimiters....

  • From: Gerrit <review@xxxxxxxxxxxxxxxxxxx>
  • To: waddlesplash <waddlesplash@xxxxxxxxx>, haiku-commits@xxxxxxxxxxxxx
  • Date: Wed, 13 Mar 2019 11:30:18 +0000

From <thisisathi@xxxxxxxxx>:

thisisathi@xxxxxxxxx has uploaded this change for review. ( 
https://review.haiku-os.org/c/haiku/+/1193


Change subject: [WIP]:Burl-remove Urlencode BUrl rejects the strings with 
delimiters. It is not possible anymore to create an URL and encode it later So 
urlencode function has to be removed. -Removed the UrlEncode and UrlDecode from 
the public API - Kept only the static
......................................................................

[WIP]:Burl-remove Urlencode
BUrl rejects the strings with delimiters.
It is not possible anymore to create an URL and encode it
later
So urlencode function has to be removed.
-Removed the UrlEncode and UrlDecode from the public API
- Kept only the static version of them, which work on a BString
- Modified all the places where a BUrl is set from a string (constructors,
  all the Set* functions, to take an extra parameter (bool encode).
- Set the default value of the parameter (encode = true), so that if the
  user does not specify anything, by default, the URL will be
  URL-encoded automatically
---
M headers/os/support/Url.h
M src/kits/network/libnetapi/DataRequest.cpp
M src/kits/network/libnetapi/FileRequest.cpp
M src/kits/network/libnetapi/GopherRequest.cpp
M src/kits/support/Url.cpp
5 files changed, 15 insertions(+), 12 deletions(-)



  git pull ssh://git.haiku-os.org/haiku refs/changes/93/1193/1
--
To view, visit https://review.haiku-os.org/c/haiku/+/1193
To unsubscribe, or for help writing mail filters, visit 
https://review.haiku-os.org/settings

Gerrit-Project: haiku
Gerrit-Branch: master
Gerrit-Change-Id: I06f06978d0d35e56d7c92b67f001856bb7dcafc8
Gerrit-Change-Number: 1193
Gerrit-PatchSet: 1
Gerrit-Owner: thisisathi@xxxxxxxxx
Gerrit-MessageType: newchange

Other related posts: