[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Clean up embedded scheme parsing/evaluation. (issue 577410045 by add
From: |
Dan Eble |
Subject: |
Re: Clean up embedded scheme parsing/evaluation. (issue 577410045 by address@hidden) |
Date: |
Fri, 31 Jan 2020 13:43:30 -0500 |
On Jan 31, 2020, at 13:33, address@hidden wrote:
>
> On 2020/01/31 18:22:47, Dan Eble wrote:
>> On 2020/01/31 17:52:45, hanwenn wrote:
>>> you can do a local alias
>>>
>>> vector<> &v = *vec;
>>>
>>> to aid readability.
>>
>> The more I think about banning non-const reference parameters, the
> more I'm
>> against it. Google's coding standards may work for them, but their
> rationale*
>> for this one is weak. How can we resolve this disagreement quickly?
> Do you
>> simply have the final say as the project founder?
>
> Can we have this discussion on a thread separate from this code review?
> I want this code to go in.
Well, you can't prevent people from replying to their email, but neither can
their replies prevent you from pushing commits.
—
Dan
- Re: Clean up embedded scheme parsing/evaluation. (issue 577410045 by address@hidden), (continued)
Re: Clean up embedded scheme parsing/evaluation. (issue 577410045 by address@hidden), benko . pal, 2020/01/31
Re: Clean up embedded scheme parsing/evaluation. (issue 577410045 by address@hidden), hanwenn, 2020/01/31
Re: Clean up embedded scheme parsing/evaluation. (issue 577410045 by address@hidden), nine . fierce . ballads, 2020/01/31
Re: Clean up embedded scheme parsing/evaluation. (issue 577410045 by address@hidden), hanwenn, 2020/01/31
Re: Clean up embedded scheme parsing/evaluation. (issue 577410045 by address@hidden), nine . fierce . ballads, 2020/01/31
Re: Clean up embedded scheme parsing/evaluation. (issue 577410045 by address@hidden), hanwenn, 2020/01/31
- Re: Clean up embedded scheme parsing/evaluation. (issue 577410045 by address@hidden),
Dan Eble <=
Re: Clean up embedded scheme parsing/evaluation. (issue 577410045 by address@hidden), Carl . D . Sorensen, 2020/01/31