Re: Patch for porting Elinks to OpenSolaris

Kalle Olavi Niemitalo <[email protected]> Thu, 19 Feb 2009 09:59:21 +0200
Newsgroups gmane.comp.web.links
Message-ID <[email protected]>
"Mark A. Carlson" <[email protected]> writes:

> +#if defined (__SUNPRO_C)
> +static inline void
> +#else
>  inline void
> +#endif
>  load_frames(struct session *ses, struct document_view *doc_view)

This looks like <http://bugzilla.elinks.cz/show_bug.cgi?id=1047>.
That bug was reported against ELinks 0.12pre1 though.
I haven't heard about such problems with 0.11.5 before.
It looks like 0.10.6 has the same bug; can you confirm?

Because ELinks is violating a C99 requirement, rather than
something specific to Sun, I don't think it's right to make an
exception for the Sun compiler.  Instead, we should make the
functions not inline and measure how that affects the speed.
If it hurts too much, then define e.g. static inline int
inline_elinks_ulongcat() and make both int elinks_ulongcat()
and int elinks_longcat() call that.  GCC should generate from
that approximately the same code as from the current sources.

_______________________________________________
elinks-dev mailing list
[email protected]
http://linuxfromscratch.org/mailman/listinfo/elinks-dev
signature.asc (application/pgp-signature, 188 B)
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)

iD8DBQFJnRFhHm9IGt60eMgRAldbAKCcguSrT/AMmBBCBgW8lOUlctVO/wCeLA2A
DTU4EcfcOr3VC3+xb85QQPY=
=s8uU
-----END PGP SIGNATURE-----