« first day (1374 days earlier)      last day (2105 days later) » 

1:55 AM
@kennytm Did I do it right? github.com/shepmaster/jetscii/commit/…
 
2:12 AM
looks fine to me; i remembered there's some errors prevented me from doing impl Pattern<H> where H: Haystack<Target = str>, let me check again
@Shepmaster no wait, the returned range should be relative to the input ~~range~~ hay
i.e. self.find(&hay[range]).map(|i| range.start+i..range.start+i+1)
 
@kennytm Then I'm missing something in my tests, do you see what?
It's a property test that I behave the same as your find
 
try using matches(...).collect()
this should start to differ from the 2nd item
ok the reason i can't do the H: Haystack<Target = str> in the standard library is because github.com/rust-lang/rfcs/pull/1672 is postponed, which again depends on chalk :|
 
@kennytm but is that just in the stdlib?
How can I exercise Consumer to trigger the same failure for forgetting the offset?
 
@Shepmaster try trim_start, the default implementation will move the range.
@Shepmaster the conflict was Pattern<...str...> for impl Fn(char)->bool vs Pattern<...[T]...> for impl Fn(&T)->bool
perhaps not stdlib specific, but certainly something that can't coexist without RFC 1672
 
3:23 AM
@kennytm ok, I'm getting the same results for find, matches and trim_start. Anything else I should use to exercise?
 
maybe match_indices. i think this should cover everything for the forward direction
 
3:44 AM
@Shepmaster thanks :D for the Consume impl you might make it more efficient by taking the .bytes() and delegate to &jetscii::Bytes?
 
@kennytm I was thinking something along those lines. I had to think through the unicode implications before I did it though.
I also figure i can try implementing the pattern for Bytes and fully delegate everything
since that's a big benefit
I'll post something about it on the RFC tomorrow (I hope)
 
yes. in my implementation both &str and &OsStr just delegate to &[u8] :p
 
but now it's 😴
&OsStr ... on non-Windows, you mean?
 
both; on windows i do check the surrogates after searching for sure :p
\o
 
 
9 hours later…
1:02 PM
Is it possible to cfg_attr on whether you're doing a doc build?
It's the simplest way to fix this doc building error on diesel (short of actually fixing documentation warnings).
And the Rust doc is… lacking in this area
 
@набиячлэвэли I think there might be a feature dox defined in a doc build...
but that may just be a rust standard library thing...
 
but you could just #![allow(intra_doc_link_resolution_failure)]?
 
@Shepmaster It's not just libstd, and it works!
great news
@kennytm I might just actually go over each doc error in the libs I'm using and PR them in tbh
 
 
4 hours later…
5:03 PM
Ay I need help.-
This doctest does not compile until 1.26.0 and I have no idea why.
 
what's the error message?
 
... Well, I think I got the problem now.
I must not extern crate for the actual crate the doctest is in.
But this is still odd, though. Can't a doctest rely on other crates?
 
downvote, questions asking for help must contain the full error
 
@Shepmaster Aye, no surprise this one still exists as a chat message. Chill, I'll be posting it on the real thing soon. ;)
 
oh noes
 
5:23 PM
@Shepmaster You know when I ask, it's srsbznss
My questions take time to mature.
 
5:47 PM
Uh, Oops.
Solved it now.
 
6:01 PM
@E_net4 Why must you not?
 
@набиячлэвэли Well, I actually can, but I must make an explicit fn main.
 
well, obviously
what were you using instead?
 
@набиячлэвэли Not obvious. It works without this since 1.26.0.
 
@E_net4 oh, really?
I wouldn't know, I always copy-and-adjust my doctests :v
 
:v
 
6:07 PM
also my doctests are always big fat lies so
Okay lads, what's the correct course of axion here
There's an issue with diesel (affects all public versions) that prevents cargo doc from working on nightly
And while I can manually patch my cached version as well as slam a patcher in my CI desc, I really would rather this get fixed upstream (documentation is a big bloody deal)
However, the issue template requires that "This issue can be reproduced on Rust's stable channel. (Your issue will be closed if this is not the case)"
But it will be far too late to fix it when current nightly (not sure if affects beta as well) hits stable
Hell, even docs.rs doesn't build the documentation
Please advise
 
I think the issue template is just that. If you were just filing the issue, then they might not worry. Weren't you going to fix it? And will your fix work on stable?
 
6:22 PM
The simplest fix was to cfg_attr it on feature = "dox"
because the errors aren't exactly helpful beyond that
@Shepmaster What do you mean by "If you were just filing the issue, then they might not worry"?
and the fix would probably work on stable
 
6:35 PM
@Shepmaster That is encouragement enough that I submitted it
 
@набиячлэвэли who has deny(warnings)?
I thought that denied warnings in upstream crates didn't stop a build
 
@Shepmaster diesel/src/lib.rs#L131
@Shepmaster Well it's a build error so they do
 
@Shepmaster rustdoc didn't support --cap-lints until very recently
 
yeah, thats the surprise
but then theoretically, building the docs on nightly should be using it
 
i think the plan would be passing --cap-lints by default since 1.29 github.com/rust-lang/rust/pull/52354
(which makes that diesel issue irrelevant on stable 🙃)
 
6:42 PM
Even more fun!
 
I'm surprised it took that long for someone to hit it / report it / fix it
 
rustdoc did not emit any lints also until very recently
 
hmmm
so, if you were to build the diesel docs with a few versions back, there'd be no warnings
then no reason to care about upstream warnings
and then they added warnings
but no one helped connect the whole upstream warning limiter
SAD
 
@kennytm What ballpark is that "several" in?
 
a full FCP will take 10 days
after reaching consesus (majority votes from the responsible team)
where now we have 2/5 votes
so if we follow the whole procedure that could take like 2 weeks
 
6:57 PM
oh, that's surprisingly palatable
(though I don't feel bad for having made this dumb fix)
 
maybe you could use RUSTDOCFLAGS="--cap-lints warn" on nightly channel
 
that sounds harder tbh (but I will keep both it and the fact that you can for some bizarre reason pass commandline arguments via environment variables in mind, thanks)
 
7:18 PM
> QuietMisdreavus - Today at 3:14 AM
then it shouldn't be a problem, tbh, since we only process intra-links on nightly
> QuietMisdreavus - Today at 3:16 AM
the lint should not fire on anything but nightly or dev channels
 
so cargo doc with diesel dependency will still work even with 1.29-beta
 
and I just hit on some unlucky timing, oh well
 
oh also,
> QuietMisdreavus - Today at 3:11 AM
i think trying to get the CLI stabilization and cargo integration in may not work out, so we'll need a smaller patch to deal with that issue
we may need to set our cap-lints to warn in the meantime
 
Thanks a lot for helping me out, fam; appreciate it
 
7:29 PM
np
 
7:55 PM
And to deal with it locally, you can just use the RUSTDOCFLAGS="--cap-lints warn" when running cargo doc
not a terrible temporary workaround
 
 
2 hours later…
9:32 PM
@Shepmaster well to deal with it locally I did the obvious thing and commented out the deny clause in the source :Р
 

« first day (1374 days earlier)      last day (2105 days later) »