ruuda / pris

A language for designing slides

Home Page:https://docs.ruuda.nl/pris/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Unable to build on 32 bit platforms

FortressBuilder opened this issue · comments

At the moment pris fails to build on 32 bit platforms. The problem seems to be in src/cairo.rs, line 231:

Glyph::new(self.0.index, self.0.x + dx, self.0.y + dy)

Glyph::new takes a u64 as the first parameter, but self.0.index is a c_ulong, which is a u32 on 32 bit platforms and thus cannot be implicitly converted into a u64. The solution is to explicitly cast it, which should work on both 32 and 64 platforms:

Glyph::new(self.0.index as u64, self.0.x + dx, self.0.y + dy)

First of all, thank you for taking the time to try Pris and to file an issue. Apologies for the late reply, I somehow missed this issue.

I’ve pushed a fix in 1f651a7. Does that solve the issue for you?

Yes, thank you :-)