If this is supposed to get the target of the symlink, then:
https://social.msdn.microsoft.com/Forums/vstudio/en-US/aaa2d1b8-b302-4939-861e-5b011b8b8a50/how-to-get-the-target-of-a-symbolic-link
(it doesn't allow linking to a section of the page, so Ctrl+F for "I think I have a solution now.")


Nope, now it'll be caught by the first one.


I have no words to tell how much I hate these SJWs and other "thought/morale police" and they're infesting everything, Reddit, GitHub, etc, I guess it's just matter of time till only way for sane people will be websites on Tor hidden services.
If code is good, I will accept commits from mass murders, ISIS, rapists, serial killers whatever, I don't care, it's irrelevant to the code.


Right. Let's keep it, as it won't conflict with anything else.


Can someone explain me what the addition of the hard-coded "sketch" folder for output files is good for?
This may break 3th party tools because the folder does not exist and according to this page https://github.com/arduino/Arduino/wiki/Unofficial-list-of-3rd-party-boards-support-urls all platform.txt files need to be updated.

    Note for core maintainers: In IDE 1.6.6 or greater the folder structure is different so maintainers should update their platform.txt according to this fix made by NicoHood


Initial spanid === traceid.


Tüm kopyaları bulup silen hale dönüştürebilir misin?


remove extraneous "is"


fix it and open up a pr.


Missing files, again.


Yeah that's what I figured as well (sorry I edited my previous post instead of writing a new one). Will have to think about how to best solve this issue.


Bunu ben de başta böyle yapmıştım fakat aynı boyutta olmasının yan etkisi oluyor. State grupları genelde en üstte durduğu için altındaki layer'ları seçmede zorluk çıkarıyor. O yüzden 1px yaptım, hatta boyutsuz olabilse öyle yapacaktım.


Did you mean to use data variable from the where clause here?


Checkmarx scan results 2015-08-06T13:17:33Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
26
17
30
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

4f27efd246e3584e896c37b5694aef1c5fcfdfb4	@amitchen22

a17e605cef0c9a86bdf46b9af412c42141162c91	@amitchen22


ohh, I was just copying; https://github.com/pyblish/pyblish-maya/blob/master/pyblish_maya/lib.py#L28


Yes - bad merge of two older functions.


Teşekkürler katkın için Özgür, 2 tane yorum ekledim, düzeltirsen merge edeceğim. Eğer ilk yorum için nasıl yapılacağını bilmiyorsan veya zamanın yoksa o kısmı daha sonra ben yapabilirim, zaten eklemeyi düşündüğüm birkaç şey var. : )


I think you meant `$.qux` here. There's no `$.quz` in the sample JSON.


Ok, sauf la capture d'écran de StreetView. Mais on va dire que c'est temporaire.


I am saddened that we couldn't have:

```haskell
do (fits,st) <- fitsOnOneLine (depend left (f right))
   let i = id
       sits = put st
   if i fits
      then i sits
      else do left
              newline
              f right
```

:joy_cat:


I don't think this is safe. `L` could be a lua thread (coroutine) and it could be garbage collected by the time that the callback runs.


I know, it should be removed from there as well. Better not pass it on.


So `protobuf.txt` would then be
```
protobuf https://github.com/google/protobuf/archive/ad8a889d1e1e2b0efd5b7579aa57ea5326cda6da.zip
```
I've never tried `*.zip` files, only `*.tar.gz` but it might work.


5058-fix-readvars-cli (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1795 of 1795 tests passed)

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


Technically this is a `.py` file and not a `.ini` file.


Thanks for putting the spaces back :+1:


It's used a few times in this patch - mainly in some ifs, where we also check other things as well.


I cherry-picked it and then re-ordered

Hm, maybe i should have cherry-picked and chnged/amended it..


Проверил, вроде работает.


this was the bug


lol


On Aug 6, 2015 9:52 AM, "Alexander" <notifications@github.com> wrote:
>
> I didn't mean that I'm passing anything as a reference. I meant I'm
having a reference to an allocated object. I'm responsible for it's
deleting, but I want it to be public, so other organisms could use that if
they have access to such organism.
> May be that could be implemented using private reference and a special
getter method, but who needs this if I can just have a public reference?

Ok. You are responsible, not the compiler. It's about trade offs.
Writing more code or unsafe code.
That said, they are still c and will eventually disappear.
But please keep in mind that they are fundamentally different approaches.

> Also, I didn't get that about spawn ->
beginning-of-class-body-until-awaits -> continuation-of-spawn... Do you
mean I can write something like that:
>
> every arg in EVENT do
>     spawn Organism;
> end
>
> class Organism with
> do
>     var _CppObject&? = _new_CppObject(arg);
>
> ?

Yes.

> That really looks strange.

As strange as awaking from events. Think of constructor as an emit.

> If you mean that I can do something like:
>
> every arg in EVENT do
>     spawn Organism with
>         this.arg = arg;
>     end;
> end
>
> class Organism with
>     var _Argument* arg;
> do
>     var _CppObject&? = _new_CppObject(arg);
>
> Then I don't see the difference between that and just having a field.

The difference is that the  allocation is correctly  inside its parent
organism and you can use finalization properly.


> custom *html control* elements?

As per spec the word `control` seems to refer only to html forms, and checkboxes and inputs are controls in that sense, but they were used merely as examples, I'm actually referring to any arbitrary html element, div, li, a, video, audio, whatever.


Checkmarx scan results 2015-08-06T13:24:01Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
26
17
30
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

b7c6882117d9d34c762881b185ba6ee69140d269	@amitchen22

4f27efd246e3584e896c37b5694aef1c5fcfdfb4	@amitchen22


Checkmarx scan results 2015-08-06T13:24:18Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
26
17
30
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

b9ddb1d17177ae1a0e190277ec18e5cc8e2e2d01	@amitchen22

b7c6882117d9d34c762881b185ba6ee69140d269	@amitchen22


> tree nodes *values*

What? You mean properties/attributes? If yes, here I'm referring to all changes that could happen, not only property mutations, but also adding and removing child nodes.


Yup. We are using Amazon Linux. I would  suspect same behavior in
centos/Redhat but I haven't validated that.

Sijis
On Aug 6, 2015 7:27 AM, "rohitagarwal01" <notifications@github.com> wrote:

> ah thanks ... able to replicate the same on linux
>
> \n works fine on mac and ubuntu
> [image: :+1:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kenzanmedia/ami-roles/commit/f267428a0bbe49b9ad0ebeeac60438aa75aea3c1#commitcomment-12563674>
> .
>


add  errno function handle


Ok – I'll revisit this. What are you viewing it in that's overriding the styles so I can test it properly in that scenario?


Is this the right order for these two? As it is, the config file will win out over command line flags.


newline before if


I wonder... Should this show dependencies for jquery and jquery UI?


FieldEditorPreferencePage lacked support to customise layout and incorporate more advanced widgets such as Groups.


Please reorder the constants to keep them alphabetically ordered.


Jawohl!


我的 PR 就是解决这个问题的,  为什么要自己实现, 不选择 merge 如果有不对的地方, 可以 评论讨论.  这还这么贡献代码 ???


Le 06/08/2015 13:18, Michael Burman a écrit :
> Why volatile in a subscriber?

That was a bad copy/paste from previous work


this returns keys not values right?


no need, the devDependencies are only needed for a NodeJS env. typically a contributor would run {code}npm install{code} and then after doing changed to "MonthPicker.js", he/she would run {code}grunt{code} to generate the minified file


Factory could be called more than once if threads compete to be the one to create the value.  Just adding this note in case this tries to go into the Kotlin std library, it is not technically correct to be equivalent to what Java 8 has as `computeIfAbsentUnsafe` which only calls factory once.


Jussi, I don't think this change is required, as null termination is not required for members of struct utmp, as the data is write to file as sequence of bytes i guess.


"Ethics".

On Thu, Aug 6, 2015 at 8:23 AM, Dāvis <notifications@github.com> wrote:

> I have no words to tell how much I hate these SJWs and other
> "thought/morale police" and they're infesting everything, Reddit, GitHub,
> etc, I guess it's just matter of time till only way for sane people will be
> websites on Tor hidden services.
> If code is good, I will accept commits from mass murders, ISIS, rapists,
> serial killers whatever, I don't care, it's irrelevant to the code.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nixxquality/WebMConverter/commit/c1ac0baac06fa7175677a4a1bf65860a84708d67#commitcomment-12564785>
> .
>

--
Kurtis Rainbolt-Greene, Hacker
Software Developer
1631 8th St.
New Orleans, LA
70115


坐标偏移  #27  也解决了....   @tudou527


@accnex569 si, lo tengo, pero lo implementé después de poner mi comentario. Aquí está mi ejemplo

```
<% if (locals.errors !== undefined && errors.length) { %>
    <span id='ErrorMsgs'>
      Corregir errores:<br/>
      <% for (var i in errors) { %>
        <span> <%= errors[i].message %> </span><br/>
        <% } %>
    </span>
  <% } %>
```

@aperezl  si te fijas mi ejemplo es diferente al tuyo. Hay que utilizar el objeto global locals que es donde se guardan las variables que se le pasan a la plantilla. Yo lo tengo así y funciona perfecto.


Once again, go back to tumblr  and gamerghazi

		 ---- On Thu, 06 Aug 2015 06:41:14 -0700  notifications@github.com  wrote ----"Ethics".

On Thu, Aug 6, 2015 at 8:23 AM, Dāvis <notifications@github.com> wrote:

> I have no words to tell how much I hate these SJWs and other
> "thought/morale police" and they're infesting everything, Reddit, GitHub,
> etc, I guess it's just matter of time till only way for sane people will be
> websites on Tor hidden services.
> If code is good, I will accept commits from mass murders, ISIS, rapists,
> serial killers whatever, I don't care, it's irrelevant to the code.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nixxquality/WebMConverter/commit/c1ac0baac06fa7175677a4a1bf65860a84708d67#commitcomment-12564785>
> .
>

--
Kurtis Rainbolt-Greene, Hacker
Software Developer
1631 8th St.
New Orleans, LA
70115
—Reply to this email directly or view it on GitHub.


> tree-view-view*?*

Since the differentiation between the tree view logic and the tree view representation in the DOM wasn't clear enough from the diagrams I had to find another way to explain how applications in a browser work. It seems that the confusion is rooted in the expectation that a browser only receives markup from the server and renders it. Which isn't the case anymore for a lot of websites.

If you have an idea how to make that clearer, I'm happy to incorporate it. I could go more into detail at the risk of loosing track of the big picture (whatever that is, no idea what all of this is about anyway) and probably not finishing it at all.

So I'm open for suggestions.


Why we included metadata into windup.test.xml file? I am trying to create XSD for ruletest and this is IMHO useless in tests or am i missing something?


:fireworks: :+1:


宽度计算误差, #27  也解决了. ........ @tudou527


Done.


That is kinda what I thought, but wanted to get your thoughts.


But, wait... Why would someone use Node.js with this plugin?


why ruleset id when ruletest id exists?


This is contrary to the referenced branching model! --> http://nvie.com/posts/a-successful-git-branching-model branches should be kept develop and master


@samccone Is `drool.flow` synchronous thought? You are calling `driver.quit()` immediately after.


Oh to contribute to the plugin.


they wouldn't, nodejs is only needed to run the grunt task that will uglify and minify the source, other than that it is not needed in any possible way


My bad, I can mess up the easiest of the commits.


yes, yes it is /o\


\*stares at fingers\*


We no longer check for vasprintf being available at all, how does this even work?


Can someone explain me what the addition of the hard-coded "sketch" folder for output files is good for?
This may break 3th party tools because the folder does not exist and according to this page https://github.com/arduino/Arduino/wiki/Unofficial-list-of-3rd-party-boards-support-urls all platform.txt files need to be updated.

    Note for core maintainers: In IDE 1.6.6 or greater the folder structure is different so maintainers should update their platform.txt according to this fix made by NicoHood

Why not have a build.output.folder={buid.path}/sketch in the platform.txt and use that instead of hard-coding "sketch" in the code and platform.txt?


Ah, no. It's 40% visible (line 118).


OK. then, now I am wondering if I should take out the minified version that is posted since there is a grunt task now.


downloading the .zip from github fixes the patching process.

the next (and possibly last) piece of the puzzle is the cmake line you linked to above: https://github.com/kineticgar/game.netplay/blob/3d9401f/CMakeLists.txt#L177

maybe a new macro named `build_addon_library()`?


it is async, however it is executed in the webdriver flow :droplet:
https://github.com/samccone/drool/blob/master/lib/index.js#L5-L11

thus it takes care of queuing actions ... if we wanted to do more async work around it we could use `.then` but for this example we do not :clap:


no you should leave, a user of the plugin would add the minified file in his/her index.html while a developer would modify the non minified file and run grunt to generate the minified file


Saying "that really looks strange" I meant that in that example `arg` is not even mentioned anywhere in `Organism`, but somehow it knows that there is a constructor somewhere in `every` where there is `arg`. Does that mean if I'm spawning that class in different places I have to pass similar set of arguments (having equal names) or all other arguments would be passed as `null` or something like that?

Saying that "I don't see the difference" I meant if the second example was correct, then I don't see the difference in passing constructor arguments and just having public fields. If you say that I don't have to declare `var _Argument* arg;` to pass constructor argument (like in first example), then I see the difference.


Reverted in fcfa17ec76fb15c28291e8f75111015fd6e10ca5.


Duplicate line


```ImmutableList.copyOf(Array.asList(nullableArguments))```


imook first-page


> up to date -> *up-to-date*

`up to date` is also possible and I have plenty dasherized words/word groups, so opted for not choosing dashes here.


websphere-mq-test would be better, IMO


Finally! No more

~~~
A copy of XYZAbility has been removed from the module tree but is still active!
~~~

errors :dancer: :tada: :+1:


We might not want to do this for some interfaces; there is a `shared.IsLoopback` and a function `isBridge` in networks.go that we could use to exclude some of the noise. Although perhaps we want to do bridges, I think it's probably safe to exclude loopbacks.


Я ничего нового для тебя не написал. Тебе нужно еще раз внимательно(!) перечитать вот это:
- создать функцию, которая принимает а) элемент, в котором поменялось значение, б) див-блок, куда будет выводится лог
- в функции определить тип элемента (инпут, селект, чекбокс)
- в зависимости от типа элемента соответствующим образом выбрать значение
- добраться до лейбла, получить имя
- сформировать лог вывод
- добавить в див-бок сформированный лог вывод

