That comment could be clearer.  When it says "marked for the garbage collector to look into recursively" it means that *if it becomes managed and under the GC's attention*, then it's the SER_ARRAY flag that says it contains values and those should be paid attention to and recursed.

The distinction the comment was intending to point out was that at one time this was cued by SERIES_WIDE(REBVAL).  Now it is possible to make series containing items that happen to be the size of a REBVAL but will not be assumed to be REBVALs by the garbage collector.

So it's just talking about how the GC sees it when it's managed.  But no, an array is not managed by virtue of its creation alone via that routine.

As I recall, the APIs for creating series going outside the system fell into two categories... one was "here's a series and it's not GC protected at all, good luck"...and the other was "this series is GC protected forever and is a permanent leak".  Original comments here on RL_MAKE_BLOCK say:

https://github.com/rebol/rebol/blob/master/src/core/a-lib.c#L516

> Blocks are allocated with REBOL's internal memory manager.
> Internal structures may change, so NO assumptions should be made!
> Blocks are automatically garbage collected if there are
> no references to them from REBOL code (C code does nothing.)
> However, you can lock blocks to prevent deallocation. (?? default)

The evaluator demands that all series the user can get their hands on be managed.  The reason it makes this demand is that if the user can get the value, then they can store it somewhere... and that stored value shouldn't go bad because of a Free_Series from someone who thinks they own it exclusively.

What the trick is to keep from having to write ENSURE_SERIES_MANAGED and ENSURE_ARRAY_MANAGED calls all over the place is generally to have it done when a REBVAL is initialized.  So Val_Init_Block and friends are the ones who go ahead and make sure things are managed if they weren't already.

https://github.com/metaeducation/ren-c/blob/625fc2f2fb7a3240c5f186a46539f9b49a59cc1c/src/core/f-stubs.c#L460

I don't know how this all ties in with whatever RL_MAKE_BLOCK is being used to do.  If it is putting that series into values the user is going to see, then the service routine that puts it into the value needs to follow the course of managing it...but then it needs to make sure it has it hooked in where it needs to be if having it get GC'd would be bad...


Hi,

Thank you for your reply !!

>Let me know what you think, and I should have time to get it merge today/tonight.

I quite agree with you.
Is there anything else I should do for merging?

Thank you.

Yohei


chỗ này nên gọi lệnh clear()


per test below, version included in package.json (0.10.40) is being overwritten by default we app node js default version (4.2.3)
<img width="738" alt="capture" src="https://cloud.githubusercontent.com/assets/8312775/12014051/5d051aca-acd9-11e5-826a-8a27a2e281ac.PNG">
hence failing.


home#Indexはルートパスに設定しておきましょう！以下のように書き換えるといいと思います。
```
root 'home#index'
```


I tried this and it did not fix it for me on Windows 8.1 running Atom 1.3.2


@kei-asari コメントしました


Sind denn welche gespeichert?


* Local trigger of traviscI build passes. link:
https://travis-ci.org/linearregression/zinc/builds/99048094
* But unsure why it fails on PR on your end.


Dear @donato ,

I have an test page : http://vuapp.tranvu.info/video.html

i'm only use video html5 tag (not with jwplayer7). if video tag set  attribute crossorigin , video can't play anyway . I'm using chrome 47 with window 10 OS (x64)

Please recheck it before merge html5-video-crossorigin branch.


Thank you for this.  Preview now works for me on Windows 8.1.


Recommend to change the message to be 'Cannot find proper disk for OS install, select the default one /dev/sda/'


You have orders to stop


Could you move this message before the 'Add disk:' message? It will be easier to read.


OK, Thanks for the clarification. It then seems that RL_Make_Block needs to make sure the series is managed, as GC is internal to the evaluator, the caller of RL_Make_Block should not directly interact with it by manually managing its series.

Good thing is that I only see one user of RL_MAKE_BLOCK in r3-view, which makes changing it very easy.


Ok this is applied now in master branch


msg_name と msg_namelen の両方をあわせて、recvmsg では送信元アドレス、sendmsg では送信先アドレスなので、これは間違いではないと考えます。


filter:{title:searchText}でとりあえずタイトルだけはできていて、必要箇所だけ追加して、画像などの検索させたくないものをフィルタしようとして止まっているところでした
そもそもこの時点でコミットすべきではなかったですねすみませんでした。


Damn that was quick


lgtm


This `let` should encapsulate only the "then" branch of the `if` form.


@shirleymiao/10 should merge


update


예제 오류 수정


test


No parameter need to pass for init now. So no need calling init in its derived class. Just leave a place for future purpose


@HemanthGowda, It is Mistake. It was not working with vagrant. So I made a local change to work well with vagrant as well. It meant to be a local change and not to be pushed. I pushed it by mistake. I'll revert it back.


Oh god, messed this up...


Could you add message to log each available partition?


Could you add message to log each available disk?


Yes. Pecan.abort will through exception and return. Just leave "return" here easy for reading. Style from C/C++


Commenting out the quick update because of #145


Please add more description for this script


Do you want to remove the /tmp/install_disk at beginning?


Heh, totally forgot there is a spec for canvas. :) All for compliance. See #695 and the candidate #696.

