Same with me @philbattos
It worked when  I removed that specific line


I like the idea of having multiple remotes.  But this is relatively non-advanced, and so probably should be the first level.


I would not put here the dependencies that are listed in `setup.py`. I know that right now they do not have the version numbers, but if we want to bump them later, we will need to change it in two places.
So, I would remove the `pyYAML`, `template-remover`, `html-linter`, `termcolor`, `nose` and `docutils`, dependencies.


Indeed. Fixed in b7a05eda6ce6796f9e58c92220b5711b81e9de0b, thanks.


why can't they just run `git checkout`?


In my testing this line was required, or otherwise an error was shown asking to set the mercurial identity.

Did something change so this is no longer required?


Awesome! Great to see you already resolved the conflicts. When I test-merged (unfortunally after doing the upstream merges), I noticed there were quite a few. :S


I'm not understanding the difference between levels 5 and 6.


Sorry, make that levels 4 and 5.


This is not the html5 tidy package.


@magnars @kaushalmodi  It should work, you just need to wrap it in `eval-when-compile`, too.


Yeah, that was an accident, accidentally git added too much. It'll be implemented soon.


What does `sudo: true` do? In http://docs.travis-ci.com/user/build-configuration/#Installing-Packages-Using-apt it says it is supposed to be `sudo: required`.


I've scheduled #120 for v0.4, so we'll deal with it pretty soon.


> but it's arguably easy to sort the trade objects before adding them to the array.

Yeah I took a look at that, but sadly where I need to use these I don't have *trade* objects, only JSON objects (as we're usually pulling data out of tradelistdb and pumping straight into a JSON object, there is no intermediate ```CMPMetaDEx``` object used)...

Thanks for the info though, very handy - I'll get stuck into seeing what I can come up with :)


@KostyaSha Thanks for pointing this out. So i fear we have to check the current docker-java code base against this API reference now.


Great!

This is another basic command, so I'd move it to the beginning even before `git reset`.

I'd recommend you build on this example by using the `git diff` command to compare two commits that aren't right next to each other.  You can take out one of the other problems to make room.


I agree - I'll play with that today and test it out - if it works I'll close out this pull request and create a new one using the node-uuid library. In doing that I'll probably want to update the package.json to include the dependency on node-uuid so that NPM doesn't blow up :)


+1 for removal, if it's useless, or not yet implemented.

But note: there is a difference between a "trade history" and an "orderbook". The latter should show active orders, while a history should show executed matches.


@gracjan I'm not sure what this thing did, but I'm pretty sure that I did no such thing in Flycheck.


Очевидно ты сменил line-ending


Still doesn't require `cherry-pick`.  If you want an example for this, talk to me in person.


great!


What happened to packages `xsltproc` and `php5`?


True, they're very rare currently. I was thinking of keeping them that way and introduce some other way for players to acquire them. Maybe there's a special level that has more books that others or some character is often carrying them.


Why is this being removed?


This looks really nice!  Great work!

A few of the levels still need some work, but overall I think you're ready to start implementing.


Maybe call it getDeprecationLogger instead since the current name seems to imply the logger returned is deprecated?


`EmitType::Reply` can be factored out of each case variant and added as the last line in the function.


You should use `locate-user-emacs-file`, really…


Yes it is.


Instead of pumping into JSON objects, some simple data transfer struct could be created, similar to those used in the UI, with only a very few members? I suggest this primarily, because I have a bad feeling about using JSON objects to store a lot of data (e.g. the pending objects), mostly because of the incident back then, when I changed some of the balance related RPC calls to use many JSON objects, which resulted in signficant lags. You may remember, otherwise I'd need to look up the details. :)


Looks good to me.


This is not correct, the leader is responsible for broadcasting the append_entries request to all followers, waiting for a quorum majority to confirm, and finally responding back to the client.  Obviously this is a lot more bookeeping.


This could just be a finally instead of a catch?


:heart_eyes: :wink:


Interessant. Ich habe versucht das früher zu verwenden bin aber gescheitert:
http://stackoverflow.com/questions/25998437/why-is-respond-with-being-removed-from-rails-4-2-into-its-own-gem

Nun, wir habe ja keine API in Ruby Level und vielleicht funktioniert es deshalb gut.


Hi,

I'm currently using this module and it works like a charme. Do you think it's possible to publish this last version of the module into the Puppet Forge ?

Thanks in advance.


@D-Alex Can you please comment on the Pull-request and not on the Comit page next time.

Nevertheless, https://github.com/rock-core/gui-vizkit/blob/03445b5abd7c35e3e7acfb98547a97cdac0177e1/lib/vizkit/vizkit_items.rb#L293

Might be the problem. There is only a check for equality and not for inheritance.
Feel free to propose a better solution. I would like to have this PR merged because it's working currently.


Ah sorry, the note above mostly applies to `getorderbook_MP` vs. `getopenorders_MP`, which I'd expect to be the same, since both tackle open orders.


Why did you change this to _'_ ? As far as I know, this shouldn't work, and you should use _`_


Changeable.


Hmm.. now I see why it's difficult to sort the objects.


I don't use this myself anymore (switched to webpack) but I'm willing to merge the request, but please take a look at my coment.


LGTM


I don't think this should happen here.  The follower should set the leader when it sees the first `AppendEntries` from the new leader.


'jepg' 오타군요. 'jpeg' 가 맞습니다. ㅠ_ㅠ


python 3.4 too


I remember :)  That was basically calling a divisibility lookup on each iteration of a loop - so for example when we did a call like getallbalancesforid_MP, it'd push out to a ```BalanceToJSON``` call once for every address but that call included checking divisibility.  Thus for a property with say 3000 addresses divisibility would be checked 3000 times (and since checking divisibility involves fetching the SP, it intro'd quite a lot of lag)...  I think we fixed that pretty easily just by checking divisibility once at the start of the iteration...

I get your point though - I'm just toying around with a vector instead - give me a few minutes and I'll be able to tell you if it works :)


why we need to bundle cc,as here


@pelargir
This fix doesn't work out
Is there any alternative??


Here I would write instead:
```python
if comment_data['severity'].lower() == 'error':
  comment_data['severity'] = termcolor.colored('ERROR', 'red', attrs=('bold',))
```

In that way the `format_pieces.append(...)` is not duplicated.


Do we need a build plugin here


@Sherekin hesch da nid gseh? :fried_shrimp:


Anything Goes also has the `Illegal` flag, and it's currently set up to only check for the double-Kyurem issue when that flag is set. Does that need to be changed?


Do we need build plugin in features


Please confirm weather we need all properties


I'm not convinced about this commit. What is doing (assuming that all linters exit with non-zero in case of messages) is that is setting the severity to error for all messages with no severity. Which I'm not sure is the thing we want to do.

For example, all messages of `pep8` or `pmd` will be now flagged as errors.


The first one is true. The second is not absolutely true. Maybe it would be best to say that it can scale up as we would expect (Ahmdals law). We can talk about the detail when we meet.


My bad. Fixed it. Thanks!


Shall we put a place holder /path/to ?


better :)


Pls change the description + T("View all organizations with 'CF_NAME orgs'"),


This commit should edit `event_edit_form.html`, not `generic_form.html`. It was a mistake during rebase against https://github.com/pbanaszkiewicz/amy/commit/600170f2ad4d58f5c521d2164605a0fe7b8410fa, so I'll probably remove this commit.


Просто обычно коммитил через TortoiseGit или приложение GitHub for Windows, а тут решил через IDE сделать это, типа тоже ведь не зря поддержка есть. Надо просто взять за правило так не делать...


Delete this intro. The reader only cares about how to set it up.  They don't care why you wrote it.


You need to give the actual commands for them to run.


In my opinion, the only difference between "enc" and "no-enc" should be the "enc". So I think that if one needs atomicity, then the other ones too. What do the others think?


So for clarity what I've done so far is:

* removed ```getopenorders_MP``` because it did nothing (wasn't implemented)
* rewritten ```gettradehistory_MP``` as ```gettradehistoryforaddress_OMNI``` because originally it just listed open trades for an address (not a history) - now it actually provides a history via the trades db
* added ```gettradehistoryformarket_OMNI``` which provides history for a pair via the trades db


There shouldn't be two versions.  All changes need to be submitted back upstream via a pull request.


The docstring also contains a short description of the function.


In the future, please submit this as a pull request to the gitlearn repo.


reset了几次这文件都会被识别为修改过，而我却没动过它。：-（，没 ignore 它上传了应该没问题吧。


Increasing the memory is not necessarily going to increase performance. Objects may start to be allocated in far away parts of the memory which would reduce the effectiveness of the memory cashes in the CPU. But the only way to see it would be trying different ones.


FYI I killed this call and replaced it with ```gettradehistoryformarket_OMNI``` :)


Remember, you are NOT writing a reference.  You are writing a tutorial.  So provide examples.  Do NOT just go through script-by-script and explain everything about it.


These are removed in commit 60880d7339a79837b07a3d6a02a0e837dccdfa5e


why not recommend the typical `Make.user` method to avoid having local modifications to version-controlled files?


These are removed in commit 60880d7


So basically make a fork of the original gitlearn and push all the current work on it?


This seems incorrect.

No exit nodes are needed when using end-to-end encryption with a download from a hidden seeding peers. Exit nodes are visible in a naked Bittorrent swarm. As everything is encrypted, no liability issue arises. Only 2 connectable peers are needed (introduction,rendezvous) with hidden seeding.


Why are you using underscore case and not lower camel case for method names?

It seems to be standard in Java to use lower camel case [1].

It would be good to use one method only and be consistent in all parts of the code. The same same could apply with classe names for tests. Sometime they start with the prefix Test and sometimes they end with that prefix.

[1] http://docs.oracle.com/javase/tutorial/java/nutsandbolts/variables.html


Well this is an interesting piece of information to capture, it shows we now make twice as many search_s calls..


Thanks Barry!  In Far, I see that we use "Internal" with capital "I", do we want to change the Far ones to lowercase, or these ones to uppercase?


inga kommentater, men det var många nya saker, borde kanske kolla igen när jag läst hela boken...


@Optimusprimeums keen for a release soon?


확인 감사합니다.^_^


I just realized the when I made the pull request it git did copy over the changes we made for gitlearn' scripts. I've included a link to the following files:
[link1](https://github.com/aso001/gitlearn_doc/blob/master/gitlearn/scripts/register.sh)
[README](https://github.com/aso001/gitlearn_doc/blob/master/gitlearn/scripts/README.md)


Hey @rsafonseca

I pulled the code and tested it, fine. However, this morning when checking some classes, I noticed that you did not add the new directory to git.

All the tests classes hare in the package "com/cloud.hypervisor.xenserver.resource.wrapper.citrix", but the directory doesn't exist.

find . -name "CitrixRequestWrapperTest.java"
./plugins/hypervisors/xenserver/test/com/cloud/hypervisor/xenserver/resource/wrapper/CitrixRequestWrapperTest.java

Since I'm busy with some code, I will fix that.

Just a heads-up. :)

Cheers,
Wilder


Great. Both are doable.


hey i m


Duh, of course!


@ccorda it might be worth dropping this repo (along with the history), and re-creating with the appropriate files/perms included... master is now the default.


public class HelloWorld{
2  -	public static void public static void main(String[] args) {
3  -		System.out.println("Hello World!");
4  -	}
5  -}


I believe we talked in person about your `register.sh` script.  I think
that's more trouble than it's worth, and not something I want in the repo.

As for your `README`, this is also not something I want in the repo.  It
provides no information beyond what's already in the scripts themselves.
Having this information in multiple places just means it's going to get
outdated somewhere.  If you really want to make more reference style
documentation, the correct way to do that would be to create man pages.
That is a pull request I would accept.

On Mon, May 25, 2015 at 11:51 PM, Andrew So <notifications@github.com>
wrote:

> I just realized the when I made the pull request it git did copy over the
> changes we made for gitlearn' scripts. I've included a link to the
> following files:
> link1
> <https://github.com/aso001/gitlearn_doc/blob/master/gitlearn/scripts/register.sh>
> README
> <https://github.com/aso001/gitlearn_doc/blob/master/gitlearn/scripts/README.md>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/aso001/ucr-cs100/commit/dfae30adbbb121fa13b6d6c92370a26de850135c#commitcomment-11357783>
> .
>


For Test in the class name to me it is most often used as the suffix, we happen to have most of our functional tests starting with Test, this I don't know why.
For the use of underscores helps reading the test name by making it look like a sentence. although here one could agree it is probably not easy to understand the tested behavior. Perhaps input_file_selector_using_job_id_is_filtered or inputFileSelector_usingJobId_isFiltered is better.

This blog post is often cited http://osherove.com/blog/2005/4/3/naming-standards-for-unit-tests.html


Format better.
For example:
```
var hits = _elasticClient.Search<Question>(
							s => s.Analyzer("standard")
								  .Query(
									q => q.MultiMatch(
											mm => mm.OnFields(i => i.Content, i => i.Title)
													.Query(searchValue)
													.Type(TextQueryType.CrossFields)
													.Operator(Operator.And))))
						 .Select(h => h.Source);