На каждом этапе пытаться понять ЧТО это значит и КАК это сделать.
Если не понятна интерпретация - пиши, я перефразирую.
Будь честным перед самим собой и старайся сделать больше, чем ты можешь.


Checkmarx scan results 2015-08-06T13:50:12Z

High
Medium
Low
Info

New
72
61
66
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

d5a07cb0b1744f15f637a4c7ffe8f2ac25f77bdd	@dudug


This is not generic; even though I understand why you are providing this it won't work for all filesystem types and thus does not seem something we ought to add.

I think this should be dropped.


ok we will offer a minified version and we will offer the grunt task for devs. However, we used the Google Closure Compiler to generate the minified version that is already posted. Perhaps we should use the Grunt task to uglify and minify then post that instead of the Google version.


а зачем так? чего просто не сделать дженерик метод?


How use easypr to opencv VideoCapture();


Cool :sunglasses:


> It does sound from the commit title like the author wanted to disable inline altogether, which is strange.

IIUC gitbook has a kramdown-like parser. Kramdown's pre-processes math content into MathJax script tags long before MathJax gets to it. IIRC, kramdown uses `$$` as delimiter for both inline and display (stand alone paragraphs force display mode).

BTW, the PR might as well remove the entire surrounding MathJax section since escaping is only useful for `$` delimiters (it enables `\$` outside of delimiters to write a plain $ in regular text).


And everything is still working?


this checkState should be done anyway. Not only if nullableArgumets is empty


copy and paste!!!


And I would rather wrap nullableArguments in Optional()


Checkmarx scan results 2015-08-06T13:51:52Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
72
61
66
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

d5a07cb0b1744f15f637a4c7ffe8f2ac25f77bdd	@dudug


@Alexandru-Dobre Please, can you check that this commit will not generate some regressions when the user change the language. It's working with search's facets and specific user given to the user profile block.


If you just set color: 'white' on any ancestor (e.g. <body>) you'll see the issue.


The scan was canceled by the user   on 8/6/2015 4:41:27 PM
Included commits

Commit ID	Owner

8bd9595c05df285efff30023ebad3628161531cb	@amitchen22

b9ddb1d17177ae1a0e190277ec18e5cc8e2e2d01	@amitchen22


Checkmarx scan results 2015-08-06T13:52:33Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
36
31
37
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

2d0494404c3b5f9f12aca923ee0f95a604f42955	@dudug

a84eb07efc81b2b508b79aca5bfce286b84281c2	@dudug


> Tree-view-node-views -> *Nodes in tree-view*

I was aiming for a unified, clear and unambiguous language, since all approaches before seemed to be too casual and confusing to the reader.

So, yeah, this suggestion confuses me a bit. I thought it was about being unambiguity, not about a pleasant reading experience?


IMX_MIRROR ?= "http://www.freescale.com/lgfiles/NMG/MAD/YOCTO/"
QORIQ_MIRROR ?= "http://git.freescale.com/source/"

# FIXME: set this to avoid changing all the recipes that use it
FSL_MIRROR ?= "${IMX_MIRROR}"

MIRRORS += " \
${IMX_MIRROR}	http://download.ossystems.com.br/bsp/freescale/source/ \n \
${QORIQ_MIRROR}	http://download.ossystems.com.br/bsp/freescale/source/ \n \
"


The scan failed.
Included commits

Commit ID	Owner

2aeb9b29b34ba431023aad6aa08d7172099192db	@amitchen22

624330688cf5f3ea97b574e55002160c33f1cdb1	@amitchen22


That method (using `this.key`) for defining the hash looks good...

... But doesn't work.
It would have be a good idea, if it worked as it seems to be.


S should be here.


Indenting change, please drop it.


This has been deployed.


Looks good


Checkmarx scan results 2015-08-06T13:54:44Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
72
61
66
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

d5a07cb0b1744f15f637a4c7ffe8f2ac25f77bdd	@dudug


#ExperienceModificationCustCA


Note to self, when it does segfault itself it all goes like this.

