Ok, parfait alors :)

On a beaucoup plus de monstres que lui, je pense que ce qui serait bien c'est de mettre a "true" le pathfinding pour les monstres agressifs corps à corps dans un premier temps, pourrais tu le faire ?


a z kolei zamianę całej listy bajerów także możesz wydzielić do metody:

```
def api():
    return Response(json.dumps(self._normalize_tasks(Task.query.all())))

def normalize_tasks(tasks):
    for task in tasks:
        yeld self._normalize_task(task)

def normalize_task(task):
    return {
        id: str(item.id),
        title: item.title,
        // itd
    }
```

znów, nie patrz czy dobrą składnię dałem, chodzi mi o ideę ;)

małe funkcje, które nazwami opisują co robią – dorzucisz nową metodę prezentującą taski (np tylko niektóre albo w innej kolejności), to masz gotowe funkcje do zamiany na rzeczy dla api


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16273](https://teamcity.mesosphere.io/viewLog.html?buildId=138325&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:09


TeamCity Civic Assembly :: Build all [Build 65](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1243&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Tests failed: 166 (1 new), passed: 6, ignored: 4 Build time: 00:06:50
### Failed tests
```
Blog categories tests: Should delete blog category new blog category 1455810788515: <no details avaliable>

Blog categories tests: Should not create new blog category if cancelled: <no details avaliable>

Blog categories tests: Should add new blog category: <no details avaliable>

Blog categories tests: Should not change blog category: <no details avaliable>

Blog categories tests: Should change blog category: <no details avaliable>

Blog categories tests: Should not delete blog category if cancel pressed: <no details avaliable>

Blogs tests: Should not create new blog if Cancel button pressed: <no details avaliable>

Blogs tests: Should add new blog: <no details avaliable>

Blogs tests: Should not change blog if Cancel button clicked: <no details avaliable>

Blogs tests: Should change blog: <no details avaliable>

Blogs tests: Should not delete blog if Cancel button was pressed: <no details avaliable>

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


Make `p0_table` a full array member instead of allocating it here?


.gitignore 파일은 git 에서 무시해야 하는 파일이나 폴더를 적어놓는 파일.
### OSX template 는 맥 운영체제가 추가하는 파일들을 써놓은 곳


In effetti "Scopri le ultime innovazioni" è più leggibile, sarei per correggere in tutti i file dove appare quella forma.


per Interface there is no 3rd arg
https://github.com/amphp/aerys/blob/master/lib/Websocket.php#L37


Good point, that could also cause an issue.

However, this commit was to fix a situation we encountered recently where the user was calling transform script with no subsequent read. Since we expect TRANSFORM_PATH to be a special directory where we keep our maps, we put that into DistributedCache. However, when the reducer tries to write to OUTPUT_DATA_PATH, there is a timestamp mismatch error on TRANSFORM_PATH.


We don't run Geom any more ?


While I like the fact that API design has a reference here, the used link does not seem immediately clarifying. Might http://wiki.apidesign.org/wiki/TheAPIBook be better? At least it gives a list of ideas that are relevant: the chapter titles. Generally, I would prefer an even more informative link thought. Perhaps an article that actually describes the important rules a contributor should follow.


both code


the pom version doesn't match the one in the whitelist


А что тут не по алфавиту?


No trouble :)
On Tue, Feb 9, 2016 at 9:01 AM Tim Schaub <notifications@github.com> wrote:

> Wow. Excuse me for this direct commit to master @geoserver/team-geoserver
> <https://github.com/orgs/geoserver/teams/team-geoserver>. In my haste to
> suggest an edit through the GitHub web UI, I completely forgot to check if
> it was creating a branch - thought this was going to result in a pull
> request.
>
> Please feel free to revert if this breaks any process. Also, I won't be
> offended if my push rights are removed. I'm happy just to submit a pull
> request if I have any other proposed changes.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/geoserver/geoserver/commit/252159f2315dffa74999a66b2c7d1e2217f0551c#commitcomment-15983258>
> .
>
--
--
Jody Garnett


These macros are added again? From https://github.com/wiedemeyer/libfreenect2/commit/5b8d15158da71097d0bc31c2a0c55e6b91f0f1b1#commitcomment-16179281


The following doc(s) have been deployed to the [629multide](http://629multide-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/multidev](http://629multide-static-docs.pantheon.io/docs/articles/sites/multidev)


그래그래그래


Works to build now... however stuck att "Starting apps." first time boot screen now xD Maybie its RR or CM  fail, but it worked before including this "fix" tree ;) Thanks for doing this anyway, really want the freaking tristate and camera to work again!


Can you put the definition just here, not all the way down the bottom?


In this example the name value should be terminated by a **.** (dot) otherwise an error is raised during Terraform apply: powerdns_record.www: **Failed to create PowerDNS Record: Error creating record set: www.example.org-A, reason: "DNS Name 'www.example.org' is not canonical"**

So the example under the _"powerdns_record"_ should be as follow: name = "www.example.com**.**"


zamiast rozdzielać sekcje pliku dodatkowymi pustymi liniami: może po prostu podziel na mniejsze pliki?


Using .exec is not ideal. Can this fix be achieved with directive priority instead?


Just open a new issue or PR and then switch to the “Preview” tab looking at the rendered Markdown. If you go back to relative URLs the links stop working, because they now link to one of:

- https://github.com/Homebrew/homebrew/issues/CONTRIBUTING.md
- https://github.com/Homebrew/homebrew/compare/CONTRIBUTING.md


This file doesn't exist in the extension, and may not exist on all sites. I think there must be a better way to get the current working directory than this.


On 18 February 2016 at 16:24, Andrew Bell <notifications@github.com> wrote:

> We're trying really hard to get rid of boost.
>

Heh, apparently, I completely missed what it is being about in PDAL. I
thought it is to move from external (user-installed) to internal pdalboost.


@ParkYoungWoong
.gitignore는 git을 쓰는데 필요한 '관리'파일이니 요것 말고
나머지가 프로젝트파일이니 하나하나 공부해서 줄마다 의미 써보자고


Hi,

up to this step 4.2 I was able to easily follow your nice tutorial. But here I´m completely puzzled: is simple-todos.js a new file in this project? It has not been mentioned before. If I create a new file with the code above I get this in browser:

> _Your app is crashing. Here's the latest log:
> /Users/hgg/.meteor/packages/meteor-tool/.1.1.10.1b51q9m++os.osx.x86_64+web.browser+web.cordova/mt-os.osx.x86_64/dev_bundle/server-lib/node_modules/fibers/future.js:245
> 						throw(ex);
> 						      ^
> Error: A method named '/tasks/insert' is already defined
>     at livedata_server.js:1542:15
>     at Function._.each._.forEach (packages/underscore/underscore.js:113:1)
>     at [object Object]._.extend.methods (livedata_server.js:1538:7)
>     at [object Object].Mongo.Collection._defineMutationMethods (packages/mongo/collection.js:927:1)
>     at new Mongo.Collection (packages/mongo/collection.js:218:1)
>     at simple-todos.js:1:9
>     at /Users/hgg/simple-todos-react/.meteor/local/build/programs/server/app/simple-todos.js:40:4
>     at /Users/hgg/simple-todos-react/.meteor/local/build/programs/server/boot.js:242:10
>     at Array.forEach (native)
>     at Function._.each._.forEach (/Users/hgg/.meteor/packages/meteor-tool/.1.1.10.1b51q9m++os.osx.x86_64+web.browser+web.cordova/mt-os.osx.x86_64/dev_bundle/server-lib/node_modules/underscore/underscore.js:79:11)
> Exited with code: 8
> Your application is crashing. Waiting for file change.

If I just insert the code on green background into simple-todos-react.jsx nothing seems to change. The input-field is still there but the list below stays empty after filling in and hitting return.

Grateful for any help.


This crashes because `parsed_ec_dump_file` is not defined:

```
ERROR: Traceback (most recent call last):
  File "/homeb/zam/alvarez/sw/easybuild-framework/easybuild/main.py", line 113, in build_and_install_software
    (ec_res['success'], app_log, err) = build_and_install_one(ec, init_env)
  File "/homeb/zam/alvarez/sw/easybuild-framework/easybuild/framework/easyblock.py", line 2327, in build_and_install_one
    app.cfg.dump(parsed_ec_dump_file)
NameError: global name 'parsed_ec_dump_file' is not defined
```


So just a `cl_float3 p0_table[image_size]` in the method, correct?


@rachelwhitton +1


unikaj takich nazw: nazwa powinna opisywać co reprezentuje, a nie jakiego jest rodzaju

tutaj mogłoby być np `api_item`

a z kolei zamiast `dict_list` mogłoby być `api_items`


As required by `allocator.h`, `Allocator::allocate()` must not return NULL. The `CudaAllocator::allocate()` is wrong and needs fixing (I forgot to update it after revising the requirement of `Allocator::allocate()`).

VAAPI does it right, see https://github.com/OpenKinect/libfreenect2/blob/master/src/vaapi_rgb_packet_processor.cpp#L149


这里译作 客户端websocket支持 是不是更合适些呢？


That is fine. Disp was the atom selection in the previous line. Now, "all" is used to designate the atom selection.


using System;

namespace IntroductionToCsharp
{
    class Program
    {
        static void Main()
        {
            double d = 123.22242433;
            Console.WriteLine("d");

        }
    }
}


I just moved them to the top, it feels wrong to have them somewhere in the middle.


This had been cropping up a lot in designs and we can't avoid adding it, imho.


ze względów bezpieczeństwa zwrotka JSONowa nigdy nie powinna być listą

czyli zamiast zrobić zwrotkę: `[item1, item2, item3]` lepiej opakować ją do np `{ items: [item1, item2, item3] }`

poczytaj:
https://github.com/symfony/http-foundation/blob/master/JsonResponse.php#L17


Ah, apparently so. That's a shame in terms of people wanting to glance between the documents locally, but not the primary use case intended here I guess. Thanks for catching this! :smiley:


为什么我将version改了，客户端还是连不上？


too legit to..


Hopefully we will get this fix integrated today.


Eso espero amigo, una opcion muy sabia.


The allocated `cl::Buffer` must be freed here unless `b->data == NULL`. `b` still must be freed even if `b->data == NULL`.


crap... @ocaisa


`task_add` precyzyjniej mówiło co robi ta akcja


OK, allora poi cerco e sostituisco.


@damianam:

should be:

```python
app.cfg.dump(repo_spec)
```


`form` może być dowolny: `new_task_form` precyzyjniej mówi co zmienna zawiera


sweet


Yes, sorry. I think I might have been confused by the view I somehow was in...


Can you convert this to a function since `_factory` was intended to give the idea that it was returning a strategy. The idea is that the function could be used to modify the returned strategy.

Did you try with just adding the `st.just(id)` without the `st.uuids().map(str)` portion?
It seems like the uid->str part should be the same concept as just st.text?


Sadly no, because OpenCLDepthPacketProcessorImpl is not defined at that point.


@SanketDG you'll want to return a proper message if code is not none, i.e. "You're not connected to the internet" something like that


typo: acknowledge


Yes


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


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


About `tox -- -v`- you're right) The idea was good but it's not good)

But with hard-coded `tests` in `tox.ini` this path will be tested always so `tox -- tests/test_foo.py` will test `tests` and `tests/test_foo.py`. You've fixed this problem with `changedir` but it changes dir (some magic for contributors, breaking tab-complete etc). So I don't know)

About `pytest` section - here could be added `pytest.ini` - http://pytest.org/latest/customize.html


Electric-Service-Refactor (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2115 of 2122 tests passed, 546 test warnings)

 * 550 tests had: AUD diffs.
 * 480 tests had: RDD diffs.
 * 3 tests had: ESO small diffs.
 * 7 tests had: MDD diffs.
 * 7 tests had: MTD diffs.
 * 3 tests had: EIO diffs.
 * 2 tests had: ERR diffs.
 * 4 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 4 tests had: Table big diffs.
 * 1 test had: SHD diffs.
 * 1 test had: SSZ big diffs.
 * 1 test had: ZSZ big diffs.

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


odczytaj urla z widoku

np jak w htmlu masz: `<form action="{{ TUTAJ_GENERUJESZ_URLA_I_NIE_MUSISZ_GO_HARDKODOWAĆ }}" method="post">`

to w JSie robisz `url: $(this).parents('form').attr('action')`


typo


shhhhhh!


It would be better to disable catch_up automatically if change and interval are 1.


Fixed in #1622


It looks like user pages aren't rendering anything
![image](https://cloud.githubusercontent.com/assets/7533441/13149525/c2e5d288-d62f-11e5-8475-443ca5adbffb.png)


I think that .bind approach is wrong. You should bind once in the component constructor, because bind returns a new function so you're creating new functions on every render.


this is invalid syntax in  visual c++ 2013

Error	2069	error C2059: syntax error : ':' [C:\foo\ace-windowsfork\WORK\Build\linphone\coreapi\linphone.vcxproj]	C:\foo\ace-windowsfork\submodules\linphone\coreapi\linphonecore.c	3909	1	EP_linphone_builder


how about an example with a line of markdown, i.e. when the link is surrounded by `<>`, also try surrounding the link by different kinds of quotes


返回一个结果为 `WebSocketClientConnection`的 Future 对象. 这句不符合 **reStructuredText** 的语法。需要在

```
 `WebSocketClientConnection` 后增加一个空格.
```


Kind of funny, I had a check for that originally :P


@MBoretto, thanks for pointing this out.
Tried to merge this into my branch but it didn't work, so I've made the changes manually and it works :smiley:


But ANY is already included in `-c help` output.


测试


That actually goes against pep8 (though I agree that it should remain consistent with the project until (if) it's decided it should be changed across the board)


IMO this should be a NORMAL, not major


config do travis precisa ser android pra ler o gradle? talvez isso faça o travis demorar mais.


Electric-Service-Refactor (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Maybe it's a bit overkill to call systemd-resolve to list this... but otoh, it works fast enough and it's easier to be consistent with -p and -t.


Unsure whether this should be type name or type ID. ID makes sense for matching, but the interface would suck. Could use Choice.value for the ID and Choice.display for the name, but none of the other filters use display, so I wondered whether there was a good reason for that, or whether there just hadn't been the need.


style updates


Minor: I'd add a `(void)` prefix to clarify that a result value is being ignored intentionally here.


Looks good – thanks!


this is wrong! The link is somewhere in the line, so you need the original line and then do a replace with the original and the new link to get the new link.


let's return -ENOTSUPP here instead?


Le problème est que ca rajoute un accès en lecture/écriture au chunk_array de l'attribut qui pour moi devrait rester privé.
Ça devrait éviter aux utilisateurs de se prendre pour des experts.
Tu entends quoi par alloué ? Ta méthode remplis toute les lignes qui sont dans le chunk_array. Normalement ces lignes sont toutes correctement indexées. Il peut y avoir des trous mais comme tu traverses la topo ensuite on s'en fou de la valeur des cases "trous", non ?


Yeah I was planning to make it consistent with the other factories, although I admit I don't understand how the function could be used like you're saying, could you clarify?

The IDs of a fasta/fastq file should be unique, which uuid guarantees. (I also suspect it's faster).


zaleca się JSy ładować możliwie na końcu pliku (tuż przed `</body>`) ze względu na szybkość renderowania

kiedy wsadzasz JSy w `head` czy na początku `<body>`, to:

1. przeglądarka na chwilę stopuje całą pracę i czeka aż się ten JS załaduje
2. wczytuje html i go przetwarza
3. w międzyczasie już reguły JSa działają

* punkt 3 spowalnia punkt 2
* punkt 1 w ogóle blokuje stronę, co jest kijowe

---

BTW, odwrotna sytuacja z CSSami: zaleca się wsadzać w sekcję head, bo nie blokują ładowania strony, a od razu wygląda ona poprawnie

jakby cssy umieścić na końcu, to przez ułamek sekundy będzie rozpiernicz w wyglądzie, zanim zastosuje reguły css


这里你顺手删掉了一个空行～还是补上吧。Python代码的格式～


name the test to what the goal is, to ensure AC power cycles keeps the file system correct.  test_ac_cycles.robot ?


ping @dblotsky
I've addressed your comments regarding making global variables local, using `path.join` for the appiumServerCommand, making bufferSize a constant, requiring `unorm` at the top of the file and making one variable decalaration on one line.


Nevermind, this is the output not the input tool xml. Ignore this comment.


Great!


@opengapps/packagers, did a force push here, please:

git reset --hard origin/master && git pull


@theod Note : this allows "normal" execution to work but I think that it breaks "play from here" feature. I leave this so that @bltzr can work but maybe it will have to be reverted afterwards.


well. the data structures do not match, so i rather leave default empty.


Are the "pythonXX" libraries existing boost.libraries? I thought there was only the Boost.python llibrary that has to be built anyway, since it's header-only. If this library then works with python2 or python3 depends on the version that "PythonInterp" returns. Am I missing something?


This appears the wrong way round, no? This returns 1 if everything is OK but 0 if not, but the function name suggests the opposite. Either rename the function, or invert the condition, please... (The function name is a bit weird btw, even if the condition is inverted, it should probably be reworked to become size_multiply_overflows() or so? After all, this has strictly speaking little to do with allocation...)

Also, change the return value to "bool" please, if this is a boolean.


That is correct. I will add that back into the list.


@keszybz i think you forgot a "not" in that sentence? or i cannot parse it...


all commented code is validation that must validate if user is a leader of current group and dont allow to add him to other group

need to think about that & implement


I think I'll move the script over to #5478. That can go in all green in one PR and we can avoid having some of the more speculative commits in this branch in the history.


Should this go on the User model, or in a utils.py file, or a class method on SQLLocation, or a method on LocationTypeManager? It made the most sense to me to have it on the User model, but if everyone made the same choice, the User model would get way too big. Happy to move it if we have a better principle.


typed_function_ordering (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (398 of 398 tests passed, 0 test warnings)

![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)


It looks like the `HasAttrs` module is also used for processing instructions (I didn't even realize processing instructions could have attributes until today!) so it would be nice if we had consistent behavior between elements and processing instructions (i.e. `attributes` always returns a hash that clients can mutate to update the node's attributes).

FWIW - Modules can define an `initialize` method that will be called during initialization much like a superclass' constructor would get invoked.


Also we have to update https://github.com/okfn/oki-py/blob/master/CONTRIBUTING.md#testing


@It  @salathe the pull request has been there for monthes.. I don't understand why you don't give the idea then, now complaint it was committed without discussion?  (with whom? with you?)


This is also from the CUDA implementation, so it probably needs to be fixed there too.


jak w kilku miejscach masz to samo, to żeby nie kopiuj–wklejkować:

* a.) utwórz podwidok, którzy załadujesz w tych miejscach
* b.) przenieś do szablonu–ojca (do `base.html`)
* c.) utwórz pośredniczący szablon–ojca (np rozszerzaj `task_base.html`, który rozszerza `base.html`)


we need const everywhere in the same time


原来的简直可怕。。。。。


well, it's because of refactoring and no tests covered this bit (which is clearly a bad idea)


Why comment it out?


"This change did not lose the response object"

I'm not referring to _this_ change. I'm referring to the fact that this change _reverts_ the signature of the log method that was changed here:
https://github.com/eclipse/jetty.project/commit/34a8da2ba26a806413f06ab18fdfad535de45cfa

de45cfa also changed the signature of the logExtended() method on line 184 in this file. I'm asking if this change should also be reverted so that it becomes logExtended(request, response, buf); again.


ustal jedną konwencję napisów w JS, np zawsze apostrofy


why did you commented it here and not in the PR? haha


:party:


@stefanofontanelli this is wrong.
We should patch the template file instead.


As specified, the `--eeout` option only works when outputting  fastq format. I encountered situations where it would be interesting to be able to output expected errors values in fasta format too. What do you think? is it easily feasible?

It would simplify the documentation: `When using --fastq_filter, include the number of expected errors in the sequence header.`


typed_function_ordering (lefticus) - i386-Windows-10-VisualStudio-14: OK (390 of 390 tests passed, 0 test warnings)

![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)


Hey @gandhikeyur sorry this deleted in the original merge.  I just was about to fix until I noticed you already fixed today.  However I did notice in the original PR you had and else statement where with "$scope.setProductAndVersion(urlParameter.product,urlParameter.version);"  Is that no long necessary?


So that's where it was...


Oh, please! And thank you! No need for a blip on these fixes as the script isn't live.


proponuję konwencję w htmlu atrybuty zawsze jako cudzysłowy (zamiast mieszać raz apostrofy a innym razem cudzysłowy)


Oh... I just looked at the "systemd-resolve -t help" and "systemd-resolve -c help" calls above. In one case you add the "any" and in the other you don't. If the output already contains the "any", then we shouldn't add it for either...

After running "systemd-resolve -t help" I see now that the invcoation for the -t case is wrong and should not list "any" explicitly...


Thanks. I figured I would test it out on a fork before messing up our real
meza repo.

On Thu, Feb 18, 2016 at 9:53 AM, James Montalvo <notifications@github.com>
wrote:

> With travis you'll have to make it build on CentOS. See
> http://stackoverflow.com/questions/32535195/how-to-run-tests-on-centos-7-with-travis-ci
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/darenwelsh/Meza1/commit/3a0b84dc105d74a809bdb676944b04d7a2082401#commitcomment-16179311>
> .
>

--
__________________
http://enterprisemediawiki.org
http://mixcloud.com/darenwelsh
http://www.beatportfolio.com


can we update to the latest fred style?


Ah, this is actually a fail-safe, not the real fix. I have no clue what causes the NaNs on that.


Sure :smile:
Just, soon as I get a moment. Its not much. Just a listener leak (if I recall correctly)


TeamCity Bender :: Deploy [Build 3.0.33.0](http://build.mikeobrien.net/viewLog.html?buildId=2517&buildTypeId=bender&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 3284, ignored: 5 Build time: 00:01:39


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1311.476616f30025f32f77255ec7353a8e178e344a0b](http://build.palaso.org/viewLog.html?buildId=69357&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 1003, ignored: 12 Build time: 00:07:08


po co `my_task`? to jest „lista Tasków”, czyli `tasks`


Yes, that was my point, something that even by chance was working with a minor upgrade it will fail hard now.


```
# enable copr for python3-nss
dnf copr enable mkosek/freeipa-master
# install python3-pyldap manually
dnf install https://copr-be.cloud.fedoraproject.org/results/pviktori/python3-ldap/fedora-23-x86_64/00128951-python-pyldap/python3-pyldap-2.4.21-1.fc23.x86_64.rpm
```


:heart:


Yeah, that makes way more sense. Thanks for the hint!


OK, sounds good to me!  Let me know when you are settled on #5478 so I can get it in.


Oh. Didn't realize the extension was calling `rb_obj_alloc ` rather than `rb_class_new_instance`. Out of curiosity is this a performance optimization to avoid calling `initialize`? Also does this mean similar changes are required in `HasAttrs` for how the `@attributes` instance variable is handled (i.e. `@attributes = {}` if the instance variable isn't initialized)?


but any float == compare throws a warning and that's not a warning that should be masked. What do you suggest? cast .value to an integer?


Ha tiens, je savais pas qu'on pouvait mettre des commentaires ici.
Sinon je ne peux pas trop te dire sur quelle version tu es mais la version officielle marche encore chez moi, testé a l'instant.
https://github.com/LordVenom/venom-xbmc-addons/blob/master/plugin.video.vstream/resources/sites/film_illimit_fr.py

Sauf videomega apparement mais j' y jette un oeuil de suite.


Could you move the key name constant out into a global var? There should be quite a few of them at the top of this file.


@alecpm I'm happy to assist on any testing related work, run the jenkins job or what not.

@vangheem thanks for the insights and planning!


 - __information__: [src/chaiscript_stdlib.cpp:0](../blob/172ab7b8e4ebfb695e86ebaf2e2aaa0b392b5ae8/src/chaiscript_stdlib.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 16 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:679](../blob/172ab7b8e4ebfb695e86ebaf2e2aaa0b392b5ae8/include/chaiscript/language/chaiscript_parser.hpp#L679) Expression is always false because 'else if' condition matches previous condition at line 677. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:683](../blob/172ab7b8e4ebfb695e86ebaf2e2aaa0b392b5ae8/include/chaiscript/language/chaiscript_parser.hpp#L683) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/172ab7b8e4ebfb695e86ebaf2e2aaa0b392b5ae8/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/172ab7b8e4ebfb695e86ebaf2e2aaa0b392b5ae8/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 683. (multiCondition)
 - __information__: [src/chaiscript_stdlib.cpp:1](../blob/172ab7b8e4ebfb695e86ebaf2e2aaa0b392b5ae8/src/chaiscript_stdlib.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __information__: [src/main.cpp:0](../blob/172ab7b8e4ebfb695e86ebaf2e2aaa0b392b5ae8/src/main.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 17 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __information__: [src/main.cpp:1](../blob/172ab7b8e4ebfb695e86ebaf2e2aaa0b392b5ae8/src/main.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/172ab7b8e4ebfb695e86ebaf2e2aaa0b392b5ae8/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/172ab7b8e4ebfb695e86ebaf2e2aaa0b392b5ae8/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/172ab7b8e4ebfb695e86ebaf2e2aaa0b392b5ae8/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/172ab7b8e4ebfb695e86ebaf2e2aaa0b392b5ae8/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/172ab7b8e4ebfb695e86ebaf2e2aaa0b392b5ae8/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


osobiście preferuję toto też wciąć – bo w końcu jest w środku `<head>` – a wywalone jak wygląda wyrenderowany html


Yes.


The comment of this method states that a default config is returned if none is present in the database, but it seems this branch actually returns an error.


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 265](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=798&buildTypeId=XbimEssentials_XbimEssentials) is now running


generuj urle zamiast hardkodować adresy


Ok.

Will do a 4.2.1 with 4.1.0 and 5.0.0 with 4.2.0

Von meinem iPhone gesendet

Am 18.02.2016 um 17:19 schrieb Gil Forcada Codinachs <notifications@github.com<mailto:notifications@github.com>>:

Yes, that was my point, something that even by chance was working with a minor upgrade it will fail hard now.

-
Reply to this email directly or view it on GitHub<https://github.com/plone/plone.testing/commit/1ff22b7a311aaa3fe367eb1d0df5eae2811990c1#commitcomment-16180158>.

________________________________________________________ The contents of this e-mail and any attachments are confidential to the intended recipient. They may not be disclosed to or used by or copied in any way by anyone other than the intended recipient. If this e-mail is received in error, please immediately notify the sender and delete the e-mail and attached documents. Please note that neither the sender nor the sender's company accept any responsibility for viruses and it is your responsibility to scan or otherwise check this e-mail and any attachments.


Ah shit. Thanks. I did that to track a problem.


This is a revision in the file do back in 2015.


typed_function_ordering (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (390 of 390 tests passed, 0 test warnings)

![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)


OK.


Nope, doesn't work for nightly either...

```
src/ui/atlas.rs:47:36: 47:50 error: the trait `glium::texture::Texture2dDataSource<'_>` is not implemented for the type `image::dynimage::DynamicImage` [E0277]
src/ui/atlas.rs:47                 if let Some(tex) = Texture2d::new(display, img).ok() {

```


This commit makes honami's backlight not to work.
For honami device, setting /sys/class/leds/lcd-backlight/brightness has no effect and /sys/class/leds/wled:backlight/brightness MUST be used instead.


@Mumfrey Is there any reason you did this except for the one usage in [VelocityDataProcessor](https://github.com/SpongePowered/SpongeCommon/blob/master/src/main/java/org/spongepowered/common/data/processor/data/entity/VelocityDataProcessor.java#L69) that could be just replaced with a cast to `Entity` instead of `IMixinEntity`? Doing that instead would match the other mixin interfaces (just add new methods not present in the Sponge interfaces, but don't extend it).


LGTM! and long. but -~300 is good


Good point about uuid on both parts.

For the factory you could at the very least specify the min_length, max_length, min_qual, max_qual
I have mixed feelings about that which is why the initial factory function had no arguments.
So after that long winded explanation, i think the only benefit is to remain consistent.

That all being said, a new idea I just had is to have statically defined strategies which come from the factory functions.

Thinking more about it, it seems odd to have the `_factory` name i suppose since it is just returning a strategy.
The idea is to just "add" new strategies to the already existing ones that come with hypothesis, so we should try to remain consistent in the naming huh?

Probably just `s/_factory//g`


testing stuff, teaching and understanding color plugins (try HSVTool, or Keyers on this, and you'll understand a lot)
For example, rotate the colors in HSVTool and you'll actually see the colors turn (gamma in ColorWheel must be set to 1 for this demo). You can also merge it onto your video files to see how the colors were affected by your comp (the background is transparent, and it can be generated at any size/position.
And I'm sure you'll find new ways of using it!


Focus: is this an adequate description for read-only configuration?


This all sounds extremely promising! I've installed helhum's version and got it working (with a minor tweak), but now I'm in the situation that certain features are missing. And those features are basically all addressed in your update.
What is the major bug you refer to? Is it the problem mentioned in the TODO comment in getUploadedResources()? Any plans for that?
Anyway thanks for your great contribution! I think the lack of proper FE upload support in the Extbase core is still a huge problem. So all efforts in that direction are highly welcome!


`high_resolution_clock::period::num / high_resolution_clock::period::num`

You'd probably want to divide by denominator. Otherwise you get 1 here.


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 265](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=798&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:01:45


For the private registry sample I created a sample URL? Is there a better example. Anything missing?


extracted a functor buildMessageIdFunc and moved init. in process()
string builder was (falsely) assuming that outboxes always same harcoded name. Now using Mailbox+Session->MailboxPath->MessageId


You'd probably want to divide by denominator. Otherwise you get 1 here.


czemu wyłączasz csrf? włącz :P


As confirmed by Joe the other day, will this suffice as a tip?


hum… dunno if I like it better than an annotation.

And it breaks the build (https://travis-ci.org/geany/geany/jobs/109620112) because it's very recent it's part of GLib.  It used to be GObject, so requiring including `glib-object.h`.


i don't know @blasdelf's latest style, what is it


Хорошо, больше не буду:D


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


Well, just pass `context` and `queue` in `OpenCLAllocator()`.

I recommend following the structure in https://github.com/OpenKinect/libfreenect2/blob/master/src/vaapi_rgb_packet_processor.cpp


fixed when outbox init. was moved into process()


Just updated the less with the mixins.


I think the rescue is a good idea, though a part of me wants to let it blow up with info (I need to think that through, and add code either way).. I did some attempts at reproducing the issue, and a missing record/store/invalid id certainly caused the error, but I could not prove that was the actual issue triggering things.  I need to dig a bit more with a fresh mind this AM.


Je suis sur la dernière 0.3.8 ;) ca avait été corriger puis ca a re bug on clique sur ilimit et rien vide :p mais je vais tester ca alors, Merci


upewnij się, że nie przyjmiesz formularza GETem – jeśli akcja ma mieć „efekty uboczne” (jak dodanie zasobu do bazki), to powinna przyjść POSTem

* np żeby ktoś adminowi nie podrzucił spreparowanego urla z jakimś skracaczem linków
* żeby różne proxy sieciowe nie próbowały tego keszować
* a przede wszystkim, bo twórcy HTTP przewidzieli tego typu problemy i tak nakazali

https://en.wikipedia.org/wiki/Hypertext_Transfer_Protocol#Request_methods


the GEN should be first so it's shown when the rule starts, rather than after it ended (which could have taken several seconds)


:heart: :rocket:


Fixes #2


`-o $@` would be better then, now it's alone


Czyli najlepszym rozwiązaniem będzie rzucenie customowego wyjątku z informacją o tym, tak?


I think that would be enough.

More generally, you are free to choose any linear combination of eigenspace basis vectors of A and B, provided they are not orthogonal. You can utilize this freedom to try to find a pair of vectors that produce the simplest projector possible. This way you may be able to reduce the size of intermediary matrices. I'm not suggesting you do this right now; I think this may be a topic for further inquiry.


Actually this is also needed for the .vapi that I generate from the .gir. Vala doesn't correctly parse gchar **.

Not sure why you would reject it gstrv and gchar ** are compatible, so no abi/api concern


This means at runtime each new frame will call `cl:Buffer`. It's probably better to not allocate `cl:Buffer` 30Hz.

You can use `class OpenCLImage: public Buffer {public: cl::Buffer;}` as a POD, and `OpenCLImageAllocator` to add allocation logic, and wrap `OpenCLImageAllocator` in `PoolAllocator`, and use regular `new OpenCLFrame` in `newFrame()`. Same as in VAAPI processor.


String Type in CSharp

using System;

namespace IntroductionToCsharp
{
    class Program
    {
        static void Main()
        {
            string Name = "\"Yodgorbek\"";
            Console.WriteLine(Name);

        }
    }
}


Could possibly work :) :+1:


@roll Ah, didn't know that a `[pytest]` inside the `tox.ini` worked as well. I think I'd rather use the `pytest.ini`, but any way is OK. My bad, I removed a line that I didn't understand and didn't check with you. Sorry.

You're right about the `changedir`. I don't think we should use it either.

I think the best solution is to remove the explicit `tests` in `[testenv]` and add a `pytest.ini` (or add in `tox.ini`) with:

```
[pytest]
testpaths = tests
```

I've just tried it in datapackage-py and it works as expected :+1:


`buffer_allocator` and `image_allocator`, two are needed.


I'd add a . after key


Watch out for these empty files! The more your files are loaded with unused things, the more tech debt you have.

`rails generate controller` is the least useful of the generators because it generates a bunch of files you'll probably never use. If you generate all of the CRUD methods with the generator, you're also writing untested code, instead of iteratively adding methods as you need them. 95% of the time, I create controllers/views manually, and I recommend that others do too. If you want to use the generator you can do this to skip the junk:

`rails generate controller BananasController  --no-assets --no-helper --skip-routes `

If you install advanced-open-file in Atom, you can do `Cmd+Alt+O` to quickly open/create new files by simply typing in the path.

I do still always use generators for models and migrations, because those do extra things I can't do manually.


@laruence I don't normally look through commits for breaking changes because normally they hit the mailing list. I just happened to see this yesterday.

It's a breaking change. It definitely does not belong in the 7.0 branch.

You may, as you state on the PR, find it weird that people might rely on a predictable stream of pseudo-random numbers, but they do have their uses.


I'm going to replace simlar assertion in ```KeystoneToken::decode_json``` as well.


@rrusso - this has been integrated into the main repo moodlerooms/moodle-theme_snap


why have I never thought of doing that? O.o


zależy jak chcesz z tego korzystać :)

ale ogólnie proponuję wyjątek

i wtedy korzystając z tej klasy robisz:

```
try:
    yt_api.get_search_results('bla bla')
exept YouTubeApiConnectionError:
    // tutaj np ładny komunikat dla użytkownika, że są problemy i żeby spróbował później
```


I was wondering if knowing that this schema is created for MySQL and knowing that MySQL is relational database why not have foreign keys where appropriate.
I did not pay attention who worked on the oracle schema or pgsql or other relational DB schemas too.


me either :sweat:


I tried to justify this in description of the pull request. Similar pattern of service+port→domain is used in DNS-SD, but with slightly different rules, and `DEFINE_PRIVATE_STRING_TABLE_LOOKUP` would not be flexible enough.


:goat:  and another . at the end of this session as well


So the parallel thread pool will be the same size as the default worker thread pool? Not complaining, just checking :)


This should be removed


Don't worry about this. When `->good()` returns false, no new calls to any methods in the OpenCLDepthProcessor should be made.


We're definitely missing a symmetric change as in `incr`


Could simplify to `resource[:owner] ||= metadata.owner.to_s`


could add a comma in section

> Use the widget in a custom form

`in class S3DirectUploadForm(forms.Form):
    images = forms.URLField(widget=S3DirectWidget(
        dest='destination_key_from_settings'
        html=(``

missing the comma between 'dest' and 'html'


Oh I see what you're saying. Makes sense to me. The naming is okay either
way for me.

On Thu, Feb 18, 2016 at 11:22 AM, Tyghe Vallard <notifications@github.com>
wrote:

> Good point about uuid on both parts.
>
> For the factory you could at the very least specify the min_length,
> max_length, min_qual, max_qual
> I have mixed feelings about that which is why the initial factory function
> had no arguments.
> So after that long winded explanation, i think the only benefit is to
> remain consistent.
>
> That all being said, a new idea I just had is to have statically defined
> strategies which come from the factory functions.
>
> Thinking more about it, it seems odd to have the _factory name i suppose
> since it is just returning a strategy.
> The idea is to just "add" new strategies to the already existing ones that
> come with hypothesis, so we should try to remain consistent in the naming
> huh?
>
> Probably just s/_factory//g
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/VDBWRAIR/biotest/commit/b21253bb70438f8773e9511f31d824851194f083#commitcomment-16180285>
> .
>


We also want to shrink the parallel queue in the same fashion?


Simon: Note that you don't need to change the processor to TCP.
syslog configurations are annoying: @localhost means UDP, @@localhost means TCP. You originally had @@localhost


Well, we did not encounter this issue previously and blocking new account registrations is frowned upon, so maybe you can rescue in the controller and send a Clerk notification rather then blowing up.

Unfortunately it's likely account specific, so I doubt you'll be able recreate it unless you OAuth with a user's real Etsy username and password.


btw, jak na to patrzę to można uprościć nazwę:

`search_result = yt_api.search('bla bla')`


I have a macro LIBFREENECT_WITH_PROFILING for this https://github.com/xlz/libfreenect2/commit/30c69be98aeb89fcb9f0a941de0249f2fd16520c#diff-af3b638bc2a3e6c650974192a53c7291R42


`, or @c NULL` shoudn't be removed, at least not unless to make `@nullable` output something special to the HTML


Ffff...


missing **;** ?


Am 18. Februar 2016 17:26:41 MEZ, schrieb Colomban Wendling <notifications@github.com>:
>`-o $@` would be better then, now it's alone
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/kugel-/geany/commit/b741ea76dfb091602e6d065c8e000a635a840d62#commitcomment-16180459

Ok but I'm gonna try to revive the other header at some future point.


Needless extra line. Let's just return the `List.partition`


strndupa! (i.e. emphasis on the "a"...) otherwise we leak memory.


Okay


Russian letters saved in utf-8 is not working when compiling on Windows (windows-1281). Use also can use code.cpp when compiling on linux.


zawsze dawaj cudzysłowy do atrybutów html


Fixed in e8cf722


tu Ci się bug apostrofowy wkręcił


Yes. Correct. I am working on this. Actualy dataTypes is big and need some work before it's ready to commit.
May be next weekend I finish.


I did it to fix the issue mentioned in the commit message, doing it this way kept the changes within a single file.


still, the "any" doesn't belong here, does it?


@leejordan :+1: :heart:


`li` powinno być w środku `ul`


Cool :+1: Not it works for all use cases.


a jak już `li` wsadzisz do `ul`, to `p` w środku będzie nielegalne – powinno znajdować się w środku `li`


Hecho sin la plantilla, queda introducirlo con el esquema dado por el profesor. Uno de los archivos esta corrupto.


That change is unsystematic (and it's also pretty ugly). As it is written `appliedTo` does not work on TermRefs. For instance it would not recognize a TermRef which has a underlying type an EtaExpansion or TypeLambda either. So I believe the right fix is to call .widen.appliedTo instead of .appliedTo whereever the problem was that caused the fix.


Группы инклюдов map-indexer-geometry. Я не уверен что их надо сортировать, сейчас проверил в других файлах - там тоже несортировано


what issue does this fix?


The #[test] attribute is badly indented now. Not that that is a huge problem, but it bothers me :)


Yes, I believe it was that TODO. I'm not working with Extbase (or PHP, in general) at the moment and it's unlikely that I will in the near future. This repository only still exists because I didn't want to just throw away the code back when I wrote it. If I see that someone continues development in his fork, or Extbase gets native support, I'll probably delete it.

So, sorry but no more contributions from me. But I'm glad I could help you at least a bit :)


ce commit me fait des erreurs avec les lignes rajouté >>>>>>>>>>>>>fba7358bd1ee0ccecc0d4c562fb9e779d634f74e


There appear to be a few others in there already. I'll clean them up in a follow up PR.


What a good question !
Anyway, no trace of SieveToMultiMailbox and description corresponds to SIeveMailboxMailet ;-)


@It okey, thanks, then may ,as this is really typo, leave the fix in master, and add a note in changlog, it is much better then only reverting this, thanks


k, lets stay with _factory for now and we can change it later so we can complete this


Oops, I was sure I emit that to the html but it's no like that. Will undo this part


I delete this loop entirely in a later commit. I wouldn't even have this in here if I was a better Git wizard.


Not API concern but C readability one.  Yes, GStrv adds the meaning "NULL-terminated", which is nice, but it also look like a non-pointer type and isn't something everyone knows about.

Alternatively an `@type GStrv` would work for GIR.

Anyway, either way a fix is required because we don't depend on GLib 2.46, so either use an annotation or include `glib-object.h`


![link](https://cloud.githubusercontent.com/assets/6268177/13150439/c71d6812-d633-11e5-9389-c35e897433d8.jpg)

The fork URLs weren't working because `ga-dc` was hardcoded in; everything's now at `ga-wdi-exercises`.


typed_function_ordering (lefticus) - Win64-Windows-10-VisualStudio-14: OK (390 of 390 tests passed, 0 test warnings)

![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)


This comment is in between arguments to a function. It makes it less clear that the line below is input to the function above. Also due to the indentation. If we can pop the list after the close paren on the line above where the comment is now and move the comment above the List.map that would be much clearer.


Any building, really


The front end sends PKs now, not activity names. This saves us a lookup.


TeamCity Civic Assembly :: Build all [Build 66](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1249&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


Booyah. Thank you @JLMcLaugh


sqlite3 (for dev) wants to store the keys as `unicode` strings. (If I try to save the keys `to_string()` value directly, I get: `sqlite3.ProgrammingError: You must not use 8-bit bytestrings unless you use a text_factory that can interpret 8-bit bytestrings`)

But `ecdsa` doesn't have a `.to_unicode()` serializer for keys. And trying to use python's `unicode` built-in gives me this:
```
(push-dev-dashboard)lcrouch:push-dev-dashboard lcrouch$ python manage.py test push
Creating test database for alias 'default'...
...EE
======================================================================
ERROR: test_validate_jws_key_invalid (push.tests.PushApplicationTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/lcrouch/code/push-dev-dashboard/push/tests.py", line 51, in test_validate_jws_key_invalid
    self.pa.save()
  File "/Users/lcrouch/code/push-dev-dashboard/push/models.py", line 40, in save
    self.jws_key = unicode(self.jws_key)
UnicodeDecodeError: 'ascii' codec can't decode byte 0xbe in position 3: ordinal not in range(128)

======================================================================
ERROR: test_validate_jws_key_valid (push.tests.PushApplicationTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/lcrouch/code/push-dev-dashboard/push/tests.py", line 41, in test_validate_jws_key_valid
    self.pa.save()
  File "/Users/lcrouch/code/push-dev-dashboard/push/models.py", line 40, in save
    self.jws_key = unicode(self.jws_key)
UnicodeDecodeError: 'ascii' codec can't decode byte 0xab in position 0: ordinal not in range(128)

----------------------------------------------------------------------
Ran 5 tests in 0.288s

FAILED (errors=2)
```


electric


This is much clearer than the above VBD_epoch_begin. Could you use the same form for that?


@davegrosenick
@Suzanna-InfoPros

This is an example notification that a TW would send to an IR person to let them know that the file is ready for IR. We would include the @mention for the IR person, simply stating that this file is ready for IR.

If you click the link below, you will be taken directly to the commit (diff) for this file. The IR person would make comments as needed, then @mention the TW to relay that the IR is complete.

-Jason


What is the recommended way of adding/removing nodes to an element? Seems like the options are:

1. Expose the necessary methods on `Element`. Currently there's a `<<` (i.e. append child) method. We'd probably need additional methods for adding nodes at a particular index and deleting nodes. In this world `nodes` would either return an immutable array or a copy of the array to prevent client's from mucking with internal state of the `Element`.
2. Allow clients to manipulate the array returned by `nodes` via the `Array` interface which supports everything we'd need. Clients are free to hang on to references to this array and mutate it at anytime.
3. A mix of 1 and 2 where the array returned by `nodes` can be mutated to add/remove child elements but convenience methods on `Element` can be used for common operations like append.

Option 3 is the world we're in prior to this change. My concern is that after this change the semantics are clients can manipulate the return value of `nodes` to change the node's children until they call `replace_text` at which point they'd need to get a new reference to `nodes` to make any further modifications. This seems error prone to me.


:blush:


Ok, I agree if that was the intended behaviour.


It looks like here you are adding 2 spaces. I suggest you move the help_text to another var.


`[yes|no]` this does not render properly in markdown...


snprintf returns an int, size_t is unsigned.


We usually use SOL_WRN() instead of SOL_ERR()


Hm, just now that I noticed that we are overwriting `cxxflags`.


Just curious, what was the error here?


I actually do it differently in a later commit. Something to look forward to!


n is size_t, it will never be < 0. Also the check is wrong, it should be if (n < 0 || n >= (int)len)


Even though this is unicode, I was getting a "not JSON serializable" error when using it with `json.dumps()`. So when creating the API response we apparently have to explicitly cast it.


@TheDiamondYT1 Use password recovery by email. PS: Push permission added.


Hey there, I've been looking at the code and I think it needs a major overhaul if we want to keep building a really good calculator.

I tried to adjust the code myself but it won't let me make a major change to the master file. If you still want to build a really good calculator I was able to find a couple of basic code forms online that would help you.

Here is one:
http://www.codeproject.com/Tips/853405/A-Simple-Calculator-in-Windows-Forms


po co `str()`? nie może zostać jako liczba?


If you leave it as is, you should probably rename the function to "has_pool_data".
But you might as well just let the function do the incrementing for you:

def get_pool_data(pool, key1, key2):
    if pool.get(key1, {}, get(key2, None)
         return 1
    return 0

Then later
data['poolThinProvisioningSupportCount'] += get_pool_data(pool, 'capabilities','thin_provisioning_support')


Electric-Service-Refactor (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (985 of 988 tests passed, 0 test warnings)

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


strlen() returns a size_t


Minor: I would note this is an example; as written it sounds like an instruction.  Aka "In this example, we show the bowerrc configuration for..."


typed_function_ordering (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-AddressSanitizer: OK (3048 of 3048 tests passed, 0 test warnings)

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


Save state support is incoming for DMG/GBC games only. GBA development is still in flux atm and would only delay a 1.0 release trying to add support for that. LCD and APU need to be serialized as well.


Why not use `ensureConforms`? Also it would be good to avoid the duplication (and shorten the line). So:

    def adaptToField(tree: Tree) =
       if (tree.isEmpty) tree else tree.ensureConforms(field.info.widen)


Common Operators

using System;

namespace IntroductionToCsharp
{
    class Program
    {
        static void Main()
        {
            int Numerator = 10;
            int Denomerator = 2;

            int Result = Numerator % Denomerator;

            Console.WriteLine("Result {0}", Result);

        }
    }
}


I think it does. It shows all available records, which is quite useful.


Do you really need this NULL check in a static function?


Please file a bug for the TODO


typed_function_ordering (lefticus) - i386-Windows-10-VisualStudio-12: OK (389 of 390 tests passed, 0 test warnings)

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


Since you are inheriting ``GroupForm`` we are you defining the widgets from scratch? You can just inherit:

```
class Meta(GroupForm.Meta):
    fields = <new fields>
```


`null` or `undefined`? Should we leave this out of the key/value altogether since it can't be serialized?


Will also work for any organization/user name now


return an negative errno?


If you use proper kernel code hal works
We have unified all devices to use /sys/class/leds/lcd-backlight/brightness


I defer to others, but the private and public registries (as you may have noted) are pretty much the same.  In other words, you can add a private registry to your group.  I am not sure we need to distinguish in an additional example here.


The code above indicates `Type` can be `null`.  Do we need to guard against that here?


...and cast only the first operand instead of the result of the integer division.


TeamCity Civic Assembly :: Build all [Build 66](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1249&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Tests failed: 166 (1 new), passed: 6, ignored: 4 Build time: 00:06:38
### Failed tests
```
Blog categories tests: Should delete blog category new blog category 1455813675109: <no details avaliable>

Blog categories tests: Should not create new blog category if cancelled: <no details avaliable>

Blog categories tests: Should add new blog category: <no details avaliable>

Blog categories tests: Should not change blog category: <no details avaliable>

Blog categories tests: Should change blog category: <no details avaliable>

Blog categories tests: Should not delete blog category if cancel pressed: <no details avaliable>

Blogs tests: Should not create new blog if Cancel button pressed: <no details avaliable>

Blogs tests: Should add new blog: <no details avaliable>

Blogs tests: Should not change blog if Cancel button clicked: <no details avaliable>

Blogs tests: Should change blog: <no details avaliable>

Blogs tests: Should not delete blog if Cancel button was pressed: <no details avaliable>

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


:heart:


Do these filters need to be key/value pairs?


This seems to need syscall.h for musl libc.


This fixes #42


Should you return NULL if it's not bluetooth?


Having this button set up as a submit causes the page to reload when a curious user clicks it. The `<form>` tag and the `type=submit` on the button cause this. Basically, for the rest of the front end class it's best not to use those since they cause the code to try and load a new page. Since we're doing only single page apps for front end, we don't want that behavior.


SOL_WRN here


Sergio ha realizado el primer cambio


@nicholasturner1 the `suffix` was never used?


You're not checking the return value, this function call can fail.


* devtools::install_github()


Fixed #39


This is a duplicate of the function in xenops_server.ml. Let's factor that out somewhere common.


If the filters or request info have changed, the outstanding request may result in a different data set. We will probably ultimately need to queue the need to reload and refresh again when the current request is completed.

There would never be a reason to queue more than one need to refresh unless the params for the request are included in the queue.


I don't think anybody will understand what *rules* you are talking about here.


Agreed. I forgot that after erasure `ensureConforms` delegates to  Erasure.Boxing.adaptToType.
I'll make the change


Do we need to apply paging here at some level so the filteredDataSet becomes 10 items (or page size)


to na pewno fajny byłby taki scenariusz:

1. ładujesz stronę główną
2. „klikasz” przycisk dodawania taska
3. uzupełniasz formularz
4. sprawdzasz, że widzisz nowego taska

taki typowy flow co robi użytkownik


So this is _more_ threads? In addition to the additional thread pool.


If it's OK, I'm going to look to make sure I understand the issue before merging this, though it seems clear that this fixes the specific issue.


removing in favor of new serialize property.


Be careful when returning a sol_vector, if you add a new element to the vector and the user is holding a reference to an element of this vector, it will become invalid! (sol vector uses realloc)


C'est juste que du point de vue perf, la méthode bourrin est plus rapide.


Congrats!


Do we still need this, now that we use autocomplete?


Yes eventually. At least for the smart data source.


anyway to append to it?


None of your test cases verify that null will be the serialized value if the active option is missing.


po zapisaniu nowego taska zrób redirect do strony z tym taskiem lub listą tasków

chodzi o to, żeby nie stworzył Ci się duplikat jak użytkownik odświeży stronę


I notice this condition is only provided for routes and outings (not for WP for instance). Is this correct?
I'm updating the Image model: do I need to add this ``inherit_condition`` as well?


Yes. I actually changed this to a _.mapValues later, which should fix it


No test verifying the exported value for displayAll


Furtado,
Estou fazendo mas não está fácil creio que vou precisar fazer um laço, depois de verificar e dividir o valor, depois e pensar como ele apresentar os dados


Alguma dica?


Yeah... seems like this data source will die once we finish the smart data source


Shouldn't we just export `null` if displayAll is true, instead of having an object with `displayAll` and `value`?


This is going to be confusing under buttons.less. Maybe a simple rename and move of `button-group` would help. Let's mull it over in the morning - over our business coffee, where we do business.


Oh really, my fault. thanks!
My bower installed v0.13.2


I was assuming that all paging would be server-side for this data source.


oh, that's perfect!


What would this be used for?


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1312.9a785abc25549352c05fc3991b6b118b1687c84a](http://build.palaso.org/viewLog.html?buildId=69358&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 1016, ignored: 12 Build time: 00:07:30


Inconsistent closure syntax. Using the style of the next line is better:

    !_.is(Deferred)


How about if we add an exception on the main query function in the database_table.php
```
 if (!$result = self::$conn->query($sql)) {
            throw new Exception("opps! something went wrong...");
        } else {
            return $result;
        }
```
and then we can add try/catch blocks on the rest of the query functions within the database files rather than the rest of the code. e.g
```
public static function get_items() {
        $sql = "SOME QUERY";
        try {
            return parent::query($sql);
        } catch (Exception $e) {
            echo $e->getMessage();
        }
    }
```
**Pros:**
- It keeps all the exception checking within the database files and every function called will automatically be checked for exceptions.
-It will return a standard and simple message to the end user that something went wrong and maybe the steps they can take to rectify things.

**Cons:**
- Since it'll be a generalized exception for all queries, it doesnt give us much information if we want to  use these for debugging.


typed_function_ordering (lefticus) - Win64-Windows-10-VisualStudio-12: OK (389 of 390 tests passed, 0 test warnings)

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


What about `tokenByName`?


Fixed in 92606cf1100e4455a864eccd587cf064f303274a .


Yes, but I this change don't come from my commit, so this fix should be in another separate commit


Yep, this: fb6185ec42cd8e7f3e494b882f4d0aaad3ce54b1


Hmmm, well I guess this is not a widely used public API for this class, so the method name change doesn't seem to be important, but where is the corresponding change to whatever was using `create_archiver`?  Did I miss something?


Yeah it's more of a "short-cut"
@mstembera I tested it at STC...but failed yellow.  2 versions:
http://tests.stockfishchess.org/tests/view/56bfed460ebc590247cdfeb1
http://tests.stockfishchess.org/tests/view/56c1493c0ebc590247cdff0e


Generally this kind of behavior is handled by a 500 error.

The `if (saved); redirect; else; render` pattern is when the user is inputting behavior that didn't work. If your app can't delete something, then that's actually an application error, not a user error, and that should be considered a failure that you let fall through to a "500 error" page.

`destroy` method in the wild is often quite simple:

```ruby
def destroy
  Bookmark.destroy(params[:id])
end
```


That works. I wasn't sure what to name it.


Thanks @teichsta !


I see, whatever transaction went through control plugin will have an attempt (and conversely, whatever transaction did not go through control plugin will have *not* an attempt).

:+1:


nit: space after `if` :)


:+1:  re changing back from check box to control


same here :)


Good catch, totally missed this one


Can you please verify that this works for `bytes10abc`, i.e. that `bytes10abc` is an identifier and not interpreted as `bytes10`? I.e. I think you also need to check that `positionX == _literal.end()`.


Did you compile using the HEAD? this macro does not exist anymore.
It's called SOL_UTIL_ARRAY_SIZE()  now


Dips?  :fork_and_knife:


these are those useless routes that the `rails generate controller` adds. Always remember to delete them! Route table parsing tends to be one of the most expensive operations for large-scale Rails apps, so it's good to be in the practice of keeping them as slim and clean as possible.


There's probably a way to attach this without using an `onsubmit` handler.  Perhaps using the `$(document).ready` handler above, and then we keep all the tracking together and could wrap it in a closure so as not to bleed the function out to the world where it shouldn't be.


upps.. must have been a miss on my side :/ I will (hopefully) do PR off all sxx ops today so I can fix it then if that is ok.


you should use `ninja install` because there are more files you will want to put in /usr


Please add documentation `@returns the keyword with name @a _name or Token::Identifier of no such keyword exists.`


刚接触，有些API还没搞明白。。。


@udeved you see at the line 31 and 68 my modification . 31 > home page and 68 > theme used . At the moment every iso created whit new structure open a fresh firefox ( 3 tab whit presentation of firefox ) so i not know if is aspected
http://kb.mozillazine.org/Prefs.js_file


oops :koala:


Is it safe to assume the meta will be the last element to the right of the heading, therefore the right margin isn't required?


Do you need to unregister if bt_enable fails?


Also, the prior code intended to memoize the archiver for any request.  Can't help wondering whether it could be a class constant.  Does this helper need to provide any args to configure the `Dor::WorkflowArchiver`?  The prior code called the `connect_to_db` and, presumably, that would raise an exception if things were not setup right?


typed_function_ordering (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (390 of 390 tests passed, 0 test warnings)

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


Do you need to unregister and bt_disable if bt_advertise fails?


Oops I made a stupid. Purge, not username... It's too early.


This was the only place `create_archiver` was used, presumably because the `connect_to_db` was unsightly enough someone wanted to bury that. Now that it isn't necessary, I don't think we need it.


Understood the meaning of sitrep. Concern is it may not be universally understandable, culturally wise.

How about 'conn_status' ?

Naming can be hard... :)


fixed, thanks


This ARRAY_SIZE is from the bluetooth API.


LGTM.


can you quote `max_bytes` and `max_objects` in the message? like
```
"'; should be 'max_bytes' or 'max_objects'"
```


There is no such function in the API...


That's a good point. I like being explicit about my styling but you're right in saying that any `li` inside of `.tabs` will be a tab. I'll make this change for the sake of making it easier to port over.


using System;

namespace IntroductionToCsharp
{
    class Program
    {
        static void Main()
        {
            int Number = 10;
            int AnotherNumber = 20;

            if (Number == 10 || AnotherNumber == 20)

            {

                Console.WriteLine("Hello");

            }
        }
    }
}


These variables are already present around line 366 as `selectionStart`, `selectionEnd` and `selectionDirection`.


} catch (IOException e) {


inputStream.close();


Useless return.


I think this `ENV` can be removed as it's defined bellow


// System.out.println("in");


How about `state`

On Thu, Feb 18, 2016 at 10:55 AM, Brent Picasso <notifications@github.com>
wrote:

> Understood the meaning of sitrep. Concern is it may not be universally
> understandable, culturally wise.
>
> How about 'conn_status' ?
>
> Naming can be hard... :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/autosportlabs/RaceCapture-Pro_firmware/commit/2285607b9fc0aa32ba9fd71578667feee6eb0722#commitcomment-16181508>
> .
>

--


I guess you mean DBG


It's considered good practices to use Lists, like

```
  <ul><li><a>111</a></li><li><a>222</a></li></ul>
```

Generally **br** is discouraged. It's functionality can be achieved by using **P** tags, **DIV** tags or lists, which will provide more structured document.


Hi, is there any particular reason to expect a comma separated list of hosts instead of a list of them?
Also, I'd like to patch this to have the possibility to also include players from config instead of discovering them, autodiscovery proved to be not reliable for me


Add an empty line before this on (I could be an offender too of this guideline…)


decr() removes a worker from the thread pool. The queue of the worker is only important in incr().


missing static?


And the credentials revoked...


not sure if we want to keep this commented code around even, not to mention modify it...


grazestrap alert. :frowning: Is there a speciifc place we're having issues with this? Perhaps there is  a specific modifier we could use for those cases to enforce ui consistency?


You can just use `CUSTOM_FORMATS.key?(params[:slug])` here for the same effect.


DBG?


// System.out.println("out");


    is (IS_…
      ^


Add an empty line before this one (I could be an offender too of this guideline…)


Le fichier a été mit a jour il y a 22 jours, donc  il te faut passer par le menu parametre de vstream puis outils et forcer la mise a jour des sources.

Ce sera plus facile a la prochaine version.


Yes, I suggest to cast to an integer. Casting to an int, would be considerably faster on all platforms.


LGTM


Do you need to remove from the bound socket list?


DBG


It has the property that there will be at most the same number of VBD atoms as there were VM operations originally, so we'll never use more dom0 memory than before in the sm layer.


Citation needed for the batch_wordstats function, wordstats function, and generation plots.
https://github.com/mylesmharrison/radiohead_lyrics/blob/master/radiohead.R


I definitely had to. Were you using an older django or somehow had it disabled (can you do that)?

It's an extra step, so if it can be left out somehow that would be great.


You don't need to inject Model, if you are using **ModelAndView** as result of action


I don't think it can be done like this, because the gradient is now on top of the text and button


state is perfect.

On Thu, Feb 18, 2016 at 8:57 AM, Andrew Stiegmann <notifications@github.com>
wrote:

> How about `state`
>
> On Thu, Feb 18, 2016 at 10:55 AM, Brent Picasso <notifications@github.com>
> wrote:
>
> > Understood the meaning of sitrep. Concern is it may not be universally
> > understandable, culturally wise.
> >
> > How about 'conn_status' ?
> >
> > Naming can be hard... :)
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/autosportlabs/RaceCapture-Pro_firmware/commit/2285607b9fc0aa32ba9fd71578667feee6eb0722#commitcomment-16181508
> >
> > .
> >
>
>
>
> --
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/autosportlabs/RaceCapture-Pro_firmware/commit/2285607b9fc0aa32ba9fd71578667feee6eb0722#commitcomment-16181566>
> .
>

--
Brent Picasso
*Autosport Labs <http://www.autosportlabs.com>*
*Technology for Race and Street*


closes #376


This is a good idea, but I think we could push it a bit further because the `render` call in the `CUSTOM_FORMATS.keys.include?(params[:slug])` branch still looks a little cumbersome.  If we rejig the hash a little we could have:

```ruby
CUSTOM_FORMATS = {
  "report-child-abuse-to-local-council" => {
    locals: {
      option_partial: "option_report_child_abuse",
      preposition: "for",
    }
  },
  "check-vehicle-tax" => {
    partial: "check-vehicle-tax",
    locals: {
      full_width: true
    }
  },
  # the rest
}

if CUSTOM_FORMATS.key?(params[:slug])
  render {partial: @publication.format}.merge(CUSTOM_FORMATS[params[:slug]])
else
  render @publication.format
end
```

Might even be able to golf it down to:

```ruby
CUSTOM_FORMATS = { # as above }
render {partial: @publication.format}.merge(CUSTOM_FORMATS.fetch(params[:slug, {})
```

Although I think that's maybe a little indirect.


why?


perhaps here we could call decr() twice.


If somebody decides to parse toString he is out of luck in the same way he is if he decides to access the tags reflectively with the setAccessible(true) hack.


What about using a FIFO/ordered-map instead of searching for the older cell each time?


I can't spell the negative word I wanted to use, so, okay?

Although it's pretty obvious what they do even without the background so maybe just delete it?


This might be a little subjective, so feel free to leave as-is.  The "trick" here is making your hash contain the exact params for a call to render.  Just need to check if the first arg name should be `partial: ` or `template: `.


@expobrain can you create a pull request for this?


Again, comments here look wrong, can we not leave this on 3 lines as:

```
Message message = new Message(
  "I", /* name */
  "will", /* data */
  "fail", /* mismatching client id */
);
```


the comment is supposed to refer to the line [ "RW", vbds_rw; "RO", vbds_ro ] just below the comment.


LOl I have the same fix in my pullreq.


fix for https://issues.apache.org/jira/browse/CXF-6783


Looks like you misspelled 'Canola'!


`rails generate controller` also makes these useless view tests. It's very rare to write view tests with rspec. Standard practice is view testing with capybara or Cucumber and model/controller/integration/services with rspec.


but why would you return -EINVAL if strdup fails?
It fails with -ENOMEM.

You may want to set r with -errno on case of strdup fail too


There wasn't any value in memoizing the connection (and, in fact, may be actively harmful for garbage collecting...). Creating the instance is an incredibly cheap operation compared to the data retrieval we do with it.


There's no rush whatsoever. Or as the saying goes "PR thrice, merge once", right?


ack


Also would be good to add a demo to the docs


there's much more code to duplicate in VBD_epoch_begin than in here.


great work


It's related to a "Stupid Quake bug"? If so, this should be fixed in VectorAngles. Also, it's necessary to fix all mods where taken in account the existence of this bug.
http://cs-mapping.com.ua/forum/showthread.php?t=37315 (ru)


I removed it because of how flaky the spec is about it. I gave the spec another read and I think I'll add it back in based on:

> Not all groups of links on a page need to be in a nav element — the element is primarily intended for sections that consist of major navigation blocks.

I'll strip the `class="menu"` from it though.


Instead of updating the scripts for each file, can loop over ones found in the dir, e.g.:
```bash
#!/bin/bash
FILES=src/properties/competitionbot/*
for f in $FILES
do
  echo "Copying file $f to roboRIO..."
  scp $f admin@roborio-2175-frc.local:/home/lvuser
done
```


Also added logo.


lulwat did you get this comment ?


Amazing commit title :+1:


Perfect, gracias. :+1:


typed_function_ordering (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-NoThreads: OK (394 of 397 tests passed, 0 test warnings)

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


You could move that further out.


@tjhiggins My bad! 083918a8b58106691474c4e52a1ea05014683ec3


I think postgres provides some operators for querying json columns. Like.. `select * from topics where topics.tree <@ '{"key": "value"}'` or something. I've never actually used them though..


"FullScreenTest", "WakeLockTest", "RenderViewTest", "ShapeTest", "BitmapTest",


public void onCreate(Bundle savedInstanceState) {


I suggest to name this "competingMethods" instead. Also, the definition can be streamlined like this:

    lazy val competingMethods = cls.baseClasses
      .filter(_ ne meth.owner)
      .map(meth.overriddenSymbol)
      .filter(_.exists)


super.onCreate(savedInstanceState);


Intent intent = new Intent(this, clazz);


Aplicaciones android


@lucasdemarchi somthing like this http://ix.io/osT


I would move the `createTableRow` function up with the other functions. and then all this code that creates a new table that goes into the `table` variable could go in a new function called *createTable*


inputStream.close();


Siiii


WakeLock wakeLock;


Why did we go for the json thing here rather than another table btw? Is there some future feature where we will have to nest them?


haha what were you trying to achieve?


those are really :cool: pictures


public class AndroidBasicsStarter extends ListActivity {


@esredson notei uma regressão nesse commit. Esse é um caso clássico onde o uso de StringBuffer/Builder (Buffer é melhor por ser sincronizado, mas não é o caso) é recomendado ao invés de concatenação.

No caso deste código, que é bem parecido com o segundo link abaixo, a cada iteração um (ou dois, se cair no if) StringBuffer é criado, o valor de acrônimo é copiado para o sb e então o valor de s é copiado também. Muitas cópias e muitas instanciações, gastando muita memória e muito processamento à toa.

No caso anterior, também há um erro: `acronimos.append("|" + s);` Aqui um StringBuffer é criado, "|" é copiado e s é colado no final, pra depois ser adicionado em outro stringbuffer. O correto seria `acronimos.append("|").append(s);`.

Leitura:
[Com relação ao processamento](http://stackoverflow.com/questions/4648607/stringbuilder-stringbuffer-vs-operator)
[Com relação ao uso de memória](http://www.yoda.arachsys.com/java/strings.html)

Sei que isso pode parecer bem bobo, mas é um caso clássico do por quê "java é lento" :wink:. Precisamos ficar atentos à esses detalhes pra construirmos aplicações mais robustas e com mais qualidade.

"The devil is in the details"


На самом деле я сам, часто, изза лени, использую БР, когда надо тчо-то отбить отступом, или сделать абзац, но это плохо.


android.R.layout.simple_list_item_1, tests));


protected void onListItemClick(ListView list, View view, int position, long id) {


correct. The interface only _mandates_ 2 args. But there _might_ be more args, whenever something (like Router) passes a third one.


right :) I also figured out a way to reduce the duplication of the scc so I think we can actually close the shi PR as the code is being rewritten a bit anyway. The only thing that changes in the instruction is the ```cond_hi/cs/etc``` bit so I will do a macro for it instead.


I don't like much this to be a KeyValuePair


Unfortunately we cannot reuse the xenopsd worker threads here. John said that this was fine for now. He has plans to have a request Limiter for the Storage module later that would fit quite well to limit global access to the sm layer.


It's really too bad that one can't concat names in macros to generate new identifiers. That would have reduced the code even more.


using System;

namespace IntroductionToCsharp
{
    class Program
    {
        static void Main()
        {
            int Number = 10;

            bool IsNumber10;

            if (Number == 10)

            {
                IsNumber10 = true;
            }

            else
            {
                IsNumber10 = false;
            }

            Console.WriteLine("Number == 10 {0}", IsNumber10);

        }
    }
}


`sql` got 2 votes which is more than enough for this bikeshed. :-) I'll have to work on a PR for it later as it touches a pretty significant amount of the docs. Also a free-standing variable named ``sql`` is less intuitive so in those cases I'm going to name it ``session`` instead and assume the context will make it clear what it is. For example `session = request.sql`.


This needs travis-ci/apt-package-whitelist#2428.


Seems like a lot of lines are claiming they've changes. Is this a line ending issue, and if so is it good->bad or bad->good?


onPause


You can only create this in the else clause, preventing to create instances when memCache contains the id entry.


This seems not to be in use anymore.


so we need to adjust the interface and add the $args into the method declaration?


@ibriano please check this commit
it really seems to me that edge == none doesn't make sense.
What is the point on having a gpio that never read its value?


Technically, the spec also doesn't mandate a description for an application. I assume you want to change the spec? (I would support that.)


setContentView(textView);


Added later.


agreed

On Thu, Feb 18, 2016 at 9:00 AM, Daniel Frenzel <notifications@github.com>
wrote:

> Yes, I suggest to cast to an integer. Casting to an int, would be
> considerably faster on all platforms.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/diydrones/ardupilot/commit/a9c985bfb3e9cff17cd7c9f6b7b1bd1348778d5a#commitcomment-16181629>
> .
>


There is a demo on the forms page


In the former implementation this `memCache` is a ObjectCache. Why have you changed to be a MemoryCache?


I didn't write a lot of this code honestly. I think this was to leave res set with the DNS resolver set to what we wanted and not reset it. More info is here: http://blog.internot.info/2015/03/specifying-dnsns-server-for-address.html


Le valgrind est pas joli par contre. Voilà le résultat:
```
valgrind --leak-check=full ./test
==30422== Memcheck, a memory error detector
==30422== Copyright (C) 2002-2015, and GNU GPL'd, by Julian Seward et al.
==30422== Using Valgrind-3.11.0 and LibVEX; rerun with -h for copyright info
==30422== Command: ./test
==30422==
> Préparation des tests:
> création des pièces...
Done.
> new_game_hr...
==30422== Invalid write of size 4
==30422==    at 0x400862: copy_piece (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==    by 0x40100C: new_game_hr (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==    by 0x401725: main (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==  Address 0x51de2b8 is 0 bytes after a block of size 8 alloc'd
==30422==    at 0x4C28C4F: malloc (vg_replace_malloc.c:299)
==30422==    by 0x400FA5: new_game_hr (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==    by 0x401725: main (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==
==30422== Invalid read of size 4
==30422==    at 0x4015E8: display_pieces (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==    by 0x40173D: main (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==  Address 0x51de2b8 is 0 bytes after a block of size 8 alloc'd
==30422==    at 0x4C28C4F: malloc (vg_replace_malloc.c:299)
==30422==    by 0x400FA5: new_game_hr (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==    by 0x401725: main (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==
Pièce [0] : x = 0, y = 3
Pièce [1] : x = 2, y = 4
Pièce [2] : x = 5, y = 3
Pièce [3] : x = 0, y = 0
Done.
> création d'un jeu vide...
==30422== Invalid write of size 4
==30422==    at 0x400862: copy_piece (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==    by 0x40100C: new_game_hr (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==    by 0x4017BA: main (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==  Address 0x51de538 is 0 bytes after a block of size 8 alloc'd
==30422==    at 0x4C28C4F: malloc (vg_replace_malloc.c:299)
==30422==    by 0x400FA5: new_game_hr (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==    by 0x4017BA: main (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==
==30422== Invalid read of size 4
==30422==    at 0x4015E8: display_pieces (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==    by 0x4017D6: main (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==  Address 0x51de538 is 0 bytes after a block of size 8 alloc'd
==30422==    at 0x4C28C4F: malloc (vg_replace_malloc.c:299)
==30422==    by 0x400FA5: new_game_hr (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==    by 0x4017BA: main (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==
Pièce [0] : x = 0, y = 3
Done.
> copy_game...
nb_pieces Done : attendu : 4, recu : 4.
nb_moves Done : attendu : 0, recu : 0.
==30422== Invalid read of size 4
==30422==    at 0x400E71: get_y (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==    by 0x40085B: copy_piece (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==    by 0x401260: copy_game (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==    by 0x4017FD: main (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==  Address 0x51de2b8 is 0 bytes after a block of size 8 alloc'd
==30422==    at 0x4C28C4F: malloc (vg_replace_malloc.c:299)
==30422==    by 0x400FA5: new_game_hr (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==    by 0x401725: main (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==
Pieces src :
==30422== Invalid read of size 4
==30422==    at 0x4015E8: display_pieces (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==    by 0x40181F: main (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==  Address 0x51de2b8 is 0 bytes after a block of size 8 alloc'd
==30422==    at 0x4C28C4F: malloc (vg_replace_malloc.c:299)
==30422==    by 0x400FA5: new_game_hr (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==    by 0x401725: main (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==
Pièce [0] : x = 0, y = 3
Pièce [1] : x = 2, y = 4
Pièce [2] : x = 5, y = 3
Pièce [3] : x = 0, y = 0
Pièces dst :
Pièce [0] : x = 0, y = 3
Pièce [1] : x = 2, y = 4
Pièce [2] : x = 5, y = 3
Pièce [3] : x = 0, y = 0
Done.
> delete_game...
i=0
i=1
i=2
i=3
Done.
> game_nb_piece...
Done, attendu = 4, reçu = 4.
==30422==
==30422== HEAP SUMMARY:
==30422==     in use at exit: 228 bytes in 14 blocks
==30422==   total heap usage: 21 allocs, 7 frees, 348 bytes allocated
==30422==
==30422== 12 bytes in 1 blocks are definitely lost in loss record 6 of 11
==30422==    at 0x4C28C4F: malloc (vg_replace_malloc.c:299)
==30422==    by 0x400F4B: new_game_hr (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==    by 0x4017BA: main (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==
==30422== 24 (12 direct, 12 indirect) bytes in 1 blocks are definitely lost in loss record 7 of 11
==30422==    at 0x4C28C4F: malloc (vg_replace_malloc.c:299)
==30422==    by 0x40175B: main (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==
==30422== 96 (48 direct, 48 indirect) bytes in 1 blocks are definitely lost in loss record 10 of 11
==30422==    at 0x4C28C4F: malloc (vg_replace_malloc.c:299)
==30422==    by 0x40166D: main (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==
==30422== 96 (16 direct, 80 indirect) bytes in 1 blocks are definitely lost in loss record 11 of 11
==30422==    at 0x4C28C4F: malloc (vg_replace_malloc.c:299)
==30422==    by 0x400F16: new_game_hr (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==    by 0x401725: main (in /autofs/netapp/account/cremi/ahalnaut/S4/Projet/rush-hour-clone/rush-hour/test)
==30422==
==30422== LEAK SUMMARY:
==30422==    definitely lost: 88 bytes in 4 blocks
==30422==    indirectly lost: 140 bytes in 10 blocks
==30422==      possibly lost: 0 bytes in 0 blocks
==30422==    still reachable: 0 bytes in 0 blocks
==30422==         suppressed: 0 bytes in 0 blocks
==30422==
==30422== For counts of detected and suppressed errors, rerun with: -v
==30422== ERROR SUMMARY: 22 errors from 10 contexts (suppressed: 0 from 0)
```


FFgthrnynty,ukuyky


I accidentally deleted this in fde4628f7fe9cbb25de2c099bff4843004932991 when trying to merge it with `create_draw`. But is `create_with_all` actually used anywhere? My git grep shows nothing.


Should we really update the version number here? We do not publish new manuals for patch releases, and do not ship it with the product. As such, the documentation is 'rolling' within a given minor release.


What do you suggest since we need the throttle counter and generated id?


Yes that would be great! Just play around in the code till something works, [here](https://github.com/marcofugaro/spacegray/blob/betterfileicons/Spacegray.sublime-theme#L652) starts the folder icon section in the code, and [here](https://github.com/marcofugaro/spacegray/blob/betterfileicons/Spacegray.sublime-theme#L723) is where I handle the setting "spacegray_fileicons" (the margins and stuff)...

[Here](https://github.com/lefoy/cyanide-theme/blob/master/Cyanide.sublime-theme#L632) is the section of the code of the theme which has already implemented the folders, and where I took the icons, [Cyanide Theme](https://github.com/lefoy/cyanide-theme).

I also will try to work on this project in the next few days if I can, so I can pull this in, thanks for the help!


This commit and other commit from 2/9/2016 were made by Namchi, but because I was still logged in, it was accidentally committed in my name.


I only follow the graysky2 patch in this first step because its more safe. But we can use -march=silvermont in all points in the next version.


Yes, there must be a description for apps - this is probably outdated in the spec because previously descriptions were supposed to come from the distribution and distro packages, so enforcing a long description wasn't necessary.


kk


yeah. this was wip; i was expecting you to review the squashed diff rather than my wip.
I think I'm going to git rebase -i and then apply your review comments.


No, because PHP doesn't like that:

`./sapi/cli/php -r 'interface x { function a(...$args); } class y implements x { function a() {} }'`
=> `Fatal error: Declaration of y::a() must be compatible with x::a(...$args) in Command line code on line 1`

It's an optional arg, not an obligatory one. If you don't use that arg, you shouldn't have to specify it....


What if someone wants to read from a GPIO only when some other condition is met? That's why I had left it that way.
The node type, on the other hand, complains about not having any trigger mode set, and always passes one that will make sense in the flow, where we have no way to read only upon user request.


i changed that to final ! to see the pics, then i didnt want to commit it, so i changed back to draft !


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


Missing language specifier: swift


d


I don't think the key order is important; the test is no spaces.
Dicts are unsorted by design.  Verifying that their string output is sorted in a particular way will break for someone.  Aiui, tox intentionally adds a bit of randomness to the sort by design, so tests need to allow for this (either sort the dict's keys before self.assertEqual(), or allow for unsorted dicts).


is this a typo? Perhaps you meant dataProvider?


Text-alignment


typed_function_ordering (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (398 of 398 tests passed, 0 test warnings)

![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)


I think this should be parameterised by the queue otherwise some pattern of incr and decr will leave the pools lop-sided because of the find_one in decr()


We shouldn't obfuscate toString(), but rather communicate via javadoc that there is no guarantee of a specific format.


see above


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


Define a ResponseModel. That can be a type inside this type, like `ThrottleCounter` that contains both the ThrottleKey and the ThrottleCounter.


Not that it matters, but this is the same as `'keyword-spacing': 2,`


Yes, it's still in between a function and it's argument. this list is the argument to the List.map


seems to be a good approach


int code;


But that would be inconsistent with the other implementations. It's threat as error on contiki and zephyr already


public void onCreate(Bundle savedInstanceState) {


setContentView(textView);


Are there any instances in the code where a persistent connection is required or multiple requests require cookie or session data to be stored in the client?


case MotionEvent.ACTION_MOVE:


break;


Then those are wrong, in my opinion. I hadn't noticed it on Zephyr, my bad. Contiki really needs some love.


Electric-Service-Refactor (EnergyArchmage) - x86_64-MacOS-10.9-clang: OK (2080 of 2084 tests passed, 543 test warnings)

 * 547 tests had: AUD diffs.
 * 477 tests had: RDD diffs.
 * 3 tests had: ESO small diffs.
 * 7 tests had: MDD diffs.
 * 7 tests had: MTD diffs.
 * 3 tests had: EIO diffs.
 * 1 test had: SHD diffs.
 * 2 tests had: ERR diffs.
 * 4 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 4 tests had: Table big diffs.
 * 1 test had: SSZ big diffs.
 * 1 test had: ZSZ big diffs.

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


Try to merge this back to Project


Private?


Not in the method which is on the stack. I mean it becomes OpenCLDepthPacketProcessorImpl::p0_table[IMAGE_SIZE].


https://github.com/tc39/ecma402/pull/78 implements this.


Oh, cool! I'll change it to do that.
On Thu, Feb 18, 2016 at 11:05 AM Jeff Jensen <notifications@github.com>
wrote:

> Instead of updating the scripts for each file, can loop over ones found in
> the dir, e.g.:
>
> #!/bin/bash
> FILES=src/properties/competitionbot/*for f in $FILESdo
>   echo "Copying file $f to roboRIO..."
>   scp $f admin@roborio-2175-frc.local:/home/lvuserdone
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/frc-2175/2016RobotCode/commit/8e7e6f959843d7f9c6eeb09bb992c2ffb4843602#commitcomment-16181774>
> .
>


@TimConnelly7 thanks for the suggestion. I had some braces with white space around and some without, I was actually having trouble deciding how to style that. Noted for the future :)


The `new date` needs to have the d capitalized to `new Date`


no I guess. can you compare against hardcoded values for now?


@j6k4m8 Unfortunately no, its required for 16-bit tiffs to be shape checkable


👍🏻 good! I will reply to your long message soon!


down


break


The default in the old consumer was true.


No need to make a difference here between real and average.  The average price should be calculated automatically in quants_move in move_validate also, so it is wrong to set the standard price here yourself.  You can leave out standard_price case too. => if move.product_id.cost_method in ('average', 'real')....


I think you just need to call initPlaceholder.

Is the plugin still useful in the modern web era? I haven't updated it in
years but I don't know if anyone would care

On Thu, Feb 18, 2016, 5:42 AM sbellone <notifications@github.com> wrote:

> Hi, just to let you know that this change breaks your plugin in all
> browsers.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jethrolarson/placeholder-augment/commit/b3e16b9168fd3e4301a421385f55e80da425a923#commitcomment-16173134>
> .
>


Very much so! I think we then could have a macro saying "implement op for all data addressing modes", so just one line per op instead of one per required addressing mode.


I think it has to do with the dependent libs... pgpy is crashing on me when i call it directly.
OSX no longer bundles openssl headers.  I'm using brew openssl, and to even install cryptography I have to `brew link --force openssl` or --no-use-wheel.  https://github.com/pyca/cryptography/issues/2138 is closed but I think because they doc'ed a workaround.


case MotionEvent.ACTION_MOVE: