onfocus="if(this.value == 'Cette tache a pas encore de nom'){this.value=''};"
onBlur="if(this.value == ''){this.value='Cette tache a pas encore de nom';}"
 onclick="if(this.value=='Cette tache a pas encore de nom'){this.value=''};"

On fait pas ca. Ca fonctionne mais c'est pas une bonne pratique car c'est du code Javascript mi dans du code HTML. Autrement du fonctionnel mi dans de la vue. La vue doit se charger uniquement de la presentation, si tu veux faire quelque chose au onFocus, tu dois faire un event sur le template dans le code Js (dossier helpers-events).

Et puis ce que tu voulais faire s'appelle un placeHolder, ca existe pas en IE8 mais on s'en fout.


Phpdoc missed


Do you have any other word?

Sent from Outlook Mobile

On Mon, Nov 30, 2015 at 2:01 PM -0800, "Wilson Chen" <notifications@github.com> wrote:

nice work, mate

---
Reply to this email directly or view it on GitHub:
https://github.com/unruledboy/WebFrontEndStack/commit/5f152c1dacab26460644e695f1a5c851ebcd7286#commitcomment-14694520


just `The (case insensitive) programming language` would've sufficed for this five-line-monster...


Phpdoc and empy lines


Il n'y a pas cette notion dans le cahier des charges


On verra ca plus tard


Phpdocs


The terminology is a bit confused about it. TransactionIdStore uses the term `closed`, although `Commitment` uses the term `applied`. This is probably a good time to unify this so that there's only one term.


on verra ca plus tard


I know it's not strictly necessary, but might it be worth explicitly returning False at the bottom of this? I found the patch method a bit confusing at first (had to refer back to the existing one) because of the different implications of returning True, False and a string value.


It would be nice to have those defs in proper header, e.g. include/spmi.h, and the core addresses can go to some mach header.


> Your benchmark probably don't show the real impact of this change; the impact should worsen with depth of hierarchies.

Yes, the benchmarks use just *some* extreme and artificial hierarchies to make relative changes in performance visible. Their absolute numbers should not be misinterpreted as representative for realistic (more complex) stylesheets and view hierarchies.

> I don't understand your comment about "don't change to iterator based approach". You did change to an iterator based approach, didn't you?

To avoid the copy of `UiItemPath` when calling `effectivePropertyMap`, its signature could be changed to `PropertyMap effectivePropertyMap(UiItemPath::iterator begin,  UiItemPath::iterator end, StyleEngine& engine)`. We don't do this here because we would have to roll it back when sharing StyleSets. Do you want me to state this clearer in the commit message?


J'aurai du le laisser en fait...


True I forgot about that, and the docs in https://github.com/plone/sphinx.themes.plone are not up to date about the integration about the version switcher .... just saying ...

I did a revert, but IMHO that means the we have to find another way to fix that, because the current setup is confusing too !


unack, could you please try to do that in general? Large comments are just bad to maintain  and a lot more to read.


@bryk: Interesting. It seems that deploy module was missing ngResource dependency after all and when it's fixed then we can just mock deploy module with 1 line.


Yep, will do!


Why you call for each item drupal_get_path, do it once before loop


why does it work here with the comments :)


Since this is effectively opening up an api, might it be worth adding some docs about the different possible return values to the class' `patch_boundary_line` method?


Teste


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


Lorsqu'on peut, on evite de mettre des buttons+onClick. Typiquement ici il ne s'agit que d'une redirection alors on passe par un href. Ca permet d'alleger le code fonctionnel JS et d'ameliorer la lisibilité du code vue HTML


Phpdocs


I would like to not deliver this failure to my Activities/Fragments. Is there a way to ignore the wohle result if its canceled? Could i catch this exception somewhere maybe?


Good catch, will do

On Tue, Dec 1, 2015 at 10:41 AM, Alex Ley <notifications@github.com> wrote:

> Can you update the docs to reflect this please?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pivotal-cf/brokerapi/commit/724bdb1cef1795fdda005e1277b364694047c5ce#commitcomment-14705024>
> .
>


Known bugs:
20 sec countdown still counting after win()
canvas got wrong position after win()


Yes, that's why is is good to have tests :) To catch such small mistakes.


Mooie commit!


It wasn't included before and I didn't want to change that. Do you want me to include it?


Tu noteras encore ici l'ajout de {{description}} pour indiquer au template que ici, il faut ajouter le contenu de la variable description de la tache.

Tu piges ca ? Tu comprends pourquoi rien ne s'affichait jamais en mode edition ?


hi


TeamCity Habitat :: Cruise [Build 303](http://build8ua1:8081/viewLog.html?buildId=762&buildTypeId=Habitat_Cruise) outcome was **SUCCESS**
Summary: Tests passed: 82 Build time: 00:01:13


Waarom heb je dit gedaan?


Indeed! Thanks!


@lislis you rock :heart_eyes_cat:


Ok, cool, then I'll remove the `--with-deps` option (just leaving `['-d', '--with-dependencies']`, and we'll say that whenever an option has something to do with dependencies, it is always "dependencies" and never "deps".


+1 go ahead with a change.


Thanks @ramsudharsan
I'm completely new to github and still trying to learn how things work here. I will try to read up and find more info on how to push to code review.


Do you mean the `qobject_cast`? Unfortunately it isn't: `qmlAttachedPropertiesObject` returns a `QObject*`...


I think those build failures seem to be with the nuget packaging stuff, and not my change?


выход очень простой, определить функцию onControlChange
и написать:
resizeX.onchange = onControlChange;
resizeY.onchange = onControlChange;
resizeSize.onchange = onControlChange;


D'oh.  Got it.  Thank you.  Sorry about that.  :(


MArray_Elim (DeadParrot) - i386-Windows-7-VisualStudio-12: OK (1981 of 1990 tests passed)

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


The idea is to have a Transform that lets you get the embedded Rotation or Position (with their own semantics) or even get directly the Transform semantics in one block (`TransformSemantics` class). But Transform should be the only class setting the semantics, unless we have no other choice. So `DynamicsComputations::getRelativeTransform()` could build the transform by the `refFrameIndex` and `frameIndex` to `ToiDynTree(...)`. I would handle the semantics in these `ToiDynTree(...)` conversion methods. this way, you will only have to do it for `Position ToiDynTree(const KDL::Vector& kdl_vector)` and `Rotation ToiDynTree(const KDL::Rotation& kdl_rotation)`: it will be transparent in `Transform ToiDynTree(const KDL::Frame& kdl_frame)`


Uploaded this version as 8.1-1.


You are refererring to the comment on lines 208/209, right? If we wanted to keep it, it should be rephrased to "emit the change notification, otherwise the QML world won't see it." But I don't think that this is necessary because this signal is not different to any other signal.


For testing only.


I am getting the following error:
An exception occurred during task execution. The full traceback is:
Traceback (most recent call last):
  File "/home/dev/.ansible/tmp/ansible-tmp-1448967232.17-139193569892154/lambda", line 2502, in <module>
    main()
  File "/home/dev/.ansible/tmp/ansible-tmp-1448967232.17-139193569892154/lambda", line 232, in main
    region=region, endpoint=ec2_url, **aws_connect_kwargs)
  File "/home/dev/.ansible/tmp/ansible-tmp-1448967232.17-139193569892154/lambda", line 2328, in boto3_conn
    client = boto3.session.Session(profile_name=profile).client(resource, region_name=region, endpoint_url=endpoint, **params)
  File "/usr/local/lib/python2.7/dist-packages/boto3/session.py", line 200, in client
    aws_session_token=aws_session_token, config=config)
  File "/usr/local/lib/python2.7/dist-packages/botocore/session.py", line 762, in create_client
    client_config=config, api_version=api_version)
  File "/usr/local/lib/python2.7/dist-packages/botocore/client.py", line 62, in create_client
    verify, credentials, scoped_config, client_config)
  File "/usr/local/lib/python2.7/dist-packages/botocore/client.py", line 270, in _get_client_args
    timeout=(new_config.connect_timeout, new_config.read_timeout))
  File "/usr/local/lib/python2.7/dist-packages/botocore/endpoint.py", line 259, in create_endpoint
    raise ValueError("Invalid endpoint: %s" % final_endpoint_url)
ValueError: Invalid endpoint: us-east-1

fatal: [localhost -> localhost]: FAILED! => {"changed": false, "failed": true, "invocation": {"module_args": {"handler": "runEc2UsingLambda.lambda_handler", "name": "ansibleTest", "path": "/home/dev/PythonLambdaDir2/runEc2UsingLambda.zip", "role_arn": "arn:aws:iam::533188012830:role/lambda_basic_execution", "runtime": "python2.7", "state": "present"}, "module_name": "lambda"}, "parsed": false}

calling to ansible as following: ansible-playbook site.yml -e realm=dev -e region=us-east-1 -vvv

site.yml lambda code:
- name: create lambda function
      local_action:
        module: lambda
        name: 'ansibleTest'
        state: present
        path: '/home/dev/PythonLambdaDir2/runEc2UsingLambda.zip'
        runtime: 'python2.7'
        role_arn: 'arn:aws:iam::533188012830:role/lambda_basic_execution'
        handler: 'runEc2UsingLambda.lambda_handler'

Let me know if you need more details.


No need to copy whole of my.cnf. "--exec echo "restart:--enforce_storage_engine=MyISAM" instead of restart_mysqld.inc should work, grep for "restart:--" for examples.


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


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


I think this requires further investigation. Would you mind opening an issue against that?


Some trailing commit I still needed to push. Note that this is the last Jena version we can support without having to change the rdfhdt library. Jena 3 introduces breaking package changes


Awesome work @nanoant! Keep up the good work.


I'd name this "percona_enforce_storage_engine_alter_table" or similar. Merging into existing percona_enforce_storage_engine is not worth it due to different server options IMHO


This was actually what I meant with issue #503. Perhaps just re-open that one?


There isn't stable/mitaka yet, you can use
http://tarballs.openstack.org/magnum/magnum-master.tar.gz
For formal builds based on tagged and released version you can use e.g. http://tarballs.openstack.org/magnum/magnum-1.1.0.tar.gz (looks like Magnum is not uploading releases to Launchpad https://launchpad.net/magnum)


Misschien wil Herman ooit nog een spegelreflex meenemen?
LGTM


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


That's the problem I ran into with eslint.


I aggree. `require` statements are a bit of leftovers here.


Yeah a movement grid is when you divide the game screen up into grid segments - the cell size depends on the game, but for an 800x600 game maybe something like 128x128 (so there would only be say 4x3 of them in the middle of the game). And then you can tab or auto-scan through each of the grid cells and select it, and it allows the player to move to that location. Sometimes the grid is visible (in which case it would be fine with this code change) and other times only the focus area is. Sort of like when you tab through an image map.


Add a check that the column has been in fact altered


Mm, great. Thanks.


Je pense que t'y a deja pense mais je te sugere aussi de crée un fonction qui converti tes parametes en tableau
genre
double param1[NB_PARAM];
PsoParam param2;
convert1To2(param1, param2);

Ca sera bien plus simple a manipuler pour le pso.


reset to this version


Weerd & Brinkkemper “Towards a reference framework for software product management”