![screen shot 2015-08-06 at 9 52 27 am](https://cloud.githubusercontent.com/assets/378969/9113187/2b6abc14-3c21-11e5-9027-a36eff838387.jpg)


Checkmarx scan results 2015-08-06T13:44:38Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
26
17
30
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

624330688cf5f3ea97b574e55002160c33f1cdb1	@amitchen22

8bd9595c05df285efff30023ebad3628161531cb	@amitchen22


Missing ending semi-colons. `;`


Yes: https://travis-ci.org/AlchemyCMS/alchemy_cms/builds/74409122


Checkmarx scan results 06 Aug 2015 16:55 UTC +03:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
1
1
2
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

791f7872c7a0155398170d39fdda41f4f4f55f49	@cxehud

126427f2bc9eb584bc6a2524232a83ef5f0d80f6	@cxehud

866abb84ce85830600babd0975f58aad7128cee8	@cxehud

3d57d8106b46655d018602cb17178aab4e20d809	@cxehud

43e518e047774cde9c979060cdebf78e1240acbb	@cxehud

0889eb7c606026a975240d5a9b9c51f31101dbd0	@cxehud

4b72aeaef585a56f19b86f9216812ae0d69bc2f7	@cxehud

04815a6091c58c7858fb69d13685f3fad2c0ab6d	@cxehud

2df19aa46451ad26f4dc72164d23cf438903565b	@cxehud

ad58b128e2fede39820ae668970bbfab38b56195	@cxehud

6b05867d4fe80a4c3b55cb36e0b6f76abc0a59f9	@cxehud

accf7e51faac6fffc26f426e5afeeaf3ed561731	@cxehud

7917c1e1165bbb3f8b67a7c2943b68d3780e2a79	@cxehud

74262a0e47571dc6ed4aa4c3a03f759fee5bd42c	@cxehud

5b96953244ae55ea53f4983dc1a9fa4d4eb872a6	@cxehud

2166d1ea7164e801468b1b1f20ff0a3ecf4c67dd	@cxehud


А давай запилим взять УРЛ из контекста или конфига


The scan failed.
Included commits

Commit ID	Owner

d3982971531b4696b570d7f732f05bce9b86d352	@amitchen22

67248ee19dce201264984aff8d54f4f1410a6029	@amitchen22


Currently the prime tower is a feature that is broken. I'm working on it right now.


sorry, it was the github system's wrong,it send a mail to notice me but i'm not the "Freezen"... ...


Kill extra whitespace!


The commit seems file however it would be good to check if it cannot be reworked to avoid code duplication for arm and ppc.

This can be done in a new patch though.


@Ninikitos DWMR. You've got good momentum going.


We only ensure that the registered Abilities are called as top level namespaces and not in Alchemy module namespace.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.466](http://teamcity.monogame.net/viewLog.html?buildId=9457&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


> If code is good, I will accept commits from mass murders, ISIS, rapists, serial killers whatever, I don't care, it's irrelevant to the code.

maybe not fro NSA, though. who knows, they might be planting back doors there.


This does not work for qoriq-arm? why? this cannot be used with imx? why?


Apparently. I'll fix that :|


Checkmarx scan results 06 Aug 2015 16:57 UTC +03:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
1
1
2
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

a546127f9b90a0b0e407280b16c7aabc093301d2	@cxehud

fbc928ac93fd95214ca45c22e564e8247d64d868	@cxehud

b96d5230998076a1a547ebcf1ed71aa0fa3a72d2	@cxehud

791f7872c7a0155398170d39fdda41f4f4f55f49	@cxehud


So I didn't! All done now :)


This is a distro thing; what are the changes on top of original busybox setup?


out no debe tener '/'
el archivo debería quedar:
.idea
out


check this.


Checkmarx scan results 2015-08-06T13:58:23Z

High
Medium
Low
Info

New
72
61
66
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

d5a07cb0b1744f15f637a4c7ffe8f2ac25f77bdd	@dudug


On Aug 6, 2015 10:48 AM, "Alexander" <notifications@github.com> wrote:
>
> Saying "that really looks strange" I meant that in that example arg is
not even mentioned anywhere in Organism, but somehow it knows that there is
a constructor somewhere in every where there is arg.

Sorry, yes you need the explicit argument, for sure.

> Saying that "I don't see the difference" I meant if the second example
was correct, then I don't see the difference in passing constructor
arguments and just having public fields. If you say that I don't have to
declare var _Argument* arg; to pass constructor argument (like in first
example), then I see the difference.

You have to declare.
The difference is that your code executes atomically between the spawn and
the use of the pointer.
There's no chance this pointer becomes invalid.
However, the nested object you create survives the spawn and can't be a
pointer and safe at the same time.


Checkmarx scan results 06 Aug 2015 16:58 UTC +03:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
1
1
2
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

ba761704ac750669c5379e312bb4b9a65409779b	@cxehud

a546127f9b90a0b0e407280b16c7aabc093301d2	@cxehud


Both these unit tests already exist in the file


Looks like SHGetFileInfo() with the SHGFI_EXETYPE is needed here.
https://msdn.microsoft.com/en-us/library/windows/desktop/bb762179(v=vs.85).aspx


Checkmarx scan results 2015-08-06T14:00:22Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
36
31
37
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

a285d23669ea0e8d6e5562a77bf673a6c4df2b0e	@dudug

2d0494404c3b5f9f12aca923ee0f95a604f42955	@dudug


This is a distro thing; please lets postpone this to when we choose where to put the distro.

Please make a qoriq-distro branch in your repository so we don't lose this patch but drop it for now.


@jedesah I can reproduce this in a simple project. Look here: https://github.com/ahjohannessen/remotely-14

Notice the differences in package for `ClientSuccess` and `ClientFailure` examples.


@PlaySpot ok!


Checkmarx scan results 2015-08-06T14:00:53Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
72
61
66
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

d5a07cb0b1744f15f637a4c7ffe8f2ac25f77bdd	@dudug


Good.


Does this solve the issue identified in #234 ?


GetEnvironmentVariable() to get the PATH:
https://msdn.microsoft.com/en-us/library/windows/desktop/ms683188(v=vs.85).aspx
Directories are separated with semicolons, so the code below will need some small tweaks


It cannot work on ARM?


I've include android build to my fork. it build fine on my ubuntu  linux and run fine on my lg-g2.
look at  mame/src/osd/retro/readme_libretro.txt to see the syntax i've used to build .


This also fix this problem:

http://www.openwall.com/lists/john-dev/2015/08/03/7

:+1:


No, just leave it 2.2.5, which is not yet released.


this is what i have already done, the minified version is now generated with grunt


so many people know the camel !


Put this to existing 2.2.5 section please, remove the 2.2.6 section


Checkmarx scan results 2015-08-06T14:02:33Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
72
61
66
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

d5a07cb0b1744f15f637a4c7ffe8f2ac25f77bdd	@dudug


> reduce -> *divided into*

Why? It was 4 steps before, and now it's only 2 steps.


@Ninikitos DWMR. The last test was intended to have you add the quote to the exsiting dictionary. But you found a way to get the test to pass doing it differently. I love it! Press on.


from: http://www.pypedia.com/index.php/convert_plink_to_Merlin


Checkmarx scan results 06 Aug 2015 17:00 UTC +03:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
1
1
2
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

dbbaea2e0baf5bf274cce9af946880a05c501e25	@cxehud

ba761704ac750669c5379e312bb4b9a65409779b	@cxehud


Indenting change, drop it.


@herrniemandHe is not here for a discussion. He is from /r/gamerghazi which is a subreddit devoted to harassing and bullying gamers and 4chan users. Since a lot of people who don't use this tool are here, I'll explain that this is the favorite webm conversion tool of 4chan and Reddit's gaming communities. The Ghazi assholes are here under false pretense of being offended because they want the repo shut down in order to screw over gaming hobbyists. Which is proven by him not being happy with the name change.

		 ---- On Thu, 06 Aug 2015 06:57:33 -0700  notifications@github.com  wrote ----
If code is good, I will accept commits from mass murders, ISIS, rapists, serial killers whatever, I don't care, it's irrelevant to the code.

maybe not fro NSA, though. who knows, they might be planting back doors there.
—Reply to this email directly or view it on GitHub.


Checkmarx scan results 2015-08-06T14:03:12Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
36
31
37
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

402fabbc19af9dae6fcdb9bb1a44cb8e7036ab22	@dudug

a285d23669ea0e8d6e5562a77bf673a6c4df2b0e	@dudug


The previous comment was more elucidating.


Thanks for testing. The exception can also be triggered by the reload option. That has perhaps not been mentioned.


thanks @dietzc
once i got the right imports there, and fixed the input make image, which was a different size than the test images (hence the array index out of bounds exception), everything worked as expected! cheers!


Good.


Array.asList won't work for primitive type array


初プッシュおめでとう！


nullableArguments is passed to FunctionInfo so it has to be created somewhere. Probably best here


Por qué pusiste tabulaciones???


fxd


In a general note, please use qemu-qoriq so it is clear it is product family specific. I did a similar thing for openssl so you can base on this to set the proper provider.


Alright now.


> *, since ...*

:confused: I am supposed to explain why doing more in a frame is less efficient than doing less?


Of course, this should be changed back to moz, once  https://github.com/mozilla-services/lua_sandbox/pull/95 is merged.


Nope. Good job.


damn, that snippets for `standard` :D

i'll fix it


sorry I'm not sure I understand. do you mean setting up a `rake spec` task?


Checkmarx scan results 2015-08-06T14:05:23Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
72
61
66
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

d5a07cb0b1744f15f637a4c7ffe8f2ac25f77bdd	@dudug


Good job.


@Konnobu ありがとう！


@krainboltgreene what ethics? I don't see there anything unethical, code is code and if it solves a problem it doesn't matter who wrote it.

> maybe not fro NSA, though. who knows, they might be planting back doors there.

@makc how do you know that contributor who sent you a PR isn't from NSA? and do you think others like China wouldn't want backdoors too?
You can't know who is that contributor and what actual intentions he have so there's no point of looking at that, but need to carefully inspect patch and always learn new things so you would notice anything fishy ;)

Anyway I think I'll better stop posting here.


a usual pitfall is that alias should end with a /
did you try
```
alias YNH_WWW_ALIAS/;
```


@Ninikitos DWMR. Concise code. Well done here.


Have you tried simply removing the `LIBRARY` part from `LIBRARY DESTINATION`? Or will that install too much?


Checkmarx scan results 2015-08-06T14:06:04Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
72
61
66
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

d5a07cb0b1744f15f637a4c7ffe8f2ac25f77bdd	@dudug


I couldn't find adequate methods for deep nested hashes there


Why did you remove this warning? I'd rather keep it in there to ensure we know something odd is happening.


ok


This commit has a bug -> the QemuImg class doesn't have a `params` attribute, causing failures at postprocess.


The scan failed.
Included commits

Commit ID	Owner

ccf8a9c24d576e188f8e519f132b05770f1b9a26	@amitchen22


The comment should be: #"/"


> "Ethics".

TIL that innocent ASCII characters have ethics attached. Wow.


Like a universal CGPA calculator. Click on any subject, and the CGPA changes.

Sent from my iPhone

> On 06-Aug-2015, at 4:05 pm, Aneesh Neelam <notifications@github.com> wrote:
>
> What do you mean by simulation?
>
> —
> Reply to this email directly or view it on GitHub.
>


Should have been `#!/logout` ... I overlooked that while resolving conflicts..


@kwhinnery This should contain a more generic URL, and don't forget about the password


Same comments as qemu one.


I like that this was changed from a `die` to a `note`, but the note should only be generated when there are no outputs in `$hashref`.


On line 34:
you are using a `if-else` statement, but if you look at the code, there is not much chaning, what is changing is just 2nd parameter. And its either `GPIO.HIGH` or `GPIO.LOW`.

So I would say this is an example where you can apply the DRY principle.
How about something like this.
```python
>>> states = (GPIO.LOW,GPIO.HIGH)
>>> print state[0] # this will print value of GPIO.LOW
>>> print state[1] # and this will print GPIO.HIGH
```

now if you consider  zip approach:
```python
def displayNumber(bitsAndPins):
     for bit,pin in bitsAndPins:
         GPIO.output(pin,states[bit])
```

There is one thing worth mentioning.
I have used `tuples` for `states` while I could have used `list`, also they can also be indexed.
But *tuples are `im`mutable* (there value cannot be changed) while `list are mutable`


this is where I meant the newline should be, or you could make it an else if to match the rest of the method.


Cool!


That's what I was thinking about. LGTM.


Ah it should be.


Ooops. So *that* was why there was a merge conflict during rebase :'3


the logic of setting this transient is decoupled now from config generation as far as this object is concerned. this should be moved to a place where we know that we can trust it will be called on its own at the correct time.

perhaps something like "prepare to receive encode" or something.


"policy": "pernament",

Typos in per**m**a**n**ent

Don't know if it matters


This works, but:
"The <> spelling is considered obsolescent."
https://docs.python.org/2/reference/expressions.html


TestScalarPolymor...


Sorry, missed this response...  So you're saying we use the provided registry to encode the metadata but not to decode it?


looks good


s/support/supports


typo, `--backing-chain option not supported`


this function calls a function that recursively calls itself n times,
it does not satisfy: "do_n that takes a function object and a number, n, as
# arguments, and that calls the given function n times"


Why not use `gears.surface` here instead and then making a copy of the returned surface?
```lua
function duplicate_surface(s)
    s = gears.surface.load(s)

    -- Figure out surface size (this does NOT work for unbounded recording surfaces)
    local cr = cairo.Context(s)
    local x, y, w, h = cr:clip_extents()

    -- Create a copy
    local result = s:create_similar(s.content, w - x, h - y)
    cr = cairo.Context(result)
    cr.source = s
    cr.operator = cairo.Operator.SOURCE
    cr:paint()
    return result
end
```


I see that now. Sorry about going full circle on you here, once or twice.
lol Thank you for discussing this with me. You do very professional work,
we are all very happy and very lucky to have your contribution.

On Thu, Aug 6, 2015 at 9:02 AM, Fouad Kada <notifications@github.com> wrote:

> this is what i have already done, the minified version is now generated
> with grunt
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fouadkada/jquery-ui-month-picker/commit/9231a141f5736e10f34a93c852b725c89b647f06#commitcomment-12565713>
> .
>


Does this work for that minnow accelerometer?


Correct - GridFSFile only returns a Document for the metadata


If info is called with default params, this will raise an `IndexError`


Some API documentation is missing that describes this and mentions that this only uses the alpha component of the image and ignores the "actual content".


Yet, the param doesn't do anything below. This param should be simply removed.


Checkmarx scan results 06 Aug 2015 17:14 UTC +03:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
1
1
2
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

2a08a0d1f388e1e7f43b1d124f36cac7e57bfd5f	@cxehud


no worries, your project came really helpful in a project i am working on


great! however, the casting is kind of dangerous. @ctrueden, any comments?


Now that you're in it, s/transaction/transactions/


first commit


this should be in-line w/ your first if
what happens if I input a n that is <= 2?


Careful with those spaces between macro content and "\" -- we're avoiding them.


Forgot to mention that the argument could be passed by copying a struct, or
each field individually. Than you'd need no tricks.


resume

自我介绍

接触编程不久，正在努力学习前端ing。
github也是接触不久，在慢慢的探索中.......


@SkyzohKey Yes, I read about it already thanks. I'll do it next time I have another 1-2 hours for coding.

Initially, I wrote the script to help some friends and people not into computing. Thus I want it to be as easy as possible for them - and in native language.


