labofsahil/github-event-dataset-2012 huggingface dataset:
Refactor passphrase

Instead of building a hash from a source file containing index and word separated by spaces this new version builds an array of words and randomly selects elements from it.
    
This will make it easier to modify the list of words since indexes don't need to be maintained by hand.

This has the added bonus of being able to splice from the array, thus guaranteeing that we don't pick the same word more than once in a single passphrase.
Some fixes and updated strings

Changed source-language to "en" to match sfSimpleForum.fr.xml (not sure if it was something wrong)
Fixed a bad closed target at line 148. 
Updated the new strings.
Add overlay to control the behavior when the user long presses the power...

... button for all dpi settings.  Setting is taken from overlay/frameworks/base/core/res/res/values-sw600dp/config.xml by keyodi; is there a reason it wasn't here to begin with?
Added Android.mk to build audio on aosp

I added the missing Android.mk which causes the makefiles in subdirectories to be built for AOSP.  This then build audio when building for the device.
o Add support for plugins contributing custom form fields
timob-10095: KicthenSink: Phone > Sound > Remote URL: touch events queued, resulting in opening multiple windows

http://jira.appcelerator.org/browse/TIMOB-10095
Test case in JIRA.
Moved config def above 1st usage.

config var used in redirect wasn't defined
Please Review: Issue #72: Add organization attribute to Task
Improve pom.xml, add content to README.md and License header

+ use latest Maven Plug-ins version
  + add content about requirements, developerment, and quick start.
  + add details in pom.xml
  + add TODO
  + ignore .settings and .project
  + add ASF license header in source files.

    please execute following command before you push to remote repository.
    ```
    >mvn license:format
    ```

    better, configure your IDE or Text Editor to *always* include ASF License
    located in etc/header.txt for every source file.
added instructions for installing with pip

easy installation instructions for pip
leontieff typo fix
squarify profile image. thanks stephen for the edit page.
about.html

I added some content to about.html for the page.

Please review and make edits as necessary.
Add syntax highlighting for the Opa language.

This plugin is directly based from the official Git repository of Opa.
Fixed typos.

Boo, typos!

All the best,

-HG
QA bug fix + Updates
grab this
Moved to new version of Module Base to improve syncing
Current code!

Current code! I'll explain it in more detail than last time when we meet up again
Try 2 - Promoted Build Cleanup, Audit trailing 

Formatting changes per request. :-)
Fix overwrite attribute
Modifs relecture
Updated council offices.

This should be an updated version of the counciloffices module, made a few tweaks from the last pull request. The module is based around what a couple of people said in the Enquirer that they wanted in the council offices.
Now that I think of it...maybe because I don't have enough experience :)
I only wanted to cleanup any unused code, therefore as you said I deleted the code that doesn't get used from my target.
Why do you do comments { like this } and not // like this?
I tried to change "vector.pxd" (because I am using it in the example) like this:

cdef extern from "<vector>" namespace "std" nogil:

and got:

Error compiling Cython file:
------------------------------------------------------------
    with nogil:
        for k in prange(m):
            if (<int> lstMatchinglist[k].size()) >(<int>20):
                                            ^
------------------------------------------------------------

feature.pyx:343:45: Calling gil-requiring function not allowed without gil

It is probably worth fixing it elsewhere ... but this is completely out of the scope of my knowledge.
No, it should only show the overlay if it is not already shown, this is the case if you're the first dialog to display.
This one will break if `%JAVA_HOME%` isn't set. It would need to check for the presence of the variable, and then assign `%JAVA_BIN%` to either `java` or `%JAVA_HOME%\bin\java`. If you can do that, it gets in :-)
Like in fmin_tnc, doesn't splitting f, g = func(x, *args) into two function call double the number of function evaluations in cases where func returns a tuple of f and g?
@indutny, @isaacs: Isn't that way too high if you're worried about hash collisions? It's over 500K operations per request if I remember my math right: `n + ((n * (n - 1)) / 2)` where n=1000.
@pboling, That repo is dead, the nearest thing I could find is https://github.com/JackDanger/simple_memoize .
Thanks! :) :heart:
And now people, LEARN how a good company behaves during such tough times. Please, read the official story of what happened:
https://github.com/blog/1069-responsible-disclosure-policy
very sorry for very long Duration time.
(zf2 don't recommend run tests All components at once, so I run each component tests as above)
Thanks, updated (koichik/node@75109c8).
trenger i lenger ned
I have tested the above patch and it does fix the logout theme issue but suspend and hibernate option are now missing.
Oops -- missed this before check in.  This change could happen any time.
the loose-not-equals is intentional here, because it allows checking for null OR undefined. making strict !== breaks the check. another problem i have with blindly following linters.

the proper check is much longer (which is why i avoided it, because of file size):  `if (typeof script !== "undefined" && script !== null) {`
Нужно переделывать!
I've pushed this by mistake, but why I did it is that for the project that I'm working on I'm using some custom tags. With that condition in place I can't add other tags without editing _coreClasses in Liquid, and I wanted to keep the library completely independent from my project.
aaaah... we will need someone more "artistic" to choose better colors
On Mon, Jan 9, 2012 at 6:20 PM, Cory C. Sitko
<reply@reply.github.com>
wrote:
> Was giving me annoying circular import issues that I didn't feel like trying to solve today.

Gotcha, no problem.


-- 
Mike Pirnat
mpirnat@gmail.com
http://www.pirnat.com/
Well, it's not wrong, but the krb5_config_get_string_default() call with statfile == NULL is wrong.

Also, a bit below we try to fopen() fn without checking if fn is NULL.  So the right thing is to return NULL if the asprintf() fails, and to return NULL if the krb5_config_get_string_default() call returns NULL.
oh, yes, it should be
ooh i feel bad for you because if you just text that 12 hours ago mmmmmmmmmmmmmmmm $
I wrote it.  If it needs a license, it's 2-clause BSD.  I should probably give it another look though if we may be looking at merging it in, as I haven't actively been using it.
We should either change the logged message to be "Unable to extract or index…" or else move the `index()` call out of the `try`.
Yes, Apple setting that value at install time would be nice. :)

