**Line is longer than allowed. (83 > 80)**
*LineLengthBear, severity 1, section `linelength`.* *Related to file `coalib/bearlib/abstractions/Lint.py`, line 103 column 81 to line 103 column 84*.


**This file contains unused source code.**
*PyUnusedCodeBear, severity 1, section `python`.* *Related to file `coalib/bearlib/abstractions/Lint.py`, line 7.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/bearlib/abstractions/Lint.py
+++ b/coalib/bearlib/abstractions/Lint.py
@@ -4,7 +4,8 @@
 import tempfile

 from coalib.bears.LocalBear import LocalBear
-from coalib.misc.Shell import escape_path_argument, run_shell_command
+from coalib.misc.Shell import escape_path_argument
+from coalib.misc.Shell import run_shell_command
 from coalib.results.Diff import Diff
 from coalib.results.Result import Result
 from coalib.results.RESULT_SEVERITY import RESULT_SEVERITY
```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/bearlib/abstractions/Lint.py`, line 37.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/bearlib/abstractions/Lint.py
+++ b/coalib/bearlib/abstractions/Lint.py
@@ -34,7 +34,8 @@

 class LinterHandler:
-    def create_arguments(self, filename, file, config_file): # TODO Also file???
+
+    def create_arguments(self, filename, file, config_file):  # TODO Also file???
         raise NotImplementedError

     def generate_config(self):
```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/bearlib/abstractions/Lint.py`, line 49.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/bearlib/abstractions/Lint.py
+++ b/coalib/bearlib/abstractions/Lint.py
@@ -46,7 +46,8 @@
 # TODO ------> Or "hybrid" approach, needed arguments are optional, if one activates
 # TODO ------> further options these get passed by kwargs
 def Linter(executable, **kwargs):
-    kwargs["executable"] = executable # TODO Sucks...
+    kwargs["executable"] = executable  # TODO Sucks...
+
     def create_linter(cls):
         class Linter(LocalBear):

```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/bearlib/abstractions/Lint.py`, line 58.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/bearlib/abstractions/Lint.py
+++ b/coalib/bearlib/abstractions/Lint.py
@@ -55,7 +55,9 @@

             @property
             def handler(self):
-                return cls # TODO Directly import functions? Rethink design maybe then...
+                # TODO Directly import functions? Rethink design maybe then...
+                return cls
+
             @property
             def executable(self):
                 return kwargs["executable"] # TODO This needs to be solved more elegant...
```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/bearlib/abstractions/Lint.py`, line 61.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/bearlib/abstractions/Lint.py
+++ b/coalib/bearlib/abstractions/Lint.py
@@ -58,7 +58,8 @@
                 return cls # TODO Directly import functions? Rethink design maybe then...
             @property
             def executable(self):
-                return kwargs["executable"] # TODO This needs to be solved more elegant...
+                # TODO This needs to be solved more elegant...
+                return kwargs["executable"]

             def _execute_command(self, args, stdin=None):
                 # TODO Also more elegance...
```


**Line is longer than allowed. (84 > 80)**
*LineLengthBear, severity 1, section `linelength`.* *Related to file `coalib/bearlib/abstractions/Lint.py`, line 46 column 81 to line 46 column 85*.


**Line is longer than allowed. (89 > 80)**
*LineLengthBear, severity 1, section `linelength`.* *Related to file `coalib/bearlib/abstractions/Lint.py`, line 58 column 81 to line 58 column 90*.


**Line is longer than allowed. (90 > 80)**
*LineLengthBear, severity 1, section `linelength`.* *Related to file `coalib/bearlib/abstractions/Lint.py`, line 61 column 81 to line 61 column 91*.


you mean the "componentsProvider.getInstallationFolder()"?
I think yes. It's possible to throw NPE.


Fix #768


These were deleted in a later commit.


same with this.


`ReadFromFile()` --> `ReadFromDir()`
`WriteToFile()` ---> `WriteToDir()`

Probably a better convention


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


s too slow!!!!!
I must accelerate the pace tomorrow.


First Header | Second Header
------------ | -------------
Content from cell 1 | Content from cell 2
Content in the first column | Content in the second column


Hello,

I am a coder in training, learning how to create amazing things.


If you add just nightwatch/commands it takes into consideration only the files in that folder, it doesn't take into account files from subfolders


NICE SOLUTION!


HttpContext.Current - надо HttpContextBase встраивать в конструктор, как сделано во всех остальных местах. напрямую HttpContext.Current лучше не использовать


Is there any documentation about the naming convention of payment types?
Is `payment-<method key>` the pattern?


desolé mais là on prends le FW et ses obligations
Oui il faut faire du générique
c'est pour cela que je t'avais mis 2 articles qui correspondent exactement à :
- des balises sémantiques avec des classes qui veulent dire quelque chose
- la possibilité de créer des thèmes sans le FW puisque le code des balises est correct ( exemple classe="article" au lieux de class="md-5 sm-5 ld-3") puisque en dure dans le code

Voila

Je te remet malgré tout les 2 articles

http://makina-corpus.com/blog/metier/2015/utiliser-bootstrap-avec-un-pre-processeur

http://blog.lesieur.name/utilisation-optimisee-de-framework-css-comme-bootstrap-avec-less/

ET SURTOUT
je met la doc de bootstrap ainsi que le lien exemple
doc
http://v4-alpha.getbootstrap.com/layout/grid/#sass-mixins
exemple
http://jsbin.com/ruxona/edit?html,css,output

c'est donc dans cette derniere version que je crois surtout avec l'arrivée de CSS variable et les CSS GRID LAYOUT
https://drafts.csswg.org/css-grid-1/


I really don't understand why `messenger` was here. Is so weird.


As you removed every usage of `timeoutLock` it is safe to remove this field from `ProtocolCount`.


`APP_ASSETS ` isn't used `build.asset.ENV` but use to bundle `css` & `js` in other tasks where `DEV_DEPENDENCIES ` and `PROD_DEPENDENCIES ` are used.


Лучше хранить бекапы в папке \App_Data\Backups\ - безопаснее


have you synced your tree recently ? it should try 'pdflatex' first there before texi2dvi, and I don't see the message that appears when pdflatex fails and it uses texi2dvi


ok !


This is great @mcm-ham :+1:


@nfeske: above are two sentences that catched my eye.


Hi! Have created a new commit in develop branch to prevent collision with other tables inside a page.


Hi. Thanks for the patch. Could you change the commit message to include the license and to remove the reference to the GitHub bugtracker, please?


Build error:
`node.ProtocolInstance().(*manage.ProtocolCount).SetTimeout undefined (type *manage.ProtocolCount has no field or method SetTimeout)`


thanks :)


Confused by this. Why does Django need to use it as a dict? Also all jsonobjects already support dict access.


Nope. Good catch.


мы напрямую SqlConnection и SqlConnectionStringBuilder не используем (без привязки к БД). Чтоб выполнить команду SQL используй _dbContext.ExecuteSqlCommand (аналогично тому как ты это уже сделал в методе BackupDatabase)


А это точно надо? просто вопрос


this should be `('openclinica', 'meta')`


`TAG_ONE_OFF` is probably a better fit


But I'm not sure if removing the lock is a good idea. Won't this field be modified by several go-rountines?


They can't all happen in parallel, since the first is necessary before we can do the others (I think—see the `associateSpaceManager` tests). However, I can put two of them in parallel.


I'd prefer to have this code synchronised. Otherwise users will be forced to use ```require.ensure``` instead of simple ```require```. And this way is error-prone.


I'm using the hash function as well, but I do think that we could use a
better algorithm.

I think a good approach might be to switch on length, and then for equal
length strings, check which character position varies most on the cases
that are left. This can be done recursively with the remaining cases.

About the AST, I've done something along these lines some time ago - the
base AST is at https://github.com/waneck/haxe/blob/gcr/gcrtypes.ml . The
idea was to make it more comfortable to work with filters, and more close
to how typed targets will see the output. It's been a long time I haven't
worked on it though


javascript метод называется DownloadBackup, а по сути делает и download и restore и delete. так называть нельзя методы нельзя. Уж лучше "submitBackupAction"


Thanks Luigi, should be better now


This should be using crispy forms: https://www.commcarehq.org/styleguide/forms/


Please note that the ALPN TLS extension is only supported since OpenSSL 1.0.2 which is likely not made available by your OS (unless you're Linux distribution is a rolling release or you already run Ubuntu Xenial or Debian testing).

I'm using the static libssl.a and libcrypto.a libraries from Ubuntu Xenial packages on Ubuntu Trusty. This is for testing purposes only, one should never ship software with statically linked OpenSSL libraries!


I'll leave synchronisation as it is and will look into other options later.


Hi @hamsterksu,

I reviewed your contribution. As you can see, password is saved in a MD5Hash format, not in plain text. In order to change the default password in the restcomm.script you need to use the same format. So please try to convert the text passed at the "$INIT_PASSWORD" flag to MD5Hash format.

Thank you.


does the storeService not have a similar `ready()` function we can use here? This approach seems a little rough to me.


This list of ciphers is extracted from: https://blog.qualys.com/ssllabs/2013/08/05/configuring-apache-nginx-and-openssl-for-forward-secrecy


seems like it already does


added.


@m-stein I adjusted the fixup accordingly, see e9a41cb. Autopilot tests for Muen look good.


05 - Image Feed/Image Feed/Models/Image.swift

Use a `NSURL` or whatever the type is.


Never mind, found it: https://github.com/commercetools/commercetools-payone-integration/blob/master/docs/Field-Mapping.md#payment-methods-covered


Thanks for the tip ;)


- Initial import under CPL (Cats Public License).
- all new code which is amended since that initial import is GPL v2


The problem isn't that the store service isn't ready, but that the service hasn't been added to the `window` object yet =/. It gets added in the created callback for `createMainWindow`, but for reasons beyond me that isn't as fast we need.


nice work


# :stop_spaceship_homepage:


Initialise the image data using [Unsplash](https://unsplash.com/developers)'s API or with raw data in a `.json` file.


This should be B3


No :scream:  I'll do `DefaultSpacesTest`, and then raise a chore to do this elsewhere.  I put it in here because I wanted to be sure the right error was coming out of _my_ test.


↓のコード追加すると、カーソルを乗せた時に★の色も変わります！ :smiley:
```php
tbody.list-body tr:hover > td.review {
	color: #fff;
	-webkit-transition: all .8s ease;
	transition: all .8s ease;
	cursor:pointer;
}
```


remove this line


needs b3 dispatch:

```
@method_decorator(login_and_domain_required)
    @use_bootstrap3
    def dispatch(self, *args, **kwargs):
        return super(LoginAndDomainMixin, self).dispatch(*args, **kwargs)