```


Your idea of sorting a vector gave me some inspiration - thanks dude :)

This seems to work OK (https://github.com/zathras-crypto/omnicore/commit/e8ac82d383d27038dbbc43848bdc2f6c440e13b7), what do you think?


[os 111] is duplicate


IsEmpty


That's where I started too. Did not work.


Reverted!


Just to be clear, this is not about this adapter, I'm writting a new adapter for PouchDB. It does have a _rev because I have added it in the ConvertoCouch Method. The problem with the story list seems to be that TW does not cares if that file exists already on the server, It ignores it in the skinny tiddlers and just tries to save it. I have reported this,but nobody seems to agree to change this behavior.


We don't need to make a `RecordingWork` here, this is done above in `_join_recording_and_works`


this should use `enumerate` to get a sequence number


Nice, thanks for the tip.


Ahh, nice idea. What about position within block?


Double space.


Moved modified dependency graph inside this repository for ease of testing, since we need to adjust dependency graph's host value.


No, that's correct. I forgot that double Kyurem-W's is blocked by `Illegal`, not `Pokemon`.

We might want to rename those to be clearer: `Illegal` means "possible without hacking" and `Pokemon` means "possible with a hacked pokemon"


Perhaps in the end this FileSelector should be named differently, just to avoid to use the full import?
Like ProActiveFileSelector, DataspaceFileSelector?


Hello!
Where you get this hash #1a4f57ad2ec0dd9b17703e13b46b96256b55b28d9e6529005e72f3d660e3f8a1"  ?


use this


Why is this line still here?


Added an empty build.sbt file to allow intellij to treat this as an sbt project.


This process is now more difficult. We need to look at the form object attached to `recording.form`, and then look at the flag `Form.attrfromrecording.  If this flag is true, we add a raaga and taala from the `tags` variable.
If not true, we get the attribute from the webservice when we get information about the work (http://musicbrainz.org/ws/2/work/41120dbe-35aa-4855-b395-9264cd26613f) - we need to make some changes to the musicbrainzngs library to read these.

We should return an error if:
 - We don't know the form
 - We can't find raaga or taala information in the place we expect to find it (based on the form)


machine:gcc, cross:gcc, sdk-cross:gcc or canadian-cross:gcc?


We don't have that info available (we don't store index in the trade DB, only the block the match occurred in).


> Ahh, nice idea

Hehe it was yours :)


I keep having kernel compilation issues. I am not sure if the failure I just had was a result of this commit or maybe because it wasn't pushed until I started the build.

Either way, here's the log (last few lines, the affected regions): http://hastebin.com/jeqelocinu.pl

This is an awesome defconfig, thanks so much


Law of Demeter violation.


It is needed in silverlight for some reason I can't explain
Den 20/05/2015 22.02 skrev "Julian Verdurmen" <notifications@github.com>:

> @Xharze <https://github.com/Xharze> why is that needed?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NLog/NLog/commit/3fa63bba67153c9b636094533ec46e3927c2b48c#commitcomment-11292269>
> .
>


Nit: `std::pair`


same here.


Yepp :+1:


Good idea, will do!


@magnars Oh, I see: `save-excursion` is a special form with its own opcode (see `byte-compile-save-excursion`).  That's why an alias would not work for byte compilation.


Ah, also my commentary there on this commit is regarding ```gettradessince_MP``` - it's basically just ```getorderbook_MP``` but filtering out trades before blocktime.

I don't see why this would be necessary, but if it is we should filter out trades on block height rather than block time and we can do so directly in the ```getorderbook_MP``` call, I don't see the need for a cloned function.


To me !...empty() is more expressive


variable re-use


Please advise if there is a reasonable prospect of this feature being merged and released OR if there is any assistance required to do this.


thanks! :)


Please, change name


good,好的


for what?


where is it used?


Mmm... there aren't any panels ? Could be useful add to the nemo desktop window an option to add a panel then? just to be more intuitive and easy.


again?


Nit: `std::make_pair`

The reason is that `std` and `boost` have quite a lot of similar elements (e.g. `boost::int64_t` vs. `std::int64_t`), and while those `using` directives in the header region can be used to avoid the need of explicit namespacing, it's imho cleaner to specify the namespace on an element basis. It's probably fine though, if we import for example the `json_spirit` namespace in the RPC files, because we use many, many `Array`, `Value` etc., and there is no conflict.


@Subjunk sure, but are you going to pull #552 before?


It's a SHA256 checksum of the archive file. You can calculate it by using 'sha256sum' or 'openssl dgst -sha256'.


Yes, I agree. And the changes look great so far! :)


Where is it used?


is stage private or public? the default is protected, which is private to the package


Thank you for removing lines of code :) 711 additions and 1,915 deletions.


https://github.com/zathras-crypto/omnicore/commit/183aa28ca1b0a663f1ed9006a9988191be29ac80 :)


This is a lame work-around.
On line 501 is a check whether the tid actually exists, so the term should be available too. If the term cannot be loaded due to some migration artifacts - just edit the node and assign the correct section term.


Thanks bud :)  I'm still running over the RPC but just trying to give it a 'clean up' as it were and make sure the calls do what they're supposed to - I should be done soon then I'll PR this branch and summarize the changes and get feedback if they're OK...


Awesome. I'm really excited about the new RPC stuff. Especially the new data population is very straight forward.


you shouldn't use code logic inside unit test.


TeamCity Ybackend :: synteny/ybackend [Build 30](http://192.168.30.225:8111/viewLog.html?buildId=1283&buildTypeId=Ybackend_SyntenyYbackend&guest=1) is now running


Should we rename `"canceltradesbypair"` to `"canceltradesbymarket"`?


修改的不错


```php
$this->assertThat($includes, $this->isTrue());
```


```php
$this->assertThat($includes, $this->isFalse());
```


Ok, thanks.


test 0526


The new paremeter *acceptSpaceBar* should be included in matcher calls(lines 432, 608)

Thanks!


```php
if ($filtered === false) {
```


```php
$this->weekIndex = $filtered;
```


Great! Thanks!


I see that it's unchanged, but it's better to use `, this` instead of `.bind(this)`


I ran the benchmark with 5M elements and 4G of RAM, and the performance for miniboxed and generic `reverse` was identical. I can't find another explanation for this other than the fact that the miniboxed vector triggered more GC cycles. Right?


Hehe - that's what I was going to look into next, in a nutshell the question:
"What's needed to move other parts of RPC to a single value (market) rather than two values (pair)"

I think given that we can always determine the other side of the "pair" from market (if it's a main eco property the opposite side is MSC, test eco it's TMSC) it's a nice simplification, but there are some unanswered thoughts rattling around my head...

If we take your example, we'd have:
```canceltradesbymarket 3``` would cancel all trades on pairs ```1 3``` and ```3 1```
```canceltradesbypair 1 3``` and ```canceltradesbypair 3 1``` offer more granular control

I'm kind of still leaning towards this ```market``` idea but I'm still musing on it :)


Obviously, the Custom Vehicle type is more suitable for VTOL aircrafts, so these functions aren't used anymore.


nice :)


this is file is changed and hence deleted


Can you replace the tabs with spaces in the file and follow the 4 space pattern


It would seem more clever to
```
#define ALIGN JTR_ALIGN(32)
```
in order to keep all work we already put in getting this portable. Now we need to start over.


Well, I didn't know until it stopped working :). If you want to add networking to the regular builds so we can use those then that'd be fine with me.


This is my `AuthConfig` #1


Don't need to close, just make the push, it will fall here.. just make sure to not send all the commits maybe.. you can convert all commits to one before pushing.. thank you..


Why we need to have possibility to pass `store` here? We already have store for CookieJar, passed into constructor, isn't it?


