Hi,

Ulf Zibis wrote:
> Hi,
>
> Am 09.02.2016 um 08:33 schrieb Jean-Pierre André:
>>
>> This is a strange cleanup, which causes unacceptable regressions.
>>
>> Sorry, I cannot accept them, they do not even pass the self-tests
>> internal to secaudit :
>
> Much thanks for the testing. I should have introduced a bug.
> How can I run the self-testing my self. I see some conditional test
> code, but I don't find any hint, how to use it.

Define the macro SELFTEST

>>
>> Finally, how did you get into the condition you are trying to fix ?
>
> There are 2:
> - broken indentation makes it more difficult to me to understand the code.
> - redundancy makes it more difficult to me to understand the code.
>

I was asking what you were trying to fix, because I thought you had
hidden something about about fixing some ACL encoding or decoding
issue. If this is not the case, I am not interested.

No patch for rewriting existing code please, even for optimizing.

I am only interested in patches limited to what is strictly needed to
fix an issue or to provide a new feature, with no unneeded change to
existing code. I also expect them to have been properly tested.

Regards

Jean-Pierre



------------------------------------------------------------------------------
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140
_______________________________________________
ntfs-3g-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ntfs-3g-devel

Reply via email to