```

https://www.commcarehq.org/styleguide/forms/#b3-migration


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


CSS мы прямо в cshtml не указываем. надо добавлять их в соответствующие css файлы


老师下节课交怎么用linux吗？弄了个双系统但是不太会用，还有就是第一次作业怎么核实啊，那个readme现在需要写什么啊？

在 2016年2月24日 下午8:32，PatYoung <notifications@github.com>写道：

> 不是关注这个内容了吗，这不是作业内容之一吗，老师能知道谁关注了吧。那个个A什么我也不清楚啊。里面好像有课上说的IP什么的，好吧，我就是不知道。老师你看见了解释一下啊。
>
> 你竟然没选这个课😑
>
> 2016年2月24日 20:09于 UncleSandwich <notifications@github.com>写道：
>
> 我的意思是如何让老师知道我们的账户，然后才能查看我们写的readme。
> 这是Python的官网: https://www.python.org/, 里面提供了Python2.7的windows版<
> https://www.python.org/downloads/windows/>，我只想问Anaconda的提供了什么额外的功能.
> 我是谁不重要，网上最好不要透露自己的真实姓名。
>
> —
> Reply to this email directly or view it on GitHub<
> https://github.com/caihao/computational_physics_whu/commit/adba974930306494dbb8ba3f1fada6733a696666#commitcomment-16294699>.
>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/caihao/computational_physics_whu/commit/adba974930306494dbb8ba3f1fada6733a696666#commitcomment-16295195>
> .
>


перенеси break на новую строчку лучше


buckupFileName называется backupFileName


I did a few changes in both the html file on here and on the copy I have on my computer  so I went ahead and just uploaded the one on my computer and deleted this one to make sure they are both the same and up to date.  I guess in the future I'll just always copy the contents of this page over what I have on my computer.


Leaf nitrogen content and nitrogen content are defined the same way. Therefore they are forced to be equivalent. Leaf nitrogen content should be defined as:

leaf nitrogen content (TO:0000543): composition and ('occurs in' some '**leaf**') and ('inheres in' some 'nitrogen atom')

Travis build failed


A enlever avant le retour en dev


<string name="qs_brightness_slider_title">Mostra cursore della luminosità</string>


@46labs yes. /s for cachedb sharing, /b for sharing through binary interface. If no flag set, profile is not replicated. Also you can't do both cachedb and binary interface replication, simply because it makes no sense.


s/emite/emit/


maybe doing an `abspath` on the config file path here might help? Have you tried that?


OK, so the whitelist it is... Given the amount of options, how about creating the `whitelist = ["loaders", "external_runner", ...] and do this in a loop as the notifications can be generic?


Bør vi ha ett eget nettverk for koha?

```
public
	- patron-client

staff
	- koha
	- catalinker
	- overview

koha
	- koha
	- koha_mysql
	- sms
	- smtp
	- sip
	- tcpsink

private
	- services
	- fuseki
	- fuseki_data
	- koha
	- koha_mysql
	- sms
	- smtp
	- sip
	- tcpsink
	- elasticsearch
	- elasticsearch_data
	- z3950proxy
```


Wasn't sure how this worked, but wanted to make sure the changes I did on my copy on my computer were up to date with what's here so I thought the easiest was to delete it and upload it again.  This copy is what's currently on the server.


Thanks!


should fetching really be the last command ?


make check-verify
观察结果并补充servicebroker相关的测试.以及发现的潜在的bug.


agreed, but out of the scope of the PR, in my opinion.


Btw, thanks to @lexmag for bringing this up! :clap:


"I'll cast a protection spell on you so the terminal won't defeat you like before." ?
and maybe the last sentence can be something like "I have enough strength for that, at least."


Quel est l'objectif de les ajouter là ?


you'll want to use a check_prerequisites method like this:

```python
    @classmethod
    def check_prerequisites(cls):  # pragma: no cover
        if shutil.which("java") is None:
            return "java is not installed."
        else:
            exitcode = subprocess.call(["java", "-jar", cls.jar, "-v"])
            if exitcode != 0:
                return ("jar file {} is invalid and cannot be used"
                        .format(cls.jar))
            else:
                return True
```

(copied from CheckstyleBear.py, AGPL)


Let's say it's ok for this time, but please consider writing more tests along the way


so ideally that function is moved out to somewhere and both just assign it or something?


Fixed


Autobind won't work here because, in this case, I need to pass a dynamic function definition with the tab parameter already filled.


I would recommend doing:
````sh
RUN apk upgrade --update \
  && apk add --virtual .build-deps \
  ...
````

The `--no-cache` option wil only prevent that the APKINDEX is not cached so every `apk` command will fetch APKINDEX from network again. Since you run `apk upgrade` it would be slightly more effective to just keep the cached APKINDEX and delete it when done (which you also do with `rm /var/cache/apk/*` down there)


I encounter a:

      PHP Fatal error:  Class 'Zend\ModuleManager\Listener\ConfigListener' not found in /dev/shm/BUILD/zend-modulemanager-fe5a156df1ae4245c7b4a7b2b9067545761f59d7/test/Listener/ConfigListenerTest.php on line 44

I will dig more (probably related to path used here)

BTW, this .travis config which use  --ignore-platform-reqs and force SERVICE_MANAGER_VERSION="^2.7.5" which is different from composer.json seems terribly ugly.


为何要撤回捏~


it does, but it also saves me a key press when I switch between the two modes (which I do frequently)


Should raise a different exception here, rather than raising VM_HAS_VGPU with a null reference.


Ottimo! Siamo pronti per le misure :D


@raiashu92 instead of sonar fixing, Please use a meaningful English sentence that says "Fixing critical/major coding standard defects in x module or a, b, c files" etc. Also add Jira ticket (create a subtask under the main task) for any fix. You can use same Jira number twice.


@gabrielamatias I'd change to "create and update plans for each account" as it sounds a bit funny with account duplicated. Also do you have another example as well as forecasting?


"Why did I ever forget it?"
> "How could I forget that?" ?


this update also force me to consider use library module to encapsulate the useful api


Project plan


I've downloaded both plugins.js and template.js files, erased the cache but table style isn't correct. Did I miss something ?


Ještě jsem si vzpomněl na další možný rozšířený požadavek: port na jiný box - EKT DID5206. Je to stejný výrobce, ale má to jiný chipset (Entropic místo Ali), takže i jiné SDK a úplně jiný builsystém (žádné gu, pkgbuildy atd). Je to celkem velký úkol asi se nestihne, je to jen nápad.


@jyp: #9


Correction MAXIM Lib


**[unitTests_php_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_branches/28102/testReport/) for this commit**


Because this delete function is in inbox.js, and deals with all types of docs, not just contacts (also reports, messages, whatever), so the message is generic.
I could do prettier by displaying a different message by type of doc. Not sure it's worth it...


"If the parts don't stay well in place, use the strings which the monsters nearby possess."
> "If the parts don't stay in place very well, get some strings from the nearby monsters and use it."
does this sound better?


These two queries are slightly different but I think this one is wrong anyway.


Okay you've convinced me, it does make more sense to do it all in memory.

Also I just realised one big problem with my approach is that if the benchmark is killed (via Ctrl-C), `logman` is still running and writing to the .etl file.

So please feel free to send a PR, it'll be great to have someone else contributing to the BenchmarkDotNet Diagnostics.

----

One thing to explain, we are currently using the old codeplex version of TraceEvent, instead of the NuGet package because the NuGet package is .NET 4.5 only and BenchmarkDotNet currently supports .NET 4.0.

I emailed the TraceEvent NuGet package owner, who turns out to be Vance Morrision (VS/.NET Perf Guru!!), he said:

> As time goes on we add features that then have references to things that are not available in V4.0 (IN particular it knows about EventSources more deeply).
It is work (with low payoff), to support two versions (one with the newer features and one that runs on V4.0)
The TraceEvent package supports V3.5 as well, and that should work on V4.0. However I have not tried actually doing this.
We are hopefully going to be open sourcing the current code (month or two), so you can create specialty versions yourself.

So they're not going to make a .NET 4.0 version, but when it becomes Open Source we could make our own if we wanted. Until that time, the best option is to stick with the codeplex project (non-NuGet package) that works with .NET 4.0 (i.e. what we have in BenchmarkDotNet at the moment)


these two methods should go in the elgot package object


Cool, I just wanted a hash function which makes infrequently enough collapsing, but short enough not to take too much of extra space since the host key is included to all logged events.


Neue Version der Datenbank


Approved.


Change simple schedule to cron schedule


It isn't, it re-runs the previous command after fetching.


I'm not sure if a non-visible setting in the translator even works properly. can you make sure that saving the translator doesn't overwrite anything? I would have suggested to keep the global setting and have it there, but this works too (if it works)


```ruby
attributes.merge!(payload_version: ContentStorePayloadVersion::V1.current) if payload_version
```
might be cleaner