But what do we do until that happens? How many people will see the README before they install MacRuby?
@richarddong 
不能，你是说命名的问题？
this is a bit of a hazard as we'll soon be ridding propertiesbuilder and moving to straight properties via rocoto.  thinking there might be a better place for this.. perhaps in the image cache? \@PostConstruct workingDirIfNotExists() or something
Ok, commit 44d8ebaece97f1744b9490cebd29b1504a4d5451 should fix this. Using a blocking call to processEvents() instead of spinning endlessly.
is the modified damage for dark evangelism working? didn't seem like it
so as I understand now `$class` can contain `array(null)`. Not nice
doesn't the bootstrap css come from main.html?
default settings - sufficient for novices. if need advanced - learn cmake.
Yes.  Tony, make sure this doesn't get merged into "master".  Might need to create another commit on the branch that removes this file.
Cool #roadblock, bro.
That is intentional. Strong naming is required by anyone who has the
requirement for strong naming (which is common in many regulatory systems).

On Mon, Jan 2, 2012 at 2:59 PM, Pedro Lamas <
reply@reply.github.com
> wrote:

> Ok, just one question: why did you sign the Silverlight projects? Adding a
> SNK signature sets a Strong Name to the assemblies and that can become an
> issue in Silverlight because we can't use Assembly Binding Redirection (and
> Publisher Policies isn't avalable also)...
>
> Lets say that I have a project that uses two libraries, one was built with
> RavenDB version 1.0.0 and the other with version 1.0.1... if you have
> StrongName in the RavenDB Silverlight assemblies, how can I have this
> scenario?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/ravendb/ravendb/commit/dabb10dd787e1ecfaba51e67ee8b875356a2738c#commitcomment-831231
>
Why is this mysterious macro being defined on Mac OS X? I can't think of a reason, but I am sure you had a good one, but shouldn't this then be explained in a comment?
Hold off until redo of analysis.
Exact, on devrait même avoir un warning je pense. Bon il suffit d'ajouter les arguments pour les fonctions. Je te laisse faire un ticket ?
Let's check redirect url and :alert/:error existence.
Mostly because I was looking through the commits and saw something where you had put in "allow support for alternative hero storage
methods in the future." Thought this was as good a place as any. So technically... no reason in particular.
<code>result</code> wird in der Methode nicht weiter benutzt, sehe ich das richtig?
I'll give you a head start ... tied up tonight. ;-)
pcm_start() is also slightly more complex than just _PREPARE/_START now, it contains some code related to mmapped buffers. However, since we don't support mmapped reads, I think this is OK.
I've removed the fields. If anything fails now, it should be very obvious why.
`for(in)` loops are pest, shouldn't really be used for iterating over arrays. 

Performance is terrible compared to regular `for(;;)` loops and also if you have some custom property on the array, it will fail.

Plus you can get different order of traversal in different browsers.
CollectionCount objects don't have an addon on them. The other models do, but you don't really want to do counts on them tbh.
i thought it was there, it is in all my other ignore files -.-
Sending a ß with newest build:
12:07:04 [INFORMATION] <drantor> ▀
>version
12:07:13 [INFORMATION] This server is running Craftbukkit version git-Bukkit-1.0
.1-R1-73-g8fee20a-b1763jnks (MC: 1.1) (Implementing API version 1.1-R1-SNAPSHOT)
This *Should* make it ready to handle the actual command
Any particular reason why you didn't use the [tempfile](http://docs.python.org/library/tempfile.html) library?
schick
I don't think it is necessary to have a constructor here.  Just rely on super's constructor.
Hi Chee Seng,

I have looked again, and indeed you are using the 'wrong' Symbols and
Types. The compiler types are a bit special: they use the so-called
dependent types, meaning that they are members of the compiler object.
Therefore, once you have an instance of the compiler, you can access them
like compiler.Tree, compiler.Symbol and compiler.Type. You can even import
all members of the compiler object, whenever the compiler is in scope:
`import compiler._`.

