In cloudpebble.net (2015-11-29) this line fails to compile:

    ./src/gbitmap-sequence.c: In function 'init':
    ../src/gbitmap-sequence.c:70:3: error: implicit declaration of function 'window_set_fullscreen' [-Werror=implicit-function-declaration]

Full log:

    Setting top to                           : /tmp/tmpY3XDb2
    Setting out to                           : /tmp/tmpY3XDb2/build
    Found Pebble SDK for basalt in:          : /app/sdk3/Pebble/basalt
    Checking for program gcc,cc              : arm-none-eabi-gcc
    Checking for program ar                  : arm-none-eabi-gcc-ar
    'configure' finished successfully (0.044s)
    Waf: Entering directory `/tmp/tmpY3XDb2/build'
    [ 1/14] Start build for basalt:
    [ 2/14] subst: ../../app/sdk3/Pebble/common/pebble_app.ld.template -> build/basalt/pebble_app.ld.auto
    [ 4/14] appinfo.auto.c: appinfo.json -> build/basalt/appinfo.auto.c
    [ 4/14] layouts.json: appinfo.json -> build/basalt/layouts.json
    [ 5/14] resource_generator_raw: resources/data/animation.png -> build/basalt/resources/data/animation.png.ANIMATION.reso
    [ 6/14] resource_namespace: build/basalt/resources/data/animation.png.ANIMATION.reso -> build/basalt/src/resource_ids.auto.h build/basalt/app_resources.pbpack
    [ 7/14] c: build/basalt/appinfo.auto.c -> build/basalt/appinfo.auto.c.6.o
    [ 8/14] c: src/gbitmap-sequence.c -> build/src/gbitmap-sequence.c.6.o
    ../src/gbitmap-sequence.c: In function 'init':
    ../src/gbitmap-sequence.c:70:3: error: implicit declaration of function 'window_set_fullscreen' [-Werror=implicit-function-declaration]
    cc1: all warnings being treated as errors
    Waf: Leaving directory `/tmp/tmpY3XDb2/build'
    Build failed
     -> task in 'basalt/pebble-app.elf' failed (exit status 1):
        {task 140637815188816: c gbitmap-sequence.c -> gbitmap-sequence.c.6.o}
    ['arm-none-eabi-gcc', '-std=c99', '-mcpu=cortex-m3', '-mthumb', '-ffunction-sections', '-fdata-sections', '-g', '-Os', '-D_TIME_H_', '-Wall', '-Wextra', '-Werror', '-Wno-unused-parameter', '-Wno-error=unused-function', '-Wno-error=unused-variable', '-fPIE', '-I/tmp/app/sdk3/Pebble/basalt/include', '-I/app/sdk3/Pebble/basalt/include', '-I/tmp/tmpY3XDb2/build', '-I/tmp/tmpY3XDb2', '-I/tmp/tmpY3XDb2/build/src', '-I/tmp/tmpY3XDb2/src', '-I/tmp/tmpY3XDb2/build/basalt', '-I/tmp/tmpY3XDb2/basalt', '-DRELEASE', '-DPBL_PLATFORM_BASALT', '-DPBL_COLOR', '-DPBL_RECT', '-DPBL_SDK_3', '-D__FILE_NAME__="gbitmap-sequence.c"', '../src/gbitmap-sequence.c', '-c', '-o', 'src/gbitmap-sequence.c.6.o']
    Build failed.


Polished https://github.com/terasolunaorg/terasoluna-gfw/commit/88a08f85af565898d89cbf1a70a33a8e9c08b71c#diff-a200dec2d3f128d3c7f16ca5c7ca73f5R334


Polished https://github.com/terasolunaorg/terasoluna-gfw/commit/88a08f85af565898d89cbf1a70a33a8e9c08b71c#diff-a200dec2d3f128d3c7f16ca5c7ca73f5R378


* Hides teamchat during intermission.
* Shows chosen class as "location" during countdown.
* Integrates (DEAD) message and hiding spec location from previous enhancements.


Polished https://github.com/terasolunaorg/terasoluna-gfw/commit/88a08f85af565898d89cbf1a70a33a8e9c08b71c#diff-a200dec2d3f128d3c7f16ca5c7ca73f5R416


Looks like this hasn't been updated since that function was deprecated. I will remove it.


Thx for integration of my fix.


It is overkill because you just set a variable in a `:init` form which is executed right away, and since you use `:defer t` the package is even not required.
So basically the expanded form of the `use-package` is a `setq` executed directly and an empty `with-eval-after-load 'comint` (since you `:defer t` and you have no `:config` block).
Even if you did not `:defer t` use-package is overkill, it can be reduced to just:
 ```
(require 'comint)
(setq ....)
```


Or update it, rather.


//#include "stdafx.h"  //поменять местами первый отрицательный и последний положительный элемент массива
#include <iostream>
using namespace std;
const int n = 10;

void main()
{
	int mas[n], index_1_otr = n + 1, index_last_pol = n + 1, k;
	//Заполняем масси случайными числами от -n до n
	for (int i = 0; i < n; i++)
		mas[i] = rand() % (n + n) - n;
	//Вывод массива на экран
	for (int i = 0; i < n; i++)
		cout << mas[i] << ' ';
	cout << endl;
	//Находим первый отрицательный и последний положительный элемент массива
	for (int i = 0; i < n; i++)
		if (mas[i]<0)
		{
			index_1_otr = i;
			break;
		}
	for (int i = 0; i < n; i++)
		if (mas[i]>0)
			index_last_pol = i;
	//Меняем местами 1й отр и последний положительный
	if (index_1_otr == n + 1)
		cout << "1 negative element is missing" << endl;
	else if (index_last_pol == n + 1)
		cout << "last positive element is missing" << endl;
	else {
		k = mas[index_last_pol];
		mas[index_last_pol] = mas[index_1_otr];
		mas[index_1_otr] = k;
	}
	//Выводим на экран окончательную версию
	for (int i = 0; i < n; i++)
		cout << mas[i] << ' ';
	cout << endl;
	system("pause");
}


Not sure I like the way I basically wrote the same thing twice in two places, but it's because it applies to both program parameters and user function parameters and I can't quite tell which is the better place to put it.


Shoud DNS be DNS server? IS DNS sever more accurate?


Don't get me wrong, your usage of `use-package` is OK, but just overkill in my opinion, I prefer to expose the  Emacs core functions directly, this is some kind of sweet spot and totally subjective :-)


Please recompile Pummelchen.dll based on the instructions provided.


Always welcome :)


point all compute nodes to their service node. Make service node as the compute nodes' DNS server.


- This isn't change the original behavior, so I didn't add a test.
- Making a configurable timeout value seems good as well. But that's more like a feature for the Session Handler implementation.

My intention is to only give the handler creator a way to disable the timeout, e.g. local store or a store based REDIS.. Actually, I believe it is only the user knows if it needs a retry or not (based on the Session Store implementation.)


One issue I ran into without this is that it would require updating `generateHpcReport` to determine the correct tix filepath based on `testSuiteInterface` again as done above for `exeName`. Extending packageTests to have that extra information like you mentioned would help with this


there are some things that must be done for all of the options.


Nice test. I learned a little from it.


qqqqq


wwww


When you install xCAT, it runs ``makenetworks`` to create network definitions for the networks that the management node


Commenting it out works for me :-)

I've found some odd problems with the next sequence API - this demo app works fine BUT dumps out errors

    [ERROR] tmap_sequence.c:336: gbitmap_sequence failed to update bitmap
    [ERROR] tmap_sequence.c:219: APNG memory allocation failed

Probably outside the scope of this demo discussion though!


Yes, I'm in favor of this :+1:


done!!


WordWrap not wrapping anymore


This 2-line block is really the only thing I changed. It shows the full thing because of the change in indentation. Just so people don't have to look through the entire thing to figure out what I actually changed :smile:


:clap: :clap:


for what purpose


How does `nameTix` depend on `testSuiteInterface`?  Seems to me like it's just a function of `stestName` / `testName`


これは本当に致命的なバグです


```ocaml
String.make num_indent '\t'
```
is shorter and removes the need for recursion


* Set the **nameservers** and **forwarders** attributes in the xCAT site table. The **nameservers** attribute identifies the DNS server hostname/ip that the nodes point to in their **/etc/resolv.conf** files. The forwarders attribute is the DNS servers' hostname/ip that can resolve external hostnames. If you are running a DNS on the xCAT MN, it will use the forwarders DNS to resolve any hostnames that the MN can't.


is this necessary? these labels are never used, use rs or enum if you want structs


honestly kid ur off the project


hand in your badge


Thanks for the fix. Strange that we didn't stumble on this before...


On a "merge" commit - meeeh. But at least I got it :p


Edit **/etc/resolv.conf** to contain the cluster domain value you set in the site table's "domain" attribute above, and to point to the same DNS server you will be using for your nodes (if you are using DNS).


@haitgo 非常感谢。
我在想可否把您的这句说明加到文章里，或代码示例里。如可能，同时也可以反馈给官方？


Nice, I somehow missed that one. Fixed in c497ac6307e87e8d4c30a07c3040b2e8dd17356a


makes NUM_HOF_TEAMS kind of useless


repurpose this code for task manager project


Tag 打算做成大小写敏感的吗？


额。。。自己理解就行了吧，这又不是好高深的东西


```
FIRST_COMMIT=3ff1fd387266d995b7547504110c9ed274a48fca \
    git filter-branch --index-filter \
    'git rm --cached --ignore-unmatch config/secrets.py' \
    "$FIRST_COMMIT..HEAD"`
