Rewrite WINGs wfont.c in Rust #9

2025-11-23T16:28:56-05:00

trurl commented

2025-11-23 16:28:56 -05:00

This is a step towards getting rid of all uses of WMHashTable and, more importantly, a first crack at rewriting WINGs itself.

trurl added 12 commits 2025-11-23 16:28:58 -05:00

Add a wings-rs crate for porting WINGs proper to Rust. 50caed30c0

Remove unused function WMIsAntialiasingEnabled. 897233cb1d

Remove unused function WMCopyFontWithStyle. b0d49171eb

Remove unused functions WMSetWidgetDefaultFont, WMSetWidgetDefaultBoldFont. 72181e4345

Use bindgen to generate Rust bindings for WINGs types. ad6df25960

We will use these to port W_Font, W_Screen, etc., to Rust.

Bindings are regenerated whenever the relevant C source files change and then
patched to make sure they will build. This is a bit hacky but will serve our
purposes: the struct definitions should be migrated to Rust sooner rather than
later, so the hackishness should be transient.

Mark wings-rs as a staticlib. db94395ecc

Put the font and y (ascent) fields of W_Font behind accessors. 8457fd61f0

Tweak build so that WINGsP.rs gets generated at the right time. dc0179fe24

Ignore target dir for nested Cargo subdirs, too. 50dad6d86e

Fix build order issues by moving wings-rs/ into WINGs/. 6a18ba06d5

Having wings-rs depend on WINGs/WINGs/WINGsP.h was not working very well if all
of WINGs/ had to be built before wings-rs/. Moving wings-rs/ into WINGs/
addresses this.

Make WINGs configuration visible from Rust. f2f961ff1e

Rewrite all functions from WINGs wfont.c in Rust. abf49492c8

This starts to establish the structure for an in-place rewrite of WINGs. An
actual redesign of WINGs may follow, but for now there are no plans to alter the
structure of WINGs substantially.

trurl requested review from cross 2025-11-23 16:29:36 -05:00

cross reviewed 2025-12-12 07:23:08 -05:00

WINGs/wings-rs/src/font.rs

						
				@@ -0,0 +41,4 @@

				    NonNull::new(unsafe {

				        x11::xft::XftXlfdParse(

				            name.as_ptr(),

				            /*ignore_scalable=*/ 0,

cross commented

2025-12-12 07:23:08 -05:00

Instead of the comments, why not give these proper names via let bindings?

    let ignore_scalable = 0;
    let complete = 0;
    unsafe { x11::xft::XftXlfdParse(name.as_ptr(), ignore_scalable, complete) }
    ...

Instead of the comments, why not give these proper names via `let` bindings? ```rust let ignore_scalable = 0; let complete = 0; unsafe { x11::xft::XftXlfdParse(name.as_ptr(), ignore_scalable, complete) } ... ```

trurl commented

2025-12-15 12:33:50 -05:00

I like the Google C++ style guide's prescription on this, but we don't have a lint, so... sure.

trurl marked this conversation as resolved

cross approved these changes 2025-12-12 07:28:39 -05:00

cross left a comment

This is certainly an improvement! I'm a bit surprised there aren't crates providing better bindings to both Xft and pango, but that's a separate matter.

trurl commented

2025-12-12 16:54:15 -05:00

There are crates that provide pango bindings, but there's some weird provenance (https://github.com/yeslogic/fontconfig-rs/issues/52), and nothing seems like a drop-dead obvious thing to drop into the existing WINGs API.

Switching to a dedicated Xft/Pango crate, or something else entirely for managing fonts and drawing to the screen (to provide independence from X11), is definitely in the cards.

There are crates that provide pango bindings, but there's some weird provenance (https://github.com/yeslogic/fontconfig-rs/issues/52), and nothing seems like a drop-dead obvious thing to drop into the existing WINGs API. Switching to a dedicated Xft/Pango crate, or something else entirely for managing fonts and drawing to the screen (to provide independence from X11), is definitely in the cards.

trurl added 2 commits 2025-12-15 12:35:48 -05:00

Add new Rust source files to Makefile.am. 3a9ea4f0c7

Forgot to do this earlier.

Name arguments directly instead of aping the Google C++ style guide. 9444c56e15

trurl merged commit 9444c56e15 into refactor/wings-rs

2025-12-15 12:36:08 -05:00

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: vitrine/wmaker#9

Allow edits from maintainers