TeamCity Ybackend :: synteny/ybackend [Build 30](http://192.168.30.225:8111/viewLog.html?buildId=1283&buildTypeId=Ybackend_SyntenyYbackend&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 77 Build time: 00:13:04


You just dropped all `#include "memdbg.h"` and introduced other problems as mentioned elsewhere. Will you keep working on this?


Got it. But my point is that `if key in ['some_value']` is a confusing version of `if key == 'some_value'`.


how did that end up there in the first place ?


@j-m-r: The problem is with GerritStatusPush: The names of the Gerrit project and the Buildbot project have to match in order to make Buildbot push reviews to Gerrit. If the Gerrit project name contains slashes, the Buildbot project name would need slashes, too, but this breaks the webview and other parts of Buildbot.


Hm, but the plugin depends on jQuery? Are you sure to remove this?


Thank you for pointing this blog post. However, it seems that not all words should be separated by an underscore char but only unrelated sentences or words. Besides, words seems always capitalized like in camel case. It would be a good idea to be consistent with all tests. It could be useful when we have to select some specific tests (with patterns) for execution.


lgtm


Fine for me, this seems a reasonable choice


```coffee
if range is "+" then 1 else -1
```
でもよいぞ


You should look at the inlining of the functions inside the reverse. There might be some that become bigger than 35 bytes in the miniboxed version but are smaller in the generic. Hence the heuristics could inline less functions and the performance degradation could be explained by the additional function calls.


Shivam,
It's great sample project.Thanks a lot.
could you please send me the document of this sample for better understanding.(purpose of each module.)
At my email id :vineetcs0110@gmail.com


Sorry,
I did't Created!!!

*Regards*
*Shivam Srivastava*

On Tue, May 26, 2015 at 1:13 PM, Vineet2121 <notifications@github.com>
wrote:

> Shivam,
> It's great sample project.Thanks a lot.
> could you please send me the document of this sample for better
> understanding.(purpose of each module.)
> At my email id :vineetcs0110@gmail.com
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/shivam01990/InventoryManagement/commit/90ee9e861eb281f8756fb6d84584f33e4b7066f2#commitcomment-11358411>
> .
>


The change should be also documented in the `Changes` file.


@ticktricktrack @adinapoli these functions might be of use to you both.


Should this be copyright to Friends of Cake?


I am always welcoming ideas when it comes to the structure as its a learning process for me too. Sometimes things do not really fall nicely into place.  I guess private members do not belong in an interface, but then, new cacheAsBitmap() is not really intended as a class.

I will look into mixins more, last time I checked these out ages ago with createjs, they were not officially in the handbook.


You need to test if the ``email`` key exists before using it, like you have done for the other


This breaks HTML5 on desktop, proposed code:

```
mw.isIOS8 = function(){
		return /Version\/8.0/.test( navigator.userAgent ) && mw.isIOS();
	};
```


@magnumripper yes, I will try to get this fixed (not very soon though) ;(


前来抢沙发，支持~


I can pull this out (the NOP) if there is objection but for some reason I thought it was nicer.


I prefer the ``isset`` function and not using the inline test function, but it's not up to me to decide ;)


I have no idea,  presume it was there before any default case


Why do not use some JSON lib for this?


No Javadoc


Well, isset doesnt allow null value. Wasnt sure null was allowed or not.


I don't agree with these modifications:

- The added lines are ok, but the master must be switched OFF just after checking the fuel level or you'll have the master ON for about 10 minutes before starting the engine, depleting the battery. This is done in the pre-fligth inspection. We should probably recover the complete "pre-flight" checklist. It was me who substituted that checklist by a single entry in the "before starting the engine" checklist. Since most of the items during the "pre-flight" are not very interesting in a simulator (check you have the key and insurance, weight and balance calculations, external inspection...), I removed the list and moved the "check fuel level" to a probably less adequate moment. I'm sorry, it was probably a bad decision.
- The deleted lines shouldn't be deleted: the master is switched ON exactly at this point of the "starting the engine" checklist.


Don't think so. Here are the numbers with 5M elements and 4GB heap:
https://github.com/milosstojanovic/miniboxing-plugin/commit/c621f5b5198756349b077acadae78cc3ac6bce29


please put a try finally around this.


素晴らしいですね…ありがとうございます＼(^o^)／


The latest commit had this fixed in 1.0.1


it is only one file and we remove it within execution of this script, I don't see how this would be a big advantage.


And addRecipientMetadata() uses inline if aswel.


Sorry, I should have deleted those functions. I do wonder though how can I secure the writing of the StabilizationSettings UAVO.


Javadocerino's


You should do that on the same branch (`1.0-uploadable`). Now you have to merge `1.0.1-uploadable` into `1.0-uploadable`, and remove `1.0.1-uploadable`.


@synctext  This might be confusing for others, but this code is correct. Every circuit ends in an exit node. Even the circuits used for end-to-end crypto. But it will not actually 'exit' any data, it's just the connectable end point of a circuit!


@ElizaSapir  i'm pretty sure we already encounter this bug and fixed it  - please make sure v1.9.x branch is up to date - Thanks @DBezemer


Will be done.


Okay


I have merged your pull request, but had to do some changes afterwards to make this compatible with the current behaviour. Today, groups are collapsed by default (except for groups in lists)

Thank you for contributing - we really appreciate it!


I removed it because the dependency did not merge with other plugins and overrides `jquery` with `jQuery`, so all other plugins are failing.
Another possibility would be to rename the dependency, but i didn't tried it.


Yeap, I thought about it. PaginatedResult is good for me. Will be refactored.


circleci is our fastest build and usually finishes within two minutes after pushing things, adding this command will make it take a few seconds more only.


The check on line 87 is about Ehcache template provided `CacheLoaderWriter`, so at line 97 we have one that we will use and we check in case the JSR-107 cache does not have loader/writer config set.
I'll rename variables to make it clearer.


seems legit? So, the JS makes a timestamp based on milliseconds, and postgres wants one based on seconds?


You are right about this.  These warnings are important. I've reverted this commit and made the necessary changes:
https://github.com/analogdevicesinc/iio-oscilloscope/commit/96d3f59886823d9d76ede591624b6082bd5ebcd2, https://github.com/analogdevicesinc/iio-oscilloscope/commit/272bf0d7cf31a48c285c8101ee23a7898c0f0ec1

Thanks,
-Dan


I guess you have already learned that ones? Do you have video recordings where you playing this compositions?


这几个词条我是在游戏里看到才去汉化的。。除非其他地方会再出现。重复进了还几次游戏，建了3个新存档没发现异常。


Please don't talk like that.


Can you please set your IDE import settings ? You can use the preset files in the build tools project.

Everybody keeps complaining about checkstyle and friends. But then, for the one thing which is not enforced at build time, we continue to see commits breaking the rules.


Same as the TxPIDSettings functions, I should have deleted this function.


Can't you just paste the comments here?


Thanks. :p


you're not setting `fullpath` when `filename` is an absolute path.


fyi  the \`\`\` can just be \` if the code's on a single line

`sucka` (single) == ```sucka``` (triple)


What does `Till` mean?


From usage in other headers, it seems the coding style is `libfreenect2::mutex` for the type.


@907th The docs are outdated now, isn't it ?

[0] http://readme.drone.io/usage/notify/hipchat/


@nicowilliams: don't think the jq(1) install path has anything to do with these lines. The `program_origin` is just the path to the file given by `-f` as set by `program_origin = strdup(program);` outside of this diff view.


Should this (and the corresponding test) be backported to 2.1 and 2.0 branches ?


Ignore this commit. Testing Jenkins build filters.


> Conflicts:
>	render/render.c

Please remove this part (the Conflicts: passage) from the commit message.


> We don't need to stat, and if we did we should fstat() (since we've already opened and read the program file here, we wouldn't want to race).
>
> If FROM_FILE and we managed to load it, then we should just use that path. We should still have an exception for /dev/ and /proc/ paths on non-Win32.

You're the pro =)


I haven't looked at the code, but I'm pretty sure that Ansible modules have methods like `should_run` that return booleans to determine whether a module can be marked as not changed and skipped. It is probably the duty of the developer to make sure that that method does the right thing. The core modules are relatively well-written in that respect; if the AWS stuff is still under development, it might be that they improve with time, or let's hope they do improve.


Actually the syntax should work with the ES7 export extension proposal, but in ES6 it should be `export { default as foo } from './path'`.


在支持requestAnimation的环境也可以强制使用setInterval


the comment about "would generate xpcompsmp.dll" wrong, as it would generate xpcomsmp.dll


@nicowilliams: guess you implemented the answers to these comments in your commits. I've pushed them with to my pull-request.


test


In JPMML-Model version 1.2.X, all plain setter methods (eg. `setProperty(Property)`) and Collection-type setter methods (eg. `addProperties(Property...)`) return `this`, which makes it possible to chain them together. For example, I would do the initialization of a MiningField element like this:
```
MiningField miningFieldCopy = new MiningField(field.getName())
  .setOpType(field.getOpType())
  .setImportance(field.getImportance());
```

It should be also possible to do such "copy construction" reflectively. However, there is no ready-to-use utility class for that yet.


Opened PR #9 for that specific point


ok thanks :+1:


@Ionic: @mikedep333: LGTM otherwise.


Because I'm running the nagios check via sudo. I set sudo /usr/local/lib/nagios/check_peering_bgp in forcecommands.cfg and only have to allow running THIS check via sudo and not some birdc 3rd party external stuff.


This is fixed by the patch


thanks to this commit, the option `--security-opt="apparmor:unconfined"` is used to avoid restore failure for me.


glad that it didn't cause any merge conflicts for my commits. :p


This is suppose to fix pth-config agains binconfig. binconfig can replace bindir but not pth_bindir
So i do that for all the major variable, that why the post install sed is not needed anymore


Trying it now:

```
[nix-shell:/tmp]$ mpv http://www.youtube.com/watch?v=NvsM9B4ac2g
Playing: http://www.youtube.com/watch?v=NvsM9B4ac2g
Segmentation fault
```

Sucks.


ww


Please dot not break PEP8: https://travis-ci.org/OCA/account-payment/jobs/63975712#L144


To be sure that we use toolchain tools. For my POV i prefer autoreconf that use project files


这个只把异步改成同步就好了嘛？


I don't see any usage of the `handle_stdout`. How is it related and if it's only for avocado, can't it be defined there?


The vmhgfs module is part of the open-vm-tools, but I didn't look into compiling them (--without-kernel-modules). I'll have a look if it'd be easy to get it working.


This is tip to get all organization name.


Kuntge apparte zinnen op apparte lijnen zetten? Git vindt dat leuk.


# Taskについて
* mapとlistの2つあるのは、複雑かつ処理の多重をしていませんか？
* たぶん、前にもいったかもだけど、2つは必要なくて、mapだけで大丈夫です。
* でも、keyをstring型でmapを使うのは非常に重いです。
* `unordered_map` を使いましょう。これはmapより高速です。
* listでたぶん、keyが同じものが入っても格納したいから使ってると思われる。
* 同じkeyで複数のオブジェクトを格納したい場合は、unordered_multimapを使いましょう。
* 現状の `find` では、mapの中しか検索できていませんが、listのは検索しないのかな？
* mapした検索しないなら、listは回すだけになっています。
* 回すだけの構造データなら、vectorのほうが高速です。

以上です。


Indeed, i only changed the actual folder in /plugins/hypervisors/xenserver/com/cloud/hypervisor/xenserver/resource/wrapper/ and not under the tests folder, but it's just semantics, all is working fine :)
Do fix that semantic issue ;)


でも今のままの方が読みやすい気がする


Die dominantie is niet biologisch, maar komt uit de omgeving. Geen evolutionair kenmerk dus.


Another option:

```ruby
community.community_customizations.where(customization_params(marketplace_name, locale)).first_or_create!
```


Yes :)


if `configuration == debug`, then define constant `WINDOWS`?


Would you please update the commit message to describe the changes, rather than generic message about some other project benefiting from this? I can see some style refactoring and new features like `_variant_only_file`.


latex dependencies ... too lot


Is this with `ffmpeg` or `ffmpeg-full`?


my default copy paste ...


Goeie verbetering!


Ok i will


same, as usual


I think this patch may be outdated by the original authors work here:
https://github.com/VictorDenisov/librsync-1


Paint.NET isn't open source anymore beyond version 3.36.


wrapper class from the core PR
without BISON_PKGDATADIR env variable, bison do the search in /usr/


Zet hier misschien nog iets `TODO` achtig bij? Een synthesevraag kan niet uit één text beantwoord worden he.


"an instance", while you're at it.


@jalemieux thanks for testing... I looked at a coupe of scripts we use internally and noticed that they solve this by actually doing two calls. The first one without a project ID and the second one with the project ID set to `-1`. I think we could/should do the same here within the `GetXXXByID` functions that also have a project ID parameter.

Additionally a `GetXXXByNameAndProjectID` still seems to make sense as searching on name is much less specific than (of course) searching by UUID. So in that case being able to narrow the search makes a lot of sense.

What are your thoughts?


:+1 (sorry @ngaskill we have to release ASAP)


The latest stable version of firefox is 38.


How would these become uppercase? Not sure as to why this is needed?


full


by default autoreconf script remove aclocal.m4, the aclocal.m4 from nettle have custom m4 script which is needed for autoreconf. Like other recipes, shouldn't we use autoreconf as default?


Hello Lucas, I disagree with this patch, because `run` is a standalone runner specific ONLY to running virt-test. It does not contain various command groups so the separation is IMO unnecessary.

To cooperate with avocado it's always possible to create mapping from avocado params to virt-test runner and supply those mapped arguments instead.


```
(gdb) r http://www.youtube.com/watch?v=NvsM9B4ac2g
The program being debugged has been started already.
Start it from the beginning? (y or n) y
Starting program: /nix/store/a0kk7j92kzsxaf7dmgsapqyavcd8rj94-mpv-0.9.2/bin/mpv http://www.youtube.com/watch?v=NvsM9B4ac2g
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/nix/store/c2zjp7bqmp5wvpkrpl7p7sfhxbdyfryy-glibc-2.21/lib/libthread_db.so.1".
warning: File "/nix/store/cpv8pyc772cx0spzz76sa6dvsf6555dh-gcc-4.8.4/lib/libstdc++.so.6.0.19-gdb.py" auto-loading has been declined by your `auto-load safe-path' set to "$debugdir:$datadir/auto-load".
Playing: http://www.youtube.com/watch?v=NvsM9B4ac2g
[New Thread 0x7fffd6495700 (LWP 9088)]
[New Thread 0x7fffd5c94700 (LWP 9157)]
[New Thread 0x7fffd6c96700 (LWP 9087)]
[New Thread 0x7fffd7497700 (LWP 9086)]

Program received signal SIGSEGV, Segmentation fault.
[Switching to Thread 0x7fffd5c94700 (LWP 9157)]
0x00007fffd7ba6a93 in deinit_responder_id.isra () from /nix/store/p32faqr1jls03xa7n39zw58vpgq8ibdc-gnutls-3.3.15/lib/libgnutls.so.28
(gdb) bt
#0  0x00007fffd7ba6a93 in deinit_responder_id.isra () from /nix/store/p32faqr1jls03xa7n39zw58vpgq8ibdc-gnutls-3.3.15/lib/libgnutls.so.28
#1  0x00007fffd7ba6ae7 in _gnutls_status_request_deinit_data () from /nix/store/p32faqr1jls03xa7n39zw58vpgq8ibdc-gnutls-3.3.15/lib/libgnutls.so.28
#2  0x00007fffe7ed8c65 in _gnutls_gen_extensions () from /nix/store/mwllg9lxgi5wjqwn0f9lcq01rlikxlvp-gnutls-3.4.1/lib/libgnutls.so.30
#3  0x00007fffe7ecb9cd in send_client_hello () from /nix/store/mwllg9lxgi5wjqwn0f9lcq01rlikxlvp-gnutls-3.4.1/lib/libgnutls.so.30
#4  0x00007fffe7ed0145 in gnutls_handshake () from /nix/store/mwllg9lxgi5wjqwn0f9lcq01rlikxlvp-gnutls-3.4.1/lib/libgnutls.so.30
#5  0x00007ffff46e5343 in tls_open () from /nix/store/j5zi48l0m8arj7a967v0fl93qiv1pwpx-ffmpeg-2.6.3/lib/libavformat.so.56
#6  0x00007ffff45feb92 in ffurl_connect () from /nix/store/j5zi48l0m8arj7a967v0fl93qiv1pwpx-ffmpeg-2.6.3/lib/libavformat.so.56
#7  0x00007ffff45fed14 in ffurl_open () from /nix/store/j5zi48l0m8arj7a967v0fl93qiv1pwpx-ffmpeg-2.6.3/lib/libavformat.so.56
#8  0x00007ffff4632dbc in http_open_cnx_internal () from /nix/store/j5zi48l0m8arj7a967v0fl93qiv1pwpx-ffmpeg-2.6.3/lib/libavformat.so.56
#9  0x00007ffff46334c3 in http_open_cnx () from /nix/store/j5zi48l0m8arj7a967v0fl93qiv1pwpx-ffmpeg-2.6.3/lib/libavformat.so.56
#10 0x00007ffff46339c9 in http_open () from /nix/store/j5zi48l0m8arj7a967v0fl93qiv1pwpx-ffmpeg-2.6.3/lib/libavformat.so.56
#11 0x00007ffff45feb92 in ffurl_connect () from /nix/store/j5zi48l0m8arj7a967v0fl93qiv1pwpx-ffmpeg-2.6.3/lib/libavformat.so.56
#12 0x00007ffff45fed14 in ffurl_open () from /nix/store/j5zi48l0m8arj7a967v0fl93qiv1pwpx-ffmpeg-2.6.3/lib/libavformat.so.56
#13 0x00007ffff4601b12 in avio_open2 () from /nix/store/j5zi48l0m8arj7a967v0fl93qiv1pwpx-ffmpeg-2.6.3/lib/libavformat.so.56
#14 0x0000000000489cf5 in open_f ()
#15 0x0000000000480593 in stream_create ()
#16 0x000000000046c8d7 in open_stream_thread ()
#17 0x000000000047082e in thread_wrapper ()
#18 0x00007ffff2856484 in start_thread () from /nix/store/i0l0jjkk82wsqz9z5yhg35iy78bjq684-glibc-2.21/lib/libpthread.so.0
#19 0x00007ffff259504d in clone () from /nix/store/i0l0jjkk82wsqz9z5yhg35iy78bjq684-glibc-2.21/lib/libc.so.6
```

I don't have time to investigate right now


This does make me wonder, then. Is it possible to hack multiple of the BW2 Kyurem formes into a game? I had simply set up that rule to check the `Illegal` flag because I assumed that's where it went.


Might not save a lot of space, but those
```c++
if (mp_obj.step1() < 0) return MP_INVALID_TX_IN_DB_FOUND;
if (0 != omniObj.step2_Value()) return;
```
... could be replaced by calling at the very beginning (or anywhere):
```c++
if (!omniObj.interpret_Transaction()) return ERR;
```

It's safe to do so ("doesn' trigger effects") and it parses the the whole packet.

I'm going to take a look at
```c++
if (0 > omniObj.interpretPacket(&temp_offer)) return;
```
... later, because I don't feel very comfortable using `interpretPacket()` to fill in some data.

I'm really not sure, where to go from here. It might also be possible to move the packet interpretation into the offer and order classes? Having all values (which are not even public) in `CMPTransaction` isn't viable in the long run imho, but I don't see a clear path right now.


I think it's probably still worth running all the code below?


> So now AFAICS, there are flag-variables in 3 places:
>
> in the Ndo structure in stack of main().

Some of the flags to tcpdump affect packet dissection, and thus, if we're going to have the dissection code be a library (usable from programs other than tcpdump), as I understand is the intent, those flags would belong in that structure.

> in int Xflag variables on stack of main().
> and in static int Xflag variables in tcpdump.c.

Other flags don't affect dissection, and thus really *don't* belong in that structure.

Whether they're local variables in `main()` or global variables is probably a function of whether anything outside of `main()` refers to them; if you'd prefer them all to be global, or all to be local and passed to the routines that use them, we could probably do that.

> Can the getopt() processing be lifted out of main() to improve readability?

It could be moved to another file not included in libnetdissect; it wouldn't belong in libnetdissect.

> (a 400 line switch-statement!)

Which options would you like to see removed?


This one I won't take since the new default that creates directories per section guarantees that this number will be way lower, and I still have not seen a single session with more than 99 videos.


Думаю, как минимум, надо изменить приоритет на ооооочень низкий, так как сторонние фильтры применяемые для `the_content` и `the_excerpt` могут сломать вашу html разметку.


I think I know what it is, I'll see what I can do.


I can't actually tell by the code, this is the one with lots of textures(for each possible state), or with couple textures rotated and overrired?


Our First Lecture


I derped lol i fixed it though


Плохо плодить 100 500 одинаковых урлов :(


+ Передаете фильтру 2 аргумента, а принимаете 1.


:clap: , vade retro, scripta supervacanea


That's the beauty of a dynamic language such as PHP. This isn't Erlang or such where we have to deal with single assignment.


Thanks for working on this. I'll reassign HWKMETRICS-101 to you


Why not just removing files instead moving them?


extract `suggestedOptionId` via `let {isListOpen} = this.state;`


You're welcome to clean it up.


Not sure if just renaming is enough. I don't really believe that this was ever called and I'll have to look at coverage info to see more.


forget it, I did it in 2.1 with r13557 -- 2.0 is not affected, there you get ERROR:  getPoint4d_p: point offset out of range


maybe do that in `_selectOption` ?


This introduced a regresion to mail encryption feature.
"if(decryptMessage && part.getContentType() != null && part.getContentType().equals("application/pkcs7-mime; name=\"smime.p7m\"; smime-type=enveloped-data")) {
                part = decryptPart((MimeBodyPart)part);
            } "

must be added after "521 +            Part part = mpart.getBodyPart(i);"
Otherwise the if the mail was encrypted this operation won't work.


WAUW!
+1
geliked en geretweet


Cool. Merged this in master!


This is finding by id, not by email.

The original error with restore password was that users weren't found by email.


Damnit.  Inccluded hilow before it was ready to go


30411/Danciu P. Diana Lacramioara/Assignment 8/Sorting/SortingAlgorithms.c

you are missing


What is this new file: reflect-metadata/Reflect.js?


it is part of the possible fields. It is systematically provided, like the id. Please see the writeMetrics method, I don't test if the parameter is desired or not


I'm sure it is not : )


The goal here is to add an offset on the servo that controls the motor tilt.


maybe a way to enable debug build for a selected recipe from command line or distro conf?


Yes i will add headers


you're right, and it's a pain to test with the database id in the response


To test this use curl like below
```
curl -X OPTIONS -I http://vngrs-challenge.herokuapp.com/api/applications -H 'Origin: *' -H 'Access-Control-Request-Method: POST' -H 'Access-Control-Request-Headers: test'
```


Ok i will. same reason that the other recipe


nettle


Maybe add some example with extra params

```ruby
      foo_csv: [:csv, load_csv, { encoding: 'utf-8', col_sep: ';' }],
```


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


my patch, the recent help2man fail on some warnings from the recipe, that just ignore that ... maybe there is a better way


Do we still need `KeyValueRowReaderFactory` ?


What's the difference here? Also the formatting feels weird.


@zenangst What has changed?


that is necessary, by default texinfo in a cross compile context (see configure.ac patch) try to build on is own, the native makedoc. We built the native one with the "file://install_makedoc.patch" patch for that


I added the time fields


@bmaschio --portName is repeated twice in the list of arguments?


Hi @rsafonseca ,

Perhaps you missed the point, but what I meant was that all the Test classes have the package declared as "com.cloud.hypervisor.xenserver.resource.wrapper.citrix", whilst the physical files are under the directory "com/cloud/hypervisor/xenserver/resource/wrapper".

Maven is building the classes in the right location, hence the tests are being executed and passing just fine. However, if you open the files in an IDE, the compile (which doesn't depend on Maven flags) will complain.

That's the current package declaration for CitrixRequestWrapperTest:

![image](https://cloud.githubusercontent.com/assets/5129209/7808441/907032ea-0393-11e5-9523-b2a8b7113929.png)

It is handy to have the source file inside a directory with the same name as int the package declaration.

I already fixed that "semantic issue". Will push in a bit.

Cheers,
Wilder


sure. native:pkg-config can't use pkg-config for finding glib :)


Shouldn't this return 1 rather than 0 ?:
```
ST_Line_Locate_Point(ST_MakeLine('POINT EMPTY'::geometry, 'POINT(1 2)'::geometry), ST_Point(-11,40));
```

As it still returns 0 here


This commit seems to have some implications for this - https://github.com/hyperoslo/Form/commit/726ed01f9b6fe1a391918d4b4b15be4e082cc408


I try to build world with the the 0.26, i failed. i can retry and give you the output


I can't do anything withthe version right now, could you open a bug for this? I'm fiddling with the version in another pull request and that would conflict. So let's resolve that until everything is merged.


never mind, as location is a fraction of length and line length is zero, the answer is still correct.


so launch files is installed in include directory ${CATKIN_PACKAGE_INCLUDE_DESTINATION}?
> ls /opt/ros/hydro/include/pr2_gripper_sensor_controller/
acceleration_observer.h  digitalFilter.h  gripper_controller.h  launch  pr2_gripper_sensor_controller.h  pressure_observer.h

shouldn't it be in ${CATKIN_PACKAGE_SHARE_DESTINATION} instead ?


Really?:((( To what license have they changed ?


https://www.webrebels.org/public/img/rebels.jpg og https://www.webrebels.org/img/rebels.jpg gir 404.


okay


OK. In this case I delete `convertedTiddler._rev` because there is no reason for it to have a `_rev` and I rely on the rev in adaptorInfo.


is this a breaking change? we started seeing errors like this "ReferenceError: Can't find variable: WebFont" over the weekend. using npm & browserify.


Oh I think I am confusing Paint.NET with Pinta. Pinta is the Mono/Linux port/fork of Paint.NET and is still opensource.


Yes


La fetch è stata introdotta perchè al momento della visualizzazione il sistema non riusciva a prendere la lista gli utenti della community


okay. Maybe that done in the wrong way but by default automake doesn't provide a way to override the default search path. This patch will check the env variable 'aclocalprefix' if it's defined will use it instead of the default prefix


What would you suggest, just remove the "Return to 3D Reconstruction Menu" button or contruct 3 buttons myself? What would be the "official" code for closing the menu?


Ok, Pinta also runs on Windows, so we should replace Paint.NET with Pinta in this list then.


La fetch è stata introdotta perchè al momento della visualizzazione il sistema non riusciva a prendere la lista delle community dell'utente


The hack in .travis.yml can be removed now, right?


bin/
.metadata/
.project
.settings/


if i correctly remember this switch doesn't exist anymore.


Now line numbers are part of methodtokens


Thanks for mistaking, it's nice to know about Pinta. :)


I'll leave `*args` here because it's passed to `super().__init__()` – to ensure nothing breaks.


Hi @gregoire-pointeau,
is there a reason why you changed the variable type to `double`, and cast it to `long` below?

Also, can you please change your editor to use spaces rather than tabs :)? Otherwise we have a formatting mess :P.

Best, Tobi


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


I am preparing a PR for this change, Paint.NET is indeed no longer opensource, see:
http://www.getpaint.net/license.html


mi tak nie działa jak Tobie wczoraj, tan. w Lista przedmiotów nie wypisuje się lista aktualnych przedmiotów jakie posiada student tylko nadal jest pusta. nie wiem jak to zrobić i nie pamiętam jak robiłes to wczoraj...


Sure, with old gcc, the gcc fails on multiple definition. The comment on the patch is quite good


yes for native:glib with old gcc


Good point. How about we just move the isArray and arraySize logic into your second switch then? I think it'll be more readable with the logic in one spot.


Hi @wilderrodrigues
The refactoring of the Libvirt broke Libvirt functionality in runtime.
Whenever spinning up a new instance in master, deploying VMs with kvm fails with no useful info in the logs.
Upon getting some debugging in, it seems that the injected LibvirtUtilitiesHelper is returning null at runtime:

LibvirtComputingResource.java:

    @Inject
    private LibvirtUtilitiesHelper libvirtUtilitiesHelper;

LibvirtXXXXXXXXCommandWrapper.java

...
final LibvirtUtilitiesHelper libvirtUtilitiesHelper = libvirtComputingResource.getLibvirtUtilitiesHelper();
...
    conn = libvirtUtilitiesHelper.getConnectionByType(vm.getHvsType());
...

Throws NullPointerException on libvirtUtilitiesHelper

Can you have a look?

:)


To be able to have both x32 and x64 bit version installed - turboJPEG's installs in two different folders...


I think we need a `terms.md` where all terms will be defined. The ROM dictionary becomes quite rich.


"546 +        return null;" is also not ok.
If you put messageMap.put(MULTIPART_MIXED, null);
This will generate null pointer exception in
result.put(BODY_RESULT, MimeUtility.decodeText(lastMessageBody));

So there should be return "" insted of null.


Sure, I'll add it.


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


:heart: commit name


Ah I think you are right. I can change these around. How do you suggest that I make these changes? What is the normal process? Shall I do another commit to update this PR? Else, if I you still want me to squash this down to one commit then I will have to force push to my forked repo which will break this PR and I'l have to create another one. What do people usually do to incorporate feedback?


A "model 3" driver was also developed for the DeltaTau PMAC controller(s) by Matt Pearson @mp49 back in 2012. I think it was rolled out and is still running on a single beamline at DLS before Matt left.

Available in the tpmac module: http://controls.diamond.ac.uk/downloads/support/tpmac


Well I guess changing it to this

```json
"dependencies": {
    "jquery": ">=1.7"
}
```

is more appropriate. Removing it is not really a solution.


Hi @wilderrodrigues
I'm just using shell, no IDE, so no complaints here :)
I understood the issue, but it i have no warnings in maven and everything is working fine, so i missed that :)


Just a commit called `test` and removing 1 line isn't so clear. It will be displayed in the original `cakephp-utils` list too...


Ah, @quintesse, so _that's_ why all my quickfixes suddenly broke all at the same time.


like i said, i'm not sure of that. because the script detect my build machine systemd and not the the toolchain one (which doesn't exist)
We can add it but do an investigation about that detection


Great!


Wow! That is a lot of variables.  I hope this is a great gain.


huhu ;) You *should* pass the torch :p


We removed `es6`, because it doesn't make sense to reference the version so it was just `es`. Changed it to `esm` because it really stands for _EcmaScript Module_, which is different to Script, and makes it a distinct abbreviation as well.


Yep, agreed.

I noted something like `/about/glossary` or `/about/concepts` on #61 which covers this. Would be very useful.


Awesome :)


