sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: go
Issue: x/net/http2: handle the case of more content received than was declared by server in content-length headers
Labels: NeedsDecision

<!-- Please answer these questions before submitting your issue. Thanks! -->

### What version of Go are you using (`go version`)?

<pre>
$ go version
1.12
</pre>

### Does this issue reproduce with the latest release?
Yes

### What operating system and processor architecture are you using (`go env`)?

<details><summary><code>go env</code> Output</summary><br><pre>
$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/asd/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/asd/MEGA/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/lib/go"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build964683646=/tmp/go-build -gno-record-gcc-switches"

</pre></details>

### What did you do?

running this code:
https://gist.github.com/anysz/e32dfd5bd501da3836ee4960f2620f68

### What did you expect to see?
data decoded correctly like in http/1.0 version

### What did you see instead?
panic: net/http: server replied with more than declared Content-Length; truncated

workaround:
`GODEBUG=http2client=0 go run` works fine
Repository: flutter
Issue: Slight flicker on the screen when loading from app overview on Android
Labels: platform-android,engine,customer: crowd,c: rendering,has reproducible steps,P2,found in release: 3.7,found in release: 3.9,team-android,triaged-android

When switching between apps to an app created by flutter on an Android 9.0 device, there might be a short shadowy (so the screen is not completely black) flicker of the screen that the app shows.

This can be reproduced by any app as far as I am aware. 
1. Create a basic app by using `flutter create app` followed by `cd app` and `flutter run` on an Android 9.0 device (it might also affect other versions, I have not tested it on any other devices, but I heard that other people are able to reproduce it on their device).
2. Launch the application and go back into the running app overview.
3. Select another app that is open.
4. Wait a little bit (this seems to make the bug more likely to happen).
5. Go back into the overview and select the flutter app.
6. A quick slight flicker will appear on the screen.
Repository: godot
Issue: Inspector leaks memory when inspecting Remote scene tree
Labels: bug,topic:core,topic:editor,confirmed

<!-- Please search existing issues for potential duplicates before filing yours:
https://github.com/godotengine/godot/issues?q=is%3Aissue
-->

**Godot version:**
<!-- Specify commit hash if non-official. -->
3.1.1

**OS/device including version:**
<!-- Specify GPU model and drivers if graphics-related. -->
Windows 7 Pro SP1

**Issue description:**
<!-- What happened, and what was expected. -->
Memory is leaked when a node from Remote scene tree is inspected.

https://www.dropbox.com/s/y488ah1jg9fmww4/inspector_leak_remote_scenetree.mp4?dl=0

**Steps to reproduce:**
Create any node, run the game and inspect this node in the Inspector.
You may also inspect 'root'. It leaks more memory.

**Minimal reproduction project:**
<!-- Recommended as it greatly speeds up debugging. Drag and drop a zip archive to upload it. -->
Not necessary.
Repository: rust
Issue: Tidy doesn't check for unused extern crates
Labels: T-bootstrap