In your case, one way to make it work is to make the TestLauncher take a
compiler instance as a constructor parameter and import it immediately
afterwards:

    ScalaTestLaunchShortcut(compiler: ScalaPresentationCompiler) ... {
       import compiler._

       def getParent(t: Tree) ...

Later, you can instantiate this class at the point where you obtained an
instance of 'compiler', and call methods on the object you instantiated.

Let me know if this helps.

iulian

On Mon, Feb 6, 2012 at 3:36 AM, Chua Chee Seng <
reply@reply.github.com
> wrote:

> Hi Iulian,
>
> Thanks for reviewing the code!  I have rewritten the getParent method using
> the method 'includes' that you mentioned, but I have strange compilation
> problem (see line 79 of
>
> https://github.com/cheeseng/scala-ide/blob/master/org.scala-ide.sdt.core/src/scala/tools/eclipse/launching/ScalaTestLaunchShortcut.scala
> ),
> I am not sure about the cause, may be the compiler is confused about the
> type?
>
> Thanks!
>
> Best Regards,
> Chee Seng
>
>
> On Mon, Feb 6, 2012 at 3:46 AM, Iulian Dragos <
> reply@reply.github.com
> > wrote:
>
> > This could be done much more efficiently if you used the position
> > information of the tree you're looking for. Positions have a method
> > `includes` to test if a position is fully included inside another. That
> > would guide your search in this case. Otherwise, you are blindly
> traversing
> > the whole AST. Have a look at class `Locator`, as I mentioned by email on
> > scala-ide-dev.
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/cheeseng/scala-ide/commit/2c7150b4d1c2437deeb9d4b02066ce6274040024#commitcomment-934642
> >
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/cheeseng/scala-ide/commit/2c7150b4d1c2437deeb9d4b02066ce6274040024#commitcomment-935106
>



-- 
 Je dteste la montagne, a cache le paysage 
Alphonse Allais
pid() or atom() where atom() is the registered name of the process
Nice! I tested out the Hubway a bit - looking great! Thanks Susana!
сомнительная практика. по поводу db:migrate ещё промолчу (хотя нигде не встречал подобного) но тесты не должны зависеть от первоначального наполнения бд
Here's some benchmarks to justify this approach for sorting in descending order.

[Benchmark script on JSFiddle](http://jsfiddle.net/aYQyP/2/).
PropertyTree xpath is actually pretty powerful.  You could probably do it all in one by using xpath = "Software/*[//@computer='%s']".  If that works it will be a smaller set because for instance you would probably get one RoxieCluster back when any node was on "@computer". 

I don't really like relying on the buildset much, I would prefer checking streq(pComponent->queryName(), "MySQLProcess").

So that means you would iterate over 
xpath.appendf("Software/*[//@computer='%s']", name);
it = pEnv->getElements(xpath);
ForEach(*it)
{
   IPropertyTree &t = it->query();
   if (!streq(t.queryName(), "MySQLProcess") && !streq(t.queryName(), "LDAPServerProcess"))
   {
         itsAKeeper=true;
         break;
}
The gunship battle crash when the other boat start to move why? :0
Maybe we should make this standard…
The problem with above setting is "branch[last] = obj", it make the tree as well as the path structure set with same object reference.

If you go ahead add additional properties to the schema to that path later, this affects the schema definition object as well.

I have a scenario, where i define a custom schema type. The custom schema type, adds some properties to the schema using a plugin. Due to the above change, when i add more properties to the schema, this changes the options also with which the schema-type was originally instantiated.

If this is the intended behavior, during the time when the schema-type is getting created, the options should be cloned.

Example :
new Schema({
  a : { type : 'custom-type' }
});

In the plugin, for any custom-type definition, certain additional properties are added
schema.add({ prop : 'string } , path);

At this point, the schema definition option { type : 'custom-type' } is also getting modified with the properties that are added. Which is a side-effect due the above change, where the references are shared.
Because the above change assumes a : { type : 'custom-type' } as an embedded definition, but it just a regular type definition.
1. Hast das mit dem Patch hier nichts zu tun.
2. Hab Ich das so noch nie in einem richtigen OS Projekt gesehen. Du?
Did the things you said!
What is sense of that?
Horen deze niet asynchroon?
4) - that's we i called jasmine tests  'integration' ones. You don't run integration tests frequently, so they may be slow,

* "test that generated file matches fixture" - we can test them comparing with fixtures in unit tests (matchers)

See you on skype :)
Shit it not always intentionally, but mistakes happen, and if we could protect users, why not.

But this is rather cosmetic than really useful.
Could this be shortened to: `@saved_env.each_pair{|k,v| cur_env[k].should == v}`
Nice work, Andy - much clearer, and great to have the JS all away with only an init to launch. I know this kind of stuff is not fun in OpenLayers!
Bug 1 fixed. It is mentioned in the CHANGES.
I was unable to save images from the repository. 
Status : fixed
distance_of_time_in_words is used in last_comment_date
this doesn't make sense
You're right, it must be false by default
thx!
ups, it must be if(areaEnabledIds.empty())
Since iOS 5, rendering the partial image in a bitmap context is no longer needed.
ah ok, you did the same error as I did in noir plugin :)

you should rebase before pushing ;-)
Hmm... maybe you're right. The problem is, that the activity doesn't finish for me when I abort it or an exception is thrown.
Hi Nathan,

Ive upload a few changes I been working in during the morning (please disregard the old FieldValidator class that is still there).
I haven't look at the widget thing yet.


________________________________
 From: Nathan Jones <reply@reply.github.com>
To: rgonzalez72 <rgonzalez72@yahoo.com> 
Sent: Wednesday, 1 February 2012, 13:06
Subject: Re: [pytrainer] Working on issue 81 (4234291)
 
Yes, a class hierachy is the OO alternative to a type code.

For each field that needs to be validated you can just instantiate the appropriate type of validator. The validator can probably be made capable of generating sensible localised error messages.

My thoughts on a custom widget were to create a new abstract subclass of gkt.Widget which wraps another widget and provides validation state and validation events. Then there will be very minimal wiring together for the UI - there just needs to be the same validation state change listener added to each widget which will add or remove an error message from the UI. As you point out though, either there will need to be multiple error messages displayed at a time or, while errors still exist, previous error messages need to be restored after errors are resolved so this approach requires a bit more thought. Maybe the validation state change handler needs to iterate over all known inputs and revalidate them when one of the inputs validation state returns to "valid". The Eclipse IDE presents input validation messages in this way (one message per screen, updated as you type, relevant to the earliest invalid field on the screen) so it could be worthwhile looking
 at the source code to 
see what pattern is used there.

Whether we use custom widgets or not though there will still need to be the string input validation type hierarchy so I suggest going ahead with that and seeing how it looks.

--- 
Reply to this email directly or view it on GitHub:
https://github.com/rgonzalez72/pytrainer/commit/4234291653c4447c5cae56a45c1844f93798eb37#commitcomment-922247
Ok got it. I'll try to rework that  out and make a new pull request as soon as possible.
Err. Think I'm still learning git!
We will be correcting it in the next or so. But thanks for a keeping an eye out on these things
thread = CreateThread(NULL, 0, playlist_stop_thread, this, 0, NULL);
hard (that's what she said, I know)
I remember reading somewhere that distributed notifications arrive on the main thread. Maybe that'd fix it?

Cya,

	David

On 27 Jan 2012, at 15:46, Dustin Rue wrote:

> It was a "debug" sleep and I've been going between two different machines so I was moving around using git.  
> 
> Here's the deal, I'm losing this battle with distributed notifications because the even though I'm trying to use a singleton that should be on a background thread it never actually receives the response so long as I'm still in limitedOptions.  The moment I leave that method the responses come in.  So in the current state, if you have Tedium running and configured with at least one destination and then you load up the new action screen it will be empty the first time and then contain data if you run it a second time.  As of right now, I'm not sure I can make this work.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/dustinrue/ControlPlane/commit/929a5ae1b8fac266cfeeb2b789264a38f47d21d2#commitcomment-908280
It’s the lowest common denominator. I regularly `ssh` into remote machines where I cannot install new shells. So for me, it’s easier to just use Bash, as it works everywhere. YMMV.
Ok, my last round of changes on googlecode get ride of any platform-specific "ifdef".

2 things have changed that should be taken in consideration by porting code:

- audio_update now needs a sound buffer pointer as parameter, audio samples will be directly written to it, with order (left channel or right channel sample first) depending on endianness. Can you confirm that audio buffer format on PS3 and X360 (which have both big endian CPU and therefore does not define LSB_FIRST in makefiles) is the same as Wii/Gamecube format, that is right channel first (higher 16-bits @ address) and left channel sample next (lower 16-bits @ address + 2) ?

- osd_input_Update changed to osd_input_update for syntax consistency


Please let me know if you come up with compilation errors now and still need alter to core code, I think it should be OK now...
This is gossip fix for mage, warlock, paladin, warrior, and druid class trainers. A couple start up errors introduced here because of two missing npc_text. I will push them this weekend.
This part is incorrect and should be moved to DamageDealt void ;)
in Siemens part of the code
Looking at the comments on FlowLifecycle I think that we invoke all lifecycle methods except RESTORE_VIEW.  This is handled by the JsfViewFactory.  I think that the code there replicates a lot of the work of the standard lifecycle (for example it notifies PhaseListeners).  My guess is that firing these new JSF 2.0 events was never added.
This method typically just returns the widget if it had been initialized & initializes it once otherwise. Here, it is cleared every time. I suggest something along these lines instead:

if (asWidget != null) { return asWidget; }
// initialization

You will probably need to keep the background svg element as field as well.
Unless Mike already has another one, I've got an SCons prescription for running Doxygen I can pull out of LSST-land to do that.
The strings "verbose" and "silent" are not enums. They are just string, but strings that make any sense at all.

Good code is written to be easily readable by humans. If the focus was to make it as editable as possible, why even bother with a String instead of a byte-flag? The most important thing to remember when it comes to coding style is that the code should be read more often than it is edited. For some good guidelines, please have a look at the coding guidelines for this project:
https://github.com/Maescool/Catacomb-Snatch/wiki/Code-Style
Accepted, but this is not the best solution.
It could happen that I will change it in the future since it does only report the first non-allowed key etc.
I would prefer to extend any error with schema information (allowed keys). And your suggestion is only relevant if you validate strictly, and in this case all additional keys are not deleted; with this way, you get a better view on the error.
I do it from PlayerController for the specific right know, it could be done in Blobby::step itself right?
it is better to use slashes to and create meaning full structure for pages as the analytics are able to give you more useful tools if you do that.
so i would use the following convention:
/b2m/ - indicates that the app starts
thanks to: Shocker for base
1st step towards implementing mmaps
Hi Oswell,

there seems to be some trouble with this feature. Could you please check the following issue on the official phonegap-tracker: https://github.com/phonegap/phonegap-plugins/issues/280

Your help would be highly appreciated!

THX in advance and best regards
Dominik
Within brackets, most otherwise special characters don't need to be escaped. Only the characters which are special to character ranges ([, ], -, \) and the character which would end the regular expression (/) need to be escaped, and if properly placed, you can avoid the need to escape some of the range specials.  So instead of:

    /[-\[\]\/\{\}\(\)\*\+\?\.\\\^\$\|]/

you could use:

    /[-[\]\/{}()*+?.\\^$|]/
woops. this change shouldn't be inside section-3d. come back to this elliot!
@stevenh, @k0rd, the new API will fix this as it will add an abstraction layer on top of the server, where plugins will just call our shit anyhow. The question is if the new API will be released before we need to change tile compression again.
cpp kode:
    void function()
    {
        std::cout << "Hello World!" << std::endl;
    }
I'll do that...
It would be nicer to make it only happen when people choose the Jribble backend, but this should be fine in practice. Is anyone really going to mind using the virtual pattern matcher, anyway? How much longer will it be experimental?
Ah! Yo decia porque pense que iban a usar este TAD para agregar los locks en los ejercicios. Entonces habría que remarcar eso en algún comentario o algo asi como para que quede presente. Quizas aclarar en el readme que no son sincronicas y como hacer para sincronizarlas.
also this?
Any reason you disabled CFQ, BFQ and SIO?
Are all these imports actually necessary for the mock class?
А давайте по-другому. Построим матрицу переходов, и возведем ее в степень.
i reverted the venc and vdec stuff that marc ported.  might have something to do with why my box didn't like it.
Stupid me ... you're right, of course. Thanks for the pull request with the fix, I merged it right away.
it's already fixed :)
I almost did this myself ;) thanks Kin. I may fork this with the reviews API provided this weekend to provide a few more options. Thanks again.
@cienislaw @Suven 

