Ah perhaps we are testing on different templates with different ping
versions?

On Monday, 7 December 2015, sanjeevneelarapu <notifications@github.com>
wrote:

> Field 7 is only giving packet as outpou. But 6 gives the actual percentage
>
> On Dec 7, 2015 6:37 PM, michaelandersen <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
>
> why has this been changed from from field 7 to field 6?
>
> —
> Reply to this email directly or view it on GitHub<
> https://github.com/sanju1010/cloudstack/commit/09409a7ac264e864c45146927b86229c8e67fbc8#commitcomment-14820240>.
>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sanju1010/cloudstack/commit/09409a7ac264e864c45146927b86229c8e67fbc8#commitcomment-14821144>
> .
>

--
Michael Andersen


can we take this chance to fix https://github.com/ceph/ceph/blob/master/src/rgw/rgw_op.cc#L1378 as well?


WlC


- "Sign a production app with a production certificate, not a debug certificate."?
- "Also make sure the certificate includes a sufficient validity period (i.e., won't expire during the expected lifespan of the app)."?
- "Google recommends that your certificate use at least 2048-bit encryption."
- "Make sure the keystore containing the signing key is properly protected. Also, restrict access to the keystore to only those people that absolutely require it."?


I'm getting OpenSSH Daemon errors on startup of OpenELEC, which then fails. On an ODROID C1+ when I upgraded to v2.2.4. Reverted and all is fine again.


:clap:


Done.  Thank you!


Can we have a full stop too?


Why `ol.layer.Image` ? If we want a tiled ?


Like


I think we still need it. Otherwise `elfeed-org` is forced in any case, even if user doesn't use it.


to use crypt() function correctly, you should add #define _XOPEN_SOURCE  or #define _GNU_SOURCE at the top of the source code, please take a look at http://man7.org/linux/man-pages/man3/crypt.3.html


This is a breaking change and should be reverted.