Looks like "is None" should be used here. At least it was equal previously.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17387](https://teamcity.mesosphere.io/viewLog.html?buildId=146860&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **UNKNOWN**
Summary: Canceled Build time: 00:21:05


using logger when we do not have a job would be ok I think, I only commented on those that did both.


Ancora non ho provato a fondo. Però se fosse tutto ok il sistema già prende il tempo e lo stampa a video.

Ciao
Pier

> Il giorno 26/feb/2016, alle ore 11:26, "cl4u2" <notifications@github.com> ha scritto:
>
> Ottimo! Siamo pronti per le misure :D
>
> —
> Reply to this email directly or view it on GitHub.
>


I wouldn't bother with this, that's just phpstorm that's complaining, it's the same in every single create() method.


Instead, try `module Foo = Agility.VMResources`


I find it hard to see when ones bday is in the calendar if it's streched over two days.
If the issue is that the happy bday post is posted to early could a delayed post fix this issue?


I started this recorder but gulp serve directs me to to http://localhost:3000/#/conf then conects to browserSynch and shows only blue stripe with "{{ navbar.title }}" and two blank inactive  buttons on the right side. How to fix it?


sharing code is good, but those two things are *not* identical. for job, you need to set the job to active/submitted, for continuous, the items.


rather do a os.path.join please! This is not platform independent. Also it's a list you should do this with all elements


The form processing code certainly assumes that the form ID is unique. Also the `FormData` table has a unique ID on the `instance_id` column so pretty sure this is safe. I think the reason for the list return is the way in which the query is made. Another approach would be:
```
FormData.objects.values_list('received_on').get(instance_id=form_id)
```

(which may throw an exception is the form is not found.


Can we change this to log occurrences to statsd?


l13 and l14 can be cleaned up.


Do we have a bug submitted upstream regarding this issue?


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


Commit Overall Website MS/PartsUnlimited copy


У меня работает код
```
- name: NPM | gulp compilation
  shell: node_modules/.bin/gulp chdir={{ site_root_dir }}
```
прекрасно, т.е. sudo не нужен даже если я запускают от menangen пользователя в sudo, плюс я упростил всё в одну строчку.


`$table.prepend('</thead>');`

instead of

`$table.prepend('<thead/>');`

isn't it ?


I would not consider it a bug, it is just not supported.


@VocrsZ


Please rephrase the commit log. Transition is the entire PR.


Each change or commit has a comment section like this. Once someone has spotted a change in the BIll, everyone can share their views on the change here


This change is accidental. I'll undo it


f


df


Besides adding the file name like here (https://github.com/resin-os/resin-odroid/commit/c9e45033d850efd650dc23d7c53aeb36f557694e) I am not sure how much more I can rephrase.


You should `require` `TokenFieldWrapper` after setting up mockery, it should work then :)


Should we do some assert_can_boot_here checking of this host here?


you mean file change tracker? it will fire if file status is changed on this.


@bahlo This file sould not contains user specific informations, if there are some, you are making a mistake.
The path should contains a relative path, if the `.sublime-project` is at the root of the project, just put `.` in the path :
```
{
	"folders":
	[
		{
			"path": "."
		}
	]
}

```


we don't have bits anymore.


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


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


Hi Roberto,

That is a start :)

Here's a link that shows the steps that are done for converting to the new security model:
http://www.open-emr.org/wiki/index.php/Codebase_Security#Plan

Check out my commit on this script for what a complete conversion to the standard security model entails(basically need to read each line of code to see if a change needs to be made):
https://github.com/bradymiller/openemr/commit/918ebcf0c32d94596964a46803ca5eefd8ab4ebc
(I haven't tested it yet)

Let me know if you have any questions on any of it. Need to ensure you understand why I made each change. Note that after you convert a couple scripts, the steps becomes more and more obvious.

-brady


Approved.


m <- NULL
  set <- function(y) {
    x <<- y
    m <<- NULL
  }
  get <- function() x
  setinverse <- function(solve) m <<- solve
  getinverse <- function() m
  list(set = set, get = get,
       setinverse = setinverse,
       getinverse = getinverse)


**[unitTests_php_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_branches/28105/testReport/) for this commit**


Capital T?


oops, will change this to Brady Miller...


Is this for future tests? You already have a product creation on the test above.


#the largest heading


I am humble.:)


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


m <- x$getinverse()
  if(!is.null(m)){
    message("getting cached data")
    return(m)
  }
  y<-x$get()
  m<-solve(y,...)
  x$setinverse(m)
  m


I think this is ok!


ok


Looks good. On the other hand, one could ask for the option at the beginning of the ```AutoDoc()``` function, where we check for the other options. That would allow the ```relativePath``` option to be in the record as well as being an option. 5e23a81c92dafe5e20d64d74ba7958a0b465f124


gr


Good one and clean :+1:


Minor Style Thing:
I think it's easier to read if you just construct it inline. i.e.
exec.Command("ip", "link", "set", "dev", dev_name, "up")


it looks sane I think if tests pass we can take it


By the way, since this parameter is called ```gaproot``` in the GAPDoc manual, I would prefer going with some name like this. But since it is only used for the relative path, maybe it is better to go with this name.


it shows my all blog


yoo noobad me in Nov. :stuck_out_tongue:


Does the result of a test run always include coverage? Or only when coverage is being collected? (If so: this should be an optional property)


add error description with str(ex) and except TypeError as ex:

more... I'm not processing toolbox expert, but probably the mesage have to be directed to QgsMessageBar or probably there would be a error message api for processing. Procesing can be used also in NO GUI context, like server context... an creating a MessageBox can generate crash


@bahlo @abrkn See [Sublime docs](https://www.sublimetext.com/docs/3/projects.html):

> Projects in Sublime Text are made up of two files: the sublime-project file, which contains the project definition, and the sublime-workspace file, which contains user specific data, such as the open files and the modifications to each.
> As a general rule, the sublime-project file would be checked into version control, while the sublime-workspace file would not.


This is a good candidate for refactoring to make more readable. e.g. The `let leaving` and `let is_leaving` lines do almost the same thing.


Да, я добавил:
```
- name: Nginx | Dist upgrade to latest
- apt: update_cache=yes
- apt: upgrade=dist
```


extracted


hm, on my mac, this kind of globbing for `rm` doesn't work:

```bash
$ bash --version
GNU bash, version 4.3.33(1)-release (x86_64-apple-darwin14.1.0)
Copyright (C) 2013 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>

This is free software; you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
~/Code/tim on master*
$ find . -name .babelrc
./.babelrc
./node_modules/react-native/node_modules/fbjs-scripts/node_modules/babel/test/.babelrc
./node_modules/react-native/packager/react-packager/.babelrc
./node_modules/react-native-image-picker/Example/.babelrc
~/Code/tim on master*
$ rm node_modules/**/.babelrc
rm: cannot remove 'node_modules/**/.babelrc': No such file or directory
~/Code/tim on master*
$ rm "node_modules/**/.babelrc"
rm: cannot remove 'node_modules/**/.babelrc': No such file or directory
```

what shell are you using? Maybe let's just leave it up to them to `rm`? Or add another option, like:
`find . -path "*node_modules/*/.babelrc" -exec rm {} \;`


I am not sure how the description reflects the change since the quota information follows the UA line and is not changed at all.


maybe this can just be made part of the while() condition?


AddressStr is in the stack no ?


You should follow the consistent copyright notice. That is, "Copyright (c) 2016 Jia-Rung Chang" in ascending order.


Thanks a lot!


I wouldn't have expected the _oa_buffer_num_samples() code to really iterate through individual samples and instead essentially just return (tail - head) / report_size;

Maybe just with a WARN_ONCE check that (tail - head) is aligned as expected before dividing


Oh, just realised this is not `squadrun` repo.

So no, These print statements make for a good log in the Cloudwatch Trail.
Not so noobad me :sunglasses:


@davidzchen : I think this has broken the python test suite, can you have a look? http://ci.bazel.io/job/protobuf/92/

/cc @philwo


Did we submit anything to ts where they said it is not supported and won't fix?


as above, maybe make this part of the while condition


You can modify the main makefile to pass -DBUILD_PULSEAUDIO=OFF  to CMake.
Here for debug https://github.com/musescore/MuseScore/blob/master/Makefile#L70
And above for release.


It's public in the sense that it needs to be called from the form validator as well as from `Page` itself, but private in the sense that it isn't a documented API that we expect end users to use. What's our convention for that? Based on your comment on https://github.com/torchbox/wagtail/pull/2027#issuecomment-163884733, I figured we were going with "underscore = not intended to be end-user accessible"...


@chelmuth Could you apply this to staging if @m-stein is ok with it? (see also previous discussion at 1623f78).


This is not necessary, I will remove later.


@umka-sev thanks for note. The check is out of the current code https://github.com/natsys/tempesta/blob/master/tempesta_fw/t/fuzzer.c#L235


No we did not, but GCC5 is clearly unsupported. I will submit a ticket to meta-ts.


so far so good


if we do go for a simpler oa_buffer_num_samples() impl. as described above then maybe bring back the line about only reading the head/tail registers to help reassure that the code doesn't need to access any mutable dev_priv state.


agreed, a pair would have fit.
But we're iterating on a input map (request.create) and building a map as output (response.notCreated).


Please the link to the issue in the comment here.


trademark?


Hack: a piece of computer code providing a quick or inelegant solution to a particular problem.


Yes, this can also be a part of while condition.

Also, Can you please check the updated patch which fixes a bug I had in this one: https://github.com/sourabgu/linux/commit/f118f6f59b5a333286db0a27f7b1ce6be15402c5
Thanks,


Thanks for the explanations Drew. I wasn't fully aware of that. I'll try to keep that in mind!


Sorry, but I updated master 30 minutes ago...


I'm also rookie on python lanaguage, everything done by Bromix.
Thank you.


Oh, right, I see.
I'd be in favour of having protected method template as it looks more modern(?) :)
Generally, I would try to reduce trickery to bare minimum (and `do {...} while(0)` is a trick IMO).
So, I pushed an update here (48f91a1aa49febd149ed6f955d1eedad60122a33) but if you are unhappy with that we can decide not to merge it with master.


Actually no, mySoundfontsPath will never be empty... it will have the value set in Preferences::init()


OH...


@Kensan Could you please open a cleanup issue for merge after the release?


this runs `elm-test TestRunner.elm`


Toto je smell. Vytvoris objekt zavolas metodu a rovno ho zahodis. Dajme tie metody `parse_procurement_announcement` rovno do existujuceho `Uvobot::UvoParser` ved tam su parsovacky nie?


copy & paste


With the STORE-1196 fix , get server URL via utils.url.


Premature abstraction, v klude to nechaj tak ako to bolo. Je to priamociarejsie a ziadna magia.


@chelmuth Yeah sure. Sorry for the hassle.


it working...


Approved.


Ah - looks like someone really messed up the form. People have been having too much fun with it! Anyway - you need to create your own form, and then get the right URL for the form and question IDs - the question IDs for the code above (`q1ID` and `q2ID`) no longer match those assigned in the form at the `baseURL`


Great idea!

The format looks a bit noisy though. If we strip all unnecessary symbols, it is still very much readable by humans and machines alike:

```
ruby ~> 2.3 Gemfile
postgres ~> 9.3
python ~> 2.7 requirements.txt
node ~> 5.5 package.json
```


these dont look very good stylized as buttons -- bad spacing


I have a doubt here: if this class is a singleton, you could have concurrence confilcts here, no?


> 1) can you please rename your buffer_reader.go to buffer.go to make it easier to review?

Done.
> 2) I believe this can be done with adding "source io.Reader" to "buffer", adding the rest of your Read function to buffer.Read, and removing Write. Optionally you can use "buffer *bytes.Buffer".

