[Mesa-dev] [PATCH 1/2] nir: some comment fixups

[Rob Clark]

On Thu, Sep 17, 2015 at 8:57 PM, Ilia Mirkin <imirkin at alum.mit.edu> wrote:
> On Thu, Sep 17, 2015 at 8:48 PM, Rob Clark <robdclark at gmail.com> wrote:
>> On Thu, Sep 17, 2015 at 8:35 PM, Dieter Nützel <Dieter at nuetzel-hh.de> wrote:
>>> Am 18.09.2015 02:27, schrieb Rob Clark:
>>>>
>>>> On Thu, Sep 17, 2015 at 8:03 PM, Timothy Arceri <t_arceri at yahoo.com.au>
>>>> wrote:
>>>>>
>>>>> On Thu, 2015-09-17 at 18:20 -0400, Rob Clark wrote:
>>>>>> -   /** list of system value variables in the shader */
>>>>>> +   /** list of system value variables in the shade (nir_variable)r
>>>>>
>>>>>
>>>>>
>>>>> The above comment is broken. With that fixed both patches are:
>>>>>
>>>>> Reviewed-by: Timothy Arceri <t_arceri at yahoo.com.au>
>>>>
>>>>
>>>> oh, whoops.. thanks, fixed and pushed
>>>>
>>>> BR,
>>>> -R
>>>
>>>
>>> Rob,
>>>
>>> you committed the broken one...
>>
>> are you sure?
>>
>> http://cgit.freedesktop.org/mesa/mesa/commit/?id=ba78260b0fbd291618df63bc7a54be0a36148b40
>>
>> looks ok to me.. I assume 'broken' == the stray character that snuck
>> in there but maybe I am missing something..  (it is quite possible
>> that I am overdue for a new prescription of eye-glasses :-P)
>
> Probably just need to wait out those hot autumn weeks until you're no
> longer thinking of shade all the time...

heh, ok.. I guess time for a larger font size :-P

>   -ilia