emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] ash, lsh: Avoid code duplication


From: Eli Zaretskii
Subject: Re: [PATCH] ash, lsh: Avoid code duplication
Date: Tue, 22 Nov 2016 18:12:08 +0200

> From: Tino Calancha <address@hidden>
> CC: address@hidden, Tino Calancha <address@hidden>
> Date: Tue, 22 Nov 2016 11:17:24 +0900
> 
> Eli Zaretskii <address@hidden> writes:
> 
> >> +Lisp_Object
> >> +ash_lsh_impl (register Lisp_Object value, Lisp_Object count, bool lsh)
> >
> > static Lisp_Object
> > ash_lsh_impl (register Lisp_Object value, Lisp_Object count, bool lsh)
> >
> >> diff --git a/src/lisp.h b/src/lisp.h
> >> index e087828..c48c2c8 100644
> >> --- a/src/lisp.h
> >> +++ b/src/lisp.h
> >> @@ -602,6 +602,7 @@ extern void char_table_set (Lisp_Object, int, 
> >> Lisp_Object);
> >>  /* Defined in data.c.  */
> >>  extern _Noreturn Lisp_Object wrong_type_argument (Lisp_Object, 
> >> Lisp_Object);
> >>  extern _Noreturn void wrong_choice (Lisp_Object, Lisp_Object);
> >> +Lisp_Object ash_lsh_impl (Lisp_Object, Lisp_Object, bool);
> >
> > No need to declare it here, as it is a static function used only in
> > the file in which it is defined.
> 
> Thank you very much.
> Here is the updated patch:

Thanks, this LGTM.  Please push to master if no further comments are
posted.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]