From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yuval Shaia Subject: Re: [PATCH rdma-core] rc_pingpong: Fix getopt processing of -t Date: Wed, 31 Jan 2018 09:17:02 +0200 Message-ID: <20180131071700.GA3220@yuvallap> References: <20180131000415.GA29879@ziepe.ca> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20180131000415.GA29879-uk2M96/98Pc@public.gmane.org> Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Jason Gunthorpe Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-rdma@vger.kernel.org On Tue, Jan 30, 2018 at 05:04:15PM -0700, Jason Gunthorpe wrote: > The -t option never had an argument, a : was erroneously added when > the -c option was added Thanks, Now i've learned how to add arguments so can implement Ram's idea :) Sorry for that. Reviewed-by: Yuval Shaia > > Fixes: 099c5aa50bc8 ("libibverb/examples: Add command line option to enable buffer validation") > Signed-off-by: Jason Gunthorpe > --- > libibverbs/examples/rc_pingpong.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libibverbs/examples/rc_pingpong.c b/libibverbs/examples/rc_pingpong.c > index b5153bceb8254b..cf5c045741b7be 100644 > --- a/libibverbs/examples/rc_pingpong.c > +++ b/libibverbs/examples/rc_pingpong.c > @@ -730,7 +730,7 @@ int main(int argc, char *argv[]) > {} > }; > > - c = getopt_long(argc, argv, "p:d:i:s:m:r:n:l:eg:ot:c", > + c = getopt_long(argc, argv, "p:d:i:s:m:r:n:l:eg:otc", > long_options, NULL); > > if (c == -1) > -- > 2.15.1 > -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html