Minor formatting and Remove unused unlocked_restoreItem()

Change-Id: I00f1e7a238176d9983290905b091ebb53177fc4c
Reviewed-on: http://review.couchbase.org/29153
Reviewed-by: Sundararaman Sridharan <sundar.sridharan@gmail.com>
Tested-by: Sundararaman Sridharan <sundar.sridharan@gmail.com>
Remove unused mutex instance.

Change-Id: I90e33818b79355e47343f5134b93575a1a5aa75c
Reviewed-on: http://review.couchbase.org/29154
Reviewed-by: Sundararaman Sridharan <sundar.sridharan@gmail.com>
Tested-by: Sundararaman Sridharan <sundar.sridharan@gmail.com>
Remove unnecessary task mutex from BGFetcher.

Acccessing the GIO APIs from the BGFetcher is already
synchronized by the lock inside the GIO.

Change-Id: I5309bc152aa52aa731cfc392df773bbaf5ef2a7b
Reviewed-on: http://review.couchbase.org/29180
Reviewed-by: Sundararaman Sridharan <sundar.sridharan@gmail.com>
Tested-by: Sundararaman Sridharan <sundar.sridharan@gmail.com>
Fix for the small error handling problem

when handleResponse is called, currentCommand is already 0xff

Change-Id: I090b1f9787935e280b88e832aed3e885f4f27c0d
Reviewed-on: http://review.couchbase.org/30225
Tested-by: Artem Stemkovski <artem@couchbase.com>
Reviewed-by: Chiyoung Seo <chiyoung@couchbase.com>
Tested-by: Chiyoung Seo <chiyoung@couchbase.com>