I added some new strings. Feel free to translate them
Yes, getVirtualFunctions remains the same. getVirtualMethods is different.

On 2/5/2012 12:36 PM, Andrei Alexandrescu wrote:
> But wait, I recall you argued to keep this as is for backwards compatibility. (I'm referring strictly to the doc changes.)
>
対象がownerのときのケース追加したいですね。
one other comment. I suppose i just copied the standards of the other parts of the rackspace code. 
see: 
  https://github.com/fog/fog/blob/master/lib/fog/rackspace/requests/block_storage/create_snapshot.rb 
def create_snapshot(volume_id, options = {})
          data = {
            'snapshot' => {
              'volume_id' => volume_id
            }
          }

          data['snapshot']['display_name'] = options[:display_name] unless options[:display_name].nil?
          data['snapshot']['display_description'] = options[:display_description] unless options[:display_description].nil?
          data['snapshot']['force'] = options[:force] unless options[:force].nil?

          request(
            :body => Fog::JSON.encode(data),
            :expects => [200],
            :method => 'POST',
            :path => "snapshots"
          )
        end

here we pass in options and use symbols.  I expect we should just continue to this standard as that is what the fog rackspace code is already doing.
I don't think this reload should be necessary, though it sounded like it fixed an issue for you.  The wait_for a couple lines before should actually be doing (more or less) reloads anyway.  That said, could you at least move the reload up a couple lines (right after the spot_request.wait_for but before the Fog.wait_for)?  That will allow it to be called just once instead of being called every time the other thing is polled.
"unactive" should be "inactive"
How would I heap-allocate w if necessary?
In order to facilitate the change below (named configs only allowing jobs/services), a new TronNamedConfig object will probably need to be created.
This validation needs to be done in parse_config, especially now that you've removed apply_config() from tronfig.  Otherwise a bad config will break trond.  Same goes for below.
Oh, I'm not talking about a redesign, the default (current) one would supply a function which does exactly what it does right now. But for this particular case (ConsistentActorRef) we should simply make a method of the following shape: def consistentHashActorRef(defaultAddress: Address): (ActorRef, Int) => Int

Wdyt?
Yes, Tuple please.

Just realized that there is a race when removing nodes otherwise, that could make you get a removed node from the `consistentHash`.
yes, that's the way
yes, that is the point, to handle restart, I'll add a comment
no, ! should not throw exceptions in 2.1
This is true - I don't see it affecting functionality but it could be confusing.  Should we guard with a current state check here?
we just decided to not use `after`, see https://github.com/akka/akka/commit/e4104cfd0687ca09943a64e8a42706e1a97ebf1c
ooo, thanks so much, for one of the cases it happens to be the sender. will of course fix
All of these imports are superfluous (after ._)
it would simplify the code, then, to have `required BarrierOp op = 2` (i.e. non-optional)
ah, looked at the whole file, now I understand the purpose of it, sending from the inside is smart. I agree with Roland, the msgs should be shuffled first.
Not sure if I actually used this.
This was barely used before and with the content refactoring it is totally gone.
care, padding_x -> padding_y
If it is, that means this line is not being tested...
Nice, this worked out so cleanly!
I used this format because it is easy to change if you add something. 

```
'...\
...'
```

I changed a few parts to your format and some to this:

```
'''
...
'''
```
More robust to use ```str(e)``` in case the exception is instantiated without any arguments.
I like this. More and more I'm digging the model of test base classes scoped to the design construct, like controllers v. itineraries.
it looks to me like it never included self.  I like the new code (but we need to get rid of the caching)
This looks weird. Doesn't query.GetField() url decode already? you can't just for the heck of it do it again as that breaks rars/zips and so on.
And my response to this apperently got lost (this is getting annoying github). No.. For the 16bit case we are filling the full 16 bit. In the 10 bit case we are using the lower 10 bits of a 16bit value. That said. the 64 is somewhat wrong since we won't be getting the full 16bit range then. ie. 03FF will turn into FFD0 instead of FFFF.
Shouldn't this be up to our VFS handler (ImageFile.h/cpp) to deal with? I don't know the context here but ideally you'd use the image:// URL as you get it (which includes the url-encoded path to the original source of the image but not the locally cached one) and let the VFS handle the lookup in the db to map the url-encoded path to the path pointing to the locally cached image.
"I mean" doesnt really sound like code/instruction language..
I think I'll need your help to figure out what is up with this.  Which one should it be, content or raw_content?
Fixed in new new patch.
hmm, should this possibly be DEBUG instead?
```return Object.keys(this.store);``` would have done it too :)
I don't think subclasses need the `NSNotification`, which simple has `NSApp` as the object and nothing more.
This method should short-circuit if `_tableViewCellFlags.selected` already equals `s`, to avoid unnecessary animations and redrawing.
2 spaces here please
You've answered your question below?
Yeah it was more of a note to self.
agreed. That was mostly a copy/paste from the original code.
I just made some changes to exploit some existing magit faces
Fixed in @b085292ea82e21bab44aa38c6573f33a72f14461 and @ee2e784e3fd383a79520883e06ca78d16edbf68e.
But the logic is correct in this case.
This `\` is redundant, since it is within a `()` block.
Or it was a micro optimization.
@njsmith: This is basically the complete script I'm using for profiling:  https://gist.github.com/62ae8b026f82b45dc597
Good catch. This is not needed.
@tenderlove @pixeltrix Or not change anything - the current implementation seems to be correct for Julian calendar.
Sorry for taking your time and thanks for educating me on differences between Time and DateTime. Though, I'm still left wondering if it's really necessary to have such differences ;)
I was just copy pasting that line, 
You can check it on your side to be sure.
Yes. I'll amend that.
Is this is part of the public API? If it's not  add a `:nodoc:` directive please.
Yeah, without `inspect` this test fails with:

```
  1) Failure:
