Re: [boost] Boost.Local Review (Nov 10, 2011 to Nov 19, 2011)

Subject: Re: [boost] Boost.Local Review (Nov 10, 2011 to Nov 19, 2011)
From: Steven Watanabe (watanabesj_at_[hidden])
Date: 2011-11-14 21:01:01

AMDG

On 11/14/2011 04:59 PM, Lorenzo Caminiti wrote:
> 
> In this email thread:
> http://lists.boost.org/Archives/boost/2011/04/179834.php
> 
> Three things where pointed out:
> 1) _ prefix is for placeholders.
> 2) _ postfix is for keywords.
> 3) Boost.Phoenix already uses _this (as a placeholder).
> 
> In the Boost.Local case _this/this_ is both:
> 1) A placeholders because it will be "replaced" with the actual object
> pointer `this`.
> 2) And obviously a keyword.
> So what do I do? _this or this_?
> 

It should be this_.  The way you're using it
is not what I would consider a placeholder.
If that were the definition of a placeholder
than every reference should be considered a
placeholder because the reference gets
"replaced" with the actual object, which
is absurd.

> I might as well be consistent with another Boost library. Plus, if
> _this is also used in the function declaration (not just the
> definition) then it really acts as a placeholder. I am only 52% in
> favor of _this over this_ and I am happy to use the name that the
> reviewers find more correct.
> 
> BTW, I couldn't find Phoenix docs that refer to _this-- does anyone
> have a link? Is there any other Boost library that uses either _this
> or this_?
> 

In Christ,
Steven Watanabe