This paper presents the reference framework for software product management. In the basic framework structure, the basic structure of the framework is based on the object. At the same time, it is based on the set of stakeholders defined in the scope of work. And then , there are some generic internal and external stakeholders. In next section, it explains the four process areas about the reference framework for software product management. Portfolio includes the decision making of existing products, through analysis the trends of the market, and product development strategy to introduce new products, decide the lifecycle of the product, and conduct partnerships and contracts. Roadmapping is a metaphor for planning and portraying the use of scientific and technological resources, elements and their structural relationships over a period of time. It is used for business oriented long-term planning and technology forecasting. Requirements management includes the activities of collection, define, revising and organizing the incoming requirements from various stakeholders. Requirements management starts from collect the requirements from the company and external stakeholders. Release planning starts from the priorities of the product requirements.  After the prioritisation, the selected requirements will be implemented in the next release.


Confusing:  "Maximum rate of sigops per byte" but the parameter is the minimum number of bytes per sigop.


@SthephanShinkufag ну как тебе? По-моему неплохо получилось.


Yes, it is. `setEngine` is still called from `onStyleEngineChanged`. `setEngine` and `mpEngine` will be removed on a later branch.


Geen idee :100:


This shot may help visualise it - in this example you can see the grid, so it passes this code anyway, but it's easier to understand this way :)