test_all_day_with_timezone(TimeExtCalculationsTest) [/home/nash/projects/rails/activesupport/test/core_ext/time_ext_test.rb:839]:
No visible difference.
You should look at your implementation of ActiveSupport::TimeWithZone#==.
Tue, 07 Jun 2011 23:59:59 HST -10:00
```
And I actually don't know why :(
I'd rewrite as @tailrec instead of while loop with stop flag.
don't you mean 'yeeeeehaw'?
"metric". I will fix it.
It's fine, I guess.
Does the new adjoint class have all of these printers?  They need to be tested in the IPython notebook and qtconsole to make sure they are working properly with ipython.
Do you want to keep a `1.0` (Float) coefficient? If so, I think you should leave this as it was.
```

>>> simplify((Mul(*[x-r for r in solve(x**3 + x**2 - 7)])))
x**3 + x**2 - 7
```
Although `sort_key` should accept the order argument, it needn't sort the args, so it should still be `self.args`, I believe. "sort" is a misnomer. It's more like "canonical_key".
convert to docstring instead of comments
I couldn't get your proposed syntax to work. I got this to work instead:

    if task.save 
        flash[:success] = "Task created." 
        redirect_to @course 
    else 
        render :new 
    end
I don't get it. How would that be?
change these two lines as I mentioned above in `hook_scianimator`
I think $forceCopy should override $this->forceCopy.
The default value of $forceCopy should be changed to null, meaning the value of $this->forceCopy is used.
If not null, the value of $forceCopy will be used, regardless what $this->forceCopy is.
This should be `composer install --dev`
Does this validate the type safety of the rule or just the syntax? If so, is there a test for this?
Fix spacing! ;)
The ADMIN_START constant is only defined when in the ACP. So it only makes a difference then.
Delete here and elsewhere.
Missing @return string
Can you change that to $this.attr('data-overlay') == undefined, as you need to consider someone using data-overlay="0" or data-overlay="" which will all result in the attribute being false but defined.
`is_a` is deprecated since php 5.0.0, should use `instanceof `
Oops, extra whitespace introduction.
For now - yes. This should better be treated as example.

When using C++11 feature one should denote it here. E.g. if you'll use lambdas you should say:
```
require_CXX11_feature("HAS_CXX11_LAMBDAS");
```

For more information please see CheckCXX11Features.cmake
Така е, изглежда, че [наистина е deprecated, вероятно от 3.2.1](http://apidock.com/rails/ActionView/AbstractRenderer/extract_format).

А иначе, трябва да се направи нещо такова в случая, за да се подаде format:
`render template: 'activity_content', format: 'html'`

Ако има нужда и от locals, тогава (примерно):
`render({template: 'activity_content', format: 'html'}, {my_local_var: 123})`

Мисля, че и това ще работи:
`render template: 'sth', format: 'html', locals: {my_local_var: 123}`

Ref: http://apidock.com/rails/ActionView/Helpers/RenderingHelper/render
won't this make a temp string?  if so, this is unsafe to send to `async_write`
В твоем варианте ты хранишь 2 записи для показа, в моем  - одну. Почему ты не учитываешь что в моем варианте можно хранить в базе сумму за предыдущие дни? И тогда будет сумма 2-х чисел всего. А когда просто нужно просуммировать 2 числа, зачем загружать крон ещё и работой по формированию статистики, когда это может сделать нода?
```shell
$ ls /nix/store/526bjh0phkwp46sv2ix90m1klhb6pg3w-gnome-python-2.28.1/share/pygtk/2.0/defs
activationmodule.c   bonoboui.defs        __init__.py
bonobo-arg.c         bonobouimodule.c     Makefile.am
bonobo-arg-types.py  bonoboui.override    Makefile.in
bonobo.defs          bonoboui-types.defs  pybonobo.c
bonobomodule.c       gconf.defs           pybonobo.h
bonobo.override      gnome.defs           wscript
bonobo-types.defs    gnome-types.defs
```

I guess `cp`ing `bonobo/*.defs bonobo/*.py` should be enough, right? But definitely not Makefile.* and the like.
Please copy only the files which are really needed to the defs directory. And is there even a more official way in the build system that could be used?
Please do not double tal:attributes (use tal:a="id val;id val")
Added RAXAJSS-366, RAXAJSS-367
You should always raise an error class, do something like this:

```raise ASQ::PermisionError```

and create a model errors.rb
```
class ASQ::PermissionError < StanardError; end
```
These test instructions are outdated, now one should run ``Theano/bin/theano-nose`` instead of ``nosetests``.
If you'd like to do a PR to make this function more widely accessible, you're welcome to. I'm not interested in expending any effort in that direction. I don't really have any more time to spend on this project and I don't think it hurts to pull it as is.
These batch tests seem pretty insufficient. We need batches that mix different kinds of ops. Reads+Writes, Replicated+Partitioned, etc. We should also test what you demoed at the iteration review, that one failed statement will cause the whole thing to rollback.
.gitignore はコミットに含めるのが普通ですか？
should be a counted version returning length with a wrapper
You're using the same .vtt file for two tests. This one and DISABLED_AmpersandInsideTagWithSubclasses. Is this on purpose?
Fair point. I will move the entire filters package into an internal package.
I had a similar feeling when I was looking at the same bug. The function is FUBAR'ed and I'm inclined to halt any further changes.
you know what, you're right. I guess I'm too tired to be making good suggestions. Sorry for the noise!
... Which is what I started with, but decided against since it looks like ancient Linear A ;)
clean new pull:

https://github.com/jquery/jquery/pull/999
This part of code is only used by UDF __cmsketch_int8_trans, whose arguments only support int8. The definition struct associated with is 
typedef struct {
    int64 args[MAXARGS];  /*! carry along additional args for finalizer */
    int   nargs;          /*! number of args being carried for finalizer */
    Oid   typOid;         /*! oid of the data type we are sketching */
    Oid   outFuncOid;     /*! oid of the OutFunc for that data type */
    countmin sketches[RANGES];
} cmtransval;

