[PATCH] Add stackframes to public libsmbclient functions

Jeremy Allison jra at samba.org
Mon Nov 19 18:44:51 GMT 2007


On Mon, Nov 19, 2007 at 10:31:16AM -0500, Derrell Lipman wrote:

> I don't have a strong opinion about that one way or the other.  If the
> exits are simple 'return some value' I have no problem with it.  If
> every return has to do a bunch of clean-up, I prefer a single exit
> point.

This is just a future patch :-).

> In this case, I'm not the original author of the code, so unless I've
> had reason to change it, I've just maintained the original
> organization of the code.  Even with a single entry and exit point,
> though, I'd still have a hard time justifying "magic" inputs to the
> lower-layer functions.  Without some strong reason to the contrary, I
> prefer the black-box concept with inputs and outputs well defined by
> parameters and return value.

This is how libsmbclient behaves to the upper layers,
and Volker's patch does the same :-).

Jeremy.


More information about the samba-technical mailing list