On 03/10/2015 10:00 AM, Gris Ge wrote: > On Thu, Mar 05, 2015 at 07:04:06PM -0600, Tony Asleson wrote: > > Hi Tony, > > All other patches looks good. With trivial changes below, it passes > 'make rpm' test in all supported OS. > >> Signed-off-by: Tony Asleson <tasle...@redhat.com> >> --- >> c_binding/lsm_mgmt.cpp | 23 ++++++++++++----------- >> 1 file changed, 12 insertions(+), 11 deletions(-) >> >> diff --git a/c_binding/lsm_mgmt.cpp b/c_binding/lsm_mgmt.cpp >> index 54a3b2d..3b91402 100644 >> --- a/c_binding/lsm_mgmt.cpp >> +++ b/c_binding/lsm_mgmt.cpp >> @@ -1849,6 +1849,16 @@ int lsm_fs_file_clone(lsm_connect *c, lsm_fs *fs, >> const char *src_file_name, >> return jobCheck(c, rc, response, job); >> } >> >> +static Value _create_fs_file_flag_params(lsm_fs *fs, lsm_string_list *files, >> + lsm_flags flags) > Should be 'lsm_flag flags'.
I wonder how this didn't fail for me when I ran the tests... Thanks for the review! Regards, Tony ------------------------------------------------------------------------------ Dive into the World of Parallel Programming The Go Parallel Website, sponsored by Intel and developed in partnership with Slashdot Media, is your hub for all things parallel software development, from weekly thought leadership blogs to news, videos, case studies, tutorials and more. Take a look and join the conversation now. http://goparallel.sourceforge.net/ _______________________________________________ Libstoragemgmt-devel mailing list Libstoragemgmt-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/libstoragemgmt-devel
