Re: [PATCH 2/2] CRED: Introduce credential access wrappers

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Ingo Oeser
Date: Tuesday, August 12, 2008 - 3:59 pm

Hi David,

On Tuesday 12 August 2008, David Howells wrote:

Ok, if that is the only reason, please mention this in the commit message.
That would be enough for me to not wonder about this issue.

But I wonder how you solve this issue if the amount of code behinde your wrappers
grows.

If you have to take them out of line, it would give me a good indication
of the costs involved in the later bodies of these now trivial wrappers,
if you do it right away. 

That way you'll find missing includes and whatnot while converting existing code
to the trivial wrappers.


Best Regards

Ingo Oeser
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 0/2] Introduce credentials API, David Howells, (Tue Aug 12, 6:28 am)
[PATCH 2/2] CRED: Introduce credential access wrappers, David Howells, (Tue Aug 12, 6:28 am)
Re: [PATCH 2/2] CRED: Introduce credential access wrappers, Ingo Oeser, (Tue Aug 12, 3:59 pm)
Re: [PATCH 1/2] CRED: Alter XFS so as to avoid namespace c ..., Christoph Hellwig, (Tue Aug 12, 4:01 pm)