From: Jie Liu <[email protected]> Simplify the code in lmNextPage via list_last_entry_or_null().
Signed-off-by: Jie Liu <[email protected]> --- fs/jfs/jfs_logmgr.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/fs/jfs/jfs_logmgr.c b/fs/jfs/jfs_logmgr.c index 360d27c..7b565d0 100644 --- a/fs/jfs/jfs_logmgr.c +++ b/fs/jfs/jfs_logmgr.c @@ -591,10 +591,7 @@ static int lmNextPage(struct jfs_log * log) * write or queue the full page at the tail of write queue */ /* get the tail tblk on commit queue */ - if (list_empty(&log->cqueue)) - tblk = NULL; - else - tblk = list_entry(log->cqueue.prev, struct tblock, cqueue); + tblk = list_last_entry_or_null(&log->cqueue, struct tblock, cqueue); /* every tblk who has COMMIT record on the current page, * and has not been committed, must be on commit queue -- 1.8.3.2 ------------------------------------------------------------------------------ DreamFactory - Open Source REST & JSON Services for HTML5 & Native Apps OAuth, Users, Roles, SQL, NoSQL, BLOB Storage and External API Access Free app hosting. Or install the open source package on any LAMP server. Sign up and see examples for AngularJS, jQuery, Sencha Touch and Native! http://pubads.g.doubleclick.net/gampad/clk?id=63469471&iu=/4140/ostg.clktrk _______________________________________________ Jfs-discussion mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/jfs-discussion
