Damien Ruscoe

[Addon] Cache commonly parsed XML files

7

## Description During a scan of libraries (Tested using Movies and TV shows) each directory is queried for 2x settings. If no settings are found for that specific directory being...

[CURL] Reduce the number of constructions of CURL objects.

1

## Description CURL objects are very expensive to create. The constructor calls `CURL::Parse` on every construction of a new object which analyses the string and parses it into many smaller...

[CURL] Refactored archive CURL parsing

2

Extracted utility function from CURL::Parse to ## Description Refactored the code for the self described "ugly hack" which converts `/path/to/file.zip/foo/bar` into `zip://{/path/to/file.zip}/foo/bar` ## Motivation and context The code was not...

Calculating and allocating the initial string buffer is unnecessary as the string is assigned to by the result of `GetWithoutOptions`. ## Description A string expected size is calculated, then allocated...

[CURL][tests] Added tests checking for differences between URIUtils::* and CURL::* methods.

8

[CURL][tests] Added tests checking for differences between URIUtils::* and CURL::* methods. Fixed issue found in testing when a service is unavailable. Check the service is non-null before calling its methods....

[CURL] Forward CURL object through interfaces.

4

## Description 2 CURL objects are created for each call to `PluginDirectory::GetDirectory`. This is unecessary as the CURL object is passed into this method. Pass this object down through the...

[FileItemList] Factor out manual index iteration while removing items into a common function.

15

## Description Factored out common code for filtering of items from a `CFileItemList` and guarded against the issues described below. Changed the existing use cases of filtering to use this...