Bracketing the loop would be:
```Python
    def send(self, lines, to_addrs):
        self.f.write(self.SEPARATOR)
        self.f.flush()
        for line in lines:
            self.f.flush()
            if is_ascii(line):
                self.f.write(line)
            else:
                self.f.buffer.write(line.encode('utf-8'))
        self.f.flush()
        self.f.write(self.SEPARATOR)
```
This is ugly, but I sometimes use it to help keep different streams synchronised when there are no other options.  For example concatenating STDOUT and STDERR within a test system for logging purposes (or for later reporting of only failed tests including all printouts, warnings, and errors).  Anyway, I remember that Python2 does not have `sys.stdout.buffer` objects - this is only in Python3 - so the solution was not good.

For the `write_str()` function, I only saw it after my last comment.   The `try:` logic seems sound.  Maybe a call to `is_ascii()` at the start would be good to default to a standard `f.write()` call?  Thought this is completely unnecessary, and I only considered it to preserve the original behaviour if only ASCII is present.  Maybe it is also worth bracketing this function with `f.flush()` calls, one at the very start and one at the end of the function, just to avoid possible synchronisation issues with `f.write()` vs. `f.buffer.write()`, if `f.write()` is ever called for some reason (`sys.stdout.write()` is called 3 times in the code, though maybe they should be `sys.stderr.write()` calls) or if the strange behaviour is with `f.buffer.write()` itself.

Anyway, the only useful suggestion I have for the `write_str()` function would be to speed the code up and simplify the logic by storing the default encoding to decrease the number of function calls:
```Python
    unicode = str

    ENCODING_DEFAULT = sys.getdefaultencoding()

    def write_str(f, msg):
        # Try outputing with the default encoding. If it fails,
        # try UTF-8.
        try:
            f.buffer.write(msg.encode(ENCODING_DEFAULT))
        except UnicodeEncodeError:
            f.buffer.write(msg.encode(ENCODING))
```


fwefwefwe


All good :)


I feel a disturbance in the force


... aici la fel, este redundant, ai deja o ditamai functie, `GetOSVersion` care fix asta face ...


@addyosmani I'm not sure to get what you mean about "show folks why they should care about it ". Are you talking about the "say what you mean" issue, the Strict Equality Comparison or something else?


@XmlRootElement and other JAXB annotations should never be added to interfaces -- becaues JAXB can not serialize interfaces in the first place.

Would you guys be open to a PR that fixes this.


Please use a generic `F: Fn(&u8, &u8) -> bool` type here so that `eat` is monomorphized.


This method will stop exploring AST on the first scope/enclousure that contains at least one matching node. Is it intended? Shouldn't it be looking up for all matching dependencies in all scopes?


So just noticed this on line 247, I made a static call `ViewModelDefaults.ReflectionCache.GetPropertyInfo(propertyLambda)`. It's not an egregious coupling of implementations (since the its just a reflection utility call) but I think it would be preferable to mark the method `GetPropertyInfo` as `static` to reflect the dependency.


Карочи, появляется кнопка "Прикрепить/открепить". После нажатия можно перетаскивать окна Ответа/настроек/Избранного/Скрытого/Видео. При этом активное окно всегда в топе z-index. Прикрепленные к панели окна ведут себя как и раньше.


Hi guys, I don't know which are all the devs, but i can see also your name in the commit message:
"Port By BlackDragon.
Used reference: scotthartbti/android_frameworks_base@06dd21f
"


Unfortunately it does not help. Still ini_set('error_log','...'); redirect full server log to selected file. HipHop VM 3.9.1 (rel)


Because:
* 8080 screamed to me: "this is a port for tests!"
* the chance of port collision with another software is way higher on 8080 than 5039
* 5039 is right after 5038 for the AMI

Would you prefer another port?


Would it be possible to describe why language bindings are needed?


You're right. Let me add some more tests...


Why was the `IXpmMarkupService` left out? He looks lonely :worried:


That is, cf. to <del>ARTH110</del> ARTH111 commit.


We will definitely do that, but for now the question is more to get everything working again in the new environment trying to change the less possible in order to avoid bugs that would be difficult to spot. Then of course we will adapt and use all the openmvg classes etc. These first commits/pushes are just for us to share the code and work. But of course the comments on what to do and which openmvg classes/function to use are more than welcome! :-)


Nice switch from static to injected dependency! I commented on this on the last commit -- shoulda looked at all the commits first!


pull request


No worries, I have just added a comment in order to put it somewhere in your mind knowing that it will be useful later on.


good catch, I can remove that from the interface and rerun all tests and push it, does that sound ok?


The thing is that to use the function `loadDescsFromFile` we need to use the `Descriptor` class.


pull & request c188776


General comment: awesome switch to Dependency Injection for the static helper. This definitely makes configuration easier, though it does _require_ the user to have a configured DI container, which is an external dependency we should think about when implementing a `static` class


