sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: go
Issue: cmd/compile: move Type.Width and Type.Align into Extra fields
Labels: ToolSpeed,compiler/runtime

This is a placeholder issue for work I plan to do, probably not for Go 1.7 at this point. If anyone else wants to work on it, though, they are welcome to.

Only two gc.Types have potentially expensive to calculate width and alignments: structs and (maybe) arrays. We should move Type.Width and Type.Align into StructType and ArrayType, thus shrinking gc.Type for the vast majority of uses.

However, Width is currently overloaded to track some typechecking state. Plan is to change Deferwidth from a bool to a bitflag and start explicitly representing that state--has a width calculation been done? Is the width calculation deferred?

Also, most callsites refer to Width and Align directly. We'll need to switch them to use the Size method and a new Align method, remove unnecessary direct calls to dowidth, and teach the Size and Align methods the constant width and alignment for non-struct, non-array types.
Repository: rust
Issue: Do move forwarding on MIR
Labels: I-slow,C-enhancement,T-lang,T-compiler,A-MIR,WG-embedded,A-mir-opt,A-mir-opt-nrvo,C-optimization

It'd be cool to rewrite chains of the form "a moves to b, b moves to c" to "a moves to c".
Repository: opencv
Issue: cleaning up the internal TLS object when the core DLL is programmatically unloaded
Labels: feature,priority: low,category: core

### Please state the information for your system
- OpenCV version: 3.1.0
- Host OS: Windows 7 x86_64
- Compiler: MSVS/VC 2012
- Options: MS memory allocation debugging used; delay load of DLLs for allowing to programatically unload them in a deterministic fashion before e.g. the main routine exits
### In which part of the OpenCV library you got the issue?
- core DLL
### Expected behaviour

all ressources should be cleaned when DLL is unloaded
### Actual behaviour

some still not released chunks of memory do pop up. (i have some idea of a solution attached but i assume several people here, the OpenCV developers, do have a deeper understanding and might pop up with an even better solution.)
### Additional description

the compiler setup was explicitely choosen like this for hardening memory consumption of the application and the used library set at the same time by successively eliminating memory leaks in all areas. 
### Code example to reproduce the issue / Steps to reproduce the issue