Not sure what you mean exactly.


0.json :
* "idEventADeplacer" :1 => pourquoi le 1 et pas le 9 ?
* "traversable": true => un mouvement est toujours possible pour un event traversable

ConditionMouvementPossible.java :
* public boolean estVerifiee() => à remplir sinon toujours false

InterpreteurDeJson.java :
* e.printStackTrace(); => ajout d'un System.err.println() explicatif


Yep, looks a bit complicated. We'll see, I'll try to decode stats properly today.


Use function size


Alejandro Moreno Cuquerella y Desirée García Soriano.


Any reason not to use ft.Stringer?


Done in #06fa668


i'll use if (s.contains("mySoundfontsPath"))...stand by....


The size test is probably not needed.
Also the sfPath key needs to be deleted if present.


When can we see [Enfield](http://enfield.getpoole.com)? Or am I jumping the gun.


ok...i'll do that...stand by....


Maybe reword this?
If it really happens to result in compilation errors, then it's fine as you will notice.
The dangerous thing is that it might silently use the other option than the one you expect, depending on the order, etc...
People should not think that the duplicate options are always fine as long as they don't get compilation errors...


But my version is simpler now:)

```
mySoundfontsPath = s.value("sfPath",           mySoundfontsPath).toString();
s.remove("sfPath");
QStringList pl = mySoundfontsPath.split(";");
pl.removeAll(QFileInfo(QString("%1%2").arg(mscoreGlobalShare).arg("sound")).absoluteFilePath());
mySoundfontsPath = pl.join(";");
mySoundfontsPath = s.value("mySoundfontsPath", mySoundfontsPath).toString();
```


ok.  you beat me!


This [line](https://github.com/textpattern/textpattern/blob/bd1390b4129ae359d92b14a5fabac61c03865cfc/textpattern/update/_to_4.6.0.php#L235) is giving me an error – "Specified key was too long; max key length is 1000 bytes” and the txp_token table is not created. This is happening with site that is already a 4.6 dev version.


yup...that does look better and correct.  I'll let you submit.


Oh sorry I didn't understood you wanted to improve this yourself -> yes, please go ahead!!


But they do exactly the same workaround:
```bash
export ALPINE_GLIBC_BASE_URL="https://circle-artifacts.com/gh/andyshinn/alpine-pkg-glibc/6/artifacts/0/home/ubuntu/alpine-pkg-glibc/packages/x86_64" && \
    export ALPINE_GLIBC_PACKAGE="glibc-2.21-r2.apk" && \
    export ALPINE_GLIBC_BIN_PACKAGE="glibc-bin-2.21-r2.apk" && \
    wget "$ALPINE_GLIBC_BASE_URL/$ALPINE_GLIBC_PACKAGE" "$ALPINE_GLIBC_BASE_URL/$ALPINE_GLIBC_BIN_PACKAGE" && \
    apk add --no-cache --allow-untrusted "$ALPINE_GLIBC_PACKAGE" "$ALPINE_GLIBC_BIN_PACKAGE" && \
    /usr/glibc/usr/bin/ldconfig "/lib" "/usr/glibc/usr/lib" && \
```


@akash1808 please review it.


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 161](https://teamcity.labkey.org/viewLog.html?buildId=305691&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 8 (1 new), passed: 43 Build time: 01:17:59
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyLearnAboutStudyDetails: <no details avaliable>

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

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

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

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

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

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

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

```


I think indeed it would be "pretty" to have the function gracefully handle $aCurators to be empty instead of returning false, which will indeed make it a necessary evil :smiling_imp:


So related to some change in restoreAutoloadFunctions.
Reverting this function to the one from 2.6 solves it.


You need to re-check your translations, especially for hyphen must follow by \ else will break.


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


Perhaps we should rename this to "LKL hijack library" and also describe the hijack script etc.


@mogoodrich it looks correct.


did you test that it works?


@tgjones well you cannot add it manually. But if its already in the .projitems it will pick it up. The only problem is that It does not show up in the IDE.. We can probably fix that by adding a <None /> Items for the Content.mgcb file.


Typo in commit log: reisn


Approved.


we don't want them in the sys path do we? Can you come up with some references to explain that this is the right thing?


Yep, I'm getting the same on a fresh install - [I posted in the forum about it](http://forum.textpattern.com/viewtopic.php?id=46843).


Should this be `*'g:bling_color_gui_bg'* *bling_color_gui_bg*` instead? My Vim has been kicking up errors when generating the helptags for this plugin since this update. Its complaining about a duplicate tag 'g:bling_color_gui_fg'. I'm thinking this might be the cause.


thanks for effort. I chose another patch for integration.


I'd rename this variable so it doesn't risk clashing with stdlib's atoi. Also, you could use g_autofree here so you don't need to g_free below =)


```
         ms_on_add_order: function(current_order){
            is_empty = current_order  && current_order.is_empty()
         if (this.config.multi_session_replace_empty_order && current_order.ms_replace_empty_order && is_empt) {
                current_order.destroy({'reason': 'abandon'})
         } else if (!is_empty || !this.config.multi_session_deactivate_empty_order
                  // keep current_order active
                  this.set('selectedOrder', current_order);
           }
```


is this a check for local? what if it is ftp? or anything other? you should check for "file" imho


выберу второй вариант


不需要，这样就可以了

2016-02-26 16:20 GMT+08:00 UlricQin <notifications@github.com>:

> 波哥这个是不是得用select+ticker，for+ticker跟sleep是一个效果吧
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/yubo/agent/commit/83a8132061e16b73a71064491b3deb016c79698b#commitcomment-16346737>
> .
>

--
喻波
小米 |  运维平台研发
电话：18601160197


thx for the comment. will update soon.


добавь

                    <field name="multi_session_deactivate_empty_order" domain=[('multi_session_replace_empty_order', '=', False)]/>


@dellis1972 Have you tried making other changes to the project in VS, and then saving? I think I remember that VS wiped out the MonoGameContentReference action and changed it to None.


@Arno-Nymous  ich habe jetzt bei mir den UpdateManager ausgestellt (hatte immer wieder überschrieben) und den ganzen ordner module aus deinem stable https://github.com/Arno-Nymous/pyload genommen in meinen gepackt. Er kommt aber immer noch nicht weiter als

`
ERROR	Error importing ExtractArchive: cannot import name Expose
`


duplicate, already in pull reqest #2


https://github.com/artur-shaik/vim-javacomplete2/blob/2950a868020c5918111c41b5660f323ba0a63442/autoload/javacomplete/classpath/classpath.vim#L63-L74
if has both pom.xml build.grald .classpath, I think if it has read classpath from `.classpath`,will not skip others


This should be `merge!`, otherwise the optional key would not be included.


备注


@steveathon If I don't pass a hotkey at all, I get the default behavior set up in the widget, which is `indent`.  I want the default behavior of the browser, which changes focus.  Instead, I just want to wipe it out of the key binding setup all together.

https://github.com/steveathon/bootstrap-wysiwyg/blob/2.0-dev/src/bootstrap-wysiwyg.js#L23


maybe we shoud change to
```vim
for classpathSourceType in g:JavaComplete_ClasspathGenerationOrder
    try
      exec "let cp .= s:Use". classpathSourceType. "()"
      if cp != '.'
         return cp
      endif
    catch
    endtry
  endfor
```


@jekhor, что осталось в пакете недоделано:

```
I: librhvoice2: hardening-no-fortify-functions usr/lib/x86_64-linux-gnu/libRHVoice.so.2.0.0`
```
* наверное, лучше пофиксить

```
I: librhvoice2: no-symbols-control-file usr/lib/x86_64-linux-gnu/libRHVoice_audio.so.0.0.0
I: librhvoice2: no-symbols-control-file usr/lib/x86_64-linux-gnu/libRHVoice_core.so.0.0.0
I: librhvoice2: no-symbols-control-file usr/lib/x86_64-linux-gnu/libRHVoice.so.2.0.0
```
* можно не фиксить, но можно поковырять на досуге; в принципе, раз библиотека очень work-in-progress, можно забить покамест

```
X: librhvoice2: shlib-calls-exit usr/lib/x86_64-linux-gnu/libRHVoice_core.so.0.0.0
```
* вот это не знаю, зависит от ситуации

```
W: rhvoice: binary-without-manpage usr/bin/RHVoice-client
W: rhvoice: binary-without-manpage usr/bin/RHVoice-make-hts-labels
W: rhvoice: binary-without-manpage usr/bin/RHVoice-service
W: rhvoice: binary-without-manpage usr/bin/RHVoice-test
W: rhvoice: binary-without-manpage usr/bin/RHVoice-transcribe-sentences
W: rhvoice: binary-without-manpage usr/bin/sd_rhvoice
```
* надо бы манпейджи написать и законтрибутить


I'd suggest seamless-immutable over immutable.js
See http://tech.noredink.com/post/107617838018/switching-from-immutablejs-to-seamless-immutable


Thanks for your response..! is this possible using php?


It's a good trick to reject the screenshot promise after 10 retries :+1:

Also add a `.catch` after this .then... so if the open fails we can trace it.

what do you think?


Approved.


So we can't get alluminium again ? that's just reverse Mask's fix https://github.com/skyarkhangel/Hardcore-SK/commit/7abb87d7ccb675c8709edc3c50a984d4eb097075


We know the promise should stop the flow, however as we discussed I feel more comfortable with also an explicit `return` here :D


multi_session_deactivate_empty_order = fields.Boolean('Deactivate empty order', default=False, help='POS is switched to new foreign Order, if current order is empty')


Probably a harmless detail, but isn't the deviceId param already trimmed? (According to the change in SurveyManagerRequest.java within this commit)


thanks. fixed in commit 0b497241aac260ed5f8fa587ac2aaa100ddf1c4f.


Docs


@nfeske minor fix above


Hallo Welt


@blakesweeney That's weird. Not sure why the same commit is listed many times.


**[unitTests_php_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_branches/28120/testReport/) for this commit**


@tgjones you know what I didn't.. darn :/ VS really need to fix that.


I assume this constraint is relaxed to allow people to use SayIt with Elasticsearch >= 1 if they want to?

Why the switch from pyelasticsearch to elasticsearch, out of interest? (We're using the latter in Pombola / YNR - I just wondered what the reason was.)


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


why? remove?


@thiagoalmeidasa Your inlined thing has several issues:

1. It installs `curl` and never removes it, so you leave `curl` and `openssl` (since it is a dependency), thus leaving unnecessary and possibly vulnerable stuff inside.
2. It doesn't have `/etc/nsswitch.conf` workaround, which is necessary for being able to resolve hosts from `/etc/hosts` list.
3. It just introduces unnecessary headaches, while I'm maintaining the `frolvlad/alpine-glibc` image in a stable and reliable way.


Although proposed change is more readable, I will keep it like this because of the SET OR HDR logic.
Can be used to show warnings about incomplete setups in the future.


I've updated the code now with working field values.


Sorry, I did not finish the pull request description :

Allow to specify if widgets are resized when calling setGridWidth.

And In the setGridWidth method i also updated the GridstackEngine 'width' attribute in addition to the opts.width. Il you do not do it you will not be able to move/resize widgets to the the new space generated if the parent grid has been expanded.


Remove this span and make it a text input using `h` (the form_for object).

Then, set the value in the javascript and it will be sent when the form is submitted.


Approved.


部署服务端应该有bug，先撤回，争取明天发个版本，同时把最近捐献的appid加上


Oh, thanks. I haven't realized that, because I do always `make config` before `make install`.


`userplugins` Ordner geleert?


confirmed to be fixed


True, it should pass seconds.


Nice! :+1:


I was thinking for the class
```ruby
def initialize(uri, token_types = nil)
```

but this works too :+1:


good news!


Why handle this different then other exceptions?


Maybe you are right. But better make loop break, to have only one function exit point.
Will add this.


I need to learn with specs for puppet modules first. I will integrate these after.


Hi,

ich bekomme bei dem hook den folgenden Fehler:

ADDON WarezWorld: Error performing periodical task | Couldn't find a tree builder with the features you requested: html5lib. Do you need to install a parser library?

Was fehlt mir da genau und wie kann ich es fixen??

Danke für Deine Hilfe!!


It seems other ul-li elements in the document start with capital?


Please try to Google something like "php convert ETRS-TM35FIN to WGS84"


ditto


If access was denied, then we never tried to do a deployment, so it makes no sense to log a failure and publish an event.


Approved.


Hopefully "Verbinden" touches more meaning of Integrations here


duplicate from pull request #3


:shipit:


I expect "validate the response" is called with response concept not a response object.


Habe jetzt checkout auf "Extract failed" aktuelle Branch gezogen. Den gesamten module verschoben

Laut https://github.com/pyload/pyload/compare/stable...Arno-Nymous:extractor-fix

sind es ja nur die beiden Extract und External . Die habe ich aus userpluglin gelöscht


I left that singular "zusätzlich als App" because it is either one for Desktop or Smartphone


I made a similar change, but for Karaf 4.0.4.  It is not enough to only include the lib/boot directory; if you look at the karaf startup script, you will see that you also need to include the lib/ext dir in lib/ext via the -Djava.ext.dirs property.  While Karaf may run with only the lib/boot directory, the full functionality of being able to drop extra jars in the intended place will not be available.


maybe factorize the code to add/remove labels into a function?


I guess "Hilfe" is better to understand here that "Unterstützung"


Thanks for raising this question. I should have added a comment about this.

This has two reasons:

a) Main reason. If a client collects a powerup, it may take a while until its
inventory update reaches the host (regular ping delay, packet lost). Even if
the inventory update would be triggered right upon collecting, it may
accidentally trigger the host to respawn items. And with primaries, these
would stay as they can't be used up.
I could optimize this by changing the timer into a byte-sized variable and
only increment it as I do with the timer. As MultiLevelInv_Repopulate is
called every 500ms, counting to 4 would be the same effect while saving 3
bytes per powerup type.

b) The timer also has the nice side effect that if a player times out, crashes
or loses a few items due to the secondary item cap during death, the level
won't suddenly get swamped by items in a single frame.

Let me know what you think. I will add the comment about the timer in the next
commit where I will address the other issues you raised.

On Thursday, 25 February 2016 18:53:21 CET vLKp wrote:
> Why not respawn items immediately and avoid the need to count time?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/dxx-rebirth/dxx-rebirth/commit/901a554e96946fec1bd5e926a3
> 957ce600da9023#commitcomment-16343581


Any suggestions for better names? I've just suffixed `blocking` with different things to have "group" of states.


fixed #54


@nfeske last minute remarks :-)


