Re: [PATCH v3 1/4] btrfs-progs: utils: Introduce function to escape characters

2016-11-02 Thread Qu Wenruo
At 11/02/2016 06:55 PM, David Sterba wrote: On Wed, Nov 02, 2016 at 09:19:10AM +0800, Qu Wenruo wrote: At 11/01/2016 06:08 PM, David Sterba wrote: On Tue, Nov 01, 2016 at 04:01:43PM +0800, Qu Wenruo wrote: Introduce new function, escape_string_inplace(), to escape specified characters in

Re: [PATCH v3 1/4] btrfs-progs: utils: Introduce function to escape characters

2016-11-02 Thread Qu Wenruo
At 11/02/2016 06:55 PM, David Sterba wrote: On Wed, Nov 02, 2016 at 09:19:10AM +0800, Qu Wenruo wrote: At 11/01/2016 06:08 PM, David Sterba wrote: On Tue, Nov 01, 2016 at 04:01:43PM +0800, Qu Wenruo wrote: Introduce new function, escape_string_inplace(), to escape specified characters in

Re: [PATCH v3 1/4] btrfs-progs: utils: Introduce function to escape characters

2016-11-02 Thread David Sterba
On Wed, Nov 02, 2016 at 09:19:10AM +0800, Qu Wenruo wrote: > > > At 11/01/2016 06:08 PM, David Sterba wrote: > > On Tue, Nov 01, 2016 at 04:01:43PM +0800, Qu Wenruo wrote: > >> Introduce new function, escape_string_inplace(), to escape specified > >> characters in place. > > > > Sorry, the

Re: [PATCH v3 1/4] btrfs-progs: utils: Introduce function to escape characters

2016-11-01 Thread Qu Wenruo
At 11/01/2016 06:08 PM, David Sterba wrote: On Tue, Nov 01, 2016 at 04:01:43PM +0800, Qu Wenruo wrote: Introduce new function, escape_string_inplace(), to escape specified characters in place. Sorry, the pointer to seq_path was misleading. The actual escape function is mangle_path and it

Re: [PATCH v3 1/4] btrfs-progs: utils: Introduce function to escape characters

2016-11-01 Thread David Sterba
On Tue, Nov 01, 2016 at 04:01:43PM +0800, Qu Wenruo wrote: > Introduce new function, escape_string_inplace(), to escape specified > characters in place. Sorry, the pointer to seq_path was misleading. The actual escape function is mangle_path and it copies one string to another. As we just print