this patch is an idea (a not yet tested proposal) on how to do a proper cleanup when the DLL is finally detached from the process it lived in.
[system-tls-delete-on-unload.zip](https://github.com/Itseez/opencv/files/221230/system-tls-delete-on-unload.zip)

add this (semicolon separated listing) for the MSVC project properties in debug build variant for ConfigurationProperties->C/C++->Preprocessor->PreprocessorDefinitions
- _CRTDBG_MAP_ALLOC

set this for the MSVC project properties in debug build variant for ConfigurationProperties->Linker->Input->DelayLoadedDlls
- opencv_core310d.dll

set this for the MSVC project properties in debug build variant for ConfigurationProperties->Linker->Advanced->UnloadDelayLoadedDLL
- Yes (/DELAY:UNLOAD)

```
// use this header
#ifdef _DEBUG
    #include <stdlib.h>
    #include <crtdbg.h>
#endif
// initially do this
_CrtMemState MemState;
_CrtMemCheckpoint (&MemState);
// first use something from the OpenCV core dll that triggers TLS item creation
// then call this function
__FUnloadDelayLoadedDLL2_i("opencv_core310d.dll");
// and finally check for any still allocated memory (=any memory leaks)
_CrtMemDumpAllObjectsSince (&MemState);


```
Repository: rust
Issue: inconsistent stepping in gdb
Labels: A-debuginfo,E-needs-test,T-compiler,C-bug

While experimenting with gdb I found I couldn't really predict what statements I might step to.  Consider this test program:

```
fn main () {
    let a = ();
    let b : [i32; 0] = [];
    let mut c = 27;
    let d = c = 99;
    let e = "hi bob";
    let f = b"hi bob";
    let g = b'9';
    let h = ["whatever"; 8];
    let i = [1,2,3,4];
    let j = (23, "hi");
    let k = 2..3;
    let l = &i[k];
    let m: *const() = &a;
}
```

If I `break step::main` and `run` in gdb, stepping stops on many of these lines, but not all of them:

```
Breakpoint 1, step::main () at step.rs:4
4       let mut c = 27;
(gdb) n
5       let d = c = 99;
(gdb) 
7       let f = b"hi bob";
(gdb) 
8       let g = b'9';
(gdb) 
12      let k = 2..3;
(gdb) 
13      let l = &i[k];
(gdb) 
14      let m: *const() = &a;
(gdb) 
15  }
```
Repository: rust
Issue: gdb needs to know what traits a type implements
Labels: A-debuginfo,A-trait-system,P-low,T-compiler,C-feature-request

I'd like to make operator overloading work for Rust in gdb.  That is, I'd like something like `print x+y` to call the appropriate method on the `Add` trait.  I'd also like to make ordinary method calls work, where the method is defined in some trait that is impl'd for the concrete type of the object.

Right now I think this can't be done.  There is no information in the debug info about what traits are implemented by a type.

Consider:

```
pub trait What {
    fn what(&self);
}

impl What for i32 {
    fn what(&self) {
        println!("{}", self);
    }
}

fn main() {
    let v = 23;
    let x = &v as &What;

    x.what();
    ()
}
```

Here `i32` is described as just a base type:

```
 <1><5e7>: Abbrev Number: 16 (DW_TAG_base_type)
    <5e8>   DW_AT_name        : (indirect string, offset: 0x46a): i32
    <5ec>   DW_AT_encoding    : 5   (signed)
    <5ed>   DW_AT_byte_size   : 4
```

I think this might be a good spot to list all the impl'd traits.  Perhaps they can be represented as DWARF interfaces; or maybe some Rust extension to DWARF would be preferable.

`i32.What` is emitted as a namespace:

```
 <2><2f>: Abbrev Number: 2 (DW_TAG_namespace)
    <30>   DW_AT_name        : (indirect string, offset: 0x48): i32.What
```

... but this isn't directly useful as it would require an iteration over all the namespace DIEs looking for matches.  Maybe I could do this; but I'd rather not as it is very inefficient.
Repository: vscode
Issue: Add support for mixed tab/spaces indentation
Labels: feature-request,editor-core,editor-rendering

Traditional Unix editors (emacs, vim etc) use a mix of tabs and spaces for indentation. The tab length is always assumed to be 8 and if the indentation size is different, indentation is created by  inserting as much tabs as possible and inserting spaces for the remainder. Right now, in order to view such files correctly one needs to set the tab size to 8, convert the file to use only spaces and then set the indentation size to the correct value. However, when working on legacy code it is desired to keep the indentation method the same to avoid unnecessary commits. Therefore VSCode should support this method of indentation, auto detect it and allow to easily convert from and to it.
- VSCode Version: 1.0.0
- OS Version: OS X 10.11.4

Steps to Reproduce:
1. Load a file with mixed indentation
2. VSCode won't display the file correctly
Repository: vscode
Issue: Improved column selection (Alt+select)
Labels: feature-request,editor-columnselect

The column selection implemented in #1515 is a start, but it's a bit awkward and unintuitive compared to other editors.

With the following editors or IDEs, you can create a column selection starting from where the mouse is clicked/dragged:
- Atom (with Sublime Style Column Selection)
- Eclipse
- Notepad++
- Programmer's Notepad
- Sublime
- Visual Studio 2015

At least Eclipse, Programmer's Notepad, and VS2015 support virtual whitespace as well, creating a complete square/rectangle.

Some of them also don't show multiple cursors, which I think looks better.

Other users were also expecting this behaviour (see [here](https://github.com/Microsoft/vscode/issues/1515#issuecomment-199464559) and [here](https://github.com/Microsoft/vscode/issues/1515#issuecomment-205910037)).
Repository: TypeScript
Issue: Runtime parameter info (name, optional, default)
Labels: Suggestion,Revisit,Domain: Decorators

<!--
Thank you for contributing to TypeScript! Please review this checklist
       before submitting your issue.
[ ] Many common issues and suggestions are addressed in the FAQ
     https://github.com/Microsoft/TypeScript/wiki/FAQ
[ ] Search for duplicates before logging new issues
    https://github.com/Microsoft/TypeScript/issues?utf8=%E2%9C%93&q=is%3Aissue
[ ] Questions are best asked and answered at Stack Overflow
    http://stackoverflow.com/questions/tagged/typescript

For bug reports, please include the information below.
__________________________________________________________ -->

**TypeScript Version:** 

1.8.9

**Code**

I'm trying to leverage decorators to create runtime contracts to check against incoming parameters.

Below is the ts code

``` ts
class Foo {

  @contract
  test (a : integer, b : string) : boolean {
    return true
  }

  @contract
  testOptional (a : integer, b? : string) : boolean {
    return true
  }

  @contract
  testDefault (a : integer, b : string = 'hello') : boolean {
    return true
  }
}
```

The above generated the following Javascript.

``` js
var Foo = (function () {
    function Foo() {
    }
    Foo.prototype.test = function (a, b) {
        return true;
    };
    Foo.prototype.testOptional = function (a, b) {
        return true;
    };
    Foo.prototype.testDefault = function (a, b) {
        if (b === void 0) { b = 'hello'; }
        return true;
    };
    __decorate([
        contract, 
        __metadata('design:type', Function), 
        __metadata('design:paramtypes', [integer, String]), 
        __metadata('design:returntype', Boolean)
    ], Foo.prototype, "test", null);
    __decorate([
        contract, 
        __metadata('design:type', Function), 
        __metadata('design:paramtypes', [integer, String]), // same as above
        __metadata('design:returntype', Boolean)
    ], Foo.prototype, "testOptional", null);
    __decorate([
        contract, 
        __metadata('design:type', Function), 
        __metadata('design:paramtypes', [integer, String]), // same as above
        __metadata('design:returntype', Boolean)
    ], Foo.prototype, "testDefault", null);
    return Foo;
}());
```

Note that the three different ts functions generated the exact sets of `__metadata` calls. The `design:paramtypes` alone cannot distinguish whether a particular parameter is optional or has default values, and thus insufficient for building a runtime contract.

It would be great for TypeScript to support a full parameter info so decorators can be made more capable.
Repository: youtube-dl
Issue: Request for supporting youku&tudou account logining
Labels: request,account-needed

## Please follow the guide below
- You will be asked some questions and requested to provide some information, please read them **carefully** and answer honestly
- Put an `x` into all the boxes [ ] relevant to your _issue_ (like that [x])
- Use _Preview_ tab to see how your issue will actually look like

---
### Make sure you are using the _latest_ version: run `youtube-dl --version` and ensure your version is _2016.04.13_. If it's not read [this FAQ entry](https://github.com/rg3/youtube-dl/blob/master/README.md#how-do-i-update-youtube-dl) and update. Issues with outdated version will be rejected.
- [x] I've **verified** and **I assure** that I'm running youtube-dl **2016.04.13**
### Before submitting an _issue_ make sure you have:
- [ ] At least skimmed through [README](https://github.com/rg3/youtube-dl/blob/master/README.md) and **most notably** [FAQ](https://github.com/rg3/youtube-dl#faq) and [BUGS](https://github.com/rg3/youtube-dl#bugs) sections
- [x] [Searched](https://github.com/rg3/youtube-dl/search?type=Issues) the bugtracker for similar issues including closed ones
### What is the purpose of your _issue_?
- [ ] Bug report (encountered problems with youtube-dl)
- [ ] Site support request (request for adding support for a new site)
- [x] Feature request (request for a new functionality)
- [ ] Question
- [ ] Other

---
### The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your _issue_

---
### Description of your _issue_, suggested solution and other information

Now youtube-dl can download youku&tudou video, and it can download videos that only open for VIP accounts with browser cookies.
However, I find the browser cookies for youku&tudou will expire in only a few hours. So I must export the cookies every time before I download the VIP-only videos. This is too annoying.
Then, I find youtube-dl supports logining with account ID in the Authentication Options as:
-u, --username USERNAME          Login with this account ID
-p, --password PASSWORD          Account password. If this option is left
                                 out, youtube-dl will ask interactively.
But youtube-dl does not support youku&tudou account logining now.
Thus, I request for this functionality.
The VIP youku&tudou accounts can be obtained in these websites: 
http://www.vipfenxiang.com/youku/
http://yk.aiqiyivip.com/
http://www.mdouvip.com/youku
Repository: TypeScript
Issue: Use function type shorthands when displaying signature help
Labels: Suggestion,Help Wanted,API,VS Code Tracked

_From @unional on April 14, 2016 23:32_

![image](https://cloud.githubusercontent.com/assets/3254987/14546974/4f88973e-025e-11e6-8053-724cc8ecc51e.png)
I would be great if when `tape((<cursor>))` it would be able to drill in the `tape.TestCase` and show the signature of the callback. :tulip: 

This is probably on `tsc`, but let's start here. :smile:

_Copied from original issue: Microsoft/vscode#5284_
Repository: TypeScript
Issue: Cannot get symbol inside class decorator (Cannot read property 'members' of un...)
Labels: Bug,Help Wanted,API

**TypeScript Version:** 

1.8.10

**Code**

``` ts
@Component({
  selector: SELECTOR
})
class SampleComponent {}
const SELECTOR = 'ng-demo';
```

**Expected behavior:**

With sample [`SyntaxWalker`](https://github.com/palantir/tslint/blob/master/src/language/walker/syntaxWalker.ts) based on `tslint`, once I visit the `PropertyAssignment` - `selector: SELECTOR` and invoke `typeChecker.getSymbolAtLocation(prop.initializer);` I should get the `SELECTOR` symbol.

**Actual behavior:** 

```
     TypeError: Cannot read property 'members' of undefined
      at resolveName (node_modules/typescript/lib/typescript.js:15307:73)
      at resolveEntityName (node_modules/typescript/lib/typescript.js:15725:26)
      at getSymbolOfEntityNameOrPropertyAccessExpression (node_modules/typescript/lib/typescript.js:28708:28)
      at Object.getSymbolAtLocation (node_modules/typescript/lib/typescript.js:28770:28)
```
Repository: youtube-dl
Issue: Site request: h-a.no
Labels: site-support-request,geo-restricted

## Please follow the guide below
- You will be asked some questions and requested to provide some information, please read them **carefully** and answer honestly
- Put an `x` into all the boxes [ ] relevant to your _issue_ (like that [x])
- Use _Preview_ tab to see how your issue will actually look like

---
### Make sure you are using the _latest_ version: run `youtube-dl --version` and ensure your version is _2016.04.13_. If it's not read [this FAQ entry](https://github.com/rg3/youtube-dl/blob/master/README.md#how-do-i-update-youtube-dl) and update. Issues with outdated version will be rejected.
- [x] I've **verified** and **I assure** that I'm running youtube-dl **2016.04.13**
### Before submitting an _issue_ make sure you have:
- [x] At least skimmed through [README](https://github.com/rg3/youtube-dl/blob/master/README.md) and **most notably** [FAQ](https://github.com/rg3/youtube-dl#faq) and [BUGS](https://github.com/rg3/youtube-dl#bugs) sections
- [x] [Searched](https://github.com/rg3/youtube-dl/search?type=Issues) the bugtracker for similar issues including closed ones
### What is the purpose of your _issue_?
- [ ] Bug report (encountered problems with youtube-dl)
- [x] Site support request (request for adding support for a new site)
- [ ] Feature request (request for a new functionality)
- [ ] Question
- [ ] Other

---
### If the purpose of this _issue_ is a _site support request_ please provide all kinds of example URLs support for which should be included (replace following example URLs by **yours**):
- Single video: http://www.h-a.no/sport/se-hoydepunktene-her?AutoPlay=true

---
### Description of your _issue_, suggested solution and other information

Hi, I don't know if I understood this new guide thing, but I'll try :-) I just wanted to request support for h-a.no, as in the above URL. :-) Thanks!
Repository: go
Issue: git-codereview: allow disabling of "foo.mailed" tag creation
Labels: NeedsInvestigation

Every time you run "git codereview mail" it creates a new tag (as per [its doc](https://godoc.org/golang.org/x/review/git-codereview)). 

I've never actually used these tags for their advertised purpose (if I did need to diff against my last mailed commit, I would just use the advertised commit hash from within gerrit), so I don't benefit from them. 

OTOH, they do interrupt my workflow. For example, the [common git bash completion script](https://github.com/git/git/blob/master/contrib/completion/git-completion.bash) prints all pending tags when auto-completing `git commit <TAB>`, and the sheer number of mailed tags obfuscates any useful information.

So, this is a feature request to disable their automatic creation.
Repository: go
Issue: debug/pe: extend package so it can be used by cmd/link
Labels: help wanted,NeedsFix

CC: @minux @ianlancetaylor @crawshaw 

cmd/link and debug/pe share no common code, but they should - they do the same thing. If package debug/pe is worth its weight, it should be used by cmd/link. I accept that this was impossible to do when cmd/link was a C program, but now cmd/link is written in Go.

I also tried to use debug/pe in github.com/alexbrainman/goissue10776/pedump, and it is not very useful. I endup coping and changing some debug/pe code.

I also think there is some luck of PE format knowledge between us. So improving debug/pe structure and documentation should help with that.

I tried to rewrite src/cmd/link/internal/ld/ldpe.go by using debug/pe (see CL 14289). I had to extend debug/pe for that task. Here is the list of externally visible changes I had to do:
- PE relocations have SymbolTableIndex field that is an index into symbols table. But File.Symbols slice has Aux lines removed as it is built, so SymbolTableIndex cannot be used to index into File.Symbols. We cannot change File.Symbols behavior. So I propose we introduce File.COFFSymbols slice that is like File.Symbols, but with Aux lines left untouched.
- I have introduced StringTable that was used to convert long names in PE symbol table and PE section table into Go strings.
- I have also introduced Section.Relocs to access PE relocations.

I propose we add the above things to debug/pe and use new debug/pe to rewrite ldpe.go and pe.go in cmd/link/internal/ld.

Alex

PS: You can google for pecoff.doc for PE detials.
Repository: opencv
Issue: minMaxIdx not available in the JNI binding
Labels: feature,affected: 3.4,category: java bindings

Hi,
I noticed that the function minMaxIdx (core) is not available through the JNI binding. Only minMaxLoc is available, which is a problem given that minMaxLoc is limited to 2-dimensional arrays.
- OpenCV version: 3.1.0
- Host OS: Linux (Mint 17.3)

This issue was discussed here: http://answers.opencv.org/question/92676/minmaxidx-missing-in-the-jni-interface
Repository: react
Issue: Iframe load event not firing in Chrome and Safari when src is 'about:blank'
Labels: Type: Bug,Component: DOM

See: https://jsfiddle.net/pnct6b7r/

It will not trigger the alert in Chrome and Safari, but it will work in Firefox and even IE8.

Is this a React issue or Webkit issue? If it is a Webkit issue, should we "fix it" in React given that we want [consistent events across browsers](http://facebook.github.io/react/docs/events.html)?

ps: The JSFiddle was based on the isuse #5332.
Repository: opencv
Issue: Mask is ignored in cv::CascadeClassifier
Labels: feature,category: objdetect

Output of mask generated by maskGenerator of cv::CascadeClassifier (see https://github.com/Itseez/opencv/blob/master/modules/objdetect/src/cascadedetect.cpp#L996) is ignored in detection step. It is assigned, but never actually used.
### In which part of the OpenCV library you got the issue?
- objdetect
### Expected behaviour

When my program provides custom (subclass of) cv::BaseCascadeClassifier::MaskGenerator object to cv::CascadeClassifier, it should be using the masks it produces to apply the detection algorithm only on areas defined by mask (and therefore speeding up the process when mask is "sparse").
### Actual behaviour

The detection is run on whole area of image anyway and it is very slow for huge images with "sparse" detection mask.
### Additional description

I have troubles to build and test OpenCV myself, but I have managed to edit "source\opencv\modules\objdetect\src\cascadedetect.cpp" and add mask check by adding following lines on line 996:

``` c++
if(mask.empty() ||
  ((x >= 0) && (y >= 0) &&
  (x < mask.cols) && (y < mask.rows) &&
  (mask.at<uchar>(y,x) != 0)))
{
```

and putting lines 996 to 1022 into the if block.
Repository: go
Issue: flag: handles unknown arguments in an unexpected less-helpful way
Labels: NeedsDecision

If the flag library encounters an unknown parameter, it discards that parameter and dumps everything else in the positional arguments bucket.

Instead, the flag library should (optionally, probably when ContinueOnError is specified) put all unknown parameters in the positional arguments bucket (returned by Args) and keep attempting to parse the following arguments. (unless encountering something that clearly indicated the beginning of positional arguments such as the naked double-dash "--")
1. What version of Go are you using (`go version`)?
   go version go1.3.3 linux/amd64
2. What operating system and processor architecture are you using (`go env`)?
   GOARCH="amd64"
   GOBIN=""
   GOCHAR="6"
   GOEXE=""
   GOHOSTARCH="amd64"
   GOHOSTOS="linux"
   GOOS="linux"
   GOPATH="/home/azani/go"
   GORACE=""
   GOROOT="/usr/lib/go"
   GOTOOLDIR="/usr/lib/go/pkg/tool/linux_amd64"
   CC="gcc"
   GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0"
   CXX="g++"
   CGO_ENABLED="1"
3. What did you do?
   https://play.golang.org/p/44mXvwTfLD
4. What did you expect to see?

list == "first", "second", "third"
flagSet.Args() == "--something", "blah"
1. What did you see instead?
   list == "first", "second"
   flagSet.Args() == "-I", "third", "blah"
   also, Parse printed an error.