fixes from review

types/part_set.go

@@ -267,7 +267,7 @@ func (ps *PartSet) StringShort() string {
 
 func (ps *PartSet) MarshalJSON() ([]byte, error) {
 	if ps == nil {
-		return []byte("nil-PartSet"), nil
+		return []byte("{}"), nil
 	}
 
 	ps.mtx.Lock()
@@ -278,6 +278,6 @@ func (ps *PartSet) MarshalJSON() ([]byte, error) {
 		PartsBitArray *cmn.BitArray `json:"parts_bit_array"`
 	}{
 		fmt.Sprintf("%d/%d", ps.Count(), ps.Total()),
-		ps.partsBitArray.Copy(),
+		ps.partsBitArray,
 	})
 }

types/vote_set.go

@@ -56,9 +56,9 @@ type VoteSet struct {
 	height  int64
 	round   int
 	type_   byte
+	valSet  *ValidatorSet
 
 	mtx           sync.Mutex
-	valSet        *ValidatorSet
 	votesBitArray *cmn.BitArray
 	votes         []*Vote                // Primary votes to share
 	sum           int64                  // Sum of voting power for seen votes, discounting conflicts
@@ -399,6 +399,8 @@ func (voteSet *VoteSet) HasTwoThirdsAny() bool {
 }
 
 func (voteSet *VoteSet) HasAll() bool {
+	voteSet.mtx.Lock()
+	defer voteSet.mtx.Unlock()
 	return voteSet.sum == voteSet.valSet.TotalVotingPower()
 }
 
@@ -424,6 +426,8 @@ func (voteSet *VoteSet) String() string {
 }
 
 func (voteSet *VoteSet) StringIndented(indent string) string {
+	voteSet.mtx.Lock()
+	defer voteSet.mtx.Unlock()
 	voteStrings := make([]string, len(voteSet.votes))
 	for i, vote := range voteSet.votes {
 		if vote == nil {
@@ -448,6 +452,8 @@ func (voteSet *VoteSet) StringIndented(indent string) string {
 // Marshal the VoteSet to JSON. Same as String(), just in JSON,
 // and without the height/round/type_ (since its already included in the votes).
 func (voteSet *VoteSet) MarshalJSON() ([]byte, error) {
+	voteSet.mtx.Lock()
+	defer voteSet.mtx.Unlock()
 	voteStrings := make([]string, len(voteSet.votes))
 	for i, vote := range voteSet.votes {
 		if vote == nil {