```
now with an actual commit to start ranging from....


This took us from 6.7GB to 27GB free. That's still not below 90% used :disappointed:


This file is under construction; I only pushed it now because I'll be at a different location tomorrow and want to work on it there. Much of what's currently there is just preparation and may end up elsewhere.


this is my code


Fixed tables of contents and images, please let me know if i missed anything.


The first error was `07:24:55 =!= | lua : script "matrix.lua" non trouvé`

then I remembered we just copied it to be "autoload", so the instructions should have been to `mkdir -p .weechat/lua/autoload/` first

Now I get this:

```
07:26:58     | lua : chargement du script
             | "/Users/fil/.weechat/lua/autoload/matrix.lua"
07:26:58 =!= | lua : impossible d'exécuter le fichier
             | "/Users/fil/.weechat/lua/autoload/matrix.lua"
07:26:58 =!= | lua : erreur : /Users/fil/.weechat/lua/autoload/matrix.lua:48:
             | module 'cjson' not found:
07:26:58     | no field package.preload['cjson']
07:26:58     | no file '/usr/local/share/lua/5.2/cjson.lua'
07:26:58     | no file '/usr/local/share/lua/5.2/cjson/init.lua'
07:26:58     | no file '/usr/local/lib/lua/5.2/cjson.lua'
07:26:58     | no file '/usr/local/lib/lua/5.2/cjson/init.lua'
07:26:58     | no file './cjson.lua'
07:26:58     | no file '/usr/local/lib/lua/5.2/cjson.so'
07:26:58     | no file '/usr/local/lib/lua/5.2/loadall.so'
07:26:58     | no file './cjson.so'
```

maybe https://github.com/keplerproject/luarocks/issues/94 explains it but I need to go now


Best commit


You might be interested in https://github.com/wincent/wincent/commit/b63a42c26a0a93c2685c3c1fa7f3f824faaffd52 too.


@titusfortner Have no idea, but I'll take a look.


That's pretty funny.


"Optionml"


`common` means it isn't related to `web`.


The test does not seem to match the error message: if the charset is empty no error is produced. This should probably be `charset == "" || strings.ToUpper(charset) != "UTF-8"`.


> I prefer following definition order.

Polished https://github.com/terasolunaorg/terasoluna-gfw/commit/afc4fdbb3d1030cf0677eab05661170b6acbbd57


Is this capability required by marionette?


ID not class


I don't think this section is needed here.

186  +Set site **forwarders** value to DNS servers outside of the cluster that can resolve non-cluster hostnames. The in-cluster DNS will automatically forward requests that it can't handle to these name servers: ::
 187  +
 188  +    chdef -t site forwarders=50.1.2.254,50.1.3.254


add help


Do we really need this? Can we make an expectation which does not depend on instance variables?


try without listing states


That's funny too :)


update with crosswalk master


add help about how it is computed


By Naresh


When the **/etc/resolv.conf** files for the compute nodes are created, the value of the **nameserver** in /etc/resolv.conf is gotten from MN's **site.nameservers** or **networks.nameservers** if it's specified. ::


I have reworked your effort into latest Grbl.  (0.9J)

https://github.com/grbl/grbl/pull/858


I would say no.


Added a test for this in https://github.com/jlongster/react-redux-universal-hot-example/pull/5


Fine with me.


Please add confignics documentation link here.


Java is in the same repo, and perl is all over the place... I'll do that.
Thanks!


У ребят из f32.me получше получились


Oops

> On Nov 29, 2015, at 22:36, Tim Dykes <notifications@github.com> wrote:
>
> ID not class
>
> —
> Reply to this email directly or view it on GitHub.
>


En mi explicación ignoren las lineas donde dicen:
```php
y
```
Eso solo significa que entre ```php y ``` se encuentra código php.


LGTM


And that's 2300 :p


Done in https://github.com/jlongster/react-redux-universal-hot-example/pull/6


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


Yay, sorry…


Noted, thanks!


Done, and I'm rewriting all commits.


@nsh87 this is the last of my tests for __measures.py__. I will be working through the rest of the similarity matrix stuff at the beginning of tomorrow now that all of the measures functions appear to be fine. Let me know if you see any errors in it.


good


Fixed in https://github.com/jlongster/react-redux-universal-hot-example/pull/7


I don't agree with this change, the timeout should be configurable. Having both timeout and retry flag is redundant. there is no test.


Applying fok-space to avoid conflict…


Have you downloaded media and display caf k30t? I forgot add this in manifest


Hello! Could you please explain, why in case when user is authenticated, but has no some particular role/permission, the request is considered as not authorized instead using of Access Forbidden?
I understand how to implement the asked behavior, but my interest is to understand why this behavior is not supported out of the box.


I am okay, as long as it has a way to disable the timeout, surely you can deprecate mine..


replace


U should spilit a big test into small tests


Bad abbreviations, making some of them too long. I don't think "pk" is a standard abbreviation for park; I'd just spell that out. I recommend:
fl.mainpkrd.wpt -> fl.mainparkrd.wpt
tx.mainpkrd.wpt -> tx.mainparkrd.wpt
tx.riogravlgrd.wpt -> tx.rgvilrd.wpt
tx.rossmaxscedr.wpt -> tx.rossmaxdr.wpt


this change does not worth a commit. could you squash it into the previous one?


因为按照字面的理解，这么做是会报错的。我觉得作为一个跟原本的叙述逻辑不太一致的东西，很有必要提点一下。


Hi, this has been changed in another commit. Renamed to Gateway.


ping++支付测试修改


0644 devrait peut-être être remplacé par le calcul umask du système ?


I think that the C++ implementation at http://git.io/v8T57 is ok, but if some error has been found, please, let me know  ;)


Hello, this is a git test file


I think it is also used in the unit tests.


Testing in production this evening


No, in fact it does not as `targetFilePath` points to upload relative path, i.e. uploading to http://server.net/upload will set it to `/upload`, which never will be resolved.

I removed the condition initially when checked the test to pass on the code without modifications, although I'm going to get it back so that no unnecessarily checks will be made in case of `CDV_TRANSFER_UPLOAD`.

Sorry for messing the commit history up.


上拉加载更多会加载2次


For those wondering, this does indeed work on Windows


Hi Morten :
    Much thanks for your sharing .
I use Visual Studio Community 2015 to open the project , there still have 6 errors after compile . And "event_to_device_service" still can't be opened .
![image](https://cloud.githubusercontent.com/assets/6659734/11465674/a8dd8e06-9775-11e5-9211-4b8dc2ec294a.png)

![image](https://cloud.githubusercontent.com/assets/6659734/11465616/46b9ce1a-9775-11e5-839c-b7d53a9d7107.png)
May I have comments , how to fix it or something I miss ?
Thanks .

Best regards,
David


gfgfd


TeamCity TripleA :: UnitTest [Build 242](http://home.k-moeller.dk/viewLog.html?buildId=315&buildTypeId=TripleA_UnitTest&guest=1) is now running


TeamCity TripleA :: UnitTest [Build 242](http://home.k-moeller.dk/viewLog.html?buildId=315&buildTypeId=TripleA_UnitTest&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 141 Build time: 00:00:34
### Failed tests
```
CashRegister.Test.Unit.dll: CashRegister.Test.Unit.Payment.PaymentControllerunitTest.Tally_StartChange1000_return1000: <no details avaliable>

```


TeamCity TripleA :: UnitTest [Build 243](http://home.k-moeller.dk/viewLog.html?buildId=316&buildTypeId=TripleA_UnitTest&guest=1) is now running


packageevaluator runs with tagged release versions of packages, so could you tag this before the next time I need to rerun it? thanks


now it added a reference to #2, heh


a


I re-used the existing logic to add `rslave` but to support the other recursive variants it might make sense to changes this logic.


TeamCity TripleA :: UnitTest [Build 243](http://home.k-moeller.dk/viewLog.html?buildId=316&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 142 Build time: 00:00:34


Fixed some shizzle.
@WouterSioen what is the best way to fetch the thrown error from ->get() in MailMotorMember->exists()?


TeamCity TripleA :: UnitTest [Build 244](http://home.k-moeller.dk/viewLog.html?buildId=317&buildTypeId=TripleA_UnitTest&guest=1) is now running


Sure, I will do it. Thanks.


TeamCity TripleA :: UnitTest [Build 244](http://home.k-moeller.dk/viewLog.html?buildId=317&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 142 Build time: 00:00:32


I'll make sure that I do.


TeamCity TripleA :: UnitTest [Build 245](http://home.k-moeller.dk/viewLog.html?buildId=318&buildTypeId=TripleA_UnitTest&guest=1) is now running


TeamCity TripleA :: UnitTest [Build 245](http://home.k-moeller.dk/viewLog.html?buildId=318&buildTypeId=TripleA_UnitTest&guest=1) outcome was **FAILURE**
Summary: NUnit error running tests in 'C:\TeamCity\buildAgent\work\71a2ca477f86049a\Software\TripleA\CashRegister.Test.Unit\bin\Debug\CashRegister.Test.Unit.dll' assembly
Loading assembly is compiled for v4.0.30319, MSIL
NUnit runner runtime is v4.0.30319, x6... Build time: 00:00:22


make search for each line and check plan_id too


啊...？

Wildsky Fann <notifications@github.com> 於 2015年11月30日 週一 下午1:01寫道：

> @RexYuan <https://github.com/RexYuan> 不要忘了盤古之白呀！
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/communityNTNU/surviveNTNU/commit/f5d5c74c321a7e7471462aa2541f2d9efb9ed268#commitcomment-14672684>
> .
>


Whoops, this also includes the translation for "Division last edited" to Ukrainian. That's missing from the commit message :S sorry! :hurtrealbad:


effect


Nice.


That was awesome!!!


tlktojk2[yojl  gpji5[1tj
t[ipk p'toj50[ti1t 'protju5tyui1]=3[t '[pruj-]5t0jkhjn'gkj


give me please test.cpp, test.h


revertig


Note: still need to implement some overloaded operators (+, +=, ==, etc.) and also need to add move operators.


Doesn't this need to be a can.dev.warn?


Thanks!


this too


and this


Actually this reading the output here would give us fsize, gotta squash these two lines together.


more love


Fixed in 0a5638f


誒？我哪裡沒加到嗎


Shouldn't we use a pthread_mutex_t here instead?


commit message XD
下次記得加就好～


自学swift的大专毕业生，大码写得不怎么滴


Hj


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.596.70e2182df8cf6e1a697e05a06a501ddd0945d633](http://build.palaso.org/viewLog.html?buildId=66674&buildTypeId=bt322) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 2149, ignored: 89 Build time: 00:16:57
### Failed tests
```
SIL.Windows.Forms.Keyboarding.Tests.dll: SIL.Windows.Forms.Keyboarding.Tests.XkbKeyboardAdapterTests: <no details avaliable>

