labofsahil/github-event-dataset-2012 huggingface dataset:
I tweaked the wording on some of these, and changed single quotes to back-ticks so we get the code font.
no need to nil check
good catch, probably forgot to clean up this piece
Would also be nice to add a comment as to why this is called with `bounds` instead of `boundingValue`.
Duplicates many times through the code. Macro/ static function instead ?
we only use the author annotation at the class level, not for each method
Don't leave uncommented code.
Hm... File holds resource so it's a type.
lines gets a range over a file, so I'm inclined to say it's like retro(x).
I can accept that Lines is typeof(lines(x)).
Plus - with new I/O I suppose lines(x) should be working on newer streams as well 
thus probably becoming an overloaded function.
Why is this message camel case? 'range is empty' is better.
maybe you should adapt your formater to the "old" eclipse one?
item should have braces for clarity.
This is a very long line with an `if` modifier, which is basically impossible to read. Can you split this down more?
You could use constants/Resources for parameter names like "userName" as seems that if u decided to change, you will touch multiple places.
Is there a reason why the intel compiler isn't used? Or will that be done next?
Yeah, I guess we might as well try and localize the change to just linux+glibc combinations.
pretty sure this fails if $properties is a list. We need to map the prefix onto each of the values of the list.
@croby: It will. Also, if .content doesn't exist, the code will still work.
`= size_t.max`
Well, the only languages I know that throw NPE/NRE on any invalid memory access are Java and C# (.NET in general). Java does it because Java doesn't have pointers in the first place, so anything else wouldn't make sense. C# does it due to Java heritage and because pointers were an afterthought in the design. In fact, I've had plenty of "wat" moments in C# when my invalid pointer accesses resulted in a `NullReferenceException` when nothing was null. I find it very unintuitive, personally, because when I write pointer operations I don't think about what they may be at a higher level; I think about what they actually are on machine level.
ditto: `cb = STARTUPINFO_W.sizeof;`
*Inefficient code*
Typical convention is to write TODO: when making a note. A number of editors (including Eclipse) pick up on it and can mark it.
I pulled the comment as it was a copy as paste fail, I made a commit with it in as it made sense.
Oh actually, disregard my latter statement about `cattr`.  They are class attributes of the class you're extending, even though they're instance attributes of the singleton class of this class.
i like constants
Updated the link to docs in https://github.com/zendframework/ZendService_Amazon/pull/32
Limitation?  I would call that a show-stopper.
commenting on the throw-away change that I won't be accepting
@ekohl Accidental.  Fixed now - thanks!
typo in deployment
please remove commented code that is not used
1. Я не понял комментария на счет "самодокументированности кода".
Писать комментарии? - мне кажется это избыточно в данной ситуации (так как не очевидного кода нет) - DRY
Писать тесты в которых описано как и что работает? - я это вроде бы написал в той или иной степени 
Писать вики документацию?  - когда я устраивался на стажировку мой код с вики документацией "обсмеяли" - DRY.
2. Если мы засунем все в функцию:
  + ограничили видимость функции
  -  я не понимаю как внутренний функции тестировать
Working bullshit.
The `http://wiki.openstreetmap.org/wiki/Using_OpenStreetMap` URL here should be put in the translations file like the partners_url one above, so that translators can point to a locale specific wiki page if it exists.
Initially the line 65 was added by me, but later I commented it out.

My reasoning was to "save" any further re-parsing of same DOM and reuse it. But, I am not convinced that there _is any execution path_ that might _reuse_ the DOM. Also, later I realized I'd rather "eat" CPU than "eat memory", as the fix is exactly about _lessening_ the memory consumption. Also, with this line uncommented, we would end up where we was before: all DOMs of all member repositories "alive" at the same time in heap.

The numbers I measured are _after_ this line commented out.
Whitespaces. 
groups = Group.objects.order_by('id').all()
JMX is not really provided, it's only a Abstract JMX protocol. JMX on it's own has no default way to Package it self. So a JMX implementation can use the JMX protocol as a base, then provide the packaging for the given env.
Changed all these methods to camel case.
I'm not sure putting bundle exec here is 1) wise, nor 2) required (if you have textmate running ruby through rvm)
Agree. The info in your comment is probably more valuable than the code itself :)
es tu sûr de vouloir ça ?
This shouldn't affect the normal flow. It is required for IP Bypss flow.
I think we should have a test case for the negative scenario of an invalid value.
Please revert if there are no significant changes to the file
expect is  commented out?
This does not actually exclude the folder 'auto-generated-pojos'
Comment as well.
I've propose a change here https://github.com/doctrine/doctrine2/pull/401/files
Needs a @return
-ABI