Thanks for the feedback Maxime, I obviously lost track of some things
after letting this patch languish for a while.
Maxime Devos writes:
> On 17-09-2022 10:05, Christopher Baines wrote:
>> + if (ioptname == SO_RCVTIMEO || ioptname == SO_SNDTIMEO)
>> +{
>> + SCM_ASSERT (scm_is_pair (v
On 17-09-2022 10:05, Christopher Baines wrote:
+ if (ioptname == SO_RCVTIMEO || ioptname == SO_SNDTIMEO)
+{
+ SCM_ASSERT (scm_is_pair (value), value, SCM_ARG4, FUNC_NAME);
How about using SCM_ASSERT_TYPE instead to improve the error message a
little?
+ opt_time.tv_sec = scm
