Remove explicit reference to _url. Always regenerate when needed.

This commit is contained in:
Gregory John Casamento 2020-04-03 17:39:35 -04:00
commit d5c163397f

View file

@ -50,7 +50,6 @@ function may be incorrect
NSString *_password; \
NSString *_path; \
NSNumber *_port; \
NSString *_query; \
NSArray *_queryItems; \
NSString *_scheme; \
NSString *_user; \
@ -2370,10 +2369,11 @@ GS_PRIVATE_INTERNAL(NSURLComponents)
- (instancetype) initWithURL: (NSURL *)url
resolvingAgainstBaseURL: (BOOL)resolve
{
NSURL *tempURL = url;
self = [self init];
if (self != nil)
{
NSURL *tempURL = url;
if (resolve)
{
tempURL = [url absoluteURL];
@ -2385,7 +2385,6 @@ GS_PRIVATE_INTERNAL(NSURLComponents)
- (void) dealloc
{
RELEASE(internal->_url);
RELEASE(internal->_fragment);
RELEASE(internal->_host);
RELEASE(internal->_password);
@ -2404,53 +2403,15 @@ GS_PRIVATE_INTERNAL(NSURLComponents)
resolvingAgainstBaseURL: NO];
}
// Getting the URL
- (NSString *) string
{
return [[self _regenerateURL] absoluteString];
}
- (void) setString: (NSString *)urlString
{
NSURL *url = [NSURL URLWithString: urlString];
[self setURL : url];
}
- (NSURL *) URL
{
return [self _regenerateURL];
}
- (void) setURL: (NSURL *)url
{
// components...
ASSIGNCOPY(internal->_fragment, [url fragment]);
ASSIGNCOPY(internal->_host, [url host]);
ASSIGNCOPY(internal->_password, [url password]);
ASSIGNCOPY(internal->_path, [url path]);
ASSIGNCOPY(internal->_port, [url port]);
ASSIGNCOPY(internal->_query, [url query]);
ASSIGNCOPY(internal->_scheme, [url scheme]);
ASSIGNCOPY(internal->_user, [url user]);
_dirty = YES;
}
- (NSURL *)URLRelativeToURL: (NSURL *)baseURL
{
return nil;
}
// Regenerate URL when components are changed...
- (NSURL *) _regenerateURL
{
if (_dirty == NO)
NSURL *u = nil;
if (internal->_dirty == NO)
{
return nil;
}
else
{
NSURL *u;
u = [[NSURL alloc] initWithScheme: internal->_scheme
user: internal->_user
password: internal->_password
@ -2471,15 +2432,49 @@ GS_PRIVATE_INTERNAL(NSURLComponents)
internal->_rangeOfPassword = URL_COMPONENT_RANGE(internal->_password);
internal->_rangeOfPath = URL_COMPONENT_RANGE(internal->_path);
internal->_rangeOfPort = URL_COMPONENT_RANGE([internal->_port stringValue]);
internal->_rangeOfQuery = URL_COMPONENT_RANGE(internal->_query);
internal->_rangeOfQuery = URL_COMPONENT_RANGE([self query]);
internal->_rangeOfScheme = URL_COMPONENT_RANGE(internal->_scheme);
internal->_rangeOfUser = URL_COMPONENT_RANGE(internal->_user);
#undef URL_COMPONENT_RANGE
}
_dirty = NO;
internal->_dirty = NO;
return u;
}
// Getting the URL
- (NSString *) string
{
return [[self URL] absoluteString];
}
- (void) setString: (NSString *)urlString
{
NSURL *url = [NSURL URLWithString: urlString];
[self setURL: url];
}
- (NSURL *) URL
{
return [self _regenerateURL];
}
- (void) setURL: (NSURL *)url
{
// Set all the components...
[self setScheme: [url scheme]];
[self setHost: [url host]];
[self setPort: [url port]];
[self setUser: [url user]];
[self setPassword: [url password]];
[self setPath: [url path]];
[self setQuery: [url query]];
[self setFragment: [url fragment]];
}
- (NSURL *) URLRelativeToURL: (NSURL *)baseURL
{
return nil;
}
// Accessing Components in Native Format
@ -2491,7 +2486,7 @@ GS_PRIVATE_INTERNAL(NSURLComponents)
- (void) setFragment: (NSString *)fragment
{
ASSIGNCOPY(internal->_fragment, fragment);
_dirty = YES;
internal->_dirty = YES;
}
- (NSString *) host
@ -2502,7 +2497,7 @@ GS_PRIVATE_INTERNAL(NSURLComponents)
- (void) setHost: (NSString *)host
{
ASSIGNCOPY(internal->_host, host);
_dirty = YES;
internal->_dirty = YES;
}
- (NSString *) password
@ -2513,7 +2508,7 @@ GS_PRIVATE_INTERNAL(NSURLComponents)
- (void) setPassword: (NSString *)password
{
ASSIGNCOPY(internal->_password, password);
_dirty = YES;
internal->_dirty = YES;
}
- (NSString *) path
@ -2524,7 +2519,7 @@ GS_PRIVATE_INTERNAL(NSURLComponents)
- (void) setPath: (NSString *)path
{
ASSIGNCOPY(internal->_path, path);
_dirty = YES;
internal->_dirty = YES;
}
- (NSNumber *) port
@ -2535,17 +2530,31 @@ GS_PRIVATE_INTERNAL(NSURLComponents)
- (void) setPort: (NSNumber *)port
{
ASSIGNCOPY(internal->_port, port);
_dirty = YES;
internal->_dirty = YES;
}
- (NSString *) query
{
return internal->_query;
NSString *query = @"";
NSEnumerator *en = [internal->_queryItems objectEnumerator];
NSURLQueryItem *item = nil;
while ((item = (NSURLQueryItem *)[en nextObject]) != nil)
{
NSString *name = [item name];
NSString *value = [item value];
NSString *itemString = [NSString stringWithFormat: @"%@=%@",name,value];
if ([query length] > 0)
{
query = [query stringByAppendingString: @"&"];
}
query = [query stringByAppendingString: itemString];
}
return query;
}
- (void) setQuery: (NSString *)query
{
ASSIGNCOPY(internal->_query, query);
if (query != nil)
{
NSMutableArray *result = [NSMutableArray arrayWithCapacity: 5];
@ -2564,6 +2573,10 @@ GS_PRIVATE_INTERNAL(NSURLComponents)
}
[self setQueryItems: result];
}
else
{
[self setQueryItems: nil];
}
}
- (NSArray *) queryItems
@ -2573,25 +2586,9 @@ GS_PRIVATE_INTERNAL(NSURLComponents)
- (void) setQueryItems: (NSArray *)queryItems
{
NSString *query = @"";
NSEnumerator *en = [queryItems objectEnumerator];
NSURLQueryItem *item = nil;
while ((item = (NSURLQueryItem *)[en nextObject]) != nil)
{
NSString *name = [item name];
NSString *value = [[item value] _stringByAddingPercentEscapesForQuery];
NSString *itemString = [NSString stringWithFormat: @"%@=%@",name,value];
query = [query stringByAppendingString: itemString];
if (item != [queryItems lastObject])
{
query = [query stringByAppendingString: @"&"];
}
}
ASSIGNCOPY(internal->_query, query); // add query string...
ASSIGNCOPY(internal->_queryItems, queryItems);
_dirty = YES;
internal->_dirty = YES;
}
- (NSString *) scheme
@ -2602,7 +2599,7 @@ GS_PRIVATE_INTERNAL(NSURLComponents)
- (void) setScheme: (NSString *)scheme
{
ASSIGNCOPY(internal->_scheme, scheme);
_dirty = YES;
internal->_dirty = YES;
}
- (NSString *) user
@ -2613,7 +2610,7 @@ GS_PRIVATE_INTERNAL(NSURLComponents)
- (void) setUser: (NSString *)user
{
ASSIGNCOPY(internal->_user, user);
_dirty = YES;
internal->_dirty = YES;
}
// Accessing Components in PercentEncoded Format
@ -2625,8 +2622,7 @@ GS_PRIVATE_INTERNAL(NSURLComponents)
- (void) setPercentEncodedFragment: (NSString *)fragment
{
ASSIGNCOPY(internal->_fragment, [fragment stringByRemovingPercentEncoding]);
_dirty = YES;
[self setFragment: [fragment stringByRemovingPercentEncoding]];
}
- (NSString *) percentEncodedHost
@ -2637,8 +2633,7 @@ GS_PRIVATE_INTERNAL(NSURLComponents)
- (void) setPercentEncodedHost: (NSString *)host
{
ASSIGNCOPY(internal->_host, [host stringByRemovingPercentEncoding]);
_dirty = YES;
[self setHost: [host stringByRemovingPercentEncoding]];
}
- (NSString *) percentEncodedPassword
@ -2649,8 +2644,7 @@ GS_PRIVATE_INTERNAL(NSURLComponents)
- (void) setPercentEncodedPassword: (NSString *)password
{
ASSIGNCOPY(internal->_password, [password stringByRemovingPercentEncoding]);
_dirty = YES;
[self setPassword: [password stringByRemovingPercentEncoding]];
}
- (NSString *) percentEncodedPath
@ -2661,41 +2655,55 @@ GS_PRIVATE_INTERNAL(NSURLComponents)
- (void) setPercentEncodedPath: (NSString *)path
{
ASSIGNCOPY(internal->_path, [path stringByRemovingPercentEncoding]);
_dirty = YES;
[self setPath: [path stringByRemovingPercentEncoding]];
}
- (NSString *) percentEncodedQuery
{
return internal->_query;
NSString *query = @"";
NSEnumerator *en = [internal->_queryItems objectEnumerator];
NSURLQueryItem *item = nil;
while ((item = (NSURLQueryItem *)[en nextObject]) != nil)
{
NSString *name = [[item name] _stringByAddingPercentEscapesForQuery];
NSString *value = [[item value] _stringByAddingPercentEscapesForQuery];
NSString *itemString = [NSString stringWithFormat: @"%@=%@",name,value];
if ([query length] > 0)
{
query = [query stringByAppendingString: @"&"];
}
query = [query stringByAppendingString: itemString];
}
return query;
}
- (void) setPercentEncodedQuery: (NSString *)query
{
ASSIGNCOPY(internal->_query, [query stringByRemovingPercentEncoding]);
_dirty = YES;
[self setQuery: [query stringByRemovingPercentEncoding]];
}
- (NSArray *) percentEncodedQueryItems
{
// FIXME
return internal->_queryItems;
}
- (void) setPercentEncodedQueryItems: (NSArray *)queryItems
{
// FIXME
[self setQueryItems: queryItems];
}
- (NSString *) percentEncodedScheme
{
return [internal->_path stringByAddingPercentEncodingWithAllowedCharacters:
return [internal->_scheme stringByAddingPercentEncodingWithAllowedCharacters:
[NSCharacterSet URLPathAllowedCharacterSet]];
}
- (void) setPercentEncodedScheme: (NSString *)scheme
{
ASSIGNCOPY(internal->_scheme, scheme);
_dirty = YES;
[self setScheme: [scheme stringByRemovingPercentEncoding]];
}
- (NSString *) percentEncodedUser
@ -2706,8 +2714,7 @@ GS_PRIVATE_INTERNAL(NSURLComponents)
- (void) setPercentEncodedUser: (NSString *)user
{
ASSIGNCOPY(internal->_user, [user stringByRemovingPercentEncoding]);
_dirty = YES;
[self setUser: [user stringByRemovingPercentEncoding]];
}
// Locating components of the URL string representation