Or if you want to return null an validation can be added in execute methdod after
                    "if(!messageByTypes.isEmpty()) {
                        lastMessageBody = new LinkedList<>(messageByTypes.values()).getLast();
                    }"
add
                    "if(lastMessageBody == null) {
                        lastMessageBody = "";
                    }"


@clbarras @johannpoignant
I just updated the Python client to version 0.6.
See CHANGELOG.md for more information.


here's the problem, if you call `python3 -m coverage` then subprocess will not throw an exception because the `python3` executable exists although it will then complain that `coverage` module doesnt, so if we use `python3 -m` here we'll need another way to catch if the module does exist.


Here


This is not the best approach, but otherwise need RedisTemplate instead of RedisConnectionFactory.


:muscle:


I'm wondering if we need this bit. SSL3_RANDOM_SIZE is in the public headers already, so can we not just require that |out| has at least SSL3_RANDOM_SIZE bytes in it? Similarly for SSL_get_server_random()


Oops, sorry about that


Improved in https://github.com/pepyatka/pepyatka-server/commit/8d9a5ce


Textures overlay above others. You have an edge texture that represents the edge.


Thoughts about this? :thought_balloon:


I like `glossary.md` it doesn't require in-depth desciption (as `concepts`), but more general then `terms.md`


mapはキー検索で登録しているオブジェクトを呼ぶために用意しました
Listは登録したオブジェクトを回すために用意しました、消すときVectorより軽いと聞いたので
mapだけでどうにか登録したオブジェクトを回す方法を探したのですが
見つからずmapとlistを使用しています
listは検索する予定はありません

