 - __information__: [src/chaiscript_stdlib.cpp:0](../blob/2a8c248167817428bfbf0f06c723d908117f0e0d/src/chaiscript_stdlib.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 16 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:679](../blob/2a8c248167817428bfbf0f06c723d908117f0e0d/include/chaiscript/language/chaiscript_parser.hpp#L679) Expression is always false because 'else if' condition matches previous condition at line 677. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:683](../blob/2a8c248167817428bfbf0f06c723d908117f0e0d/include/chaiscript/language/chaiscript_parser.hpp#L683) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/2a8c248167817428bfbf0f06c723d908117f0e0d/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/2a8c248167817428bfbf0f06c723d908117f0e0d/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 683. (multiCondition)
 - __information__: [src/chaiscript_stdlib.cpp:1](../blob/2a8c248167817428bfbf0f06c723d908117f0e0d/src/chaiscript_stdlib.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __information__: [src/main.cpp:0](../blob/2a8c248167817428bfbf0f06c723d908117f0e0d/src/main.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 17 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __information__: [src/main.cpp:1](../blob/2a8c248167817428bfbf0f06c723d908117f0e0d/src/main.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/2a8c248167817428bfbf0f06c723d908117f0e0d/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/2a8c248167817428bfbf0f06c723d908117f0e0d/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/2a8c248167817428bfbf0f06c723d908117f0e0d/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/2a8c248167817428bfbf0f06c723d908117f0e0d/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/2a8c248167817428bfbf0f06c723d908117f0e0d/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


```php
$obj = new \stdClass();
$obj->state = true;
$instance->put($obj, 1);

$pairs = $instance->pairs()->toArray();

var_dump($pairs[0]);
var_dump($pairs[0]->key);
var_dump($pairs[0]->key->state);

$pairs[0]->key->state = false;
```

```
object(Ds\Pair)#2901 (2) {
  ["key"]=>
  object(stdClass)#2907 (1) {
    ["state"]=>
    bool(true)
  }
  ["value"]=>
  int(1)
}

object(stdClass)#2907 (1) {
  ["state"]=>
  bool(true)
}

bool(true)
```

`RuntimeException: Creating default object from empty value`


It said cwilen in the lower left corner of the commit window.  If that
doesn't mean we are committing as that person, then that is a horrible
interface.

On Fri, Mar 4, 2016 at 2:20 PM, patzinak <notifications@github.com> wrote:

> As far as I understand there are two different things: GitHub account and
> git configuration, You may have logged in as yourself on GitHub but pushed
> the committed the changes without modifying the git configuration of the
> local repo.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/McDermott-Group/servers/commit/ff773bb7d8fc7570694040f0f915bd607ec53169#commitcomment-16509736>
> .
>


This is an interesting idea. However, with the react material-ui components it seems difficult to implement as it requires overriding some existing CSS rules and it creates a not consistent UI in which some sentences are too flush against the end of the element containing them. I will forgo implementing this at this current version but will consider it again in the future.


I only kept it because I wasn't sure what to do with the comment.  Does it still apply?


vhcl.setSociete(null); na77it beha el recursivité ,makenech t3awed tserialisi rou7ha l'entité
we tjini exception m3awda 1000 marra fil console
===> same situation for all the other attributes setted to null


As far as I understand, Issue.ts does contain an `id` field but it is not set from firebase, where it is also not stored. This is perhaps a misleading issue with the Issue class.


Remove the old code? I saw somewhere a note that doing so is part of the Drill coding standards...


All `ident` have been changed to `ty` now.


![twilight s_friends_waving_goodbye_s4e01](https://cloud.githubusercontent.com/assets/7750/13542559/de712bb6-e263-11e5-9cb6-9d5c4973f67d.png)


The method has been reimplemented by the backend team.


See note above. The logger is a static pointer, and so is not destroyed at the end of the run. As a result, the output file may not be closed and, when writing other than to stderr/out, the output buffer may not be flushed.


Have documented the expected behaviour of division and remainder for signed integers.


does auto fix the problem for the mini boss on screen


We shouldn't be printing in tests. Maybe assert expected output instead?


You could also use our good friend ButterKnife here:

@Bind(R.id. tvPetFavName) TextView mPetFavName;
@Bind(R.id.tvPetFavSex) TextView mPetFavSex;


Remove println.


@tholman, why is there a ` here? Is it a mistake?


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.9: OK (397 of 397 tests passed, 0 test warnings)

![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 wonder if it's worth keeping these sorted and using binary search.


and then these lines would go away and be replace with code to bind the UI elements in our view with our view holder:

```
ButterKnife.bind(this, view);
```


Yep, mistake.


This rand() use is crap... Need to fix with rand48 or something


this method returns a json file that contains  a company  and the list of its vehicles
@YoloWF  you will be using this mtehod
i will upload some screenshots


vhcl.setSociete(null); na77it beha el recursivité ,makenech t3awed tsérialisi rou7ha l'entité
we tjini exception m3awda 1000 marra fil console
===> same situation for all the other attributes setted to null


Fixed.


Oh, wow! Thanks @gadicc, great pickup


did you have a bad merge? We already have a file called CritterFinderApplication in the project.


How about
 tpaige1271

Sent from my iPhone

On Mar 4, 2016, at 5:09 PM, Andy Chan <notifications@github.com<mailto:notifications@github.com>> wrote:

Can we do name-prefix or suffix instead? With a name, it is very easy to have name collision since duplicate name with KMS key is not allowed.

-
Reply to this email directly or view it on GitHub<https://github.com/TimeIncOSS/terraform/commit/42f95353edf9981261e8d3001c7f85fca56cf38a#commitcomment-16511834>.

Please be alert for any emails that may ask you for login information or directs you to login via a link. If you believe this message is a phish or aren't sure whether this message is trustworthy, please send the original message as an attachment to 'phishing@timeinc.com<mailto:phishing@timeinc.com>'.


Interesting that P* algorithm. Would one need a weight(t) distribution for every edge?

And will it give you a range for the ETA or what exact benefit over an exact time-depending calculation would one get?


TeamCity LabKey :: 16.1 Release :: Community :: CDS postgres 9.5 [Build 14](https://teamcity.labkey.org/viewLog.html?buildId=307946&buildTypeId=LabKey_161Release_Community_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 8 (1 new), passed: 39, muted: 4 Build time: 01:14:49
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyFilters: <no details avaliable>

org.labkey.test.tests.CDSSecurityTest.verifyUserPermissions: <no details avaliable>

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyStudyAndTreatmentVars: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyLogGutterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyGutterPlotBrushing: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisFilter: <no details avaliable>

```


Looks in good order @mrpatrickwest
One question, in the xsl:choose for each template, the default choice is <xsl:text> </xsl:text>, is there a specific reason for that?


We recently fixed some bugs in projection pushdown that were causing problems like this. It may no longer be relevant.


develop (energy-plus) - x86_64-MacOS-10.9-clang: OK (1574 of 1574 tests passed, 0 test warnings)

![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)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18386](https://teamcity.mesosphere.io/viewLog.html?buildId=154610&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 250 Build time: 00:04:54


Visit project here: [http://codepen.io/BakMamba74/pen/dGxmpP](url)


Let me know if you need help figuring out how to use the CLI tool! It's particularly tricky around ambiguous imports (where the user has to select from a list).


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.9-NoThreads: OK (393 of 396 tests passed, 0 test warnings)

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


@zack-shoylev Carina is currently on `DOCKER_VERSION=1.10.1`


Hard to read. Since we do nothing in the "else" case, might be clearer to "short circuit" this and say:

if (initialized || path == NULL || ! m_filepath.empty())
  return;
...


Ah, I copied this from checksum file on S3, we built binaries for armv6, armv7 and finally armel so they show up as we see here. Anyway, we can ignore the order here since there is only one record for each binary @petrosagg


This is still very repetitive with importFunctionType. Can we fold more of this logic into that, perhaps passing down the DeclContext we're importing into and information about which parameter of the function stores "self"?


Visit project here: [http://codepen.io/BakMamba74/pen/OMeXOv](url)


We could do that... If it were a free form text value then I would have not made it required, but since it is using allowable values I thought it was simpler to make it required with a default value, this way it will never be empty, and then it only gets used if the SSL Context Service is set.


Added links referring to rust's reference about signed machine types where the two's complement representation is listed.


Why change this ? This is from 0.3.0


Again, I feel like we have a lot of repeated code here with the normal VisitFunctionDecl path, and I'm concerned that we won't be able to keep them in sync.


nit: The #region seems unnecessary.


Possibly, I currently want to implement it correctly first, then maybe we can start to fiddle around with bitwise operations to speed it up.


I should have split this commit in multiple ones... But having 2 minutes to write, commit and push made me do this... :-1:


brutal replacement failure hahaha


release-5.x (lefticus) - Win64-Windows-10-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-92.56%25-yellow.svg)


will fix


@JasonGlazer You need to keep this line.  One at the top for the single-pass, and one here for the InitOAControllerSetUpAirLoopHVACVariables block which loops over all the oacontrollers.


Should probably update the copyright header to 2016 for all the changed files.


There is another counter in the getMessage() method which counts individual messages received, so by having this one it makes it easy to see how the batching is working from the counter page in the UI. Basically, if you set batch size to 100 and sent 1000 messages, the other counter should be 1000 and this one should 10.


In the docs/release notes, I think we could emphasize "testing" as the reason for this feature. After all, if Django's commands need to be tested this way, chances are that third-party commands have similar needs.


I think we can remove it.


@ravnon `Issue.ts` [does have an ID field](https://github.com/aaronnech/CheckYourBias/blob/master/src/common/Issue.ts#L25). Are you not able to access it?


change the 'add my bug fix over here' to 'add my bug fix over here via bugfix branch'.


Bueno,

Pues les queda de tarea, ya tienen conocimientos de sobra no sólo para
mejorar la aplicación sino también para arreglarla!

Abrazos.

On Fri, Mar 4, 2016 at 7:46 PM, Ezequiel Tojo <notifications@github.com>
wrote:

> creo que rompimos algo porque ahora no puedo crear items desde el html
> aunque desde postman le pega bien
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/surfercoder/node-mvc/commit/967dfb41b93c59cbb4b7edb70228084048e0f558#commitcomment-16512582>
> .
>


Agreed. In fact, if the file can't be opened, the code will crash on line 75 when we attempt to use m_pOutFileStream. If the file can't be opened, either need to throw an exception or default to using stdout (and writing a long message that the requested file can't be opened.)


I am using python 3.
In order to be able to compile I had to add link to X11:
-L/usr/X11/lib -lX11


Если используешь мастер страницу, то все html, head, body пишешь в мастер странице, а здесь только уникальный контент. Я завтра покажу всем как делать то, что ты пытаешься сделать здесь с помощью head. Но он генерирует неверную разметку здесь, хоть и выглядит что она выполняется верно.


This is wrong, since the ports change everytime...


Hey @JamesRandall Thanks for the update!  Any idea when this will be ready?


Fair point


A nit. But, it is helpful to write the path to the file that can't be opened to give the user a bit more info when tracking down the problem.


Looks good!


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18387](https://teamcity.mesosphere.io/viewLog.html?buildId=154617&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 250 Build time: 00:05:05


Thanks! Looks good.


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-5: OK (397 of 397 tests passed, 0 test warnings)

![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)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18388](https://teamcity.mesosphere.io/viewLog.html?buildId=154618&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 250 Build time: 00:05:41


What took so long? ;)


release-5.x (lefticus) - i386-Windows-10-VisualStudio-12: Build Failed

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


We were just told that we can't open the file, but now we are deleting the file? Suggestion: if we can't delete the file to create a new one, we can't delete it. Perhaps omit this line.


Meglio
_________


The is_open check is unecessary. We already checked that above and delete the pointer (and set the pointer to null.) The file shouldn't close between then and now...


Needs an empty line above.


Done in 762da4a.


gzdfhdghg


vei, o que está comentado deleta.


@miguelportilla You have correctly discovered some inefficiencies in the implementation with respect to `JobTypeData`. But I can do one better, why not just have the `Job` store a `JobTypeInfo*` as a data member? Then we only need look it up once. Anyway, a full refactoring of `JobQueue` is beyond the scope of my changes.


Does this fix the Bungeecord bug?


Loosened up tolerance in 4cd2da8.


Yeah, I agree that the safest thing is to use std::atomic. However, that will be a little bit clunky, because it will mean modifying all of the accesses to that variable (which should all be memory_order_relaxed in order to avoid a performance penalty).


Why is this not final?


Added in 4521adf89059d5600e241ba9c8a2fdc9a2c8a1b4.


vei, o qeu está comentado deleta, nao vamos deixar nenhum codigo comentado, em comentario soh comentario


Can we get this pushed as a new release? All the builds my company does are failing since we have GitBook configured to the latest release via npm. v2.1.0 works for us but 2.1.1 fails when executing `gitbook build`.


@evsh I think you answered to the wrong comment.


Done in a667005.


So how do we go about exposing ports when we don't know which until the container starts????


The default:;
is normally written as
default:

; not needed.


comment


default:;
looks kinda weird;
default:

is more normal.


Default html page is index.html or index.htm not **Index.html**


Okay.  I'll delete the comment and move the function into `backjump`.


This line may attempt to write a null if I called init( null ). Do you want to say:

... << (path == null) : "stdout" ? path


lazemni nzid'ha sinon maynajemch yfassa5 entité detaché ,
t9olli 3lech détaché => n9ollek 3la 5ater List_compte() 7atit feha em.clear()
t9olli 3lech clear() ==> ahbet louta taw ta3ref


> On Mar 4, 2016, at 2:43 PM, rjmccall <notifications@github.com> wrote:
>
> This is going to force us to eagerly emit type metadata for anything with an archetype in it, regardless of whether it's just an archetype or not; e.g. [T]. That seems unnecessary and expensive.

What doing an isa<ArchetypeType>() on the desugared type be a better condition?

> Also, I thought the problem here was that you specifically needed these things emitted in the prologue.

Forcing the metadata to be emitted before the debug intrinsic was good enough as a quick fix to unbreak the LLDB testsuite because the debug intrinsics for function arguments will be inside the prologue.

In addition, we can force-emit the type metadata for all primary archetypes in the prologue.
I think together this should cover all cases that can reasonably be expected (until I can figure out a way to emit DWARF expressions for getting to the type metadata).

-- adrian


but why


type doesn't have to be any, right?


The process id is written only for windows. The C standard library has a function to get the pid for Linux systems: getpid( ). In fact, this function probably works for Windows as well...


delete this ;


Gute Idee mit der Anleitung :+1:
Vielleicht langt es den Link auf "SSH Key" (bei ... ein Passwort oder einen SSH Key hinterlegen ... ) zu legen, ohne einen zusätzlichen Satz einzufügen


@rj0495 Not as all, just a simple cleanup.


@sledgehammer999 , indeed! Thank you!


@marijnh This test is flaky. If I let it run in the browser it fails. If I step through with a debugger, it passes. I assume this may have to do with how ProseMirror flushes to the DOM asynchronously. Have you had to deal with this in other tests?


FixCrashWithZoneAndSystemERV (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1021 of 1021 tests passed, 0 test warnings)

![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-30.13%25-green.svg)


This is just a small optimization, does not affect the behaviour of the plugin in any way.
Short answer, no.


I see only single place where it is deleted, while in the second one you assign a new value after deletion. The second case is even simpler with a smart pointer. The first one also is not a problem, just a reset() call will do what we need. What is more important, in my eyes, is that the type of the smart pointer bears owning strategy information.


Might this going to cause any harm?


FixCrashWithZoneAndSystemERV (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


I'm basically referencing #117


release-5.x (lefticus) - Win64-Windows-10-VisualStudio-12: Build Failed

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


em.clear() bech tnadhef el session ..we ki nbaddel fil les entités fi des classes o5rin (kima 3malt fil rest we 7atit des attributs null) maysirech MAJ lel les valeurs  fil base de données
Rque :Select_Compte fait appel a cette methode(et à em.clear())


If compiler reorders a test and a lock, there is certainly a bug in it :) My point is that you can return early without obtaining the lock, which is very expensive operation comparing to the check.


@coten leave a line note somewhere if you want me to check something out


same problem

declare then as const, don't case them this way.


This closes the file output stream if we opened it, which is good. But, there is a hole.

getLogger( ).init( "foo" )
getLogger( ).close( );
DRILL_LOG( level ) << "I crash";

Will crash because m_pOutStream still points to the now-deleted file.

But,

getLogger( ).init( null )
getLogger( ).close( );
DRILL_LOG( level ) << "I work";

Works, because m_pOutStream still points to stdout.

To fix this, maybe put m_pOutStream back to stdout in the close? And, set initialized back to false?


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Added a json error for handling a connection error with mysqli.
Changed following codes to suit (OCD wins out).
Added function call for freeing up result of mysqli prepared statement binding.


update_travis_toolchain (lefticus) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


5tart .size() bech el relation lazy tssirelha chargement mel base ....


same here. The const should not be lost.


good catch!


el test lazem makenech ken ma3andouch des véhicules ==> exception


If it's passed in as const, casting it to non-const is the wrong way to go. YOu need to fix the declaration.


nice trick!


The redirect can now be set by specifying a `newversionredirect` parameter in a page's front matter. Someone will have to manually add that parameter to pages that don't exist in newer docs. Hope that's a good compromise.


Leave a line note


I'm not entirely sure if this is thread-safe, in the sense that if we get multiple requests, it might not create an instance of the variable for each one. I might be wrong, but from what I remember from MVC, I think using HttpContext.Session["variableName"] to store information for the current session might be safer.


:warning:  @coten super important - dont do this, there is a bug!!!  don't return a multi return with one item in the dictionary, just return a list of strings instead.


Лучше делать строготипизированные представления. В шапке писать @model ТипДанных


I would be hesitant to put it into a common file since it is only used once in this file but perhaps we can find a solution of creating a non common file that holds all the literal copy.


check this out if you want to name the port file paths:
https://github.com/DynamoDS/Dynamo/wiki/Zero-Touch-Plugin-Development#documentation-tooltips-and-search


1.4 is the latest


Pretty useful changes


![life2015-2016 004](https://cloud.githubusercontent.com/assets/17658051/13543054/8a99fece-e21c-11e5-8331-cb4f6da7676b.JPG)
## A clear sky


If the token is secret/sensitive, use the protected settings to pass the commandToExecute property


it would be best to reference the protogeometry dlls from the extern folder


Please keep the parameters at top for consistency with other templates and makes it easier to read the flow of the template from top to bottom


typo - use canola


This has nothing to do with #117 , and not with any error at all.


btw cb is a piece of crap and I'd be just as happy if you submit a CL with
rm -rf sys/src/cb
in it.


as above - just return your list directly.


Yeah lol


@bmalets here it comes 812318ff6358a485fed81fe157c13130d3eedcbe


Hey Les, I tried a horizontal arrangement, but with our main page's restricted width and navigation bar I couldn't find anything that I was happy with.
-
Jakub Swiatczak
Technical Writer
Stormpath

#:  416-884-1728
@:  JakubSwiatczak

On Fri, Mar 4, 2016 at 2:34 PM -0800, "Les Hazlewood" <notifications@github.com> wrote:

Nice work!  Just a minor comment about the Tenant diagram:

I think of Accounts and Groups within a directory as peers - not in a hierarchy.  I'd probably represent the membership as a horizontal line between an Account box and a Group box, not a vertical one (as that usually implies hierarchy or communication flow).  Dunno if you'd find that useful. HTH!

—
Reply to this email directly or view it on GitHub.


you probably want to use AlmostEqual


The code is fine, but I wonder about the design. This is a poor little client that just wants to submit a query. Why do I need to link with and connect to ZK? In the future, perhaps offer some intermediate service so that the client connects to that service, gets a drill bit, and then connects to the drill bit. This isolates the client from ZK.


I would suggest a conditional operation here via the ? operator to handle cases where the _recentIp is not determined. Might happen if the session expires, since this data is quite volatile.

On that note, minor suggestion: if you could rename it to _recentIp, that would be great, just to follow the Camel Case convention. No big deal, though, just pointing it out.


Oh Jonah, your kittehs model is spectacular. I shall accept this pull request.


Stupid git client.  :(


Oof. Good catch!


Well that's the exception. I'll preform some of my own testing now.


Nice!


Looks good! You can also use the `labal` tag to add titles for each field, as opposed to the header tags. The `label` tags `for` attribute should have a value equal to the input's ID without the hash tag.


This function needs a test


Does it support nesting?


Yes, come to think of it this was basically the test.

Passed on my Linux machine as well, but not Travis...


lol it'll help me a lot cuz i'm still a php noob


this branch is using another database structure , so each time you checkout a branch make sure you reset the database @YoloWF


Why are we statically linking libunwind? This seems like it will cause more problems than it might solve


lol //technical stuff


@crisbeto Seems like this line is now wrong?


Much better :+1:


does this actually rotate the line correctly?


Nice job using the `toggleClass` method. Looks like you just have to add the remove item functionality. I'm marking this assignment as complete. Good work!


@petrosagg we use 4.x and 5.x node binaries from official distribution for armv6 and armv7 so don't need to list them here


Can these three lines just be removed?


HarryPotter не очень понятное название для  класса. Так как он генерирует список книг, то название класса не должно зависеть так явно от того, какие данные генерируются. Лучше BooksFactory


Ты делаешь меню через мастер страницу, но здесь явно ее обнуляешь, поэтому меню нет.


missed this line


 - __information__: [src/chaiscript_stdlib.cpp:0](../blob/0eee23109e7d47dc5185e1578b167a0fa79cea66/src/chaiscript_stdlib.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 16 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:679](../blob/0eee23109e7d47dc5185e1578b167a0fa79cea66/include/chaiscript/language/chaiscript_parser.hpp#L679) Expression is always false because 'else if' condition matches previous condition at line 677. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:683](../blob/0eee23109e7d47dc5185e1578b167a0fa79cea66/include/chaiscript/language/chaiscript_parser.hpp#L683) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/0eee23109e7d47dc5185e1578b167a0fa79cea66/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/0eee23109e7d47dc5185e1578b167a0fa79cea66/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 683. (multiCondition)
 - __information__: [src/chaiscript_stdlib.cpp:1](../blob/0eee23109e7d47dc5185e1578b167a0fa79cea66/src/chaiscript_stdlib.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __information__: [src/main.cpp:0](../blob/0eee23109e7d47dc5185e1578b167a0fa79cea66/src/main.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 17 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __information__: [src/main.cpp:1](../blob/0eee23109e7d47dc5185e1578b167a0fa79cea66/src/main.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/0eee23109e7d47dc5185e1578b167a0fa79cea66/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/0eee23109e7d47dc5185e1578b167a0fa79cea66/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/0eee23109e7d47dc5185e1578b167a0fa79cea66/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/0eee23109e7d47dc5185e1578b167a0fa79cea66/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/0eee23109e7d47dc5185e1578b167a0fa79cea66/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


and the first reference on this one


*references*


Thank you for the feedback. It has been addressed in #63


Not thread-safe as written due to hidden static state ot strtok( ).


Updated errors


LaTeXDummy (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Isn't SectionAddr the key? it is unnecessary and slightly confusing to have it as part of the value also


i'm cool with this though; that's not a thing i like, need or anything
:+1:


@chris-mike @cptvitamin Here's what I came up with to fix the most egregious issues.

The first rule, `.response__submission__status__title`, applies to subtitle text that was intended to be very small (see https://github.com/edx/edx-ora2/blob/master/openassessment/xblock/static/sass/oa/views/_oa-base.scss#L555, which eventually leads to https://github.com/edx/edx-ora2/blob/master/openassessment/xblock/static/sass/xb/elements/_typography.scss#L112).

Before:

After:

The second rule applies to cases where we have an `h3` inside of a `header` element, both of which end up getting `font-size: 1.5em;` from different places. This is why we were seeing h3 elements of varying sizes.
Before:

After:


Здесь меню будет выводится, но разметка будет сгенерирована неверная, так как получится что берется информация из мастер страницы вся(все теги htm, head, body) и будут добавлены теги из этого представления. То есть если используеш мастер страницу оставляешь только уникальный контент, то есть тегов htm, head, body не должно в нем быть


if it works with 0.9 and 0.10 might be better to do `> 0.9`?


@jvillarba Looks good except post schema should reference User not Post.


@jvillarba check comments


defaults to the nanny's namespace


typed from website word 001


I tested on mac/windows/linux.. everything passes. Weird!


Sorry I wasn't clear -- you need the full copyright block (see pretty much any other go file for an example, e.g. https://github.com/kubernetes/kubernetes/blob/master/pkg/util/flock/flock_other.go#L3-L17).


It really depends on why you're doing it.  Do you just want to make sure the archtypes involved are bound, or do you care specifically about the compound metadata?  Because, for the former, you can do something like:

  // Force all the archetypes referenced by the type to be bound by this point.
  // TODO: just make sure that we have a path to them that debug info can follow
  if (type->hasArchetype()) {
    type.visit([&](Type t) {
      if (auto archetype = dyn_cast<ArchetypeType>(CanType(t)))
        emitTypeMetadataRef(IGF, archetype);
    });
  }


@DarkSilencerZA What do you mean can you maybe send screen shots?


One of the files in the nanny package should have a package level comment.


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (398 of 398 tests passed, 0 test warnings)

![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)


Miight be a bit clearer to use short-circuit here:

if ( this->m_bIsConnected )
  return CONN_SUCCESS;

Although, really, if I'm connecting when already connected, this should be an error, or the previous connection should be closed and a new one opened. Else, I'll may be talking to a service different than the one I requested.


We hold onto the connStr, but accoding to the strtok docs (used in parseConnectStr) the string will be overwritten. See http://en.cppreference.com/w/c/string/byte/strtok


you can use `fmt.Errorf()` here to avoid nesting `errors.New()` and `fmt.Sprintf()`


I see, if I were you I would reconsider this solution. With scrolling containers you introduce a whole new set of issues and use-cases that you don't have to even worry about with standard page scroll, just to mention one restore the scroll position when you do pjax navigation or the one from @shaunandrews' comment. Not to mention that scrolling container don't work well in *older* mobile devices.


I made a PR: #3209


this comment should be moved up to line 60 where the accompanying code went


I think we'll have to chat about this - I'm mostly understanding what you'd like to do but need to think about how to best do it. - to make sure I'm getting it:

you want to look at every strut and rotate it so that its X axis points to some vector. I get that part.

Why do you want to construct a new list of nodes?

I'd much make this a private method that cannot be used as a node, and have it update the nodes instead.

In general if you're creating a node it should create new objects so that the graph makes sense, but is that necessary here. I'm worried about the expense of this with many nodes and struts. BUT if you do want to create new nodes, try something like this:

```
foreach(var node in nodes){

foreach (var strut in node.struts)
{
//do you rotations here creating a new set of lines

}

}

//now outside of your for loops create a new set of nodes, instead of calling this method foreach node.

newNodes = Node.ByPointsLinesAndGeoOrientationStrategy(nodes.centers,newlines,6,AverageVectors)


Here, might be easier to read if we said:

if(!strcmp(protocol.c_str(), "zk"))
   connectToZk( ... );
else
   connectToDrillBit( ... );

And move the detailed code into the private implementation functions.


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (390 of 390 tests passed, 0 test warnings)

![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)


Remove dead code?


Do we handle the case of an empty list?


if you can next time a PR is easier to make changes to.


New features binary with skipblanks functionality, countlines / nolinenumbers


release-5.x (lefticus) - i386-Windows-10-VisualStudio-14: OK (389 of 390 tests passed, 0 test warnings)

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


We don't call the zook.close( ) if we had an error. Is this a problem?


Also remove "+ get a free t-shirt"


Note issue of strtok( ) in a multi-threaded environment.


Please don't change the spaces to tabs, this will break all the ``[codeblock]``s as the rst converter removes XML tabs and the actual spaces are using for indentation in rst.

Also many of those changes are bogus as it converted double spaces that Americans tend to use after dots in sentences to tabs.

Also some things were reordered in a weird way, and I'm pretty sure running doctool would invalidate half of the changes. So all in all it would be better to just run the doctool with doing post processing with your IDE :)


Если используешь мастер страницу, то нет необходимости писать опять всю разметку. Представление получается это элемент, который полностью вставляется в RenderBody. Сейчас разметка генерируется неверная.


lol @Asif6474 come here more often XD


please export the routers


If I understand correctly, should this line not be:
dfdx:add(wd, x:abs())?

Won't negative parameters decay to negative infinity otherwise? This comment applies to the per-parameter case too.


Hi, let's leave this out of the general config template. It will just confuse our users, and it's just for the testing (i.e., us). I actually think we can prevent the setting as a whole, but we don't have to do that now. I guess it would be better to append this setting to the config.ini.php file when this is copied and filled in, so it stays out of the template. Not that much work for us, but it saves us from having to document it, and possibly answering user's questions about it :wink:


realm -> a Realm


What will commit 1000 be? :O


@fchollet
for samplewise, why is it axis=1 and not axis=(1,2) to get the moments of the channel?


True.  The original intent was that there might be an attribute that we would want to always run every time, whether another wanted to block or not.  However, the 4 attributes/methods that ended up utilizing it could easily just return after 1 blocks it without the need to run the rest of the handlers.  I will update this shortly, thanks.


Funny thing — I knew it :tired_face: :palm_tree:


@ajjl This defines the macro, but does not use the new macro anywhere.


That's what I did. I ran the doctool and then converted spaces to tabs, which I guess wasn't so smart. I need to repair this commit :smile:


specification
we need to see the original exception too


chop


Should m_zh be reset here after close?


Nit: we do:

    int rc = ZOK;
    rc = zoo_get...

Can we just do:

   int rc = zoo_get...


Good catch, have this change in my local repo as well


Thanks for your interest!

The implementation of the P* algorithm is part of my thesis which is about weather-aware routing.
The basic idea is that there are multiple data sources to gather weather data (or generally any other routing impacting data), so there will be a weight probability distribution for every edge.

This enables three queries like proposed in the P* paper:
1. `what are the paths from Berlin to Munich whose amount of rain is at most 5 l/m^2 with a probability of at least 90%`
2.  `what are the top-k paths from Berlin to Munich with the highest probabilities to have at most 5 l/m^2 of amount of rain?`
3. `what are the top-k shortest paths from my Berlin to Munich whose amount of rain is guaranteed by a probability of at least 90%?`

For that to work I still need to figure out how to add multiple weight values for each edge, kind of like a vector/map/array[][] with all available weather values (e.g. rain, sunshine, temperature) and their corresponding probability distribution.
Can I achieve this by expanding the `FlagEncoder` or do I need to get to the low level API like `EdgeAccess`?


Is it legal to create a button with POV angle as the button number?


Idk if we ever need these, but they are in the xmls


Version - master 3/4
Verified fixed


Thank you for that. I've installed the patch but still getting the same error.

http://new.tinygrab.com/6407caa59e9415834b98b4549c4a780125b6436f7d.png

I added a console.log('FIXED') just before line 15 to ensure there was no caching from the old plugin.


LaTeXDummy (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2165 of 2165 tests passed, 0 test warnings)

![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)


1. Yeah, I agree with that. If we were to go with `instanceof` it would be better to use `WorkingTreeIterator`
2. I think the `Enum` approach is definitely better than using `instanceof` so I'll go with that.


For both Capybara selenium and webkit, setting the window size (on the tests that do so) is necessary to prevent errors related to overlapping elements. Is there anything equivalent with phantomjs? Will `page.current_window.resize_to` still work? That's the new syntax which works for both selenium and webkit (unlike previous browser-specific syntax we used).


It is not :). That has been fixed, merely not pushed.

> On Mar 4, 2016, at 5:48 PM, Mikhail Kyraha <notifications@github.com> wrote:
>
> Is it legal to create a button with POV angle as the button number?
>
> ―
> Reply to this email directly or view it on GitHub.
>


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-AddressSanitizer: OK (3048 of 3048 tests passed, 0 test warnings)

![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, I see. Maybe try 1920x1080?


@FransBouma Just submitted a PR :-). Was thinking about doing it earlier but I messed up few things which resulted in massive diffs. Just applied the changes on a new branch based on you master branch and submitted it.


$ (document).ready(function() {
    $("#id").jCarouselLite({
        vertical: true,
        hoverPause: true,
        btnPrev: "#id",
        btnNext:"#id",
        visible: 3,
        auto: 3000,
        speed: 500
    });


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18389](https://teamcity.mesosphere.io/viewLog.html?buildId=154628&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 250 Build time: 00:06:23


Yeah, I'm not entirely sure what's going on with that. The indentation is definitely not like this when I view the code in IntelliJ.


Well, technically, most porn portals show "more than one obviously infringing video", i.e. copyrighted videos from paysites and/or parts of commercial DVDs .... so I've heard.

IMHO youtube-dl should be what wget is for http-URLs: A downloader, regardless of WHAT is downloaded. But that's not my call.


LaTeXDummy (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1019 of 1019 tests passed, 0 test warnings)

![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-30.13%25-green.svg)


Yeah, it was yuck, which is why I'm not doing it in this version, but I left it in as it didn't seem like it was causing any harm and made the javadoc on rawSocket make more sense as it was no longer referencing something that no longer existed.


I think there was only one instance of an overlapping element, which I've yet to screenshot or look into. A couple failures are due to API differences with alerts and attribute checks. And some I have no clue yet.


LaTeXDummy (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


Again, just a style issue. Might be clearer to do:

if ( drilbits.size( ) == 0 ) {
   // Handle no drillbit case
}

// Normal case

The return statements are a bit awkwrd to read as they are...


Oh, it's because I'm using space-based indentation, but their indentation is tabs. So it shows up fine in my IDE but not elsewhere


Not sure this is worth committing as the @dlebu change will replace it anyway. Otherwise looks fine.


Yeah, that makes sense. 1400 wide should be plenty. Maybe that extra 80 height is breaking the one test that overlaps.


Не самое лучшее название для метода. Имена стоит делать отражающими действительность


Should we throw an error for a negative limit? Or just leave it as a way of the user disabling all undo/redo functionality on an `undoable` reducer.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18390](https://teamcity.mesosphere.io/viewLog.html?buildId=154632&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 245 Build time: 00:04:18


This shouldn't be needed. See google/google-api-nodejs-client#459


FixCrashWithZoneAndSystemERV (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (2134 of 2135 tests passed, 0 test warnings)

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


We can get an error here (err == -1) if no drillbits. But, later, if err != 0, we return a zoo keeper error. The problem is, zk is fine, it is the lack of a drillbit that is the problem.


release-5.x (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (390 of 390 tests passed, 0 test warnings)

![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)


Sure.


Here's what I meant by it not being straight forward. Is that comment supposed to be true now? Should this strip the `-rc` bit?


[ ] a


The following doc(s) have been deployed to the [ttrowellht](http://ttrowellht-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/htaccess](http://ttrowellht-static-docs.pantheon.io/docs/articles/sites/code/htaccess)
- [docs/articles/sites/code/redirect-incoming-requests](http://ttrowellht-static-docs.pantheon.io/docs/articles/sites/code/redirect-incoming-requests)


Done.


Dones.


LaTeXDummy (Myoldmopar) - i386-Windows-7-VisualStudio-12: Build Failed

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


Man, I tried this a while ago (https://github.com/shua/r2_dol) and I couldn't for the life of me get r2 to recognize separate vaddr and paddr when bits was set to 32 (16 and 64 worked fine). I have no idea if that was a bug with r2 or something I was doing wrong, but I'm glad to see someone with more expertise finally implement this. I'll have to drop in on #radare again. I'm interested in adding support to common nintendo binary data files, or maybe some of the extra gekko/flipper assembly commands if radare doesn't already support them.


Not intentionally but I guess when you fork it disables it by default. I've re-enabled it.


actually the comment doesn't make sense to me. It'd would be better to always use the script location rather than rely on CWD.

And... in which case does it fail?


It may be good because it's just the first boot only to download the docker binary.
Forget it. :)


<?php
	filesize();
	pathinfo();

 ?>


Note that this fixes the error with coarse hi.tiff. The issue didn't have to do with cell quality, it had to do with the size of the domain and how I was assigning top height. We weren't getting enough vertical layers. Now we scale the top height based on either on dx|dy or dz (whichever is greater).


这项目再过两个月应该可以彻底活过来了，这几年我都没更新


@alimaredia It looks reasonable.  Unfortunately, I can no longer execute test/ceph_objectstore_tool.py from src directory because CEPH_BIN and CEPH_ROOT must be set.


this use of tqdm is kind of contrived, I must say, since the loop is printing out on every (rapid) iteration which makes the progress bar kind of useless.  Let's either ditch the example or say something abstract like `process(r)` within the loop


If delete datasets fails, the workflow will show as SUCCEEDED. Shouldn't it be failed, then?


I was trying to be clever with `self.assertEqual(..., debug)`, but that can only test for one of the two possible conditions. I'm rewriting it to test for both.


fu_blastfurnace.object  error "moonstoneore" : "moonstonebar",


10.1.3  在hibernate.cfg.xml文件中配置实体类映射
实体类还需要配置到hibernate.cfg.xml中，以便Hibernate初始化实体类与数据库表的映射关系。如果只配置了映射关系，而没有配置到hibernate.cfg.xml中，Hibernate是无法解析实体类的，因为Hibernate无法自行判断哪些是实体类。
如果实体类是使用@注解配置的，需要用<mapping class=””/>配置，而如果是用XML文件配置的，需要用<mapping resource=””/>配置XML配置文件。hibernate.cfg.xml文件中配置实体类映射的示例代码如下：
<!-- 使用XML映射文件映射实体类的配置 -->
<mapping resource="com/cn/vo/UsersVo.hbm.xml"/>
<mapping resource="com/cn/vo/DeptVo.hbm.xml"/>
<!-- 使用@注解映射实体类的配置 -->
<mapping class="com.cn.vo.UsersVo "/>
<mapping class="com.cn.vo.DeptVo "/>


not have ,


If your reason for suppressing the exception is to at least delete the other dataset instances, maybe try logic like this:

https://github.com/caskdata/cdap/blob/develop/cdap-app-fabric/src/main/java/co/cask/cdap/internal/app/runtime/batch/dataset/MultipleOutputs.java#L344-L367


@Arodang, did you intentionally leave this commented out? if so, why?


话说我前两天还复习了一遍呢，哈哈～谢大的文字风格还是那么的“清新脱俗”

发自我的 iPhone

> 在 2016年3月4日，11:02，Yihui Xie <notifications@github.com> 写道：
>
> 这项目再过两个月应该可以彻底活过来了，这几年我都没更新
>
> —
> Reply to this email directly or view it on GitHub.
>


with this commit, is this build fully functional to compile?


I did - this is the command we're going to need when we re-enable our E2E testing, so I commented it out for later use.


Interesting, there is no "raw"-variant of the [`selinux_check_access`](https://github.com/SELinuxProject/selinux/blob/58dcbcd0db4b498e70cf97d381394e3bcf289211/libselinux/src/checkAccess.c#L35). So, we always [translate](https://github.com/systemd/systemd/blob/6344f3e28dd6ae341015d62163437f6cb1add749/src/core/selinux-access.c#L254) contexts.
Are there any `libselinux`-developers?:)
cc @fishilico


is there a way to get around all of the "../" chunks in paths?  can't you set a root directory somewhere and have all paths be evaluated relative to the root directory?


`asyncForm.js` -> `async-form.js`


I will look into doing this over the weekend, that's a good idea.


@mattstauffer any change you have a link to some examples for testing the token exhange


Moved this chunk to htaccess.rb


Moved this chunk to mimetypes.rb


can you do

    for(var i in resources) {
?


Is Ctrl+C not sufficient? And as far as I know `top` doesn't have Ctrl+D quit either.


Moved this chunk to httpdconfig.rb


Updating the spring dependency fixes this issue.  See my fork at [https://github.com/rpridemore/activator-akka-java-lambda-spring](https://github.com/rpridemore/activator-akka-java-lambda-spring)


@asofold Quick hacked together block list for 1.9, may help you in the official version. :)

BlocksMC1_9.java - to go here https://github.com/NoCheatPlus/NoCheatPlus/tree/master/NCPCore/src/main/java/fr/neatmonster/nocheatplus/compat/blocks/init/vanilla
```
package fr.neatmonster.nocheatplus.compat.blocks.init.vanilla;

import org.bukkit.Material;

import fr.neatmonster.nocheatplus.compat.blocks.BlockPropertiesSetup;
import fr.neatmonster.nocheatplus.compat.blocks.init.BlockInit;
import fr.neatmonster.nocheatplus.config.WorldConfigProvider;
import fr.neatmonster.nocheatplus.logging.StaticLog;
import fr.neatmonster.nocheatplus.utilities.BlockFlags;
import fr.neatmonster.nocheatplus.utilities.BlockProperties;
import fr.neatmonster.nocheatplus.utilities.BlockProperties.BlockProps;

@SuppressWarnings("deprecation")
public class BlocksMC1_9 implements BlockPropertiesSetup {

    @Override
    public void setupBlockProperties(WorldConfigProvider<?> worldConfigProvider) {

        // ---- New blocks ----

        // 198(END_ROD
        BlockInit.setAs(198, Material.END_ROD);

        // 199(CHORUS_PLANT
        BlockInit.setAs(199, Material.CHORUS_PLANT);

        // 200(CHORUS_FLOWER
        BlockInit.setAs(200, Material.CHORUS_FLOWER);

        // 201(PURPUR_BLOCK
        BlockInit.setAs(201, Material.PURPUR_BLOCK); // TODO: red sandstone / prismarine ?

        // 202(PURPUR_PILLAR
        BlockInit.setAs(202, Material.PURPUR_PILLAR); // TODO: red sandstone / prismarine ?

        // 203(PURPUR_STAIRS
        BlockInit.setAs(203, Material.PURPUR_STAIRS);

        // 204(PURPUR_DOUBLE_SLAB
        BlockInit.setAs(204, Material.PURPUR_DOUBLE_SLAB);

        // 205(PURPUR_SLAB
        BlockInit.setAs(205, Material.PURPUR_SLAB);

        // 206(END_BRICKS
        BlockInit.setAs(206, Material.END_BRICKS);

        // 207(BEETROOT_BLOCK
        BlockInit.setAs(207, Material.BEETROOT_BLOCK);

        // 208 (GRASS_PATH
        BlockInit.setAs(208, Material.GRASS_PATH);

        // 209(END_GATEWAY
        BlockInit.setAs(209, Material.END_GATEWAY);

        // 210(COMMAND_REPEATING
        BlockInit.setAs(210, Material.COMMAND_REPEATING);

        // 211(COMMAND_CHAIN
        BlockInit.setAs(211, Material.COMMAND_CHAIN);

        // 212(FROSTED_ICE
        BlockInit.setAs(212, Material.FROSTED_ICE);

        // 255(STRUCTURE_BLOCK
        BlockInit.setAs(255, Material.STRUCTURE_BLOCK);

        StaticLog.logInfo("Added block-info for Minecraft 1.9 blocks.");
    }

}

```


expect(resources).to.be.a('object');


delete extra nl


forgot to change this string! "The StorageService".


多了个 `>`


inconsistent use of 's


> Hm.. and why you both started doing it this way

whaa? lol "started doing it this way" is kind of an exaggeration don't you think? I have like 500 projects, and I think 2 or maybe 3 of them do this lol.

_it makes sense when you want to allow the user to customize the **class**, not just create an instance_.  Like how we allow the cache property in cache-base to be customized. It's just a really easy and much cleaner way than doing this kind of thing:

```js
function Cache(prop, options) {
  if (prop) {
    this.prop = prop;
    this[this.prop] = {};
  }
}

Cache.prototype.set = function(key, val) {
  if (this.prop) {
    this[this.prop][key] = val;
  } else {
    this[key] = val;
  }
};
```

anyway, that said, I agree that it probably isn't necessary in verbalize.

> which cause all instances to be modified.

not if the implementation is doing `new Verbalize()`. If that doesn't work because of how underlying libs work, then we might want to change how the libs are instantiated in verbalize. anyway I guess it seems like something that should be considered in implementations and how they use verbalize, but shouldn't really matter to verbalize. we should just do something like what @tunnckoCore was saying:

```js
module.exports = new Verbalize();
module.exports.Verbalize = Verbalize;
```

@tunnckoCore we're making changes to this a lot right now. it's always good to get feedback on a project, especially early on when it's easier to change, but maybe try to make the tone a little less aggressive so it's easier to have a conversation about it


https://github.com/ant-design/ant-design/issues/1122

`hideAction` 应该只有在 combobox 时才是 `blur`。


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18267](https://teamcity.mesosphere.io/viewLog.html?buildId=153854&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 245 Build time: 00:04:52


Precisa melhorar a redação.


Is this how you wanted it?


Shouldn't you check that gathering resources actually increased the resource count?


pySend-my-ip


I saw it [here](https://github.com/blog/2111-issue-and-pull-request-templates), and it seemed like good practice for an open source project, but you're right, it's probably a bit too much. If you think it's unnecessary, you can reduce the amount or just remove it entirely.


Did you leave this commented out on purpose? If so, why?


Да, точно, я к этому ещё не привык :)


Vile scum.


Yeah, I like that at the moment.  It makes the flag/option variables very obvious... and since in the current version they are pointers, I like it. :-p


great！ let me star


asasf


Thanks @cdzombak !


Refer to commit 5c526a5<https://github.com/JaydeepIMG/musl-1/commit/5c526a57a0b3b3725cbb18e3bf5a8fbf933b5585>

From: rofl0r [mailto:notifications@github.com]
Sent: 03 March 2016 PM 07:02
To: JaydeepIMG/musl-1
Cc: Jaydeep Patil
Subject: Re: [musl-1] MIPS64 N64 port. Cosmetics changes. (759436c)

tab

—
Reply to this email directly or view it on GitHub<https://github.com/JaydeepIMG/musl-1/commit/759436c54ae5d622f80f7e171ee1a1ca991f5eb6#commitcomment-16473058>.


Refer to commit 5c526a5<https://github.com/JaydeepIMG/musl-1/commit/5c526a57a0b3b3725cbb18e3bf5a8fbf933b5585>

From: rofl0r [mailto:notifications@github.com]
Sent: 03 March 2016 PM 07:02
To: JaydeepIMG/musl-1
Cc: Jaydeep Patil
Subject: Re: [musl-1] MIPS64 N64 port. Cosmetics changes. (759436c)

space

—
Reply to this email directly or view it on GitHub<https://github.com/JaydeepIMG/musl-1/commit/759436c54ae5d622f80f7e171ee1a1ca991f5eb6#commitcomment-16473061>.


When subscribing to test.mosquitto.org there were instances when the fromCString would fail leaving topic as nil. In that case the attempt to unwrap would crash the app.
So yes, in that case it was a real issue.

Not sure I like dropping the message, but doing more got pretty invasive.

The other change to payloadString was backed out because it was unnecessary, and as you state if it needed shouldn't have been mixed with this change.


Refer to commit 5c526a5<https://github.com/JaydeepIMG/musl-1/commit/5c526a57a0b3b3725cbb18e3bf5a8fbf933b5585>

From: rofl0r [mailto:notifications@github.com]
Sent: 03 March 2016 PM 07:04
To: JaydeepIMG/musl-1
Cc: Jaydeep Patil
Subject: Re: [musl-1] MIPS64 N64 port. Cosmetics changes. (759436c)

tab+space

—
Reply to this email directly or view it on GitHub<https://github.com/JaydeepIMG/musl-1/commit/759436c54ae5d622f80f7e171ee1a1ca991f5eb6#commitcomment-16473110>.


This error appears again, maybe there is conflict with cm11.0 when using ccache. I will tryy to setup a difference ccache folder for cm12.1.


alright, it's in the DayZ-1.8.7-Rebase branch.


great. thanks!


The error I get now is
> Notice: Undefined offset: 0 in C:\Users\admin\OneDrive\Documents\Programing Stuff\Web development\serverRoot\elemukulek\lib\Transphporm\TSSFunction\Data.php on line 43

tss with issue
```css
main { content: template('calendar.xml'); bind: data(model.gettingdata()) }
```

xml
```xml
<div id="calendar-title" class="row">
    <h2 class="title five columns">November 2015</h2><!-- style="padding-right: 30px; float: left; margin: 0.5ex 0;" -->
    <form class="controls calendar seven columns" method="get">
        <select name="month" id="month">
          <option value="1">January</option>
          <option value="2">February</option>
          <option value="3">March</option>
          <option value="4">April</option>
          <option value="5">May</option>
          <option value="6">June</option>
          <option value="7">July</option>
          <option value="8">August</option>
          <option value="9">September</option>
          <option value="10">October</option>
          <option value="11">November</option>
          <option value="12">December</option>
        </select>
        <select name="year" id="year">
          <option value="2015">2015</option>
        </select>
        <input name="submit" type="submit" value="Go" />
        <br />
      <a href="events/" class="previous"><< Previous Month</a> | <a href="events/" class="next">Next Month >></a>
    </form>
</div>
<div style="overflow-x: auto;" class="row">
<table class="calendar">
  <thead >
    <tr class="calendar-row">
      <th class="calendar-day-head">Sunday</th>
      <th class="calendar-day-head">Monday</th>
      <th class="calendar-day-head">Tuesday</th>
      <th class="calendar-day-head">Wednesday</th>
      <th class="calendar-day-head">Thursday</th>
      <th class="calendar-day-head">Friday</th>
      <th class="calendar-day-head">Saturday</th>
    </tr>
  </thead>
  <tbody>
    <tr class="calendar-row">
      <td class="calendar-day">
        <div class="day-number">1</div>
        <div class="event">
          <a href="events/5">Event 5</a><!--   -->
        </div>
      </td>
    </tr>
  </tbody>
</table>
</div>
```


Fixes #12 . Well, hopefully it clarifies it at least!

Scott


@ricardoquesada  :smile:


Scummvm? :)


Revert?


I've just updated from tmux 1.9 to 2.0 and would like to know to what function is it redundant ?

In an environment where you just don't want anybody to access any session, even newly created, you lock the entire tmux server, if you want to lock only just one session, then lock-session ...


Some identifiers were automatically changed by the conversion tool, and because they were never actually used from the C++ side the C++ headers weren't always updated.  So they probably should be upstreamed.


Damn lol, everywhere I comment in a past year or two I'm "aggressive", I can't understand the problem srsly - maybe my english, maybe some culture diffs, maybe really the world out of Bulgaria is **too** soft, haha.
But okey I'm trying :) :+1:

I'm not aggressive, just asking, i'm curious dev. I'm always trying to help, always share issues and ideas.

> is kind of an exaggeration don't you think

nope, lol. I just meant that I just noticed it in a past few weeks. And I just don't see sense and value on it.

What's the problem to do something like this in a tests

```js
var Logger = require('verbalize').Verbalize
beforeEach(function() {
  logger = Logger()
})
```

To clarify, it's the same, but without the need of wrapping. And I'm talking in general, it's not for the case of verbalize, i can't remember but it is on few more places.


Nice catch, Anders. At first I thought that this wouldn't be needed, because `python-requests` is a dependency of `python-pip`... So anyone who uses `pip` would have it installed.

What happens is that nearly every Python dependency of bootstrap-vz is available (the exception is `pyrfc3339`) from `jessie` and later releases. So, if one install them using `apt-get`, they maybe end up in a situation where `requests` isn't available.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18268](https://teamcity.mesosphere.io/viewLog.html?buildId=153865&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 245 Build time: 00:05:03


All the formatting stuff is great, but why are we doing this?!  I showed you how to do this properly the other day!


ack


2 biến, đơn giản quá :)


Oh, my. Looks good but I'll boot a few times with this patch to if everything works as expected.


这个log不需要了吧


这个log也不需要了吧


What's the source for this import path?

According to https://github.com/fsnotify/fsnotify/issues/108#issuecomment-185815848, it was an unexpected finding that it works (probably because the github org was renamed, and that setups redirects):

> I tested `go get gopkg.in/fsnotify.v1` and it still works, which I did not expect.

https://github.com/fsnotify/fsnotify#api-stability does not mention pkg.in and suggests using /vendor/ for commands.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18269](https://teamcity.mesosphere.io/viewLog.html?buildId=153870&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:04


I saw it on gopkg.in's page for the project. Whoops! Good catch.

> On Mar 3, 2016, at 19:49, Dmitri Shuralyov <notifications@github.com> wrote:
>
> What's the source for this import path?
>
> According to fsnotify/fsnotify#108 (comment), it was an unexpected finding that it works (probably because the github org was renamed, and that setups redirects):
>
> I tested go get gopkg.in/fsnotify.v1 and it still works, which I did not expect.
>
> https://github.com/fsnotify/fsnotify#api-stability does not mention pkg.in and suggests using /vendor/ for commands.
>
> —
> Reply to this email directly or view it on GitHub.
>


为什么需要`go l.retry`呢？我上个comment的意思是，Logger.retry可以不需要了，把Retry for 循环写在Log.flush里。


这个有啥用？


Would use the variable this, but I have no idea how it works.


謎のブランチ内コンフリクトが起きてしまったためrevert使ってしまいました。


zazawang


What can I use in new version of tmux instead of choose-list for quick menu like these:

    choose-list -l "return,client,server,session,window,pane" 'source ~/.tmux/menu/main-"%%"'


参考
変数の宣言(var文) - 変数 - JavaScript入門 ： http://goo.gl/dtWh


find_byメソッドは引数にnilが入ると最初の要素を取ってきてしまうかと思いますが大丈夫ですか？


looks like will work fine if we use `class: 'radviz-tooltip'` instead of id;  problem is when using `htmlwidgets` often we'll use more than one in a single `DOM`.  This will result in a conflict, since `id` must be unique.  We generally have to go with a hash `id` or work with `class`.  Fortunately, the way you designed we can just change here and in the css.  I'll be happy to test and submit pull to take some work off of you.


Very nice! I look forward to testing this out!


The formatting? I think that's unnecessary.


jaguarbs05 aqui hay una lista de canales actualizada pero algunos estan e 1080p necesita como 20 megas

https://mega.nz/#!I9wwjAAY!Podqly-08oN0D1UAvIsXtmndKOjhvx0Ieq0q8oTJVx4


同じようなことをコントローラーでやっているので、シンプルに、またviewにロジックを書かずに記述したいですね。


log可以在下面检查不通过时方便查看几个运行参数


Hi Gavofyork
How to make or build wallet Ether

Thanks and Bestregards
Rusdi


同上


ここは他と違ってLIKEという文字がないような気がしますが大丈夫ですか？


Bug 2796: Change default value: snmp_send to false.


Not for this PR, but should we use a `HashWithIndifferentAccess` for `_view_hooks`?


I noticed [`00`](https://github.com/toji/webvr-samples/blob/gh-pages/00-hello-webvr.html) doesn't have these meta tags. is that intentional?


@justineqiu
Nice


отлично!)


Looks good.  What do you think about tying the `before(:all)` and `after(:all)` to an RSpec `type`?  I like the convenience of it, but could see an argument against considering it's less explicit.

For example, `RSpec.describe Foobar, type: :split_accounts_feature, do`


Attempting to use the macro `CHPrimitiveProperty(...)` resulted in a build error because a semicolon was missing in its implementation. This fixes it.


Oops. This was for #166.


Hm, seems like this could still execute if `start == end == 0` on an unsigned int?


我觉得retry逻辑单独提到一个函数里面更清晰，可读性更好，以后可能会根据运维的情况调整重试的策略，放到单独的retry函数中好修改。


個々で一回終わり。


そんで、ここのカッコを最後に持ってく。


外部ドメインタイトル習得ファンクションの定義。


Looks good. The only thing I would point out is that you seem to have branched off of #730, which is a little odd, and could lead to merge conflicts.
If you want to unwind this, you should be able to do so relatively easily at this point. There's probably a way to do it in command-line git, and that would be cool, but you could also just copy this one file over into /tmp, kill the branch #734 and make a new one, and then copy the file back from /tmp into .../tests/scheduler

Magit probably makes this pretty easy, but if you're not using magit yet, do it the easy way.


I honestly do not recall the discussion, which is my fault. We looked at the registry and it was not obvious what we should change there. We tried using files from an older commit, it didn't work. We also tried to run everything without the AC Bridge with Multiplexer but it didn't work either.


removed


ｈｔｍｌ３として、あとに出る関数様の変数を定義しておく。


这里有个bug：l.retry的参数不能是*[]，而是得是[]。因为entries是定义在flush的stack里的，所以一旦flush返回了，对应的stack frame就不存在了，此时&entries就是dangling pointer了。

这个bug从一个侧面说明retry逻辑和flush逻辑的密切联系——两者都调用PutRecords操作同一个slice，没有什么可以分开的。

另一个侧面是，这两个函数很多代码都是重复的，逻辑上也拆不开；重复的代码的可读性也不好。所以我认为应该合在一起。


タイトルの実際の表示HTML.


unit test不是用来输出人可看的内容的。除了写这行log的人，其他人也不会想到要去看输出结果。如果这些结果真的很重要，应该写成 assert.Equal之类的形式，让机器去检查。


One thing - I think I set a bad precedent with this:
```
from tests.factories import gbe_factories, scheduler_factories
```

I'm starting to strongly prefer explicit imports:

```
from tests.factories.gbe_factories import ProfileFactory
```

or for multiple imports from a single source:

```
from tests.factories.gbe_factories import (
  ProfileFactory,
  VolunteerFactory,
)
```


Instead of removing the old annotation, replace it with `@LegacyConfig(...)`. This will allow servers to continue to work with the old config option (but will warn in the logs when that happens).


But the website is not reflecting this update... Am I missing something?


That was my thought for the next step if we're going to share across files.
Just didn't want to go too far down that road if you didn't like this idea.

On Thursday, March 3, 2016, Ben Linton <notifications@github.com> wrote:

> Looks good. What do you think about tying the before(:all) and after(:all)
> to an RSpec type? I like the convenience of it, but could see an argument
> against considering it's less explicit.
>
> For example, RSpec.describe Foobar, type: :split_accounts_feature, do
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tablexi/nucore-open/commit/38dd1fa57e6bca13c9a660aa5076a62ca58c40d2#commitcomment-16491737>
> .
>


My bad. Not sure about the Maven project structure. :p


Vous avez modifié le fichier pom.xml, ce qui fait que le build (mvn clean test) ne fonctionne plus. Vous devez rajouter les 4 lignes supprimées (balise <modules> pour que je puisse valider).


Try refreshing.  The [page](https://marklodato.github.io/visual-git-guide/index-en.html) shows `git diff b325c da985` now, with the arrow still pointing the same way.


This commit [breaks installation of `gcloud-python`](https://github.com/GoogleCloudPlatform/gcloud-python/issues/1570).


I wasn't that concerned because it's only accessed internally, but that
might make the internal code cleaner. I'll play around with it. Good
suggestion.

On Thursday, March 3, 2016, Ben Linton <notifications@github.com> wrote:

> Not for this PR, but should we use a HashWithIndifferentAccess for
> _view_hooks?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tablexi/nucore-open/commit/38dd1fa57e6bca13c9a660aa5076a62ca58c40d2#commitcomment-16491711>
> .
>


[Ayyyyyyyy](https://github.com/MLG-Fortress/VanishNoPacket/releases/tag/3.20)

And I personally use [Cnaude's fork](https://github.com/RoboMWM/VanishNoPacket) anyhow


// CHANGED HERE - UN COMMENTED getinteractionweights.


I'm fine with this either formatting, = to revert


I would like to ask other's opinion for the rename: for me previous one (```LKL_HIJACK_NET_TAP```) is not useful.

I don't understand why ```IFNAME``` doesn't make sense to you: indeed dpdk doesn't use interface name but for tap it's useful.


// added variable current


//see change in variable content


//see the .hide() statement


let's remove this


ah, my mistake: it was already commented and I only applied to dpdk code. will fix it.


Oh I was looking at the line below by accident. Can fix this


This file is merged from 5 different language files. All credits are granted here to original authors respectively:
* MatrexsVigil/harvestcraft#44
* MatrexsVigil/harvestcraft#51
* MatrexsVigil/harvestcraft#54
* https://github.com/sb023612/Localization-Database/blob/master/HarvestCraft/HarvestCraft%201.7.10e/zh_CN.lang
* http://www.mcbbs.net/thread-523781-1-1.html


> But okey I'm trying :)

I know you are :) no worries.

> maybe my english,

your english is good, but you're right, there might be some subtle differences that aren't intentional. that's okay. I always keep this in mind and hope that devs who have another native language will do the same for me.

> and sorry if this post again is aggressive, i'm not, i just write this way. I'm fun and happy guy in reality.

don't over-worry. I'll try to explain but part of it is the overall "tone".

In the spirit of trying to help you overcome this, since I do think you are definitely not doing it intentionally, here are a couple of things that made me think your tone seemed aggressive (these are my personal interpretations, I don't know what anyone else might think. please don't worry, I didn't think it was really aggressive or a big deal, it's more that I've seen you make a lot of progress over the last year, and I'd like to see you achieve what you want):

- you said: "it's normal to not be the best always." To me, this translates as something like: "It's normal to see bad code". or, maybe a nicer interpretation: "I don't expect every line of code to be perfect". Either way, this sounds like you're saying "I'm right. You're wrong". Usually that will make someone feel defensive, and if you're trying to learn about why they made certain code choices, it's less likely to result in a good discussion.
- later you said: "I know and I'm agree." but you gave the following example in your first comment, and asked why we didn't do it this way:

```js
module.exports = new CacheBase()
module.exports.CacheBase = CacheBase
```

The answer to that is, _because it won't work_. Point being that (if you were trying to learn), then if you understood what @doowb meant about why we used that pattern in `base` and `cache-base`, then you must have also realized that you were wrong in your initial comment. This led me to believe you sounded defensive when you said "I know and I agree". Again, because you didn't seem to "know" in your first comment. This contributed to the overall sense that you were being aggressive. but maybe that's too strong a word.

fwiw, it's much easier to have normal, candid conversations when you can hear someone's voice, or see their face. and it's hard to control how you're interpreted in a text conversation, but it is what it is. we just need to be extra sensitive of how others might interpret what we're saying. (believe me, I've said things online I wish I could take back, so I'm not criticizing).


woot!


@kashyappanda very much thanku


Thanks for this!

Damn, `make` from MinGW didn't work? It did for [this other person](https://github.com/mathquill/mathquill/issues/446#issuecomment-123898788) a while ago, I wonder what happened.


I would suggest that rather to remember `<from> -> <to>`, just use *chronologic* order, `<past> -> <present>`. The timeline points right by itself :stuck_out_tongue_winking_eye:


I agree but I'd like to also ask to @tommie-lie since this is from his PR. any comments @tommie-lie ?


Neat! Didn't know about this.


Cool. Looks like we were using `Actions.sh` when we could have been doing `sh` all over the place. Anything else that we should tweak @KrauseFx?


haha whoops nice catch thanks!


for future use, you can add a height to an element instead of adding a bunch of `br` tags


You're right.  The arrows are wrong too, meaning the old text was consistent with the arrows, it's just that both were backwards from what you'd expect to do (as you said in your initial suggestion).  Thanks again for helping me understand.

Fixed in dd3b538 and pushed.


other thoughts:

- if you want to learn (as I do) about why programmers make the decisions they do, so that you can become a better programmer, then that is amazing! keep doing that the rest of your career, in everything you do. (my mom used to say: "he who knows **what**, will always have a job, but he who knows **why**, will always be his boss")
- Don't tell people they're wrong, and don't say "I'm right". Even if you are. You will **always, always, always** go further by allowing the truth to become self-evident. You want people saying, "Now I see why he was asking that question". Not "but he didn't have to be a dick about it" lol.


related to #289


scripts/checkpatch.pl didn't tell this as an error but agree: will add a space before ```malloc(...```


*Helping me understand*, hahaha :stuck_out_tongue_closed_eyes:
I learnt more from you than you from me :+1:


you can remove "= 2" in this line, it is not necessary.


@hunter-packages/sdl2 I tried making it work with CONFIG mode, mind checking if it seems ok? Also how would I test such a thing? (I'm new to CONFIG mode packaging...)


OMG

![screenshot 2016-03-03 23 36 53](https://cloud.githubusercontent.com/assets/2119355/13518141/d5e64fba-e198-11e5-8651-094b43f21877.png)


okay: will fix it and the other places as well.


Just use `[bend left=25,auto=left]` instead :grin:


So I should also update the length in clif_send call, right?


As far as I can see this variable is not used


trying to make the pull showup


Yup, in the picture above it's `git diff b325c da985` :grimacing:


No available in CF


Yay! :grin:


You're the best.  I figured out the `bend left=25` part, but it would have taken me a while to realize I needed `auto=left`.  It's been six years since I wrote this...

Fixed in 9a06fb7.


Will not compile in CF


No need to use `LIB_SUFFIX` here - it will work without. Also there is a chance that it will not work on some platforms


AAAAAAAAAAAAAAAAAAAAA


You suggested that to me [here](https://github.com/MarkLodato/visual-git-guide/blob/13c5fa169295a683c53dbb415d5569fa7c4893cc/diff.tex#L18) :wink:


Ohh right forgot to remove it, sorry.


for testing


This is a nice improvement :+1:


I'm fine to fix it but again I would also like to ask @tommie-lie for this comment.


okay, will remove it.


HF solver for project 1B


done


Increase space of not selected day for Tap.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18271](https://teamcity.mesosphere.io/viewLog.html?buildId=153886&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 245 Build time: 00:08:50


Many of the underlying methods do something like this:

```js
Logger.prototype.addFoo = function(name) {
  Object.defineProperty(Logger.prototype, name, {
    get: function() {
      var fn = function() {};
      fn.__proto__ = Logger.prototype;
      return fn;
    }
  });
};
```

so when doing the following code:

```js
var logger = new Logger();
logger.addFoo('bar');
console.log(typeof logger.bar);
//=> 'function'
```
`bar` is added to the `Logger.prototype` so doing:

```js
var logger2 = new Logger();
console.log(typeof logger2.bar);
//=> 'function'
```

I'm using `.create` to return a new **class** that will basically give a clean prototype so it can be used in tests.

I know in some places we do something more like this:

```js
Logger.prototype.addFoo = function(name) {
  var self = this;
  Object.defineProperty(this, name, {
    get: function() {
      var fn = function() {};
      fn.__proto__ = self;
      return fn;
    }
  });
};
```

I'll probably try that out to see if it makes a difference and still works since the instance is the thing that's being used instead of the `.prototype.`. If it does, then I'll remove the `.create` to simplify the code and make it more readable/maintainable.


Can we change it back? I don't really get why it was modified in the first place? @alschm


> mind checking if it seems ok?

Looks good in general, see some notes.

> Also how would I test such a thing?

Install this package to some temporary location and create trivial example with `find_package(SDL2 CONFIG REQUIRED)` + link SDL2 library. Temporary location can be passed using `CMAKE_PREFIX_PATH` for example, see https://github.com/forexample/package-example#boo-use-installed-foo

So it will be:
```bash
[sdl2]> cmake -H. -B_builds -DCMAKE_BUILD_TYPE=Debug -DCMAKE_INSTALL_PREFIX=/.../temp/install
[sdl2]> cmake --build _builds --target install
```

Using:
```bash
[sdl2-example]> cmake -H. -B_builds -DCMAKE_BUILD_TYPE=Debug -DCMAKE_PREFIX_PATH=/.../temp/install
[sdl2-example]> cmake --build _builds
```


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18272](https://teamcity.mesosphere.io/viewLog.html?buildId=153887&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 245 Build time: 00:09:47


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18270](https://teamcity.mesosphere.io/viewLog.html?buildId=153885&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 245 Build time: 00:09:53


don't change it here


Alright will fix, thought it was relevant given SDL uses it.


看看这个(来自wiki)
```
位元0-3（1、2、4和8）决定了药水效果。目前最高的有效数为14十进制 = 1110二进制。
位元0-5决定了药水的名字。
如果位元5（32）为真，那么药水是II级药水（或在某些情况下为还原版药水）。
如果位元6（64）为真，那么药水是延长版药水（或在某些情况下为还原版药水）。
位元5和6皆为真的药水无法被酿造。
位元13和14（8192和16384）决定药水是饮用药水或是喷溅药水。如果位元13为真，则是饮用药水。如果位元14为真，则是喷溅药水。
位元7-12和15会被无视，因此很多拥有不同伤害值的药水都有同样的属性。
```


nah that sounds strange, since I don't really touched the EC order, but just changed to CC use this same order too. On RunUO the buy/sell menu uses the same code for both classic/enhanced clients, and it works perfectly

but on sphere, even if I try to make it perfectly match the runuo code it will not working, so maybe something still wrong. Anyway can you upload this 'old' enhanced client? Maybe it will help with something :)


Looks like this commit message somehow conveyed to Google :smile:


fixed


Should use htmlspecialchars($usersname) here.


Ok thanks,  we'll do that.


This movePiece works for the way we currently have it implemented (with Event Listeners and the local user) and also with our unit tests. Having it work to update for other users through the server would require some changes and possibly another or a different function (hopefully the same). We will have to figure it out with you and Kai to see what all is needed to make sure it is communicated properly with all the information to the server. I think the selectedPiece should have all of the information stored that we need... or we could just be on different pages and need to talk it over.


It actually should be done in a background service, and need to survive upon phone restart, etc.


Same here.


totally right, this must be a remnant of a previous phase.


added.


offset can't be negative, but it can be zero.


好的，我再看看 谢谢


Remove commented out code.


Wasn't my intention, no.


aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa


ありがとうございます！


the idea is that if anyone in the future wants to implement a srp class not based on deferreds, this should be anything she needs.


Yes, we could do that. However, I don't the maintainers for most of the modules.


Thanks, I missed that.


Since we are using ES6 classes for the core library I think we can use them here as well.

I guess the only difference is that we have a constructor instead of `getInitialState()` function.


@jyotisingh — did you really intend to remove the resources? When agents auto-register, they'll not get the resources.


Thanks! Didn't know I could access the children in the constructor :)


@doowb makes sense. since `__proto__` is really only necessary where we want to give the user the ability to use a function that is also an object AND has the prototype methods of a class, it seems like that should only be done at the implementation level. e.g. in `base-logger`. but in lower-level methods we would only need to use more expressive methods. not sure if that helps


should we do something in place of the `PosyArray` assert, like check length?


`p_subbed.value` does not work here? Maybe it ought to?

ps -- great inline comments above though


Ok, but how can we do something like that with classes?

```
var createSimpleWrapperClass = function(domName) {
  var myClass = {
    render: function() {
      return React.createElement(domName, this.props, this.props.children);
    }
  };
  return React.createClass(myClass);
};

var OnsButton = createSimpleWrapperClass('ons-button');
var OnsToolbar = createSimpleWrapperClass('ons-toolbar');
```


feels like maybe the attaching of units to the solution should happen in a method designed for that purpose, as opposed to here. Or this method should be renamed


hmm, this now feels out of place. It's too specific / special case relative to the docstring and method name.


Shouldn't this docstring say `dicts` not `lists`?


Maybe we can name it `DryConfig`?


By Jie Hao & Xin Zou, Shanghai Jiaotong University, China


You probably could, although if you were super-paranoid, would that not work if R13 was close to the very high end of the address space, and SI, being near that plus something, overflowed? Sure, here the something in "plus something" is very small so overflow is extremely unlikely if not impossible the way memory is mapped in practice, but in general, I think it's prudent (and cheap) to normalize wrt src_base.


Haha, yea. Very thanks! :)

> To me, this translates as something like

Oo no, no. Haha. I meant totally different, but yea, as I said... my not trained english. I meant that my above example is not the best solution. As we said, some code make sense in one situations, another in another situations. That's why I edited my first post with "_edit: actually, okey, my above example isn't correct_", exactly because the example with CacheBase, because there such thing make sense.

> then you must have also realized that you were wrong in your initial comment.

Actually, yea, that's why I edited it when realized that the CacheBase is not for here and it not mess with what I'm talking.

> my mom used to say

Haha, she's absolutely right. My dad also say that, but a bit different.

Actually, @doowb, good point. Maybe have some reason and maybe I don't understand how exactly shitty mocha's beforeEach works - I never had a need to use it, only few times just to try it.
Your examples make sense and I understand them, it's normal, but written this way, yes.

```js
var Logger = require('verbalize').create()
var logger = new Logger();
logger.addFoo('bar');
console.log(typeof logger.bar);
//=> 'function'

var logger2 = new Logger();
console.log(typeof logger2.bar);
//=> 'function'
```

But what about if it is like that (pseudo-code)

```js
var Logger = require('verbalize').Verbalize
var logger = null

beforeEach(function () {
  logger = new Logger()
})
it('add foo', function () {
  logger.addFoo('bar')
  assert.equal(typeof logger.bar, 'function')
})
it('test bar', function () {
  assert.equal(typeof logger.bar, 'undefined')
})
```
Should work as you expect? If not, then I really don't understand the beforeEach and don't see value on it.


Hi Tony,

Something insane happened on this commit(you changes all image files). I recommend reverting this commit:
git commit revert 0414acda7aefddd0d96f845cca9bce5a36632673

And then trying the convert from title-custom again.

-brady


何かプッシュした状態でgit reset を使うと同一ブランチ内で競合が起きてしまうようです。
コミット時に自動プッシュさせてしまっているせいで、これがよく起こるので、自動プッシュ設定を解除しました。


lgtm.


omg the English lol "enforcing gracious prof whenever possible" nuuuuu that makes us sound not as good change it ples :cactus:


Hm. Wait, I think I got it. haha


Alright its workin on my end, now I just need to figure out how to make it respect CMAKE_C_FLAGS because is I understand the documentation from [here](https://github.com/ruslo/hunter/wiki/usr.adding.new.package), CMAKE_C_FLAGS should not be written to.


I love the design of using an implicit class to add abortable behavior. I'm just not sure the signature `pipelineFlow.withAbortable` sends the right meaning. I'm thinking along the lines `pipelineFlow.abortable` alone would communicate better. What do you think? We can discuss.


It shields this particular exception from being rethrown as a `SQLException` in the `catch` below. An example of why `catch (Exception e)` should be avoided.


you can do it if you want


@hustldw @sasa150 @lvyb @FreeTymeKiyan @JoannaCode @forest5013


Nice & clean. Thanks very much. Just the few comments above.


> CMAKE_C_FLAGS should not be written to

Yes, I think it may cause troubles with non-trivial toolchains like Android or sanitizers.


Yes. I did not intend on merging this to master. I was just testing out the sensitivity of sensor.


OK. It fits.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18273](https://teamcity.mesosphere.io/viewLog.html?buildId=153925&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 245 Build time: 00:08:24


This got changed later in the PR. For a while I was taking the approach of using the CommCareTask error catching (https://github.com/dimagi/commcare-odk/blob/master/app/src/org/commcare/tasks/templates/CommCareTask.java#L35-L47) to send the error for processing in `CommCareTask.deliverError` but then I felt like that was making the execution path of error handling very unclear, so I decided to be more explicit and pass values instead of using error handling.


I think in this case we should just write it like this. I guess you could do it with decorators and classes but I don't really want to use decorators since it's not standard yet.


git reset をした時点で、自動プッシュされていたコミットが１つあったことが原因で余計なコミットが一つあります（これ）。下に同一名のコミットがありそちらが、正しいコミットです。


assdf


ok!


You're right, good catch! The commit message is wrong, I was trying to make it fail fast. This commit is reverted later in the PR because I realized it was really wrong to this (would break saving the form as incomplete :cold_sweat: )


done


Major issue in this commit. See here for details:
https://github.com/practiceprovider/openemr/commit/0414acda7aefddd0d96f845cca9bce5a36632673


This ist Not the best because forecast support only Languages like "en" "de" or "ist"
But angularjs also like "de-ch" "de-de"


It turns the terminal values of the recursion, which we assume are lists, into arrays...


Invoke function already checks if the selected object is a mesh.  No need to check again in execute function.


It certainly is! I was hoping there'd be a better way to do it, to be honest.


That doesn't work if offset is in [8, 16): if you're copying length=16 offset=10 from just after the 'j' in:

abcdefghijxxxxxx

then you want to end up with

abcdefghijabcdefghijabcdef

instead of

abcdefghijabcdefghijxxxxxx


Yeah. I tried several other places. This was the most elegant.


no, yes.


추가로 다이어 그램 삭제와 그에따른 전체 길이 줄였음


thanks for your work，I am the author of TranslateApp,and I am thinking  about make a pr in this moment, but I found you add TranslateApp firstly ,it is so coincidence , thanks ~ have a nice day ~


Great point. Added with f46d22274


fixed


@dvklopfenstein somehow this code breaks the build

![image](https://cloud.githubusercontent.com/assets/106987/13518883/6093406c-e187-11e5-846d-3848f34706e0.png)


p_subbed.value? by the end, p_subbed is either a posynomial or a Quantity


SDL seems to be handling cases where an android toolchain is used, same with emscripten though.


This change has also been deployed.


This is due to some craziness in how exceptions are thrown through Android's `ContentProvider`s. As far as I understand, only certain serializable exceptions can be thrown through the content provider `InstanceProvider` calls like `insert` and `update`. I think this is because these calls are delegated through the Android system. So, I throw `IllegalStateException`s to designate a form instance transaction issue, catch the `IllegalStateException` as soon as possible and rethrow as `FormInstanceTransactionException` for better readability later. I think the error class message gets hard-coded through the exception serialization that is done by Android's content provider code.

Phew, this is a sorta messy PR, not super proud of the way this is being done but given the constraints I think it is acceptable. Sorry for not making this clearer in the PR description!


Do you have any tips on how to fix this kind of thing?


As in line 69?


1684_CrashOnEditMeasure (macumber) - i386-Windows-7-VisualStudio-12: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


This commit also fixed a bug where `{{#if}}{{{foo}}}{{/if}}`, `{{#if}}{{#each}}...{{/each}}{{/if}}`, `{{#if}}{{#if}}...{{/if}}{{/if}}`, etc would get out-of-sync with its inner (dynamic) bounds and fail to clear them on re-render when the outer conditional changes.


this is just filler so change it to wtv u want


hmm maybe it's that i'm a bit tired, but...
_get_uri/get_method need to get the action_name to be able to construct the differently named functions during the initialization of the class (at __new__). otherwise we cannot invoke it, and I think the _actions dict is the most compact representation of the mapping I can imagine.


1684_CrashOnEditMeasure (macumber) - Win64-Windows-7-VisualStudio-12: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


Changed.

2016-03-03 12:25 GMT-08:00 Matteo Pagliazzi <notifications@github.com>:

> @piousbox <https://github.com/piousbox> you should use a translatable
> string for this. It should be added to common/locales/en/api-v3.json and
> can be used in the API by calling res.t(stringName). While in tests you use
> t(stringName)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/piousbox/habitrpg/commit/140313041b5152b1613b8d806cf474402a823272#commitcomment-16484545>
> .
>

--
Victor Pudeyev
piousbox@gmail.com


Having used $0 this ends up grabbing the whole path of the script and trying to use that as the tmp filename which isn't working on my system. Shouldn't this be $1 ie the name of the iso?


IIRC, the compiler can change the order of instructions in order to optimize. It's not that critical place to bear my brain, trying to bring it to perfection.


local lfs = require("libs/libkoreader-lfs")
lfs.attributes("/sys/devices/virtual/input/input1/neocmd", "mode") == "file"

can provide same function without opening the file itself.

And using a local variable to store the long file name would be better.


Getting this message with a freshly compiled version. Not outdated.


How is it that the tests are run? I think having a `contributing.md` would be good. I would love to help but I'm sure where to start, haha.


> Do you have any tips on how to fix this kind of thing?

Can you add a link to the exact line with code


The below is not valid `shell` (because of the `$`), so you can't use it. Apparently `console` works in this case, but is rarely highlighted anyways, so YMMV.


pat_mega_branch_rc (macumber) - Win64-Windows-7-VisualStudio-12: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


Can't use numpad shortcuts anymore

- custom shortcuts stopped working ( + and -)
- Setting numpad shortcuts shows letters instead of numbers and +-/.


fixed


test


Ah, yeah, I didn't realize it was reverted. The hazards of reviewing commit-by-commit...


CMAKE_C_FLAGS gets initially modified [here](https://github.com/hunter-packages/SDL2/blob/hunter/CMakeLists.txt#L151) but it's also modified at various other places throughout the file.


Why is this even an instance method?


`let` => `const`


Well...that kinda sucked.


You're not even going to allow modifying `ItemStack` instances before the server started? Why is that? It would make sense for `Entity` and `TileEntity` instances to be illegal to exist before the server actually started, but it doesn't make sense not being able to generate some easy things like `ItemStack`s outside of the server context (like on the client before the server started up).


Hello ,

Thanks for taking pain and writing tome.

I am trying to produce the POC for BRMS for decision table capability  for
last three days with BRMS 6.0.2 and JBOSS 6.4.

Can i download  brms-coolstore-demo-master.zip at "c:\temp" and execute the
file init.bat than it will install product(EAP and BRMS)  and demo together
? Do i need to setup any prerequisite ?

BTW , wiith my current setup ( EAP 6.4 and JBoss BRMS 6.0.2and able to
clone the repository )  , I am able to bring the environment but not able
to bring cool store web app war file is missing so i can not invoke the
BRMS environment .

Any help will be much appreciable sir.

Thanks
Tarun

On Fri, Mar 4, 2016 at 7:15 AM, Eric D. Schabell <notifications@github.com>
wrote:

> First, the get started uses all the latest versions of JBoss EAP and BPM
> Suite, see the project you downloaded: installs/README for the details.
>
> If you need to use 6.0.2 then the way to do that is to switch to tagged
> version that matches, JBoss EAP 6.4 does not work with JBoss BRMS 6.0.2 so
> see the RELEASED VERSIONS at the bottom to pick what you want to use:
> https://github.com/jbossdemocentral/brms-coolstore-demo
>
> If you want to use the most recent then you need JBoss EAP 6.4 + patch 4 =
> EAP 6.4.4 and then JBoss BRMS 6.2.0.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jbossdemocentral/brms-coolstore-demo/commit/403cf0429463f4ae428cc10dd2adbb5a6178e9a5#commitcomment-16490376>
> .
>


Good catch!


Done as commit 5f1c01d9.


These are because in the previous commit I had done some formatting issues, so reverting them it shows a diff compared to the previous one.