allanjude commented on this pull request.


> @@ -653,43 +654,28 @@ zfs_read(vnode_t *vp, uio_t *uio, int ioflag, cred_t 
> *cr, caller_context_t *ct)
 static int
 zfs_write(vnode_t *vp, uio_t *uio, int ioflag, cred_t *cr, caller_context_t 
*ct)
 {
-       znode_t         *zp = VTOZ(vp);

Why are you changing the style of the variable declarations here?

> @@ -1217,18 +1352,10 @@ zvol_dumpio(zvol_state_t *zv, void *addr, uint64_t 
> offset, uint64_t size,
 int
 zvol_strategy(buf_t *bp)
 {
-       zfs_soft_state_t *zs = NULL;

Again the style of the variable declarations seems to change for no reason.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openzfs/openzfs/pull/664#pullrequestreview-137245466
------------------------------------------
openzfs: openzfs-developer
Permalink: 
https://openzfs.topicbox.com/groups/developer/Ta167a88bbab5c009-M54f1a841d1fc828b6010c049
Delivery options: https://openzfs.topicbox.com/groups

Reply via email to