![image](https://cloud.githubusercontent.com/assets/164476/11499043/2d82646a-981b-11e5-9bb6-3e504d1155f4.png)


В итоге получилось достаточно хорошее приложение. Требования к разработке были выполнены.
В моём понимании, приложения такого рода должны быть максимально простыми в использовании, чтобы не тратить много времени. И здесь удалось это реализовать - при первом же использовании не возникло никаких вопросов и заминок. Особенно хочется отметить то, что приложение не имеет огромного количества окон (типичная ошибка подобных программ).
Двумя недочётами для меня стали:
1. Отсутствие функции удаления событий.
2. Отсутствие выделения текущего дня на таймлайне.
Для дальнейшего развития могу предложить, в первую очередь, устранить эти недочёты, а также улучшить таймлайн в сторону быстрого нахождения нужной даты (например, разделение по неделям с подписью даты начала недели).


Is there a reason you increased the restart intensity here?


# :exclamation:  **warning** :exclamation:

if you have **private plugins** that users should not use then remove this plugin from the `plugins` folder!


Is the removal of this file from the git repo intentional? If so, why?


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


I guess this is copied from openstack-nova.spec - it should be updated there and all its copies, it's rather confusing with too much internal details. I'd suggested to simply put
   # Let RPM handle the dependencies
like in, ahem, openstack-keystone.spec. Better wording is of course welcome!


Add tests that
- OPTIMIZE TABLE on an InnoDB table does not change its SE (it is mapped to ALTER TABLE internally)
- ALTER TABLE with more than one clause, one of which is ENGINE, works or fails as expected


With this commit Verify.call methods have all the older method capabilities.


As a side note: I initially wanted to protect the accessor `TransformSemantics & getSemantics()`, or only keep the `const TransformSemantics& Transform::getSemantics() const` and for some reason I didn't ...


TODO: Start cache mechanism from original dfa.cc?


Since the child process is, itself, a supervisor, shouldn't the shutdown time be set to `infinity` based on http://www.erlang.org/doc/man/supervisor.html#type-child_spec ?


@tymochko пробела между названием функции и ёе вызовом быть не должно, то есть так нужно: `prompt()`


"This are" ?


it can be done by setting the variable "release" to "4", instead of "4.3".
That still leaves question, though, should it be 'newest first', so 5, 4, 3
or 'numerical', so 3, 4, 5.

I would think 'newest first' is more logical, but open for discussion.


The code from this file can be divided into 3 parts:
- drivers/spmi/spmi-uclass.c - SPMI bus read/write with dm_scan_fdt_node() in .post_bind() - bus device
- drivers/power/pmic/pm8916.c - PMIC read/write and the keys code - bus child
- drivers/gpio/pm8916_gpio.c - gpio driver - child device - provide proper compatible - pmic child

For the above you will need proper device-tree description.

In .bind() method for this pmic, you can put pmic_bind_children() or dm_scan_fdt_node() for child nodes like the pmic GPIO.
The purpose is to have proper relation and dependency, between spmi bus, bus devices(bus children) and their children (pmic gpio).


объявление переменной лучше вынести из цикла, а в цикле только назначать новое значение.


Ben c'est plus joli pour l'utilisateur


Maybe `"the option `#{option}` exists but it is of the wrong type"`?


@tymochko то же самое, где ты взял этот пробел?


Yes, using a mock object.

However the use of private properties/methods (instead of protected) makes that rather difficult.


aa


然并卵


I tested it and no error appeared

Milovan Zogovic <notifications@github.com> schrieb am Di., 01.12.2015,
11:35:

> I don't think this commit will solve the issue. It is *this* line that
> raises: Cannot read property 'instance' of null. (
> https://github.com/wout/svg.js/blob/master/src/svg.js#L94)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wout/svg.js/commit/74bb0144ccda4bf3cc4f34b497e696a0a7eb99ec#commitcomment-14704907>
> .
>


Yes, that is a good optimization.


@hseeberger, at first sight, it's clear that this scenario is also feasible in Consul coordination. However, as usual, this is a little more involved than in the case of etcd. If you remember, what the consul coordinator does during the refreshing state is to refresh a session which is associated to the key. So, what could happen is that the session  expires. In that case, I suppose that the best we can do is to return back to the `AddingSelf`stage in order to repeat the registry process getting a new `sessionId`.

On the other hand, thinking about this issue, I realize that maybe we're not currently supporting the case when, for whatever reason, the member could be removed from the cluster.

WDYT?


i've not seen that pattern before, does it not want to be

logger.error {err: error, channel:channel}, "error parsing JSON"


@tymochko тут достаточно задать `true` для переменной `checkUserName`


@JMSerruya Thanks for updating the release zip, but could you also check the framework on master as you've done in the past. Also, update the Demo Project to reflect all the new goodness you've added?


It's a reference, so the object it refers to is changed.


I think libpng++-dev must be done last. I.e. after libpng-dev:i386 is installed otherwise it won't pick the correct dep.


I do not know.


same as other message


@tymochko и здесь проверять только переменную `checkUserName`, сравнивать её ни с чем не нужно, так как она уже булевая


dfa.cc returns null in such state. In what case this might happen? Does this mean pattern problem or it can happen during search? It it happens during search, then probably no exception should be thrown, right?


@tymochko здесь по-умолчанию задать `false`


I am fine with both, we just should have it the same through all versions and we do not have that currently, see the pic:
![version-mix-docs](https://cloud.githubusercontent.com/assets/358860/11499255/07ea8778-9825-11e5-9167-ad20ffa86a9f.png)

And please would you mind to add the above statement to the theme docs ?


This formatting looks strange to me. From my PoV either the end `)` should be also on separate line (the same way as it is done with `{   }` parenthesis or part of the attributes should be also at the same line as `(`. Personally in this case I prefer option 2.


add get method for the extraPostData


why not `tasks_ids.is_a? String`? Much more common and readable IMO.


make it return void since you always return the `params`


coffee script will turn {foo, ...} into  {foo: foo, ...}. Just a nice short hand. I use it a lot in log lines since we often just want to log out variables with the same names.


I think it's so wonderfull.


as


Thanks for the hint, good catch!

Regarding the implementation for the TCP connection: I've never tried because my openvpn servers were all running on UDP. Should be roughly similar, though.

The TLS support is the thanks to the great work by @andiwand =)


this should be rebased on bytes RE2J branch


What exactly will happen, i.e. what response (status code) do we get back, when the session has expired and we try to refresh? Could you please try to find out?

Not sure what exactly you are asking wrt removing the member from the cluster. I think this is covered by the TTL / session, not? I mean instead of explicitly removing an entry we simply delegate that task to etcd/Consul, because that's needed anyway if a node crashes and hence can't remove itself.


pass


Is this method empty by design?


indent


needs a space


remove ""


This will allow for more additions to the complex widget (like #179 for example).


Trying to attack some bower-resolution-hell root causes


Very much looking forward.


Nice catch!


`No, I forgot to delete it... Thanks !


Just read the comment a second time - not sure what happens here. You are running this in Python 2 on Debian 8, right?

Regarding the implementation for the TCP connection: I've never tried because my openvpn servers were all running on UDP. Should be roughly similar, though.

The TLS support is the thanks to the great work by @andiwand =)


What does the second item "add new topics, ...." mean?


Please leave only this:
CONFIG_ARM=y
CONFIG_ARCH_SNAPDRAGON=y
CONFIG_TARGET_DRAGONBOARD=y
CONFIG_DEFAULT_DEVICE_TREE="dragonboard"
CONFIG_SYS_PROMPT="Dragonboard # "

# CONFIG_CMD_IMI is not set
# CONFIG_CMD_IMLS is not set
# CONFIG_CMD_XIMG is not set


Ok, looks like an issue of `__c3_error`, and setcookie actually writes a header...
Anyway, looks like the code here `!file_exists(dirname($current_report))` returns true in your case so it gets into error.

I think your directory actually exists and code doesn't work as it is expected to. Could you provide path to your dir so I could check it locally and fix that issue?


Would `!isset($value)` do the same job?


dsdadas


Wenn du das schon so definiert hast, könntest du hier auch `&db5ns;` nutzen, oder?

(Abgesehen davon: Funktioniert dieses Stylesheet auch für DB4?)


I'll try that later, it won't take long anyway.


The idea under reusing _SYSTEMD_UNIT was to have the exact same output from CentOS and Ubuntu, so it is easier to parse and we don't have to deal with exception.
I have changed it in CentOS as well. I am assuming it is not using anywhere yet - at least it is not used anywhere (outside the tests) in the flocker codebase


`webpack.production.config.js` does not existed.


From this config you need to remove:

CONFIG_CMD_DM
CONFIG_DM
CONFIG_DM_DEVICE_REMOVE
CONFIG_DM_SERIAL
CONFIG_DM_STDIO
CONFIG_SIMPLE_BUS
CONFIG_SYS_MALLOC_F
CONFIG_SYS_MALLOC_F_LEN


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7415](https://teamcity.mesosphere.io/viewLog.html?buildId=83332&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout Build time: 01:00:07


The DDL firing logic has been moved into  `com.sandy.jovenotes.processor.dao` package. I will discuss the design point with you.


Fixed: c5c27a6.


I think we should keep both test cases: one with a simple `partial` and one with a nested `partial`.


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


It all comes down to `sinon.assert.calledWithExactly( appendSpy.lastCall, 'ckCsrfToken', CKEDITOR.tools.getToken() );`


I'd put a line break before this line, but if that's against your house style, it's fine as-is.


You should write `webpack.production.config.js`  by yourself. A easy way to do it is just copy ``webpack.production.js``.


We've already checked this out. The status code will be `Not Found`, i.e. the session will not exist anymore. So, we'd have to acquire a new one. Basically, the same process we do in `AddingSelf`.

And yes, as expected, both the session expiration process and the existence of the companion key are managed by consul itself. That works like a charm, don't worry :). What I was wondering is, what happened if it's the Akka cluster itself that removes the member, i.e. the cluster considers the member to be unreachable. I suppose that this is properly managed by the Akka cluster engine trying to reconnect, but to be honest, I didn't test this aspect.


Extra empty line :)


Looks good for me.


You can increase this, since will not affect the init time.


This also fixes not filed issue about selected directory wrong state when user clears the filter with items selected and directory selection doesn't go away


Needs a bit more thought, since "release" is also used elsewhere, for instance in the PDF.

Simplest would be to have a new variable "version_selected", which does what it says on the tin, and change the versionswitcher accordingly.

PR coming up once local test completes


Sorry for breaking tcp! I didn't test it and thought I didn't change anything about it. Thanks for fixing it!

I don't really know why tcp+tls support would be necessary, because checking the port would already tell us that the service should be up. Or do also want to check the response to be sure it is working fine?

I cant try to test things about tcp. But it may take a while since I dont need it yet.
You can leave an issue with it, I think @liquidat is fine with that too?


Should this commit fix #28 ? It seems like it doesn't. See https://bugzilla.redhat.com/show_bug.cgi?id=1287034 for a related bug report with rpmconf 1.0.15 on Fedora 23, backtrace included.


first "de"


abc def


this probably can be removed until we implement proper MATCH_ALT


bootstrap


update


Nice .. thanks.. love this little module, using it for a small campaign to save wildflowers. http://www.plantlife.org.uk/wildflower_garden/pledgeaplant/


Test had to be updated after renaming the journalctl output to _PROCESS_NAME to keep consistency in the logs


Could be me but `is_special_choice` seems more natural for a boolean return type.


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8080/ui#review:id=5)


Your code should check if mkdir() succeeded and handle result accordingly.
Suppress warnings with @  `@mkdir()`


remove unused file


Hi All,

Here are the command checks I'm using with Icinga:

define command {
command_name check_openvpn_udp
command_line /usr/lib/nagios/plugins/check_openvpn -p $ARG1$ --tls-auth
$ARG2$ $HOSTADDRESS$
}

define command {
        command_name check_openvpn_tcp
        command_line /usr/lib/nagios/plugins/check_openvpn -p $ARG1$ -t
--tls-auth $ARG2$ $HOSTADDRESS$
}

I think the tls-auth is redundant in the TCP check, but I left it there
anyway.

adam@icinga:/usr/lib/nagios/plugins$ python --version
Python 2.7.9
adam@icinga:/usr/lib/nagios/plugins$ uname -a
Linux icinga 3.16.0-4-amd64 #1 SMP Debian 3.16.7-ckt11-1+deb8u5
(2015-10-09) x86_64 GNU/Linux
adam@icinga:/usr/lib/nagios/plugins$

The TCP support with tls-auth would go that step further as you say by
checking the response.  It shows that the server accepted your ta.key
rather than just an open port.

Good work all round though, thanks for the work spent on this plugin it's
working fine so far.

Best Regards,

Adam Parker

On 1 December 2015 at 11:30, Andreas Stefl <notifications@github.com> wrote:

> Sorry for breaking tcp! I didn't test it and thought I didn't change
> anything about it. Thanks for fixing it!
>
> I don't really know why tcp+tls support would be necessary, because
> checking the port would already tell us that the service should be up. Or
> do also want to check the response to be sure it is working fine?
>
> I cant try to test things about tcp. But it may take a while since I dont
> need it yet.
> You can leave an issue with it, I think @liquidat
> <https://github.com/liquidat> is fine with that too?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/liquidat/nagios-icinga-openvpn/commit/c8a8fecbfcd42af85c02071429cd144d0e90ce5b#commitcomment-14706019>
> .
>


Thanks, good point.


I'll look into it but I fear that adding an extra level of communication & synchronisation significantly complexify things. Note that the fitness and pso are already (or will be) separate functions so that's not an issue I believe.


The IP address can be absent from network settings if the container is not on the default network - presumably that presents as `info.NetworkSettings.IPAddress` being the empty string. Should we treat that as an error?


With thousands of layers, this request would timeout.


NOTE: after discussing it with Itamar, the decision was to use tail (note that the problem was that in the documentation I was referencing the use of trial instead of tail - my bad for writing docs and running test at the same time!).
The regex that will match the filename is now more restrictive, so it doesn't match anything accidentally (it includes the "==>" and the "<=="
With a single tail call we:
*Don't have to worry about manually giving name to the processes, as they will be taken from the log name
*Don't have to make 5 or 6 tail remote calls
*Tracking more logs would be as easy as adding it to the command; we won't need a function per logfile we want to track


Si besoin est, oui. Mais je pensais d'abord voir si un vieux reinterpret cast fonctionne. :-)


Oops! You got me.

Never heard we can mention someone in the commit message. :grimacing:


After this modification, the GetFields() won't return public fields of the type, which is, in my opinion, incorrect!


As suggested, now both, journalctl function and tail function return the deferred.


You kept the original call of optionsOnRemove(event), so you call it twice.


You can also autoinsert the current version here, using some helper glue in Makefile


Making sure we gather all the deferreds


Fix fieldset borders.


new line


spacing issue


My bad, the newer pylint does not complain, will fix this


Missing new line


Hi,

thanks for your note. I am not sure if I understand. in what case would that be useful? thanks!


Yes, I think we thought about this. IIRC we didn't do it because `mergeInheritableProperties` is a void function with an out parameter. But thinking about it again, this would be no reason to move the optimization inside.

But I'd like to propose the opposite: keep the optimization here and inline/replace `mergeInheritableProperties` with `PropertyMap::insert` which seems to do exactly what `mergeInheritableProperties` wants to achieve. What do you think?


@camiloborges - Just discovered a setting in VS to prevent this from happening again: Open Tools > Options, select Powershell > Tabs and change the Tab setting to "Insert spaces" :-)


think it might be a bit cleaner as:

if len(spec) == 4:
  spec += ([], None, {})
self.args ... = spec


All right, I could get the same behavior, i.e. a `NotFound`, for etcd with adding `prevExist=true`. I'll open a separate ticket.

Wrt removal of member nodes I think you are right. Member removal can be initiated gracefully, and in such a case we either should remove the node from ConstructR's KV or shut down the node. I'll open a separate ticket.


Un momento, que estamos teniendo problemas al tocar todos el mismo JSON


please don't use \ to break lines, wrap with parens instead


in python 'with open' handles the opening AND closing of the file.
So:
     with open(STATE_DATA, 'wb') as output_file:
            pickle.dump(self.data, output_file, pickle.HIGHEST_PROTOCOL)

Only the code within the with open section has scope of the open file. Once the program is outside that bit of code then the file is closed.

Could you amend this commit without those comments and I will merge.


You will need to check out and compile the projects.  They both use maven for the build process, so something like the below command executed for each project should do the trick:

`mvn clean install -DskipTests=true`

You'll find the jar files in the respective 'target' folders.


:-1: jag tänker fiffla, var är libreJS


yes would prefer AutoCloseable


Where do I unzip it? ~/go?


Some comment after the review creation


That makes sense. Thanks to consider this. I'll raise proper tickets as soon as I see your approach.


Today I took a look on delete email and works well to all behaviours.


Add too a readme file :D


from API point of view, the timeout would be better to define in constructor.


@mstembera
as @lp-- Time.elapsed() > Time.available() doesn't just mean that an easymmove triggered signals.stop. it could also be due to stillAtFirstMove logic.
In both cases I don't understand why a thread with higher completedDepth should not be selected. Afterall we are not selecting an easyMove from the other thread. We are selecting a move from the other thread that is selected by searching at an even higher depth than mainThread.
Maybe I'm missing something but I don't know what it is...


Ваше сообщение не является инструкцией по необходимому запуску программы


Yes, probably an empty address should be an error.

Note that Docker 1.9 will return blank `info.NetworkSettings.IPAddress` for any container which has been explicitly attached to a different network, even if it is still attached to the default "bridge" network.


pl add favourite color


@cloudnautique I get this when starting a fresh rancher/server: `time="2015-12-01T11:47:15Z" level=error msg="Error unmarshalling rancher-compose.yml under template: zookeeper/0, error: yaml: line 9: did not find expected key" `


These same changes have been made by colormotor (whoever that is).

std::sqrt is correct but needs <cmath>
Casts have been added to avoid pedantic warnings.


agree


Does anyone else have an issue with this?

I had to add -ContentType ""application/json" onto the Invoke-RestMethod to connect

And also it should be noted the channel has to be the handle and not the room name (spaceless)


Golang y u no partial function application


Version 1.0


this is from test.txt


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


Ah, right. So the first one is about the storage as a whole and is basically a sanity check to prevent moving back in versions at the storage level.

The second one is more a trigger about a specific resource.

I wonder if these 2 should be different from the version point of view. I mean, one thing is the Storage::VERSION and a different thing is the Resource::VERSION, which are independent one from the other.

I think that my main confusion is that both are pivoting on the same global version number and that should not be the case as those are 2 different situations and require different ways of handling it. The former probably some kind of migration at gemstash level as a whole, the latter something that happens underneath without even notifying the user.

So maybe it is `check_storage_version` and `check_resource_version` ? (`s/check/validate/` if it reads better) and the error message something like "Gemstash storage engine version #{version} does not support this cache folder version #{cache_version}, please check the documentation" kinda thing, and the other something like "Resource #{resource_name} version #{resource_version} is not supported by this storage engine #{storage_version}"

It is a cryptic enough situation to handle, so I think it makes sense to provide good information and just point to the documentation for fixing.


the rest of the file tries to stay within 80 columns (also please no trailing slashes)


Ok


good job :+1:


is this needed?


i spent two hours trying to deal with the Visual designer .... what a pity , that editor is a shame .
lol
yeah , the rsult is far for good , but we have the buttons to change the pages ...


Why having Util in the name? Util classes are usually classes providing only static methods, this is not the case.

Plus add javadoc.


Nice thing this `--track-fds=yes`!


Passing the semantics annotations (with default unknown values) to the conversion function definitely make sense, and it is a convenient solution.


:sunglasses:


Привет. Посмотри на стиль кода в скриптах, которые уже есть в проекте. Важно придерживаться одного стиля. Исправь, пожалуйста, код с учетом этого.


Разработанное приложение было успешно протестировано согласно заявленным аспектам. По результатам тестирования можно сделать вывод, что приложение является качественно спроектированным и реализованным, что делает его конкурентоспособным среди аналогов.
[Rezultaty_testrirovania.docx](https://github.com/PavelEvsaev/WeatherToGo/files/48461/Rezultaty_testrirovania.docx)


This is not the official spelling :smile:


This change breaks incanter:

java.lang.IllegalStateException: var: #'clojure.core.matrix.dataset/dataset? is not public, compiling:(incanter/core.clj:99:10)


Используй для строк один тип кавычек. eslint (см. файл с его настройками в корне проекта) в помощь.
Нужен пробел между `"` и `+`.


this commits break the kivy GUI;


I think(?) you want parens here:

       (allowed.args + allowed.kwonlyargs)[0]


ok


Makes complete sense and helps remove all this redundant code! Good Stuff! :+1:


:sweat_smile: me ha corrompido el poder


The point of that was to get natural sorting.  In at least some cases I would get lists like 1,10,11,12,2,3...  I think we should likely convert to int on import for numeric fields.


Ты уверен, что в JS, если взять условие в скобки, то тебе вернется `boolean`? Если да, то зачем сравнивать буквально с `true`?
Это выражение лучше переписать как `if (Array.isArray(a) && Array.isArray(b)) {`


**[phpUnit_trunkUnit run failed](http://qa-s2.wikia-prod:8080/job/phpUnit_trunkUnit/9003/testReport/) for this commit**


> I've made some changes to addlang.txt for better wording, if you agree on it I'll run the helper, then you can add translations.

@garvinhicking
Did or do you wait for something specific to run the helper for https://github.com/s9y/Serendipity/blob/master/lang/addlang.txt ?


Bje je sais pas, je connais pas ce site (meteortips). Dans le doute, reporte toi à la vraie doc officielle :  http://iron-meteor.github.io/iron-router/


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7430](https://teamcity.mesosphere.io/viewLog.html?buildId=83391&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:00


saw approval from andytill
at https://github.com/basho/riak_kv/commit/96acc252617d36cfc7eeed124a9d64ff0b6b8c72


merging basho/riak_kv/fix/az/handle-backend-and-qry-worker-timeouts = 96acc252 into borshop-integration-1283-fix/az/handle-backend-and-qry-worker-timeouts


basho/riak_kv/fix/az/handle-backend-and-qry-worker-timeouts = 96acc252 merged ok, testing candidate = 5b47755a


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


I'll see about your advice.Thank you for my a favor.
if you don't mind, Can I ask you a question when it is not know?


@Khelek ок. Это справедливо.


Un type de task ? Mais pour quoi faire ? On en parle jamais dans le cahier des charges. Tu t'en souviens ?


sudo_users  =  {109722245}


Haaa, mais c'est pas le type de task pour les preferences d'affectation ? Si oui, on fera ca plus tard


Будет не лишним, если ты переменную `i` объявишь сначала как объявлял другие. Но в данном случае можно сделать и `for (var i = 0; ...`. Только не забудь, что цикла у тебя два.


Shellcheck complains: https://github.com/koalaman/shellcheck/wiki/SC2086


After this modification, no public fields returned.


Perhaps it's better to use `$()`: https://github.com/koalaman/shellcheck/wiki/SC2006


Sure, I'll be happy to answer


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


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7433](https://teamcity.mesosphere.io/viewLog.html?buildId=83396&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Failed to create a directory Build time: 00:00:00


On peut faire ce qu'on veut en CSS. Typiquement, Bootstrap fournit des classes CSS pour faire de jolis bouttons. Ne te soucie pas trop de l'esthetisme pour le moment


You should update .gitignore file.
.idea/ and *.iml files should not be added to the repo.


Why not use `crypto.Sha3`?


Send me a link, when you're done - I'll review, if you want


This is how the built in formatter does it, the line is too long.


I like this feature very much. Thanks!


You could also refactor this to

```go
sha := sha3.NewKeccak256()
for ... {
    hash := sha.Sum(buf)
    ....
}
```


P.S. Another option is to use `[[` instead of `[`:
```bash
$ FOO='test test'; [[ -f $FOO ]] || echo OK
OK
$ FOO='test test'; [ -f $FOO ] || echo OK
-bash: [: test: binary operator expected
OK
```


Should be `<a>escaped as code point</a>` to make Bikeshed linkify this correctly. Currently there’s no link at all.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7434](https://teamcity.mesosphere.io/viewLog.html?buildId=83397&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Failed to create a directory Build time: 00:00:00


It's `title_video` no need for `section` as it doesn't explain anything.


> end of enumeration

end of *the* enumeration maybe?


It's a mistake. First of all sections should be with id and not class as we don't really have more than one on page. But even if you kept class as you probably didn't... you misspelled the class name.


I'll never be confident with `-`-separated words, but could this be the case of user**-**supplied function? :smile:


Ok try this:
1) Move all other Startup python to one side, except for the new ThumbnailExportTask.py and ThumbnailExportTaskUI.py in dotStudio
2) In the Export dialog, create a new preset, with Content = Thumbnail Export
3) Export and see what happens...


Completely useless selector without section. It will clash with contact form.


Very cool. Maybe we should consider changing the rdfhdt library in time (for JSON-LD support etc.).

I wonder if we can bring the Java server more closer in functionality to the JavaScript server. Preferably without too much duplication. Would be awesome if we could reuse the templating mechanism. Maybe there is room for that with Java 8 and Nashhorn.


Should we use this solution or add a `_next_hero` attribute and change player's hero on his next spawn without killing him?


Oui mais sans parler de CSS. C'est juste qu'à mon niveau, je comprends pas pourquoi on passe pas directement ça dans un bouton puisque de toute façon on le fera plus tard. Je me dis juste que je vais peut-être faire des erreurs plus tard quand on remettra le bouton


Harmful `cms:eval` which nicely fills error log.


Oui c'est ça


Lowercase comment needs to be written properly.


Sure will take care of this. This patch is still WIP so not done yet. Thanks for pointing it out.


This wasn't necessary before. Why is it now?


This looks great, Thank you very much @adilek !


The cloning happens in without_cached_constants, so I am essentially doing as much at once as possible. I have to add the inputs to the function graph before returning from the function.


Requires docs and I'd keep the naming consistent (e.g. `GetCtx`)


dito


Nas tags que estão no rodapé, os # não seriam substituídos por um ícone?


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


I sent an e-mail to check with Dima B.


Why did you changed the formatting of this line? I have never seen a PHPDoc line wrapped like this.


Again, this can be the empty string, and we should probably return an error in this case.


`var sha3_nil = crypto.Sha3Hash(nil)`


I can't get the inject to work as i'd expect. Advice appreciated /cc @jamesgorrie @theefer


Yay. I'm special!


Лучше создать через newSingleThreadScheduledExecutor()
И в параметрах передать TaskThreadFactory чтобы потоки были daemon

Так же на unbind() хорошо бы остановить шедьюлеры


Nimam komentarjev na trenutno verzijo.


@joao-manywho I don't think this is going to do the trick. You'll also need to change the way the width of the input is calculated in the input component itself. I set the width of the input box in the test flow to 3 digits (with the yeti) theme and its broken


Why did you removed this test? Please re-add it. It will fail because you also removed the exception being thrown when the filter array contains an invalid type.


(I've had to put the comment on the wrong line because github) - again, the final return statement of this function should probably be changed to return an error if `NetworkSettings.IPAddress` is the empty string.


To follow other methods on `sol_util_file.h`, I think that `sol_util_move_file()` is more appropriated.


This is used in the EntityGroup and Workflow assignee code. Admins/managers would assign users to tasks in a workflow, and assign users to manage a group of elements (a collection of samples from different projects to process in the lab, for example). No reason this couldn't be changed for an individual user search if there are security worries...


Agreed.


@Olympic1  you can remove the "?" after "index.php" and the link will still work

http://forum.kerbalspaceprogram.com/index.php/topic/116058-/


Теперь инициализация session тикета происходит асинхронно в другом потоке, в теории первые запросы могут работать без тикетов, я бы тут их еще синхронно установили и зашедьюлил с начальным периодом


I think this *could* be left as is - both wordings seem acceptable.


Btw, the problem at hand aside - I can't image who'd consider `C-c C-m` convenient for frequent usage. :-) `C-c RET` is not super handy either... Should be aiming for something as convenient as `C-c C-c`.


should be `:operation<->`


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


should be `:operation<*>`


should be `infix:<n>`


has tornat a posar un bucle for a dins d'una funció que retorna dades en funció de cada iteració del for... un error.


Oops gotta remove noob stuff


@davidjgonzalez Here we've a challenge. HttpCacheHandlingRule has hooks other than onRequestReceive which may not be possible to implement in HttpCacheConfigImpl. I made this cache config accept only the service pid of rules and does all the check in the Engine. I committed the code. Please do check and let me know.


@mariokorte Ah, yes that's a good idea. The header just looked a bit cluttered is all. I'll see how it looks in the footer text later on today. Thanks!


I think 'original' code was not commas.  The original code parsed format types from the ```./john``` with no params output.  It was different format than the new --list=formats value.  The new code gets into the tight loop in the if statement, so never has to deal with finding the start of the format strings, and then the end of the format strings.  So that $in_fmt variable is actually a pretty good check, for the new block.  BUT your regex is also pretty good self documenting code also.


Cache config taking a set of service pid of rules.


Handy method which lets the engine know if this config accepts the rule with given service pid.


@schlomo
could you add a comment why dev/oracleasm is excluded
so that other users can see what that is about
(not everybody knows about dev/oracleasm)
plus the https://github.com/rear/rear/issues/721 URL
so that the original issue is linked there?


valgrind-3.11.0-1 est dans gnome depuis le 10/11


@ketan as a TODO we need to rethink the model_mixins way of extending the model.
Perhaps change that to

```
m.model(function(){}).withUUID().withErrors();
```


FWIW, I use `C-c r`. It works well for me because the clj-refactor uses plain letters for the bindings. So I just release Ctrl right after C-c. I could see `C-c RET` working well for the same reason.


We should get the list of all owner details instead of just the first one


That seems to have done the trick, thanks!


All available rules bind to the engine. Engine stores them in a map with service pid as the key.


jQuery(".albumThumbnailImage")
    .attr('href', data.result.url)
    .find("img").attr('src', data.result.src)
    .end().show();


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


Apply the rule only if it's a configured globally at engine or configured at cache config level.


I use C-c C-m myself and find that quite convenient.  The only issue I have is that I seem to hit C-c m about 5% of the time :p


Same case here but for deliver cache hook.


...for cacheResponse hook.


use `SELECT id` and `mysql_num_rows` instead


what happens on error?


**Code Quality - Nitpick**

In this case, there shouldn't be a space between the function name and the parenthesis.


TODO: implement inline optimizations from dfa.cc


@hartmannaf The files needed to construct a server are all public. But hosting the files on internet and offer the link might lead to legal issues so i wrote those in order to let them build the server themselves, privately. The reason for no my server file on gbatemp, mainly dues to ronhero's working server. He does provide a paid service for n00b and said he would get it public once the $ had reached a limit. O...
However once used a spoofed package that all contents included claim to be version 32767 and in fact a 9.2 pack, with the server and launched in SysNand with SigPatch, would permantly stop it from updating.


Наверняка для сервиса тоже надо полный путь написать, с учётом разных appId наших для дебаг-бета сборок.


Done.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7428](https://teamcity.mesosphere.io/viewLog.html?buildId=83370&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout (new) Build time: 01:00:07


Am I doing something wrong with my build here - the sym-link to the XHP class means I'm getting a self-referential 'name already bound' error:

```
$ hh_client
src/site/xhp/a_post.php:3:13,19: Name already bound: :a:post (Naming[2012])
  guides/hack/24-XHP/01-introduction-examples/a_post.php:3:13,19: Previous definition is here
````


The fallback URL is wrong. I created a pull request to fix this: https://github.com/luksak/media_entity_embeddable_video/pull/1


By default, there is no restriction to access http://nas/adminer/index.php. Lots of users open the tcp/80 and/or tcp/443  port on their routers and don't check anything about security.
Adminer can connect to DSM's internal PostGRE SQL server which allows connections from localhost (127.0.0.1) only, with user admin and an empty password.
Remove the .htaccess will be a huge security issue (drop database, ...).
The same problem could appear with MariaDB when using low security accounts.

I will a backup/restore of this file (backup on pre-upgrade, restore on post-upgrade).


@Nullable ?


Remove all the commented code please.


Done


Thanks for your reply. Can you post your boto config( ~/.aws/config) and relevant env vars (AWS_*)?

It looks like ansible/boto3 is trying to connect to the endpoint using the region name. This will not work of coarse.

Have you also tried other modules like route53_facts?


Hmm, nice.


Используй forEach.


nice


HOWTO:

1. 在resources/locale-*.json中定义对应的多语言字符串资源：如"dashboard.CPU": "处理器"，其中"dashboard.CPU"即为resource ID，是整个程序唯一的。

2. 替换原有code中对字符串的使用，有如下几种处理：
    a. HTML template中字符串，用translator filter处理，如：`<h3 class="z-h3">{{"dashboard.DASHBOARD" | translate}}</h3>`
     b. object中的字符串域，用如：`Translator.addProperty(cpu, 'name', 'dashboard.CPU');`方法添加getter
     c. kendo-grid中的column，用如：`title: '{{ "dashboard.CAPACITY NAME" | translate }}',`方法添加filter


Issues with `maximus.py`:
* raises error, when you try to call `best_move()` beyond a terminal game state
* game state evaluation doesn't work properly, yet

However, the proposed `mediator.py`, should function correctly and without inner errors, while remaining fairly simple. Tests should be written, though.


Напротив, @NonNull


Good point. I've intended map_variables to be used *before* compilation, but I suppose there are other avenues by which an fgraph could come to own some of the replacement subgraphs. To be safe I could simply clear the cache after the outermost call to map_variables. The code in map_variables doesn't do any fgraph modifications (I hope) that interfere with its own correctness, and the docstring could warn the user to keep the fgraphs on replacement graphs constant (i.e. don't have fgraphs or don't change them or reconstruct the replacement at every call, etc.).


This may leak `hn`


OK!


Erm... pass?


parentDirectory = location.markdown + languages[i] + '/'; -> parentDirectory = `${location.markdown}${languages[i]}/`


빌드는 됩니다만 부팅이 안되는 원인을 모르겠네요..


The code does not comply to PEP8.

*Detected by PEP8Bear with severity 1 while running section `python`.*

*The issue occurred in the lines 2 to 2.*

The issue can be fixed by applying the following patch:

```diff
--- a/test.py
+++ b/test.py
@@ -1,4 +1,6 @@
 def main():
     pass
+
+
 def other():
         pass
```


The code does not comply to PEP8.

*Detected by PEP8Bear with severity 1 while running section `python`.*

*The issue occurred in the lines 4 to 4.*

The issue can be fixed by applying the following patch:

```diff
--- a/test.py
+++ b/test.py
@@ -1,4 +1,4 @@
 def main():
     pass
 def other():
-        pass
+    pass
```


Я про возвращаемое значение.


I get you, but i still have the same error with 2.2.3 version. Perhaps it is some other type of error, I'm not sure.

Let me guide you through my callstack:

1. `img.onload` is called
2. `self.parent(SVG.Pattern)` is called (https://github.com/wout/svg.js/blob/2.2.3/src/image.js#L19)
3. `parent = SVG.adopt(parent.node.parentNode)` is invoked where `parent.node.parentNode` happens to be `null` (https://github.com/wout/svg.js/blob/2.2.3/src/element.js#L176)
4. trying to call "instance" on null (https://github.com/wout/svg.js/blob/2.2.3/src/svg.js#L94)


Тоже самое forEach


Yeah, this test is correct. I thought this was tested already, but that doesn't seem to be the case for relative imports.


forEach


ULONG_MAX and UINT_MAX are defined in stantard header <climits>


@tatsuhiro-t  An alternative might be to use [`GetFileAttributesEx`](https://msdn.microsoft.com/en-us/library/windows/desktop/aa364946%28v=vs.85%29.aspx) on Windows, which does not require opening a file at all.


трішки заплутався, бо ніби після FOR ставлять пробіл


url = `../${text}`

Заменить везде где есть конкатекация строк.


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


I call this "group identifiers" in all other places.


Good question.


:star:
Yeah this doesn't make sense to me now either. I'll change it.


Please re-add the removed `else throw` here.


ok


github clone下主分支，用xcode编译出窗口，拖拽文件在10.11上convert成功，但是在查词的时候，查不出来，显示“Format not supported”，请问这是什么情况？


then line 109 must be

gpio = read8(MCP23008_OLAT);


Ok, should be looked into. In JS code we always do it like that, afaik.


Yes, for the person who hosts it it will still be a legal issue, but I think it would be a lesser problem for normal users. Also I don't think it would be a problem to simply give the users the domain, but maybe I'm wrong. Would still be easier for users to update this way with something like 2.domain.com or 5.domain.com for 9.2 or 9.5 :)


Problem 1 - reverse a string with void reverse(char *str).


The code does not comply to PEP8.

*Detected by PEP8Bear with severity 1 while running section `python`.*

*The issue occurred in the lines 2 to 2.*

The issue can be fixed by applying the following patch:

```diff
--- a/test.py
+++ b/test.py
@@ -1,4 +1,6 @@
 def main():
     pass
+
+
 def other():
         pass
```


The code does not comply to PEP8.

*Detected by PEP8Bear with severity 1 while running section `python`.*

*The issue occurred in the lines 4 to 4.*

The issue can be fixed by applying the following patch:

```diff
--- a/test.py
+++ b/test.py
@@ -1,4 +1,4 @@
 def main():
     pass
 def other():
-        pass
+    pass
```


This method does much more than "adding filters". Therefore the name is misleading.


fast-forwarding end-to-end/timeseries to borshop-integration-1283-fix/az/handle-backend-and-qry-worker-timeouts = 5b47755a


The `[0]` actually selects the first result of the list comprehension, but I do agree that using parens here might improve the readability.


There is an explicit type hint now, so the exception should still be thrown. I agree that the test shouldn't have been removed, though.


`| O_CLOEXEC | O_DIRECTORY`. If `fsync()` will work with `O_PATH` (not sure), pass that flag as well.


Yes...


Great, it sounds nice.  Could you make a PR?


Yeah totally. It would make sense in some cases (I dont see the way to specify that on the upper constructors, but it would fit ok for the templates below).

On the other hand Wire is written in c++0x rather than pure c++11, so I'd like to keep it MSVC2010 friendly still.
(it has been a long time since I used MSVC2010 last time, so I hope it is still compiling without errors :)


Awesome that you have tested that it actually throws an exception. What about the test that indicates that it actually post emojis successfully


createTherdLevel -> createThirdLevel


REMOVED validation


It is, but it's not worse than `setFilters` was, imho. @Jonas-WMDE complained about this, too.


То есть, ты ждешь, егда картинка полностью загрузится, и тогда уже считываешь её размеры. Я егда-то давно хотел так сделать.
Прилично выглядит.
Такая проблема с размерами, по-моему, только на `2chan.net`, или еще где-то?
Теоретически, должно везде работать.


I'll test if this actually works first (or if GetFileAttributesEx has the same problem as _wstat) and do so then.


This would not be necessary since there is a type hint, right?


@fmrl Sorry for the delay. Can you explain the error this is fixing? Just tried and I can verify the current `seq.fst` without this change:
```
fstar.exe seq.fst
Verifying module: FStar.FunctionalExtensionality
Verifying i'face: FStar.Set
Verifying i'face: FStar.Seq
Verifying module: FStar.Seq
All verification conditions discharged successfully
```


Just trying to require the path directly, rather than testing for absoluteness, will also allow for layouts such as:

```
- someDir/webpack.config.js
- package.json
```
with `./someDir/webpack.config.js` specified as the config.  I don't vouch for the sanity of this layout, but I believe referencing the file like that in most other loader situations would work correctly.


Are you sure about this?
I have never had any issue because of `wire::format()` up to date


Ca pourrait effectivement faire le tricks


Might be a good idea to use `strndupa(new_path, PATH_MAX)` here instead.


This misses a type hint.

Apart from that, I actually like the basic idea of your refactoring. It fixes an actual issue and is still easy to read, if not better.


Shouldn't these files be opened with `b` mode as well?


layauts -> layouts


The Delete emoji test has not be been acounted for


@tymochko просто `if (checkUserName) {`


Ещё на мидоричан: #868

Хотя, по большей части,я её запилил из-за каталогов, где отсутствие разворота картинок (ни в каком каталоге нет размеров картинок) было самым большим недостатком. Теперь там скрипт может работать (после запиливания поддержки) вполне полноценно.


Prima gewerkt, goed resultaat.


fixed in 73625b949ff7c73b41794257762edc0449837d6a


The chmod should happen before file copying takes place to avoid possible race conditions.


Here we are initializing an empty array if the ownerdetails is null.


Щас расширение залью, глянешь чё я набодяжил.


The lines have a "-" must be taken from the code or leaves?


Love it!


About the style choice, you are right. This probably got refactored a couple of times and the fact that is still in mm and that it wasn't the same everywhere were leftovers of some chaotic refactor session :/
Didn't make the effort to change it to int tho..


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 145](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=550&buildTypeId=XbimEssentials_XbimEssentials) is now running


En fait pas si sur car maintenant c'est une classe à cause du constructeur. Il y a peut etre quelques ajustements à faire.


This method is a little brain twister. Could you elaborate on lines 80 and 81?


Pourquoi?


This patch comes from https://github.com/robbyrussell/oh-my-zsh/issues/4565#issuecomment-152763146


Dafuk snakker du om?


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 145](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=550&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:01:14


In the case of blacklisted package the journal log looks like follows:
```
Dec 01 13:28:44 dhcp-24-115.brq.redhat.com kernel: will_segfault[12967]: segfault at 0 ip 0000557601367a00 sp 00007ffd7b750dd0 error 4 in will_segfault[557601367000+1000]
Dec 01 13:28:44 dhcp-24-115.brq.redhat.com abrt-hook-ccpp[12968]: The executable '/usr/bin/will_segfault' is blacklisted
```
If crash the blacklisted executable, the `log()` function has adequate priority log level in context of other log messages because there are no more log messages and user can easily find the message in journal log.


1. Ставишь stable верисю лисы (42), если у тебя её нет.
2. Накатываешь (jpm)[https://developer.mozilla.org/en-US/Add-ons/SDK]:

```
npm install jpm --global
```

3. Заходишь в `../dollchan-extension` и запускаешь расширение. JPM стартует стабильную лису со временным профилем:

```
cd dollchan-extension
jpm run
```

4. Пробуешь запускать разные сайты. Я там расставил немного логов, они отправляется прямо в git-консоль.


why you have different conditions than in dfa.cc?


Tu peux l'enlever ça.


Tarefa #190


Good call - read through and looks great.

Btw - does your Json test work w tee print writer approach?


You can transplant by yourself.
Originally it is writing by Code::Blocks.


Shouldn't that be the same as path.join-ing the package location with a relative path?

Attempting to require a relative path directly would resolve it relative to this source file, I believe. So I think what I've written supports what you're describing. (I meant for it to, anyway 😅)


Делает `gulp make` и копирует es5-версию в папку с расширением. Попутно исправляется один неприятный баг (по-моему, он егда-то уже был). Править оригиналы что-то не хочется.


Extract
```
    DFAState nextState;
    nextState = currentState.next.get(lastRune);
    if (nextState == null) {
      nextState = runStateOnRune(currentState, lastRune);
    }
```
as separate method


@daid: Handled that in commit 345c599062dae64f41ab3c6a4998433062c1270e

The case where the count =  zero should not occur, but now that case is also handled.

Frontend changes which enforce the count to be more than zero: 4c668c9a1e39e1481b2d3c9abe4a819a3a956133


this isn't actually calling with_args, is this right?


Честно говоря мне не сильно это нравится.

structure = [
    {second_level:[
              {third_level: [
                                  {third_level_data: [{}]}
                                  ] }
                           ]
    }
];

Тебе не кажется, что это слишком сложно?


George, makes sense on a *nix system! So, no need to change any code?

Ross

On Tue, Dec 1, 2015 at 2:05 PM, gt7810 <notifications@github.com> wrote:

> Hi Ross,
> Finally worked out the problem with messenger and module loading.
> Base path is Case sensitive. Doh!
>
> George
>
>
>
>
>
> On 25/10/2015 9:23 PM, Ross Parker wrote:
> >
> > Hey George. Thanks for this commit...but I am not sure commenting out
> > the code is the best long term solution. The include in question is
> > generally (but not always) included in index.php, and so should not be
> > re-declared in /modules/Messenger/messenger_post.php. However, if it
> > is not included in index.php, then it should be included. That is what
> > the test is for.
> >
> > This same test is used elsewhere in Gibbon with no issues, and I have
> > not had any issues on our servers with this page. I guess your server
> > is doing something different with slashes in the path.
> >
> > I am going to set this file back to how it was before your commit:
> > would you mind looking a little more at what is causing the issue on
> > your side? The question is why is the $include entry (drawn from the
> > entry $includes) not matching to
> > "/modules/Messenger/moduleFunctions.php" on your server?
> >
> > Thanks!
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/GibbonEdu/core/commit/2211482a297820c1b677ca622b0ec45b546f968a#commitcomment-13971773
> >.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GibbonEdu/core/commit/2211482a297820c1b677ca622b0ec45b546f968a#commitcomment-14700405>
> .
>

--
*Husband, father, son, brother, friend, coder, teacher & coordinator of
ICT.*
Blog .  http://rossparker.org
Tweets . @rossdotparker <https://twitter.com/rossdotparker>
Opensource .  http://gibbonedu.org
--


@whatyouhide you think it's unnecessary to `inspect` the value? BTW, I agree with your proposal, I already changed to this:

```
"the option `#{option}` exists but it is of the wrong type: `#{inspect value}`"
```


Also change commit message :)


Flowerpot still change to wood when place it..


On system like ROS, the default socket timeout differs from the standard infinite value. This causes exceptions. We fixed this forcing its value to None.
From ROS code:
```
    # 2) xmlrpclib doesn't give us any way of affecting the
    #    timeout other than affecting the global timeout. We need
    #    to set a timeout to prevent infinite hangs. 60 seconds is
    #    a *very* long time. All of the rospy code right now sets
    #    individual socket timeouts, but this could potentially
    #    affect user code.
```
The exception happened:
```
Exception happened during processing of request from ('127.0.0.1', 51866)
Traceback (most recent call last):
File "/usr/lib/python2.7/SocketServer.py", line 582, in process_request_thread
self.finish_request(request, client_address)
File "/usr/lib/python2.7/SocketServer.py", line 323, in finish_request
self.RequestHandlerClass(request, client_address, self)
File "/usr/lib/python2.7/SocketServer.py", line 638, in init
self.handle()
File "/usr/local/lib/python2.7/dist-packages/remi/server.py", line 158, in handle
if not self.read_next_message():
File "/usr/local/lib/python2.7/dist-packages/remi/server.py", line 169, in read_next_message
length = self.rfile.read(2)
File "/usr/lib/python2.7/socket.py", line 380, in read
data = self._sock.recv(left)
timeout: timed out
```

Now fixed! Thanks to @awesomebytes


oh it's a typo


Do you mean after `fopen` but before any `fwrite`?


Надо бы этот фай


Надо бы этот файл как-то автогенерировать на основе скрипта.


minor nit: I tend to prefer leaving the hanging ( on the previous line:

        .. should_receive(
          'stuff').with_args...


В начале файла было сделано denodify, тогда зачем эта функция?


Dosya olusturuldu


Стрелочные функции.


А как-нибудь убрать скрипт `data.url('boards-getter.js')` из страницы в случае несовпадения возможно? Чтобы песочница место не занимала.


"瞭解 Free Code Camp"
space between fullwidth and halfwidth characters for better readability


Был в `core.js` в тамперманки. Тоже была проблема с тем, что библиотека воспринимала `self` как объект `window`.


This method is for writing an (extruder switch) retraction and the resetExtrusionValue() is called before writing the retraction.
The method is still called in the function switchExtruder, so the question is why would we call the method here, before the retraction?


这一条改不回来了……


Hi,
I am using the following AWS env vars:
AWS_SECRET_ACCESS_KEY=XXXXXXXXXXXXXXXXXXXX
AWS_REGION=us-east-1
AWS_ACCESS_KEY_ID=XXXXXXXXXXXXXXX
AWS_PATH=/home/dev/.aws
AWS_ACCOUNT_ID=XXXXXXXXX

I don't have the boto config file (~/.aws/config)

I never used route53_facts module, but I used for example the ec2_elb_lb as following:
-  name: load balancer
      local_action:
        module: ec2_elb_lb
        name: "{{ realm }}-elb-{{ service_name }}"
        state: present
        scheme: internal
        subnets: "{{ public_subnet_ids }}"
        region: "{{ region }}"
        security_group_ids: "{{ elb_sg_id }}"
        connection_draining_timeout: 120
        cross_az_load_balancing: yes
        listeners:
          - protocol: http
            load_balancer_port: 80
            instance_port: 80
        health_check:
          ping_protocol: http
          ping_port: 80
          ping_path: "/healthcheck"
          response_timeout: 5
          interval: 20
          unhealthy_threshold: 3
          healthy_threshold: 2

Note that I am using region as variable, hence I am using the -e region=us-east-1 when calling ansible-playbook command.
It looks that the endpoint (ec2_url) is returned from the get_aws_connection_info function which exists in module_utils/ec2.py

I am appreciate your help...


Could you add a `Q_EMIT` here please?


done :+1:


https://developer.mozilla.org/en-US/Add-ons/SDK/High-Level_APIs/page-mod

Там есть метод `destroy()`. Уничтожает весь объект. Не знаю, удаляет ли сам скрипт.


It has several versions. :-)


:-1:  jag är faktiskt 12.89 :^(


Подтянул коммит 18a31b4f465a4217a705ac38211751e8cb258da4
Ничего не изменилось.


Можно проверить при помощи `about:memory`. Сегодня возможно займусь. У меня до сих пор специально сконпеллированный браузер лежит для анализа потребления памяти.


Ha non non, on passera jamais par un <button> pour ce genre d'action. En revanche on ajoutera du style CSS qui le fera ressembler à un beau boutton.


Ошибка. Убери пробел.


I don't understand this. Your metro/fonts looks the same to me as your metro-dark/fonts.


А так неплохо. Осталось теперь запилить глобальное хранилище, ну и можно обмен данными между табами c разными доменами ( можно сделать на основе https://developer.mozilla.org/en-US/docs/Web/API/Broadcast_Channel_API , благо для аддоноскриптов он тоже доступен \o/ )


Ok, I can fix that.


fix


Hi,
is your SystemVerilog grammar complete and working? I am not sure how to interpret the commit message. I started to write a SV grammar for ANTLR4 myself, but I'm stuck in the middle as it is WAY too much work and I've already lost all hope that the grammar will actually work once I'm finished. So maybe you could leave a short comment about the implementation status of this grammar?
In any case, very impressive work!
Thanks and kind regards, Max


The `assertRaises` method calls it with `1` as argument (the line below).


On va avancer par iteration avec comme premier gros objectif d'obtenir à peu près les memes fonctionnalités que l'existant. Il ne faut pas essayer de tout faire en meme temps car ca fait de trop grosses iterations. Il faut maximiser le feedback pour avancer au plus vite.
Si cette notion n'est pas clair, je peux en parler mercredi apres la reunion.


"歡迎來到 Free Code Camp。我們是學習寫程式和幫助非營利組織的開源社群。"


ah sorry about that, been awhile since I've touched this code :)


**Code Functionality**

Line 40 has nothing to do with line 41. So you are actually not testing a single emoji here, you are testing `/emojis` from line 14 because that's what the `$this->response` returns.


see https://github.com/NativeScript/android-runtime/issues/291


Ой вей. Кусабу убрать нафиг можно жи есть. Нафиг я ее добавил, детект по делформе же.
Не знаю, я сильно вчера упоролся егда сочинял.

Короче, нам нужны `qDForm` из `ibEngines`, и содержимео полей `ibDomains`.


No worries. Thank **you** :wink:


Shouldn't this go to syslog INFO?


Lost space character


Hi Vadim,

Would you mind trying DT2W on your kernel?
Don't have my environment ready for building kernels and maybe the community can enjoy it.

https://github.com/Nothing-Dev/MaxiKernel_condor/commit/5a48c315e19c4857e3119ea6e40cfd246c57882d


OK


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


Actually, scratch that, GetFileAttributesEx also returns 0 for symlinks, the MSVCRT fixed that by making _wstat open the file as well, but only in later CRTs that we don't link again. I'll just fix the potential handle leak.


TeamCity Habitat :: Cruise [Build 304](http://build8ua1:8081/viewLog.html?buildId=765&buildTypeId=Habitat_Cruise) outcome was **SUCCESS**
Summary: Tests passed: 97 Build time: 00:01:14


**Code Quality - Critical**

Only Instantiation of the Client should happen in the `setUp` method.

The request made to `/emojis` should happen in the method  `testGetAllEmoji` which is testing that all emojis be returned


I find the semantic of this function weird.  This hands out either (a) an empty ("uninitialised") props objects or a (b) already set.  It is thus like a heap with named cells.  The initialisation (filling with properties) is done via StyleSet in StyleSetProps.  Given this: Is this really a public API on `StyleEngine`?


Хотел тебя добавить в "contributors", но это поле в `jpm` чому-то deprecated.


`2.0-M2` is already available


Hey, we have the same problem with our [GDG application](https://github.com/gdg-x/frisbee).
What did you do to replace AppState? Why is it suddenly removed from GMS?


** Code Quality - Nitpick**

You don't need this method, this is a negligible test, what does it do again?.. just tests URL?


See https://github.com/Ableton/aqt-stylesheets/commit/8936be3193b2dd6e442cbfdc5908c0f9a99caba6. With the following branches in mind, the simplest is here to just replace `mergeInheritableProperties` with `std::map::insert`.


maybe we need try-catch here


Just an observation, I haven't made a pull request with my version of this type of addition to keep from stepping on peoples toes, but the ^ shouldn't be mapped to the current version of lineBegin.  Instead unless my vim fu is suspect that should be mapped to 0 (zero).  The ^ actually goes to beginning of first non white space.  In mine I have created a seperate function in Cursor like below to map the ^ too:

```
	static nonBlankLineBegin() : vscode.Position {
		let pos = this.currentPosition();
		const linetext = TextEditor.ReadLine(pos.line);
		if (linetext.length !==0) {
			var fc = linetext.length - linetext.replace(/^\s+/,"").length;
			return new vscode.Position(pos.line, fc);
		}
		return new vscode.Position(pos.line, 0);
	}
```


Вообще вот https://developer.mozilla.org/en-US/Add-ons/SDK/Tools/package_json, если что.
И да, аддон не будет работать в лисе <38. Там `jpm` аще не завезли.


@milmazz no no sorry, I messed up :) I agree with inspecting, I just meant to use the wrong **type** instead of **value** :)


OK, thanks!


Small typo: "tilting"


That was my first implementation, but I remember that there were use cases, when StyleSets can survive StyleEngines and wanted to reconnect.  I don't remember though which.


@backseatviolist yup. And the beauty of the free software is that I have another options. All my code have 2 spaces indentation and there are forks of JSLint that still read the indent option. I have freedom to determine how many spaces for indentation in my code, and only need the lint to show me actual errors instead of telling me that 2 spaces indentation is an error that must be corrected.


:+1: Bro!


I'm a bit concerned about null-driven development in Scala, especially when we don't have to deal with Java APIs. Can we use `Option` here?


Ok, let's put it into documentation.


I was done


Yeah I noticed this as well. Fixed in https://github.com/w3c/csswg-drafts/commit/30caf88a3b028844b0e27ee60d0e5802fe5dece9


**Code Quality - Critical**

 `$request = $this->client->get('/emojis');`
  `response = $request->send();`

This should be happening here, then your assertions can now test for the response and response status code


Changed to `Path on the datastore, where the glance images will be stored'`


> Там есть метод destroy(). Уничтожает весь объект. Не знаю, удаляет ли сам скрипт.

Конкретно этот - нет. Он просто отключит внедрение новых скриптов. А вот `worker.destroy();` после `console.log('No board.');` как раз сделает то, что нужно.
https://developer.mozilla.org/en-US/Add-ons/SDK/Low-Level_APIs/content_worker
>**destroy()**
Destroy the worker by removing the content script from the page and removing all registered listeners. A detach event is fired just before removal.


maybe need try-catch here


list.h isn't included here. need to include.


We need an example illustrating these options.


When executed in Java newer than 7 the test on the 6.9.4 version is executed twice.


Still getting the validation error, digest seems to be missing...


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


When executed in Java older than 7 the test on the 6.8.7 version is executed twice.


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


Нормально. Как раз 38 это ESR.


Ah d'accord, je savais pas


You meant options.getGroupByInstances here


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


See https://msdn.microsoft.com/en-US/en-en/library/1kt27hek.aspx
> vsnprintf, _vsnprintf, and _vsnwprintf return the number of characters written if the number of characters to write is less than or equal to count; if the number of characters to write is greater than count, these functions return -1 indicating that output has been truncated.
> (...)
> If buffer or format is NULL, or if count is less than or equal to zero, these functions invoke the invalid parameter handler, as described in Parameter Validation. If execution is allowed to continue, these functions return -1 and set errno to EINVAL.


@whatyouhide no problem :+1:


I've solved my problem, just changed the simulator's language environment. I've also tried your way to solve my problem, it worked. but there has a small problem when the first time it would not change the 'cancel'  to other language...No matter how, thank you very much.


It looks like it supports it: https://github.com/francoispluchino/composer-asset-plugin/blob/master/Resources/doc/index.md#project-scope-installation


test


Зачем поддерживать альфа-версии некробраузера? Мб просто указать `>=38.0`?


Yes, right after the file is created would be better, but not entirely correct: one could still obtain a handle to that file before mode changing takes place. The ideal thing, instead, would be to use the three-parameter version of `open(2)`, passing the mode directly. Using `O_SYNC | O_WRONLY | O_TRUNC | O_CLOEXEC | O_CREAT` as the flags parameter.


Аще одно замечание. Скрипт стартует по `doc.readyState === 'interactive'` в условии

```js
if(doc.readyState === 'interactive' || doc.readyState === 'complete') {
	needScroll = false;
	async(runMain)(true, null);
} else {
	var cfgPromise = null;
	if((aib = getImageBoard(true, false))) {
	...
```

`contentScriptWhen: 'ready'` в этой строке аналогично `DOMContentLoaded` (ну мы ведь делформу же ищем). теоретически можно искать домены по `contentScriptWhen: 'start'`, и если домен не найден, то
запускать второй объект `page` с `contentScriptWhen: 'ready'`, котрый будет искать делформы.


With this the `isValid()` function is undefined.


You don't check for r < 0.


Да не. Тогда будут подключаться фактически три скрипта (для поиска доменов, форм и потом сама кукла). Думаю так пойдёт.


This is a lol wrong commit. hahaha


I export the following env var:
EC2_URL="https://lambda.us-east-1.amazonaws.com"

and now it looks OK. :-)

I guess that if it does not find EC2_URL env var it uses AWS_REGION env var


Same comment as for the `styleSetProps()` method above.  This should not be a public method.


@meag - It was 8 years ago :) I do not recall exact reasons, but it obviously some privacy considerations (binded team_say vs manually typed in the console). I think I had some long conversation with Renzo about it, you probably could ask him on IRC (or PM on quakeworld.nu), maaay be he recall details.

As I got we decide what manual mm2 should NOT go in MVD/QTV in ANY case. And what you suggest somewhat conflicts with it.

Also, it looks like ezquake issue which you trying to fix with KTX? Isn't it a bit bad idea? :)


Ingbrzy this fix cause FC


Было бы няшно, если бы пейсать список доменов можно ыбо сразу сюда. Но увы. Доступна только строка с одним доменом.


@thomashan Could you remove this change? I will update the tutorial HTML accordingly.


Could you define and use a proper typedef?


This `clear` is not necessary.  `oldPropertyMapInstances` will be destructed anyway at scope end.


Oh yep - you're quite right. I got fixated on the isAbsolute without properly going any further. Ahem!


Это долгий разговор, но tl;dr:

* Пакеты должны быть `go get`-able;
* Зависимости не должны нуждаться в сборке;
* Для сборки не должны быть необходимы дополнительные утилиты.

Я тебе скидывал интервью Брайана Кернигана и Алана Донована, где они объясняют что такое diamond dependencies, что в Google они долго решали эту проблему, и решили зависимости просто версионировать. Поэтому ни в Google, ни в Go нет пакетных менеджеров — не существует правильного способа это сделать.

По той же причине, ни в одной серьезной компании никто не пользуется npm, pypi, Docker Hub, или любым другим hosted-решением. Максимум, естественно — локальный сервер. CI-сервер должен иметь возможность достать все, что нужно для сборки проекта из локальной сети.

Репозитории/пакеты удаляются, переименовываются, меняют лицензии. Это неприемлемо для работающей компании. Вы не хотите завтра узнать, что автор переименовал пакет, и сегодня вы не можете собрать вообще ничего и выкатить критичный багфикс. В срочном порядке нужно обновлять вообще все.

Хорошо, представим даже что это локальный, внутрикорпоративный репозиторий, который был твоей зависимостью. А автор его уже за-`force push`-ил, и ты не можешь собрать свой. Это еще неплохо, ведь даже если автор на другом конце света, то можно через пару дней как-то этот вопрос разрулить. А если вы не можете понять почему процесс падает в кору, а собрать с той же версией не можете?

В компаниях вроде Google вопрос стоит не только в версии зависимости, но даже версии системных библиотек и компилятора — там нужны reproducible builds, чтобы иметь возможность 1-в-1 пересобрать конкретный бинарник.

Когда мы скачиваем зависимость из npm, мы не считаем что нам нужно не забыть зайти в `node_modules/$depname` и собрать ее, верно? Не всегда, вон Facebook свой Flux Dispatcher очень долго в несобранном виде выкладывал. Они на полном серьезе говорили, что нужно зайти и руками его скопировать к себе в проект. И это был кабздец, честно говоря. Поставил зависимость, а тебе еще нужно не забыть ее транспайлинг настроить, ведь там ES6 и прочая красота.

Иначе говоря, вопрос не в том почему генерируемые файлы в Git-е, а почему все остальные зависимости еще не в нем.

Именно так делают Canonical с Go-сборщиком [gb](https://www.youtube.com/watch?v=c3dW80eO88I), а в Go 1.5 теперь есть GO15VENDOREXPERIMENT, который скачивает зависимости как подмодули в папку `vendor/`. Я лично до этого `GOPATH=$PROJECTPATH:$GOPATH` пользовался.


И, кстати. Для расширения (учитывая что оно работает на лисе 38 и выше) можно отключить регенератор, половину фич `core.js` и убрать кучу фич компилятора `babel`. Компилировать скрипт можно будет только во время обновления расширения.


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


"我們的一些學員在多倫多一起寫程式時的合影"
營員 is not a common word. Suggested translation: camper = 學員 (They are learning to code.)


Хотя в данном случае нам это и не надо. Делформа ищется везде.


this should go to: Compile.java:compileRegexp and use Frag to obtain start and startUnanchored indexes


I added this workaround because of https://github.com/packsaddle/ruby-uri-ssh_git/blob/12fb9d9d17dfc0ca76a756f118b4dfa963cdda68/lib/uri/ssh_git.rb#L35
But I can't remember why I do this.


Hmmm. Wouldn't the `O_SYNC` make things slower than doing `fsync` at the end?


If this function is not called outside Validations.cpp, then it does not need to be a public member in the base class.


Then put the file in var/ and link to it in web/adminer. Also check other packages for the backup/restore code.


Mais c'est une bonne idée d'etre passé par un <button> malgré tout.

Il faut pas que tu sois découragé de faire des betises, au début c'est bien normal. Il faut en revanche etre curieux, allez chercher des solutions sur internet, expérimenter, tester sans avoir peur de tout casser.
J'ai l'impression que tu as tout ca alors ca va bien se passer !


You mean the ``getenv()`` call above? Leftover TODO msg?


It is called from `PeerImp`.


Hihi, we discussed this here. Why do you prefer this? `decltype` seems to be as clear as possible to us in this context.


Update styles.css


Your work with root_path, "" path looks more like a hack here.
Also this function is overcomplicated already. Making it more complicated is a bad way.


Thanks :angry:.  This has broken `M-x list-packages` for me on Emacs 25.1…

Can please refrain from advising essential core functions?  That's a horrible thing to do, and a darn stupid idea, no matter what issue this could possibly fix.


Hello, I get stuck with the kfpcs. I modified the test_kfpcs_ia.cpp to test kfpcs works or not, but it seems doesn't work.
I have loaded two images to generate source and target pointclouds as the input. and the program stuck there without any return ). Can you tell me what's wrong? Thanks a lot.

source code:

//#include <gtest/gtest.h>

#include <pcl/point_types.h>
#include <pcl/io/pcd_io.h>
#include <pcl/registration/ia_kfpcs.h>
#include <opencv2/opencv.hpp>

#include "test_kfpcs_ia_data.h"

using namespace pcl;
using namespace pcl::io;
using namespace pcl::registration;
using namespace std;

typedef unsigned short ushort;
//typedef pcl::PointXYZI PointT;
//typedef pcl::PointCloud<PointT> PointCloud;

const double camera_factor = 1000;
const double camera_cx = 325.5;
const double camera_cy = 253.5;
const double camera_fx = 518.0;
const double camera_fy = 519.0;

PointCloud<PointXYZI> cloud_source, cloud_target;

//////////////////////////////////////////////////////////////////////////////////////////////////////////////////
void test()
{
  // create shared pointers
  PointCloud<PointXYZI>::Ptr cloud_source_ptr, cloud_target_ptr;
  cloud_source_ptr = cloud_source.makeShared ();
  cloud_target_ptr = cloud_target.makeShared ();

  // initialize k-fpcs
  PointCloud <PointXYZI> cloud_source_aligned;
  KFPCSInitialAlignment <PointXYZI, PointXYZI> kfpcs_ia;
  kfpcs_ia.setInputSource (cloud_source_ptr);
  kfpcs_ia.setInputTarget (cloud_target_ptr);

  //kfpcs_ia.setNumberOfThreads (nr_threads);
  kfpcs_ia.setApproxOverlap (approx_overlap);
  kfpcs_ia.setDelta (voxel_size, false);
  kfpcs_ia.setScoreThreshold (abort_score);

  // repeat alignment 2 times to increase probability to ~99.99%
  const float max_angle3d = 0.1745f, max_translation3d = 1.f;
  float angle3d = FLT_MAX, translation3d = FLT_MAX;
  for (int i = 0; i < 2; i++)
  {
    kfpcs_ia.align (cloud_source_aligned);
	std::cout << "kfpcs works " << std::endl;
    // copy initial matrix
    Eigen::Matrix4f transform_groundtruth;
    for (int i = 0; i < 4; i++)
    for (int j = 0; j < 4; j++)
      transform_groundtruth (i, j) = transformation_office1_office2[i][j];

    // check for correct transformation
    Eigen::Matrix4f transform_rest = kfpcs_ia.getFinalTransformation ().colPivHouseholderQr ().solve (transform_groundtruth);
    angle3d = std::min (angle3d, Eigen::AngleAxisf (transform_rest.block <3, 3> (0, 0)).angle ());
    translation3d = std::min (translation3d, transform_rest.block <3, 1> (0, 3).norm ());

    if (angle3d < max_angle3d && translation3d < max_translation3d)
      break;
  }

  std::cout << "test exits " << std::endl;
  //EXPECT_EQ (static_cast <int> (cloud_source_aligned.points.size ()), static_cast <int> (cloud_source.points.size ()));
  //EXPECT_NEAR (angle3d, 0.f, max_angle3d);
  //EXPECT_NEAR (translation3d, 0.f, max_translation3d);
}

//////////////////////////////////////////////////////////////////////////////////////////////////////////////////
int main (int argc, char** argv)
{
	std::string bathPath = "C:/Users/phd/Desktop/mrmorning/";
	std::string sourceDepthFile, targetDepthFile;
	std::string sourceRGBFile, targetRGBFile;
	cv::Mat sourceDepthImg, targetDepthImg;
	cv::Mat sourceRGBImg, targetRGBImg;

	std::string sourceID, targetID;

	std::cout << "*******************************IA-KFPCS***********************************" << endl;
	std::cout << "Input Source Image and Tatget Image IDs : " << endl;
	std::cin >> sourceID >> targetID;

	sourceDepthFile = bathPath + "depth/" + "frame_" + sourceID + "_DepthDataConvertedSmooth.dat.png";
	targetDepthFile = bathPath + "depth/" + "frame_" + targetID + "_DepthDataConvertedSmooth.dat.png";
	sourceRGBFile = bathPath + "rgb/" + "frame_" + sourceID + "_RGBImg.jpg";
	targetRGBFile = bathPath + "rgb/" + "frame_" + targetID + "_RGBImg.jpg";

	sourceDepthImg = cv::imread(sourceDepthFile, CV_LOAD_IMAGE_ANYDEPTH);
	targetDepthImg = cv::imread(targetDepthFile, CV_LOAD_IMAGE_ANYDEPTH);
	sourceRGBImg = cv::imread(sourceRGBFile, CV_LOAD_IMAGE_COLOR);
	targetRGBImg = cv::imread(targetRGBFile, CV_LOAD_IMAGE_COLOR);

	for (int m = 0; m < sourceDepthImg.rows; m++)
	for (int n = 0; n < sourceDepthImg.cols; n++)
	{
		// 获取深度图中(m,n)处的值
		ushort d = sourceDepthImg.ptr<ushort>(m)[n];
		// d 可能没有值，若如此，跳过此点
		if (d == 0)
			continue;
		// d 存在值，则向点云增加一个点
		pcl::PointXYZI p;

		// 计算这个点的空间坐标
		p.z = double(d) / camera_factor;
		p.x = (n - camera_cx) * p.z / camera_fx;
		p.y = (m - camera_cy) * p.z / camera_fy;

		// 从rgb图像中获取它的颜色
		// rgb是三通道的BGR格式图，所以按下面的顺序获取颜色
		p.intensity = d;

		// 把p加入到点云中
		cloud_source.points.push_back(p);
	}
	// 设置并保存点云
	cloud_source.height = 1;
	cloud_source.width = cloud_source.points.size();
	cout << "point cloud size = " << cloud_source.points.size() << endl;
	cloud_source.is_dense = false;

	for (int m = 0; m < targetDepthImg.rows; m++)
	for (int n = 0; n < targetDepthImg.cols; n++)
	{
		// 获取深度图中(m,n)处的值
		ushort d = targetDepthImg.ptr<ushort>(m)[n];
		// d 可能没有值，若如此，跳过此点
		if (d == 0)
			continue;
		// d 存在值，则向点云增加一个点
		pcl::PointXYZI p;

		// 计算这个点的空间坐标
		p.z = double(d) / camera_factor;
		p.x = (n - camera_cx) * p.z / camera_fx;
		p.y = (m - camera_cy) * p.z / camera_fy;

		// 从rgb图像中获取它的颜色
		// rgb是三通道的BGR格式图，所以按下面的顺序获取颜色
		p.intensity = d;

		// 把p加入到点云中
		cloud_target.points.push_back(p);
	}
	// 设置并保存点云
	cloud_target.height = 1;
	cloud_target.width = cloud_target.points.size();
	cout << "point cloud size = " << cloud_target.points.size() << endl;
	cloud_target.is_dense = false;

	test();

	system("pause");

	return 0;
}

//////////////////////////////////////////////////////////////////////////////////////////////////////////////////


Lige nu er den her altid disabled


Why the if? You already checked for NULL.


This could go before the function as a constant value, it doesn't have to be redeclared each time the function is called.


@lunaryorn is right. That's a very very bad idea...


Скопировал не глядя из туториала, это минимально поддерживаемая `jpm` версия. Можно исправить.


Ah my bad you are right!  If you already have the logic for it written up as you have here then don't worry you are not stepping on any toes; feel free to make PR with it.

Sent from my iPhone

> On Dec 1, 2015, at 8:06 AM, Richard Ashwell <notifications@github.com> wrote:
>
> Just an observation, I haven't made a pull request with my version of this type of addition to keep from stepping on peoples toes, but the ^ shouldn't be mapped to the current version of lineBegin. Instead unless my vim fu is suspect that should be mapped to 0 (zero). The ^ actually goes to beginning of first non white space. In mine I have created a seperate function in Cursor like below to map the ^ too:
>
>     static nonBlankLineBegin() : vscode.Position {
>         let pos = this.currentPosition();
>         const linetext = TextEditor.ReadLine(pos.line);
>         if (linetext.length !==0) {
>             var fc = linetext.length - linetext.replace(/^\s+/,"").length;
>             return new vscode.Position(pos.line, fc);
>         }
>         return new vscode.Position(pos.line, 0);
>     }
> —
> Reply to this email directly or view it on GitHub.
>


`fileInput.$.form` is faster :)


