Results 341 comments of m4b

This is an I interesting idea. The tool could be a dev dependency and perhaps we have a feature flag enabling “regression suite”, it then downloads as a setup everything...

Yea that was exactly what I was thinking. Except probably it’s on a mod though and we put all manifest/external file based tests in there. It might also be useful...

I believe this is correct; the idea is that it returns whether the symbol is something a dynamic linker could import; relying on section header information, while nice, is not...

Was this ever resolved ? Is it still an issue ?

Can anyone help out elast0ny or clarify is this a bug we need to fix? @jan-auer, @willglynn @philipc maybe?

hi @Maimonator thank you for investigating! if you want to push your patch, we can review (a diff with the changes is easier to read); even better is if we...

So I thought the PE structs implemented FromCtx (reading) ? However for I don't think IntoCtx has been implemented yet (writing) That would be pretty awesome :) I think the...

@indygreg first off, thank you for the kind words! :) So re `.tbd` files, the short is that I would/could be persuaded to include them in goblin. The long is...

So i'm wondering if a simple, more flexible approach here is just to detect the tbd (glancing at your code, this seems somewhat complicated/tedious), and just return a `TBD(&'a str)`...

> n order to avoid a future API break, do you think it would be worth declaring a new `TBD` type - initially an alias or thin wrapper around `&'a...