If we define args as array of Datum, on 32-bit machine, args will be an array of pointer. This leads security problem.
Actually, we consider this situation. The problem is that the our function ```SRF_invoke``` needs to return Datum due to the two MACROs ```SRF_RETURN_NEXT``` and ```SRF_RETURN_DONE```. Therefore, the function signature is different with the ```cxx_func```. We search the code, and find that it only be cached. We didn't have a case to use it. So, we didn't set the function pointer. If later, we have the scenarios use this pointer, then we can simply add a new function pointer for the SRF function. Thanks.
nit: spaces around the else, please.
I'd call this parameter "mode" or something
> How do I do that?

Thinking about it I have just merged my rework branch for OC 5.0 (mostly client side code, but also namespace changes), so you would get too many errors trying. Otherwise it would be ```git pull --rebase origin master```.

> Not the best response, as it should better respond with a 403 Forbidden.

I'm not sure what the proper way is. @evert should we throw a ```Sabre_DAV_Exception_Forbidden```?
let's put up a complete linear gradient solution if we're taking the time to do something fancy. please also add the standard ``linear-gradient`` as well as  ``-ms-linear-gradient``. I would also add a directx gradient filter for ie < 10:

```css
background: linear-gradient(#88f, #fff);
background: -ms-linear-gradient(#88f, #fff);
filter: progid:DXImageTransform.Microsoft.Gradient(
            GradientType=0, startColorStr="#88f", endColorStr="#fff")
```
Is the `suite()` definition necessary? Usually I define the tests, then add

