Re: [PATCH 0/4] Missing Support of SASL Sign or Seal using Data Security Layer

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 10/8/22 14:52, Thomas Reim wrote:
On 10.08.22 03:04, Ian Kent wrote:

Thanks Thomas, this functionality sounds useful and the code looks good.


Mostly the comments I'll make later (once I've looked at it a few times)

are about my coding style preferences but there was one place where I

had questions about some function call behaviors. I'm tempted to make

the style changes myself but it's your code and I'd probably make a

mistake or two, ;)


Ian

Dear Ian,

thank you for your feedback. Looking forward to your comments. I tried to align

the style to yours but of course it's just a guess. Either way is fine for me. Most

important is a working SASL data security layer. So if you like I can also perform

style updates, once I have an idea :-), test all updates and send the reworked patches

to the list.


Yeah, I got distracted today but I'll get to it and have a

look at the log changes too.


Ian





[Index of Archives]     [Linux Filesystem Development]     [Linux Ext4]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux