I hadn't noticed the .fr, thanks for pointing this out. I have switched back to .com.
As for the layout, I don't think it matters as long as all the needed strings are present!


The cache wouldn't be the problem. The problem is detecting changes in time without using to much performance.


I did, but I couldn't come up with a good way of moving the ImportFrom-specific code out of this method that didn't duplicate enough code as to be not useful.


Java, Javascript. Same thing amirite?


orly


Perfect! :+1:


I agree with @redonkulus; this should have been a bigger change in your semantic versioning. It also broke our build.


Sorry, I am not sure what you meant. We need to guard against the scenario where SetValue  succeeds but the Assert after that fails, causing DeleteValue() to be not called. To do that we use the following pattern

```C#
Registry.LocalMachine.SetValue("Test2", expectedValue);
try
{
    // this will execute only if SetValue succeeds.
   Assert.Equal(expectedValue, (int)Registry.GetValue(Registry.LocalMachine.Name, "Test2", defaultValue));   }
finally
{
     // DeleteValue will be called irrespective of whether the Assert fails or passes.
     Registry.LocalMachine.DeleteValue("Test2");
 }
```


Check ahoul be

iif(cfg.flags.SingleOutputDir==true,

So it is properly detected when removeflags is used

Guess it is same case for other flags


OK. I will correct the english in the ``INSTALL.md`` :fast_forward:


Sure, all my fault. I was the only one working on the code. *cough* Mason was totally there, don't let him fool you, he was working along with me, basically telling me what to type *cough* *dies*


@SunsetBlade @sstone93 @michaelcatt ... how do I remove this file? sigh :pouting_cat:


Ok. I know you asked previously, but this should definitely deserve its own ticket. At least for sake of traceability.


Are you talking about loopback 1.0? 1.0 doesn't have replication.


If you install trough package.el it should generate autoloads for you. Isn't `define-namespace` defined in your emacs?


Hahaha
Le 25 mars 2015 20:49, "Anthony Scemama" <notifications@github.com> a écrit
:

> OK. I will correct the english in the INSTALL.md [image: :fast_forward:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TApplencourt/quantum_package/commit/7332f5c6558ad3dbd5d0fd78f2c171b11e3e92ba#commitcomment-10398453>
> .
>


Looks good!  Proceed with pushing this change to production!


But is required in previous versions, so needs to be left in or made optional based on arduino version.


Amit this page may look good on mobile but now it doesnt looks good on desktop.
We have to make it responsive not lower its width.


I reproduced the issue and fixed/tested with this case:
https://github.com/bkaradzic/bx/blob/master/scripts/unittest%2B%2B.lua#L10


It is working both ways, we can keep it as it is. Justs want to inform you, if you have concerns.

My best wishes to get your health back!

Jahto and I must finish important things first we detected, so no hurry for beta and flyspray.org until solid fixed.


Whatever we predicted together (in the past) are coming real. And Whatever we wanted to build are .....  :'( :'(


Thanks for the entry but participants were not supposed to generate their own title options. Everyone of us had their 3 title generated by someone else. So here's three new ones:

* 8-Bit Rainbow Armada
* Hardcore Sloth School
* Wild West Goth Trivia

Please pick one of those and change your pull request accordingly or just open a new one and close this one here.


Ok. Will check tomorrow, away from computer now
On Mar 25, 2015 8:52 PM, "Branimir Karadžić" <notifications@github.com>
wrote:

> I reproduced the issue and fixed/tested with this case:
> https://github.com/bkaradzic/bx/blob/master/scripts/unittest%2B%2B.lua#L10
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bkaradzic/genie/commit/ce8f85701187fa48521ef0a6d38ac0f13726ae7b#commitcomment-10398491>
> .
>


ast_optimizations (lefticus) - x86_64-MacOS-10.9-clang: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-0.0%25-red.svg)


I think this needs to be `d.response.pages.item` based on how you were setting the `path` before. In any case, the best way to debug what's going on is to try `console.log` out to view what this function is returning. You need it to return and array of the objects you wish to iterate over to generate the single views.


Voce ja viu um homem enlouquecer, por causa de um pedaço de código em uma tela?


ast_optimizations (lefticus) - x86_64-MacOS-10.9-clang-Debug: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-0.0%25-red.svg)


What happens if you visit `aggressive-indent.el` and run `M-: (lm-header "version")`?


I couldn't run my dataset.  Program replies
 "Warning message:
In data(spthin_cvs2) : data set ‘spthin_cvs2’ not found"
I guess something wrong in my csv input. How can I see the csv input example of this package ?
Tnks!


Sounds good


I still see 2 is_spam in the master branch in Config/Schema/schema.php


See https://github.com/joomla/joomla-cms/pull/6581


Agree! Smartphones are not computers! :)


Исправлено в a03f6edd16c0f7d04dc08dbd2d115a9be56822ef


The javadoc for `Operator.finish()` says "This method will not be called if the Task is already failed or canceled." so it seems odd to delegate to `finish()` here. Looking at our other operators that implement `close()` there seems to be a fair bit of inconsistency, with some delegating to `finish()` and some `finish()` delegating to `close()`. Could you look at all of those too and see if they're correct?

It seems to me that it would be best to instead change the Operator interface to remove that sentence, and instead always call `finish()` before `close()` which could then be done in `Driver.destroyIfNecessary()`


You are right, it still works when `num_examples` is integer. Thanks. Anything else is alright?


Use better variables than "x"


develop (kbenne) - x86_64-MacOS-10.9-clang: OK (1160 of 1163 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.74%25-yellow.svg)


Hey JR. Am getting an Uncaught TypeError: Cannot read property 'hasOwnProperty' of undefined. Which backend are you on? thx


does this method get called anywhere ?


Yup, that's right, will add a docstring


They're defined right above. `new RoundingRule(2)` is a custom rounding rule. The canned ones look like `RoundingRule.FOR_SCIENTIFIC_NOTATION`, etc. `RoundingRule` is in [com.github.nateowami.solve4x.config](https://github.com/Nateowami/Solve4x/blob/master/src/com/github/nateowami/solve4x/config/RoundingRule.java). And for the canned rounding rules, you can compare them with `==` because there can't be duplicates of canned rules floating around in memory (unless someone does some nasty cloning).


Summary about directory "win32".
"win32"目录摘要。


XXX, this is not done, it should test for the existing tools


Thanks a lot for this code. Could you give an example how the WP8 URL has to look like?


What is in the `@topics` object here? I wonder if you actually meant to use `collection` instead?


right again - pushed the change to my branch.


use a better variable than "x"


Some spacing issues here


spacing issue to fix


有时候命名是件又纠结又麻烦的事情


I nuked an eclipse file...shouldn't have been there anyways


@ehmry You forgot to commit this patch


Thanks!


Where can i reach u?


@defunctzombie can you explain this change


Hey Aeramis,

Great job on the midterm. You used object oriented programming, and were able to successfully access 3rd party APIs. I left some inline comments, but mostly they were just small things, like using better variables and fixing some indentation.
I would say however, that since it is pretty hard for a user to actually enter a key word that matches any of the stories that came back from the API's, I would add a little more information for the user. For example, "a total of zero stories returned for 'cheddar cheese' key word search". I was not sure your program actually worked until I got lucky with some random word.


K(X, Y) = 2*<X, Y> / (||X||²+||Y||²)


Artifacts


"in a CN" is unnecessary, in as much as any domain present in the CN *MUST* be present in the subjectAltName.

Thus any certificate with a wildcard domain in a CN is by definition a subset of those certificates with a wildcard domain in a subjectAltName.


When you care about battery performance, come check out my Premium Version:
http://christocracy.github.io/cordova-background-geolocation/


HotFixPurchasedAirUnitTest (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (656 of 656 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-4.66%25-green.svg)


Looks better, at least I think.

I'm assuming the reason you changed all the references to "INI Files" to "Config Files" is because the RDB file is no longer Project64.ini; it's Project64.rdb not .ini as well as the other config files that don't have INI/RDB extension with Project64 anymore.  So that looks like a good idea.


Передавать мир через параметр тут излишне, он и так передастся на сервер как реквизит формы.


actually... not so weird. Renamed `QualifiedMemberName.ModuleScope` to `QualifiedMemberName.QualifiedModuleName`, per http://codereview.stackexchange.com/a/84893/23788


Would you mind fixing the docstring to say "This function excludes shared variables that store the state of Theano random number generators" or something similar? "random shared variables" is a little vague.


Thanks :)


How about
```c
L"Cheat has been changed.\nDo you want to update?");
```

I follow the traditional habit of writing 2 spaces after the period ending every complete English sentence, so normally if it were me I'd have probably written `changed.  Do...` with two spaces after the period, not one.  But at least this way, with the newline moving the question "Do you want to update?", we can be neutral to that debate and use the free message box space to keep the question to the user on its own line.


As worded, this mentions nothing about the ICANN vs PRIVATE domain syntaxes.

*.appspot.com is a valid certificate to potentially issue, but only to an applicant who can demonstrate control over appspot.com (with no left-label portion)

*.google is a valid certificate to potentially issue, but only if the Applicant can demonstrate they're the IANA contracted registry (so you would need to validate the Subscriber's Organizational identity, and thus would need more than simply Domain validation)

Just making sure we're explicit between the proposed implementation (only check ICANN) and the wording (check both ICANN and PRIVATE)


Simply amazing. o.O    when it will be in tc?  soon? who will be the oompa loompa? xD


Did I miss a comment on why this isn't wrong?  I'm kind of curious why tests on this aren't failing.


I *think* this `find` can be avoided entirely. I can think of two purposes it might have:

1. Expand directories in the `git status` output to the list of files below them.

   If I’m not mistaken, the only case in which `git status` does return directory names is to summarise untracked subdirectories. You can ask it to not do that by passing `-u all`. Then every path it outputs will be a file.

2. Filter out deleted files.

   If `@changed` contains only files then you can just `@changed = grep -e, @changed`.

And if the `find` invocation goes away then the “fucking shell” exception is unnecessary too.

Yes? (I’m not sure I’m not missing something.)


Nice use of rescue .... something I bet I won't see in any other student assignment.


fagagsadgasdg


