labofsahil/github-event-dataset-2012 huggingface dataset:
Evaluation order of function arguments

According to TDPL p. 50 (section 2.3.5.3 Function Call) function
arguments are evaluated left to right. This commit fixes the examples in
the documentation where it is assumed that the evaluation order is
unspecified.

I do not know what's the current behavior of dmd regarding function argument evaluation order and whether d-programming-language.org concerns the specification or the current reference implementation. Depending on this you may ignore this request.
Multiple bugfixes

- replaced resTypeToString array with function, for bounds checking reasons (I encountered game data with type ID '6' which is out-of-bounds)
- switch to using unsigned 8-bit PCM audio instead of signed, which works around bugs with PulseAudio in most currently available SDL releases (maybe the most recent SDL release contains the fix, I'm not sure). Fixes issue #2
- LP64 platform compatibility. Long-Pointer 64-bit platforms use a 64-bit 'long', which breaks endian.h (which assumes that 'long' is 32-bit, and typedefs 'uint32' as 'unsigned long'). This causes the game engine to crash at startup.
- A better fix for LP64 (and more universal compatibility) which switches the 'uint8', 'int8', et. al. to use the types defined in stdint.h (uint8_t, int8_t), which are guaranteed to be the correct sizes, so no preprocessor magic or configure scripts are needed to ensure that the data types are the size we expect.
- Replace '-O' with '-Os' to optimize for code size. Sort of in the spirit of the original game, and also generally GCC emits better code with this flag.
- Drop some warning flags which are meaningless for C++ code (-Wstrict-prototypes, -Wimplicit)
- A bit of whitespace cleanup, particularly in terms of indentation and comma placement on line continuations in resource.cpp.
- Switched a binary OR to a logical OR (the binary OR made no sense given the context and generated an irritating compiler warning)
- Clean up an unused variable, initialize an uninitialized variable used in a comparison, etc.
Updated screenshots to VS 2012 RTM & Windows 8 RTM + bug fixing
Fixed animation and speaker notes of VIP and upgrade domain

Fixed animation and speaker notes of VIP and upgrade domain
This block will be rendered as one line. Have an empty line between if you want it to separate. If not, properly wrap them together.
The API and impl look great. A question - how this monitoring is envisioned to be weaved into the CAS overall system i.e. is there an idea to set up some kind of scheduling beans to periodically poll these monitors and asynchronously report the statuses via some kind a remote destination e.g. email, messaging topic, [place you favorite destination here]?

Cheers,
D.
The on_redirect block doesn't get called if follow_location is set to true - is this expected? I was hoping to get curb to follow the redirect for me but let me know what the url it was redirected to.
vdrfan - the c&p thing wasn't against you - but against myself. Just for clarification :)
Haha wat

On Jan 1, 2012, at 10:58, Rossco1337<reply@reply.github.com> wrote:

> are you a wizard
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/ceol/pypkm/commit/dc276c8462de97df176d0197078fc3189906a9d0#commitcomment-830329
@rgrove I feel this updated implementation is more solid and `_url` might be useful in other places in the future. Let me know what you think…
A set per definition does not have order, right?
@zydio Oops -- too late I've spotted a bug: 

${jsConditionalForIe(9, '&lt;script type="text/javascript" src=" + h.url_for_static('/scripts/vendor/html5shiv/html5.js') + "&gt;&lt;/script&gt;')}

Is missing ' in two places ... should be:

<pre>
${jsConditionalForIe(9, '&lt;script type="text/javascript" src="' + h.url_for_static('/scripts/vendor/html5shiv/html5.js') + '"&gt;&lt;/script&gt;')}
                                                             ^                                                             ^
</pre>
Yes, so that people are aware of it, and dont change the protected fields to private, or remove virtual attibutes.
a) I don't like `ap`'s output
b) this helper seems to provide slightly different functionality, mainly by printing to STDOUT or logging depending on context and adding the "[DEBUG]" prefix
c) this helper proved to be very useful (to me, anyway) in another project

I'm not intimately familiar with either of your suggestions though, so I'm happy to be convinced otherwise...
xulijian2
@piotrga good catch ... drop the sync optimization as it will result in less confusing behavior under error conditions. Need more time to think about alternative optimizations but we can also introduce them later without breaking public contracts.
Je nutné v tomto repository tak často přepisovat historii jako zde 0c5a67d86? Je to přinejmenším neslušné.
it is bug
This was a test, to see if auto Update on Transifex work ;)
Shouldn't declare a default value for $parameters?
It gives a notice on production server.

