gh-test.txt

sent
"Guess there is a typo here: `translate`."""
"@arturoc: If you multiply-include `gst.h`, wouldn't it be good to add some include guards, so that we don't include it several times, driving up compilation/linking times?"""
"Thank you Vlad. Your contribution to Mangos and TC will be remembered. I admire you so much."""
"@opdenkamp Hi Lars, I'm afraid that you forgot to upload the file to the server. If this is automated and just takes some time, I apologize."""
"Ok, so let's be paranoid."""
"Instead of duplicating warden logic, maybe the test could simply call warden.authenticate!() in MyController's before filter passing the strategy?"""
"Oh sorry!   Just need to add a reference to the youtube page in docs/source/userguide/settings/plugins/index.rst. That's this page: http://www.thinkupapp.com/docs/userguide/settings/plugins/  Also the developer guide you wrote isn't linked anywhere in"""
"It is trivial, it can be changed if you consider it appropriate... in fact, I just added this here for consistency I suppose.   Cause the easiest way to 'hack' into generating enum values was to use PHPMyAdmin's approach: to use the (already available) """
"yes, :( Last time i specially check mangos.sql Db version at real use with server start. So must be fixed now... and no other simialr problems"""
"one thought i did have would be to use it in config.ru, but then we lose the ability to use AppConfig :("""
"Should cache `jQuery.support.elementTraversing` to a var locally"""
"@trustin it's named """"race_fix"""" .... Thanks :)"""
"Would be to easy ;) and to unobfuscated :D  Fabian"""
"This breaks many plugins which use Players as keys in HashMaps and keep them even if the player logs out & back in again. Maybe you should add a way to make a player's entity id persistent as long as the server is running? (I'm opening an issue as well if """
"You can avoid using the HashSet with something like ```   ids.flatMap {     // return None if we don't want to include the id     // return Some(id) if we do   }.toSet ```"""
"DQL injection is possible here. you should not pass the value (which is a user input) using string concatenation. You should use a query parameter here and set the value for the parameter"""
"The docs here describe behaviour that is pretty different from what is implemented in the code. The passed value gets constantized if it does not respond to #delivering_email, which actually is a bit weird."""
"This breaks php-git build :("""
"Use ruby 1.9 hash syntax"""
"I hate you guys. :( I removed all tabs at once and re-done them manually as I copied the code from a version with too many tabs!"""
"given that this could be arbitrary data, do we want to do a hex dump instead? (ChannelBuffers.hexDump(commandName))"""
"https://github.com/EdGruberman/CraftBukkit/commit/9c2f7ab206aa1e580dd8f3cbd34a202c59fbc64a >what's in 'a' name? that which we call a merge...  :-)"""
"Since I really don't understand this cryptic lines I just post the error output for you to review: ``` 2>C:\Development\xbmc\xbmc\utils/GlobalsHandling.h(184): warning C4346: 'xbmcutil::GlobalsSingleton<T>::?$Deleter@V?$shared_ptr@$RX@BAAB@@boost@@' : depe"""
"Umm, there's plenty in there that don't :P"""
"tradition with waypoint record ^^"""
"Is this threadsafe?"""
"The bundler fix is great, but `gem install` will still give a confusing prompt about rails overwriting its own stuff :'("""
"you need a blank line after the title so that the next few lines go to `\description{}`; BTW, usually I do not end the title with a period"""
"hmm this is weird, is this the autoformatter?"""
"According to http://tools.ietf.org/html/draft-ietf-httpbis-p5-range-22#section-4.2, Content-Range is undefined on response codes other than 206 and 416.  Unless chrome needs the Content-Range header to be present on this 200 response, this set_header call"""
"Heh, yeah :)"""
"Probably costs as much to do:  for(i <- v.inbound.averageValue; o <- v.outbound.averageValue) yield ((i,o))"""
"You did, I don't want to live on this planet anymore :("""
"Don't you mean Lives?"""
"Also, sorry for this, I should have revert earlier, I just thought that I will be able to fix it quicker."""
"why not remove autoStart parameter and class property?"""
"There is no way for us to know the root of your Octopress directory, as `lib/` will be distributed as a gem, much like `jekyll`. Do you know of a better way to find this directory?  As you can see from the diff, this used to be an absolute directory rel"""
"default should be from the config not in code"""
"Yep... I guess this should be faster  Am 07.07.2013 um 10:30 schrieb Trustin Lee <notifications@github.com>:  > We can do that, too. We'll have to duplicate and retain it every time. >  > On Sun 07 Jul 2013 04:18:22 PM KST, Norman Maurer wrote: > > """
"Ah ... sorry. Okay :/"""
"The sentence is weird. It should probably be rephrased to be clearer."""
"wouldn't inject be more succinct?  e.g.  <pre><code>   filter_chain.inject(Array.new) do |filters, filter_item|     filters << filter_item if filter_item.before?     filters   end </code></pre>"""
"Why does it require a HashSet?"""
"You can configure any provider to be icon provider by setting login_form: ```yaml pam:   login_form: false ``` Though, it the wont pass  app_id and app_secret as first parameters... Why do those providers require them to be arguments instead of bein"""
"very helpful, thanks! (especially for creature and gameobjects)"""
"Since it's going to be a hack anyway, can the Tempfile compatibility issue be solved like below?     def is_a?(klass)        Tempfile.ancestors.include?(klass) || super     end"""
"All of it. :-) Use two spaces, not four."""
"md5 mismatch   Binarycrayon:Library yudi$ brew install pyqt Also installing dependencies: sip ==> Downloading http://www.riverbankcomputing.co.uk/hg/sip/archive/4.12.2.tar.gz File already downloaded and cached to /Users/yudi/Library/Caches/Homebrew E"""
"Ok, as always i wrong read post, anyway not care about any crashes without useful info..."""
"Tbh no idea, i don't have sniffs for these, just based myself off the sql comment blocks :P  [EDIT] Sems a bit more logical though."""
"XemsDoom: +1  This commit is terrible mistake, and I'm not going to continue in this %#@& !"""
"The emphasis is on IF. :) Anybody can simply add that spell to eg. SAI. The crash log should come from some kind of fun server.  Anyway, feel free to drop out that (maybe hacky?) check, but in that case you should drop out the check in Spell::EffectActi"""
"Since nobody seem to care that we break working addons from Dharma.....  Then i think we atleast should make getcwd() fail the same fashion i made chdir() fail. That way we don't get weird subtile errors from addons using getcwd(). Imho they have no busi"""
"~90 LOC just to avoid a goto?  I think that speaks for itself :)"""
"Good call, it wasn't encoded before, looking into why..."""
"Hurray!"""
"But lying on the class makes it useless."""
"Of course it did... I assumed everything was in proper order. Thanks for the heads up"""
"Well, in that world `Number.isNaN(obj)` will work correctly too :)"""
"oh.. and I pulled the plug on dwarfforges... http://dev.bukkit.org/server-mods/dwarfforge/"""
"the else if must be in its own line, add a linebreak after the }"""
"move this in SD2/EAI - hack much more, then current commit :) all simple object interaction like this must be realized by spelleffects or procs - see sniffs.  PS curious why you started to worry about the quest, which in the current core is still impossib"""
"Can we add a default return? throwing an exception sounds a bit too much for view related code"""
"MAke it so that the dispatcher to be used is specified through the constructor, in that way it'll be easier to configure it later."""
"Apparently missing the `s` interpolator."""
"DoCast(colossus, SPELL_MERGE, true); is too hack :P  but that problem is in CheckCast for SPELL_EFFECT_CHARGE (i think is bad pos z)"""
"I agree, I will change it."""
"@ry you should read nodejs-dev from time to time. :) I cherry-picked it from badger/c-ares@bffd67f."""
"the naming of these 2 services is weird. The first one uses the extractors but the second one is not an extractor. It uses loaders"""
"also this issue closes #504 (forgot that sorry!)"""
"Sorry, guys. Yes individually tests pass. I am working on a fix which includes moving this form constructor to beforeFind, to ensure correct ordering. Sometimes tests fail because the find results are incorrectly ordered. I have setup postgres now, so i ca"""
"Maybe this was a bad idea"""
"Then we're on the same page :-)  Alright, could you do me the favor of squashing the commits into 1 so it's easy to cherry-pick it into 2.0.4?  Great work!  √"""
"Creature* c > Creature* creature  Heh. D="""
"Yay nice fix."""
"Why is the Sanitize class deprecated? Is there anything else that replaces it?"""
"Actually it *should* still work on pre-3.0 I just have to test it. There is no checks done on what flags you are setting so since the values are `static final int`s they'll be compiled in the code. They'll be ignore as flags on those platforms but you shou"""
"I whisper this one: looks like a huge step to ahbot implementation. Yay ;-)"""
"depends lines go after md5, before patches"""
"Hi,  I not find tests, but they start issue https://github.com/FriendsOfSymfony/FOSFacebookBundle/issues/79 but fix not work :)  2012/3/22 Drak < reply@reply.github.com >  > > @@ -79,11 +110,35 @@ public function regenerate($destroy = false) > >       */ """
"win32: 'sqrt' : ambiguous call to overloaded function while trying to match the argument list '(unsigned int)'"""
"I think it should be either `has been removed` or `was removed`. Also, wdyt about adding the commit examples to the changelog?"""
"I know Twig supports ``none`` as an alias for ``null`` because of its Python inspiration, but does it make sense to keep it in this ExpressionEngine ?"""
"ok, no idea how to format code blocks in github comments :-/"""
"A empty query(using where) should just ignore the condition returning to normal query, intead of a exception or return 0 elements. Do not you think?  Post.where({}) is somethink like SELECT * FROM posts (WHERE NO CONDITIONS).  If a have a dynamic hash """
"Well, I guess I'll be taking my leave like @V10lator.  I need NBT and it's clear after more arguments that it will not be supported, probably along with an AI API and packet API.  Edit: @mbax That was the reason why I was planning to include independent """
"This is inconsistent with the other UNIX implementations. The caller is responsible for not passing in NULL."""
"@dmethvin: that sounds fine to me, but in my opinion in that case the name `isEmptyObject` is not adequate (that is why #7413 exists). Perhaps we can use `isObjectEmpty` instead, and deprecating `isEmptyObject`?"""
"I don't like that syntax at all. It's not type safe, and therefore doesn't prevent silly misstakes or help in refactoring      """"foo"""" must be('defined)     cluster.convergence must be('definod) """
"because hack shit methods must die."""
"Thanks to have already fixed this bug."""
"if (events.ExecuteEvent() == EVENT_DEATH_YELL) """
"Adding the parameter in the middle of the signature is a BC break"""
"typo here"""
"Unstable for Public servers? They are not supported by Trinity xD"""
"Yeah, just add additional commits."""
"this is wrong. you should not talk about the service id but ask a csrf provider. the bridge is usable outside the full stack framework"""
"You are right."""
"@benschwarz as far as I lives in Russia, I have to tend to develop interfaces for at least two languages - Russian and English. I18n is best for it ;)"""
"Component(1) was totally working before =( could you bring it back?"""
"Relevant: http://twitter.com/ajpiano/statuses/148842137751003136"""
"@markstory Why do you trim the dot from the domain? It can cause security issue. The `cakephp.org` cookie domain is accessible only by the `http://cakephp.org`, but the `.cakephp.org` cookie can be accessed by `http://cakephp.org` and `http://bakery.cakep"""
"great:D """
"I request a revert of this commit :("""
"Because that keyword only has use for constructors accepting 1 argument without a default value."""
"Needs to be documented that it's blocking"""
"im just sayin .  why you taking time to respond lol . i have no idea how this works , im just here for cookies"""
"This is a BC break as the signature is not the same anymore. You should add another method that enables to switch on/off the HTML extension."""
"And it's an import so I can't make it dynamic... Wow.  New plan. Import org.* and import net.*  Horrible idea but fixes the issues... Now to just fix the bytecode issues..."""
"The session_id() call is probably a better solution than setting $_COOKIE directly. I'll change that."""
"Yeah, sorry about the username non-sense. I registered as """"rwldrn"""" a long time ago. I also have """"rwaldron"""" but that doesn't have any of my project stuff :\"""
"do you mean 1f96ccdfe24ffedd9eb303f73f5985a3a127e203 by cyberium? saw this while trying to backport schmooz' patch and cyberiums is missing on mangos one :)"""
"Yes, they are still necessary to support HTTP pipelining (server side) and connection pool (`Agent`, client side). `_buffer()` keeps data until a socket is assigned. When a socket is assigned, `_flush()` writes the buffered data to the socket.   The follow"""
"I can see a decent reason to use the `DjangoJSONEncoder` as the default as it gives us `Decimal` and `datetime` support for free. These are very common use cases for django sites (especially datetime!)."""
"Ah, true. I forgot about that. Browsers... :("""
"This method is synchronized since javax.security.auth.login.Configuration:getConfiguration() depends on system property """"java.security.auth.login.config"""". We need to make sure that the login configuration object is based on the given storm configuration, """
"the default name should be changed to something more intuitive"""
"Ah... No - the pathInfo contains the stuff between the servlet-mapping and the query-string. From the spec:  \[pathInfo\]: a String, decoded by the web container, specifying extra path information that comes after the servlet path but before the query stri"""
"Flame Leviathan is bad example, because it always follows one particular target at time, regardless of threat levels."""
"Welcome back @homakov. FWIW you got your message across. Not sure you could have done it any other way. Take this as a lesson all you devs, take all your bug reports seriously."""
"Could we rename this to `newBoss()` like other factory methods in our public API?"""
"This is most likely a bad default"""
"The rule you guys used in the past with the ie.less was the following:  a img { border: 0; }"""
"I'd probably prefer a simple if:      val interval = routeeProvider.settings.retryLookupInterval     if (interval.isFinite) // Retries enabled        retryTask = Some(context.system.scheduler.schedule(interval, interval, self, RetryTick)(context.dispa"""
"scratch that... it's under 70 seconds now.  (hyperthreading is weird.)"""
"Thanks @trondn for the fix."""
"yay, doxygen!! \o/ thanks Gotisch :)  @vlad: we know, that _YOU_ have the whole mangos code in your head (you can probably even run it in your head without need of any compiler...), but almost all other persons need some kind of documentation... ;) """
"base64.b64encode would be even more efficient, I was simply reporting the bug"""
"@jnicklas, as I wrote my message I wondered how many people from the EU would make comment.  What I really wonder is why the authentication library should handle the i18n at all. Surely this could be handled by rails.   In my opinion the generated views"""
"No sorry, insider42 based. dont this problem exist in clean core?, maybe Pala and DK spells related  Best regards"""
"eclipse formatting due to the `;` break it to two lines, or enclose in `{}`"""
"yep, it's work, but i need to add user and password for proxy connection=( Can u added this feature. May be --proxy=login:pass@server:port"""
"yeah okay my bad... disregard last 24hrs of comments"""
"How would you disable CSRF on a form then ?  There are probably better solutions than what I have proposed, you should submit a PR when you have a working one."""
"Aww man, it's a cruel world, eh?"""
"this is wrong IMO as you are loosing part of the data in case of a composite key without saying anything to the user"""
":confused: """
"sorry -- this is my first formula and that was an extra line of copy/paste -- Deleted"""
"I see. In fact, I worried this change. (because break FOSFacebookBundle and ...)"""
"Can we revert it now on master? =)"""
"`$this->kernel->getStartTime()` cannot return null. So you can safe a condition check. Btw unrelated, kernel->getStartTime() returns -INF, which is probably also problematic as INF not always supported and it violates the phpdoc which says, an integer is"""
"off in development mode"""" -1  When I am going through the hell of fixing IE rendering bugs I am working in development mode. I want pages to look exactly the same when in production. I don't want to add a IE hack to find out it creates another bug in pr"""
"sigh...it is not a question of syntax preference. It's an abstraction that adds no value to the underlying scripting language. It's not worth having the argument of whether it's nice or not (which is entirely subjective)...it obscures without adding value """
":) cool, thanks."""
"@vlastv Please read about the form-urlencoded serialization of form data for checkboxes: - when they are checked, they appear in the submitted data, and the value is the one set in the input (which may even be an empty string IIRC) - when they are not ch"""
" Uhm oh yeah. Hmm I would just go with Name (Handle)"""
"nice catch :)"""
"Hmmm, I just downloaded it on my desktop and sha1sum says your checksum.  But https://github.com/rapid7/metasploit-framework/wiki/Downloads-by-Version shows the b9d6dbe54b23448da9a939cf3e1f95cb7dc0e363.  Though the download link for *that* file is http://"""
"`if event.target.nodeName isnt 'A'`"""
"Not sure if I like this idea. I get how having a variable for this adds some flexibility, on the other hand it isn't likely that you will have one font color throughout the whole site anyways so I'm not sure how useful it is.  Using the color set in a pare"""
"I also find setElement a little confusing, perhaps it reveals the inner workings more than necessary but not sure what would be a good name, pointTo?.  Also i liked the old syntax you had where you got objects for each element you wanted to modify but p"""
"Can any tell how much is CPU load in % on clean mangos, with ~100 players?"""
"cofirm @digz6666 double loot... """
"This is just amazing. Thanks for adding this code to use in bukkit."""
"I think BytesRef.toString is too verbose since it uses 4 chars per byte ('0xc3' for example). Maybe we could directly call `currentPayloads[j] = Base64.encode(curPaypoad.bytes, curPaypoad.offset, curPaypoad.length)`?"""
"very thx ^^"""
"you mean the problem lies in their implementation using WeakReferences and their associated extra GC cost, right?"""
"I'm afraid it also contains a fair bit of Engrish, both in spelling ('avaible', 'build-in', etc.) and grammar."""
"Formula updated to address this (would be nice if brew audit warned about this...)"""
"Since all of the cases where `format_object_header()` could have returned an error code are now handling by asserts, it should never fail (except by asserting), so I don't both to check the return value for < 0 anymore. Does that seem okay? """
"wrong realization AKA hack. see spells 48586-48589. """
"Thank you so much!"""
"This comment is pretty useless"""
"I wasn't aware that expressing an opinion on committed code based on information from the PHP manual in an open source project was """"spamming"""" everyone. I also wasn't aware that it notified every single collaborator, so cram your elitist attitude and have s"""
"It probably won't make any noticeable difference but a continue would be in place, indeed. I actually thought of it but then forget about it again, so nice catch. :)"""
"There is a problem when running this new FaceMaterial example with WebGLRenderer.   I'm already too tired to sort it out now, but the problem is apparently coming from faces which do not get randomly assigned material, which means their material is undef"""
"Sorry, I didn't make it clear that such a test should be performed only for Opera. Other browsers already are burdened in different ways for their own errors. But I get what you're saying... too many issues with too low a success rate to be worth it.  I'"""
"Actually, here is [one example](http://travis-ci.org/#!/thoughtbot/paperclip/builds/67058)."""
"Hehe, hadn't quite grokked that, thanks :-)"""
"True! Is not `unescaped`, is `unreplaced` or `escaped`. Then, should I change it?"""
"We may want to use `_isfile`, just for extra sanity?"""
"Hate to add noise, but I would really prefer to have links to the issues as well. """
"Blank lines before and after `depends_on ...`"""
"you could also write ... if (roll < 73) ... to prevent calculating the random number in this line - one might be enough :)"""
"Bad Scott! Bad!"""
"Thanks for your comments and tests!  (Aptana is driving me nuts, I'm currently searching for another IDE)"""
"make that N330 pretend it's a single core with no HT and you should see any issues :)"""
"this param don't exists in the method signature"""
"@trustin ups thanks... stupid osx :/"""
"note: this is a hack, if you have a better way then please tell us"""
"Please, use 1.9 hash syntax, thanks :)"""
"but but but  ferals are fluffy, not overpowered :'("""
"can you tell me what issue(s) this was causing (like which api you were using)?"""
"Standard way of determinating euclidean distance. If you used it once you will recognize it :) http://en.wikipedia.org/wiki/Euclidean_distance"""
"Glad you fixed it"""
"I guess my comment was vague, I was sleepy. There seemed to be some compile, errors. I'm at work now I'll post them when I get back home."""
"As i know warlock's pets is permanent pets and should follow to owner if owner is mounted (this how it works on blizz servers), so with this                  if (pet->IsPermanentPetFor((Player*)this))                     ((Player*)this)->UnsummonPetTempo"""
"Why did you chose to seed with 0 instead of a counter?"""
"yes it'd be more complicated I think: I really don't know the details so don't trust me but as I understand it if you want to sync audio on video you'll have to feed it with timestamps and set some compensation. The compensation / async is set at swr_init """
"Actually, maybe not. I see that encode actually encodes all the attributes if body is blank. What if I want an empty post?"""
"Sure, though I'm sure we'll only ever have one hehe."""
"pet::Update call Creature::Update and in creature update before chnages exist explcit assign different value to regentTimer In some cases Pet regen timer updates conflict with unexpected Creature updates and timer never set less that normal diff values. """
"Long live the thread-safe code! :)"""
"great - okay, makes sense.  thanks for going through these all - yay, less yellow triangles! ;)    feel free to merge!"""
"this commit is going to make writing tests so much more difficult :("""
"thank you for improving movement system. now, i can see the face of boss when combat : ) """
"Then how do we get the current planned drops? I need it for my graveyards plugin  edit: Nevermind, i'm a blind idiot"""
"Shouldn't this and the following methods be called `openSession`, `closeSession` etc. to be consistent with the naming in the rest of the framework?"""
"Oh, I've misread commit title, my bad :dancers: """
"ups.. missed it. Thanks :)"""
"Nice name, buddy."""
"yea forgot the space after if mate :]"""
"That password isn't actually the password, it's just a random string, so I don't think I'll filter-branch it out.  Thanks for your vigilance, though."""
"It's always sad to see a reference like that go, but it was probably a good move."""
"Actually, with the removal of case insensitive comparisons, this is obsolete!"""
"LOGIN_DEL_EXPIRED_IP_BANS double?"""
"This appears to be only used in log output. If so, we should avoid calculating it in normal operation."""
"Why this change? :smile:  """
"Instead of sniffing for IE, you can use the IE-specific [`document.documentMode`](http://msdn.microsoft.com/en-us/library/ie/cc196988\(v=vs.85\).aspx) property."""
"I think this was the only comment you skipped =)"""
"Thanks, this issue has caused us serious confusion on numerous occasions. """
"Hack:p"""
"+1 This looks super useful!"""
"ah, sorry"""
"`const` also helps the compiler generate code that makes more efficient use of registers tiny function though, probably doesn't make a difference in this case"""
"That's great! Love to hear that ;)"""
"`brew audit frei0r`, this version tag might not be needed."""
"I think its quite clear when the person has a look at the interface whats what :) Another alternative would be to rename the DecoderInterface into EncoderDecoderInterface to make it even clearer."""
"I hope it is not patronising if I offer this : http://gist.github.com/329172  Thanks : DBJ"""
"weird tabbing and bracketing here"""
"Just use issue tracker."""
"Well, we had problems with rawurlencode, because it encodes '/' and other that cause issues with Apache just returning a 404 or 500 (not sure anymore) without even hitting PHP. There's a ton of different ways to plug PHP in a webserver, and every webserver"""
"For users, it probably don't have to be mentioned (it's a documentation). For developers there are unit tests (+docs) :)"""
"It's more disappointing how long it takes @Subv to add a simple Typeid check :)"""
"@tadd agreed - I wasn't aware of that syntax. I've added your suggestions and pushed. thanks!"""
"You mean semicolons? :P"""
"I like this name change!"""
"Please add parentheses around the `and` block, precedence isn't obvious."""
"Sorry for the bad commit message."""
"Any chance we can get definition duped on the way in? These deletes can cause some bizarre issues at times. See http://github.com/ernie/paperclip/commit/ef2e871ac1015a67e27b7a8a8c0da35842ed97bb for one fix."""
"sorry, totally tried to be @MrZyx, because he said postgres i think... sorry all."""
"Well, I think we should clean the CS. The CssSelector component was originally written with underscores when porting it from Python but the CS have been fixed latter."""
"Yes, much more descriptive.  Done!"""
"Does this regular expression work if I have ? within quotes? I also don't know why we are doing the replacement rather than leaving it to the server? This seems pretty dangerous."""
"@Dinnerbone you should rollback this change apparently it crashes clients. I tried it now and it's true... sorry :("""
"You right i not remember about UNIT_FIELD_CRITTER (in fact first time read about), BUT I not """"put hack"""". I update _existed_ field to ObjectGuid. Start use UNIT_FIELD_CRITTER field if it related to minipet can be good step but independent from this or late"""
"weird, when I tried last time, it feared all npcs around me..."""
"That sounds like a record!"""
"They are all just lines.  Thus I would make some utility functions that created sets of lines that could be rendered using the standard Line.js class.  But I am unsure if you are looking for additional functionality on top of just generation."""
"man, am sorry but who you are to ask developers do anything?"""
"I dunno where can be problem :( I have in fog just cave part, but no the """"souls effect"""" around. I also think Spirits in Frozenmourne are bugged. In 1st Frostmourne had no spawns, in some of attempts attack instantly after teleport and sometime work perfect"""
"Ideally we want all style in the stylesheets"""
"From what I understood, if `tryTupleApply` returns true, then the subsequent logic in `tryNamesDefaults` will choose tuple application and won't go for named or default argument applications. Right?"""
"No need for the extra line."""
"Awesome work everyone, it's beautiful!"""
"Also `man1.install 'gpg-key2ps/gpg-key2ps.1'` IIRC."""
"Yeah that was the question of tge TODO... Do we want this?  Sent from my iPhone. Excuse any typos....  Am 03.01.2013 um 16:27 schrieb Trustin Lee <notifications@github.com>:  > In transport/src/main/java/io/netty/channel/ChannelHandlerUtil.java: > """
"That's not 'broken', that's ack 2 working as expect. Report it upstream."""
"// somedev1 -  6/7/02 Adding temporary tracking of Login screen // somedev2 -  5/22/07 Temporary my ass  jk ^_^ couldn't resist"""
"If there are no references to it, it means that it's been terminated, if it has been terminated then DeathWAtch has been triggered. No need for WeakHashMap"""
"I don’t think you want the `!dragging` here—this behavior should only be controlled by the clamp setting."""
"Crap..."""
"Ouch, that is not a very happy thing for me. I use tons of datasources that do not extend the base class :("""
"trail and error has the downside of making backports for the subprojects real hard. For example its almost impossible to backport lots of stuff from TC because you never know if the backport is wrong or the commit simply was experimental stuff. Trust me, I"""
"@spastorino, please don't forget to push this to exceptron as well!"""
"Watch out, you'll be crossing paths with [MCFTIDE](https://github.com/scala/scala/commit/a205b6b06#L40R128)"""
"Brilliant, I didn't realise that homebrew would generate the checksums automatically. I've amended the file again to include this. Do I need to create a new pull request or should my changes be fine now?"""
"Unfortunately that would still be too much: as symbols mostly come from different SourceFiles grouping is not benefitial."""
"well since it's an undocumented feature, I'd say break it. To do it like Django does ;-)"""
"GOs with gossip without hacks? epic!"""
"`brew audit gcutil` and remove this version tag if it isn't needed."""
"Sniff it! :)"""
"what a shame.... :("""
"Good call. Simplified."""
"WTF ?????"""
"And WOW, how the crap did _that_ happen? Apologies for the double-post. I blame sunspots."""
"@josevalim are you sure that we need this `indifferent_hash.rb` file? :)"""
"Spawned GO still in mangos not have real size... and ofc, geometry also... unknown way calculate it :( """
"Multibyte::Chars is designed to work with UTF-8.  Some references: https://github.com/rails/rails/blob/master/activesupport/lib/active_support/multibyte/chars.rb#L7 https://github.com/rails/rails/blob/master/activesupport/lib/active_support/multibyte/char"""
"total 392   8 -rw-r--r--   1 andriytyurnikov  wheel    1734 Oct 11  2011 ACKNOWLEDGEMENTS   8 -rw-r--r--   1 andriytyurnikov  wheel     983 Oct  4 01:29 AGREEMENT 280 -rw-r--r--   1 andriytyurnikov  wheel  140111 Oct  9 01:54 CHANGES   8 -rw-r--r--   1"""
"I've wanted a method like this for so long!  I'm always writing little bits of code to do this. Thanks!"""
"We've never allowed these, sorry."""
"Is there an elegant solution for this problem or we should drop it on the floor like we do now? Poor java guys..."""
"@normanmaurer I think you or trustin may write an post on you website about the release() and retain(); this may be important things to understand in netty4,and you may need to make it clear to the readers. Thanks."""
"IE9x64 on my Windows 7 VM does ~6200 reps/s of `jQuery('<div id=""""foo""""><span>bar</span></div>')` without this change and ~5800 reps/s with ([source](https://gist.github.com/990134)). You'll probably be able to run more accurate performance tests than I hav"""
"he collected data on exceptions"""
"No... you haven't got me correctly. Method is gonna always exist on `Request` object. The problem is that if the argument exists, it returns it. Otherwise, it throws an `InvalidArgumentException`. Look at the implementation of method `Request::getTruste"""
"I'm worried that this change will break existing sites with no way to recover the old behavior if they are relying on it. Thoughts?"""
"Perhaps I'm being pedantic, but I'd prefer the new validation to be added in a separate commit from the reshuffling. Ditto for other new code this commit that I might not have seen."""
"evil fix!"""
"Love what you did there."""
"I think it stands for unsigned long long."""
"pull a _why and make everything as cryptic as possible, that's what good programmers do. job security ftw"""
"Actually, looking more closely, this CSS isn't specific to login/register? or is it? Right now it's everywhere Bootstrap is enabled, but it looks pretty login-specific. What do you think?"""
"Thank you. <3"""" - Alien Blue users  This issue caused a massive uproar on /r/alienblue."""
"Ah. Cool deal. Thanks, John!"""
"Nice spotting, sir. Thank you!"""
"@rehv https://github.com/Bukkit/CraftBukkit/commit/8f12382e8efc8c39a919af9180dd884caf3720ff#commitcomment-2274858 >I'm fine with whatever solution that doesn't force devs to recompile every CB version or so  Good news, this won't make any dev recompile """
"One assumes you're aware of the following:  Set<T> weakSet = Collections.newSetFromMap(new WeakHashMap<T,Boolean>));"""
"That's why this is called a learning project. We are sorry that your servers don't work as immaculate as they should. Things are done with trial and error, and believe it or not this is still volunteer work, so no one should really answer to every cry for """
"Done, tests pass without the whitelist now"""
"coding style: brace on next line"""
"val NATFirewallAddresses = getStringList(""""akka.remote.nat-firewall-addresses"""").asScala.toSet"""
"Oh my... I remember I stopped and looked at the word, it looked weird but kept going... =_="""
"I will.  we have support for List<T> and Dictionary<T>   coreclr safe deserialization now.  Unfortunattly we did not find a way to support HashTable deserialization without making its protected constructor public, which I expect is enough to not want that """
"grats - first stable thing in eden i guess :D *running*"""
"Oh, sorry didn't see that :/"""
"shouldn't this be an Error not a warning?"""
"Comment: Short circuiting is available if:  - no full comparison requested, only validation for a certain ordering (ordering ne FullOrder)  - state Same has already violated (see states and allowed transitions in my comment below) and the requested orde"""
"This line saved my ass. Thanks!  I did a fresh install and after that, the SSH Keys added by the users didn't appear in git's authorized_keys file for some reason. (No errors where shown)  With this line it works like a charm :+1: """
"I can't find SAI for www.wowhead.com/?npc=23859... didn't you forget to convert his script?"""
"I'm getting this on linux: ../../../src/game/Object.h:521:7: warning: no newline at end of file"""
"Unfortunately, I found someone using this [in the wild](https://github.com/nbudin/devise_cas_authenticatable/blob/master/lib/devise_cas_authenticatable/routes.rb#L16).  I'll file a ticket up stream, but should we make this a warning?"""
"A single-use non implicit Ordering is an unlikely sweet spot. Skip the Ordering and write ``` sortBy (_.name.decoded) ``` instead of sorted treeOrd. """
"Can this use the preprocessing util?"""
"> Weird, this is what I get:  > http://twitpic.com/3agtdb/full  My bad, I reverted my changes, rebuilt and now I see it (Lucy has strongest effect). Must have been some caching issue.  > By the way, I'm a bit confused with the ratio the refraction us"""
"Thanks Mike.  You said it better."""
"Did you intend to only whitelist the AbstractDownloadStrategy class? Pondering replacing with  ```ruby when Class then @strategy ```"""
"die-hard designers would probably rage over SASS/LESS/Stylus as well, however css does have the fundamental issue of not being dynamic, so I think that is a different argument"""
"Oh, I didn't realize! Should have checked the audit. I've removed the version string."""
"It's ALSO for display. Although I doubt the practical use of your case, especially considering what this fixes (copying from URL encoded VFS's), reverting maybe a little drastic. I think a far better fix is simply adding a check performing an IsInternetStr"""
"hmm  ``` for (uint32 resistance = 1; resistance < 10, r>=probabilitySum; ++resistance)     probabilitySum += discreteResistProbrability[resistance]; ```"""
"Ofc, but my secretary (monkey) in random typing by keyboard not always type proper texts to commit notes. ^^"""
"looks like this didn't work :("""
"Congrats with your first commit and developer status at mangos! Do we know you from forum? Good Luck and thanks for joining!"""
"It wouldn't be in net.minecraft.server or obfuscated if it were new :) The changed lines are always marked with //craftbukkit."""
"code is better than documentation :-)"""
"even better would be to add a comma at the end."""
"`positionPercentHack(elem, name, value)` => `positionPercentHack( elem, name, value )`"""
"@jeresig - Right I get that *(and auto converting rgb would be fine)*. I just don't dig swallowing all potential errors :/"""
"MD5 issues here too :("""
"I don't think this is correct. @arrbee?"""
"Oh, sorry guys. Got distracted by other things. I'll split my submodule branch and push up some options today."""
"So move the yelling to a central place? Great idea."""
"I think that translator is missing one fallback search level. In my opinion there are 3 levels of fallback  I've set in config.yml the default translator fallback to en I've done some translations in English (en) and Italian (it). mybundle.it.yml mybundl"""
"That's a good thought. Hm."""
"Looks great! love `OF_TTF_MONO`, etc"""
"So how will the writeHead merge handle multiple headers?  If it's an array then append, otherwise overwrite?  There's some nuance here maybe.      res.setHeader('set-cookie', 'single-value');     res.writeHead(200, { 'set-cookie': ['more', 'cookies']});  I"""
"Heya u sayed Fix multiple permissions and remove multiple config option… but where is the config option for RBAC ?"""
"i know malsup's Taconite plugin replaces jQuery.httpData. maybe that's why it stopped working when i tested 1.4.3pre a few months ago :\"""
"http://2.bp.blogspot.com/-Boq9ZwPglK0/TlQUl5VZ7tI/AAAAAAAABqw/AaicADRN2hA/s1600/404_Not_Found_nginx.jpg i want speaking about that :D  pony are everywhere"""
" i don't get it, why do you override DateTime#to_time?       DateTime.now => #<DateTime: 2012-01-25T11:33:21+01:00 ((2455952j,38001s,996469000n),+3600s,2299161j)>        DateTime.now.to_time => 2012-01-25 11:33:25 +0100   looks like offsets are allrig"""
"finally :)"""
"i'm realy confused with github... i can't edit my edited file"""
"Yes I was reusing Rack initializer but that way I'm passing NullSessionHash creators the responsibility to define a null store which should be in NullSessionHash. So agree.  Fixed https://github.com/rails/rails/commit/e3b9b25aa3cbe6344425a346e6e73073ca6"""
"re: """"Use Props.empty instead guys."""" Genious. And obviously it's in the docs... I just read it... :)"""
"In RFC 2109 an explicitly set domain must always start with a `.` (section 4.2.2). Only if the domain is omitted will the  leading `.` be missing (section 4.3.1). Based on the domain matching examples, the rules seem like a cookie's inclusion in a request"""
"This looks intresting, now.. hmm, to get up with some nice scripting idea using this :D"""
"Thanks for fixing it, you saved my OCD :P"""
"My bad example is at fault for that one, sorry @antonio!"""
"The issue is that a lot of the people starting these debates are too strongly opinionated when they really don't have a say. """
"unsigned int"""" in order to get rid of the sign-compare warnings in line 240 and 242."""
"Gratz Zergtmn! Very good job!"""
"may offer this: https://github.com/rsa/mangos/commit/fdf656d23a114f5dd3fd17ec77b7ae766bade4a2 not need with current mangos code, but required for thread-safe. tested, worked fine."""
"`decodePdu` should be lazy wrt. decoding the actual payload, since we might throw it away in untrustedMode (it might be a mem bomb …)"""
"*sigh*.  I'll fix. :'("""
"What about local XSS attacks?  With unescaped output, something akin to the MySpace worm is enabled by this change.  Granted, Rails coders *always* remember to escape output!  :)  But was the intent to remove a layer of defensive programming?"""
"Epic fail... let's build enterprise so?"""
"Thanks Ben!"""
"Yeah, what the crap is going on with that last line!  Let me fix that right now."""
"`Oldie` insures no past or future browsers that have broken parsers could ever possible apply the IE hacks, but I like `lte-ie8` better.   I would advise just using php(that's my preference) to convert between the classes and hacks on the fly.  Just right """
"oh, I missed the comment bits, my bad."""
"\ No newline at end of file"""""""
"This breaks gossip options :("""
"Hey! This problem prevents all European folks from usage of paperclip... I am sad about it :-("""
"Nice catch. Leftover from the evolved development :)"""
"I think the reason why we don't have ::Parse() URLDecode is that it could cause problems when manipulating the CURL filename member and then re-using the CURL object. It would be better then to put this in URIUtils::GetFileName(), I guess.  Since I didn'"""
"    if (Initialized() && !Finalized())         full spline is sent"""
"session.reset() is invalid in this scope. Lets remove it and then work on a different approach.  In my ExampleServlet I had to use `this.request.getSession(false).invalidate()` to get it to reset the JSESSIONID. Simply using `session.invalidate()` was no"""
"Ok thanks very much for your explanations. And thanks LordJZ too :)"""
"You have a point there, it would be good to get rid of this.  However, the opposing constraint is that when there is no custom Hashing object, some objects produce poor hashcodes for typical object ranges - for example (n: Int).hashCode == n. The problem """
"Hope you didn't hate me too much for not having commented there in the first place :/"""
"My bad. Suggestion was only pseudo code. It doesn't handle 'folder creation failed' and returning: true or false.  I meant something like: https://github.com/itofzo/Sick-Beard/commit/247339e45aef0a915b0208c27de8c8621bdc57e0 """
"Hi Gina,  `oauth_access_token-client_id=12345-client_secret=dummy_secret-redirect_uri=http%3A%2F%2Fmytestserver%2Fmythinkup%2Faccount%2F%3Fp%3Dfacebook-state=testiferricous-csrf-token-code=thisisatestfauxcode`  The length of this file name is **testife"""
"I also tried it, and hardly less than you:) so far obtained. though of course very easy to write ..."""
"I'm updating to Rails 2.2 and I got a lot of deprecation messages like:  DEPRECATION WARNING: @change will no longer be implicitly assigned to change.  This deprecation check seems to be a little overzealous.  If I set an ivar @change in the action and"""
"Thank you, guy-who-probably-lives-in-the-Balkan."""
">I'm pointing out the fact that  Gee, you make it so tempting to feed. Guess I'm going to have to resist though :("""
"One more point for you on the unlimited """"why we love you""""-scale. Have a nice day."""
"I'm curious about the performance implications of adding another call to the stack. """
"Good idea ;)"""
"@Machiavelli - I WILL NOT SIR! Are you trying to get my hands burned even more!! You crazy man you!"""
"You don't really needed to fix these problems because jzlib was on the exclusion list but on Windows it didn't work because of different file separators.  It's good to clean things anyway. :-)"""
"i hope the next change will be fixing a bug that aura 269 affects resilience =)"""
"I think that it may be better to handle the """"Trick"""" (roll_chance_i(50) returns false) through spell 24714 which requires script effect. Your """"Trick"""" code looks like it would work perfectly there though :)"""
"I don't understand the thinking your response. I helped you avoid a bug. How does that make me """"upset""""?"""
"At least, Visual Studio compiler remove any ASSERT in release mode, so the function is never called. I Had a hard time with that :("""
"Progress progress!"""
"Yes, this is best:  ```javascript exports.PORT = +process.env.NODE_COMMON_PORT || 12346; ```  Hooray!  A bikeshed!  Let's ALL paint it!"""
"LNK2019: unresolved external symbol """"protected: virtual void __thiscall ByteBufferException::PrintError(void)const """" (?PrintError@ByteBufferException@@MBEXXZ) referenced in function """"public: __thiscall ByteBufferException::ByteBufferException(unsigned int"
"Why should we then even trigger the SEND_HEADERS event? We can also register a listener on MVCs EVENT_FINISH and forget about SEND_HEADERS event. You can also remove there unneeded/ unwished headers."""
"holy damn >_>!"""
"This seems to have awaken Issue #1789 again, why do you changed scroll to offset ?"""
"    s""""cpuCombined must be between [0.0 - 1.0], was [$x]"""""""
"I think there's a mistake in this method, `columns_hash` is not correctly being used, as @tenderlove has fixed in d29235d2a4df94b6db7d3e45ab15a6799363384e for branch 3-2-stable. I'm going to forwardport his commit to master, @yahonda can you check it agai"""
"Unfortunately, the contents of the checkout lines in the reflog are not guaranteed to be a branch name. In my reflog for libgit2, I found:  `<sha>^0`, `<sha>`, `<sha prefix>`, `FETCH_HEAD`, and `<branch>` """
"/* just ignore this shit. TODO write more code */"""
"Cannon spells bugged..."""
"kandera, congratulate to join team!!!!"""
"Thanks"""
"MaNGOS have nowadays gold era :)"""
"Hooray!! Thanks everyone. :)"""
"Yeah I think we should make the interface return `Future[Unit]` :\ I'll file a ticket"""
"Okay. A comment would be nice, otherwise LGTM."""
"Recompiled version works fine, sorry."""
"Hey what happened to the EncryptedCookieStore?"""
"closes #10517"""
"This line is so confusing. :cry:"""
"good catch, thanks!"""
"Remove these comments"""
"This change breaks routing that worked with subdomains in the past (2.0.4) as it redirects a failed login request to /users/sign_in as opposed to /[subdomain]/users/sign_in.  Can this change be removed for the 2.1.0 release?  It's the only breaking chang"""
"`/** */` to make a documentation comment."""
"the test or in the implementation(ActiveModel::Serialization#serializable_hash)? If we do it in test, we are not actually checking any order. this test will always be passing because of `sort` call.  """
"lol, nice commit message dave"""
"Well, we didn't have a migration in a while but it is in the usual update workflow. Rails will also print a pretty clear error message in case you miss it. So I don't think that's necessary :)"""
"That's not right, `X509_STORE_add_cert()` takes ownership of the certificate."""
"Give one of them a different name.  (What that name should be, I don't know.)  Also, the first new gunzip method should probably also be gunzipStream for consistency (and with the same signature collision problem)."""
"Fixed the syntax for checking the presence of email and password with mongoig"""
"There goes my weekend. I beg, BEG, you to revert this."""
"Agreed `DjangoJSONEncoder` would be better default encoder class."""
"line endings: http://help.github.com/line-endings/"""
"if i have my server set to hardcore people will be banned on death ??"""
"BURN HIM!!!"""
"That's right. Forget me, I'm silly when I wake up."""
"yeah, you're right, sorry, didn't noticed that there is only one amp symbol."""
"broke jasmine:ci :("""
"Fun trick: you can use `Object` instead of `function(d) { return d; }`. But the identity function is probably more readable. :)"""
"I'm at Railsberry conference now. If someone can provide a clean fix - it will be great :smile: """
"Should the signature be (d, i)?"""
"It might be best to do a source compatibility hack here instead of copying the constant.  In Analyzer.scala, there is a class Compat.  Maybe add:  ```scala val DummyValue = 0 def hasMacro(s: Symbol): Boolean = {   val MACRO = Flags.MACRO // will be DummyVa"""
"crazy, I remember writing this code awhile back... I put it up on the mailing list but never got around to writing a test for it... """
"You're welcome. I was just about to remove this completely as I found that in fact it was already fixed. :) Cool you chose it instead!"""
"Don't use `Zend\Json\Decoder`. Use `Zend\Json\Json::decode()` instead, as it uses `json_decode()` by default. `Decoder` should only be used in rare occasions, typically when the JSON is known to be in an encoding other than UTF-8, or where you know JS lam"""
"we should remove authorize() and just have login() with an optional set of permissions. in most cases apps will want to tos the user and request permissions in one shot."""
"Shauren did forget something ... *screenshot* :p"""
"Gridmonster fools us all !"""
"haha, no. I don't trust a global search and replace on something like this. Now to do our plain Makefiles that are not autogenerated."""
"Another bug introduced by alias abuse. Tests for this file fail."""
"@coelho Great point :+1: """
"I miss C++ :("""
"Sorry, I thought my comments were sufficient:  ``` If the code includes a valid, prologue position strict mode pragma, execute code by injecting a script tag into the document. ... Do the injection.   Otherwise, avoid the DOM node creation, insert"""
"yes i noticed that after i send this pr -.-"""
"don’t use `Any` in Java-facing signatures, Paul warned that there is pain lurking on that path"""
"Hi, thanks for the fix. but what about the MultiSelect?! it's just working with single select :8ball: """
"good, this one makes simplier doing backup"""
"The methods `get()`, `set()`, `has()`, `all()`, `clear()`, `remove()`, `replace()`, `isEmpty()` and `keys()` [are missing](http://symfony.com/doc/current/contributing/code/conventions.html)."""
"I can start on that right now and make that the next push. I realized that I was trying to do it all at once and I need to break it up. Something that should have taken me a day ended up taking a month :(  This is the order of how I was thinking of doing"""
"Thanks for the link. Will fix all related issues."""
"How do I go about benchmarking the strict build to see what the impact actually is? It feel wrong to me to take decisions without metrics. Never used our benchmarking suite so I'll need some help here."""
"+1 :D (i never used that)"""
"Whoops, amazing that this lasted a year in the repo :(  I fixed things up in [27ab9093b5be7ca16192982d4f6640130d963421]"""
"the problem is: this is an overwrite of a reversion-admin function and i wouldn't like to change the signature of the function"""
"yup  I considered writing my own, but the benefit isn't very obvious :-)"""
"What an entertaining read. Nice!"""
"Happy to see you going by hand, but the Ragel parser had the advantage of the graphical Dot state machine output. Are you maintaing a similar FSM for this version, and if so, would you please copy the .Dot file into the repository? Thanks! """
"i dont get it what is this for ?? sorry for my noobish question"""
"Sorry, maybe this was the formatter that I used too (as the indentation), was not my intention to make this... I'll rollback... Thanks"""
"@jeresig: committed and made pull request (http://github.com/rkatic/jquery/commit/452c292daa8de2d45cf283bd51cc98d90e04327f)."""
"@leprechaun This should be:  ``` BotoConfigLocations.append(os.path.join(os.getcwd(), """".boto"""")) ```  This avoids an invalid path on Windows.   Also spaces after or before parenthesis aren't preferred according to PEP8 which I believe @garnaat is trying to"""
"I donno. I would start by seeing if this is still an issue. It seems odd to check for only the primary window *(as if only the primary window object would bug out, and not say an iframe window object)*. I don't see any other place in the `add` method that """
"FYI - this breaks rspec-rails' monkey patch of AC::TC, so rspec-rails-2.0.0.beta.8 does not support rails from this commit forward. I just addressed this in rspec-rails, and will try to get an rspec-rails beta 9 out soon.  http://github.com/rspec/rspec-rai"""
"we try to remember, but when we push broken code we need to fix it as soon as possible (we don't have a versioning scheme) and in that stressed moment forget about ci.  also, it's difficult to explain ci to open source commiters, and when a commmit needs"""
"Why do you do this either way, ```options['login_form']``` is ```true``` or ```false```? Shouldn't this depend on ```icon```, instead? Something like: ```ruby if icon   Gitlab.config.omniauth.form_providers << provider end ```"""
"I've absolutely not read all of the comments but I just have to add that we should have done this commit WAY earlier.  The whole point of Bukkit is to *NOT* provide you access to the Minecraft internals directly. This changes makes it so that you *HAVE* """
"Really appreciated this!"""
"i only fixed an obviously broken script, not debated its blizzlikeness"""
"I still don't understand what this stuff has to do with WebKit.  Anyway, would be cool to actually find the root cause: the change I made here was just in the way I set the HTML for the frame.  I can only assume that the frame requires a """"physical"""" fil"""
"Yes, they certainly should!"""
"@Ne0nx3r0 https://github.com/Bukkit/CraftBukkit/commit/8f12382e8efc8c39a919af9180dd884caf3720ff#commitcomment-2305605 >@mcfacilitator When we're given permission to discuss this on bukkit.org I'm sure most people would be glad to do so. It would be nice t"""
"Isn't this weird?"""
"Because, a fully functional retail like AH with simulated seller to represent a working economy would make TC even more blizzlike :)"""
"improvements*"""
"yes, sorry the ifdef is incorrect, it's fixed now"""
"Oops, my bad sorry. I was using 2.4.0 rc1 ... this is correct for 2.4.1 (which is now the default installed by pip)."""
"haha, you have made enough good commits already so no sweat ;)"""
"We don't do interim builds, but I'm preparing for the next NuGet release which will also be the starting point for the v3/v4 split. WebHostUrl can be explicitly set to point to a preferred external url, which if it is it will use that instead. """
"In order to get the most out of it the decoder can decode frames in advance (the thread does not block in flipPage). This way the one or other decoding cycle can even last longer than refreshRate without the need of dropping a frame.  vdpau decodes, the"""
"Yes, please."""
"@ADmad Very true! testing with ```array_key_exists()``` now.  @markstory @ravage84 It does, but I targeted master because I considered a bug: . throwing undefined index when a key is not in the ```viewVars``` array . returning a JSON encoded array whe"""
"Bad :|"""
"Yes, as I was `git add`ing it, I thought to myself """"maybe you should rethink this."""" I'll fix it. :smile:"""
"Getting weird errors in WebKit with this error. Will be investigating further tomorrow morning."""
"That's weird. It works for me locally:  ``` mbp:activerecord godfrey [3-2-stable_restore_frozen_state_on_rollback] $ rvm 1.9.3 mbp:activerecord godfrey [3-2-stable_restore_frozen_state_on_rollback] $ ARCONN=postgresql ruby -Itest test/cases/transactions_te"""
"This causes a style guide violation and makes it harder to move towards `onevar`"""
"@riking very nice."""
"nwsw is correct, there is an abstraction layer, and you simply need to use BaseFacebook and provide the three hooks to set/get/clear session data. Using $_SESSION makes it work out of the box, without having to store the access_token in the cookie."""
"I had it implemented that way originally. The `rijndael` function uses substrings so I decided to make this the same. Being more strict is an option though."""
"ok now again :D where can i get new map,vmaps extractor for windows ? compiled ones plz.. dont say me to compile etc.. i have reasons."""
"@Runemoro Clients crashing because of a packet change is the BEST CASE scenario. It could have spaghetified the world. It could have deleted the whole server folder. The trouble with using the obfuscated NMS code is that you can't control what your methods"""
"Hm, where are coderay options used in kramdown? http://kramdown.rubyforge.org/options.html"""
"Players are opening the chest with spell 61437, might be possible to force that to be casted by the player on gameobject use. And tbh, I rather see sniffed positions then """"working"""""""
"You're right, sorry. I should have redirected stderr to avoid the spam.  I was checking in this case because new Macs apparently don't have an unset path - rather, they have `/Developer`, which is invalid but not unset."""
"hooooooray :)"""
"On Thu, 10 Jan 2013 11:03:10 -0800, rsa wrote: > @billy1arm [1] , If you have not realized, this """"developer""""  > completely > (by some last commits) destroyed the possibility of further work on > the code - that you yourself have yet to learn. As for workin"""
"Why is the `-d dist` needed? I thought dist is the default?"""
"I am geeking out over the bug and the fix. srsly awesome."""
"Yep, will add in a separate commit.  Thanks for reviewing the code, it is really appreciated :)"""
"I think this entire special-case block and corresponding expense of `.index` can be eliminated if the `jQuery( this ).remove()` on [L296](https://github.com/PaulBRamos/jquery/blob/6fb68d22f4b136f0b467cf060ea3286f8cc6fcce/src/manipulation.js#L296) moves af"""
"The current and advance callbacks share the same type signature so the empty iterator just reuses the same function for both, hence it doesn't follow the naming conventions that the `ITERATOR_SET_CB` macro requires. I could probably rename `empty_iterator"""
"@Seldaek: Using such a path pattern, the `?` gets encoded anyway. So it wouldn't generate a query string."""
"@whitethunder, I told them, but they don't trust me. Seriously, project is an excellent name. More, I'm looking forward for a method like """"user"""" or """"company"""". Also a great idea would be methods """"topic"""", """"post"""" and """"comment"""". =)"""
"indeed. we could either make the tests use the new methods instead (has to be done some day anyway) or simply switch to static calls.  But working with E_STRICT is kind of impossible at the moment anyway with the current 2.0 head. too much stuff not yet"""
"No, it wasn't; nice catch. Fixed."""
"Yup, knew I'd forget something..."""
"Isn't it weird that it works with java.lang.Integer but not RichInt?"""
"Ruby should really optimise these kind of things automatically :cry:"""
"why allocate a new String instance?      def apply(name: String): Node = hash(name)      def fromHash(hash: String): Node = hash"""
"Why not a simple check for `null`? As the provider is injected in the constructor the object will be type hinted?  Is this to guard against people doing crazy things while extending the class?"""
"Hmm, sorry for forgetting credits - lost them while rebasing :(  First and biggest credit goes to virusav, as he wrote an original patch. Also credits goes to NeatElves and Infernos for problems research Also to crackm for inspiration and tests!"""
"woot - really awesome to see you in here !"""
"can you use Ruby 1.9 hash syntax?"""
"fixed"""
"Forget a """"}"""" I doubt it because the log gives me problems on several lines. I'll see to resolve this and I'll let you know. ;) All went well for the other files it's strange anyway =)"""
"Although I love warpten, I am sorry if I go to fast. code=trinity"""
"Style: we use license boilerplate like this:  ```c++ // Copyright Joyent, Inc. and other Node contributors. // // Permission is hereby granted, free of charge, to any person obtaining a // copy of this software and associated documentation files (th"""
"sorry, it's fixed now"""
"`if($connection && $connection->getFieldValue() ==  'close') `"""
"Could you elaborate why this is needed? That is, why is a static constructor needed. No idea why so I am curious :)"""
"At this stage I'm not sure the rails core wants or cares about this fix. :( cc @fxn @steveklabnik """
"oh nice find, that's been bugging the crap out of me"""
"He-he  http://filebeam.com/ab2603ca3af8b3130bbcac8da807677c.jpg  I wrote: """"Sorry, wrong patch)   Please delete this issue """" )))))))))))))"""
"Thanks a lot ^^"""
"@tarcieri I don't appeal to the authority. We are talking about formal languages.  In Ruby 0 is true. That is a fact. Given that fact, the one that has to justify that it is a bad idea that 0 is true is you. And you need to argue with language designers."""
"> that's really old and annoying bug.   It's not bug until it not propertly reported but just user _personal_ problem.  Must be fixed in [11255]"""
"WTF???? OMG ((("""
"What's up with the really amazing commits lately? Thank you!"""
The issue is only with passwords that got the high bit set on some chars for some specific combination of chars (not even valid utf8 chars). I think many could live with that minor risk.  I myself would want to use this code also on pre 5.3.7 platforms
"There are always odd things like `--single-branch` that come up and you should be able to implement without hacking the library. A client might want to copy default config over or do other weird stuff. `git_clone` is great, being able to do it yourself if"""
"Yes, it is! And I want to remove one from `node_crypto.cc`, but @isaacs disallow it."""
"This is just another of the things I noticed and fixed while this pull request was waiting. I don't believe I can create multiple pull requests with the different bits so it got included here.  The reason for this change is that the cookie_expiration_pe"""
"Will this also do scheduling?  If it could monitor loads then prioritise queues based on that would be pretty awesome."""
"What does """"the names are encoded"""" mean to you?"""
"This is not very readable plus has unnecessary merge with empty array. Better to replace with this: ```php if ($merge) {  $this->_inputDefaults = array_merge($this->_inputDefaults, $defaults); } else {  $this->_inputDefaults = $defaults; } ```"""
"yes... i'm sorry... but this bullshit.. any stable commit for 1 month) I should be happy? ofcourse - i have be terrible of your emotions... i want stable working of my server and nothing more.. How can break an off repo? I can not express it in words...."""
"My CI server agrees :)   --  Kenneth Reitz   On Friday, April 13, 2012 at 4:39 AM, Bradford Toney wrote:  > Python 3.x doesn't support .encode('hex') >  > ---  > Reply to this email directly or view it on GitHub: > https://github.com/kennethreitz/requests/"""
"i will do a new PR to fix this. sorry!"""
"ok, sorry for my reaction, fixed in recent commit"""
"That is how it was working before, I tried using model() but got several errors. I'm merging this as it is passing the tests, but I'm open to suggestions for improving it"""
"I spent almost an hour thinking about this code snippet.  I wrote some comments, then I erased them 3 times. Seems to be ok.  Since we now have the verification cache, and that cache proves to be fast, maybe the safest thing would be not to do anything in """
"Good to see that we use DBC's in more cases :)"""
"ah okay - didn't realize that the MAC_OS_X_VERSION_10_7 isn't define on 10.7 running 10.6 SDK.  nevermind then - what you propose is fine!"""
"There may be too many APIs related to path handling. Sorry! I just extracted this one since it was such a common pattern in the code."""
"makes me more sad to see people complaining that we fix our own mistakes..."""
"@stloyd wow, I was not aware of this method! I update the PR now, thanks."""
"Confirned by testing in game. Thank you :) I look currently how this can be used in code..."""
"It should only search for 'core/Loader.php' as the Loader class can be extended."""
"It helps debugging, and no security is violated as there is none :)"""
"Lol."""
"when tests running with Netbeans (all tests in folder) - this is problem because in Mvc/Controller/TestAsset/ exist the ZendTest\Mvc\Controller\TestAsset\Request class. """
"Knock yourself out :-) Even something that's replaced later that does the job would be better than nothing :)"""
"seems you even didn't tested this change. This bug already fixed by this commit (see DB part)"""
"i like InnoDB more :D"""
"This documentation disappeared in the process :-("""
"thanks! I've learned something today. :)  ``` ruby case record_or_options when Hash   options = record_or_options   options.assert_valid_keys(:etag, :last_modified, :public) else   record  = record_or_options   options = { :etag => record, :last_modified ="""
"Agreed with Vincent-Michael, at the moment many of those scripts were written people did not know how the spell system should work, as an example you can take script for righteous defense, which is an extreme violation of spell system rules."""
"why not ``lang_template['hide_untranslated'] = getattr(settings, 'CMS_HIDE_UNTRANSLATED', False)``"""
"To avoid `isInstanceOf` on every `computeHash`, I could replace the `hashingobj` with a custom internal `Hashing` which does the mangling if it's a class returned by `Hashing.default`, yes."""
"ScriptName removal won't be needed? -> http://pastebin.com/zF5xks2v (Still got to find out how to put proper comments in here) Ha, this should be it<pre> -- Inquisitor Hallard scriptname removement UPDATE `creature_template` SET `ScriptName`='' WHERE `entr"""
"That describes what options get generated for the homebrew formula itself.  We want to add the guard to the line because if the user doesn't request lua, we don't want vim to pick it up on accident (which it might do if it is already installed and we se"""
"Sort this constants with the old first"""
"Not really related to this particular commit, but why do you use DIRECTORY_SEPARATOR (even in it's shorter DS form)? It seems unnecessary, as using / will work on all OS, Windows included. I think your code would look better, and be easier to read, if you """
"Indeed, that's why I asked you :)"""
"Oh right, I didnt see it, ty  =)"""
"I considered it, but currently s3 only offers one encryption option so a bool was enough to get it done.  However I can see them adding other options in the future in which case passing a symbol that indicates type could make sense.  Lately I've been tryi"""
"Neat.  I wouldn't pass up the opportunity to use a crazy trick!  :)"""
"Thanks bro, missed it! :smile: """
"rsa, i glad to heard that you can configure your own compiler and, stop talking about multithreading, clean mangos sources has no MTMaps,   """
"thank you very much :)"""
"I didn't know about libev. I'll check it out! Thanks."""
"@isaacs  ```js function Deflate(opts) {   if (!(this instanceof Deflate)) return new Deflate(opt);   Zlib.call(this, opts, binding.Deflate); } ```  it should be `new Deflate(opts)`  (The commit is to big to add proper line comments ;o)"""
"sorry i just got this wrong... this iteration is just for applied areaauras... but this must be related to some aurahandling changes afterall"""
"Edit - just checked with my python script, I got 1350 records for ytdb 619 :) http://paste2.org/p/1981677"""
"WHEN REPORTING ERRORS, INCLUDE THE GOD DAMNED ERRORMESSAGE/COMPILERERRORS!"""
"This is included in the next line automatically."""
"Indeed it should :)"""
"`g.src=('https:'==location.protocol?'//ssl':'//www')+'.google-analytics.com/ga.js';`"""
"alil late a dollar short, nay .... the moments passed... :D nice language btw... i got over it... so np <3 ;)"""
"If this is only used in one place for one usage I'd rather just use ConcurrentHashMap and make sure that whatever gets put into it only has identity equality defined"""
"I guess we should move it back. This is pretty weird tho"""
"I've just updated this feature."""
"I agree with @grumpydev here. There should be no ctor just for testing purposes. If it's difficult / cumbersome to create a fake certificate then maybe we should provide some test helper for that instead. If the Ctor was already there then it must've been"""
"if we have  GRANT USAGE ON * . * TO 'mangos'@'localhost' IDENTIFIED BY 'mangos' ... So use login """"mangos"""" more correct than root.  No lol."""
"I'm sure you didn't mean to do this :)"""
"Method of """"trial and error"""" - the most efficient in the case of the development of open source. Welcome to the real world :)"""
"Removing `Apply` subclasses will break implicit highlighting in the IDE, in the worst case forcing us to fork the IDE for 2.9 and 2.10. The change in itself is fine, but let's not slip this in one day before a milestone goes out."""
"Don't put this on the prototype.  People will get clever with it."""
"I agree this is a big ugly hack that will break more than it will fix. The original script should be kept for other systems.  If necessary you can use uname to discover distro/version and switch proper logic using that. (but I guess there are better solu"""
"Oh, no, see what you mean now. Fixed."""
"yaaay :)"""
"Damn sorry for messing/merging in your area theuni ..."""
"@syamn don't worry, plugin authors can easily update their plugins, just like when a new minecraft version comes out :)"""
":cat: :cat: :cat:"""
"Thanks a lot seph for fixing this :)"""
"@mbax, @EvilSeph, here's a simple solution that does the work:  First implement a version checking mechanism is CB that looks for """"target-versions"""" in plugin.yml. (If current craftbukkit version is not in the list, then not load plugin and print """"XXX pl"""
"nice Unit::GetUnit is so ugly!"""
"Oh, I just noticed that. Good point, sad times :("""
"Good call, removed!"""
"Obfuscated code :)"""
"Graz! ;)"""
"Thank you, boyz :) Next target - prepared statements :)"""
"No need to repeat the name of the documented element.  A factory to create MessageQueues for an optionally provided ActorContext"""
"Is this code path ever followed? I mean, will an unsigned integer ever be larger than (unsigned) -1? """
"they are right, i have sniff checked"""
"@Shauren 0bb86a3e97cf720512ae00c9bee209a30c5fb928 This patch still needs some further fix."""
"Is something like this: https://sourceforge.net/apps/trac/xbmc4xbox/changeset/30905/trunk acceptable?"""
"Oh noes, code style. :( I'll be super careful about that in the future. Too many projects here."""
"That was the easiest way to recognize a h5bp project :)"""
"Thanks for your amazing work, Malcrom ^_^"""
"Test case needed:  ```scala // BEFORE scala> val x = """""""" x: String = """"""""  scala> ((x: x.type) match { case a => a }) : x.type <console>:9: error: type mismatch;  found   : a.type (with underlying type String)  required: x.type               ((x: x"""
"@Razer6 you are right. I did not noticed #4594 :("""
"@elupus should we set the same strategy for HasAudio? Currently audio visualization pops up for a short while when starting a video. I think this is the reason for that."""
"Yes, it would be really cool if you could update the wiki. But don't forget to say it will only work from version 1.4.0 forward!"""
"ok, thx for taking the time to answer ;) great work!"""
"Oh for Ruby 1.9 hashes! :crying_cat_face: """
"Fair enough. I'll redo the entire thing, it was a bit hefty in the end :)"""
"@V10lator  ``` try http://pastebin.com/FcNGtFDV ```  @mbax will be happy then ;)"""
"Sorry, that was lazy... :/"""
"Maybe Jlogger?  cheers On May 14, 2013 7:30 PM, """"Parker Moore"""" <notifications@github.com> wrote:  > As the commit message says, it was a hack to get around pygments.rb's use > of the Ruby stdlib Logger. I guess I should rename this class, but I > co"""
"I thought you meant 3-bytes, sorry."""
"Yes please do, would appreciate it =)"""
"Commit [11000], jeez! :) ...I joined when mangos rev was only [1527] and now it is 11k - nice progress, I'd say :)"""
":+1: @holman.  but who knows how many apps have this issue and don't even realize it. :("""
"that wouldn't work, cause it would return a symbol hash key, which is not a json primitive"""
"This means we are modifying the original options, no?"""
"Great! Thank you again!"""
"i cant translate french :["""
"Assuming you don't depend on deserialization and/or base64 decoding for verifying integrity (see my other comment below), then maybe you could defer deserialization until after you have verified expiry.  You don't need the deserialized object to verify in"""
"Also why enable this one by default and not the other ones? It's pretty annoying having to enable them in every new project."""
"@Wolvereness  Oh sorry i meant CB 1.1-R8 haha... Anyways, i wasn't talking about bukkit versions, i was talking about official minecraft releases.  Also! Now i seem to be getting a lot of crazy people complaining about my projects not working on the lat"""
":x: This not follows your own method signature. Expected string passed Exception instance"""
"This line break our apps :(  The only solution I know is change the session key to make their cookies expired. """
"Why?"""
"I agree - lets fix it up there as well...."""
"The documentation wasn't updated. So encode has to_xml's documentation.  Also, why is the API changing in a point release (2.1.1)? Shouldn't this have waited until 2.2, and a deprecation warning added in 2.1.1? """
"fyi I don't think this has worked since this change, no flash messages :("""
"@grum  >The other solution is to apply random obfuscation to any CraftBukkit build on everything except the main methods and the Bukkit API. > >Maybe we should just do that.  ...Wow, are you serious? Im speechless.. as @rehv said, that makes me scared"""
"@davidben and @anthonyryan1 I landed this since there was a good patch (even saved 1 byte gzipped) and I'm glad it solves the problem. To ensure we don't regress on this we'll need a unit test; contributions welcome. I've opened [ticket 12040](http://www.h"""
"Do you really expect the client to disconnect (and the player to be unloaded) between lines 33 and 43, or between 43 and 50? If yes, then there's now an exploit to avoid damage by disconnecting between 44 and 50..."""
"Because the old text """"src/qt/res/src/*.svg"""" had covered the newly generated src/qt/res/src/bitcoin.svg i had to """"rewrite it arithmetically"""". :) I think it's okay like this."""
"seriously, stop discussing your bugs on completely unrelated commits"""
"Wow... Mangos rullz :) It's a great day for our community."""
"onlinetime is spammed anytime worldsession opcodes are handled... its not like it is supposed to work i intend"""
"Don't like it, either. Ok, we'd better stay with deserializing parameters before passing to the `Invoker` - it's much cleaner. But I'd get rid of static extension methods in favor of something like. `IParameterResolver` interface + implementation, that'd """
"`_protect_identifiers()` is supposed to be a protected method, but currently it's public because all the drivers use it for DB forge. There's a `protect_identifiers()` method (no underscore) that I believe exist solely for this purpose - can we use that i"""
"@CelticMinstrel Agreed, not impossible, but nearly impossible. :)"""
"I'm disappointed by this commit, unless it fixes a bug. Does it fix a bug? I also wanted to get rid of the code duplication, but @rwldrn had asked me to wait till grunt was integrated before making further changes to the test suite. My plan was to have php"""
"yes that could be hard :)"""
"`GitHubAccountAuthenticator` sounds reasonable."""
"I'd like to add you to my professional network on LinkedIn.  - Sam  Sam Rose Student Ambassador at University of Glamorgan Chester, United Kingdom  Confirm that you know Sam Rose: https://www.linkedin.com/e/-15j9z5-gs0w7njf-1k/isd/4048244323/Yqb0l5AJ/?hs=f"""
"If the method signature is the same as the parent there's no need to redefine it.  I'm not sure why you're even trying to do so. :-)"""
"`depends_on` should come after the checksum and a blank line, i.e.  ```rb homepage ... url ... md5 ...  depends_on ... depends_on ... ```"""
"ThrowTypeError"""
"This should probably be moved to the `debug.xml` file as the `ErrorHander` is only registered when debug mode is on. We don't want this listener to run in the production env."""
"Nothing there is related to this commit. I would guess you do not know what are you importing, mixing old character and world updates together."""
"Looks like automatic conversion to me (also the original script author is an idiot)"""
"crash ! :("""
"Well done, thanks for not being stingy on the mc-dev update :)"""
"Nice solution. Definitely should start using this method."""
"lololol good catch"""
"@akaariai small comment typo."""
"I think this is checking false to avoid keys with `nil` value to be a failure"""
"Different styles used in the file. I selected this one, as `@see` is not related to the function signature.  Where can I read guidelines for docblocks? Especially regarding aligning different parts of docblock."""
"nice to hear this, good job !"""
"Ahhh yes. Brain fail, sorry!"""
"All I gotta say's we got passionate people here and I look forward to all your commits and releases... I guess this one wont go away without a """" You $%#^ Semicolon Nazi* """" and thus voila!  * http://en.wikipedia.org/wiki/Godwin%27s_law )"""
"cool"""
"If you want you can, if not I can go through and fix the old code. There isn't a phpcs test for this which is why it is inconsistent."""
"It basically tells the server to convert all int's into strings. This is necessary because we use 64bit ints on the facebook side, but many PHP installations are still 32bit -- and fail terribly when trying to json_decode 64bit ints. By requesting strings """
"Why this change? I have a big aversion against """"new"""""""
"Why is this being removed? It's only a """"duplicate"""" to improve performance."""
"Botched the commit message on this.  This just fixes browser support information."""
"Yea, all this is extremely weird."""
"i don't know ruby, what does this syntax error mean ? Library/Formula/chkrootkit.rb:32: syntax error, unexpected $end, expecting kEND """
"case m_targets.m_targetMask & TARGET_FLAG_SOURCE_LOCATION lost in this implement :("""
"Makes sense, I haven't tested getters/setters yet"""
"I think a type alias that is used only once `MultiHashMap` is probably not that helpful. I'd just inline it so there's one less thing to lookup when one is reading the code.  I think the same goes for the `Responses` alias. In particular I prefer to rea"""
":metal: Love getting rid of the duplication"""
"@davidwindell I forgot that this was on develop. :)"""
"You didn't :) i was giving an explanation why it does not have comment and some code is commented out :)"""
"... but when `statisfied?` is called, the build environment is not yet set up, so it is not satisfied :-("""
"In principle LGTM but a regression test would be good, I'm not able to trigger a memory leak with getPeerCertificate()."""
"Ooh, a reason to bring mmap and map2 together in mmap2!"""
"No moar YouTube and vimeo? No moar hearts? :("""" +1"""
"@wsoltys  Sorry bother you but i still get no git version http://paste.ubuntu.com/782072/ also got some annoying virus warning. see xbmc forum"""
"I see two issues here:  1) It's easy to overflow length on platforms where size_t is 32 bits. Not directly exploitable but something to keep in mind. 2) length is unsigned here but the length argument to SetIndexedPropertiesToExternalArrayData() is a s"""
"hmm, how can we make a route in that file public if the standard security setting sets everything in it as non-public?"""
"Can you make this red?"""
"I can't put this together right now, but would greatly appreciate someone else doing so."""
"cheating, stupid GM's... additional checks - not bad."""
"I don't think it should be a notice. The fallback from fr_FR to fr is totally normal for instance, and this would be spamming your logs."""
"Ops, that seems to have [failed the build](https://travis-ci.org/rails/rails/builds/6066789) =("""
"If the git:// protocol fails for some reason, try again using http://  This will make the fall back discoverable for people who are not aware that they have a firewall blocking git"""
"That last sentence also reads a bit odd."""
"Why are we renaming these functions? If we stick the original func name, we will have less code to worry about. No?"""
"I was thinking last night we should maybe build some of this into appconfig, so we could just say AppConfig.service.twitter? or something like that.  I have made many typos with these damn hashes.. :P  #justathought"""
"Because the build system changed to require them since the last release (over a year ago) and while we could include them as general dependencies, they aren't needed for the release version. When FontForge is next released, they will be needed as general """
"@Schmoozerd  thx for a fast, good review and your help in develop"""
"Also, I'm not sure how I feel about this being within the ServiceBusMessageBus class, feels more like a ServiceBusConnection function.  However it is this way in other scaleout providers :("""
"no equals and hashCode for this one? why not case class?"""
"I think this is not necesary because buff is given by the mount. Also if you remove this line the """"fly hack"""" motion without mount is fixed. """
"I wonder if it wouldn't it be better to implement this in `Geometry` instead. Then, at the end of the `JSONLoader` call `geometry.applyMatrix( new THREE.Matrix4.makeScale( scale, scale, scale ) )`."""
"@CorrieKay   I'm aware, I'm just really entertained by the screaming contest above. :D"""
"I like this mutable keyword. Only the cache is updated for better performance next time round, so perfect use case I think. I'm happy to do this, but how does it sit with OF convention? Does it work on all OF compilers? Also, the bHasChanged flag will be """
"Style."""
"ALTER TABLE `character_queststatus_seasonal` ENGINE=MYISAM; in world misc ?"""
"abused as poor man’s option"""
"rbac_group_roles"""
"<button onclick=""""alert('speaking of security....')"""">hhahaha WHOOPS"""
"This line might be going a little too far. It obfuscates whats going on by putting a really really big ternary in the way. Can you roll it back or try to make it a little more readable?"""
"when I write this I was wondering using the set or add.thanks ,the set is more good."""
"nah it's a none-node related C project, thanks I'll check it out!"""
"I renamed the methods to `addViolationAtPath` and `addViolationAtRelativePath` in the latest commit."""
"Yeah, I think so"""
"yeah. I also liked the way it was.  I think that the default 404 is going to cause the most confusion, It makes me think """"I dont want to return a 404!"""", and I'd probably delete it.  Saying that, a magical """"super"""" is probably equally as brittle.  A"""
"looks like soon we will have a fully unstable core :))"""
"Ah, good. Okay, leave it as is."""
"lol. nice to meet you. Didn't actually expect you to reply.  It's nothing really, just your events (specifically BlockPlace) is such a godawful event. :(  I was just venting a bit."""
"yes :)  Edit: Well - most spells should actually work correct with dbc-values. the rest could be considered exceptions or custom."""
"This is ugly..."""
"Congratulations, Lightguard! A new dev in the house! /cheers"""
"rsa i not meaning as fix your horiable hack written wihout attempt understand real problem.  My patch in gist fix _real_ porblem with wrong """"future"""" msecs time in object last update field. But not resolve all porblems.  You pointed """"fix"""" just """"recorrupt"""" b"""
"Dude, ytdb sucks."""
"@Garrett: Perhaps you missed my thanks in the commit message? Definitely appreciate the input.  @jdalton: When you get the volume of questions, support requests, possible bug reports, and other miscellany that I do related to jQuery it significantly help"""
"Thanks a ton for taking this on, @domenic.  I just signed the CLA.  I think I've forgotten most things about the V8 API since I wrote Contextify a couple of years ago, but I'll check over the code and help any way I can.  I've accepted various pull reques"""
"Oh God. I'm having trouble with PRs on GitHub. Awkward. I'll better make another one using a fresh copy of Symfony."""
"Changed it."""
"This is a bit hackyish :)"""
"I'm very sorry, it's my mistake. I removed this line, could you please review this again?"""
"Dennis, you rock. Seriously."""
"When using the `useUploadName` option, should we be doing any sanitizing of the $_FILES['name'] value which will be used as the target filename?   The `basename()` should help, and I hesitate to change the filename too much if the user is expecting all """
"I wish PHPStorm had context specific docblock template... :("""
"I'm really worried about SEO and color : transparent; maybe i shouln't ?"""
"Cool ^^"""
"Yup, agreed, and indeed I was mislead by the other issue (which was the UrlMatcher not decoding). I tested and it seems to work all good now, with full encoding. I just decode the / back and turn %20 into + because it looks better and seems to work just fi"""
"Is camera example supposed to be working? It's very confusing now ;/"""
"Let's go with `ensure_reset_password_token!` here too. You can also change the line 47 of this file to call this new method. :)"""
"mysql is weird, this shouldnt work as it does  WHERE (npcflag & 0x10000000) != 0  is way more readable (obvious)"""
"That Commented out code should be removed, if not used :)."""
"@coelho Because Bukkit is an open, friendly community that encourages discussion and feedback."""
"why run `data_size = str->Utf8Length()`, just to check for ascii and run it again?"""
"They require it.  toIndexedSeq doesn't because IndexedSeq is covariant."""
"d'oh! that's one down. don't forget about """"mail"""" => """"male"""""""
"why is a hack for me work excellent , is like other script (sindragosa), and don't need more complications An answer all spells which are in dbc(like are defined) have theyer attributes and should be using for script of the boss/npc?"""
"evil is such a judgmental word. :("""
"again, another style violation for no gain."""
"This was pulled from the old makefiles and gitian descriptors. It's required for deterministic builds."""
"Thanks for being so responsive to the suggestions!"""
"off course, sorry bother you, i return to 7906 so i cant provide data :("""
"Good work qaston"""
"is `/` allowed character in generated cookie? Then it should be enclosed in quotes, otherwise config will barf."""
"I think that would be great! shall we call the color uniform color_uniform ?   Shall i send a PR? - I'd also add  a fix for a typo in matrixStack.getModelViewPorjectionMatrix() if that's ok =)"""
"This whole commit looks totally wrong."""
"Had it that way first, but decided it belonged in the same test block. Next time we won't. Good to know. Thanks!"""
"`prefix` is the formula's own prefix, and libiconv and gettext aren't linked into `HOMEBREW_PREFIX`, either. It would need to be `#{Formula.factory('libiconv').prefix}`, etc, since libiconv and gettext are keg-only brews."""
"... and the solution:   gem install rake-gemcutter  Sorry! :$"""
"So this was the missing path. Nice work!"""
"This can be done as a subformula instead, so that it will be protected by a checksum. See e.g. the `git` formula for how this is done."""
"I see you worked on strings, but you could cut down on the number of required strings if you made them """"Godmode: %s"""" and 2 strings """"Enabled"""" and """"Disabled"""""""
"I agree about that in that case. But i'm talking about the patch in general. There are several places where you replace a disabled state with a timed out state."""
"Objects have their SHA-1 sum which determines their filename, though you could argue that a collision is not something that we handle."""
"There turns out to be one more issue here -- this deleted line defined $owner, which is used below.  So seems that it needs to remain.  It's used by       Session::completeLogin($owner)   @AaronKalair , @ginatrapani"""
"Changed it to just be `length > 1`. Not really useful to anticipate weird stuff here."""
"we are talking about a config/ directory under a bundle. How many config files do you here? One for the routing? One for your services? One for the doctrine metadata (and perhaps of few more if you use one file per entity)? That's not a lot of them."""
"Maybe It was already solved: 687e68d88f5e1a7bb6a2d876fbbc5b8c4571d853 #9803"""
"Glad to see this land!"""
"Style: Long line. Wrap at 80 characters max. I believe I pointed that out last night too. :-/"""
"`!=`"""
"Eclipse claims that this type violates the """"Unique Particle Attribution"""" principle for XML schemas. http://en.wikipedia.org/wiki/Unique_Particle_Attribution  Can we reformulate this type to avoid this?"""
"http://www.trinitycore.org/t/trinitycore/ticket/307"""
"It was fine before this commit, this commit broken :("""
"This is bad for cherry picking too much happening in one commit.  Can this not be don in 4 maybe even 8 separate commits?  I lost the plot when the cache changes started =("""
"Because a hidden input field is still affected by positioning rules, like another element would appear with `visibility: hidden;` being set."""
"darn, no I only noticed: 4a136cc is the SHA of the commit we had trouble with last time - this shouldn't be in the repo anymore... maybe one of those machines you used still had the polluted repo on it?"""
"May I ask why you decided to split the regex?"""
"You could also use something like: ```ruby column_names_with_alias.each_with_object({}) do |(cn, an), hash|   hash[cn]= row[an] end ``` which is more idiomatic but avoids the array allocations."""
"Finally :)"""
"Breaks lot's of tests.   <pre>   1) Error: test_highlight_should_sanitize_input(TextHelperTest): NameError: undefined local variable or method `allowed_attributes' for #<HTML::WhiteListSanitizer:0x1010af260>     /Users/arun/checkouts/rails/actionpack"""
"Well this is bad, every update I'm going to have to wait about 3 months for all the plugins i use on a daily bases to update.. I really hope this gets reverted :/"""
"Ah I forgot about that and didn't test properly, sorry. The buildout directory is now included in the install whitelist. I've squashed the commits."""
"Hmmm, it looks nicer like this but now it's very slow on my notebook :(. Curiously now it's OpenGL that is worse than ANGLE.  Multimaterials are costly and lines are costly too. I used flat shading instead exactly for this reason."""
"Well, damn ...  Quote from the `str_replace()` manual: >If search and replace are arrays, then str_replace() takes a value from each array and uses them to search and replace on subject. If replace has fewer values than search, then an empty string is used"""
"Okay. That should save some time, and it doesn't break the existing tests, so I guess it's okay. I wasn't sure how hard to make diff try on this, so I just turned everything on. :smile: """
"no clue ... i refactored the packaging script at one point to match up with ios and atv2 - i guess since then you lost it ;)"""
"ah, now I get you; hmm, too many Runnables nested here"""
"The FormRegistry being new in 2.1, it seems weird to have a deprecated method in it"""
"Thanks for the commit finally :D  I would rather see the account and char ban commands actually work before the vehicles patch :P"""
"It is needed for LATM AAC. It can be removed once we update ffmpeg, but I'm waiting for some Windows developer to test it. (https://github.com/xbmc/xbmc/pull/18)"""
"@chopmo Argh! You're right. It seems I didn't imagine that case. Let me take a look..."""
"yes, but unsafe and useless rename now, batter let been as-is. We can't rename internal sql update guards anyway..."""
"That struct looks pretty good. For the basic version (i.e. `git_clone`) all you should need to provide is how to tell you about progress and what branch to checkout. I think we might want to add a flag saying whether we want to ignore SSL errors."""
"See, exactly the way you respond to my post shows me what an ignorant you are. Thanks for writing a comment though."""
"Typo."""
"There are tickets actually. They are all related to Firefox not executing the string passed to globalEval synchronously when using script tag injection and non-async scripts are being loaded (jdalton confirmed to me this was fixed in firefox a few months a"""
"The first thing is if you URLDecode you need only URLDecode the filename, not the entire thing, right?  After all, the rest of the path comes from a filesystem that is NOT to be URLDecode'd.  The second is whether it should be done at all.  As elupus point"""
"revert this commit fix ^ http://gist.github.com/619376 crash. very strange."""
"interesting fix... ;) """
"@BangL Check out my message about 3 hours ago in this comment section. :)"""
"@vicb: I somehow fail to come up with an elegant solution regarding explode etc. - any suggestions?"""
"So we can't put file larger than 4GiB in index? Sad."""
"I understand that these functions are designed to mimic String equivalents, but I think that Array#from should either return [] or raise an exception if it is invoked with an index that is out of bounds.  """"pete"""".from(4) # => """""""" """"pete"""".from(5) # => """""""" %"""
"Bound functions are particularly horrific in V8 for some reason. http://jsperf.com/argument-slicing/2"""
"@TnTBass I don't disagree with a detection mechanism, no. I disagree with the use of dynamic package paths that cause errors at a random time, not fitting it's purpose. Add a plugin.yml setting, it is all that is needed to do what you want plugins to do."""
"Wow!  Before:      real 8m54.220s     user 7m18.483s     sys 0m50.399s  After:      real 7m46.224s     user 6m13.979s     sys 0m44.507s"""
"for the cookie on websites, we use """"fbs_{app_id}"""", the idea being that only instances bound to the same app id should share data"""
"If i disable cases 0, 1 and 5 crash move to case 8!!  http://pastebin.com/NMhSRN5D  How i can set GCC to UTF8 encode?"""
"Using event delegation, the dom can be interacted with before onload fires.""""  Can you explain this briefly?  Thanks!"""
"@mislav Andrew and Tobie have done plenty of code consistency commits. Consistency is a good thing."""
"hehe. I was waiting for the loolz. Thanks :-)"""
"hmm ok or is it from the GM status? though .gm off and .gm visible on  """
"RSA want to say, that if fixed one thing, it would be nice fixing it by more wide way for future development, something like this =) http://paste2.org/p/833860 """
"Congratz and thanks for everything!"""
"AGPL == epic fail. Don’t count on getting all that many contributors."""
"This permits SQL injection. User data should never be interpolated into the keys of a find condition."""
"+++++++++++++++++++++++++++++1. Awesome commit."""
"Another interesting update to this method in the same threat http://forum.openframeworks.cc/index.php/topic,6092.15.html"""
"yes, that contains strictly more information. I'd like to use FiniteDuration more in method signatures which expect finite durations …"""
"I have a problem in AbstractToken.php  Notice: serialize() [function.serialize]: """"id"""" returned as member variable from __sleep() but does not exist in PATH/vendor/symfony/src/Symfony/Component/Security/Core/Authentication/Token/AbstractToken.php on line """
"AUTHORS is alphabetized by last name"""
"Aw man, you can't grow trees instantly anymore :("""
"It should be quite easy to implement something more sophisticated using ``preg_split``."""
"Yes you right"""
"Oh sorry ..."""
"So we could leave it as `.php`."""
"what will workerID do? worker.uniqueID looks weird"""
"more SmartAI power ^_^ thanks for this"""
"Wow! Thanks. Maybe the patch and has no relation to the infinite spawn chests, but he solved it perfectly."""
"Yay! :yellow_heart:  """
"Where is .destroy() defined?  I'm confused..."""
"Oh, sorry - disregard, please."""
"This list is useful to me. I always forget what are the supported methods and I always look this guide."""
"I have taken the bold step of enshrining this advice. https://github.com/scala/scala/wiki/Coding-Style"""
"The idea with a factor of something else is that there is less knobs to learn how to turn. But I'm fine with either. """
"testing"""
"Nice commit! This will help me for encounters event :)"""
"I think the output parameter really needs to be a `git_config_backend_iter **` not just a pointer, so that a new value can be allocated.  Also, I suspect some backends may want to support the """"match"""" behavior natively instead of relying on the wrapper t"""
"I agree, QAston, he really does a good job, I was not forgetting that. But maybe the word """"gratefull"""" was a bad choice for expressing myself :p"""
"he means more working quests, hackfixed ofc"""
"Whoops! Looks like something I added as a temporary hack during debugging, and then accidentially left in place. This explains why I had two places stripping quotation marks: this one and my own line 452. I guess I'd better remove lines 442 through 444. M"""
"Try fix Gargoyle, and Ill take a screen :D"""
"Hmm ... but on the second though, I think I'll make the `copy.avatar` to have a `@file` assigned to it first, and assert that `copy.avatar.file?` Then, I'd assign original.avatar to it, and `assert !copy.avatar.file?`"""
"Does not compile on FreeBSD 7.2 :( http://gist.github.com/648972"""
"sniffed ofc"""
"ok, makes sense. i only use the name in the line below, to replace in the driver name pattern, i don't need the actual manager service name.  regarding the problem of default might not be named """"default"""", i would inject two parameters then? the paramete"""
"great idea this. could be used to script arenas/bgs/opvp/wg instead of hacking up the core"""
"Next time, don't in this PR, leave trailing commas in array elements"""
"Shouldn't it take which claim to authorize?"""
"Sorry.. this Bug that i mean, caused by commit https://github.com/TrinityCore/TrinityCore/commit/f3ec76faeb63009ddff357a93b61220fc6039f47"""
"Actually I don't think anyone would write `.where(""""something like 'some value%'"""")`. It is more likely to be something like `.where(""""something like ?"""", params[:name])` to protect against SQL injection. For this particular case I see how """".like"""" would be use"""
"Double sad face :<<"""
"Cool, I'm looking at this now"""
" @burnham There is not a single creature that have deathstate set :) There was one before this commit (34297), but i could not find that creature in any sniff (a full sniff of kologarn encounter), so he was removed in 2011_06_27_06_world_creature.sql."""
"Wrong fork. :-)"""
"this should check if the cookie is already set, if so, don't set it as otherwise it will bust upstream caches..."""
"Oops sorry holic, added an extra l by mistake >_<"""
"ok, bcs I have old core and some issues after fixing are not closed :) anyway thanks"""
"If you are signed in and you are redirected to new_session_path, it will redirect you to after_sign_in_path_for automatically. :) So it does an extra hop but for the user it works the same."""
"Have you tried pulling in the same version of finagle that we use for the rest of the project? Too different?"""
"urllib3 does the right thing with threads so it shouldn't break, more connections will be spawned and pooled correctly from what I've seen in the code."""
"haa here it is, i get it now, would never had find this //helpers part def alone honestly . ok thanks to both of you guys and ofc no hard feelings NoFantasy :)"""
"This loop will continue until it reaches the target or hits a cycle, so it won't fail on precision. However, we might run into problems from something like `font-size: 100em` or `opacity: 100`. And even if we don't, I suspect (without evidence) that it's """
"@bergerkiller No!!!! :("""
"@just3ws in your situation, and any other situation with heterogeneous environments, I would recommend adding the bin directory to your `.gitignore`.  @dhh's position above is that such environments are in conflict with Rails conventions, so the `.gitign"""
"Also, assigning vars inside a transformation?"""
"This line is missing in the latest version, causing `uploaded_file` always nil. Is this intended?"""
"Why not a use a hash in the test?"""
"Redundant.  `@Before` already tells enough information."""
"Nullcheck 'e' here. Yeah I know it's a rare case but it prevents huge bugs in the future.  Also, can we change 'e' to 'event'? Obfuscated code and all :)"""
"I know that style guide is not ready yet, but we used to discuss that variables should be named in lowerCamel case.  Variable Gender makes me feel uncomfortable."""
"Nice, less typing is always welcome :-)"""
"Shit that was my fault sorry @EvilSeph"""
"so why the reload?"""
"crash log ends with """"ACCESS VIOLATION"""", nothing is there... i just catched that """"VisitNearbyCellsOf """" in console output"""
"Cool. I'm out of town until Tuesday, but I support both this change (with comment about i <= 1) and re-deprecating isResolved/isRejected. :)"""
"All my players can use all gm commands :( I've just run the sql updates :("""
"Should proberly be next instead of end."""
"Please indicate if there is a failing test case, or if the problem is related to an overlapping commit.  I can re-factor the exception use in authenticate_or_request_with_http_digest easily enough. All other exceptions are used to provide useful failure """
"lol"""
"There is some code duplication here. I would have written :     return $this->has($name) ? $this->attributes[$name] : $default;"""
"BAD. ASS."""
"On Wed, Dec 21, 2011 at 5:24 PM, viktorklang < reply@reply.github.com > wrote:  > > > >    // FIXME: Dispatchers registered here are are not removed, see ticket > #1494 > > -  private val dispatchers = new ConcurrentHashMap[String, > MessageDispatcher] > """
"And no even if the encoder produce nothing there is overhead as we need to store it on every encode as we not know if the next call will maybe not produce something"""
"Sorry, I'm an idiot and can't copy/paste correctly."""
"all auction code expected executed in main world thread only, not in map threads. So remark about not safe just lol."""
"It passes.  But instead, I included a test that is divorved from the standard library (ie, it doesn't depend on the signature of `Option#getOrElse`). It was more convenient to test for an error due to failure to eliminate a tail call than to check for s"""
"yes, swimming case missing, but in pet code possible :("""
"whoa. great job!"""
"cool ! now in a """"official"""" mangos. thanks to Lynx3d for a long time vmap developing in fork! """
"If we're only storing the first 32 characters of a 64-character hash, isn't it possible to have two hashes that are not unique?  Why not add a unique index to the activation_token field and catch a PDOException on insert that indicates a unique key prob"""
"why ? i used this rev, and i get this error on this rev, my commit is ok for this rev or ? i dont know with rev makes this compile error"""
"Awesome. A migration guide already. :wink: """
"Can this be handled in the actual database adapters? I.e. by putting PG specific code in the PG adapter.  Maybe by adding e.g. a `:fixed_string` entry to the `native_database_types` hash, and using `""""fixed_#{type}"""".to_sym` if fixed is true."""
"Or maybe this one is vehicle seat? Just got the struct from client"""
"That's a great idea. Would be a way to also solve Diego's issues with context switching."""
"Everyone hates this commit. What possible reason could you have for not removing it if everyone hates it? There is no good reason..."""
"Formatting of this documentation page is broken from """"## Not run"""" on"""
"Wow.. Thank you for putting this together.  I have been pulling my hair out for a bout a day or so now.  My application was failing to properly clear logged out users...etc.  I am going to download this and check it out to see if it solves the problem I """
"this needs a comment in `ActorCell.newUid` so that nobody gets crazy optimization ideas which then break hashing"""
"@lanthaler @schmittjoh you're absolutely right. I'll propose a PR soon with this only feature (the accept-* headers parsing) and I'll use `preg_split` which is awesome."""
"@kandera: I know that 3rd party scripts are not official supported - That should be only a information for user that has ChaosUA script in use :-) -> I like this script and it's IMHO great and stable. """
"i dont know :[. i only tested it for a few things."""
"piece of art :smile: """
"oh I finally see the problem. Well thats not how its supposed to work for sure :). We should be able to solve this better though by redesigning the tracker a tiny bit. Essentially we want to ask the tracker if we need to render + flip or if we can stay on """
"@cptspiff: Yeah this would be the best future solution although this can't be easily/quickly be implemented. @elupus: Do you mean we should move the URLDecode() call + URL check to URIUtils::GetFileName() instead to make it more generic?"""
"![bike shed](http://farm5.static.flickr.com/4154/5206490512_73bc75b5f2.jpg)"""
"Merging commits from 1.3 has this problem that we loose the visibility keyword for functions :("""
"As a fair warning, we **won't** be accepting pull requests that converts from 1.8 to 1.9 syntax (like the hash or proc syntax). It will just mess Git history and add a bunch of noise when we must focus on 3.2 regressions."""
"So much changes per commit is bad..Hidden issues can be..."""
"Odd, it might be the test being wrong... I remember spending some time fixing the multi insert in sqlite so I was quite surprised it did not work."""
"Was that it? THE THING THAT CAUSED SO MUCH HORROR. D:<"""
"Leave this in the `git_clone` signature. Feels very unnatural here. It's an output parameter."""
"Fixed in #1695. The fix seems to have been so obvious, we did it the same way. ;) I also added comments to make this special case more obvious. :)"""
"Is this thread-safe?"""
"So write a tiny bit of code to rename the nms/obc references to their versioned counterparts and unleash it on your 'trusted' plugins. Feel free to make a public tool out of it as well and perhaps keep a 'you can hackily update the plugin using this tool'-"""
"Cool, it's great to see this headed toward Promises/A!  I think it's gonna become the norm for multiple promise implementations to be in play in any reasonably interesting app.  I found a situation where I believe it still isn't fully compliant, so I creat"""
"Feels good man."""
"Sleeping is a weird variable name what about receivingMessages or ignoringMessages"""
"yes :-( I know, just saying that while it is broken, there should be other way to fix it (although I don't know how)"""
"poor guy"""
"And in all fairness, I'd also leave `origin_remote` and `local_path` in the function signature. They are not optional, and they are not going to change. So why not leave them there?"""
"@jonleighton you're right!  Sorry about that.  :(  Looks like this code is new in 3.1.  Are you comfortable with this going out for rc6?"""
"this should not use concatenation but query parameters (you can have SQL injection through the username currently). Btw, doctrine DBAL supports binding an array as parameter (which is not the case of PDO itself) so it is easy: ``WHERE s.identifier IN(:ids"""
"@opdenkamp: can we make CEC also less spammy ? On the PI it logs a lot."""
"Use 1.9 ruby hash syntax please"""
"Because the hostname will often be used to set a port or connect to a socket it could make sence to parse the hostname about port and socket, too"""
"Ach.  Lying comments.  Will fix shortly."""
"here we have an utf-8 string assign to a CStdStringW, are you sure it is assigned with the correct utf-8 conversion?"""
@adamv I notice you cleaned this up -- I had actually copied the string interpolated version from existing Formula...  If this is the preferred way would this be worth adding to brew audit ( I want to poke around in there for separate reasons anyway...)
"Nice work. :)  However shouldn't we have a fail check if the MMaps are enabled but the folder isn't found? Similar to the Vmaps check."""
"Lol, yeah. Damn headers. Would be very nice if someone took a look at the code though. Not sure it was the best implementation."""
"We need only one copy of it - it perfectly fits to be static.  Other solutions looks like a workaround/hacks: make variable non-static and create static instance of class to have only one unique non-static member variable.  We don't need constructors, d"""
"Renamed, though ... both look weird to me."""
"Nice! That actually makes it much more readable, thanks!"""
"shouldn't 'Doctrine\\Common\\DataFixture'  be declared before 'Doctrine\\Common' ? (like 'Doctrine\\DBAL\\Migrations' is declared before 'Doctrine\\DBAL' )"""
"https://github.com/TrinityCore/TrinityCore/blob/master/src/server/shared/Logging/LogWorker.cpp Check LogWorker::svc() it's deleted there. """
"Yeah, it's pretty nice.. it'll be nicer once it's an actual gem (pretty big git dependency, my Heroku slug size is up to 97mb) but I'm not complaining :grin:  While I haven't tested this, I have a feeling anyone using TheRubyRacer instead of Node.js as t"""
"Yeah, add another file that's blocking parallel download. No, really. Don't do it. The proposed solution isn't solving the problem, since """"people like me"""" wouldn't remove the file. We also need colors. We would just change them, or not. Please use sane def"""
"Six?  WTF man?  No oriental sites.  And oh yes, point to your own milk-toast, blog-only and fuckall-else  sites.  Nice work.  Seriously.   At least point to the list in the wiki.   """
"Noooooo! I actually prefer the old style hash syntax because the new one doesn't support objects as keys. :-(  ``` { author: person, person: [posts] } ```  won't work for example.  ``` { author: person, person => [posts] } ```  is a horrible mi"""
"ROFL, hard, but need =)"""
"I had no good reason for removing this comment - it must have been my mistake :)"""
"Usually we set event identifiers in `setEventManager()`, which ensures that they happen on injection. That also simplifies the `getEventManager()` implementation, as it only needs to call `setEventManager(new EventManager())`.   On Wed, May 8, 2013 """
"Yes, it's the node hash."""
"You'd need to escape every regexp special characters (think . for instance) in s.data so that you don't have false negative. Talk about a """"simple"""" hack gone wrong in the convoluted department (not to mention performance and memory issues in older browsers"""
"sure, I am fine to change it to whitelist mode UNLESS it is development or test"""
"You need to put this to require as well because its required for the tokengenerator. Without it the csrf component is not really useful. Also you probably do not need ~2.4. I guess something like ~2.2 is enough (needs examination). Furthermore, its miss"""
"This would have been perfect as a plugin, considering the potential performance issues. I will no think twice before using .data()"""
"`node.createDocumentFragment && node` could easily be misinterpreted as a code bug (it took me several minutes to figure out that its intent is to ensure `node` is assigned and not simply that operands got inadvertently swapped); I’m strongly inclined to """
"<offtopic> sorry for this post here :( In forum you have very nice post about Mangos in production servers make by Kero, is possible to move it to the mangos' wiki? </offtopic>"""
"Didn't really need to do anything special as anyways rendering was already done per material-specific parts, remember geometry groups?  Maybe we can just forget about this for now, anyways before it wasn't possible to have object with different face orient"""
"Seems like it should be set as an error that the controller can, optionally, check.  EDIT: validator -> controller"""
"I'd fix it myself, but I'm on my way out the door."""
"Maybe explain that pixel arrays are like unsigned byte arrays but with values that are clamped rather than wrap around."""
"argh.  sorry for the double post, timed out on the first."""
"Yes, I removed it as I was thinking the signature root.add(d, i) might be confusing.  I think it’s fine though, so I’ll add it back everywhere. :)"""
"I know the patch is only about the model... but my eyes hurt and I run screaming after seeing so many explicit hashes like { """"value"""" => """"1"""" } in the method calls. """
"![](http://images1.memegenerator.net/ImageMacro/7266902/EVERYONE-STFU-GET-BUSY-WORKING-OR-GTFO.jpg?imageSize=Medium&generatorName=Scumbag-Steve)  You don't HAVE to use coffeescript. I see a bigger waste of time bitching about it. I also see people bitch"""
"I trust you"""
"on the contrary, that was a great suggestion, I just simplified the code because of that"""
"This is Rating (unk1)"""
"You are right. Not sure why I added it there. Thanks for fixing."""
"There is a backdoor in this commit, i'll give a cookie to the one who finds it first:P"""
"Please revert this merge. The assets helper, which requires an HTTP request, is now directly injected whereas before it was lazily loaded. This is causing all sorts of regressions."""
"'EventId' is not a class or namespace i've get this while compiling"""
"version is scanned from tag name"""
"@Vincent-Michael  Two time you owned me, I'm so stupid ..."""
"I just spilled out a little for my fallen OmgOmgz :("""
"This sounds like a """"major"""" bug. Would you mind to fill a issue for it an close it ? So we have a better history of what bugs got fixed ?"""
"This is great. API's for the win.  I hope we start seeing this trend more and more, rails providing the interfaces, and devs dropping in the implementations based on whatever requirements they have. Can't install Redis in your stack? use delayed job. nee"""
"Sadly not,  the assignment works, but then after the code has executed the assigned value is lost again :("""
"move it to namespaces like `lib/tasks/gitlab/update_hooks.rake` and also include it to gitlab:app:setup"""
"omg qaston this is awesome!"""
"Im So Confused Right Now """
"As one of those """"stupid server admins"""" many are so fond of bashing, I applaud this change.    What does this means to me, as an admin? * I can see instantly which plugins may break between Minecraft updates.  This allows me to update faster than before"
"I'm tested and now get runic power, but if not have a target also get runic power (when damage), doubt is in description:  """"Patch 3.1.2 (2009-05-19): If no target is available for Blood Boil to hit, the spell will now cast and consume a blood rune but ge"""
"Aww I was using that :("""
"that seems to do the trick.  ... now some cuke is failing, though :/"""
"@jacknagel Got it. But for reinstall to reuse this code, install_formula should really pass on this exception, so I'm planning to put the catch part in the block from install in next commit."""
"encrypted?"""""""
"this was suggested initially but it is a BC break"""
"hello again rbac"""
"True true :)"""
"Please remove the priority for the encoders. The order of these 2 is irrelevant (they are handling different formats anyway) and I don't see a reason to make them use a high priority."""
"rather than add some real html and then selecting it, you should do what you did in your other tests, create some arbitrary html with jQuery, and append it to `#qunit_fixture`. I know that you're just following examples in the tests here, but it's somethi"""
"Then we can be happy that other browsers don't follow the spec, and hope for a fix to the spec instead."""
"Thanks so much for implementing this. <3"""
"@JamesNorris You're an annoying troll, and he was joking about using bytecode bypasses."""
"This is a silly decision that does not represent the majority of the rails community. Official plugin? Absolutely. Core? **No fucking way**."""
" Lutik, you are right. this is not problem base at fact that currently we use 0 as highguid for players, but anyway wrong way extract low guid. Specially because at off servers used <>0 high guid for players and possible we also switch to at some moment."""
"Thank you for that Malcrom :)"""
"well actually this doesn't optimize anything since the profiler is turned off by default.  public static boolean a = false; // The first field in MethodProfiler  so instead of making this accessible grum just made it worse :("""
"I see. Yeah. It's weird that we call 'em options when they go in and configurations when they come out."""
"Better not give the code sniffer reasons to complain :smile: """
"I have uploaded the libgcrypt-1.4.5.tar.bz2 to our mirrors already, we can look at all those once the restructure is done"""
"aawwww... damn. note to self: don't cherry-pick your own commits from 2010 without thinking...  updated: https://github.com/mangos/mangos/commit/f5f52d9a123a3a8f9250634be0fc433f436f4fda"""
"Oh, that's bad. Thanks for reporting, I'll have a look into this. I thought we already got rid of these :("""
"Oops.  I'm getting the same thing.  :-("""
"Forgot to implement it. :-) Done."""
"`Relative` would suggest to allow accessing the parent node as with `..` in file systems. We should either support such feature or rename the method to `addViolationAtSubPath` in my opinion."""
"This is so badass I don't even know where to begin"""
"@DouweM But if it were his own repositories and posts he would be allowed to delete and commit... And if he created a second account we would have probably violated the ToS as well and somebody would hang him for that... dafuq dude :(  bbl..."""
"Don't do this. This will break horribly if node.js ever adds multi-isolate support."""
"Can you please rather give us a test case that causes NullReferenceException? In your code there seems no chance that """"msg == null"""" could occur and I wonder if it really happens.  Also it's great if you can follow our coding style, so that we can use """"me"""
"The md5 doesn't seem to be consistent. It changes each time it is downloaded. Is sourceforge the only download url?"""
"@SuperSpyTX or perhaps, just maybe, this is a place for discussion not internet memes."""
"Github ate my comment. Let's try this again: ================================  I don't understand this commit. I just did: <pre><code> >> alternate = ActiveSupport::OrderedHash[1,2,3,4] => #<OrderedHash {1=>2, 3=>4}> >> alternate.class => ActiveSup"""
"You're right, misunderstood the setup. Thanks!"""
"So everything fine here :)?"""
"Oh yeah!"""
"I originally found that CSS snippet on StackOverflow and the comments came with it.  I've updated the comments, and added a blurb about why border-box is used.  FireFox8 still defines -moz-box-sizing so I left that entry. It appears that the latest Chrome"""
"bad hack :( your must set stacks on max (already true limited) for triggered only, not cyclic recast in real (see sniffs). @zergtmn - triggered spells true selected by spelldifficulty - 66228, 67106, 67107, 67108."""
"well great googly moogly"""
">I'm really glad to see that filter gone  So am I!"""
"Well, you can already do it with a instanceof check: ``if ($child instanceof self) {``. It will exclude buttons which are using a different implementation"""
"Nice, one less fork!"""
"Thank you, Neo2003. Nice patch :)"""
"Also, it is simplier setup for new pod runners, as SSL is required.  If you want the optimization of doing it via reverse proxy, you can enable it in the settings, but this is about creating the right defaults for Diaspora in a domain we can control."""
"Yeah sorry. If there's weren't a ton of other changes here I'd fix it but it's not worth it at this point."""
"Oooo... My God! Great job!"""
"and more safe. Thanks :)"""
"Returning early for `if (Configure::read('Cache.check') === true) {` would be great too!"""
"I don't trust it"""
"I AGREED TO US THIS PROGRAMN"""
"Agree, in this case, it's an optimization not needing to calculate the hash."""
"Spaces are not encoded here. Neither with + nor with %20. Why do you not want to follow the rfc? Just encode every component with rawurlencode."""
"So how would I pass the error to the error handler configured outside the factory? Something like this?      $this->prevErrorHandler = set_error_handler(array($this, 'handleBC');     [...]     public function handleBC($errorNumber, $message, $file, $l"""
"Yeah, I made it private to and renamed it to limit what could get here. I can't think of any valid need to `newDefDef` or `newValDef` with an inherited symbol."""
"Add a call to ```session_id($new_id)``` here to make sure we really start the new session. This will cause a new cookie to drop, but that is fine as long as our ```sess_time_to_update``` window is larger than our ```sess_forward_window```. Also, if we have"""
"@antirez : I noticed you never but the reported-by/signed-off-by lines to your commits. My humble suggestion is that this might be a wonderful opportunity to do so:  ```` .git/COMMIT_MSG:  [...]  Reported-by: foxx <email> Tested-by/Acked-by: charsy"""
"Sorry, my mistake - haven't noticed it was dropped."""
"django-reversion, not django-revision."""
"use my workaround :)"""
"nope Brian, that still didn't correct libraries still gone missing...using win server 2k8 / vc100.   whats missing, that got removed, that has now killed auto locating of SSL on win systems?  and remove the time/date on your patches, u just  pushed and it """
"Because I trust @Seldaek who spent time doing benchmarks on real-world apps."""
"How it pissible translate this messages?"""
"The JSON gem, ActiveSupport and yajl-ruby's JSON gem compatibility API all override that method. If someone includes 'yajl/json_gem' *after* ActiveSupport has been loaded, the above code will work but not the way the caller expects since the options hash f"""
"I wonder how much of a performance hit we'll be taking for this... strict mode is has significant performance costs that are attributed to the runtime checks that are required.   JFTR...  -1"""
"It is probably best to stick with `java.util.LinkedHashSet` until fix from https://issues.scala-lang.org/browse/SI-5767 is available on all versions we compile against. (I don't think we call remove, but still.)"""
"I think i write in commit note reasons.  I at 100% agree that idea good but i can't fix it without testing and reproduce. :("""
"As you know we love contributions ;) And thanks again for all the work!"""
"yea,right..silly me for not checking another commits..sorry :)"""
"`brew audit` uses this to audit patch URLs, which is broken now."""
"Reason for this hack is the same as the one for ICC teleport spells, which is documented here https://github.com/TrinityCore/TrinityCore/blob/master/src/server/game/Spells/SpellMgr.cpp#L3399"""
"It's [contradictory](http://pretty-rfc.herokuapp.com/RFC2617#representation-of-digest-values). I think that if the WWW-Authenticate Response Header provides a algorithm field we should return it."""
"I don't like exposing this idiom in our public API.  Let's just make retain() non-final and override return types manually."""
"Yes, this is true for the exisiting code. However, I can't see why the existing code used case insensitive comparison. I don't think that this is necessary."""
":O Thanks."""
"make it more clear: If a formula spam defines a 'without-foo' option another formula can `depends_on 'spam' => 'with-foo'` and it works as expected. Similar to how `build.with? 'foo'` did it already. I know that what you suggest is something different, tho"""
"sexy"""
"Omgf, i heard about this exploit TODAY :O  And what can be done about people who already used this exploit to benefit from passive auras? is there a way to """"clean"""" it out? maybe reseting everyone's talents or by deleting certain auras from the character """
"I'm offended that anyone would want to contribute something useful.   Also, I'm going to punch a random stranger in the back of the head tomorrow and yell """"that's for hot pink!""""  Based on this I'm happy to have immortalized hot pink in print:  !['hot"""
"Useless comment."""
"This is cool, I didn't realize lib is under versioning. Then I think the two lines can be simplified to `lib.install Dir['lib/clang']`."""
"What do you mean by that? I mean the order is not violated as far as I see, only there are holes."""
"I like that idea even though it has nothing to do with this bug :smile: """
"Previously everything included sortfileitem.h (which included SortUtils.h) for the SORT_METHOD_* enums, while actually used SortUtils.h for the sorting.  SortFileItem.h will (eventually) be dropped completely - it is now included only here (for the enums)."""
"Deleting this message would be rude. All my other messages were already deleted.  I got the message - I'll be sure never to post around here again - sorry for bothering all of you with my ignorance."""
"boo!  sorry dudes."""
"It doesn't return a dict but rather a bool. But I get the idea. Thanks."""
"wow! great work."""
"This is a bad idea for security imo. The use cases for reading cookies in JS are extremely rare and should be conscious choices, because it's a big attack vector combined with an XSS attack."""
"Eh, srsly, now that fucking dash that I left space after is killing SSH. Enough core stuff from me, going to quest batches..."""
"hmm, actually, we already have 1 compiler pass using a different namespace than in bundles :("""
"@atnan Prior to this patch we were not using Passenger on production, and all our applications were running under subdomains. You're right, maybe setting up this in two places can take people to confussion, in fact, a couple weeks ago on the spanish mailin"""
":)"""
"sounds great"""
"Would this work?:  ```ruby object.is_a?(Hash) && object.all? { |k, v| k =~ /\A-?\d+\z/ && v.is_a?(Hash) } ```  I'd also move this method below the `each_element` one, assuming it's needed as an extracted method at all."""
"@josevalim Do you mind commenting as to why we have the bin file in both the railties gem and the rails gem?  Maybe if we get to the bottom of that, we can get the rubygems team to help us out. :("""
"What's with all the weird tags Ry?"""
"because it wouldn't be blizzlike if I did :)"""
"Fixed, good point."""
"Thanks :) ^^"""
"@fmela That is not enough to enable SPDY support. Installing with `brew install nginx --with-spdy` and have a `nginx.conf` with a `listen ssl spdy;` and proper ssl statements will start with the following warning and disables SPDY support: `nginx: [warn] n"""
"Is this preferred? for (unsigned int i = 0; i < m_items.size(); i++)"""
"Yes :("""
"Oh yeah you are right.. missed that. Sorry!"""
"@ben @arrbee Bam! It now returns fully fledged references when the expression looks like a reference :wink: """
"Awesome, I'll play with this when I get home to make use of more og love. :)"""
"As I've said before this is a custom hash-code class for chunks.  Tell me how to fix it, but telling me to use class <OtherClass> is unhelpful."""
"`protected[akka]` is weaker than `protected`, so what are you saying? Subclassing applies to ActorSystem or ActorCell, so I don’t see a problem which emerges from this line."""
"Yeah, well i just asked out of curiosity, since the code is ez to maintain i just didnt see the reason^^"
"Oh, indeed. That's a bug, really..."""
"According to @stof [comment](https://github.com/symfony/symfony-docs/pull/1686#discussion_r1543373) no exception should be thrown or the exception should be caught in the 'PreAuthenticatedListener'."""
"Yep! :smile:"""
"It invokes `git-tracker help`, which just spills a message to standard out. The test then assert that the expected help text was written to standard out.  We used to have sub-command, `test-command`, which did basically the same thing, and only existed """
"Yes, not match) i  found only 69412 in 3.3.2 useful spell . With this commit this spell should work correct.Like many other in 3.3.3 with old EFFECT_CREATE_RANDOM_ITEM that moved to EFFECT_CREATE_ITEM_2   Thx. )) BTW, it is hard to stop spam in log (you c"""
"Good point about the copying - However, I find it rather worrying that copying a world and loading both the copy and original is now a valid way to crash your server..."""
"Good thing he has Git Extensions. So easy to remove ;-)"""
"Yay finally, thank you =D"""
"oh crap.  I totally missed that.  The tests passed though...  strange."""
"Awesome"""
"Very deep ;-)"""
"^^ well done."""
"ugh again... credit whores"""
"Maybe a note would be suffiecient, which tells the user to checkout the branch in the latest stable version?"""
"Why you think using ImmediateEventExecutor is wrong ?"""
"it was copy-pasted from the old code :\ thanks"""
"Alright, let me know when things are ready for review, I've got an insane amount of code to review"""
"@UltraFaceguy  >The commit removes some outdated and inactive, but trusted, plugins that are integral to my server.   You just made a fool out of yourself and this proves why this commit is needed.  How can you 'trust' something that bypasses the API"""
"it's documented on ActorPath, and it's used for cheaply injecting the transport address into its string rep (for local Addresses)"""
"Have to say I really appreciate this as one time I needed jQuery as a string and manually escaping all these was a pain.  Don't ask why I needed jQuery as a string..  :D"""
"It's their responsibility to manage uploads."""
"yes. all OutDoorPvP may be easely rewrited with this code - not in core, in SD2. but need work :)"""
"I feel like with there being a link to the official OAuth site *and* a link to the `requests_oauthlib` repo, the change makes sense"""
"Have you did a pull request? This is blocking me to use rails 3.0.8. :("""
"A new age of of The Internet has started.... sad times! :("""
"Was just working on that. Thanks :)"""
"Doh!!! Sorry guys - first push here - forgot to rebase."""
"... to clarify: the only instance where """"holdSurface"""" is called in DXVA.cpp is by CDecoder. Now for software decoded we don't pass through this channel so in that case we will are using """"unheld"""" surfaces potentially being released while you still need the"""
"I see. I tested using rand(8, 4) and with rand(4, 8) the first only show values between 6-8 and the second show values between 4-8  What I want to say? I want to say that the order matters so, IMHO, you have three options:  * throw an exception when the v"""
"hmm... then i'm really confused... because since this commit i get """"not declared in this scope"""" errors at compile for these ones... :-(  EDIT: i'll make a clean compile and let me surprise...  EDIT2: but it's defined as -> class AuraScript : public _Sp"""
"This is a bug in another library, not ours. ZLib does not touch the data pointed by `next_in` (it only touches the pointer itself, to move it forward), but even with that the signature of the pointer is not constant.  There is nothing we can do about this"""
"I am not overly excited about this, but the syntax differs too much to apply this directly in the abstract adapter. :-("""
"yup.  i thought of that too but wanted to avoid introducing a new option. but i agree it's cleaner and less hacky.  I'll adjust the PR.  Thanks."""
"I truly hate this function. """
"after checking the byte-code for Ryan’s “Catcher” variant I think that is the best option right now"""
"You forget about one slash ;-)"""
"Should be `if (`and `) {` as per the [coding conventions](http://book.cakephp.org/2.0/en/contributing/cakephp-coding-conventions.html)"""
"You can use the meta, but it's just much better to send it as an http header."""
"Why this change? (I don't see any mention in commit message about this)."""
"Extract the above into a separate method, and then call and return from that method: ```php protected static function createAdapterFromItems($items) {     // do the work from inside the conditional, and then:     $adapter = static::getAdapterFromMana"""
"epic :D:D:D keep going :) good job"""
"I came across this issue, and just applied a base64 sledgehammer to the output (which is what the Rails patch mentioned in the Trac issue does also).  I did consider the Vigenère-cipher-style approach, but at that point I had already done most of the wo"""
"adding a new method in an existing interface is, unfortunately, also a BC break :("""
"Doesn't it close #2729 ?"""
"In fact suggested unrelated patch also rejected, becasy we need in general fix how tartet mode pairs wotj with SELF2 target mode instead adding hacks."""
"Will probably revert this, has some bad side effects"""
"nice, i thought was my fault and was going crazy"""
"Hahaha, conflict resolution sucking, fill fix :)."""
"Looks like the book got pulled, link doesn't work for me. :("""
"finally some changes to that awful functions"""
"I see. I'v only looked at the diff, sorry."""
"Who knows :) Will change."""
"That'd be great - thanks!"""
"That's what's so weird: I have the timezone line in my config file, and it still throws the error. If I switch to my staging branch, I don't get it, so I assumed it was this change, but maybe not?"""
"Hahaha, totally agreed I believe one is morally allowed to refactor touch typing while looking somewhere else :smile:."""
"I was hoping to see this get added since I read the forum topic. Good to see it made the cut. :)"""
"There's [240 more](http://bugs.jquery.com/query?status=assigned&status=closed&status=new&status=open&status=pending&status=reopened&type=feature&resolution=notabug&resolution=patchwelcome&resolution=plugin&resolution=wontfix&resolution=duplicate&col=id&co"""
"I'm still hoping we can think up a way to avoid this happening, but I'm at a complete loss right now..."""
"i keep getting segmentation fault o.0 wtf is causing that??"""
"But FYI this is set up for trouble and you should probably revisit it. Taking the memberType of the symbol owner means it will be wrong whenever the symbol is inherited. When the type signature is """"Symbol => TypeTree"""", the right thing cannot be done."""
"It was designed to work for all symlinking, not just homebrew. This way Basho can use it in their build process for other products.   On Jul 13, 2013, at 9:50 AM, Adam Vandenberg <notifications@github.com> wrote:  > In Library/Formula/riak.rb: >  > """
"I think it would make """"more sense"""" to let 08 extend from 07. At least it sounds more logic, even if it will not matter :)"""
"I wanted 'second' when I'm iterating hashes.  But 'third' to 'tenth' sugar doesn't seem to taste that sweet. :)"""
"Classic JGit.  Oh well. Thanks for taking the time to look into this. I've reverted the assert to the old one. :)"""
"nope. float is fine. float is also signed and the melee damage one works right"""
"Well, since you have such a cute and proud dog as your best friend, I trust that you've done your numbers right. People with animals tend to have the necessary empathy and holistic thinking needed to properly assess complexity and eliminating bias.  Cons"""
"I think this is a non-issue, I'm more scared about `nodeFor`."""
"Why signed if you don't allow values < 0 anyway?"""
"that's beautiful, finally it will be impossible to fork-bomb yourself."""
"Yeah currently the end developer has to manage cookies between requests on the same client instance.  The more I think about it, the more that feels like a bad idea.  Once I'm done with the authentication strategies I'll implement a per instance cookie jar"""
"I <3 you Kirk. Now I just have to let Machi down easy. Seriously dude, some really amazing work you have been pumping out."""
"that would be my idea as well yes."""
"Maybe each of these could be split into separate descriptive methods and put together to achieve the sanitized URL? String sanitation is a notoriously ugly process and maybe breaking things out into separate private methods could aid in elucidating the pr"""
"`hashMapping` it will be"""
"shouldn't these lines be:  raise """"You must set config.cookie_secret in your app's config"""" if config.cookie_secret.blank? request.cookie_jar(:signing_secret => config.cookie_secret)  since config.secret was changed to  config.cookie_secret"""
"I think that it is perfectly valid to only implement a decoder. This would be """"closing the door"""" where we don't have to. It's more flexible to have only one interface, and Lukas' use case in the rest bundle can still be supported.  On Mon, May 9, 2011 at """
"This is not correct. Because ```javascript var a = """" ' """" + ' """" ' ; ``` changes to ```javascript var a="""" ' """" + '"""" '; ```"""
"The OS X executable is called Bitcoin-Qt"""
"Yes. Thinking about it again the best approach might be to remove the `Send` and, in the `encrypt` method, copy only the next `maxBufferSize` bytes from the write ByteString into a ByteBuffer from the pool and have the SSLEngine `wrap` these bytes into an"""
"uh, cause I cut and paste like an idiot. will fix..."""
"If I didn't miss anything, then the `getInstance` pattern should hold for all singletons, including Kill, GetRoutees, and some others I'm currently forgetting. Differences between docs and code will then have to be settled by changing the code ;-)"""
"Cute name :)"""
"Why does it seem that there is no feasible reason to continue this rant about something that makes a developer responsible for their actions? Christ I'm sick of the email spam. Guess what. This commit is good and effects 75% of the plugins I've wrote. It m"""
"Nit: pass hash by reference."""
"That's what I told to myself, but it is much faster and easy to read this way. I don't like this kind of naming convention either but I found it better than injecting 4 services or add a dedicated section in the loader to get tagged services. I might be w"""
"So what if you didnt put it there? Be the hero Corey!!!! :D"""
"I don't think we need this comment."""
"Thank you Ambal for the patch. Go ahead for the next goal!! ¡¡Prepared statements!!  Cheers Axxus AKA Undergarun ;-)"""
"CraftPlayer implements Player extends OfflinePlayer.  OfflinePlayer is poorly named, because of compatibility issues; it should be, Player is OnlinePlayer and OfflinePlayer is Player. OfflinePlayer isn't restricted to just offline people, it's a handle to """
"@fcurella you mean `request.set_cookie(key, '""""param1:param2""""')` right? If so that makes sense. But yes, adding a test and ensuring this doesn't break anything else would be awesome."""
"What if the collections' size is Integer.MAX_VALUE?"""
"no its just wrong"""
"Indeed, I didn't really seek out the """"second order simplifications"""" which this must have enabled in pattern matches."""
"I'm not a big fan of the injection of the input here as it means that the helper becomes stateful. I know that because of BC, we don't have many choices, but another possibility would be to just inject the value of the interaction flag. It would still be """
"Field with initial underscore _paginator"""
"Yay! I always thought this was strange."""
"awesome documentation in here!"""
"If you're going to implement these, you should likely also implement the EventManagerAwareInterface, which would make it possible to get this auto-injected via the service manager."""
"No, I am going to write the creator about this though, and will update the formula if they agree."""
"Maybe Javascript is better than C about this, but you're both re-assigning n and using its value in the same expression.  That seems like it relies on undefined behavior in the Javascript implementation."""
"You have to because it defaults to `element.last` so if it goes to `extract_values_from_collection` it will go to the last value which is incorrect. Like I said, the current default value for for `value_method` when you don't pass a value is `:last` Pleas"""
"Fixed in the commit https://github.com/derekmahar/akka/commit/0e0c55010af9476e9b2981e4b3c2030641b03f10."""
"typo"""
"Also... seems like a weird thing to nitpick."""
"WOOT this bug was annoying!"""
"read the commit message ;P"""
"`it is...`"""
"My patch tested and working 3 month more). But we must fix it in blizzlike way) BTW animation is well. And it seems this item created by blizz for transmogrif"""
"No problemo :)"""
"The problem is that we haven't specified a public API but the documentation and examples show it is ok to inherit from the sanitizer and override a method like `process_removed_attributes`. Considering that many people are writing plugins due to the mass a"""
"The `$GLOBALS['_SERVER']['_']` is not set all the time. It will give a notice of undefined index '_', causing some tests to fail or not show the real error from the test case.  The same issue happen in the `GlobalState.php`. """
"Bug Reported !!! and you fuckers say i don't do anything around here ... >.>  -Jason"""
"Wouldn't that be out of the scope of this pull, given the two other existing places in Cookie.php that already invoke GMT rather than the preferred UTC?"""
"@nickdesaulniers It prevents nss from being automatically symlinked into /usr/local/bin, yes. You can find its sandboxed prefix by doing `brew --prefix nss`, or you can use `brew link nss` to link it manually."""
"We had a solution (ffmpeg event) but unfortunately this part was not accepted upstream. Michael said that it would break some other functionality. Comparing new and old state is the only idea I have so far."""
"No, it is not. It's low level, but not internal nor expected to disappear or change anytime soon."""
"I don't like the default value: if the parameter is optional, you should use null. And if the value is null, you set a default value. I don't think that we need to define a default priority (we don't have a default priority in our priority filter)."""
"Cool, thx =)"""
"```diff -m_afFirstNagaCoord +FirstNagaCoord ```  Should be done everywhere. :("""
"Might want to include some more debug info about which actorref didn't manage to activate within how long."""
"That not have any sense. You don't create a test with accidental wrong values. Check this because seems that you have a lack in the test design."""
"Another possible solution, but (at least to me as non-rubyist) just as cryptic if not worse. What's wrong with the `if`-block?"""
"Strange indentation here"""
"Does not make the code any more readable IMHO."""
"Yep, see earlier comment about keeping things the same for this PR.  My hope is to add shader-based GL as a new renderer. This way we could have gl2/gl3/gl4 selected at runtime based on hw capabilities. Iirc @jmarshallnz has a GL shader implementation a"""
":O EVENT_TYPE 73....? Not in the wiki..? What is this sorcery!"""
"Agreed. We need to recheck all generated test to be free of this issue.  This will be the first success story of `AR::B.mass_assingment_sanitizer=` feature. :)  I ll start working on this."""
"\o/ Thanks!"""
"This was added to fix the error on http digest - it tried to reauthenticate when questionmark was added to the query and also it was trying to reauthenticate on params change. Can you provide an example of failing request?"""
"I am very sad to see this change: it totally reverses the idea of self-contained mini-require for natives and again includes it into the module system itself. That was the whole point of it existing. requireNative and internalModuleCache should have staye"""
"h5bp/ant-build-script@be424f7 solved my only complaint with this commit, so thanks! I love you all <3"""
"In truth, I don't really think we need this any more, since we figured out how to get gyp to generate xcode projects to build with a .node extension. I'm happy to generate a PR that adds the condition you've suggested, or backs out the change."""
"No real reason to introduce _one time_ variable."""
"problem in duel in elvin forest - its not sanctuary zone"""
"And about one `@param`..."""
"@envygeeks It would seem that you know more about regular expressions than I do. I wasn't aware of that, thank you. I'll go ahead and revert this change. If you'd like to audit any other regular expressions in Octopress, I'd appreciate your input."""
"Because the address we depend on is loaded on demand. We are guaranteed to call this lazy val only when the address is initialized."""
"What is wrong with strncpy? snprintf is possibly vulnerable to format-string attacks."""
"Should it have a blank line above?"""
"Its git hash. We dont use rev ids. """
"Does it work the same way for numeric keys ?"""
"This is not necessary - it's a developer-supplied value, not user input. Also, the next two lines do enough to sanitize the value."""
"@rsa In this case do shut up and do not bother us from your worm den. Thanks for the url anyway."""
"Indeed the fieldList is a whitelist. I too think it should be a protected method."""
"Interesting. In which case, it's glib's configure that sucks."""
"Whichever way you slice it, it would be good to see unit tests for equals/hashCode consistency of this class, in addition to the functional tests like `blame-aye-triple-eee.scala`."""
"Maybe we should add the dependency as you said and assume an installed version from bundler as a backport. I can't see a better solution.  @rsampaio, ideas?"""
"Shauren, ALARM!!! There is bug again =( Try enter vehicle directly from another vehicle... """
"Well, now users do have a good reason to cheat.  I would leave an """"official"""" hole with `debug.setmetatable` for hackers, but I believe that others should be plugged in now.  1. Add `mt.__metatable = """"redis.strict`"""" to plug `setmetatable()`. 2. Use `ge"""
"That would violate the **DRY** convention. That is how the encoder retrieves options at the moment, but only for a single value. I don't think it makes sense to repeat the same block three times if they are all going to do the same (getters/setters)."""
"no ive checkd it. you should be able to use the parachute without falling and once you start falling it opens up."""
"`/>` is unnecessary with this doctype ;)"""
"It is, but it also returns a result set. See the `optimize_table()` and `optimize_database()` logic."""
"Yeah I was kind of irritated about the m4 directory.  I forgot about set."""
"Here's the fix to an issue I alerted the Plupload folks about.  Yay improved security! Lack of alerts from moxiecode to their customers regarding this issue...not so awesome... :/ Lack of credit to me for finding this (in the commit message, a tweet fro"""
"QAston You do not feel sorry for people who use the repository?"""
"Actually, better yet, ditch the arg-sniffing in C++ and create `binding.writeBuffer()` and `binding.writeString()`."""
"Nevermind, totally misread the change. Sorry. </github_spam>"""
"Thanks for the advice.  Changed it to an options hash."""
"Thanks!  I think this is all good stuff.  We should now just make it half as long :)"""
"As the attribute should be clean on the sender side, there should not need any trim(). But in case of doubt, we could put the trim again (in the decodeAttributeMultipart then). WDYT ?"""
"Looks like a typo was introduced here.  Probably not a big deal, though."""
"Mm, I'm not sure. I don't see why we need to link to the official OAuth site."""
"I don't understand this change. According to Mathias' post, this new solution still has most of the cons of the original.  The CSS hacks are much less readable, obvious, meaningful, etc. They're harder to notice and more complicated to remember and use. """
"`$str` here"""
"This masks the original exception backtrace and outputs the true error on stderr instead of in the exception message. Instead, this should rescue the gem activation error and raise a `BcryptMissing` error with this message.  Better yeah, get rid of the r"""
"what happens if negative or 0?"""
"Ahh, I don't know how I feel about this... Paul Irish would not agree. =P lol I joke of course. =)"""
"I find this weird. Why do we only recur on mode == Poll?"""
"You called it herobrine! WIN!"""
"uploadNewCredentials is not an operation that is currently supported by nimbus.  We can put it back in as part of a future pull request."""
"As you see we frequently forget to update these references, master should be fine for now. But it might be a good idea if we start moving functionality."""
"I agree that it should be immutable! If you can conjure up something in the next hours then it might make it into 0.16 =)"""
"YAY!"""
"the concept of having it done but looking like total abomination doesn't exist here.. """
"I think the current """"sets"""" directory node is now also showing 1-movie sets, which is pretty annoying.  It can be solved by passing true to GetSetsNav in the CDirectoryNodeSets::GetContent"""
"I asked about this before, got answered """"omitted in purpose to avoid bad implicit casts"""""""
"Sounds like something we should avoid, creating brittle code to satisfy testability seems wrong."""
"nope: this is not an Ordering, try out `Int.MinValue compareTo Int.MaxValue` to see what I mean"""
"Add in the deprecated reference, so that I can test & merge this ;) Nice work as usual :)"""
"I'd need to jarjar them in any case.  If no extra dependencies are needed can this stay here or do you think extra module is right in any case?  I'll get rid of them. From Guava I am using LoadingCache but that can be done with locks and HashMap (same as """
"Awesome commit-hash: ``dead730`` xD"""
"rsa, passing NaN values as input parameters will definitely cause issues with any geometry functions. Just because they expect them to be valid. RandomMovementGenerator<Creature>::_setRandomLocation() is the last function in call stack which triggered call"""
"Weird formatting here!"""
"Just pushed update with a comment detailing that section =)"""
"I think MaNGOS startup-time is O(1), so the progress bars are really a must have :)"""
"arg :( why's that?"""
"Why not, what's the problem supporting """"/""""?"""
"EFFECT_IMPLICIT_TARGET_CASTER is not used anywhere? Why not use TARGET_OBJECT_TYPE_PET to replace EFFECT_IMPLICIT_TARGET_PET and make """"implicit target type"""" a bool?"""
"Please try https://github.com/TrinityCore/TrinityCore/commit/28d1cc9cfe8ba1b7f30f0d918b40eaf6d54170b3  @QAston, please pop on IRC when you get some time so we can hash out the details."""
"good!"""
"I got a bug after this update - whenever the basket don't get used it never despawn till server restart, it should dissapear on umbrella dissapearing. Can anyone confirm this bug or it is my error only? Like you get ~ baskets all over dalaran or other plac"""
"Found a site on which all the fields have been identified and commented... http://www.madx.dk/wowdev/wiki/index.php?title=CreatureDisplayInfoExtra.dbc not sure how much of it is trustfull, but it might help identifying more of the fields :)"""
"Didn't you mean `:as => :minus_path`? Actually, you don't need the `:as` option at all for that test."""
"NIce"""
"Not really.  They're now needed when building from the tarball too :("""
"crap, I'll rework it, I don't want a dangling else just waiting to f**k us over if someone does not watch what they are doing."""
"yeah ok i used only the latest source ce7cc421f505f72ba477 ( not your changed ) i wanna only inform u, and the latest rev same error ( fc5e92413851835879f0 )"""
"It was originally that way, not sure why. After refactoring per your suggestion below, not sure that does anything, so removing."""
"as a side comment... dwarfforge broke several times due to bukkit changes... not because of nms...  so let's see what is discussed here  a) A commit which will require plugin developers to recompile and change their plugin when mc version change every """
"@carlosantoniodasilva this is correct.  In master it's a method call, in 3-2-stable it's a hash."""
"Doesn't matter at all. Just remove that line."""
"Hate the double colons.  Bring back the old style!"""
"I think we can just remove it..   Am 01.10.2012 um 10:42 schrieb Jestan Nirojan <notifications@github.com>:  > In codec/src/main/java/io/netty/handler/codec/sctp/SctpMessageToMessageEncoder.java: >  > > + > > +package io.netty.handler.codec.sctp;"""
"Its safe, at least from a unit testing perspective! I also don't see why it wouldn't be safe. An I missing something?"""
"So yes, ```libssh2_userauth_publickey_fromfile``` wraps ```libssh2_userauth_publickey``` but if you have already read and parsed the key, no need to go back to the disk and parse it again."""
"I dont have a problem recompiling my plugins. I do every few days anyways with some bug fix or new feature. Whats bothering me the most is everyone else is being forced to do this as well now and its going to slow down the DBO queue badly. I wouldnt care, """
"Oh, for future reference, this was introduced in #7155."""
"Same problem here."""
"OK, that's it - no committing before bedtime for me, I tend to break more than I fix. :/"""
"No, it is not normal to see req objects leak in this case.  The agent.requests collection is merely requests that are pending or in progress.  The role of the agent is to keep a pool of **sockets**, which are re-used between different requests.  The reques"""
"Thank you sooooo much ;)"""
"Agh.  That was a dumb mistake.  Thanks for fixing that up.  I'll go back through and audit my other uninitialized member changes to make sure I didn't make another similar mistake."""
"Holy crap."""
"It was intended that the VML element be present for all tests. If it causes failures, that indicates additional places where jQuery needs to apply workarounds."""
"sorry i was wrong, you are right on this"""
"Oof. Got scared."""
"Sure, but adds 1 call to the stack only the first time for any property. It's actually a win on repeated calls because it's only one property access instead of 2, but I hear you. Any suggestions on how I might detect that a browser has no vendor prefixes """
"I see, instead of having a seperate closed flag just clear the timeout here in addition to in stop.  Lastly do it inside the if(this === connection.socket) conditional"""
"looks weird """
"I think there are enough tests about forms using the formats machine to sanitize input values. I'd rather specifically tests `sanitize_separators` in a separate `test_sanitize_separators` test."""
"This function still feels like a big hack to me. I don't think you should create another `abort` method to pass the data you need. I'd recommend creating an exception class (similar to [`ForbiddenError`](https://github.com/reddit/reddit/commit/633a7104913"""
"changes like this without spells == hack. 13 spells only in DBC, by sniff his much more."""
"Probably because it's set using GET_R() et. al.  You could certainly change it to GLfloat and divide by 255.0f when it's assigned above."""
"I felt a great disturbance in the Rails, as if millions of core extensions suddenly cried out in terror and were suddenly put to better use."""
"Doesn't this need indifferent access? Wasn't that the reason for #7478?      self.session = HashWithIndifferentAccess.new"""
"If we want to give people a choice about how long the hashes are (using a setting in bitcoin.conf), then surely this function would need to be re-introduced again, wouldn't it?"""
"In case it wasn’t obvious from my article, I’m all for reducing the number of conditionals used. :) The use of the `_` and `*` hacks is a no-brainer, really. Love the `.oldie` class by the way – I was thinking of using that when I wrote the article but dec"""
"the better question would be where RichInt can actually come from? Is this not a little over-designed?"""
"Excuse me if I'm missing something, but doesn't the []= method on Hash return its argument anyway, so that prev would in fact be set to the rightmost member? However, it seems to me that `prev` is not used for anything here, and could be removed."""
"Would HeartbeatsPerKeepAlive be too wordy of a name? I'm worried that people will assume that KeepAlive represents a time span in seconds without reading the docs.  Also, is there any reason we shouldn't just make this a uint autoprop instead of doing a"""
"Agree with @bajtos. I use the double-indent style for function signatures (but not function calls) too."""
"Josh,  Not sure if this makes sense, but we've been using CookieSessionStore with a persistent session_id for a few months to be able to track features such as visitors to resource X also viewed resource Y etc.  Sole motivation being API compat with ot"""
"moron how i here you steal go to scool so do not call me kid coz i can be your dady"""
"Crap, this was meant for wip-props only :( I'll fix it."""
"I'll simply restore the patches for now, then, until I'll get some sample which they break."""
"@jeresig - Welp sucko, when they fix it then something like a weak inference + version sniff will do. As for them not fixing bugs because it will affect a js lib; that is my new pet peeve, browsers should fix bugs and let libs fix there own bugs :P"""
"I think I spoke too soon here. I can't really find an example of obfuscated overloading outside of net.minecraft. So nevermind. But maybe some commenting when changes are made would be a good idea. That way it's easier to figure out the how-and-why of a ch"""
"     Can't compile X64 under vs2010, returnng this error :   5>d:\mangos\mangos\src\game\spellmgr.cpp(1226): error C4716: 'DoSpellProcEvent::AddEntry' : must return a value"""
"Saved another 12 with that event.js whack, so minus 48."""
"@Skixbmc @Karlson2k  ```""""%s\%s""""``` should be ```""""%s\\%s""""``` right?"""
"To prevent what i described in my previous comment, all routes must now explicitly define a default value for _locale in their definition."""
"...JavaScript already has this capability..."""" """" ... it would be interesting to see cases where a version check actually saved you code...""""  I think I am (finally) sick and tired of this kinder-garten ..."""
"I were able to find vim from homebrew-alt:  >/usr/local $ b info vim >vim 7.3.462 >http://www.vim.org/ >/usr/local/Cellar/vim/HEAD (1256 files, 22M) >https://github.com/adamv/homebrew-alt/commits/master/duplicates/vim.rb  The local install is from """
"You based good looks off of my posts? BEST. DAY. EVER. <3 :)"""
"I'm usually in the """"moar comments!"""" camp. But yeah, WTF."""
"Just out of curiosity, why not define the `to_s` method directly instead of this alias?"""
"sorry to say but still crashes when making new char.  http://pastebin.com/2mx4dmCu"""
"This is absurd. 659 comments and it has still not been reverted? There is absolutely no point to this commit. It simply makes the developers and server owners lives harder.  You do not compile your plugin to be smart and choose the correct Minecraft vers"""
"You're taking the address of a stack-allocated local here, aren't you? If that works, it's purely by accident."""
"I am getting confused with what is considered 'best practice'?  In the Boilerplate template you advocate paulirish.com/2008/conditional-stylesheets-vs-css-hacks-answer-neither/   In this revised reset, normalize you are now reverting back to CSS hacks for """
"It does look weird. I assumed this is the way it should have been displayed when looking at the FAQ. I have changed it to one line now."""
"The first example was supposed to be: <pre> categories:  - foo  - bar  - xyzzy </pre>"""
"Aliasing types like this is a cardinal sin in that it makes things more obfuscated rather than less.  (""""What's this ExtType type?  What values can it have?  Oh, it's just v8::ExternalArrayType - why don't they just say so!"""")"""
"nice"""
"Ah, that makes sense (I didn't realise it was required, even though I tried it that way during testing).  I've opted for a new named feature, `requires_literal_defaults`, and then a couple of new lines of code in `column_sql` and `alter_field`, along wi"""
":heart:"""
"Ok makes sense.. I will take care!"""
"This should be changed to dynamic units via ``model->getOptionsModel()->getDisplayUnit()``, remember my recent pull :)."""
"Done, sorry but visual studio does not add a space on if/while/for etc. :)"""
"I think there is some drift between the comments and the current function signature (filename is not a current param...). I know this did not happen in this particular change, but maybe we should we clean this up?"""
"Yes, @viktorklang We can probably reuse one `HashedWheelTimer` for the whole `NettyTransport`.  So @drewhk correct me if I'm wrong. This seems to be per outbound association. Are we creating this for every connection to a new remote system?  """
" /Suicide...  I just hate RBAC ._."""
"Isn't it kind of weird that you're checking for `merge` here and then not calling it?"""
"I have the null setting when the next decode phase starts at the top. Do you mean nulling it out immediately after the out.add() is called so if something happens in between it doesnt throw?"""
"In repl, `:javap -app MyApp` needs to locate the relocated code.  (By convention, `:javap -app MyApp#` means `MyApp$delayedInit$body#apply`.)  I suppose if I ever had trouble finding the delayedEndpoint$ enmanglement, I could inspect the closure.  In fact"
"-Itest adds to the load path so you don't have to hack relative paths in every test. This isn't external loading of dependencies at all; it's a simplification."""
"if we change this signature to `string this[string requestDirectory, params string[] allowedExtensions]` then `allowedExtensions` can be passed along to the delegate as well and maintain the same functionality as before"""
"As long as we're nit-picking this check-in :) and wearing my autodoc hat, a doc string on IKVReduce and kvreduce might be nice (assuming it's supposed to be available for public implementation). To see an example of how this comes out in the autodoc, see h"""
"Yeah. I think it's good to handle it but still complain. I simplified the message tho."""
"Haha, the original comment used """"will let us use"""", which I incorrectly copied in 6b8de05d."""
"It wont delete the ones after, that is not possible in PHP. Any locked buffer will prevent access to any buffers stacked under it (there exists ugly hack to access them, but ugly I said).  The $level !== $previous check will just prevent an infinite loop."""
"finally...."""
"@peterc, letting testing concerns seep into the core object like this gravely offends my sense of aesthetics. Big part of the reason why I dislike rspec syntax so much."""
"should this be unsigned?"""
"That shit took 2 days to find. I hate it with every inch of my body."""
"Arg.. It was capitalized. Fine. It's not like I can't handle the correction of a capitalization error that *should* stay somewhat permanent after this..."""
"Do you wanna say it's right script? O_O Ungly hack at all. Shoud be something like: moveinlineofsign -> cast SPELL_SCARED_CHICKEN->use net -> root\stun ->set gossipflag ->gossiphello->cast SPELL_CREATE_QUESTITEM -> despawn. and on reset cast SPELL_INVIS_CH"""
"Please use 1.9 hash style."""
"hi spp. please help me http://img836.imageshack.us/img836/4622/99076218.png flood in console"""
"No problem. Just see this page: http://forum.xbmc.org/showthread.php?tid=141158 I see you already signed up on Transifex. I added you as a coordinator of the Dutch translation team.  Cheers, Attila """
"Not really, just a few billion times faster. :)"""
"Instead of using ``else`` here, you should return the array at the end of the ``if`` IMO"""
"use 1.9 hash syntax please"""
"Yeah, this was pretty much just getting rid of some postgres syntax errors"""
"@normanmaurer +1"""
"@Tux2 We're working hard on ensuring the concern over waiting for approval is minimized :)  As for compatibility, abstraction is your best friend."""
"Finally! Thanks for the fix!"""
"Oops .. yeah, I overlooked that one. I've been using 1.9.2 for too long :bomb:"""
"very nice!"""
"Indeed, it should not. Turn Evil has MECHANIC_TURN but not MECHANIC_FEAR. These mobs are not immune to that mechanic."""
"Could you update http://udbwiki.webhop.net/index.php/Main_Page ? I'll be greatful. :)"""
"I agree that `setMessage` is not a good solution add should be removed/deprecated because the `addViolation` does the same and is more versatile. If we do that, I wonder what influence does the return value of the `isValid` method have at all? At the mome"""
"How do you suggest that be done?"""
"It's an awful browser... pretending to be modern, just a fraud.  If this issue doesn't exist in 5.1.7, we shouldn't be special casing it."""
"This test suite looks sexy. :+1:"""
"Thanks."""
"thats going to reduce the table character_queststatus significant... good job!!"""
"If the class isn't found there should be a runtime error. The user has done something wrong in this case and should be notified – silently using the default serializer is confusing and could be dangerous (e.g., the user was expecting encryption). """
"its commit fix this issue? https://github.com/TrinityCore/TrinityCore/issues/2356"""
"I agree with drogus. Seeing something specific to IE here seems fine to me."""
"Hmmm. I don't have an environment at home to test with but I was thinking... why do I have to decode() here??? Shouldn't the object already be in unicode? I remembered thinking about the DeleteMarker and Key objects where they parse xml and pondering the """
"Excellent work. More often to such commits instead of cutting away never stopped anyone from fields in the database ..."""
"This points to something weird: if I do `dumpdata --pks 2,3,4 app` I'll get _any_ object of any type with PKs of 2, 3, or 4. That's... weird. I think a better approach would be to make `--pks` only work if you've specified a _single_ model. So `dumpdata -"""
"I see the """"destroy()"""" method not existing now. I did that quickly just based on how I saw it somewhere else. Sorry about that. :("""
"I second this. Please remove this :("""
"Why are people still complaining about this commit exactly?  Yeah I complained and hated this commit, no shit.  But think about it, hack clients have to update on every MC update anyways.  And for those who don't use MCP (me) have to update anyways!  D"""
"with movement opcode xD Thx ^^"""
"ah sorry, you use it in the bundle class. But this seems like a weird way"""
"Still crashing like hell http://pastebin.com/KkUUK0K9"""
"> yay, less yellow triangles! ;)  do you even see those? ^^ I thought warnings were disabled on XCode (and the build server [seems to agree](http://videosynthesis.net/oftesting/testruns/20130214_164427-v0.7.4/))?"""
"was thinking of doing this myself since a long time... good job."""
"That source is not reputable, as they have published numerous erroneous statements and frequently allow code examples to become out dates. Furthermore, Chris Heilman is not an authority on the subject matter, nor does he back his claims with even the sligh"""
"Well, 1 was added back already."""
"DOH! I started that test before I realized that there was already a test, (named testDeactivatedUser), that just needed to be updated. I should have deleted the testLoginWithDeactivatedAccount function. I'll fix and send through another pull request."""
"oops, I missed with my dupe pull req. ignore my previous comment (already deleted). sorry"""
"Can you also remove this change? since Sundown has been deprecated, Redcarpet will never pull code from Sundown anymore. Thanks!"""
"[ 85%] Building CXX object src/server/scripts/CMakeFiles/scripts.dir/Northrend/FrozenHalls/PitOfSaron/boss_ /opt/RC_TC_BUILD/src/server/scripts/Northrend/FrozenHalls/PitOfSaron/boss_forgemaster_garfrost.cpp:56:31: e /opt/RC_TC_BUILD/src/server/game/Misce"""
"A perfect timing. Just a second before my commit :)"""
"It's ugly hack, not solution."""
"plz don't forget FreeBSD here. or use _LINUX or TARGET_*"""
"This seems like a poor idea, and probably should not exist."""
"Pinning something seems more like bookmarking or saving a post, but that's not same as liking a post. I want to let the other person know that I appreciated it, it made me laugh/smile, or enjoyed it. Pinning does not convey that. I agree with Rekado also t"""
"omg sorry, can't read diffs anymore ... minor cerebral blackout  - red line means """"deleted"""""""
"Style issues: unnecessary newlines, spaces between operators, long lines."""
"What are `$hash` and `$header` for?"""
"Not by me :P i only diffed the fix they are from MitchesD  :)"""
"I can understand your worries, the current suggestion was the least of all evils:  Actor == The behavior self    == The identity context == The context"""
"11018"""
"@oriolgual there already is https://github.com/oscardelben/words-about-code/blob/master/2012/04/rails-internals-secure-password.md"""
"could be rewritten as `[ -d $PID_PATH ] || mkdir -p $PID_PATH`"""
"Nop,vehicles didn't behave this erratic before Machi and Shauren started poking in the code :) """
"Yes, you're right. It actually is kind of weird looking. Should I change it?"""
"all i can do is edit the acm calendar... sorry bro"""
"I still prefer RAII. Make the compiler do the work of making it leak proof. Safer *and* more convenient."""
"There are two `when`. Also, it's not clear what `its smallest key is larger than the number` means? I think it's just because your `units` hash does not contain the searched key, not actually because of `smallest` vs `larger`, not?"""
"What do I need (what tool, settings, manual, knowledge) to get it fully working like authenticator?"""
"Oh, my apologies... :( I will remember that."""
"same problem  cant use this update :(      8749 01 mangos mail loot template sql """
"I found out that setting an invalid -proxy leads to not even entering the BOOST_FOREACH loops, as vhListenSocket and vNodes are empty. That means hSocketMax simply keeps the init value of 0, which leads to 10022 spam.  I also did some research for the fir"""
"Corrected in SHA: 58a13d7"""
"EDIT: Laise is too fast, posted his response before I asked something similar about it. :P"""
"Wow, this is pure brain failure on my part."""
"Point taken, I think instead of modifying the code I'll document what you said under the scripting documentation, and show that it is possible to easily hack the scripting engine to create global variables, but that we just avoid that this is done accident"""
"Looks like some kind of funny injection testing :)"""
":) Thanks again, @renanbr."""
"Hmm, kinda unnecessary, but OK. """
"I am observing races here. After a flushBuffer CurrentVideo/audio.dts is NOPTS_VALUE -> InputState is updated -> dts and time_offset sent to players are invalid.   We have a update timeout in UpdatePlayState and here :("""
"Tx for restore working TC.  Machiavelli - if U need testers- i ready to become a tester"""
"@raggi we need to get in touch. I don't want to start hacking on some of that stuff if you've already done the hard work.  Please email me, josh at joshpeek dot com"""
"Thanks for pointers, actually /home/lechup/.virtualenvs/django-sprint/local/lib/python2.7/site-packages/django/ is a symlink to /home/lechup/github/django/django/ .  I'll try that .pth thing, but I think it is not the case.  PS: Pastebin? I though abou"""
"true enough - already on it and will be fixed in next commit! Thank you for reporting!"""
"Shouldn't this decorator live in `django.contrib.auth` somewhere instead of here?"""
"WTF on master branch ? i thought TC would never add it oO"""
"Thx for the suggestion. Thought it might be the perfect place since it's contextual, but now that you mention that I realize I'm probably polluting core members' notifications :\"""
Err:508
"Sorry for the hassle folks!"""
"Niiiice!"""
"Nice, didn't know what to say about this argument in the tutorial :)"""
"Theres no need to be rude.    And I think you just want to use an instance variable, that is how we do the """"@me"""" stuff (sorry to github user @me)"""
"The line between `md5` and `head` should remain."""
"ya +1 for clearer code comment :D"""
"This change makes me a bit nervous, and I see the possibility of very large performance issues on the database servers limiting to 1,000,000 rows.  How about a rough sane limit of 25?"""
"Whoops, something went terribly wrong here! Fix is at 7bf32c0."""
"Indeed, I just used what was there. Sometimes I have not that wide view/knowlegde you have in the area of coding :)."""
"god i fail"""
"Pretty simple script for a TBC boss I say. I wonder who did it originally..."""
"It's to be consistent so you can use the same layout for pages and for posts if you want.  - [convertible.rb](https://github.com/mojombo/jekyll/blob/master/lib/jekyll/convertible.rb#L100) - [post.rb](https://github.com/mojombo/jekyll/blob/master/lib/je"""
"(BTW, the application.js file is no longer shipping with .coffee added for the simple reason that it'll have no code. But if coffee is available, we'll be generating JS stubs for new controllers (like helpers) with the coffee extension on)."""
"@larzconwell it's dead simple to do something like that in previous rails versions, by doing for example `ActiveRecord::Base.attr_accessible nil`, the thing is that if you have a big app that does not whitelist attributes, it will all break if you turn it """
"great to see railsadmin in master :)"""
"Ohh, I didn't realise this. Sorry my bad - I will amend now."""
"If you could figure out how Hoptoad used to figure out a way to jump to the top of the middleware stack, we should do that for SSL."""
"Still need the hash syntax update?"""
"WORST COMMIT EVER. THANKS OBAMA."""
"The parameter is passed by reference and hold the value how many args have been replaced. Since it is not read after the call it is not necessary.."""
"This looks like a perfect opportunity for dependency injection. Have you considered passing the loader and recorder instances in as arguments? If you wanted to go all the way, you could even remove the dependency on `connection` and instead pass in a schem"""
"It's good to know that this in stone, it'll help people move forward one way or another. """
"Thanks @andrew-d Given the apparent rize in spammers making GitHub account and trying to sneak stuff in via Pull Requests, this might have come off as sketchy had I even noticed it ;)"""
"Why is this being removed?  Thanks"""
"Hmm, this seems odd to me.  If the directory is scanned (and I assume scanned with """"use foldernames for movie lookups"""" enabled) then the item itself should have metadata assigned, and thus the videoinfotag should have correct m_strFileNameAndPath available"""
"This is not a good approach to fixing issue #750. See pull request #757 for a better approach to fixing the issue. My script fails to do any real integrity checks. If someone uploads a malicious script (eg: A .exe with the extension .jpg) it will fall thro"""
"maybe `onTimeout()`"""
"@UltraFaceguy Don't keep arguing. I learned fast. I actually joined the chatroom, and one of the staff members **belittled** me. Shows how much respect they have for us, compared to the respect we have for them (which **was** considerable)."""
"You can't load a script using script tag injection synchronously."""
"This is a bit of an issue in the long run though - not that I have any idea what the solution is.  See this commit for instance: 24154b216682dfef7ff647a5c2f698b89ba429a6 - doesn't have anything to do with #225"""
"woot! one more exploit gone!"""
"IMO the entire path substitution is a utter hack. I've only looked at it cause amet nagged so hard. Issue @ http://trac.xbmc.org/ticket/11952"""
"It looks so much cleaner :-)"""
"@paulrosania: In fact the twitter worm did exactly what NoKarma is describing.  XSS is the stack-smashing of web attacks, if you're attacked you're done.  Nothing can save you """
"Need to update the checksum!"""
"all good, that's how we ride."""
"WOW!!!! rev 9999!! any surprise in rev 10.000? =)"""
"Thank you :-)"""
"Would adding libelf as a dependency be a cleaner way to provide this header?"""
"Trust me, I'm not a Rails committer, and I've been on the receiving side of that badgering. :-)"""
"formatting got a little weird here..."""
"i cant speel :["""
"http://checkstyle.sourceforge.net/config_modifier.html"""
"Is there some way to configure this (keybindings perhaps)? I find it super annoying that XBMC steals my native volume up/down buttons."""
"Epic commit indeed! VMAP needed a revamp and this is a huge step forward! Many thanks to all the devs involved in this one!"""
"Alright, I'll revise the code accordingly."""
"+1, Thank you"""
"oooh … yeaaah …  sorry, you’re right, of course. Damn but these Futures can be confusing."""
"it does kinda seem weird. Maybe it's a bug within the minecraft server itself that needs to be done this way to work?"""
"I am ashamed."""
"We can only compare our results to json.parse on browser that have the native method. I agree it's not optimal to test this everywhere but where we actually use it. Perhaps we dump this test's attempt to compare since we don't care about the native result."""
"it is for adding fields to a query or to the whitelist?"""
"... but if it's a headache we can just revert. It was just because of #2524."""
"Sounds good to me! Thanks."""
"There are a large number of Components that us it :) Templating, Translator, Config, Console, DepencyInjection, Routing, Security.  Use this regex ```@(return|param)\s+[\w\d]+\[\]``` to search there usage."""
"I'm also a bit sad to see this go.  But if it's too taxing on the servers, it's too taxing ..."""
"Please stop censoring my comments...."""
"Aren't all those datas stored in hash maps ? And thus, does this really fix a huge waste because of empty entries ? :/"""
"and the valid names must be passed to the `compile()` method. The goal being to not do anything special in the compiled string."""
"Readme says Django>=1.2 — which is it? :)"""
"what's the point of this dependency?"""
"How annoying. I'm hoping to move away from storing configuration in the rakefile at some point soon. I don't like the idea that people have to deal with merge conflicts on their configurations. Sorry about that."""
"If the default is callable, then the default shouldn't stay in the table. This function may be too low-level to handle this, but it hints that once we've got the specific value, we """"forget"""" that it was callable. I think that in this case, column_sql shoul"""
"I did that and ended up with massive project which I don't need and incorrect deps. I dont fancy shaving the project and fixing deps every time something changes in the project. Multi-module setup simply sucks...:("""
"shouldn't you check that it is a subclass of Request ?"""
"sorry, forget what I said, *braces* can be removed"""
"@raggi, The language does have a standard, it's terrible and specifies that the parsers implement these crazy rules.  If you understood the history of JavaScript and how it was never really thought through or designed it would make sense.  Ryan has made """
"Actually, is there anyway to detect this?  We have a bunch of widgets that still work on older jQuery versions that depend on a setter.  I'd like them to check if this is available before grabbing our plugin.  Thanks!"""
"thank's isaacs, sadly it took us several ridiculous comment ping-pongs to come to some meaningful dialogue such is the power of """"social coding"""", I suppose ..."""
"Why do you need the strSettings (one presumes you can't have that if strContent is empty?)  I presume strHash can be set on intermediate folders where strContent (or exclude) is not set, so that one seems OK."""
"Well, it's pretty safe to say there are different opinions :) I'll make a post in the forums so we at least can see if we are able to make a unified style on the SQL (SQL that goes in to the log files was my main concern)"""
"hoooooly crap."""
"Gotta love auto-updating browsers :) What’s up with this commit though? What exactly do you mean by “no longer necessary with the release of Opera 11”? Older versions never existed?"""
"enforce it configside with GM.LoginState = 1"""
"Sadly, I have to agree with @Elevim because of too many broken spells now :("""
"*hides*  Anyway, there were no sniffs available for this quest, I just found the invisiblity spells and figured this way was MUCH better than the previous one. As for @vlad852, they should ALWAYS have the gossipflag, but since they walk really fast due SPE"""
"will be useless in cata"""" Your comment is pretty useless too."""
"ah, forget it"""
"Is this needed because oauthlib is not currently up to date on pypi?"""
"The colon is not part of the version here"""
"Wow, could have probably come across a little friendlier there, Phil!"""
"Thanks a lot.  We will have to remove `ObjectDecoder(int)` otherwise. :-)"""
"Oh, sorry! I did not know it existed, but he had to feel the fail tests and help pages respectively. """
"Indentation gets a bit weird here"""
"This is not just useful for hacking scalac, but is also consistent with Java's behavior. As SLS 4.7 hints,   ``` The import clause import p._ is equivalent to import p.{_}, i.e. it makes available without qualification all members of p (this is analogo"""
"@jameskilton rogue Ajax requests cannot produce CSRF attacks, so this was just an annoyance to people hand-writing JS."""
"Right now i doubt that this pays off.  Anything you really want to know?"""
"Do you mind adding another comment before this line saying something like """"JRuby will returns the value before typecast as integer"""" or something like that? I think we might forget why this condition has to be exists at later date."""
"I would love to see this get pushed."""
"u broke tests :("""
"it worked, weird. thanks."""
"I would say take out the province of China thing since it offends so many people. Also, the """"what about Kosovo?"""" argument, well, that's a what if, the whole point of Getting Real is very anti-what-if. Deal with that if and when it happens. Worst-case scena"""
"I mean definitely not of course ;-) see [this](https://github.com/symfony/symfony/pull/3403/files#L3R147)"""
"-1   I prefer the code that was reverted, agree with @tarcieri """
"@fabpot shouldn't it save resources, in case a array is given as $files, to use it as-is instead of transforming it into an Interator and immediately afterwards transforming the Iterator back into an array?  In case you like the idea, I could prepare a s"""
"here, too"""
"Doesn't this belong to the other `WithBackendTest`?"""
"(in other words, my mind is melted down and I trust my basic arithmetic skills even less than before)"""
"so nevermind.. this commit looks right, it's the original code that looks totally fishy"""
"I **LOVE** this change."""
"bad idea :( why 2 spaces was not enough? Now code becomes wider and will be harder to read"""
"I can't imagine how operator void* can replace it, because &m_guid is never null."""
"as @ADmad and I pointed out, you need to respect the coding conventions. too many whitespaces here inside () for example."""
"wonderful :) so problem still there but no solution... will try more things tomorow x_X impossible that we are alone with this problem...  edit: @VladimirMangos """"git diff remotemangos/master"""" is also empty so far and as already told i fetched a new cl"""
"Using the system libraries is intentional so we won't include this patch."""
"this is kinda lame, still not progressive"""
"Finally"""
"Yes it is still possible, but note that both HttpPostRequestDecoder and HttpPostRequestEncoder have the HattpDataFactory as argument, which should be setup to use (or not use) this RFC conforming rule, since this is where (HttpDataFactory) this rule has t"""
"Yeah, it was mentioned in the [pull request](https://github.com/mxcl/homebrew/pull/11731). So I added a [new commit](https://github.com/docwhat/homebrew/commit/ec71fda146ec27c98a135c0cf07ded3f09417a58). I shouldn't have committed that change. :-(    """
"I kind of want to get rid of all of these... the smileys in particular look worse than ascii smileys."""
"Now this is just scary."""
"haha this is some hella-awkward misunderstanding of operator precedence, yay parens """
"Interesting that you get key violation errors during runtime on DELETE and UPDATE queries. I'll investigate this, thank you for your feedback."""
"Yup, it's commendable effort ;).   I was also tempted to start from scratch, just it's so daunting task. There are many hacks cumulated in WebGLRenderer, it just grew organically with each demo :S"""
"I believe it should be InetAddress.getByName(null)."""
"Yes, if you wish. :-) On Oct 23, 2011 10:36 AM, """"Norman Maurer"""" < reply@reply.github.com> wrote:  > @Trustin: > > What do you think about also commit this to 3.2 branch ? Its a low risk > change, so I think we should do. > > WDYT ? > > -- > Reply to this e"""
"So we gave up testing with the minified version? So sad."""
"Ah, sorry about that, noooow I get it. :(  Good call. But please add a comment about things like that so happy cowboys don't Boy Scout too much in the future."""
"@jdalton - I can't comment on that, I haven't reserched those browsers for significance specifically (but note: Firefox has much slower conversion rates)."""
"@Darkhand81  They are refusing to answer that. :("""
"@jdalton: Although they're far more likely to find them baffling and unnecessary - as the bevvy of tickets in our bug tracker indicates :) The user is expecting it to do nothing in IE - this change handles that particular case (we already do this for heigh"""
"I actually checked that, it is always an   OrderedMemoryAwareThreadPoolExecutor"""
"This wouldn't work because getting responseText for a request fetching binary data throws an exception in IE9 :P"""
"good job!"""
"@WestLangley I think the only thing that stands out is the following:  * Shouldn't Matrix4.multiplyVector3Array also be made obsolete in favor of a applyProjectionVector3Array or applyMatrix4Vector3Array?    * Same issue with Matrix3.multiplyVector3Arr"""
"yes, my bad"""
"Typo: should be """"that's only"""" """
"Wrong. There is absolutely no reason for the form data to be Doctrine Collection. The only possible phpdoc is ``mixed``"""
"Ah, sorry this diff isn't very clear. The above is for the documentation for the ``fields_for`` method. Here's what it looked like before:  ```ruby       #   <%= form_for @person do |person_form| %>       #     First name: <%= person_form.text_field :"""
"Here we could make decision based on admin status or authentication provider."""
"@umpirsky: Thanks. Colleague of mine was making a translation I just quickly went trough it. Of course I also saved file in ANSI encoding :|"""
"Oh, nice! Thank you for your ultra fast response :)"""
"Oops. Sorry about that!"""
"Cool. Looks good."""
"Why was this reverted? :("""
"I don't like that idea because the zone index handling is already awfully long. I defined a new macro `UV_PLATFORM_HAS_IP6_LINK_LOCAL_ADDRESS` and protected the helper function with `#ifdef` instead.  Perhaps we could move `uv__parse_zone_index` to plat"""
"You threw eveything in the enumerator except for these, which you commented in a fugly way. Y U DO DIS"""
"Oops... forget about this !"""
"This is making it so no one can log in. You're caching the authenticity tokens, which don't cache. :("""
"@justinbmeyer: Yes, and even corrects for whichever box-sizing you're using :-)"""
"True, I just noticed the use of ActiveSupport::OrderedHash in other parts of the lib, so I followed that. By switching to a plain hash @wangjohn 's comment would also be solved. """
"If you mean #any_instance, you were better off not knowing.  It's a nasty code smell."""
"@josevalim What was the rationale for this change? It breaks nested conversion from BSON::OrderedHash."""
"Afforess, no. This class already existed. It already did that. All I changed is an exploit allowing them to specify any name they want, despite them not actually being that account. Basically, log in as Afforess&meepmeep will let you through, even if your """
"Yes, fixed on the next commit, sorry about that."""
"No!!!! Damn you rubygems!"""
"This is a dead store to i"""
"This wouldn't really break any security configurations I don't think.  The hinclude tags just wouldn't work until you changed the access control:      { path: ^/_internal, roles: IS_AUTHENTICATED_ANONYMOUSLY, ip: 127.0.0.1 }  to:      { path: ^/_internal/"""
"Oh. I see you're exporting this. Hm. Add a `git_` prefix then. :)"""
"Fixed in [11508]. Thank you. VS accept old version, lol."""
"Save here :)"""
"The comments talking about the """"identity map"""" should be reworded. Some ORMs may not have an identity map (or have a different name for it)"""
"oh, good luck getting paid in that case. hahaha"""
"Yeah, totally useless."""
"Ugh, sorry. I guess we're using  ```ruby `which pg_config`.chomp.empty? ```  in formula. My bad."""
"Yeah. I copied this from timer_wrap :-). Will change. """
"There's still the other file =)"""
"Fix that the problem while saving characters ?  cause when you log out and then login all your things are not saved and you have to start again"""
"I could disj it at the worker.clj level, so that system executor doesn't heart beat.  The semantics of this supervisor function are still consistent  (defn matches-an-assignment? [worker-heartbeat assigned-executors] Yes you need to disj to do matching"""
"So for example RSA's scripts where they developed a Boss_Spell_Table is useless or will be not compatible?"""
"Didn't we move this method to FrameDecoder ?"""
"This test got failed with Oracle adapter. Oracle database handles equally NULL and empty string ( I know it's not an standard but it is what it is.)"""
"It does not work :(  ```bash $ time php UnitTest-e8af5d1.php | grep -E '^F' F 0/abc F abc/0 F abc/0/def ```"""
"will be replied *with* ;)"""
"Since `delegate` [uses][1] `module_eval` internally it doesn't buy us anything.  [1]: https://github.com/rails/rails/blob/3b7947ea31489afd020419e3c22ee01ce4809103/activesupport/lib/active_support/core_ext/module/delegation.rb#L115 """
"Glad I could help! :-)"""
"This line means I can no longer have a URL from an interpolation look like so, note the ? for the params. I have worked around the issue by defining a singleton method for #escape on the returned string. The subject of the commit seems to indicated this sh"""
"Warning about Coding Standards: - no tab - ident with 4 spaces - bad syntax for if look [PSR-2 5.1](https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-2-coding-style-guide.md#51-if-elseif-else)  ```php if (is_resource(/* ... */)) {     /* ."""
"Why is it  string?"""
"I guess Scmoo's talking about e. g. `.password foo bar bar` command. The password will appear in Server.log, am I right?"""
"Ooooh no :("""
"Nice. Easy to forget :-)"""
"Ah that makes sense :)"""
"nice, thanks ;)"""
"Most enter vehicle commands are CMSG_SPELLCLICK. I'm working right now on removing the vehicle hacks related to that and add proper spellclick data for vehicles. """
"Yep. I actually edited my comment. :-)"""
"Great, let's merge!"""
"This is the signature used in the constructor. That is why I used it in the setter."""
"And everything else that came in that expansion and before too."""
"This should be fixed now:  https://github.com/xbmc/xbmc/commit/d446c0a01f3460c1726c61e43c649188b981350a"""
"Breaking semver change makes iD developers sad. :("""
"Thanks.. I missed this while refactoring :("""
"I like the var. Now when people ask if we have any priors, we can say yes. :)"""
"Githubs eats my comments =("""
"This patch looks fine, but just for my edification - what can be done with the AdditionalInfo parameter?"""
"Last I spoke with @tenderlove on this subject, he suggested to make visit_AddColumn public for now. Aaron, what do you think about adding another object to schema_definitions to handle this?"""
"What the fuck. We ported this from a BSD..."""
"It seems that this commit has broken again the timers on MacOS (issue 534). They were fixed in 82e9da9fb7fb79bf730d  See https://github.com/ry/node/issues/closed#issue/534 Cheers, Jorge."""
"As this discussion is now officially over, I can finally resume my opinion without having to be neutral!  This commit was liked by staff so they would get fewer complaints, and liked by terrible developers because they never use CraftBukkit anyway. Thats"""
"@alloy Good call. You can create a patch if you want.  But this is kind of temporary/experimental direction. We want to be more friendly about namespacing your application so multiple rails apps can exist in the same space."""
"Oooops totally my fault. Yeah, couldnt find pending, ill look into xit and xdescribe once I'm on a computer."""
"@markstory yes, it seemed odd..."""
"Use `django.contrib.auth.views.redirect_to_login` to redirect to the login page (see https://github.com/divio/django-cms/blob/develop/cms/views.py#L136)"""
"Add HttpListener"""
"There's also still a test missing for blank credentials instead of `nil' and how about one with multibyte characters? (“café"""" vs “cafe”.)  Cheers"""
"(apologies for the weird formatting there, i didn't realise it was going to use Markdown...)"""
"I think this is a great idea, but it's going to screw over a LOT of people who followed bad jQuery tutorials online that all seem to love recommending $() as a shortcut to $(document)."""
"Pondered it, shifted it around at the back of my head, and I still lean towards making it explicit that a `throw` is happening here. I’ve added one sentence above, WDYT?"""
"Also can be written as:  ```ruby class UserLogin < UserBasic   expose :private_token end ```"""
"This should be probably in some trait instead. One responsibility is getting the capacity values, while another one is assigning weights. WDYT?"""
"Doesn't hurt but kind of superfluous. :-)"""
"Welcome back :)"""
"Can you write the """"hack"""" for me ? :p I need to fix this problem"""
":+1: :tada: :yellow_heart: """
"This config comment is confusing"""
"I want to call attention to this bit right here. Once auth gets to this level (i.e., we've already gotten a 401 once), this is how the overrides work. The credentials in the payload **always** win if they're present. """
"I'm shocked and appalled by the change.  Please revert the entry for Taiwan to """"Taiwan"""", which follows the standard practice of every major, important and recognized web site and software service of the world. Neither Amazon, IBM, Microsoft or Apple, to """
"this looks inverted to me after these changes after a quick look. 0 timeout means no timeout means handle eof like normal eof, and don't return false in IsEOF()"""
"Wow.... *blink* nice catch ;) I'll fix it in a pull soon."""
"they click on the link above, and pick one http://repo.typesafe.com/typesafe/snapshots/com/typesafe/akka/akka-actor_@binVersion@/"""
"Could you please reword this commit message to something like """"Check mock modhash value for testing""""?"""
"I can't remember why this is lazy. Seems like a waste. """
"Ok - I take it. Forgive my ignorance - what's wrong with starting children that way? Is this violating the rule of not sending behaviour to an actor?"""
"Thanks this fixed my issue!"""
"Fuck. I failed. Sorry, don't know what went wrong.  Have to repair. """
"bad work. rsa fully rigth. spell_dbc used loooong time, and don't have any problems."""
"But this commit message makes me sad :cry:"""
"Why does the remember-me cookie default to `httponly` but not the session cookie?"""
"one more developent, this error is related to the openssl bundled with the osx, however if CommonCrypto is being used (--with-darwinssl) tis error seams resolved.  Maybe it will be a best option to use '--with-darwinssl' as default option on osx - first"""
"Should I go and buy a can of paint ? :)"""
"my editor is the culprit!"""
"Possibly it will break. And we can easily add these things. I suspect though that such bugs were oddities in GCC that no longer apply. Superenv only does `-Os -march=native` anyway. What I should do I guess is put a warning in for these so at least we can"""
"Makes sense.  Thanks for clarifying."""
"Needed for incoming RBAC commits (Role Based Access Control)"""
"Exellent work. I think, the next step (step by step :) should be transfer Pet Action's from PetHandler.cpp to Unit.cpp (for use in CallForAllControlledUnits). Tested more than three months (based on Laise code). Need publish clean patch? ;)"""
":'(  I'm keeping my hot pink!!!!"""
"Yeah.. Will backport just been busy   Sent from my iPhone. Excuse any typos....  Am 30.07.2012 um 10:08 schrieb Frédéric Brégier<reply@reply.github.com>:  > @normanmaurer should be set also in release 3 in Cookie.java and DefaultCookie.java and Cooki"""
"@SaitoWu Also why we need shared repositories if we have umask?"""
"This totally broke uses where a song doesn't have an album assigned or has an empty album (when not using musicBrainz). Was this EVER tested with something else than musicBrainz? And who signed off on this? This MusicBrainz merge really broke too much shit"""
"What's a hashcode?"""
"This is pretty dangerous, don't you think so?"""
"problem is that /xbmc/linux/PlatformDefs.h defines 'byte unsigned cha' at 158, RPI's /usr/include/linux/i2c.h uses 'byte' as param name at line 26. reordering includes fixes compilation, as long as param is included (somehow ) before define."""
"Awesome!"""
"Not yet, but planned :)"""
"fixed. I think that tests for Rails core is bad idea. Don't you think so too? Have you read good article from DHH about it?!"""
">>This seems disastrous.  >makes me nervious, i don't want to be the guy who breaks stuff, if we go with your solution – it would definitely will not break anything, if we go with mine – it would definitely will, affected amount of code can be relativel"""
"Awesome commit !"""
"I discussed this with cptspiff on IRC before committing it and we were both of the opinion that the sets node should show all the sets in case we want to extend support for sets within XBMC to e.g. allow adding movies to a set which will not be possible if"""
"post n.40 is dated 03/09/2009 so it's for 3.x"""
"Hmmm, nevermind, sorry, too tired."""
"ha ha i get your point :) but still they wont remove it from disables it until its 100% blizzlike so i guess you will have to w8 then :D"""
"Okay, I'm totally relaxed now ;)"""
"Notch should add this in the vanilla server. It crashed our vanilla server twice because Notch forgets to check things >_>"""
"You damn atheists! There is no hope for you in the afterlife."""
"Luke had a pullreq some time ago which introduced this abstraction, yes. I think it's overkill. Also, I want to extend it to transaction hashes too."""
"Did you mean to include the iso9660:// part for all systems?  Prior to this it was only for non-Windows.  Previously:  ``` GetDiskUniqueId: Trying to retrieve ID for path D:\VIDEO_TS CMediaManager::HashDVD - Adding file content for dvd file: VIDEO_TS"""
"missing space between descr. and return"""
"Do you have a reference to that bug in clang ?"""
"Also, `data[cipher_offset + 1]` looks like it could be an out-of-bounds read."""
"whoa, weird..."""
"As mentioned in PR desc. according to the docs of Lighttpd, `https` is always first when request is encoded."""
"Now it is even more crashing with this https://github.com/TrinityCore/TrinityCore/issues/1422"""
"Note: """"CRYPTO_THREADID and associated functions were introduced in OpenSSL 1.0.0 to replace (actually, deprecate) the previous CRYPTO_set_id_callback(), CRYPTO_get_id_callback(), and CRYPTO_thread_id() functions which assumed thread IDs to always be repres"""
"@stefansaraev I knew it was something simple. Thanks."""
"Merry Christmas everyone"""
"You're welcome, glad to help :)"""
"Yeah, looks good. Reminded me that I wanted to make Duration.Inf and Duration.MinusInf compare equally to itself."""
"http://paste2.org/p/1701727 Crashing like hell"""
"Fixed"""
"What about literals such as $null, $true and $false?"""
"you are right,my bad..."""
"Confirm, this breaks compile. :["""
"Thanks!!!! =)"""
"blame CastedCreatureOrGO removal xD https://github.com/TrinityCore/TrinityCore/blob/7776016106e748fee8cb3e052ec3c8e575e84a55/sql/updates/world/2013_08_30_06_world_eai_sai.sql"""
"sorry for the multiple comment here, but this problem could also be avoided by first of all doing a clean separation between DXVA Decoder and Renderer - as proposed in isidrogar's PR 346 (see https://github.com/xbmc/xbmc/pull/346)"""
"@bigbasher I'm sorry to cause you any trouble. I'm not really sure what you mean. Was the download link incorrect? I pushed a tag of v2.1.0 which is the most recent version of the sdk."""
"whitespace change not needed"""
"ahh also I'm right :)"""
"Argh.. annoying. Suppose we must blacklist ATI here then. Can we get driver version somehow and blacklist 13.04 and older?"""
"Yep :)  Sent from my iPhone  On Jul 12, 2012, at 2:05 PM, Viktor Klang (√)<reply@reply.github.com> wrote:  >> + >> +.. includecode:: code/docs/camel/Introduction.scala#CamelActivation >> + >> +The above code shows that you can get a ``Future`` to"""
"I was planning to change this to use binascii.b2a_hex tomorrow morning (which is all .encode('hex') does anyway), expect pushing!"""
"+1 the code must remains simple, consistent, readable and maintainable. It's the only way we can avoid stupid bugs."""
"i'm crying. this is awful please, please undo :'(!!!! it was lovely to control before, now it's like trying to control a balloon with a fan some of us need to aim the camera quickly and accurately!"""
"quest one :/"""
"Nice one, I like it a lot. :)"""
"Could be simplified to `return type == v8::kExternalUnsignedByteArray;`"""
"A bonus of script injection+eval is you can capture the return value from the eval(....) code too"""
"Sorry, I tend to deal with a mixture of 2,3,4 spaces and/or tabs for indenting between JS and Perl.  Drives me crazy..."""
"No problem, thanks for bringing up this issue though. Happy coding!"""
"good to see this on master repo after so much time"""
"Well, it would be safer to use `assoc = Hash.new { |h,k| h[k] = [] }`"""
"case 9:            if (this.results_showing && !this.is_multiple) {this.result_select(evt);}             return this.mouse_on_container = false;             break; Do not forget break statement , otherwise on tab control will not be transferred to next"""
"This very well could break stuff. Its the worst code in the world :("""
"One drawback to outputting directly is you cannot send headers/cookies/sessions which makes it no different than using the command line runner."""
"Yes, it's the strategy for the router itself."""
"Its already happening. Doesn`t works! :("""
"no. when you /cower at him he roars. when you /angry at him he cowers. this is correct"""
"Does this fix this crash? http://pastebin.com/KkUUK0K9 Or i have to set a report?"""
"I've thought about this and think it may not be such a good thing security wise. What if https:// is temporarily down, but https:// works? Then the secret url to post a key would be exposed on the internet... (I know i've done this in sparkleshare-invite-o"""
"hahaha :)"""
"No worries. Thanks for getting this fix in!"""
"This would be BC break, and it's to late to have such in 2.2."""
"The default implementation for javax.security.auth.login.Configuration is dependent on system property java.security.auth.login.config. see http://docs.oracle.com/javase/6/docs/jre/api/security/jaas/spec/com/sun/security/auth/login/ConfigFile.html  Auth"""
"Haha, come on, you can declare the `git_tree` here instead of doing this."""
"SMART_ACTION_ENTER_VEHICLE should be removed and its implementations by SMART_ACTION_CAST_SPELL (sp?) and cast the appropriate vehicle riding spell. SMART_ACTION_LEAVE_VEHICLE should remove the aura on the target vehicle from the specified caster. Not sur"""
"For some reason this commit did some really weird things to the diff making it somewhat hard to read."""
"Fixed as well. *Very* nice catch!"""
"The more I think about this, the more isolated shared-nothing threads seem reasonable."""
"AFAIK {@inheritdoc} doesn't work with the API viewer. @fabpot ?"""
"My bad - misread!"""
"Very nice man!"""
"All classes used in the styles of the toolbar should start with ``sf-`` to avoid conflicting with the styles of the page. We don't control the page as this is injected into the app (the same is true for the name of the keyframes btw)"""
"Looked too expensive..."""
"What about injecting the Application and adding a `getInput` method to the application?  This means that the application is statefull of the input, but it is already quite statefull of the current command (`Application::$runningCommand`). And it means """
"Use ruby 1.9 hash syntax please"""
"Sure, `CC.strip().split()` would be fine. Change it to that and I can merge. Have you signed the CLA?"""
"@JamesNorris https://github.com/Bukkit/CraftBukkit/commit/8f12382e8efc8c39a919af9180dd884caf3720ff#commitcomment-2307508 >Staff - You need to stop copying/pasting the FAQ. We have ALL read it by now, and if we continue to debate, it means we don't agree w"""
"true, but then i would also need to be host, methods and schemes attributes instead of elements. see https://github.com/symfony/symfony/blob/master/src/Symfony/Component/Routing/Tests/Fixtures/validpattern.xml"""
"oh, you're absolutely right -- sorry for my confusion! That makes sense, it should add `word.length` to get the next match."""
"OTOH you made me thinking about if that is really a good idea to add the terminator in the encode step. It should be either a separate option, or just up to the user completely."""
"really? damn"""
"yup, could keep them together in a Tuple2 also  This has the down side that several competing threads would do the same hashing work, this is on the sender side so it's probably not usual to have many threads using the same actorRef  I think it would be p"""
"Guys, it's over. They do not want to hear your arguments. They will delete posts that come up with examples of where this commit will lead them. And they try to muzzle me because I already busted a gigantic hole through this commit with my artillery (NoVer"""
"This solves map, but can I use the same hack for all the other *maps?"""
"break the array values in multiple lines.  You can see the coding style here http://framework.zend.com/wiki/display/ZFDEV2/Coding+Standards"""
"fixed (i will install code sniffer quite soon!)"""
"I have updated this test to include a test count as it would """"pass"""" as no test would run. Should i add a separate pull request for this? Or you happy to just include it with the next push? """
"violated, mutilated, maimed, …"""
"IMHO it’s still a matter of… will people read the comments / documentation and remove what they don’t want or need? History has shown that generally, they won’t."""
"I give my congratulation. :)"""
"could we change this (and `inAny` / `inAll`) into vararg methods?      def inNone(x: Mode, xs: Mode*) = this & (xs :\ x)(_ | _) == NoMode"""
"If you only expect one name, why do you accept varargrs? `super` (`Enumerable`'s `group_by`) doesn't accept args at all (other than block), so there is currently no need for method signature to accept more than a single name.  Granted, it might be inter"""
"It is needed and is set in [constructor](https://github.com/ADmad/cakephp/blob/8ae3934378e75b2bdc7ac850f7593467fa7e7fc3/lib/Cake/Controller/Component/Auth/BasicAuthenticate.php#L55). I removed the overriding of `$settings` property so that if its keys are"""
"@vrybas: instead of explicitly skipping the filter you should do:   protect_from_forgery :except=>[:aircrafts_by_manufacturer]  I'm also reasonably sure that you could do something along the lines of  protect_from_forgery :if=>:some_method_which_retu"""
"WDYM? Are you ironic? What is wrong? Please provide understandable feedback. """
"We should always use real declarations. Using unused args as var declarations is hacky. """
"Bad commit name.. Of course this *prevents* people from gaining fishing skill when wielding fishing pole in offhand slot (titan grip)"""
"My bad, it's not """"dasherize"""".  The issue is line 223 of hash/conversions.rb - adding """"underscore"""".  Same problem, though, and a major change in functionality."""
"I ended up using `strtotime()` as the various formats that I saw in the wild and in the specs resulted in icky code. However, strtotime() seems to be able to sort everything out."""
"I've been working on overhauling the JS code organization, and the mishmash of initialization code in `reddit.js` is best considered legacy. Could you please move this initialization code to its own file/object like `r.timeago` (see `r.analytics` as an ex"""
"also libavfilter/vsrc_buffer.h header is not copied by ffmpeg install script so the file is not present with external ffmpeg. AND av_vsrc_buffer_add is not present on all ffmpeg version so you can't use it :("""
"oops.  sorry - looks like I introduced a few little artifacts."""
"Great that this was started!"""
"@Kapoeira, yeah. Vehicle Id, not creature entry  @Tassader, both spells will be in that case then ;) that's why it's there"""
"Thanks for this fix. Its definitely better than the actual way. """
"Please write the exact method signature, e.g.  ``` ruby def first_or_create(attributes = nil, options = {}, &block)  ```"""
"\o/ I'm a hacker!"""
"boss_xt002.cpp(813) : warning C4018: '>=' : signed/unsigned mismatch :)"""
"^^ https://github.com/settings/notifications go gooo :D"""
"Hehe, yeah looks good."""
"Updated AS guide to include this new methods: https://github.com/lifo/docrails/commit/7a1189a2c75b16f86a11c1d2379c50b11a7a2d66"""
"mhm, okay"""
"I suggest you add a comment as to why this works (first write goes into ClientHelloParser and has an 18 kB upper limit, followup writes go into the BIOs as usual.)"""
"@DziNeIT Ermagherd doomsday!"""
"@vmg you know I love the picking of nits.  They have been moved."""
"also tested with playerlimit=0 by 4 same time login, work fine."""
"Well if I am not mistaken that letters refers to the structure of the tables(sql). i : stand for integer s: String  f: float"""
"not work on Debian In file included from /home/mangos/sources/Trinity/src/server/game/Entities/Object/Updates/UpdateData.h:22,                  from /home/mangos/sources/Trinity/src/server/game/Entities/Object/Object.h:24,                  from /home/ma"""
"sorry - its my error. previous comment not deleted with """"There was a problem with your request, please try again"""""""
"Add it to this PR directly."""
"I am sorry. Can you provide any reference to how to cross referencing these sections from the docs in another project?"""
"We already have the possibility to script instanced map, no reason to script it more than once i assume :)"""
"I don't like micro optimizing things that is obviously not a bottleneck, but I will not argue more. You know we have a AtomicReference in Dispatcher?"""
"just verified that the bytecode generated with and without `final` is identical … (vals are always private final fields)"""
"Damn, you are right. Fixing..."""
"Nice catch, didn't know we have such a function, while working on the Qt5 patch :)."""
"PVS Studio. Also it's free (month license) for opensource projects (http://www.viva64.com/en/b/0092/)."""
"Why not just a one liner: ring.rangeImpl(Some(hash), None).headOption.getOrElse(ring.head)._2 ? """
"i just hit an issue with this code today - https://github.com/ahoward/rails_default_url_options/pull/1 - and also - https://github.com/customink/central_logger/pull/28  it's faster, but it breaks the interface completely: passing HashWithIndifferntAcess """
"That post was doing well until it invalidated itself by claiming that strict mode is """"parse time"""""""
"Just rechecked. Except this place, disabled state is replaced with """"Infinite"""" state (disabled state in other words). :)"""
"What's this?"""
"I use """"domain"""" as a query parameter throughout my application.  Do I need to modify every route, url_for, link_to, and redirect_to to use something different?  :-("""
"@daeltar yes, I've implemented the same solution sometimes: a toggler let's the user see what he types if he wants. No email or password confirmation fields. I don't like them personally, they are tedious and add fields. No complaints by users so far.  """
"This breaks every formula which uses `gfortran`. See #15103."""
"So this """"if user_signed_in"""" should be on the tag page. I'll move it."""
"It seems like you got quite a few braces-on-their-own-line around this file. Can you push them up? ^^"""
"I'm not convinced that this is actually an improvement:  1. There is a still a dependency of name in the condition and inside the conditional, except now it is 'request_id' 2. Adds an assignment in the if condition, which is arguably a risky idiom 3. I"""
"We will need to agree in that we disagree :).  For a public API, you can read """"in_seconds"""" either in the parameter name or the documentation. It's about the same effort. And you are not even going to write the parameter name! But on the other hand having"""
"also additional compile errors at linux  if g++ -DHAVE_CONFIG_H -I. -I../../../src/game -I../..  -I../../../dep/ACE_wrappers -I../../dep/ACE_wrappers  -I/usr/include/mysql -I/usr/kerberos/include    -I../../src/shared -I../../../src/game -I../../../src/g"""
"Oh and all those animated gifs in my timeline now, DRIVE ME COMPLETELY CRAZY. I really hate them…"""
"I can read this, but I'm sure others will find it difficult. Would you mind expanding this hash syntax out?"""
"My Hands :D"""
"Yay!"""
"Shouldn't this always be `""""1.0""""`?  The rest of the `tornado.auth` code is pretty much hardwired to handle either version 1.0 or 1.0a, and if we always need to send 1.0 in the 1.0a case, all cases are covered.  This also makes the `_oauth_version_to_send()"""
"Maybe this should be available as hook for others template engines :)"""
"There's a way to rethrow an exception so that you don't lose the stack. There's http://msdn.microsoft.com/en-us/library/system.runtime.exceptionservices.exceptiondispatchinfo.aspx. I'd recommend using that so we don't just lose the stack by rethrowing the"""
"@Tie-fighter That's the thing: he should have reported, not exploited. What if everyone at GitHub had had a day off today? GitHub would be open to be exploited by anyone. He would've been able to get his point across just as well, were he to post a blogpos"""
"Pretty much hackfix, rather specify knockback immunity in spellauraeffects.cpp"""
"This is busted busted busted.   With this code I can precompute likely bitcoin wallet passwords and apply it over and over again against every wallet I steal. I can even make a nice wallet rainbow table.   You've misunderstood how salt works and what its f"""
"Cool I've made something similar http://www.emoticode.net/c/inject-any-command-inside-an-elf32-gnulinux-executable-file.html"""
"`brew audit pydoop` should mention this is not needed"""
"Whoops.. That ain't good though. We can't really drop support for lucid yet.  We could have used sha routines in ffmpeg instead. Might be more complicated I suppose. On Mar 27, 2012 12:40 PM, """"Arne Morten Kvarving"""" < reply@reply.github.com> wrote:  > for s"""
"sweet hopefully no more gcd exploits"""
"Much better! :thumbsup:"""
"Oh yes, I just realized there was more potential for shortening! :-D ```ruby def gh_token   gh_token ||= (pn = Pathname.new('~/.brew').expand_path).file? ? MultiJson.decode(pn.read)['token'] : '' end ``` Beautiful!"""
"your don't understand me :( see https://github.com/mangosR2/mangos/commit/7d2fb98deda53695056c5861105c9acfb4c68c50"""
"I have the same error as starpeak :("""
"btw, activeae was a godsend. the push model = no more low-latency hacks :)"""
"My mistake; I failed to copy the trailing slash and it redirected somewhere weird."""
"Mike: I'm willing to move openldap to duplicates."""
"Can just use the above hash map until it becomes an issue with an increasing amount of migrators I guess."""
"Horrible wording :thumbsdown:"""
"haha :D happy new year ;)"""
"this part was already in script file, only reordered, so diff looks very colored :D"""
"saving the save is sexeh.  gj!"""
"Oh man, thanks for this fix! <3"""
"I agree in principal, but the implementation is honestly a bit ugly and confusing, which is not your fault, just a byproduct of what you're trying to accomplish (https://github.com/symfony/symfony/pull/99/files#L0R52).  I'm also not sure I agree that the """
"Sure,   Can you share some additional information with me:  * a failure message * what are your thoughts on the reason * how did you detect this issue?   Thanks for detecting this. Can't even imagine how hard it could be to detect."""
"What happened to most of the images? This needs to be immortalized :("""
"@pixeltrix nah, I'll just revert it.  I suppose we should figure out a test for this. :("""
"@irae: I see that your posts aren't coming through. You should file a Github support ticket, sometimes people get put in the spam filter accidentally and need to be freed."""
"No toe-stepping bobo1on1 - it's open-source and team-work :)   Until the DSP's and other enhancements get in, do you mind if I tweak a few things? The volume amplification can happily stay in the stream, but I do think the limiter should run on the fina"""
"I agree. It should never get called given the rest of the code so there's not much point in it being there."""
"Thanks, true. I added tests for all 3 Java base classes now."""
"Take this out of the loop to cut out 16383 unnecessary field dereferences"""
"you can just commit and push your stuff... the pull-request is updated automatically. Just let us know once we should review again. And thanks again!"""
"Well in my case it still works, perhaps a specific things of ATI that would remember the normally available screens.  Got numbers of other problems with all video playing at 2 or 3x speed but this is another story."""
"and 10 001 will revert this commit back :D"""
"Yes! I'll do that.  On Fri, Sep 21, 2012 at 12:57 PM, Kevin Sawicki <notifications@github.com>wrote:  > In app/src/main/java/com/github/mobile/AuthorizationClient.java: > > > @@ -0,0 +1,68 @@ > > +/* > > This class can be removed now right? > >"""
"Oh, I didn't know."""
"Too late now but you didn't meant to return `nil` here. Perhaps it should have been `""""""""`?"""
"My research is based on sniffs and, sometimes, on common sense. Anyway, if the Hover Disk shouldn't work like that it can be corrected later, when Malygos fight is implemented.  About the 2nd issue, it will be corrected soon."""
"please remove before committing"""
"Having a destructive get() seems like a bad idea.  Why not just call set() afterwards to clear the block.  I'd like to avoid duplicated functionality."""
"I have never edited the user guide so far. Maybe if one could familiarize me. """
"A vehicle commit ?! That's nice ^^"""
"I hereby associate myself with this epic commit thread."""
"Am 10.01.2013 01:10, schrieb Lars Op den Kamp: > In xbmc/pvr/PVRDatabase.cpp: > >> @@ -609,8 +609,8 @@ bool CPVRDatabase::RemoveStaleChannelsFromGroup(const CPVRChannelGroup &group) >>    if (!group.IsInternalGroup()) >>    { >>      /* First remove chann"""
"This is the question to whom? If to me - i have no hostility towards anyone :)"""
"100% bug, shocked it wasn't caught before"""
"@heldopslippers Very little of what's in Rails is """"that hard to create"""". That's not the metric by which we gauge progress. Rails is progress by a thousand improvements."""
"Good call - I was just hunting down that issue! Landed."""
"It is interesting this reference does not include version , culture and public key token. Any reason?"""
"Indeed, great work as always, nice to see such optimizations in mangos!"""
"Nice :))"""
"I had it return true, not the same as having it actually work :angry: but then as you said if we write a compiler then this is a micro optimization that is not really worth spending much time on"""
"unique... :)"""
"FWIW, I requested the ability to test for strict mode compatibility without actually using strict mode: https://github.com/jshint/jshint/issues/504  It was apparently closed 2 months ago with no explanation :-("""
"Yeah, that's right. My bad. Commit 788a7e4 corrects this issue."""
"Could you add a <remark> tag that explains why this is here or we'll just end up forgetting why and removing it 6 months from now =)"""
"This is certainly more consistent. Consistency is good."""
"Preferably use `-fPIC` instead of `-mimpure-text`. The latter is a linker hack that makes position-dependent code work in a shared object."""
"Thanks Amaranth <3."""
"If you add `TODO:` in the comment, many code editors will show it in an overview. Might be useful, and more clear to readers that it isn't just commented out code (otherwise, someone that doesn't know might remove it)."""
"Nice. I didn't know this package existed. I like keeping logic out of my templates. :)"""
"OK, but I just discovered the built binaries this way behave differently. :( Seems like further investigation is needed."""
":+1: to this. More understandable that way too. The current line just makes my head hurt."""
"Yeah I got this fixed on my local copy as well, weird the previous commit would send changes without trying to build first."""
"i'll stop repeating all of these, search/replace:  `s/foreach(/foreach (/g`  `s/if(/if (/g`"""
"Why not python-requests.org? It's the same thing just with a registered domain."""
"I would name it Logger instead. Or at least I would try to avoid exposing the name trait on it as it is different from interfaces in that they act very much like a class"""
"@LihO: fixed many spells (mostly with trigger spell effects), remove of many ugly hacks., effect handling is now logical and consistent."""
"Fuuuu ! copy paste fail :'("""
"array_merge is not recursive - it will not even descend and process the problematic numeric array - that's why it 'passes' but the original purpose of the Hash::merge -- that is not fully clear to me anyway -- is probably not fulfilled by array_merge..."""
"Oh! I think it will be my favorite commit :)"""
"one line is clearer here"""
"Can definitely recommend this powerful method: http://docs.oracle.com/javase/1.4.2/docs/api/java/util/Collection.html#isEmpty()"""
"Oh, crap ... thanks for noting that.  Edit: https://github.com/EllisLab/CodeIgniter/commit/a8262ba2fe0e11302b8d81e1afba71d4f96cd6d7"""
"There, I fixed the two cleanup-issues :) Sorry about the sloppyness!"""
"It is required to return HashWithIndifferentAccess instead of just plain regular Hash??? I mean what value does it bring in to the code???"""
"not going to open the browser killer PR again, so email answer. will  check :)  Op 10/15/12 23:28, alanwww1 (Attila) schreef: > > @opdenkamp <https://github.com/opdenkamp> > I can't seem to find where we use this string (#19161) > Cannot this be deleted ? """
" > What is the purpose of this object?  Contains message classes, constants and a handful of functions that are   used at several places."""
"Im glad someone finally did something about this instance. Many thanks to Trista and fellas!"""
"I would DRY this up a bit and calculate the width before calling `ImageRow.new` within the if statement."""
"Thanks for updating these!"""
"On this note.. My thoughts here has always been to separate m_decode and m_render. dvdplayer decodes, calls flippage, it signals app thread new frame, but doesn't make the new frame available for render yet. FlipPage blocks until a new decode buffer i ava"""
"You need to sanitize this in order to avoid path injections like these:      /%2Fetc%2Fshadow     /../example"""
"Need to update the checksum as well!"""
"I would still prefer nanoTime because it resists from system date shift. The perf difference is negligible IMHO."""
"Very nice work mate. I know of a fair few quests that hold conversations between NPC/player and/or several NPCs. I can see them being a lot easier to do for you guys now. :)"""
"Actually, I kind of agree with James. It's not a surprise so many huge plugins are pulling out of Bukkit."""
"Great stuff, after these last tweaks I think it's ready to go in.  Have you've signed the cla? (Will be needed before I can merge it in: http://www.typesafe.com/contribute/cla)  Cheers, √"""
"Yeah, I was going to enhance the performance of the augmentWidthorHeight function by doing exactly what you have there. Having to call getComputedStyle multiple times sucks. I need it to also grab the box-sizing as well, so it needs to be a bit less naive"
"@dddeeefff That feature in demand lately is being implemented in the ItemMeta system. So yes, that is being done. :)  @Darkhand81 The 'burden' put on developers is negligible with exception of huge hacky libraries that break every update anyway (meaning"
"I usually use single quotes for Hash keys -- it's easy to change them if you prefer."""
"Oh, that's sad :( I don't know if I supposed to do so many commits, but I will do it anyway."""
"Thank you so much for clarifying that, @EvilSeph!  I think those changes are really needed."""
"Hmm, how can there be collissions ? If it's the same object class then each object has its value-unique hashcode, if they're equal then they're equal.... unless String's hashcode isn't that reliable.  Or can you suggest a mathematical hashcode for id an"""
"Alright, I'll change that now"""
"Remove empty line"""
"Moving CallingThreadDispatcherModelSpec.scala broke the build on my machine. Does it still work on yours?"""
"same problem ... buf changes should be completely reverted if you ask me!"""
"You could use an interface, I don't have a good grasp of whether an interface is better, or whether the current approach is better. I thought that the if statements that were added wouldn't be required if it were a 'pure' decorator. Something like:  ```"""
"This is a public method so we shouldn't casually change its signature.  It should continue to take a single `arguments` parameter, and then `_on_request_body` should merge `body_arguments` and `query_arguments` into `arguments`."""
"@Vasago Vehicles were really fucked up before Machiavelli and Shauren brought in their changes, they were good changes, they just didn't interface very well with the hacks that were already there before, it has been cleaned up since then, but they still ne"""
"Yep :) Google Webfonts doesn't have Melbourne, which is the open source font I'm using in the image. If I can't find that font hosted externally by a reputable CDN, I'll probably stick with the image (base64 encoded)."""
"Ah, no problem really. I kind of started hacking away without taking a look at conventions and code style :sweat_smile:"""
"Thanks, I was able to reproduce it that way.  Now investigating."""
"Then shouldn't `lib/action_dispatch/journey/core-ext/hash.rb` be removed as well?"""
"This is sooo much better than the `ldap_prefix = ldap ? '(LDAP)' : ''` line of code :)"""
"Really great addition, will save me a lot unnecessary hacking  :)  @tilsammans  As far as I know protected_from_forgery, is for securing the frontend users, from other users/sites/scripts, who tried to pretend  to be the frontend user. I mean that a per"""
"I'm holding off till the auth stuff is all done refactoring.  Then I'll revisit everything.  The reason I didn't do the icon-with-link route is because this is meant to be silent SSO -- e.g. you just visit gitlab and if you're already logged into openid"""
"Ah, sure. Thanks for explaining and sorry for the noise! :-)"""
"Thanks, I missed that new feature. Awesome."""
"Is it possible to use the Thrift isSetParent_id (or whatever it is called) here instead? As mentioned the id !=0 is fine as a hack, but if possible we should figure out the correct solution."""
"Shouldn't the assigned value be `jQuery.fn.toggle`?"""
"Doh. Fixed!"""
"Why `window.String` instead of `String`? They're reliable globals in the language (not just in the browser API), also whitelisted in JSHint.  Or is this to allow better minification? Afaik it can only optimise the host object name that is locally aliased"""
"I believe that Chunk.java line 47 is causing an NPE :( - http://mibpaste.com/PCUVYv"""
"I know this is possible and I am doing it but making mounts working on all areas for gm would be good if something needs testing with it. Aso you cannot even move on fly mount with gm and player not!"""
"This one looks weird - use a tab for the indentation."""
"2012-01-03 22:23:45 CRASH ALERT: Player Smetano has viewpoint 0 4 when removed from world  2012-01-03 22:23:45 CRASH ALERT: Player::CreateViewpoint: Player Smetano cannot remove current viewpoint!  spam in console"""
"@mbax Yes. Innocent until proven guilty should apply for code in 99% of cases."""
"@Ascathor, not necessary, this caused by the github's pull request policy, we can't select which to pull, hope github could solve this problem in the following update, forget it, please view the """"files changed"""" tab, haven't this kind of thing happen to you"""
"Can you remove the `!!`, and don't mutate the `options` hash that's passed in?"""
"I need to correct myself: since the result of `nextSeq()` always goes into a `Some()` boxing will happen immediately anyway; an optimization would be to encode the optionality of a SeqNo by reserving `Long.MinValue` as undefined."""
"@mcsnetworks WOW That looks amazing! Will most likely be moving to this some time in the future."""
"Why reimplementing minitest's [`refute`](http://bfts.rubyforge.org/minitest/MiniTest/Assertions.html#method-i-refute)?"""
"workers are assigned to the `cluster.worker` by there uniqueID, this makes them very easy to find again. If we change this to an array people will get a lot of `undefined` values in there for loop. Also i don't like to have method like `.push()` that indi"""
"this seems like a very weird fix..."""
"So, honest question for whoever can answer this..  Dont you guys already tell the ticket submitters that its not your problem if the plugin derping is using NMS/OBC?  If so, then wont the workload increase for having to validate all of the new file sub"""
"Yeah, I’m feeling so BAD today."""
"@asofold  Sadly, this is probably good news for hackforums.net's swarm of hacked minecraft clients. Maybe it's a good time to join them and sell hacked clients. (/sarcasm)"""
"I wanna give @cowboy a hug :D"""
"I think as @freakboy3742 made in an earlier comment - these hints aren't all that useful, any error should hopefully be sending people to our docs, so let the docs do the talking. I would say it is helpful to include what the value is, when it is not an a"""
"from?"""
"@jeremy thanks SO MUCH for that writeup, explains why it was decided to be not fully baked, with a convincing and reasonable argument, and also explain what you are thinking about future directions.   It is VERY useful for us Rails-using developers to ha"""
"this doesn't helps, the problem just modified a bit (for example in one time all totems, all temp enchant and mob corpses are disappears/deleted/despawned in one moment at whole server). I found the originally source of problem - it's 10687, with reverted """
"ok thanks ^^"""
"Is this different than the """"default viewmode for 4x3 material"""" ?"""
"this might want to use the new aroundReceive which is being discussed in the context of `akka-persistence`; we should either await that before going forward with this or create a ticket so we don’t forget."""
"Ew, sometimes I forget that there are bindings for languages not written in C. :/  I'll fix this in the dev branch, thanks."""
"Sir, yes sir! :smile:"""
"thx, corrected. Its pretty good if thats the biggest flaw youve found :p"""
"What is `subline`?"""
"We need to think about optimizing here for the most common case. Which is a single alias. In this case, we don't really need to allocate a list. Might be an overkill, but stil.. :)"""
"This issue should be reported upstream."""
"I thought I had a test that asserted just the opposite. """"after initialize"""" should be like it says *after*, not before middleware initializers run."""
"this issue was originally raised in ticket #13364 http://trac.xbmc.org/ticket/13364. the patch attached there is a hack and should be ignored, however the rest of those details provide an example of the problems that can arise due to the current order of i"""
"`Concurrent::Cache#fetch` is slightly different from `Hash#fetch` and behaves similar to `AS::Cache::Store#fetch`: it stores the return value of the block, so the `[]=` call at the end is unnecessary."""
"Removing debugging seems like a bad idea."""
"99c8399772701af9bf3c79e4c514aead57cdc7d6"""
"offsetWidth/height are unreliable cross-browser - especially when it comes to inputs. http://jsfiddle.net/jboesch26/Gtm5H/"""
"actually, this *will* wrap at 2038 on linux if it uses 32 bit signed time_t by default (which I think it does)."""
"Alright, one more try at 51a02409c867eb0e7e3965f13a48e7fb6b154bcd"""
"So much win with the [NoScope fix](http://msdn.microsoft.com/en-us/library/ie/ms533897\(v=vs.85\).aspx#1) here *(minus the UA sniff)*. +1 from [the future](https://github.com/jquery/jquery/pull/718#issuecomment-4796946)!"""
"You're a smart cookie @jonleighton."""
"Haha, switched, have you?"""
"Which method is """"faster"""" is hard to say as the JIT of course is very magical :)  But I'm not too concerned about that -- I'm more curious about your comment above:  """"Change all access or definition to HttpDataFactory to the new AbstractHttpDataFactory"""
"Looks like rubygems took out the warning. I think to be safe you need to have the executable in both gems. :-(  See https://github.com/rails/rails/issues/2400"""
"To save the trouble of doing this, I just kept ```ldap_enable?``` and added a new ```shibboleth_enable?``` for the same purpose. Nevertheless, I agree with this approach as it is cleaner."""
"Here, you are not checking whether the ``rewriting`` key exists in the options. but the signature of all methods of the interface declares ``$options`` as being an optional argument. This is not consistent."""
"Oh, yes - you'll need compatibility with boost filesystem v2 though... for now"""
"And I got the error. :("""
"No, it's not architecture, it's CPU family. In log we get """"x86 64-bit"""". I think it's better than """"x86_64 64-bit"""" :)"""
"And a long time to wait vehicles, which also work normally in a patch..."""" One big hack can't be """"normal"""" patch ;) without dependence work it or not..."""
"I agree this needs to be worked out. However, we currently have a request to use custom hashing on TrieMaps. I think we should at least enable the TrieMaps to take custom Hashing objects for now. Given that we don't do this with the rest of the collection"""
"what problem with its? except clean horribale hackish way for detection... only one good: maps not big differen in diff versions, so one time need do..."""
"Why are you obfuscating your browser sniffing.  Better to be honest about it with .browser"""
" Because the fields in db are tinyint and not int"""
"I suppose that this code should be moved elsewhere, no?"""
"CMake Error at cmake/macros/FindGit.cmake:24 (message):           Git was NOT FOUND on your system - did you forget to install a recent version, or setting the path to it?       Observe that for revision hash/date to work you need at least version 1.7"""
"whenever i see deleted code, i get all happy and stuff."""
"this way to retrieve the salt seems like a really hackish implementation IMO."""
"did we ever make it part of the CS to not have a return inside an else in cases like this?"""
"But it's not the same semantic. AwareTrait are means to inject something. Here we really provides options, not make the filter aware of something. Mmh... I wait for the others :D."""
"I'd actually argue here we import MessageDescription, and change the signature of dispatch to: ```php public function dispatch(Message $request, Message $response = null); ```"""
"Removing this class seems to have broken ServiceStack.OrmLite.SqlServer:  Could not load type 'ServiceStack.Common.Extensions.ReflectionExtensions' from assembly 'ServiceStack.Common, Version=3.9.38.0, Culture=neutral, PublicKeyToken=null'."""
"**Even better .... Thanks ;o)**"""
"sure, but we are compatible with Doctrine ORM 2.2+"""
"that exception in postRestart would be the direct equivalent of a `require()`; I’d include it to be sure"""
"Ah sorry, so the ask implementation does it for me, then I'll just use recover."""
"rsa, how your multi-threads related to clean mangos sources? Object created in same threads that Map::Update run. If muti-threading will be added to mangos core it will be per-Map mutitreading and then Map::Update still will called in same thread as objec"""
"Yep, it will be 130 :("""
"glad to see that you accept invitation to TC team and community ;) congrat to you man."""
"Yeps. Idea is that email is required only to create new user. Authentication do not require it (if user is found)."""
"I was just about to do that.. :( oh well."""
"Good documented. """
"I disagree with this change. With conditional classes, it was very clear what styles were applied to which browser. CSS hacks don't have inherent meaning, which decreases maintainability in my opinion. They're probably fine if you're the only one writing t"""
"Isn't there a MurmurHash in Scala libs? In the nightlies there is a scala.util.hashing.MurmurHash3. Is ours customized somehow?"""
"I think they should be the other way around. It's the `consistentHash` that should be in the atomic since you don't want to overwrite it with a stale `consistentHash`. Overwriting the `consistentHashRoutees` with stale values will just trigger a new updat"""
"Sounds kinda hackish."""
"Should be `The CSRF token is invalid. Please try to resubmit the form`"""
"The old behavior was to only use the events toggle if the first *two* arguments were functions, since that's its signature. It may not matter, but it's worth noting."""
"Thanks for this   =D"""
"@int007 you are an idiot. removing that line doesn't leave anything undefined. Go learn C++ and come back when you learn the difference between defining a variable and passing it to another function."""
"The BC break is necessary since it is currently not working properly :worried: """
"Oh, sorry it was my mistake."""
"epic :)"""
"I loved it :("""
"Way to many caching issues, memory leaks, and other complications :(  We can always take another stab at it for 3.0."""
"agree with Betaman2k... where is summon fix ?) we can`t find the problem :("""
"gah, sorry for the double post. /me fails at github"""
"No worries. I often forget the docblocks too."""
"My thinkings about this: 1) It makes the lives of some devs harder then needed. Not only do I avoid NMS/CB code cause of such breakages, but also I try to use deobfuscated variables/methods only if possible. By doing that I avoid breakages pretty good, no"""
"@lsmith77: The code is different from the one we have removed some time ago. Here, I've just fixed a bug because the Store set a wrong `Content-Length` header."""
"`i` should be unsigned (-Wsign-compare)"""
"$appapiid = 'xxx'; $appsecret ='xxx';  $facebook = new Facebook(array(   'appId'  => $appapiid,   'secret' => $appsecret,   'cookie' => true, ));   $session = $facebook->getSession(); var_dump($session);  results in: NULL"""
"Gah. I'm an idiot. I undid this particular change in subsequent commits. In case it's not painfully obvious, I'm still figuring out git."""
"its full of hacks (including that +200 to LK height)"""
"I think it's kind of weird that a method named validate_response! actually validates @request... Changing the name to validate_request! would be less confusing."""
"yup - I just did that. but unearthing some more potential bugs.  this is like a gift that keeps on giving ;)"""
"Crap, I thought I had set the author right... *confused*  This was entirely done by Lordron"""
"Don't forget to rerun cmake, guys."""
"Thank you for this fix!!"""
"And yet I'm on me:) I went through all the code 2 times nothing to report on that side I raise a good compilation and I see I handed the file on my rest can goo better ^ ^"""
"all this reverts are bad as I already updated 99translations. So much lost now :("""
"U removed because some compilers didnt support that? Thats odd..."""
"@renanbr, great point! Why not just change it to look like this, though?  ```php $paths = array(   // for git clone git@github.com:sebastianbergmann/phpunit.git && cd phpunit && composer install   __DIR__ . '/../vendor',    // for composer install -"""
"shouldn't this use round_int and implement that using round() on arm. that way, if it's ever implemented in a faster version, we don't have to touch other code to have it used."""
"Reverting this without fixing the issue in #7608 just breaks all versions of Opera again. The reason for why curCSS is split up should probably be commented here too in order to avoid forgetting why it exists in the first place. rwldrn had an alternative p"""
"But even without space reuse this is wrong because there's space on both sides. The naive algorithm would be `images.inject(0) {|sum, img| sum + img.width + 2 * img.spacing}`. Also, do we need spacing on the edges? I don't think so."""
"Shouldn't we validate the max-tries value?"""
"![](http://img689.imageshack.us/img689/7277/15620874.jpg)"""
"Looks good.  Mind if I add a CityHash implementation in here?"""
"this line is throwing an error on iphone, GL_UNSIGNED_INT is undeclared."""
"Don't forget: V8 is smarter than us all :P"""
"Again `unsigned int *` instead of `int *`."""
"HOLY F*CKING CHRIST!"""
"Don't add new whitespace."""
"Oops. My mistake !"""
"That's a lot of fixes to my patch, sorry for that."""
"OMG, the C# compiler was able to figure out the generic type parameter? Color me impressed for once. :)"""
"LOL"""
"Well, no. The forward slashes are creating the problems, they're supposed to be hashes. Or do you mean the names?"""
"Does this show up elsewhere?   Seems a useful method, but I could be wrong."""
"Ah crap - sorry I thought original_listitem_url was only used for UPnP & plugins, will fix this afternoon"""
"@randx Shouldn't 'Resque' be changed here as well?"""
"Because I always forget. :'( I'll put it in next time."""
"Being explicit is faster then coercion, I suspect that extend is a perf sensitive operation..."""
"Yes, sounds good. I'll file a bug for it, so that I don't forget it. Done: https://issues.scala-lang.org/browse/SI-7624"""
"We need basepoint cache/saving :-("""
"I agree that we should do a smooth change, although it will not break production environments (a already set password with a length of 6, still works, right?) . So, I propose we do this:  * New applications come with this new definition (but it can be chan"""
"Shush :("""
"was fixed in https://github.com/TrinityCore/TrinityCore/commit/4bc983d7b756d235477eef8b7529cacfb7a1f247 sorry for that mess up guys."""
"You are correct -- beforedeactivate only exists in IE.  propertychange also only exists in IE.  Would you like me to remove the browser sniffing, and while I'm at it, clean up the code as per JQuery_Core_Style_Guidelines, and rebase it (or whatever the co"""
"Sorry, that's my fault.  I'll revert this cleanup and add few more tests for those cases."""
"Yeah, I know, but he's bound to want to fix it he he finds it later, I'm just finding it for him now. :P"""
"don't forget xbmc.gui/xbmc.json api bumps here too as (if?) needed, i guess one for the whole shebang is enough.  I would guess that lots of skins have hard-coded hacks for .tbn stuff?"""
"ok. its clear now. Also i'm glad we got rid of `@history_path`. I'll merge it now - hope it works :)"""
"Seems like something is leaking memory :("""
"I think it's ok that you are the author. To me the author is the one who had the idea/designed it conceptually or contributed large parts of the code."""
"http://img692.imageshack.us/img692/3598/missilulft.jpg PICTURE UNRELATED.  Some people may know where this comes from :p"""
"Use the stable download; this will break checksums when upstream pushes more commits."""
"You realize that you post on 7 month old commit?"""
"That is indeed a good idea, but my time is just running out (and I am only on smartphone anyway). Will add the test as soon as possible!"""
"Missing capital letter for the first word and missing dot at the end."""
"In [11327]. Thank you :)"""
"No, old way was incorrect, maybe someone can do a config for this allowing to don't send whispers instead blizzlike way."""
"no only season 11 no pvp damage gems and its weird on linux the damage is atleast working (lower dmg but its working) but on windows its just keep missing each attack for me :/"""
"Models is fine, they can be isolated, and are easy to talk to directly though the class registry anyway  If a normal request has the same event manager for everything but models, then I'm a happy man for a crud plugin perspective :)"""
"Damn, I screwed up my mental math. You're right."""
"We don't like patches from gists very much."""
"I have no idea either, I just trust the spray guys"""
"Oops, I accidentally committed my work-in-progress liveHandler rewrite in this commit :-/ sorry about that! I'll be landing the correct code soon."""
"Looks good. Our usual idiom here would be `!error` instead of `error == 0` - would you update that? Then I think this is good to merge. Nice catch!"""
"Yeah yeah I've seen that, it's an awesome work BTW :) But I was wondering what does the statement `name = name;` stands for ?"""
"We need to signal """"done reading but no data yet and not EOF"""" for crypto streams."""
"That's why it's so nice to have that many pairs of eyes looking at this code :) Thanks!  https://github.com/jquery/jquery/commit/1c77259d06e49ccaedbd0f7adf968e10748dacc0"""
"maybe """"urand"""", instead """"frand"""" ? :)"""
"My bad, it's gone now"""
"Good catch, I missed that. :)"""
"Epic commit. As always, when magic commit number comes :))"""
"Passphrase"""" was just fine IMHO. We really want users to use a longer sentence instead of just a password. Also, """"Verschlüsselungs-Kennwort zur Dekodierung"""" has too much redundancy."""
"Are you sure of these Gameobjects IDs? I don't see them in sniffs, at least for map 469."""
"We could probably go either route here successfully.  """
"Stumbled on this quite a bit, good job. """
"I agree in theory with the interface, but how would that look and what would the interface be? Having an interface with only an """"execute"""" method signature defined feels like a weak interface. I'm not saying it's a bad idea - but I'd like to see how it woul"""
"Before you couldn't change the realm after loging out, you had to relog. Now you can change realm after logging out."""
"OK, fair enough, it was like that before your edit.  I guess my question is: is it logical to send an not-yet-authorised user to ```loginRedirect```?  From the class comments it appears that ```loginAction``` is used to typically display a login form, wh"""
"@mislav  @adamv  creating a ~/.cabal/config will happen as a matter of course when installing any haskell libs. that is a bad reason to use --global.   --global will create confusing problems for none expert haskell users """
"Going to leave this place now but will be back online soon ;) I usually go by DiogoSnows Thanks a lot!"""
"what about 15   EVENT_T_FRIENDLY_IS_CC ? DispelType masks are still missing in docs:("""
"Actually for this particular check, I think there may be a simple solution.  When AUTHENTICATION_BACKENDS is set only to the modelbackend (the default), consider a non-unique username-field to be an error, otherwise a warning that mentions your custom b"""
"the indentation is weird on this line"""
"I use it for debugging, ex. in IE :) where I can't use firequery."""
"I already ended up with short because 32767 waiting threads sound plain  wrong :-)  On Thu 09 May 2013 12:39:48 PM KST, Norman Maurer wrote: > Or maybe even int? Should have notice It :/ > > Am 09.05.2013 um 01:41 schrieb Trustin Lee <notifications@github."""
"oops, my fail :("""
"Note:  `MurmurHash`  exists in `scala.util`.   (a) Should we provide a default `MurmurHash` instance?  It's an instance of `T => Int` (b) Should we adapt `MurmurHash` to extends `Hashing`? (c) Given that we have two classes, should they both be in the sam"""
"Forget, discussion on #7418 :)"""
"hm, that is a bit vague indeed. Maybe it should just throw an invalidargumentexception instead? It doesn't make much sense to pass -5 or 10 as a value anyway. Allowing it only works confusing"""
"@stevenh512 Do we even know that GitHub was not using attr_accessible? They could have just opened up columns that are only accessible to admins to all users because of the bluntness of attr_accessible (somewhat remedied by the new role based feature).  """
":-)"""
"Deep Wounds damage still doesn't scale according to damage modfiers and/or doesn't stack right. That's another critical problem."""
"Alright, thanks. I don't have access to my dev environment at the moment, so I'll have to fix what you commented on tomorrow or the day after.  Thanks again, Trustin :)"""
"Lol, i think in mangos many missing things that let wait for you with this ;)"""
"Permanent Feign Death is permament! ;)"""
"It seems that `time` is in fact required to be a `FiniteDuration`, because e.g. `isOverdue` fails if `time` is not a `FiniteDuration`; if so, this should be checked statically by making `time`'s type `FiniteDuration`. Consequently, you'd have to move `fro"""
"Unsure if this is correct. I don't see how this would have worked previously, though."""
"Oops, sorry :/"""
"I needed to update it about a month ago :)"""
"Fixed. """
"so this fix is also related to my post here i see: http://www.trinitycore.org/f/topic/5197-can-this-cause-indeed-crashes/ thanks for the fix!"""
"It's not *that* severe if we allow wallets to be stored at arbitrary locations on the file system as in multiwallet-qt. It would be quite severe if _path happened to be the datadir :)"""
"That's true - though not with the new syntax.  (Also, Erlang and Imagemagick are detecting versions wrong right now, will have to fix.)"""
"That is bad news. We were just updating our app and migrating from attachment_fu / AWS::S3 to paperclip because we were getting sick and tired of the problems with european buckets :("""
"Why by default set 1 ?"""
"Y U NO LIKE? :("""
"We remove `/download` from sourceforge URLs (`brew audit` will warn about this)."""
"Do we need to check for undefined or null result here? we do pretty much everywhere else"""
"I'm -1 on this. It's imo easy enough to do Mailer.expects(:welcome) and then keep mocking from there. I don't like having two ways of doing the same thing purely to make it marginally easier to mock."""
"Build failed for the updates since #965 :-("""
"It *is* a magic number, and... no. The choice of how good a fit to demand while protecting ourselves against nontrivial loops is arbitrary."""
"Losing this is unfortunate... I think it helped a lot with not leaving missing translations around."""
"Aah, this is thwarting my attempts to get formated output from redis-cli over a ssh command."""
"This is needed to control dropping. Consider we just dropped a frame in decoder, this is at the end of the queue. How should player know that it has already catched up? You can't by just looking at the front of the queue."""
"It is only a security risk when used with TLS/SSL, so maybe this could be:  Note that GZipMiddleware is considered a security risk on sites that use HTTPS"""
"I think I prefer: import org.jboss.netty.akka.util.{ Timer, TimerTask, HashedWheelTimer, Timeout ⇒ HWTimeout }"""
"Right. Since it is currently fixed size anyway an array would fit perfectly."""
"It is what bridge methods are for; we don't have enough bridges. The erased signature can only differ between the subclass and the implementing class if the backend will handle the calls and forwarding correctly; it is not a pretty picture when you get to"""
"We can drop this. It's more or less a relict from the process of creating this formula."""
"Great! No need to hold off, go ahead and work on this branch and we'll rebase later. Thanks Amy!"""
"might be possible, indeed :) """
"These actually look rather terrible and are frequently broken for me. Here's a screencap of what I'm seeing:  ![buggy button](https://f.cloud.github.com/assets/263432/1054875/9fa97904-111b-11e3-9123-da1220b289d4.PNG) """
"Forget about it :-) all fine."""
"Let me know :)"""
"I'm glad you see these things. It's easy to make mistakes or overlook things when you are cleaning up several files :)"""
"Thanks, Vladimir. Nice update, I haven't thought of it :)"""
"It does harm! You hide `const` problem with this type cast. Type cast potentially hides a lot of problem types.  Typecast in C usually points to a hack or a bug. It's better to avoid type cast wherever it's possible. You're writing code for people, not for"""
"... Did you not see the bukkit commit? https://github.com/Bukkit/Bukkit/commit/74a6fb983408c3f22a669c33c78ff3ef17524ffc"""
"Unless there's something removing items from that list this could be a real pain in the ass for very dynamic, long-running apps... like games =/"""
"Why did you remove the check of valid ssl packet ? I think it makes sense to have them in place in case a non secure client tries to write something"""
"Questgivers don't offer daily/weekly quests anymore :-("""
"why does it matter?"""
"Doesn't that do a copy of the original collection? Would it be preferable to have a """"foreachChild""""-method instead?"""
"moved into encoders/decoders"""
"I understand. In that case, we would need to have a gui setting to hide/show 1-movie sets. IMO, to most users it won't make much sense having their sets list polluted with those..."""
"that is a good idea."""
"Just out of curiosity, why `uv_buf_init` should take `unsigned int` instead of `size_t`?  On my machine, the compiler complains about > warning: implicit conversion shortens 64-bit value into a 32-bit value  using in `uv_alloc_cb`."""
"This is so good !!"""