leave the deliberately unused, not-for-use method inline

doing so means code is not generated for it seeing as its not used, moving it
out means code is generated for it and its detected as unused code

Change-Id: I72edc89446bdfac50069e0b509fc7f90540e0d4a
callcatcher: remove newly unused code

and rework reads to just return the read value

Change-Id: I5d2f01064465c65859ec4ba031ec9dfa16403487
leave this Clone as inline

so code isn't generated for it unless the new_clone is required,
which it isn't under gcc.

Change-Id: I07e0b3e4b2e07625509b51ba7d37e24440f56cfe