double check that this output is what you expect.


@mikepetersonccv , I'm changing the FinancialTransaction model so that it no longer has the CurrencyTypeValueId field.  It's being moved to a child FinancialPaymentDetail table (see my comments on [this pull request](https://github.com/SparkDevNetwork/Rock/pull/1123) for more details) This is obviously a problem for this migration that includes code that references that field. I can update the code to use the new FinancialPaymentDetail child model's field, but when this migration runs, the model in code, won't match the database since the migration to update the database would run much later than this migration.  Thoughts?


Great test.


@brianreavis can you briefly explain why we have this `common` + `options` trick and not just a simple `options`?


Good.


This commit has been posted for review (see issue #85).


@hbun DWMR (done with my review). This is good.


here you specify that hsqldb can be stream connected to by using the create_stream_socket_perms, however, never in your policy is there specified that maintains a socket. This inconsistent.

Either hsqldb maintains a socket that can be used to stream connect to hsqldb or hsqldb creates unix stream socket to be able to stream connect to someone else. In that case it should be "create_socket_perms" instead of "create_stream_socket_perms"


this is part of "auth_use_nsswitch()" and thus duplicated


@hbun DWMR. This is spot on. Good.


for some reason the xml wasn't being returned without it.


nop. However the bug will not longer be visible.


Why did you put it in a namespace and why that name?


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.466](http://teamcity.monogame.net/viewLog.html?buildId=9457&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 538, ignored: 6 Build time: 00:23:27


adding some i18n example


Just switched to it (https://github.com/alexarchambault/argonaut-shapeless/commit/2da55adf1b4fcbc3da0f2fd7eeb9779840313712), thanks!


It is OK to always be 2 because after the AKE finishes we generate a new pair of DH keys.


@mickeyren Let's make sure that new methods are added with tests.


Either you explain the usage of a view-view or you just leave it at a
view. I would use the word view instead tree-view-view. The same with
tree-view-node-view: node in tree-view

Am 06.08.2015 um 15:43 schrieb Danny Arnold:
>     tree-view-view/?/
>
> Since the differentiation between the tree view logic and the tree view
> representation in the DOM wasn't clear enough from the diagrams I had to
> find another way to explain how applications in a browser work. It seems
> that the confusion is rooted in the expectation that a browser only
> receives markup from the server and renders it. Which isn't the case
> anymore for a lot of websites.
>
> If you have an idea how to make that clearer, I'm happy to incorporate
> it. I could go more into detail at the risk of loosing track of the big
> picture (whatever that is, no idea what all of this is about anyway) and
> probably not finishing it at all.
>
> So I'm open for suggestions.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/despairblue/furry-octo-dangerzone/commit/36d2576511d93faaf4b94ccd360650156beaef1c#commitcomment-12565199>.
>

--
Matthias Lenk, M.Sc.
Technische Universität Bergakademie Freiberg
Universitätsrechenzentrum
Bernhard-von-Cotta Straße 1
09599 Freiberg

phone: +49 (0)3731 / 39-4789
mail: lenk@tu-freiberg.de


Give me a link please


@hbun You could simplify like this - `return @"1 cheese";`


that's nice!


Leave me a link please


@pavanka Just applied your suggestions.


@hbun DWMR. Well done here. Press on!


You missed a rename here it seems


I can't get this to work.

I created the file "assets/tinymce-config.json" with the content:

``{"test":"THIS IS A TEST"}``

and changed the ``tinymcerte.external_config`` setting to the file path.

My custom config option does not appear in the config.


nice to see u


I noticed that I'm not checking if ```$remap->{range_request_handling}``` is defined, although it doesn't check for the Edges as well.


You mean pixel offset within a tile?


this needs to say something, python thinks you wanted `def check_fermat_ints():` indented under the `else:`, hence the error.

for now try (and see if you can figure it out):
```python
    else:
        pass
```


I'm going to be adding all this to the readme. The idea is that you can put your own keys in .env and then run the specs once and the vcr cassettes will be created for you. VCR periodically allows requests anyway to make sure the cassettes are up to date, so you need the keys loaded.


I have written sample code for storing the data into cassandra while reading and processing it through Storm.

There are 5 files

1. Connection.java   (making the connection with cassandra)
2. CassandraConnector.java  (inserting and creating the column family)
3. CassandraBolt.java (Bolt for processing the stream receiving from spout)
4. CassandraSpout.java (reading the file and sending it to bolt for processing)
5. CassandraTopology.java (driver file to run the code.)

USAGE: storm jar storm-cass.jar com.strom.cassandra.CassandraTopology  [filename]


Checkmarx scan results 06 Aug 2015 17:19 UTC +03:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
1
1
2
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

e0475c32af6628c6c9f8c995d077c5689e741068	@cxehud

6b170c26e8750d31fdc922d9fed7028cce70ef32	@cxehud

03b2408a92d606bb3ab4a6fb52c0c955f8a3a9f2	@cxehud


@hbun DWMR. :+1:


@mmun hopefully the test for this will be in Ember core by then https://github.com/emberjs/ember.js/pull/11218/files#diff-bfaf63740e7a9670d5dbd48c684341e2R57


This commit is being reviewed as a part of review issue #86.


One thing that wasn't addressed here is dot output, which I find helpful for understanding what is going on in the graphs. I'll use http://www.graphviz.org/doc/info/attrs.html#k:portPos and other attributes to try to communicate the edge orientation.


The patch looks OK, but I'd make a user handle for an I2C request (that is cast internally to the worker thread or timeout), not leave it as void *.


It is about beeing unambiguity, but I was confused. It was not clear no
me, why you use the word. You could also explain why it should be called
a Tree-view-node-view, but explicitely say that you will refer to it as
a nodes in tree-view.
Or, however make it clear why it is a Tree-view-node-view.

Am 06.08.2015 um 15:52 schrieb Danny Arnold:
>     Tree-view-node-views -> /Nodes in tree-view/
>
> I was aiming for a unified, clear and unambiguous language, since all
> approaches before seemed to be too casual and confusing to the reader.
>
> So, yeah, this suggestion confuses me a bit. I thought it was about
> being unambiguity, not about a pleasant reading experience?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/despairblue/furry-octo-dangerzone/commit/36d2576511d93faaf4b94ccd360650156beaef1c#commitcomment-12565463>.
>

--
Matthias Lenk, M.Sc.
Technische Universität Bergakademie Freiberg
Universitätsrechenzentrum
Bernhard-von-Cotta Straße 1
09599 Freiberg

phone: +49 (0)3731 / 39-4789
mail: lenk@tu-freiberg.de


saw approval from macintux
at https://github.com/basho/riak_kv/commit/248314c4ba16bcd081533a6170fdbb4a5b7dee53


merging basho/riak_kv/fix/az/assorted-dialyzer-issues = 248314c4 into borshop-integration-1164-fix/az/assorted-dialyzer-issues


basho/riak_kv/fix/az/assorted-dialyzer-issues = 248314c4 merged ok, testing candidate = 017e003c


@hbun DWMR. Concise and correct. Good.


I thought about that later :). Thanks for taking a look at it late at
night, really appreciate it.
On Aug 6, 2015 1:11 AM, "danielsgriffin" <notifications@github.com> wrote:

> Interesting solution. It works!
> I was expecting:
>
> def do_four(f,s):
>     do_twice(f,s)
>     do_twice(f,s)
>
> But, good job.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Neeragrover/HW02/commit/7c243488372bceffd4618089cc26ecf0f659be47#commitcomment-12559187>
> .
>


Yo me equivoqué en el nombre, puse NodIO en lugar de NodEO en la figura.
En algún lado vi nodio y se me grabó así.
Busqué y es el nombre de los logs:
new (winston.transports.File)({ filename: log_dir+'/nodio-'+date_str+ "-" + sequence+'.log', level: 'info' })
Sorry lo cambio.


check

Am 06.08.2015 um 16:02 schrieb Danny Arnold:
>     reduce -> /divided into/
>
> Why? It was 4 steps before, and now it's only 2 steps.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/despairblue/furry-octo-dangerzone/commit/36d2576511d93faaf4b94ccd360650156beaef1c#commitcomment-12565730>.
>

--
Matthias Lenk, M.Sc.
Technische Universität Bergakademie Freiberg
Universitätsrechenzentrum
Bernhard-von-Cotta Straße 1
09599 Freiberg

phone: +49 (0)3731 / 39-4789
mail: lenk@tu-freiberg.de


Good.


@hbun DWMR. Success. Lessons learned - value gained - you have pushed back the curtain of the unknown. Good. Continue.


Pulled latest last night and hooked to ashscan and Chatsharp crashed like within 10 minutes. I had to switch back to older Nuget package by @khawajaumarfarooq . You might wanna test it. I have kept the log in my box at home. I will open an issue and share that as soon as I can.


At some point, therre might be a trade off between checking for changes
or just do the whole thing without checks.

Am 06.08.2015 um 16:04 schrieb Danny Arnold:
>     /, since .../
>
> :confused: I am supposed to explain why doing more in a frame is less
> efficient than doing less?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/despairblue/furry-octo-dangerzone/commit/36d2576511d93faaf4b94ccd360650156beaef1c#commitcomment-12565789>.
>

--
Matthias Lenk, M.Sc.
Technische Universität Bergakademie Freiberg
Universitätsrechenzentrum
Bernhard-von-Cotta Straße 1
09599 Freiberg

phone: +49 (0)3731 / 39-4789
mail: lenk@tu-freiberg.de


Should just use Array.isArray here...


LGTM


Also had trouble with the other runner - among other things it'd run the tests using stale outputs (from target-path instead of the fileset). Hence building my own :stuck_out_tongue:

Submitted #16


sfsfsfsd


sfs


Was there a regression test for the ordering?

When I've had this in Rails, I found it useful to create fixtures that had the ordering info, but were created out of order.


This commit is being reviewed as a part of review issue #84.


This commit is being reviewed as a part of review issue #87.


Checkmarx scan results 2015-08-06T14:18:23Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
72
61
66
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

d5a07cb0b1744f15f637a4c7ffe8f2ac25f77bdd	@dudug


Checkmarx scan results 2015-08-06T14:18:20Z

High
Medium
Low
Info

New
72
61
66
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

d5a07cb0b1744f15f637a4c7ffe8f2ac25f77bdd	@dudug


Tests good to me.


That didn't work. Fixing it...


:trollface: @cabol


develop (mbadams5) - x86_64-MacOS-10.9-clang: OK (1269 of 1269 tests passed)

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


check

Am 06.08.2015 um 15:49 schrieb Danny Arnold:
>     up to date -> /up-to-date/
>
> |up to date| is also possible and I have plenty dasherized words/word
> groups, so opted for not choosing dashes here.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/despairblue/furry-octo-dangerzone/commit/36d2576511d93faaf4b94ccd360650156beaef1c#commitcomment-12565361>.
>

--
Matthias Lenk, M.Sc.
Technische Universität Bergakademie Freiberg
Universitätsrechenzentrum
Bernhard-von-Cotta Straße 1
09599 Freiberg

phone: +49 (0)3731 / 39-4789
mail: lenk@tu-freiberg.de


Neither is "accountto" :stuck_out_tongue:
I think this could be worded better. It's not very clear as it's written. Another FAF account on that computer is linked to steam, so this other account must be as well? But to a different steam account?


@Snkz
`TEST_USER` in `local_config.py` is finally a `dict` rather than a `str`.


How are we avoiding continuous skips in case of a busy connection? won't skip count get stuck at a high number meaning this cache won't grow / shrink at all?


Can you give the stack trace? I'm running the lastest ChatSharp in production on 4 networks without any trouble.


The scan failed.
Included commits

Commit ID	Owner

30d36644f235e87d3dbb12e889665dee3417ce0b	@amitchen22

327536ba65aab5675dd2d593ae812b42000dca42	@amitchen22


The scan failed.
Included commits

Commit ID	Owner

184f11a6d2d8593a75164e93cb2879f9dcbed868	@amitchen22

30d36644f235e87d3dbb12e889665dee3417ce0b	@amitchen22


This commit is being reviewed as a part of review issue #50.


Should be more explicit, PROTOBUF IMO..


Same here..


It was a quick solution for equality checking in tests.  If you want to offer a better solution that's fine I just don't imagine any practical code doing equality checks on this.


This is a temporary change - will look into the code and probably move towards changing this to basic_consume with start_consuming called after that, but need to analyze if it will be appropriate for all the hsn2rmq use cases.


then: custom html elements?

Am 06.08.2015 um 15:33 schrieb Danny Arnold:
>     custom /html control/ elements?
>
> As per spec the word |control| seems to refer only to html forms, and
> checkboxes and inputs are controls in that sense, but they were used
> merely as examples, I'm actually referring to any arbitrary html
> element, div, li, a, video, audio, whatever.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/despairblue/furry-octo-dangerzone/commit/36d2576511d93faaf4b94ccd360650156beaef1c#commitcomment-12564985>.
>


Checkmarx scan results 2015-08-06T14:21:38Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
26
17
30
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

2ee51e5a66b19396841f123fe398cfbf3f34ae3a	@amitchen22

c2b97a1229cba22fac5b751dcc85f7be278f0955	@amitchen22


Using `return isNaN(item);` likely works better


Good job. Although, s11 is an odd variable name, you can keep it as s1.
Also, you can do this in one line:
```python
s3=int(input("enter third side:"))
```


Sobre o nome: respond-above, above what?


Good.


Thanks. I'm going to rebase this to fix a few typos in the commit message.


2015, but not copyright Apcera, this should be you, etc.. Also, file should be names protobuf_enc.go.


I'm sure I don't know what you're talking about:
https://github.com/FAForever/server/commit/2c9da4933a9d4d9c2a14f433e3fc5425f1b2ded7


Use const?


They changed the structure of how the sketch compile in 1.6.6 to fix a few issues. Thats all up to the arduino team, not me.


Damn, so many copy-paste errors here that I failed to see.  Thanks! Fixed


The purpose of the change is for the PostgreSQL integration. Apparently for certain types it needs to know what the data type is. The specific use-case was for null, but I made this larger change as I think it will help keep things a little clearer for the database going forward.


Checkmarx scan results 2015-08-06T14:23:23Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
26
17
30
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

c2b97a1229cba22fac5b751dcc85f7be278f0955	@amitchen22


Ugly :(


In general looks good, not sure if we should introduce the external dependency though.


No need to get a float array and convert it since it goes into a Color directly


I agree. Can we delete this function then? Otherwise we have to add it everywhere else too.


Not wrong, but actually, values of the nodes are changed.

Am 06.08.2015 um 15:35 schrieb Danny Arnold:
>     tree nodes /values/
>
> What? You mean properties/attributes? If yes, here I'm referring to all
> changes that could happen, not only property mutations, but also adding
> and removing child nodes.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/despairblue/furry-octo-dangerzone/commit/36d2576511d93faaf4b94ccd360650156beaef1c#commitcomment-12565020>.
>

--
Matthias Lenk, M.Sc.
Technische Universität Bergakademie Freiberg
Universitätsrechenzentrum
Bernhard-von-Cotta Straße 1
09599 Freiberg

phone: +49 (0)3731 / 39-4789
mail: lenk@tu-freiberg.de


Rebased to e7be304


Thanks. Which bug does this fix?


@michaelcahill, you didn't say how you found this one, but it looks like an easy mistake to make in the future, and a difficult one to debug.

What would you think of doing something like this in gcc.h:
```
static inline uint64_t
__wt_atomic_cas8(uint64_t *v, uint64_t old, uint64_t new)
{
        return (__sync_bool_compare_and_swap(v, old, new));
}
static inline void *
__wt_atomic_cas_ptr(void **v, void *old, void *new)
{
        return (__sync_bool_compare_and_swap(v, old, new));
}
static inline uint64_t
__wt_atomic_store8(uint64_t *v, uint64_t val)
{
        return (__sync_lock_test_and_set(v, val));
}
```
and so on? Then we know the argument is only evaluated once.

A positive is we get type checking (the WT_STATIC_ASSERT in the underlying macros goes away).

A negative is we have to specify the address of the object to the inline function, and we have separate functions for pointers and 8B objects.

Thoughts?


The scan was canceled by the user   on 8/6/2015 5:24:42 PM
Included commits

Commit ID	Owner

3556bde7d0ed75634921c292229ef83c19c67996	@amitchen22

6f0f92b97d3ddd96a7be9ca7182ff05461fd32fe	@amitchen22


@pekohl DWMR (done with my review). Good work here. Press on!


stripTreeTable is not a suitable name. stripe != strip.


Right now there are 3 sub-namespaces with the same constants but different types (Color, char*, avd float[4])


Yeah, but you'll end up with one failing test you'll have to fix.


`no enabled` should be `not enabled`


This build http://hydra.nixos.org/build/24297107/nixlog/1.


Good.


nice :+1:


responde above medium :wink:

Medium is 720px

'small': 480px,
'medium': 720px,
'large': 960px,
'x-large': 1280px


Это почему так ?


yeah :(


db
----
latest_movie : timestamp
latest_serie : timestamp

code
-----
$latest_serie_fetch = $this->cache_m->getLatestSerieFetch();
$backlog = array();
$steps = 10;
if ($latest_serie_fetch){
	$n = 0;
	$sr_backlog_piece = $this->serieprovider->fetchHistory($n + $steps); // with state 'downloaded', limit = $n + $steps

	if (sizeof($sr_backlog_piece) > 0){
		$sr_timestamp = $sr_backlog_piece[0]->date;
	} else {
		$sr_timestamp = $latest_serie_fetch;
	}

	while ($sr_timestamp > $latest_serie_fetch){
		foreach ($sr_backlog_piece as $piece){
			if ($piece>date > $latest_serie_fetch){
				array_push($backlog, piece);
			} else {
				error_log("Updated latest serie fetch to " . $piece->date);
				$latest_serie_fetch = $sr_backlog_piece->date;
			}
			$sr_timestamp = $piece->date;
			$n = $n +1;
		}
	}
} else {
	$backlog = $this->serieprovider->fetchHistory(); // with state 'downloaded', limit = false
}

foreach ($backlog as $log){
	$this->serieprovider->addEpisode($log->data);
}

..

addEpisode:
	- load appropriate info from thetvdb
	- insert data in database table episodes (and series if needed?)
	- run cache_m on episode and/or serie


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/2262


I think you mean composer install instead of update ;-)
It works now, thx


Checkmarx scan results 2015-08-06T14:26:44Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
26
17
30
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

327536ba65aab5675dd2d593ae812b42000dca42	@amitchen22

2ee51e5a66b19396841f123fe398cfbf3f34ae3a	@amitchen22


Good. But try to keep your function calls in `main()`


Ah, neat :)
On Aug 6, 2015 7:32 AM, "danielsgriffin" <notifications@github.com> wrote:

> Good job. Although, s11 is an odd variable name, you can keep it as s1.
> Also, you can do this in one line:
>
> s3=int(input("enter third side:"))
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Neeragrover/HW02/commit/7c243488372bceffd4618089cc26ecf0f659be47#commitcomment-12566491>
> .
>


I don't think this is the right fix, let me check.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.894.902f1b56bb30c0b78065d8c0d08b7229ef2750da](http://build.palaso.org/viewLog.html?buildId=60694&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 397, ignored: 4 Build time: 00:04:41


indeed


You don't like degenerate triangles? They don't fall into the category of triangle? :(


Good, but above.


Я думал, что в сете могут быть только элементы enum. А это что ?


At least it compiled and started for me.


Shouldn't we be using `0` here? That's what we use when we create new customers. Plus `-1` has the possibility of being 'absint` to `1` if an extension dev does something with it.


Yes, ``composer install`` is the good choice !


You're right in fact the constructor ComponentImpl() should take a type and a FileAttributes as parameter, and then I'll be able to use primitive boolean


have you published this update to the dev mxds?


@pSub the fact that it compiles doesn't mean the dependency is right


Yeah, go ahead and explain to me how it's ethical to (theoretically) work
with ISIS. Pleeeeeeease.

On Thu, Aug 6, 2015 at 9:07 AM, Xavier Mendez <notifications@github.com>
wrote:

> "Ethics".
>
> TIL that innocent ASCII characters have ethics attached. Wow.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nixxquality/WebMConverter/commit/c1ac0baac06fa7175677a4a1bf65860a84708d67#commitcomment-12565875>
> .
>

--
Kurtis Rainbolt-Greene, Hacker
Software Developer
1631 8th St.
New Orleans, LA
70115


this will lead to the function being called only n-1 times (probably my fault w/ the random `print_hello()` function call in `main()`)


On the Instagram.php page-  how come you removed pulling all new posts and saving them to the Database?  That is one of the core features of this as when you are subscribed Instagram doesn't send the full data you have to go out and get it.


Good.


@zml2008

@simon816 already had a PR for this. Check the PRs next time. Either revert this or work to add something that may be missing to @simon816 's and merge THAT one.


indeed, I need to recopy it.


:scream:


Yes, the WidgetBuilder returns a JSON with the first, and only, key is WidgetBuilder. The build function is on that.


indeed


Closer matches spec from https://url.spec.whatwg.org/#concept-urlsearchparams-new --- but still missing copy constructor support


@pSub: seems like another example of http://lists.science.uu.nl/pipermail/nix-dev/2015-July/017751.html. Bit me too.


ahem


Tive alguns problemas com isso no começo e acabei gerando dados duplicados. Achei melhor, só pra não acontecer mesmo, fazer essa verificação.


transports_directory appears twice


oups, indeed


yuck!


Ugh, tl;dr on this separation for the string case. Why, in a tweet? Also, did you test all lcd grove samples again to be sure we're safe?


형도 머리아프고 나도 머리아프곸ㅋㅋㅋㅋㅋㅋㅋ


You are right, except i think it defaults to 1, but it says it is required in W3 documentation.  Good catch.


yep


This is a little scary. I'd be more comfortable if:

- `GROUP_EDGE_ROLES` had a comment above saying that the order matters and that new roles must be appended
- Anything that references these roles should have a Foreign Key constraint, if possible (e.g. AsyncNotification below)


we want this to be in the function itself


spraying


Added comment and handle exception at import location. That results in an extra early return though.


I have it all on log file I will be able to share it at 7 PM EST.


Alright, thanks.


Thanks: I *confirm* hereby that it fixes my reported #3893 bug.


@levicole DWMR. :+1:  Long press is a good solution.


Still not getting it. What do you mean with `value`?


good catch with the additional %step.
verified that this is correct with a range of N for step = 4 in the example below
```python
In[1]:  (4 - np.arange(5, 15) % 4) % 4
Out[1]: array([3, 2, 1, 0, 3, 2, 1, 0, 3, 2], dtype=int32)
```


glad someone's keeping me honest


Why shouldn't it be sent out?  It's not a security risk.

> (you can anyway see the URL, should you really want it, in the jenkins job parameters list)

But you can only see that once (*if*) the Jenkins job got triggered, and then it's an extra two mouse clicks.  And before the job gets triggered, you have no way of seeing any progress at all without hunting for the relevant pr-trigger job.  So it's way more usable this way.


Los archivos eran una ensalada de espacios y tabulaciones. Así que unifique todo


Finalmente português correto.


this is backwards


Since pomf is dead, here's a screenshot reupload: http://i.imgur.com/CR6idYo.png


woa you're alive and editing this! I hope you've been making a bunch of other cool games


okies!


@rosskarchner what was the purpose for converting this to a tuple? It's causing TypeErrors in django-sheerlike [here](https://github.com/cfpb/django-sheerlike/blob/master/sheerlike/__init__.py#L72) and [here](https://github.com/cfpb/django-sheerlike/blob/master/sheerlike/query.py#L278)


you can just do `else`, if the if above is not met then automatically you know it is the opposite.


Really like this because the API is the exact same! :+1:


good.


I didn't remove it, I just pulled it out into a method on the InstagramSubscription class (updatePosts) so that functionality could be reused.


"doesn't ~~work~~ apply"


That looks absolutely great, thank you very much.  I would just like to verify that in line 382, that is adding a hotkey for Ctrl+1, Ctrl+2, etc. for quickly opening the previous wallet files, yes?


fixed! thanks


Also the error on hydra is very clear: `/...-binutils-2.23.1/bin/ld: warning: liblcms2.so.2, needed by /...-colord-1.2.3/lib/libcolord.so, not found (try using -rpath or -rpath-link)`

colord is missing lcms as propagated build input as required by the .pc file. That means some of the deps of simple-scan dropped the lcms propagation, and nothing else propagates it.


Good.


Nomenclature aside, how can it be a triangle when it fails to have three
sides? It's a line! :{

I guess I don't like them, ahaha... Guilty.

On Thursday, August 6, 2015, danielsgriffin <notifications@github.com>
wrote:

> You don't like degenerate triangles? They don't fall into the category of
> triangle? :(
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dinabseiso/HW02/commit/31194a2f2dc16ac8828bd8c13aa9afd589e4559e#commitcomment-12566693>
> .
>


Removed debug and added bind to all hosts


Well, for my purposes it was more important to allow the config itself come from the command line...so in that respect, the command line wins.


I have been seeing a lot of these commits. Does this really need to be in MPWorks or can it be in a database? The long list of these commits make it extremely difficult to find any old commit in MPWorks.


Do you mean change the name from PROTO_ENCODER to PROTOBUF_ENCODER?


I tried to define it in MVC terms, where the scene-graph is the model, the tree-view is the controller and the rendered tree-view is the view. Since the problem seems to stem from using view twice, how about calling the tree-view just tree-controller and the tree-view-view just tree-view?

Other terms would be renamed to
tree-node-controller and tree-node-view.


it was erroneously committed--


@jedesah I think I've pinpointed to what is causing this, `Gen`:

```scala
  def liftField(c: Context)(field: Field[Any]): c.universe.Tree = {
    import c.universe._
    q"_root_.remotely.Field(${field.name}, ${field.typeString})"
  }
```
Not sure what is the best solution other than making the constructor accessible.


:+1: Thanks for satisfying my curiosity :wink:


@levicole case sensitivity?


nice


missing ;


Ok, I'll stop sending Pull Requests for now.


github 일단 하나 프로그램은 돌아가는데 좀 이상하게 돌아가는 에러들 내일 잡고 내일 서버랑 클라이언트 그냥 쉭쉭 나누면될거같은데
토욜날 전화나때려라

2015-08-06 23:44 GMT+09:00 HyunWoo <notifications@github.com>:

> 형도 머리아프고 나도 머리아프곸ㅋㅋㅋㅋㅋㅋㅋ
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/masinogns/C-language/commit/081aae72ba3a11952f637e8306618f9436abe9e0#commitcomment-12566828>
> .
>


Checkmarx scan results 06 Aug 2015 07:51 UTC -07:00

High
Medium
Low
Info

New
0
4
25
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

755745fcd2269350f9fecc9d2413712cf3c4b7cb	@dortal


@mfonville , I missed the re-order part of the changes...I'll add that to the "knowledge" base!  Have fun on holiday!


You should consider the performance here, storing the entire model will consume a lot of memory


Yeah the tree-view-node-views where missing from the terminology. I somehow thought the system was self explanatory after already composing the other terms in that way. Like if it ends on view then I'm talking about the parsed html/DOM.


@levicole DWMR. Not sure how you sorted this soltuion out, but I want to know. You've implemented a simple and concise soltuion. You should be proud of it. I'm proud of you. :100:


develop (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


Tossing future away, concerned for demo code that users will copy this in their code


Good.


╮(╯_╰)╭


You can also do `for (role_type, user_name), user in self.my_members().iteritems()`

...assuming that's what those fields mean.


@vitaly-t I needed to use a custom pool https://github.com/curzonj/spacebox-nodejs-common-native/blob/master/src/pg-wrapper.js#L15


Partially address #9


Comment? What's this?


Whenever you change an API, please add the description of the new parameter to the document comment above the API definition.


yes, thanks


Thanks for the mention! I just refactored out the FBP Protocol support but it will reappear in my GitHub repository by next week.


Redundant else

```
factory, err := ...
if err != nil {
    return nil, err
}
context.ClientFactory = factory
```


I had missing "... to compute **measure(s)** when..." when I suggested this description


Maybe this should be called `new_role` for clarity below?


nope, it needs an explicit conversion of type, if we also want to apply with plain text messages.


this only calls print_hello nine times


This returns false.


good.


shouldn't this method be called getMeasureComputer**s**?


Okay, this was the solution. Delete the repository from my harddrive (it was a carbon copy anyway) and cloned it again, tadaa... problem solved.
You'd really think there'd be an easier solution to this :(


Could you write an example for the docstring (such as the one in cn_test) ?


You don't like degenerate triangles?


fixes #4


Checkmarx scan results 06 Aug 2015 07:57 UTC -07:00

High
Medium
Low
Info

New
6
12
116
0

Recurrent
0
4
25
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

e3c65557712c475ed74a387f75d080c157c72718	@dortal


warning! `filter` does not return a collection, but a view! you need to call `toList`


Good.


heyyyya I'm actually still pretty upset that I haven't finished this since
I said I would lol. Basically the project wasn't designed to be finished so
I really hit a wall. I started again by actually making a design document
so I can like, maybe finish it and make it a real game.

I've made some silly little nondigital games for a class last semester that
was a bunch of fun.

what have you been up to?
On Aug 6, 2015 9:46 AM, "Emily Pan" <notifications@github.com> wrote:

> woa you're alive and editing this! I hope you've been making a bunch of
> other cool games
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/interactivecomputingkapp/Kisetsu/commit/c52698d98f7abbc13606d0ebc1d2f879415f48ea#commitcomment-12566935>
> .
>


in visual-line mode (V), the navigation doesn't seem to work when soft line wraps are enabled. If I select a line that is wrapped, I cannot move up.


@rsivasundaram Once the feature/CCLE-5298-recycle-bin-refinements branch is merged, this will probably need to include a step to click "Yes" on the confirmation dialog.


you can certainly write these lines with `Preconditions.checkArgument`, can't you?


The scan failed.
Included commits

Commit ID	Owner

d5a07cb0b1744f15f637a4c7ffe8f2ac25f77bdd	@dudug


you could inline these tests


this complicates a bit unnecessarily


LGTM


try running this script


develop (mbadams5) - i386-Windows-7-VisualStudio-12: OK (1272 of 1272 tests passed)

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


my comment (nadav)


I'd rather use "~1.9.0" in order to lock the version in. Actually applies to all dependencies.


check to ensure n > 2


Checkmarx scan results 2015-08-06T14:51:53Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
72
61
66
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

d5a07cb0b1744f15f637a4c7ffe8f2ac25f77bdd	@dudug


good.


Un peu violent le hardcode :p


fixes #5


you should check the argument before setting the property
in addition, you're not checking for null values yet


good, though "order" is an interesting variable name
like an how many orders of fries?


Ah, gotcha. I wasn't sure :)


Good.


Ah! Didn't think of that. Will fix it!


I've added note about required values in web page. I'm not sure whether adding the (*) beside field names is the most aesthetic way of flagging them (especially in all languages) or whether there is some more suitable symbol to flag them with?

Out of curiosity, when I add an extra text item such as here should I ask for translations to be made or do the translators have some kind of automated watch when new items are added to the GeoServerApplication.properties files?


You don't like degenerate triangles?


in the build method, you can check only that the property is not null (if you apply my comment above), it's enough to tell the set method wasn't called
conclusion: no need to do the same verifications in the set methods and the build method


Not entirely...will we go over it in class?

On Thu, Aug 6, 2015 at 12:53 AM, danielsgriffin <notifications@github.com>
wrote:

> This works if run in the interpreter. But the goal is to be able to call
> the entire script and run it, calling right_justify from within the main()
> function.
>
> Make sense?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sgvolkov/HW02/commit/3e311c9921896b5316b174e4ae453b52f675b428#commitcomment-12558867>
> .
>


Checkmarx scan results 2015-08-06T14:54:12Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
72
61
66
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

d5a07cb0b1744f15f637a4c7ffe8f2ac25f77bdd	@dudug


or `range(n)`
or `range(0,n)`


I guess you didn't meant to commit the `debug` part, right?


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 271](https://teamcity.labkey.org/viewLog.html?buildId=264912&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:10:18
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

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

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

```


this isn't supposed to be here


should have a MeasureComputer argument and be static, imho


why aren't these still a, b, c, n?
they are unique within this scope and won't change what is in `check_fermat`


Hallo


This value is never used.


hfdsahfdg


xdbfdashs


Is this fixing transparency for the menu (on mobile/smal displays)?


Looks good I will merge in. Thanks for the update!


Good.


good


I'd put construction of the error message in a method, for clarity


Should be bash-it and not Bash it.


We should fail fast here and break out of the loop as soon as we've detected a failed task.


I wanted a `do_four` function... :(


develop (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


Hm,

after installing nvidia-331-updates-uvm it seems that I can at least manually "rmmod nvidia" and after that it survives hibernation (suspend to RAM that is)


True that. Though scenario's can explore these cases which is not always a happy, bad or sad path.


Why did you edit this class? can you undo the changes here?


Because I am using some functions from `hhvm/asio-utilities`, specifically `HH\Asio\vm`

See: https://github.com/hhvm/user-documentation#composer-dependencies

I can add a comment to the code.


Thx for fixing the bug: "Fixed a bug where using the wrong encryption key first caused the right key to be seen as invalid."


minor nit, but I'd swap these conditions and check if we failed rather than if we haven't failed.


Habíamos quedado que usábamos espacios en vez de tabulaciones. Dos espacios de indentación por nivel.


Yes, that is why we are putting examples in their own directories so they can at the very least be type-checked. That is a good point though. I can return some `string` from this function so it will type check.


why do you need the "new Buffer"? Update allows a second param to indicate the string encoding...


:)

Let's say


I'm using the Sauce gem to successfully test on mobile browsers using appium 1.4.7 on Sauce Labs. This is the trick:

`["iOS", "Safari", "8.2", {:deviceName=>"iPad Simulator"}]`

Or use my [sauce platforms gem](https://github.com/bootstraponline/sauce_platforms) + sauce_ruby:

```ruby
config[:browsers] = [
  Platform.ios.ipad.v8_2
]
```


I changed it to continue. I was a bit lazy about naming variables, but I guess if other people need to read the code it should be clearer. Also I tried changing continue == "n" || continue == "no" to continue == ("n" ||  "no"), but then it stopped working. Last night in the IRB it looked like that would work though.


use a positive test for clarity


I know :)


Unfortunately display: table breaks vertical alignment, I'll take a look if there is another solution.


develop (mbadams5) - Win64-Windows-7-VisualStudio-12: OK (1272 of 1272 tests passed)

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


Part of the logo displayed, and using adobe dreamweaver to find proper css file (PF.Base/theme/default/flavor/default.css) I assumed it would be simple to adjust size to accommodate my logo, but it just wouldn't change (cleared cache locally and in cache manager) also tried on different browsers. Are there more settings or files I need to address?
Thank you
Marty Cota
![screenshot-whookka com 2015-08-06 11-05-36](https://cloud.githubusercontent.com/assets/13561300/9115239/08e610ca-3c2c-11e5-8b81-4b7fcdf6ca90.png)


you don't need to test if `measureComputer.getInputMetrics()` is null, method `setInputMetrics` should do it for you
(in addition, `measureComputer.getInputMetrics()` is not even supposed to return null)


I agree! I ended up not even using these functions as there was way too much DFU output to the terminal.


I didn't worry about it since the color change was so subtle. I thought
that it could just wait until we published next time.

On Thu, Aug 6, 2015 at 8:40 AM, steveoh <notifications@github.com> wrote:

> have you published this update to the dev mxds?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/agrc/wri-web/commit/bff3542f819a366c8f67b3ed5272ab918c0920c0#commitcomment-12566727>
> .
>

--
*Scott Davis*
Utah AGRC <http://gis.utah.gov/developer/>
stdavis@utah.gov
@ScottAGRC <https://twitter.com/ScottAGRC>


Paper is hosted both on [Overleaf](https://www.overleaf.com/read/gqgnjhxdvbqg) and here
Following [this tutorial](https://ineed.coffee/3454/how-to-synchronize-an-overleaf-latex-paper-with-a-github-repository/)


오 파이썬 단에서 이렇게 제외시킬 수 있네요! 이걸 전 프론트 단에서 제외시켰는데 이러는게 더 좋을듯!


`remove` возвращает того, кого поремувил, и `null`, если никого не было


Add password for input type


same comment as for prod code, could use `checkArgument`


That's fine. Turning it into a generator was a bit of an experiment.


That was needed when I had a hash. But that's indeed not anymore needed, so it's better like that :-)


`Route` will match all verbs (GET, POST, etc.).


Actually, it should probably return a `tuple()` given that it's meant to be an editable collection of cycles.


test-s3-fixes (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1272 of 1272 tests passed)

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


Создание пустого `EnumSet` выбранного класса енума.Т.е. ни один из флагов не выставлен.


FYI I branched from here and merged `master` to 3e5f77740461ce088035bdf38cec0f8933b51060.


Assign the env keys to a protected variable at the beginning of the class for scalability purposes.


I does help to give good names to variables, it can even help you to make sure you know what it stands for.

I just tried in the IRB and got:

2.0.0-p645 :001 > "no" == ("n"||"no")
 => false
2.0.0-p645 :002 > "n" == ("n"||"no")
 => true


Yay!


Checkmarx scan results 06 Aug 2015 08:18 UTC -07:00

High
Medium
Low
Info

New
6
16
141
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

2afdc40b2cc24d7174250d3d1fd5dea957e284a2	@dortal


What's "12275"?


It doesn't write to the mini-statement file. So for transfers, it only writes 'Transferred X from account to account'. Otherwise it wrote the transfer, and the deposit and withdrawal that made up the transfer. I could probably avoid writing new functions if I put the file writing code in main.rb under each option, rather than in account.rb & bank.rb.


You are not working with ISIS by merging some patches into your project, even if those patches are written by ISIS, because the code is supposedly under a FOSS license and has no owner. Whoever wrote the patches, it has the same effect on your project.


develop (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


yes..


"Ethics"

On Thu, Aug 6, 2015 at 10:18 AM, Xavier Mendez <notifications@github.com>
wrote:

> You are not working with ISIS by merging some patches into your project,
> even if those patches are written by ISIS, because the code is supposedly
> under a FOSS license and has no owner. Whoever wrote the patches, it has
> the same effect on your project.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nixxquality/WebMConverter/commit/c1ac0baac06fa7175677a4a1bf65860a84708d67#commitcomment-12567947>
> .
>

--
Kurtis Rainbolt-Greene, Hacker
Software Developer
1631 8th St.
New Orleans, LA
70115


I still have a ways to go before the bugs are squashed and the UI looks good, but it's getting there! I hope to have a beta version by the end of August, but it depends on how busy I'll be with other stuff.


I've changed it to display_current_balance_msg, though I'm not sure that's much clearer. It seems that I can't put the "#{current_account.balance}" part in lib.rb, so I kept all the variables in main.rb.


i tested the commit fix in oc 6.0.2 with postgresql and lighttpd using php 5.3.4 and works like a charm, also in oc 7.0.4 with postgresql and nginx using php 5.3.10 and also works perfectly


This looks like a typo. I get "error: 'REIDS_NOTICE' undeclared" from this line when running make.


This commit lead a no method error:
```
NoMethodError - undefined method `exposures' for API::Auth::Entities::MyEntity:Class
```


Oops. Thanks.


I put them both on the same line to help me visualise how it outputs, since they're printed on the same line. Does it look better in the code to keep everything on separate lines?


day03，写手机防盗模块。


@msund - we can't just delete URLs like this. please add a `redirect_from` yaml variable to whichever page you want /line-and-scatter to redirect to. Here is an example of a `redirect_from` variable in action: https://github.com/plotly/documentation/blob/source/_posts/r/2015-07-30-shiny.md


> Yeah, go ahead and explain to me how it's ethical to (theoretically) work with ISIS. Pleeeeeeease.

@krainboltgreene okay, but only this time and I know you're such a troll.

I mentioned several groups like mass murders, etc. but you're only interested in ISIS? Nazis killed more people and I would accept their commits too but all you care is ISIS this, ISIS that.

Okay so do you know that doctors/physicians are obligated to abide by [Declaration of Geneva](https://en.wikipedia.org/wiki/Declaration_of_Geneva) (like Hippocratic Oath) and there's a point
* I will not permit considerations of age, disease or disability, creed, ethnic origin, gender, nationality, **political affiliation**, race, sexual orientation, social standing or any other factor to intervene between my duty and my patient;

So they are obligated to help everyone, even ISIS. If a member of ISIS is about to die in their hospital they'll do whatever they can to save him. And this in fact is ethical, while not saving would be unethical.

You're saying I should discriminate people (eg. like ISIS) but I won't discriminate anyone, same as a doctor wouldn't discriminate anyone.
Also it's not working with ISIS, they would be working for me by contributing to my project and adding useful features who would be used and appreciated by users who would find them very useful.


@krainboltgreene Your trolling strategy needs some refinement. Try actually making arguments from time to time instead of repeating ad-hominem, vague statements and insults.


`&:inline?`


@adeguet1 Sleeping that long gives reproducible errors (failure to reset) on my workstation. I didn't do extensive testing, but 0.1-0.3s seem to work; not sure why any sleep is required, but it is.


nice ;)


jjj


It's not checking for changes. That's actually something angular does for it's two-way data-binding, they use dirty checking for this.
The approach I explained is actually event driven.

Oh, I guess you talk about the diff algorithm react uses? If so, then that hasn't been mentioned yet.

But yeah, let's say that you have to completely different trees you want to render to the DOM and every time you render you render the tree that is not already in the DOM, then of course the diffing might be useless.
Although it's hard to say whether applying patches to make one tree into another one or removing one tree from the DOM and replacing it with another one is more efficient. That is very implementation depended. Could be that one approach is faster in one browser and in a another browser the other is faster.
After all it sounds like premature optimisation.

Oh, or you're talking about a huge tree where every thing must be diffed just to find one small change and whether the diffing needs more computational power than just replacing the whole tree. Maybe, maybe not. No one knows. These things actually can happen, if you try to render a hug amount of data on a mobile device, but then why would you put more information on a page then a human can comprehend and why would you rerender the whole site every time and not just the part that changed?

If you had immutable data structures then the diffing wouldn't need much computation, since then it's just comparing memory addresses. Sadly this does not work in this case since the model (the scene graph) is in the DOM, and the DOM is and always will be mutable.

```
var node = document.querySelector('material')

var node2 = node.setAttribute('color', 'red') \\ actually mutates the node and returns undefined instead of returning a new node and leaving the old one alone
```

Coming back to breaking the site into smaller components. Assume every node in the tree observes its corresponding node in the scene graph. Should it receive a mutation event it marks itself as dirty, just that, nothing more. The diff algorithm then skips all non dirty nodes. Bam, best of both worlds.

Sorry I may have misused this comment as a scratch pad.

But back to what the problem was. You were mixing pieces of the react approach with the ones from the event driven approach, right? If not, what is it you don't understand?


Ohh crap!! I made a mistake while copying it here.. Thanks!!


Rather than having this here, we should extend the eslint config [like we did with component-omniture](https://github.com/the-economist-editorial/component-omniture/blob/master/package.json#L21-L26)


good catch


Our naming convention for driver tests is to make the method names literate separated by underscores.  This is not really a java convention just something we do.  So this should be 'should_prepare_once_when_configured'


How about calling it `to_nd_index` as a slightly terser version? "Multidimensional" is just such a long word.


What is the error?


:+1: so cool


Yes...I realized this while making the pattern :)


利用css3绘制三角形，似乎是只要绘制相关的边就可以了。border-left: 16px solid #fff;border-top: 12px solid rgba(255,255,255,0);border-bottom: 12px solid rgba(255,255,255,0);加入rgba(a,b,c,0)最后设置为0就是透明的。就可以了。


Yea it should be n==0. Thanks!!


You mean write "print '+' + (printpattern('-','+')*2)" for the 2by2 square?


There is also a logoSelector directive https://github.com/geonetwork/core-geonetwork/blob/3.0.x/web-ui/src/main/resources/catalog/components/edit/logoselector/LogoSelectorDirective.js used for the editor (for gmx:FileName needed for organisation logo). Not sure if those 2 can be merged ?


I've noticed that a TLB that contains Vista+ APIs (for example), will prevent the EXE from running on XP; it appears that connections to all APIs used in the EXE (whether behind version checks or not) are made upon start-up which will prevent you using Vista+ only features via TLB if you want to continue to support XP. Perhaps I'm doing something wrong, but basically even with `MsgBox` as the first line in `sub Main`, the EXE still crashed with missing function error. For my TLB, I'm moving version-specific APIs to a support module.


Lol. I didn't know I was supposed to include them. Technically they are just line segments. Haha


I neglected to share this with you before, but can you add javadoc for each of the test methods similar the format shown [here](https://github.com/datastax/java-driver/blob/2.1.7.1/driver-core/src/test/java/com/datastax/driver/core/TokenIntegrationTest.java#L100-L111)?   We document the possible test_categories [here](https://docs.google.com/spreadsheets/d/1FX2H9-filbzGPK5iXOZkDSntzpGItOXbPSH0brZOxew/edit#gid=1192185630) (prepared_statements:preparation seems good here)


Totally! Already noted the copy/paste mistake, but couldn't fix it until now.


This looks fine to me, do you have a test for it as well?


hi my names jeff


hello android


LGTM. Seems like the same thing doesn't happen in UserByName, because you don't hold a lock while blocking on the update.


LGTM. Confirmed on http://docs.openstack.org/developer/swift/middleware.html.


Yes, right, I'll change it.


klavye kapanması için


I'm sure, but I don't really know how. If I changed the end to: end while !(another_service?) || answer == "exit", it gets to else disp_menu_error first.

Even if I take out the menu error code, end while !(another_service?) || (answer != "exit"), doesn't work either.


"Ethics"

On Thu, Aug 6, 2015 at 10:23 AM, Xavier Mendez <notifications@github.com>
wrote:

> @krainboltgreene <https://github.com/krainboltgreene> Your trolling
> strategy needs some refinement. Try actually making arguments from time to
> time instead of repeating ad-hominem, vague statements and insults.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nixxquality/WebMConverter/commit/c1ac0baac06fa7175677a4a1bf65860a84708d67#commitcomment-12568042>
> .
>

--
Kurtis Rainbolt-Greene, Hacker
Software Developer
1631 8th St.
New Orleans, LA
70115


Not very cross-platform, but I suppose you're the only one running `prepublish`.


棒棒哒(≧▽≦)

Peter Cai <notifications@github.com>于2015年8月6日周四 16:06写道：

> Atom大法好！每次打开一个文件都自动帮我删掉无用的空格和tab = =
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PaperAirplane-Dev-Team/BlackLight/commit/0b47db81128994f4dcc17329ad4a36a401c88f73#commitcomment-12559103>
> .
>


#3 in fact .... that's why this plugin is important....


r+


Thanks @sjstebbins - Here's the correct link:
https://github.com/callemall/material-ui/blob/master/src/svg-icons/action/grade.jsx


Yes, if the `String` has not been consumed that means that the parsing failed. I didn't know about `Distribution.Text.simpleParse`, I'll take a look at it, thanks.


It is just test. I still can't prevent executing `prepublish` on local `npm install`. Maybe you have some ideas?


Thanks, fixed once I saw it red in the CI test, but was not able to push on Github because of server timeouts. Did it now.


We need to add a new empty line at the end of all .R files in the repo.


saw approval from jedmccaleb
at https://github.com/graydon/stellar-core/commit/a698f5034457f5deb9d7ceec130b94e7559eebb0


merging graydon/stellar-core/bug-670-short-history-write = a698f503 into auto


graydon/stellar-core/bug-670-short-history-write = a698f503 merged ok, testing candidate = 49345b34


Like line 805, 810, etc. use TimeUnit.SECONDS.toMillis(1) in place of 1000.  The compiler optimizes the call away, and it makes it trivial to see that ``idleTimeout`` is in milliseconds.


thanks


So this is an empty function now?


this assumes that the hardcoded "@mag-fh-cfg03:5014" would be accessible from any server that is using monit for its processes. can we make it optional ?
Thanks


Thank you very much for that one :-)


Fix #179


Noted


syncPlan.startTime = new Date('09/09/1999 13:00:00');


What do you think about the presence of a `discriminator` as a hint to the java generator that the base will be an interface and requires a concrete class?


love this function :heart:


what is this, remove plz


`Number`?


@kholk
i just discovered that i made a logical fault here. Rev. 2 incoming tomorrow.

Thanks for the input, will update the pr accordingly.


Couldn't these just be in vm_manager.cpp and made `static const`?


[Chris G. noted](https://forum.civicrm.org/index.php?topic=35125.msg151316#msg151316) that this patch excludes having custom templates and suggested *not* commenting out lines 92 to 99 so it can find the custom template directory.


Vec3 meanPoints = vX.colwise().sum() / nPoints;


Nice!


I don't believe this change is worthwhile.  For one thing, methods that are invoked after this assignment, like ``checkException()`` called from ``closeOpenStatements()`` will end up with logs that contain the ``ClosedConnection.CLOSED_CONNECTION`` rather than the real connection where an issue occurred.

What did you see as the benefit of this change?  We're making an additional assignment, for what benefit?


this seems to fix #215


No idea why the other changes in the code, Github gave me those automatically for some reason.


They're used in other files in later commits.


This needs the product_type as this is where the service name is abstracted, but that won't work for origin now that I think of it.


Yes, that was my thinking -- we'll rethink background matching as part of the work on coaddition coming up in the next cycle, so let's punt on that until then.


@jnjackins we are requiring `rsyslog` but we didn't install `rsyslog` in the module. something like
```puppet
ensure_packages([ 'rsyslog' ])
```
is needed in the module.
Thanks


Is the syntax correct here?

```diff
- --add-module=/${temp_dir}/nginx-${NGINX_VERSION}/headers-more-nginx-module-${HEADERS_MORE_VERSION}
+ --add-module=/${temp_dir}/nginx-${NGINX_VERSION}/headers-more-nginx-module-${HEADERS_MORE_VERSION} \
+ --with-http_ssl_module
```
> Note the `\` at EOL


Hmm, that's a bit strange. It looks like it works for && though:

2.2.2 :008 > "no" == ("no" && "NO".downcase)
 => true
2.2.2 :009 > "no" == ("NO".downcase && "no")
 => true


Again, an early reassignment of ``delegate`` can cause issues, like here ... where the debug log would contain an incorrect connection.  ``delegate`` should hold the real connection until the last possible instant of ``close()``.


建议保留链接，翻译和校订作者的链接也请保留，翻译和校订作者的链接可以从 zh/doc/PLAN.md 的最后找到，写法可以是：

```
> 原文：[eLinux.org](http://eLinux.org/Legal_Issues "http://eLinux.org/Legal_Issues")
> 翻译：[@zxqhbd](https://github.com/zxqhbd)
> 校订：[@lzufalcon](https://github.com/lzufalcon)
```


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1269 of 1269 tests passed)

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


I wasn't sure if I should add logging here saying we're going to try to generate a valid url by escaping input


Yes, I was wondering what exactly you were trying to do here, and planned on asking.

Do you want a default set of values (level is 1, xpReward is 100, etc.) and a way to customize it by applying modifiers?


`#` 号后面加一个空格吧，全文都加以下吧。


code is :moneybag: :)


(And the less you have, usually the less bugs it generates).


Linux 作为通用语，首字母大写比较好。


Kernel 只用 GPL V2 协议授权？


Do you have the three test cases mentioned above?


removing `LIBRARY` has a positive effect on win32. the configure step no longer fails, but the network code doesn't compile on win32 yet so I can't test the install part.

removing `LIBRARY` on osx doesn't seem to have any effect. the helper libraries are still installed to the wrong path:

```
Install the project...
-- Install configuration: "Debug"
-- Installing: /Users/garrett/Documents/kodi/addons/.//game.netplay
-- Installing: /Users/garrett/Documents/kodi/addons/.//game.netplay/addon.xml
-- Installing: /Users/garrett/Documents/kodi/addons/game.netplay/game.netplay.1.0.0.dylib
-- Installing: /Users/garrett/Documents/kodi/addons/game.netplay/game.netplay.15.0.dylib
-- Installing: /Users/garrett/Documents/kodi/addons/game.netplay/game.netplay.dylib
-- Installing: /Users/garrett/Documents/kodi/tools/depends/target/binary-addons/macosx10.10_x86_64-target/build/depends/lib/kodi/addons/game.netplay/resources/libXBMC_addon-x86_64-linux.dylib
-- Installing: /Users/garrett/Documents/kodi/tools/depends/target/binary-addons/macosx10.10_x86_64-target/build/depends/lib/kodi/addons/game.netplay/resources/libKODI_game-x86_64-linux.dylib
[ 90%] Completed 'game.netplay'
[100%] Built target game.netplay
touch .installed-macosx10.10_x86_64-target
```


1: M100 needs more reviewers
1: M101 signature is missing
1: M111 unrecognized component name: requirements


[anitaziegner]
company: Whizz Education
num_female_eng: 1
num_eng: 6
last_updated: 2015-08-06


I see, however 2 lines would be more conventional.

Also it might be a good idea to replace the this:

def disp_current_balance
	print "\nYour current account has "
end

def disp_savings_balance
	print "\nYour savings account has "
end

With one method looks like this:
def display_balance(account_name, balance)
...
end


Signed-off-by 这个是一个通用标记，建议保留后面打括号加翻译或者反过来，例如：

署名行（Signed-off-by）和。。。


We are using Aurora on Ubuntu 14.04, and had similar issues- can discuss in person.


Why require that `T: Copy`? I guess because in a concurrency-oriented future, that would be handy?


If that is the case, we should add a comment justifying the`T: Copy` bound.


also, can you help with the platform/arch tests in 87ac8f7? I'm not sure what values to compare in the if statements


I cannot reproduce the issue with the following code:

	var _buttonGroup:ButtonGroup = new ButtonGroup();
	_buttonGroup.gap = 10;
	_buttonGroup.verticalAlign = ButtonGroup.VERTICAL_ALIGN_MIDDLE;
	_buttonGroup.horizontalAlign = ButtonGroup.HORIZONTAL_ALIGN_CENTER;
	_buttonGroup.direction = ButtonGroup.DIRECTION_HORIZONTAL;
	_buttonGroup.distributeButtonSizes = false;
	_buttonGroup.dataProvider = new ListCollection(
	[
		{ label: "One" },
	]);
	this.addChild(_buttonGroup);

A single button is displayed, as expected.


reverted in https://github.com/mapbox/node-sqlite3/commit/4955b79f04509196a15c360e612f9c0c393c3609 - refs https://github.com/mapbox/node-sqlite3/issues/459#issuecomment-128416060


You mean the `emacsWithPackages` wrapper, aren't you? AFAICS there's nothing in `emacs24` that adds these lines to the `site-start`.


there are a few changes like this that snuck in, they'll be gone after review


## 后加空格，Linux 大写


Fixed.


In cases like this, where you have two methods which share logic, you can actually create a third method which has the duplicate logic. Then call that method from the original two.

It is likely that you would want this new method to be private. This means that it can only be called from within an instance of the class: http://rubylearning.com/satishtalim/ruby_access_control.html


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.467](http://teamcity.monogame.net/viewLog.html?buildId=9461&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


I'd like to point out that `packetReplyTimeout`, `keepAliveInterval` and everyting else besides `startupClasses` and `optionalStartupClasses`, will not be respected by Smack 4.1.

I recommend removing everything from L15 on and using a static initialized block like
```
static {
  SmackConfiguration.setReplyTimeout(30000);
}
```
instead.


EDD core has always used `-1` to identify a guest. Good or bad that's how it is right now


This way we would loose the TYPO3 6.2 compatibility.

What about something like this:

````
'depends' => array(
		'typo3' => '6.2.0-7.3.99',
	),
'conflicts' => array(
	'typo3' => '7.3.0',
),
````


initial commit of readme.md


Land in the most recent relevant branch and then cherry-pick to older versions. :)