unordered_mapはキーをstringで登録してもmapより軽いということでしょうか。ω。？


@post渡すの分からなかった


fixed it.


It's not exactly breaking. This adds native support for browserify and AMD. You should be able to `npm install webfontloader` and use it your code:

```
var WebFont = require('webfontloader');

WebFont.load(...);
```

This previously wasn't supported, so any custom step you might be using to use the web font loader with browserify might be broken. I hope this will simplify using the web font loader with browserify and require.js.


TYPO3 6.x doesn't have that 'M' parameter, so how about changing this to this in order to support this on 6.2.x?
```php
$module = GeneralUtility::_GET('M') ? GeneralUtility::_GET('M') : 'web_layout';
```


@goldhoorn Did you have pkgconfig for TurboJPEG?


@benjaminfaustino please review


actually i think's ok :p

there is no chance for a statement to appear both in statement and concurrent_statement, and this is the correct way to write it


Great, thanks!


@goldhoorn can you change this if you have the libusb pkgconfig?


@rsafonseca

We running several Marvin tests (component/smoke) with KVm before sending PRs. We execute test_routers, test_vm_life_cycle, test_private_gw, test_service_offerings, test_vpc_routers, etc. It working fine,

Someone made the same comment before, but unfortunately we cannot reproduce it as it's just working. Below screenshot from VM that is under tests now:

![image](https://cloud.githubusercontent.com/assets/5129209/7808846/5e69c7f4-0396-11e5-8855-291bb4855d3c.png)

Could you please tell me what test environment you have and the steps you used to reproduce it?

Cheers,
Wilder


The normal process is making another commit to update the pull request. And then squash the commits into one.
Why does it require you to force push? Is it because you do not have the latest changes from upstream?


Corrected even though, some uses 4 spaces other uses 2 :)


`search` vs. `find` in order to be consistent with `jsonpath-rw`?


Hi @ryanjduffy
Unfortunately, it breaks layout of slider.
I need this line to use `moon-simple-picker-button` css class