I think it should be like this:
public function raw_query($query, $parameters=array()) {
![whoo hoo!](http://f.cl.ly/items/3h431B0g2p1N432K2b0i/Homer04.jpg)
It's not a test of the method itself but a way to test other methods that call getName() without having to rely on having a real Plugin instance.

What was the rationale of making this method final? It's not like a plugin developer couldn't put anything in the name within plugin.yml, no?
I'm ruthless in keeping the complexity of the implemention as low as possible. Will you kindly explain the need for this branch, other than it avoids a simple string concatenation?
헉. Major한 오류네요. 반영하겠습니다.
if region info is migrated but splitA/B is not, that is kinda weird.  would this code be dropping the splitA/B - they need to be moved to the new row.
The line before is `ok = file:delete(...)` which will return `ok` anyway. The return value of an assignment or pattern match is always the value of itself, in this case that's `ok`.
Could we generalize this class?
Nice tests! The spice of TDD is flows inside you now.

Just as a reminder, instead of $this->assertTrue(isset(...)) you can do $this->assertNotNull(...)

Also, I think we would benefit to add some of our own custom assertions, like assertTrueString and assertTrueInt
you can make it one with contextlib.closing
http://docs.python.org/library/contextlib.html#contextlib.closing
I'm gonna head out for tonight, but I'll can definitely reformat the coding style.

The StatsdClient is taken from the etsy/statsd:
https://github.com/etsy/statsd/blob/master/examples/StatsdClient.java

They have a license at the root of the project that at a glance looks compatible. You want me to prepend their license into the code? I guess I could also ask the author if it's cool to place it under ASL.
https://github.com/etsy/statsd/blob/master/LICENSE
Well, the general idea of the minion logging is so a local user that was trying to figure out what happened on a system would have a much easier time of just looking at the salt minion logs to see that Joe Admin ran this command to modify such and such file at midnight last night.

I work with a big team of SREs and this is a very valuable "convenience" item to keep them from having to constantly jump on the master and wade through all the commands and try to figure out if it applied to their specific minion.  Especially if someone tries to get creative with regular expressions or the like.

Which part breaks the backwards compatibility, just the call to the user key?  How about wrapping that in a try/except and reverting to the original log message format if no user was passed?
this is still pretty rough, but it's a start.

TODO: 
1.) fix marker labels incorrectly displaying after zoom 
2.) jasmine tests
3.) documentation updates
Fair enough you win. The fact that many other images are needed broke my last argument.
> It costs very little, but it adds nothing. So it still has a very bad value-for-cost ratio. We've got dozens of graphical resources that are simply required to be there for the client to function, which is why we gain nothing by making these two arrows optional.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/Bertram25/mana/commit/396ad0d3a39fdd18c6a538c95ad5da899acc7cbc#commitcomment-907890
>
Wow! Textual has been built a lot.
I am sorry, I'm not sure if b-mind deleted his comments, or I just can't see them... however, changes are still unfinished, I wanna make it up a lil bit different way then I did it on ArcEmu forums.
don't forget to revert that...
Thanks, fixed.

On Thu, Feb 9, 2012 at 4:37 PM, Thomas Freudenberg <
reply@reply.github.com
> wrote:

> According to `ConfigOptionDocs`, this should be
>
> ```csharp
> AvailableMemoryForRaisingIndexBatchSizeLimit = Environment.Is64BitProcess
> ? 512 : 128;
> ```
>
> Just to keep @e-tobi happy ;)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/ayende/ravendb/commit/0d8e738fa9a6d083a2dfaea4290b3cbcf3f78163#commitcomment-949658
>
I have no experience writing tests in Python the 'proper' way, but I can give it a go.
It seems like this now uses a result variable which is no longer declared?
уже описал xml ранее
Why is this synchronized?  Isn't this only run once, when all of the threads are finished?
Need to translate all of these. Since these will outputted as normal text (and not as html attributes), use the xlt() function (rather than the xla() function) from the library/htmlspecialchars.inc.php file:
$dateRanges['1_day'] = xlt('1 Day From Now');
I believe we had to add this line so that it would work right when clicked, because it's set in a modal box. If that helps...
Cheers RedSquare, fixed. But more importantly, thanks for explaining -what- I did wrong, not just giving an answer. Much appreciated :)
yet https://jenkins.mozilla.org/job/marketplace/204/ :)
It's the StageManager's job to manage the BackStage, but not to construct it. 
If you want to know where an object comes from, it comes from its context. The system is easier to test and more flexible by composing it with single responsibility objects.

What happens tomorrow when BackStage needs to accept Actors and strip off events from InputManager. Instead of having to update the construction code in the middle of StageManager, you can just go update BackStage with one line of code to specify the new dependency.

Feel free to IM me for further discussion.
Ok... The slipstreams stopped working? I can't imagine...
Hey people, what do you continue asking about database?... are you not reading Spyder message or need glasses?
no! this mobs don't scripted in C++ and must be in SAI )))) ( include bosses in other scripted Dungeons ) I don't script in SAI mobs who already in C++
in the commit note i said i added line diff, but i meant to say scm-diff
Keeping the flisp parser would be like keeping around a C compiler written in assembly language. At some point, not really necessary.
You just have to be careful implementing syntax changes; you have to do a bootstrap before you can use the new syntax. And if the new syntax is not backwards-compatible, you have to first implement it, then bootstrap, then change all the files to the new syntax, then bootstrap again. If something goes wrong you get wedged and have to roll back, but that's not such a big deal.
On my mac, it had found BLAS in vecLib that Mac OS X ships. I noticed it, and forced it to use our BLAS.

-viral



On Jan 28, 2012, at 12:50 PM, JeffBezanson wrote:

> I get
> 
> ```
> checking for sgemm_ in -lblas... (cached) no
> configure: error: Cannot find BLAS libraries
> make[1]: *** [arpack-ng-3.0.2/config.status] Error 1
> ```
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/JuliaLang/julia/commit/2029caaeab85b17e31d0cea53738e47265c05d8b#commitcomment-910559
je sais :D

Le 19 fvrier 2012 12:19, Olivier Hoarau <
reply@reply.github.com
> a crit :

> Bien vu !
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/Olwaro/Web-loto/commit/06ddda64a22589e10303be0e43c168386ef01418#commitcomment-982397
>



-- 
Ludovic LEFEBVRE.
Elve 1re anne ENSC.
I noticed that type checks are actually mentioned [in the coding style](http://docs.jquery.com/JQuery_Core_Style_Guidelines#Type_Checks) that Aza prescribed.
This speaks to a broader issue I want to handle, I want consumers to be able to manage nameservers on their own, external to what the platform provides. I just haven't come up with an interface that I liked. It should be a mechanism to add/remove {server:,port:} such that when servers are available they set name_server_ready and checkReady so that the client portion is notified it can send queued requests.

The reason parseResolve doesn't take a filename, and parseHosts does is because I knew the logic could probably be mostly reused on all platforms. On the contrary I knew parsing a resolv.conf wasn't going to be possible on all platforms so I planned on having a separate function specifically for windows, I hadn't investigated what to do in that case. So anyway, adding the switch in parseResolve is not the right solution but adding a new function that's only called from the original switch is what I would prefer.

I want this to work on windows, and I'm fine with bootstrapping to the point where it doesn't except on start.
well, pretty sure this is caused by multithreading changes - which I know of nothing and hence cannot discuss (yet)
Neede some fixes here
if swap disabled, this result always appeared on linux servers (without mangos) dependent from LA. on LA 1-2 after 24 hours, in LA 0,2 after one month.
check with swapon -s command.
Use temporary this: https://github.com/Undergarun/mangos/commit/72a2a8bfe55f0e13e18d82f094f59b78906abf46 but all your sd2 scripts must be moved to object guid use as soon as you can.
я уберу вообще. во первых пользы с него мало, во вторых винда, в третьих оверлок все же не есть хорошо...
@beberlei So, it is possible to use doctrine2.2 with Symfony 2.0.10? After this commit?
The reason for having the CompressedBytes type was because uncompress2 requires having the length of the string being passed in, so I was storing that in the zlib_pack struct.  Also, it allowed me to create the finalizer to manage memory (though -- I can probably do that anyway -- so it's really just the first point).
Poor swig won't handle "-D VAR=1"
edit 'do' to send command to all focused switches
Ab sofort immer GitHubViz schreiben!
You are still using .32 kernel ????
Yannick, for me this one looks better : 
$sOutput = str_replace("\'", "'", $sOutput);

but I am not sure if your other problems also is fixed with that :
forum thread you wrote something about
-> "<a href=\"detail.php?id=7054\">Cisco_environment</a>",
We'll have to rework the structure, the DomainRecord is not really the place for a GUI element
Puppa vecio, appena capisco come si usa questo github caricherò qualcosa!
> Also replace Xs with &times;

pig disgusting
After writing the previous line I'd be dizzy, too ;)
Alright, I'll revert this.
Changing priorities didn't fix anything for me, only adding a delay to Linkify Plus did.
In cases where there's a small number of big boxes, the text stays tiny. Is this normal?
Jigoshop v1.0 replaces the quantity picker with a dash on all downloadable products, actually

Nice, though :). Not sure how the download limits could affect quantity, though. I don't believe we accounted for that in 1.0.
Too wordy.

If this is reusing existing strings and we don't want to re-work copy, that is cool.

@skinny probably has a better idea, but:

BrowserID requires cookies
Close this window and [enable Cookies](link to Cross-Browser SUMO article for BrowserID and cookies).
Can you please submit the patch upstream and link to the bug report in here? Thanks.
you're already checking this on [line 245](#L2R245)
There is an issue with no longer checking the pane.outerWidth()
By returning if this test is succesful one can end up in the situation where:
* contentWidth == previousContentWidth
And then you never get out of that, even if pane.outerWidth() does change.  
(because you don't update contentWidth to reflect that until later in the code)


So if you give the nested content a min-width that is equal to the panewidth, and then let it grow dynamically in width (but not in height): then the scrollbar never appears (while it should when the content-width goes beyond the pane-width) 

I'm working on a demo that shows this. Basically it is a horizontal variant of the vertical-auto-reinit sample.
Hey, tearDown() is protected by default in PHPUnit! Why do you changed its visibility? :-)
Text here should be an array and it should be declared as a field of the class because we need to take out the information from it later on.
Why this line? Do you wanna throw an error or something if statSync fails?
I'm nit-picking but I would prefer these to end in dot-log so it's obvious that they are log files.
You should implement the spaceship operator and include Comparable to get everything else. DRY FTW!

http://www.ruby-doc.org/core-1.9.3/Comparable.html
Nice, 2 thumbs up on moving the built-in CSS functions into the same matching as LESS's math/color functions.
If you want, send me a pull request and I'll merge in your work!
this test fails on ie8. The issue is that you can not create a script tag with content in it programmatically in ie. I spent some time on it, and just gave up and disabled the test for ie.
I added a hello.html page; not welcome.
Use the same header as in the rest of the Speedometer code (not all rights reserved, but Apache 2.0 license.)
To make completions for `bem-tools` available in your bash, run following command ( ensure that you have `bash-completion` installed, first )
Ideally, requiring `lib/sequenceserver.rb` from any where should setup the load path and load all the dependencies. And Bundler was designed to do just that. However, SS currently  calls `require 'bundler/setup'` (which asks Bundler to do all its dependency magic for SS) in `bin/sequenceserver`. It should be moved to `lib/sequenceserver.rb`. That will solve your problem too.
@IngCr3at1on We've done extensive testing on this using our build cluster. http is much more reliable. ssh protocol was extremely flaky and would cause sync hangs and failures across all the build machines.

http is also the preferred protocol of github. (I can't recall where I saw that post at the moment though)

What secondary remote fix are you talking about? I don't see that in the aosp ics manifest.
Sorry but this doesn't have the right aesthetic attributes to follow the current Etherpad style.  This commit provides little to no value to the end user so will not be included.
@steveklabnik interesting stuff!
Hey,
The main reasoning behind nesting etc is consider the following code:
```coldfusion
#a#
<cfoutput>
    #b#
   <cfoutput>
        #c#
   </cfoutput>
   #d#
</cfoutput>
#e#
```

#a# and #e# are not going to be parsed as hash variables so will be output literally... i.e don't highlight them
#b# #c# and #d# are all within cfoutput tags so will be parsed by CF - i.e highlight them.
without nesting #d# gets left out because the closing tag after #c# kills the cfoutput "scope".

It's basically functionality that I implemented in my Notepad++ ColdFusion Plugin early on as a visual distinction of being in a cfoutput tag or not (especially if the cfoutput tags are not within view), and one of the big things I've missed since using Sublime. 

There are still other places that would need it to be picked up, e.g cfquery.
Also, you need to update the version.php file (increment the $v_database value by one)
use xlt() function
Good first version - it'd be even more preferred to split on word boundaries, but this is definitely a good improvement so we stop sending invalid data to the server. :)
Oh hah, I was just testing out editing that there, didn't realize it was
actually sending it to you. It should be in both previous and next -- it
makes buttons, see "Buttons" under
http://twitter.github.com/bootstrap/#forms



On Monday, January 30, 2012, Gina Trapani wrote:

> What does the btn in the title do? And how come not in previous link?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/ginatrapani/ThinkUp/commit/e01021ab1131536554b3479fc089c375be7efbe8#commitcomment-915313
>


-- 
________________________________________
about: http://www.anildash.com/
im/twitter: anildash phone: +1 646 833-8659
este cambio lo realice para intentar subir los cambios que habia hecho en la rama keyla a master realice los siguientes pasos:
 me coloque en master
git checkout master 
git add .
git commit -m "cambio a readme desde la rama keyla"
git push origin master
In the short run why not replace 'class_name.underscore.gsub(%r{/?#{klass.model_name.param_key}($|/)}, '').gsub('/', '_')' with 'refinery' so the dashboard stops crashing.  The only test that fails when that is done is the one that expects edit_x_y_z_admin_y_z_path which is not the paths that are being generated so you are currently swapping passing tests for a failure rendering the activities.

I recognize the ongoing discussion, but it seems like until the decision is made making the test pending and aligning the code with the actual paths would make working with the rails-3-1 branch easier.

My two cents worth would be that the model doesn't belong before the admin i.e. 'edit_refinery_blog_admin_blog_post_path' rather than 'edit_refinery_blog_post_admin_blog_post_path'
@pksunkara No. The files being separate here is meaningful. Think: blacksmith.
double en-US
Hmm... this API prevents users from providing their own method sorter implementations.

How about this:

    public interface MethodSorter {
      Comparable<Method> create();
    }

Then you can define value like this:

    Class<? extends MethodSorter> value() default DefaultMethodSorter.class;
This doesn't work in the 'dist' bundle, each asset gives an error 500:

Caused by: java.lang.IllegalArgumentException: URI is not hierarchical
	at java.io.File.<init>(File.java:363) ~[na:1.6.0_29]
	at controllers.Assets$$anonfun$at$1$$anonfun$apply$5.gd1$1(Assets.scala:57) ~[play.play_2.9.1-2.0-SNAPSHOT.jar:1.0]
That's normal; it's not a bad thing. I guess it doesn't need to be shown
when compile is called implicitly; maybe the next version of Leiningen
could suppress it while jarring.
wtf.... 手误
not sure if you noticed or not, but status_graph.php hasn't worked 100% since your first set of changes i believe. Usually it would display the IP of the host with traffic measurement to the right, but it doesn't. It would with the original status_graph.php from an iso.
which version  or branch did you install?
iuiuiu
Well, it didn't help.
It's a lein issue: http://myminutes.wordpress.com/2011/12/01/clojure-lein-gives-error-exception-in-thread-main-java-lang-runtimeexception-java-lang-nosuchmethoderror-clojure-lang-keywordlookupsite/
`resource_managed` is a staging area, `classes` is the packaging area.
The output subfolder mimicks the input.
So to put your compiled file in public/stylesheets, move your source into app/assets/stylesheets
Those divs serve different purposes. The new _zoomDiv is the d3 technique of having an offscreen div element to simulate a scroll event and normalize the wheel delta value. The mouseDiv in my fix is a shield that sits on top of the map parent to give continuous mouse scroll events, otherwise mouse wheel momentum seems to get interrupted by DOM changes. Note that the `MM.addEvent` call for wheel events is listening on map.parent in the current version - that's what causes the glitches.

Basically it should be possible to "throw" the zoom of the map and let mouse wheel momentum do its thing, just like it does if you scroll a long web page.
Accidental double if
You are correct, I will merge this as soon as I can fix up internal projects that use the gem
I just remembered now, that you can connect to different driver with DB::setup( $string, $username, $password ), I'll cherry pick some change from the multiton version, and put back setup as public method:
https://github.com/rainphp/rainframework/blob/20ba62a61a5a943f7e839b3a149ed6ca2a1e1362/library/DB.php

Thank you so much for noticing that!
It is not the problem I encountered.

The problem is: 

* We want to disable midje in prod mode
* We can control that with `*include-midje-checks*`
* The idea was: set it to false in the entry point of the program (`main`)
* Because `main` is always used in prod, it works fine

But it doesn't work: 

* In prod mode the program load `main`
* But for `main` to work, it needs to load other modules (`util`, `core`, ...)
* And the loading occurs **BEFORE** executing the code in `main`, and this code contains midje unit tests

I think we need to check code examples, or ask Brian on how to do it.
Shouldn't this:

  create_remote_file agents, p,p 'notice("hello")'
 
be:

create_remote_file agents, pp, 'notice("hello")'
As you know, I've experimented with JSDoc 3 and think it is a reasonable alternative to ND.  I'm not hooked on that solution, but have a strong preference for going back to JSDoc style comment annotations.
Done, but with a few changes.
Same goes for this one
Mais attend donc le rdv avec le prof...
Sinon ça va faire comme l'AO, c'est à dire un dossier ou la moitié est hors-sujet...
In this case the pipes opened in line 256 don't get closed.
有没有查找文件的api啊
I didi it in branch: https://github.com/itteco/iframely-chrome/commit/c5646d0bd081980a58176bdd5d549f4a324d6243#L0R10
The class was deprecated and scheduled for removal from the Platform's
12.1 release.  It's still present in the CMS 2.5 release, so no worries
there for now.  For CMS 3.0, I think you'll be using JRegistry instead.

On 1/16/12 10:00 PM, "Matias Aguirre"
<reply+c-872114-2f3589db71ed9d3e0a494495394fde52a32cce69-368545@reply.githu
b.com> wrote:

>Hi Michael,
>
>I Have a question about this. Why the JParameter class is removed? What
>about if i need this class into the platform to manipulate params from
>joomla-cms?
>
>Regards
>
>--- 
>Reply to this email directly or view it on GitHub:
>https://github.com/joomla/joomla-platform/commit/0e70125bf18b5dc6c552114e1
>cac51dd7c089564#commitcomment-872114
You should rebase your work using git rebase origin/master instead of copying the parent pom.xml
Yeah, that seems to be an artifact from the period when skipping was [handled by signals](https://github.com/pculture/mirocommunity/blob/7a9333c9f51fca6f71bed9fe2ae765604981cdfc/localtv/tasks.py#L167) which checked internally whether source_import was `None`. The check wasn't transferred over when the signals were removed.
> That's *not* anti-git

YL, is this where "igittigitt!" comes from ... ? ;-)
ow darn well then ok, yeah ^^
Well, it's only going to be a pile containing maybe 7 cards by the end of the round. Being able to tell if there are three Crises in there shouldn't be _too_ hard. 

I also like that your injuries can 'flare up' now and then on later turns. If you WAKE UP with your Broken Arm and Punctured Lung cards in your hand, you're having an awful day, and you're going to seek medical attention and easy Locations.
BACKTRACE

(gdb) bt
#0  0x0000000000000000 in ?? ()
#1  0x0000000000939d35 in Unit::_UnapplyAura (this=0x7f7375b20000, i=@0x4c65de40, removeMode=AURA_REMOVE_BY_DEFAULT)
    at /home/user/tc_source/TrinityCore/src/server/game/Entities/Unit/Unit.cpp:3322
#2  0x0000000000bd03ac in SmartAI::RemoveAuras (this=0x7f7375ab7640)
    at /home/user/tc_source/TrinityCore/src/server/game/AI/SmartScripts/SmartAI.cpp:441
#3  0x0000000000bd08b5 in SmartAI::EnterEvadeMode (this=0x7f73c7cbe860)
    at /home/user/tc_source/TrinityCore/src/server/game/AI/SmartScripts/SmartAI.cpp:462
#4  0x00000000009436a2 in Creature::SelectVictim (this=0x7f7375b20000)
    at /home/user/tc_source/TrinityCore/src/server/game/Entities/Unit/Unit.cpp:13016
#5  0x0000000000bd18c5 in SmartAI::UpdateAI (this=0x7f7375ab7640, diff=<value optimized out>)
    at /home/user/tc_source/TrinityCore/src/server/game/AI/CreatureAIImpl.h:563
#6  0x000000000088c797 in Creature::Update (this=0x7f7375b20000, diff=232)
    at /home/user/tc_source/TrinityCore/src/server/game/Entities/Creature/Creature.cpp:522
#7  0x0000000000a506f7 in Map::VisitNearbyCellsOf (this=0x7f73ac86d000, obj=<value optimized out>,
    gridVisitor=@0x4c65e020, worldVisitor=@0x4c65e010)
    at /home/user/tc_source/TrinityCore/src/server/game/Grids/Notifiers/GridNotifiersImpl.h:44
#8  0x0000000000a55c55 in Map::Update (this=0x7f73ac86d000, t_diff=232)
    at /home/user/tc_source/TrinityCore/src/server/game/Maps/Map.cpp:534
warning: (Internal error: pc 0xc76820 in read in psymtab, but not in symtab.)

warning: (Internal error: pc 0xc76820 in read in psymtab, but not in symtab.)

#9  0x0000000000c76821 in MapUpdateRequest::call (this=0x7f733543ca00)
    at /home/user/tc_source/TrinityCore/src/server/game/Maps/MapUpdater.cpp:54
#10 0x0000000000cf3601 in DelayExecutor::svc (this=0x7f73bb69d540)
    at /home/user/tc_source/TrinityCore/src/server/shared/Threading/DelayExecutor.cpp:52
#11 0x00007f73c93fea87 in ACE_Task_Base::svc_run (args=<value optimized out>) at ../../ace/Task.cpp:271
#12 0x00007f73c940015b in ACE_Thread_Adapter::invoke_i (this=<value optimized out>) at ../../ace/Thread_Adapter.cpp:161
#13 0x00007f73c94001e5 in ACE_Thread_Adapter::invoke (this=0x7f739fb2acf0) at ../../ace/Thread_Adapter.cpp:96
#14 0x00007f73c7ccafc7 in start_thread () from /lib/libpthread.so.0
#15 0x00007f73c7a4064d in clone () from /lib/libc.so.6
#16 0x0000000000000000 in ?? ()
`g//d` should mean the previous pattern. I'll merge your change.
I can confirm that this fixes the error with running import/test_hiddenimport that I mentioned in [#503](http://www.pyinstaller.org/ticket/503) on windows, ActivePython 2.6. I don't have CPython 2.6 to test against though. Cheers.
I'm not sure how SourceTree works.  You can check to see if you have a .gitignore file on  your local computer, though.  If not, can you "pull" changes from github?  On the command line, you'd just do `git pull`
(Y)

--
Nando Vieira
http://nandovieira.com.br



On Mar 5, 2012, at 9:10 PM, Rafael Mendona Frana wrote:

> Done.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/plataformatec/responders/commit/160a074571ee39e4454164e275f20b0df70ba0ed#commitcomment-1045527
thx, merged
Ya, the purpose of putting it inside the loop was just in case the files changes while the script is running. What would be the advantages of running it outside the loop? Just efficiency ?
What is the point of accepting 32bit integer samples (when INTEGER_SAMPLES is defined) when soundtouch has to convert them to 16 bit anyway? Why not let the BitDepthAdapter do the conversion and remove all conversions from SoundtouchEx? SoundtouchEx need only do the interleave/deinterleave which could perhaps be optimized if it only moves data around instead of also converting them on the fly.
Yay! As a fan of shoulda-context, I'm really happy someone took over maintenance! Thank you Travis!
not used, please remove
```python
print "3.",

lastnum = (listtotal-1)+2
print lastnum
for num in nums:
    print nums
```
Is not how you remove the last element.

`nums.append (5)`
Putting a space before a `:` or the `()'s` is highly discouraged. Don't do it, I'm surprised it worked.
Michael mentioned separating dataset into data & set...I prefer the single word, happy to refactor.  Agreed on the 'find_' - would we want them for data_source and external_dataset as well?

Also - there are other functions (i.e. get_dataset_handler) that may or may not belong in this service - something to discuss tomorrow or next week.
Also, when the tests are ran a...file? Or something? called 'localhost' shows up in the project root. It's very bizarre.
ruby 1.9.x doesn't downcase non ASCII chars

``` ruby
1.9.3p0 :005 > "ХОЧУ ВСЕ МАЛЕНЬКИМИ БУКВАМИ".downcase
 => "ХОЧУ ВСЕ МАЛЕНЬКИМИ БУКВАМИ" 

1.9.3p0 :006 > "ХОЧУ ВСЕ МАЛЕНЬКИМИ БУКВАМИ".mb_chars.downcase
 => "хочу все маленькими буквами"
```
Any chance this could get pushed to rubygems?
Thank you for your contributions!!! Keep it up!

Do you think this is ready for a version bump + new release?
I'd drop all this.  Anything that doesn't have a `@classmethod` decorator should implicitly be considered an instance method.
Artem,

A pull request would probably work if you made the "FOR UPDATE" optional 
(so it probably needs to be exposed in the namespace).  Testing is a 
problem still, but there are some integration tests that you should make 
sure are OK with all the platforms you can.  Also try with HSQL (to make 
sure that you can switch off "FOR UPDATE").  Mark Fisher will decide on 
the pull request ultimately, but I think it's a good idea if it isn't 
changing existing behaviour by default.

Dave.

On 24/02/12 12:08, Artem Bilan wrote:
> Dave, I understand your opinion.
> I've just tested it on Derby and Oracle with PROPAGATION_DEFAULT&  READ_COMMITTED without FOR UPDATE.
> My Oracle 11 sayd that it support only READ_COMMITTED&  SERIALIZABLE...
> However other my changes (without FOR UPDATE) for **JdbcMessageStore** are still there.
>
> What do you think now? Is it make sance to make PR on this?
> Thank you for your time.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/artembilan/spring-integration/commit/d5af04c225bf33c782e90ae2570c5fcab9aa7b78#commitcomment-1007158
>
Deleted was unused
reply...
No magic, it appears to be an error, there should be a for loop to create terms and pass it on the to taxonomyManager.  Thx for noticing.  I will fix it this week.
It should be ofDrawGridPlane?
wrong commit name!
Manque un point à la dernière phrase. O:)
With this and the following lines you have lost 1g, at boot it is 800, if you set it to 1g in setcpu it gets set to 800
#bikeshed that is the single most important "." in the history of ever;
I never did find the actual root cause but I found another curb issue with the Couchbase client regarding complex keys and views as well:

http://www.couchbase.com/forums/thread/passing-complex-key-ruby-client-20-preview-3

We've moved away from Couchbase for now. To me, it didn't seem right that you should have to set the resolve_mode in your client directly hence my feeling that curb might be the culprit here.
Is it `arg<0>` or `arg0`? It's unclear from both the generic structure used both in the JavaDoc and this sentence. Personally I prefer `arg0`
That change is from another phone,
Is this an extra s?
many thanks , i try it
I could add a message to do that.
Just a note, I couldn't run the rake task as I got `no file to load - rdoc/task`
just as a tip, y ou can use the `done()` callback instead of providing a high number of assertions. For example, see here: https://github.com/ded/sink-test/blob/master/tests/tests.js#L74
Ээ ну да, про первую я ничего не говорил - там остается =)
I also removed this `Y.UA.ie` sniff, and instead only set `withCredentials` (a CORS feature), when CORS is supported. This way if an older IE is using `XDomainRequest`, it won't try to set `withCredentials` on it.
I believe that the resize event is throttled under the hood already.
Can the validation be simplified by checking if `$groupname == elgg_get_friendly_title($groupname)`? Is that the only check we need?
Sure, I can. Though code conventions talk about *testcase extending junit.framework.TestCase*. So I will change that also...
Missing copyright header block.
I think I see what you mean. At the time I did this I was worrying about trying to ensure that the PETSc4py defaults got used, but it actually makes more sense to enforce our own defaults on the solver so that it will be consistent across backends.
Then the stash is empty already.
eliminates this
the better question would be where RichInt can actually come from? Is this not a little over-designed?
I'd prefer an algorithm that instead of creating strings and caching them compares the Addresses first (falling back to the selfAddress) and then compares the path-names.
Ah, the good old `if (...) always (...)`
* @param maxNrOfRetries the number of times an actor is allowed to be restarted, negative value means no limit
 * @param withinTimeRange duration of the time window for maxNrOfRetries, Duration.Inf means no window
 * @param decider mapping from Throwable to [[akka.actor.SupervisorStrategy.Directive]], you can also use a
 *   `Seq` of Throwables which maps the given Throwables to restarts, otherwise escalates.
 */
case class OneForOneStrategy(maxNrOfRetries: Int = -1, withinTimeRange: Duration = Duration.Inf)

Since you use defaults you're saying unlimited restarts, but that's not true, since actors are either stopped immediately or escalated (which in it's turn stops the children, right?)
shouldn't this be `apply` in `Metric` companion instead?

    object Metric extends MetricNumericConverter {
       def apply(name: String, value: Option[ScalaNumber]): Metric = ...
Hm... may be this method should be moved in the FILE helper?..
So here we're testing that unauthorized users and non-sysdamin users can't create other users? Does this have something to do with organizations?
At very least, the verbiage on the help is wrong. It's not outstanding bindings, it's in progress binding-related actions (not saying to use that phrasing, just mentioning that the bindings themselves aren't outstanding if it's an unbind action that's taking place).
Eh, nothing looks right. If the user doesn't know what an underscore or a hyphen is, we probably don't want them using Pulp in the first place. Removing entirely.
Appreciate you fixing these when you come across them. There's a massive brain fart of these all over the CLI.
huge chunk of these changes are in js that really should be in a gem (or otherwise somehow ignored here)
oof, I see you made half of the cases require this behavior and missed several others.  this will break "deny" for example, which does not have card errors.   please review every instance where "errors" is called and be sure there will always be errors on @card.
should do the same with Resolver
This might be better implemented with a case class. apply (and unapply) would be implemented, and it might avoid using implicits
import order ++
these tests should apply by passing the corresponding schema
If the path exists and is a folder, then you still ought to do a `git fetch -a origin` in it, so that you pick up new refs.
this is too optimistic
we can't assume that all ways of entering text are covered (e.g. undo from menu, or customized keybindings)
so it's better to check start and end of the string to not add \x00 into document
Did you test this? As far as I see the second instance of your name should refer to your commit author and so should be your old name.
I assume you just meant the console.log's? I'll remove them in the next push.
yep... but this way you get a nice diff in the test log, allowing you to see what went wrong easier.
Rather than checking the message name, could we do this by checking a property on the event, something like "showingOnly"?  Then modify the source of the onresize/onpostresize to include this in the event structure.  This would make it easier to add more events like this in the future and also make this mechanism available to devs.
duh found it already, sorry
We should pass the metrics object to the callback as a parameter and not rely on a global ref.
why? its equivalent
This needs to be prefixed.
New code should not synthesize properties, since it's no longer necessary for the compiler.
Don't use `ntohs()` here.
Another thought:  If loading DLLs is platform-dependent, then maybe we ought to have utility classes, one per platform, that are responsible for loading and unloading libpd.  For example, we might have LibPDWin32Loader.Load(), which loads the DLL via kernel32 and initializes libpd.  Having just one loader for starters is fine.

The upshot is, I just don't like the idea of an API that might leave the library in an inconsistent state.  I want one call to load and initialize, and one call to unload and release.
Fixed and updated commit. Thanks Paul.
Actually it might be good to modify the standalone deploy client to pass an exit code. Maybe in a later pull request.
Maybe throw an IllegalArgumentException ?
перенести к остальным полям в конец класса, добавить префикс m
Looks like you attempted to use a tab instead of spaces for the leading whitespace on lines 107-112.  The indentation appears off.
Also why not use the `include_content_headers?` method?
New CHANGELOG entries go to the top :)
Anything with `message.prototype.* = ` are class properties. They're available without initializing the object.
Can't you store the static symbol somewhere? I'm worried linear search might be too slow, plus it's comparing strings, which isn't fast either. Whenever the static symbol is created, you could maybe attach it to the original.
I don't like the way constructor bodies blend seamlessly with member definitions and intersperse arbitrarily; sometimes I use locally only to bring emphasis to the fact that a constructor exists.
Add a space after each comma.
Add a full stop at the end of the sentence.
sttart += step

stop -= step
`@since` has to be 1.1.11 ;-)
same for dropPk...
Neet :D Didn't knew that!
Nitpick: the word "immiscibility" is probably better.
Yeah I'll get back to this.
Having a separate module would not prevent module-level importing of werkzeug.

You should make an initial check and only import werkzeug when a http_route is defined.
>= now means the self.retry_timeout hasn't occurred yet
now = 1334087011.951727
retry_timeout = 30
marked_down_at = 1334087001.951727 (10 seconds ago)

marked_down_at + retry_timeout == 1334087031.951727 (20 seconds from now)

I originally had it >= now and it broke the retry tests... I had to do the above to figure out what was wrong
To be a little more refined on this test, what's important for `connect()` is that it's called with the values of `self.port`, `self.host`, `self.prefix`, etc.  So you might want to write the test more this way:

``` python
_RiakClient.assert_called_with(host=self.conn.host, port=self.conn.port, prefix=self.conn.prefix, mapred_prefix=self.conn.mapred_prefix, client_id=self.conn.client_id)
```

This helps your test be more expressive on what the actual behavior is, and helps insulate your test against unnecessarily breaking when things change.
Another personal preference, but I would have defined this as an inner private (and static) class in the test class, to reduce the surface area :)
Use an on-stack variable, easier to reason about.
I think in this scenario, the data points are best specified using theories rather than with the test helper.
I'm still working out the best approach to manage Dev/Deployment with this. The uglifyJs significantly reduces the size of the payload, but makes debugging in dev impossible. I'm hoping to find a solution where the agent run under jetty:run has unminified resources, but when packaged up will compress and minify them all.
still very ugly :|
The ```// <![CDATA[``` should be inserted here? And ```// ]]>```before ```</script>``` ??
Should probably use the `sql_create_table` method in `phpbb_db_tools`
This code is from another pull request and not related to this one. Please make sure you work in branches.
Not sure `sql_inject` is the best name here. While technically correct, it is easily confused with the sql injection attack.
Use multiple lines.
Instead of returning when template_var is empty, we should throw an exception if template var is empty and return_content is false. As that is an invalid method call.