Either one's fine I guess.

I know I used to say "plug-in" with the hyphen in it, not "plugin", because FireFox's spell-checker dictionary was severely lacking in completeness, but in modern standards "plugin" is an official word so who cares.


```Объект.Мир = ОсновнойОбъект.Мир;```


hm that's weird I was pretty sure I fixed this, one moment


develop (kbenne) - Win64-Windows-7-VisualStudio-12: OK (1164 of 1166 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.83%25-yellow.svg)


Whatcha think, @shannanf89?


Aqui agrego 2 push_back al codigo.


Here's *a* Pyret definition of round-half-to-even (IEEE) in terms of the current num-round. I'll add it in a test suite file.

```
fun round-half-to-even(n):
  n-neg = (n < 0)
  n-abs = num-abs(n)
  var n-round = num-round(n-abs)
  if num-abs(n-round - n-abs) == 1/2:
    if num-modulo(n-round, 2) == 1:
      n-round := n-round - 1
    else: nothing
    end
  else: nothing
  end
  if n-neg:
    n-round := 0 - n-round
  else: nothing
  end
  n-round
end
````


Great Midterm. I can't seem to ever find things to criticize in your code. I'm trying.
I'm happy to see you used the spotify API, which was a little more difficult to access than the APIs we gone over in class.


:beers:


解决checkJson方法中， $option 变量名未定义的错误


{
body
 hexid:10px;
}


Could you please restore my copyright? I'm sure you didn't mean any harm, but removing my copyright is actually a violation of the license. Thanks!


I may have overridden Brandon's changes to the database data, whatever they were.


I said program in my commit message.  I meant to say file.


develop (kbenne) - i386-Windows-7-VisualStudio-12: OK (1164 of 1166 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.83%25-yellow.svg)


lol


rax retry tests


Only comments here are that you should consider changing the home page for the app. In your routes, add a root path. Also I see you have some GIFs , but I didnt see those actually show up on any of the pages.
Good Job.


I'm not opposed to the change, but I'm also not sure why it is necessary or relevant to RFC 5737, as this testcase is not "documentation". Both IP blocks "do not legitimately appear on the public Internet.  These addresses can be used without any coordination with IANA or an Internet registry." according to at least RFCs 1918 and 5735. Those stipulations alone allow 10.0.0.0 to be a viable testcase, regardless of RFC 5737's designation of a subset of "IPv4 address blocks that are provided for use in documentation".


Does this only work for Nullable<bool> What about the other null able types, like Nullable<long>, Nullable<int>,Nullable<double> ...

How about this:
if (excludeDefaults && (value == null || (!fieldDef.IsNullable && value.Equals(value.GetType().GetDefaultValue())))) continue;


ah ok. glad it is was a simple bug.


There's an RPN calculator towards the bottom of radtrack/RbUtility.py (look for `def rpn(expression):`). Perhaps this could be of use there (or vice versa).


Mas @tianp, oke dengan hasil akhir ini?


where's WITH_ART_SMALL_MODE := true ?


Here you are instantiating the MyDate object and passing a date. Looking at the code for your MyDate class, the initialize method does not take any arguments, so you are always going to get some errors.


Nice, yesterday i thougth that i had updated with this badge but seens no.. :+1:


I'd love to get this pushed to npm


def initialize(user_date)


Из-за этого сброса кэша, не будут работать тестовые методы с зависимым контекстом, если их выполнять по одному. Или сохранение контекста в самой обработке не гарантируется?


Stop using default texts


@timoxley I cannot remember ... :(


Thanks!


用一个int就够了，我写个贴给你。


Still trying to figure out this commit and branches thingy...


I left some inline comments on your midterm. You are missing the 'user_date' argument on the initialize method for your MyDate class. I think this is the root of the problem here. Fix that part and see if it works.
Awesome use of regex !


This reference to the Mobile Services SDK should stay against the 1.x line (probably 1.3.1 right now).


R2b: Before working on the Factory


I tried to grep my project folder for use of my specific fork but so far have turned up nothing.. so long ago and a shit terrible commit message on my part.


yes


half of it


Obvious enough, thanks zerty!


@jstedfast I think this commit would solve my *folder.Search(SearchQuery.NotSeen)* problem with Cyrus-imapd.

Any change to get a new build out to use it ?


why? in PR its edited.


And I still get the below error after I remove it myself:

petchaw$ ./cake schema create --plugin Comments

Welcome to CakePHP v2.6.3 Console
---------------------------------------------------------------
App : app
Path: /Applications/MAMP/htdocs/hcrc/app/
---------------------------------------------------------------
Cake Schema Shell
---------------------------------------------------------------
Error: The chosen schema could not be loaded. Attempted to load:
- file: /Applications/MAMP/htdocs/hcrc/plugins/Comments/Config/Schema/schema.php
- name: Comments


Does this solve the problem of ambiguous error messages for invalid credentials?  I think that was the reason why https://github.com/niumainframe/JESftp/issues/1 was made-- because some students would just wig out when ftplib stack traces would come out.


This has no useful content.


For what it's worth, it was Angel's Envy Rye that I mentioned. ;-)


I don't want a 'nbproject' directory.


Oh that's interesting. Is that assuming that the new form's timeEnd and/or submission time is near the old one's? Or does this actually end up sorting the case update back to the original update's position in the list for some other reason I'm missing?


90022444-FewReleasePrepThings (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Just to make sure I understand what is incorrect (as I attempted to fact check these things), is this part incorrect? env does in fact print out a list of variables if called on its own. I just had trouble locating solid use cases, as you noticed. For example chroot's functionality suggests it as a sandboxing mechanic, yet its manpage suggests it does a poor job as one due to the reasons I outlined in its description. Also was there anything incorrect with the use of PATH there? Testing that example yielded a different set of paths if shell was set up with a new environment set to /bin/sh rather than /bin/bash.


Загвоздка в том, что картинки должны загружаться асинхронно, и на старте экрана их ещё нет, а плэйсхолдеры для них надо расставить сразу. Можно было бы передавать imageURL в ячейку, но такой вариант мне не нравится. Я попробую подумать над этим вопросом при реализации асинхронной загрузки картинок.


This changeset is too large.


@mjeaton sorry! fixed in a1ed485


Useless assignment to variable - `other_roles`.


Use a guard clause instead of wrapping the code inside a conditional expression.


It gives `"1.0"`


That was my original design when I first split this out, and I'm fine with moving back to that if you think it's better.

What bothered me about it was that it seemed unintuitive to me that the constructor would perform all the analysis immediately. In my head, constructors are more lightweight, and just initialize the object, whereas having an `analyze_files` function seems to better imply that running it does the (relatively, at least) heavy act of parsing and analyzing the files.

I also felt that having an `analyze_path` function on the analysis object didn't match up since you can't really just pile more paths onto the analysis once it's done. And with two of the three functions outside of the object, it seemed to make sense to move the third one off for consistency.

Perhaps just having the `_` in front of the method names would be enough of a signal to avoid them, and maybe `analyze_paths` could be a class method?


I don't think I want to merge the deletion of a file I don't think I have.


Please use the `new-test` script to create a new testcase with the necessary boiler plate and proper file number.


I'm not sure about this, should examine more adodb code. But I think that when not using offset and limit, it internally just fetches all the result set into memory, so trying to loop over it row by row does not solve any problem.

If adodb would put the full result into memory/variable, it would be a bad db abstraction layer, not?

yea, exportdb.php seems unused, maybe it was used at the beginning in creating the install xml files first.

I wonder if adodb xmlschema is good enough to contain all the information of a table definition. For example, MySQL supports comments on defining fields, so we could add a comment there for what reason field 'list_order' exists. Like automatic documentation for the database.


:tada: :fireworks:


Agreed, but this is a good safety check.


Test


hey its the first vim revision


ast_optimizations (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-AddressSanitizer: OK (335 of 335 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Some of us do the manual install, as explained on the project's home page. For several reasonr, I tend to prefer manual install to package installs.


Damn, didn't see that pr first. sorry we can fix it. I'll be back on in a bit. I just wanted to clean those up so no one else had issues, then I was going to fix when I got back on. Want to redo the pr to add it back? Or I can try to do it in a little bit


Favor `unless` over `if` for negative conditions.


Yeah I mean I added it here to break up the monster function in main.js


 - __warning__: [include/chaiscript/language/chaiscript_eval.hpp:542](../blob/62337062bf365be4b702c7fa18f9244bab29aaf6/include/chaiscript/language/chaiscript_eval.hpp#L542) Member variable 'Equation_AST_Node::m_oper' is not initialized in the constructor. (uninitMemberVar)
 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/62337062bf365be4b702c7fa18f9244bab29aaf6/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/62337062bf365be4b702c7fa18f9244bab29aaf6/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/62337062bf365be4b702c7fa18f9244bab29aaf6/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/62337062bf365be4b702c7fa18f9244bab29aaf6/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/62337062bf365be4b702c7fa18f9244bab29aaf6/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


This seems to be taken from the GNU libstdc++ (https://gcc.gnu.org/onlinedocs/libstdc++/libstdc++-api-4.5/a01023_source.html). Thus we need to reflect their copyright and the license in the header of this file:

~~~
// Core algorithmic facilities -*- C++ -*-

// Copyright (C) 2001-2014 Free Software Foundation, Inc.
//
// This file is part of the GNU ISO C++ Library.  This library is free
// software; you can redistribute it and/or modify it under the
// terms of the GNU General Public License as published by the
// Free Software Foundation; either version 3, or (at your option)
// any later version.

// This library is distributed in the hope that it will be useful,
// but WITHOUT ANY WARRANTY; without even the implied warranty of
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
// GNU General Public License for more details.

// Under Section 7 of GPL version 3, you are granted additional
// permissions described in the GCC Runtime Library Exception, version
// 3.1, as published by the Free Software Foundation.

// You should have received a copy of the GNU General Public License and
// a copy of the GCC Runtime Library Exception along with this program;
// see the files COPYING3 and COPYING.RUNTIME respectively.  If not, see
// <http://www.gnu.org/licenses/>.

/*
 *
 * Copyright (c) 1994
 * Hewlett-Packard Company
 *
 * Permission to use, copy, modify, distribute and sell this software
 * and its documentation for any purpose is hereby granted without fee,
 * provided that the above copyright notice appear in all copies and
 * that both that copyright notice and this permission notice appear
 * in supporting documentation.  Hewlett-Packard Company makes no
 * representations about the suitability of this software for any
 * purpose.  It is provided "as is" without express or implied warranty.
 *
 *
 * Copyright (c) 1996-1998
 * Silicon Graphics Computer Systems, Inc.
 *
 * Permission to use, copy, modify, distribute and sell this software
 * and its documentation for any purpose is hereby granted without fee,
 * provided that the above copyright notice appear in all copies and
 * that both that copyright notice and this permission notice appear
 * in supporting documentation.  Silicon Graphics makes no
 * representations about the suitability of this software for any
 * purpose.  It is provided "as is" without express or implied warranty.
 */
~~~


Yep, thanks for the help!


yarly


I'll make PR now, just run verb on your side for readme, please :D


Addressed in a6316c8fee8b372a28653de238e5d1a192fda5a4


That's a hell of a lot of scrolliing, The passwords I beleive are encrypted, right?


damn, so used to ruby


return true   is missing


Thank you


Hmm, I thought that SAGA handles empty values (or `None` for that matter) just fine?  I actually tested this... :)  So what triggered the commit?


Yea, you can add reporter, dev, whatever roles to each project.

Having global roles for all projects just make it easier if you don't need that granularity.

If you need that granularity, drop global roles feature.


thanks!


você pode usar o:

```
textarea{
  &:focus{}

  @include placeholder {
    color: white;
  }
}
```


If you could add your entry alphabetically, that would be great.


And I don't even think any of us are supposed to see this


Salut, je suis débutant dans les podcast et je fais des codes en python, java et C.
Abonne toi à ma chaine pour me soutenir


It seems as if in emacs 25.x, `(lm-version)` is not run inside the `aggressive-indent.el` buffer but somewhere else. I get that error if I run it in, for example, the scratch buffer. But `M-: (lm-version)` runs fine in the `aggressive-indent.el` buffer and returns `"1.0"`.. all in emacs 25.x


saw approval from aberghage
at https://github.com/basho/yokozuna/commit/b08d199e7683ecf9c175561ebec7842648c96607


merging basho/yokozuna/test/zl/extra-pb-tests-for-custom-schema = b08d199e into borshop-integration-474-test/zl/extra-pb-tests-for-custom-schema


basho/yokozuna/test/zl/extra-pb-tests-for-custom-schema = b08d199e merged ok, testing candidate = b17dfc6a


ast_optimizations (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-ThreadSanitizer: OK (335 of 335 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


this looks like it belongs in a method.


Uma coisa que é aconselhável é deixar as cores em variáveis dentro de um arquivo colors ou algo do tipo :smile:


##### Deployment available online:
#### - [app.js](https://s.brightspace.com/apps/ipa/dev/bda55faaf37884e4986c00823bcaa767dc12fc24/app.js)
#### - [appconfig.json](https://s.brightspace.com/apps/ipa/dev/bda55faaf37884e4986c00823bcaa767dc12fc24/appconfig.json)


mycomment


It's still waiting approval -.-

Bigger plugins take more time.


This works for me.
Is this commit really necessary: https://gerrit.omnirom.org/12810


`border-radius: 0 0 0 0;` ta duplicado :sweat_smile:


Didn't know the Shaman stuff came with it. Was going to do each class as a separate merge request as i go through and clean them up.

Re: Shaman stuff. The Mastery got changed in 6.0.


I didn't do a detailed comparison but `package-activate-1` in `package.el` is much different in 25.x than in emacs-24 branch (who am I telling this :P).


If you are going to be doing a system version check, you should probably not be calling `floatValue` on a string, since a version number is not the same as a numerical number. You should probably use `-[NSProcessInfo isOperatingSystemAtLeastVersion:]` instead.


ast_optimizations (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (335 of 335 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


hmmmmmmmmmmmmmmmm


TBH : I am really not found of the extraction of this code here. This sounds way too procedural and not object oriented.
This method should probably be defined in the MethodInvocationTree but we would want to avoid mixing InvocationMatcher with the tree.


Why was this merged with this test?


You consider to PR that? Btw. we should have 0 by default I think.


java 7 arg


Ok, I've added it back in a way that should do what I wanted but still work for you guys.


Let me know if it works.


This line runs the unit tests. Don't think we want to remove it?


Oops, that's iOS 8+. Use `-systemVersion`, but do a string comparison instead. You can see an example in Swift here: http://nshipster.com/swift-system-version-checking/


This cause a problem when adding the background color on labels with the generated notes "Added x label" "Removed x label"


@FernetMenta: ping - I think we should not apply DRC at all. According to: https://www.ffmpeg.org/ffmpeg-codecs.html#ac3 1 means: full compression.


ast_optimizations (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (335 of 335 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


* Выделить печать отдельной записи в отдельную процедуру в классе Tworkings.
* Tworkings.AddRecord
* WriteToFile перезаписывает файл, а не дописывает в конец


ast_optimizations (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg)


I misunderstood you first. I thought at first that you meant something like project groups having subprojects (I think it has been a requested feature) or permission groups in a project having also subgroups, but we haven't anything like that. What I've been trying to implement has just global groups and one level of project groups. And exactly because we can have several different groups on project level that do have not any kind on relation to global groups, I can create them and assign users to them at will and according to my needs, and I'd really expect to have the project specific permission to override any global permission just for that project only.


It does have it's own info, in the local_address attribute, but it shouldn't try to contact itself I think


I'm not convinced this was tested before it was committed. Mainly because you are using two different variables `failed` and `failure`. Also this will still write out the files in the event of a failure.


> va_list is not necessarily char* - it's implementation defined and might as well be struct ;)

Well we are MSVC only I doubt they will change implementation anytime soon. But fine I made a reference there.


nope! wrote from my phone. apologizes. will update shortly.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (335 of 335 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Ya I think I have messed up little but but sure I will not do the same thing again and my work is under progress by the end you will find it every thing clean :+1:


Did you chose rep['original_name'] to stay as consistent as possible with old dbase field name? I find rep['latin_name'] more logical...


all tests pass:
success: http://buildbot.bos1/builders/test-yokozuna/builds/1050


Ah...it was merged last week. I just closed this PR now; Doug pointed that I had a couple of PR I forgot to close. I'll send a PR to remove this test after branching.


Hi. You can see the example data that comes with spThin using the following commands.

```
data( package = "spThin" )
data( Heteromys_anomalus_South_America )
Heteromys_anomalus_South_America
```

Good luck!


pass the exception (at least the exception message) into the logger  ?


I understand your idea, but please give us some days thinking before making such radical change!


why have/had you removed all that (encode/decode serie_name and apparently few others things) ?  merge prob ?


Correct.  :(  Sorry - that was a typo I created while trying to fix other spelling errors.


i would not add a JIRA issue into an error message.


it better to use bootstrap as a gem


Super mega update. Îmi place!


A few more logos....hopefully named OK ;)


Oh I see OK!


Can you put your gist on the same line as your topic? Thx :)


Hey, global roles can't be dropped! Every user must belong to a global group. I just need and want a way to occasionally override that global permission if needed. Makes group and permission management a lot easier.


Do you really need this file?


If left in it would not compile on arduino 1.6.1


One is used to get the display name used in a muc (needed for pinging people), the other is the jabber account name (needed for PMs).


It looks like, this file is unnecessary


sorry you're right. they get stuck at the end because they get sorted by server modification date. https://github.com/dimagi/commcare-hq/commit/e51627948764015d59581647e7e884cbe2597c64

I wonder if that's "correct" behavior or not. I suspect not. :hankey:


Nitpicky  but could we add the suffix to the logger message: logger.error("Exception when updating module definitions for the stream "  + " with the following error " + e );


gcc throws an error to me. if i understand correctly, you cannot swap an rvalue, pls correct me if i'm wrong.

Isn't it the best and simplest solution to write
```
if (!impl_)
{
	impl_ = std::make_shared<implement>();
}
```
instead?


lol


I will read http://www.flyspray.org/manual/
too to understand why it is made the way it is.


no provisionamento do Vagrant, você já é root – então não precisa de `sudo`


 - __warning__: [include/chaiscript/language/chaiscript_eval.hpp:542](../blob/c32a944b9db21a6a1502d5caf329fa148dd86764/include/chaiscript/language/chaiscript_eval.hpp#L542) Member variable 'Equation_AST_Node::m_oper' is not initialized in the constructor. (uninitMemberVar)
 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/c32a944b9db21a6a1502d5caf329fa148dd86764/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/c32a944b9db21a6a1502d5caf329fa148dd86764/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/c32a944b9db21a6a1502d5caf329fa148dd86764/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/c32a944b9db21a6a1502d5caf329fa148dd86764/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/c32a944b9db21a6a1502d5caf329fa148dd86764/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


Typo: should be 'activityInterGroups'


Yes, a lot has changed in package.el in Emacs 25, but I don't see what could cause your issue.
I've pushed a new commit labeled "Try to fix the version", which should solve your problem. Let me know it works or not.


use model scope for this select-query


It must be other message: "Dialog No button adds class **fail** to fixture".


closes #25


__API Blueprint syntax error on line 1__: Expected "---", "HOST:" or empty line but "#" found. (Autogenerated message from apiary.io)


Done.


closes #18


Interesting patch. Wish you success. What is the meaning of the new phase values?


or `if (!impl_) impl_.reset(std::make_shared<implement>());`


Did not spot that one during code review : why is owner of "super" symbol for classes not symbol ?


Yea I tested this it works almost in same time so no use!


shatup


What about dependent: destroy? Do you have a logic related to moving books from one shelf to another?


j


If ep_flags is not set then we have to generate completion by default. Same comment applies for writemsg, readmsg and atomics below.


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1164 of 1167 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.74%25-yellow.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-69.77%25-green.svg)


use link_to instead of tag `<a>` with `href` attribute


Why is this section commented out? The commit message doesn't really indicate it and I've been using a fork with this for quite some time now with no issues to the feature.

For developers often using sandboxes rather than production orgs, having to append `-i=test` is rather tedious as opposed to `force login` always redirecting the user to whatever they used last and letting the use change that by specifying something explicitly ie. `-i=login` or `-i=test`.

If I submit a patch for this would it be considered or was this a design choice?


I think this is good enough. It's not like the number of samples is very large. :)


Ah, I suspected README.txt was a generated file but I didn't see where it came from.

Done.

Note: It added several new entries as well. Let me know if you want me to remove those as well.


develop (lefticus) - x86_64-MacOS-10.9-clang: OK (335 of 335 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


but then its a `shared_ptr<shared_ptr<implement>>` if i'm not mistaken.
it should be
```
impl_.reset(new implement);
```
instead


tx_attr:op_flags should be checked for FI_COMPLETION and passed along to fi_ibv_msg_ep_sendmsg when calling send, sendv, etc.


that seems proper, yes!


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-AddressSanitizer: OK (335 of 335 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


@dczlonka I would suggest to rename it to `URL_PARAM_PAGE_LAST_PK`, because having two names for one thing will be confusing. Let's stick with the name of parameter that API uses.


@kkaefer are the the corner padding pixels being handled?


Do we need the type annotation?


I don't think they fired as anything but UnknownEvent, but I can check it to be sure. I'm using Asterisk 13. Maybe that makes a difference?


Want to avoid having a dependency on Font Awesome (especially since it's just to get a spinner).


Is this cfor still used?


intellij complained without it, but no we don't really need it. it will speed up compilation though!


Don't leave the dead code.


*sold*


you can use `@books.present?` or `@books.any?`


I think i ran optimize imports - i will double check.


TeamCity xBIM :: XbimExchange :: Build [Build 147](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1211&buildTypeId=XBIM_XbimExchange_Build) is now running


TeamCity xBIM :: Xbim Geometry :: Release [Build 46](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1210&buildTypeId=XBIM_XbimGeometry_Release) outcome was **FAILURE**
Summary: Compilation error: Xbim.Geometry.Engine.Interop\Xbim.Geometry.Engine.Interop.csproj (new) Build time: 03:24:19


does it boost the performance? Intuitively I'd say the old version is fast as in the usual case there aren't many app level styles but a lot supported styles - I might be wrong though. Do you have a benchmark for this?


yeah, i guess it's still used.


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1150
success: http://build.servo.org/builders/linux2/builds/1149
success: http://build.servo.org/builders/android/builds/1184
success: http://build.servo.org/builders/mac1/builds/1363
success: http://build.servo.org/builders/mac2/builds/1366
success: http://build.servo.org/builders/gonk/builds/332


also `@books.blank?` and `@books.empty?` for negative conditions


TeamCity xBIM :: XbimScripting :: Build [Build 11](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1212&buildTypeId=XBIM_XbimScripting_Build) is now running


TeamCity xBIM :: XbimExchange :: Build [Build 147](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1211&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:01:47


These icons look marvelous.


Ok. And it would be better to bring the others into this discussion too. Let's do it tomorrow, I want to finally enjoy my few beers instead of fighting with each other. Or do we fight, at least you understand my idea, just don't think now would be the correct time to implement it?


TeamCity xBIM :: XbimScripting :: Build [Build 11](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1212&buildTypeId=XBIM_XbimScripting_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:01:09


We can move it somewhere else but Config is actually written as a state file (i.e., state.json) and does include other run-time data such as process ID, start time, namespace paths, etc.  It's also a file that libcontainer already writes out, so we don't have to add another file just for C/R.


@Pr0curo Ack, right again. That was untested...


First - it should be called `last_pk` not `sinceId`, second, we shouldn't really set it by ourselves, but rely on the API instead.


Its the one that's already been used.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-ThreadSanitizer: OK (335 of 335 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Yes but if removed it will not compile on older versions. So it needs to habe a #ifndef for the current version.
I would like this to not require the very latest arduino to compile.


fast-forwarding master to auto = 644dc405


Hmm, perhaps I should not have made that change so cavalierly. I guess the word scalar means different things to different people (and I probably use it in different ways). If memory serves, it was due to some assertion in trans where bare-fns would have been appropriate, I'll investigate.


I can get a release out this weekend, but probably not before then (it's a bit of a pain to roll a release)


gemeint war #42


I don't know man, you're the one who gave dropck the codes 366 and 367 :)

Or perhaps this just means errors were removed? I'll renumber.


Changed Product class so that soldBy attribute is of type String for testing.


Да, вы правы. Исправлю.Спасибо за code review!


Or `log << pk;`


fast-forwarding develop to borshop-integration-474-test/zl/extra-pb-tests-for-custom-schema = b17dfc6a


I didn't see the cfor 2 lines up....


Oh and, yea, it will probably work around your problem since you are using Cyrus which is the server I wrote this workaround for.

https://bugzilla.cyrusimap.org/show_bug.cgi?id=3890


"that check" meaning the types of the arguments? It is now occuring in the caller, at line 228 (I stripped away all the dead code there, which also allowed me to remove the "autoref args" logic and so forth).


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (335 of 335 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Don't you think that using `brnanch` makes sens in this case? E.g. *"use the most recent version in the 2.3 branch"*. Which word would you use instead branch?


While you are changing this class, can you also add a GuardedBy("this") for allSplits?


Fixed path names so that they are compatible with any operating system.  Added processing of BEA file "detailnonres_stk1.xls" for section 2.3.1.


saw approval from jdm
at https://github.com/snf/servo/commit/e1f34b827d0d2122750818e380438218e727ad3e


merging snf/servo/main_thread_runnable = e1f34b82 into auto


snf/servo/main_thread_runnable = e1f34b82 merged ok, testing candidate = 7f587f6c


Keep bigger condition code before smaller. Example:
bad:
```
if condition?
  puts 'resolved!'