@ktrzeciaknubisa I'm not sure if 'int' is sufficient to handle return value. Why don't you try STD_TO_NUMBER instead of STD_TO_INTEGER at 579 ?


I think either here, or in `relations.md` (`mapper.md` etc.) sections we should describe the difference between `rom.commands(:users)` and `rom.command.users` and so on. I was confused when looked at them first.


Which commit or pull request changed the pattern format?


Doesn't have to do with nested IFs? what setting or editor I'm using to make indention?


Great!


This happens in all my environments (Ubuntu, Fedora21, Centos6, Centos7)
All marvin tests run fine in all environments.
Upon doing an actual install and going through the setup wizard, issue occurs after initializing host and attempting to deploy systemvms.
The only failure that gets logged is in management server logs stating that response is not of type StartAnswer or something like that. It's actually of time UnsupportedAnswer, because it gets created as such due to any exception ocurring in the Wrapper's execution.


Done!
I force-pushed to my master branch. I can rebase it on yours if needed.


* unordered_mapは、keyがstring型の場合は、mapより高速です。
* listはvectorより速いのは、間違いです。
* まぁ、今のスペックではそんなに変わりません。
* でも、listとvectorの使い分けは調べましょう。
* mapでも、登録したオブジェクトも回せます。


http://ja.stackoverflow.com/questions/5997/vector%E3%81%A8list%E3%81%A9%E3%81%AE%E3%82%88%E3%81%86%E3%81%AB%E4%BD%BF%E3%81%84%E5%88%86%E3%81%91%E3%81%BE%E3%81%99%E3%81%8B

これを見るといいかも


where did i go?


This is where ANY exception gets thrown into an UnsupportedCommandAnswer, with no information about the actual exception being sent back to the agent:

LibvirtComputingResource.java:

    @Override
    public Answer executeRequest(final Command cmd) {

        final LibvirtRequestWrapper wrapper = LibvirtRequestWrapper.getInstance();
        try {
            return wrapper.execute(cmd, this);
        } catch (final Exception e) {
            return Answer.createUnsupportedCommandAnswer(cmd);
        }
    }


Okay po kuya carey.
Gang ngaun hindi ko pa din po mapagana ung localhost mail function. Hehe


Hi Marko,  it was  edd81d3;
I wrote it in #1959, but then forgot to refer in this PR, sorry :)


This is a comment.


Later in `mappers.md` and in commit to `relations.md` i noticed the alternative syntax. Then started to try it out using `pry` and found there are some ways to do it via `setup.relation(:users)`, or `ROM.relation(:users)`, or `ROM.boot.relation(:users)`. AFAIK there are just different ways to access `@boot` variable.

I think there should be some description of how to access this varialbe "conventionaly" between `ROM.setup` and `ROM.finalize`.


Comment to commit.


should read `don't resend request, failed to get cookie`


here

Alain Escaffre
Director of Product Management, Nuxeo
Tel: (+33) (0)6 29 37 04 51 - Twit. @aescaffre

2015-05-26 10:56 GMT+02:00 Guillaume Renard <notifications@github.com>:

> Here
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nuxeo/nuxeo-liveconnect/commit/ed31d09503d33640d08242ad17c7a38d300c7a15#commitcomment-11359543>
> .
>


type


We should log this failure


This is my first comment


Hello @igor0511. Thank you. I haven't tested this yet. My first question would be, do you really need to create an own ticket view for this module? Would it be possible to either a) use the existing small view or b) just have a small internal ticket list that is not configurable etc.? This seems a lot of code for a small feature.


saw approval from shino
at https://github.com/basho/riak_cs/commit/2f19b509cc6f0ec33795806b69ab91bf87d91ea9


merging basho/riak_cs/bugfix/gc-manager-batch-race = 2f19b509 into borshop-integration-1148-bugfix/gc-manager-batch-race


basho/riak_cs/bugfix/gc-manager-batch-race = 2f19b509 merged ok, testing candidate = 8044cd0b


@Tobias-Fischer  I changed it to remove a warning :) !
I don't care at all if we change it back, but warning on visual.

Concerning the editor I'll change it asap, but i was sure I had already spaces...


We should perhaps add something about the memory management of this function, i.e. callers should not free this memory. Probably should be added to the SSL_get_ciphers() description immediately above as well!


没发现异常最好了，应为看到只是说明游戏中显示了，但是代码中也有可能用这些作为一些关键变量的值来进行判断什么的。


@niligulmohar In it's original form that took in an Array, multidimensional-array meant a multiple bargraph see [0] for sample data input. I converted it to work with JSON such as [1] so I maintain the key loop. This is as to use the same bar graph for both single column bargraph and multi bar graph.

[0] https://github.com/Freeboard/freeboard/blob/62501c9004562e2cda143a6e8b20e741240d2162/plugins/freeboard/freeboard.widgets.js#L585
[1] http://localhost:8080/v0/d0ff50efbc53/modbus.elmatare_pump.CG_EM24.3.62/deltas/2015/Africa/Nairobi


所以保险起见我取消了，并且加上了“NoT=2”的标记（不知道是否有问题，为了保险起见暂时不汉化标记），将来只要翻译找到他们还是很容易的


Yes, as mentioned above, I am not a stranger to it.
I know that many consider it to be an improvement,
Personally, I do think the code readability goes down by far when doing it.
I know that it is in fact a performance enhancer to do it, however I think it´s barely measurable today :)
If we are to do it , we need it to be 100% correct via the MVC model that needs to call a pure template with  .html extension from our core files.
I do not think we should lightly mix file extensions with random code content.
My point is that we either do it 100% correct or not at all.
If 25% of the files is changed but the other parts remain unchanged it will only cause confusions.
The current standards is to not mix, so we should avoid it until changed overall.


typo


@gregoire-pointeau: Ahhh, that's cool with me if it removes a warning. I didn't get any with g++, that's why I wondered.

Regarding the tabs/spaces: in your recent commits it's definitely all tabs. Would be great if you have a look.

Best, Tobi


ack


LGTM


Small typo: You forgot to change the homepage to NL

Rest is perfect :+1:


can't say, needs testing :p


@floriank This breaks the generated JS syntax in the default case of the case statement above.


Sure, and that's the beauty of a public website such as Github, as the language doesn't have this visible, a comment does show.


If you're increasing the reload time, then isn't this a nerf, rather than a buff?


There is another thing codacy.


I'm not sure what you are hinting at here. Can you elaborate? :-)


ack.


Tu peux mettre les vrais icones au passage stp ?


bug not fixed:
plugin output length 1533:PROCESSLOAD OK - WorkingSet is 7.131e+04kB | Allowed_Audio_Editor_Licenses=99.0 Allowed_Carts_Player_Licenses=99.0 Allowed_Crash_Recorder_Licenses=99.0 Allowed_Ingest_Manager_Licenses=99.0 Allowed_Media_Logger_Licenses=99.0 Allowed_Media_Manager_Licenses=99.0 Allowed_Multi_Media_Editor_Licenses=99.0 Allowed_OnAir_MultiLayout_Licenses=99.0 Allowed_OnAir_Player_Licenses=99.0 Allowed_Quality_Control_Licenses=99.0 Allowed_Res17_Licenses=99.0 Allowed_Res31_Licenses=0.0 Availability=1;;1:;0;1 Average_Transaction_Duration=1.0 Client_Communication_Errors_per_Minute=0.0 Connected_Clients=4.0 ElapsedTime=699790.0s;;;0;9223372036854775807 HandleCount=352.0;;;0;16744434 PageFaultsPersec=0.0;;;0;2000000 Peak_Queue_Duration=0.0 Peak_Total_Call_Duration=1.0 Peak_Transaction_Duration_per_Minute=1.0 PercentPrivilegedTime=0.0%;@75:80;@80:;0;100 PercentProcessorTime=0.0%;@75:80;@80:;0;100 PercentUserTime=0.0%;@75:80;@80:;0;100 Queue_Duration=0.0 Queued_Calls=0.0 ThreadCount=17.0;;;0;3200 Total_Call_Duration=1.0 Transactions_per_Minute=1.0 Used_Audio_Editor_Licenses=1.0 Used_Carts_Player_Licenses=1.0 Used_Crash_Recorder_Licenses=1.0 Used_Ingest_Manager_Licenses=1.0 Used_Media_Logger_Licenses=1.0 Used_Media_Manager_Licenses=1.0 Used_Multi_Media_Editor_Licenses=1.0 Used_OnAir_MultiLayout_Licenses=1.0 Used_OnAir_Player_Licenses=1.0 Used_Quality_Control_Licenses=1.0 Used_Res17_Licenses=1.0 Used_Res31_Licenses=0.0 VirtualBytes=169592.0kB;@1500000:1700000;@1700000:;0;2000000 WorkingSet=71312.0kB;@1500000:1700000;@1700000:;0;2000000

I got only "header": PROCESSLOAD OK - WorkingSet is 7.123e+04kB
without metrics


:shipit:


@mhaskel Is it possible to get this added to the release list.


Are you sure you are allowed to use the _xmlDoc type ? Leading underscore usual mean private.


あざます！
制作中のゲームに反映できるかわからないけど見てきます。ω。


Error from me, my $_SESSION was wrong


Thanks, but first I need to integrate the changes.


как


I did manual tests as well.

Doing again now...

![image](https://cloud.githubusercontent.com/assets/5129209/7809305/75f5e738-0399-11e5-9a84-1505f5d13da2.png)

What is the version of you ACS agent on the KVM host?

Cheers,
Wilder


Super, jesteś w świetnym miejscu, ogólnie masz referencje do przedmiotów studenta, ale nigdzie nie dodajesz do niej przedmiotu ktory chcemy dodac


思ったけどこれそもそも`-`と`+`以外が来た時に通しちゃいますね。
書き直します／(^o^)＼


Faktycznie ze dokladnie to samo wczoraj pisalismy :)


__API Blueprint syntax error on line 658__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


done


This isn't KC705-specific (NB: I will likely merge your patch and this can be addressed with a second one)


Please, remove all project files from PR and leave only files which are related to test. It would be easier to follow what is going on


not applicable anymore


I'm not exceedingly familiar with the spec, but can't these magic numbers be named constants?


Why make `vstr` private and then friend x86 anyway? The whole class here is private to `x86`. I'd just keep everything public like with the rest of the inner classes in here.


I don't think it brings much.


This, on the other hand, is KC705-specific and should not be. I thought the artiq_flash.sh script was capable of storing that image now?


`@boot` is private, you shouldn't access it directly. See `ROM::Global` API docs which hopefully explain it :)


Yeah, fixed in https://github.com/spring-projects/spring-boot/commit/676b7d713a2886c00ea83be58ab11a98c2fa604f


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1171


so can I change the commit before I do the pull request?


I need to integrate the changes so I reviewed them. We're using the class that got removed (we're using composer as a library).


I have a local patch that adds this feature to artiq_flash.sh, but last time we discussed it I understood you didn't want it. Maybe I got it wrong, I can push it and adapt the documentation then!


Hab ich doch extra runtergeschraubt weil viel zu viel raus gekommen ist.


Great commit Geddy!


Why did you merge it in the first place?


__API Blueprint syntax error on line 2093__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


тут нужно исправить


удали блок


__API Blueprint syntax error on line 2118__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


все


Yeah. It would look like

def age
  @age
end

def age=(new_age)
  @age = new_age
end

On Mon, May 25, 2015 at 9:21 PM, Ian Schoonover <notifications@github.com>
wrote:

> Do you know how to write a getter and setter method without using
> attr_accessor?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jackorourke/ixperience/commit/3da357105b77720d44460b2a04a3825b7027824f#commitcomment-11353619>
> .
>


Oops, fixing.


done


__API Blueprint syntax error on line 2126__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


__API Blueprint syntax error on line 2127__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


__API Blueprint syntax error on line 2138__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


good


[![LGTM](http://lgtm.in/p/cSimZjhpq)](http://lgtm.in/i/cSimZjhpq)

_Activity.javaはcayennemodelerによる自動生成ファイルなので変更不要


Well, it is. 86b5f0c0532042b6d45579671139a1dc3f881685


Ow.... yeah using composer as a library isn't the best of ideas at the moment. I mean, it hasn't reached a stable release yet, so you have to be very careful with pulling in new changes.


The idle experiment would be a better example. See my post on the mailing list that explains how to compile and load it. (Again, this can be addressed with a second patch)


__API Blueprint syntax error on line 2162__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


Could you at least document that mkXmlDoc source source is either a document xml or a file ?


Storing token in HttpSession won't work well with default expiration period of 30 minutes.


Hi @rsafonseca

The snippet you pointed in the previous comment (i.e. try/catch) with the Exception being caught is not cool, right. However, did part of the code was not changed during the refactor. If you look at the history you will see that it has been catching Exception since the beginning. I'm not saying that it's right, but it cannot be pointed at as the root cause.

In my previous comment I showed that VMs can also been created manually (via the wizard). Could you please try to update your agent on the KVM host? You can download it from here: http://jenkins.buildacloud.org/job/package-centos7-master/ - that's the one I'm using.

Did you built the agent based on the latest code? If so, I will do the same and see how it goes.

Cheers,
Wilder


This is the only place in the library where these numbers will be found, and there is exactly zero chance they will ever change, so I don't really see the benefit of naming them...

A comment would be in good style, though, if only for documentation. It will be so in the next commit.


@guyharris
>  if you'd prefer them all to be global, or all to be local and passed to the routines that use them,
> we could probably do that.

Great, that would improve cleanliness in `main()`.

> Which options would you like to see removed?

None really. Just move `getopt()` processing to a new file (`parse_cmdline.[ch]`). And maybe make all options `const` for other files (`tcpdump.c`) including `parse_cmdline.h`. Something like:

In `parse_cmdline.c`:
```
  #define INSIDE_PARSE_CMDLINE
  #include "parse_cmdline.h"

  CONST netdissect_options ndo;
  CONST tcpdump_options    tdo;

  int parse_cmdline (int argc, char **argv) {

    while ((opt = getopt_long(...)) != -1)
      switch (opt) {
        ...
      }
  }
```
And in `parse_cmdline.h`:
```
  #ifdef INSIDE_PARSE_CMDLINE
    #define CONST
  #else
    #define CONST const
  #endif

  extern CONST netdissect_options ndo;
  extern CONST tcpdump_options    tdo;
```


PLEASE PLEASE PLEASE add header guards.


ログイン機能がないから、とりあえず、１を設定してる？
こうゆうコードは、なぜそうしているか、コメントを記載しておいてください。
読み手が考えそうなことを先回りして書いておくと、読みやすいコードになると思います。

TODO を入れておくとさらにベストです。
//TODO ログイン機能を追加することになったら、セッションからログインIDを取得する


Oh I see, yeah that would be a little cleaner. I'll probably clean this up but only for the 2.x version, as this is in mostly maintenance mode after 1.1 anyway.


これでredirectしてくれるの？
もしそうなら知らなかった。勉強になりました。


I really thing we should not do that. This header only shows up in the embedded case, i.e. the user is supposed to only see the specified process. This link would allow them to break out of that. I suggest to either have no link or to link to the process instead.


fast-forwarding develop to borshop-integration-1148-bugfix/gc-manager-batch-race = 8044cd0b


TeamCity xBIM :: XbimExchange :: Build [Build 295](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1691&buildTypeId=XBIM_XbimExchange_Build) is now running


TeamCity xBIM :: XbimExchange :: Build [Build 296](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1692&buildTypeId=XBIM_XbimExchange_Build) is now running


TeamCity xBIM :: XbimExchange :: Build [Build 295](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1691&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:02:04


Yes, either use your Git Client or edit it online here:
https://github.com/gmkey/homebrew-versions/blob/thunderbird-nl/Casks/thunderbird-nl.rb

Click the little edit icon, change the text. The Pull Request will automatically list your newest commit to the brach, so a merge would include it :)


全体的に言えることですが、フィールドは、privateにして、getter、setterを用紙しましょう。
getter、setterは自動生成できます。
eclipseのメニューで、Limyから行ってください。


useless comment.


Hi @wilderrodrigues
Everything is built on the latest code in my testing, so agent is 4.6.0-SNAPSHOT as well.

The issue happens going through the basic startup wizard and deploying system vms (ssvm and console proxy), try to reproduce using that :)

I just mentioned that uncool try/catch because it was preventing proper debugging whenever the issue occurs, this was just a heads up to help you debug the issue once you are able to reproduce it, not an indication of the root cause :)

I can also give it a try with the version you're referring to (not the lastest) to see if the issue was already present when that package got built, anyway, i can only do it later tonight, as my dayjob is gonna keep me busy the whole day today :)

Cheers,

RF


@akbertram would you please review this merge commit? It seems I've resolved conflicts in wrong way which leads to export acceptance tests failure.


Hi, when I compile this, it says:
hardware/samsung/ril/libril/ril.cpp:142:5: error: 'RIL_SOCKET_ID' does not name a type
     RIL_SOCKET_ID socket_id;
     ^
hardware/samsung/ril/libril/ril.cpp:153:5: error: 'RIL_SOCKET_ID' does not name a type
     RIL_SOCKET_ID socket_id;
     ^
hardware/samsung/ril/libril/ril.cpp:167:45: warning: 'rilSocketIdToString' initialized and declared 'extern' [enabled by default]
 extern "C" const char * rilSocketIdToString(RIL_SOCKET_ID socket_id);
                                             ^
hardware/samsung/ril/libril/ril.cpp:167:45: error: 'RIL_SOCKET_ID' was not declared in this scope
hardware/samsung/ril/libril/ril.cpp:170:11: error: 'MAX_SOCKET_NAME_LENGTH' was not declared in this scope
 char rild[MAX_SOCKET_NAME_LENGTH] = SOCKET_NAME_RIL;
           ^
hardware/samsung/ril/libril/ril.cpp:254:37: error: 'RIL_SOCKET_ID' has not been declared
 static int sendResponse (Parcel &p, RIL_SOCKET_ID socket_id);
                                     ^
hardware/samsung/ril/libril/ril.cpp:320:34: error: 'RIL_SsServiceType' was not declared in this scope
 static bool isServiceTypeCfQuery(RIL_SsServiceType serType, RIL_SsRequestType reqType);

how to solve it?


