Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GString support #257

Merged
merged 2 commits into from Dec 18, 2018

Conversation

Projects
None yet
3 participants
@philn
Copy link
Contributor

philn commented Dec 16, 2018

No description provided.

@EPashkin

This comment has been minimized.

Copy link
Member

EPashkin commented Dec 16, 2018

Also problem on 1.28.0

  --> D:/eap/rust/0/gdk\src\atom.rs:38:26
   |
38 |     pub fn name(self) -> glib::GString {
   |                          ^^^^

error[E0658]: access to extern crates through prelude is experimental (see issue #44660)
  --> D:/eap/rust/0/gdk\src\event_setting.rs:15:38
   |
15 |     pub fn get_name(&self) -> Option<glib::GString> {
   |                                      ^^^^

error[E0658]: access to extern crates through prelude is experimental (see issue #44660)
  --> D:/eap/rust/0/gdk\src\keys.rs:10:43
   |
10 | pub fn keyval_name(keyval: u32) -> Option<glib::GString> {
   |                                           ^^^^

warning: unused import: `gobject_ffi`
  --> D:/eap/rust/0/gdk\src\event.rs:10:5
   |
10 | use gobject_ffi;
   |     ^^^^^^^^^^^
   |
   = note: #[warn(unused_imports)] on by default

@philn philn force-pushed the philn:master branch from 17ba9ef to d7a9a6c Dec 16, 2018

@philn

This comment has been minimized.

Copy link
Contributor Author

philn commented Dec 18, 2018

@EPashkin @GuillaumeGomez can you re-run CI for this?

@EPashkin

This comment has been minimized.

Copy link
Member

EPashkin commented Dec 18, 2018

Thanks and 👍

@GuillaumeGomez

This comment has been minimized.

Copy link
Member

GuillaumeGomez commented Dec 18, 2018

Thanks!

@GuillaumeGomez GuillaumeGomez merged commit ac9aafa into gtk-rs:master Dec 18, 2018

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.