```
if __name__ == '__main__':
    unittest.main()
```

As written, the tests will all run when imported, which doesn't play nicely when wanting to have a long-running test runner find tests, import them, and run some/all of them based on what the runner is trying to do.
See `exports.testSettingHeaders` for an example of testing request headers.
could this not be folded up in to the line above?
I feel like instead of defining the function here, just use the call to the Utils module directly above.

Also I think if a method call is not in declarative style (e.g. `has_many :something` or a `puts` or `log` statement) we usually provide parenthesis for the arguments, reads better imo.
Is there a reason why you have another class 'playlist-visible' instead of overriding the 'hidden' class on the button, so you can call `playlistButton.hide()` and `playlistButton.show()`.
This comment is not true is all cases, so it needs to be updated.
How about moving ``test</p>`` to the next argument? We don't show any attribute even inside the end tag anyway. So I think it would be better if we're testing around the begin tag.
Now that we have this handy column indicator :) can you wrap this comment to 80 (or so) columns?
You're right! 1 sec works on both platforms.
this adds a new vbzen every time 
maybe change to
```js
ide.addEventListener("enterzen", function enterzen() {
    ide.removeEventListener("enterzen", enterzen)
...
```
Almost asked how to handle mongo/django form field errors then but by some freak coincidence figure it out just in time... You are testing me way too hard here. :)
This goes for gender.
What does this mean?  You can't really switch focus like this.  You can set focus on something that is a child of your current focused element.  I'm not sure this will do what you think.
Check the specific error message
Capitalize "unfortunately" and fix spelling.
Just return early if it's not there instead of doing a deeply nested if block underneath.
Why are we adding this?
Does ssl use anything under the covers?  E.g., not read, write, sigreturn, exit? (My guess is retries, etc become harder?)

If so PR_SET_SECCOMP, 2 on newer kernels (3.5 rcs now, in ubuntu precise) might help fill this gap.  Happy to submit patches for this.

It's also worth adding PR_SET_NO_NEW_PRIVS if the host kernel supports it.
What's up with this commented out code?
invalid option
可以先不动，我也倾向于FreeBSD下的用户不会用--with-jemalloc选项。
s/ietmadm/ietadm/
pkg-config could give us this one
Not sure why this got changed, but the visible check removes the hidden user.  I believe this should be changed to also check for readable users.

User.visible.readable.each  - @jlsherrill can you confirm?
```ruby
# i would add
validates_uniqueness_of :keyname, :scope => [:informable_id, :informable_type]
# or do we want same keys with different values?
```
Comment above function is just comment. But if you put it one line down (bellow def get_produc()), then it will became docstring, which can be automatically extracted by epydoc and document our code.
use destroy()  because it calls hooks in models. delete just send DELETE query.
This is not from basho_expect, just me wanting read/write functions with more options:
* Expect a not found to test deletes
* Pluggable K,V generators

I can try to merge with the main ones if you don't mind.
No need for two abbreviated lines instead of one :(
From IRC:
If we are redefining NODE_NETWORK (==!fClient) to be a classic-style archival node not necessarily a full node (for pruning), then Im fine with defining a SUPPORT_SPV_CLIENTS to be has full blocks and supports mempool/filters/etc then Im fine with that.  As long as we dont have partial support for the final SUPPORT_SPV_CLIENTS-type flag in one version and finalize it in the next.
My head has difficulties parsing this English early in the morning.
Maybe something like "Returns the next sequential transaction order id" would be clearer? I think the increment is implicit in "next".
The link was wrong... :P
Same as above, please revert to existing code.
Couldn't you use constants for these values?
More funny whitespace here, and on the next line.
Shouldn't this be `application/json`?
I see, so it doesn't matter in the simulations in which you're being fed by a SourceFacility, which are most we'll run at this point anyway. Got it. Thanks!
That begs the question of whether these things need to be unwrapped at all or not.
It is usefull for raw cell, where you can start on `diff --git`.
End, I'm not sure.
Is it significant that these examples are for cases where a user is editing herself? If so I'd wrap them in a `context` that says so. 

Also I'd recommend moving `permission_regex` down into the describe (or whatever the narrowest scope is where it's needed). Since the assertions that use it are fairly wordy, it might be better to define a helper method or custom matcher.
This should be separated out into a separate pull request.
For the purposes of diff, I am going to need a function that easily gets me the in-memory contents of a blob from the ODB with the filters applied. Is there a chance you could make a non-static function out of this and move it somewhere we can share it.

I think what I'd really like is something relatively high-level, a la the `blob_contents_to_file` but that gives me the data in-memory, preferably in a `git_buf`. If we had that function, then I think the `blob_contents_to_file` routine could become much smaller (and this `apply_filters` function would just be absorbed.

I'm not sure where the right place for such a function would be; perhaps in `filter.h/c` as a higher-level interface to the filtering functionality. Open to alternative suggestions if you think of a cleaner/more appropriate place.

If you don't want to get into the now, I can tackle it when I get around to crlf handling for diff. Just let me know...
Minor point, but I think you could dump the second buffer and simplify this to just:

```c
git_buf_puts(buf, "Authorization: Basic ");
if (base64_cred(buf, username, password) < 0)
    return -1;
git_buf_puts(buf, "\r\n");
```
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.
we need it in the HCHARS too.

I did the same mistake before; )
It appears that this page is en-US only, but would it hurt to i18n it anyway?
The one that got away. Thanks, removed it.
This is so we see new data and not cached data after an upgrade happens.
Expressions are trees. Example: The range function uses three other expressions. Filter is a method on all expressions, used to find sub-expressions in the tree - aggregate expressions deeply nested inside other expressions.

Makes sense?
I was wondering if I had to handle this in Buffer or not. Thanks.
doesn't this belong outside the for-loop so it isn't re-initialized on every line?
хак винеси в style_ie7.css
https://github.com/fre5h/fwdays/blob/402c3f48d388b2a9b003d6793ea021714c46e71f/web/css/style_ie7.css
-1 for adding an argument to `create` for clarity and consistency (It should be in `AuthenticationException`). I would add the fuild setter everywhere also for consistency... but this whole change might be only personal preferences.
IIRC you already introduced BC break by changing `HttpKernelInterface::handle()` so I think it would be better to change it to "proper" and more consistent version than using "pseudo" BC ;-)
The previous version was buggy because it would have strippen heredoc.

With the new version, I need to fetch tokens inside the `do...while` (see below) which is why `foreach` is not an option. Does it answer your q ?
you should sue ``info()`` to mark it is deprecated instead of a comment, so that it appears when dumping the reference
I would suffix the classname with `Driver`.
But this is how it's done by the BrowserKit component:
see https://github.com/symfony/symfony/blob/master/src/Symfony/Component/BrowserKit/Client.php#L85