```


бред какой-то


I know!! Can't believe it myself lol


На мой взгляд - хуже. Окошко у аккумулятора в оригинале уже.


I think the GPSR runs a custom middleware.launch that includes the reasoner. @svddries probably knows.
But I found this error running $ rosrun challenge_gpsr gpsr.py mockbot, so I think its good to run that sort of test before pushing to master.


Reviewed


TeamCity TripleA :: UnitTest [Build 246](http://home.k-moeller.dk/viewLog.html?buildId=319&buildTypeId=TripleA_UnitTest&guest=1) is now running


TeamCity TripleA :: UnitTest [Build 246](http://home.k-moeller.dk/viewLog.html?buildId=319&buildTypeId=TripleA_UnitTest&guest=1) outcome was **FAILURE**
Summary: NUnit error running tests in 'C:\TeamCity\buildAgent\work\71a2ca477f86049a\Software\TripleA\CashRegister.Test.Unit\bin\Debug\CashRegister.Test.Unit.dll' assembly
Loading assembly is compiled for v4.0.30319, MSIL
NUnit runner runtime is v4.0.30319, x6... Build time: 00:00:22


Brady I think you missed the next line, it is already replaced with strip_escape_custom() :)


Implementing the RepositoryContent in a simplified way would not be so much a problem.
However, the features Deployer and related code do manage downloads in a certain way, so that bundle downloads can be started asynchronously, cached locally, etc... (this is done using the DownloadManager).
I don't think we should bypass this processing, so implementing RepositoryContent would not bring much imho.


Why is it "safeDealias" and which one should I normally choose? If this is for subtyping only it should go in TypeComparer or people will get confused when they look at Types. We are probably going to refactor TypeComparer anyway in the near future. Let's keep this change in mind to possibly roll it in then.


@torkelo @mtanda is there a plan for custom metrics?

I've been using the original CloudWatch data source plus a cronjob that calls `aws cloudwatch list-metrics` and it's been pretty effective.

Downside is that configuring the location of the json file is a little strange from the web UI, plus you also need the aws-cli installed.

Another thought would be to just do it in memory via `time.NewTicker`. You're making more calls as you scale up grafana instances, but at once per hour I doubt it'd incur much cost.

I'll plan to stay connected to #grafana over the next couple weeks and maybe we can catch up to talk it out?


same here :)


Doesn't this fail with a KeyError if the screenplay has no title? I think it should be `if 'Title' in screenplay.title_page`.


One question:

In what __order__  `@:build` macros will be called? Same as metas or?..


TeamCity TripleA :: UnitTest [Build 247](http://home.k-moeller.dk/viewLog.html?buildId=320&buildTypeId=TripleA_UnitTest&guest=1) is now running


just noticed that the exporthow.t test is bogus it should be testing declaring a controller in the EVAL.


TeamCity TripleA :: UnitTest [Build 247](http://home.k-moeller.dk/viewLog.html?buildId=320&buildTypeId=TripleA_UnitTest&guest=1) outcome was **FAILURE**
Summary: NUnit error running tests in 'C:\TeamCity\buildAgent\work\71a2ca477f86049a\Software\TripleA\CashRegister.Test.Unit\bin\Debug\CashRegister.Test.Unit.dll' assembly
Loading assembly is compiled for v4.0.30319, MSIL
NUnit runner runtime is v4.0.30319, x6... Build time: 00:00:21


@uzysjung  혹시 tv 동작 안하던데.. 추가하신건지요?


should by LonLong


already modify the value format to use WT_ITEM.


Run 바로 안되는거 수정해봤습니다. 잘되나 확인해보세요


ECAL gap is now included by default, excluded for 2012 ee.
2015 ee ntuples should be recreated. Please check electron eta distributions for discrepancies in the EB-EE transition region (|eta|~1.5)


update of doxy file + descriptor


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


would require QVariant::DateTime too
would be nice to add QVariant::Date with another icon maybe (I did not see the date icon yet though)


What changes?

2015-11-30 4:06 GMT+01:00 Zoltan Grose <notifications@github.com>:

> 6.2.2 vs 6.3.1 (note the white box in 6.2.2)
>
> [image: screen shot 2015-11-29 at 7 03 46 pm]
> <https://cloud.githubusercontent.com/assets/5044847/11462670/3802f67a-96cc-11e5-880a-a1dbfa338e6e.png>
>
> [image: screen shot 2015-11-29 at 7 04 34 pm]
> <https://cloud.githubusercontent.com/assets/5044847/11462676/3da1667a-96cc-11e5-9fae-cb5be4b7203e.png>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vasl-developers/vasl/commit/f91804a6de52a55724a17036eedb861491f65e2d#commitcomment-14672088>
> .
>


@alexbruy thanks for this.
Please see the 2 comments and pull a  request afterwards.
you can assign me for review :)

best wishes


@sompylasar - I tried reproducing the server hang issue you mentioned - to no avail. How did you do it?


btw, are you sure the icons use the same text font?


[`generateHpcReport`](https://github.com/commercialhaskell/stack/blob/8c0228532853597490285a5d7b075bf3bc6a9b92/src/Stack/Build/Execute.hs#L1240) only has access to test names (in ex. `testname`) and for detailed-09 test the tix file has to take into account the generated testStub filepath.  I remember changing only `nameTix` to use either name resulted in a broken `updateTix` or `generateHpcReport`

The env var was a quick way to have guarantee of the filepath to get it work, but extending packageTests to have that extra information would help out updating `Coverage` to better support different test-suites


Нужно один раз запустить скрипт add_quadkeys.php


Could you please publish the 3.4.2 release to NPM?


Andi, are you doing your own tests with binutils from
https://www.kernel.org/pub/linux/devel/binutils/
or
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;a=commit;h=cb8abd63853c3b8b74e17d7d12a8e5ed04926688
?

Because as far as I can understand, your work on Linux & LTO doesn't work on latest binutils-hg.
I have to override LD with scripts/gcc-ld to avoid plugin error and even if I do so, I get lots of linking error on both x86, arm, arm64.

I kindly request you to test this with latest binutils-hg.
I'll add error log if you want one.


`luarocks install lua-cjson` allows me to start matrix


Now I realize that I was confusing testPackages as being of some type of `NamedComponents`... Hmm, so it still requires to use `resolvePackage` to have that information available so I guess using the env var looks like a cleaner solution :P


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


ESP8266 not ESP9266


Ah! It's enabled for pure aesthetic reasons, but I've made it work with a bit of a hack. You can set the `Error Document` of the S3 bucket's website hosting to `index.html`, which will make all 404s etc serve index.html:

![image](https://cloud.githubusercontent.com/assets/903976/11466220/a64f27fc-973f-11e5-8a44-bcc322547b77.png)

 I realize this isn't part of the instructions in the readme so I'll update that!


Oops. I keep doing that. Must just have a single "git push and publish" script, or maybe a github hook


Début Série 6.


@pilz I think it might be a good idea to restrict the listener to only listen to `pat-update` if it was thrown by a  `pat-stacks` parent.

Like so:
```
$container.parents('.pat-stacks').on("pat-update", null, this, utils.debounce(equaliser._onEvent, 100));
```

I didn't test the above line, but it should work.


Ok...I'll partial revert it, retaining the ShortcutPickHelper


Great Stuff Igor..It works as a charm..Any Error messages are now being upfront shown on the console.

Many Thanks
Simran


It's true.


@GrahamCampbell can you merge?


It's pretty late here, I'll take a look after work tomorrow though.


Please help review the code. Thanks a lot~
@Richard-Yunchao


This triggers a lint error:

[0] /Users/glennr/src/glennr/react-redux-universal-hot-example/src/client.js
[0]   42:31  error  'location' is missing in props validation  react/prop-types
[0]   42:47  error  'params' is missing in props validation    react/prop-types


#  a layer for mobile


Fais un petit effort sur le message de commit :p


This needs to be removed - it's a test dependency.


Cmake policy CMP0042 was introduced in cmake version 3.0.  I have cmake version 2.8.12.2

I have removed that line from CMakelists.txt

If you change any of these files in the future please make sure you are only using stuff introduced in 2.8.12.2 and below.


Dont do logic in templates. Always call controller's methods. Otherwise such code is hard to follow and test.

Just move this logic to isDeployDisabled() (and rename is needed)


здесь не могу вспомнить для чего, возможно действительно лишний


без него работать ничего не будет. код который открывает всё находится как раз в том модуле. там просто была переделана целая функция и в эту пределанную функцию я добавил код обрабатывающий открытие всего


You're always allowed, but sometimes plain old JS is better. E.g., directive definition objects: this should be a function that returns an object. But in case of callbacks or when you `.bind(this)`, use arrow functions which are harder to introduce bug in.


fixed :)


Во, так лучше :)
В идеале жанры должны разделяться фильмом в его конструкторе (т.е. в методе initialize — чтобы `movie.genres` всегда были массив)


Okay


This looks like an unrelated change.  Why is --disable-static removed?


A explained in the ticket.... https://tracker.phpbb.com/browse/PHPBB3-14234


Action Plans index.html


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


Looks good


`sort_by(&@@hsh_of_sorts[key_block])` :)
Или, если это выглядит слишком запутанным:
```ruby
block = @@hsh_of_sorts[key_block]
sort_by(&block)
```


yes, it is a copy-paste error.


*cant...  cool!!


I changed variables
```
    int ttl_jobs = 500;
    int per_thread = 30;
