On 20 Feb 2003, Jean-Marc Lasgouttes wrote:

> 
> I cannot comment on the content of the patch itself, but I have two
> observation on its form.
> 
> 1/ the header forms.h is AFAIK automatically generated from the
> various header files in lib/include. So you should not modify this
> files directly, but change the relevant sub-header file. I would think
> that it should be Basic.h, but I am not sure.
> 
> 2/ the following part is not good, because (1) // is not a C comment
> and (2) you should remove the line completely instead.
> 
> --- xforms-1.0-release/lib/flinternal.h       2002-05-21 04:38:39.000000000 +0900
> +++ xforms-1.0-i18n/lib/flinternal.h  2003-02-20 21:42:27.000000000 +0900
> @@ -411,10 +411,10 @@
>      int tooltip_time;
>  #ifdef XlibSpecificationRelease
>      XIM    xim ;                /* input method           */
> -    XIC    xic ;                /* input context          */
> +//    XIC    xic ;                /* input context          */
>  #else
>      void  *xim;
> -    void  *xic;
> +//    void  *xic;
>  #endif
>      unsigned int  navigate_mask;       /* input field           */
>      long reserverd[6];
> 
> Hope this helps.


Thank you for the corrections. You are helping!


cghan 

Reply via email to