Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
    Denilson das Mercês Amorim
    @thelink2012
    ehh why am I writing here instead of the issue?
    Denilson das Mercês Amorim
    @thelink2012
    hmm am I using the issue tracker correctly?
    David Campbell
    @dcampbell24
    Yeah. I think what you are doing is good. If you have general questions that don't fit well into a specific issue, I think this is a good place, otherwise it is helpful to create issues to keep threads of discussion in one place.
    Denilson das Mercês Amorim
    @thelink2012
    You seem to no be Watching rust-iup-sys!? Well I have opened a issue over there, better notify you here since you won't receive a mail notification eh?
    Denilson das Mercês Amorim
    @thelink2012
    Ohhh and by the way the issues I /cc'ed you in iup-rust itself, you didn't leave a opnion, what does that mean in general?
    Did I use /cc wrongly?
    David Campbell
    @dcampbell24
    Either I didn't read it, or read it but didn't have too much to say at the moment. I'll try to make more of an effort a point of getting back to you when you @mention me.
    I don't know what /cc is if that is a seperate thing from a mention.
    @thelink2012 I am now watching the repository, not sure why I was not watching from the start.
    David Campbell
    @dcampbell24
    @thelink2012 would it be helpful if I add you as a co-owner for the iup-sys crate? Changes to the API should mean bumping the crate version and uploading the new version to crates.io. I can also make rust-iup into a crate and make you a co-owner if you want. I have been holding off for the time being because the crate is not very stable right now.
    Denilson das Mercês Amorim
    @thelink2012
    Sure, I also agree rust-iup itself is not so ready to be released.
    About the cc thing I'm not sure too, I just see it used a lot in some repos (like rust-lang/rust) itself to make the attention of some, and when searching it's meaning on google I got into this page https://guides.github.com/features/issues/

    which says

    We like to use the /cc syntax (an abbreviation for carbon copy) to include people in issues:
    /cc @kneath @jresig

    David Campbell
    @dcampbell24
    Do you already have a crates.io account?
    Denilson das Mercês Amorim
    @thelink2012
    not really, let me create one
    David Campbell
    @dcampbell24
    ok
    Denilson das Mercês Amorim
    @thelink2012
    done
    David Campbell
    @dcampbell24
    ok added you
    Denilson das Mercês Amorim
    @thelink2012
    also the issues I required your attention are #17 tru #23 ('xcept for #18) if you lost the notifications
    David Campbell
    @dcampbell24
    oh thanks
    Denilson das Mercês Amorim
    @thelink2012
    lol interesting I can reference issues from here
    David Campbell
    @dcampbell24
    yeah that is pretty cool
    Denilson das Mercês Amorim
    @thelink2012
    so #17 #19 #20 #21 #22 #23 are 'em
    hmm by the way you have done a few updates on README.md [yesterday?], was that the updates required to conform to the 'new state' of the library (i.e. not a plain wrapper of the C library)?
    David Campbell
    @dcampbell24
    nah I didn't make that update it was just related to splitting the crates into separate repos.
    I can probably update sometime today though before I go to sleep
    maybe just say that things are in flux but we are moving toward a traits and struct based arch
    Denilson das Mercês Amorim
    @thelink2012
    ah fine
    David Campbell
    @dcampbell24
    So regarding 17 the From trait is idiomatic now, but not really a high priority thing to do
    Denilson das Mercês Amorim
    @thelink2012
    well it's a 10 seconds change :P
    David Campbell
    @dcampbell24
    :)
    Denilson das Mercês Amorim
    @thelink2012
    I didn't get over those issues already because I was waiting for you feedback, I don't feel alright to do those moves without asking you
    hell sometimes I feel a bit bad for forming the basic direction of the library code instead of you
    David Campbell
    @dcampbell24
    well I don't mind, you should probably be listed as an author
    I just created it for a simple app I was writing and didn't have any lofty goals
    I like the direction it is going
    If you are making smallish changes I would feel free to just do it or make a pull request I am not likely to object
    regarding 19 I am not sure it is really safe to have multiple objects sharing a pointer
    Denilson das Mercês Amorim
    @thelink2012
    it seems to be I asked a question on reddit on my first days of Rust
    and for handle likes objects it seems to be
    let me find it
    TL;DR: it's fine :P
    David Campbell
    @dcampbell24
    but I made something copy when I was prototyping I thought and it led to a crash
    Denilson das Mercês Amorim
    @thelink2012
    hm weird
    David Campbell
    @dcampbell24
    so I would just say test it out
    Denilson das Mercês Amorim
    @thelink2012
    (Y)
    no such emoticon, goddam
    David Campbell
    @dcampbell24
    I will respond on the tickets actually
    Denilson das Mercês Amorim
    @thelink2012
    and give the doubt if the contained handle is really part of that trait.
    i.e. a Handle can wrap any element and such element may or mayn't allow hierarchy operations, and the user would be able to call hierarchy operations from a Handle.