From: Po Lu <luangruo@yahoo.com>
To: Andrew De Angelis <bobodeangelis@gmail.com>
Cc: emacs-devel@gnu.org, Qiantan Hong <qthong@stanford.edu>
Subject: Re: Volunteering to help on etc/TODO item: Improved xwidgets support
Date: Wed, 19 Oct 2022 09:33:24 +0800 [thread overview]
Message-ID: <87k04wa97f.fsf@yahoo.com> (raw)
In-Reply-To: <CAP5CrM34=apHuad4Py9NywncmnHQJBH1OZekZ-NFsf_iESzfCA@mail.gmail.com> (Andrew De Angelis's message of "Tue, 18 Oct 2022 11:55:28 -0400")
Andrew De Angelis <bobodeangelis@gmail.com> writes:
> Hello everyone and thanks for all your work!
Hello, thanks for working on Emacs.
> This is regarding TODO item: "Things to be done for specific packages
> or features/NeXTstep port/Missing features/Improved xwidgets support"
>
> I've started working on the NS code for xwidget-webkit, with the aim
> of bringing it up-to-date with the changes to the X11 and GTK code
> (you can check my as-yet-still-very-minor changes at this fork). I'm
> sending the email to: 1) check if someone is already working on this
> 2) make sure I'm going about it the right way 3) inquire about the
> current X11/GTK implementation.
>
> Regarding 1: I haven't found many recent matches for 'xwidget' in the
> mailing list, but if you're aware of someone already working on this
> effort, please let me know
Sure. It would be good if you coordinated your efforts with Qiantian
Hong (copied), who is also making changes in that area.
> Regarding 2: As noted in the Contributing node of the manual, I'm
> making you aware of my planned improvements and I'd like to know if
> you have any suggestions/advice. My current plan is to go through the
> xwidget.c code, take note of any functions/subroutines that are
> defined for GTK but not NS, and add an NS implementation in xwidget.m.
Thanks. It would be nice if someone resolved the more crucial issues
first, though:
- many procedures in nsxwidget.m crash when encountering killed
xwidgets (see the doc string of `kill-xwidget' for more details.)
- nsxwidget.m has apparently been written with Objective-C Automatic
Reference Counting in mind, and thus leak memory, as Emacs cannot
use ARC.
> I will do my best to complete this so that the NS code will be fully
> up-to-date. If there are any planned changes to xwidget.c or
> xwidget.el for the upcoming 29.1 release, please let me know.
I think it will probably be too late for Emacs 29.1, which will start
the pre-release process soon, at which point changes that don't only fix
regressions (and possibly the MS-DOS build) will not be allowed. Unless
you can complete this work before November, that is.
> Regarding 3: I do not have a Linux machine available at the moment,
> which would be valuable to get a better sense of the current GTK
> implementation (I'm working on finding additional volunteers to help
> on this). Is there a standard-procedure I can follow to ask questions
> here about the GTK implementation?
Just send an email to this list, with me copied in. BTW, the canonical
implementation is not a "GTK implementation", but rather two
implementations combined in a single file through ifdefs: one for the
regular X11 build, and the other for a GTK build that does not use any X
Windows specific interfaces.
> Is there a point person I should contact specifically?
Me.
> I would like to keep the two different implementations as consistent
> as possible, while also making sure that common bugs are addressed.
> One question I have regarding this is on the
> `xwidget-webkit--loading-p' variable: in my build, I see that this is
> set to true when creating a new session, but it is then never updated
> to nil (even long after the web page has fully loaded). Since this
> variable is not present in the C code, I'm not sure if this is a
> limitation of the Lisp code (and therefore common regardless of the
> underlying framework, GTK or NS), or if it's handled correctly in
> other builds.
I think the timer stops by itself, the variable is not all that
important.
next prev parent reply other threads:[~2022-10-19 1:33 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-18 15:55 Volunteering to help on etc/TODO item: Improved xwidgets support Andrew De Angelis
2022-10-19 1:33 ` Po Lu [this message]
2022-10-19 3:43 ` Andrew De Angelis
2022-10-19 4:45 ` Po Lu
2022-10-19 4:55 ` Andrew De Angelis
2022-10-19 5:11 ` Po Lu
2022-10-19 18:54 ` Qiantan Hong
2022-10-20 2:23 ` Andrew De Angelis
2022-10-20 2:46 ` Po Lu
2022-10-20 11:13 ` Daniel Martín
2022-10-21 23:29 ` Andrew De Angelis
2022-10-22 0:07 ` Po Lu
2022-10-22 2:13 ` Andrew De Angelis
2022-10-22 4:49 ` Matt Armstrong
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87k04wa97f.fsf@yahoo.com \
--to=luangruo@yahoo.com \
--cc=bobodeangelis@gmail.com \
--cc=emacs-devel@gnu.org \
--cc=qthong@stanford.edu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.