else
  do_some_code
  do_some_another_code
end
```
good:
```
if not condition?
  do_some_code
  do_some_another_code
else
  puts 'resolved!'
end
```


Read the manual too. I don't agree with just ORing the permissions, but that will be the way it's handled, because I don't have time to rewrite the manual too. Case resolved.


That worked!


Ended up with a
```
#PBS -A
```


It should be version >= 38 as in FF Developer Edition it works as well.


This has to be made a variable, otherwise it's bad for 4k systems.


the pom looks great


I'm surprised you didn't have to change segment alignment.


oh, I missed this pr which is fine. as mentioned on the forum, ffmpeg does apply the correct default. 1 means that the information coded into the stream is considered. playback is done the way intended by the recorder. a value of 0 means manipulation.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (335 of 335 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


you can use each_with_index to avoid additional variable


cool


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


this seems to be corrupt


@FernetMenta read the post i linked to - it suggests the proposal in the stream metadata is for "when the user wants to apply DRC" ... in that case a 0 would mean "default" and 1 "ahh you want DRC? then please use those proposed DRC values..."


Do we still have those restrictions now that we are using getDefaultGeometry?


Yeah, it's in the CQL filter.


https://github.com/xbmc/xbmc/pull/6820 <- sorry - i though this was a comment to this PR - it contains the link i mentioned ...


I chose this formulation because I thought it best reflected the spirit of the original test. The problem here is that the type of `return n + 1` is a type variable (which defaults to `()`). If I annotate `0` to be `0_i32` (or something), that still leaves the RHS as an unconstrained type which we cannot infer (as there exist multiple impls that apply to i32, e.g. `impl Add<&i32> for i32` and `impl Add<i32> for i32`). Using a type parameter avoids this.


But perhaps it is not the most faithful translation, thinking harder, because it will use overloaded `+` I guess.


`./configure --debug --gdb` Didn't change anything, I'll try to get gdb/lldb configured properly tomorrow.


Make sense.  Thanks.


I think we might be better off passing the individual fields, e.g. `AnyPublicKey` and `int seq` rather than making the member function extract them. We already deserialized (since we have an STObject). Alternatively we can write this function:
`Manifest makeManifest(std::string const&);`
which does the deserialization and extraction. `Manifest` would be a struct containing the fields of the manifest object.


master (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (334 of 334 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


the same but more simple `@book.shelf = @shelf`


I recommend you to use Java Enums instead of Strings


Missing an indent here.


This sub is temporary. We're hoping to collect metrics about how Picard is called in practice before I start messing with default version handling.

I'm hoping that wrapping in an `eval` and passing a timeout to `IO::Socket::INET->new()` is sufficient protection against this sub taking down a whole build or getting stuck waiting on a socket.


oh, this IS the code in trans -- my bad. I'm actually ... not entirely sure why I had to add this, but given that `type_is_scalar` returns true for ty_bare_fn it seemed appropriate to me (and a bare fn is just a pointer, after all). I can dig a bit deeper.


You could change the multiple "if"s for a switch or apply a design pattern like command or chain of responsability :-).


blah indent is wrong now.. fixing


It's working


shouldn't this be opts['home']?


Look at what Encapsulation is in OOP and apply it to your class! :smile:


I worked on similar functionality for HashJoin in Cascading https://github.com/gerashegalov/cascading/commit/dd880d30f25d9f6767a19284617f16aabb6ae4de . And there we encountered changing a file name extension may make the path unusable for some InputFormats. Therefore I suggest to change the renaming scheme here as well such that the file name extension is preserved, to be safe.


I can change yr


Discussion continued on https://github.com/Malabarba/aggressive-indent-mode/issues/44#issuecomment-86214912


Oops, didn't notice it.


#312
#313


yes - knew that, but part of the AngularJS philosophy is to not use
anything else.

Moving along into the future, do you see us progressing to the point where
Rails is server API only and ERB templates go away?  I would vote for that
type of progression, but it means Angular would take over routing.
However, I've really really wanted to find a way to use the Iona framework
to build mobile apps with Angular....  Thoughts?

- james

c: 206.953.0593 | e: james.vastbinder@gmail.com

On Wed, Mar 25, 2015 at 8:21 AM, Jason R Alexander <notifications@github.com
> wrote:

>
> FYI, jQuery was already included in the original forked project. So
> decided to use it.
>
> Jason
> _____________________________
> Jason R Alexander
> Sent as binary to your brain!!!
>
>
> > On Mar 25, 2015, at 7:14 AM, James Vastbinder <notifications@github.com>
> wrote:
> >
> > I am fine adding jQuery if we are not using Angular, but Angular has
> validation libraries we should use if you decide to move to Angular.
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/landbaronclub/landbaron/commit/a57d40151757fc112f4d49ab1bd9d19c4d1414ae#commitcomment-10392351>
> .
>


Thank you!

Just tried an upgrade of my prod fs, with on purpose deleted vendors. Vendor install failed on local machine. It seems like it need the cacert certificate as well (cacert.pem). Will look at it when i'm better


What's the difference between `:foodpedia` and `:FOODpedia` resource? Maybe they should be merged in a single resource.


Good catch.


@mignolowa I think, you've forgot to add the empty prefix as we discussed earlier.


Hmm, seems you're right!

https://wiki.asterisk.org/wiki/display/AST/Asterisk+13+ManagerEvent_DialBegin

Looks like it's been separated out into two events (see link). So sorry, as you were!


This ```Start: 0``` looks superflous.


This was causing issues when logging in the first time.  Current endpoint was grabbing either nothing or a should be hidden file (on mac) with no errors in the method call.  If you uncomment that out, and then delete the .force folder on your drive, you can see the behavior.  I only commented it out because I like the idea, but didn't have a chance to put in a proper fix.  Whatever is returned from CurrentEndpoint needs to be determined to be a valid value. If not, then the production endpoint needs to be used.  This also may have implications when switching users. Really the current endpoint needs to take the current user into consideration too.


This too.


perhaps api_valid? otherwise client.valid? might be a bit confusing. but that's that important.


This resource is never used, because instead `<http://lod.ifmo.ru/resource/Person145348>` is used, right?


LGTM except for a minor minor comment.


You're right. It's supposed to be in place of that len() == 0 comparision. Probably forgot to update or it's an artifact of quite a few rebases. Good catch!


Looking at the jsperf again the regression is still there.


Do we still need these two line with keycheck implementation?
It seems to work without them.


エラー記録しなきゃログ取る意味が‥‥


What I see, after allowing a 32bit ARR to be used, is that previousTime < currentTime.


Well the license text seems clear about that but who knows...


> Though it will not be truly static, but each allocation for each compilation unit.

I don't think this is that much critical. But if we count bytes ...

> Logger.cpp approach is probably the best after all.

I wanted to do this compact in one header and I really don't like explict template initialization. But if you prefer this approch. Here is the patch https://github.com/kasper93/mpc-hc/commit/0dd3f308e01245e4239d9f1d982d5e1bdff8d468 comments are welcome.


2015-03-25 10:30 GMT+01:00 philipperuiz <notifications@github.com>:

> why use plyr:::renames, 1) the function rename of dplyr was choose for
> all the other function and works great.
>
No it doesn't: it generates warnings during package check. The only
available fix (I saw you used it) is to initialize a dummy variable, and
that is really ugly.

> 2) when the rename function of plyris imported in the NAMESPCE like in
> line 64, it overwrite the previous import of the function rename of dplyrand
> return a warning message when the package was built and tested.
>
True, I don't like it either

> 3) if the function is imported by the NAMESPACE plyr:: is not necessary.
>
I was not sure which of plyr::rename or dplyr::rename is meant if both plyr
and dplyr are imported by the NAMESPACE. Anyway, the prefix doesn't hurt
even if useless.

> 4) I prefere use only dplyras far as possible.
>
You may be right that we should not be using the two simultaneously.

Could you write a fix without using plyr::rename nor dplyr::rename (issue
#34) ?

> --
> Reply to this email directly or view it on GitHub
> <https://github.com/pveber/morse/commit/3fa48709b4e2c835e7b0997a8197eacccffdcba6#commitcomment-10385074>
> .
>


What's the rationale for this change? How's it related to supporting MSYS2, which is the topic of this commit?

If this change is intended, the comment in line 51 should be changed, too (or omitted, as it's just repeating in prose what's very obvious from the code).


check this out. I think it is pretty instructive. the colorscale needs adjusting the the time interval between pivots, but otherwise I think it is good.


Agreed.  Thanks for clarification.


OK


These muzzle ```Start: 0``` can go, too


1個目の解答でok.
前も言ったけどミドルウェアはリクエスト処理用なのでテストだと処理されない。だから処理してるフリしなきゃいけない（自動でやれよという話だが、メッセージ使ってない人もいるから自動にしてないのかもしれない）


Another one that can go.


@Thetruecrow Thank you :+1:


oh, that's a mistake! This line should be removed.


Thanks.  They express the amount of influence each piece has in determining the phase of the game.


fiestas :D


My intent is to build lVL as a free way to
improve your learning and understanding of a subject.
Because I wish I had something like that.
I should remove the trading part though.

The main concept is to provide information ask questions
and provide very quickly answers to these questions.

Probably,
Anki is a good tool with which something like that could be achieved.

On 25 March 2015 at 14:29, Egor Kovetskiy <notifications@github.com> wrote:

> Why you doing this?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lanastasov/lVL/commit/24e8cc543de0e8c347f21e9eddf8a46a5e11362a#commitcomment-10388250>
> .
>


Allowing a 32bit value to be passed and eventually set fixes PPM input on the SPRacingF3 (TIM2) but it's still broken on the STM32F3Discovery (ChebuzzF3 target) (TIM1) and the workaround on line 167 is required for the Chebuzz target.


added justwarp capability for all warp spaces


master (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-AddressSanitizer: OK (334 of 334 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


r+


@khadijaibrahim  @samaasallam


Shouldn't this be av_opt_set_double?
It's a floating point value from 0..1 so might be nice to allow this to be set to e.g. 0.5?


1: M100 needs more reviewers
1: M101 signature is missing
1: M110 missing component name


Brilliantly simple solution and it led me to another that feels even more like what I was shooting for.

```
<div ng-model="someFunc" ng-model-options="{ getterSetter: true }" ng-click="$scope.context = elem">
```

This gives me a reliable context that can then be used in the getterSetter.

Thanks


Well, TIM2 (only) is 32bit timer .. i overlooked that. I'll check what needs to be fixed ...


this is nice - however it is a breaking changes for users. Anyway to maintain backwards compatibility?


We can't do this as we need to save the external pipe names that Go runtime sets up *before* CRIU executes.  In other words, we cannot wait for CRIU's completion because by the time CRIU has exited the restored process may have already closed or moved its standard file descriptors.  The current loop implementation is wrong too.

What we need is an "init" process that would in turn start CRIU and wait for its completion.  The init process itself will execute only after it has synced with its parent.  This is exactly the way docker containers are started via dockerinit.  From this point of view, restore is similar to docker start.

Please see https://github.com/SaiedKazemi/docker/blob/cr/vendor/src/github.com/docker/libcontainer/namespaces/restore.go for how it was done in the previous version of libcontainer.


Why does free need to be implemented? Shouldn't there already be an implementation in stdlib.h?


Por que es neceario agregar esta linea de codigo.
Lo digo por que cuando entra a produccition settings.DEBUG tiene que ser false, y aparece error 500.


Remove every tabs. m64p use 4 spaces. :)


:+1: no problem!


Ack :/ It looks like I typo'd this in my last report on the issue, though @pdf had it correct. I only noticed when I tried to remove my custom "System -> Advanced -> System Tunables" setting after upgrading to 15.1.8

net.inet6.ip6.rfc6204w3
(interfaces.inc is missing the 3; net.inet6.ip6.rfc6204w isn't a valid oid according to sysctl)


evertthing below this point in this file in this commit is whitespace cleanups.


this will cause static linking to break..


I'm sorry to say this, but I think this is pretty much ... awkward. What used to be a simple git-wrapper is now a resource editor. What functionality next, formatting USB sticks? (SCNR)


Unnecessary tabs.


master (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-ThreadSanitizer: OK (334 of 334 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


This line appeared twice, only one was needed.


I try to do some work on the view of admin permission pages for 1.0 on weekend using the current OR-permission mechanics. And add some more interesting demo data sql, so we can test corner cases together.


Ah, not formatting USB sticks comes next, but an icon editor!


My pull request should be now in the condition that you could try to merge it with your fork/branch and test with mysql also.


Enabling the debugging of the PPM edge times helped identify the problem.


master (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (334 of 334 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


This is really dangerous for various reasons. It might mask huge gaps by filling in zeros and after the rest of the processing this is no longer obvious while the data is still corrupt. `method=-1` is the only save merging method. Otherwise either figure out why there is a gap in the data in the first place or throw it away. If its only a couple samples you can use `method=1` and `interpolation_samples=N` which is likely ok.

Out of curiosity: When is filling in gaps that are larger than one or two samples ever a good idea?


rax retry tests


saw approval from kellymclaughlin
at https://github.com/helium/plumtree/commit/2d3184898579dc8cf97645c77cbdaa05653581d3


merging helium/plumtree/bugfix/cluster-ops = 2d318489 into he1000-integration-7-bugfix/cluster-ops


helium/plumtree/bugfix/cluster-ops = 2d318489 merged ok, testing candidate = 52e6bf56


Hi Yiyang, this should be params[:team][:name] instead of params[:team_name]. I have fixed it. -Yang


this is okay as the first iteration.  Some disabled sTring  methods accept both strings and reveres, so we'll need finer-grained conditionals later on


Hmm not sure what is best here, let's see if @alexmarsev has an opinion.


master (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (334 of 334 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


master (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


Already tested. It's working. :grin:


I think this ending '\n' should be removed, but it's not important.


I suggest you do one commit for only one functional change in the future. This commit is fine, you do not need to revert it.


Ref #107


>We need to guard against the scenario where SetValue succeeds but the Assert after that fails, causing DeleteValue() to be not called.

PR feedback addressed. `SetValue` doesn't succeeds.


Yep, you're right... it's me that like that algorithm quite a bit... :D


whoops! good find


I haven't got around to working on this but I made an issue (https://github.com/fatih/vim-go/issues/366) so maybe someone else can take a look.


@davidsansome Looks like you wrote this a looooong time ago :p (commit c7f4a091ed1055a01a1278da571fbe32d67171c0)
Looking at the Gstreamer documentation, I believe it's pretty safe to remove this, but the "Save configuration" makes me doubt it is really useless...


I dont understand github at all... anyway, for this to work you need the Tabs.png picture and change the pathname for it to whatever it is for you
![tabs](https://cloud.githubusercontent.com/assets/10598160/6835937/160d973c-d336-11e4-8378-232b0e275899.png)


also in regards to using `PATH` to avoid `bundle exec` - this would be RVM specific solution and if was it using `$PWD/bin` then check http://niczsoft.com/2013/10/please-hack-my-rails/

`rubygems-bundler` is generic reusable solution and it is also happily used by @postmodern s https://github.com/postmodern/chruby users, the only thing distinguishing it(or rather it's dependency: https://github.com/mpapis/executable-hooks) from a hack is merging https://github.com/rubygems/rubygems/pull/590 to Rubygems


The kernel cannot use libc. If you think about it, the functions in libc are often fancy syntactic sugar over system calls (to the kernel...)
Everything that we need must be implemented in a special klibc that is exclusively used for the kernel.


method_missing (arBmind) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (335 of 335 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Instead of using `app.generateKey()` I'd suggest adding something like the following to *content-staging.js*:

    $( "#id-of-the-button" ).click(function(event) {
        event.preventDefault();
        // Call generateKey() from here instead.
    }

The secret key could be generate directly using javascript, avoiding the AJAX call.


all tests pass:
success: http://buildbot.helium.io:8010/builders/test-plumtree-build-debian-7-64/builds/36


fast-forwarding master to he1000-integration-7-bugfix/cluster-ops = 52e6bf56


Where do I use this script in which way? I would like to add this to the explanation.


update_sqlite_library (kbenne) - i386-Windows-7-VisualStudio-12: OK (1675 of 1676 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.94%25-yellow.svg)


`typeof settings.onUpdate !== "function"`


`decoration = "flowers:"..name,`


libart-compiler makes my build freeze, dunno how's with yours


activity_app_like_"faild" failed
activity_app_unlike_"faild" failed


update_sqlite_library (kbenne) - Win64-Windows-7-VisualStudio-12: OK (1675 of 1676 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.94%25-yellow.svg)


And what do you mean with "Broadcast Games"?  game group played... Is it for the watched videos?


no problem, glad it worked!


comment from @joshrowley...

I think this needs to be `d.response.pages.item` based on how you were setting the `path` before. In any case, the best way to debug what's going on is to try `console.log` out to view what this function is returning. You need it to return and array of the objects you wish to iterate over to generate the single views.


yeah, I've put a comment for split, probably there are more


@kiriakosv  Διόρθωσα μερικές ανωμαλίες στο pdf που βγαίναν πολύ εκτός του ορίου ...


might want to add extra information to the message (i.e., on which line the endcode was appended)


i had to scroll for ages to find the peripheral.joystick repo among all the updated game add-ons. not the worst problem in the world haha. in the future i'll adapt my old script that generates these settings.xml and strings.po files


@joshrowley - when I set it to `d.response.pages.item`, I get the following error...

`Error: Error: collection must return an array`

`Potentially unhandled rejection [4] Error: collection must return an array`


What's the reasoning on this? Just don't like it?


We want a min and a max? and use 1.0 by default with < 1.0 above in the comparison?


Add `default:`, just in case.


Shall we consider a warning like this from PC-lint http://gimpel-online.com/MsgRef.html#1556 ?


Thanks @cjmaio, really nice!

I've added a couple of comments to the code. I'd suggest starting with submitting a subset of this pull request containing only the Settings_Ctrl and the settings.php template.

I'd like holding of adding dependencies on Bower and Font Awesome. I like the idea of enabling developers to hook in third-party libraries though, but it needs to be done in a non-forcing way.


Small typo on word "conventional".


Please take a look at commit e160ba448e2abc840dfa5fbfe5d9c84e13b0440e. This makes it error out if the feature was explicitly requested but is not supported.


As talked on IRC you want to have a special lower bound to use what ffmpeg wants - fine with me.


No big deal, but you may have accidentally included

this.$$watchersCount = 0;

in the 1.3 branch (I believe $$watchersCount was introduced in 1.4)


Nitpick: space between > and the name _resourceMap :P


90946736_WSHP_#4567 (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1674 of 1676 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.88%25-yellow.svg)


Why not 32768? :P


I am not a fan of this cop, either - I prefer to always grab the correct arguments in a block even if I'm not going to use them.


Bem Vindos ao SisConcurso!


method_missing (arBmind) - x86_64-Linux-Ubuntu-14.04-clang-3.5-AddressSanitizer: OK (335 of 335 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Hmm, there you already have the iterator, and then delMesh() does another find() on the map? That's a bit unecessary...

Is there a reason you don't just call erase() right there? (Of course, you need to make sure you're not ++ the erased iterator then; erase() returns the next iterator in the sequence for that reason)


saw approval from aberghage
at https://github.com/basho/yokozuna/commit/9ab9f2b0b9dff8d489267b80e14af6779bcc2bb0


merging basho/yokozuna/nix/zl/move-tools-scripts-to-sh+ = 9ab9f2b0 into borshop-integration-475-nix/zl/move-tools-scripts-to-sh+


basho/yokozuna/nix/zl/move-tools-scripts-to-sh+ = 9ab9f2b0 merged ok, testing candidate = 0114c561


References #80


This looks reasonable, could work for the other commit where the "useless" block was removed?


90946736_WSHP_#4567 (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


Oups, sorry, I was thinking this was a PR but realize it's your own repo.

Network in mupen64plus is a huge resquested feature since years so I'm very happy to see you working on this.

bsmile32 did good feedbacks to help you to have a "smooth" PR. I would add:

* m64p core is in pure C. No C++ allowed (you didn't used C++ but just in case).

Don't hesitate to create an issue with _Discussion_ label on the main mupen64plus core [issue tracker](https://github.com/mupen64plus/mupen64plus-core/issues). This way you can discuss with other devs if you have any question or feedback in how core could be enhanced.

Code is quite clean (except tabs :P ).

What you've got for now seems very promising and I really hope you plan to push this in the core once it's finished and everybody agree.

Keep the good work!


References #79


По-умолчанию i-bem.js смотрит аттрибут `data-bem`, - логично, что и тут лучше сделать так. В документации по i-bem.js тоже `data-bem` используется https://ru.bem.info/technology/i-bem/v2/i-bem-js/.

Поправил, чтобы при `_optJsAttrName != "onclick"` значение было без `"return"`,например `data-bem='{ "my-block": { "name": "ya" } }'` вместо  `data-bem='return { "my-block": { "name": "ya" } }'`


I'm okay with not having an install rule while we figure out what to do with loading/unloading x11glvnd. It might be better to put it into a subdirectory, though, or at least give it a name that makes it obvious that it's an example file.


Yes, it's not necessary, I just copied everything at the beginning


Minor nitpick here as well: space between the > and the name


I second @Nielsomat, can we get a bugfix release for this?


The update broken the general usage:

./ffmpeg -loglevel info -re -i /root/bunny.mp4 -vcodec nvenc -preset hp -fflags +genpts -vb 24000k -minrate 24000k -maxrate 24000k -bufsize 48000k -muxrate 26000k -cbr 1 -2pass 0 -r 50 -g 100 -pix_fmt yuv420p -acodec aac -strict -2 -ac 2 -ar 48000 -ab 256k -flush_packets 0 -packetsize 188 -y -f mpegts out.ts

./ffmpeg -loglevel info -re -i /root/bunny.mp4 -vcodec nvenc_h265 -preset hp -fflags +genpts -vb 24000k -minrate 24000k -maxrate 24000k -bufsize 48000k -muxrate 26000k -cbr 1 -2pass 0 -r 50 -g 100 -pix_fmt yuv420p -acodec aac -strict -2 -ac 2 -ar 48000 -ab 256k -flush_packets 0 -packetsize 188 -y -f mpegts out.ts

FFmpeg stucks, does not work for both.

Before with nvenc work working fine with above commands. It looks like -g parameter, -muxrate, -flush_packtes stuck FFmpeg with this update.

Below line works:

./ffmpeg -loglevel verbose -i /root/bunny.mp4 -vcodec nvenc_h265 -preset hp -r 50  -y -f mpegts out.ts


isn't erase returning the next interator a C++11 feature? I thought so anyway (which is why I haven't used it).
delMesh could, in theory, be called elsewhere other than the cleanup (which is why there's a bit of a redundant find() in that case).


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1151
success: http://build.servo.org/builders/linux2/builds/1150
success: http://build.servo.org/builders/android/builds/1185
success: http://build.servo.org/builders/mac1/builds/1364
success: http://build.servo.org/builders/mac2/builds/1367
success: http://build.servo.org/builders/gonk/builds/333


I thought this was backwards compatibility problem but I weren't up-to-date ; sorry


Oh, right, this is a string assignment - changing ENV later on won't affect `dll_path`.


Here's where my idiocy and lack of knowledge of Node... where do I run `console.log`?


Have included the TOS_UID and TOS_CALL_VERSION in this count? Perhaps you could list the calls in the commit message body.


method_missing (arBmind) - x86_64-Linux-Ubuntu-14.04-clang-3.5-ThreadSanitizer: OK (335 of 335 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


While this is something that can be configured in `my.cnf` having it in here makes the script a bit more bulletproof out of the box. Thanks for this.


Hm, I'd rather default the scaler to normal3x, because for at least 95% of the games, none or 2x is much too small.


If `mapView`’s tint color changes after the dot is initialized, how does the dot keep its tint color in sync?


Likewise here the erase() thing.

To avoid code duplication of the deleting, maybe add a private method delMaterial() (and likewise with the Mesh) that takes an iterator and returns the next iterator, and call that from both cleanup() and the old delMaterial()?


Cool. I'll look into fixing that bug first. Validating endpoints wouldn't be very straightforward since there are custom domains to consider. I'll investigate a couple error scenarios though with bad endpoints and make sure they fail in a graceful way before submitting.


fast-forwarding master to auto = 7f587f6c


This patch should be squashed into the previous one.


No caso do Vagrant talvez não, mas se rodar esse script entrando na máquina
com "vagrant ssh" para ir testando "live", ou até rodar dentro de algum
outro box linux não root, talvez vale a pena pedir permissão root no começo?

On Wed, Mar 25, 2015 at 5:31 PM Carlos Villela <notifications@github.com>
wrote:

> no provisionamento do Vagrant, você já é root – então não precisa de sudo
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/servicosgovbr/guia-de-servicos/commit/16ada18d3ed4ffddc974afdf4b9a64d8635d0288#commitcomment-10399140>
> .
>


Right button.


I'm proposing to redo this in #507, and I'm not sure I like/understand the difference here. Is rubocop basically saying that `@@`  variables are not recommended for use anymore?


method_missing (arBmind) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (335 of 335 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Errrm, I wanted to say that I'm pretty sure that's not a C++11 feature, but http://www.cplusplus.com/reference/map/map/erase/ only shows void()? Maybe I'm confused something here?


@Hoverbear it won't be stable for 1.0


sdfsdf


saw approval from jdm
at https://github.com/aweinstock314/servo/commit/cb9cfe2b22e6d80e8b30ad64f2cff5e22a61c284


merging aweinstock314/servo/moz-events = cb9cfe2b into auto


aweinstock314/servo/moz-events = cb9cfe2b merged ok, testing candidate = 9d10d0d7


Should be fixed now with this PR: https://github.com/joomla/joomla-cms/pull/6581
Please test...


InfluxDB 0.9.0 supports tags.  We no longer need to encode properties in series names.


@ericbn, shouldn't update() in Model use dialect to generate query?
insert() uses dialect to generate statement, I suppose update() should as well...? updateParametrized() ?

Also, toUpdate() ignores dirtyAttributeNames

And there is an issue in test:
Line 502 saves a model, then sql is executed (can have errors, whatever, model is already saved with changes)

G.


shouldn't say "iLO" here.


Why so lenient in an API endpoint? I'm not even sure this will make a difference (see [this note](https://docs.djangoproject.com/en/1.5/ref/databases/#sqlite-string-matching)). Also, consider this case:

| username | password |
|---------------|-------------|
| mikE          |    abc123 |
| mike          | abc123    |

With a request that had username `Mike` and password `abc123`, this might have undesired results (logged in as someone else).


*When updating : set Permission "Use internal database authentication" to 1 for all user (update DB)*

Pending

*Really strange to see 'Allow LDAP' and 'Allow webserver' in Permission even if this plugins was deactivated*

Done: https://github.com/LimeSurvey/LimeSurvey/commit/3f6898fb7ad9fd8bb1f74bcb12fe2a2f94da8900

*What for another Authentification method ?*

This is a temporary solution. All this staff will change in LS3. IMHO more improvements aren't worth.


ditto -- this is a generic implementation.


method_missing (arBmind) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (335 of 335 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


This standardizes the noun "state", and removes "quick save", "instant save", "state save", "save state"

This make is easier to work with adjectives and verbs.
**Examples**
with the verb *load*:
 "Load the State" vs "Load the Save State"

and also with the adjective *saved*:
"Load a saved State" vs "Load a saved Save State"


oh, or here....


all tests pass:
success: http://buildbot.bos1/builders/test-yokozuna/builds/1052


method_missing (arBmind) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


Rejecting because this uses the phrase "ilo", when it should be generic.


See [this](http://influxdb.com/blog/2015/02/24/influxdb-v0_9_0-rc4_released_now_with_tags_and_more_unicorns.html)  blog entry.


Not sure if we change this line because we are still supporting Joomla! 2.5 with php 5.3


processing speed bump


At the very least, gcc on this system will insist upon -std=c++11 before it's used.


r+


develop (lefticus) - x86_64-MacOS-10.9-clang-Debug: OK (335 of 335 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Oh, okay, seems like for std::map, this is a C++11 feature. For std::list, this existed prior.

I didn't know that. :D


I like this idea. Should've thought it of myself.


Passed in payment ID, and made both the payment ID and affiliate ID available on the affwp_allow_referral filter


This doesn't have to be inside the lock.


Thanks for the fix. Ran through the options and everything seems to work just fine.


no prob, thanks


great, thanks for checking it out


Right, forgot to include the information that this fixes a crash whenever Git calls a pager inside `git-cmd` and `/cmd/git`.


Thanks :)


I don't have a package called golang.org/x/tools/go/ast/astutil


Please add a test


That's not sensible in my opinion - the entire page will fail instead of being caught in the `__toString()`. Can you instead make sure that it throws the exception instead the render method - in which case `renderTable()` starts making sense.


why you cannot use scalaBinary here?


its there since 11.1 , and J2.5 shipped with that, so it should be good


en español plis


develop (kbenne) - x86_64-MacOS-10.9-clang: OK (1163 of 1165 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.83%25-yellow.svg)


I just noticed this error while loading the package:

*Error loading completion: Error trying to parse file: Unexpected character, expected a comma or closing bracket in Packages/Red/Red.sublime-completions:45:3*

Is the *?* char allowed in names?


Having trouble reproducing this -- all tests still appear to be passing for me and I don't have the package you referenced above, but Mercurial claims to be up-to-date with golang.org/x/tools (??)


Damn that s right, said nothing ;)


Makes sense! I'll start working on your suggested changes. I do agree with
all of them.

Thanks for the code review!
On Wed, Mar 25, 2015 at 4:43 PM Joakim Stenberg <notifications@github.com>
wrote:

> Thanks @cjmaio <https://github.com/cjmaio>, really nice!
>
> I've added a couple of comments to the code. I'd suggest starting with
> submitting a subset of this pull request containing only the Settings_Ctrl
> and the settings.php template.
>
> I'd like holding of adding dependencies on Bower and Font Awesome. I like
> the idea of enabling developers to hook in third-party libraries though,
> but it needs to be done in a non-forcing way.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cjmaio/content-staging/commit/b279d79cbc07c81869e5b626db45c6f6055b9473#commitcomment-10400403>
> .
>


I like this approach. The empty directory is used only for this test, so "use it and delete it" is ok. However, I would like to put creating/deleting dir into before/after methods. Thanks that, directory will be deleted after tests, even when filesToLoad throws exception. @bandrzejczak What do you think?


Nice patch, thanks! I've extended this a bit so that blank names are handled properly, too: 1194957d81ab6b00ab69b742203e001a0feec1df


Should be fixed in the most recent commit - I cleaned up set_amom_bbox quite a bit (and renamed it), and added some comments clarifying that while all the external interfaces are PARENT, some of those within SdssShape.cc are LOCAL.


This is copied directly from our previous login code. I think the intention is to reduce login blocking from miskeying. Happy to remove it if it seems problematic.


They moved to git, try adding git remote and pulling.
On Mar 25, 2015 3:04 PM, "godoctor" <notifications@github.com> wrote:

> Having trouble reproducing this -- all tests still appear to be passing
> for me and I don't have the package you referenced above, but Mercurial
> claims to be up-to-date with golang.org/x/tools (??)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rdallman/godoctor/commit/9665a816476ab94cd9082d82ecba6eddccf2946f#commitcomment-10400780>
> .
>


Yep, that self shouldn't be needed since `socket` is an instance method on Mysql2::Client.


Are you sure this will work?

It takes some time to create and get the machines in a "Running" state. (Not uncommon to take like 10-20 seconds from my experience, which is why I recommended putting this in the cloud init file instead.)


@dudekm I już ta klasa jest OK.

Tak, jest tylko „pośrednikiem” do danych – ale przede wszystkim jest reprezentacją pewnego „pojęcia domeny”: opisuje coś co nazywamy „użytkownikiem serwisu”.

Nieważne czy dane tego użytkownika trzymamy w bazce czy gdzie indziej – ta klasa się nie zmieni i nadal będzie reprezentować użytkownika.


ok so we can let it commented for now


Where is this used?


`then` !? What is this, 2006? :)


@dudekm Zrobiła Ci się kopiuj–wklejka tworzenia użytkownika. Wyciągnij ten kod do metody prywatnej: `createUserFromQueryResult()`


fast-forwarding develop to borshop-integration-475-nix/zl/move-tools-scripts-to-sh+ = 0114c561


Did it not load Prosilver files automaticly if they are missing from the Spring theme?


this doesn't have to be inside the lock. Only execution and removal of operation from queue has to be synchronized, not peeking the next operation.


That's Trello website:
https://trello.com/b/VpE3Gp28/sd19-webforms-c-asp-net


@dudekm rozróżniaj między `$result` i `$this->result` – ten kod nie zadziała


write description


ast_optimizations (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (335 of 335 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


@dudekm A co jeśli wynik jest faktycznie NULLem, czyli brak wyników?

Dwie opcje: zwracaj NULLa albo rzucaj wyjątkiem. Proponuję to drugie:

```
if (null === $result) {
    throw new UserNotFoundException();
}
```


:+1: I'm loving shellcheck!


Modifications for Honami: Disable Selinux, enable Tomoyo, activate disabled by default ciphers, hashes and random number generation tools, enabled CPU voltage table, enable syn cookies.


put in the lock


In the previous version, there was a bigger bug that caused them all to be set to NaN, I think.


working on this now


Ah, yes, I can see how this would be confusing.  I'll add a comment explaining the difference.


ast_optimizations (lefticus) - i386-Windows-8.1-VisualStudio-14: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-0.6%25-red.svg)


Este bloco não está igual aos demais. Depois copia do bloco de baixo, pois adicionei mais formatos.
Cumprimentos


nguyễnviếtsơn


By the login - it needs the list of facilities to login.


I was wondering under what conditions `callback` is called (after a successful response with no redirect and always after an error response). Add an explanatory comment?


Okey dokey :)
On Mar 26, 2015 1:53 AM, "Łukasz Domeradzki" <notifications@github.com>
wrote:

> IMO there's no need for new thread as cm-12.0 will be abandoned once
> cm-12.1 will get stable.
>
> And it's too early to think of it now. I want to see cm-12.1 branch before
> I even consider making a build of it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ArchiDroid/android_device_samsung_i9300/commit/6520eaab3ad677ede57c5bbb399778dff421eb06#commitcomment-10394551>
> .
>


90946736_WSHP_#4567 (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (657 of 658 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.85%25-yellow.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-4.87%25-green.svg)


Why is the context changed in `getTaskStats()`?
I would expect that methods like `getXX` do not alter the state of an object.


I should have clarified: I think these attributes should be initialized in the ViewModel constructor. That way, if you want to know all the state variables you can access from the VM, you only need to look in one place.


Sure.


Hey @bvaughn could you take a look at this? For some reason it's not opening the browser.


I think it's "GameShark" without the space, actually.

It's one of those slang proper nouns where you capitalize while also co-joining the terms as a compound word without the space, but I don't really care either way I guess.  Because English does that an awful lot anyway.


A simpler test would be to block the deletion of operation from the store and attempt an operation on the same item. If the operation succeeds then we have an error, otherwise test passes. You can put a timeout on second operation (because it won't get access to lock as the delete of operation is blocked).


@erikvullings lets move the out discussion here : https://github.com/TypeStrong/atom-typescript/issues/206


No it will print a protocol message. Which probably doesn't help the end user figure out that there credentials were not accepted. I should print a more user friendly message to the user.


You should make a list of system constraints and issue if this comman fails.


It would be useful to have some instructions about building TLK-D with a pre-built TLK. Also, with NEED_BL32=1 in the TLK-D makefile, it will not be possible to compile the dispatcher code without at least specifying a prebuilt TLK binary on the command line. So some instructions, in perhaps the referenced pdf will be useful.


Looks like the `app` value changes based on OS: chrome, google-chrome, google chrome


Maybe ditch `app` and let the OS use the default browser?


what can be done?


Video prove for commit:
https://plus.google.com/u/0/photos/+PavelBondar/albums/6130301247557109089/6130301246506974610?pid=6130301246506974610&oid=110691682057328261107


I think you may also need to specify an *actual* file instead of just a directory.


May be problematic in an edge case, but mostly I think since we're using SQLite that the inexact match is either redundant (because SQLite will quietly look for inexact matches anyway) or useless ("For strings containing characters outside the ASCII range").


la même page passe ainsi de 183013 octets à 203329
(en gzip, de 38437 à 51932)
et de 123 hits à 83.


Is there any reason that the mapping is not saved using `name -> id` instead of `id -> name`?


This is in the method `JESftp.retrieveJob` the caller actually checks for an exception in main.  The caller catches `JESftpError`


http://dictionary.reference.com/browse/fullscreen

I was fine with "full screen", though maybe it should have really said
`"Change emulation from window mode to full-screen mode."` with the hyphen in it.

without the hyphen to begin with when it said:
"to full screen mode"
...it sounds open to interpretation, like is it to **A** full mode related to a screen?  Is it to a full screen with a mode?  Is it to a full-screen mode?  The hyphen or lack thereof makes a difference.


do you have time to give it a spin?


Don't merge to master yet!


Hello!
There I find another issue maybe there is a connection or something missing from my database.
When I choose upfront method then buyer can not sign order, there error shows up "invalid transaction." -- but only on upfront when I select a escrow method then signing working correctly.
Do you have any ideas what can be wrong?
And please it is possible to check the correctness of value "Address Index" at bitcoin admin in BitWasp (there is some warning: Only change this if you know what you're doing! {I never changed this, but I do not know where can error be})?


Ești pe val!


Will just nix it then.


I'm at the office so I can't right now.


*Command.formatSyntax()


:+1:

For reference:
* Content-Type: http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.17


Am Note 4 hat diese geholfen.


ast_optimizations (lefticus) - Win64-Windows-8.1-VisualStudio-14: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-0.6%25-red.svg)


heh


Greg you are absolutely hilarious.


I second this. It's unnecessarily unsafe and makes little sense, considering one could just use StringID as block ID and NBTTagInt as schematic-internal ID.


@Hendy Are there any "magic" variables available for the XPath? E.g., `$currentPage` or `$parentPage` - then I'll add them here.


1) We use UPPER CASE for SQL keywords -- `COUNT`, `SUM`, `CASE` etc.
2) More importantly, I am nervous about the performance impact here. If optimizer would decide to use table merge instead of subselects here, we would be in trouble.
For this reason, either derived table `(SELECT ... WHERE RD.req_id=R.id)` without `GROUP BY`, or two explicit subselects might be preferable.
Also, perhaps guard this feature by `$is_root` check to access the impact without immediately dropping performance for normal users.


Thanks for the reference. On second thought I will put in a more standard-friendly fix and rebase this.


I'd prefer to see the ChatCommandFlag moved to the second parameter in the ChatCommand ctor.
It will start to get messy when we start adding more aliases and localized aliases to the end.


ok. please take a look when you had time. I spent too much time fiddling it!

None of the advice here worked for me https://github.com/stevelacy/gulp-open/issues/2


commit ddddd


This repeated part is rather long now, perhaps extract it in a manner similar to `$no_de`


Thanks for the feedback - could you please check with the latest code. If that doesn't improve please open a ticket including what settings you are using for remarketing so that we can try to replicate. Also what's your php version?


I thought specifying the plumber was a bad thing, wasn't there an issue about this somewhere along the way?


I'm not really sure whether "mempak" is a proper noun or just some shitty common noun that Nintendo made up.  I didn't even think "pak" was a word that could refer to packed memory or whatever those people were trying to convey.  I think it's just slang.

I think if grammar had the most priority there, people would call it "controller pak file".
http://dictionary.reference.com/browse/pak
Still sounds dumb though.  It's N's fault.


ast_optimizations (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-AddressSanitizer: OK (335 of 335 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


About everything else looks like good improvement I guess.


NANANNAA


This part duplicates subquery in `Console.pm`
This is probably what I was talking about during our meeting.
Perhaps at least add a TODO comment?


master (lefticus) - x86_64-MacOS-10.9-clang: OK (334 of 334 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Good .. That I'm not nuts on this one!

Actually I think there are some more issues with those query's ... trying some stuff now but will need you to take a close look at the changes.  The basics are; the left joins on those updated query's I think should all be inner joins since they are exclusive not inclusive.

LOL on 1.1, yeah this one will require a separate PR ot that branch to even have a change, I'll do that once we feel these are fixed.


It's a toss up. Many programs "fullscreen", and many others use "full screen".
For example:
VLC, PCSX2, Dolphin, and PPSSPP, Firefox use "fullscreen"
WMP, YouTube, Jnes use "full screen"

Personally, I like as a compound word -- Fullscreen


@jemmybutton I've pulled in your examples, hope you don't mind. I did change showField -> showfield to be consistent with other parameter names.


Display this part conditionally (`[% IF total_count %]`), make sure `accepted_count` is not empty/undefined instead of zero.


I checked "View" menu under my Palemoon Firefox, and it says "Full Screen", with the space.

But whatever.  Over the years the English dictionary keeps expanding with more and more added words and contexts, like "Googling:  do a Google search".  Maybe "fullscreen" becomes a word too.


This line still needs a 2.3.0.0 entry.  Otherwise LGTM.


Ой, абсолютные пути в конфигах это не гуд.


ast_optimizations (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-ThreadSanitizer: OK (335 of 335 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Is there a charge if an instructor arranges a workshop without assistance from SWC foundation? eg for their own research group, class, dept, or otherwise?


I think it would make more sense to call this IMutableUFNode to fit the pattern of IUFNode and UFNode.


Should this be a call to `read_member_named`?


Однострочные комментарии взрывают дифф :( Лучше бы {- ... -}.


Did you mean to get rid of the border that designated these as buttons? When you hover over them, they look like buttons (dark background). I'm not sure there would be an appropriate icon for `Deselect All` unless you can think of one. If we're going to have the icons change along with the text, let's get that working before we merge this in.

![Buttons](http://i.imgur.com/qTtYEEy.png)

We also might think about having the select button on the left and the expand button on the right, so that they mimic the individual buttons.


What's this let[] for?


`remainder` это хорошо, обобряю!


In that case, may be you can do this, the `startswith` is not needed anymore.

```
cherrypy.request.method == 'POST' and cherrypy.request.headers.get(
'Content-Type', '').split(';', 1)[0] in (
    'application/x-www-form-urlencoded',
    'multipart/form-data', 'text/plain'))):
```


Why do you pass `$self->{debug}` here but not in previous call? Did you test this code?


So neither of the things I suggested above resolved it?


ast_optimizations (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (335 of 335 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


I'm pretty sure this is the reson our new builds are failing!  #286


What about:

```javascript
describe('"formats" option', function() {
  it('allows adding alternative syntax for comments', function (done) {
  });
});
```


Nope - only for those we help to arrange.  This is mentioned in point #2
of http://software-carpentry.org/scf/workshops.html - should the FAQ
repeat that, and link to workshops.html ?
Thanks,
Greg

On 2015-03-25 6:49 PM, David LeBauer wrote:
>
> Is there a charge if an instructor arranges a workshop without
> assistance from SWC foundation? eg for their own research group,
> class, dept, or otherwise?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/swcarpentry/site/commit/3a3b2d984afa9429253bfa1ea0a5bbc3e654bf3c#commitcomment-10401399>.
>

--
Dr. Greg Wilson    | gvwilson@software-carpentry.org
Software Carpentry | http://software-carpentry.org


These can be reduced to one line:

```
margin: 50px 0;
```


`{ $_[0]->{dir} }`


@ehazlett `machine --version` is still returning the old version number `0.1.0` even though it has the correct SHA aeb89ff

```bash
$ machine --version
machine version 0.1.0 (aeb89ff)
```


[`vistrails/gui/resources/images/disclaimer.png`](https://github.com/VisTrails/VisTrails/blob/ad9e4ec0acb74382d1be2b1c613d624dd796b2cc/vistrails/gui/resources/images/disclaimer.png) needs to be updated


А какой должен иметь смысл `mplus` для парсера с вашей точки зрения?


ast_optimizations (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (335 of 335 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


This block is called inside the loop, which means it can potentially be executed more than once, and potentially leak memory (as subPreQueues isn't free'ed then).

Wouldn't it make more sense to call it before the entire seeking loop?


ast_optimizations (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg)


Bien! cierto fue error de copy paste :+1: