good point :)


my french.txt if you are interested to add it to your VitaShellCustomization. Tested OK.
Anyway, very good job! Keep going  ;)

[french.txt](https://github.com/TheOfficialFloW/VitaShell/files/444546/french.txt)


Still failed with this patch.

It seem's that the result get from cache is a basestring in my py2.7 env, no matter whether your put it as unicode or not, into the cache.

I noticed you've switch to ``PyLibMCCache`` , is that the course of the problem? I have to stay with ``MemcachedCache`` because it's almost impossible to install ``pylibmc`` on Windows.


Your method should be
boolean deleteUser(User user) or void deleteUser(User user) throw YourException


Select 4th icon(difference chart) from list of charts.
**Flow:**
Injects `<bar-chart>` into `<polymer-d3>`
Executes callback method `setDiffrenceSettings`, which lives inside `<bar-chart>`
Callback sets `this.configurator` as `PolymerD3.barChart.difference`, which returns settings and draw.
Inside draw, `conf` is set as `this.configurator.conf()`
Translations are set as `this.configurator.transformations()`


Strange I can edit your comments Dan...


@masaanli we've just release alpha.3 with the modal service: https://ng-bootstrap.github.io/#/components/modal


gpl code cannot be added to mpl. needs your explicit licensing of this fils as mpl.

On August 30, 2016 11:04:07 AM GMT+01:00, jaredo <notifications@github.com> wrote:
>Hello!  The `RandomSVD.hpp` code was released under GPLv3 in one of our
>[software packages](https://github.com/Illumina/akt). I am not sure if
>GPL code can be added to MPL codebases (although the converse seems
>true). Do you know?
>
>
>
>
>
>--
>You are receiving this because you commented.
>Reply to this email directly or view it on GitHub:
>https://github.com/jaredo/jeigen/commit/88df1ef86adc2f4323c0fd970f84ceb8d1f79d41#commitcomment-18827075

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.


I'm sure this was not happening in the past. I assume this commit is really in your repo, dan?


TelNumber addTelNumb(TelNumber telNumber)


We need 3 namenode units because we need to have 3 journal nodes. You are right about the interface being off. I do not see this upper limit being enforced. We should update the README of the interface.


This will return true if any value in an array ends/starts with a string value?


Your data layer method shouldn't use in.nextLine();


Curiously these messages are not @ https://github.com/danpoltawski/moodle/commit/cf36c893dd84c2c1e2a62afbed13341b43b8e210 only here, under moodle/moodle.


I get them all too. I think everyone who has 'watched' the gihub.com/moodle/moodle repository will get them.


Is performance important here? I think we can do this without locking (on this level) if we go with a ConcurrentMap.


:thumbsup:


Now you can remove the threading :P


@bjansen I think it's a better practice to explicitly write:

```ceylon
shared actual Document commonDocument {
     "should never be called"
     assert (false);
}
```


Det ser fint nok ud :)


Enum PhoneType{
MOBILE,
HOME,
FAX

}

class Address{
private String phoneNumber;
private PhoneType phoneType;

.....
}


Correction: this turns out to be one of those bugs where bisection doesn't help 😢. By doing seemingly random changes to the code (commenting certain parts in or out), I can get the test to work or to fail. It even seems to depends on the exact circumstances in which I build the library. I can't say for sure anymore that the bug didn't exist before 3.0.0. I've spend yesterday all day to chase the bug but I couldn't find it. The only thing I know with relative certainty is that it only happens when called from Java.


Haven't really thought about this I must confess, but I thought some NativeString abstract should be possible here as well. I'm just not  big fan of having the implemantation in the generator or in the Boot class.


nice!


You can have multiple logged in user. So you can keep
Set<User> loggedUsers;


I get them :)

On Tue, Aug 30, 2016 at 11:13 AM, Richard Oelmann <notifications@github.com>
wrote:

> I get them all too. I think everyone who has 'watched' the
> gihub.com/moodle/moodle repository will get them.
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/moodle/moodle/commit/cf36c893dd84c2c1e2a62afbed13341b43b8e210#commitcomment-18827186>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ACB8vjcGzV9FbXHKHA-dhmTCyjd2C5v_ks5qlAI7gaJpZM4JwWzp>
> .
>

--

[image: Moodle] <http://www.moodle.com>
Mary Cooch
Community Educator
e: mary@moodle.com
p: +61 8 9467 4167 w: moodle.com <http://www.moodle.com>
[image: Facebook] <https://www.facebook.com/moodle.lms/>
[image: Twitter] <https://twitter.com/moodle>
[image: Google+] <https://plus.google.com/+moodle/posts>


"List<User> friends;"


Right, I agree this is a better idiom.


1. should keep interface for data layer class.
2. class name isn't good.


It seems to fix the issue completely! LGTM :tulip:


Okay thanks for the details. If you are confident in this diagnosis, please feel free to send me a pull request that makes this check not happen on Linux. Just be sure to confirm it works with automount, and add comments explaining why we don't do it on Linux.


i'm not a fan of those things either.
Unfortunately i failed to solve the constructor issue. Which makes it impossible to define normal class for `String`
```haxe
class String {
   public function new (s:String)  {
        this = s; //Haxe compiler error: cannot assign this
   }
}
```
Do you have any ideas?


[çorlu yurt](akademipansiyon.net) - [çorlu erkek öğrenci yurdu](http://www.akademipansiyon.net/2015/09/corlu-erkek-ogrenci-yurdu.html)

çorlu merkezde akademi erkek öğrenci yurdu +90 282 651 81 82


[Çorlu Yurt.pdf](https://github.com/akademi59/corlu/files/444570/Corlu.Yurt.pdf)


The parser is an external library so it either successfully loads the JSON or it doesn't, so it's not part of the app. If you have markup problems in your config I suggest running it through an external [JSON validator](http://jsonlint.com) to help figure out the issue.


Started work on channels, updated quest status enum, you can now see quests correctly in minimap & in world.


@im85288 @iainsmacleod
I've fixed a lot of the playback issues. There was an out-of-order problem with the thread that runs on playback. The currentChannel was being deleted when the first thread stopped after another channel started playing.
The epg visibility and current channel is now set when Stop is pressed.
This might mean that something needs to be done about streams that just stop unexpectedly.

Until everything is stable with playback the thread that controls the Up Next feature is disabled.

If you want to help track down the problem you can uncomment some of the XXXXXX log print statements to see what is going on.


should has interface


Aqui o nome do método deveria ser `findByGroup`


` public User create(String username, String password) ->  public User create(User user)`


Você alterou o método `publc Date getCreatedAt() {}` acima mas nesse você não removeu o `_`.


@CoderMJLee 真牛逼


Pansiyonumuz çorlunun merkezinde belediye binasına yakın konumdadır..
namık kemal üniversitesi mühendislik fakültesi belediye otobüsüyle  5dk.
namık kemal üniversitesi meslek yüksek okulu belediye otobüsüyle 10 dk.
 çevresinde bankalar, ptt,otobüs durağı, camii, alışveriş merkezi,
eczane, sağlık ocağı vb. bulunmaktadır..

ısınma sistemi doğalgaz
her katta güvenlik kamerası
24 saat sıcak su
wireless

1-2-3-4 KİŞİLİK ODALAR

ERKEN REZERVAZYON AVANTAJLARINDAN YARARLANIN..
Cemaliye Mah. DR. Hakkı Görenli Sokak No: 6/1 Çorlu / TEKİRDAĞ

[çorlu yurt](http://www.akademipansiyon.net/2016/08/corlu-yurt.html)


I felt this approach was "good enough" - but it's not quite as accurate as the referrer check since you could change the expiration and this would still potentially pass. Regardless - it's better than nothing - and about the best we can do without hacking into the driver / injecting nasty spies. 👍


Acho que podemos pensar em criar uma tabela `group` que possui várias actions e o usuário passa a ter vários grupos, o que acham?


```
private List<String> phoneNumbers; -> private List<PhoneNumber> phoneNumbers;
private List<String> friends; -> private List<User> friends;
```


main now greets user with "hello " + last name


Quando alterar o nome do método no repository, não esquecer de alterar aqui.


@dirk This fixed the deprecation warnings?


@viniciuslopeslps Não esquecer de usar aquele parser de JSON que vc comentou quando for implementar pra web.


here also should have type
private AddressType addressType;
Enum AddressType{
Mobile,
Home,
Fax
}


Boilerplate.sh - To give a default template


AddFriend -> addFriend


已修复


test


I'd recommend not to do mkdir inside npm start scripts, instead create the directories, and commit the following `.gitignore` inside them:

````
# Ignore everything in this directory
*
````

(code comment)


この bugfix はどのような bug だったのでしょうか？
元のままでよさそうに見えるのですが…。


Hey man, did you intend to use GPLv3 on this project? I would have thought MIT or Apache would've been better?


Good Job Man Thanks!!!


(I'm going to file a bug with github, subscribe to https://tracker.moodle.org/browse/MDLSITE-4732 if you are interested). Locking this thread now.


ah, yes, thanks!


I've sent a mail to them:

> Hi,
>
> I'm an admin/owner of the moodle/moodle repository.
>
> Since some hours ago we (owners, watchers...) are receiving notifications (both mail and web) from any clone of the repo.
>
> So, for example, this commit performed in a clone: https://github.com/danpoltawski/moodle/commit/cf36c893dd84c2c1e2a62afbed13341b43b8e210#commitcomment-18827133
>
> Is seen, commented and notified to everywhere here (and the commit is not available in that repo at all):
>
> https://github.com/moodle/moodle/commit/cf36c893dd84c2c1e2a62afbed13341b43b8e210#commitcomment-18827133
>
> I'm not aware of this happening ever, maybe I'm wrong... so just asking and sharing for your consideration, in case it's some regression your side.
>
> TIA and ciao :-)

Let's see, maybe it has been that way since ever, although I've not noticed it till today (the "ability" to comment on any fork commit from master).

In the mean time, let's stop here because we are spamming a good number of people, sorry all!


```
boolean addUser(User user); -> User addUser(User user); Your returned user should also contain generated id
void editUser(User user); -> User edtUser(User user);
void deleteUser(User user); -> boolean deleteUser(User user); or void deleteUser(User user) throws Exception;
```


```
private ArrayList<String> phones; -> private List<PhoneAddress> phones;
private ArrayList<String> friends; -> private List<User> friends;
```


Changing this to `display: none;` hides the modal


Can you set a package to a specific version in requirements? If you note, we're using regex 2015.11.22 for CI because it has a tendency to break on us.


Here will be good if you will use for loop. You can keep all your commands to collection.


Oups, totally right , NEVER translate DEV message. sorry and thanks


Why can 'append only' event-store  have any deadlocks?


I checked on my laptop and the std:abs gives the same results of normal abs so it is ok.


Thanks :)


We need synchronization across two maps here:

1. the local `storedValues`
1. the `storedValues` in `parentStore`

So I don't see how `ConcurrentHashMap` will assist us with that.


alternative: do the mkdir inside `postinstall` script.


Great ! Thank you (for other factor out too )


ここは別スレッドで実行されるため、メインスレッドのStateが読み込めないのだと思います。
下記にあるように、キューを作る方法が良いと思います
https://github.com/stssg526/lis/blob/lisver2/unity-sample-environment/Assets/Scripts/AIClient.cs


Please see my comments above regarding synchronization across _at least_ two maps.


あー。これはなんでしたかね。。ちょっと見なおすのでお時間ください。


Hey, would you guys be interested in sending a pr with this change?
It makes sense and I'm happy to merge it with only few changes.


I compiled this commit and although I have not captured any new Mereti ship since then, I have noticed that none of the Mereti ships that I had previously captured have this AI chip.
Shouldn't this commit authomatically install reasoning nodes in all the Mereti ships that the players have in their fleets?


Тут можно было бы функционально через stream api написать, и думаю лучше бросать exception вместо null


not printable, but `quote_char => "^?"` or DEL char 0x7F


Pas top les IP en dur... `"ws://" + document.location.hostname + ":8998"`


Startup screen mockup


Blog comment mockup


ce fichier étant dans votre _frontend_, il ne sera au final pas caché. Il n'est pas nécessaire de vous donner tant de mal :-)


:tada:


SHouldn't it be == instead of = ?


Надо бы еще проверить, что в массиве 2 элемента, а еще можно проверить, что они разные.


Visual Regression Testing found a difference between new version and the baseline.
Check the diffs [here](http://jenkins.wikia-prod:8080/job/design-system-reference-page-pr-check/305/).
Accept the new baseline [here](http://jenkins.wikia-prod:8080/job/design-system-reference-page-update-baseline/parambuild?branch=XW-1620).


See #263


Не сравнивай  c nil,  если выражение вернуло nil, то это воспринимается как false, а значит, можно заменить все на  `unless REGEXP_STATION_NAME.match( @station)`


Валидация должна быть ДО того, как ты добавляешь объект в хеш, иначе, там будут невалидные объекты


Опять же убери сравнение с nil и замени if на unless


@Nadahar You give me doubt, so i checked the GUI and read "CPU threads", so in your case 6x2, if i understood correctly the algorithm 8-/
I was mistaken by the explanation given "Choose the number of processor cores that should be used for transcoding" in fact it's the threads number that is displayed. So perhaps we should reformulate that.


Also add code to
-read your transition state and product coordinates from PDBs.
- read the indices of the protein and substrate atoms involved in the CV from PDBs

Create also a brahan-utilities github project on michelab where you can upload your scripts and input files used to test that your code is working.


Fixes #2


Done


What about making String an abstract? I know it's against coreApi but for PHP it might make sense.


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 2.0.152](http://build.clear-measure.com/viewLog.html?buildId=10301&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) is now running


Ok, I'll switch it.


I did not do this on purpose, because:
- Like this all Oszis have to have the same commands / same order of commands
- Generally I prefer _only_ changing what has to be changed in a script otherwise:
    - It can be dangerous when using Sourcemeters, Power supplies, etc. One should as few commands as possible.
    - Here one nevertheless has to check the Oszi at least once if the Signal is ok, so no real benefit
    - It is more error prone, when settings are e.g. ignored by the Oszi (invalid range, etc.)
- Lab devices have buggy communication you do not want to talk more  to it than needed
- 20 % more loc


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 2.0.152](http://build.clear-measure.com/viewLog.html?buildId=10301&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 72, ignored: 2 Build time: 00:02:07


If no problem, then less work (docs, UI, translations) changing it, so I vote to keep it, since its not incorrect.


Might be useful to get in the habit of doing pull requests rather than straight commit? Makes code review easier?


Haxe compiler relies on String being a class. So do users' macros.


@pkozlowski-opensource Thank you, gonna implementate it! And maybe giving back some suggestions or thing where I'm stuggling about.


Hi,

I think I get multiple copies of the emails – can you remove me as well? Any email that has “paulo” in it.

Thanks

Paulo

From: Seth Tisue [mailto:notifications@github.com]
Sent: Monday, August 29, 2016 14:44
To: NetLogo/NetLogo <NetLogo@noreply.github.com>
Subject: Re: [NetLogo/NetLogo] Update about, versions, faq (c7ec2f5)

It was nice to be kept on the list for a while (I think Uri suggested it), but I am now officially requesting that you remove me.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub <https://github.com/NetLogo/NetLogo/commit/c7ec2f570bd0adecc49a7d8db9de87a943dd4ad0#commitcomment-18820469> , or mute the thread <https://github.com/notifications/unsubscribe-auth/AB6dHRNLVk06j8VKCATtXPS8LQywaApVks5qk1KJgaJpZM4Jv6Fd> .  <https://github.com/notifications/beacon/AB6dHcFLCkRbLpeBNdPrzZMvD3hlWKthks5qk1KJgaJpZM4Jv6Fd.gif>


Fixes #21


That's redundant with the previous bullet. The `ls` description should be merged into the previous bullet, or all three extensions should get a bullet with a short description.


Should be fine until Pattama is ready to merge her code.
I would be good to figure out how to create an issues tab for michel lab plumed2 code changes. I don't know why this issue is lacking in the current interface.


To me it looks like the call to `putStoredValue` in `getOrComputeIfAbsent` could as well be a `ConcurrentMap::computeIfAbsent`. Assume two threads call `getOrComputeIfAbsent`:

* if there's a value in the store, both will see it
* if there's only a value in the parent store, both will see it
* if there is no value at all, one of the default creators will be called and both will see the created instance

Looks pretty much like what happens with the manual synchronization. But I would need to further investigate and maybe test to be sure and it is surely more subtle. Hence the question whether performance matters, because if not, we don't have to spend time figuring this out.


It's not full async unfortunately, because each request still takes a thread for no good reason. There may be ways to address that though. In the meantime I will focus on getting a synchronous service working smoothly.

Another repository I started: https://github.com/biosustain/protov


> Hello David,
>
> I managed to track a problem down with our hardware to this commit. We've found that 2 out of 10 preproduction units are randomly rebooting after this commit was introduced. The devices need to be stressed to reproduce the problem (however WiFi iperf3 tests seem to be enough to get it to fail).
> Just to clarify, the problem doesn't exist with old DDR initialization code?
>

Previous initialisation code is working fine, I was considering patching the master on my fork by reverting to the hardcoded ar333x_dram.c initialisation code just to be sure (to be sure)... I expect this to be a worse case "quick fix" for our production issue.

> The problem seems to be related to temperature, since if we cool the board (below 50degC) it seems to run a lot longer, but eventually it will lock-up and reboot. The device will also lock-up in the boot loader at times just after the watchdog kicks in, and unfortunately the device needs to be power cycled in this case.
> Have you checked where exactly it lock-ups in U-Boot? I suppose that you don't have access to JTAG on your board?

No it's actually locking up in Linux first and foremost, occasionally (rarely) it will fail during startup in uboot (what I assume is the) watchdog resets. I don't have access to jtag on this hardware :(

>
> Have you had any reports of instability yet with this particular combination?
> I got some reports about problems, but only with AR9341/2 in combination with DDR2 and I still haven't got chance to confirm and eventually fix that as I don't have any board with such h/w. I'm able to make some tests on Carambola 2 (AR9331-AL1A + Winbond W9751G6JB-25) and maybe other AR9331+DDR2 boards if I find some in my deep boxes ;)

hehe, ok no worries. I referenced a lot of my changes from Carambola for adding support for our hardware design.

>
> I think that the most important change, with biggest impact, is DQS TAP values calculation. This could be reason or at least one of the reasons and I would check that first.
>

Agreed, I did find one thing reviewing the initialisation code, i'm still working on it, but will comment the specific areas above.

> For now, could you please:
>
> Tell me what firmware you are making tests on?

LEDE (reboot) bleeding edge, and Openwrt Chaos Calmer (15.05.01).

> Provide sources and/or binaries of the U-Boot you are having problems with (I don't know what your code is based on and what changes you made)?

My changes are minimal but in my forked repository - https://github.com/TouchStar/u-boot_pepe2k/tree/vh401_support

> Check what TAP values (registers QCA_DDR_TAP_CTRL_0_REG, QCA_DDR_TAP_CTRL_1_REG) where setup on your board before and after this code introduce?

I will do that tomorrow, I bricked hardware just before I headed home and need to desolder flash with programmer (not a biggie).

> Tell me what clocks profile you are using and if setting lower clocks (like 300/300/150 MHz) helps?

It's the default profile - CPU-400/RAM-400/AHB-200/REF-25/SPI-25 MHz. I originally thought the problem was simply the SPI differences, but it proved to be more complicated that that.

> Also, does your code include below changes:
> d9ba54f
> d450805
> 878acc1
> ?

I've confirmed the problem doesn't exist in the commit before this one, and is present since this commit, up until latest master, so it wasn't fixed with any changes made there after. Those changes/fixes made after do complicate things, but I'm concentrating on this commit at the moment.

>
> Are you interested in a fix given that the problem seems intermittent and inconsistent across hardware?
> Sure, just need to reproduce it on my hardware and track down the real reason.

Understood, i'll try my best to track it down, but i'm quite green with this hardware. I'm surprised how little documentation is out there for such a popular device :)

>
> Regards,
> Piotr

Thanks for the quick response!


This register doesn't seem to exist on AR933x (copy paste bug?)


Ditto as above.


nice


renamed back to map_ from map


Added read me file


Supports EV Training


I would name the alternatives `greaterThan` and `lessThan`, to keep the `Ordering` distinct semantic, with distinct messages


@retrue that's impossible


@Tobias-Fischer I definitely agree on that.


Was the "+" sign intended?


a jak myslisz? : >

Odpowiedź jest w słówku "new" :D Które oznacza... no cóż - new, nowy! ; )


++ thanks


fixed. There is an issue tab now.


Hm. OK. I'll think about it. Also haven't really looked into your implementation yet...


So the synchronized specialization is indeed going to make it into RC1. So I'm rolling back this commit.


`thanks


Per our discussion:
```python
In [1]: from xml.sax.saxutils import quoteattr, unescape

In [2]: quoteattr('>')
Out[2]: '">"'

In [3]: unescape(quoteattr('>'))
Out[3]: '">"'

In [4]:
```


@Hendy Just to let you know that I've added my Umbraco-tagged RSS feed back on my website.

<https://leekelleher.com/tag/umbraco/feed/>


This should go somewhere we can use it as a ready solution, IMO


Yes, it is necessary for passing the jobserver into the command.


why not use
for (var part in pathParts)


Regarding these: -

> d9ba54f
> d450805

I noted that hacky reset fix that persisted through some attempts to get rid of it. It seems the original engineering team added it as a "TODO" ->

/*
 * WAR: Hornet 1.1 currently need a reset once we boot to let the resetb has
 *      enough time to stable, so that trigger reset at 1st boot, system team
 *      is investigaing the issue, will remove in short
 *
 * TODO: verify this problem, add execution based on revision
 */

> 878acc1

Reading up on the fractional dithering, again, very little documentation :( But noted it could contribute.

Lastly... Remember this has occurred on 2 out 10 devices in our preproduction run, so I don't expect it to be easy to reproduce on a random hardware selection. The two devices CPU's were from two manufacturing runs so I thought it was worth spending time digging into it, as other people could be experiencing issues as well.


Excellent work sir.

Aside: I noticed tab was killing the playback yesterday, but you may have
fixed that - or it may have just been me.

On Aug 30, 2016 6:24 AM, "primaeval" <notifications@github.com> wrote:

> @im85288 <https://github.com/im85288> @iainsmacleod
> <https://github.com/iainsmacleod>
> I've fixed a lot of the playback issues. There was an out-of-order problem
> with the thread that runs on playback. The currentChannel was being deleted
> when the first thread stopped after another channel started playing.
> The epg visibility and current channel is now set when Stop is pressed.
> This might mean that something needs to be done about streams that just
> stop unexpectedly.
>
> Until everything is stable with playback the thread that controls the Up
> Next feature is disabled.
>
> If you want to help track down the problem you can uncomment some of the
> XXXXXX log print statements to see what is going on.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/primaeval/script.tvguide.fullscreen/commit/899d13d67af87f9e75e4c02c481b6266112d7884#commitcomment-18827298>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AEAHoqnnWkCPvGnfs6NVJuxRAg5TMhntks5qlATOgaJpZM4JwX2f>
> .
>


> I think I get multiple copies of the emails – can you remove me as well? Any email that has “paulo” in it.

(@tltlaboratory) Hi Paulo! This conversation was not about email lists, but if you want to stop receiving notifications, just log in to GitHub, go to the NetLogo repository and click "unwatch" at the top of the page. Let us know through feedback@ccl.northwestern.edu if you have more questions.


shouldnt this be ===


first commit


Template function definitions need to go in the .h file not the .cpp file. The definition needs to be visible at call sites or else how can the compiler generate the correct function?


I did this also on purpose.

I don't see any problem, I will add another script (with similar name) and keep your script as it was.


This is a [MERGE] commit, not a [IMP]. My bad...


@sugy なにか意図があったような気はするのですが、Printlnが残っているところを見ると雑な状態でコミットしてしまった感があるので a57c39fe でRevertしました。
また、 `rttMicroSec` といいながら値はミリ秒なので、 aad4af8 で変数名を修正しました


Вообще то такая конструкция:

			if (DialogService.ShowModalWindow(departmentSelectionViewModel) &&
				(SelectedDepartment = departmentSelectionViewModel.SelectedDepartment != null ? departmentSelectionViewModel.SelectedDepartment.Department : null) != null)
			{
				...
			}

может быть представлена так:

			if (DialogService.ShowModalWindow(departmentSelectionViewModel))
			{
				SelectedDepartment = departmentSelectionViewModel.SelectedDepartment != null
					? departmentSelectionViewModel.SelectedDepartment.Department
					: null;
				if (SelectedDepartment == null)
					return;
				{
					...
				}
			}

Разве нет?! И это не совсем то, что ты предлагаешь.


It should be handled more carefully. As you know, the Notepad saves file in GBK by default. And BOM might present, which may cause problems.


appr


Although performance is always important, it is not a critical issue at this point in time.

Correctness is far more important.

That's why I followed the KISS principle in the first place. 😉

And I think we should continue to follow the KISS principle until the current implementation poses a serious issue for a user.


不知道怎么回事，commit失败，但是github客户端也没提示我失败，我是登录github网页才看到的。这次commit之后，评论还是可以使用。


arent we overcomplicating the logic?
shouldnt it just be

completepath.indexof("*").lastindexof(path.sep) <-- wont this give us what we want?


ofcourse use above with proper null checks etc


actually this is a `trait` so we can't add that to 2.5.x. anyway? so wouldn't that be too bad to change the default to Ordering?


> I think we should continue to follow the KISS principle until the current implementation poses a serious issue for a user.

Makes sense. Especially since the current version is much easier to understand and verify than the one I proposed. But it is hard to leave the read-locking untouched... ;)


***Code Quaility: Critcal***
You should endeavour to use prototypal inheritance pattern or the javascript classical es6 pattern.
This index and it's prototypal method should be re-written like this
```javascript
class Index {
    constructor(){
       this.wordIndex = {}
        this.currentFile = '';
        this.crruentDataIndex = {}
// other variable intializers too
   createIndex() {
 // your methods
}
```
Or it the prototypal inheritance way like so
```javascript
function Index(){
          this.wordIndex = {}
        this.currentFile = '';
        this.crruentDataIndex = {}
}
Index.prototype.createIndex = function() {
// your methods should be here and all others
}```


but body and post data is missing.. waiting


using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
class Problem5SortingNumbers
{
    static void SortNumber(int n)
    {
        List<int> sequence = new List<int>();
        for (int i = 0; i < n; i++)
        {
            int temp = int.Parse(Console.ReadLine());
            sequence.Add(temp);
            sequence.Sort();
        }
        for (int i = 0; i < sequence.Count; i++)
        {
            Console.WriteLine(sequence[i]);
        }
    }

    static void SortNumberArray(int n)
    {
        int[] numbers = new int[n];
        for (int i = 0; i < numbers.Length; i++)
        {
            numbers[i] = int.Parse(Console.ReadLine());
        }
        Array.Sort(numbers);
        foreach (var entry in numbers)
        {
            Console.WriteLine(entry);
        }
    }
    static void Main()
    {
        int n = int.Parse(Console.ReadLine());
        SortNumber(n);
        SortNumberArray(n);
    }
}


Почему comaprator только по String?


As an additional workaround I just added a task in root's crontab to change the data/ permissions to "ludumdare:www-data" every minute.


> And I think we should continue to follow the KISS principle until the current implementation poses a serious issue for a user.

👍


fixing path for emotional intelligence blog


引用的digit阵列


Dear Msweier, Many thanks for this code. Can you be a bit more specific how I can use it ?  I'm woking with the Plugin QGIS-Drainage-Channel-Builder, it starts running. (See picture one). However in middle of the 2D Progress while generating " Inserting synthetic grid into DEM where channel elevation is less than DEM"  there is a error message "python.exe has stopped working". Funny enough, if I click on "closing program" I can see in the background that there result has been placed into a raster file. Of course after that step QGIS crash and take the result with it. I feel I'm so close but I have no idea what I do wrong...   How comes that you are proposing a vector layer name as output for the results in the raster layer ? Sounds a bit confusing for me. Or is it just for convenience, so that if your path - as a vector has a name, like in my case from Point 4 to Point 99  -> C4toC99 you want to have the results the same name?
[20160830_NL_Channel_builder_C4toC99.pdf](https://github.com/msweier/QGIS-Drainage-Channel-Builder/files/444719/20160830_NL_Channel_builder_C4toC99.pdf)
![20160830_qgis_drain_raster_result](https://cloud.githubusercontent.com/assets/21243059/18087585/9f894342-6ebe-11e6-9529-96502a8c77ae.JPG)


@kaneki1985 I'm sorry, what are your comments about? Keep in mind they notify commit authors.


yeah, we will need to do this for both '*' and '?' characters


May I know the functionality inside 'in-view' which was required at the top of file. Also some info about enter and exit events used here. Thanks.


***Code Quality: NitPick ***
Change this function calls to be more descriptive and simpler to read. You can use lodash  utility method for this
[isObject](https://lodash.com/docs#isObject)


why is it getVarInt? i don't see that function in that commit though


Deal


Перенеси это все в JodaDateValidator


***Code Quaility: Nitpick***
Since you are returning boolean its always good to return early and seperates isEmpty from getindex as it seems you are mutating a global variable while it should just be checking for an empty object


Typo? Should this be '1.0' as in the previous change?


***Code Quality: Nitpick***

This utility function might not be needed when you are clearing up your codebase


7k credits, sad.


Всё совсем не так.
Вот тут можно посмотреть то, что изначально было предложено:
https://github.com/xbadcode/Rubezh/commit/3c6f3fff4ad52846ab7c0a78acf92bdeb93f4269
Изначально было предложено перенести общую на уровень выше, дабы 2 раза одну и ту же проверку не делать, а не переносить двойную проверку на уровень выше.


***Code Quality: Critical***
This might be a lot of overehead as the O(N) seems to be n**n


##CSS修改
桌面版->460px
移动版->400px


If they are not secret - yes. Bookkeeping is always better than no bookkeeping.
I do some calculations and checks on my 'mons now and then, but always "on paper", one by one.
Point is - this hack will make it into next release of non-ruby version, so example of how this can be used will be nice.


***Code Quality: Nitpick ***
Prefer arrow functions to avoid setting the lexical this to another variable


is this nessesary?


Just not documented (which is nothing unusual for Qualcomm/Atheros documentation).
It was taken from here:
https://source.codeaurora.org/quic/qsdk/oss/boot/u-boot-1.1.4/tree/board/ar7240/ap121/hornet_pll_init.S?h=1.1#n387


***Code Quality: Nitpick***
A function should only do 1 thing. This functions seems to be doing more than one thing.
Split down this functions to different helper functions for easier testing and readablity


As above.


If `pin_factory` is now a class, rather than a method, should it be capitalised as `PinFactory` instead?


TeamCity OTA+ :: rvi_sota_server :: ci [Build 858](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=11855&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


Multiple reasons, including the fact that the API is not only provided for single threaded use, it is provided for the server implementation to use as well. Using the law of least surprise, if all the other classes are thread-safe, then this class should be thread-safe as well. Finally, the purpose of thread-safety isn't so that you can pass a ChatComponent between multiple threads and _modify_ it, but for data consistency when this is passed from the API to the implementation [thread]. It's completely thread-safe or not thread-safe at all, and the latter should be opted for whenever possible.


Done


@mondain  Hi, thank you for the clarification. I was confused also due to looking at the commit dates and not seeing one that was in August 2016 :-) .  I see though that the param was there from 2014 :) :+1:


TeamCity OTA+ :: rvi_sota_server :: ci [Build 858](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=11855&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:03


***Code Quality: Nitpick***
Always prefer map forEach and other higher functions in favour of loops over object and arrays
This can be refactored to
```javascript
Object.keys(this.wordIndex).forEach((key)=>{
         if(path.win32.basename(indexKeys) === fileName || path.win32.basename(indexKeys, '.json') === fileName) {
      result = this.wordIndex[indexKeys];
}
  }
```


'is build' -> 'is built'


add try/except here. `__contains__` should silently fail as I suppose.


I will publish it then. It's a bit of mess now so I have to clean the layout. The possibilities are endless for calculators like that actually when you can export pokemon data from the game itself.


you are using the return value of optJSONObject without check - better look through all of decoder


TeamCity OTA+ :: rvi_sota_server :: ci [Build 859](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=11859&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


TeamCity OTA+ :: rvi_sota_server :: ci [Build 859](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=11859&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:48


***Code Quality: Nitpick***
You should use meaningful variable names
Functions should only do one thing
This functions has a lot of overhead and do many things. You should try seperating different functions as delegate them as the name implies.
You should use straight forward methods when neccessary e.g
```javascript
_.get(that.wordIndex[indexKey], queryString);
//This shouold change to
that.wordIndex[indexKey][queryString;
```


@manishrawat0212 Create list elements first then add text


This line is now wrong, as the newer Zeros have one CSI connector but still zero DSI connectors.


Oh I see. But the unit is correct now. Does it mean that we need to replace the test .h5 files? Setting the phase to π/4/180*π is awkward.


@manishrawat0212 Why did you use detach? why not remove?


Requires the latest of the latest premake5, you might even have to compile it yourself.


Please revert that


TeamCity OTA+ :: rvi_sota_server :: ci [Build 860](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=11861&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


Fixed the spacing and refactored to adhere with PEP8 standards


Nope. The opposite: it will return true if at least one of the needles is found at the end/start of the haystack string.

Do you see any drawbacks in that?


Actually in M, it's true, so i change it to true. Commented out reboot function to be sure if something is not crashing from svc. And it's not, so I will revert it.


> I noted that hacky reset fix that persisted through some attempts to get rid of it. It seems the original engineering team added it as a "TODO" ->

This TODO is also mine ;)
And it will be there probably forever as I don't think that there is some better, or any at all, way to fix this bug.

You can refer to Atheros SDK based low level initialization for AR9331 here:
https://source.codeaurora.org/quic/qsdk/oss/boot/u-boot-1.1.4/tree/board/ar7240/ap121/hornet_pll_init.S

Just be aware that there are many hard-coded register addresses and values... and some of them I have just skipped in new code, so maybe that's the reason...

> It is likely a hardware bug.

Yes, AR9331 seems to be really broken (another example: USB controller is going crazy when WLAN is changing frequencies, ex. scanning channels), but still powerful and very popular.

> No it's actually locking up in Linux first and foremost, occasionally (rarely) it will fail during startup in uboot when (what I assume is) the watchdog resets. I don't have access to jtag on this hardware :(

U-Boot doesn't initialize and use watchdog at all and full chip reset disables watchdog.

I have seen similar problem with hang in kernel as soon as it tries to touch WMAC/RTC registers, solution is here: https://github.com/pepe2k/u-boot_mod/blob/master/u-boot/cpu/mips/ar7240/ar933x_pll_init.S#L119, but I don't think that it's not the reason for your problem as you wrote that it hangs when WLAN is working.

> I've confirmed the problem doesn't exist in the commit before this one, and is present since this commit, up until latest master, so it wasn't fixed with any changes made there after. Those changes/fixes made after do complicate things, but I'm concentrating on this commit at the moment.

OK, so the bug is somewhere in DDR controller initialization, this is valuable information.


got it - fun fun fun...

```
/* Sync mode , Set Bit 8 of DDR Tap Conrtol 3 register */
    set_reg(AR7240_DDR_TAP_CONTROL3, 0x10105);
```


TeamCity OTA+ :: rvi_sota_server :: ci [Build 860](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=11861&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 210, ignored: 3 Build time: 00:07:41


TeamCity OTA+ :: rvi_sota_server :: migrations container [Build 45](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=11864&buildTypeId=ota_plus_RviSotaServer_MigrationsContainer) is now running


@nfeske: m-stein's contribution


@nfeske: my contribution


No I did but when I load the server it says missing dependency


TeamCity OTA+ :: rvi_sota_server :: migrations container [Build 45](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=11864&buildTypeId=ota_plus_RviSotaServer_MigrationsContainer) outcome was **SUCCESS**
Summary: Running Build time: 00:01:17


ok fine by me


I'm fine to add xcode7 build.  It could catch bugs related to buggy std::make_shared.


create


typo resovleSourceContainer --> resolveSourceContainer
Still time to fix it before releasing a new version


This commit was part of https://github.com/SAP/openui5/pull/1077


Would you mind adding the screenshots to this pull request?


Works!


Previous commit


lol.


> Vs Team Service - Release Manaegment

`Team Services - Release Management`

I think the convention is to either spell out "Visual Studio" or drop "VS" from the name.  Plus there was a typo in "Management".


Carambola 2, with current version of my code:

```
u-boot> md.l 0x18000028 1
18000028: 00010105    ....
```


Hello
I wonder why other error code than 400 and 500 are not considered as valid error code. According to SOAP 1.2 adjuncts, other HTTP status code are valids as code with SOAP Fault payload: https://www.w3.org/TR/2002/WD-soap12-part2-20020626/#tabreqstatereqtrans

I have troubles with a server replying 401 Unauthenticated code (with a SOAP Fault enveloppe) that I cannot handle thanks to SOAPHandler<SOAPMessageContext>::handleFault because current JAX-WS implementation produces ClientTransportException.

Another issue, this ClientTransportException is not expected to be caught specifically as it is not public API: http://stackoverflow.com/questions/35917173/how-to-properly-catch-clienttransportexception

According to standard, isErrorCode should also include all 4xx and 5xx code, requiring body to provide a SOAP Fault payload, at least to allow valid 401 response with SOAP Fault to be processed properly as a SOAP response.

Another option may be to add a MessageContext client property to switch this check from one implementation to the other, so that JAX-WS user may choose between SOAP Fault handling or ClientTransportException without any effective error handling.

What is your opinion about it ?


@manishrawat0212 Why closest? why not parents?


it is a HACK and that call cause AL bad params error...


Oh and if the release triggering also works with TFS, it would probably be `TFS/Team Services - Release Management`.


Er... I fixed that: a8d8a8a7edc34f1d6f7d82f5704ba8d292c31383


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


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


That's not actually correct. A ChatComponent which is written on one
thread, and contains ArrayLists and standard non-volatile non-atomic fields
will be safe to read in another thread if passed to the other thread in a
safe way, for example via a thread-safe queue. Java's memory model means
that the ordering enforced by that queue is enough to ensure correct
results in the consuming thread, if the ChatComponent is not modified after
it is added to the queue.

If the ChatComponent is modified after it is added to the queue, the use of
thread-safe collections here and volatile will not actually result in
well-defined behaviour, you would need additional synchronisation with the
consuming thread.

Making fields atomic/volatile in the way which is currently done is only
resulting in a performance hit.

If you haven't already, I suggest watching this talk about the java memory
model:

https://shipilev.net/blog/2014/jmm-pragmatics/
https://www.youtube.com/watch?v=noDnSV7NCtw
https://www.youtube.com/watch?v=Ky1_5mabd18

On 30 August 2016 at 12:44, Pierre C <notifications@github.com> wrote:

> Multiple reasons, including the fact that the API is not only provided for
> single threaded use, it is provided for the server implementation to use as
> well. Using the law of least surprise, if all the other classes are
> thread-safe, then this class should be thread-safe as well. Finally, the
> purpose of thread-safety isn't so that you can pass a ChatComponent between
> multiple threads and *modify* it, but for data consistency when this is
> passed from the API to the implementation [thread]. It's completely
> thread-safe or not thread-safe at all, and the latter should be opted for
> whenever possible.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/TridentSDK/TridentSDK/commit/ac2750e8602b3c39d21afac40571638173a120da#commitcomment-18828210>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AAvwaAS9Mm_b7G2rj_AmoJOpxXhgcqGRks5qlBeAgaJpZM4JwWnW>
> .
>

--
Ross Allan


Thanks man : ) I've been hunting for a json validator! I think, having
successfully ironed out my uci issues, I'm just gonna have to put in the
time and effort with the backend (actions, etc), that's probably what's
killing me.  Thanks again, I can't believe you didn't make this a paid app
given its universal popularity ; )

On Aug 30, 2016 4:23 AM, "Andrei F." <notifications@github.com> wrote:

> The parser is an external library so it either successfully loads the JSON
> or it doesn't, so it's not part of the app. If you have markup problems in
> your config I suggest running it through an external JSON validator
> <http://jsonlint.com> to help figure out the issue.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/AndreiLux/Synapse/commit/8391a5a584034c1c7333a4e87fde78c11259c89b#commitcomment-18827292>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ACjHoK8GjLq3CeVAvx8s36RvZPhLgClMks5qlASHgaJpZM4Jv3Z8>
> .
>


Cool, I'm back to not having much time at all for development I'm afraid (back at work) but will install and test later on tonight.


There's a problem with that: constructor signature says it waits for IConfig $appConfig, but $c->query('AppConfig') returns an OCA\Richdocuments\AppConfig() object. => problem, my owncloud instance explodes.


> The plugin has a post build step - "VS Team Services Continuous Deployment".

I recommend re-wording and updating with the new name:
`This is done via the **Trigger release in TFS/Team Services** post-build action.`


- [x] @mentions, #refs, [links](), **formatting**, and <del>tags</del> supported
- [x] list syntax required (any unordered or ordered list supported)
- [x] this is a complete item
- [ ] this is an incomplete item


TeamCity OTA+ :: rvi_sota_server :: ci [Build 857](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=11840&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **UNKNOWN**
Summary: Canceled (Exit code 137 (new)) Build time: 02:43:35


TeamCity OTA+ :: rvi_sota_server :: ci [Build 861](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=11868&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


TeamCity OTA+ :: rvi_sota_server :: ci [Build 862](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=11869&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


Maybe a word of explanation in comments why it was needed?


@Tobias-Fischer thanks a lot for the painful job :smile:

### Take-home message for @gregoire-pointeau
_Reverting_ in git means to **introduce a new commit that invalidates the previous one**. Despite it's easy to push a button in a GUI and do a revert, this operation tends to mess up the history, especially when it's executed right after a merge. The **history in git is a precious thing** we should always try to keep neat.

Since you're working on your own branch, you can modify the public history of that branch without caring too much of the consequences of this operation. To this end, use the command **`git rebase`**. In particular, the idea is to put whatever you did on your branch `feature/inserm/narrative` on top of `devel`. Hence, the right command is **`git rebase devel`** launched from within your branch. Then, to publish these modifications you have to **`git push --force origin feature/inserm/narrative`**. The `--force` options here is required because you'd modify the public history of `feature/inserme/narrative` (but - as said - no one minds in this case).

Remember that when you are about to perform a `git rebase` you ought to make sure that your two local branches used in rebasing are up-to-date (e.g. `devel` and `feature/inserm/narrative` in this case).
Thus, do:
```
git fetch origin
git checkout your-branch-1
git pull --rebase origin your-branch-1

git checkout your-branch-1
git pull --rebase origin your-branch-1

git checkout your-branch-2
git pull --rebase origin your-branch-2
```


In gpfs.h: #define GPFS_USRQUOTA     0
In sys/quota.h: #define USRQUOTA  0

Couldn't use USRQUOTA as gpfs.h has macro for GPFS_USRQUOTA and not USRQUOTA. The USRQUOTA macro is in sys/quota.h, which cannot be included if gpfs.h is included as that leads to compilation errors because both header files have some macros with same names.
Errors seen when we include both sys/quota.h and gpfs.h
In file included from /root/rpmbuild/BUILD/nfs-ganesha-2.4-dev-29-0.1.1-Source/FSAL/FSAL_GPFS/export.c:50:0:
/root/rpmbuild/BUILD/nfs-ganesha-2.4-dev-29-0.1.1-Source/FSAL/FSAL_GPFS/include/gpfs.h:2728:0: error: "Q_GETQUOTA" redefined [-Werror]
 #define Q_GETQUOTA      0x0300  /* get limits and usage */
 ^
In file included from /root/rpmbuild/BUILD/nfs-ganesha-2.4-dev-29-0.1.1-Source/FSAL/FSAL_GPFS/export.c:39:0:
/usr/include/sys/quota.h:120:0: note: this is the location of the previous definition
 # define Q_GETQUOTA 0x800007 /* get user quota structure */
 ^
In file included from /root/rpmbuild/BUILD/nfs-ganesha-2.4-dev-29-0.1.1-Source/FSAL/FSAL_GPFS/export.c:50:0:
/root/rpmbuild/BUILD/nfs-ganesha-2.4-dev-29-0.1.1-Source/FSAL/FSAL_GPFS/include/gpfs.h:2731:0: error: "Q_SETQUOTA" redefined [-Werror]
   #define Q_SETQUOTA    0x0400  /* set limits */
 ^
In file included from /root/rpmbuild/BUILD/nfs-ganesha-2.4-dev-29-0.1.1-Source/FSAL/FSAL_GPFS/export.c:39:0:
/usr/include/sys/quota.h:121:0: note: this is the location of the previous definition
 # define Q_SETQUOTA 0x800008 /* set user quota structure */


It does need the background thread for dead stream detection but I need to think and test a bulletproof solution for crappy cheap Android tv boxes. ;)


> Yes, it is supported from 1.3 version onwards.

Was "1.3" the version of the previous plugin?  Maybe the answer can simply be "Yes"?


Fix committed via 5ea3bcc3492ca6d4b32fda4d8458eaee912748b4


Yeah I guess so – I felt like this would be useful somehow but I'm having a hard time defending it. Feel free to make that change.


test.cc was removed as of e4fd51195d742924128d8576f4bd22b33ba8febb


You need to revert the groovy code in the -spring-boot-starter-remote-shell` as well.


Also, given the question is just above the next "FAQ" section, what would you say to moving the question there?


This seems like pretty weird behaviour on Julia's part.


No valid flags with GPFS structure gpfs_quotaInfo_t. It takes both zero and non-zero values as valid change request.
Fore reference:
typedef struct gpfs_quotaInfo
{
  gpfs_off64_t blockUsage;      /* current block count in 1 KB units*/
  gpfs_off64_t blockHardLimit;  /* absolute limit on disk blks alloc */
  gpfs_off64_t blockSoftLimit;  /* preferred limit on disk blks */
  gpfs_off64_t blockInDoubt;    /* distributed shares + "lost" usage for blks */
  int          inodeUsage;      /* current # allocated inodes */
  int          inodeHardLimit;  /* absolute limit on allocated inodes */
  int          inodeSoftLimit;  /* preferred inode limit */
  int          inodeInDoubt;    /* distributed shares + "lost" usage for inodes */
  gpfs_uid_t   quoId;           /* uid, gid or fileset id */
  int          entryType;       /* entry type, not used */
  unsigned int blockGraceTime;  /* time limit for excessive disk use */
  unsigned int inodeGraceTime;  /* time limit for excessive inode use */
} gpfs_quotaInfo_t;


Thank you for reviewing.


Today's call notes :

1. Project purpose
2. CMS use for project : Magento
3. Number of pages : 5
4. Required Contents : product images, video.
5. Slider images limit : 20 images.
6. Browser support : safari, chrome, mozilla firefox.


yeah, test.cc wasn't supposed to go in and was garbage. As for the defaults, it was a mostly automated process aided by the clang modernizer. .h vs .cc for defaults influences if/how to inline, so no difference there, but not how the code functions


ignored .imr iml and so on idea suffix


TeamCity OTA+ :: rvi_sota_server :: ci [Build 862](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=11869&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 210, ignored: 3 Build time: 00:07:34


Hi Eddie,
Snapshot 1.1 seems to be no longer active. You should change it to 1.2 else you get a dependency error in maven install.
It should also be insignificant where you place the pi4j dependency.

Cheers


This breaks the _Generic selection, since size_t and uint64_t (at least with 64 bit, I imagine uint32_t elsewhere) are apparently indistinguishable. Same of course for derived pointers.


@caugner Yes, it does and that why I was hesitant to merge it. I made the decision because I didn't want the typo to persist over the lifetime of the project... better to fix it now, in 0.2, than 1.2.

That's a good idea about updating the Readme. I'll do that. Hopefully, the change will not cause too many headaches for people.


> @gajus should I have named the commit differently?

We are using https://github.com/conventional-changelog/conventional-recommended-bump

Therefore:

* `fix:` prefix will bump the patch version
* `feat:` prefix will bump the minor version
* anything that includes `BREAKING CHANGE:` in the footer will bump the major version.


> also can this file be edited manually? i.e. does the script only prepend new releases, or does it re-generate the file, replacing any manual edits?

It re-generates the file each time. I'd like to avoid manual edits.


Premier message


I'm confused about this check for a semi-colon. I can imagine it being useful for the old, set-of-all-users-from-events method, but it seems like you could remove it now that you have an explicit list.


第一次


Should not we have "moduleH1" here?


@nateberkopec is this `@width` doing any work?


really?


Respond with `res.forbidden` in this case.


fo -> for


Unauthorized if no token -> Unauthorized when you are not logged in


Commiting ParentProj to MAster


401 -> 403


We do not.  I'll delete it

On Aug 30, 2016 12:39 AM, "M. Teufel" <notifications@github.com> wrote:

> We don't need this entire SSH section anymore, do we?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/SuperTux/supertux/commit/55f9b6062ea9b6817ae80cd3afd86eaa71d7145f#commitcomment-18824438>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AC7N4YUGA6yMMoGRdu8VkkqbCpBHS9K2ks5qk8IJgaJpZM4JwKet>
> .
>


@mosheBelostotsky I would word it this way:

**Hubot Enterprise** is comprised of a set of middleware modules and scripts that facilitate writing `hubot` integrations with the following features:
* Security flows (authorization and authentication)
* Help mechanisms
* Standardized, multi-chat platform adapter APIs.


Why remove this check?


Forbidden for regular users trying to access machines not assigned to them.


this should be $footer = strip_tags(.... etc ....


Can I use that on my site?


The deploy provider does not allow dynamic version numbers.  I also only
want to call CPack if I'm going to deploy it

On Aug 30, 2016 12:39 AM, "M. Teufel" <notifications@github.com> wrote:

> Why do you continue to use a script when this all should be possible with
> a deploy provider?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/SuperTux/supertux/commit/d8614ef875cc35ab900bbb0579d77132f4760be9#commitcomment-18824433>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AC7N4QVZrljOwFzP2utgf3yJV1WcYq_rks5qk8HygaJpZM4JwKeA>
> .
>


machine -> machines. You are expected an array of machine. Please use plural.


Pushed the wrong directory, should be ok now


Related https://github.com/UKMNorge/UKMvideo/issues/8


Good catch.


work on naive bayes.


not be authorized -> should be unauthorized


Hi, can you make a PR with your diamond shape for the ngImgCropFullExtended? We would love to have more features and people developing on it.


admin -> admins


@caugner Done. There also might be a way to migrate the old value. Don't really have the time to look into that.


@Sami32 The GUI text doesn't "prove" what is correct: 0774d3ad6f8a56e3a26be187a34d761b3476a870. This needs to be figured out properly. I don't remember what investigation I did prior to that change.


this definitely changed the look of the animation, and it looks mucho bueno now, but it's still having a little graphical stutter at the end of the animation. it's pushing itself back up. i feel like it's the margin-top css added to tab-content. I can't do any actual debugging right now because I'm at school and don't have brackets, but oh well.

shoutout to brackets.io


pls add new line thx


I accidentally made a space where the "Default" phrase is in,  commit 47f1e81:, and pushed that commit in the repo.  This commit overwrites that change.


there is a lot of work needed to get it working properly. i hope bluetooth will work again on cm14.0
good luck, guys!


Corrected to check if the student already exist on a certain date.


wow such descriptive name, many confusions


Thanks! I merged and reviewed the texts. My changes are https://github.com/nfeske/genode/commit/9d8d4e630c2d253a2f49e454c4d7b7d406a7e35a


Testa en annan bakgrundsfärg


font sizes in px pls


There are two tables `Stream` and `Messages`. When you append with `ExpectedVersion.Any` and the stream row doesn't exist, it is created, before rows are inserted into `Messages`. The stream row maintains the version number sequence and maps the string ID to a int for easier joins / faster queries. There is a deadlock condition when you have two or more threads appending to the **same** stream with `ExpectedVersion.Any` at the same time, where each thread attempting to insert the same `Stream` row (insert if not exists). It's a really narrow window, but load tests showed this to occur.

Now if one of the threads loses, they will retry and because the `Stream` row now exists, it will not attempt to insert it and will succeed as usual.

Does this make sense? (My commit message was clearly insufficient; noted).


前面减号表示本行内容被更改


@rickerd Hey there, I would love to submit a PR with this, but unfortunately although the shape of the image preview is a diamond, the resize handlers are still those of the rectangle shape which I simply copied and edited. I consider this to be incomplete at the moment. If you think this is something that would add value to the ngImgCropFullExtended, I would love to work some more on this and try to make it fully functional before submitting a PR, by the end of this week(sunday). :)


Because this is actually used as a submodule for Ardupilot, which has a different submodule for Nuttx.


s/different/separate/


@jrah For this fix to work, `responsive-nav.js` needs to be copied into a local `/js` directory, otherwise it may only work on a local setup with `wfp-ui` Bower package installed.


aint this the same as line 55? combine pls


Look at ExtractVariables policies in request flow and include parameters in specs. This covers params in:

1. Header
2. QueryParam
3. FormParam


I got this error
oops, something went wrong! TypeError: readAll(...).then is not a function.

any advise ?


@oceanstack

```git submodule sync``` should help!


why style the same thing with display none and inline block 🙃


@mosheBelostotsky @rqc - Perhaps we should move this info to the Wiki, and keep just a quick introduction here?


Ofcourse, every feature is of value towards the plugin.
Make sure you have the latest version, we're at 0.6.0 now.


unneeded, pls change to 0


missing space


See https://github.com/ternjs/acorn/pull/435 – by adding a `paths.acorn` option that points to `../dist/acorn.js`, the parent module is considered external when bundling, but points to a relative file in the generated output, which is what we need


padding: 0;


Will do a git pull & merge locally before updating my forked repo. :)

> On 30 Aug 2016, at 15:44, Rick <notifications@github.com> wrote:
>
> Ofcourse, every feature is of value towards the plugin.
> Make sure you have the latest version, we're at 0.6.0 now.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub <https://github.com/corasla/ngImgCropDiamond/commit/4c464904fb492ca00c3f1cb350f57e094e5cfcca#commitcomment-18828849>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AFSWpmLdj1SMeG2OVC8zSzq5RaJfxjAhks5qlCW2gaJpZM4JwefM>.
>


The OP was unable to provide steps to reproduce.  He was on the latest Chrome and said to close the issue, he's using his own workaround for now.  Reverted via https://github.com/qzind/tray/commit/9c60f1e968cdeed41361ef849325d9c1345219c4.  We can reopen if we have the proper steps to reproduce this problem.


margin: 8px 30px 0 0;


@AnonXS This should be reverted since for example 9991 will never call this function.


Wouldn't this implement Zombie?


@kvangent I just noticed we are using the same key for Google Static Maps API and Google Maps Geocoding API. Don't they require 2 different keys? The 1st one is a browser key and the 2nd is a server key


Thank you for the careful review! :-)


Говорил же, что лучше логарифмическую шкалу использовать =P

Т.е, по оси OY рисуй не `t`, а `log(t)`.


There is something wrong with this patchset: my loolwsd is configured with --disable-ssl , and my loolwsd.xml has both ssl.enable and ssk.termination at false.  But, at some point a query to /loleaflet.html is enforced as HTTPS, resulting in the breaking of the HTTP legacy mode of loolwsd !


don't need this or line 19


Can you, please, show how you use it so I can help?


:+1:
Спасибо!
Постараюсь перевести на англ.


GG @papandreou


saved my day


pls use font sizes in px and then components in rem


Give me the output of the patching commands so I can fix it more easily


Sorry, my fault. @gladhorn force-pushed and I squash-merged and forgot to edit the actual commit message.


For multi you actually need .begin(100000, SERIAL_8E2) since it uses a bit different framing. I am wondering if SBUS really works since also has the same SERIAL_8E2 encoding.


@matthewmorek I didn't want to make changes outside of `index.html`.

This was intended so you could see what I was doing with minimal noise. I will add `responsive-nav.js` into `/js`.


Ok thanks to you two.
I guess case closed !


Awesome! Thanks, saves me the trouble.


@jrah Can you confirm if this fix works locally?


Not sure why req_py_majver and req_py_minver always need to be set to 2 and 6, respectively. On a CentOS 6 system where EasyBuild was  built with system python 2.6 any build of a python package will fails. EasyBuild checks for python commands and (in our case) finds
   - the self-built Python 3.5.1 in a path .../bin/python
  -  python2 and python2.6 in /usr/bin
it then checks all the 3 pythons agains req_py_majver etc... and eventually decides to go with python2 which clearly is not python 3.5.1. As a workaround I have now set req_py_majver and req_py_minver to "None" to make python package builds work again.


Now how the hell would I use Github to merge this instead of copy/paste?


dddd


Hi ittala,

The plugin needs some bug fixes, and I've been meaning to look into it.  I don't think you are doing anything wrong.  What version of QGIS and GDAL are you using?  Also you should get an error in your Python section of your Log panel (under View->Log).  Do you know what that says?  I will try and look into this in the next couple weeks.

If you got output in QGIS, then you should have a folder called that is written to your desktop or home folder called QGIS2DrainageChannelBuilderFiles with the output files in it.  This will allow you to see what was done if QGIS is crashing.  You should get several files all with your vector layer name as a suffix after the underscore.  See the Output section of the documentation.


Thank for ur response. I did like u suggest and met another error:
```TypeError: Attempted to wrap undefined property handleLike as function```
in this sinon function:
```handleLike_spy = sinon.spy(Person.prototype, 'handleLike');```
this error appears only if i use ```@observer(['profile'])``` . If i remove `(['profile'])` - no more error.

i also did `console.log(Person.prototype)` :
```
// Prototype
Constructor{
	constructor: function (props, context, updater) { ... },
	__reactAutoBindMap: Object{getDOMNode: function () { ... }},
	getDOMNode: function () { ... }, render: function (){ ... },
	mixins: null,
	statics: null,
	propTypes: null,
	contextTypes: null,
	childContextTypes: null,
	getDefaultProps: null,
	etInitialState: null,
	getChildContext: null,
	componentWillMount: null,
	componentDidMount: null,
	componentWillReceiveProps: null,
	shouldComponentUpdate: null,
	componentWillUpdate: null,
	componentDidUpdate: null,
	componentWillUnmount: null,
	updateComponent: null
}
```
and in case when i remove `(['main_store'])` the output is next:
```
Bar{
	componentWillMount: function (){ ... },
	componentWillUnmount: function (){ ... },
	componentDidMount: function (){ ... },
	componentDidUpdate: function (){ ... },
	shouldComponentUpdate: function (e,t){ ... }
}
```


ывф


@edagan I kind of like to treat documentation as code. When we make changes to the source code, let's say in a pull request, we should make any documentation changes that occur as part of those changes.


following the main issue about render the image without the same fonts, I did all the above changes but I got the error.
Do you want me to share the updated code ?
or this code ?
`var node = document.getElementById('elm');
		domtoimage.toPng(node)
	    .then(function (dataUrl) {
	        var img = new Image();
	        img.src = dataUrl;
	        document.body.appendChild(img);
	    })
	    .catch(function (error) {
	        console.error('oops, something went wrong!', error);
	    });`


oh. this shouldn't be monitor.


Updates to ships don't affect old ships for the sake of compatibility.


You ROCK!


Thank you for explanation!


please avoid using ids as css selectors


@matthewmorek

Yes, with issues:

- `>767` navigation disappears
- On load the menu briefly appears but transitions out
- `main-nav` is applied inconsistently to `nav`, rather than its containing div


This patch is not based on the current master branch, so while Dmitrii verified it, it was done only against the old IDE version, although I'd expect just fine with master as well. It also seems that it implements the check only in the compiler plugin - what about the debugger?

It also doesn't follow our coding style guidelines, as described in https://github.com/foss-for-synopsys-dwc-arc-processors/arc_gnu_eclipse/blob/master/JavaEclipseFormatterForArcGnuIde.xml. This code style was based on "generic" Java style in Eclipse with a few changes. We probably should avoid using a custom style and instead use a Google one, from https://github.com/google/styleguide/blob/gh-pages/eclipse-java-google-style.xml . Luckily both of them sahe the main trait - usage of whitespaces instead of tabs.


TeamCity Glyssen :: Glyssen pull requests [Build *.*.225.321ec93855d3af142031033dd1ac2f54af2ba480](http://build.palaso.org/viewLog.html?buildId=76221&buildTypeId=Glyssen_GlyssenPullRequests) outcome was **SUCCESS**
Summary: Tests passed: 1284, ignored: 17 Build time: 00:04:03


Annoying, but in Firefox, Ctrl-I opens the Page Info page, and this code does not sem to be called. Not sure what to do about it, though.


this is some WIP on the way to solving the problem as discussed here https://irclogs.baserock.org/%23baserock.2016-07-08.log.html

and here
https://irclogs.baserock.org/%23baserock.2016-07-12.log.html


Good job, guys! But where are the function descriptions?


@cecca00  Nope, same key. You just need to have the Static Maps API for it.


This would break because url is not there. let us scoped to the context. You can again get the url here. or define  let url on top.


6080 ? 3384 ?


@mchurchward - Is the reasoning behind the table structure to make it easier to merge the sub plugin providers with the ones that ship as standard?


Changed the design for error dataset.
Error dataset will now have two fields: 'record' and 'ts'. And record itself will be the key for the error dataset, in order to avoid the overriding of keys(which was set earlier using recordCounter approach) in case of multiple reducers.
Same error/dropped record coming multiple times, will be entered only once.
Let me know your thoughts.


mongoDB 에 데이터를 싹 다 지우고 다시 시작 할 것 !


todo: check for error from this call


todo: <= 0


Автонумерация студии


@mchurchward Will the source field be used to track whether it came from oembed.com / plugin / custom provider?


@jfoug the src/x86-64.h changes shouldn't have been committed!
Don't use `git commit -a`!


Двойная отрисовка?


The unrar changes should have been in a separate commit.


下面这两行代码中，温度符号希望可以更改为℃。
                    String.format("↑ %s °", weather.dailyForecast.get(0).tmp.max));
                    String.format("↓ %s °", weather.dailyForecast.get(0).tmp.min));


@FernetMenta what do you think about this as a fix for the problem discovered here https://github.com/xbmc/xbmc/pull/10333#issuecomment-243243601 ?


I do not agree with 'cli command helper' wording.  CLI already means **C**ommand **L**ine **I**nterface.

What about "Command line tool to manage your local development environement for YunoHost" ?


I did not use commit -a   I used --author=""  but these were picked up :(


I can't find `pi_revision` anywhere. Was it supposed to be `pi_info`?


Did you forget to "git add" the actual changes?


dear @roelandjansen @sijskes  temporary no function is using this flag.


In fact, I will correct it.
I will do a commit with other modifications.
Thx.


lol


We should see if we can get a BMUnit test for this, something that pauses the OutputStream.write until the openInputStream() call completes. What's actually going to happen here is it will copy the file completely, then open an input stream for it...and the fact that it's running in a separate thread won't help us much as a result.

What we need is to have the copy operation happen in the thread, but not the openInputStream() call. The method should spawn the thread, and wait for it to start (using some sort of synchronized block on an object to wait() and notify() in the thread, if that makes sense)...then openInputStream and return it while the thread is still executing.


the unrar changes are not complete.  Some of it was debugging, trying to find some issues.  Some likely would be better (the -num vs -1), but in the testing I did, all num passed to function was 1, so I think the code is not going to crash.


fix #64


product/域名服务/域名备案/备案简介/备案限制.md

这里没有改过来^^


this is wrong as it is.


20150916 and 2090916?
I'm guess you are wrong here. Looks like type, should be 20151104


Can the `{strict:true}` option be used to pass *any and all* JavaScript strings through “he” to achieve either valid output ot an error?

I need a truly generic content to HTML escape library, and this one was [recommended](https://github.com/angular/angular.js/issues/15061) to me, but if it has *any* constraints on input, it cannot be used.


Do you have some unusual settings in your git config which caused this?

You might want to check `git config --list`


You might be interested to see some other changes we made on our waterfall patch https://github.com/WaterfallMC/Waterfall/blob/master/BungeeCord-Patches/0008-Upgrade-to-netty-4.1.patch


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


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


$ git config --list
user.email=jfoug@openwall.net
user.name=jfoug
push.default=simple
merge.ff=only
core.repositoryformatversion=0
core.filemode=true
core.bare=false
core.logallrefupdates=true
core.ignorecase=true
remote.origin.url=git://github.com/magnumripper/JohnTheRipper
remote.origin.fetch=+refs/heads/*:refs/remotes/origin/*
remote.origin.pushurl=git@github.com:magnumripper/JohnTheRipper
branch.bleeding-jumbo.remote=origin
branch.bleeding-jumbo.merge=refs/heads/bleeding-jumbo


Is it OK that you add 2016.4 files with Aldec's 1.3 version when 1.6 is already merged?


I used tuple_try_out instead of in the book were it says Fibs, I got confused.


@sud80 this would be invoked in case of process.env.VERBOSE only, which is defined above in case of process.env.VERBOSE on line 163


I'm still confused about this.


@captn3m0 hope this won't cause any issue on production.


I have just done a fresh build of OBH for Duo2, which was fine.
But on first setup wizard, it crashes:

<   238.986> Traceback (most recent call last):
<   238.986>   File "/usr/lib/enigma2/python/Plugins/Extensions/LCD4linux/plugin.py", line 8569, in rcKeyPressed
<   239.001>     self.k = int(LCD4linux.KeyScreen.value[:3])
<   239.001> ValueError: invalid literal for int() with base 10: 'off'
<   239.001> [ePyObject] (CallObject(<bound method UpdateStatus.rcKeyPressed of <class 'Plugins.Extensions.LCD4linux.plugin.UpdateStatus'>>,(352, 0)) failed)


Man you guys are some BEASTS! I hope you get can this implemented soon, would make it that much safer as far as bans go (i believe). Thanks for all the hard work and determination. I know this is a fun/free time project for you guys, which in my eyes makes it that much amazing on what you have already done! CHEERS!


Kay. I will post later(before night maybe).


Weird, now it works. Ignore me for now.


# suggestions

I think this looks much more readable, especially if you can put the ` p => ...` in it's own function :)

## variables
Keeping variable declarations separate and at the top of a scope is a good idea. And keeping them each on a separate line makes it look much less cluttered.

## functions and reusability
Also if you use something twice or more it should probably be a method/variable.

``` cs
public List<Result> Query(Query query)
        {
            var results1;
            var results2;
            var result;

            results1 = _win32s.AsParallel().Select(p => p.Result(query.Search, _context.API));
            results2 = _uwps.AsParallel().Select(p => p.Result(query.Search, _context.API));
            result = results1.Concat(results2).Where(r => r.Score > 0).ToList();

            return result;
        }
```


@FernetMenta what do you think about this as a fix for the problem discovered here xbmc#10333 (comment) ?


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#YizwGJ1MTJipqdlThg2ntA


I think that the program didn't know what to do so just freaked out and gave me an error.


same as the last one.


@c-schmitz: This change breaks SPSS export. The exported file only contains the line:
`Database inconsistency error` (probably from line 101 of the same file).

 I am using the latest version and a PostgreSQL database.

Thanks, Ramon


@FernetMenta what do you think about this as a fix for the problem discovered here xbmc#10333 (comment) ?

(sorry for the noise with the previous commits which were just "accidents")


and agin.


@maths22 Use `after_script` or `before_deploy` with an if check if you don't want to call CPack all the time. As I mentioned on IRC (in private message?) dynamic file names are either supported using env variables or by defining `file_glob: true`: <https://stackoverflow.com/questions/25929225/how-to-deploy-to-github-with-file-pattern-on-travis>.


LOL


Looks like this was just released in doctr 1.2, so i was looking at newer-than-released source =)

https://github.com/drdoctr/doctr/blob/master/docs/changelog.rst#12-2016-08-29


@rufer7
I get an error here:
`Precondition failed: !string.IsNullOrWhiteSpace(condition)"`

Am I using it in a wrong order maybe?


TeamCity Glyssen :: Glyssen pull requests [Build *.*.226.1e1afe6843378bac831ed2a02d09261bf94a042c](http://build.palaso.org/viewLog.html?buildId=76223&buildTypeId=Glyssen_GlyssenPullRequests) outcome was **SUCCESS**
Summary: Tests passed: 1490, ignored: 17 Build time: 00:03:39


Documentation for `after_success`/`before_deploy`: <https://docs.travis-ci.com/user/customizing-the-build/>.


Major props to all the DEVs - You folks are doing an amazing job!


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 179](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=1178&buildTypeId=Xbim_XbimExchange_XbimExchange) is now running


ignore


io


Hello,
I am trying to use your widget and after this commit I get
ORIGINAL EXCEPTION: TypeError: this.textMaskInputElement is undefined

on load when adding to the page, as described in the tutorial
<input [textMask]="{mask: '(111) 111 1111'}" [(ngModel)]="myModel" type="text"/>


tried using something that was in a different file.


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 179](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=1178&buildTypeId=Xbim_XbimExchange_XbimExchange) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:12


A few more to do further up the file


@pbimpisidis you first have to execute the continue action, as there is no open state transition


`font-size`不要用奇数啊


Just FYI: For immutable lists without Guava (and without copying the elements), you can always use `Collections.unmodifiableList(list)`.


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


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


@curtwagner1984 Did you change your mind at some point? I was wondering why it was there the other day... =P


@lzjqsdd 最近确实比较忙，准备找工作，想赶点自己的项目，这部分还剩三分之一，这周末赶一下，九月初弄完。麻烦了


TeamCity OTA+ :: rvi_sota_server :: ci [Build 863](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=11884&buildTypeId=ota_plus_RviSotaServer_Ci) is now running