Re: [PATCH 3/3] btrfs-progs: remove meaningless debug info for restore

2014-08-31 Thread Gui Hecheng
On Fri, 2014-08-29 at 16:49 +0200, David Sterba wrote: > On Thu, Aug 28, 2014 at 10:25:55AM +0800, Gui Hecheng wrote: > > The printf of @offset enlightens the user little. > > And the restore cmd is not a debugging tool, so just > > remove the debug-info-like printf. > > I'd like to let the restor

Re: [PATCH 3/3] btrfs-progs: remove meaningless debug info for restore

2014-08-29 Thread David Sterba
On Thu, Aug 28, 2014 at 10:25:55AM +0800, Gui Hecheng wrote: > The printf of @offset enlightens the user little. > And the restore cmd is not a debugging tool, so just > remove the debug-info-like printf. I'd like to let the restore command be more verbose as it's potentially working with a broken

[PATCH 3/3] btrfs-progs: remove meaningless debug info for restore

2014-08-27 Thread Gui Hecheng
The printf of @offset enlightens the user little. And the restore cmd is not a debugging tool, so just remove the debug-info-like printf. Signed-off-by: Gui Hecheng --- cmds-restore.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/cmds-restore.c b/cmds-restore.c index 918f1fb..648fbc9 1006