Izveidoju [PR](https://github.com/hybridauth/hybridauth/pull/535). Patiesībā būtu baigi feini, ja kāds šo klasi pārrakstītu. Ne tikai tā ir neglīta, bet arī knapi strādājoša.


100 kr. er for lavt imo.


Ok, I'll work on this later.  For one test per issue, that is more suited for [unit tests](http://www.nmr-relax.com/api/3.3/test_suite.unit_tests-module.html).  However these appear to be [system tests](http://www.nmr-relax.com/api/3.3/test_suite.system_tests-module.html) which check the operation of the entire system.  At those links I use the rule of one unit test = one issue, and then overload the system tests to catch as much as possible without slowing things down (the test suite for this software takes >10 min on the fastest computers today to cover the [full code base](https://www.openhub.net/p/nmr-relax)).


>  Maybe a call to is_ascii() at the start would be good to default to a standard f.write() call?

I don't see a need for it. Well, I'm making the assumption that any decent encoding is a superset of ascii, and in this case, there's no harm in calling `.encode()` on it. Testing `is_ascii` before takes more code and may even be slower (not that we really care, but it goes through the string twice).

> speed the code up and simplify the logic by storing the default encoding to decrease the number of function calls:

That's an option, but it sounds like a micro-optimization that we shouldn't care about (is `sys.getdefaultencoding()` that slow?). Actually, it even adds an unnecessary call when if don't use `write_str`.

One real issue with `write_str` is that it does "fall back to UTF-8" on a per-call basis, hence it may end up mixing encodings if some characters can be encoded in the default encoding and some cannot.


Deleted comment regarding old code


Sure thing.


Hi @blacktrash

Win10 OK (Edge/Firefox/Chrome play on click)

Chrome Android Lolipop: NOT OK (touch to play does not play, player is in "playing" status, touching pause/play or seeking does not help playing)


Yes, thanks!

I think you might need to add the `@XmlRootElement` annotation to the implementation classes, though.


Thanks Paco!! This is most helpful!!


Sounds good!

Am 01.09.2015 um 11:20 schrieb Denis Chenu:
>
> Yep, Sam have right. Same for other Quota :first one by order is the
> default. Updatebale via plugin event.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/LimeSurvey/LimeSurvey/commit/bccdf8e70e6d788bd64dfa803be37c6b78d9c36b#commitcomment-12994092>.
>


yes but i'm not sure it will works right away with the voctree, it may require to write some specialisation of the template functions inside vocabulary_tree


Why is `\0` special here? I don’t see corresponding spec text.


Indeed but thats not all, much more work is done by me and brett rogers.
i also informed brett rogers about this. Just look at temasek, aicp, resurrectionremix and other sources
there you will find the original commits by me and brett rogers for the floating stuff. Like i said if you want i can give you access to my private repo for you to see what is the truth.


i added libqt5gui5, qt5-default as a issue to the approval of travis ci, lets see how this works out...


May as well test if it can cope with filepaths with whitespace in them...


配列の添え字にもスペースをあけましょう！


Hey Sia, no worries.

I looked at the next commit and it all seems good, though I noticed you opted not to rename the methods (which is fine by me, though I'm neutral on the names, so feel free to rename if you want!)


Så lav det om.


That's specific to h5e, `\0` is special in the tokens themselves (see the corresponding branch in emit_char()). If I don't do this, `\0` isn't properly handled where CDATA sections are expected.


See also the comment on emit_chars():
```rust
// The string must not contain '\0'!
```


@thockin The LB and proper iptables rules are created and destroyed.  I have run into an issue that I have yet to diagnose completely.  Attempting to run an OpenVPN UDP VPN fails because the source ports and IPs are not consistent.  Even if I manually delete and recreate the LB Pool to have IP+Port Session Affinity, the VPN will complete TLS negotiation with the same IP+Src Port, but then the IP+Src Port begins changing.

This means that either the GCE UDP LB implementation or something in k8s is leading to to a failure of UDP session identification, but I haven't had time to run that down.


This test has absolutely nothing to do with the IO threadpool (it's only a Begin/EndInvoke), so I guess it's more a random failure than anything else.


:hand:


Yep, I'm hitting the compilation problems right now...


looks like all commands implementing this interface already have @XmlRootElement


@warden The raw install counts by version, updated monthly, are here: http://stats.jenkins-ci.org/plugin-installation-trend/installations.json

I built `capabilities.json` on top of that to show how many installs are at a given version *or newer* to show how many instances could even use a plugin requiring that parent version.

In this case, increasing the minimum from 1.560 (102k supported installs) to 1.609.2 (50k supported installs) means that you're throwing away support for half your current possible audience.

> I think that we should encourage people to upgrade to the recent versions rather than look to support some old stable releases, because otherwise we'll not be using the latest and greatest features/fixes/performance fixes and the development of the "plaform" will be slowed down.

In general, people will update when it works for them. There may be good reasons to not stay on the latest possible LTS release, (e.g. updating Jenkins every few months simply cannot be scheduled on busy instances, or they rely on some behavior that has changed and they need to adapt first), so *arbitrarily* excluding them from updating the plugin, when it doesn't use any new core API or rely on core fixes, doesn't help anyone. Note that the update center will not offer plugin releases that won't work for their core version at all f they're on 1.596.x or older, so they don't even know they're missing out. The plugin will just remain broken for them (and if you care about the platform, this isn't a good signal to send).


Что то не работает `null`
![image](https://cloud.githubusercontent.com/assets/13660819/9604356/c7b4fd0c-50bf-11e5-9d39-bb27d9548366.png)


@emord, please just eyeball this and let me know if I'm missing a situation where we need the `save-date-picker` class, or need to remove the `hasDatepicker` class. It seems to work as-is, but I'm worried I'm about to break something somewhere else.


Thanks!


thanks! I copied and pasted from code that was already there. Thought it is some sort of guideline


Well, I am planning on using Elasticsearch soon, so why not?...  I've opened #13243.


I've never had that happen before, thanks for the PR.


This can now be done without the pointer indirection using the `NoInit` tag:
```cpp
_textures = Containers::Array<Texture2D>{Containers::NoInit, _swapTextureSet->TextureCount};
```

It won't call any constructor on the objects, so you then can use placement new instead of the `.reset()` call:
```cpp
for(UnsignedInt i = 0; i < _textures.size(); ++i)
   new(&_textures[i]) Texture2D{wrap(...)};
```

(The `NoInit` tag was added quite recently.) I can do the change myself, though, but only blindly -- I wouldn't be able to test it anywhere.


very nice! :+1:


Guys, I don't want access to private stuff, if you want I can just remove the feature, reset the branch or change the commit name to give proper credits or do anything you want, no problems ;)


cool


Fixed that back in June: https://github.com/brucepreston/cucumber-ruby-tcl/commit/c772b607cb14609f98a6ba1da8ca927b044d7e4a


also vp8 encoder shoud be removed too. i didnt sow it on caf.
I tested it and it worked good.


Reinstall your dependencies, it has been moved to quokka-flask-mongoengine package. working on development.quokkaproject.org


Hi @baodang. Done.
+ webpack-split-by-path@0.0.3


Seems like overkill for two instances that are side-by-side already.


Invited to the team. Welcome! Feel free to ask anything by e-mail: vasily (dot) v (dot) ryabov (at) gmail (dot) com.


I think control would be better. Maybe we should make the `RejectOrders` trait upgradable.

That being said, maybe it's better to accept this PR as-is for now and deal with the orders on their own or when we migrate the tick tank and artillery.


@p8pepe `getAmount` kojarzy mi się bardziej z kasą... o ile dobrze rozumiem jest to ilość punktów gdzie można oddać rowery tak? Może coś jak `count()` albo `getNumberOfDockingStations()` ? ping @skowi


`sys.getdefaultencoding()` should not be that slow.  Maybe if `write_str()` is called for every line of a commit with diff generating a multi-megabyte email, it might be noticeable.  I just spend a lot of time with my number crunching software performing such optimisations, so by default I tend to shift all function calls to the highest namespace possible.

On most systems, `sys.getdefaultencoding()` will return ASCII.  This should be safe to mix with UTF-8.  However if `sys.getdefaultencoding()` returns ISO-8859-1, then mixing this with UTF-8 could be problematic.  I didn't think of that scenario, but these encodings are not compatible with each other.  The result will probably be a lot of garbage printed for each ISO-8859-1 character, as the entire block of text would be detected and treated as UTF-8 by most software.  Would it be safe to just encode everything as UTF-8 (well, only in Python3)?


The basic idea was to keep the nested lazy seqs unlazified. But this does the job.


Good catch.


format raw is causing tinymce to return unclean content containing

 and data-mce-bogus attributes in empty fields?


Yep. Though yeah, maybe that's a reason not to inject it, so it doesn't look like we need IPython to run the examples...


sizeof(msg)


sizeof(buf)


hydra-derivatives can use libreoffice for .doc .docx thumb-nailing


Ok cool man, just remove his commit and add the commits that you can find on aicp source, there the authorships and credits are correct. also you'll get a better working floating windows in return. Thanks for understanding mate.


Thank you for the credit.

There are a lot of cool things implemented in forks of jquery-backstrech. It would be nice if we could easily see them and perhaps merge them all. Wait, is that what you are doing?


Kierowałem się notacją, z metod powyżej, wolałem się dopasować niż zmieniać - oczywiście poprawię.


sneaked in in rebase?


Agree on the latter, although only control would be the way, upgrading `RejectsOrders` wouldn't solve the issue - attack orders. :P


Yes, that's it :-). I merged most of them. There maybe are two or three that I couldn't because they are too different than the original version and then completely unmergable with this one...


Line 276 has one extra space (pep8 error)..


It decides if it looks above or below based on the icon state.


u r stuped


why?


The token is only added to standard design, should be added to admin design too (when using ezplatform with legacy-bridge)?


Oh, wow, never seen *that* before :D I learn something new every day :P So, `Containers::Array<Texture2D>` doesn't even need to be `Containers::Array<Texture2D*>`?


Usunąć <code>@Column</code>? Czy zostawić tak jak jest?


:fire: :wink:


I was thinking about running only one job on 18.0 (as we have for 17.4) and most of the jobs on R16B03. If we want to change the major Erlang version we test on I think this should be 17.5 but only after resurrecting coverage report on 17.x (as it was broken in 17.x before 17.5 and that's why disabled).


@hvelarde tentei executar a build citada por você mas a branch dela não existe mais. No entanto executei uma build minha que ainda instala o jshint/csshint pelo buildout e funcionou sem problema:

https://travis-ci.org/plonegovbr/brasil.gov.portal/builds/74907015

Será que não foi um problema temporário?


@clebertsuconic It might be worth adding HornetQ protocol to the picture.


remover esse caralho


Since no transactions, there is no roll-back mechanism. Better handle it the library
try:
    bulk.execute()
except BulkWriteError as bwe:
    pprint(bwe.details)


Thanks!


Print errors if you do not get the type of message you expect?


we should also use an image of the german book though :no_good:


added to main feeds conf a14f49d80d637ec4a6829ea5bf7beee593beaf17


Removing this assignment because i don't see any use for the udata.


nit: I'd move that statement and put it just below `int sock`.


@l3l0 dokładnie - jest to ilość stacji dokujących


Do we want event writes to be ordered? motor has a parallel but unordered way of doing this that is promised to be faster than ordered bulk insert.


@esdrubal fwiw, I've seen quite a lot of hangs of the runtime tests on Linux amd64 for at least a month, might just be another one: https://bugzilla.xamarin.com/show_bug.cgi?id=33283.

I can reproduce the delegate2.exe timeout by running it in a loop like shown in the bugzilla ticket.


@vadimcn I do my best to keep it working on nightly. Travis tests run on it every day. It should work on yesterday's nightly, and I should have it updated some time today.


Can you format xml?


lol, oops


copyright header missing


Sure. But current setting allows to check if indeed everything works. In final version it can be only `mnesia` job fo 18.0.


"These Maven plugins were initially created to manage server instances across our examples suite.  The suite first creates a server  and then starts it.  Once started the server is managed via..."


A promise is not always a Promise. For instance, a bluebird instance, is not instanceof native Promise. Maybe checking that `this.obj` is an Object and `this.obj.then` is a Function, is enough here ?


For example...


better but the line's too long


@KozhinM @blacktrash Thank you for the explanation!  I get it now.  I will work with my Envivio contact to figure out why their packager is doing this.


cool , man


strncpy (not that it should really matter here, but, well...)


prova


Небольшая заметка, как правильно оформлять коммиты
https://github.com/blog/926-shiny-new-commit-styles


I don't know if it needs mentioned, but It took about 10 minutes for Android Studio to finish indexing (or whatever else it was doing) before the "Tools > Android" menu item was available (during which time I was somewhat perplexed)


Gotcha. How would you enable this using a command-line flag or env variable to make?


Ajout d'attributs dans l'entity User, postCode et Region, dans le but de permettre ultérieurement un tri des invitations par user et par region


Added tests for _sort_by_source_priority: https://github.com/brucepreston/cucumber-ruby-tcl/commit/7ee5e019124b713baa79171d112a1b332453de63


This command creates a server with configuration specific to the the supplied arguments.  It is also possible to install some additional libraries for external modules, such as ...


Could you add an example to each command.  It will make it a lot easier to understand.


That's longer than 80 chars.


> sneaked in in rebase?

sneaky heads...


This is good - try to make pull requests for changes of the same type/theme.


Create
```javascript
function isSessionSorage() {
  return  settings.storageMethod == 'sessionstorage' && window.sessionStorage !== undefined
}
```
Use
 ```javascript
 if(isSessionSorage()) {
```


The reasoning behind my "try default and fall back" is that if the user has a, say, latin1 encoding on his system, then outputing everything in UTF-8 yields a lot of badly encoded characters (Ã© & friends). Trying the default and falling back to UTF-8 will output correct characters most of the time, and only non-latin1 characters will be badly rendered. But after all, someone with a latin1 encoding trying to display non-latin1 characters gets what he asked for ;-).


Now facebook returns only id and name fields in user info by default, if you need other fields, you have to define these fields in your request.
e.g.
in the [facebook] section of /conf/authentication.conf
change
protected_resource_url=https://graph.facebook.com/me
to be
protected_resource_url=https://graph.facebook.com/me?fields=name,email,first_name,last_name


poundmass?
as in meterlength? ;-)


1747-SetpointManagerScheduledCrash (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


man page says:
> The functions snprintf() and vsnprintf() do not write more than size bytes (including the terminating null byte ('\0')). If the output was truncated due to this limit then the return value is the number of characters (excluding the terminating null byte) which would have been written to the final string if enough space had been available. Thus, a return value of size or more means that the output was truncated. (See also below under NOTES.)


no tabs


now that reads nicely..


O, my mistake. I'll look into that.


Missing `var` keyword here: that produce un-consistent behaviors between browser and node context (by overriding native Promise support in the browser)


why not make this a companion method?


Opa. Corrigido! Valeu! :)


Can you add a line of comment above describing the function and its parameters?


Undoing the patch?


We'll want to clear this `setInterval` as well when the component unmounts.


Let me know when the Petulant2D Engine is released. :dancers:


Wsortuj metryki aby były zgodne z dokumentem WCAG.
Czyli 1.3.1 na początek a 2.4.10 w środku
+1


Yeah, that would help readability.


ok


Why do we have `name` and `username` ?


Is "project" still configurable? If so, how?


When I initially opened the project I got this android-19 target error - I can add/change this, I just wanted verify first:

![image](https://cloud.githubusercontent.com/assets/3090809/9605002/f1998c3c-5088-11e5-8fd6-a3490cd52e62.png)


I know the previous implementation also printed the pid, but, do we really care, actually?


Fix the indentation here, It's so inconsistent!


githubのアドレス追加。


Instead of `empty` + `append` you could just `html`


static imports first


To clear a setInterval, we have to save the return value of the setInterval and call clear on that.

ie.
```
this.setTimeInterval = setInterval(this.setTime, 1000);
clearInterval(this.setTimeInterval);
```


It may be a good idea to have a default here in case `data-text` is not set.


**FAILURE** - TeamCity HotRod C++ Client :: Pull requests monitor - RHEL6 [Build 250](http://ci.infinispan.org/viewLog.html?buildId=29891&buildTypeId=HotRodCpp_PullRequestsMonitorRhel6&guest=1) for 7db6aea02be77d30a43e97b9ddf7490652c5a344
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:00:18

Process exited with code 127


I'll prefer you work with Laravel Collective's Form Facade. It's shorter, cleaner and more presentable


8: M102 unrecognized bullet/signature
9: M102 unrecognized bullet/signature
19: M102 unrecognized bullet/signature
20: M102 unrecognized bullet/signature
11: M130 no bullets are allowed after signatures
15: M130 no bullets are allowed after signatures
17: M130 no bullets are allowed after signatures
22: M130 no bullets are allowed after signatures


Missing copyright header.


I'd remove the `img` completely and put a background on the class quote in the css. e.g.

    .quote {
        background: url("../images/testimonial_bg.jpg");
        height: 394px; /* just got that from the image height in finder ;-) */
        color: white;
        text-align: center;
    }


@kenmux try `site.enforce_ssl` instead? See template above.


Why?


これはもう要らんやろ。。


何か意図があるのならそのままでもいいけど、色を決めてから描いたほうが自然。


For those tests, I have a slight preference for `< 0`, makes it clearer IMHO.


No, it doesn't need to be.

If the count was fixed to, say, 3, the above would be equivalent to `Texture2D _textures[3] = {wrap(0), wrap(1), wrap(2)};` (all objects properly initialized on construction).

It's slight improvement over the following (notice the missing `NoInit` tag):

```cpp
_textures = Containers::Array<Texture2D>{_swapTextureSet->TextureCount};
for(UnsignedInt i = 0; i < _textures.size(); ++i)
   _textures[i] = wrap(...);
```

This involves unnecessary GL texture creation and deletion (because all items are default-constructed in array constructor and later completely replaced using the output from `wrap()`). The [`NoInit` tag](http://mosra.cz/blog/corrade-doc/classCorrade_1_1Containers_1_1Array.html#ad5d81995c79fb79441ff143342e513f3) helps to prevent that.


do not test for null failure it is not done anywere.


license is missing


:zap: :+1:


What's your opinion on when to do `.method_name` vs. `#method_name`


Installed android-19 & then got this:
![image](https://cloud.githubusercontent.com/assets/3090809/9605153/fcb1999c-5089-11e5-970d-7f9759788684.png)


できれば.ruby-versionに統一してもらいたいです、、
https://rvm.io/workflow/projects#project-file-ruby-version


:cry:


Solves issue https://github.com/ErasmusMC-Bioinformatics/edger_galaxy_wrapper/issues/4


I'd prefer a name like "getBytesFromChunk".  The validation aspect can be implicit.


Why not shared ? This way the lib can still be shared between `test-platform` `remote-process.so` and `libparameter.so`


Change the first `p` to a `blockquote` and tweak the padding on the .quote class

    .quote blockquote {
        padding: 100px; /*You choose the values that look best */
     }


Instead of chunkIndex, how about expectedChunkIndex?


I did that in my patches, strange you also have to do it.


80 chars


Remove blank line


fsdfasdfasdf


What warning is being suppressed here anyway?


You could create paragraphs inside the blockquote or even a `p` and a `footer`.


Point クラス作ってみた


when trying to reference the podspec while developing against 2.0rc2 I receive and error from cocoapods

[!] Invalid `player-sdk-native-ios.podspec` file: undefined method `link_with' for #<Pod::Specification name="player-sdk-native-ios">. Updating CocoaPods might fix the issue.

In this case, I'm not sure that this line should actually be in this podspec.


Why ?


Thanks!
- I didn't know (or forgot) about \N. I now mention both.
- You are right, I forgot that in Latex one should use \divides instead of | for the binary relation. That will appear in the next commit.
- My PDF file and the one online don't have the red boxes. I have had this problem in the past. Sometimes `make dist-clean` followed by `make` helps; I think it causes the system to update pygments, or something like that.


Probably to distinguish between pound (mass) and pound (force).


It is still need by
```
    <dependency>
      <groupId>org.jenkins-ci.plugins.workflow</groupId>
      <artifactId>workflow-durable-task-step</artifactId>
      <version>${workflow-jenkins-plugin.version}</version>
      <scope>test</scope>
    </dependency>
```
Or is this dependency not required?


The send lock will be hold during the receive below. Can't that cause a deadlock? Isn't it better to reduce the scope of the mutex to just the send statement?


fixes #12


mServerSuccess ? mServerResult ?


**FAILURE** - TeamCity HotRod C++ Client :: Pull requests monitor - RHEL6 [Build 254](http://ci.infinispan.org/viewLog.html?buildId=29899&buildTypeId=HotRodCpp_PullRequestsMonitorRhel6&guest=1) for 693d8fc6795fb6c03cbdfd0f55c0f1922b76a5e6
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:00:06

Process exited with code 127


For the Get it Now and and Signup buttons use hyperlinks i.e. make them `a` anchor tags and style them with

    display: inline-block;

and give them a `width` and `height`, a `border-radius` and you are well on the way.


**FAILURE** - TeamCity HotRod C++ Client :: Pull requests monitor - RHEL6 [Build 255](http://ci.infinispan.org/viewLog.html?buildId=29900&buildTypeId=HotRodCpp_PullRequestsMonitorRhel6&guest=1) for c9767c1a4db0d69336ab862a6d200e131f632e10
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:00:07

Process exited with code 127


It does make sense, yes.


Beautiful tests!


Unnecessary.


Hi there!
Just curious why did you remove Free?


good catch


Huh, interesting ,-) We've been staying away from Linux-specific polling functions actually, and use only POSIX stuff...

In any case, since this will mostly be used by one single app (or multiple ones but for very short amount of times), I think it would be ok to keep it single-threaded. Otherwise `epoll` looks much nicer that `select`...

@dnschneid: Any opinion?


通过**options 向url_map中添加url


i noticed that the template xcode projects have been moved to a new `standard` folder.
this makes it harder to make changes to them, since we can't build the projects from those directories.
wondering if this folder change is necessary?


Wrap the `img` tags in hyperlinks and you can wrap the Copyright into a `small` and use CSS to change the `display` to `block` so it will go on a separate line.


Her trengs det rydding :-)


`__init__.py` im Wurzelverzeichnis sollte nicht notwendig sein, da "wdhypermedia" ein Modul (~Datei) ist, kein Paket (~Verzeichnis).


Jeżeli jest błąd i widzisz to - poprawiaj.


All right, I will change the package imports to be less specific.


Don't forget to add the "resolved" tag to the corresponding issue!

Cheers!


Nicht besser *addLogin()*?


Revert this and add this https://github.com/temasek/android_frameworks_base/commit/28b98b559cfd3a2e35e02dcd6352a61e42b6c2fa


The name of the lock is misleading. It should be 'socketLock' as there are concurrent send/receives. The receive must be protected as well.


`__init__.py` im Wurzelverzeichnis sollte nicht notwendig sein, da "wdhypermedia" ein Modul (~Datei) ist, kein Paket (~Verzeichnis).

Wobei es durchaus überlegenswert ist, das zu ändern, um die Übersichtlichkeit zu erhöhren.


This repo doesn't contain a nixops folder! Is it supposed to live elsewhere?


Hey ezio, he is just lying, indeed he stole me work saying he used it as reference, I can explain you better when I get home.  Im at school now just don't do anything


Don't use the PR naming format for commit messages. We use a special commit message format when merging branches to master. All other commits should use normal commit messages.


I had a checkout of NixOps as a subdirectory :-)


we already do that, `has` is probably not necessary here.


Yes, I tried `site.enforce_ssl` from the very beginning.
I did the modifications showed above, and the variable `site.url`.
It seemed that the script works, but the blog gave no response after redirecting...


Guys, i'll add both your teams to the commit title and message, then please solve the thing on your side ;)


Add an assert in the code for this precondition.


You have already used Guava, then why not doing something like that:
```java
        return FluentIterable.from(Splitter.on(getSeparator(rawMapping)).split(rawMapping))
            .transform(new Function<String, String>() {

                @Override
                public String apply(String input) {
                    return input.trim();
                }
            }).toImmutableList();
```


We should then change selectSurroundingToken to call function rather than repeating the code here


**SUCCESS** - TeamCity HotRod C++ Client :: Pull requests monitor - WIN_S2012 [Build 230](http://ci.infinispan.org/viewLog.html?buildId=29889&buildTypeId=HotRodCpp_PullRequestsMonitorWinS2012&guest=1) for c9767c1a4db0d69336ab862a6d200e131f632e10
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:06:14


thank god that's implicit, the extra parameter to pass once in a while would kill... =P


shall bail out if `len < 0`.


@kenmux your _config.yml should just have `enforce_ssl: ` and `url` (the site prefix is not necessary in _config.yml)


Suggestion: If we go above the total limit (some TBs), we can't know how much we're exceeding if the excess is too small (i.e. some GBs, I guess) because of the rounding which is TB-order (e.g. I have 3.82TB/3.82TB = 100% on my instance, if I use 1 GB above the limit). Maybe we could tell (explicitly) the user how much he is exceeding.


0.13.4.  - it's in master right now if you want to build and use the SNAPSHOT version.


or even better, `= { 0 };` as you did in the server.


Ok :wink:


For all I know LFG might even match up 5 Tanks and better log that case if it happens.

> this just makes it more complicated to check logs imho.

You have a chance to check if this makes indeed logs more complicated by fixing the issues reported at https://github.com/TrinityCore/TrinityCore/issues/14331


Any specific reason you're going with a CFileItemPtr (and the additional re-definition due to forward-declaration) here? It's already pass-by-reference-to-const so you guarantee that the item isn't modified so you may as well use a  `const CFileItem&` or a `const CFileItem const*` or am I missing something?


XD


perhaps


adasdad


 #[zstack Test Bot Report] P A S S
 Integration Test Result:
Test Summary:

 TestSuite|TestCase | Pass | Fail | Timeout | Skip
 ---------|---------| ---- | ---- | ------- | ----
 basic_test|__suite_setup__|0|1|0|0
basic_test|_basic/test_create_vm_|0|0|0|1
basic_test|_basic/test_stop_vm_|0|0|0|1
basic_test|_basic/test_start_vm_|0|0|0|1
basic_test|_basic/test_reboot_vm_|0|0|0|1
basic_test|_basic/test_add_volume_|0|0|0|1
basic_test|_basic/test_add_vol_to_stopvm_|0|0|0|1
basic_test|_basic/test_add_volume_reboot_vm_|0|0|0|1
basic_test|_basic/test_add_multi_volumes_|0|0|0|1
basic_test|_basic/test_add_volume_negative_|0|0|0|1
basic_test|_basic/test_crt_temp_from_volume_|0|0|0|1
basic_test|_basic/test_crt_vm_with_volume_|0|0|0|1
basic_test|_basic/test_vm_securitygroup_|0|0|0|1
basic_test|_basic/test_create_iso_vm_|0|0|0|1
basic_test|_account/test_crt_vm_by_normal_account_|0|0|0|1
basic_test|_account/test_crt_vm_by_normal_account2_|0|0|0|1
basic_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
basic_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
basic_test|_suite_teardown_|0|0|0|1
virtual_router_test|__suite_setup__|0|1|0|0
virtual_router_test|_host/test_reconnect_host_|0|0|0|1
virtual_router_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
virtual_router_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
virtual_router_test|_novlan/test_create_vm_|0|0|0|1
virtual_router_test|_novlan/test_reboot_vm_|0|0|0|1
virtual_router_test|_novlan/test_ping_vms_|0|0|0|1
virtual_router_test|_host/test_reconnect_host_|0|0|0|1
virtual_router_test|_vlan/test_create_vm_|0|0|0|1
virtual_router_test|_vlan/test_reboot_vm_|0|0|0|1
virtual_router_test|_vlan/test_snat_create_|0|0|0|1
virtual_router_test|_vlan/test_dnat_reboot_|0|0|0|1
virtual_router_test|_vlan/test_ping_vms_|0|0|0|1
virtual_router_test|_vlan/test_create_3vms_|0|0|0|1
virtual_router_test|_vlan/test_start_vm_|0|0|0|1
virtual_router_test|_vlan/test_3l3s_vm_|0|0|0|1
virtual_router_test|_vlan/test_remove_dhcp_|0|0|0|1
virtual_router_test|_vlan/test_2l3_vm_|0|0|0|1
virtual_router_test|_vlan/test_crt_vms_with_limited_ips_|0|0|0|1
virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.0_|0|0|0|1
virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.1_|0|0|0|1
virtual_router_test|_vlan/test_start_vms_simultaneously_|0|0|0|1
virtual_router_test|_vlan/test_crt_static_ip_vm_|0|0|0|1
virtual_router_test|_vlan/test_crt_vm_with_hostname_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_stopped_vm_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_reboot_vm_|0|0|0|1
virtual_router_test|_vlan/test_chg_static_ip_vm_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_with_static_ip_|0|0|0|1
virtual_router_test|_volume/test_add_volume_|0|0|0|1
virtual_router_test|_volume/test_add_m_vols_|0|0|0|1
virtual_router_test|_volume/test_add_volume_vr_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_volume_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_2_volumes_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_24_volumes_|0|0|0|1
virtual_router_test|_volume/test_volume_ops_on_stopped_vm_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_from_dvt_|0|0|0|1
virtual_router_test|_template_image/test_crt_temp_image_|0|0|0|1
virtual_router_test|_template_image/test_del_temp_image_|0|0|0|1
virtual_router_test|_sg/test_ingress_|0|0|0|1
virtual_router_test|_sg/test_egress_|0|0|0|1
virtual_router_test|_sg/test_icmp_|0|0|0|1
virtual_router_test|_sg/test_udp_|0|0|0|1
virtual_router_test|_sg/test_stopped_vm_|0|0|0|1
virtual_router_test|_sg/test_vm_ops_|0|0|0|1
virtual_router_test|_sg/test_2vm_egress_|0|0|0|1
virtual_router_test|_sg/test_2vm_ingress_|0|0|0|1
virtual_router_test|_sg/test_4vm_complx_|0|0|0|1
virtual_router_test|_sg/test_detach_l3_|0|0|0|1
virtual_router_test|_pf/test_2l3s_pfs_|0|0|0|1
virtual_router_test|_pf/test_attach_pf_|0|0|0|1
virtual_router_test|_pf/test_delete_pf_after_destroy_vm_|0|0|0|1
virtual_router_test|_pf/test_del_vip_before_add_pf_to_vm_|0|0|0|1
virtual_router_test|_pf/test_multi_pfs_sg_tcp_|0|0|0|1
virtual_router_test|_pf/test_multi_rules_|0|0|0|1
virtual_router_test|_pf/test_pf_after_destroy_vm_|0|0|0|1
virtual_router_test|_pf/test_pf_sg_tcp_|0|0|0|1
virtual_router_test|_pf/test_pfs_tcp_udp_same_vip_ports_|0|0|0|1
virtual_router_test|_pf/test_pfs_tcp_with_same_vip_|0|0|0|1
virtual_router_test|_pf/test_tcp_|0|0|0|1
virtual_router_test|_pf/test_udp_|0|0|0|1
virtual_router_test|_pf/test_vm_ops_|0|0|0|1
virtual_router_test|_eip/test_2_eips_|0|0|0|1
virtual_router_test|_eip/test_2l3s_eip_pf_|0|0|0|1
virtual_router_test|_eip/test_2l3s_eips_|0|0|0|1
virtual_router_test|_eip/test_attach_eip_|0|0|0|1
virtual_router_test|_eip/test_create_eip_vm_|0|0|0|1
virtual_router_test|_eip/test_del_vip_before_add_eip_to_vm_|0|0|0|1
virtual_router_test|_eip/test_eip_sg_tcp_|0|0|0|1
virtual_router_test|_eip/test_pf_eip_attach_|0|0|0|1
virtual_router_test|_eip/test_vm_ops_|0|0|0|1
virtual_router_test|_eip/test_vm_ops2_|0|0|0|1
virtual_router_test|_lb/test_create_lb_alone_vr_|0|0|0|1
virtual_router_test|_lb/test_create_lb_source_|0|0|0|1
virtual_router_test|_lb/test_create_lb_|0|0|0|1
virtual_router_test|_lb/test_lb_health_check_|0|0|0|1
virtual_router_test|_snapshot/test_create_snapshot_|0|0|0|1
virtual_router_test|_snapshot/test_create_sp_after_delete_|0|0|0|1
virtual_router_test|_snapshot/test_create_image_by_snapshot_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_for_both_root_..._|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm2_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm3_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm_|0|0|0|1
virtual_router_test|_snapshot/test_delete_snapshot_on_ru..._|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_live_vm_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_from_vol_crt_f..._|0|0|0|1
virtual_router_test|_snapshot/test_change_max_sp_depth_l..._|0|0|0|1
virtual_router_test|_regression/delete_sg_with_2_attache..._|0|0|0|1
virtual_router_test|_regression/rm_nic_sg_on_stopped_vm_|0|0|0|1
virtual_router_test|_regression/test_data_volume_with_vm..._|0|0|0|1
virtual_router_test|_vr/test_ping_vms_reboot_vr_|0|0|0|1
virtual_router_test|_vr/test_reconnect_vr_|0|0|0|1
virtual_router_test|_vr/test_start_vrs_simultaneously_|0|0|0|1
virtual_router_test|_host/test_reconnect_host_|0|0|0|1
virtual_router_test|_suite_teardown_|0|0|0|1
local_ps_virtual_router_test|__suite_setup__|0|1|0|0
local_ps_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_ps_virtual_router_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
local_ps_virtual_router_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_novlan/test_create_vm_|0|0|0|1
local_ps_virtual_router_test|_novlan/test_reboot_vm_|0|0|0|1
local_ps_virtual_router_test|_novlan/test_ping_vms_|0|0|0|1
local_ps_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_create_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_reboot_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_snat_create_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_dnat_reboot_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_ping_vms_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_create_3vms_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_start_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_3l3s_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_remove_dhcp_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_2l3_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vms_with_limited_ips_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.0_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.1_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_start_vms_simultaneously_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_static_ip_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vm_with_hostname_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_reboot_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_chg_static_ip_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_with_static_ip_|0|0|0|1
local_ps_virtual_router_test|_volume/test_add_volume_|0|0|0|1
local_ps_virtual_router_test|_volume/test_add_m_vols_|0|0|0|1
local_ps_virtual_router_test|_volume/test_add_volume_vr_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_volume_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_2_volumes_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_24_volumes_|0|0|0|1
local_ps_virtual_router_test|_volume/test_volume_ops_on_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_from_dvt_|0|0|0|1
local_ps_virtual_router_test|_template_image/test_crt_temp_image_|0|0|0|1
local_ps_virtual_router_test|_template_image/test_del_temp_image_|0|0|0|1
local_ps_virtual_router_test|_sg/test_ingress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_egress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_icmp_|0|0|0|1
local_ps_virtual_router_test|_sg/test_udp_|0|0|0|1
local_ps_virtual_router_test|_sg/test_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_sg/test_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_sg/test_2vm_egress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_2vm_ingress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_4vm_complx_|0|0|0|1
local_ps_virtual_router_test|_sg/test_detach_l3_|0|0|0|1
local_ps_virtual_router_test|_pf/test_2l3s_pfs_|0|0|0|1
local_ps_virtual_router_test|_pf/test_attach_pf_|0|0|0|1
local_ps_virtual_router_test|_pf/test_delete_pf_after_destroy_vm_|0|0|0|1
local_ps_virtual_router_test|_pf/test_del_vip_before_add_pf_to_vm_|0|0|0|1
local_ps_virtual_router_test|_pf/test_multi_pfs_sg_tcp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_multi_rules_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pf_after_destroy_vm_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pf_sg_tcp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pfs_tcp_udp_same_vip_ports_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pfs_tcp_with_same_vip_|0|0|0|1
local_ps_virtual_router_test|_pf/test_tcp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_udp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_eip/test_2_eips_|0|0|0|1
local_ps_virtual_router_test|_eip/test_2l3s_eip_pf_|0|0|0|1
local_ps_virtual_router_test|_eip/test_2l3s_eips_|0|0|0|1
local_ps_virtual_router_test|_eip/test_attach_eip_|0|0|0|1
local_ps_virtual_router_test|_eip/test_create_eip_vm_|0|0|0|1
local_ps_virtual_router_test|_eip/test_del_vip_before_add_eip_to_vm_|0|0|0|1
local_ps_virtual_router_test|_eip/test_eip_sg_tcp_|0|0|0|1
local_ps_virtual_router_test|_eip/test_pf_eip_attach_|0|0|0|1
local_ps_virtual_router_test|_eip/test_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_eip/test_vm_ops2_|0|0|0|1
local_ps_virtual_router_test|_lb/test_create_lb_alone_vr_|0|0|0|1
local_ps_virtual_router_test|_lb/test_create_lb_source_|0|0|0|1
local_ps_virtual_router_test|_lb/test_create_lb_|0|0|0|1
local_ps_virtual_router_test|_lb/test_lb_health_check_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_create_snapshot_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_create_sp_after_delete_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_create_image_by_snapshot_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_for_both_root_..._|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm2_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm3_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_delete_snapshot_on_ru..._|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_live_vm_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_from_vol_crt_f..._|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_change_max_sp_depth_l..._|0|0|0|1
local_ps_virtual_router_test|_regression/delete_sg_with_2_attache..._|0|0|0|1
local_ps_virtual_router_test|_regression/rm_nic_sg_on_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_regression/test_data_volume_with_vm..._|0|0|0|1
local_ps_virtual_router_test|_vr/test_ping_vms_reboot_vr_|0|0|0|1
local_ps_virtual_router_test|_vr/test_reconnect_vr_|0|0|0|1
local_ps_virtual_router_test|_vr/test_start_vrs_simultaneously_|0|0|0|1
local_ps_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_ps_virtual_router_test|_suite_teardown_|0|0|0|1
local_nfs_virtual_router_test|__suite_setup__|0|1|0|0
local_nfs_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_nfs_virtual_router_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
local_nfs_virtual_router_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_novlan/test_create_vm_|0|0|0|1
local_nfs_virtual_router_test|_novlan/test_reboot_vm_|0|0|0|1
local_nfs_virtual_router_test|_novlan/test_ping_vms_|0|0|0|1
local_nfs_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_create_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_reboot_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_snat_create_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_dnat_reboot_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_ping_vms_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_create_3vms_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_start_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_3l3s_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_remove_dhcp_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_2l3_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vms_with_limited_ips_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.0_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.1_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_start_vms_simultaneously_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_static_ip_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vm_with_hostname_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_reboot_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_chg_static_ip_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_with_static_ip_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_add_volume_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_add_m_vols_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_add_volume_vr_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_volume_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_2_volumes_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_24_volumes_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_volume_ops_on_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_from_dvt_|0|0|0|1
local_nfs_virtual_router_test|_template_image/test_crt_temp_image_|0|0|0|1
local_nfs_virtual_router_test|_template_image/test_del_temp_image_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_ingress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_egress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_icmp_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_udp_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_2vm_egress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_2vm_ingress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_4vm_complx_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_detach_l3_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_2l3s_pfs_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_attach_pf_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_delete_pf_after_destroy_vm_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_del_vip_before_add_pf_to_vm_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_multi_pfs_sg_tcp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_multi_rules_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pf_after_destroy_vm_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pf_sg_tcp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pfs_tcp_udp_same_vip_ports_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pfs_tcp_with_same_vip_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_tcp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_udp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_2_eips_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_2l3s_eip_pf_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_2l3s_eips_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_attach_eip_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_create_eip_vm_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_del_vip_before_add_eip_to_vm_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_eip_sg_tcp_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_pf_eip_attach_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_vm_ops2_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_create_lb_alone_vr_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_create_lb_source_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_create_lb_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_lb_health_check_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_create_snapshot_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_create_sp_after_delete_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_create_image_by_snapshot_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_for_both_root_..._|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm2_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm3_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_delete_snapshot_on_ru..._|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_live_vm_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_from_vol_crt_f..._|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_change_max_sp_depth_l..._|0|0|0|1
local_nfs_virtual_router_test|_regression/delete_sg_with_2_attache..._|0|0|0|1
local_nfs_virtual_router_test|_regression/rm_nic_sg_on_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_regression/test_data_volume_with_vm..._|0|0|0|1
local_nfs_virtual_router_test|_vr/test_ping_vms_reboot_vr_|0|0|0|1
local_nfs_virtual_router_test|_vr/test_reconnect_vr_|0|0|0|1
local_nfs_virtual_router_test|_vr/test_start_vrs_simultaneously_|0|0|0|1
local_nfs_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_nfs_virtual_router_test|_suite_teardown_|0|0|0|1
Total|304|0|4|0|300


oh no you didn't è.é


These fixtures files can be safely removed.

For future projects, to avoid the test-unit library being setup, when `rails new`-ing the project, pass the `-T` or `--skip-test-unit` option.

For this project, to avoid the `test/*` cruft from being generated by `rails g ...`, see this SO answer http://stackoverflow.com/a/30379845/67834


`paramTypes` should really be removed anyway. Parameters should store symbols in them.


? that's the only thing Graphviz can deal with


plz do


For discrete.series model, the parameter D can be both positve and negative based on simulation procedure (using uniform distribution to simulate decomposition rate). Thus, it is possible that two negative symbols (--) appear in the formulae at the same time. Please revise it.


to nie koniecznie musi być prawda, lepiej byłoby jakbyś pierwszą pętle zaczynała od pierwszego niezerowego miejsca w tablicy A, bardziej formalnie od najmniejszego takiego i dla którego A[i] > 0


ouch...


:trophy:


Хм. Попробуй потом еще раз, как только #10 исправишь :)


Try a config.ru now but require 'rack' in the sinatra file.
    require 'rack'

# config.ru
    require File.expand_path '../runner', __FILE__

# runner.rb
    require File.expand_path '../sinatra', __FILE__

    /home/mock-sinatra$ rackup

The server will crash but you'll see the messages just like Ruby ran.


thanks


What is new:
* support for new api that includes:
 * basket
 * discounts
 * vouchers and coupons
* last webkit support that includes:
 * support for multiple languages
 * plenty of configuration options to PowaTag QR code
* support for both new and legacy error codes
* extended logging
* support for carriers modules
* Italian translation
* bugfixes


No technical reason really. But I was planning to move the rest of guiinfomanager to shared_ptr as well so this is just "future-proofed". Doesn't do anything for threading but at least then you know the object will be around so it's a bit safer.


Great point - was not sure if I could reuse existing data structures. I will refactor & push it. Thanks!


Now _empty()_ method may be
```java
    public static MappingsImpl empty() {
        return builder().build();
    }
```


I doesn't pass tests


Hi @lorenzo! I've just seen this and I'm guessin if this (installer-name) is still required.

Sigo en castellano si no te importa que me expresaré mejor. El caso es que yo tenía esta línea en mis plugins de CakePHP3 (el de Less y el de Bootstrap) y en un proyecto que tengo con Cake3 me estaba fallando. Siempre que hacía `composer install` me creaba los plugins en los directorios indicados por `installer-name` pero entonces CakePHP no era capaz de encontrarlos.

Eliminando el extra de mis plugins parece haber solucionado el problema, por eso me pregunto si esto es necesario todavía.


Thanks for the fixes @wwmayer ... I still have a hard time figuring out what is needed and what is not when subclassing...


I'm not sure about the details of what you are trying to set up here.
By definition, valid sort direction means compatible with the ORDER BY clause -- this simplicity is the main strength and the main weakness in the current model of plan node output ordering.
The closest you can get is a not-quite-valid but still useful index scan output ordering that matches some or all GROUP BY columns (for the serial or partial agg win) but can not eliminate an order by. Then, (possibly? always if there's a limit, maybe always always -- see side note below?) the agg output can be ordered with an inline limit and mergereceived OR (possibly? maybe whenever there's no limit? maybe never?) received out-of-order and ordered only on the coordinator.

So, the interesting test cases for me (which you may or may not have covered elsewhere) are
(prefix?) key(s) of the index match (prefix?) key(s) of the group by AND match (prefix) key(s) of the order by -- YET no two sets of keys (index, group, order by) are identical.
A variant of the above where the matching keys are an arbitrary non-prefix subset of the group by keys -- ideally, on principle, our code always considers permutations when processing group by keys.
I'm not sure that permutating the ORDER BY (prefix) keys with respect to the INDEX (prefix) keys is likely to cover/uncover anything new or interesting. I'd expect that it simply produces just another case of a clear mismatch.

Side note: sorting before or after sending in the no-limit case is a tricky trade-off that depends on how "amenable" the process architecture is. Parallelism (sorting earlier on many sites) is theoretically a winning strategy EXCEPT when there is an alternative centralized resource (an abundant thread/memory pool) that could be appointed in such a way that the sites are immediately freed up for concurrent single-threaded work -- I don't think our current process architecture usually allows for this -- at best I think that it's a situational thing that we can't generally (quite literally) "plan" on.


done in a later commit


Are those needed? Maybe for some js hooks? Otherwise, I'd gladly get rid of them altogether ;)


I can confirm this issue, created an issue on https://forge.typo3.org/issues/69455 and set a patch to review.


Yes. You can read more about it here:

http://developer.android.com/guide/topics/resources/more-resources.html#Dimension


ok  done


Just cosmetic, it looks like the indenting got doubled-up here.


This is EXACTLY why you need to make tags if you really want your plugin to be usable.
Running a composer update of the "dev-master" branch crashes with updated configuration format.

I have no choice but to fork it to ensure stability.
Please mark it as stable and provide tags quickly ;)


(reviewing my own code, I don't like it anymore ,-))

I like your solution above to have a slightly oversized buffer to handle this. Using `sizeof(outbuf)-1` in `popen2` would make this nicer.


I was originally considering that, but since that's a runtime error and something we can technically handle with this little snippet it seemed like a better solution just to handle it.  After all, they can insert a null but when it comes back it's None as it should be.


You're missing the opening part here


Here too


不要用aaaaa


hardcoded 64 isn't great, but I like the idea.


唐家轩-项目开发想法.md

这几个问题都很有价值。感谢你的建议


Looks good


Wir machen Urlaub.

Unser Büro ist deshalb vom 31.08. bis einschliesslich 06.08.2015 nicht besetzt.

Sollte es in diesem Zeitraum zu Problemen mit der Erreichbarkeit Ihrer Webseite oder des Emaildienstes kommen, wenden Sie sich bitte an die Firma Mesos.
Sie erreichen diese telefonisch während der üblichen Bürozeiten unter 0 221 - 48 55 798 - 0

Für alle anderen Angelegenheiten stehen wir ab dem 07.09.2015 gerne wieder für Sie zur Verfügung.


@zauker: Looking like we'll likely do a release this Thursday evening.


Missing the opening tag


项目初步开发思路by张泰然.md

行，稍后把概要设计和详细设计也思考一下。


Well proof is there, me and brett rogers already went in discussion with siracuervo in the past. but he is an little child who has no respect for other people and cant communicate.
Our main commit for floating window is days ealier then his (proof is there). So how can we used his commit as reference, its the other way around!! He multiple times asked me or brett rogers for help on his floating window stuff that he, i know for sure pulled from us @aicp. I was crazy to help him out with it. this is the thanks we get for that. its up to you who to believe, a 13 years old kid or real devs that are mature and have years of experience working on android.


Missing the opening tag here and on the line below


This is more idiomatic Puppet - `defaultto(:present)`

https://github.com/puppetlabs/puppetlabs-acl/blob/master/lib/puppet/type/acl.rb#L152

Would you mind making that adjustment here?


Aaaaaah ! `Kernel#fail` is an alias for `Kernel#raise`


:shipit:


I don't think `social` is appropriate as the name of the view it should redirect to. It should redirect to either `home` or `dashboard` or something very appropriate.


Can you configure your editor to not automatically add the ending newline? It's not that I mind having a newline, it's just that it doesn't feel important, and it adds noise to the PRs. Thanks!


You should update your search to also include `b,` (maybe update the regex to be `\bb\b`?


Change the name of the route to something much more appropriate like i mentioned in earlier comments


嗯，期待你的表现


Im sorry to say but we are not aicp guys. i am BlackDragon from Fusion rom and Brett Roger from Cyanide rom. we work alot together. We both have private repo's so we pushed our work to aicp source
to share this with others.

Greets
BlackDragon


I see you found the answer here. :)


To stop people like Jamie from scraping it. I originally had this implemented but removed it in a bugfix commit for some reason.


:+1: I'll merge this as @mhl is away.


I think nested if's are better in this case.


You are absolutely right! My bad. Back to square one.

It seems like for some unknown reason CCIFG flag is not set and the code gets stuck in there. And the code I submitted does not solve the problem. Another curious thing is that if I do anything in the while loop above such as reading TBCCTL6 register for debugging. The problem disappears.

And, I also think that setting SCS flag may be needed.


This probably shouldn't be here. We can't set up dsc_credentials yet.


s/money filed/money field


You actually can put this in the same selector as the color one, so it looks like this:

```
.links-container .navbar-link:active {
  color: rgb(233, 50, 117);
  background-color: white;
}
```

Also, I think the format of your background-color is not right. What were you going for?


My first git hub repository on Github


I really like this test infrastructure.


Work with @andela-oogunjimi and integrate the social media auth into his design. You can do that by creating a branch off his `auth-page-design` and integrating your work.


My2cents: both you guys have added code to this commit (also each one mentioned the other in both sources), so why just not living with both names in the commit title? :)


This also shouldn't be here until we have a way of accepting credentials.

I think you may need to do a scrub in all type generation to make sure that the word "credential"  is not matched before you decide to add the param. This is crude but `/^.*credential.*$/i`


We should not include this type. It's not supported.


So I want to accept these changes, but don't seem able to do this.


This type is also not supported.


We've recently added `helper.getIndex()` -> I think we want to use it here as well.


Similarly in the spec generation we'll need to ignore bringing in credential types.


@magnusvk um...yes, I don't know how I completely dropped the ball on that for this method.  It was actually broken so I've fixed it and added specs here: https://github.com/jordan-brough/solidus/commit/70d0aadcb071519c8bb9d3ebf60a618f02a28108
Thanks.


that's better :)


@liewegas. In face for op of linger it's no need call oncommit_sync->complete().  It only need decreate the num_un*.


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


@jhawthorn @magnusvk any thoughts on the potential change above? Shall I finish fleshing that out?


you're not just building the question here, you're also executing it. I'd change the name accordingly


The setup is such that, there is an rtimer that fires periodically for every scheduled slot.
Each slot is lets say 15 ms. and next rtimer value will depend on the next scheduled slot.
Lets say if there are 25 slots in a slotframe and only the first and the last slots are scheduled.
the rtimer will be fired in the first slot and the second rtimer value will be setup such that it will be 23 slots later. It such a case, the msp430_sync_dco routine in msp430.c gets stuck in the first ( while(!(TBCCTL6 & CCIFG)) )  loop since the CCFIG flag is not set.


do we really need to specify the name of the config file? ConfigFactory scans for this name automatically: https://github.com/typesafehub/config#standard-behavior


Hi @liuguofang - very good point!  i've tried a fix, I think it works, but can you test it and confirm? I will make the pull-request now. Let me know if it works for the cases you describe


This line is breaking the entire skin, as seen in Brickimedia/brickimedia#413


:ok: :1234:


I'd rename to "appendResult", since it's a writer you should follow their nomenclature


You mean in api/checkouts_controller and frontend/checkouts_controller?  I feel like it's mostly chance that they are the same at the moment (and likely that they will diverge soon) so it didn't seem like something work abstracting out between them.


no brackets {} needed


2个问题或建议啊：

1、下面这种表情如果要显示是不是ok的啊？直接按照你的IMG标签写地址就行了吧？如果想放大图片是不是把width指定大一点就行了？
http://res.mail.qq.com/zh_CN/images/mo/DEFAULT2/1.gif

2、我看了图片防火墙机制，很强大。但是如果用户用微信上传了很多自己拍的照片，并且在网站内进行保存，该如何在弹幕中显示呢？该网站在一个固定的域名下面。


我看到1.0.2版客户端已经发布，是不是可以下载使用了？


what does .count() do? if it indeed counts, you could do allAnswers.size. If it doesn't, the method needs a better name


same here


rename me :)


You can change certain things about it.  For example you can change the
icons.  But Android makes sure that the menu looks and works pretty much
the same for all apps.  This makes Android more user friendly because the
user does not need to figure out a new menu structure for each new app.

You can read more about it here.  The menu is part of the Action Bar (bar
at the top of most Android apps).  The menu items are called Action
Items/Buttons.

http://developer.android.com/design/patterns/actionbar.html
http://developer.android.com/guide/topics/ui/actionbar.html#ActionItems

On Tue, Sep 1, 2015 at 2:07 AM, Newton Toussaint Nigere <
notifications@github.com> wrote:

> Can one change the layout of the menu.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/e-Insight/AndroidApp/commit/d2213a7afaf0024ca34d9c395704630fcf01ffea#commitcomment-12988074>
> .
>

--
David MacCallum
https://www.linkedin.com/in/davidjohnmaccallum
079-965-2942


love this name


Oh, ok. Lets stick to just using one of the two aliases then. I've only ever seen `raise` outside of unit tests, and it gets special treatment from more syntax highlighters, e.g.:

```ruby
raise Error, 'desc'
fail Error, 'desc'
```


good.. always put main method/entry point as first method of a class


There was already a `break` right here. The indentation was intentional.

What small bug were you correcting?


Awesome! Nice to see this committed! I was about to submit a PR... just hit the same issue in 1.9.2.


You may find an even better name for this method


extract variable for groupName.slice(0, 2).mkString("/") with a good name


is this where instances metadata is pulled from openstack to register agents ?
where can this be set?


101789044_gridviewFilters (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2288 of 2343 tests passed)

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


I'd call it SnippetHTMLTemplate


@skeptical Thanks for fixing this!
Concerning GIT, how do *I* end up performing this commit? :hushed:


Should that instead be named `exclude_admin_only_addons`? I understand that you're filtering them out, but when you do `MyModel.objects.filter(foo=bar)`, you're keeping the ones that have `foo=bar`, so it makes this name a bit confusing.


Defaults to True? Did you mean "defaults to None"? It would then make sense to test explicitely against False. But then, why doesn't it default to False instead?


Alterado o tipo do campo de string para lowercase.


huh. I'm not sure why your shell's stderr isn't utf8 but I just fixed it anyway


LGTM!


Thanks for your help Matt.


Is this where you actually want the "}" to be?

What are you actually trying to close off.

Hint: Chrome's error is "Unexpected token else"
What this means is that when the javascript is running, it comes across one of your "else" statements when it's not expecting to.


this file is currently not working at all, is it? (since there's no binding and all)


why is this change related?


avatar
![13434282](https://cloud.githubusercontent.com/assets/4997583/9606465/d7d8ecda-50c3-11e5-9bed-dfa80fc441e5.png)


I still find the stateless `TCPv2EncodingImpl` class to be a bit confusing. There's no state in the decoding class (great!) but the caller has to manage the state of the decode on its behalf (not great).

It reminds me of patterns from C where it's used because the state cannot be encapsulated. That doesn't mean it's a good idea. I'd definitely give each connection its own instance of a stateful encoder/decoder following the pattern of the previous one, because that encapsulates the state neatly both conceptually and in the code.

I'm not insisting we change it, as I think this will work, but I'd like to understand the motivation for choosing this pattern.


you meant container_dashboard, right?


If I were to do this I'd be creating properties in the leader class, referring to the member  i.e.
@property
def name(self):
    return self.member.name


the `+` means a union? I don't think that's immediately clear (for me `[a,b]+c` would mean `[a+c,b+c)`).
Maybe:
// Casting to an int64 leaves the range [0, 1<<63 - 1] intact and moves -1<<63 to math.MinInt64.


this is irrelevant, those are topology specific structs


JavaDoc on isAllowedToImpersonate method missing?


relevant only for topology


Good call, I'll change it.


This function just gets a filtered version of the environment for `flux-wreckrun` to put into kvs. Actually removal of `FLUX_URI` probably isn't required here, but it doesn't hurt either since `FLUX_URI` will be added back by wrexecd


i dont get it..


Perfect! Sorry I didn't get time to do this


1. 是的，但是WIDTH仅限宽，高是和文字等高的
2. 拍照发弹幕也太过分了吧？！这里这个图片弹幕设计是为了给表情用的，图片配弹幕是什么鬼啊？！如果你只是想要图片飘过的话……反正也没几行代码，你可以自行用正则凑活下就行……
3. 是的。
4. 建议把图片存在本地，虽然我这里对每一个加载过的图片都有做缓存机制。


By Alloc_Node, I think you mean Make_Node. Likewise in the comments of Free_Node below.


> this is irrelevant, those are topology specific structs

I left these as placeholders.


why change lambda to the -> thingie? they're basically the same thing, but doesn't work on ruby < 2.0


I think you should just pass return directly rather than wrapping it.


Cool, I see `Dir.mktmpdir` works but when was that added?  I don't see it in ruby-doc: http://ruby-doc.org/core-2.2.2/Dir.html


Do we ever want to allow_secondary when checking has_symbol? The secondary can either be correct, or stale - would there be any value in querying the stale state?


> this file is currently not working at all, is it? (since there's no binding and all)

nope, more placeholder. I didnt know what to replace it with.


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


Addition of `setenvf.h` doesn't seem to be needed here?


To fix the error in travis, you should do mode= theano.compile.get_mode("FAST_COMPILE")


Hello, do you have a minimalist example for using ActiveOverlay? I can't seem to work it out.

(PS: sorry to intrude here but I couldn't find a place to write to you.)


Looks good. Suspect it would be safer to always run has_symbol against the primary


hiihih


Yeah I can see how that is confusing. I'll reword.


Can't you get the queue counts from the context also? It seems they're already provided, right? From https://github.com/mstriemer/olympia/commit/456d1debd03cb4fc23399c475eaff1f948cfa09f#diff-0392dae3003d324d2ec9ae994dc53f3eR54

If so, you don't need the `admin_reviewer` here, and thus not in the context either.


Because stabby lambda made us do it http://i.imgur.com/pHH7mLC.png


Is this list likely to be very big?  OCaml won't fuse these maps and folds, so you will make three trips through the list.   If it might be big, it may be worth fusing them by hand.


fixes null pointer `entry` which may come from https://github.com/NativeScript/android-runtime/commit/290a095ace7e1e97cf924f5baf63af323494a9d4


Not a big fan of the "http-codes/" folder. I'd rather have 404.twig and 500.twig at the root of the views/ folder.


(but seriously, to be consistent with https://github.com/theforeman/foreman/pull/2605)


@benjaoming 0.7.2 has been released

On Tue, Sep 1, 2015 at 9:25 AM, benjaoming <notifications@github.com> wrote:

> bumping version for another release @reinbach ;)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/django-money/django-money/commit/55b7d4570268f1a6ab7e4db6b3910d5a5ffcad47
>


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

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


Test code for 3.0 : http://pastebin.com/xKGMFcS5


I guess `return true` is not a permanent solution? If so, we should perhaps consider adding a `// FIXME` or `// TODO` and say clearly what is missing.


this is in response to the issue i created:
https://github.com/thinkaurelius/titan/issues/1135


yes because it's not used at all in the file because mach_vm_protect is commented and it's used vm_protect instead. I will review also this file with the time because there is TODO inside and I would like to implement the io plugin for app://. I'm working now in all of your comments.

The test seems broken due to conflicts with master maybe, i'll review anyway ;)


懂了，谢谢


As you want, I can move them asap.


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


I had a look at this and 'benchmark-test.xml' configuration doesn't need to be specified here. Let's move it directly to ConfigurationWithScenarioInFileTest. Apart from that having abstract method getBenchmark() with implementation in subclasses will be a bit more precise after the change.


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


OK let’s do it.

--
Jérôme Velociter

From: Johann Pardanaud <notifications@github.com>
Reply: 46cl/wp-boilerplate <reply@reply.github.com>>
Date: September 1, 2015 at 4:16:39 PM
To: 46cl/wp-boilerplate <wp-boilerplate@noreply.github.com>>
Cc: Jerome Velociter <jerome@velociter.fr>>
Subject:  Re: [wp-boilerplate] Fix #12: Add 404 and 500 templates (32bf14e)

As you want, I can move them asap.

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


it should make it out in 1.9.3 and 2.0


:clap: :+1:


This is minor, but you can use standard @author annotation.


subscribe to done event


Sat 6.1 uses it, so we must support 1.9.3 syntax :cry:


:+1:  Looks good


> relevant only for topology

more placeholders


Hi, @albins!

First, thanks for double checking.

I found this commit by accident. If you find errors in the IOOPM assignments, the best way to fix them for everyone is to send a pull-request (PR) to the main fork of the repository. If you could send a PR for this, it can be merged for all students and everyone wins!

If you don't know how to do that, find me in the lab on Thursday and I'll happily show you :)


I know you just moved it from below, but I don't see the point in using this `dict` notation. Makes it unnecessary more difficult to read


anyway
```ruby
lambda {|foo| function}
```
is exactly the same as
```ruby
->(foo) {function}
```


Just GET and DELETE, no PUT


I'm treating `None` as `True` in case this gets called with `queue_counts({}.get('admin_reviewer'))`. I wanted it to default to `True` because that's how it worked before and I wasn't sure if I had found all the call sites.

Looking through `git grep` it looks like I may have them all so I can flip that around.


this causes "EMFILE, too many open files 'app/scripts'" warnings on watch, because it's including all bower_component templates as well.  Maybe better to use:

'<%= appDir %>/scripts/verifiedpixel*/**/*.html',


Forget what I said, I just found out it has been merged. Fantastic :)


These 2 methods seem redundant to me. Can't you turn them into one called "parseAnswer" that returns Some(true) for a yes, Some(false) for a now and None if there was no answer ? This would make a lot more sense I think.


```
vagrant@katello-centos7-devel ~ $ ruby --version
ruby 1.9.3p448 (2013-06-27 revision 41675) [x86_64-linux]
vagrant@katello-centos7-devel ~ $ irb
1.9.3-p448 :001 > ->(foo) { puts foo }
 => #<Proc:0x00000002c27a48@(irb):1 (lambda)>
```


this paragraph can be improved through the change above


  is good the game  Guerra24

2015-08-30 12:54 GMT-06:00 Guerra24 <notifications@github.com>:

> I forgot to rename from Mod to ChunkHanlder
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Guerra24/Voxel/commit/8585b8f2b783d0deb1f1a239ae8204ed96f889da#commitcomment-12966622>
> .
>


Made updates to existing code and merged domain from SincFun's domains and Conformal maps.


Tnx, will fix it ;)


Null check


this may cause a NP. why was the check removed?


Probably incredibly minor and unlikely to come up in practice, but shouldn't webEnvironment be set to false when spring web is not available? The only case I can see where that would impact things is if someone has manually called setWebEnvironment, but at least then the behavior of that variable would be consistent regardless of whether spring-web is on the classpath.

In other words,

```java
if (!isSpringWebAvailable() || !WebApplicationContext.class.isAssignableFrom(applicationContextClass)) {
      this.webEnvironment = false;
}
````

instead of

````java
if (isSpringWebAvailable() && !WebApplicationContext.class.isAssignableFrom(applicationContextClass)) {
      this.webEnvironment = false;
}
````


Move this to the top with the other simple assignments


I agree, I was just shooting for a smaller diff. There's definitely some weirdness in this code. I'll flip it to `{}` since I was thinking the same thing.


*composable* って「組み立て可能」と「合成可能」でいつも迷うんですが、どちらが良いんですかねぇ＞識者の皆様


Why not using logger?


this is wrong. it should be "_Cgo_"


"Because the unit has the rest of the bits zero, it can pick the zeroness
 any one of those bits to signify a free state."

I am confused. AFAIK, the free state of a node is determined by its existence in the "free" list of the pool (pointed to by the "first").


sweet, I was sure this is a 2.0 feature, my mistake. please feel free to ignore this entire thread :+1:


I don't really have anything to do with it, you probably want to comment on that foreman PR.


@vanthonguyen  thanks !


Look closely and in combination with the loadProperties methods


Roos: This is not exactly the latest status_update, but the latest_activity. E.g., could be a blog post from the activity stream (as in team Binary's activity stream). Not sure if this is a bad thing or super good?
Anyways, it shows content so you have ipsums to work on the layout :-)


個人的には effect は *side-* がついてなくても「副作用」と訳す派なのですが、こちらも識者のご意見求む


`queue_counts` gets called 6 or 8 times on each page editor load (I don't remember which but leaning towards 8). It's pretty awesome. I thought about doing that but didn't really want to mess with it. I'll try it out and see what happens.


Move these above after `getName()`


I wonder how can one be sure that `servletContext` is set before `getRealmConfiguration()` is called?


ok, 1de8f254b920417484f249343eec7c9e28c9f7ee


This is jb's sample text


Typo `SELECt`


Renaming "last" to "lastShape" and renaming "shapes" to "lazyShapeList" would be clearer to me.


ah, the "isNotBlank" catch


It is _cgo_ on my Linux machine (Go 1.4).


Are you sure this is right?  It seems with this logic that if the overall shape string had, say, a Polygon, and then a MultiPoint, then the result would be one ShapeCollection with the Polygon and every point in the MultiPoint.  So the MultiPoint as a container is lost.  Right?  Are you sure we should do this?  If the spec says so but I suspect we don't want to do this.


You could add a bunch of groups here (see benchmark-cs.xml). It doesn't really interfere with scenario import, but it would improve test coverage in general & should be relatively easy to add new tests, wdyt?


I think we should try and have at least a javadoc sentence at the class level for all non-test source files.


good catch.


Thanks guys for this discussion, brings a lot of good stuff.
@oleg-nenashev: afair the matrix dependency was needed by some classes when core was bumped. iirc the matrix was part of core previously and now was moved to a separate plugin, hence the added dependency.
In my understanding, major version change *of plugin* means plugin incompatibility, in this case SOAP to REST switch. In semantic versioning I would probably change the number from 1.2 to 1.3, here however the versioning is not using semver, hence bump to 2.0.

@daniel-beck: thanks, now i undestand the numbers! Obviously I undestand that arbitrarily excluding users might be bad, but on the other hand you mentioned "if it doesn't rely on core fixes". Well, I don't think we are able to test that, that's the idea of upgrades and continuous improvement for me and for the same reason I am trying to brag LTS version instead of latest-gratest master.

I think that we have two scenarios were either:
* we keep backward compatibility with as much as possible but this will not help maintain clean codebase, usage of consistent APIs and cleanup of deprecated ones, etc...
* we try to bump occassionally to a newer version of core to keep it "more up to date" (not necesarrilly updating to the newest LTS on the day, but e.g. making sure that the requirement is not older than - I don't know - a year?)
What do you think about this attitude?

For now, as this is critical: the best consensus is to in fact release with the previous requirement (1.560) as @jglick suggests and then to bump to a newer LTS release afterwards. That's what I'll do.


I agree, Fixed.


One more though: I am wondering if we are able to somehow track "trends of upgrades". (evil mode on) - I mean, if we would release this with the newest LTS requirement, I would like to see if that would speed up user upgrades (evil mode off) :)


I guess it changed in 1.5: https://github.com/golang/go/commit/2c2770c3d4dea11ce96f63d8fcb40ca5dd230e33#diff-6255934d72e047dd802d046c5748b331R83


Why changing `@Subscribe` to `@Listener`? Only because of Guava having the same annotation?? I think `@Subscribe` describes better what it's for.


What if end == text here (if it can)? end[-1] will certainly be UB, then. Could you comment more on this?


This can be inlined:
```java
public String explain(Statement statement)
{
    DataDefinitionTask<Statement> task = getTask(statement);
    checkArgument(task != null, "no task for statement: %s", statement.getClass().getSimpleName());
    return task.explain(statement);
}


The constructor `PrintWriter(OutputStream)` uses platform's default encoding, which is never a good idea to rely on. We probably should do something like `new PrintWriter(new BufferedWriter(new OutputStreamWriter(out, "utf-8")))` here.


Is the cast necessary here?


@wesleycho Great, thank you for your fast reply.


@chandrasekharv   I do not see any  necessity  of calling getCellStyle method as static, since we are not using method  outside  of this classes / anywhere else.


This is a constant, so it makes more sense to make the field public.


Do you reckon we should just standardise this for the sakes of the tests? Or should we have different messages for each impl. I had taken this code directly over from the old impl and not realised new fields had been added btw, hence no setting of the description and name in the current HEAD.

```
        SystemStatus status = new SystemStatus();
        status.setId("apiman-gateway-api-vertx3"); //$NON-NLS-1$
        status.setName("Vert.x 3 API Gateway REST API"); //$NON-NLS-1$
        status.setDescription("<some useful description>"); //$NON-NLS-1$
        status.setUp(true);
        status.setVersion(engine.getVersion());
        return status;
```


Fixed.


Also thanks again, my apologies for the extra work you have with this. it should not have been this way if
everybody is just honest, and give proper authorship and credits. also preserve the written comments by the devs. The comment he wrote is also no good, he did not made the port to lp.


Could you please fix indentation?


license is missing


Shouldn't these two `5000` s rather be coming from Accounts configuration?


The generics in `createExecution()` aren't necessary either ... maybe they were needed in Java 7? Or just bad code.


```sh
git commit --amend --author "Philippe <xxx@gmail.com>"
```

It could be dangerous :).


No problems guys, no apologies needed, instead it's correct to give proper credits ;)
Pushing the fixed commit in a while, tnx.


Correct, removed.


This is `DELETE /organizations/NAME/policy_groups/POLICY_GROUP_NAME`


property based testingってxxxCheck系のお話だと思うのですが、いい訳が見つかってなかった気がします。


yeah that would be great.
its helps if we can build those projects.


@grncdr I think this needs to be `[cma_token]`


You should use ATTR_URL instead.


@Saurutobi Hey how were you planning on doing the reports. Were you planning on using a certain library, or what. Because write now I'm using a pdf writer and its about to get to the point where all this code is getting unwieldy and basically bloated.


I don't see why this is better.  Without the try the exception bubbles up to the caller which has a chance to handle it (or die).  Catching it here just suppresses the exception.


訳案「我々が必要とする柔軟な抽象化の源は関数型プログラミングだ。なぜなら、データ処理において使われる数学は関数型プログラミングの基礎でもあるからだ。」


Cheers, I've learned something today :bulb:


Wow cool, thanks for the help :)

Yes this will definitely be useful!


Hm... we cannot guarantee that `inputStream` is closed in all cases. We either need two nested `try` blocks or we need to move the `inputStream` assignment to `try()` at line 99.


Good point.


Mainly I wanted to have both the encoder and decoder in the same object to have them together and also to be able to share the byte<=>method map. The encoder is essentially stateless because it can always emit the whole result in one chunk. So I tried to come up with a stateless decoder.

In the previous design the decoder had it's own internal state machine, probably closer to how you envision it. The parsing was also more complicated so it was definitely worth it.

I'm happy to discuss alternative designs as an exercise


we are sourcing external to the write process both a time and a sequence number which are what we use to determine order.  I don't think we care about the in-to-mongo write order.


I think this needs `@SuppressWarnings("unchecked")`


Will do. Thanks bro


Nice code!


Thanks! I always wondered why it my modules didn't load, haha.

Anyway, I really appreciate the help :)


I realize we can't always make this equality assertion, but sometimes we can. What do you think about making this additional assert be an extra boolean arg, possibly with a convenience method named assertRoundTripAndEquals  ?


> Is there a reason not to use the same for the PPSD and tfr?

To pick up on this argument.. Since right now we're changing a lot of things in PPSD, I would even advocate to change the PPSD plot default to viridis as well. The PPSD colormap is *really* bad and I think it's time to break with the old PQLX style. We can leave a clear note in the release message and change log and docstring how to switch back to PQLX looks. See #1108, I'll make the change there..


Sorry, should have mentioned this here but I cut 0.1.6 immediately after integrating this change so it's live now!


ser retirado


If valid and len > 1, then end is > text


You pulled from the master branch. You want to pull from the [last tag](https://github.com/lodash/lodash-fp/tree/0.10.2).


LGTM post this change.


I think we should test this.  If for some reason you couldn't test this, I think it's better to use an \@Ignore to make it more obvious in test runs that a test was skipped.


"which are not ..."?


Interesting note. Are you implying _memsetting_ to zero is different than _assigning_ to zero? Because assigning a pointer to zero will make it NULL.

ref: http://stackoverflow.com/questions/9894013/is-null-always-zero-in-c

C99 6.3.2.3:
"An integer constant expression with the value 0, or such an expression cast to type
void *, is called a null pointer constant."


Condition reversed there comparing with the original one.


This should probably be `IllegalArgumentException` and thus `checkArgument`


PS: take a look at this lines, they fix a Sysui FC: https://github.com/ezio84/frameworks_base/commit/e9237114b81362d516fa201db7b4ff0a5e7c694b


Yes, but I won't make any assumption about how is defined gunichar.


Obviously, this is now counted toward my space allowance, and will be applied elsewhere...  :)


Why didn't developer ID work out?


mysql functions are deprecated, currently working on a rewrite for all classes, will be no need for sanitization


What kind of exception is thrown?


workbench URL?


Accelerated mode in MP:
Unless all players will go into this mode, it won't affect the gameplay anyway. All computers "simulate" the game world separately, and after each turn they are "synced" with user input from packets. If one machine will delay sending its packet, others will have to wait.
To conclude, I don't think frameskip in MP is a problem.

Testing MP on one PC:
Yes, you can do that. I remember running such game on my PC, but I can't remember how I did it in detail. If this will help in testing MP, I could check that and prepare an instruction of some kind. Let me know if you need it, Loobinex.


@chandrasekharv

Looks  like both the for loop  iteration is doing same stuff, Is it possible to create one method and reuse it by passing necessary attributes as parameter (cell type, column width).


Done


@kashike

Do tell what that is.

To everyone else. Its a name. It was changed to avoid confusion with Guava in-which isn't going away. You subscribe to events, you listen to events, you handle events...many choices for names here. We chose Listener because that seemed to have the fewest conflicts.

@jk-5
EventListener was a possibility but seemed to be more verbose than needed.


7.0 is better


The new code that works for some reason is as follows ( I put nop())

  TBCTL = TBSSEL1 | TBCLR;
  TBCCTL6 = CCIS0 + CM0 + CAP + SCS;

  /* start the timer */
  TBCTL |= MC1;

  /* wait for next Capture, avoid an overflow */
  TBCCTL6 &= ~CCIFG;
  while(!(TBCCTL6 & CCIFG)) nop();
  last = TBCCR6;

  TBCCTL6 &= ~CCIFG;
  /* wait for next Capture, avoid an overflow - and calculate difference */
  while(!(TBCCTL6 & CCIFG)) nop();
  diff = TBCCR6 - last;


because soon nightly will be 7.1 so you won't even be testing on 7.0...


This could be:

```js
hasha.fromStream = function (stream, opts) {
  return new Promise(function (resolve, reject) {
    if (!isStream(stream)) {
      throw new TypeError('Expected a stream');
    }

    opts = opts || {};

    stream
      .pipe(hasha.stream(opts).on('error', reject))
      .on('error', reject)
      .on('finish', function () {
        resolve(this.read());
      });
  });
};
```

…if the `return Promise.reject(…` bugs you


So yeah, right now everything is stored in /home/$username/autoslack, but this will change soon-ish to the proper directory.


Noticed a mistake: here the R vector contains the opinion of all nodes but it is not right notation, as vector R represents the aggregation of neighbor's opinions


This is an old commit :stuck_out_tongue_closed_eyes:


this will now misalign the slot representations, and make them less useful as a visual aid...


`Root context.` и достаточно. Обрати внимание, что важно ставить точку в конце первого предложения в javadoc-е.


And by proper directory I mean it's just going to call $path/autoslack $i instead of this monster of a gowno-code.


New proposal for the whole paragraph:
"Because Windows does not provide real-time behavior, it is recommended to make use of dedicated hardware [reference to the PCIe solution]. Otherwise, for purely software-based openPOWERLINK solutions, the achievable minimum cycle times will be around 10ms and above. This number depends very much on the used system and cannot be guaranteed."


FP in Scalaでもそのままカタカナでしたね・・・


I think I'd rather leave it as is than deprecate the middle parameter.


Ну.. почти идеально же было. Этот комментарий бесполезен. Итак видно, что класс пустой.


Тут тоже точки не хватает. Посмотри как у тебя это будет выглядеть в javadoc-е.


Good point.
Created name and conn_url properties.


What's wrong about that?


Yeah. We already have a module that's shared between them, so I'd prefer not duplicating that code. We can have it diverge when we need to. And this code may well stay the same even when `massaged_params` changes.


Hmm I don't think you should swallow the exception here. That will definitely lead to NPE on l.122. Instead, those exceptions could be just propagated outside. parseConfig method is declared to throw Exception anyway so it's up to client how to handle it.


ok. hope it's positive changes.


At some point I'd like to pull the croutonfindnacl functionality in here directly...


Maybe overcomplicated? Could it be done in a simpler way?


Ah, it's used like a variable, nevermind :-)


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.942.f93ed9899d6e6adb44ddbf39957e86487f1c136d](http://build.palaso.org/viewLog.html?buildId=62470&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 486, ignored: 6 Build time: 00:05:53


Отлично!


Do we have a better link? I doubt we are updating this after release.


I must have missed the comment then


Лишние переводы строк.


license is missing


:+1:
Having it in the <head> was harmful indeed.


+1

Also, how about supporting sed-style \U?


Fix: Error in RequestHeaderMiddleware if headers are missing #19


Do we want:
  if (directory == null || directory.getFileName() == null)
here?
If getFileName == null it means that directory.toString is "" and I don't think we want to add that to srcFolders.


Вижу, что ты уже понял, что properties на русском должен быть специальным образом закодирован с помощью `native2ascii`. Или это за тебя сделала IDEA и ты не заметил?


This method belongs to ConfigSchemaGenerator (not used here).


why not just remove chris's bio? it's bad to have commented-out stuff in the live site.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.943.5dbc0fc345cccfdb1bc3cc954e3b1cc3bd855db9](http://build.palaso.org/viewLog.html?buildId=62471&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 486, ignored: 6 Build time: 00:05:06


Здесь конечно уместее было бы использовать не `delete`, а `move`.


Instead of print, we would retry for x times with timeout t there. Die trying?


That's what I thought. I am implementing the unordered version


Возвращай интерфейс.


This should be in a separate section above


Oh wow, didn't know about this method of setting `TEMPLATE` config. I like it!  Is it new in Django 1.8?


Can you  use one try catch block ? so that if   "file not exception" is caught the  "workbook.write(out);" method wont execute. And also add finally block and call the "out.close()" method or else  use "try-with-resources"


So that chmod(0770) is still accessible by the unprivileged user (due to group access).

I think the group should be set to `video` though, to handle the multi-user case.

You probably should set the umask instead of using chmod later.  And explain the whole permissions scheme in comments.


細かいですが、「特に、」の代わりに前の文を受けて「この発表では、〜のような人気のツールを使った例を見ていく。」みたいな感じにすると繋がりが良いかもです。


Nit: move function to .c, and check memory allocations.


Let's rename this - the usage is not tied to files only.


What does that mean "connect to Presto as the 'hive` user"? Is this the Presto username? How does that relate to HDFS permissions? It should only be used for the Hive metadata table owner.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 473](https://teamcity.labkey.org/viewLog.html?buildId=269424&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:08:43
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

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

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

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

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

```


Hi Dan, thanks for your excellent daterangepicker. I'm looking for a way to disable certain dates so they can't be picked. From the looks of it 'isInvalidDate' may do this? However, my JS isn't advanced enough to create code without some examples or more advice in the documentation. If I'm going along the right lines are you able to show a quick example here and/or in the documentation so that I can get the gist of it. Is there some sort of array of date that I send using this function?

Thanks


`NameError: undefined method `response_headers' for class `Faraday::Env'`
Faraday version: 0.8.10

Keeping this up may become a huge pain, but as long as it doesn't crash it we will be fine :)


Ты уже второй раз очень умело обходишь потенциальные проблемы :)

Обрати внимание на название метода -- `messageSource`, если метод переименовать, то локализация должна сломаться. (Аналогично, если не ошибаюсь, будет и с view resolver-ом.) Дело в том, что название бина получается из названия метода, который этот бин создает. А в некоторых случаях, таких как этот Spring ищет бины по фиксированному названию.

Я в таких случаях указываю название явно с помощью `@Bean(name=)` в т.ч. и чтобы подчеркнуть, что наименование бина важно.