-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Now is the priority which should be immediately printed as referenced
by the comment. Seems perfectly sensible, especially given that the
other items might be stored later.

On 03/24/2014 11:43 AM, DavidVorick wrote:
> In src/common/log/prioritylog.go:
> 
>> +type PriorityLog struct { +	// now control the levels of
>> priority which are immediately printed +	// and stored to print
>> in case of an error. +	dispose bool +	now     uint +	records
>> *recordHeap +	out     io.Writer +	// lock protects the above
>> fields +	lock sync.Mutex +	// stop stops creation of new log
>> entries. +	// THere are two mutexes solely to prevent deadlocks. 
>> +	stop sync.Mutex +	// Every logging function counts as two in
>> pending +	// The first for formatting the message, the second for
>> creating the record +	pending sync.WaitGroup +}
> 
> This could also be organized a bit better. |now| is a confusing
> variable name. Stop and pending should probably be isolated by a
> newline.
> 
> It just helps to make the code more readable.
> 
> — Reply to this email directly or view it on GitHub 
> <https://github.com/Siacoin/Sia/pull/48/files#r10890091>.
> 

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.22 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQEcBAEBAgAGBQJTMF1NAAoJEHP1nN3uZoHusPIIANljoNXvtH3GC/KOJzBUUBAS
bjgkzxmnril6LYkmiFqxMLlw/og7R2y32hTzizBFIsGBvwfT6gKMYWXyVuebZHqT
iFYit3+mKyB5Gmlw1LoZleHXL8TVjFzrcOwBAEKR0Zi+Cgu/p1J0HPRwR7TL1jPD
bGJU5gC6JavgYH2Dch95nICuML43sNZWTyab7bboGjsjYGm2f8S9BFzhNtLiDmKY
ur32RtxxkMcKPRD4g1ERUAfkE17ES5a3zbSAg9JyqUFXj4hG46JnBK74gSThEHKa
sndG9wmq+CiZy7yFm0sBtpbaW/1HS2SIwE5pc3jV5movfdSqpGi6eIq2o54CGWo=
=M60k
-----END PGP SIGNATURE-----