Since 44a01b8a54b078d15620d1133b94ee21ee7a6915, tidy no longer emits unused crate warnings -- i.e., Cargo.toml has a dependency but the associated library does not. Due to use of 2018 edition implicit imports, this is no longer an easy thing to do (any file within the crate can use the crate, and extern crate isn't required).

I'm thinking we can probably just remove this tidy lint for now in favor of something like https://github.com/rust-lang/rust/issues/57274 in the future.
Repository: go
Issue: cmd/cgo: documentation for C.GoString could be more complete
Labels: Documentation,help wanted,NeedsFix,compiler/runtime

Documentation for `C.GoString()`, `C.GoStringN()`, and `C.GoBytes()` should probably explicitly document behaviour for "special" values (e.g. `NULL` pointer, negative integers).

https://golang.org/cmd/cgo/#hdr-Go_references_to_C

#### System details

```
go version go1.12.6 linux/amd64
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/nschelle/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/nschelle/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
GOROOT/bin/go version: go version go1.12.6 linux/amd64
GOROOT/bin/go tool compile -V: compile version go1.12.6
uname -sr: Linux 4.4.0-146-generic
Distributor ID:	Ubuntu
Description:	Ubuntu 16.04.6 LTS
Release:	16.04
Codename:	xenial
/gnu/store/h90vnqw0nwd0hhm1l5dgxsdrigddfmq4-glibc-2.28/lib/libc.so.6: GNU C Library (GNU libc) stable release version 2.28.
gdb --version: GNU gdb (GDB) 8.3
```
Repository: terminal
Issue: A few StateMachine unit tests don't fully test what they purport to
Labels: Product-Conhost,Help Wanted,Issue-Task,Area-CodeHealth

I'm not sure if this is right place to report this, because it's not a bug in the app itself, but it's an issue I noticed in the unit tests, which I think may be considered a bug.

As a specific example, consider the [`TestCursorKeysMode`](https://github.com/microsoft/terminal/blob/9b92986b49bed8cc41fde4d6ef080921c41e6d9e/src/terminal/parser/ut_parser/OutputEngineTest.cpp#L1162-L1177) test in OutputEngineTest.cpp:

```c++
TEST_METHOD(TestCursorKeysMode)
{
    StatefulDispatch* pDispatch = new StatefulDispatch;
    VERIFY_IS_NOT_NULL(pDispatch);
    StateMachine mach(new OutputStateMachineEngine(pDispatch));

    mach.ProcessString(L"\x1b[?1h", 5);
    VERIFY_IS_TRUE(pDispatch->_fCursorKeysMode);

    pDispatch->ClearState();

    mach.ProcessString(L"\x1b[?1l", 5);
    VERIFY_IS_FALSE(pDispatch->_fCursorKeysMode);

    pDispatch->ClearState();
}
```

The second half of this test is assumedly meant to prove that the given escape sequence will reset the cursor keys mode. However, the `_fCursorKeysMode` flag is false by default, so you could replace that escape sequence with almost anything, including removing it altogether, and the test would still pass.

There are similar problems with `TestCursorBlinking`, `TestCursorVisibility`, and possibly also `TestAltBufferSwapping`.

When I added a similar test, I got around the problem by explicitly setting the flag I was testing to the opposite of what was expected, but I'm not sure if that is the best approach. Another possibility might be using `std::optional<bool>` instead of just a `bool`, so the default value could then be a `nullopt`, and it wouldn't be possible to pass the test based on the default value alone. The downside of that approach is the `VERIFY` steps become a little more complicated.
Repository: youtube-dl
Issue: Is it possible to abort if one of the formats is unavailable?
Labels: question

## Checklist
- [x] I'm asking a question
- [x] I've looked through the README and FAQ for similar questions
- [x] I've searched the bugtracker for similar questions including closed ones

## Question
When I request a download from Youtube in formats "137,140" (for video and audio, without merging, because it is lengthy and useless), the following happens:
- Trying to download 137... not available, whatever, moving on
- Trying to download 140... success!

Your video is successfully downloaded!
(...but there is no video, only audio.)

Is it possible to do anything about this situation?
- Maybe add an option to abort if one of the requested formats is not available?
- Maybe add an option to disable merging when downloading in "137+140" form?
- Maybe download the video in both formats at the same time, merging them on the fly, and then saving only the output file?
Repository: flutter
Issue: Inline Widgets should have modified/ignore safe area changes for keyboard.
Labels: a: text input,framework,P2,team-framework,triaged-framework

When nesting a whole app inline, bringing up the keyboard causes the app to display a big empty area corresponding to the safe area.

This safe area may or may not be relevant for the inlined app.

![Screenshot_2019-06-21-17-52-56-196_io flutter demo gallery](https://user-images.githubusercontent.com/1887398/59957471-82d68400-944d-11e9-9f8c-a6cf4e8ae7bc.png)
Repository: godot
Issue: Convex decomposition leads to inaccurate shape for simple meshes
Labels: bug,topic:core,confirmed,usability,topic:3d

**Godot version:** 05a0a68c72cc16c443301398ab93e8d838401ac0


**OS/device including version:**
Windows 10 x86_64


**Issue description:**
Using the old quickhull method, when using the Mesh->Create Convex Collision Sibling, a simple scaled cube mesh would produce a perfect convex hull collision shape. Now, with the new convex decomposition, the same simple cube mesh produces an inaccurate shape:
![shape](https://user-images.githubusercontent.com/11659668/59957943-0b066900-94e2-11e9-9ccf-4ffba715eb2a.PNG)
This issue is not that much of a problem for this example (only a small difference), but for more complex convex shapes it becomes a major problem.

**Steps to reproduce:**
Import this gltf file to a godot project:  [cube.zip](https://github.com/godotengine/godot/files/3316424/cube.zip)
Use Mesh->Create Convex Collision Sibling(s)
See that the collision shape does not line up with the mesh instance (use wireframe mode for a better look)

**Minimal reproduction project:**
[test.zip](https://github.com/godotengine/godot/files/3316426/test.zip)
Repository: terminal
Issue: Add support for turning off cursor blink
Labels: Help Wanted,Area-TerminalControl,Area-Settings,Product-Terminal,Issue-Task,Priority-2

I don't link blinking cursor on terminal.
so I have tried to stop blink of cursor from settings but there is no option to do it.
How to disable blink of cursor?
Repository: TypeScript
Issue: IntelliSense signature help forgets inferred parameter names in completed call
Labels: Bug

<!-- 🚨 STOP 🚨 𝗦𝗧𝗢𝗣 🚨 𝑺𝑻𝑶𝑷 🚨

Half of all issues filed here are duplicates, answered in the FAQ, or not appropriate for the bug tracker. Even if you think you've found a *bug*, please read the FAQ first, especially the Common "Bugs" That Aren't Bugs section!

Please help us by doing the following steps before logging an issue:
  * Search: https://github.com/Microsoft/TypeScript/search?type=Issues
  * Read the FAQ: https://github.com/Microsoft/TypeScript/wiki/FAQ

Please fill in the *entire* template below.
-->

Per https://github.com/microsoft/TypeScript/issues/31845#issuecomment-504615727, posting this as a new ticket for posterity.

<!-- Please try to reproduce the issue with `typescript@next`. It may have already been fixed. -->
**TypeScript Version:**  3.6.0-dev.20190621

<!-- Search terms you tried before logging this (so others can find this issue more easily) -->
**Search Terms:** n/a

**Code**

```ts
function frob<T, A extends any[]>(f: (...args: A) => T, ...args: A)
{
    return f(...args);
}

function blub(foo: string, bar: number) {}

// press Ctrl+Shift+Space inside the parentheses here:
frob(blub, "banana slamma", 812);
```

Parameter names are correctly displayed while initially typing out the `frob()` function call:

![image](https://user-images.githubusercontent.com/4097841/59400643-2517ad00-8d66-11e9-8c37-d613b4f867b9.png)

**However**, once all the arguments are filled out, it reverts to generic names such as `args_0`, `args_1`, etc. ☹️

![image](https://user-images.githubusercontent.com/4097841/59400700-58f2d280-8d66-11e9-9aa6-08aa204ab46f.png)

**Expected behavior:**
Parameter names originating from an inferred parameter tuple should be preserved in IntelliSense.

**Actual behavior:**
Parameter names are preserved while typing out the function call, but once the call is completed it reverts to using generic parameter names and remains that way for all subsequent invocations of signature help for that call (unless an argument is deleted).

**Playground Link:** <!-- A link to a TypeScript Playground "Share" link which demonstrates this behavior -->
[⏯ Click here](https://typescript-play.js.org/#code/GYVwdgxgLglg9mABMATnARgHgCoBpECCiApgB5TFgAmAzogIZgCeA2gLoB8AFMAFyJcAdMPooA5jX4EAlIgC8HRHkTDBoiVOkAoAN5bEBxCmJQQKJMCEjxNaQG4tAXy1bQkWAkToANiHQ84OH4aKBQYMDF8dFF+MBAAW3RiFFkdZy0AegzEAAdjGjoAYVDvAGoAZQALGGAoCpz6CGJEcJoYKmaoSuaG4zAu4hpBxG7jXlc0fx8-fAAiaLBGekQab3p4+PpZ-AAOAEYAJnstIA)

**Related Issues:** <!-- Did you find other bugs that looked similar? -->
#31845
Repository: go
Issue: net: a smart way to check internal/poll ErrTimeout
Labels: NeedsInvestigation,FeatureRequest

Hello guys.

It looks like we have no smart way to check the internal/poll ErrTimeout when we set the deadline on the connection.

The `net.Error` interface is not good enough as following

````golang
An Error represents a network error.

type Error interface {
        error
        Timeout() bool   // Is the error a timeout?
        Temporary() bool // Is the error temporary?
}

````

The `Timeout()` method may mislead caller think it's deadline error if the error is actually caused by `sycall.Errno` (`e == EAGAIN || e == EWOULDBLOCK || e == ETIMEDOUT`) rather than internal/poll ErrTimeout

Maybe we need a smart way to check the deadline error?

Many thanks in advance:)
Repository: terminal
Issue: Feature Request: Smoothly scroll in new lines
Labels: Issue-Feature,Help Wanted,Area-Rendering,Area-TerminalControl,Product-Terminal

# Summary of the new feature/enhancement

Some hardware terminals allowed smoothly scrolling in new lines. It should be possible to implement this kind of feature in a software terminal emulator as well.

YouTube demonstration (VT525 smooth scrolling):
https://www.youtube.com/watch?v=Iju_pOQM0a0
Repository: rust
Issue: Lifetime error's span is extremely unhelpful
Labels: C-enhancement,A-diagnostics,A-lifetimes,T-compiler,E-needs-mcve,D-papercut

# Without explicit lifetime
```
error[E0621]: explicit lifetime required in the type of `expr`
   --> typescript/checker/src/analyzer/expr.rs:14:84
    |
14  |       pub(super) fn type_of<'e>(&'e self, expr: &Expr) -> Result<TypeRef<'e>, Error> {
    |  _______________________________________________-----________________________________^
    | |                                               |
    | |                                               help: add explicit lifetime `'e` to the type of `expr`: `&'e swc_ecma_ast::Expr`
15  | |         let span = expr.span();
16  | |
17  | |         match *expr {
...   |
340 | |         }
341 | |     }
    | |_____^ lifetime `'e` required


```


# With explicit lifetime
```
error[E0623]: lifetime mismatch
   --> typescript/checker/src/analyzer/expr.rs:14:91
    |
14  |       pub(super) fn type_of<'e, 'a>(&'e self, expr: &'a Expr) -> Result<TypeRef<'e>, Error> {
    |  ___________________________________________________--------_____--------------------------_^
    | |                                                   |
    | |                                                   this parameter and the return type are declared with different lifetimes...
15  | |         let span = expr.span();
16  | |
17  | |         match *expr {
...   |
340 | |         }
341 | |     }
    | |_____^ ...but data from `expr` is returned here


```

Error's span points the whole function, and in my case, it's 300 lines.
Repository: terminal
Issue: Feature: Add support for Infinite Scrollback
Labels: Issue-Feature,Area-TerminalControl,Product-Terminal

# Summary of the new feature/enhancement
I know at least gnome-terminal supports infinite scrollback as a checkbox option in profile settings; it would be great to have that here (I'd love to be able to replace X server+gnome-terminal for WSL).

I checked to make sure that there wasn't an issue for this already, and I saw that you have a TODO comment in the code for this, so I figured I'd open a GitHub issue so that you can use the 👍 to figure out how to rank it.


<!-- 
A clear and concise description of what the problem is that the new feature would solve.
Describe why and how a user would use this new functionality (if applicable).
-->

# Proposed technical implementation details (optional)
Looks like the plan is for users to set HistorySize to `-1`
https://github.com/microsoft/terminal/blob/900d0c3cce39fa191e16b0224fa32c3441f8de24/src/cascadia/TerminalCore/Terminal.cpp#L88
Repository: flutter
Issue: Support shorter dev/testing cycles
Labels: a: tests,c: new feature,framework,P3,team-framework,triaged-framework

<!-- Thank you for using Flutter!

     If you are looking for support, please check out our documentation
     or consider asking a question on Stack Overflow:
      * https://flutter.dev/
      * https://api.flutter.dev/
      * https://stackoverflow.com/questions/tagged/flutter?sort=frequent

     If you have found a bug or if our documentation doesn't have an answer
     to what you're looking for, then fill our the template below. Please read
     our guide to filing a bug first: https://flutter.dev/docs/resources/bug-reports
-->

## Use case

<!--
     Please tell us the problem you are running into that led to you wanting
     a new feature.

     Is your feature request related to a problem? Please give a clear and
     concise description of what the problem is.

     Describe alternative solutions you've considered. Is there a package
     on pub.dev/flutter that already solves this?
-->
For those of us that use TDD or heavily rely on unit testing, we need to be able to re-run our tests quickly and view the coverage report quickly. Opening the browser to view the coverage report takes to long to support short dev cycles.

## Proposal
Pull some features/use-cases from the [Jest](https://jestjs.io/docs/en/cli#running-from-the-command-line) testing framework like watching, filtering, and coverage output.
<!--
     Briefly but precisely describe what you would like Flutter to be able to do.

     Consider attaching images showing what you are imagining.

     Does this have to be provided by Flutter directly, or can it be provided
     by a package on pub.dev/flutter? If so, maybe consider implementing and
     publishing such a package rather than filing a bug.
-->
Repository: godot
Issue: select multiple particle2d and change same property bug
Labels: enhancement,topic:editor,usability,topic:particles

**Godot version:**
3.1.1

**OS/device including version:**
mac 10.14.4


**Issue description:**
can't change all selected particle2d's property at the same time.


**Steps to reproduce:**
1. select two or three particle2d node.
2. change the gravity of them.
3. check each other gravity , found that only one particle be modified.

**sugession:**
1. [good] change all particle2d property with the same time.
2. [bad] hide all property when mutiple selection.
Repository: terminal
Issue: Mouse pointer should be text and not arrow one
Labels: Help Wanted,Area-TerminalControl,Product-Terminal,Issue-Task

<!--
This bug tracker is monitored by Windows Terminal development team and other technical folks.

# Environment

```none
Windows build number: 10.0.18362.175
Windows Terminal version (if applicable): 0.2.1715.0

Any other software?
```

# Steps to reproduce
Move the mouse over some text inside the terminal.

<!-- A description of how to trigger this bug. -->

# Expected behavior
The mouse pointer used should be the text cursor (Ꮖ) one since text is selectable, not the default arrow (↖) pointer.

<!-- A description of what you're expecting, possibly containing screenshots or reference material. -->

# Actual behavior
It's the default arrow pointer that's used.

<!-- What's actually happening? -->
Repository: terminal
Issue: Feature Request: CornerRadius on Cursor Shape
Labels: Help Wanted,Area-Rendering,Area-Settings,Product-Terminal,Issue-Task,good first issue

# Summary of the new feature/enhancement

When setting the cursor shape to **filledBox** or **vintage**, it would be nice if you could round the corners of that shape.

# Proposed technical implementation details (optional)

If the cursor is drawn from a shape, then you could add a setting for `cursorCornerRadius: "2, 2, 2, 2"` and apply that to the shape.  If however the cursor image is drawn from a font in the glyph - then perhaps CascadiaCode could have rounded boxes for the cursors.
Repository: rust
Issue: E0515 searching internet for the phrase "owned value" does not give helpful results
Labels: C-enhancement,A-docs

https://doc.rust-lang.org/error-index.html#E0515
> Consider returning an owned value instead:

Searching for this phrase does not define it within the first page of a google search.
Repository: scrcpy
Issue: QtScrcpy
Labels: gui

 Amazing project. Thank you!
I reimplemented scrcpy with different technology stacks (Qt + Opengl) and added something:
- an interface for interaction.
- connecting multiple devices to the same application

Of course I want to share it. [QtScrcpy](https://github.com/barry-ran/QtScrcpy)
![win](https://raw.githubusercontent.com/barry-ran/QtScrcpy/master/screenshot/win.png)
Repository: TypeScript
Issue: Allow inference of class generics when declaring an interface 
Labels: Suggestion,Awaiting More Feedback

## Search Terms
interface, inference, extending class

## Suggestion
I'd like to be able to strongly type an interface such that when it is used on a class that has a generic I can infer the type of that generic to then use in the declaration of the interface.

## Use Cases
<!--
What do you want to use this for?
What shortcomings exist with current approaches?
-->
I want to use this within a library that exposes both an abstract class and an optional interface that developers can opt-in to use. The abstract class has one generic parameter, and in order to correctly type the interface without duplicate generic declaration I want to be able to infer the abstract class' generic type.

Currently the inheriting class declarations must duplicate their generic type declaration.

This issue is similar to https://github.com/microsoft/TypeScript/issues/26242 I think, but I don't see how the resolution of that issue will fix this one.

## Examples

<!-- Show how this would be used and what the behavior would be -->
```ts
abstract class Base<T> {
    // implementation omitted for brevity
}

// ⬇️this is the proposed new syntax ⬇️
interface Remap<T> extends Base<infer U> {
    remap(input: U): T;
}

class Foo extends Base<string> implements Remap<number> {

    public remap(input: string): number {
        return input;
    }

}
```

## Checklist

My suggestion meets these guidelines:

* [x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
* [x] This wouldn't change the runtime behavior of existing JavaScript code
* [x] This could be implemented without emitting different JS based on the types of the expressions
* [x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
* [x] This feature would agree with the rest of [TypeScript's Design Goals](https://github.com/Microsoft/TypeScript/wiki/TypeScript-Design-Goals).