```
and test passed without errors.

But there are some other "windows specific errors":
```
whenURLendsWithPercentDoNotExcape(com.openpojo.reflection.java.packageloader.impl.URLToFileSystemAdapterTest)  Time elapsed: 0.016 sec  <<< FAILURE!
org.junit.ComparisonFailure: expected:<[/]apps%> but was:<[D:\]apps%>
	at org.junit.Assert.assertEquals(Assert.java:115)
	at org.junit.Assert.assertEquals(Assert.java:144)
	at com.openpojo.reflection.java.packageloader.impl.URLToFileSystemAdapterTest.validateURLtoExpectedFilePath(URLToFileSystemAdapterTest.java:66)
	at com.openpojo.reflection.java.packageloader.impl.URLToFileSystemAdapterTest.whenURLendsWithPercentDoNotExcape(URLToFileSystemAdapterTest.java:49)
```


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


Updated to provide the missing teleport/set pose feature


Move all logic to the `isDeployDisabled` method.


А зачем этот if? Мне почему-то кажется, что можно обойтись и без него.


They're actually lists of associations already! :)


This method now looks very nice :)


Good catch! Will make note of it. :+1:


ca marche, fix small typo in comments ?


Should you do `|| -1`? Then, in case this returns 0 you'd default to -1. I'm not sure this is correct. According to docs, the method returns -1 anyway when the search character is not found.


@bryk: Is there a way to pass form object without giving it a name? I'm just wondering.


These constants are never used here. Nobody cares about them in this file. How about moving them to the controller (where they are used) and adding a quick comment why we need them?


Delete newline?


Right


Cool so there was no need to parse the cabal file _again_. `package` already has that information after updating `packageTests` to a `Map Text TestSuiteInterface`


我晕........你手脚真麻利.......我还在重新写这里，因为tag原先的设计有问题，无法支持tag/tagId的浏览


Cool


created function in formdata.inc.php

function check_magic_quotes() {
  return get_magic_quotes_gpc();
}


Don't need `@class`. You can, however, add `@final`. It is a good practice for model classes and in general when you do not design for inheritance.


'correct_user' doesn't seem to be a proper name for this method. `current_user_post` sounds better


Fwiw, the current debian stable (jessie) ships maven 3.0.5..... so i'm not sure this was a wise move.


Why do u need that empty file?


Perfect, looks nice. A few comments and let's merge this.


Propose to rename "error" -> "usage". Also propose to use QT by default. As far as I understand this is the main platform.


I don't see any reason why i've added this. I guess during refactoring this method has changed somehow and this became wrong. I'll fix it.


Alerts and notices should be at the top, between header and main content


"My przedstawimy tutaj kilka z nich, gdyż elegancko są one opisane na manualu! Wybacz nam, ale musimy nauczyć Cię jak korzystać z manuala i szukać w nim informacji."
Nie podobuje mi się :( Można dać odnośnik do manuala, ale nie okazywać tego tak publicznie.
W zamian można zrobić mini rozdział albo do wstępu info dać o tym, że wiele informajci będzie odnosiło się do manuala


No. Give it a name "ctrl.deployForm" (or similar) and in controller write `/** @export (..type..) Initialized from the template */ this.deployForm`. With this you'd have access to the controller


Fixed


Merci :). Ça fait plaisir de voir des gens s'intéresser à la prémod en tout cas


static pages shouldn't include any logic


Added QVariant::DateTime with corresponding (still ugly) icon.
All new icons are there, don't know why GitHub does not show them as icons, instead an SVG XML is displayed.


Might want to use kind_of? here to stay consistent with the rest of the code in this method. And if there ever would be a subclass of Project, the check would still work


Pourquoi avoir supprimé cette ligne ? Le fonctionnement n'est plus celui attendu.


@dockimbel I wonder if it would be safe to move this line after the block bellow and remove the head resetting after insert call.


because u're using this method only from ```current_user```, u don't need it. The same result can be achieved by just ```current_user.posts```


Thanks.


Just checked, and all icons with text use DejaVu Sans Mono, 9pt


@bryk: Sure. My thought was that model related const should be kept near model class.


I must say that I'm surprised by the outcome of these tuning sessions.


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


this is really really weird: if I comment out the `'test'` event firing the following one (`'createConnectedUsersList'`) doesn't produce any effect on the client. It is even more weird considering that the response to `'test'` event on the client is commented out!!


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 630](http://build.prxm.it:8811/viewLog.html?buildId=4993&buildTypeId=Jarvis_DocumentStore_CI) is now running


There's an error message from getopts itself:

    $ ./aa2mp3.sh -a
    ./aa2mp3.sh: illegal option -- a


Yes, see the test case.


Israel:
En el caso de Python puedes mejorar el desempeño con estos cambios:

def compute_fitness(chromosome):
    return sum(chromosome)

def mutate(chromosome):
    return chromosome.invert()

En el mutate actual no mutas todos los bits solo uno.
mutie[mutation_point] = not mutie[mutation_point]

Por último en el onemax.py cuentas el tiempo de generación del cromosoma random,
vi que nadie considera este tiempo, solo el del calculo del fitness.


Added the check back in fe81475829c08ce6be6a393691e5ee59d0246c0f


I don't get why here I should use `socket.emit` instead of `io.emit` (which doesn't work)...
I think we should talk about this on Tueasday.


Ich bin immer noch für die Suffix-Variante ^^


Quelque chose comme : `Post : Grid layout - Fix typo in comments`.


I don't think that U need to rewrite this method of ```warden``` to login the user


Whoopsie, sorry.


Hi, I am not sure what is the actual fix there. Could you elaborate?


IMO it would've been safer for these to go in a `finally` block in case someone in the future decide to catch and ignore possible exception


Better safe than sorry...


Rather use protoc-gen-combo

```
regenerate:
	go install github.com/gogo/protobuf/protoc-gen-combo
	protoc-gen-combo --version="3.0.0" --proto_path=../../../../../:../../protobuf/:. castvalue.proto