! this.config.multi_session_replace_empty_order  && ! (is_empty && this.config.multi_session_deactivate_empty_order)


Is that still required?


invisible


forgot to add `break` sorry


no more updates ??????


This should be replaced by the maven dependency plugin, which does not truncate the long path name.  The pom for the remote karaf module does this.  See my commit at https://github.com/Steve973/arquillian-container-osgi/commit/be0303d52df56ce74c027bc73a0e50987e649994


Hi, sorry, my mistake. Happy to have you close this request. Thanks for the reply anyways.


I am taking a purely pragmatic approach.  I am basically moving the logic from the codegen to the cppast.  Along these lines, I will probably end up with something exactly like what simon suggested with   (int list * tcppexpr) cases.  As to whether it is better to represent the non-int cases as a CppIfs or a CppNonIntSwitch, I think it depends on if I need to do any reasoning about the call chain (in which case cppif is probably better), or just dump it out.  If I only need to dump it out, it comes down to a question of practicality - will the generated c++ code be nicer, and will to ocaml code be nicer.

I too use the neko hash function.  The "getField" code currently uses a c++ switch in length, and then a series of "if" statements.
But what I was thinking, is instead of switching on the length, do a binary search on the hash, like:
```
var tmp = String( whatever);
var hash = tmp.getHash();
var found = true;
if (hash<0x8000000)
{
    if (hash<0x300400)
    {
         if (tmp=="value1")
             doValue1()
         else if (tmp=="value2")
            doValue2()
        else if (tmp=="value3")
            doValue3()
        else
            found = false;
    }
}
else
{
    if (hash<0xa000101)
    {
    ...
    }
    else
    {
    ...
   }
}

if (!found)
   doDefault()
```
Calculating the hash thresholds optimally to create groups of 2 or 3.
Cpp string constants have the hash code built-in, so this should be fast.

In the case of multiple case values, these could be split, and the code duplicated, or they could be done separately first.
Floats and ints should also work this well this way.


что это? зачем это здесь?


I'd like to clarify what "reject" means here. What the "validate the response" steps are rejecting on errors is the update promise rather than the response. May something like "If they throw an exception or reject a promise returned to the calling algorithm, .." work? Or if you expected SW does something else, please let me know.


I've reviewed the PR up to this commit, and all fine from my side. Please notify us when you have finished making Travis pass and when this PR will be in a needs-review state again.


I took your advise of #13 and made updates to translation for chat history


lol. Thank you.


ditto


        },
        ms_on_add_order: function (current_order) {
            is_empty = current_order && current_order.is_empty()
            is_frozen = ! current_order && current_order.ms_replace_empty_order
            if (this.config.multi_session_replace_empty_order && is_empty && ! is_frozen) {
                current_order.destroy({'reason': 'abandon'})
            } else if (is_frozen || !is_empty || !this.config.multi_session_deactivate_empty_order) {
                // keep current_order active
                this.set('selectedOrder', current_order);
            }
        },


teste


BTW, if we are going have an explicit list of arguments to be supported, the saving of those args can go through the same list. It'd be interesting to experiment if it's possible to:

* save those arguments in text format, as it's done with urls and config
* save only the supported arguments


а вот я сомневаюсь, что это будет работать... по крайней мере - не всегда :)


@kartheek3011 HTML id attribute should not contain white spaces.


good job


This looks great thanks


From the previous comment, with better location context: It'd be interesting to experiment if it's possible to:

* save those arguments in text format, as it's done with urls and config
* save only the supported arguments


@kartheek3011 audio and video files are missing.


To be honest, I find https://github.com/hairyhum/pics/blob/master/new.png to be difficult enough to reason about that I cannot come up with new names for at least several states and transitions.

Some suggestions for those that I believe I understand:

 * `blocked_flow` => `blocked_on_internal_flow`
 *

Also, please submit a PR so that the whole idea can be discussed. This discussion does not belong to #499 and I am not convinced this refactoring is actually improving `rabbit_reader`. It seems to put together two simple (and reasonably separate) state machines into one that's pretty complicated.


Are you saying it should be relaxed? Probably should be, yes. Otherwise, I don't follow :)

Haystack switched from one to the other in version 2.2.


Why do we need this temporary variable?


NOSETESTS_PATH should be just NOSETESTS.


+1 Made my day today.


ok

Thanks & Regards,
Deepu

On Fri, Feb 26, 2016 at 7:10 PM, Julien Dubois <notifications@github.com>
wrote:

> Oh sorry I didn't understood you wanted to improve this yourself -> yes,
> please go ahead!!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jhipster/generator-jhipster/commit/11265ac3cb77483e5b4b5efacb6ee0d555202a99#commitcomment-16349745>
> .
>


