labofsahil/github-event-dataset-2012 huggingface dataset:
No caching

It isn't required in our environment and should stay off while developing a design anyway.
Updated council offices.

I have updated the council offices module. It now contains a little extra functionality which means that you get a rough idea of how many monsters are surrounding your outpost, and which outpost is in the most danger and the amount of monsters surrounding it. Is outpost-specific flavour text.
Cool I'll mess around with it.  One of the biggest criticisms of pbs is
that it's too magical, so anything I can do to lessen that is good...

On Sun, Jan 29, 2012 at 10:57 PM, nemec <
reply@reply.github.com
> wrote:

> Yes it does. It does the exact same thing just without all the hackery so
> I didn't bother submitting a pull request.
>
> Actually, you may want to take line 388 out. I experimented with solutions
> for `from pbs import *` without using exec, but it doesn't look like it's
> possible. I think I forgot to take __all__ out, though I'm not sure it
> affects anything.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/nemec/pbs/commit/071a98b40fb564cba352b7e2d4281b35c83240ca#commitcomment-913118
>
you are right of course, I missed that... will fix asap...
I'm sure the JVM can do this just as well as us, if not better. Less code for us to handle, so everyone wins!
New development should happen on master, ideally.  If there's anything else left behind that you'd like merged, let me know, and I'll bear the burden of porting it.

Landed this on 7ae0d473a6b6ac8cb3e55d665528667566cd8e60. Thanks!
goleki jquery sing lebih singkat,,,goleki neng api.jquery.com/browser/...."selektor gt"
Nice +1 on this :-)
what this is changing?
move this to common, osx/ios uses system
Perhaps.  I guess it depends on if we want to keep rewrite as a "cheap" operation or not.
I personally don't like the optional parameter, it's something completely not semantic. You will need something like:

```stylus
color: mix(red, main_color, 30%, true)
```

What does that true means? should we use a predefined literal like `all-channels`? I've not seen something like that in stylus and IMHO it's horrible.