Also, I would generally use a comparison operator instead of empty, because empty implies an isset which is not needed here.
How much of a performance gain are you expecting to see by moving these 35 queries into a transaction?
When would this condition be true?
好的，这里修改一下
Can you also check that it's a valid sector id (ie SmrGalaxy::getGalaxyContaining() !== false), just so we don't give links that will try to plot outside of the universe
Just deleting unused stuff. I couldn't help myself.
`npm test` will now just run the unit tests. We can debate whether it should also run `mojito test` as well, but I feel that it shouldn't run the functional tests since they take around 20 minutes to run.
Ну не место загрузке файлов в Dao. По-хорошему контроллер должен был вызвать специальный сервис, который бы все проверил, сохранил картинку и вернул бы её урл, который конроллер записал бы в объект баннера. Плюс у меня мало опыта в написании зпгрузчмков картинок, я не уверен в безопасности этого кода. Может быть стоит загрузку картинок отдельным pull request?
Need to declare `listTabs` as a var to avoid hoisting.
EmailLogger
Plus button should not be part of the repeating form (currently, a new "+" button is created each time your press "+"). Instead, put "+" button along with drugSubmitButton
Yes! I get it. Thank you. :-)
[10, 20, 30, 50]
border-width: 10px 20px 30px 40px;
is this variable used?
Yes, there is no other place to use it.
I'm still curious if I can mix a replicated read with a partitioned read and get detected as single partition. Am I missing something or is that case not handled?
To avoid confusion, use a more local variable (than "verb_args") to shuttle arguments from parse_args to process_verb_arguments.
It's never the same verb_args that gets passed to ParsedCommand.
To be useful, this check should come before m_index->is_countable_index_ is accessed.
Maybe reopenClass?
Наверное, нужно возвращать массив из одного элемента?
You can name the method - searchForJobsByCategory
It should, but when the table was converted to sqlite3, it's looks like the A and Z column labels were transposed. I agree that the table should be fixed to properly label the columns. This fixes the immediate problem in which the numbers actually retrieved are incorrect. Inspect the mass.sqlite table for details.
nit: Could we move this above parseCaseLabel just so they are closer together?
Well, I had a check for { first, but wanted to make it as explicit as possible. I'm open for "as simple as possible" as well. :)
Can you add a Before/After example for a typical catch phrase? 

    catch(PDOException $e)

After:

    catch(DBALException $e)
To minimize size, I sorted all the class fields by descending alignment order, so to keep that going, these two fields should probably go to the end of the object.
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 here you. Any suggestions on how I might detect that a browser has no vendor prefixes without browser sniffing?
`Server` sets `@inprogress = true` internally when doing IO; we should probably be setting it here as well.
It is OK for formulae to clash.
it would be great to define "query_params" in a doc block here and elsewhere in the class.
was passiert hier, falls ein File mit encoding != 'UTF-8,ISO-8859-1,WINDOWS-1252' auftaucht?

müsste ja wenigstens ne Warnung geben oder?
If there is a reason this cannot be part of `may_view()`, please indicate why in a comment
Any reason this changed from `elif op == 'preview'` to just `if`?
While we're still on this topic, why does psycopg2 still need to be a requirement? Syte itself doesn't use PostgreSQL anywhere, does it?
@tfwright Not sure how I would access the client without adding an ivar to get at it
I don't know what's wrong with my IntelliJ but it kills a lot of white spaces. And the diff show a lot of thing that git doesn't show me... This is the best I can do.
Let's take off "and will be removed in the next release".
:)
I like using ternary operator, but many people do not. :(
Yep, I think it makes sense.  It means you can share the schemas between tests.  Though I think at present it'd parse the files each time, but that's not too a big deal for now.
this can probably be a foreachlimit of 10 or so, parallelism here is fine.