Do some small cleanup in IceTLS.h and add some markup for doxygen

Do some small cleanup in IceTLS.h and add some markup for doxygen to it.

BUG=
R=jpp@chromium.org, stichnot@chromium.org

Committed: https://gerrit.chromium.org/gerrit/gitweb?p=native_client/pnacl-subzero.git;a=commit;h=af5d44c6560e3fcd9ccb9c1f094369b05fcb28ef

[rkotlerimgtec, 2015-11-09 22:06:00]

Description was changed from

==========
Do some small cleanup in IceTLS.h and add some markup for doxygen
in it.

BUG=
==========

to

==========
Do some small cleanup in IceTLS.h and add some markup for doxygen to it.

BUG=
==========

[rkotlerimgtec, 2015-11-09 22:06:01]

rkotlerimgtec@gmail.com changed reviewers:
+ jpp@chromium.org, karlschimpf@gmail.com, stichnot@chromium.org

[John, 2015-11-09 22:36:58]

lgtm

[rkotlerimgtec, 2015-11-09 22:47:52]

I am not able to check code in so someone else will need to manually do
that for me.

Tia.

Reed

On Mon, Nov 9, 2015 at 2:36 PM, <jpp@chromium.org> wrote:

> lgtm
>
>
>
> https://codereview.chromium.org/1430273004/
>

-- 
You received this message because you are subscribed to the Google Groups
"Native-Client-Reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to native-client-reviews+unsubscribe@googlegroups.com.
To post to this group, send email to native-client-reviews@googlegroups.com.
Visit this group at http://groups.google.com/group/native-client-reviews.
For more options, visit https://groups.google.com/d/optout.

[Jim Stichnoth, 2015-11-09 22:49:52]

https://codereview.chromium.org/1430273004/diff/1/src/IceTLS.h
File src/IceTLS.h (right):

https://codereview.chromium.org/1430273004/diff/1/src/IceTLS.h#newcode22
src/IceTLS.h:22: /// @defgroup /IceTLC Defines 5 macros for unifying
thread_local and pthread:
IceTLC?  Should this be IceTLS?

https://codereview.chromium.org/1430273004/diff/1/src/IceTLS.h#newcode26
src/IceTLS.h:26: /// Declare a static thread_local field
Reflow the remainders of all these comments to 80-columns.

[Jim Stichnoth, 2015-11-09 22:52:34]

stichnot@chromium.org changed reviewers:
+ kschimpf@google.com
- karlschimpf@gmail.com

[Jim Stichnoth, 2015-11-09 22:53:04]

Changed the reviewer list to use the right email for Karl.

[rkotlerimgtec, 2015-11-09 23:59:50]

https://codereview.chromium.org/1430273004/diff/1/src/IceTLS.h
File src/IceTLS.h (right):

https://codereview.chromium.org/1430273004/diff/1/src/IceTLS.h#newcode22
src/IceTLS.h:22: /// @defgroup /IceTLC Defines 5 macros for unifying
thread_local and pthread:
On 2015/11/09 22:49:52, stichnot wrote:
> IceTLC?  Should this be IceTLS?

Done.

https://codereview.chromium.org/1430273004/diff/1/src/IceTLS.h#newcode26
src/IceTLS.h:26: /// Declare a static thread_local field
On 2015/11/09 22:49:52, stichnot wrote:
> Reflow the remainders of all these comments to 80-columns.

Done.

[Jim Stichnoth, 2015-11-10 00:01:46]

lgtm

[Jim Stichnoth, 2015-11-10 00:53:01]

Committed patchset #2 (id:20001) manually as
af5d44c6560e3fcd9ccb9c1f094369b05fcb28ef (presubmit successful).

[rkotlerimgtec, 2015-11-10 22:33:32]

Patchset #3 (id:40001) has been deleted