Pinging @visionmedia, and waiting it's *answer* on the *querelle*. I'll gladly correct this PR with whatever comes out of this discussion.
No. The Minecraft class is Ozelot. (The Swedish spelling of ocelot.)
How about using toString ? toString().match(/^function\s(.+)\(/)[1] should return the constructor name....
This was the disadvantage that plugins must be direct descendants...
How about at least trying to test with a single property like pluginType instead of testing for each type?
Or a brunchVersion prop with a syntax similar to package.json's
There's two semicolons here.
@EdGruberman
The potions can already be given directly with current command, but you need to know their metadata. 
You can find it out on sites like, for example, http://www.minecraftinfo.com/IDList.htm
I do not understand git fully either, so i dont know how to "cherry pick" commits "as it should be" if you understand what i mean
it was about time to see such change..ty
@myabc one tip if you want your 1.9 code to work with 1.8 is to use the backports gem. Granted you can't use 1.9 syntax like stabby procs or the new symbol key syntax in hashes, but otherwise you can write the code as normal. The backports gem basically updates the 1.8 core/stdlib to work like 1.9's.

Granted, this is more applicable to libraries, but still not bad for apps.. since you can use things like ree which may still perform better than 1.9 in certain situations.
When I look for `cancelEditPage` in this file, I also find the following line:

`cancelEditPage(true);`

So won't this cause an issue?
``wget -O -`` is the last caractere expected?
I can't. Even if you're right, without that part, it turned out that the client is crashing with the rest applied.
It's defined upper in this file. Right [here](https://github.com/KL-7/travis-ci/blob/master/spec/spec_helper.rb#L13-16).
Nice chance!
Should not have used a regex here, `delete` takes a string.
Cóż za faux pas. :) 
Już poprawione.
This is pretty awesome, but could you add some tests for it?
there should be no check for options ( we should just use size from kEmbedSettings
you should be setting `margin-right: 0` too
Zitat von celeron55  
<reply@reply.github.com>:
hello,

you're right the first version of the patch wasn't really good. In  
case of fog disabled for example there wasn't a difference between  
water and air anymore. That's why I did a rework to use the special  
texture instead of not drawing a face at all. By now in core this  
texture is same as the "normal" texture. I didn't finde a case where  
it's currently used so i replaced it by a plain color texture to be  
used as face in those cases the first patch didn't draw a face at all.

Ugly is that I didn' get the changes right on first time and those  
fixes are in 3 commits based on old one too

https://github.com/sapier/minetest/commit/aacbfc6a912af0e4076af4883600f801bfd67a66
https://github.com/sapier/minetest/commit/27500b764fe566ce0f10baaffd69ef3fb0e9f165
https://github.com/sapier/minetest/commit/3acc6fd40efed0a967381a575783e721240cc18b

> You don't know what liquid is behind glass with this, right? It is bad.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/sapier/minetest/commit/080c4d78fe5c32234487313fc005aaade13962cc#commitcomment-993398
>



----------------------------------------------------------------
This message was sent using IMP, the Internet Messaging Program.
yay!  I love pages of red diffs!
This method static as it accesses no state on the class, consider marking it static.
With this change, I find that my wifi connection drop each time I do make 'install-gaia' and I have to adb reboot to get it back.  So I'm rebooting on each install-gaia anyway...
041b1ba
Well, I'll revert the code as the diff says, then I'll pass it to you to fix that problem, ok?
You can also just change it to : 

		$var = array_map('strip_tags', $this->getParameter('var', 'array', array()));

simply supply an empty array as default value
I'll feed my GCC to keep it happy ^^
@xarly is 100% stable...Now 20h and y continue...but with a little lag.
I would like a test that does the opposite, assigns someone to a different group and they don't get superuser status. It would suck if we gave it to everyone.
No, it should stay at the root namespace, because it's not really a configuration option.. it's outside of configuration, and is there primarily for more complex development.

---
Jeremy Jackson
http://github.com/jejacks0n



On Mar 5, 2012, at 12:05 PM, Doug Puchalski wrote:

> Should this be ```Mercury.config.debug```?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/jejacks0n/mercury/commit/e1e936ede0f54f665b93e05805fcae92371edf9f#commitcomment-1044357
or alternatively we could have a separate method for this but for now i'll just change the docs
There's probably a bug in the algorithm that makes it not increase
size properly. I remember seeing that a few times, can't remember what
I did about it :(

It's unfortunately optimised for the all-tiles case, since that's the
data I had to work with. I might see if I can tweak it tomorrow?


On 01/02/2012, at 11:08 PM, techtangents
<reply@reply.github.com>
wrote:

> In cases where there's a small number of big boxes, the text stays tiny. Is this normal?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/TheSpyder/fewtile/commit/f9e52daa2989070efcad5cad3f670426035a4381#commitcomment-922370
is active support really needed? it overrides DateTime#to_time with it's own (faulty) implementation.. https://github.com/rails/rails/blob/6372d23616b13c62c7a12efa89f958b334dd66ae/activesupport/lib/active_support/core_ext/date_time/conversions.rb#L68
Thanks for the report. Fixed here https://github.com/hakimel/reveal.js/commit/ba8fc0bc35c6715f2918d6f18cf15d5bad7f8644
`all()` is unnecessary here
ah ah.
Thank you, Mark!
[Constructor(DOMString url, optional DOMString protocols),
 Constructor(DOMString url, optional DOMString[] protocols)]
interface WebSocket : EventTarget {
  readonly attribute DOMString url;

  // ready state
  const unsigned short CONNECTING = 0;
  const unsigned short OPEN = 1;
  const unsigned short CLOSING = 2;
  const unsigned short CLOSED = 3;
  readonly attribute unsigned short readyState;
  readonly attribute unsigned long bufferedAmount;

  // networking
  [TreatNonCallableAsNull] attribute Function? onopen;
  [TreatNonCallableAsNull] attribute Function? onerror;
  [TreatNonCallableAsNull] attribute Function? onclose;
  readonly attribute DOMString extensions;
  readonly attribute DOMString protocol;
  void close([Clamp] optional unsigned short code, optional DOMString reason);

  // messaging
  [TreatNonCallableAsNull] attribute Function? onmessage;
           attribute DOMString binaryType;
  void send(DOMString data);
  void send(ArrayBuffer data);
  void send(Blob data);
};
This doesn't seem to work for me. If I do chest.getInventory() on a double chest, it always return a 27-slot CraftInventory, no matter how I place it.
Are you sure that "z" should be hashCode, not getLocation().getBlockZ()?
anyway,about my problem i have the strange feeling that after  i extracted the new maps for mmaps sistem it started to crash  on every player death.
and it shows me the code from above, i think that 
#define MAX_NUMBER_OF_GRIDS      64  from "GridDefines.h" shout be set to other value.
I'd like to accept your pull request, but the variables and methods should be more semantic (dom_fnc, $$)....
Thanks Raimonds!
change year 2012 (minor)
Totally embarrassing.
in chera O ro red neshoon mide !?
Do you end up leaking this or does zeitgeist_log_find_events free it ? If so that's an unusual API...
Oh yeah,  sorry about that. I'll just send another pull request. Thanks!
Does this test still pass after 8c4cd24?  This test shows the behavior I thought would be convenient, but the removal of lines 103-106 in 8c4cd24 make me think this test would no longer pass?
The config options need clearer names.  !osd_use_tmap means that the osd must ignore old tmap values.  osd_auto_upgrade_tmap causes the osd to upgrade old tmaps.  The osd will always use omap for new objects.  This should only be observable by users using read rather than TMAPGET to get the contents of an omap.
fedena 2.2
They weren't being used. Theming would be the appropriate way to do this now anyway.
sa-weet!
That (static method) would also avoid the confusion I had. I've now looked up the language specs and seen that I was wrong. Static members are initialized on class loading and therefore are done before members of normal classes are created (like described above), but enum values are initialized before static members, and the most interesting thing (at least for me): the initialization of enums and static members is in textual order, not like in the code.

@blafoo: Would you change that to a static method then?
I think preferred the name 'uploadDelegate' as it renforces that it is a swappable delegate pattern.
Things did not line up nicely. Now they do.
oops, sorry for the noise with the added final to parameters and sort member actions ;s
Are there any accessibility issues when setting font-size to 0? Thought I read somewhere that some screen readers have problem with that...
You can put everything here eg.

    examples.append(read_file_or_die(assert_file_exists(arg)))
shouldn't the annotations be final classes ?
There are two domains in play here:

plone.tinymce (custom plone translations)
tinymce (upstream tinymce translations that we abuse)

Thus, commit should be reverted!
congrats (and thanks!)


On Mon, Feb 6, 2012 at 11:11 AM, Juan Maiz <
reply@reply.github.com
> wrote:

> wow
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/softa/festivaldeideias/commit/6fa09210147b7ce13e285781f1e9388a800ee7fe#commitcomment-936441
>
la cosa, es que lo que hace ese commit es cargase esa spell no?
You got over what exactly?
You can create a minor BC break since this will be only merged to 2.3.
All you can do is enforce the **$type** to be set.

Just don't forget to add that to CHANGELOG for 2.3
Johannes: Is the NH saga persister supporting multiple [Unique] attributes?

All: Given that its very rare to need multiple unique properties. Should we
restrict users to only support one. This makes it easier to implement new
saga storage's. Users needing more than one should can be guided to set
their own constraints in manually?

On Fri, Jan 13, 2012 at 2:32 AM, Jonathan Matheus <
reply@reply.github.com
> wrote:

> No, I was saying that we can use session.Delete. We can also support
> multiple properties pretty easily. I can do both tomorrow
>
>
>
> On Jan 12, 2012, at 3:51 PM, Andreas hlund<reply@reply.github.com> wrote:
>
> > So you're saying that we can support multiple unique properties?
> >
> > On Thu, Jan 12, 2012 at 6:18 PM, Jonathan Matheus <
> > reply@reply.github.com
> >> wrote:
> >
> >> We can now that we're using a UOW. We previously opted not to rely on
> >> Raven change tracking, but recently changed our minds. I'll change it.
> >>
> >> ---
> >> Reply to this email directly or view it on GitHub:
> >>
> >>
> https://github.com/NServiceBus/NServiceBus/commit/44f79a38a54aebfcfe7d3bc070fd90cce523b884#commitcomment-861627
> >>
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> https://github.com/NServiceBus/NServiceBus/commit/44f79a38a54aebfcfe7d3bc070fd90cce523b884#commitcomment-862610
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/NServiceBus/NServiceBus/commit/44f79a38a54aebfcfe7d3bc070fd90cce523b884#commitcomment-863209
>
TrinityCore does not and will not condone public illegal servers. Plain and simple. You want something « stable » ? Fix it yourself.

> DDuarte
> Dear aios,
> GTFO
> Yours sincerely, Nay

Totally.
WARIO
Also fixed null value test when checking script parameter values, $null instead of ""
Anything is possible when using named instances. FreeTDS says they work but I never have, use the host/post. This is why I put that same info in the read me.
I guess this change should not be applied to Autstralia (AUD)? In that case you should move the BRL line to the top.
You do understand that almost ALL of our conditions are now broken, right?
Nah, it's the most structured way to do it and I've already bastardized the Settings API as it is.
We where adding too much, yet not complete information here: is better to simplify it and then show all valuable information when viewing full content description
I doubt Salat will work with 3.0 yet, I need to work with Rose on that.

On Wed, Feb 1, 2012 at 11:17 AM, Richard Wallace <
reply@reply.github.com
> wrote:

> We're using casbah 2.1.5-1.  I just tried with 3.0.0-M2, but Salat breaks.
>  I'll try patching that and see what I find.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/mongodb/mongo-java-driver/commit/e7b58d101d0307d0a3c10a2af92d1ce31784a043#commitcomment-922901
>
This ends up being _really_ verbose, possibly to a fault, but I suppose worse things have happened.  It will be easy enough to parse and write, which is the important thing.
@dapi Mail on iPhone
When I saw it, I was like 'whoah, that's _way_ prettier.' I'm in 100% agreement, here.

And now that I think of it, I guess my problem is really an IRB display issue more than anything else; they _are_ exactly equivalent.

I wonder why IRB displays it all strangely...
Don't check in .rbx directories, they're meant to be runtime caches.
Sure.

I have now wiped workspace for both sample projects on Jenkins: master and
stable. They are rebuilding now.

It may take a while, since at least one of them use a private maven repo
inside the (now wiped) workspace. The wipe causes maven to re-download the
entire internet on first build after that... :)

Thanks Johan for donating from your spare time and looking into this!
Den 8 feb 2012 19:23 skrev "Johan Lindquist" <
reply@reply.github.com
>:

> Hmm, seems to not clean out the obj directory correctly for 'mixed'
> projects (project is of type apk but also contains native code), will fix
> that.  In any case, I removed the obj manually and it builds fine for me.
>
> Hugo/Manfred, could you remove the obj folder manually and re-run a build?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/jayway/maven-android-plugin/commit/ee807a227a95827d1684f968bd90bab049a54ceb#commitcomment-946259
>
Ok,  if you'd like to extend my code to support `Timestamp`,  I'll trust that you get the timestamp logic correct.   Assuming you use parseTime,  you'll need to choose the conversion based on the type,  which the current doConvert et.al. don't really support.
we already boot @ 1.20
No, I already fixed it. Just re-merge
In order to make it backwards compatible and have an easy migration, for those sites with staging enabled and using a logo, after the upgrade they will have (logo=true logoId=0), and in this situation we will use the live logo.
@jeffmccune there's some inconsistency with the shortcuts (customer support and manual). I'm thinking we should always append `-foss` or `-pe` to the URL, even though the actual link may redirect to the same page. But at least that way we can make them redirect to different pages if need be.

The PE shortcuts are as follows:

<pre>
Explore the Puppet Enterprise Community.url
[InternetShortcut]
URL=http://links.puppetlabs.com/windows-installer-community

Get Puppet Enterprise Support.url
[InternetShortcut]
URL=http://links.puppetlabs.com/customer-support

Puppet Enterprise First Steps.url
[InternetShortcut]
URL=http://links.puppetlabs.com/windows_installer_starting_out

Puppet Enterprise Manual.url
[InternetShortcut]
URL=http://links.puppetlabs.com/windows-manual-pe
</pre>

The FOSS shortcuts are:

<pre>
Explore the Puppet Community.url
[InternetShortcut]
URL=http://links.puppetlabs.com/windows-installer-community

Get Puppet Support.url
[InternetShortcut]
URL=http://links.puppetlabs.com/customer-support-foss

Puppet First Steps.url
[InternetShortcut]
URL=http://links.puppetlabs.com/windows_installer_starting_out

Puppet Manual.url
[InternetShortcut]
URL=http://links.puppetlabs.com/windows-manual
</pre>
In this case, it was actually a bugfix, the file wasn't included anywhere, either a dev oversight or merge error.
I am the author of the previous changes that Marcello is referencing; please contact me so we can work together on merging the changes. You can PM me on the pfSense forums and we can work from there. Thanks.
Overriding `hash` to do all this complicated stuff, then overriding `isEqual:` to do object identity (which is already done that way by `NSObject` anyway) seems pointless, although harmless. Did you originally intend to have more complicated equality testing? I'd suggest either implementing that, or if object identity is the way to go now, just deleting both of these methods.
yes its this one
what did you add to ramdisk for init.d?
removed unused commands from plugin.yml
axel, please pull your commit into my cooper device too, so i can build other version cm ( with my customize ) for cooper :D
Please help me wirh cooper
It dos't work(
bootloop on samsung picture
This change was reverted. It was only used to find the break in the screen on animation. It is not included in the actual history when building.
Yes, it is not required but rather just a nice to have. It is just useful to put there whenever we do a `bundle exec irb` so that pry would be included and can easily be used within irb. Will remove if need be.
Could we change class name to GeneSigDBInputValidator? - to avoid any confusion in the future that we are validating gene signatures.
I think maybe the module should be `VCR::Deprecations::RSpecMacros` so it's more clear that it's for RSpec.
Most of this function is the same as ShowCommand(). Can they be merged?
Can we combine 2 & 3 into a single step? Something like:

    2. Complete the required fields and submit the form.
       Note: Your application must have a unique name.
       We recommend: "<your handle>/t".
`sync()` is called before the `render` event is fired…
This section is now duplicated from the ValidConfig class. Can it be extracted into another function so we don't have to repeat it?
I don't understand, this will be executed all the time?
Yep, that's been the plan all along.
If it's just internal, just put the impl inside the ComactByteString companion object?
That makes perfect sense!
Yes, these classes related to DummyTransport are not threadsafe. This is  
something I intend to clean up, but only after the design of the remoting  
is settled.
I agree, I don't like it either. I hate symbols in general.
… which I’ve done
Could it fit into `AskSupport.ask`, since that is what the user imports?
Removed, remove comment later ...
Could you fix the docs and update the PR?
This will need to be `utils.i18n.getText` not `i18n.getText`
0.7 doesn't need to be much later than 0.6. I think we will be able to release much sooner than previously now that we have a more solid design. But it would be nice to have a bit more feedback from the community as well before adding guis. If you think that you can have this is in and tested before the end of October, that's a different story. We can talk about it tomorrow in the hangout. We could concentrate more on the tracking for this hangout.
Ok, that's fair. We're stretching the concept of DELETE as it is, so I can see this behavior.
Except, ya know, git history.
Didn't know I could make reference to the function that is being defined. Thanks!
Weird.. I was sure i replied to this by mail.. Just turn on deinterlacing.
got that working, will be in the next push of this PR. I shouldn't be so wary of touching the DB code...
That would be okay too.
like add-extends!
By the way functions do not leave a space after the function name, e.g.
```
_Chain_Initialize_empty( &the_alut->ALUT_mappings );
```
This is because functions that are implemented as preprocessor macros must not have a space after the macro name.
Doesn't "*" add a restriction? With "cp -R ~s" we don't restrict the Source wildcard.
Same for rm_rf_unix.
No, it has to be >= 32000 (as the doc says)
These lines should be tab-indented.
This behavior should go into `setHideKnobTimer:`.
Agreed- I just didn't know which unicode identifier it was. Thanks!
Isn't `NSMidX(self.animationView.superview.bounds) - (NSWidth(self.animationView.superview.bounds) / 2.0)` the same as `NSMinX(self.animationView.superview.bounds)`?
it used in createentrypoint.cmd.php line 155
Yes. `wrap == NULL` means we're looking at a bug in Node. Masking it as a system error is very poor style.
Wouldn't the `in` operator be more appropriate in such cases? I can imagine cases where you would want to have inheritance for settings. (This isn't a concern for this pr, but more for node in general.)
Honestly I was basing this on line 1111 of the same file, in the connect function. Could I remove the "|| {}" there as well?
Ok thanks for the suggestion. We will add a Mapper for this and research the hasDependencies flag.
`readSuspended` is probably correct?
I read somwhere that System.nanoTime() is not really something you should rely on.
Let me try to remember where ;)

