Re: [Nfs-ganesha-devel] Change in ffilz/nfs-ganesha[next]: Remove unused fsal_read2() and fsal_write2()

2018-02-23 Thread William Allen Simpson

On 2/22/18 1:32 PM, GerritHub wrote:

Daniel Gryniewicz has uploaded this change for *review*.

View Change 

Remove unused fsal_read2() and fsal_write2()


I've reviewed, but the write showed up in my inbox before the read,
followed by this cleanup.  But the write changed things added by
the read changes.  And most of this cleanup was added by one of the
other patches.  Stuff appearing and then disappearing was confusing
to me.  Probably others, too.

So I've recommended squashing the 3 patches.  Will make future
bi-section easier.  Not that we'll ever need it, as otherwise
this seems like good work.

--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel


[Nfs-ganesha-devel] Change in ffilz/nfs-ganesha[next]: Remove unused fsal_read2() and fsal_write2()

2018-02-22 Thread GerritHub
>From Daniel Gryniewicz :

Daniel Gryniewicz has uploaded this change for review. ( 
https://review.gerrithub.io/401041


Change subject: Remove unused fsal_read2() and fsal_write2()
..

Remove unused fsal_read2() and fsal_write2()

Change-Id: I6bf6495190eadb3f0d0dcf8ff74823fb0235e18b
Signed-off-by: Daniel Gryniewicz 
---
M src/FSAL/fsal_helper.c
M src/include/fsal.h
2 files changed, 0 insertions(+), 205 deletions(-)



  git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha 
refs/changes/41/401041/1
--
To view, visit https://review.gerrithub.io/401041
To unsubscribe, visit https://review.gerrithub.io/settings

Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-MessageType: newchange
Gerrit-Change-Id: I6bf6495190eadb3f0d0dcf8ff74823fb0235e18b
Gerrit-Change-Number: 401041
Gerrit-PatchSet: 1
Gerrit-Owner: Daniel Gryniewicz 
--
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot___
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel