[sane-devel] [RFC] mksotemp
Klaus Kaempf
kkaempf at suse.de
Mon May 9 13:32:17 UTC 2016
* Olaf Meeuwissen <paddy-hack at member.fsf.org> [May 09. 2016 15:10]:
>
> Alessandro Zummo writes:
>
> > On Mon, 02 May 2016 11:57:58 +0900
> > Olaf Meeuwissen <paddy-hack at member.fsf.org> wrote:
> >
> >> That leaves zero flags to be passed, so the mkostemp() call reduces to
> >> mkstemp().
> >>
> >> I would simply call mkstemp() unconditionally. That way you also don't
> >> have to add it to AC_CHEC_FUNCS() in configure.in anymore.
> >
> > I agree with you, but not being the original
> > author I didn't want to remove it all :)
>
> Simplicity is a virtue. There is no benefit in making the code more
> convoluted than necessary. Just call mkstemp() unconditionally. If
> you write a commit message that documents the rationale, I doubt the
> original author will complain.
:-)
No worries, just go ahead !
Klaus
--
SUSE Linux GmbH, GF: Felix Imendörffer, Jane Smithard, Graham Norton, HRB 21284 (AG Nürnberg)
More information about the sane-devel
mailing list