Beside that what you think of this workaround?
Sent from my iPhone. Excuse any typos....

Am 26.08.2012 um 13:54 schrieb Shay Banon <notifications@github.com>:

> In src/main/java/org/jboss/netty/channel/socket/nio/AbstractNioWorker.java:
> 
> > @@ -220,7 +253,32 @@ public void run() {
> >              }
> >  
> >              try {
> > -                SelectorUtil.select(selector);
> > +                long beforeSelect = System.currentTimeMillis();
> As far as I know (and I researched it a bit) System.nanoTime is cheaper OS call wise compared to System.currentTimeInMillis. Also, if you use nanoTime you get considerably better precision which is what you are after here (compute elapsed time).
> 
> —
> Reply to this email directly or view it on GitHub.
>
They shouldn't be including a full HTML page, that's bound to break
things anyway. They should be using an iframe or a popup. I feel no
sympathy for them if they're doing that, things can break.

On 21 July 2012 23:04, Johannes Schlüter
<reply@reply.github.com>
wrote:
>>               php_info_print_hr();
>> -             php_info_print("<h1><a href=\"");
>> -             php_info_print_request_uri(TSRMLS_C);
>> -             php_info_print("?=PHPB8B5F2A0-3C92-11d3-A3A9-4C7B08C10000\">");
>> +             php_info_print("<script>(function () {\n");
>> +             php_info_print("    'use strict';\n");
>> +             php_info_print("    window.onload = function () {\n");
>
> Won't this cause strange behavior if somebody is embedding the phpinfo() page in a page already overwriting window.onload? People are embedding phpinfo() in installers or configuration areas for different applications.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/php/php-src/pull/132/files#r1211248



-- 
Andrew Faulds (AJF)
http://ajf.me/
I think you'll be mutating internal objects here, even when using `clean` without `!`.
You don't need this require.
This should now be something like:

    if config.active_record.delete(:migration_error) == :page_load

And it should be moved up, at least [before this initializer](https://github.com/schneems/rails/blob/a52fd948781f947d30ef2ce8b9af86b1d764c6f1/activerecord/lib/active_record/railtie.rb#L62).
We can use `args.extract_options!` here to get rid of the `Hash` check (http://apidock.com/rails/ActiveSupport/CoreExtensions/Array/ExtractOptions/extract_options!)

```
options = args.extract_options!
if options[:on]
  #...
```
@rafaelfranca thanks for the suggestions. This comment i did not quite understand: did you mean to rename `fixtures/file_test.rb` to `fixture_set_test.rb`?
user ()
it'd be nice to expand the description for that issue here
val rhs1 = transformed.getOrElse(rhs0, rhs0)
Looks like someone beat me to it: b924c41.
That's much cleaner. Thanks.
You might consider using np.squeeze for kpos.
It would be nice, but I do not know enough the internals of SymPy to attempt to do it.
Let's add the note from http://suif.stanford.edu/suif/suif1/docs/suifmath_34.html

```
If a single solution exists to this equation, then the particular solution is simply the one solution. If infinite solutions exist, then the particular solution is the solution with all free variables set to 0. In general, all possible solutions to the equation can be represented as the sum of the particular solution and a solution to A*x = 0.
```
Spaces between commas here. You can run a pep8 tool over the code and correct these.
Is there a way to have a tensor expression that isn't entirely distributed?
I agree. Otherwise this looks fine. One comment, though...why are you using expand in the test. Is this in anticipation of teh non-expanding 2-arg Mul?
Since _latex is the standard interface for the latex printer system,  so just removed this example from doctring.
Would it be better if this completely flattened in one step, like `T(a, b, T(T(c), d))` to `T(a, b, c, d)`?
This line should be ordered properly. Also mention yourself in the very end of the change line.
use ConfigurationUtil.toConfiguration()
I really like the restructuring, only using the quote var once, not using jq. Much cleaner. However some valuable parts of the script have been lost. There were a series of string methods that resulted in words not being broken. Also, the concatenated content included an ellipsis that would sit on the end of the string, plus an A link that was floated right to sit apart from the string. I believe this makes this UX clearer. How about this:
It does, but the problem is that a person may have multiple projects, but none are the default. This is check is only if the person has absolutely 0 projects (soon to be lists hopefully).
link to journlaing document
we should break this example in to sections and annotate what's happening? 


issue the following commands to create a directory start mongod. 

    foo

Wait until you see a the following content in the log, and then use C-c to stop this instance: 

    bar

Move the directory: 

    baz

Celebrate!
`/u/{{ pageuser.userprofile.display_name }}` should be `{{ url('profiles_view_profile', pageuser.userprofile.display_name) }}`
What is the full index name (either "it" or too long version)?
I think we need a subdirectory because the script moves chunks into this directory and in the case that an upload is interrupted, we need a way to distinguish which files can be removed.
dammit missed that one
What does a post id if 0 mean? If nothing use null probably.
This was an API change. The first parameter was size and is now min.

I don't see dimension being used anywhere in phpbb tree, therefore you may be able to do this but you need to get approval from more developers.
Also use __DIR__ rather than dirname(__FILE__) as we require PHP 5.3 in 3.1.
Is `$undelivered_user` still required? Looks like you could just send the quey right away without storing everything in an array.
Yep I almost forgot about quueeenooows. Also be advised that configuration.h is a temporal solution until #3 is closed.
I was also thinking about ease of adding a logger outside of the `shoes` gem. It seems like it should be possible to register your own logger without monkey patching our stuff.
you can use: `schema.add( _slug: { type: String } );`
It might well be.  (The `more` relates to streaming... indicating whether there is more data that needs to be rendered in the template (true) or whether this is the last time data will be pushed through the template (false).)

Regardless, changing the `render()` API is outside the scope of this pull request.
админы будут заливать левые файлы? файлы льются уже после добавления баннера в базу
does this replace white spaces, or remove them entirely? Why are we doing this?
Prefer to create this in yours stores (your store can point to a json doc) rather than inline in your view. helps keep stores/models separate from views. see how this is currently being done for stores in OPD.
This doesn't guarantee that the given projection is the projection of the axes...
This was necessary as the attributes in one test cube were appearing in a different order to those in master.
Apologies, forgot which one but it was loaded from a .nc file.
This is strictly the only change that is needed, right?
same here, timeout?
No need to break this into several lines, it fits in 80 characters.
application:unload() can occasionally fail with an exception that cascades to the (IIRC) the application controller and then crashes the OTP kernel app, which then stops the VM. Tests interpret the VM shutting down as a failure, which I think is a good feature.  I forget the details of how it works, but it does happen, at least with R14B04.  I haven't tried risking the ire of R15B01.
It's not table leak - the problem is that somehow the 'schema' test app ends up being requested by the test run of syncdb even when INSTALLED_APPS doesn't contain it. I think the problem is just in the other test, but until now the other apps have had tables existing so it hasn't tried to create them again. I'll look into it.
What are the other `input_type` possibilities? Please add some parameter documentation.
NOTE: don't get too detailed on this run, as I'll have to redo it anyway.  When we agree on options and I do it for real, then we can/should nit pick.
This makes a simple variable declaration list nearly unreadable. It actually looks like there is important program logic here.
That argument is the "scope". It goes into the error messages, and changes the behavior a little. It's undefined for the main function call, and then defined when the function is re-called the secondary times.
Yes, we may change it. It is a scale factor for random initial values...
Not just as an ordered aggregate, but also ordered by the same expression as the input parameter.  

E.g. this is good: ```select agg(x order by x) ...```, whereas these are not ``select agg(x order by x desc)```, ```select agg(x order by y)```.

It would be good if the database had a better way to express the relevant properties so that a user can call the function simply, unfortunately it does not.
Yes. With plpython, we can create real internal functions and hide them from users. This is really better than plpgsql. 

And the speed is faster. I agree.
Nice! I didn't know about this, and we should be using a new enough jQuery to use it.
Both possible. As you like. `alias_method :j1, :deparallelize`
This entire block belongs to an ERB formatter.
But if the shared folder is part of the root instead of the path it won't detect that the file is on the shared folder
I'm assuming this introduces a different way of sending long messages that we multipart ourselves before sending to the SMSC? If so can we document this somewhere?
I kind of expected migration methods to take an `old_version` and a `new_version`?
Hmm, takhle se všechno narve přímo do "globálního" exportu. Nebylo by lepší to mít jakoby v balíčku?

Něco jako:

  exports.error =
    BaseError: BaseError
    UserError: UserError
    ...

Nebo jak jsem zjistil nedávno, pomocí hezkýho CoffeeScriptovýho zjednodušení:

  exports.error = {
    BaseError
    UserError
    ...
  }
Why aren't we writing singletons and instead we're using the "old" revealing module pattern ?
I don't think so. If setAccessible(true) fails, we get a `SecurityException`. If it doesn't fail, then the `IllegalAccessException` is impossible.

As an aside, I don't like security managers. They don't provide security and they add friction to regular trusted programs. Applets are dead, and we should shovel as much dirt on top of their coffin as quickly as we can.
> [ so now github email integration no longer works ... :-(]

I haven't been using it. I can wrap my commit messages, though. It annoys me, too, that they are too wide for the terminal in less. They look better unwrapped on GitHub, though.

> Find one thing worse than the other?

Le sigh. Is there a way to tell perltidy in a comment or something to leave some block of code alone?

I am starting to wonder if this is worthwhile, if before every commit it makes changes that annoy…
Just an improvement. This is because of going to the stack is faster than computing the field reference.
You can see this for instance in jdk impl of concurrency framework.
Same doubt here...
According to the Brackets style guide, strings should be in double-quotes.
Last one!  This `return` can also be removed too.
This is confusing.  "keyIn" from smart auto complete is actually trigged on "keyup"

I tried combining the two, but I need to do some things on key down (like esc and enter keys) and then others on keyup because the key needs to be commited to the search field. To make things more clear I got rid of keyIn and replaced it with "keyup". I then added a comment explaining that hanleKeyUp() takes care of cases where the character needs to be commited to $searchField first since the logic operates on the current query.

-----Original Message-----
From: Peter Flynn [mailto:reply@reply.github.com] 
Sent: Thursday, May 03, 2012 3:17 PM
To: Ty Voliter
Subject: Re: [brackets] Quick Open round two, no caching (#734)

> +        */
> +        
> +    };
> +
> +
> +    QuickNavigateDialog.prototype._handleKeyIn = function (e, query) {
> +        // extract line number
> +        var gotoLine = extractLineNumber(query);
> +        if (gotoLine) {
> +            var from = {line: gotoLine, ch: 0};
> +            var to = {line: gotoLine, ch: 99999};
> +            
> +            EditorManager.getCurrentFullEditor().setSelection(from, to);
> +        }
> +
> +        // Remove current plugin if the query stops matching

Do _handleKeyIn() and _handleFilter() both run every time the query string changes?  It seems like this case is covered in both methods, so it feels like one of them is redundant code.  _handleFilter() walks the whole list of plugins, and if none of them return true from match() it sets currentPlugin to null.  So it also covers that case.

Does _handleFilter() only get called some of the times query string changes?  E.g. it doesn't get called on backspace?

--- 
Reply to this email directly or view it on GitHub:
https://github.com/adobe/brackets/pull/734/files#r771268
Not sure--I can't test it right now either. I don't think we need to hold up merging the tests for this, but I'll add a task to my todo list to check when I'm back in the office.
Should we make addMenuDivider() generate its own id by calling that method automatically?  Based on the usage here, it looks like extensions will be unable to add menu dividers because that id-generating utility is private.
I'm still seeing both here.
The tests above for attr names follow this with a verifyAttrHints() helper function that checks the list contents, to test the list filtering.  Should we do something similar here?

Unlike verifyAttrHints(), we probably couldn't have the 2nd arg be optional since the value list is so attribute-specific, but that seems fine to me.
Thanks for taking the time to write the test. Do you think you can use one of our T()//TEquals()/Tisnull() functions to do assertions of things working as expected instead of the throw here?

In this case it could look like

    var seq_type = typeof data.seq;
    TEquals("number", seq_type, "seq type should be 'number'");

Cheers!
On Thu, Jul 26, 2012 at 2:25 AM, Aaron Meurer <
reply@reply.github.com
> wrote:

> > @@ -106,6 +106,9 @@ def flatten(cls, seq):
> >                you should only consider them when your code is highly
> performance
> >                sensitive.
> >
> > +              Don't try to optimize by removing 1's from the list of
> factors
>
> I thought this was handled in `AssocOp.__new__`.
>
>
It is. I updated my docstring edits. But what your work has shown is that
if you can avoid even getting there it's going to be better.
This is the interface that log provider adapters need to implement. I felt the 2 line interface would be easier to implement than creating a facade that directly matched [NLog's Logger](https://github.com/jkowalski/NLog/blob/master/src/NLog/Logger.cs). [Sample providers](https://github.com/damianh/Logging/tree/master/Logging/SampleProviders) can be part of the documentation or a KB article.
please keep all arguments on the same line for the method declaration
You don't need to do these != checks. It's fine if they're equal!
Instead of creating it and then find it, just set it from what's returned by create
Oh, I see, that is what @jiho wrote.

However, the behavior in 0.9.0 was that range is *not* changed by `breaks`.  In my opinion, the 0.9.0 behavior is correct.
I think you should be able to use a Visibility property. Visibility.Collapsed or Visibility.Visible instead of strings, and those will bind correctly. I could be wrong though. If I'm right we should probably use the Visibility.XX.
加上 form 標籤
`%form.form-inline.input-append`
其他地方也是同理
This will get ugly... There are so many cases then. What is one limit is type of `datetime.datetime` and the other `datetime.date`... I am trying to figure out how to write this as cleanly as possible.
Mind cleaning this up?
We need jQuery?
This can be consolidated to `.image-plugin-hidden`.
Remove this extra newline.
"to subscribe to"?  Maybe, "the type of notification to determine which subscribers to notify"?
`instance` is a poor name, since you're potentially working with a map instance, a popcorn instance, and a plugin instance.  Rename so it's clear what this is.
Awesome catch. Spotting my laziness for sure.
Appreciate this is not a new addition, but the `__class__` here is redundant. Calling class methods from within class instances should resolve the method correctly, without needing to do it yourself. Unsure how I feel about static data behind function calls, but that's a discussion for another time.
OMG WTF HUGE BUG plz add spaces so the assignments line up.
Honestly, I do think that __opts__ is ripe for a refactor so it is more explicit what configurations are supposed to be exposed to users and which are supposed to be protected. It is dependency injection because the configuration code will inject a variety of values into that object. 

I was not suggesting to use a full blown parser, but I would like to see it not traceback for strings such as "mybadstring".
Why maintain state here? Is there a downside to having buffering enabled?
well yes, and so was nexus-test-common before that - but in fact in some cases it seemed direct declaration of dependency was required in the past -  some cases a compile scope dependency was depending on junit version x and that was overriding nexus-test-common transitive dependency of same artifact at scope test.

so in this pull I went for the minimal changes I thought would be unlikely to change the dependency graph

I can take another look to see what removing all the explicit test dependencies does this time around
Normal comment only //
list->part == cur? 好像有这个可能？
would be clearer if styled like:

```
if (...) {
  ...
} else {
  ...
}
```
Well as you noted it is indeed maybe easier to define it completely.

Now normally C# does a complete increment for you when you assign a type, for a byte this would be:

public enum test : byte {
   enumthing1 = 0,
   enumthing2,
   enumthing3,
}

this is the way i learned : http://img17.imageshack.us/img17/1172/enum.png (Slide from my highschool, TI bachelor course)
I know you didn't probably introduce this error, but isn't the format supposed to be like:    _(" string  %s") %  item     and not  _("string %s" % item)   I think petr mentioned this earlier today.
Yeah that was a mess with that ['context']['context'] crap. See dd169e3 which changes it to ['context']['exception'] and checks that it's present before calling `Raven_Client::captureException` (otherwise it will fall back to calling `Raven_Client::captureMessage).
Ouch, somehow missed that one
Can be collapsed.
Niasam, pravilo je tu...ne kuzim...

Date: Wed, 17 Oct 2012 15:00:05 -0700
From: notifications@github.com
To: uskoci@noreply.github.com
CC: vatavuk1@hotmail.com
Subject: Re: [uskoci] Eventi komunikacija + još par evenata (#39)

In design/backend/src/main/java/com/randombit/uskoci/game/GameControllerImpl.java:
> @@ -101,37 +104,26 @@ public void playCard(int playerId, int cardId) throws ActionNotAllowedException
>          removeCardFromPlayersHand(cardPlayed, playerId);
>      }
>      
> -    private boolean playerPointsTooHigh(int playerId, Card cardPlayed){

Maknuo si ovo pravilo van?

            
              —

              Reply to this email directly or view it on GitHub.
Trebalo bi ove stringove Bozja volja stavit u onu GameConstants klasu i koristit od tamo ...
This is probably OK for now, but I think usually closing a tab goes to the previous tab rather than the first one. To be fair this isn't specified in the wireframes though.
fixed - thanks!
这种方法名。。。我真的瞬间凌乱了 @fcrosfly
I think this should be "real-time" here.
Then that is another bug.
Is there a reason for this? I see it only set but never read, which seems pointless?
Yes it is a personal favor, but I wanted to express my dislike here :-D. I think it's also harder to read.
Is this intended to be non-modal?
Isn't the nonce optional, if you don't want a pong?
On second thought perhaps its best to keep separate functions for each, make for a more readable API.
you should either keep using the short class name as previously or add a leading ``\`` to make it a FQCN. Otherwise, it will break the completion in IDEs resolving the relative class names in the phpdoc (PhpStorm 4 does for instance)
This looks like a query-replace-o.  copy-sequence doesn't accept an end2 keyword argument.
Remove this (since it is already being done above). You may have to change the above to `put` instead of `post. And replace any reference to `close_motion_voting` with `close_motion`.
Doesn't matter but thought I'd mention that for this I'd do either os:cmd("mktemp") or just erlang:phash2(make_ref()).
Better to have the code that's constructing this log it, and provide context (see exc_info=True).
s/depreciated/deprecated/
Is the equality really true in this case? Would there be any differences in terms of performance? I have left .toJSON() in place, but if you say that get_text() is better, I could change that.
make this Optional Map instead
Correct; my mistake.
s/MSGPACK-RP/MSGPACK-RPC/ ?
oops. didn't mean to leave that in there :)
CamelCase makes me sad :disappointed:
Leftover "store"
This is a problem for branches that contain the string "-g". I tried creating a branch named "this-is-not-good" and `git_revparse_single` could not resolve it (because it was trying to resolve "ood").

I think you either need to look much more carefully to see if the spec looks like `git describe` output, or probably even better, you need to fall back to the original spec if treating it as `git describe` output fails. Actually, I think you should do both - require at least slightly tighter matching before trying as describe output (perhaps looking back from the matched "-g" to make sure it is preceded by 1 or more digits and another dash, but that may not be the best choice) - and then if it failed, try again with the original spec.  You could probably code it like:

```c
    substr = strstr(spec, "-g");
    if (substr && further_checks_on_spec(spec, substr)) {
        if (revparse_lookup_object(out, repo, substr + 2) == 0)
            return 0;
    }
```

Using this recursive call (just one, so we don't really have to worry about too much nesting) makes it easy to continue with the normal path if the describe fails to match.

What do you think?
I am not entirely sure of whether GCC will be able to extract this multiplication properly. I'd advice towards keeping a constant pointer outside of the loop and increasing it by `stride` each iteration.
`@see changes` (see above)
Also, other `@see` refs should update
Oh, trailing comma here ^
This is missing:

Number of active Army of Awesome contributors

Definition: The number of people who have contributed to the Army of Awesome in the last 30 days.

Definition of contribution: replied to a tweet.
Good catch. That one could have been caused some of our nasty to reproduce waitFor bugs when the while loop didn't check the result before the next interval is fired.
`dsn = dsn_template % ('template1',)`
Why not `self.get_raw_dumps(...)` here?
You could probably just do an assertion here for brevity. The test(s) will fail either way.

    assert not (mobile and desktop), 'An app could not be only both desktop and mobile.`
bob wins.
Right on, I hadn't looked at optimizing things yet. I'll do this now.
Just checking, I agree with keeping this as strict as possible. Which actually leads me to a separate thought. Can someone override these built in parsers by redefining them?
You should add the use statement for the class directly
please return a `ValueGuess` or `break` to be consistent with all others `case`s.
public methods should be defined before protected ones
You could probably use `-size` here.
post.author.userID != [CPUserDefaultsHandler currentUser].userID
I see this all over the place.. suggest adding isCurrentUser to User.
Ahh good point.  I would guess that a grep like that would be unreliable.  I guess there's not much to be done here.
变量名为什么一会带_一会不带？如果是内部使用不对外暴露最好统一下
如果错误的传入selector 这里都不考虑容错判断的么？
N-avem nevoie de ID, pentru ce?
Are you sure about that? I could be wrong, but I don't see how one is more immune than the others. It's all threads under the hood. How is PoolWorker.run any different than the above approach?

Also, the current approach requires two hacks (looping to 1000 looking for results, and raising an error - that could mask real errors - if there are no results).
This assert needs to be on one line since our stripping code is pretty naive.
Yeah, sorry about that. Will remove.
before we start listening to the chanel, we unsbuscribe to attach a new function that will only get fired once. i cant remember exactly but im pretty sure if we dont unsubscribe first if the user tries to import while the file is importing it will start fireing events twice because we then would ahve two functions listening to the channel.