這個判斷應該是多餘的，這會導致 memory leak。


Theoretically, yes, there should be a failsafe for this loop. And I should
also add one for the player selection in Bounty mode, where I took this from.

However practically, there should never be an endless loop as at least the
player calling this (the host in this case) will be playing. Soon as the
reactor countdown begins, all item respawning will deactivate.
So I do not see a case where this can happen but I'll ass a counter that will
bail as soon as we spend too many iterations in this loop (N_players*2 or
something should be enough I guess).

On Thursday, 25 February 2016 18:54:33 CET vLKp wrote:
> This could be a `do { } while` loop.
>
> Some places in the code suggest that there might be a way for all players
> not to be `CONNECT_PLAYING`.  Can that happen?  If so, this loop would hang
> forever.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/dxx-rebirth/dxx-rebirth/commit/901a554e96946fec1bd5e926a3
> 957ce600da9023#commitcomment-16343593


It should at least go to 1800mA I think, I didn't really found any HW limitation.But who knows...
I tried several approaches and didn't get past the 1500mA, maybe I missed something idk.


@nfeske: some more corrections


una forma interesante de evitar este encadenado de if, o un posible switch case es un map.

public class DBConnectionFactory extends AbstractDBConnectionFactory {

    private Map<String, AbstractDBConnectionFactory> ConcreteFactories = new HashMap<>();

    public  DBConnectionFactory () {
        ConcreteFactories.put("MySqlConnection", new MySqlConnection());
        ConcreteFactories.put("SQLServerConnection", new SQLServerConnection());
        ConcreteFactories.put("SQLiteConnection", new SQLiteConnection());
    }
    @Override
    public SqlConnection getConnection(String conType)throws IllegalFormatException {
        return ConcreteFactories.get(conType);
    }


Hi Boris, when looking at the code changes in this commit i had the suspicion the VFAS voltage cell is only changed for the X4R SmartPort Telemetry protocol. Turns out that is true. Just build the sourcecode and installed it on my naze. Since i'm using a D4R, the VFAS voltage is still displaying the battery voltage on my Taranis. It there a reason why you only changed the SmartPort Telemetry and not the FrSky Telemetry?


#FILEDATE toegevoegd in 'elif statement' voor het geval #STARTDATE niet voorkomt.
#PROJECTID opgeknipt in drie functies:
- Get_ProjectID_Name
- Get_ProjectID_Code
- Get_ProjectID_SubCode


Was going for consistency, but you're right, they really shouldn't be listed as dependencies. And I should probably move grunt to devDependencies also, so I'll do that next.


@kartheek3011 form action value will be a HTTP get or POST paths. Read more on this https://developer.mozilla.org/en/docs/Web/HTML/Element/form


Regarding the Travis config:

- `--ignore-platorm-reqs` existed to allow testing against php 7 before composer supported the `||`notation for dependency constraints. IIRC, it's also needed to allow installation on HHVM.
- forcing the version is done to allow us to test against both v2 and v3 releases. I tried using the "prefer lowest" flag, but ran into issues with the test tooling whereby composer was unable to resolve all dependencies. The approach used here works reliably.

Regarding your statement:

> Reverting this function to the one from 2.6 solves it

Can you explain what "this function" refers to? Where are you seeing an error, exactly?


Oh, this FIXME is preexisting, from `trans_operand_into`.


Well, I will try as well a bit more in detail in the next weeks.
Just ordered an USB amperemeter to have proper data...


Esto puedes er una interfaz


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


shouldn't we also test that the queue is in fact resumed and working as expected?


Please make sure to explicitly close conditionals, like:
```
if(CONDITION)
    …
endif(CONDITION)
```
That makes nested conditionals easier to understand and/or to squash out problems.

Everything else LGTM!  =)


and you can use it here as COMMAND ${NOSETESTS}.  That will make sure nosetests is call with its full path, just in case it's not in system's default $PATH…


What value do we get of using this function over the well-named constant `CLEAR_SCREEN`?


Non non !!!


Gotta love those easy fixes hehe


What about
```channelSession.flush();```, ```session.clearReadBuffer();```, ```channelSession.readable();```
calling outside try block. How i may understand this calling this is user scope task or not?


Si no tenes abstract methods, salvo por l aoportunidad de instancair, esta clase no se diferencia de una común.


Decoding done :)

Tonight fq_codel support will be finished, probably codel also.


Aktuell funktioniert alles bis auf das entpacken.

Userplugin ist natürlich sauber

1. Versuch
Ich habe dein stable genommen und den Ordner module in meinen verschoben ->
Error importing ExtractArchive: cannot import name Expose

2. Habe dein Branch genommen
Geht auch nicht und auch der Crypter fehlt dort noch

3. Bin wieder auf deinem stable (NUR module)

4. Deine Idee


Thanks!


remove


Right, I understand the point. `map[string]interface{}` wasn't a real well-thought suggestion, it was there just to compose the argument of the my first comment.

I understand that having proper well-defined types makes the library better and is the standard -- and it really should be! --, but perhaps considering that this is not much of a library, but more of a thin wrapper around the Trello API, and considering that people using this will surely need to know about the Trello API (for example, the documentation for the methos of this library don't explain anything, it just shows links to Trello API documentation), perhaps we don't need all this (again, considering the amount of work and hugeness of the code base that well-defined types will require), perhaps, then `map[string]string` (you'll object to this also, I think, but hey, we don't even need to check the existence of keys or anything like it, just  serialize to JSON)?

Converting a `map[string]string` to JSON is worse than converting a struct, I know, but I'm talking about a trade-off here. Does that make sense?


https://github.com/svinota/pyroute2/blob/codel/pyroute2/netlink/rtnl/tcmsg.py#L574

It's a bit tricky, but works somehow.


Not related to the accept header, and we don't need to re-fetch if we already fetched the resource. But, we may need to fetch if the previous `type` was an unsupported type, which made us avoid preloading.

So we would need different language to handle type here, but we need to add such handling, as `type` changes can impact loading.


한국어 추가. 이 Branch의 앱 실행은 테스트중


That's a pity, I was hoping I was wrong :( It would be perfect otherwise. I still use shared projects, and I still put the mgcb physically into the shared folder, and then I add it (as a link to existing item) to each platform-specific project in VS.


Well I thought about that too, but this is more flexible in case we extend the whitelist.

Plane text would be nice, but we store processed params. So the only way I can think of is to iterate through `key:values` and store per line `key: pickle.dumps(value)`. That way you see and theoretically alter the values easier than when you store everything. The downside would be speed and I don't think it's worth it.

What could be improved is IMO the multiplexer. I'd be nice to create a yaml file out of the tree. That way it'd be possible to run remote tests using the stored multiplexer...


This is an ancient version, which doesnt even get built.
I have backported zcache from 3.6 tree, the zcache-v2 from 3.11 tree requires both cleancache and frontswap, which is tough to port.
I can create a PR for zcache if you want,.


Ooooh,that's coool!


@kiljacken Agree on that point, I can't not hear it as "datas" the s doesn't say structures to me. Imo collections is the best option, and with tooling it should be fine, i.e. auto importing or the editor understands the module layout with an auto completer, etc.

Another idea, just throwing it out there; ADT for abstract data types?


Potrebujem tam spracovavat iba tie spravy co su pritomne pre dany format. Bez toho aby notifier musel vediet o nejakych formatoch. Spracuje vzdy iba hash ktory moze mat rozne kombinacie znamych klucov.  Takze to nepokladam za premature abstraction. Nemoze byt tak ako to bolo ak nechces vidiet pri niektorych formatoch zbytocne "Nepodarilo sa extrahovat." spravy.

Ako inak by si to filtroval?


Hmm, doesn't look like that for me. Let's see if next commit changes anything...


コメントに対してコメントができる。


Don't seem to be used?


これはピヨピヨしてるね～とか言うのか


Yup that is what I used to do.. I was trying (desperately) to make it
easier for our users.  In XS this works like a dream and its soooooo easy.

On 26 February 2016 at 11:56, Tim Jones <notifications@github.com> wrote:

> That's a pity, I was hoping I was wrong :( It would be perfect otherwise.
> I still use shared projects, and I still put the mgcb physically into the
> shared folder, and then I add it (as a link to existing item) to each
> platform-specific project in VS.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mono/MonoGame/commit/1d4ff0cef576273e4342fb86001e21dcd27a174e#commitcomment-16350536>
> .
>


Probably the first time in a while where the XS experience is better than
VS :P

On 26 February 2016 at 12:00, Dean Ellis <dellis1972@googlemail.com> wrote:

> Yup that is what I used to do.. I was trying (desperately) to make it
> easier for our users.  In XS this works like a dream and its soooooo easy.
>
> On 26 February 2016 at 11:56, Tim Jones <notifications@github.com> wrote:
>
>> That's a pity, I was hoping I was wrong :( It would be perfect otherwise.
>> I still use shared projects, and I still put the mgcb physically into the
>> shared folder, and then I add it (as a link to existing item) to each
>> platform-specific project in VS.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/mono/MonoGame/commit/1d4ff0cef576273e4342fb86001e21dcd27a174e#commitcomment-16350536>
>> .
>>
>
>


but I'm confused where is TestRunner.elm


This only applies to the pollinate command which we already pass the --insecure (-k)


What is OpenStack? OpenStack is a cloud operating system that controls large pools of compute, storage, and networking resources throughout a datacenter, all managed through a dashboard that gives administrators control while empowering their users to provision resources through a web interface.


Are there any plans to include this soundfont into other release packages? Would be really handy to have this soundfont included in the Windows installer or .zip package - great improvement in MIDI quality compared to the quite crappy Microsoft MIDI synthesizer.


Esto no anda bien.
Pegale una mirada a etse builder: https://github.com/juanmf/java-bootcamp-2016/pull/1/files

lo grave:
  ++ tu Clase a construir no tiene los atrivutos que se supone que el builder setea. en el cnostructor deberia setear los atributos con los pares en builder
  + definis un atributo de instancia en la clase, para el builder, paradoja. Porque el objeto a construis debe existir antes que el builder para que funcione, no tiene sentido.
  + la referencia al builder no se mantiene, después del constructor ya fue.
Lo no grave:
  el builder queda mejor como static inner class. asi hacer new obj.builder().


Ten hlavny parser je spolu s detail parserom dohromady uz moc velky. Nedaval by som to vsetko na jednu kopu.  Ak ti nesedi jednorazovy objekt, tak to mozem prerobit na staticke metody detail parsera.


To jest poprawna zmiana


The thing is that the constructor of PdfCleanup is not used in the tests... (Unless both me and my search function are blind.)


vm.change with a base64 value also failed with 0.3.0 because of  '==' . Can someone please build a new release?


Should be a def.  Also why is it an Option when it seems Errors always need a reason?


El biulder debería aceptar parametros, a lo sumo definir defaults. este esta hardcodeado.


Should be a def. Also why is it an Option when it seems Errors always need a reason?


Idk, I just copy paste them from wikipedia :p


otra paradoja, espera recibir ins instancia del objeto a  construir. esta se crea al final. Cuando ya te setearon todos los avlors obligatorios al menos.


As these are never constructed with a different reason, they should be case objects, defined like this:

```
  case object VersionsRetrievalFailed extends HandshakeError {
    override val reason: String = "Failed versions retrieval."
  }
```


Good


Mira tu builder.
Lo demás son observaciones de mejoras.


Caused by: MediaException: MEDIA_UNAVAILABLE : c:\Users\Dubs\Documents\
NetBeansProjects\RocketDog\src\splashReduced.mp4 (The system cannot find
the path specified)

Change the path into your absolute path to RocketDog src folder. The Media
constructor uses URI syntax instead of the easier one that Image uses so it
is still set up to serve that file out of my computer. We'll have to put it
online somewhere or something so that it has one universal uri

On Fri, Feb 26, 2016 at 2:09 AM, nsefcik <notifications@github.com> wrote:

> I got this when trying to run the program (any ideas what happened?):
>
> Executing
> C:\Users\Noah\Desktop\RocketDog-AddCutScene\dist\run279851548\RocketDog.jar
> using platform C:\Program Files\Java\jdk1.8.0_45\jre/bin/java
> Exception in Application init method
> java.lang.reflect.InvocationTargetException
> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> at
> sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
> at
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> at java.lang.reflect.Method.invoke(Method.java:497)
> at
> com.sun.javafx.application.LauncherImpl.launchApplicationWithArgs(LauncherImpl.java:389)
> at
> com.sun.javafx.application.LauncherImpl.launchApplication(LauncherImpl.java:328)
> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> at
> sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
> at
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> at java.lang.reflect.Method.invoke(Method.java:497)
> at sun.launcher.LauncherHelper$FXHelper.main(LauncherHelper.java:767)
> Caused by: java.lang.RuntimeException: Exception in Application init method
> at
> com.sun.javafx.application.LauncherImpl.launchApplication1(LauncherImpl.java:912)
> at
> com.sun.javafx.application.LauncherImpl.lambda$launchApplication$152(LauncherImpl.java:182)
> at
> com.sun.javafx.application.LauncherImpl$$Lambda$50/1343441044.run(Unknown
> Source)
> at java.lang.Thread.run(Thread.java:745)
> Caused by: MediaException: MEDIA_UNAVAILABLE :
> c:\Users\Dubs\Documents\NetBeansProjects\RocketDog\src\splashReduced.mp4
> (The system cannot find the path specified)
> at javafx.scene.media.Media.(Media.java:405)
> at edu.uco.sdd.rocketdog.model.CutSceneIntro.(CutSceneIntro.java:34)
> at edu.uco.sdd.rocketdog.model.SceneFactory.getLevel(SceneFactory.java:32)
> at
> edu.uco.sdd.rocketdog.controller.RocketDogGame.init(RocketDogGame.java:55)
> at
> com.sun.javafx.application.LauncherImpl.launchApplication1(LauncherImpl.java:841)
> ... 3 more
> Exception running application
> edu.uco.sdd.rocketdog.controller.RocketDogGame
> Java Result: 1
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sharris40/RocketDog/commit/675cbfc9d3ac7fb0b4f12f08d0ae972fba9d2269#commitcomment-16346604>
> .
>


Mais dis-donc, qu’est-ce que c’est que ça ? :smiley:


ah indeed, sorry


@carlocamilloni do you know if this always works? If so, I would backport it to v2.2


У ЧтениеТекста есть Получить(), поэтому этот цикл можно заменить на "Рез = Текст.Прочитать()")


