fix: export RawElement type

[tychoish]

Missed that exports had to be explicitly re-exported in #449.

[tychoish]

@abr-egn, I found this when trying to use this in addition to #462. I missed that the exports of symbols here needed to be explicit, so while you can use this, if you want to pass the iterator around explicitly, you can't because the symbol isn't exported.

[abr-egn]

Yup :) Isabel's taking a look at this.

[isabelatkinson]

Just one minor request, going to tag in @abr-egn as well. On an unrelated note, it doesn't seem like our documentation lint task is catching that there's missing documentation for this type and its methods. I filed RUST-1868 to fix this.

[isabelatkinson]

Thanks for catching this clippy error! I think it would make more sense to call the len method size since it's exposing the number of bytes in the element. That would remove the need to have this is_empty method, which likely won't have any utility for the users of this type.

[tychoish]

I'm fine with that if you are. I think it is technically API breaking because you could have called the len() method before even though the type wasn't exported.

I don't care, but you might.

[isabelatkinson]

Oh that's a great point, in that case I think we'll want to stick to len for the name and change it with the next major release (RUST-1869). I still don't think it makes that much sense to have an is_empty method, so I suggest silencing the clippy error with #[allow(clippy::len_without_is_empty)]. Sorry for all of the back-and-forth here!

[tychoish]

done!

[abr-egn]

LGTM!