[Piglit] [PATCH 1/2] general: Correct some @file names.

Matt Turner mattst88 at gmail.com
Thu Sep 1 17:54:41 UTC 2016


On Thu, Sep 1, 2016 at 12:01 AM, Pekka Paalanen <ppaalanen at gmail.com> wrote:
> On Wed, 31 Aug 2016 13:05:31 -0700
> Matt Turner <mattst88 at gmail.com> wrote:
>
>> ---
>>  tests/general/scissor-depth-clear.c | 2 +-
>>  tests/general/scissor-many.c        | 2 +-
>>  2 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/tests/general/scissor-depth-clear.c b/tests/general/scissor-depth-clear.c
>> index 9832270..b07e2e8 100644
>> --- a/tests/general/scissor-depth-clear.c
>> +++ b/tests/general/scissor-depth-clear.c
>> @@ -25,7 +25,7 @@
>>   *
>>   */
>>
>> -/** @file scissor-copypixels.c
>> +/** @file scissor-depth-clear.c
>>   *
>>   * Tests that glScissor properly affects glClear(GL_DEPTH_BUFFER_BIT).
>>   */
>> diff --git a/tests/general/scissor-many.c b/tests/general/scissor-many.c
>> index 17eb232..9892b6f 100644
>> --- a/tests/general/scissor-many.c
>> +++ b/tests/general/scissor-many.c
>> @@ -25,7 +25,7 @@
>>   *
>>   */
>>
>> -/** @file many-scissors.c
>> +/** @file scissors-many.c
>>   *
>>   * Tests drawing to each individual pixel in the drawable using glScissor.
>>   *
>
> Hi,
>
> why not simply remove the file name from the @file command?
>
> "If the file name is omitted (i.e. the line after \file is left blank)
> then the documentation block that contains the \file command will
> belong to the file it is located in."

Thanks. I wasn't aware of that. I'll send a new patch.


More information about the Piglit mailing list