Merge lp:~zorba-coders/zorba/set-uri-components into lp:zorba

Proposed by Chris Hillery
Status: Merged
Merged at revision: 11656
Proposed branch: lp:~zorba-coders/zorba/set-uri-components
Merge into: lp:zorba
Diff against target: 67 lines (+8/-0)
1 file modified
src/zorbatypes/URI.cpp (+8/-0)
To merge this branch: bzr merge lp:~zorba-coders/zorba/set-uri-components
Reviewer Review Type Date Requested Status
Luis Rodriguez Gonzalez Approve
Chris Hillery Approve
Review via email: mp+190740@code.launchpad.net

Commit message

Ensure calling set_XXX() followed by toString() works.

To post a comment you must log in.
Revision history for this message
Chris Hillery (ceejatec) :
review: Approve
Revision history for this message
Luis Rodriguez Gonzalez (kuraru) :
review: Approve
Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :

Validation queue starting for the following merge proposals:
https://code.launchpad.net/~zorba-coders/zorba/set-uri-components/+merge/190740

Progress dashboard at http://jenkins.lambda.nu/view/ValidationQueue

Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :

Validation queue succeeded - proposal merged!

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/zorbatypes/URI.cpp'
2--- src/zorbatypes/URI.cpp 2013-08-02 19:21:03 +0000
3+++ src/zorbatypes/URI.cpp 2013-10-11 18:06:57 +0000
4@@ -1099,6 +1099,7 @@
5
6 theScheme = new_scheme;
7 set_state(Scheme);
8+ invalidate_text();
9 }
10
11
12@@ -1109,6 +1110,7 @@
13 {
14 theHost = new_host;
15 set_state(Host);
16+ invalidate_text();
17 }
18
19
20@@ -1119,6 +1121,7 @@
21 {
22 thePort = new_port;
23 set_state(Port);
24+ invalidate_text();
25 }
26
27
28@@ -1130,6 +1133,7 @@
29 uri::encode(new_user_info, &theUserInfo, false);
30
31 set_state(UserInfo);
32+ invalidate_text();
33 }
34
35
36@@ -1149,6 +1153,7 @@
37 set_state(RegBasedAuthority);
38
39 unset_state(Host);
40+ invalidate_text();
41 }
42
43
44@@ -1170,6 +1175,7 @@
45 {
46 initializePath(new_path);
47 }
48+ invalidate_text();
49 }
50
51 /*******************************************************************************
52@@ -1179,6 +1185,7 @@
53 {
54 theQueryString = new_query;
55 set_state(QueryString);
56+ invalidate_text();
57 }
58
59 void URI::set_opaque_part(const zstring& new_scheme_specific)
60@@ -1193,6 +1200,7 @@
61 theOpaquePart = new_scheme_specific;
62 set_state(OpaquePart);
63 }
64+ invalidate_text();
65 }
66
67 /*******************************************************************************

Subscribers

People subscribed via source and target branches