[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: feature/comp-use-ctors 1b48e8dde5: src/comp.c: Use constructor expre
From: |
Vibhav Pant |
Subject: |
Re: feature/comp-use-ctors 1b48e8dde5: src/comp.c: Use constructor expressions when possible. |
Date: |
Fri, 14 Oct 2022 18:28:39 +0530 |
User-agent: |
Evolution 3.44.4 |
On Fri, 2022-10-14 at 20:10 +0800, Po Lu wrote:
> Vibhav Pant <vibhavp@gmail.com> writes:
>
> > branch: feature/comp-use-ctors
> > commit 1b48e8dde5bfee65c036fbfdc32b68c9093c2e06
> > Author: Vibhav Pant <vibhavp@gmail.com>
> > Commit: Vibhav Pant <vibhavp@gmail.com>
> >
> > src/comp.c: Use constructor expressions when possible.
> >
> > * src/comp.c:
> > - Add declarations for creating constructor/initializer
> > expressions
> > when supported.
> > - (emit_coerce): Use a struct constructor expression to
> > create a
> > Lisp_Object value instead of creating a new local variable.
> > - emit_limple_call_ref: Emit a single constructor expression
> > for
> > initializing tmp_arr.
>
> This is not how we write commit messages; it should be:
>
> * src/comp.c: Add declarations for creating
> constructor/initializer expressions when supported.
> (emit_coerce): Use a struct constructor expression to create a
> Lisp_Object value instead of creating a new local variable.
> (emit_limple_call_ref): Emit a single constructor expression for
> initializing tmp_arr.
>
> Keep in mind that the commit message must also be filled to 64
> columns.
Ah, knew something about it was off. Thanks :)
--
Vibhav Pant
vibhavp@gmail.com
GPG: 7ED1 D48C 513C A024 BE3A 785F E3FB 28CB 6AB5 9598
signature.asc
Description: This is a digitally signed message part