2c... Be careful for the getField hashing, you should measure things on ARM
because I suspect a switch( and subnested ifs ) may cause much less
caches/branch misses than cascading ifs on the "average case" but only
measurement will prove it...

2016-02-26 12:44 GMT+01:00 Hugh Sanderson <notifications@github.com>:

> I am taking a purely pragmatic approach. I am basically moving the logic
> from the codegen to the cppast. Along these lines, I will probably end up
> with something exactly like what simon suggested with (int list * tcppexpr)
> cases. As to whether it is better to represent the non-int cases as a
> CppIfs or a CppNonIntSwitch, I think it depends on if I need to do any
> reasoning about the call chain (in which case cppif is probably better), or
> just dump it out. If I only need to dump it out, it comes down to a
> question of practicality - will the generated c++ code be nicer, and will
> to ocaml code be nicer.
>
> I too use the neko hash function. The "getField" code currently uses a c++
> switch in length, and then a series of "if" statements.
> But what I was thinking, is instead of switching on the length, do a
> binary search on the hash, like:
>
> var tmp = String( whatever);
> var hash = tmp.getHash();
> var found = true;
> if (hash<0x8000000)
> {
>     if (hash<0x300400)
>     {
>          if (tmp=="value1")
>              doValue1()
>          else if (tmp=="value2")
>             doValue2()
>         else if (tmp=="value3")
>             doValue3()
>         else
>             found = false;
>     }
> }
> else
> {
>     if (hash<0xa000101)
>     {
>     ...
>     }
>     else
>     {
>     ...
>    }
> }
>
> if (!found)
>    doDefault()
>
> Calculating the hash thresholds optimally to create groups of 2 or 3.
> Cpp string constants have the hash code built-in, so this should be fast.
>
> In the case of multiple case values, these could be split, and the code
> duplicated, or they could be done separately first.
> Floats and ints should also work this well this way.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/HaxeFoundation/haxe/commit/9d361c6d14e1f6c004a4f9de0a4b8768afcc40bd#commitcomment-16350336>
> .
>

--
David Elahee


NotVersionsRegistered doesn't make sense to me


Why is the commit only added to 2.0.3 and not master?  If a user upgrades from 2.0.2 to a version greater than 2.0.3 (i.e. skipped over 2.0.3), then would they encounter this compatibility code?

I'm verifying that I've compiled 2.0.3 with this commit on by my win 8.1 x86-64 and ArchLinuxARMv7, and the upgrade works as expected:  If I start the new mscore & close it right away, then then .ini gets the sfPath line removed.  If I start again and modify preferences, then the .ini gets a "mySoundfontsPath" line.


Cim je velky? Ja prave mam radsej nech vsetko parsovanie je na jednom mieste.


**[unitTests_js_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_js_branches/25438/testReport/) for this commit**


@MovingtoMars Why in particular?


closes Factisio/factis-store-hexastore/issues/2


There will be no automatic upgrade path from 2.0.X to master. They will write the preferences file in a different path.


I don't know much about linux, but if all distros define `unix` and/or `__unix` as BSD forks do, we can get rid of that `defined(linux) || defined(__linux)` and just unify to `defined(unix) || defined(__unix)`.


Alebo to moze byt aj modul.


this branch is used for web services that is used in the app.


I understand that but if someone goes from 2.0.2 to 2.0.4, then will they encounter this code?


Both are in use, one is clearly dominant. I'd prefer we stick with that most people use.


Was this meant to be changed?


Or figure out how to serve it from any local computer

[https://docs.oracle.com/javase/8/docs/api/java/net/URI.html]


Reported at: https://github.com/embeddedarm/linux-3.14-pxa16x/issues/1
Reported at: https://github.com/embeddedarm/u-boot/issues/2


autor: AbcChanelMC


Oh...I think I know.  Because 2.0.4 (or whatever comes after 2.0.3) will be branched from 2.0.3....


vsetky tieto funkcie by podla mna mali mat prefix `parse_` hovoria lepsie o tom co robia.


if a 2.0.4 happens, it will stem from the 2.0.3 branch, just like the the 203 branch stems from the 2.0.2 one.


Thanks!!!


The method would return null or empty string as configured for the password field.

It is probably to allow for passing only a username to the URL, however I think there is an implementation error it the code will still attempt to resolve the user's password. It probably should be Validator.isNotNull(password) and the "if (password == null)" should be moved out of the block. . In any case, it is highly unlikely that it will be null because of the following code preventing this method from being called if it is.

https://github.com/stian-sigvartsen/liferay-portal/blob/aa52d27cd1d00d51d71382da32c27d19eeee89fe/modules/apps/iframe/iframe-web/src/main/java/com/liferay/iframe/web/display/context/IFrameDisplayContext.java#L219


Experimentalne odobratie sa zjavne neujalo :). Vratim tie prefixy.


cpu_, gpu_ 要就是 (1) 都是 nullptr (2) 都是某個記憶體區塊
第一個 assertion 就是在保證這件事
所以這個應該沒有 memory leak
不過如你所說，delete/cudaFree 丟入 nullptr 的確是安全行為
這個判斷的確是不必要的


@juliabackports this and 4ec891892b591f38c564080d221d2bc910834627


`if (empty($_GET['skip_checks'])) $_GET['skip_checks'] = '';`


@nhpatt @topolik articleId is missing in this check


Oh, now I got it...


Another regex casualty?


Hopefully fixed in 99ef7f8143549f798c94d27715459b2ab1d283eb. Sorry for the mix-up


oopss....


Nevermind, dam to na hromadu.


Yup. Please see my fixes in later commits. Thx


adt is good with me, but i fear it might not appease Liam, as ds didn't
work for him

On Fri, 26 Feb 2016 at 12:59 Felix Angell <notifications@github.com> wrote:

> @kiljacken <https://github.com/kiljacken> Agree on that point, I can't
> not hear it as "datas" the s doesn't say structures to me. Imo collections
> is the best option, and with tooling it should be fine, i.e. auto importing
> or the editor understands the module layout with an auto completer, etc.
>
> Another idea, just throwing it out there; ADT for abstract data types?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ark-lang/ark/commit/2952a0935545829dad9402a08b20cfe1f34894e0#commitcomment-16350578>
> .
>


**[unitTests_php_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_branches/28128/testReport/) for this commit**


copy paste error?!


hacky


**[unitTests_php_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_branches/28129/testReport/) for this commit**


@kiljacken I dunno, I think ds is too short, ADT is a meaningful, well known acronym that is just right imo.


Brady do we have any thing related to smarty implementation in library/smarty.class.php or steps?


man, we should try to :skull: `SupplyPointCase`


The treeset could be built outside the Metadata object. There is no need to modify this class.


Those rogue regex


Great! Hadn't thought of this yet.


@mhgoldman nicely done with removing the id dependencies from the tests :+1:


were these just not necessary? or what replaces this logic?


Why serializing to JSON? The API does not accept JSON but only `application/x-www-form-urlencoded`… I don't see any advantage of a map against a well written library / wrapper… Sure, you might spare some minutes not writing a struct with defined parameters but imho that's only technical dept… You will need to keep the documentation open all the time you're using the library looking up which parameters you can pass in. With a well defined interface you have the property names to see what you can pass in and what type the value has. (Sure with changing types like `float32` and `string` for position that's hard too but thats the implementation of the API which we can't change)


I was not able to access the md object from outside the Metadata class thats why I did it here


back


This is independent from the move to 2.x please base this on the master branch and not the one you used for 2.x
Also could you please format the code before sending a PR? It would make it easier to review your contribution. Simply do `mvn java-formatter:format`. Thanks


conceptually it seems like we should avoid calling `cs_buf_data_data()` if not in cs_mode so think this check can instead move up into `stream_have_data__unlocked()` to avoid treating a non-existent CS buffer as being 'empty'.


@jnioche ok, will do so soon


Approved.


Yeah, this works great for me. Thank you.

On the synchronization issue, note that `return sync.await(properties.parse(source, sync.defer()));` will also make this synchronous (and therefore will appear be easier to call for callers), but is safer because it won't modify global objects. The `sync(properties, 'parse')`-version can fail for many reasons. (For example, if `properties` module desides to call `Object.freeze` on the exposed object or if some other module makes some other similar modifications on the `properties`-module.)

So although `sync(properties, 'parse')` will make the code here slightly simpler, it will increase coupling between modules, complicate the global semantics and make programs a bit more fragile, thus increasing global complexity.

But as I said, this is a sidenote and not really relevant. Thanks again for a prompt fix!


Once, twice, three times a lady


Coordinate typo seen here.


By that you are changing
"3 - sweatshirts" to "1 - set of sweatshirts".
To avoid dispute I'd favor for
"1 - set of 3 sweatshirts"

Something like:
`$_[ 'item_name_' . $id ] = Shopp::__('Set of %d %s', $Item->quantity, $_[ 'item_name_' . $id ]);`


you could use `mapping.map(...).orElseThrow(...)`


Oh I see - I actually removed the tests from this project as I didn't write any, and forgot to remove the `test` task from the Gulpfile.


aaa


Boom


just call the keyset method on Metadata
[https://github.com/w0mbat/storm-crawler/blob/b0e1d6a101dd9c0978eddc364fc2e07be3004dcd/core/src/main/java/com/digitalpebble/storm/crawler/Metadata.java#L161]


adding new files to the repository


Hi,
du brauchst noch das Modul `html5lib`. Du kannst es über
```
pip install html5lib
```
nachinstallieren, dann sollte es laufen.


shouldn't you break execution here and notify that configuration is broken?


The current version uses a whitelist. You will need to add nbd to it.


Zacinam sa chytat, v tom pripade by nestacilo nieco ako toto?

```ruby
DETAIL_MESSAGES = {
  amount: '**Cena:** %s',
  procurement_type: '**Druh postupu:** %s',
  project_runtime: '**Trvanie projektu:** %s',
  offer_placing_term: '**Lehota na predkladanie ponúk:** %s',
  procurement_winner: "**Víťaz obstarávania:**  \n %s"
}

def compile_detail_messages(details)
   details.each_with_object({}) do |(type, value), messages|
     messages[type] = DETAIL_MESSAGES[type] % (value || 'Nepodarilo sa extrahovať')
   end
end
```


notably being the implementation of `i915_oa_can_read()` (with the only difference being that it can run without the perf lock) it could be good if the name better matched `i915_oa_can_read()` - maybe `oa_can_read__unlocked()` (skipping the 'i915_' prefix since it's not a method implementation nor exported outside i915_perf.c)


Ack.


I changed it to SUCCESSFUL instead of SUCCESS because of this already existing definition of SUCCESS which caused me problems when trying to compile (as i was cleaning up the double includes which existed at that time):

QEMU/hw/nvme.h:#define SUCCESS 0x0


@giovannibussi

as far as I can see it works, and also in principle I don’t see why it shouldn't.
I have tested on three different cases, plus using driver and sum_hills and it pass
travis and the regtest locally.

> On 26 Feb 2016, at 13:07, Giovanni <notifications@github.com> wrote:
>
> @carlocamilloni <https://github.com/carlocamilloni> do you know if this always works? If so, I would backport it to v2.2
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/plumed/plumed2/commit/1aa68bd2727de89bedf79bf85d4a777cea596c05#commitcomment-16350718>.
>


Test


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `bears/java/CheckstyleBear.py`, line 21.*

The issue can be fixed by applying the following patch:

```diff
--- a/bears/java/CheckstyleBear.py
+++ b/bears/java/CheckstyleBear.py
@@ -18,7 +18,8 @@
         "WARN": RESULT_SEVERITY.NORMAL}

     prerequisites_args_list = ["java", "-jar", jar, "-v"]
-    prerequisites_fail_msg = "jar file {} is invalid and cannot be used".format(jar)
+    prerequisites_fail_msg = "jar file {} is invalid and cannot be used".format(
+        jar)
     check_prerequisites = classmethod(CheckPrerequisites.check_prerequisites)

     def run(self, filename, file):
```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `bears/CheckPrerequisites.py`, line 2.*

The issue can be fixed by applying the following patch:

```diff
--- a/bears/CheckPrerequisites.py
+++ b/bears/CheckPrerequisites.py
@@ -1,5 +1,6 @@
 import shutil
 import subprocess
+

 def check_prerequisites(cls):
     if shutil.which(cls.executable) is None:
```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `bears/scala/ScalaLintBear.py`, line 9.*

The issue can be fixed by applying the following patch:

```diff
--- a/bears/scala/ScalaLintBear.py
+++ b/bears/scala/ScalaLintBear.py
@@ -7,6 +7,7 @@
 from coalib.misc.Shell import escape_path_argument

 import bears.CheckPrerequisites as CheckPrerequisites
+

 class ScalaLintBear(LocalBear, Lint):
     executable = 'java'
```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `bears/scala/ScalaLintBear.py`, line 30.*

The issue can be fixed by applying the following patch:

```diff
--- a/bears/scala/ScalaLintBear.py
+++ b/bears/scala/ScalaLintBear.py
@@ -27,7 +27,8 @@
         "java", "-jar", jar, "-c",
         scalastyle_config_file, ".", "-q", "true"
     ]
-    prerequisites_fail_msg = "jar file {} is invalid and cannot be used".format(jar)
+    prerequisites_fail_msg = "jar file {} is invalid and cannot be used".format(
+        jar)
     check_prerequisites = classmethod(CheckPrerequisites.check_prerequisites)

     def run(self,
```


**Line is longer than allowed. (84 > 80)**
*LineLengthBear, severity 1, section `linelength`.* *Related to file `bears/java/CheckstyleBear.py`, line 21 column 81 to line 21 column 85*.


**Line is longer than allowed. (84 > 80)**
*LineLengthBear, severity 1, section `linelength`.* *Related to file `bears/scala/ScalaLintBear.py`, line 30 column 81 to line 30 column 85*.


```
alert('我叫Rachel');
```


why aren't we checking jobEnvironment.variables[maskHint.value] in the second check?


maybe a switch() would be better self documenting so it's clear here that the last else corresponds to `CS_BUF_HAS_DATA`


Had now idea you could write methods in-line like that.  Thanks for the education :-)


@czechboy0 Nice catch! It's not, it should be just one.


yes, we know that, but we don't want to change the name of the default profile; I think this could be fixed if you hide the base profile from `QuickInstaller`.


The reason to have a local version, arrays in parameters, was fixed. But a new reason is coming, galleries are not implemented in python-500px yet.


@ace2nutzer , that's weird. In my case games are  still lagging with stock LCDCLK (seems that we have to fix in kernel), but with 57 MHz I haven't noticed any tearings.
Btw, reading HWComposer code from that repository I found that some devices have SW Vsync (like in our case), but some use HW one as well. For now I forcibly enabled SW Vsync here: https://github.com/ChronoMonochrome/android_hardware_u8500/commit/f39cc6d266942ff1e97b3eeaf0288fd98c5bf303#diff-032b12c46ce5f3b5b012bbaa1a965168R1412 because it seems that we don't support HW Vsync.  Do you have any ideas, where HW vsync has to be enabled? I guess it's somewhere in kernel, but so far I haven't found any specific changes to do that.


buen trabajo!


Hmm that's wrong, fixing it and adding more tests.


Hello,
I thinkt it was not very good to set the console readonly. There is no option present to clear the console window.
Please think about this.

Thanks Rene


@dxli hypot(hypot()) will call twice the sqrt() ?


I've seen how their carat/exp method works. They simply push a button labeled "carat" it instantly sends them to zombie survival mode and their character install suicides and dies and they gain carat coupons I believe or something on the lines of that or they gain direct carats I've never seen the final outcome but I do know the way it works. @neetVeritas


You are public figure, Jack, please be careful :smiling_imp: :smile: