sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: go
Issue: runtime/race: unexpected race when Conn.Close used for synchronization
Labels: RaceDetector,compiler/runtime

go version devel +a714470 Wed Sep 27 16:29:18 2017 +0000 linux/amd64

The following program reports a race even though the read of x
cannot happen in parallel with the assignment to x because
we wait for the connection to be closed before reading it.

https://play.golang.org/p/ufvoScCbSm

	==================
	WARNING: DATA RACE
	Read at 0x00c42008a220 by main goroutine:
	  main.main()
	      /home/rog/src/tst.go:24 +0x288
	
	Previous write at 0x00c42008a220 by goroutine 7:
	  main.main.func1()
	      /home/rog/src/tst.go:16 +0x7a
	
	Goroutine 7 (finished) created at:
	  main.main()
	      /home/rog/src/tst.go:14 +0x16d
	==================
	2017/10/04 16:48:53 1
	Found 1 data race(s)
	exit status 66
Repository: rust
Issue: Missing stack trace on Windows
Labels: A-debuginfo,T-compiler,O-windows-msvc,C-bug

This unfortunately is not reliably reproducible - I can get a stack trace from small test projects, so the only way I can demonstrate it is cloning [this repo](https://github.com/parkovski/scifiweb/tree/5ef233e9454906e115ab674b1d9e4499282eb042) and running (with RUST_BACKTRACE=1) `scifiweb.exe run vm/test/noresolve.scifi`. I have VS2017 on Windows 10 - I don't remember this happening with 2015, so I'm not sure if it's a change made in VS or Rust.

The error I get is this - there is nothing printed after `stack backtrace:`.
```
thread 'main' panicked at 'Can't take immutable awake borrow while mutable borrow is active', util\src\graph_cell.rs:19:4
stack backtrace:
```

The VS debugger does show a stack trace, so the PDB must be generated correctly.

I'm willing to work on a fix for this if someone can point me toward how to debug the stack trace generation code in Rust.
Repository: react
Issue: Type Fiber by tag as disjoint union
Labels: Type: Enhancement,Component: Reconciler,React Core Team

@sebmarkbage tried this in https://github.com/facebook/react/pull/8545 but Flow couldn't handle it well.
Might be worth trying again someday.
Repository: flutter
Issue: Benchmark sub-systems of a11y processing
Labels: framework,c: performance,a: accessibility,P3,team-framework,triaged-framework

Get performance numbers for the following:
* Updating the semantics tree in Dart
* Transferring semantics tree update to native side
* Processing semantics tree update on native side
Repository: rust
Issue: Tracking issue for `#[doc(spotlight)]`
Labels: T-rustdoc,B-unstable,C-tracking-issue

Initially implemented in https://github.com/rust-lang/rust/pull/45039, this attribute puts your own trait on every function doc if its return type implements it.
Repository: vue
Issue: Export flow types in dist file
Labels: feature request

### What problem does this feature solve?
If people want to use Vue flow types, they can import them from the Vue npm module. 

### What does the proposed API look like?
We would include the flow directory in the npm dist files

(I'm happy to make a PR)

<!-- generated by vue-issues. DO NOT REMOVE -->
Repository: go
Issue: x/build/devapp: flag new contributors in CL list
Labels: Builders,Proposal,Proposal-Accepted

I was speaking with someone today who mentioned that they read some of the CL's and issue responses and felt intimidated. Specifically they were worried that they would submit a CL and there would be a mistake with it and they'd get unfriendly feedback.

It's easy to see how this could happen. If I make a dumb mistake it's probably due to carelessness and it's worth calling out, where if a new contributor makes a mistake, it's probably due to not knowing the context or the build tools well enough. But that can be hard for a reviewer to see, or a reviewer might not know to switch modes before reviewing a CL.

We have the "Thanks for your first contribution" commit message, but maybe we could add something else to flag for reviewers that the contributor is new to the project? Maybe gopherbot could add a "New-Contributor: +1" label to CL's where the submitter has submitted less than five CL's. Or a gopherbot message that says e.g. "Note to reviewers: This CL was submitted by someone new to the Go project! They might not have all of the context that others have."
Repository: TypeScript
Issue: Detect duplicate nodes during printing
Labels: Suggestion,Experience Enhancement

While rewriting code for refactorings and code fixes, it is quite easy to inadvertently insert a single `Node` into multiple positions in the tree (i.e. when it should have been cloned).  It would be very helpful if the printer (or other, more appropriate, component) detected this, rather than behaving subtly incorrectly or throwing an (apparently) unrelated exception.

One way to do this would be to clear `__pos` and `__end` before traversing the tree and then throwing if they are set a second time during the traversal.
Repository: TypeScript
Issue: Add node flags for blank lines before and after a node
Labels: Suggestion,In Discussion,API

When inserting a node (e.g. during a code fix or refactoring), it would be very helpful if there were a way to say "ensure that there is a blank line following this node", rather than inserting one, since inserting one might result in a pair of blank lines.  The determination of whether or not to actually insert a linebreak would be made in the printer.
Repository: electron
Issue: Menu.popup: implement positioningItem for all OS
Labels: enhancement :sparkles:,platform/windows,platform/linux

It looks like [`positioningItem`](https://github.com/electron/electron/blob/master/docs/api/menu.md#menupopupbrowserwindow-options) for `Menu.popup()` is only supported on macOS. It would be great to have it on Linux and Windows as well.
Repository: vscode
Issue: Customizable status bar item locations
Labels: feature-request,workbench-status,keep

Hello

It would be really nice if one could customize the location of the items in the status bar at the bottom of the editor.  For instance, I would want to have git branch information on the right, text encoding and position on the left, etc.

Maybe allow people to sort things with something like 
```javascript
"window.statusBar.items": {
    "left" : ["cursorPosition", "tabWidth"],
    "right": ["textEncoding", "gitBranch", "feedbackButton" ] 
}
```
Repository: vscode
Issue: Input box completion support
Labels: feature-request,quick-pick

- VSCode Version: 1.17.0
- OS Version: Windows 10 x64

I'm not sure if I'm searching for the right terms, but there doesn't seem to be a way to get an extension to show an input box with completion entries, similar to the 'Show all commands' dropdown, or the 'Go to symbol in workspace' dropdown:

![image](https://user-images.githubusercontent.com/5137410/31306887-c0be31c0-aba4-11e7-93c7-52a484d0222d.png)

The closest that I can get is to use `showInputBox` with a `validateInput` input validator, but this only allows you to return one entry, which doesn't work well if there's multiple options that could be possible. There's also the quick picker, but as far as I know, it doesn't allow you to dynamically update the available options depending on what has been typed in.

Is it possible to expose a similar sort of functionality in the vscode API?
Repository: rust
Issue: Decouple NetBSD/amd64 builds from dynamic libgcc_s
Labels: T-compiler,O-netbsd,C-feature-request

The prebuilt binaries for NetBSD x86_64 require libgcc_s.so, but the base NetBSD system can be built as a "pure" clang build; in such cases the system will not contain libgcc_s.so.

To cover non-gcc NetBSD systems the bootstrap kit should be built using -static-libgcc.

See https://gnats.netbsd.org/cgi-bin/query-pr-single.pl?number=52584
Repository: youtube-dl
Issue:  Unsupported TV Provider NBC free profile to view full episode with new Tokens System
Labels: tv-provider-account-needed

Microsoft Windows [Version 6.1.7601]
Copyright (c) 2009 Microsoft Corporation.  All rights reserved.

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', '--no-check', '--ap-mso', 'NBC', '--ap-usernam
e', 'PRIVATE', '--ap-password', 'PRIVATE', 'http://www.nbc.com/today/video/today
-oct-5-2017/3590873']
Usage: youtube-dl [OPTIONS] URL [URL...]

youtube-dl: error: Unsupported TV Provider, use --ap-list-mso to get a list of s
upported TV Providers

I've **verified** and **I assure** that I'm running youtube-dl **2017.10.07**

Using latest version youtube dl sorry i did in last issue include verbose, NBC changed the policy for viewing the video, now needed to create a NBC profile to view free to view video with lock icon you can create a free account here

https://www.nbc.com/sign-up

before this change you did not need auy profile and download on youtube worked fine, if you removed (s) from https

when done you get 3 free credits for new account, each credit allow you view a full locked episode,, you will need a US ip/proxy to view as it is geo locked, but i have tested this and it works the video plays if you are logged into free nbc profile it does not need to be linked to a US TV provider as long as you tokens to unlock a episode.

https://www.nbc.com/today/video/today-oct-6-2017/3590881

i need some help and understand how to use the commands --ap-mso --ap-username --ap-password

does these command support this new NBC profile and token system nbc had changed too since start of this week

hope someone can look into all this.
Repository: vscode
Issue: Can we support CR line endings as well as LF and CRLF?
Labels: feature-request,editor-textbuffer

VSCode supports LF line endings, and CRLF line endings, but it doesn't support CR line endings.

Any chance of support for this? There _are_ some exotic platforms which still require them, such as e.g. pre OSX MacOS, and the amount of work involved ought to be trivial. This is ff=mac mode in vim.

I've tried manually setting files.eol to "\r", but the value is rejected as being invalid. 

Right now, if you try to open a CR file in VSCode it gets normalised to CRLF, which is incorrect. At the very least stopping this from happening would be nice.
Repository: flutter
Issue: UserAccountsDrawerHeader - Integrate account management
Labels: c: new feature,framework,f: material design,package,P3,team-design,triaged-design

Would it be possible for the UserAccountsDrawerHeader to be integrated with [AccountManager](https://developer.android.com/reference/android/accounts/AccountManager) in Android and in settings bundle/within app in iOS (whichever is better)?

This would help standardize the sign in/ sign out flow, or atleast provide a template to implement this most common of all functionalities. This would also help volks understand the best practice for how to check if the user is signed in before showing a page/screen, and react if the user signs out while an action is in progress. An analogy for this functionality is the router hooks (CanActivate et al) in AngularDart [1].  The Flutter router Fluro [2] does not seem to have the hooks/guard mechanism.

In case this mechanism is better implemented via the Observer pattern/Streams/Rx*, that too would be a welcome discussion. Some guidance is better than nothing.

Filing this request since I could not find an issue for this. Apologies in case there is already one. Also, in case it is relevant, this assumes that the authentication mechanism does not use Firebase.

[1] - https://github.com/dart-lang/angular/blob/master/angular_router/example/lib/admin/admin_component.dart
[2] - https://github.com/goposse/fluro
Repository: flutter
Issue: UserAccountsDrawerHeader - Implement/document user account switching
Labels: framework,f: material design,d: examples,P2,team-design,triaged-design

UserAccountsDrawerHeader does not seem to switch user accounts when another user account image is tapped on. May be the animations need to be triggered via some calls, may be the relevant calls are missing, may be everything required to do this is already there and I cannot see it.

In any case, the expected state is that when the user clicks on the small CircleAvatar, the user accounts should switch. Relevant animations, if any, should be performed. Not sure about the animations part because the material spec does not seem to suggest any [1], but I am sure that the user account switching needs to be communicated to the user through some visual mechanism.

With regards to the actual user account switching within the app, and the relevant changes, the app needs to be notified about the account switching. Would this be best implemented as some kind of an event delivered on a user account lifecycle subscription, in conjunction with #12447 ? Apologies if this does not make any sense, I am trying to wrap my head around this myself.

[1] - https://material.io/guidelines/patterns/navigation-drawer.html
Repository: godot
Issue: Inconsistent execution order of _integrate_forces
Labels: bug,confirmed,topic:physics

**Operating system or device, Godot version, GPU Model and driver (if graphics related):**
v2.1.4 stable

**Issue description:**
The standard 2d physics step performs force integration before velocity integration. Rigid Bodies have a method binding called `_integrate_forces` that's designed to replace force integration with custom logic. However, within a standard step, the `_integrate_forces` method executes _after_ the velocity integration stage, rather than before.

This is mainly a consistency issue. It can partially be worked around by inserting integration code into the `_fixed_process` method, but this can result in minor race conditions if other nodes attempt to modify a rigid body's physics parameters during the fixed physics stage.
Repository: neovim
Issue: startup: precedence of :syntax/:filetype autocmds
Labels: defaults,ux

<!-- Before reporting: search existing issues and check the FAQ. -->

- `nvim --version`: NVIM v0.2.1-915-g01e53a5
- Vim (version: ) behaves differently? no
- Operating system/version: WSL
- Terminal name/version: conhost/Windows 10
- `$TERM`: xterm-256color

### Steps to reproduce using `nvim -u NORC`

```fish
~> cat minimal.vim
autocmd FileType md set syntax=markdown filetype=markdown
~> nvim -u minimal.vim
:set ft=md
:set ft #prints markdown
:set syntax #prints md
```

whereas

```fish
~> nvim -u NORC
:autocmd FileType md set syntax=markdown filetype=markdown
:set ft=md
:set ft #prints markdown
:set syntax #prints markdown
```

Vim does not exhibit this behavior when started up with `minimal.vim` containing the same.
Repository: TypeScript
Issue: Customise extensions that import intellisense recognises
Labels: Suggestion,In Discussion,Domain: Completion Lists,VS Code Tracked

_From @Rich-Harris on October 6, 2017 15:7_

As of VSCode 1.17, typing `import x from './` offers autocomplete suggestions of local files and directories. However, the list of file extensions that are recognised is somewhat arbitrary — .js files are recognised, as are .ts and .jsx, but .json isn't (despite the fact that the Node handles .json files natively).

Meanwhile, when building web applications with bundlers like Webpack and Rollup, it's common to import files with extensions like .css, .html, .vue, .tsv and so on.

Is there some way to customise the list of extensions that are recognised? An alternative approach could be to recognise all file extensions rather than attempting to filter them.

_Copied from original issue: Microsoft/vscode#35731_
Repository: go
Issue: encoding/json: Multiple issues with decoding of nulls into ",string" tagged fields.
Labels: NeedsInvestigation

I've been playing around with the `megacheck` tool, and found that `NullTestStrings` type in `encoding/json`'s tests is unused. It seems to be aimed at checking decoding of nulls into `",string"` fields, but there were no tests, which is the first issue. The second is that if you create one, it fails: https://play.golang.org/p/5XjdhoDDkq.

```
--- FAIL: TestUnmarshalStringNulls (0.00s)
	main.go:126: Unmarshal of null values failed: MustNotUnmarshalJSON was used
	main.go:138: Unmarshal of null did not clear nulls.Map
	main.go:141: Unmarshal of null did not clear nulls.Slice
	main.go:144: Unmarshal of null did not clear nulls.Interface
	main.go:147: Unmarshal of null did not clear nulls.PRaw
	main.go:150: Unmarshal of null did not clear nulls.PTime
	main.go:153: Unmarshal of null did not clear nulls.PBigInt
	main.go:156: Unmarshal of null did not clear nulls.PText
	main.go:159: Unmarshal of null did not clear nulls.PBuffer
	main.go:162: Unmarshal of null did not clear nulls.PStruct
	main.go:166: Unmarshal of RawMessage null did not record null: 123
FAIL
```

It seems to me that there are some inconsistencies between decoding `null` into a simple field and decoding `"null"` into a `",string"` field.

Formalities:
```
$ go version
go version devel +ca360c3 Sat Oct 7 22:12:36 2017 +0000 linux/amd64
$ go env
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/ainar/go"
GORACE=""
GOROOT="/home/ainar/go/gotip"
GOTOOLDIR="/home/ainar/go/gotip/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build220310604=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
```
Repository: flutter
Issue: We should exclude flutter_tools dependencies from flutter API docs
Labels: tool,d: api docs,P2,team-tool,triaged-tool

In particular, having the "file" package in there is very confusing.
Repository: go
Issue: cmd/compile: consider adding sibling links to generated DWARF
Labels: NeedsInvestigation,Debugging,compiler/runtime

The DWARF .debug_info section generated by the current Go compiler
does not make use of sibling links at the moment. As gc-generated
DWARF DIE's begin to get more substantial/complicated, this will make
it more time-consuming for debugger as it tries to locate the DIE for
a specific entity within the .debug_info section.

It might make sense to look into introducing siblings to DIEs such as
the subprogram DIE and possibly the structure type DIE (for use in cases
where you have a struct with a large number of members).

Note that such a change would provide little benefit for Delve users
without a corresponding change to Delve (looking at the Delve tip
source code, it appears that the DWARF reader is not set up to exploit
sibling links).

@aarzilli @heschik
Repository: opencv
Issue: IMREAD_LOAD_GDAL produces images with RGB band order
Labels: incomplete


**System information (version)**

- OpenCV => 3.3.0
- GDAL => 2.2.1
- Operating System / Platform => Linux 64 Bit (CentOS 7)
- Compiler => GCC 7.2

**Detailed description**

Hello, when reading color raster image using GDAL (IMREAD_LOAD_GDAL flag), the band order is RGB, which is reversed from opencv standart one -> BGR. This is not a big problem, but still challenging when I want to save image after some manipulation/or overlay something on it and then write it using imwrite. 


**Steps to reproduce**

just open any image with IMREAD_LOAD_GDAL flag - imread(file, IMREAD_LOAD_GDAL). 

Thank you
Repository: rust
Issue: Grammar ambiguity with left shifts and fully qualified (UFCS) paths
Labels: C-enhancement,A-grammar,A-parser,P-low,T-lang

Playground: https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=35f2e9b9c6d48c33199bc878363a15df

```rust
trait Trait {
    const CONST: u8;
}
impl Trait for *const u8 {
    const CONST: u8 = 1;
}

fn main() {
    0<<*const u8 as Trait>::CONST;
}
```

`0<<*const u8 as Trait>::CONST` here unambiguously means `0 < (<*const u8 as Trait>::CONST)` because it can't be parsed as a shift, but rustc tries to parse it as a shift and report an error:
```
error: expected expression, found keyword `const`
 --> src/main.rs:9:9
  |
9 |     0<<*const u8 as Trait>::CONST;
  |         ^^^^^
```

Note that you can't detect this situation using limited lookahead in general case, for example
```rust
0<<Trait1 + Trait2 + ... + 'static as Trait>::CONST
```
is unambiguously a type too, but we don't know it until we see `'static`.

This syntax allows to construct fully ambiguous examples as well, e.g.
```rust
x<<y as z>::a
```
can be interpreted as both `((x << y) as z) > (::a)` and `x < (<y as z>::a)`.
Repository: TypeScript
Issue: [typescript] dom definitions not up to date
Labels: Bug,Help Wanted,Domain: lib.d.ts,VS Code Tracked,Effort: Casual

_From @yukulele on October 5, 2017 12:7_

<!-- Do you have a question? Please ask it on http://stackoverflow.com/questions/tagged/vscode. -->

<!-- Use Help > Report Issues to prefill these. -->
- VSCode Version: 1.16.1
- OS Version: macOS Sierra 10.12.6

Steps to Reproduce:

* Create this code in typescript file
```typescript
const elm = document.createElement('main')
```
* `elm` is typed as `HTMLElement` instead of `HTMLMainElement`

`lib.dom.d.ts` is not up to date

how to update it without wait for vscode update ?

<!-- Launch with `code --disable-extensions` to check. -->
Reproduces without extensions: Yes

_Copied from original issue: Microsoft/vscode#35620_
Repository: go
Issue: proposal: spec: find a way to export uncased identifiers
Labels: LanguageChange,Proposal,LanguageChangeReview

https://github.com/golang/go/issues/5763#issue-51284151 observes “It is very strange to use, say Z成本 or Jぶつける as identifiers.” In that issue we discussed potentially changing the default export rule, but as of https://github.com/golang/go/issues/5763#issuecomment-333669811, which seemed to have general agreement, we decided against that.

Even so we do want to find a way to export uncased identifiers, or at least consider ways, in order to address the original observation.

This issue is for discussion of non-breaking ways to export uncased identifiers.
Repository: go
Issue: proposal: x/net/sctp: new package
Labels: Proposal

# Background

This proposal will make easy to develop SCTP based applications such as HSS or MME in cellular networks.

I'd like to add this proposal to core library, because the one-to-one style SCTP interface on Linux is very similar to TCP interface. If we make an another package or add this to golang.org/x/net, it will make 
large copy of code from `net` and `internal/poll` package.

# Proposal

Add one-to-one style SCTP support to net package.

* Support sctp/sctp4/sctp6 in net.Listen and net.Dial.
* Add 3 functions and 3 structs to net packages.
    * structs
        * SCTPAddr
        * SCTPConn
        * SCTPListener
    * functions
        * func DialSCTP(string, *SCTPAddr, *SCTPAddr) (*SCTPConn, error)                                                      
        * func ListenSCTP(string, *SCTPAddr) (*SCTPListener, error)                                                           
        * func ResolveSCTPAddr(string, string) (*SCTPAddr, error)

This proposal supports only one-to-one style interface with `SOCK_STREAM`.