Skip to content
This repository has been archived by the owner on Jun 8, 2021. It is now read-only.

Deignoring #319

Merged
merged 5 commits into from
Apr 25, 2016
Merged

Deignoring #319

merged 5 commits into from
Apr 25, 2016

Conversation

EPashkin
Copy link
Member

No description provided.

@GuillaumeGomez
Copy link
Member

Great, thanks!

cc @gkoz

}
}

//pub fn get_display(&self) -> /*Ignored*/Option<gdk::Display> {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like we should add Gdk.Display to the list of manual objects.

@EPashkin
Copy link
Member Author

Updated

@EPashkin
Copy link
Member Author

@gkoz If you merge gtk-rs/gir#248 first, I update this

@@ -6,7 +6,7 @@ GIR_FILES = gir-files/Gtk-3.0.gir
gir : src/auto/mod.rs

src/auto/mod.rs : Gir.toml $(GIR) $(GIR_FILES)
$(GIR) -c Gir.toml
$(GIR) -c Gir.toml -s
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, you meant adding to the makefile. Well, it doesn't seem like something useful in a regular workflow. Isn't the run time in single digits for you now?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IMHO this way is better defended from sudden time regression.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It doesn't appear in any automation. Locally a sudden slowdown should be easy enough to notice.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Undo it?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only sys generation builds automatic, stats unneeded here.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, please undo.

@EPashkin
Copy link
Member Author

Done

impl Clipboard {
pub fn set_text(&self, text: &str) {
unsafe {
ffi::gtk_clipboard_set_text(self.to_glib_none().0, text.to_glib_none().0, text.len() as i32);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since we're passing the length, CString conversion can be avoided, like in text_buffer.

@EPashkin
Copy link
Member Author

EPashkin commented Apr 25, 2016

Fixed. Also separate wrong union of Regen and manual

@EPashkin EPashkin force-pushed the deignoring branch 2 times, most recently from 45234e3 to d81647f Compare April 25, 2016 00:08
@EPashkin
Copy link
Member Author

Fixed again: using c_int and split line

@EPashkin
Copy link
Member Author

@gkoz, restart AppVeyor please, it has clone error, or merge PR

@GuillaumeGomez
Copy link
Member

Appveyor restarted.

@EPashkin
Copy link
Member Author

AppVeyor build succeeded

}
}

//pub fn get_current_device_grab(&self, device: /*Ignored*/&gdk::Device) -> Option<Widget> {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like we've missed the need for Gdk.Device

@EPashkin
Copy link
Member Author

EPashkin commented Apr 25, 2016

Fixed all &mut Tree*, added FileChooserConfirmation

@EPashkin
Copy link
Member Author

AppVeyor build succeeded

@EPashkin
Copy link
Member Author

Adding final part of enum as i32. See gtk-rs/examples#88.

@EPashkin
Copy link
Member Author

Again forgot build examples with latest gtk :(

@GuillaumeGomez
Copy link
Member

Restarted.

@gkoz
Copy link
Member

gkoz commented Apr 25, 2016

Everything looks good now. Thanks @EPashkin

@gkoz gkoz merged commit be5c86c into gtk-rs:master Apr 25, 2016
@EPashkin EPashkin deleted the deignoring branch April 25, 2016 15:43
alex179ohm pushed a commit to alex179ohm/gtk that referenced this pull request Oct 21, 2019
…reference_in_enum

Remove unused uses in generated enums.rs
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants