[PATCH] fileread for the std VMOD

Sanjoy Das sanjoy at playingwithpointers.com
Wed Nov 24 08:01:38 CET 2010


Hi!

> Can I get you to put fileread into a separate source file ?  It is
> sufficiently complex for that.

Done.

>
> Some nits from FlexeLint:
>
> ../../lib/libvmod_std/vmod_std.c  288  Warning 529: Symbol 'aligned_sz' (line
>    207) not subsequently referenced
> ../../lib/libvmod_std/vmod_std.c  207  Info 830: Location cited in prior
>    message
>
> (Use the autogen.des script to get developer settings and GCC would have
> told you this)

Sorry; was not aware. The patch now builds without error with the
Makefiles generated by autogen.des.

-- 
Regards,

Sanjoy Das.

http://playingwithpointers.com

Public Key at http://playingwithpointers.com/custom/public_key.txt
-------------- next part --------------
A non-text attachment was scrubbed...
Name: vmod-fileread.patch
Type: text/x-patch
Size: 4613 bytes
Desc: not available
URL: <https://www.varnish-cache.org/lists/pipermail/varnish-dev/attachments/20101124/d8b39eb2/attachment-0006.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: vmod-fileread-testcase-patch.patch
Type: text/x-patch
Size: 2769 bytes
Desc: not available
URL: <https://www.varnish-cache.org/lists/pipermail/varnish-dev/attachments/20101124/d8b39eb2/attachment-0007.bin>


More information about the varnish-dev mailing list