TeamCity xBIM :: XbimExchange :: Build [Build 296](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1692&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:39


TeamCity xBIM :: XbimExchange :: Build [Build 297](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1693&buildTypeId=XBIM_XbimExchange_Build) is now running


//Not allocate


is there a bug without this condition? Doesn't seem necessary since graveyard activation is only possible if card is in the graveyard. Just as card activation is only available when card is in hand.


TeamCity xBIM :: XbimExchange :: Build [Build 298](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1694&buildTypeId=XBIM_XbimExchange_Build) is now running


TeamCity xBIM :: XbimExchange :: Build [Build 297](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1693&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:25


done


Oh, yes, thats what i mean, not `override` but `overlay`. Because this code is SO compact so i didn't understand it at first.

Totally steeling it and converting to Scala =P


.gitignoreを編集して、JSPからjavaに変換されたファイルを除外してコミットしてください。
既にコミットしたファイルは、git rm で削除する必要があるので削除してください。
ググって調べてみてください。


As far as I can see, `<iostream>` is only used if `SYNCOPE_DETECT_DEADLOCKS` is defined. It would be a good idea to only #include it in that case as well, as it would reduce namespace pollution.


> Method that checks that the xml document has been correctly *parsed*.
then should the method be called `isParsed`?


Fixed: https://github.com/combak/ShatteredWorld/issues/58


@adaikalaraj This violates hierarchy abstraction. Ideally the pager shouldn't call any activity apis. It is the job of the fragment. The pager should raise an exception for 403 forbidden and ExamsListFragment should call logout. Do not pass the activity handle to ExamPager. Catch the exception raised by the pager in the fragment. The pager should raise exception to inform the above caller about 403.


TeamCity xBIM :: XbimExchange :: Build [Build 298](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1694&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:40


TeamCity xBIM :: XbimExchange :: Build [Build 299](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1695&buildTypeId=XBIM_XbimExchange_Build) is now running


It would be nice for people to be able to go back home


Or just use NOVA and get it for free! block.add(new ConnectedTextureRenderer)


Hi @rsafonseca

So you built the Agent. I will do the same and give it a try.

 Concerning the version from jenkins.buildacloud, I will check with @DaanHoogland why it's not building the latest code. We should be able to get latest snapshot from there, but looking at the date I realised last one was built on 11th May, which is pretty old.

The try/catch can be dealt with afterwards. Let's try to reproduce this problem first and get it fixed.

Cheers,
Wilder


should be

    } else {
        setAnswer(item, question);
    }


TeamCity xBIM :: XbimExchange :: Build [Build 299](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1695&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:37


TeamCity xBIM :: XbimExchange :: Build [Build 300](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1696&buildTypeId=XBIM_XbimExchange_Build) is now running


こじまです


findByIｄは、各サービスクラスで定義されており、冗長なため、AbstractServiceクラスに集約しましょう。
ジェネリクスをうまく使えば、集約できます！


hi. sorry..
I'm find your git and have interest to use this for build working rom with vc4 driver - may be you help me?

My english bad...((


I do not understand why we need this. It seems as a hard coded thing that works for our tests. This should be covered by the directembedding + preprocessing for slick.


We're considering that still.


TeamCity xBIM :: XbimExchange :: Build [Build 301](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1697&buildTypeId=XBIM_XbimExchange_Build) is now running


TeamCity xBIM :: XbimExchange :: Build [Build 300](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1696&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:35


Results in "Ignoring name shell2.example.org, not in weave.local." when I try it.


Where is `$shell2` supposed to come from?  There is a command earlier that sets up `$shell1`, but not `$shell2`.


ik denk dat je bij unfavorite lesson wilt kijken of er in de database geen entries meer zijn waarbij het lesson_id die van de net verwijderde les is


but it used to be in the global scope (after require('webfontloader')), and now it isn't? you don't consider that a breaking change?


`xurls *.txt` will be extended (via globbing) by the shell. The program receives the list of files matching the glob pattern already, so if we ever receive `*.txt` as an argument it's because the user definitely wants to read `*.txt`, e.g. they executed `xurls \*.txt'.


done


Yeah, backward compatibility is hard! But no worries, if that's the price
to pay to have a language that evolve I'm happy to pay it :-)

Cheers

On 22 May 2015 at 00:53, Andrew Cowie <notifications@github.com> wrote:

> It's frustrating to have to do this sort of thing. We want the platform to
> evolve, but in doing so we create this "dammed if you do, dammed if you
> don't" situation. Thanks for taking care of this!
>
> AfC
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/aloiscochard/machines-io/commit/8630b5b7bcaa9af32fc11db137d4daa501356805#commitcomment-11317961>
> .
>

--
*Λ\ois*
http://twitter.com/aloiscochard
http://github.com/aloiscochard


Can we guarantee that the functions with `@preserveInvocation` will typecheck after the translation?


TeamCity xBIM :: XbimExchange :: Build [Build 301](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1697&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:28


TeamCity xBIM :: XbimExchange :: Build [Build 302](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1698&buildTypeId=XBIM_XbimExchange_Build) is now running


But "home" in this context would be the process, not `/`. I also remember talking to the designers about this and they said there should not be a link. Could you please check with them again?


@redcamel smith 감사합니당 역시 고수의 손길로.....살아났네요..


@littleguy77 should we remove this? From what I saw on paulscode.com it seems like @gonetz removed the boost dependency.


It's an interesting question. The key can appear syntactically valid to the client but fail when it gets used on the server. Should I test the key against the server on connection? Otherwise the client won't know the key is broken until they do something with it.


I'd personally just make this `unique_ptr<Owner[]>` and follow the Rule of Zero. It seems that in the code as is currently there isn't a problem with failing to follow it, but it's a simple change that is future-proof.


This doesn't quite follow from the first sentence, which establishes that we want to use a name other than the hostname.  Maybe "You can also use dns-add to add the container's configured hostname and domain, simply by omitting `-h fqdn`"

(also I found it ambiguous whether I am supposed to give `-h` and omit the fqdn, or omit both)


Why do we need this?


What is this for?


Hi Dan,

Seems like this commit turned our site into returning "true". All brackets are closed and all commas are there. You see anything funky?


Missing comma. Making a new commit.


What about the other roles in this array?

How come this happens when the "Rebuild automatically" is not checked?


TeamCity xBIM :: XbimExchange :: Build [Build 302](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1698&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:02:45


TeamCity xBIM :: XbimExchange :: Build [Build 303](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1699&buildTypeId=XBIM_XbimExchange_Build) is now running


@jgoppert Think there should be a visionInitCount here.


It's how it has been done since the beginning. It is ok to use them even though it's true that there are typedefs for them in libxml. If you don't like this, please submit a patch (not the purpose of my pull req).


build with I9300


Параметризовані тести - супер!

Робота з БД мені зовсім не по душі. Я маю на увазі DBUnit. Довго його ковиряв і знайшов дещо інше як альтернативу.

Є новіша і елегантніша штука DbSetup. Там непотрібно створювати xml файли, всі операції проходять буквально в пару строк. Набагато зрозуміліше для людей. На тестову роботу з БД можна подивитися в моєму бранчі.

Але то як шановна громада вирішить :)

З тим щоб ділити так проект(принаймні main) я не згоден категорично.

Дуже круто робить Оля Ніцович, вписуючи пейджі в функціональні пекеджи залежно від їх "принадлєжності".

Вона якраз і робить так, як ми всі задумали з самого початку. На мою думку це і є мікс Page+Functional.


TeamCity xBIM :: XbimExchange :: Build [Build 303](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1699&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:02:09


TeamCity xBIM :: XbimExchange :: Build [Build 304](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1700&buildTypeId=XBIM_XbimExchange_Build) is now running


I have started some stuff too, project skeleton and quickly CoAP client integration but we could collaborate each other ;-)


Why don't put this in this file: https://github.com/scaleway/image-app-torrents/blob/master/patches/etc/php5/fpm/conf.d/50-scaleway.ini ?


TeamCity xBIM :: XbimExchange :: Build [Build 305](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1701&buildTypeId=XBIM_XbimExchange_Build) is now running


TeamCity xBIM :: XbimExchange :: Build [Build 304](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1700&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:25


TeamCity xBIM :: XbimExchange :: Build [Build 306](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1702&buildTypeId=XBIM_XbimExchange_Build) is now running


TeamCity xBIM :: XbimExchange :: Build [Build 305](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1701&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:16


I prefer to keep double line separators between sections


TeamCity xBIM :: XbimExchange :: Build [Build 307](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1703&buildTypeId=XBIM_XbimExchange_Build) is now running


TeamCity xBIM :: XbimExchange :: Build [Build 306](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1702&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:16


If we can be sure that `@preserveInvocation` will work I am good with this approach. I just do not know how to be sure about that.


TeamCity xBIM :: XbimExchange :: Build [Build 307](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1703&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:13


TeamCity xBIM :: XbimExchange :: Build [Build 308](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1704&buildTypeId=XBIM_XbimExchange_Build) is now running


TeamCity xBIM :: XbimExchange :: Build [Build 308](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1704&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:17


TeamCity xBIM :: XbimExchange :: Build [Build 309](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1705&buildTypeId=XBIM_XbimExchange_Build) is now running


thanks! you're a great help. I'm also trying a new pull request for the remote-desktop-manager package that included (not by design) my original translation effort...


The idea was that if the message publishes ok then no exceptions were thrown and the connection works. Is there something else I can test for?


TeamCity xBIM :: XbimExchange :: Build [Build 310](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1706&buildTypeId=XBIM_XbimExchange_Build) is now running


TeamCity xBIM :: XbimExchange :: Build [Build 309](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1705&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:20


TeamCity xBIM :: XbimExchange :: Build [Build 310](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1706&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:15


TeamCity xBIM :: XbimExchange :: Build [Build 311](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1707&buildTypeId=XBIM_XbimExchange_Build) is now running


Double checked my sanity, not a missing comma.


TeamCity xBIM :: XbimExchange :: Build [Build 312](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1708&buildTypeId=XBIM_XbimExchange_Build) is now running


TeamCity xBIM :: XbimExchange :: Build [Build 311](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1707&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:15


TeamCity xBIM :: XbimExchange :: Build [Build 313](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1709&buildTypeId=XBIM_XbimExchange_Build) is now running


TeamCity xBIM :: XbimExchange :: Build [Build 312](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1708&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:16


TeamCity xBIM :: XbimExchange :: Build [Build 313](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1709&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:15


TeamCity xBIM :: XbimExchange :: Build [Build 314](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1710&buildTypeId=XBIM_XbimExchange_Build) is now running


if codacy shows up it will do comments anyway, you think its better to mention it explicitly?


the mtu could be a different value than 9000, but still have jumbo frames. I suggest to use != 1500 (or > 1500)


TeamCity xBIM :: XbimExchange :: Build [Build 314](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1710&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:14


TeamCity xBIM :: XbimExchange :: Build [Build 315](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1711&buildTypeId=XBIM_XbimExchange_Build) is now running


TeamCity xBIM :: XbimExchange :: Build [Build 315](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1711&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:14


TeamCity xBIM :: XbimExchange :: Build [Build 316](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1712&buildTypeId=XBIM_XbimExchange_Build) is now running


TeamCity xBIM :: XbimExchange :: Build [Build 317](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1713&buildTypeId=XBIM_XbimExchange_Build) is now running


TeamCity xBIM :: XbimExchange :: Build [Build 316](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1712&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:16


TeamCity xBIM :: XbimExchange :: Build [Build 318](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1714&buildTypeId=XBIM_XbimExchange_Build) is now running


TeamCity xBIM :: XbimExchange :: Build [Build 317](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1713&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:19


@leo here is acceptance test for AI-1041. Would you please correct what is wrong here?


TeamCity xBIM :: XbimExchange :: Build [Build 318](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1714&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:14


TeamCity xBIM :: XbimExchange :: Build [Build 319](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1715&buildTypeId=XBIM_XbimExchange_Build) is now running


make it default to 1500 ?


done


TeamCity xBIM :: XbimExchange :: Build [Build 319](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1715&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:19


TeamCity xBIM :: XbimExchange :: Build [Build 320](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1716&buildTypeId=XBIM_XbimExchange_Build) is now running


That's how it was called before.


do you guys still use this?


merge


Hmm, it's a tough call. The Web Font Loader never officially supported browserify or require.js, and the existing interface has not changed. When I added the functionality I did not consider people were already using the Web Font Loader with browserify, so from my point of view it wasn't a breaking change. I can see how it is a breaking change from your point of view.

Do you feel strongly about rolling back this release and doing a 2.x?


TeamCity xBIM :: XbimExchange :: Build [Build 320](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1716&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:19


TeamCity xBIM :: XbimExchange :: Build [Build 321](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1717&buildTypeId=XBIM_XbimExchange_Build) is now running


@yuriyz the issue here is specifically with dates and your test includes no dates!


and iface["mtu"] != "1500"


TeamCity xBIM :: XbimExchange :: Build [Build 321](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1717&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:18


TeamCity xBIM :: XbimExchange :: Build [Build 322](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1718&buildTypeId=XBIM_XbimExchange_Build) is now running


Occassionally.
________________________________
From: Ben Brunton [notifications@github.com]
Sent: 26 May 2015 10:58
To: coulson84/dirtylog
Cc: Joe Coulson
Subject: Re: [dirtylog] Removing google analytics code (5613566)

do you guys still use this?

—
Reply to this email directly or view it on GitHub<https://github.com/coulson84/dirtylog/commit/5613566d3b3e84eaf275c20fa3682885b9e2fdcb#commitcomment-11360596>.


TeamCity xBIM :: XbimExchange :: Build [Build 323](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1719&buildTypeId=XBIM_XbimExchange_Build) is now running


TeamCity xBIM :: XbimExchange :: Build [Build 322](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1718&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:21


TeamCity xBIM :: XbimExchange :: Build [Build 323](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1719&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:15


TeamCity xBIM :: XbimExchange :: Build [Build 324](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1720&buildTypeId=XBIM_XbimExchange_Build) is now running


@akbertram It do include the dates for AI-1082


TeamCity xBIM :: XbimExchange :: Build [Build 324](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1720&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:18


TeamCity xBIM :: XbimExchange :: Build [Build 325](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1721&buildTypeId=XBIM_XbimExchange_Build) is now running


TeamCity xBIM :: XbimExchange :: Build [Build 326](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1722&buildTypeId=XBIM_XbimExchange_Build) is now running


TeamCity xBIM :: XbimExchange :: Build [Build 325](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1721&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:17


TeamCity xBIM :: XbimExchange :: Build [Build 326](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1722&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:16


TeamCity xBIM :: XbimExchange :: Build [Build 327](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1723&buildTypeId=XBIM_XbimExchange_Build) is now running


(if i recall correctly, before slicing the lock is also used only for retrieval)


LOL, I was pushing two identical commits.


please put this under the catch in a 'finally' block! :)