instead of MatchKind, RE2 instance should be stored here.
Assume:
re2.longest = true -> kind = LONGEST_MATCH
re2.longest = false -> kind = FIRST_MATCH


Come segnalato in precedenza. La traduzione inglese cita "New installed app"


Logcat:

java.lang.IllegalStateException: Target host must not be null, or set in parameters. scheme=null, host=null, path=
at org.apache.http.impl.client.DefaultRequestDirector.determineRoute(DefaultRequestDirector.java:596)
at org.apache.http.impl.client.DefaultRequestDirector.execute(DefaultRequestDirector.java:298)
at org.apache.http.impl.client.AbstractHttpClient.execute(AbstractHttpClient.java:560)
at org.apache.http.impl.client.AbstractHttpClient.execute(AbstractHttpClient.java:492)
at org.apache.http.impl.client.AbstractHttpClient.execute(AbstractHttpClient.java:470)
at com.android.quicksearchbox.xiaomi.XiaomiSource.query(XiaomiSource.java:178)
at com.android.quicksearchbox.xiaomi.XiaomiSource.getSuggestions(XiaomiSource.java:250)
at com.android.quicksearchbox.xiaomi.XiaomiSource.getSuggestions(XiaomiSource.java:55)
at com.android.quicksearchbox.QueryTask.run(QueryTask.java:68)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1112)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:587)
at java.lang.Thread.run(Thread.java:818)
at com.android.quicksearchbox.util.PriorityThreadFactory$1.run(PriorityThreadFactory.java:45)