TeamCity Habitat :: Pull requests. Cruise [Build 6](http://build8ua1:8081/viewLog.html?buildId=845&buildTypeId=Habitat_PullRequestsCruise) outcome was **SUCCESS**
Summary: Tests passed: 109 Build time: 00:01:18


In trying to figure out frame overflow issues I'm seeing (e.g. #234) I started playing with this function. It bothers me a lot that adding arbitrary values here (say adding ` - 100` to the end of the line) shrinks the frame (as shown by showframe) but does not reflow the content in the frame. Also this function is newer called on the last page of the document even if there are footnotes. I don't know exactly what this is supposed to do, but something doesn't seem right about these results.


page-header


page-header__top после нужен отцентровщик .container


лого не может быть в меню, он в хедере. Далее привожу правильную структуру с классами -

              Информация

 далее остальные пункты меню по аналогии

Блог путешественников, пользователей Пинк


[CWE 310: Cryptographic Issues](http://cwe.mitre.org/data/definitions/310.html), [CWE-326: Inadequate Encryption Strength](http://cwe.mitre.org/data/definitions/326.html)


page-main


3 класса тут -  .container .page-main__inner .page-main__inner--blog


page-footer


.logo .logo--footer


ссылка logo__link, изображение .logo__image .logo__image--footer


.social-links__item .social-links__item--tw


весь хедер по аналогии как уже писал выше


I also cherry-picked this into 0.13


If you had configured root `welcome#index`, you don't need `get welcome/index` in here anymore, if you wanna go to index page, you can use `root_path`.


Why use BUFF + 25, BUFF + 75 and so on declaring vectors? There's no need for this since BUFF is not used in any other place all over the code. by the way, this use force the processor to calculate the value of "BUFF" plus another number like 75 and so on. It's not necessary once that there's no improvement


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


.page-main__inner .container, отцентровщик это просто класс container, а рядом идет класс для стилизации с модификатором если нужно


филдсеты начинаются со второго блока с заголовком, этот блок состоит просто из дивов без филдсета


nit, could use `dump_bool()` here.


form__wrapper form__wrapper--bg-gray


Nice routes configuration! :+1: .


There's no reason for declaring f (int f) every loop cycle over and again once the f was already declared outside the loop itself.


photo-item


page-main


сначала блок внутри container


сначала блок внутри container, блок лучше назвать price, rate чаще всего связано с рейтингами и оценками


```php
$members = $teams->pluck('nom');
```


price-table


.container .page-main__inner


please add a comment so we remember to remove this in the future


`public/functionPHP.php`

Ce code doit aller soit dans un contrôlleur (au pire) soit dans un modèle (au mieux).


Project seems to already extend [Cloneable](http://docs.oracle.com/javase/7/docs/api/java/lang/Cloneable.html).


Leave a footprint in the start of amazing Swift


test branch


Yes, forgot to remove that. Thanks for picking it up.


Please, use Fragments rather then Activities for different screens


TeamCity KB-QS :: content-editor :: build [Build 7](http://teamcity.nice.org.uk/viewLog.html?buildId=108786&buildTypeId=KbQs_Editor_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:07:01


>
> I wasn't arguing that the compiler should accept other file
> extensions—just that it's a bit weird that parsing stops if one of the
> files doesn't have the correct extension, but *doesn't* stop if parse(Path
> const&, Config const&) returns false for one of the input files (which is
> somewhat contrary to your intention to "parse all files and emit as many
> syntax errors as possible").
>
> Now that you mention it, though, there's something to be said for not
> requiring input files to have a certain file extension. For example, the
> C++ standard library headers are extension-less (though I know it's
> implementation-defined whether the library headers are actual files).
>

Sure... and if I had an unlimited amount of time, I might think of a better
way to handle this. In the meantime, it's fine as it is.


@MagicDroidX Well, I will test


Fixed #70


Not true, this has been added for a long time in https://github.com/stsundermann/gtk-sharp/commit/8e307d8eaa0db7cfb9aaac76324c24d7e9486db4


OK, you are right. I will prepare a special note in `NEWS.md`


I tend to only put .sh scripts in bin/, but that's not a strong opinion :)


Nice!


ah ok


oh sorry, I copied the wrong link :disappointed:


não sei se esse é o jeito mais correto de dar o return mas foi o que deu pra fazer


hm, ok let's go with that then. Let's see how it plays out.


This is incorrect I think, because you test an input character at an output position (i). I think I'd remove the white space in the input before translating it.


sure


Ok. Will update it when I have time. I just added this as a placeholder to suppress the warnings when building the site.


Do these files exist in the folder /assets/img/ ?


This might be helpful if you didn't see it already :-) https://github.com/adnanh/webhook-contrib/tree/master/docker


You are right. I will change it.


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


`GravatarImagesDirectory` -> `Directory`


Pretty old year


I'd make sure that it defaults to `NORMAL`, if e.g. the severity isn't contained in the string I don't want to mention it in the regex so I think defaulting to `NORMAL` would be good


"by the"


"of the"


:+1:

btw, do we have any upcoming raffle selling events?


Hey Linus, sorry about this commit. My Github broke and I couldn't commit to any of my projects anymore. It kept giving me an error message about 'limenet' not having permission.
So I thought it might be a key collision with you. Seems like that was the case because I can push to your projects nicely, just not my own!! wtf?


Fixed #59


- "Attackers can tamper with or backdoor apps and then re-sign them."


Also `"/GravatarImages/"` -> `/Cache` or something like that :)


Did you add changes to static/styles/src/wpp.admin.less ?
You must work with less files when it's required.

On any changes to wpp.admin.less file your direct changes to wpp.admin.css will be removed.
You must install [grunt](http://gruntjs.com/) and run `grunt watch` from wp-property root directory.
It will listen all changes in less files and do compilation ( generates new css files ) automatically.

You can look all css files, which are being compiled from less files in `gruntfile.js` :

![image](https://cloud.githubusercontent.com/assets/308489/11629995/ea9c32b4-9d03-11e5-9d7b-db2bcbafb368.png)


If you are not checking that you have at least one argument here, and do not catch an IndexError, then calling `pubs` here will display an horrible stack trace rather than a helpful usage message.


hm, I think it would be good to provide a `run` method and let the "user" just subclass it, override the `__init__` and leave the rest as it is


Hey there, this is a really really old version of the library, so I expect it is broken :) The latest code is here: https://github.com/orf/django-choice-object/blob/master/django_choice_object/choice.py


This program breaks contigs into 2 kb fragments with 100-bp overlapping regions.


Yes, thank you for your advise, I forgot to remove it. :D


Since *"Only the Paranoid Survive"* this should probably be a [ConcurrentHashMap](http://docs.oracle.com/javase/7/docs/api/java/util/concurrent/ConcurrentHashMap.html).


+1, we should have done this a long time ago.


m9, u need dis code


In some case, your text is too long to display, maybe you need to use [truncate](http://apidock.com/rails/ActionView/Helpers/TextHelper/truncate), your text will be slip with exactly length.


Current way of assigning read_socket and write_socket in
```
if (read_socket && read_socket[0])
```
returning false every time. As a result, the push method never evaluates the error returned by APNS, and notifications fail without returning any error.


що це?


We tend to use `} else {` in the project instead of
```
}
else {
```


The additional benefit of removing space in the input is that there is only a single word boundary (8) between two words, so you don't need the second part of this test.


Випадково додав?


This looks good.


TeamCity Habitat :: Pull requests. Cruise [Build 7](http://build8ua1:8081/viewLog.html?buildId=847&buildTypeId=Habitat_PullRequestsCruise) outcome was **SUCCESS**
Summary: Tests passed: 114 Build time: 00:01:25


we could restructure this without async also by using recursion and pop, for example, this would work:

```javascript
    module.exports.reset = function(done) {
        var collections = mongoose.connection.collections;
        var deleteProcessor = function(obj) {
            obj.deleteMany(null, function() {
                if ( collections.length > 0 ) {
                    deleteProcessor(collections.pop());
                } else {
                    done(null);
                }
            });
        }
        deleteProcessor(collections.pop());
    };
```


Okay, I will try this.


As far as I can see you still fetch this image after it was read from directory. Is there any reason/edge case to do that?


Yeah, but every time I run `grunt amsf-update`, it gets copied over. So I thought it's easier to just leave it here. Perhaps you can add `git checkout _deploy.yml` as the final step of `grunt ams-update` to keep the local version?


@vpisarev To prevent trash like this, 1 PR it is 1 commit to merge, no more.


Aren't we supposed to be passing the wrappers around, and the routines themselves unpack them with `#to_model`?


- "Attackers can tamper with or backdoor an app, re-sign it and publish the malicious version to third-party app marketplaces. Such attacks typically target popular apps and financial apps."?


is that the problem? the async library? what's wrong with it?


Oh, try removing the dot

so the css file is farmingtonrobotics.org/assets/css/hour-of-code.css

./something refers to something in the same directory, so in our case farmingtonrobotics.org/assets/css/something

if we just do /something, it refers to the root directory. The root is farmingtonrobotics.org, so that would be farmingtonrobotics.org/something


helal ama confirm de iş görürdü :smile:


logger.error


Check [this](https://docs.google.com/spreadsheets/d/1KAXHHfOZQ5_7_-VtrOkZcBWhAcUuIxTc_PUHB1QP6YM/edit#gid=0&vpid=A3) document. You could add the ones whose dates aren't in the past.


Drop the dot before `/assets` in each url and you should be good.

The dot tells the web browser to search for the image relatively based on the current file.

(Oh dang, Giselle beat me to it.)


:warning: seul MS Windows utilise des `\` pour séparer les différents éléments d'un chemin.

Et `include` est à bannir de votre vocabulaire PHP. PSR-4 s'en occupe pour vous: http://www.php-fig.org/psr/psr-4/


Also this appears to be a Lev design issue where there isn't a way to `run` subroutines with `perform_later` -- please try to be proactive about bringing these to my attention as I was lucky to catch this in a merge conflict


Note you can use `$(pgrep -d, ...)` which is handled already


Maybe a ``@warn`` fits well here.


Worth noting here for future readers that the next patch actually does the work this debug statement describes.


@danfowler Apart from fixing the Tabular Data Packages' schema, is there anything else remaining to commit this into master? I've been bitten by this bug :/


First of all, the BLAS implementations all disagree on their treatment of NaN. NaN behavior is not specified by the standard. Second, our behavior does agree with OpenBLAS.

The real reason though is that in the BLAS routines, a 0 scale factor means "pretend the data in this slice doesn't exist". Take Dgemm for example. It computes C = beta * C + alpha * A * B. If beta is 0, C is directly overwritten with the multiplication data. It seems that the BLAS commands should be consistent. Either a 0 value means "perform the floating point operation times zero", or it means "ignore the existing data". It seems like BLAS should be consistent in this meaning, which leads to the behavior here. If the scalar is zero, the data is overwritten.


+1 on this being an issue for us too @jDeppen @josevalim, we're upgrading an old app from 2.2.3 to 3.5.2 but don't have an `email` field on our resource.

Is there an issue or PR raised around this?  #3774 doesn't look like it would solve the issue for us, it would still blow up on `email_changed?`


This does not in any way make the notifications work .. @s-fanou was going through the code and made some general corrections along the way


захардкоджена айдішка?


aaa


Ok. It's more automatic and practical, but less flexible, and less deliberate. I think actually that a hybrid method, with the `path` argument as overriding conf.filename is best.


I think it might be worth to at least explain in the comments that this new `record.id` is the composite id, since you're removing the intermediate variables. (The whole matter of original id vs the "composite" one needed to make it globally unique for ember data is confusing enough; we shouldn't remove explanations for the sake of compactness).


Am I reading this wrong? Do you add this same signature twice? Seems like something the go compiler should complain about...


Внятное название, которое пишется в админке


This space is not needed.


Это не DEV точно, может что-от типа название твоего проекта. Что бы все модули были в кучке


Again *"Only the Paranoid Survive"*, to avoid race conditions better use:
`return projects.remove(checkNotNull(projectId)) != null ? true : false;`


Maybe s/data/vector element's data/


Delete this line.


Ditto.


Не то что бы ошибка, но так красивше


@lexmag when you can't beat them, join them :stuck_out_tongue:


This section seems like it should go near or be rolled in-to ##Managing new releases above.


s/is an is a/is a/


ADdasdasdasda


s/like/a/


:+1: lgtm


Тут будет динамический текст который лучше не переводить таким образом. Редактор повешается такое переводить


I'd love to be able to unit test this function. We should dig into the Ember Data source code to see how they test it (if they do).


1 - Ни одного коммента
2 - Было 3,стало 4. Чем это обусловлено ?
3 - Отступы


Ну ты понял да :)


good


Так более читаемо


Changed


rebase this out


I think this sentence could be easier to read if it was re-written as:

Origami modules are generally installed based on a [semver](link-to-semver-definition) range. To ensure new releases don't affect the current users of a module we have a set of requirements for non-major releases:


i don't have context on cloudcare, but...why are there responses that trigger the success function but are actually errors?


I think this is unnecessary. Simply, if `return lhs.priority < rhs.priority;` does not preserve FIFO order (maybe it does, I'm not sure right now), then `return lhs.priority <= rhs.priority;` does and vice versa.

And the operator can be integrated into the struct if you wish:
```C++
struct RQItem{
  Arrival* arrival;
  int amount;
  int priority;

  bool operator<(const RQItem& other) const {
    return priority < other.priority;
    //return priority <= other.priority;
  }
};

typedef PQUEUE<RQItem> RPQueue;
```
Because the default comparator is `std::less`.


All xEdit record names are singular if you haven't noticed with very few exceptions. I will update when CK is out.


and ``@c NULL`` on errors. It might be valid to add a ``@warn`` there telling not to store NULL pointers willingly on the vector, since it may suppress errors on this function (I used a custom pointer myself when coding one of the form nodes).


well, fly fucking again: much more clean not to if_add_queue before populate. Queue ifp only on populate success.

Regs,


@fracmak I personally don't have any issues with async, however I could agree with not bringing extra dependencies, if they are not utilized in the project enough, for example, in this instance we could remove it, if this is the only instance we will use it in, but if we are planning to restructure some of the code with async (which I would not like, I prefer to structure code by utilizing event, rather than utilizing async), also some people are more conscious about libraries that they use, and some companies have approval process for each library, so async is HUGE, and its dependency tree is enormous, and approval process for async is less than optimal.


Perhaps add a test here to check that the default config file was not created as well.


xD


A newline is needed.


Mentioned in Slack: Perhaps filter the filename the media is saved at when setting up the attachment to ensure something unique. VVV's test db still had the attachment from when it was uploaded by a WP phpunit run, which Travis wouldn't have had, but other devs likely may have.


No, this is the current version.  The comment (and the whole `if hasattr()` block) is still present and unchanged.  I found this commit via `git blame`. ;)
[django_choice_object/choice.py#L24@master](https://github.com/orf/django-choice-object/blob/master/django_choice_object/choice.py#L24)


Ditto for this and other functions returning void *.


Ditto WRT data part for these 2 too.


This change isn't on CPAN yet altough the version number (0.401) is the same.
I found out by having a redirect loop between a page and /login/denied


@heikoheiko the TypedStorage._valid_types is supposed to be a private class member so by convention it might be changed without notice. The idea is not to rely on any private member from outside the class. Do the project's code standards allow that?


Mentioned in Slack: Maybe add/remove a `pre_get_option_` filter the date-based upload folders option to dispense with the yyyy/mm folder.


If we switch to `ConcurretHashMap` better use [putIfAbsent](http://docs.oracle.com/javase/7/docs/api/java/util/concurrent/ConcurrentHashMap.html#putIfAbsent(K,%20V)).


Oh, that was a bug. Nice catch.


so - what if the section is reduced simply to " Resources and Sub-Resources MUST be referenced as plural nouns." @EasyAsABC123


Do you really want to add the CPU to the list of CPUs if you can't figure out whether or not it is online?


I looked at it, but I think all I need is cache control. When I have some images, I will just add one more line to set a long max-age for images. Seems not worthwhile to introduce more dependency just for this.


This function seems redundant to the `available` property, no?


removing the header results in implicit declarations of the guidance_h_module functions...


it's difference between a successful call to touchforms vs. a failed connection/500 error. this "error" can be something like not filling out a required question. i agree this isn't the correct way to do this. ideally we'd return a different status code, but that'd take refactoring on the backend. god knows how  much refactoring can be done in touchforms. it's a gold mine


So I will remove this tag.


Action take when a user is **in a** DNSBL:


My bad, you're right.


ah, ok, makes sense. What about this implementation? It maintains the async nature of the code (yours is serial), and avoids recursion

```javascript
        module.exports.reset = function(done) {
            var collections = mongoose.connection.collections,
                countdown = collections.length;
            collections.forEach(function(obj) {
                obj.deleteMany(null, function() {
                    countdown--;
                    if (countdown === 0) {
                        done(null);
                    }
                });
            });
        };
```


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


sorry that's not true, required questions i believe are handled differently, but it's the same idea. it's essentially touchforms was able to handle the request, but the computation resulted in some caught error


y u do dis! D:


Why is this good? On what it help? What i was lock on files, i dont see a reason for this except for someone it should looks nicer have double -- as #. but also i breake tabs lines, what is also zero affect on funkcionality, but something potencial nicer for something less nicer....


It has "arguments" because you talk about `the window dimensions of the current top-level browsing context`. I guess if you like my comment could be rephrased as "context". Or are you agreeing with me? I'm not sure :)


LGTM :+1:


asdfasdf


按先后顺序执行请求并返回结果


asdfas


I think we'd better tell the users how to handle this: to have data forming the list nodes as structs with the 1st members being a sol_list.


It doesn't really matter because those operations are presumably idempotent within a given step and therefore running them once or more than once is black-box equivalent. So you would expect a real implementation to elide the repeated call.


IIUC, these are caused by invalid user input to limits.cpu. It seems like we should report errors in all these cases, at least via log.Error or something.


Oh I apologize! I saw the `print` statement below and didn't think it was
the latest. Yeah, that comment needs updating, I need to find some time to
give this project some love I think.

On Mon, Dec 7, 2015 at 3:14 PM, Michael <notifications@github.com> wrote:

> No, this is the current version. The comment (and the whole if hasattr()
> block) is still present and unchanged. I found this commit via git blame.
> ;)
> django_choice_object/choice.py#L24@master
> <https://github.com/orf/django-choice-object/blob/master/django_choice_object/choice.py#L24>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/orf/django-choice-object/commit/4acf70d0b2bf56c50b0e7b9a82e6aa3fa74c723e#commitcomment-14823180>
> .
>


What do you expect.?


I think the better name is file_path


I might be valid to mention that only uint16_t integers are accepted.


Best name 11/10-ign


Or int16_t, it seems.


Fine as long as the UT passes with this.


Hey Rasmus,

Thank you very much for letting me know, I really appreciate that! I think I know what the problem is/was: I downloaded your [Vagrant box for PHP 7](https://github.com/rlerdorf/php7dev) a few days ago and to access one of my private repos (for PHP 7 testing purposes), I needed an SSH `git clone`. Since there already was an `id_rsa` private key, I [created a private key](http://askubuntu.com/a/53555) based on that one and [linked it with my GitHub account](https://github.com/settings/ssh). Thinking about that now, it was a pretty stupid mistake!

Now I don't know much about SSH but from the way I see it, that was the cause for this key collision (which, under normal circumstances, is extremely unlikely). I deleted/unauthorized that SSH key from my GitHub account. Could you please try to push to one of my repos again to make sure it's now fixed?

Sorry for the mess I've created! Let me know if I can be of any help.

Linus


Here and elsewhere, project setters can be chained:
```java
private static final com.google.api.services.cloudresourcemanager.model.Project COMPLETE_PROJECT =
    new com.google.api.services.cloudresourcemanager.model.Project()
        .setProjectId("completeProject")
        .setName("full project")
        .setLabels(ImmutableMap.of("k1", "v1", "k2", "v2"))
        .setProjectNumber(DEFAULT_PROJECT_NUMBER)
        .setCreateTime(DEFAULT_CREATE_TIME)
        .setLifecycleState("ACTIVE");
```


I really look forward to an RPM release with this commit. I am running Portus on such a network, and the usability of the UI is severely degraded when these assets can't be accessed.


Is this define doxygen-documented too?


ha, this reads like cc-odk Java


will do thanks


Here and above, I think you could use `clone` instead.


user's


What about extracting some kind of "utils::remove_char_from_string"?


Should this be `/**` ? Doesn't seem to be zipping the contents of the folder.


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


Nothing wrong :P, I just assumed that `TCPServer` was a new style class


my idea here is more general, but was not sure where to provide this kind of functions. I imagine this working in an easy, and for now simplified version of http://api.rubyonrails.org/classes/ActiveSupport/Inflector.html#method-i-classify I guess moving it to some kind of support module makes sense, but not sure where to put it in the context of the actually plugin manager layout. ideas?


I think you only ever write to this, not read from it.


This function can be trimmed down a bit - check all the code which is duplicated and squeeze it


ok


Replaces the tuple with a class containing only a constructor and public fields.


I'm pretty sure that's not your name ;)


great thing you have this "cluesMatrix'!


I agree


I have some sympathy for that thinking.  It’s also clearly stated in the beginning of the spec that the algorithm steps are just provided as guidance, and that what matters is the implementation provides a similar _output_ as the steps would provide; so optimisations are allowed.


lol


hooray


Related issue: https://github.com/PTAnywhere/ptAnywhere-widgets/issues/10


I don't understand what this is all about. Isn't this a local variable that's never used again?


It might be valid to mention that dst must have enough space to hold the contents of src.


isn't it weird thait doesn't raise anything?


do pice to jak je napisane?


Hello, this is rather inconvenient for testing (mocking specifically). What exactly is the issue here with letting Guzzle write the file?

Regards
Rene


You need to share your config file


double again?
"a similar thing"


Should be `type`.


@chuckbook yes, more control over the RTC initialisation is a good idea.  Might even want control to select (and reset) LSI or LSE manually.

Feel free to provide a PR for bypass mode.

Good to hear about the TCXO's.  I'll keep it in mind for the next run of boards.

And I thought of another way to improved robustness of RTC auto startup: the issue with DFU is that the RTC is not even clocking, even though the registers say that it's initialised.  This is easy to detect: you just have to see if the RTC sub-tick-counter changes or stays constant in an appropriate period of time (measured by the HSE, eg pyb.delay(5)).  It adds a bit to the startup time, but can be optimised because the value of the RTC clock is all zeros in this DFU reset case, and one can check for that before waiting to see if the RTC ticks over within 5ms.


Yeah, this should work instead of #1 :-)


Even worse now. Should be

```php
return empty($args[0]);
```

in my mind... See #1


@jangregus kurva to si co zase spravil? ta vyjebem z firmy este raz daco dojebes


You already have a `KeyEventDispatcher` here so you don't need it in the `GameUI` class (or the other way around


@adityamenon great! I will merge! :)


枠
馬番
うま名
性齢
馬体重
増減(text)
騎手名
斤量
父馬名
母馬名
母父馬名


enne


Дополнительно. Можно ли обойтись без этой переменной?


ah, got it, thanks


I'm glad it works, but I don't understand. If I'm correct, the code does exactly the same. It just doesn't use the `Pathname` class


这个地方漏了。。。传给 log4js 会自动打出堆栈


@slavapestov Sorry, but I don't know.  I didn't test it with cycles longer than this case here.


si uplny kokotko alebo co robis? :D dopice ved mne prislo asi uz 30000000000000000000000 mailov kazdy s nejakou tvojou jebnutou picovinou co tam jebnes


Oh, never mind. I get it now.


LGTM :+1:


Hm, I will check


@naholyr c'est du point de détail mais si tu peux mettre un nom de message de commit en rapport avec le post: `Post : object literals - fixed typo`


missing port 3000


Sorry :confounded:

Tired brain saw "new statement", not "function argument".  I can revert it in another pull request if you want.


@fracmak this approach opens a possibility for a race condition, where last call completed before previously issued delete requests, in this instance if you want to implement parallel processing, and guaranteed completion of every queued function, then we would need to add second internal callback function that would account for every callback.


IIUC, doesn't this have the same problem as before, where if I set limits.cpu = 2 for 20 containers, all 20 get scheduled on the first 2 CPUs, instead of spread across all the CPUs?


should be just srcset - we don't need the picture tag since we are only changing resolutions and not adding different pictures for art direction - https://css-tricks.com/responsive-images-youre-just-changing-resolutions-use-srcset/


Why not `new RiakNode.Builder()` ?


For the two functions _at(), it's not really "insert", that gives an idea of shifting things around. That's why it had "set" before.


Thank you @DeveloperNube for implemented "npcshopclosed" packet parser.

Now we can perfect fix #790 and #652 .

###Step 1.
implemented "npcshopclosed" packet parser
https://www.dropbox.com/s/153z5hqnklql2wu/callshop.patch?dl=0

###Step 2.
find out script.c
```
	} else {
		st->state = CLOSE;
		st->mes_active = 0;
	}

	clif_scriptclose(sd, st->oid);
	return SCRIPT_CMD_SUCCESS;
}
```
change to (thank you @aleos89 in https://github.com/rathena/rathena/commit/cef2f9ea63aabd45bc5dda294a37fe2f96821275)
```
	} else {
		st->state = CLOSE;
		st->mes_active = 0;
	}

	if (!sd->npc_shopid) // Don't close if a shop was opened.
		clif_scriptclose(sd, st->oid);
	return SCRIPT_CMD_SUCCESS;
}
```

###Step 3.
find out script.c
```
	if (sd)
		clif_scriptclose(sd, st->oid); // If a menu/select/prompt is active, close it.

	return SCRIPT_CMD_SUCCESS;
}
```
change to (thank you @aleos89 in https://github.com/rathena/rathena/commit/cef2f9ea63aabd45bc5dda294a37fe2f96821275)
```
	if (sd && !sd->npc_shopid) // Don't close if a shop was opened.
		clif_scriptclose(sd, st->oid); // If a menu/select/prompt is active, close it.

	return SCRIPT_CMD_SUCCESS;
}
```

Done!


![snip_20151207152921](https://cloud.githubusercontent.com/assets/986185/11630897/7f5a904c-9cf7-11e5-88ff-f7099f2899cd.png)
![screen shot 2015-12-07 at 15 30 38](https://cloud.githubusercontent.com/assets/986185/11630896/7f586fb0-9cf7-11e5-833e-6f74714f83c2.png)


You should be able to remove references to the label_filter and menu_filter as well – those are, if anything, more dead than the rewrite_filter is (they just don't have visible on-page effects).


я думала оставить или helping или filtered, мне казалось что лучше фильтеред, но вообще могу =)


ma jebes uz @jangregus takychto buerantov jak si ty tu mame kopu ja ta vo firme nechces ked mi zatvaras issues jak keby si bol dajaky manazeris do pice.
som ti napisal ze to mas opravit a ty si s kludom zavres issue jak kebyzez sa nic nedeje :D


Next week we are going to do that.

@jordimassaguerpla I've added a card inside our our backlog about that.


Here's another TypeScript oddity I couldn't quite figure out:

I'm creating a function, `memoized`, and assigning properties to it (in this case, `cache`).

If I declare or reference it as `memoized.cache`, the compiler gives me this error:
![image](https://cloud.githubusercontent.com/assets/389077/11630871/12ba92b6-9cc5-11e5-8405-2f7eb6216099.png)

So I ended up declaring & referencing it using the `memoized['cache']` notation. Is there a better way to handle this?

/cc @nicknisi on #1


suggest adding isTest() to Source class


Done.

This was very good because I realized something about the way I used the image gallery in the showcase that was not ideal. So if at all possible, let's not just revert but amend 0c6aefe4482fb02d16858221d52cbb9ea578ec20, if necessary.

I would like to do the *Features* section differently (without screeshots), but we should discuss that in the forums, I guess. It's the better place for that. Also, the blogs screenhots need to be optimized, but I don't have time for that now.


Review this sentence.


replaced


Thank you very much.


--
Manfred Touron
Trucs à la con
08 36 68 65 65


This breaks my email logging in log file


You didn't remove the /imu_compass_asl.cpp from the cmakelist file. This doesn't compile


A very neat little trick you could do here is create something like a `Map<Character, String>` which will hold entries like:
```
Map<Character, String> icons = new HashMap<Character, String>();
icons.put('*', "wall.png");
icons.put(' ', "lane.png");
// etc.
```
This, you can place in the `FluffyFileReader` class for example (or in a different one) and after you fill this map with your icon file names, you will be able to write:

```
for (int i = 0; i < rows; i++)
			for (int j = 0; j < colums; j++) {
				gameLabelMatrix[i][j] = new JLabel();
                                gameLabelMatrix[i][j].setIcon(icons.get(gameMatrix[i][j]));
                                gameLabelMatrix[i][j].setText(String.valueOf(gameMatrix[i][j]));
 }
}
```
What do you think?


Does thsi version resolve the problems discussed with the previous version? https://www.doomworld.com/vb/freedoom/66837-burning-city-f-sky2/


I'm not following, the done() callback is happening inside the deleteMany's callback. Why wouldn't it be done with the delete at that point?


NUL was right.


I figure after almost three years of that being untrue, it's time for it to go.

On a more substantive note, there is a problem with libffi with enums; expand-to-foreign-dyn-indirect doesn't expand them correctly. I have a hackish fix in which the expansion looks correct but it doesn't execute correctly. Very odd.


Cool... le dépôt est bientôt à moi :-)
Le 7 déc. 2015 16:32, "Manfred Touron" <notifications@github.com> a écrit :

> Manfred Touron
> Trucs à la con
> 08 36 68 65 65
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jeannedhack/programmingChallenges/commit/a17d308cb1d2b1e75dfb542dbaa8849974a1c54c#commitcomment-14823678>
> .
>


could be a constant to avoid creating a string for every request, not even used in most of them


Isn't this just duplicating ItemStack.isSimilar() ?


are you TDD-ing? Come on, we're practicing. Please be strict with yourself


Good class design, non-existing package design - why?


you can call this FAKE_GOOD_VERSION but that is a micro micro nit.


Yeah, it always confused the heck out of me, and apparently I wasn't alone :)

While converting to the new names I found 2-3 places where we were using the wrong ones, mostly on container copy or snapshot restore, basically leading us to set the local container config of the new container to the value of the expanded config of the source. The result is a working container so we never really noticed, but it does mean that any change to the profiles wouldn't affect the container...


Только вот дошёл прочесть вдумчиво.
А как связаны генерируемый файл и зависимости? Про go-way с версионированием я внутренне согласен (тут дизайнеры Go следуют своей всюду проглядываемой логике: если для проблемы все решения плохи, то она останется без решения). Но при чём тут генерируемые файлы? Если мы
обеспечим reproducible генераторов, то и генерируемое совпадёт. Генерируемые файлы иногда бывают огромны -- сотни мегабайт (в компиляторной разработке -- AST, к примеру).
Можешь кинуть ссылку/пример, где это обычная практика (включать ген. файлы в репозиторий; видео не смотрел, там есть?)? Единственное, что я придумал -- их включают как раз для контроля изменений внешних зависимостей.


Well, if I clone this repository as-is, it doesn't build, and that's the error message... Do I have to regenerate the bindings or something? Maybe I'm misunderstanding how to use this...
I'm trying to build because ubuntu ships a super-old version I think? The unofficial AUR package for arch linux uses http://cgit.freedesktop.org/gstreamer/gstreamer-sharp/snapshot/gstreamer-sharp-1.4.tar.gz, but it doesn't build without modification either, because it expects gmcs... What is the correct upstream to use to get the newest version?


Drat, I thought I had already committed this to develop and here it is sitting on an unfinished feature branch.

Looks like it changes the actual behavior but is compatible with the documented behavior. (It was documented that the functions should return the new state, but in practice the return value was not always used.)


works for me.


K, no problem :) I merged it in.


@phlax, what about set priority to `1.0` for all units and than update only changed priorities?


Thanks Tony,
I downloaded the zip file from 'browse files' above, the change is in js/grid.grouping.js but not in js/jquery.jqgrid.js.  I'm not at all familiar with the build process, there is a gradle.bat, I tried running that, ang got the following:

Exception in thread "main" java.net.UnknownHostException: services.gradle.org

Are there parameters that need to be specified?  Or do I just manually concatenate all the grid.*.js files?


Good point, this was a quick hack!


Va fan!


Mäh, tjockis!


@adityamenon New version is up on atmosphere.


JIRA missing in commit message
reading "generic parameter" in commit message, I wasn't expecting this kind of change (more a Java generic change), I'd rather say "constant for ps parameter"


TeamCity Habitat :: Pull requests. Cruise [Build 8](http://build8ua1:8081/viewLog.html?buildId=848&buildTypeId=Habitat_PullRequestsCruise) outcome was **SUCCESS**
Summary: Tests passed: 114 Build time: 00:01:15


Oh yeah, I can probably untangle a rebase where I merge both the CPU commits together if that'd help.
Or at least rename that commit to make it clear that this only introduces the monitoring bits and that the next one adds the actually balance logic.


First Commit!


Code formatting: opening braces should be on the same line as the method signature


perfect


Yeah, but lets not get the `active_support` deps for now.. ;)

Since this use is localized in the template, I am fine to define it in here..


I am merging this to develop now, since it doesn't seem to have much to do with the branch it's on (which I want to rebase and PR).


I'll edit this commit to fix it. Looks like a rebase issue to me and since this gets renamed to CGroupSet in the next big commit, I probably didn't notice.


why not using MAX_LIMIT here ?


Yes. cpu0 and any CPU which doesn't support hotplugging won't have the online file. Lack of online file means that the CPU is online but cannot be hotplugged.


You'd better check this out: https://github.com/pbiecek/archivist/issues/162

2015-12-07 15:56 GMT+01:00 Witold Chodor <notifications@github.com>:

> OK, you are right. I will prepare a special note in NEWS.md
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wchodor/archivist/commit/bbfdee7bc5ff217919f0f7502627117e98873648#commitcomment-14822723>
> .
>


OK as discussed on IRC removing space from the input won't work either of course because it will mess up the translation. As for where to do the word detection and where to remove or insert spaces (input, output or both), I don't think you need to break your head to much because as said in practice it will make little difference. As long as it's properly documented so there can be no confusion, it's fine.


只传 string 的话 log4js 会包一个 error 来打堆栈？


why not using MAX_LIMIT here?


no there shoud be only AssertionError


For future translation concerns, I'd rephrase to say "You just reference
Resources and Sub-Resources as plural nouns."

Not that we're translating right now, but consider it future-proofing. :)

On Dec 7, 2015, at 9:15 AM, Kris Chant <notifications@github.com> wrote:

so - what if the section is reduced simply to " Resources and Sub-Resources
MUST be referenced as plural nouns." @EasyAsABC123
<https://github.com/EasyAsABC123>

—
Reply to this email directly or view it on GitHub
<https://github.com/salesforce-marketingcloud/SFDC-MC-REST-Style-Guide/commit/ff400fac2450019d1135ab211f234871b935135b#commitcomment-14823206>
.


Nice catch, will drop it. It was used in a previous iteration of the balancing code, before I moved to the sortable struct list.


This line here causes your `NullPointerException` when the program starts.


HI,

I am getting following error when I use it the without the fopen(..., "w"):

PHP Fatal error:  Uncaught exception 'RuntimeException' with message 'Unable to open C:\Users\h44z\Downloads\seaf-php\seaf-php\test\DL_1449502425.txt using mode r+: fopen(C:\Users\h44z\Downloads\seaf-php\seaf-php\test\DL_1449502425.txt): failed to open stream: No such file or directory' in C:\Users\h44z\Downloads\seaf-php\seaf-php\vendor\guzzlehttp\psr7\src\functions.php:275
Stack trace:
#0 [internal function]: GuzzleHttp\Psr7\{closure}(2, 'fopen(C:\\Users\\...', 'C:\\Users\\h44z\\D...', 283, Array)
#1 C:\Users\h44z\Downloads\seaf-php\seaf-php\vendor\guzzlehttp\psr7\src\functions.php(283): fopen('C:\\Users\\h44z\\D...', 'r+')
#2 C:\Users\h44z\Downloads\seaf-php\seaf-php\vendor\guzzlehttp\guzzle\src\Handler\StreamHandler.php(141): GuzzleHttp\Psr7\try_fopen('C:\\Users\\h44z\\D...', 'r+')
#3 C:\Users\h44z\Downloads\seaf-php\seaf-php\vendor\guzzlehttp\guzzle\src\Handler\StreamHandler.php(108): GuzzleHttp\Handler\StreamHandler->createSink(Object(GuzzleHttp\Psr7\Stream), Array)
#4 C:\Users\h44z\Downloads\seaf-php\seaf-php\vendor\guz in C:\Users\h44z\Downloads\seaf-php\seaf-php\vendor\guzzlehttp\guzzle\src\Exception\RequestException.php on line 51

Somehow guzzle tries to open the file in read mode...


and anything means raise incoming exception


so lets say you have 4 collections that you are deleting, then forEach would iterate through each of those 4 collections, and issue a command to another entity (in this case database server), one after another (without waiting for completion call), with registered callback, and lets pretend that 1st collection has million records, and rest of the collections have few hundred, so what will happen is that 1st collection would take lets say 10 seconds to get deleted, and 2, 3, 4 would take 1 second, so in this implementation, your done() callback is going to be called before 1st collection is completed, it relies on last collection to complete last, which is not going to be the case in some instances.


Not sure what's in that package, but gstreamer# master + gtk# master compiles and works fine. Master is no longer using gmcs but mcs which is what recent mono versions have.


Code formatting: everything between .filterQuery() and .end() should be indented to an additional level deeper, as they represent an embedded logical level. It's easier to grasp it on first sight that way. See testExample1().


Are they also things you're worried about crashes in, or are they just dead?  Because I want to make as few changes here as possible.


It is the code text. We have removed the translation.


On Mon, Dec 07, 2015 at 07:35:57AM -0800, Stéphane Graber wrote:
> Oh yeah, I can probably untangle a rebase where I merge both the CPU commits together if that'd help.
> Or at least rename that commit to make it clear that this only introduces the monitoring bits and that the next one adds the actually balance logic.

No worries, once I figured it out it was fine. Just left a note there
in case hallyn reviews :)

> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/stgraber/lxd/commit/c0894a46d15953f466382260edcb5fc7ddf25bbf#commitcomment-14823797


I wondering why you choose `reviews.create` in here, you should use `review.build(review_params)` to initial object you want, don't persist it by create. In some case, you have to handle result of `@review.save` .


000-bugg ovanför Nyheter i index.php


Changed to "for" in next commit. Two "fors" in succession still scans better than "targeted to".


I don't think so, I'll issue a pull request for this branch.


Yeah, sounds reasonable. We certainly shouldn't fail to rebalance in that case but reporting the error is fine.
Ideally any such error would lead to us adding proper checking in validateConfig() so that we can catch them when the user sets the config as opposed to CPU rebalance or container startup.


On Mon, Dec 07, 2015 at 07:37:40AM -0800, Stéphane Graber wrote:
> Yes. cpu0 and any CPU which doesn't support hotplugging won't have the online file. Lack of online file means that the CPU is online but cannot be hotplugged.

I see, ok. A comment to that effect would be useful.

> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/stgraber/lxd/commit/3e0fba234173cd2faa2c79315ef87ecfe9f9279e#commitcomment-14823842


@seddy we do want to fix it and i agree the patch wouldn't solve it. It should just be a better of adding a `respond_to?(:email_changed)` check.  Can you please send a PR and copy me on it?


for whitebox testing this is totally okay. you need to deal with the internals to some extend. e.g. in this case it's actually important that the test reflects updates of that list.


bad commit message :-1:


No, it doesn't. See the sort.Sort(usage) call. The usage CPU list is sorted from least used CPU to most used CPU.


On Mon, Dec 07, 2015 at 07:39:59AM -0800, Stéphane Graber wrote:
> Yeah, sounds reasonable. We certainly shouldn't fail to rebalance in that case but reporting the error is fine.
> Ideally any such error would lead to us adding proper checking in validateConfig() so that we can catch them when the user sets the config as opposed to CPU rebalance or container startup.

Yep, some code in validateConfig() would be good, but not really
necessary to merge this.

> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/stgraber/lxd/commit/3e0fba234173cd2faa2c79315ef87ecfe9f9279e#commitcomment-14823914


@ph I will not bring active_support in, I'm not crazy for now. But having classify alike method here makes sense as we should generate this patterns for the templates.


Do you foresee a use for this in tests?


I guess there is a type in commit message
"WS using SearchOptions ~~h~~ as ~~a~~ page size limit of 500"


This `0` here is a magic number. You should create constants for which players turn it is, something like:

`public static final int PLAYER_1_TURN = 0;`
`public static final int PLAYER_2_TURN = 1;`
or something like that


Was going to fix "keep in mind", but that already appeared at the start of the first paragraph.


hmm, still not understanding. I only decrement the countdown on completion of each deletion. So it doesn't matter what order the deletions happen in, it'll only call done() when the countdown reaches 0. I could rename countdown to "remaining" to make it clearer.


As above: do you foresee a use for this in tests?


this change seems unrelated to this commit
in addition, why removing it?


Would be great if you define IT in other module, as this can be tested in a more properly way with more parameters.


Also, apparently you have this line here which makes the previous `playerTurn = 0` totally useless


Arg(Arg()) and Arg(Const()): minor code duplication. Can we avoid it and get most of the performance by declaring the function that handles CFGArg as `inline`?


Oops, english back :)


unit tests for added limit missing for most actions


what is `a`?
what is `b`?


Couldn't work out what you were trying to say here - so went with what I might have said.


can just write:
```Swift
return getMathExpressionFromInput(inputArray).stringResult
```


On Mon, Dec 07, 2015 at 07:40:52AM -0800, Stéphane Graber wrote:
> No, it doesn't. See the sort.Sort(usage) call. The usage CPU list is sorted from least used CPU to most used CPU.

Oh, heh. I just assumed that sorted by id :). Sounds good.

> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/stgraber/lxd/commit/3e0fba234173cd2faa2c79315ef87ecfe9f9279e#commitcomment-14823934


You said try to reduce the nr of global variables!!


Recuerda que el segundo parámetro que se le pasa a view es un arreglo asociativo.
Por lo tanto lo que tienes que pasarle como segundo parámetro debe ser algo así:

```php
$this->load->view("home", array("usuario", $this->usuario));
```


I think this class is more that a bunch of tests for `LocalResourceManagerHelper `, basically you are testing the whole `rpc` layer, should it be called something else?


So the `uint-js` module doesn't have a TSD definition. As such I couldn't use the `import { UInt } from 'uint-js';` notation. Is this the only way to import NPM modules that don't have a type definition file, or is there a better way?

#1


Or constants don't count?


Can you add a comment to avoid deleting this line? It seems we're keeping this for compatibility.


I think this can safely be a `String[] gameType` not an `Object[]`


This seems to be a class utility method -- should be static, yes?


Also a comment over this line, would be great.


What code would have the better behavior?


I believe this method should be rename to `getIndexOfOperatorFromInput`


Esta asignación está mal. ```php $this->usuario->encontrar() ``` no devuelve nada (fíjate en la función). Por lo tanto **nada** queda asignado a ```php $this->b```


Improved readibility (I hope).


What is this doing in the maintenance....


should be `fromIndex:toIndex` to avoid confusion. `first` and `last` have different meanings


After this merge, I'm seeing lots of errors like this:

    -- Performing Test VCL_HAS_BOOL
    CMake Error at CMakeLists.txt:13 (add_executable):
      CXX_STANDARD is set to invalid value ''

Do now need to manually set CXX_STANDARD somewhere?


윈도우즈 깃을 까실때 commit as is, push unixpath 로 하시면 거의 문제 없어요.


should we set `self.priority` here too?


Redundant if it's in the Factory already.


@rodrigopavezi Just wondering, what does this section do?


You already have a class variable with this exact name and value


Thank you so much!


Why can't this `aux` variable be local?


Right, true. It's something that we have to watch out for but we would have had to come back when the model changed and add the new data or remove old data members either way.

This way doesn't use node and it also decoupled from the server code.


@rodrigopavezi This should be `prod` and not `test`. Otherwise we end up with a test version that doesn't work with in-app Storage in rva.risevision.com.


Good catch.


Shouldn't pauseAccount be: public static method pauseAccount(Account $account) on AccountFactory?


@josevalim - sure thing, will send a PR through if it's helpful :)


Updated to use {} notation


@weltling this might be a ABI break? anyway, I committed this into php7, since this is only used in our own extensions.. what do you think?


Ohhh, is this why they used `!=` because it'll turn zeros into falses into nulls into undefined etc?

@esad, can you open a new issue about the code that's in there now. Make sure it describes exactly what goes wrong and show the code that fixes it.

I still don't fully get how we can change the order of things and expect everything else to work the same though. Can you help me understand that in a comment on the new issue?


I had to specify `email: "someone.else@gmail.com"` because the factory was creating `user` and `other_user` with identical information, causing the following error:

```bash
Failure/Error: other_user = create(:user)

     ActiveRecord::RecordInvalid:
       Validation failed: Email has already been taken
     # /usr/local/rvm/gems/ruby-2.2.3@rails424/gems/factory_girl-4.5.0/lib/factory_girl/configuration.rb:14:in `block in initialize'
     # /usr/local/rvm/gems/ruby-2.2.3@rails424/gems/factory_girl-4.5.0/lib/factory_girl/evaluation.rb:15:in `[]'
     # /usr/local/rvm/gems/ruby-2.2.3@rails424/gems/factory_girl-4.5.0/lib/factory_girl/evaluation.rb:15:in `create'
     # /usr/local/rvm/gems/ruby-2.2.3@rails424/gems/factory_girl-4.5.0/lib/factory_girl/strategy/create.rb:12:in `block in result'
     # /usr/local/rvm/gems/ruby-2.2.3@rails424/gems/factory_girl-4.5.0/lib/factory_girl/strategy/create.rb:9:in `tap'
     # /usr/local/rvm/gems/ruby-2.2.3@rails424/gems/factory_girl-4.5.0/lib/factory_girl/strategy/create.rb:9:in `result'
     # /usr/local/rvm/gems/ruby-2.2.3@rails424/gems/factory_girl-4.5.0/lib/factory_girl/factory.rb:42:in `run'
     # /usr/local/rvm/gems/ruby-2.2.3@rails424/gems/factory_girl-4.5.0/lib/factory_girl/factory_runner.rb:23:in `block in run'
     # /usr/local/rvm/gems/ruby-2.2.3@rails424/gems/factory_girl-4.5.0/lib/factory_girl/factory_runner.rb:22:in `run'
     # /usr/local/rvm/gems/ruby-2.2.3@rails424/gems/factory_girl-4.5.0/lib/factory_girl/strategy_syntax_method_registrar.rb:20:in `block in define_singular_strategy_method'
     # ./spec/controllers/goals_controller_spec.rb:69:in `block (3 levels) in <top (required)>'
```

Any ideas what would be causing that? It's not a problem I remember having before.


```php
<h1><?php if(isset($usuario)) echo $usuario->getNombre() . " " . $usuario->getApellido()?></h1>
```


I think you mean "but its assumption", other than that this looks fine, and documents why we need to so this ourselves in this case. Thanks.


Compiling a regex for a "regular" string, and not using it "many many" times (really imprecise here, I know) is usually slower than using simple string operations. Nonetheless, this is pretty minor, and the filter functionality is neat. So, ACK!


@Maxopoly this is a huge step in the right direction, awesome.

I know a few common utilities are:

containsAtLeast(List<ItemStack>)

remove(List<ItemStack>) or perhaps better -- remove(ItemMap)

I could envision using a few variations to getDifference() --

isContainedBy(), testing if this map fully contains the passed map

canFit(Inv) testing if this map can fit into the passed inventory

Do you envision any way to specify an ItemMap directly in config? If so, might be worth looking at ConfigurationSerializable, as you can very easily define it in that fashion.


Also, s/executabe/executable


> How different are they?

The build command puts the binary just in the working directory while install puts the binary into `${GOPATH}/bin/`


An interesting discussion on the forum regaring the use of the current implementation
https://our.umbraco.org/forum/developers/api-questions/72856-override-defaulturlprovider-not-working-when-new-constructor-is-used


This is also code text from CCDA


limit is tested in SearchOptions


I'm not sure I should test all actions for the limit mechanism, the same way the limit was not tested on all actions


You can't assume the GRUB architecture is i386-pc.  It could also be i386-coreboot.  See commit d1bd566da0a06bb0c3e1aab5cd5a8965994a286f.


the file is deleted because it was not used anymore


WoW ! You find the Docker module :eyes:
This first release v0.0.1 displayed a simple logo. I should release a real version this weekend.


@bcoke I think we're nearing 'functional prototype' levels! :sparkles:


Good catch, but soon I'll push more mode options.


Исправил

19e0878


That's why it has `cp '$(GOPATH)/bin/gogs' `.


We should also not restart Camel context if not necessary, as it reinitializes all resources, like connections. Is there any reason why would you like to do it?


Also have a problem in switch Network data when we switch three times or more.


Всё чотко, кроме нереального кол-ва знаком табуляции которую я говорил **не использовать**!

> Засчитываю, но исправить можно.


This didn't work as expected. Fixed in a later commit.


@Qu4tro
Temos de fazer uma gramatica para planear audições, é assim que na plataforma do nosso trabalho serão inseridas as audições.
Esta é a minha tentativa acompanhada de um exemplo de teste.
As strings estão a escaxar não percebo pq :(


Hurray ! Thank you @sebastianbergmann !


Only way I could get the embedded backticks to show up correctly.


K


done by yourself ;)


removed


@Qu4tro

Todos os commits que fiz neste dia são relativos a extrair a informação do xml e colocar numa bd SQL. Também fiz forms para alterar, remover e adicionar informação (na bd).


@ssalinas24 It makes the var names longer, but I've been trying to put `-color` at the end of anything that is a color, since we also have vars for things like font names, margin measurements, etc., the idea being that someone seeing the var name in isolation can understand what kind of thing it is defining.

So in this case, `@rxDatePicker-background` would become `@rxDatePicker-background-color`, `@rxDatePicker-border-top` would become `@rxDatePicker-border-top-color`, etc. In the cases of `border-top` and `disabled-text`, it's easy in isolation for someone to misunderstand what those define -- the `border-top` could be a measurement, or `disabled-text` could be a font face definition. Adding `color` provides some clarity, in my opinion, even if it makes the names longer :+1:


You have just duplicated the dependency from devDependencies, that doesn't do anything.
You'd need to remove `^` – see https://docs.npmjs.com/misc/semver

Anyway, this was fixed already, so I will just bump redux-logger to the latest version.


r+


@jwittenbach Woohoo! Look at all those muscles!


Nice spacing and comments. Easy to read through. I am still tweaking my own version, it's nice to see how different people think and how simple some things can be when I try to over complicate them.


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


> The 1.2.1 language module requirement will prevent that running on 1.2.0.

I'm still not sure understand what I'm getting at. I want to be able to do:

    $ /path/to/ceylon-1.2.0/bin/ceylon compile mymodule
    $ /path/to/ceylon-1.2.0/bin/ceylon run mymodule/1.0

and

    $ /path/to/ceylon-1.2.1/bin/ceylon compile mymodule
    $ /path/to/ceylon-1.2.1/bin/ceylon run mymodule/1.0

without changing a line of code.
I specifically do *not* want to compile in 1.2.0 and run in 1.2.1 or vise versa.

It's obvious that the first example would work because all the imports are for 1.2.0. So I don't see why the same shouldn't work for 1.2.1 since @tombentley did the work on the overrides.

There are no APIs to check because each compiler will compile against its version and it will either compile and run or it will not.


:beers:


For reference: anyone who runs into the same issue and wants to have this fixed, please add your vote to the following bug report in the Adobe issue tracker:

https://bugbase.adobe.com/index.cfm?event=bug&id=4096824

Thanks a lot in advance!


Please use short syntax


Fixed, thanks.


:heart:


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


When I suggested doing this to @joewalker he said it was not acceptable.. /cc @campd why is it ok to do now?


My only pushback (last one I promise) is that, yes we would have to change
it either way, but when using the model, we'd likely have more immediate
feedback and could implement error checking into the model's constructor
rather than trying to track it down when something behaves erratically
after a change and we can't figure out why.

Something to consider for the future, I guess. Not sure what the
disadvantage of using Node vs. Mongo is here, especially for a setup script
like this. But I trust your judgement. Feel free to enlighten me more on
why you chose this. I always like to learn more.

On Mon, Dec 7, 2015 at 10:48 AM, Yash <notifications@github.com> wrote:

> Right, true. It's something that we have to watch out for but we would
> have had to come back when the model changed and add the new data or remove
> old data members either way.
>
> This way doesn't use node and it also decoupled from the server code.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/YashdalfTheGray/flintandsteel/commit/d9eb7d05db1e9d95504c025cadfafcb561fcdeaa#commitcomment-14824129>
> .
>


Ok. I will remove `= "mvn"` part


Ой вей. Не туда.


This seems to have radically sped up REST API calls (requests were 10x slower than this before):

```
$ curl -o /dev/null -s -w %{time_total}\\n http://dspacetest.cgiar.org/rest/handle/10568/68999?expand=all
0.307
$ curl -o /dev/null -s -w %{time_total}\\n http://dspacetest.cgiar.org/rest/handle/10568/68999?expand=all
0.176
$ curl -o /dev/null -s -w %{time_total}\\n http://dspacetest.cgiar.org/rest/handle/10568/68999?expand=all
0.172
$ curl -o /dev/null -s -w %{time_total}\\n http://dspacetest.cgiar.org/rest/handle/10568/68999?expand=all
0.209
$ curl -o /dev/null -s -w %{time_total}\\n http://dspacetest.cgiar.org/rest/handle/10568/68999?expand=all
0.173
```

:+1:


@bry4n @CITguy Another thing that @ssalinas24 discovered is that if the selected date appears more than once in the calendar view, both instances get the selected treatment. For example, if the user has selected "Dec 1", and "Jan 1" also appears at the end of December's dates, the "Jan 1" instance will also get a blue circle. This could happen with the dates at the beginning of the month or at the end (since the dates we show on either side of the current month display include the final dates of the preceding month, or the first dates of the following month).


ah you are right, I missed countdown variable inside a callback, my mistake, I misread it, sorry


@donnapep this is for node 4.2.2. When doing a clean build the npm install will run a make which requires a newer version of the gcc. cheers


Generally looks good except for i386-pc vs. i386-coreboot. You need to check the $onie_firmware variable to see which GRUB architecture to use.

It looks like there is another coreboot bug (unrelated to your changes) on line 222 for the $core_image variable.  Similar coreboot bugs for the $core_image variable also exists in demo/installer/x86_64/install.sh and installer/x86_64/install-arch.


@ssalinas24 Could you attach a screen shot here to illustrate this issue? I think that'd probably be easier to understand than my wordy explanation :stuck_out_tongue:


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


Верно


Code lởm quá


I think having a completely trivial test like `assert(Rational(1,2).toAlgebraic.toRational.get === Rational(1,2))` would do no harm. Same for toReal `assert(Rational(1,2).toReal.toRational === Rational(1,2))`.

Otherwise, LGTM.


why?


Найс.


I need to refresh metatype properties at least
```
 PropertiesComponent pc = camelContext.getComponent(RhiotKuraConstants.PROPERTIES_COMPONENT,
                 PropertiesComponent.class);
 pc.addFunction(new RhiotKuraMetatypePropertiesFunction(m_properties));
```
it is why I need to refresh full camel context to be sure and everything restart correctly,
Any idea to refresh or improve it ?


Because I want to reference the review object to dish object. I use nested route in this case. If we use build_params will it get exactly dish_id?


r+


@mentioning Hubyeti.


This looks good :+1:


I can check if we add any new or modified properties and update (restart CC) if necessary only ?


:heart:


I think an indent level too much^^


@josevalim Can you explain the concerns behind this change please? Is it international characters? Just trying to learn :)


:+1:. Guess I should set aside some time to work on automating the mirror uploads sooner rather than later :smile:.


@laruence They're marked ZEND_API. I don't think we can change those in 7.0...


Hi  ced0609,

as we saw you made some work on Oracle DB support on that documentation update, are interested to share more about this support ? We still have ourselves some work around Oracle DB support.

Regards


A mistake


There's an AI loop problem with this one - #174


Right! I did not realize that we deal with it later. This looks good.


Well you're compiling the same module under two different releases and the outcomes are going to be different incompatible `.car` files. I'm not sure why you want to do that.


String.downcase handles all unicode stuff which is a cost we don't need to pay here. :)


On the same note, I've had some time to tinker and I think what is generally happening is:

On x86-64:

1. Before call, stack is (must be) 0 mod 16. (aligned)
2. When entering procedure, stack is be 8 mod 16 (because `call(q)` is `push %rsp; jmp`.
3. When the prologue is done, stack is 0 mod 16 (aligned again). Usually this is done with `push %rbp` (saving the old stack base pointer).

I'm not sure yet how this interacts with the SysV 64-bit ABI claim that the stack base pointer is superfluous and can thus be omitted (-...omit-frame-pointer).

Perhaps it aligns the stack some other way, when the frame pointer is omitted in this way.


LGTM.


+1, will fix. Thanks for noticing this.


It looks more like my mistake than yours actually. I had generated a new rsa key for the image. Not actually sure why I need a key there at all actually. I should get rid of it.
But, I left that old 1024-bit DSA key around. One I had switched away from years ago. But it looks like I forgot about Github when I updated everything to a 4096-bit key.


Originally I wrote it this way because I thought I was looping over an iterator into the actual `handleForProfileNameByIdentityId` object.

But `Object.keys()` returns an honest-to-goodness array, so it's safe to mutate the `handleForProfileNameByIdentityId` object while "looping over it".

So +1, will fix.


@donnapep thanks, I will make the change.


I means you should use `@review = @dish.reviews.build(review_params)`, so that you can handle  result of `@review.save` is successfully or failure, you can render view or redirect correspond with each case. If you use `@dish.reviews.create(review_params)`, you can't handle if it has been failed.


Shouldn't be `data.colsToMerge` instead of `colsToMerge`? If yes, I will push a fix soon.


"The installation is simple, and there are detailed steps for many different configurations on the..."


"Now you can *start* to create..."


Why not use `$request->attributes->get('_route')` to be more explict


I have a pull request out to fix this issue. Not sure if it's the preferred approach, but it works.

https://github.com/vxl/vxl/pull/37


fixed: 6010cf9726e37b28b66c161daee08cf037f365cf


Indenting is incorrect


`*` position


A może lepiej bez podzapytań:
$customerstats = array(
'tickets' => $DB->GetOne('SELECT COUNT(*) FROM tickets WHERE customerid = ?', array($customerid)),
'domains' => $DB->GetOne('SELECT COUNT(*) FROM domains WHERE ownerid = ?', array($customerid)),
'accounts' => $DB->GetOne('SELECT COUNT(*) FROM accounts WHERE ownerid = ?', array($customerid)),
);


Names take up a fair amount of space. Consider making a PipeRegistry with custom numeric IDs for Neptune.


cool!!!


Coding style is `if ((a == b) && ...)`


Trouble in SIM1 Network data when switch three times or more,
because still used SIM2 APN settings.
We can learning more from Radio log.


aggressive find-replace ftl


Unfortunately, it's hard to understand that nothing was changed in `relate_unit` code.
Can we split this commit?


Хрен его знает что у него не так, но раз так раздражает. Теперь надо закрыть и открыть webm-файл для применения новой громкости.


"Finally, the extension constructor should be added to either L{_universalExtensions}, if the parser can be used for client and optionally server extensions. When the client and server extensions have completely different forms, you should add client form to the L{_universalExtensions} and the server form to L{_serverExtensions}."

I would re-phrase it as follows ("either" is missing its "or" since it is quite complicated paragraph):

"If the parser can be used for client and optionally server extensions, the extension constructor should be added to L{_universalExtensions}. Otherwise, when the client and server extensions have completely different forms, you should add client form to the L{_universalExtensions} and the server form to L{_serverExtensions}."


Fajnie by było gdybyśmy przyjęli konwencje wyciągania tego typu liczb do stałych. Tak, żeby nie trzeba było się zastanawiać co ta 7 oznacza :)


can do - it works as is on mlo in a reasonable time frame without taking too much resources, so im minded not to change it - and its run-once code


Hi Simon,
I'm not sure to understand the sense of recent commits about the application-specific passwords. I'm still using it as a regular password. It there a new way to support the use of two-factor authentication (like with a token)?


formatowanie coś nie tego ;)


@FadeMind
```
-- Looking for Q_WS_WIN
-- Looking for Q_WS_WIN - not found
-- Looking for Q_WS_QWS
-- Looking for Q_WS_QWS - not found
-- Looking for Q_WS_MAC
-- Looking for Q_WS_MAC - not found
-- Found Qt4: /usr/bin/qmake-qt4 (found suitable version "4.8.7", minimum required is "4.6.0")
-- Found Qt4: /usr/bin/qmake-qt4 (found suitable version "4.8.7", minimum required is "4.6.0")
CMake Error at /usr/share/cmake-3.4/Modules/FindKDE4.cmake:71 (message):
  ERROR: Could not find KDE4 kde4-config
Call Stack (most recent call first):
  qt4/CMakeLists.txt:19 (find_package)
-- Configuring incomplete, errors occurred!
```


Are you refering to the "(REDIS)" text ?
If so, its automatic, i've added a new property 'requireredis'=>true


sure, ill add


formatowanie coś nie tak ;)


chmdo .... ahhh LG... type existing commands.


Nee to also remove VXLBuildSetting.cmake from the install command below you'll get this error when installing:

    -- Installing: /usr/local/share/vxl/cmake/VXLConfig.cmake
    CMake Error at config/cmake/export/cmake_install.cmake:36 (file):
      file INSTALL cannot find "/tmp/vxl/build/VXLBuildSettings.cmake".
    Call Stack (most recent call first):
      cmake_install.cmake:42 (include)


by doing that i think its going to be much harder to read the entire bodies of code


Hi Brother Jones,

Thank you for your patience.  I think we can have the new game and the sort
in place by Wednesday.  I'm not sure by tomorrow but I will email as soon
as the lesson #9 requirements are finished.

I know we still have lessons 10 and 11 to do yet.  We are working hard but
it might all be late.  My goal is still to have it finished and working
according to requirements by the end of the course, even though I know the
project will be penalized for being late.

I am not making excuses and do very much plan on learning this material to
finish the course requirements.  Real life just got in the way this
semester.

I am sorry for the interruption this causes to your schedule of grading.

Betsey

On Sun, Dec 6, 2015 at 10:04 PM, Craig Jones <notifications@github.com>
wrote:

> Is this ready to be graded? You were suppose to add createnewGame in
> GameControl.java and it is not there yet. I have got to grade this soon so
> that we can move forward with the course. We are almost out of time. How
> close are you for me to grade week 09?
>
>
> ________________________________
> From: Betsey Delorey <notifications@github.com>
> Sent: Saturday, December 5, 2015 9:52 AM
> To: delorey-betsey/CIT260NoCake
> Subject: Re: [CIT260NoCake] List Arrays Added (fb09cc9)
>
>
> This commit shows the three list arrays. When the user selects a location
> that has an item on one of the three lists, he can choose to mark the item
> off the list. Then user can select a list to view the list and items found
> will be displayed in the second column titled Found. We will alphabetize
> the found items for our sort. We need to add that and the runtime error
> processing and the streaming IO to save the game yet. Will work on that
> next week.
>
> -
> Reply to this email directly or view it on GitHub<
> https://github.com/delorey-betsey/CIT260NoCake/commit/fb09cc9b948ae8c71fb414f65bc2e973a82a3a0b#commitcomment-14804531
> >.
>
> [https://avatars2.githubusercontent.com/u/14282285?v=3&s=200]<
> https://github.com/delorey-betsey/CIT260NoCake/commit/fb09cc9b948ae8c71fb414f65bc2e973a82a3a0b#commitcomment-14804531
> >
>
> List Arrays Added · delorey-betsey/CIT260NoCake@fb09cc9
> Read more...<
> https://github.com/delorey-betsey/CIT260NoCake/commit/fb09cc9b948ae8c71fb414f65bc2e973a82a3a0b#commitcomment-14804531
> >
>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/delorey-betsey/CIT260NoCake/commit/fb09cc9b948ae8c71fb414f65bc2e973a82a3a0b#commitcomment-14812731>
> .
>


we don't need it because I think query_pushes_by_pushid_range can replace it.


Yes, sure, just a type. Thank you!


Sorry i wanted to change something, but ended up not changing it, and forgot to remove the imports


The logError error string needs an enter "\n"


i can update the commit message to say - that it used some of the code from the relate_unit method, rather than merged with - which is probs more accurate.


@nikic yeah, but there are typos...... it is too bad that we didn't notice this before release :<


The goal of this PR is to make sonata-admin compatible with SF 3.0.

If you think your suggestion should be added, feel free to make a PR. :+1:


@pjvohra DWMR. Keep pressing ahead!


This seems weird: VnidStartMaster() goes out of its way to only call CreateNetNamespace() for namespaces that don't already exist, but at the same time, you change watchNamespaces() to *stop* doing that, and call CreateNetNamespace() even for namespaces that do already exist.


А зачем эта проверка?


No thanking me? :'(


arguably the default here should be empty and then on prod/india we override this with the new router. probably will cause the least amount of pain to devs locally


Actually, I need only this lib. Exe files are just byproducts.


JS file is not operating yet. Please don't try to refactor or change it in any other manner. I'm going to finish it ASAP.


@oberon2007  https://pastebin.com/raw.php?i=7qeJnVVw


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


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


This code inside the class should be probably the (static) function.


@askb Can't we just `return true` here?


@askb And `return false` here?


Personally I think it is much easier to review if we first have a commit that moves `virtualfolder/models.py:relate_unit` to `virtualfolder/receivers.py:vfolder_unit_postsave_handler` and then another one that adds new receivers.


NOTE: In this PKGBUILD I don't build qtcurve-kde4 anymore (see conflicts)


the whole condition check should be moved to the first argument of REMOVE_VIOLATING_FLAGS( )


You should here use PinGenerator.


Pull request added to address mleotta's issue.

https://github.com/vxl/vxl/pull/38


I'm guessing you didn't mean to commit this debugging code?


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


query_push_by_revision can replace it totally. we could just use push.changesets to get it.


no its the description, I added a note about the recommended feed engine and visualisation tips, there are quite a few changes all listed in this commit


Мне не очень нравится привязка к строковым значениям.

Во-первых, метод, звучащий как update_vote (обновить голос), не должен его удалять (точнее то, что он может удалять – совсем не очевидно).
Лучше для отмены (=удаления) сделать отдельный метод.

Во-вторых, гораздно логичнее передавать в update_vote само значение (т.е. +/-1). В будущем тебе может понадобится добавить другие значения голоса (например, голос пользователя с более высоким рейтингом может считаться за два).


we will return push instead of revision


Could you somehow name these "magic" strings?


@e0d I thought that if a role needed common in order to run, it would have a dependency on it still.  It seems like if a role expected to have pip and git, it would inherit from common still.  My concern is around losing the ability to run a role and get everything that roles needs to be run setup.


@ret2libc this is a comment I forgot to remove. The idea is that when there is not section header it doesn't mean there is not section info, in that case the info is in the program header. Once that we parse correctly the info from phdr the helpers functions such in this case were relying on the section header too. To resolve this inconvenient I had to add a static global variable `g_sections` to use it in the code when needed instead of trust on shdr.

This variable is initialized at `init_elf` calling the get_sections function


Maybe got somewhere else of compatibility. Removing...


In chroot everything is at state of latest unstable, yes!


Кого раздражает? Вроде все работало.


Why has this change not been rolled back since Aug 2014? It clearly breaks the intent behind https://github.com/marionettejs/backbone.marionette/pull/745/files

Is there some other way to have a separate data model for serializing data to the API and what is bound to the Handlebars templates for rendering that I'm not aware of, or is current version broken still?


Well It's not regular comparison, we might want to use some special filters. First I wanted to use `fnmatch` to support `*` only, but this seemed more-flexible.


Just confirmed that gcc does do this for non-leaf functions. Even when it doesn't `push %rbp`, it does something similar to `sub $0x18, %rsp` to align the stack frame.

At function entry the stack frame is always 8 bytes off, as indicated by Agner Fog (8 mod 16).

You probably already knew all of this, but for me it's educational. I guess the hard part is knowing all the differences between platforms.


I think you miss a point. The call (in your case foo) need to save some registers (ebx,edi,esi on 32 bits). So honestly you don't care if you're 8B or 16B aligned when you enter the function. If you dump some reals code, you will see various add/sub/and on the stack.

On 64 bits you can completely omit the full stack due to the red zone but otherwise that the idea.


@MATsxm could you revise line 418


I wouldn't change that.  OpenSSL has all kind weird magic values...


Can you write comment about:
1) what functionality from KITKAT is required,
2) what is the behaviour if you run app on low-end phone (below KITKAT)


Hi,

Signature showing as not in signature folder.


В треде же кто-то жалуется. Да и если подумать: слушаешь ты в фоновой вкладке музычку, а тут решил посмотреть тихий `webm` и начал увеличивать громкость и фоновая вкладка как заорёт!


Ммм. Ну да.


Т.е. синхронизация оставлена, просто теперь надо перезапустить webm, чтобы он её подхватил.


What if intent was not NULL?


unobsoleted == resurrected?


You here get the data, but what you do with this data afterwards?


@hjmjohnson Can you have a look to see if this branch is right for what you want to do? Considering it fixes a CMake configure issue, it's rather important.


Hi, there are new files


Seconding @feanil's comment; a role should run alone without errors on a base box. Perhaps this indicates we need to separate `common` into several smaller roles?


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


Please sync with current master, then submit a pull request from this branch to master.


@oberon2007 Add `kdebase-runtime` as depends. (in lib32 too) in globally.


Looking above, the existing diff already blocks these filters.


I fixed it already in another commit, thanks!

-Brad


Hello,

Please go here: https://github.com/tonytomov/jqGrid and there is a button Download ZIP (at right in the middle of the screen) click on it. Alternatively go here https://github.com/tonytomov/jqGrid/commit/232ba0abed8b9ab4bbab2d591256b73a40018248 download the file.


geia sas paidia


I'll take it, but I prefer one-liners myself. C is tricky, it's very easy to shot yourself in the foot. Thanks for the fix!


> Actually, I need only this lib

I don't see headers installed in this case... Also as I've mentioned this guide might be interested for you in case you want to install `*Config.cmake` module so library can be found easily and include directories added automatically:
* https://github.com/ruslo/hunter/wiki/example.find_package.config


Can you try to separate view layer from model layer? I mean things like displaying text "2 hours ago" are clearly view things and things like notifying data set changes are clearly model things.


How can it be achieved?


Just a heads up: this patch won't do anything without the corresponding tor patch that implements the FORGETHS command. I'll be trying to get the tor patch merged upstream shortly. The command may be renamed from FORGETHS to HSFORGET, in line with other commands that have been added to tor since this patch was first created.

The tor patch against 0.2.7.5 is here:
https://code.briarproject.org/akwizgran/briar/blob/master/patches/tor.patch


This change looks reasonable to me, but I'd like to talk through no longer creating the `CONFIG_DIR`.  I'm guessing that the motivation is that the config file must already exist.  If we don't find a config file anywhere, then we cannot boot.  So there's no point in creating a directory where we think the config file might go; the config file will still not be there.

Am I in the ballpark?  Thanks.


And this?


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


Damn.  I wonder if this is what was causing me problems with CloneKill.


This gives me a "Font from origin http://fonts.gstatic.com has been blocked from loading" on Chrome. Had to add 'https'


According to the SSL_CTX_set_verify man page, verify_callback should return either 0 or 1.


Thanks,
I will inlcude in the next relaase of ldap3.


Well, yes, but since I compiled `foo` in a separate module, it can't know which alignment is true when it enters. So that's what the ABI specifies, and the ABI specifies 8 mod 16, I think.


@jwieder I am a Team Lead.  Feel free to list yourself at least as an Advisory Group Member.

@davbre can I put you under Advisory Group Member for now (unless you'd like to a take a lead role)?


I'll try to let this be my last post, it's about the statements. I'm assuming `m_base_frame = true`.

```c
// Stack is (8 mod 16)-byte aligned here (entry of function)
xPUSH( ebp );       // Now it's 16-byte aligned.
xMOV( ebp, esp ); // Move esp into ebp (set new base pointer).
// Save the register context
xPUSH( edi );        // Stack is 4 mod 16
xPUSH( esi );        // Stack is 8 mod 16
xPUSH( ebx );       // Stack is 12 mod 16
xSUB( esp, 12);    // Stack is 24 mod 16 = 8 mod 16.
// Stack is unaligned.
xCALL()  // Unaligned xCALL
```

Is my math wrong? Is the stack actually aligned before the call?


Why did you add this?


:+1:  membership_and_combat_stats() is only used in duel.php (aka... ...the bathhouse...)


Sounds good!


@lburgazzoli Sure thing


Did you update the sensor specifications to be in Kilo Ohm as well (so the UI can display it correctly)?


actually in this particular case would it not be better to read password from stdin?   That need not be done in this change, however.


Is this a precedence error?  `not` binds more tightly than `and`, so this is equivalent to `if not(A) and B`, rather than `if not(A and B)`.  Ditto for the next comparison.  If that's what you intended, then please ignore this.


That looks a bit dangerous. Everything above 7.0 forever and ever?


We should do this part (`createRecord('riak-node', ...`) in the Explorer service.


Smart :)


I renamed the variables for consistency so I could c&p some parts of the code and since it was named ```klass``` in the first module I did, everything became ```klass```.


+1

Very well written and immediately working. Thank you @yees-cds - ready to merge


Nope.. What if redstone wire is placed on a redstone block? Or a block is powered and you place redstone next to it? Every side must be done


OK, I'll push a separate fix.


Is checking the promise state necessary, given the `Ember.RSVP.allSettled(` above?


ligne de commit


Якщо ти про захардкоджений мейл, то наразі для тесту: не можу нормально скористатися мейлсервісом зі своєї машини.


Суть в чём: в скрипте есть защита от повторяющихся тредов (чтобы не ломать `pByNum` и прочий ливер скрипта), однако может возникнуть такая ситуация, что загруженная страница окажется состоящей исключительно из повторяющихся тредов, а значит пустой. Теперь скрипт будет загружать следующую страницу, если обнаружит такое недоразумение.


ok, make sense! thanks!


```
Is my math wrong?
```
Ofc.  Try again ;)


Thanks for your help with this. Unfortunately I'm still having trouble... I've installed https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=gtk-sharp-git so I'm at version https://github.com/mono/gtk-sharp/commit/89b2dae6d82bfb22ee8641e5bbd6d42dcbab8c17, and I'm trying to build gstreamer-sharp at version https://github.com/gstreamer-sharp/gstreamer-sharp/commit/bf18e534a96079ce46b34a2e2afd7d8d23aef915. gapi3-codegen from gtk-sharp-git is being used by make, however:
```
make[1]: Entering directory '/home/m/develop/gstreamer-sharp/samples'
/usr/bin/mcs  -out:playback.exe -r:../out/gstreamer-sharp.dll -r:/usr/lib/pkgconfig/../../lib/mono/gtk-sharp-3.0/glib-sharp.dll ./Playback.cs
./Playback.cs(31,24): error CS0117: `Gst.Parse' does not contain a definition for `Launch'
/home/m/develop/gstreamer-sharp/samples/../out/gstreamer-sharp.dll (Location of the symbol related to previous error)
```

Am I missing some other dependency?


I want to replace these with a call to Player/PC static or something.

Opened as issue: https://github.com/BitLucid/ninjawars/issues/314


K


I don't think csgolounge supports HTTPS.


@yifan-gu So, for now, we'll fail the pod at runtime?


Arg. OK, that's it, I'm not thinking well, count me out for today and sorry for the disturbance.

I forgot to take into account that on 32-bit, the %eip register is only 4 bytes. So it would be:

```c
// Stack is (4 mod 16)-byte aligned here (entry of function)
xPUSH( ebp );       // Now it's 8 mod 16 aligned.
xMOV( ebp, esp ); // Move esp into ebp (set new base pointer).
// Save the register context
xPUSH( edi );        // Stack is 12 mod 16
xPUSH( esi );        // Stack is 0 mod 16
xPUSH( ebx );       // Stack is 4 mod 16
xSUB( esp, 12);    // Stack is 0 mod 16
// Stack is aligned.
xCALL()  // aligned xCALL
```

Everything appears to be correct :).


Hm I suppose a `perform_later(routine_name, args)` method would be good.
Passing wrappers to perform_later does not work right now because they are not serializable. Need to include the GlobalId module and implement a global self.find method.


`self.known_query_parameters` can't be renamed to `self.known_parameters` here, because the old name is used elsewhere in the code (and it's a public variable, so there's no way of knowing which other modules might refer to it).


так, поправлю. дякую.


Because my development environment and my runtime environment are not the same. And our tools don't really make it easy to switch between the two. But even if they did it's like asking why someone would want to compile my code with Java 7 and Java 8 when the output would probably be different .class files.
To me it seems that we would want code to be as compilable as possible with the largest amount of systems, without having to force users to use one specific distribution or the other.


i think that just makes it more confusing - the state is OBSOLETE - so i would say that the verb is better with that explicitly


Looks good to me.  We will need a separate task to ripple out the use of this across the whole of the CLI.


Of course you did


Right, good point! Good thing I did not mess with it! :)


what's this magic?


Тестирование проведено для работы напарника. Результаты проведенной работы в документе.
[TestPlan_-_Onliner_currency_conventer.docx](https://github.com/Suvorova-Anastasia-350504/Onliner-currency-conventer/files/54263/TestPlan_-_Onliner_currency_conventer.docx)


I think this can be improved by using a default value, like this
```
WEBROOT="/var/www${1+/${1}*}"
```


I do not believe mdfind with just filename will always return just a single entry as mdfind finds all entries containing the filename. For example:

```
[~]# mdfind -name lsregister
/Users/me/Library/Caches/Metadata/Safari/History/http:%2F%2Fkrypted.com%2Fmac-security%2Flsregister-associating-file-types-in-mac-os-x%2F.webhistory
/System/Library/Frameworks/CoreServices.framework/Versions/A/Frameworks/LaunchServices.framework/Versions/A/Support/lsregister
[~]# mdfind -name lsregister -count
2
```

Perhaps the man page could suggest using:

**mdfind -onlyin /System "kMDItemFSName = 'lsregister'"**

as it is less likely that Apple would have lsregister in more then one location of the /System directory though not impossible.


Экзотический случай, тем не менее.


Follow the php7 project on github and you'll see the micro nightmare
they're patching up because some issues simply aren't found until the first
major release gets them noticed. Point being, wait for the 2nd release.
First is usually worst.

On Monday, December 7, 2015, mikaoras <notifications@github.com> wrote:

> @younes0 <https://github.com/younes0> php7 really sent A LOT OF RCs
> and put out the stable version in a hurry
> can't imagine is there still some problems in current version
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gadelkareem/wp-w3-total-cache/commit/e71030e89f4cd666742ae2ec39451782beb42f3a#commitcomment-14819146>
> .
>


Looks good. Need to change where the files/controllers are.


@rally25rs Try https://gist.github.com/jmeas/0deede9f425f491c0fb5

As far as rolling back.  This addition was added 6 months before anyone at Backbone thought it was a bad idea.  When Mn added this behavior it was the recommended Bb way.  To remove it is breaking, so it will never be rolled back on a 2.x branch.  3.0 have a release candidate shortly and it will include this fix.


Nooooo! @gluxon and @gisellegk it didn't work! :(


ありがとうございます。別途メールさせていただきました。


@rally25rs, we don't break backwards compatibility. We decided against adding this breaking change in 2.0 because of all of the other major breaking changes that were added. This will be added in v3, which hasn't been released yet (but should be released in the near future!)


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


Would it even be possible to use:
```
import com.redhat.ceylon.typechecker "1.2.0";
import com.redhat.ceylon.compiler.js "1.2.0";
```
on 1.2.1, without somehow introducing classloader isolation for internal Ceylon libs? Or is it that you want some sort of version resolution to bump these to `1.2.1` when running or compiling with `1.2.1`?


Save probably no longer needs to/should call Load.


@Zayelion
1. I believe that the setcode should be the same as the setname in-game, since it is directly displayed in the edit page.

5b5c484
a6f3d01
Basically the principles are:
(a) setcode<=0xff (base)
Adding '#' means this code is reserved but not in use now.

(b) setcode>=0x10ff (extended)
If the setcode is not used by current OCG/TCG cards, then it is deleted from the list.

2. The user can determine whether the system should display the setname row  or not.

3. The option name are different in Percival's version and Japanese version, so currently we use the name of those items in the source code.


- [ ] Need `@DateTimeFormat` ? I think `@DateTimeFormat` is unnecessary.


Extraneous whitespace


I think this is a problem. You're going to copy the artifact, removing the version number, then the Ant deploy task is going to copy the artifact again when it deploys it to Liberty. Maven needs the version number in there when it 'installs it to the local repo'. Ant shouldn't need to strip version numbers. So I think a better approach would be to tell Ant the artifact filename and the filename to deploy it to. So this needs an update to the Ant task as well.


так надо


@Xekyo Can you do a fixup of this one as well? Because this is more than just " Added a test for the pruning of extraneous inputs after ApproximateBe… "


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.903](http://teamcity.monogame.net/viewLog.html?buildId=11997&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 560, ignored: 6 Build time: 00:26:08


This file shouldn't be part of the PR. No need to add new unused 'using's, change default colors, or add a blank line :).


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


I categorize the returned links by iframe / absolute / relative / protocol.


Well again, with a `--release` flag we may be able to make something that makes sense, but without it not. Open an issue about it if you want.


Needs to be updated to latest rev. The PR has conflicts currently.


Переставало работать в новой странице, которая была бы загружена после пустой. Там не хватало `lastThr` потому ломалась цепочка постов. Последние три строки патча как раз это исправляют.


Mmmmm... alright.


Can you change the supervisor file so that there's a single main `supervisord.conf`, and then we create a folder in the same directory called `supervisor.d`, and then use the [include section](http://supervisord.org/configuration.html#include-section-settings) to have an absolute, globbed path to all supervisor files in the directory (i.e.. `supervisor.d/*`). This makes it easier to have many programs all included in the single instance of supervisord. All the paths should be absolute, in this case.

Thanks!


SERVER_EDITION__PERMANENT_KEY -> SERVER_EDITION_PERMANENT_KEY


@tboerger There's substantial difference between `go build` and `go install`. The 2nd not only builds and installs the binary into `$GOPATH/bin` (`gogs` in this case), but also builds and installs all dependencies into `$GOPATH/pkg`, so next time you call `go install` (and `go build` too), all dependencies that have been installed to `$GOPATH/pkg` are just reused.

You may see the difference when (with vanilla empty `$GOPATH`) you do `get get -u -v ./...` on checked out Gogs project, then you do several times:

1. `go build -v` - you will see that every time you do all dependencies all rebuilt all over again, because these has not been (yet) installed
2. `go install -v` - after 1st run doing initial full build, 2nd run is doing essentially nothing, and only if you modify so piece of code, then it is recompiled


此命令写错


Add this line to 2nd Line :(


Понял.


Было проведено тестирование приложения напарника, которое увенчалось успехом.
[Test_Result_-_Shopping_list_and_maps.docx](https://github.com/Diana-Kupriyanova-350504/Shopping-list-and-Maps/files/54271/Test_Result_-_Shopping_list_and_maps.docx)


The initialisation of `TrustRemote` is based on user specified trusted subnet information in a later commit.


Add this line to 3rd line :) :+1:


It looks like you have two css values for one property. The `no-repeat` value is for the `background-repeat` property.

If you're trying to use the `background` shorthand property which allows you to have multiple values, then it should work after removing the `-image` portion.


Is this the 'malicious code' that was problematic on curseforge.com and bukkit.org? I hope everything gets sorted out since I love HomeSpawn! :)


Done.


Интересно, скрипт вообще запустится на пустой доске или упадёт с какой-нибудь ошибкой?


Can we just assign a default null hypothesis and make that an error, for clarity & simplified security?