is there a correct way to format `HashMap`

[zhiburt]

is there a correct way to format `HashMap`

[CGMossa]

Yes I'll totally make a pr. Let me use it for something.

…

On Fri, 29 Oct 2021, 18:56 Maxim Zhiburt, ***@***.***> wrote: And yes, this is not written somewhere that I could find I think it right, I wonder if you wan't to open a PR, with an update of README.md, maybe some example? *Of course I can do it as well if you wish 😄 Is it possible for you to write a little overview of what options are available for the derive-macro? I saw in the examples that there are inline and other functionality in there.. I think that all options are covered in README.md. But I can imagine it might be not as descriptive. Where do you feel it can be improved or be noted? I mean in rust doc or README.md etc. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#28 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAIDVSGJFX6OM6QX2QW73JTUJLG3ZANCNFSM5G4YADSQ> .

[zhiburt]

Hi @CGMossa,

So all collections from std must support tabled.
For example in case of HashMap once key and value implements Tabled, everything will be smoothly work.

#[derive(Tabled, Hash, PartialEq, Eq, PartialOrd, Ord)]
struct Distribution {
    name: &'static str,
    is_active: bool,
    is_cool: bool,
}

impl Distribution {
    fn new(name: &'static str, is_active: bool, is_cool: bool) -> Self {
        Self {
            name,
            is_active,
            is_cool,
        }
    }
}

fn main() {
    let mut data = std::collections::HashMap::new();
    data.insert(Distribution::new("Arch", true, true), "not used");
    data.insert(Distribution::new("Manjaro", true, true), "used");
    data.insert(Distribution::new("Fedora", true, true), "in use");

    let table = Table::new(&data);

    println!("{}", table);
}

+---------+-----------+---------+----------+
|  name   | is_active | is_cool |   &str   |
+---------+-----------+---------+----------+
| Manjaro |   true    |  true   |   used   |
+---------+-----------+---------+----------+
|  Arch   |   true    |  true   | not used |
+---------+-----------+---------+----------+
| Fedora  |   true    |  true   |  in use  |
+---------+-----------+---------+----------+

Though I probably would agree that it is not well documented how tabled works with HashMap, BTreeMap etc.

---

Did my answer help you?

[CGMossa]

This is great! And yes, this is not written somewhere that I could find.. But of course it makes sense as soon as I read that keys, and values has to implement Tabled.

Is it possible for you to write a little overview of what options are available for the derive-macro? I saw in the examples that
there are inline and other functionality in there..

[zhiburt]

And yes, this is not written somewhere that I could find

I think you are right,
I wonder if you want to open a PR, with an update of README.md, maybe some example?
*Of course I can do it as well if you wish 😄

Is it possible for you to write a little overview of what options are available for the derive-macro? I saw in the examples that there are inline and other functionality in there..

I think that all options are covered in README.md.
But I can imagine it might be not as descriptive.

Where(How) do you feel it can be improved or be noted?
I mean in rust doc or README.md etc.