```

That way we get every combination of the marshaler and unmarshaler tests generated.
Also the code is only regenerated if the protoc version on the machine is above 3.0.0, which allows all the tests to always run given every combination of go and protoc.


References:
[(Stack overflow)How to set up factory in FactoryGirl with has_many association (JellicleCat's answer)](http://stackoverflow.com/questions/6963298/how-to-set-up-factory-in-factorygirl-with-has-many-association)
[(Thoughtbot)Remove Duplication with FactoryGirl's Traits](https://robots.thoughtbot.com/remove-duplication-with-factorygirls-traits)
[Factory Girl - Getting Started](https://github.com/thoughtbot/factory_girl/blob/master/GETTING_STARTED.md)


Please use same format as the one used for computing '#today' key.
date('j.F.l')


LGTM


I do not think that catching a generic RuntimeException in this place is the right approach to fixing this problem. If we want to handle the unexpected Record content we should do so in the processHandshakeRecord and throw a HandshakeException, don't you think?


Agree about usage, Regarding default configuration, I actually never heard about that. SO propose to leave as is


Added https://github.com/terasolunaorg/terasoluna-gfw/commit/a6743a2f94590c5d4d1fa8069312dc50ac4535ae


I do not understand how to set the filter the first time if your layer is invalid.


I meant with respect to the labeltext icon.
It would be nice to have all 4 consistent.
Can you maybe redo the one for the text and name it accordingly to the others?


Removed default value https://github.com/terasolunaorg/terasoluna-gfw/commit/a6743a2f94590c5d4d1fa8069312dc50ac4535ae


Jeez. I really need a way to make VS annoy me about these.


Yes, it would be better, but such analysis needs to use as less code as possible. So some good upper bound estimations would be better than accurate calculations.


This is good, but for linux builds you need to remove it from global options, too.
```diff
diff --git a/build.py b/build.py
index e04aac5..d16add5 100755
--- a/build.py
+++ b/build.py
@@ -240,7 +240,6 @@ class PhantomJSBuilder(object):

         configure = [configureExe,
             "-static",
-            "-static-runtime",
             "-opensource",
             "-confirm-license",
             # we use an in-source build for now and never want to install
```


`crowbar_databags_path.join(file).read` ;)


@donnykurnia Hard-coded style is exactly one of the core ideas of JSLint. From the README: "Conforming to a consistent style improves readability, and frees you to express yourself in ways that matter. JSLint here plays the part of a stern but benevolent editor, helping you to get the style right so that you can focus your creative energy where it is most needed."

It would be counter to this philosophy to provide options allow you to enforce your own custom style. JSHint/ESLint/JSCS allow you to do that. JSLint instead makes all of its users stick to a single style. I agree with JSLint's stance. In my opinion, there is no value in having more than one coding style.


Is this approach correct?
I am passing the `room` collection and store it as an attribute of the `chatModel`, together with the `chatMessages` array (which is an attribute of the same collection); then I'm creating a view for both the room (which is the list of connected users) and the chat messages.
Am I actually passing the right objects to the model/views? I don't understand even if I'm logging everything out...


So the bug was because pip repository updated pylint version?


Regarding Qt as the default I am not sure so lets keep as it is


Add `PARAMETER` https://github.com/terasolunaorg/terasoluna-gfw/commit/a6743a2f94590c5d4d1fa8069312dc50ac4535ae

As for `CONSTRUCTOR`, I don't think adding it makes sense because the usecase is really rare. `CONSTRUCTOR` would not be used unless `CharSequence` is extended.


No, it would not be safe, as the `/head` field could contain garbage, so setting it before calling any action is mandatory.


it was on both line, sorry....


@manojo -- this breaks some of the other HTTP parse tests. Did you intend this tweak to be optional or should the other check files be updated?


This is the old commit. I did a force push. Check the fresh sources: https://github.com/ariya/phantomjs/blob/master/build.py#L242


You are missing slot type (re)setting: `str/header: TYPE_STRING`


@mariosky me han hecho tantos cambios que ya no sé bien ni lo que hace.

Una pregunta: si hago mutate de esa forma... no estaría ejecutando el mismo que en el resto de versiones... ¿algún problema con eso @JJ ?

Gracias @mariosky


right. and actually the entire logic of the method was not okay after having a second look. should be much better now.


Hmm does `testsToRun == Map.keys (packageTests package)` ?

That way packageTests can be passed to `Coverage` functions instead to handle tix file manipulation correctly


@bryk: Ohh... I know why it's here. When you are filling the container image field and then erase all fields it becames undefined and there is an error in console saying that 'can not read property lastIndexOf of undefined'. This way there is no error because -1 is assigned.


m_objectId is required for cases where the original object uml has been destroyed and need to stay.


![screen shot 2015-11-30 at 12 34 27 am](https://cloud.githubusercontent.com/assets/2508142/11467112/e2491c08-96fa-11e5-9569-bd208eb5f4c9.png)
![screen shot 2015-11-30 at 12 35 48 am](https://cloud.githubusercontent.com/assets/2508142/11467115/e40ed898-96fa-11e5-8f99-332b6897ea07.png)


The object may be destroyed on perfoming the undo action. To avoid dangled pointer please use QPointer<UMLObject> obj.


Little derp here :)


Are there any cases where we want to show fields that are not quantities? Pivot tables, charts, maps, should all only include quantity fields.


I have to fix this anyway because when container image filed get erased after filling it then this.containerImage becames undefined.


can with program detect key press or hold??how??


1) please rebase your change
2) set response = False

apart from that LGTM, good catch!


Hey, how about you write a test for this? This would be a nice quick exercise for you. And we should start writing tests anyway, because the codebase is stabilizing (e.g., I'm writing tests for brackend at the moment :) )


hey, it's already it the https://github.com/redmaner/MA-XML-CHECK-RESOURCES/blob/master/MIUI7_auto_ignorelist.xml,  ?


This need to be performed in the following order:
1. Try to find the umlObject by searching m_objectId
2. if not found return obj


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


produces a cycle in the build path (on master OH2 with master ESH), when I leave it out, it compiles.


I'm not sure whether this is model related constatnt or controller constant. I see this like: this is a model class - it does not care how it is used. Then there is controller which has special case for name and version and it uses it in a special way.


Hi,
This item and items of this nature should not be translated. Note should only be translating Labels.
-brady


@arantius

![comment](https://cloud.githubusercontent.com/assets/2373486/11467194/be794572-9746-11e5-9d9f-56b3e154928b.png)


Build error
make[3]: *** No rule to make target `all'.  Stop.


can you also add the QVariant::time case?

for icons, what do you think of:
for date and datetime: http://uxrepo.com/icon/calendar-by-icomoon ?
for time: http://uxrepo.com/icon/clock-by-modernpics ?


Should definitely not be translating this and other things of this nature.


Ye, that's a good idea. ;)
I've changed this to `(this.containerImage || '').lastIndexOf(':');` This way there will be no undefined error.


Ah, ok... Sorry, missed that. Thanks a lot.


Hello
Your JSManipulate - Javascript Image Filters & Effects project is too great.

But I want to convert this source projects to PHP.

Please Help me
Plase.


super


lol, missed that


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 630](http://build.prxm.it:8811/viewLog.html?buildId=4993&buildTypeId=Jarvis_DocumentStore_CI) outcome was **FAILURE**
Summary: Tests failed: 6 (5 new), passed: 296 Build time: 00:18:07
### Failed tests
```
Jarvis.DocumentStore.Tests.dll: Jarvis.DocumentStore.Tests.SelfHostIntegratonTests.DocumentControllerIntegrationTests("v1").can_upload_document_with_name_greater_than_250_char: <no details avaliable>

Jarvis.DocumentStore.Tests.dll: Jarvis.DocumentStore.Tests.SelfHostIntegratonTests.DocumentControllerIntegrationTests("v1").verify_de_duplication__not_delete_original_blob_before_15_days: <no details avaliable>

Jarvis.DocumentStore.Tests.dll: Jarvis.DocumentStore.Tests.SelfHostIntegratonTests.DocumentControllerIntegrationTests("v2").can_upload_document_with_name_greater_than_250_char: <no details avaliable>

Jarvis.DocumentStore.Tests.dll: Jarvis.DocumentStore.Tests.SelfHostIntegratonTests.integration_out_of_process_tika_long_name("v1").verify_tika_job_with_long_name: <no details avaliable>

Jarvis.DocumentStore.Tests.dll: Jarvis.DocumentStore.Tests.SelfHostIntegratonTests.integration_out_of_process_tika_long_name("v2").verify_tika_job_with_long_name: <no details avaliable>

Jarvis.DocumentStore.Tests.dll: Jarvis.DocumentStore.Tests.SelfHostIntegratonTests.integration_out_of_process_tika_password("v1").verify_tika_job_with_password: <no details avaliable>

```


tst


again lol, missed that :)


s/endup/end up/


Why if I emit the event here, when the room should be not empty, I get this error?
![screen shot 2015-11-29 at 7 22 02 pm](https://cloud.githubusercontent.com/assets/12717225/11467058/8410133a-96fa-11e5-9d2e-6b21f2505e8d.png)


ah right :)


@alexbruy are you ok if I push your stuff to master with new icons?
might be faster ;)


looks good


just '/index' route is not REST'ful . U can use '/users' instead which was defined above by ```resources :users, :only => [:show, :index]```


Do not remove m_objectID, the object may be destroyed on perfoming the undo action.


Oh sorry! Thx, now I see it :)


And no space after the .


Maybe get_network_node_from_neutron_attributes? Horrible, I know :-)


Sure, feel free to push it.
Regarding font, i used existing "labeltext.svg" as base and font is the same in all icons. BTW, pull-request already opened


setItemRegistry and getItemRegistry as mentioned in persistenceresource.xml?


hey, its like comment, but not the comment sign here.....


상수를 하드코딩말고 define 된 수를 쓰는것이 좋을듯...


On Sun, Nov 29, 2015 at 11:52:09PM -0800, Park Ju Hyung wrote:
> Andi, are you doing your own tests with binutils from
> https://www.kernel.org/pub/linux/devel/binutils/
> or
> https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;a=commit;h=cb8abd63853c3b8b74e17d7d12a8e5ed04926688
> ?
>
> Because as far as I can understand, your work on Linux & LTO doesn't work on latest binutils-hg.
> I have to override LD with scripts/gcc-ld to avoid plugin error and even if I do so, I get lots of linking error on both x86, arm, arm64.
>
> I kindly request you to test this with latest binutils-hg.
> I'll add error log if you want one.

The Linux binutils (the first link) are needed. The FSF binutils are
not supported for LTO because they don't support ld -r for LTO properly.

Note that the compiler also needs to use the correct linker as plugin-ld
(there's no way to override this at runtime short of replacing the
original linker or rebuilding the compiler)

-Andi

--
ak@linux.intel.com -- Speaking for myself only.


Idea was to set it like that.

Put defines at function start (cpufreq_powerstats_create)
#ifdef CONFIG_OF
	struct device_node *cpu_node;
	char device_path[16];
#endif

#ifdef CONFIG_OF
snprintf(device_path, sizeof(device_path), "/cpus/cpu@%d", cpu);
cpu_node = of_find_node_by_path(device_path);
	if (cpu_node) {
		ret = of_property_read_u32_array(cpu_node, "current",
				powerstats->curr, count);
		if (ret) {
			kfree(powerstats->curr);
			kfree(powerstats);
			powerstats = NULL;
		}
	}
#else
kfree(powerstats->curr);
kfree(powerstats);
powerstats = NULL;
#endif


NUM_OF_DATA 가 30으로 정의되어있는데 카운트는 60까지 셈.. define의 수를 바꾸거나 해야되지 않을까?


Again in file puppet-nginx/spec/classes/service_spec.rb, restart command is being used. Shouldn't there too we need to make this change.

As in case upstream is wrong still shoots error as  execution of /etc/init.d/nginx restart command failed. That pointed out issue.

Please have a look.
Thanks,
Pallav Joshi.


Hey John,
It seems you left out the autotranslate(me) on the load event of dlgChisqure. Consider having it in.
Thanks


While I agree that using non-minified file is better for dev purposes, most of the guys I know (myself included) don't ever really bother to check what bower installed so it might be good to leave a smaller version for install-and-forget purposes.

On the other hand, most people who use bower also use CSS magnification of all bower files through grunt/gulp/webpack/other tasks when building for production, so yeah this might be a good idea.


Isn't it basically moved from https://www.kernel.org/pub/linux/devel/binutils/ ?

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;a=blob;f=release.binutils-2.25.51.0.4;h=623c48e326cdebfc52c85718e815f2beeff22741;hb=cb8abd63853c3b8b74e17d7d12a8e5ed04926688 indicates it's purely for Linux.


Because u've put a unique constraint on ```name``` column, u also need to put a validation inside User model, like ```validate_uniqueness_of :name``` to display this error for the end user. Right now it will just raise database error and user won't understand what's going on


[TestResults.pdf](https://github.com/Brel-Egor/BBorders/files/47081/TestResults.pdf) Приложение работае без каких-дибо проблем на Android версии 5.0.1


Yes, it is possible. Available is not precise name. Primary purpose is to decide if to start new iteration, maximum_iteration_start_time would be more precise but too long.


disabled code is bad because people don't know what to do with it, also it is not checked for syntactical correctness. How about using proper logging with different levels and possibility to en-/disable id?


(He corregit l'enllaç)


I think we can break this out into it's own .scss file, make the structure a little more modular


[TestResults.docx](https://github.com/ArtuhovichVladislav/SoundTube/files/47087/TestResults.docx)
Тестирование проведено, особых замечаний выявлено не было. Один тест провести не удалось из-за техгических неполадок.


Maybe we should add FBOX here too? Just for consistency and to not run into trouble if and when those get start to get used again?


In any case the layer is not deleted, although I didn't try what happens in this one. I'll check it this evening, AFAICS there's no ownership sip annotation for QgsQueryBuilder ctor.

If dlg has neither parent nor Qt::DeleteOnClose set, probably it won't be deleted as well.


On Mon, Nov 30, 2015 at 12:51:21AM -0800, Park Ju Hyung wrote:
> Isn't it basically moved from https://www.kernel.org/pub/linux/devel/binutils/ ?
>
> https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;a=blob;f=release.binutils-2.25.51.0.4;h=623c48e326cdebfc52c85718e815f2beeff22741;hb=cb8abd63853c3b8b74e17d7d12a8e5ed04926688 indicates it's purely for Linux.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/andikleen/linux-misc/commit/0ec62f0db60ed1025df5ad4422604718c1460cef#commitcomment-14675596

Ok you're right. It's the Linux binutils tree.

I currently use an older version (two years old or so), not testing the
latest versions.  Maybe something broke. I'll check.

-Andi

--
ak@linux.intel.com -- Speaking for myself only.


Yeah, keeping them as two separate views is starting to create naming inconsistencies among our URLs...I still like the idea of keeping them both under /reports. Thoughts?


@jebbdomingo, this seems to be vulnerable to a js gotcha called "var hoisting" (https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/var) . If more than one ckeditor fields are initialized, all the var declarations will be hoisted on top, that means prior `defaultConfig` and `prefferedConfig` values will always be overwritten by latter var declarations for those variables.


As mentioned below, naming inconsistencies are starting to pop up. Why is there a 'report' state, and also a 'reportview' state, you know?


@yogaswara gistha ini di update ke semua template ya..


That's kind of old habit from java where if we have a predefined object then we can store it as static within the class. Something like: BigInteger.ZERO.


On peut obtenir le même résultat en ajoutant la classe "hidden-xs" dans le DOM.


look like it's too high here, my phone shutdown after 4 hour of full charged


Yup, I know now that rebasing can be "dangerous" if a commit has already
been pushed out to a shared audience.  I wanted to blog about this some
time, but I couldn't fit that blog post into a day :P

On Mon, Nov 30, 2015 at 2:19 AM, Don Okuda <notifications@github.com> wrote:

> @eric-hu <https://github.com/eric-hu> Yeah, I did some rebasing and
> merging which gave this commit a new SHA. I think GitHub sent you a
> notification because I mentioned you in the commit message and it thought
> this was a new commit.
>
> lol rebasing.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/donokuda/donfiles/commit/f3440081e1e259ae38db4d1bf6b8ff3849bd0938#commitcomment-14669812>
> .
>


Technically that wasn't a fail-safe I guess.
```
if(r < rF
else if(r >= rF)
else ...
```
The ifs are a little wrong, but you do need 3 cases. For "doesn't know skill", "novice", and "rF+". I thought I added that... you need to talk to Edern to get Blacksmithing to rF, don't you?


This is a T4 template thing that's driving me nuts i cant figure out where its adding these lines.


It should be done in caller not in `copy_recursive` like https://github.com/clear-code/tdiary/blob/0fbbcfd8b9ed477b976972cf92d7c5aec37985b6/html-archiver.rb#L602-L604 .


got a compile error and didn't feel like fixing it. hope people don't ever make any maps starting with mp/siege_hoth


이 부분도 count가 선언한 배열의 원소 개수와 일치하지 않음. define하여 사용하면 이런 실수를 줄일 수 있을듯..


I was rather confused right now, why I was mentioned in a very outdated commit xD"


Here, maybe we can break this part out of main.scss into reports.scss


함수명에서 get은 매우 간단한 연산일 경우에 사용.. 다른 단어를 사용해보기..


no more cheating here :)


megre


함수명에서 get은 매우 간단한 연산? 에서 사용.. 다른 단어로 변경하기..


[TestResults.docx](https://github.com/ArtuhovichVladislav/SoundTube/files/47099/TestResults.docx) сайт успешно прошел тестирование. Не было возможности провести тест на "Поддержку большого количества пользователей на сайте одновременно", так как тестирование проводилось на локальном сервере.


Bonjour,
MOn rpoblème est toujorus d'actualité.
J'ai rajouter les 11 linges dans le fichier setup, et delete les 5 dans le fichier formdisplay mais quand je veux accéder au formulaire en anonyme, j'obtiens un "loading fail" sur mes champs*.


Thanks :D


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


This fixed issue #2


this command doens't work, the package has a different name on packagist


Copy paste leftover?


@USBhost
Now, we have Adaptive LMK and Process Reclaim.

Adaptive LMK
echo 1 > /sys/module/lowmemorykiller/parameters/enable_adaptive_lmk // enable it.
echo 53059 > /sys/module/lowmemorykiller/parameters/vmpressure_file_min

Process Reclaim
echo 1 > /sys/module/process_reclaim/parameters/enable_process_reclaim
echo 100 > /sys/module/process_reclaim/parameters/pressure_max


Bonjour,

Avez-vous fermer puis réouvert votre navigateur web ?
Ces information son en session et ne sont donc rechargées qu'une seule fois à l'ouverture du navigateur (et de la page).


i created an additional html file. is this too confusing?


@Khelek а, здесь другая история. Нажимаешь удалить, АПИ возвращает 200, а вот срабатывает всё равно error, а не success. Можешь неполениться, проверить. У меня башня поехала от этого.


Did you fix the "main" field in the lib repo, as I mentioned in the issue?


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.843](http://teamcity.monogame.net/viewLog.html?buildId=11757&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependencies failed: 2 Build time: 00:00:00


I would rather go to strtotime() comparison since the date format in the $item['value'] might change.


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


А можно разбить на 2 перекрытия?


aAaah, you are entirely correct, sorry for the noise!


Possibilité d'ajouter des entreprises partenaire au site, elle s'ajoutera ensuite sur la page d'accueil


On Mon, Nov 30, 2015 at 01:07:57AM -0800, Park Ju Hyung wrote:
> Thanks :D

I tested 2.24.51.0.3 and it works on x86. So nothing obvious broke.

Double check the compiler's pluginld. If that is wrong all kinds of
strange things can happen (e.g. errors for every assembler symbol)

-Andi


Possibilité de modifier une entreprise partenaire, de lui ajouter une description et design du panneau d'administration


PS : Et avez-vous essayé les modifications du commit suivant : b474996
Ce commit est également en lien avec votre problème.


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


ahh okay that would be good


Meh. Mock dependency and no requirements.txt updates :(


Cities call should be put inside of countries-then. Otherwise they will work in parallel


Need to make a version variable in property tag


아래 커밋코드와 비교해볼때 i가 1부터 60까지의 값이 들어올 수 있는데
i가 1일 때 2, 3, 4번 원소에 값이 들어감.. 60일 때 179, 180, 181...
원하는 바는 0~179까지 180개의 원소를 받으려 했으나
현재는 2~181까지 180개의 원소를 받음.. 다시 생각해보아야 할듯.


I have no means to test kyleprods. You could try to revert to stock values.


Resources based on XMLHttpRequest is not a good idea. The right way is to implement TClass.getResourceAtStream() so that (selected) resources will be embeded in generated JavaScript.
I and Jarda have same opinion and that is it should be implemented in TeaVM because resources is basic thing in Java.
Controls.js don't use resources in any non-standard way so when TeaVM will support resources the Controls.js will work with TeaVM.


I'll fix it!


2 lines above are redundant, and make the _resizeEvent() do nothing at all, instead of resizing the canvas, when cocos is inside a frame.


Nah, creating an additional html file is fine. It's the state names that can get confusing as we start adding more, does that make sense?


Could also use:
```
if (p_debug)
    fname=debug_binary;
else
    fname=release_binary;
fname+=use64?64:32;
```
But actually after writing it I'm not convinced it would improve the code :P


rather not store a reference to redis in $this. Rather
```
$redis = $this->Redis->redisConnect();
$statsKey = $this->_getStatsKey($database);
$stats = $redis->get($statsKey);
```


megre


Hi Fabio,

I'm very sorry, this should not happen at all... :-/
I will look into it, but my first guess would be that a "$:scram b"
could solve that problem maybe...

cheers,
Marcus

On 11/29/2015 03:58 PM, Fabio Colombo wrote:
>
> Hi, I was getting the following fatal error when running with these
> committed changes:
> "The TMVA interface currently only supports float-type input variables!"
> therefore, for safety, I disabled by default the
> MVATestMethodsProducer in the pipeline sequence during my commit
> 8029dd5
> <https://github.com/cms-analysis/HiggsAnalysis-KITHiggsToTauTau/commit/8029dd546681088987401cb51bc0682cf01ba42c>.
> We can check together what is the reason for this and try to solve it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cms-analysis/HiggsAnalysis-KITHiggsToTauTau/commit/f2ce0e61500b2264dbbbdbe1943ddd77f3b0e773#commitcomment-14668862>.
>


위의 코드는 temp 와 humid 각 단일 변수를 출력하는 함수...
하지만 필요한건 temp와 humid가 배열 일때가 아닌지..?


Thanks!


Why this patch? I'm not sure to understand.


Why is the clone needed? Couldn't it just move into the `CommandHandler`?


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


Hi John,
It seems you omitted the autotranslate(me) in the load event for the dlgChisqure.
Consider putting it up.
Thanks


Maybe a ```String&``` would be better for these two functions? I'm a C++ beginner though so I might be mistaken :)


Hello @cemolcay,

Could you update the podspec with a new version in order to ship your dependency into private podspec ?

It could be really nice of you because we cannot specify any commit inside a podspec.

Thank you very much,

Best regards,

Lorenzo.


모든 데이터를 출력하는것은.. 원하는 데이터 개수가 모두 찼을 때 출력을 하는게 어떤지요


Yoda conditions you shall not use.
Replace with `mAvatarGrid =! null` as seen around the rest of Topeka.


下个版本加进来，好了


These methods are unnecessarily wrapping hide and show calls to the FAB.
Please remove them.


Very nice cleanup overall, thanks a lot!


Well the latest seems to be 2.25.51.0.4.

On Mon, Nov 30, 2015 at 6:13 PM, Andi Kleen <notifications@github.com>
wrote:

> On Mon, Nov 30, 2015 at 01:07:57AM -0800, Park Ju Hyung wrote:
> > Thanks :D
>
> I tested 2.24.51.0.3 and it works on x86. So nothing obvious broke.
>
> Double check the compiler's pluginld. If that is wrong all kinds of
> strange things can happen (e.g. errors for every assembler symbol)
>
> -Andi
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/andikleen/linux-misc/commit/0ec62f0db60ed1025df5ad4422604718c1460cef#commitcomment-14676003>
> .
>


Please add more context on the value that your getFixedVisible... methods add.


oh lol. yeah it makes sense that it gets confusing. we can do /reports/create and /reports/view, this looks cleaner


What's the value added by this?
If it does, this PR is more than a mere bugfix and probably should be multiple commits.


The original is correct.

One name server _uses_.
Two or more name servers _use_.


Hehe, thanks! :-)


1. Empty semicolons (without any statements before them) break compilation with MSVC 9.0 Express Edition.  I'm not sure about other versions.
2. snprintf() function could not be resolved under MSVC 9.0 Express Edition.
3. The original PKCS11_login() implementation allows for pin to be NULL.  https://github.com/mtrojnar/libp11/commit/ecb83dad73ae39afb24b3cea294d44b7f3e3a6f1 breaks it by introducing prev_pin defined as char[64].  My patch fixes this regression by changing prev_pin to char *.  This also eliminates the need for snprintf().


@pedrobaeza si merece la pena cuando no hay puntos que convencen, yo no he obtenido una respuesta clara de porque cambiarlo a python, y menos de porque cuando hice el commit no se sugerio y se debatió. Aún sigue sin cuadrarme porque los módulos sepa no siguen también el punto de cambiarlas a clase python, es una clase abstract que podría ser una muy buena solución intermedia.
Yo no he eludido, lo que no me parece lógico es que unos módulos verticales propios hagan la adaptación otros, pero sin embargo yo si he respondido las matizaciones que has indicado tu, dándole una solución técnica más correcta a tus sugerencias sin perder la herencia de Odoo.

@rafaelbn sin duda que hay que seguir unas normas de calidad, y yo intento seguir las normas de OCA, y si en alguna me he colado, como ha pasado en otras contribuciones en otros repositorios que no son este, me, pero este punto no es de calidad en si, sino de programar de una forma diferente, y siento no encontrar donde se indica que deben clases python y no heredar de models.


I used the semi-colon on purpose, since there is a list coming up it is possibly confusing to know which things are grouped if a comma is used.


Nevermind, it's fixed already in the second commit.


Remove this.. Must ahve been added during a merge conflict


If you want to use "Case 1", "Case 2", and so on, then the `style="numbers"` needs to be removed (I put this in so the options would be numbered automatically, which they are, and putting "Case 1" is then duplication).


Yeaaah...not sure haha. That's the problem of having them point to sub-routes


so report.css and reference it in style.scss?


都删除target


@manojo -- This is not enough to disable the computation (with the if statements) of the overflow check. It simply means it keeps parsing in any case.


report.scss, and yeah, import it in style.scss


As formatting against a standard is a tedious work, can I suggest we agree a common standard and use an automatic code formatting tool before commits?

Formating standard can be GNU C (http://www.sourceformat.com/pdf/cpp-coding-standard-gnu.pdf)?

One potential tool I just tried that supports GNU C is:
http://sourceforge.net/projects/astyle/?source=typ_redirect

Some comments on this tool here:
http://linuxpoison.blogspot.ro/2010/02/astyle-source-code-indenter-formatter.html


kylepro and kyleprods are 100% same, you can even use kyleprods ROM on s7580 and it's detect 2 sim (1 hidden sim slot) but cannot working because lack of sim 2 IMEI.
how's those thing work on your phone? "low battery" should be 3.5v but now it's increase to 3.7v, you're missing a lot of capacity of battery


lol. maybe just a page that has a description of what the user can do with reporting, kinda like a help page


Problems or programs?


CV V2


@Khelek works! thanks!


Addirittura i commenti sulle linee di codice, bello github


I agree. I'll fix if you don't want to send a PR? :)

On Mon, 30 Nov 2015 08:32 Krzysztof Magiera <notifications@github.com>
wrote:

> IMO it would've been safer for these to go in a finally block in case
> someone in the future decide to catch and ignore possible exception
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/facebook/react-native/commit/2b22d22a8384a800043bc0141b3a36eaa59c0716#commitcomment-14675347>
> .
>


버블소트 말고 qsort는 없나요??


TeamCity Open-source Libraries :: NotDeadYet :: CI [Build 0.0.21.0](http://build01:8888/viewLog.html?buildId=13377&buildTypeId=OpenSourceLibraries_NotDeadYet_CI) is now running


А зачем ты тут заинклюдил ReposModule? Есть ли в этом необходимость?


I'm sorry?


Issue #9


TeamCity Open-source Libraries :: NotDeadYet :: CI [Build 0.0.21.0](http://build01:8888/viewLog.html?buildId=13377&buildTypeId=OpenSourceLibraries_NotDeadYet_CI) outcome was **FAILURE**
Summary: Tests passed: 3; exit code 1 (new) Build time: 00:00:55


Good call. We usually design problems with our programs we can fix them later on.


This patch doesn't allow selection of move from other thread not only for easy moves but also for first root moves. Maybe this is also  good, not sure


Okay, if it's not clear to you then it will be confusing to other people too. I'll update the text after merging. :)


Issue #4


TeamCity Open-source Libraries :: NotDeadYet :: CI [Build 0.0.22.0](http://build01:8888/viewLog.html?buildId=13379&buildTypeId=OpenSourceLibraries_NotDeadYet_CI) is now running


@danielhickman We can't link to the cyanogen theme store, right?


This is bad design. For every request there will be two calls to database.
We should do one call and, depens on relusult, send NotFoundExeption


TeamCity Open-source Libraries :: NotDeadYet :: CI [Build 0.0.22.0](http://build01:8888/viewLog.html?buildId=13379&buildTypeId=OpenSourceLibraries_NotDeadYet_CI) outcome was **SUCCESS**
Summary: Tests passed: 3 Build time: 00:00:31


why my appname is nil here


:+1: I guess there's no type symbol for that.... Lol. Hmm.


I think it's the header format in general, not a specific field.


Done


This line should be reverted.


This only reverts the logging message.


See comment here as to why we can't memoize the attachments: https://github.com/dimagi/commcare-hq/commit/5465ebfc86daae50cc3a7f075f17cb59bbb8f6ff#diff-d28c016fb16e34a86ad9ce844e326789


@pedrobaeza si merece la pena cuando no hay puntos que convencen, yo no he obtenido una respuesta clara de porque cambiarlo a python, y menos de porque cuando hice el commit no fue sugerido y no se debatió. Aún sigue sin cuadrarme porque los módulos SEPA de OCA no siguen el punto de cambiarlas a clase python, es una clase abstract que podría ser una muy buena solución intermedia a no perder la herencia de Odoo.
Yo no he eludido nada, lo que no me parece lógico es que unos módulos verticales propios hagan la adaptación otros, lo que hacen esos módulos es extender los conceptos de los recibos. Sin embargo yo si he respondido a las matizaciones que has indicado tu, dándole una solución técnica a tus sugerencias sin perder la herencia de Odoo.

@rafaelbn Sin duda, estoy de acuerdo en que todo hay que publicarlo junto y que hay que seguir unas normas de calidad, y yo intento seguir las normas de OCA, y si en alguna no se ha realizado correctamente, como ha pasado en otras contribuciones en otros repositorios que no son este, me las indicado en el PR y se han corregido siguiendo las indicaciones, aquí no se hecho ese paso. A parte, que este punto no es de calidad en si, sino de programar de una forma diferente, y siento no encontrar donde se indica que deben ser clases python y no heredar de models. Yo soy el primero que todo sea apoyarse, llevo muchos años aquí para no saber que es el camino del software libre.


This commit has the addition of app.config because only .config.js was ignored by .gitignore


Issue  #6  #8 #5


Convention is to pass an `Actor self` as the first arg, even if we don't use it.  I'm wondering whether we should drop this convention at the same time we migrate to explicit interface implementations, since that old practice has been almost completely superceded by an actor reference in each trait.


Very nice....


Good job, thanks. Could you push this upstream to the repository you forked? https://github.com/chrisbanes/cheesesquare It is very hard for developers you share your code with to collect and update pieces from different places and get the licenses right.


Do you need the `.Length`s here?  Distance comparison should work fine on the `WDist`s themselves.


>>>Why? Isn't the read of a POD is an atomic operation to begin with?
depending on the underlying hardware architecture it may or may not be atomic. As far as C++ is concerned there are no guarentees


I was wrong... I thought `dlg` was a class member... As it has no parent and it's a local var, `dlg` is deleted when it goes out of scope. And the same should happen for `layer`. BTW, I'll try this evening and let you know.


depending on the underlying hardware architecture it (read of a POD) may or may not be atomic. As far as C++ is concerned there are no guarentees


How about making this a `BlockingHeight { get; }` property instead? Suggestion, not a review request.


So it works completely???


Hmm, I guess you could put the links on the /reports page


About the general question whether to use non-const references or not: The issue is that if you call the method, and pass arguments via non-const reference, you have no idea that the passed value can be modified ([argument repeated by a KDE guy](http://developer.kde.org/~wheeler/cpp-pitfalls.html)). This made Google [forbid it](https://google.github.io/styleguide/cppguide.html#Reference_Arguments) for its internal codebase.

So generally I'd avoid non-const references, and consider them bad style :)

And second argument, references can't be NULL, and here I use that fact by checking err for NULL. This way one of the two places it is used in each exporter class doesn't have to pass an err param which it doesn't use later on.


Замени на 'getNames'


Thank you! Its got a bunch of options like Bitcoin now so folks who can't PayPal/debit card can buy it. International problems :)

--
Phil Sturgeon
Sent from my iPhone and there's probably typos because I'm probably at the pub.

> On Nov 29, 2015, at 11:22 PM, Hannes Van De Vreken <notifications@github.com> wrote:
>
> Different link to @philsturgeon's book.
>
> https://twitter.com/philsturgeon/status/671214215806496768
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/madewithlove/blog/commit/ee33d7a31ed702c56776a2fb36c424333a28d2bf


@mlubin, NonlinearModel now wraps the LinerQuadraticModel and exposes the relevant functionality.


Because this is a long-lived collection, you should `.ToArray();` it so that the subsequent enumeration is cheaper.


The line 30 should be fixed as proposed.  The others are "ok" to me.

30行目は明らかに逆なので直したほうがいいですね。他も私はいいんじゃないかと思います。


This should be something like `.Where(Exts.IsTraitEnabled).Max(b => b.GetHeight())` to match the behaviour ingame.


But yeah, not a long time ago I've gotten this explained myself after I've submitted a PR that did precisely such non-const references.


interesting... I suspect the problem is very few folks outside palantir are running the version using http password based auth, most people are on auto-generated SSH keys now.  As such, that codepath is not well tested (by me or others)


Could you move the IBlocksProjectiles commit earlier, and then merge this one in with the main gate implementation commit?


Это не исправлено


I never do it, I changed to do it like you and respect Topeka style:
https://github.com/googlesamples/android-topeka/search?utf8=%E2%9C%93&q=%22null+%21%3D+%22&type=Code
 ```
      if (null != start || null != end) {
            return getStartEndView(start, end);
        }
        if (null == mAnswerView) {
            mAnswerView = createEditText();
        }
```


extract System.map to $TEMP/minkernpackage


no, this reverts ```continue```, which is key here


rename System.map file in minkernpackage


it check for renamed file in minkernpackage dir's parent directory


make sure to look at the minkernpackage directory for the new system.map


should fix https://github.com/xamarin/test-cloud-live-stf/issues/125


These are calibration data. It forces the phone to low calibration mode if battery voltage drops below 3700 mV or notified capacity is below 15 per cent. The phone shuts down when it reaches critical voltage which corresponds to 3480 mV and 2 per cent of capacity according to ss_kyleprods_cutoff_cap_lut. The low calibration mode means that the driver tries to keep the difference between coulomb-counter based capacity and open-circuit-voltage capacity within 1 per cent. In high calibration mode the difference should be up to 3 per cent.

I am now discharging my phone from full charge. I charged it to full about 19 hours ago. It now reports 49 percent left (notified capacity) and 18 hours left. Debug parameters: ocv capacity is 45 per cent, unusable capacity (uuc) is 6 per cent, state_of_charge estimation error (soc_error) is -10 per cent, adjustment factor is -21. You can check those parameters with any file explorer as the driver saves those parameters is sys/kernel/debug/bcmpmu59xxx/fuelgauge


а если getList вернет error? Нужно добавить обработку


Let's separate the issues solved by different commits.


the groupinfo participants list is broken, is that caused by this changes?


python has multiple inheritance, so use it:
```python
class ArtistModelMixin:
    model = Artist
    fields = ['first_name', 'last_name', 'nickname']
    form_class = ArtistForm

class ArtistDetailView(ArtistModelMixin, DetailView):
    template_name = 'artists_app/artist_details.html'

class ArtistCreateView(ArtistModelMixin, CreateView):
    template_name = 'artists_app/artist_add.html'

class ArtistUpdateView(ArtistModelMixin, UpdateView):
    template_name = 'artists_app/artist_edit.html'

class ArtistDeleteView(ArtistModelMixin, DeleteView):
    template_name = 'artists_app/artist_delete.html'
    success_url = reverse_lazy('artist_list')
```

in models:
```python
class Artist(models.Model)
    # ...
    def get_absolute_url(self):
        return reverse('artist_details', kwargs={'pk': self.pk})
    # ...
```


а зачем это? Какой смысл в двух null?
нужно проcто callback(null) или callback(error)


Need to remove unnecessary log line


@Elginior This is a temporal workaround, 'cause I'm pretty sure this dependency has to be imported along with some others, like JUnit or Spring Test. But I saw there are some exclusions. I decided do not touch them and put dependency independent.


maybe it's mine battery suck or something is eating battery, i'll test it again with clean ROM


better use ```:followings```


I personally do not agree with the idea of adding Microsoft.*.dll as dlls to be skipped because the switch "nonsystem" does not imply skipping them (at least it is not obvious). On top of that, I don't think Microsoft.TeamFoundation.Build.Client.dll qualifies as a "system dll"; one may want to know if they have a dependency on it.


@Elginior Agree with you, BUT:

- do you have any suggestions how to do that?
- can we leave it as it is now, 'cause we don't have any problems with database performance at this moment?


Why not just to use "Follow" class instead?


Is this line intended?


Here you have an example.


Essential feature. Thanx :) !


so clicking a single report will open a new page with just that report and all its details?


Ok, the blog_id was a mistake I think, I'm not using it.


The problem that I was encountering was that once I ran valgrind.vim, My
preset errorformat was being wiped out and other commands were not working for
me. However, upon closer inspection, I realize that the way I did this, the
length of errorformat would increase each time valgrind.vim was run. Perhaps I
should try to put this in a guard so that it will only run once. The only
other possibility that I can think of (although there might be some that I am
missing) is to remove it from the function and place a comment in the readme
telling users to set this for themselves for any filetype for which they will
run this plugin.

Let me know your thoughts and I'll be happy to help with a solution.

On Mon, Nov 30, 2015 at 01:17:12AM -0800, Nicolas Cornu wrote:
>Why this patch? I'm not sure to understand.
>
>—
>Reply to this email directly or view it on GitHub.*
>


That's a good point. What about using a const reference though for Strings? Though I understand that here you need a full copy to be able to check for NULL, so I'm more asking generally to know what to use next time I want to manipulate strings.


In general, you shouldn't make database queries in the views


My need was to look at *just my assemblies*. There are definitely other approaches to achieve this. Like a search term that I could provide. An alternative keyword additional to nonsystem (e.g. my).

what would be your suggestion?


Hm, that's true, that might work, as it's only used in one command handler at a time...


Comment


odd '>>' characters.


btw, you don't even need to define `template_name` everywhere - by default it is defined as `{app_label}/{model_name}{suffix}.html`, where suffix is one of `_list`, `_detail`, `_form`, `_confirm_delete` (depending on base view class)


Sorry about that. I thought I had changed them all!


ok


Wow, good spot.


shice, hallo, in dem container den ich befülle muahahahah


:warning: props mutation! props are RO!


Ah, my bad – sorry for the noise.


It's not just a matter of if reading a POD is atomic or not (it is not as far as C++ is concerned). Say thread t1 decrements the counter - it might do a 'load into register, update value in register, store to memory'. When thread t2 reads the POD it could read right before thread t2 has executed' store to memory'. It's clear that this situation could be problematic.


this line is redundant


Yeah. I think right now, the service just has a report.getAll() function, and is missing a report.get(id) function. Maybe you could implement that as well?


Yeah I wasn't sure about this one, will have to refactor.


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


u don't have to specify ```controller: :comments``` unless it name differs from the resource name, because Rails will try to match those name automatically


네 동작합니다.


I think a very useful addition to AsmSpy could be to provide support for -include and -skip switches; for instance:

    asmspy . -include "MyNamespace.*" -include "DependencyNamespace.*" -skip "Microsoft.*" -skip "System.*"

or a shorthand like:

    asmspy . +"MyNamespace.*" +"DependencyNamespace.*" -"Microsoft.*" -"System.*"

What do you think?


i know about default template_name that was for studying only


i would already cast to string here


There is no need in @user variable. Even if u need this on a view, u can always use ```current_user``` helper method there


git tool vet fails at line 222 where extra %q's are introduced. Any reason to include those?


Why did you call the variable `CustomFields` starting with a capital? That seems to produce two JSON properties: `customFields` and `CustomFields`.


Is this intentional? Seems weird to return CUBEB_ERROR* for this function


I like the +. If first occurrence is detected a white list could be built to output the results


There's no code path for accessMode == MORE_WRITE


Yes, I've been somewhat inconsistent with that in Topeka, so both exist in the repository right now.
The overall idea is having easier readable conditions for newly added code, which excludes Yoda conditions.


I have some clues, but not exact solution
We can leave it for now. My proposal is to make issue in JIRA. Then, when we have time to dig into it, we will return to this.


Oops, indeed, this is a the text email, not the html email.


np


has_key is deprecated and removed in Python 3. Use ``python:'banner_title' in banner``


Possibilité de visualiser le document joint à l'offre


Thanks! Fixed here f717e123e869c14b8264fced63a577eb644f9f42

On Mon, Nov 30, 2015 at 3:52 AM, Ulric Wilfred <notifications@github.com>
wrote:

> Mistype here: Unk*n*ownObjectError :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sebpiq/WebPd/commit/24f1f74d648e028824c587ce98ee32b872a12ffa#commitcomment-14671714>
> .
>

--

*Sébastien Piquemal*

 -----* @sebpiq*
 ----- http://github.com/sebpiq
 ----- http://funktion.fm


Looks good to me :+1: