refactor: build the librrd argument list without the dummy slot - #16

Open

somethingwithproof wants to merge 17 commits into

php:masterfrom

somethingwithproof:refactor/argv-builder

Commits on Aug 21, 2026

ci: build the test fixtures before running the suite
somethingwithproof
committed
fca1950
tests: refresh the rrd_tune and rrd_xport expectations
somethingwithproof
committed
109a553
fix: initialise the RRDCreator startTime parameter
somethingwithproof
committed
def9748
fix: validate object state before building rrd arguments
somethingwithproof
committed
c3f5957
fix: size the timestamp buffer for 64-bit time values
somethingwithproof
committed
f18be81
fix: reset the remaining creator state on re-construction
somethingwithproof
committed
d3c14d3
fix: apply open_basedir to RRDCreator::save and RRDGraph::saveVerbose
somethingwithproof
committed
ea62c3e
fix: stop rewriting the caller's options array
somethingwithproof
committed
f01390f
fix: abort argument building when a value cannot be converted
somethingwithproof
committed
19a881c
fix: pin the checked path and the options array across userland calls
somethingwithproof
committed
51c7997
fix: hold a reference to the graph options while building the argument list
somethingwithproof
committed
4412812
fix: clear file_path on release and stop walking past the fetch result
somethingwithproof
committed
39def9a
tests: cover duplicate data source names, re-construction and unbuilt objects
somethingwithproof
committed
d00ead3
chore: replace three macros PHP master has removed
somethingwithproof
committed
e18e4d3
ci: add an allowed-to-fail job against php-src master
somethingwithproof
committed
019fd68
chore: probe for librrd's argv constness instead of assuming it
somethingwithproof
committed
49835d3
refactor: build the librrd argument list without the dummy slot
somethingwithproof
committed
d5a4842

Navigation Menu

Uh oh!

refactor: build the librrd argument list without the dummy slot - #16

refactor: build the librrd argument list without the dummy slot#16
somethingwithproof wants to merge 17 commits into
php:masterfrom
somethingwithproof:refactor/argv-builder

Commits on Aug 21, 2026

ci: build the test fixtures before running the suite

tests: refresh the rrd_tune and rrd_xport expectations

fix: initialise the RRDCreator startTime parameter

fix: validate object state before building rrd arguments

fix: size the timestamp buffer for 64-bit time values

fix: reset the remaining creator state on re-construction

fix: apply open_basedir to RRDCreator::save and RRDGraph::saveVerbose

fix: stop rewriting the caller's options array

fix: abort argument building when a value cannot be converted

fix: pin the checked path and the options array across userland calls

fix: hold a reference to the graph options while building the argument list

fix: clear file_path on release and stop walking past the fetch result

tests: cover duplicate data source names, re-construction and unbuilt objects

chore: replace three macros PHP master has removed

ci: add an allowed-to-fail job against php-src master

chore: probe for librrd's argv constness instead of assuming it

refactor: build the librrd argument list without the dummy slot