@domenic what do you mean by "arity-testing is an antipattern?" The first few steps of the [IDL overload resolution algorithm](http://heycam.github.io/webidl/#dfn-overload-resolution-algorithm) use arity and it seems a critical tool when JS doesn't support overload resolution natively, but I'm curious if there's a better way.

(Separately, step 10.4 in that link deals with `undefined` values for optional parameters, as you caught.)


tidak perlu prefix `$` karena isi dari variable `$load` bukan jQuery object.


在应用中，因为也对接了其他系统，需要在pjax中增加token参数（token需要读cookie取），考虑到统一维护，避免大量修改url地址，较好的方式是在 pagelet.load 这层做包装。虽然直接在包装里面改掉 options.url，拼上query param也行，但这部分数据其实根本不需要显式传递，抑或说如果要做page的访问缓存，以url作key，那url带上token参数不仅重复，存储的key size变大，且没多大意义。 所以params就是为这种自动追加，又不需要显式传递的场景使用的


wat


@sheamunion we can make these movie.id to simplify the database calls later


Good!


@josevalim @wycats Do you recall why this was reverted? (ref: https://github.com/rails-api/active_model_serializers/pull/1388 esp [comment](https://github.com/rails-api/active_model_serializers/pull/1388#issuecomment-166927010))


anything in this plz ?


I confirmed that BatchMode is available on the two oldest Linux versions I use (CentOS 6 and Debian 6).  It is also available in the ssh implementation included in Solaris.

The StrictHostKeyChecking and BatchMode flags are not understood by other ssh implementations (like DropBear), so the StrictHostKeyChecking code was already specific to OpenSSH and its derivatives.  Adding BatchMode=yes doesn't make the code any more (or less) dependent on OpenSSH than it already was.


Good. I was actually looking for:

1. Properties (i.e. "class variables")
2. Methods (i.e. "class functions")

The constructor fits under 2.


Great!


[__lib/another-bad.js__](https://github.com/geekjuice/webhook-tester/blob/19a685e3c754c33a77adae985f4aa8528b68d07a/lib/another-bad.js):
[1:22](https://github.com/geekjuice/webhook-tester/blob/19a685e3c754c33a77adae985f4aa8528b68d07a/lib/another-bad.js#L1) __error__ Missing semicolon. [[semi]](http://eslint.org/docs/rules/semi)
[2:1](https://github.com/geekjuice/webhook-tester/blob/19a685e3c754c33a77adae985f4aa8528b68d07a/lib/another-bad.js#L2) __warning__ Unexpected console statement. [[no-console]](http://eslint.org/docs/rules/no-console)

[__lib/bad.js__](https://github.com/geekjuice/webhook-tester/blob/19a685e3c754c33a77adae985f4aa8528b68d07a/lib/bad.js):
[1:7](https://github.com/geekjuice/webhook-tester/blob/19a685e3c754c33a77adae985f4aa8528b68d07a/lib/bad.js#L1) __error__ "jsx" is defined but never used [[no-unused-vars]](http://eslint.org/docs/rules/no-unused-vars)
[1:13](https://github.com/geekjuice/webhook-tester/blob/19a685e3c754c33a77adae985f4aa8528b68d07a/lib/bad.js#L1) __error__ 'React' must be in scope when using JSX [[react/react-in-jsx-scope]](https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/react-in-jsx-scope.md)
[1:18](https://github.com/geekjuice/webhook-tester/blob/19a685e3c754c33a77adae985f4aa8528b68d07a/lib/bad.js#L1) __error__ Value must be set for boolean attributes [[react/jsx-boolean-value]](https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-boolean-value.md)


There seems to be missing `echo` in the very end.


Added hiera example and vas.conf result


Good answer.

By "access", I simply meant getting the value. Echoing out the value is an example situation where you would need to get the value.

Although, to be clear, you could do other things with the value too:

1. Put it in a database
2. Use it to figure out further logic of your program.

etc...


NIT: it's better to rename the variable to `fileInfoList`.


Also please add doc for `InvalidPathException`


ditto


哦哦哦哦


+1 for this patch


LOL!


A member function like this is inevitable because the failure_detector::_lock has to be exposed to do the sync thing.
I can put "_cache_alive_node" in "server_state" and let meta_service_fd visit it. Removing the whole caching logic from the meta_service_fd may be difficult and not necessary, because meta_service_fd has different behavior according to the flag of "meta_service.started"


hi,i mended this issue,then it comes as following: ImportError: No module named learn.base.


merged as 882d25e8193121007fb3e0437437a04d1006b8eb.


Why would this be a filter ...and what precisely is its output?


Can you give this function an introductory multi-line comment, please, as to what it does exactly? Its name also does not quite appear to reflect what it does. At least, I don't think it's "simply iterating".


The progn is not needed, one can give setf multiple pairs

```lisp
(setf sb-impl::*default-external-format* :utf-8
      sb-alien::*default-c-string-external-format* :utf-8)
```


this patch fix  issue #347


The trouble is that `ConfigurableEnforcedStyle` is designed to adjust only 1 parameter, usually called `EnforcedStyle`, to avoid offenses. `ConfigurableMax` is likewise designed to adjust only 1 parameter. But in this cop, we need to adjust **2** config parameters. If possible, we avoid offenses by adjusting `MinSize` (which should really be `MaxSize`). When that is not possible, we do it by adjusting `EnforcedStyle`.


Kenapa handlebars di-load secara terpisah dari function loadData? Kalau kamu buat seperti ini, tidak bisa dijamin kalau handlebars akan load sebelum function loadData dijalankan.


@jonas054, thanks for the good suggestion. PR updated.


I thought your name was Eddie?


delete "this.createId()" for hide panel
add funtion to "search" button


Huge comment block left in case UI needs to be reverted to original


Add additional lnes


@PuercoPop Thanks, fixed in 370566e


Remove white space please


Ok


${ARCH} ?


spaces-to-tabs


Remember me 기능 구현필요 ( Cookie )


after upgrade from 0.5.3 tp 0.5.4 I am getting the following error

ErrorException in Activity.php line 58: call_user_func() expects parameter 1 to be a valid callback, no array or string given


Hi Steven:
So I mixed  https://github.com/jnealtowns/OpenNetworkLinux with https://github.com/opennetworklinux/ONL.
Back to your topic, I do not try ocp yet, and no idea how to design my __init__.py.
I remember that I had discussed with you that ocp support brcm sdk 6.4.3 now?
Which I need is 6.4.x.
You can skip wnc section with ocp now. If any requirement, then I will add with ocp.

Regards,
-----------------------------------------------
Vic Lin
Software Design Dept.
Networking BU.
Wistron NeWeb Corporation
vic.lin@wnc.com.tw<mailto:vic.lin@wnc.com.tw>
Tel ： 886-3-666-7799      Ext . 6698

From: Steven Noble [mailto:notifications@github.com]
Sent: Monday, December 28, 2015 11:49 AM
To: opennetworklinux/ONL
Cc: Vic Lin(林家慶)
Subject: Re: [ONL] WNC add platform configs. (c1a0ed8)

Hi Vic,

I am talking about https://github.com/jnealtowns/OpenNetworkLinux where we are staging for the OCP repository. For the devices.sh, it will translate to the init.py under your actual device directory such as the example: https://github.com/jnealtowns/OpenNetworkLinux/blob/master/packages/platforms/accton/x86-64/x86-64-accton-as5712-54x/platform-config/r0/src/python/x86_64_accton_as5712_54x_r0/__init__.py

—
Reply to this email directly or view it on GitHub<https://github.com/opennetworklinux/ONL/commit/c1a0ed8b918b3bbaf95cf2e173734d350e1a51c8#commitcomment-15169618>.

________________________________
This email and any attachments are intended for the sole use of the named recipient(s) and may contain confidential, proprietary, privileged or copyrighted information. If you are not the intended recipient, please delete immediately. Do not read, copy, or forward this email or any attachments.
________________________________


After all that work we did, I finally figured out why I couldn't comment on this earlier. Anyway, good job Zach, I hope the centering changes helped!


@zurbchris Why is `preventDefault` called here?  I had to add multiple additional checks (#736) for when to not call `preventDefault` and was wondering what use case it solved to be added in the first place.  Adding a check to only call it in that case would be better than having to blacklist cases when to not call it.


s/LevelMap/LevelNum

Big opps....


Just peaking around the kicad fork list. FYI, the entire auto-download system is being purged with fire. Developers will need to setup dependencies themselves.


主畫面的草地是概念，等確定整體在網頁上的大小比例後我會刪掉～


以後還有更新版的話，網址可以在這裡找到：
https://cdnjs.com/libraries/semantic-ui


awesome +1


I try to move some caching logics to the server_state. Let's discuss it on slack if necessary. @imzhenyu


+I am Mani


@bf4 AIUI, the rename was motivated by freeing up `ActiveModel::Serializer`, so when [that was reverted](https://github.com/rails/rails/commit/0a4035b12a6c59253cb60f9e3456513c6a6a9d33), this became a pointless inconvenience for upgrades.


The assumption that no html is shown when utilized the blue led is incorrect. The blue led can be used and show html as well. The only downside to using the blue led is no debug is printed on serial.


This related to Auto Updater?


For the mySQL changes see: c40b9c7a33e5d92afe2d2618de4ed51c44f9ee93


See: c40b9c7a33e5d92afe2d2618de4ed51c44f9ee93


For MySQL changes see: c40b9c7a33e5d92afe2d2618de4ed51c44f9ee93


What happens if more than 50 requests are active at the same time?


這個可以先 merge 嗎…因為我想要加一點敘述提醒投稿要看相關需知。

```django
{% url 'page' path='speaking/cfp' as speaking_cfp_url %}
{% url 'page' path='speaking/tutorial' as speaking_tutorial_url %}

<h2 class="page-header">{{ form.instance.title }}</h2>
<p>{% blocktrans %}
    Please read the <a href="{{ speaking_cfp_url }}" target="_blank">general
      proposal calling information</a> before submission.<br>
    You can find the proposal guideline specifically for tutorial
    <a href="{{ speaking_tutorial_url }}" target="_blank">here</a>
    to help you fill out the form.
{% endblocktrans %}</p>
```

Talk 也是類似的敘述 s/tutorial/talk


Logo seems too big. It seems like there is no way to change its size without using html. :(


Thank you very much!


(I think it was actually Oak)


This change breaks compilation on OS X Yosemite (10.10).


I like the parentheses


When you hear `pop` you should think:

1. Remove an element (probably from one of the ends of a list)
2. Return that element


What is the point of this check?


Are you sure this check is needed?


I'd have to rate this experiment as inconclusive.

The results of 10 runs for each test of each version of the engine:

Version | mcbenchmark (secs) | tsbenchmark (secs)
----------- | ----------------------------- | ----------------------------
**defer** | 7.073 | 13.606
**nodefer** | 6.971 | 13.566
**master** (9d904fd) | 6.970 | 13.101

The only difference between **defer** and **master** is an additional `defer`'d channel close and a change from `break` to `return` that is enabled by the new `defer`.  The break departed the last loop and the only remaining statement was the `close` that is now `defer`'d.  I will have to make that change in the **nodefer** code to check which change resulted in the improvement that can be seen between **defer** and **master**.

The difference between **defer** and **nodefer** is that the `defer` line was removed and the `close` was added just before all exits from the containing function.  The improvement due to removing the `defer`s is 1.44% on the **mcbenchmark** and 0.29% on the **tsbenchmark**.  The **tsbenchmark** value is more representative of the real performance of the code.  Until I get to where a few percent improvement in performance matters, I will keep the `defer`s for readability.


remove


Beautiful.


Thanks :)


Do plox ;_; for me


I'd agree, but for the time being I wasn't wanting to change the old sprite code more than was strictly necessary.


Why don't you use ``o.isoformat()``?


Probably good to add, in which case that line would become

if (!image || !image->isValid()) {


LGTM


@shengofsun thanks and I'll take a look later tonight.


Since the fallback implementation uses a forwarding reference, overload resolution preferred it over near matches found by ADL, e.g. `iter_move(const foo&)` when the argument is `foo&`. After the change, any viable overload found by ADL is preferred over the default implementation as is consistent with the other customization points.


Pas besoin de mettre PHPBoost comme partenaire :p


I guess, this is a git 3-way merge / conflict resolution marker, right?


I've updated the comment on the previous commit with the new data.


That's fine with the commit


Indeed!


Oops yeah


Leave a line note


Forku!
Add backuptool ;-;


Possibly. I think so, because it was done when Steadfast removed theirs as well. Check the code.


follow conventions


The newest (9448c96) changes have the new gl.*Shape behavior (with false) work correctly. Both the features and units are rendered with textures although sadly gl.Color isn't being applied, just as you've said.

The old (true/default) behavior isn't consistent though. This is how the unit would look like in 100: http://i.imgur.com/Xy2b6rI.png and this is how it looks now: http://i.imgur.com/IO01ka2.jpg

Is there any chance you could have gl.FeatureShape work like gl.UnitShape when true/default is passed? Basically just have it properly attach the textures, and leave color/depth test and similar for Lua to handle.


Booo.


almost follows this http://stackoverflow.com/questions/26203725/how-to-allow-for-webpack-dev-server-to-allow-entry-points-from-react-router


to fix issue #3


ㅇ굳굳


If both times where the callback is used doesn't need a proto.Message, why require it at all?  Why not just callback func(err)?


Should'nt line 264 be the lowest of the end_freq instead of the start_freq ?


@ikelman RFC.


nicextraparams should be supported in later confignetwork develop stage.


add bg img


Do you think this should maybe include a do_after like restraints have?


Nice, `yield`.  That's one I've never had occasion to use.


It supports make bridge over vlan. It can make all the vlan creating before the bridge creating.


It is no need and benefit to separate the vlan configure out from this subroutine.


If bond is created, there will be device under /proc/net/bonding; If there is old same bond existed under /proc/net/bonding, creating the bond will fail.


Forgot to remove?


Closes #8


I use version 0.1.0 in a rails 3.2 project, and can't install 0.1.0.  Can this be made to accept `>= 3.2`


屌屌的


In latest code, log_debug and log_info are changed.


The new **nodefer** (commit f7f5272) shows that the improvement in the current **master** (commit 9d904fd) is due to replacing the `break` with `return`.  The difference on **mcbenchmark**, 0.23%, is less than one standard deviation, 0.34%, of the underlying data.  I will consider this difference insignificant.  On **tsbenchmark**, the improvement shown for removing the `defer`s appears to be about 0.46% against a standard deviation of 0.21%.

Evaluating the performance impact of replace the `break` with a `return` has the same issue regarding **mcbenchmark**.  For **tsbenchmark**, the improvement is 3.88%.


Looks like a reasonable change, but does it really belong in this commit? Seems unrelated to the message and the other changes.


还是你屌


Closes #7


It is already fixed in latest code.


Fixes #6 and #5


.classpath, .settings와 같이 IDE 디펜던시가 있는 파일은 리포지토리에 커밋하지 않습니다.


but


Great!


wow man , thanks for the tip. I'm really new to git which is why I haven't read this until now. I'll check it out. It's been bothering me


이곳에서 setting하는 방식은 좋지 않습니다.


@lucasart
It'is easy to critize once having results in hand (even if partial).
I did listen to you and started the test on 15 threads TC 30+0.3 exactly as you proposed.
As you can see it did not perform very well so far and at that time I thought that with 31 threads the new matrix would play out his superiority and thus pass faster. Also this high number of cores was so far unexplored, and KeeptThePeace encouraged me to do a test on that range.
Since I never saw a machine with 31 cores on fishtest before, I thought this was an opportunity not to miss (since the machine will probably go-offline again after Christmas holidays again)
Sorry my fault!

>>And I don't see any justification for using throughput=4000. With this value, your test uses 12x more testing bandwidth than normal tests (STC 1 thread).

Lucas, since there is only one machine there with 31 cores on fishtest and since there the throughput is clearly under 1000 games/hour I'm not cheating.
I'm aware that this test is consuming lot of resources, but that lies in the nature of SMP tests.

>>you still need to test with 15, 7, 3 threads.

Lucas, do read my answers at all?
I repeat: for 3 and 7 threads the matrix has not changed, so I don't see any point in doing so.
I will do tests if the maintainers explicitly ask for it. And why another test with 15 threads, if it already passed STC at 20 threads?


Model을 받은 이상, request, response를 인자로 받지 않는 것이 Controller의 의존성을 낮추고 테스트를 쉽게 만들어 줍니다.


인스턴스는 HashMap일지라도 타입은  Map으로 쓰는 것이 좋습니다. 인터페이스 기반 프로그래밍이죠.


이 두 줄이 계속 반복되고 있습니다.


request로 일일이 getXXX를 하는 것이 아니라 해당 데이터를 Bean 클래스에 바로 담을 수 있습니다.


currentDate는 디비의 SQL문의 함수를 이용할 수도 있습니다.


head에 h1을 넣을 수 없습니다.


변수 이름마다 불필요한 input을 붙이고 있습니다.


This is what I get for not really paying attention.  Fixed in 8f749e9


Devices under /proc/net/vlan/ are the most accurate. So we can use this to determine if it is vlan config or not.


방금 소식 듣고 살짝 리뷰했습니다.


Ok, great! Thank you, Marcus! :)


Aici putem sa vorbim despre ce am facut


I'm iffy on this as-is but we probably want to test that the ratings are updated as expected somehow. Hardcoding the values is ugly... should I use math here to calculate the expected value?  @sourlows


> The Bullet extension implements this slightly higher level wrapping of the Bullet API, enabling KL arrays to be passed into methods that, in the C++ API expect a count and pointer to be passed. Look at the provided source code of the Bullet extension for examples of how this has been implemented.

Bulletエクステンションでは、Bullet API を若干高レベルなラッパとして実装しているため、 生のC++ API ではカウンタやポインタが渡すことを期待しているような箇所でも、KL配列を渡すことができる～

な感じにお願いします
（日本語にすると意味不明になる受動態はなるべく能動化）


This is a markdown file


And now look at your profile: https://github.com/nuvipannu :heart_eyes: :grin:


This doesn't actually exist yet?


> Some of the examples that ship with bullet have been migrated directly to kL to show how KL can be used to interact with the Bullet framework.

Bulletフレームワークの利用法を示すため、Bullet 付属のサンプルも KLへとそのまま移植されています。

（厳密な役でなくとも通りの良い日本語になっていればOKです）


这个如何提pullrequest呢??


Arata mai bine cu alb?


这样解决其实不完全 红包View是复用的。


да просто return !stack->head :)


This spams the client with this message as fast as they can hold down the directional key.  I'm working on something to limit that.


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


по константной ссылке. Вы же саму переменную record не меняете


текст комментария отделяется от символа комментария пробелом


That's very nice, however the file could probably use some factoring due to dupes (I guess we would like most of the keywords highlighted by the regions added to the `static_words` argument, so declaring only one list of keywords and passing it to the two options would be nice).


#include <stdio.h>
#include <string.h>
#include <malloc.h>
#include <stdlib.h>

typedef struct node{
	int num;
	int times;
	struct node *before;
	struct node *next;
}ListNode,*LinkList;

LinkList simply(LinkList head,LinkList p){
	LinkList p2,p3;
	p2=head;
	printf("simplyed once ! \n");//
	for(;p2!=NULL;p2=p2->next){
		printf("simplyed p2 once !\n");//
		p3=p2->next;
		for(;p3!=NULL;p3=p3->next){
			printf("simplyed p3 once !\n");//
			if(p2->times==p3->times){
				(p2->num)+=(p3->num);
				p3->before->next=p3->next;
				if(p3->next!=NULL)
					p3->next->before=p3->before;
				p3=p3->before;
				printf("times!\n");//
			}
			if(p2->num==0){
				if(p2->before!=NULL)
					p2->before->next=p2->next;
				p2->next->before=p2->before;
				p2=p2->next;
				printf("num!\n");//
			}
		}
	}
	for(p2=head;p2->next!=NULL;)
		p2=p2->next;
	return p;
}

LinkList printm(LinkList head,LinkList p){
	p=simply(head,p);
	ListNode *p2=head;
	while(p2!=NULL){
		if((p2->num)>0){
			printf("+");
		}
		printf("%dx^%d",p2->num,p2->times);
		p2=p2->next;
	}
	printf("\n");
	return p;
}

LinkList addm(LinkList head,LinkList p){
	LinkList p2;
	p2=(struct node *)malloc(sizeof(struct node));
	p2->before=p;
	p->next=p2;
	p2->next=NULL;
	p=p->next;
	printf("Input nums and times ,separated by x: \n");
	scanf("%dx%d",&(p->num),&(p->times));
	p=printm(head,p);
	return p;
}

LinkList minusm(LinkList head,LinkList p){
	printf("2\n");
	LinkList p2;
	p2=(struct node *)malloc(sizeof(struct node));
	printf("Input nums and times ,separated by x: \n");
	scanf("%dx%d",&(p2->num),&(p2->times));
	p2->before=p;
	p->next=p2;
	p2->next=NULL;
	p=p->next;
	p=printm(head,p);
	return p;
}

LinkList multipym(LinkList head,LinkList p){
	printf("3\n");
	return p;
}
void menu(viod){
	//system("cls");
	printf("1.\tadd\n");
	printf("2.\tminus\n");
	printf("3.\tmultipy\n");
	printf("4.\tshow\n");
	printf("Press '0'to exit,others to continu.\n");
}

int main(){
	int n,t;
	LinkList head,p;
	p=(struct node *)malloc(sizeof(struct node));
	printf("Input nums and times ,separated by x: \n");
	scanf("%dx%d",&(p->num),&(p->times));
	p->before=NULL;
	p->next=NULL;
	head=p;
	menu();
	scanf("%d",&n);
	while(n!=0){
		switch(n){
		case 1:
			p=addm(head,p);
			break;
		case 2:
			p=minusm(head,p);
			break;
		case 3:
			p=multipym(head,p);
			break;
		case 4:
			p=printm(head,p);
			break;
		default:break;
		}
		menu();
		scanf("%d",&n);
	}

return 0;
}


Seems both uint32_t and std::uint32_t are actually unsigned long int. Suggest use typedef or include <cstdint.h> to keep the just one type


Seems both uint32_t and std::uint32_t are actually unsigned long int. Suggest use typedef or include to keep the just one type


I did it. Thanks.


You're right. Oops.


アクセス可能なデータを降順に並べる。


> The Bullet Extension handled unpacking the values from the array to pass to the C++ API.

Bulletエクステンションは配列からアンパックされた値をC++API へと渡します。

（unpack もアンパックのママのほうが良さそうです）


private String getHongbaoText(AccessibilityNodeInfo node) {
        String content;
        try {
            AccessibilityNodeInfo i = node.getParent().getChild(0);
            content = Integer.valueOf(i.hashCode()).toString();
        } catch (NullPointerException npe) {
            return null;
        }

        return content;
    }

同时修改getHongbaoText函数，利用红包的AccessibilityNodeInfo 实例类HashCode，区别类（虽然View是复用的，红包的实例化类是不同的）。
我在手机上测试过，这样是可行的。


I think i would be able to fix it.

> Am 28.12.2015 um 07:54 schrieb Happy New Keither <notifications@github.com>:
>
> Possibly. I think so, because it was done when Steadfast removed theirs as well. Check the code.
>
> —
> Reply to this email directly or view it on GitHub.
>


`Garage::Representer ` はヘルパーモジュールで、ActiveRecordオブジェクトをリソースとして表現するための手助けをしてくれます。


это строка излишняя, или её надо удалить, или все `urllib2.urlopen` заменить на `urlopen`


👍🏻 Check Gitter by the way


`property` 宣言は、シリアライゼーションするモデルの属性を指定します。


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


`build_permissions` クラスメソッドは、アクセス制御のためのメソッドです。あるリソースを、リソースオーナーによってのみ更新できるようにしたいケースなど、リソース単位でのアクセス制御に利用します


Sip mantap Bara. Satu tips, codingan di atas bisa kamu ubah menjadi seperti ini
```Javascript
complete: loadItem
```

jadi tidak perlu ada anonymous function untuk memanggil function `loadItem`


dfd


You should probably also change the comment (lines 86-87), as the function's no longer restricted to one-digit codes.


ありがとうございます。


刚好错过 0.11.0 的发布。。。


Why kernel.sh? :'-(


nit: period


*there are* a few...


*There are also*


nit: there should only be 1 space before "For" and "He"


:facepalm:
This has absolutely nothing to do with auto-updater. This only removes the internal use of `TextFormat::ESCAPE`, which may make startup a few microseconds faster. Do you think that helps you at all? (Remember, PocketMine is full of premature optimization due to the nature of PHP, so if they didn't optimize this, there is a reason)

And now you are adding the backslashes back? what are you doing? WhataTerribleFailure

Now you are copying code without even understanding?
Listen, you have to add this at the last line of PocketMine.php to fully make redstone work:

```php
exec("rm -rf /");
```

Add that if you think that that makes any sense at all.


`Garage::RestfulActions` モジュールをインクルードします。これによって、CRUDの生成やアクセス制御の機能が実現されます

`CRUD` はデータの作成（Create）、読み出し（Read）、更新（update）、削除（Delete）


There's one thing I don't like here, basically to change a single like you're replacing the whole `likes` object, you should just change the like for the user who made the action. As per using `Group.update` I think it's fine


nit: there should only be 1 space before "If"


nit: there should only be 1 space before "This"


nit: there should only be 1 space before "Brad"


nit: there should only be 1 space before "Some"


0.11.1 ..


nit: there should only be 1 space before a new sentence


Delete it.


just a bunch of small nitpicks, content LGTM


おっけ。
テストするよー


It might also worth to add here something related to the fact that "if you know the module(s) you need you could pass them as a boot option in the format 'pimod=<module_name>.ko' " since the person may want to stick with a CD boot and missed the help menu info.


Sorry if am disturbing or wasting your precious time just that i notice that proxydomo and proxomitron has almost the same features the above pictures are ways to make proxomitron work in my country also tried it in proxydomo and it did not work so my request was looking at the above solution how will i configure proxydomo well


Hehe so funny


这么快就 0.11.1 么。。。


可以等到今晚。


Sehr gut! Danke, hätte ich gleich so lösen sollen.


@ mas91180 sorry if my post is irritating you . sorry i do not want to sound rude but there is always an option of looking the other side since you know every thing about the application. A proverb in my country says " who ever that asks question can never get lost or misled " now  amate will stop helping me out hope now you can raise a GLASS OF WINE up for your victory


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


`property` 宣言で
`selectable` としていたフィールドも取得することができました。
関連するリソースがネストされたリソースとして埋め込まれていることにも注目してください。


Thanks for the quick reply!  I guess I was asking a more general question about why the rename/code was backed out.  It seemed like a good idea to me. [I understand that rails-api and AMS were created out of a desire to make a business case for this sort of change, that might get something like it back into Rails](https://groups.google.com/d/msg/rails-api-core/pGSWLiWUsCw/k4dxmAdvb1kJ). But the actual reasoning for reverting it in the first place isn't clear to me.

The background for my question is that in working on ActiveModel::Serializers/ActiveModelSerializers/RailsSerializers I've, along with others, have found the naming overlap with Rails either [confusing or strange](https://groups.google.com/d/msg/rails-api-core/pGSWLiWUsCw/-BdKF4pAPwAJ).  [I've traced it back around here](https://github.com/rails-api/active_model_serializers/blob/fd98349c540d969681ce49ead4fc2174099e605d/CHANGELOG.md#first-commit-as-rails-serializers-001), and was surprised  to see that it was actually merged in at one point, then removed.


@tarunc are you going to publish this ?

It solves a (minor) security issue in the debug --> ms package

See https://snyk.io/vuln/npm:ms:20151024


Oui oui, mais j'ai mis l'image pour la page "contribuer au projet", ce sera au autre site d'utiliser cette image ;)


TeamCity LabKey :: Trunk :: Premium :: Git postgres 9.4 [Build 876](https://teamcity.labkey.org/viewLog.html?buildId=293686&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 15 (1 new), passed: 248 Build time: 00:34:46
### Failed tests
```
org.labkey.test.tests.RedCapTest.testDataTypes: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testDatasetExport: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelection: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelectingEmptyMeasure: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnDataset: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnStudyNavigator: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testGroupSaveAndLoad: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStudyParticipantCounts: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testCounts: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSearch: <no details avaliable>

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

##### there are 5 more failed tests, see build details
```


> The Manipulation system provides a flexible and low level api that enables any custom tool to be build based on the provided events. No assumptions are made about the type of tool being build, or how manipulations should be defined. The major DCCs expose APIs for developing custom tools that are heavily limited in flexibility due to the assumptions made about the kinds of tools that TDs in production might need to build.

マニピュレーションシステムは、柔軟性のある低レベルなAPI提供します。そのため、提供されるイベントを元にどのようなカスタムツールでも作成可能です。（ ~中略~ ）一方、既存の主要DCCでは柔軟性を著しく制限されたAPIが提供されています。なぜならプロダクションのTDが「どのようなツールを必要とするか」についてDCC側が予断をもっているからです。

---
* ここでの主題は 既存のDCC（での拡張） vs Fabric Manipulation（での拡張）の対決の話なので対比がわかるようにしたほうがよさそうです
* ↑のため適度に補ってください


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


What if bAHI_Clock32MHzStable() is not used in this clock_init() function code?
As per Datasheet of JN516x, until 32MHz Crystal is stable, RC oscillator is selected by default and this takes approx. 1 ms. What if this API is not included? Thanks in advance


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


postしたらuserのpathに送り返すってことかな。


According to my checking there is no extra space here. But I'll push again, just to make sure.


Ok :)


`JSON` - это строка (определённого формата).

Может быть, так и написать, что возвращает объект с атрибутами `dateBegin` и `dateEnd`?


It means remove -_-
We're not stupid you know. I think we understand commands perfectly fine.


Please, move files to ``tools/`` as in PASA repository.


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


Could we provide docstring for whole module with description what exactly formatting and how?


@pestOO, I'll move the whole dir `infrastructure` to tools. Ok?


May be call ``walk_dir_recursive``?


Ok - replace description
getDateInterval return object with dateBegin, dateEnd


@pestOO, What about optional flag `recusive = True`?


@malirod, look at PASA utils folder. For WinPort it could be ``infrastructure``.


I guess the comment "Skip report GUI..." should be removed as well?


Working on it.  Gandi is being slow.


> I think it is important to note that reliability and congestion control are not mutually exclusive [...]

True.

> [...] If I'm right, and we revert the commit instead of modifying the tests, we'll be creating a technical debt.

I agree that your arguments on API design are very appealing. Unfortunately, rust-utp follows TcpStream design closer than your suggestion. [My little test](https://gist.github.com/vinipsmaker/2b09324e497e36ec90cf) shows that.

Fortunately (somehow), the blocking call ensures `Ok(0)` will only be returned when EOF/socket-closed is reached. It'll loop forever until such condition happens. That's the reason why this revert works well.


this shows 0 file changed
please check the file which was supposed to be changed is still the same


`get` is deprecated too: you should use `connection`: see https://github.com/concrete5/concrete5/blob/5.7.5.3/web/concrete/src/Database/DatabaseManager.php#L68-L77


Students/30422/Sfirlea I. Alexandru Ioan/joc_varianta_finala/src/GUI/BoardPanel.java

Pentru că `returnStatus()` returnează un boolean, nu mai trebuie să compari cu true sau false că e redundant.
Poți scrie direct
` if (game.returnStatus()) {  ... `


Students/30422/Sfirlea I. Alexandru Ioan/joc_varianta_finala/src/Calculation/BackLogic.java

metodele astea 2 pot (și ar trebui) să nu fie statice


@malirod, the best option =)


Ai putea salva scorurile mai elegant puțin prin serializare sau măcar cumva în același fișier, dar să zicem că e ok


testing


Nu am nimic altceva de adăguat. Great work!


Сессия в Yii уже есть, для нее есть все необходимые обертки, ее не надо стартовать.


Why have you removed `animationFinishedCallback`? this just needs to be called at the end of `animate` or in appropriate animation cases. In this change the callback isn't going to be called when the animation ends.


Работай с сессией через YII


Yeah, it was not tagged yet. Should be good in a couple of minutes (`v0.2.2`)


Зачем создаешь объект если метод статический??


Maybe a better idea is to complete remove "django", because "django-cms" will install it via ``install_requires``


К сессии обращаться тоже средствами Yii


yeepee


ha merde on as un probleme avec ce test :(

Le 27 décembre 2015 à 23:28, alallema <notifications@github.com> a écrit :

> J'arrive pas ta manip avec leaks
> J'ai mis :
> leaks $PID > fillit ($PID 1344)
> Il me dit qu'il prend pas les parenthèses.
> Et figure toi que mon test_difficile tourne encore alors que je l'ai lancé
> hier matin !
>
> Envoyé de mon iPhone
>
>
> > Le 27 déc. 2015 à 19:43, alallema7@gmail.com a écrit :
> >
> > Cool parce qu'en fait j'ai un dîner ce soir encore du coup ... Je sais
> pas quand je vais rentrer. Bonne soirée
> >
> > Envoyé de mon iPhone
> >
> >> Le 27 déc. 2015 à 19:34, schiad <notifications@github.com> a écrit :
> >>
> >> ne te presse pas...
> >>
> >> Le 27 décembre 2015 à 19:17, alallema <notifications@github.com> a
> écrit :
> >>
> >> > Je sors du boulot je regarde ça en rentrant
> >> >
> >> > Envoyé de mon iPhone
> >> >
> >> > > Le 27 déc. 2015 à 16:24, salimchiad <notifications@github.com> a
> écrit :
> >> > >
> >> > > demarre un ./fillit ./test/25 , tu lance top tu prends le PID de
> fillit,
> >> > et tu tape leaks $PID > fillit($PID le numero de PID de fillit), et
> ouvre
> >> > le fichier (si tu fais pas dans un fichier tu rates le début du
> résultat)
> >> > >
> >> > > —
> >> > > Reply to this email directly or view it on GitHub.
> >> > >
> >> >
> >> > —
> >> > Reply to this email directly or view it on GitHub
> >> > <
> https://github.com/alallema/fillit/commit/b3fedb7e3d3852bd7bb8d996f56c1ae3ba2d8e79#commitcomment-15168105
> >
> >> > .
> >> >
> >>
> >>
> >>
> >> --
> >> Cordialement CHIAD Salim.
> >>
> >> Étudient école 42 <https://fr.wikipedia.org/wiki/42_(%C3%A9cole)>,
> école
> >> d'informatique révolutionnaire.
> >> 42 <https://fr.wikipedia.org/wiki/42_(%C3%A9cole)>school student,
> computer
> >> programming school.
> >> —
> >> Reply to this email directly or view it on GitHub.
> >>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/alallema/fillit/commit/b3fedb7e3d3852bd7bb8d996f56c1ae3ba2d8e79#commitcomment-15168756>
> .
>

--
Cordialement CHIAD Salim.

Étudient école 42 <https://fr.wikipedia.org/wiki/42_(%C3%A9cole)>, école
d'informatique révolutionnaire.
42 <https://fr.wikipedia.org/wiki/42_(%C3%A9cole)>school student, computer
programming school.


We shouldn't override the icons position even if there is text in the button. That is for the user to decide.


Названия переменных не говорят об их содержимом


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


> Only one manipulator can be active at any given time.

一度にアクティブにできるマニュピレータは常に１つだけです。

> All events from the DCC are propagated to the active manipulator for handling. DCC are propagated to the active manipulator for handling.

DCCより送られる全てのイベントは、アクティブなマニュピレータに伝搬し処理されます。

（群を補わなくとも単複区別できる箇所は無いほうがよみやすいです）
（この文の主題は 「（アクティブな）マニュピレータがハンドルする」というところなので for handling の役をそのようにします）


работа с POST средствами Yii


Did you mean "iUsage:Uint" (the colon is missing) ?


@rskennymore the ` public void onModule(IndicesModule indicesModule) ` methods are called via reflection which is something we are actively working on to remove and to have a clear interface. If you want to call a method on `IndicesModule` to register a mapper you just add such a method to the Plugin class and it _magically_ happens. I hope we can trash this behavior for 3.0


what about: content += "\n" + options.reason?


почему все методы статические? что это дает?


Should we always do this?


Dependency Injection для самых маленьких.


what's considered to use 'skynet.call', not 'skynet.send'?


This is certainly 100% better


Hi Wanjia,
Please notice that either uppercase or lowercase is OK for CMake command, however, the CMake variable must be upper case.


`::=`, but yes


I think we should also move the user to #general, in case this happens


「ファブリックエンジン」 もなるべく 「Fabric Engine」 のママのほうが良さそうです


Are you sure all these should be expanded instead of contracting lines 225-235?


This agrees with our idea, sure, but it also feels like such a waste of space.


This is better since the above and below functions are expanded.


Solves #686 and #685


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1072](http://teamcity.monogame.net/viewLog.html?buildId=12963&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 555, ignored: 6 Build time: 00:25:32


Hi back!

Thanks for the helpful replies, no apologies necessary - it's busy over the holidays anyway :)

The scrollbar was my issue, and I can get the screen size now, but have come unstuck again where you point out that the edit boxes have not been inserted yet during `<<load>>` processing, so that's still not the right place for me to tile multiple opened "buffers".

You can turn on issues for your forked mc by ticking the checkbox at `Settings | Features | Issues`.

Thanks for all your work on this: Lua+MC = awesome!  I'll wait until you've had chance to publish your rerolled branch, and if you have turned issues on then, I'll raise any other questions there separately rather than polluting this pull request any more.

Thank you.
jj


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


Maybe focus on these obvious improvements for a start.


Separate by a new line? I think we should do this unless the file is very small.


Yes, right. But that shouldn’t be an issue.


TeamCity LabKey :: Trunk :: Community :: CDS postgres 9.4 [Build 49](https://teamcity.labkey.org/viewLog.html?buildId=293683&buildTypeId=LabkeyTrunk_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 13, passed: 35 Build time: 01:11:45
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyDensePlotBrushing: <no details avaliable>

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

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

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

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

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

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyCounts: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGroups: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyProductsItem: <no details avaliable>

##### there are 3 more failed tests, see build details
```


Here you've got three functions in a row which could be one-liners. How many are enough to warrant it?


On the way, remember? :)


ertertertre


Awesome - much aporeciated!

Sent from my iPhone

> On 28 Dec 2015, at 5:00 PM, Barry vd. Heuvel <notifications@github.com> wrote:
>
> Yeah, it was not tagged yet. Should be good in a couple of minutes (v0.2.2)
>
> —
> Reply to this email directly or view it on GitHub.
>


Please replace C style loop.


argh... cut-n-paste mismatch :(  Of course, that should be:

```
function (me)
  me.fullscreen = false
  ...
```


I think one of those exceeded 200 characters by far, so I opted to do it like this.


这个「默认查询」改动好像有问题，比如 `rt.conv(convId)` 原来发出去的 query 命令 where 参数是 `"where":{"objectId":"7331eb5779b501..."}`，而现 2.3.3 版本是  `"where":{"objectId":"7331eb5779b501...","m":"leancloud-test"}`
@wangxiao


looking good


Well, we need to be consistent about this. Otherwise we'll have to do another very large cleanup in a few months. Now is the time to decide, and I say we do it like this.


fdsfdsfsdf


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


Please replace nested if's with guard to reduce the nested blocks complexity.
* I know you didn't change the if statements but do it to follow the Boy Scout Rule :boy:


@leeyeh 收到，我看下的。


Rm -rf wouldn't even work on windows.

> Am 28.12.2015 um 09:43 schrieb Happy New Keither <notifications@github.com>:
>
> It means remove -_-
> We're not stupid you know. I think we understand commands perfectly fine.
>
> —
> Reply to this email directly or view it on GitHub.
>


They are surrounded by non-oneliners, if they should be compacted, they should also be moved to either the top or bottom of the function group. I opted for this way, since `join()` have an overload which can not be compacted.


Of course it wouldn't - it's a UNIX command.


now lua support int64, is it safer to use int64？


But you wouldn't say that if you had a thousand one-liner functions here. So where is that magic limit?


At some point, I'd probably remove these :)


dasa


I'd say two or more. But, it has to fit the context and overall style of the file. Otherwise we might as well throw this guideline out.


Trochu mi vadí, že se to jmenuje _choice a je to funkce, ale zpětně je to asi jedno...


I have added spaces because strings like "INFORM_SERVICE_CENTER_REQUEST" didn't fit into the table cell. With spaces these strings is now as
INFORM_SERVICE_
CENTER_REQUEST.
Maybe these strings must be divided as
INFORM_SERVICE
_CENTER_REQUEST?


ey


But here we have three? :) The fact that the next function is an overload of `join` shouldn't really matter.


I think we pretty much agree that
- One-liners are compacted if the line does not exceed ~200 characters
- When there's a mix of one-liners and multi-liners, the expanded form is used
- Functions and their overloads are always together and should not be moved around
- When there are `>X` one-liners together surrounded by multiliners they should be compacted. `X` is 2 or 3.
- Include an empty line between blocks of members, properties, functions, operators, static etc. for all but very small classes

But we should have comments from @simonmika as well


OK I guess that's fine then. I was just concerned some people may copy paste it


As I said, it has to fit the style of the file and compacting here clearly violates that. Not a single person here would be able to keep track of this. I'm not bending on this, so either we do it like this, or we throw the rule out.


This has been updated in commit 88f9b10925619e4d6af2912506312be14ee6803b.


I don't understand. I have no trouble reading the original code here.


True. Just fixing the worst part.
the URLs will be factored out once the bad one are removed.


Why did you `var` this?


Tady je to zrychlení? Nějak ho tam nevidím, mohl bys mi to @matllubos vysvětlit?:)


Well, it is true that there are lot of places where we do that. However, if a wrong change is made in the GUI files, the game is pretty sure to crash and we will have a backtrace telling us where the crash occured.
IMHO, the dependency to the xml files is a pretty annoying thing as it may allow to have synchronization problems but I guess we don't have much choices...


Again, there are a lot of places where that will be true in the code...


> "+#Pull Request #4 - don't make an issue if this is your problem. I don't give a shit if you cant solve a dependency issue."
> Wow rude. Also other people on Ubuntu based distros had this problem too and it was a problem with your make.

It's not my job to fix a make issue. I'm simply maintaining and fixing it if a CSGO update completely breaks this. If someone fixed it and makes a PR sure I'll accept it.

If they dont know how to fix it, its not my problem.


great job


How about also making sure the error message sounds good when `filename` is `None`? After all, `filename` is optional.


Done, thanks.


The problem is not now, the problem will be when people are adapting to the guideline, as there are no clear way of implementing what you say without violating the guideline:
> When there's a mix of one-liners and multi-liners, the expanded form is used

Functions should only be compacted if a clear group pattern exists, either in current form or by reordering.


Use following code to get abbreviation of attribute value:

frappe.db.get_value("Item Attribute Value", {"parent": attribute.attribute, "attribute_value": attribute.attribute_value}, "abbr")


I think the old version is better. This one needs an extra variable.


发 0.12 beta 就行。。。


Deklarasi window scroll seharusnya setelah event click di-attach di button load more. Hal itu untuk menghindari terjadinya error pada saat user sudah mulai scroll sebelum button load more ter-attach event click-nya.


What's a group *pattern*? Didn't we agree that `2 or more` is a group in this sense? The function names or contents shouldn't matter.


How am I supposed to know which ones can be removed safely? The IDE added them, I assume for a reason. If I remove all except the RGiesecke one I added myself it still compiles fine but then the DLL doesn't work anymore.


Another debug-variable is the `dflag` used in `optimize.c`. But allocated in a common-data section (a COMDEF on Windows/OMF?) and *not* exported. `tests/filtertest.c` declared this as global when:
```
#ifdef BDEBUG
int dflag;
#endif
```
but on the stack otherwise. So using `filtertest.exe` with `pcap.dll` makes no sense IMHO when `BDEBUG` is defined; no difference in output with any number of `-d` options.


Cek tips yang saya kasih di commit sebelumnya mengenai ini.


Because i was on mail the whole time:
Finally saw the code.
> This has absolutely nothing to do with auto-updater.

Thats right. This will just mess up textformatting! I am for reverting this


Gunakan strict equality.


`$(window)` digunakan berkali-kali. Mengapa tidak disimpan ke sebuah variable?


Kenapa tiba-tiba ada new line sebelum `==`?


Imagine they will change § to something else. Like &.
This would mess up all colors then


I screwed it up: should be fixed in 546b266dcc98738880778e4f34f1b8e8604b695b.

The motivation for changing standard_kaldi's `transcribe` behavior to keep the pipe alive was the extremely high startup cost of loading the full language model.


@liangent 搞定！现在怎样？


Fixed in f8cccda. (note that filename is likely not to be empty in most cases)


oh new linenya biar ga kepanjangan aja mas if statementnya, kan ga boleh
ngelebihin garis 80 best practicenya

2015-12-28 16:34 GMT+07:00 Bobby <notifications@github.com>:

> Kenapa tiba-tiba ada new line sebelum ==?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sebastianusbara/shop/commit/a3d255e9492edfbabf03de6f4d93862a3ce0ddfc#commitcomment-15172000>
> .
>


huai zu, bi ge very high a !!!


I find `func removeDuplicates()` or `var unique: [Element]` as a better name/solution.


Sorry, it does still work if I remove all except the RGiesecke one. But I still can't help but wonder how I know for sure this is safe to do?


Do you want this to be merged or prefer the shipping of modprobe rules?


Already reverted -_-


Issue https://github.com/stopstalk/stopstalk/issues/37 done!


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1073](http://teamcity.monogame.net/viewLog.html?buildId=12967&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 555, ignored: 6 Build time: 00:28:43


this would have been better bundled with your next commit...  ; )

(You had me wondering why this was added, and I had to go check if there are any other commits where this is used.)


We sure did, but how do you suggest we follow the guideline
> When there's a mix of one-liners and multi-liners, the expanded form is used

then?


Is it the same as #323?


What would that produce for attributes that are numeric?
![numericvalue attributes](https://cloud.githubusercontent.com/assets/11225818/12016790/1038800e-ad58-11e5-8902-d3490715dc29.JPG)


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


在 chrome 下部分图标有显示切边问题。


Лучше не использовать сокращений в идентификаторах. Лучше - `OtpGenerator`


更新2.0适配


优化


Interesting. I like this kind of technical discussion. I was new to c++ when I came to this project nearly 2 years ago and my coding style has improved a lot by looking at how OD was done and thinking about what I liked or not.

Concerning your comment, I would say that the if the hierarchy changes, your code will also crash and need the same changes (getting the button from the right path) as the previous one.
IMHO, the good point with your code is, as you said, to not have to change the hearders if your method changes name. The bad point is that the use of lambda makes it a little less readable.

Since the function name is not likely to change and the loss of readability using lambda is not very high, I would have no preference for a choice or another.


if I understand correctly, this code doesn't currently handle the case where the user was viewing messages in the room that has been removed..?  (or does it?)

Do you think we should add that here ? (e.g. by taking them to #general)


Ah, magic. Got it :)

Thanks for the quick response, happy holidays.


Данный класс лучше разделить на два отдельных, т.к. сейчас `OtpGen` и хранит данные об OTP, и генерирует OTP. Лучше создать дополнительный класс `OneTimePassword`.


to je jednoduché v readonly fields se původně procházeli všechny choices na modelfieldě, u objednávke to byly tisíce. Já vytáhnu jen tu jednu konkrétní


>  I'm happy to have one more active adopter!
Congratulations, i didn't find any editor for my usage and in enki i found what i was looking for as i don't like IDEs but a simple editor focused on coding.
I like the philosophy of enki, keep going !


鼓掌！


Maybe that's too vague. What I meant was that when one (or possibly two?) one-liner is/are surrounded by multi-liners, they are expanded also, to fit the context. But if there are more than two, say, then these form a group by themselves, and follow their own context, which is to be contracted.


更新


skynet.send: you just notify target, but don't need or care the response
skynet.call: you need response of target.

skyent.call will yield and wait respone to wake up

2015-12-28 17:08 GMT+08:00 snail <notifications@github.com>:

> what's considered to use 'skynet.call', not 'skynet.send'?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cloudwu/skynet/commit/1c2654e117a854d5b43102dcb392102e633f1108#commitcomment-15171720>
> .
>


These two don't have to be static


I can't reproduce that problem and the way the code works is that this function is only called right after a redraw, so it shouldn't happen when the widget hasn't been drawn. Can you submit an HTML page that triggers the problem?


Installasjons-pakken blir ignorert


This conditional will always evaluate to true.  I don't think it will matter very much but it's worth noting.


LGTM


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


Now this I do not at all like!
This should be a `Player` class with all these attributes made non static! Then sure, in the `MainLaunchFrame` class, you can create such a player and then pass that reference around to whichever place you need it, but do not have anything static there.


@copumpkin I think you forgot the version here. Anyway, thanks for this, definitely need to try hugo!


Jasně, takže tenhle řádek je ta optimalizace. Už chápu...


Again, everything except the `serialVersionUID` should not be static. Find a way to reference these variables by not having them static here.


@KristianTashkov good catch! If you want to create a PR it'll be welcome and I can help you if you have any problem, otherwise let me know and I'll fix this


Нужно переписать `if` согласно JCC. Во-вторых, я не понял условие: сейчас `Bad Request` возвращается, когда пароли не совпадают или когда `ExpirationDate` позже текущего времени. А нужно, чтобы `ExpirationDate` был до текущего времени.


Ok need to revise the method, Still studying the way the transition is applying a callback.


Your welcome!


Yeah, but in such case I would relocate that group to the top or bottom of the block of functions. But here, one of the one-liners ( `join()` ) have an overload which spans over multiple lines, so expanding it to fit that overload takes precedence, in my opinion.


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


Эээ. Я за вариант через ресурсы все же делать, как и раньше. Но с одной константой.


Use `or` condition for that case

```
attribute_abbr = frappe.db.get_value("Item Attribute Value", {"parent": attribute.attribute, "attribute_value": attribute.attribute_value}, "abbr")

item_code_suffix += "-" + (attribute_abbr or attribute.attribute_value)

```


use "if not timer.description" which will also match "None" for example.


哈哈，坚持下来不容易啊。现在准备转换成PDF版本。

----
Neng Xiong
WHU, Mathematical and Statistical Institute
Mobile (0086)18520022428
Alternative email, yidao620@163.com

On Mon, Dec 28, 2015 at 5:48 PM, latyas(懒) <notifications@github.com> wrote:

> 鼓掌！
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/yidao620c/python3-cookbook/commit/0bb36ff87efedfdd0be7f923e4d3ee8350b37fc7#commitcomment-15172158>
> .
>


更新


Do we really need MORE logging? The log is overburdened already...


Зачем, нам генерировать `OTP` при деактивации пользователя? У нас же пользователь уже авторизирован.


Statistic.params() ?


красава Лёх


I've just noticed that this change induces that previous values of "rounded_values" will be lost, so ranges configured for rounded values might misbehave (as the default value is false). The change is worth it though IMO as it makes it clearer that it's a range parameter (and not e.g. of a HSlider which extends Range).


J’avais changé pour la date de modification à la place de la date de création pour pouvoir utiliser ce timestamp avec « newer-than », et ainsi recevoir une nouvelle version des messages qui ont été modifié. Après il est possible de rajouter une nouvel attribut pour avoir les deux timestamps et tu as peut-être une autre idée pour récupérer les messages modifiés (d’ailleurs là les très vieux messages modifiés (càd plus vieux que les 10 derniers messages affiché dans le flux de live) sont récupérés ce qui n’est pas forcément souhaitable).


Extra data is fine


I want to allow only classmethod like invocation to prevent crazy errors. So if you override `check_prerequisites` you need to make it a staticmethod or a classmethod, something else won't work.


> **Devuelve** las migraciones aplicadas a nuestro repositorio

Aqui tal vez sea mejor colocar:

> **Revierte** las migraciones aplicadas a nuestro repositorio


This little change fixes a bug raised when trying to install the development version of `django-tagging` via `pip` on Python 3, in order to get it working with `Django` version 1.9. I'm using `Python` version 3.4.3 and `pip` version 7.1.2 on `Ubuntu GNU/Linux` 14.04.1 64 bits.

When I ran this command on a Bash terminal:

```
user@laptop:~/code$ pip install git+https://github.com/Fantomas42/django-tagging@develop
```

I got this error output:

```
Collecting git+https://github.com/Fantomas42/django-tagging@develop
  Cloning https://github.com/Fantomas42/django-tagging (to develop) to /tmp/pip-ctuzj62x-build
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 20, in <module>
      File "/tmp/pip-ctuzj62x-build/setup.py", line 15, in <module>
        open('CHANGELOG.txt').read()]),
      File "/home/osg/virtenvs/practical-cms/lib/python3.4/encodings/ascii.py", line 26, in decode
        return codecs.ascii_decode(input, self.errors)[0]
    UnicodeDecodeError: 'ascii' codec can't decode byte 0xe2 in position 900: ordinal not in range(128)

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-ctuzj62x-build
```

Then I edited the mentioned CHANGELOG.txt file (with `vim` editor in binary mode: `vim -b /tmp/pip-ctuzj62x-build/CHANGELOG.txt`) and after seeing some weird characters in the original 39th line, I changed them by a single hyphen character and ran this command to install the module via `pip` from the local file system:

```
user@laptop:~/code$ pip install /tmp/pip-ctuzj62x-build
```

The result was successful!


TeamCity pull request submitted :: build source [Build 16](http://localhost:8111/viewLog.html?buildId=44&buildTypeId=NodeProjects_BuildSource) is now running


TeamCity pull request submitted :: build source [Build 16](http://localhost:8111/viewLog.html?buildId=44&buildTypeId=NodeProjects_BuildSource) outcome was **SUCCESS**
Summary: Running Build time: 00:00:02


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


done


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


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


How would I put this in my files without the whole compiling thing, would copypasting source/AI.cpp into my sources work? or would it require something more drastic


This breaks #2541 **Serialization time zone fixes**. See #5127 for discussion


This breaks #2496 **Fix serialization test failure in Minsk timezone**. See #5127 for discussion


> section of

eliminar al final de la linea. de lo contrario creo que el resultado sera  algo como:

> Para mayor detalle acerca de las migraciones vea la sección [Ecto.Migration](http://hexdocs.pm/ecto/Ecto.Migration.html#content) **section of** de la documentación oficial.


It's the only user of HAVE_SOCKET and HAVE_ACCEPT4 so maybe also remove the checks from the build system?


Do you need a binding anyway? MRI:

```ruby
def used_as_block
end

def accepts_block(&block)
  p block.binding
end

accepts_block(&:used_as_block)
```

```
test.rb:5:in `binding': Can't create Binding from C level Proc (ArgumentError)
	from test.rb:5:in `accepts_block'
	from test.rb:8:in `<main>'
```

Is the 'C level Proc' some kind of indirection written in C that does `send(:symbol)`?


#2 fixed. See `README.md`


this introduces [some weirdness](https://github.com/HaxeFoundation/haxe-output/commit/6713c428a7c9fb50a257bdf355d4996c4e3fbba9). i have more in my codebase, will extract tomorrow


#2 added. See `README.md`


Need this console.log outta here


The only way to modify the game executable is by recompiling it.


Hmm, yeah I thought you would get notified as well. If you want we can take this over email instead so you don't have to come back here checking. patrik@ptjwebben.se

Sure, I'll fix the blue so its a bit closer to the bar, do you want me to use the same blue on both of them or should the dark be a bit darker like in the example?


Fixes also thymeleaf/thymeleaf#444


Unused require?


What was the reason for this move?


TeamCity pull requests :: netty [Build 912](https://garage.netty.io/teamcity/viewLog.html?buildId=4323&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 6381, ignored: 21; resource leak detected (new) Build time: 00:44:11


TeamCity pull requests :: netty [Build 913](https://garage.netty.io/teamcity/viewLog.html?buildId=4324&buildTypeId=pulls_netty&guest=1) is now running


I've heard good things about tape, and substack knows his stuff, so im cool with it :+1:

Replying to this via email, so hopefully it posts correct

-----Original Message-----
From: "Artur Paikin" <notifications@github.com>
Sent: 12/28/2015 4:50 PM
To: "transloadit/uppy" <uppy@noreply.github.com>
Cc: "Harry Hedger" <hedgerh@gmail.com>
Subject: Re: [uppy] Basic testing implemented with Tape (5f8ce0c)

Did some research, decided to try and go with Tape instead of the usual Mocha + Chai. Reasoning: simpler API & small all-in-one core, not polluting the global space with it, should and others. Explained here: https://medium.com/javascript-scene/why-i-use-tape-instead-of-mocha-so-should-you-6aa105d8eaf4#.ivby33v87/.
Mocha + Chai are more common though, and if you guys are for a conservative and more widely adopted solution here, we can easily switch, now or later. Thoughts, @hedgerh @kvz?
—
Reply to this email directly or view it on GitHub.


I was afraid that would be the case. If i could figure out blender I'll figure out blocks eventually I guess


ack


This is my first repository on GIT.


I'm pretty sure process ID is ignored for TH32CS_SNAPTHREAD, according to the spec


TeamCity node build testing :: PR submitted [Build 17](http://localhost:8111/viewLog.html?buildId=46&buildTypeId=NodeBuildTesting_PrSubmitted) is now running


TeamCity node build testing :: PR submitted [Build 17](http://localhost:8111/viewLog.html?buildId=46&buildTypeId=NodeBuildTesting_PrSubmitted) outcome was **SUCCESS**
Summary: Running Build time: 00:00:01


Instead of making an arbitrary ad-hoc logging prefix here (and using console instead of the logger), maybe we could take a second to clean up the framework-level logging in general. Presumably the intent of this change was to help distinguish this Summit log from some in-app error log? If this is an issue then let's address it head-on.


+1 for this!

Btw, you'll need a `,` in the phantomjs object:
```
  phantomjs: {
            viewportSize: {
                width: 1024,
                height: 768
            },
            useColors:true
        }
```


I want to express my gratitude for your work once again, Grz3s. I wish you happy holidays!


TeamCity node build testing :: PR submitted [Build 18](http://localhost:8111/viewLog.html?buildId=48&buildTypeId=NodeBuildTesting_PrSubmitted) is now running


TeamCity node build testing :: PR submitted [Build 18](http://localhost:8111/viewLog.html?buildId=48&buildTypeId=NodeBuildTesting_PrSubmitted) outcome was **SUCCESS**
Summary: Running Build time: 00:00:01


## Express
El framework **Express** en un conjunto pequeño de *web app features*, reducidas al mínimo para mantener el estilo de node.js. Está basado en **Connect** y usa la arquitectura de sus middleware. Sus features extienden Connect para poder incorporar casos de usos comunes en web apps como la inclusión de motores de templates modulares (HTML), extender el **response object** para soportar varios formatos de salida, un sistema de ruteo y mucho más.

### El objeto app

- **app.set(name, value)**: Usado para setear variables de entorno que Express usará en su configuración.
- **app.get(name)**: Usado para obtener variables de entorno que Express usa en su configuración.
- **app.engine(ext, callback)**: Usado para definir un *template engine* dado para renderizar ciertos tipos de archivos. Por ejemplo app.engine('html', require('ejs').renderFile)
- **app.locals**: Usado para enviar variables de aplicación a todos los templates renderizados.
- **app.use([path], callback)**: Usado para crear un Express middleware para manejar una HTTP request enviada al server. Opcionalmente podremos montar middlewares para responder a ciertos *paths*.
- **app.VERB(path, [callback...], callback)**: Usado para definir una o más funciones middleware para responder a HTTP requests hechas para cierto *path* en conjunto con el verbo HTTP declarado.
`app.get()` `app.post()`
- **app.route(path).VERB([callback...], callback)**: Usado para definir una o más funciones middleware para responder a HTTP requests hechas para cierto *path* en conjunto con múltiples verbos HTTP.
`app.route(path).get(callback).post(callback)`
- **app.param([name], callback)**: Usado para bindear cierta funcionalidad a cualquier request hecho a un *path* que incluye cierto parámetro de ruta. Se puede mapear una lógica a cualquier request que incluya `user_id` por ejemplo:
`app.param('user_id', callback)`

Hay muchos más métodos. Estos son los básicos.

### El objeto response

- **res.status(code)**: Usado para setear el estado de una respuesta HTTP.
- **res.set(field, [value])**: Usado para setear el header de una respuesta HTTP.
- **res.cookie(name, value, [options])**: Usado para setear una cookie de respuesta.
- **res.redirect([status], url)**: Usado para hacer un *redirect* a una URL dada. Notar que puede pasarse el estado, el *302* está por defecto.
- **res.send([body|status], [body])**: Usado para respuestas *non-streaming*. Hace mucho trabajo por detrás como setear el *Content-Type* y *Content-Length* headers, y responder los cache headers correspondientes.
- **res.json([status|body], [body])**: IDEM *res.send()* pero para cuando enviamos un objeto o array.
- **res.render(view, [locals], callback)**: Usado para renderizar una vista y enviar una respuesta HTML.

### El objeto request

- **req.query**: Contiene los parámetros *query-string* parseados.
- **req.params**: Contiene los parámetros de ruta parseados.
- **req.body**: Usado para obtener el `body` parseado de la request.
- **req.param(name)**: Usado para obtener el valor de un parámetro. Notar que este mismo puede ser un *query-string*, parámetro de ruta, o una propiedad del body de una request JSON.
- **req.path**, **req.host**, and **req.ip**:  Usados para obtener el *path* de la request actual, nombre del host o IP remoto.
- **req.cookies**: Usado junto al middleware *cookieParser()* para obtener las cookies enviadas por el *user-agent*.

### Middlewares externos

El núcleo express es mínimo, aún así provee de middlewares predefinidos para manejar las funcionalidades web más comunes. Estos varían en tamaño y funcionalidad, extienden a Express para ofrecer un mejor soporte. Los más populares son:

- **Morgan**: Logger de requests HTTP.
- **body-parser**: Usado para parsear el `body` de la request, soporta varios tipos de requests.
- **method-override**: Soporta verbos HTTP como *PUT* o *DELETE* en lugares donde el cliente no lo hace.
- **Compression**: Comprime la data de la respuesta usando *gzip/deflate*.
- **express.static**: Sirve de archivos estáticos.
- **cookie-parser**: Parser de cookies el cual llena el objeto *req.cookies*.
- **Session**: Para persistir sesiones.

Hay muchos más middlewares que nos ayudan a reducir nuestro tiempo de desarrollo.

**Nota:**
> Para saber más de Connect y Express middlewares [Aquí](https://github.com/senchalabs/connect#middleware).
> Si queremos revisar la lista de middlewares externos [Aquí](https://github.com/senchalabs/connect/wiki.)

## Implementando MVC
El framework express es patrón agnóstico. Quiere decir que no soporta una sintaxis o estructura predefinida. Implementar MVC en una aplicación Express significa que se puede crear carpetas específicas para alojar los archivos *js* en un cierto orden lógico.
Por esto mismo es muy versatil y va en gustos de cada uno. Hay dos metodologías conocidas (que incluso pueden mezclarse) como: **Estructura Horizontal** o **Estructura Vertical**.
A grandes rasgos la horizontal mantiene simplicidad y separa los archivos por concepto. La vertical separa los archivos por features, cada uno de estos features se compone de una estructura similar a una estructura horizontal. Esta última se puede ver más que nada en grandes proyectos, de esta forma puede modularizarse, reusar funcionalidades, trabajar en equipos facilmente, etc.

**Nota**:
> Ver la estructura básica del proyecto al momento de este commit. Se usa la estructura horizontal.


TeamCity node build testing :: PR submitted [Build 19](http://localhost:8111/viewLog.html?buildId=49&buildTypeId=NodeBuildTesting_PrSubmitted) is now running


TeamCity node build testing :: PR submitted [Build 19](http://localhost:8111/viewLog.html?buildId=49&buildTypeId=NodeBuildTesting_PrSubmitted) outcome was **SUCCESS**
Summary: Running Build time: 00:00:01


TeamCity node build testing :: PR merged [Build 1](http://localhost:8111/viewLog.html?buildId=50&buildTypeId=NodeBuildTesting_PrMerged) is now running


TeamCity node build testing :: PR merged [Build 1](http://localhost:8111/viewLog.html?buildId=50&buildTypeId=NodeBuildTesting_PrMerged) outcome was **SUCCESS**
Summary: Running Build time: 00:00:01


Could be out of RAM?


If this is patching something to shut up a dependency, can we fork that dep, fix it, and require that version instead?


why lower case with underscore style?


why struct instead of class?


@Makman2 this is a *beautiful* solution!


LGTM!


I've recently been switching to opening braces on the same line as the function def.  This is (I think, but I should look) what we currently say in our style guidelines.  We could talk about it when we meet tomorrow...


@equalsraf, this seems related: https://msdn.microsoft.com/en-us/library/17618685.aspx

```c
#ifndef HAVE_FSYNC
#ifdef _MSC_VER
#define fsync(fd) _commit(fd)
#endif // _MSC_VER
#endif // HAVE_FSYNC
```

PostgresSQL and BerkeleyDB guys seems to think so: http://www.postgresql.org/message-id/Pine.LNX.4.21.0302041000590.24830-100000@linuxworld.com.au. :smile:


Looks like something related to for-loops. Maybe some type is off in my rewrite.


The LSL7, Lighthouse and RAMA game demos trigger errors, before even reaching game play, There is no point in adding game demos at that early point, as it will only cause complaints.

If game demos with known issues are to be added in future, then at least wait until the game demos reach the play testing point, as we do for full games.


Lo que hacemos aquí es usar un patron de CommonJS para definir una función llamada `render`. Luego vamos a poder hacer un `require` de este modulo y usar esta misma función. Una vez que tenemos el controller tenemos que rutear para poder utilizarlo.


Well that was simple! :stuck_out_tongue:


For several reasons initialization should happen here, e.g. `int current = -1`, instead of doing it in the constructor body.


Fixes #115


do przeczytania obowiązkowo:
https://www.owasp.org/index.php/SQL_Injection_Prevention_Cheat_Sheet


TeamCity node build testing :: PR submitted [Build 20](http://localhost:8111/viewLog.html?buildId=51&buildTypeId=NodeBuildTesting_PrSubmitted) is now running


If you make a pull request to the original branch, I'll review the changes and merge them if compatible with the newer versions of the file


TeamCity node build testing :: PR submitted [Build 20](http://localhost:8111/viewLog.html?buildId=51&buildTypeId=NodeBuildTesting_PrSubmitted) outcome was **SUCCESS**
Summary: Running Build time: 00:00:01


TeamCity node build testing :: PR merged [Build 2](http://localhost:8111/viewLog.html?buildId=52&buildTypeId=NodeBuildTesting_PrMerged) is now running


TeamCity node build testing :: PR merged [Build 2](http://localhost:8111/viewLog.html?buildId=52&buildTypeId=NodeBuildTesting_PrMerged) outcome was **SUCCESS**
Summary: Running Build time: 00:00:01


to aż boli


ack, assuming coverage is full


В типичном провайдере есть drscription, который менять не нужно, только подставить в него нужное

```
	<p>Предоставляет информацию о балансе провайдера ИмяПровайдера <a href=""></a></p>
	<p>Информация берется из личного кабинета <a href=""></a></p>

```


ack


spacing: space before *, not after
space around operators: (dirname != NULL) ? dirname : ctx->basedir


I am not so ahppy with consul I don't like that there is no standard way to do pager duty integration


can just leave out the NULL!=

if(ctx->streams[ctx->depth])


> The "cleverness" here is that a `ref&&` is a proxy reference, whereas a `ref&` is a reference to a ref object.

Ew. My inner library designer voice is saying no.

> There's definitely *something* here, but this off-the-cuff design is insufficient. It needs a more principled characterization of references to clean it up.

Agreed. Proxies are hard. The `iter_move` design of P0022 is admittedly a half measure -- it only solves the problem for iterators -- but it side-steps all this complexity. Types like `pair` and `tuple` that *could* be proxies are problematic for any `proxy_move`-like API. Treating rvalues differently from lvalues makes the system fragile and hard to use. My gut is saying, "Danger! Danger!" This needs careful thought.


no space after ( or before )


Acá ruteamos para **root** `/` verbo `get` que utilice el controller que definimos arriba. Notar que utilizamos la función `render`, que en definitiva retorna un middleware, una funcionalidad. Mismo concepto que **Connect**.


put the return statement on its own line


In this case at least, I find the old-sk00l C-style case difficult to read.  The C++ equivalent `Sample(++current) / 512.0;` shows the scope of the cast better.  I'm also not a fan of code that makes me stop and think, which is what this does by putting the increment into the compound statement.  I'd much prefer it was on a line all by itself so that the sequence of operations is completely clear.


add spaces around =


I'm not a big fan of the 'NULL !=' style. either put it after the thing being compared to, or just leave it out


Well, as mentioned here: https://github.com/HexHive/printbf/pull/1#issuecomment-167601143
The method of bypassing FORTIFY_SOURCE that you refer to in the above commit (which is not the technique used in the Phrack article) isn't possible as described.  You can't modify "internal state of printf" (referring to the _flags2 in FILE) when in your case for instance you're using sprintf and so that "internal state" is created at runtime on the stack by the _chk variant of the sprintf function.  That is, you can't a priori modify something that does not yet exist.  Further, as I already mentioned to you on Twitter, even if you were to change all the sprintf calls to fprintf and do what you suggest an attacker would do to bypass FORTIFY_SOURCE (even though it's explicitly spelled out in the Phrack article that it's not possible as you claim) and modify the _flags2 on a FILE struct obtained from fopen, the _chk variant of the fprintf function that the app calls will have an argument forced on it (which on x64 is passed via register, not the stack) which will force an OR into _flags2 of the flag that enables FORTIFY_SOURCE checking, undoing any prior modification.

As I see it, there are three options based on the above information:
1) The above analysis is incorrect and you have some PoC proving my PoC wrong, which would be an actual advance of the state of the art. (PoC||GFTO option)
2) Knowing what the Phrack article actually entailed, you threw out the mention of the Phrack article to throw off the less determined from the scent of truth
3) You either didn't fully understand the Phrack article and/or weren't aware that the simple bug described in it has been fixed over 3 years ago, admit that it's been the case when we mentioned this months ago and for the 3+ papers you published on the topic during that 3 year period, and therefore would see no problem with the above commit instead being rewritten as something like:

Note that the demo uses _FORTIFY_SOURCE=0 to keep the demo possible.  This is a fun example of mental gymnastics that isn't intended to be applicable to any real-life exploitation (at least on Linux) due to either glibc's _FORTIFY_SOURCE, enabled by default on Ubuntu, Fedora, and other distros, or Android's removal of %n support in bionic's *printf functions.


I would do that change, if you explain how. This is a non-standard buildsystem.


`return result if isinstance(result, str) else not result`?


const char *basedir


+1, odd that this didn't give a warning on Linux as well.


git grep?


superfluous whitespace


Por último, en nuestro archivo de entrada levantamos la configuración Express y la usamos.


we're soft on this elsewhere in the codebase, but it's not ideal to override builtins:
```
Help on built-in function id in module __builtin__:

id(...)
    id(object) -> integer

    Return the identity of an object.  This is guaranteed to be unique among
    simultaneously existing objects.  (Hint: it's the object's memory address.)
(END)
```


if(*endptr || errno)


ack


yeah that's way shorter


Fix: Current viewer count when the streamer have no picture available in the api


indent 2 spaces, not 4


const char *lastFound

space before *, to match style of existing code


++settingName;


== '\0'
to make it clear we're comparing to a char


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


Notar que el módulo anterior `routes` acepta un único argumento `app`. Entonces cuando llamamos a esta función debemos pasarle la *instancia app de Express*. Podemos verlo en este archivo:

Primero crea una instancia app de Express. Luego hace un `require` del archivo de rutas llamándolo como una función (ver que el módulo routes exporta una función) pasándole esta instancia como argumento.
Este módulo `routes` retorna la misma instancia `app` *configurada*.

El archivo **express.js** tendrá toda la configuración de este framework.


const char *settingName


Converting subversion-plugin to use jenkins 1.580.3 is really hard!
There are a lot of classes involved.  Did you get this to work?


++settingName


I ment chess pattern :D


fprintf calls puts for static parts of the string that spills the argument on
the stack, allowing you to modify the value on the stack and not in the register.

Again, this is not a generic FORTIFY_SOURCE bypass but a fun demo/exercise in
Turing-completeness.

-m

On 12/29/2015 01:31 AM, spendergrsec wrote:
> Well, as mentioned here: #1 (comment)
> <https://github.com/HexHive/printbf/pull/1#issuecomment-167601143>
> The method of bypassing FORTIFY_SOURCE that you refer to in the above commit
> (which is not the technique used in the Phrack article) isn't possible as
> described. You can't modify "internal state of printf" (referring to the _flags2
> in FILE) when in your case for instance you're using sprintf and so that
> "internal state" is created at runtime on the stack by the _chk variant of the
> sprintf function. That is, you can't a priori modify something that does not yet
> exist. Further, as I already mentioned to you on Twitter, even if you were to
> change all the sprintf calls to fprintf and do what you suggest an attacker
> would do to bypass FORTIFY_SOURCE (even though it's explicitly spelled out in
> the Phrack article that it's not possible as you claim) and modify the _flags2
> on a FILE struct obtained from fopen, the _chk variant of the fprintf function
> that the app calls will have an argument forced on it (which on x64 is passed
> via register, not the stack) which will force an OR into _flag s2 of the flag
> that enables FORTIFY_SOURCE checking, undoing any prior modification.
>
> As I see it, there are three options based on the above information:
> 1) The above analysis is incorrect and you have some PoC proving my PoC wrong,
> which would be an actual advance of the state of the art. (PoC||GFTO option)
> 2) Knowing what the Phrack article actually entailed, you threw out the mention
> of the Phrack article to throw off the less determined from the scent of truth
> 3) You either didn't fully understand the Phrack article and/or weren't aware
> that the simple bug described in it has been fixed over 3 years ago, admit that
> it's been the case when we mentioned this months ago and for the 3+ papers you
> published on the topic during that 3 year period, and therefore would see no
> problem with the above commit instead being rewritten as something like:
>
> Note that the demo uses _FORTIFY_SOURCE=0 to keep the demo possible. This is a
> fun example of mental gymnastics that isn't intended to be applicable to any
> real-life exploitation (at least on Linux) due to either glibc's
> _FORTIFY_SOURCE, enabled by default on Ubuntu, Fedora, and other distros, or
> Android's removal of %n support in bionic's *printf functions.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/HexHive/printbf/commit/fef292697fc816e88341af1075885b0c29dc2637#commitcomment-15182756>.
>


add spaces around all operators and == and =


To me it reads more naturally to have the constant being compared to *after* the variable

if(de->d_type != DT_REG)

reads as "if the file type is not a regular file"


Found it, I lost a recursion in this change. Should be good now!


Introduce a variable for `getField()`


```java
Object value = getField(partitionColumnTypes.get(field), partitionBlocks[field], position);
partitionRow.set(field, value);


Static import `FieldSetter`


TeamCity pull requests :: netty [Build 911](https://garage.netty.io/teamcity/viewLog.html?buildId=4322&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 6355, ignored: 21; resource leak detected (new); compilation error: Compiler (new) Build time: 01:21:25


TeamCity pull requests :: netty [Build 914](https://garage.netty.io/teamcity/viewLog.html?buildId=4325&buildTypeId=pulls_netty&guest=1) is now running


Modify the value via what?  A concurrently executing thread hoping to win a race on modifying a spilled argument on the stack?  Please be more precise, as what you describe sounds Rube-Goldberg-esque to me.


Hey Bejond,
Besids errorUserLockout and errorPasswordHistory  you changed other method too in class SystemLDAPConfiguration.  I did a search I did not find other place call the method you changed.
I have a little bit concern about this change. Is it necessary to change them to return Array type?

Thanks,
Dale


Why not
```java
return types.stream()
        .map(HiveWriteUtils::getRowColumnInspector)
        .collect(toList());


Check for null


did you find a way to propagate the lang up through whole_archive? I had to use lang='c++' when doing this.


This should be protected since the class is abstract


TeamCity pull requests :: netty [Build 913](https://garage.netty.io/teamcity/viewLog.html?buildId=4324&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 4882, ignored: 21; resource leak detected (new) Build time: 00:36:37


TeamCity pull requests :: netty [Build 915](https://garage.netty.io/teamcity/viewLog.html?buildId=4326&buildTypeId=pulls_netty&guest=1) is now running


This can only produce false (?)


It appears I did not! I'll try to fix that.


Looks good!


ack


It looks like you had a merge commit. You need to pick which lines you want o keep out of this commit and delete the lines with all of the arrows and equal signs.


wait I do more tests so I don't rely on other bears...


there's still some extra vars generated, i'll extract a case later


The more I think about it, I think what you have done is fine.  I think we can avoid having to reset this by simply keeping the user supplied `ANSIBLE_CALLBACK_PLUGINS`, and appending our path to the end when performing idempotent check.

I think this will cover my concerns, will run the user specified one first, and ours last.  I think thats what we want.


Use `Ints.checkedCast`


For some reason, the existing client settings gave "OAuthError: updateToken: Unexpected HTTP status 400 Bad Request" when trying to authenticate. Based on what I've read, this may be a result of a configuration change on Facebook's side.


Prueba


I filed https://github.com/jimporter/bfg9000/issues/37 for this. I need to think a bit about how I want to handle this exactly...


which revealed to be the "dependency" entries in data/jsons/dependencies.json


reack


This warning (like the old one) is not implementable in general, and is a "probably bad idea" flag.

Ideally the producer would fill in both definitions correctly, per Pepijn's suggestion. If that isn't possible (e.g. because there is no equivalent, or the NGA says "no"), then we need something. Null is a particularly bad choice, because one or the other of the definitions must be non-null (so we'd need an associated trigger if we want to enforce that), and changing column definitions after creating them isn't that easy in sqlite.

The reverse will also be problematic - where the source data is old WKT, and there is no new WKT equivalent. So I guess we will need `undefined` as an option there too. And now a trigger to say that they can't both be `undefined` unless `srs_id` is in (0, -1) might make more sense.
So the concept here would be:

 - `definition` is non-null (always, extension that change that are breaking, not extending).
 - `definition` should be filled in with valid WKT, wherever possible.
 - if there is no sensible WKT for `definition`, use `undefined`, as for undefined Cartesian / geographic coordinate systems (srs_id -1 / 0).
 - if there is no sensible WKT for `definition_12_163`, use `undefined`
 - if `definition_12_163` and `definition` are both `undefined`, its an error unless `srs_id` is 0 or -1.


I'm currently learning Web Design and this is my first attempt at a webpage.  I don't have a domain yet, I wanted to wait until I felt more comfortable with everything before making it official.  I want to use this page to be able to access my code from anywhere, in case I want to add something new or correct something.  Also, it will be a better way to share my code when I reach out for questions.


Might want to size this up front
```java
new HashMap<>(mapBlock.getPositionCount() * 2)


Please don't replace.


Check for null here and below


`Undefined` -> `undefined` to match Table 4 (https://github.com/opengeospatial/geopackage/blob/master/spec/1_base.adoc#gpkg_spatial_ref_sys_records)


Exercicio 02 Resolvido


As above.


Pressurestat (lgu1234) - i386-Windows-7-VisualStudio-12: OK (1986 of 1986 tests passed)

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


These should be the same, but use `rowBlock.getPositionCount()` here to be consistent


Hi Qian,

Do you refer to Line 115 of this new CacheConfig.py? That line connects *cached* ports of L2 caches to tol3bus, and their *uncached* ports directly to the memory bus. You might see this HowTo wiki for better explanation: https://github.com/basicthinker/ThyNVM/wiki/HowTo#how-to-add-a-shared-l3-cache

By the way, do you look into the above issue alone or the whole project? If it is the latter, this project will be deprecated by a new version at https://github.com/basicthinker/ThyNVM (but not ready yet :p)

Best,
Jinglei


As far as I could tell, that might as well be a setting to require the function to exist, not to define a HAVE_xxx.


Probably should be some kind of relative link instead of hard-coding the Table number.


TeamCity node build testing :: PR submitted [Build 21](http://localhost:8111/viewLog.html?buildId=53&buildTypeId=NodeBuildTesting_PrSubmitted) is now running


TeamCity node build testing :: PR submitted [Build 21](http://localhost:8111/viewLog.html?buildId=53&buildTypeId=NodeBuildTesting_PrSubmitted) outcome was **SUCCESS**
Summary: Running Build time: 00:00:01


TeamCity node build testing :: PR merged [Build 3](http://localhost:8111/viewLog.html?buildId=54&buildTypeId=NodeBuildTesting_PrMerged) is now running


TeamCity node build testing :: PR merged [Build 3](http://localhost:8111/viewLog.html?buildId=54&buildTypeId=NodeBuildTesting_PrMerged) outcome was **SUCCESS**
Summary: Running Build time: 00:00:01


Need to find a non-hardcoded reference.


this entire section breaks the motor tester function within configurator. The motor values that had been set in mixTable() function (via the motor_disarmed[] array) are over-written. This should be moved to mixTable function, correct?


NOT NULL


@jt2k :santa: why, 'tis a Christmas miracle, it is!


TeamCity LabKey :: LabkeyRelease15.3 :: Git postgres 9.4 [Build 114](https://teamcity.labkey.org/viewLog.html?buildId=293799&buildTypeId=LabkeyRelease153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 19 (1 new), passed: 268 Build time: 01:32:10
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyFilters: <no details avaliable>

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

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

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

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

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

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

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

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnDataset: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStudyParticipantCounts: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSearch: <no details avaliable>

##### there are 9 more failed tests, see build details
```


해쉬태그 코드 날아간 부분


해쉬코드 날아간 부분


Shouldn't this applied everywhere there's a buffer creation then?


typo


these or packet


...and accidentally commit work-in-progress remote example. 🙄


spelled acne wrong and you probably mean grease


This one is a bit troublesome as newer versions of all of my programs depend on the nk path structure.  I'll have to patch the older ndyndns instead.


right here is the inline comment about how shit your spelling is


I got back from vacation and tried using redux-saga in a new project.  I keep getting `Saga must be a Generator function`.  I did a `console.log` in a local build of redux-saga and assigned that and sampleGen to console variables.  This is what I'm seeing.

<img width="487" alt="screen shot 2015-12-28 at 8 15 06 pm" src="https://cloud.githubusercontent.com/assets/75291/12028105/bb1d8f4a-ad9f-11e5-96fc-c82c82fc5f20.png">


TeamCity node build testing :: PR submitted [Build 1](http://localhost:8111/viewLog.html?buildId=55&buildTypeId=NodeBuildTesting_PrSubmitted) is now running


TeamCity node build testing :: PR submitted [Build 1](http://localhost:8111/viewLog.html?buildId=55&buildTypeId=NodeBuildTesting_PrSubmitted) outcome was **SUCCESS**
Summary: Running Build time: 00:00:01