True! Thanks! See https://github.com/Ableton/aqt-stylesheets/commit/d50f40b347f79c6ad10a7f09e5a0e24d31197374


HI phdsky,

please use our mailing list at http://www.pcl-users.org to ask questions.


C'est quand même bien décourageant de passer 3 jours à te casser les pieds pour réussir quelque chose, et qu'après en une heure tu as changé à peu près tout.


Please keep the essence of this comment with the `properties()` function in the previous commit.


PullCtrl har ikke denne funktion, synes godt det kunne være en del af denne her task når det nu er en html validator


Forms have native API to access elements, which is much faster than `querySelector` (`form.elements[ 'ckCsrfToken' ]`).


pullData tager 3 argumenter ind


Hm, yeah, there's a chance this might happen.


This line launches a migration POST with a unique migration url.


Je trouve que c'est une bonne méthode. On peut aussi utiliser la formule du produit. Comme souvent, il y a plusieurs façons de faire.


Would it be better to have a separate, I dunno, `contentRoot` property instead of assuming index 0?  The tile would still be in `children[]` and would need to be updated on unload and reload, but it is explicit and perhaps more obvious.


Hey, hi!
how about that: https://github.com/shirou/gopsutil/commit/4f6e4f73f3096777e6b48f69ebedae76c30a15d4


OK , Thanks a lot : )


this should go to DFA.java. MatchKind argument should be removed