First Pull Request


Lint: Multiple inherent impls rust-lang/rust-clippy

We may want to have a lint that checks for [multiple inherent impls](https://play.rust-lang.org/?gist=5798fd8736e09adb74da&version=stable). Why those are not problematic from a language standpoint,...

Created - 0 comments - E-easy L-lint T-AST

suggest struct constructor instead of mutable object + field mod rust-lang/rust-clippy

If you have a struct where all fields are public, then you should never do `let mut a = A::default(); a.x = 42;` but instead use a constructor with `.. A::default()` at the end: ``` rust #[derive(...

Created - 0 comments - E-easy L-lint T-AST

lint: transmuting known null pointer to ref rust-lang/rust-clippy

`mem::transmute(std::ptr::null())` – this is undefined behavior.

Created - 0 comments - E-easy L-lint T-AST

collapsible_if shouldn't apply if there is a comment before rust-lang/rust-clippy

In this example there is comment which applies to both branches in the `else` block. ``` if entry.end > entry.base { prev = entry_; // Go to the next entry } else { ...

Created - 0 comments - E-easy L-enhancement

improve collapsible if in complex cases with same expressions rust-lang/rust-clippy

``` rust if a { if b { f(); } } else { f(); } ``` could be ``` rust if !a || b { f(); } ```

Created - 0 comments - E-easy L-enhancement L-lint

float_cmp: ignore when comparing with constant? rust-lang/rust-clippy

Consider: ``` const ONE: f32 = 1.0; fn main() { let v = 0.9; let _ = v == ONE; } ``` These are probably cases where you want to check for the exact constant, because it is a common argume...

Created - 0 comments - E-easy L-enhancement

Lint std::panic::catch_unwind rust-lang/rust-clippy

<!-- Hi there! Whether you've come to make a suggestion for a new lint, an improvement to an existing lint or to report a bug or a false positive in Clippy, you've come to the right place. If you ...

Created - 0 comments - E-easy L-lint T-AST

Lint: Implicit Deref rust-lang/rust-clippy

Basically, an `#[allow]` by default lint that tells you when you are referencing a value implicitly (e.g. `String::new().chars()` is really `(*String::new()).chars()`. Rationale is that somebody o...

Created - 0 comments - E-easy L-lint T-middle

Lint to warn about str::as_ptr() usage in ffi call rust-lang/rust-clippy

Hello. It would be nice to have a warning when using `str::as_ptr()` in a ffi call instead of using `CStr`. For instance, in the following code: ``` rust unsafe { printf("Hello".as_ptr() as *cons...

Created - 0 comments - A-correctness E-easy L-lint T-middle

Add topics to GitHub rust-lang/rust-clippy

See https://github.com/blog/2309-introducing-topics. Looks like only the owner can do that @Manishearth smile We should have *Rust* and *linter* at least.

Created - 0 comments - E-easy L-documentation

Detect errors on calls to *MakeCurrent tomaka/glutin

For the moment, calls to `wglMakeCurrent`, `glXMakeCurrent`, etc. can silently fail. The return type of `make_current` should be a `Result<(), MakeCurrentError>`.

Created - 0 comments - bug E-easy

Implement multisample render buffers glium/glium

Add a `new_multisample` for render buffers. There's no need to create new types.

Created - 0 comments - E-easy T-enhancement

Implement default tessellation level glium/glium

When you submit patches but don't have a tessellation control shader. This should be done by adding a field in `DrawParameters`.

Created - 0 comments - E-easy T-enhancement

Add test for multiple textures sampling glium/glium

Test if textures are binded correctly when there are multiple textures involved.

Created - 0 comments - E-easy

Write tests for glium_macros glium/glium

Now that `glium_macros` isn't used in tests anymore, it needs to have its own tests to see if it works.

Created - 0 comments - E-easy T-enhancement

Use https://github.com/simnalamburt/obj-rs for the examples glium/glium

Instead of reinventing the wheel, use https://github.com/simnalamburt/obj-rs which has support for glium.

Created - 0 comments - E-easy T-enhancement

Default Sampler settings are undocumented glium/glium

Had to go digging around in the source to find out what the defaults are.

Created - 0 comments - E-easy T-enhancement

Switch to `Into`, `From`, etc. traits glium/glium

This library has several traits named `IntoFoo`. They should be replaced by `Into<Foo>`.

Created - 0 comments - E-easy P-low-prio T-enhancement

Make get_proc_addr unsafe tomaka/glutin

Because it requires the context to be current.

Created - 0 comments - E-easy

Implement polygon offset glium/glium

By adding an element in `DrawParameters`. Allows alteration of the depth value of the fragments.

Created - 0 comments - E-easy T-enhancement

Add tests for compute shaders glium/glium

Also test if the work group is too high.

Created - 0 comments - E-easy T-enhancement

Don't fail compilation on unsupported targets tomaka/glutin

Use a dummy implementation and return an error during creation instead of failing compilation.

Created - 0 comments - E-easy enhancement

Add correct support for GL_EXT_multisampled_render_to_texture glium/glium

The code in fbo.rs should be adapted.

Created - 0 comments - E-easy T-wrong

Add a bytecode loader gluon-lang/gluon

This would allow the vm crate to be included and used separately though limited to only loading pre-compiled bytecode. For starters it doesn't need any checking for bytecode validity but eventually...

Created - 0 comments - beginner enhancement vm

Displaying issues 1 - 30 of 984 in total

License

Language

Labels