Use of possessive quantifiers such as ?+ requires perl 5.10, thus breaking compatibility with perl 5.8.x:

```
$ make test
PERL_DL_NONLAZY=1 /usr/bin/perl "-MExtUtils::Command::MM" "-e" "test_harness(0, 'blib/lib', 'blib/arch')" t/*.t
t/basic............skipped
        all skipped: test parameters not provided in test.txt
t/body_string......
#   Failed test 'use Mail::IMAPClient;'
#   in t/body_string.t at line 14.
#     Tried to use 'Mail::IMAPClient'.
#     Error:  Nested quantifiers in regex; marked by <-- HERE in m/\G(?:"((?:\\?+ <-- HERE .)*?)"|([^()\s]+))\s*/ at /builddir/build/BUILD/Mail-IMAPClient-3.36/blib/lib/Mail/IMAPClient.pm line 2201.
# Compilation failed in require at (eval 6) line 2.
# BEGIN failed--compilation aborted at t/body_string.t line 14.
# Looks like you planned 3 tests but only ran 1.
# Looks like you failed 1 test of 1 run.
dubious
        Test returned status 1 (wstat 256, 0x100)
DIED. FAILED tests 1-3
        Failed 3/3 tests, 0.00% okay
```


remove the `-`


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 383](http://192.168.30.225:8111/viewLog.html?buildId=2041&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:11:52


Actually the whole `foreach` loop becomes unnecessary, because one could just do
```C++
BtConstModuleList unindexedModules(CSwordModuleSearch::unindexedModules(modules()));
```


Also, servald "status" will show you the actual http port;
https://github.com/servalproject/serval-dna/blob/development/server.c#L920

As does "start";
https://github.com/servalproject/serval-dna/blob/development/server.c#L820

On Fri, Aug 14, 2015 at 7:08 PM, Jeremy Lakeman <Jeremy.Lakeman@gmail.com>
wrote:

> I mean that line there is only for backwards compatibility for nodes that
> don't send the (now ancient) HAS_PORT flag. Which we always send;
>
> https://github.com/servalproject/serval-dna/blob/0a061e0fe8b63882eb5fda028def23a91aed75ed/rhizome_packetformats.c#L159
>
> Any nodes that doesn't send it, will by definition, not support the new
> config value and will implicitly be using HTTPD_PORT as their web server.
> So using the config value there will actually break backwards compatibility.
>
>
>
> On Fri, Aug 14, 2015 at 5:46 PM, gardners <notifications@github.com>
> wrote:
>
>> Hello,
>>
>> I made this adjustable so that I could easily run multiple instances on a
>> single machine for working on the multiple radio rhizome synchronisation
>> work, and know for sure which instance was being talked to by which
>> Rhizome
>> HTTP API client.
>>
>> I'm open to a better solution.
>>
>> Paul.
>>
>> On Fri, Aug 14, 2015 at 3:03 PM, Jeremy Lakeman <notifications@github.com
>> >
>> wrote:
>>
>> > If the port can be configured to anything by any client, this no longer
>> > makes any sense. Other old clients will be using HTTPD_PORT, as any
>> > configurable client will pass HAS_PORT below. But we should probably
>> > document when HAS_PORT was added if we want to break compatibility.
>> >
>> > —
>> > Reply to this email directly or view it on GitHub
>> > <
>> https://github.com/servalproject/serval-dna/commit/4dbd6f4695dfc606427249155dfdab3ffec41305#commitcomment-12704310
>> >
>> > .
>> >
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/servalproject/serval-dna/commit/4dbd6f4695dfc606427249155dfdab3ffec41305#commitcomment-12705763>
>> .
>>
>
>


Why remove the `const` here?


Indeed homebrew already manage this...
But I do not understand your previous error here https://github.com/tuttleofx/TuttleOFX/issues/442
Do you try to install the last tuttle formulae without the ```git submodule sync```?


Not sure how good this is. I'm not exposed to this kind of tasks so any help/recommendation is welcome.

cc/ @Kartones


Here a bug sneaked in: it should be x = ...


petla sprawdza od 0, a B[0] zawsze bedzie false, bo wartosci w A są wieksze niż 1


ef


Here we should do some research if multiply with carry can be replaced with shift/add. Several implementations of kiss RNGs can be found in the net


Thank you, reverted with commit ea3b984ec006ad57332dd03a547f7edf456d8c85


I understand what you mean. It could be a good improvement. I think we will clean this application, and after that we will move and generalize the decoding part in avTranscoder library :sunglasses:


Command name MigrateWithStorageGetDestPathsCommand  and MigrateWithStorageDestPathsCommand are too similar. Why not update MigrateWithStorageCommand object and use it instead of defining MigrateWithStorageDestPathsCommand.


Hello,

What version of PHP are you using? Im on 5.6 and it has no errors. I'll drop it to 5.4.x

Andrew


You are of course free to do any change you feel like in your fork of PropEr, but I am just wondering: what exactly is the purpose of this change?

The code snippets which appear within the NO_MODULES_IN_OPAQUES if-def macro are there only to be able to  build PropEr with Erlang/OTP releases where these types are built-in (and not module-qualified).


I think adding hub is great but not for replacing the git alias


在這邊就new timer感覺  不太好,應該放在onCreate比較恰當
if(!timer){
    timer=new Timer();
}


@tawan Könnte es sein, dass in [mandrill_api_key.rb](https://github.com/klappradla/im_graetzl/blob/master/config/initializers/mandrill_api_key.rb) ein Typo ist? Auf ElasticBeanstalk haben wir als ENV-Variable nur **MANDRILL_KEY**, nicht **MANDRILL_API_KEY**.

Würde zu dem Error passen den ich in den logs bekomme (missing mandrill api key, etc.)


Hi, I'm the author of this module.
FYI I've written a shorter version for the random function:

```js
function random() {
    var x = Math.sin(.8765111159592828 + seed++) * 1e4
    return x - Math.floor(x)
  }
```

I though you'll like it!


إجراء النقر على الزر


`return Survey.has_rows`


الوحدات المستخدمة في الفورم


العناصر الموضوعة على الفورم


Something going wrong. [Site](http://dlang.org/phobos/std_experimental_logger.html) renders items in this table twice.


This commit looks great at a glance, thanks!


Yep agreed with that ;-)


Hi, I'm the author of this module.
FYI I've written a shorter version for the random function:

```js
function random() {
    var x = Math.sin(.8765111159592828 + seed++) * 1e4
    return x - Math.floor(x)
  }
```

I though you'll like to know it!


Yes but there are other existing methods in the Python API that already use `std::vector<>*` as a return type and there python addons rely on the fact that the result is always a vector/list (even if empty) and not `None` so this change will break the API and therefore addons using these calls.

There are other ways to get `Dialog::multiselect()` to return a vector or `None` (e.g. using `Alternative<>`).


Nice, thanks for this :)


Commit message: workaread vs workaread vs workarea


Why does this use copies? It is assumed that item stacks are mutable, I'm pretty sure @Mumfrey said something like that in regards to the inventory API.


Creature linking to despawn on death / evade. Does that work?


Don't you have the same problem with qt libraries?
Anyway, this ```qt4_include``` variable can be deduce automatically from ```QTDIR``` build parameter (like sndfile and boost libraries). And the path should be create by ```os.path.join``` to keep portability.


Here you are @newlawrence :smile:


Can you add translations to rr strings  instead of cm ones . better to do not touch cm strings


You forgot to switch this to use the new `run_cmd` :-)  Also please can you change the commit message to give an example of a gerrit reference (`refs/changes/61/261/3` will do just fine) to make it clearer why they are not fetched by the original clone.


I'd use `Carto::User` ActiveRecord new model, is lighter and allows `in_database` too


update fireball-x/fireball#211


Looking good. Thank you.


The regexp compile invocations here and below look like something that could be a bit time consuming (cpu). Can't this be done once and perhaps stored in a static field?


Why not filter on `key == future` already here? It would save a long iteration later, through all collected stack frames.


Because GitHub pull requests don't fast forward when merged. If you manually force it to, it doesn't close the pull request.


can we have old submissions referencing a deleted questions? that would fail here


something astray with indentation here?


actually, forget about it, leave this one as I would call just before looping a `user.reset_pooled_connections` because I don't trust Ruby 1.9.3 nor the ruby pooler.

Try first at a staging but that should not do any harm as only drops the user connections.


I used underscore because that's the name of the profile (monotouch_watch), and because underscores were used before (--with_profile4_5, for instance).

But I don't care much, both work fine for me.


Can't this be simplified by being a normal local function? `def findMessageOrdinal(elem: IVariable)` = ...?


I can`t see this fix the issue. I even tried new clean install but still the same!


Please don't do that; it just accumulates unused code which reduces maintainability.  The Rails app already has a locking mechanism so I don't see why we would need another one in the future.  And it's already beginning to unnecessarily cost us time by forcing discussions about how to share code between the two subclasses.


Awesome! Your boilerplate is really helpful, thank you.


The formula without this line does work now. I cleaned my git cache so i cannot reproduce the error anymore. Lets keep it like this for the moment.


Be advised that I've just pushed `atom-linter` support for both streams


don't you want to call it?


Why do we need to have run dialyzer for test and examples folders?


I don't understand the logic in this whole function below. Initially I thought it would be enough to modify `stackFrames` to use a pair as a key, `(ObjectReference, Int)` (maybe `Long` would be safer for long-running applications). Why do you need to walk *all* stack traces to locate the right one, while before it was a simple lookup in the map?


convert_to_unicode (unless it's a bool/number)


I guess here I should use Data::Transpose::Validator


Hm, weird, but okay. Just be sure, or whoever else is doing the backports, that it's 30% in vanilla.


won't work once they are removed (check `.__class__.__name__`)


I don't think it's 10.10 and iOS 8, but rather 10.9 and iOS 7.


Oh cher...


I think you miscalculated your values here. For example, mbonus should be: `3,5,13,27,47,73,105`.


RadianceRefactor (rpg777) - Win64-Windows-7-VisualStudio-12: OK (2190 of 2215 tests passed)

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


Heads up, this is already a directory in the LabAnalysisCodes repo: https://github.com/leslielab/LabAnalysisCodes/tree/master/DNA%20in%20pits

In order to work on it from there, you can click on the button towards the top of any page in the LabAnalysisCodes repo which reads "Branch: master", type in a branch name, and click on Create Branch button which shows up automagically; you will then be able to commit to that branch freely while ensuring that everything on master still runs smoothly.


@ropilz Perfect, thanks!


cc @sameersbn :)


:fearful:


If you look at line @597, we access the values only when rr is true. As rr = (relative_rank(Us, s) - RANK_2) * ((relative_rank(Us, s) - RANK_2) - 1), we acces them only when the relative rank is at RANK_4.

2 * (2 - 1) - 1 = 1 > 0, then true from now on.

However, thanks for checking my code! I know that I often make mistakes.


Test looks right to me.  Thanks for including the negative test on line 54.


can you squash this so there aren't failing commits in the history?


@atotic sure that will be very helpful!


sysobj.r seems to be where args: is started out to `none`, both in the script context and the options context.  Looks like switching that to just `[]` would take care of the NONE! arg'd cases.


Also, you do realize that the G4 is 8992 not 8994, right?


需要合并到MASTER分支。


It is missing the cleanup
delete the account and opportunity


Also, if you look at the test signature, it is the same as the last master.


Ooph, that document could be written a lot better. The doc tries to communicate two different things:

  1. The first two snippets are plausibly complete code (notwithstanding the ellipses). They show all major steps and symbols (class names, object references, properties, methods).
  2. The next six snippets are not executable code -- they list the names of various properties/methods in an abstract shorthand. Implicitly, you need to get a reference to the actual `$response` object (as demonstrated in the initial snippets).

Trivially, their example doesn't work here because we don't have a reference to `$response`. To use `$response`, we'd need to lookup a reference somewhere (maybe in the container) or buffer the headers (eg in a static `CRM_Utils_System_Drupal8::$bufferedHeaders`) until `$response` becomes available at some future point.

However, there's a deeper disconnect. The `header()` function is fundamentally intended to output immediately. It's based on the assumption that each PHP process handles exactly one request+response. Most of the places where Civi uses it are like:

```php
header('Foo: bar');
echo $data;
exit();
```

It's important the header be sent over the wire at exactly that moment - if it's after `echo`, then it's too late to output headers. If it's after `exit`, then it will never run!

The Symfony architecture (with `HttpKernel`/`Request`/`Response`) discards this idea of *exactly* one request+response. (You might, for example, run sub-requests, unit-tests, or daemonized queue-workers.) To make this work, your application logic stashes all its output in a `$response` object... but it doesn't know or care about the exact time at which the response is sent. The decision to send will be made by the framework (eg D8's HttpKernel).

These two patterns -- `header()+echo()+exit()` vs `HttpKernel+Request+Response` -- do not intermingle freely. It would be counterproductive to stash headers in a `Response` while keeping the other elements the same.

So... +1 for removing this. Just use `header(...)` as in `CRM_Utils_System_Base`.

(To be clear... `HttpKernel` is a great pattern, and I'd be happier if Civi used it, but I think it's outside your intended scope.)


Did you try @FindBy(id ="username") ?
Review all your xpath locators


@topherfangio  Need to add documentation for this.


I would argue that if your ide is that out of sync with the command line build tools, you are doing it wrong. File locking and race conditions between the 2 tools is a valid concern though.


I am not sure if all your pages should extend of TopMenu class


If you can use by id or name,.....
Please apply it!


Ah yes. First two values of the arrays are not used because of `if (rr)` on line 597.


I think that the driver.findElement(By.id("calMonthPicker")) should be defined in the Page Factory


Same as above


do we have a better flag for checking whether units are installed?


You created a LookupPage class
Why aren't you using that class?


`gpkit.units` works, if you import gpkit


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 329](https://teamcity.labkey.org/viewLog.html?buildId=266599&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 30 (3 new), passed: 2 Build time: 00:11:15
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


Looks great. My only complaint (and not something that'll prevent a merge) is this is a bit of a catch-all commit.

Would be great to have broken it up into separate patches for the various parts. E.g. the .gitignore change should be in one patch etc.


It is not necessary to commit the .idea folder and binary files *.class


Please use the Singleton class in order to get the WebDriverWait instance
wait = WebDriverManager.getInstance().getWait();


Why do you need those?


Pointing out the obvious I suppose: this is where the rargs->do_arg would be tested, and if it's set then exclude the filename (if the script were to be excluded)...


Ok, thanks. Now reason is clear.
Recently got nice catch of ignoring such options: test in TeamCity that ensures TC understand OOM exception (with 'PermGen space' text) in java builds. Ran that on my linux dev machine: everything hanged (until kernel killed something) because of jre 8 and test application contains expected memory leak (on string#intern, or something similar).

So if plugin have memory leaks would they be found if tests run in jre 8?


Yeah, completely understand that.


It's in master: https://github.com/facebook/react-native/commit/7d19ff3dcb3f5e51cc8c3abe8f2638b2a394253c


should be #270


Should be #270 ?


[I attempted to write this comment before but I no longer see it.  Sorry if this is a duplicate.]  The tests make sense to me.  Thanks for including the negative test case on line 58.


Please create a Singleton class


In order to avoid having to do this again in the future, let's change the URL to this format:

    https://github.com/FortAwesome/Font-Awesome/archive/v4.4.0.zip

The github URLs should be more stable.


Sweet. https://github.com/facebook/react-native/blob/master/Libraries/Modal/Modal.js for the source


The Account and Opportunity class have similar methods 'save'
Please create an Abstract class in order to re-use code


[I attempted to write this comment before but I no longer see it.  Sorry if this is a duplicate.]  I don't think this test precisely lines up with the description of the commit: "This allows a TrustSet transaction to create a trust line if the _only_ _thing_ being changed is setting the `tfSetfAuth` flag.".  The way I read this is that _at_ _least_ the `tfSetfAuth` flag must be set.  It looks like other things may be changed as long as the `tfSetfAuth` flag is being set.  Is that your intent?  Or am I misreading the way the test works?


@GianlucaGuarini ,
with a clean clone of riotjs, running `npm install` triggers this warning:
```
The peer dependency phantomjs@>=1.9 included from karma-phantomjs-launcher will no
longer be automatically installed to fulfill the peerDependency
in npm 3+. Your application will need to depend on it explicitly.
optional dep failed, continuing fsevents@0.3.8
```
I think we must include phantomjs in dev dependencies.


typo: `one` should be `on`?


Please add docstrings to about how to use this class and the methods inside it.


Use `.ReadToEndAsync()` and add a `using` statement for the stream reader to dispose of it properly


@atotic I fixed the PR. Would you mind trying it?


@robertbaker What exactly would you like me to document? :-)


Prompt the user for a valid Bluetooth address and how to obtain it (e.g. `hcitool scan`)


TeamCity pull requests :: netty [Build 36](https://garage.netty.io/teamcity/viewLog.html?buildId=354&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Execution timeout (new); tests passed: 5076, ignored: 20 Build time: 00:45:09


@hostilefork here you go: 4e27b28e823e69dc3c7fb2e38fb94534fd65e2a4.


nupic imports before local imports, right? the second one is a local import, isn't it? https://github.com/numenta/nupic/wiki/Python-Style-Guide#import-grouping


Ugh.  What a pity workarounds like these are needed...


I misspelled "Routes" as "Roots" in the commit message!!!


Thx! your beer :beer:


You no longer need to call `roslib.load_manifest()` with catkin packages, please remove any line like this.


This closes #73


I think I found the right place to bring these things up:

https://plus.google.com/+JoeGibbsPolitz/posts/7xLaiAMUP8q

I'll see if any useful discussion happens there.


its part of #114  check the badges on:
https://github.com/locationtech/spatial4j/tree/code-coverage

(though they won't actually work until it is on master :(


I think, it's arbitrary nul-terminated strings. It's OK to expect something sane from the platform and the parent process.


rename to ca.crt


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


Yes, the problem is some PeerNode variants override the computation method, but don't do the status-table book-keeping.


In particular, this fixes Kalya's "Rain of Arrows" skill 102 defined on weapon.lua around line 294. When there was dead enemies, choosing the skill would sometimes fail, and sometimes it would select enemies that were already defeated. Note this would also prevent something like a "revive all parties" attack, if for some reason you wanted to select all the enemies so you could bring them back to life. Why would you do that? Zombie enemies could change sides for the duration of a battle. That might be helpful, unless a roll of the dice made them simply made them angry at your character and start taking vengeance for their own death.


Oh, and by the way, I added you to the lg-devs g4 team, so you now have full access to the G4 repos on there. Let's try to keep all the G4 AOSP development together, so it gets done faster and cleaner.


hh


I saw the same warning. :+1: for adding phantomjs to dev dependency.


*probable


@hostilefork I would do it in C, after the issues at hand are resolved.


or an instance method on workshop


nice stuff, i'm thinking of adding them to the reagent_containers with a nanobot deployer for use.


just curious, why is this better @alexpaluzzi || @robdodson


@@ -0,0 +1 @@


Just saw it today, sorry for taking so long ppl.

So, @bolshakov, on your example you simulated a best case scenario, but there are ppl using different serializers to serialize the same kind of `object`, in this case you would have different roots, which is kind of inconsistent, so after discussing with part o Rails team we decided to go for it.

We will still work on this in order to make it compatible with multiple conventions, it's already part of a [PR](https://github.com/rails-api/active_model_serializers/pull/1029).

I'm not saying we will keep it this way, we are always open to discuss it :smile:


Straight-forward.


Looks good.


Changing GetAccountInfo to AccountInfo to conform to the document "Effective Go" (https://golang.org/doc/effective_go.html). Also, proposed to add the Email field to the Account struct in case an app needs to communicate with it's users.


Code is reviewed OK. Tested with 2.8.7 and 2.9.


Thanks! Was about to do the same. :)


Well seen!


Was this the subtle issue plaguing mode?


For CentOS 6.x it would be helpful to add the kernel and kernel-dev packages, dscussion is in https://github.com/dotless-de/vagrant-vbguest/issues/141

This would avoid the workaround I need today to unblock vagrant-vbguest:
vagrant up ; vagrant ssh --command "sudo yum -y install kernel-devel kernel" ; vagrant reload --provision


PlantOperationSchemes (kbenne) - i386-Windows-7-VisualStudio-12: OK (2209 of 2235 tests passed)

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


Niklas,

I really appreciate you providing CreatorBuilder for gen_srcs.py, you may think you've done a quick and dirty implementation, but your python skills are way beyond mine!! I'll make a cleanup pass after we get creator working well.

Another thing, what I'm looking for is not so much how long it takes for cmake, meson or creator to run the first time, but more importantly how long it takes to just do the build steps there after. In your case you're hiding ninja behind creator so your code is part of the build step, but that's not the case with cmake or meson. So what I'll be looking at is the performance after running creator the first time and currently ```./gen_srcs.py XXX test 1000 1000``` is my base test. [Here](https://docs.google.com/a/saville.com/spreadsheets/d/1gbNIkvF3vmkvzdiFkLcWLj261urXWcK1la1ZiDYyvhE/edit?usp=sharing) is a spread sheet with the timings on my desktop.

I have run into a couple of problems one is to fix line 562 and the other appears to be an interaction between threads and dependencies, I don't know how to fix that. See details below.

With the fix for line 562 I could generate 1000 libs just fine:
```
$ rm -rf test
wink@desktop:~/prgs/test/gen_srcs (NiklasRosenstein-master)
$ ./gen_srcs.py creator test 1000 1
wink@desktop:~/prgs/test/gen_srcs (NiklasRosenstein-master)
$ cd test
wink@desktop:~/prgs/test/gen_srcs/test (NiklasRosenstein-master)
$ creator

[2002/2002] gcc -std=c99 /home/wink/prgs/test/gen_srcs/test/build/obj/testapp/main.o /home/wink/prgs/test/.../wink/prgs/test/gen_srcs/test/build/libs/libL999.a -o /home/wink/prgs/test/gen_srcs/test/build/bin/testapp
==> creator: exporting to: build.ninja==> creator: running: ninja -f build.ninjawink@desktop:~/prgs/test/gen_srcs/test (NiklasRosenstein-master)
```

But if I tried to generator 10 libraries it fails:
```
$ cd ..
wink@desktop:~/prgs/test/gen_srcs (NiklasRosenstein-master)
$ rm -rf test
wink@desktop:~/prgs/test/gen_srcs (NiklasRosenstein-master)
$ ./gen_srcs.py creator test 10 1
wink@desktop:~/prgs/test/gen_srcs (NiklasRosenstein-master)
$ cd test
wink@desktop:~/prgs/test/gen_srcs/test (NiklasRosenstein-master)
$ creator

[12/22] gcc -std=c99 /home/wink/prgs/test/gen_srcs/test/build/obj/testapp/main.o /home/wink/prgs/test/gen_.../wink/prgs/test/gen_srcs/test/build/libs/libL009.a -o /home/wink/prgs/test/gen_srcs/test/build/bin/testapp
FAILED: gcc -std=c99 /home/wink/prgs/test/gen_srcs/test/build/obj/testapp/main.o /home/wink/prgs/test/gen_srcs/test/build/libs/libL000.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL001.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL002.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL003.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL004.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL005.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL006.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL007.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL008.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL009.a -o /home/wink/prgs/test/gen_srcs/test/build/bin/testapp
gcc: error: /home/wink/prgs/test/gen_srcs/test/build/libs/libL000.a: No such file or directory
gcc: error: /home/wink/prgs/test/gen_srcs/test/build/libs/libL001.a: No such file or directory
gcc: error: /home/wink/prgs/test/gen_srcs/test/build/libs/libL002.a: No such file or directory
gcc: error: /home/wink/prgs/test/gen_srcs/test/build/libs/libL003.a: No such file or directory
gcc: error: /home/wink/prgs/test/gen_srcs/test/build/libs/libL004.a: No such file or directory
gcc: error: /home/wink/prgs/test/gen_srcs/test/build/libs/libL005.a: No such file or directory
gcc: error: /home/wink/prgs/test/gen_srcs/test/build/libs/libL006.a: No such file or directory
gcc: error: /home/wink/prgs/test/gen_srcs/test/build/libs/libL007.a: No such file or directory
gcc: error: /home/wink/prgs/test/gen_srcs/test/build/libs/libL008.a: No such file or directory
gcc: error: /home/wink/prgs/test/gen_srcs/test/build/libs/libL009.a: No such file or directory
[12/22] gcc -c -std=c99 -I/home/wink/prgs/test/gen_srcs/test/libs/L002/include -o /home/wink/prgs/test/gen_srcs/test/build/obj/L002/L002.o /home/wink/prgs/test/gen_srcs/test/libs/L002/src/L002.c
ninja: build stopped: subcommand failed.
==> creator: exporting to: build.ninja==> creator: running: ninja -f build.ninjawink@desktop:~/prgs/test/gen_srcs/test (NiklasRosenstein-master)
````

The problem appears to be that the dependencies aren't defined strictly enough to allow ninja to use multiple threads. I say this because if I execute 'ninja -j1' it works:
```
$ rm -rf build
wink@desktop:~/prgs/test/gen_srcs/test (NiklasRosenstein-master)
$ ninja -j1
[22/22] gcc -std=c99 /home/wink/prgs/test/gen_srcs/test/build/obj/testapp/main.o /home/wink/prgs/test/gen_.../wink/prgs/test/gen_srcs/test/build/libs/libL009.a -o /home/wink/prgs/test/gen_srcs/test/build/bin/testapp
```

Then if I execute 'ninja -v -j14' I see the failure that libL001.a, libL003.a and libL004.a haven't been created yet. And very oddly step [21/22] is the link step for testapp while step[22/22] is creating archive for libL003.a, this definitely feels wrong.
```
$ rm -rf build
wink@desktop:~/prgs/test/gen_srcs/test (NiklasRosenstein-master)
$ ninja -v -j14
[1/22] gcc -c -std=c99 -I/home/wink/prgs/test/gen_srcs/test/libs/L000/include -o /home/wink/prgs/test/gen_srcs/test/build/obj/L000/L000.o /home/wink/prgs/test/gen_srcs/test/libs/L000/src/L000.c
[2/22] gcc -c -std=c99 -I/home/wink/prgs/test/gen_srcs/test/libs/L001/include -o /home/wink/prgs/test/gen_srcs/test/build/obj/L001/L001.o /home/wink/prgs/test/gen_srcs/test/libs/L001/src/L001.c
[3/22] gcc -c -std=c99 -I/home/wink/prgs/test/gen_srcs/test/libs/L002/include -o /home/wink/prgs/test/gen_srcs/test/build/obj/L002/L002.o /home/wink/prgs/test/gen_srcs/test/libs/L002/src/L002.c
[4/22] gcc -c -std=c99 -I/home/wink/prgs/test/gen_srcs/test/libs/L003/include -o /home/wink/prgs/test/gen_srcs/test/build/obj/L003/L003.o /home/wink/prgs/test/gen_srcs/test/libs/L003/src/L003.c
[5/22] gcc -c -std=c99 -I/home/wink/prgs/test/gen_srcs/test/libs/L004/include -o /home/wink/prgs/test/gen_srcs/test/build/obj/L004/L004.o /home/wink/prgs/test/gen_srcs/test/libs/L004/src/L004.c
[6/22] gcc -c -std=c99 -I/home/wink/prgs/test/gen_srcs/test/libs/L005/include -o /home/wink/prgs/test/gen_srcs/test/build/obj/L005/L005.o /home/wink/prgs/test/gen_srcs/test/libs/L005/src/L005.c
[7/22] gcc -c -std=c99 -I/home/wink/prgs/test/gen_srcs/test/libs/L006/include -o /home/wink/prgs/test/gen_srcs/test/build/obj/L006/L006.o /home/wink/prgs/test/gen_srcs/test/libs/L006/src/L006.c
[8/22] gcc -c -std=c99 -I/home/wink/prgs/test/gen_srcs/test/libs/L007/include -o /home/wink/prgs/test/gen_srcs/test/build/obj/L007/L007.o /home/wink/prgs/test/gen_srcs/test/libs/L007/src/L007.c
[9/22] gcc -c -std=c99 -I/home/wink/prgs/test/gen_srcs/test/libs/L008/include -o /home/wink/prgs/test/gen_srcs/test/build/obj/L008/L008.o /home/wink/prgs/test/gen_srcs/test/libs/L008/src/L008.c
[10/22] gcc -c -std=c99 -I/home/wink/prgs/test/gen_srcs/test/libs/L009/include -o /home/wink/prgs/test/gen_srcs/test/build/obj/L009/L009.o /home/wink/prgs/test/gen_srcs/test/libs/L009/src/L009.c
[11/22] gcc -c -I/home/wink/prgs/test/gen_srcs/test/apps/testapp/include -I/home/wink/prgs/test/gen_srcs/test/libs/L000/include -I/home/wink/prgs/test/gen_srcs/test/libs/L001/include -I/home/wink/prgs/test/gen_srcs/test/libs/L002/include -I/home/wink/prgs/test/gen_srcs/test/libs/L003/include -I/home/wink/prgs/test/gen_srcs/test/libs/L004/include -I/home/wink/prgs/test/gen_srcs/test/libs/L005/include -I/home/wink/prgs/test/gen_srcs/test/libs/L006/include -I/home/wink/prgs/test/gen_srcs/test/libs/L007/include -I/home/wink/prgs/test/gen_srcs/test/libs/L008/include -I/home/wink/prgs/test/gen_srcs/test/libs/L009/include -std=c99 /home/wink/prgs/test/gen_srcs/test/apps/testapp/src/main.c -o /home/wink/prgs/test/gen_srcs/test/build/obj/testapp/main.o
[12/22] ar rcs /home/wink/prgs/test/gen_srcs/test/build/libs/libL009.a /home/wink/prgs/test/gen_srcs/test/build/obj/L009/L009.o
[13/22] ar rcs /home/wink/prgs/test/gen_srcs/test/build/libs/libL005.a /home/wink/prgs/test/gen_srcs/test/build/obj/L005/L005.o
[14/22] ar rcs /home/wink/prgs/test/gen_srcs/test/build/libs/libL002.a /home/wink/prgs/test/gen_srcs/test/build/obj/L002/L002.o
[15/22] ar rcs /home/wink/prgs/test/gen_srcs/test/build/libs/libL006.a /home/wink/prgs/test/gen_srcs/test/build/obj/L006/L006.o
[16/22] ar rcs /home/wink/prgs/test/gen_srcs/test/build/libs/libL007.a /home/wink/prgs/test/gen_srcs/test/build/obj/L007/L007.o
[17/22] ar rcs /home/wink/prgs/test/gen_srcs/test/build/libs/libL000.a /home/wink/prgs/test/gen_srcs/test/build/obj/L000/L000.o
[18/22] ar rcs /home/wink/prgs/test/gen_srcs/test/build/libs/libL001.a /home/wink/prgs/test/gen_srcs/test/build/obj/L001/L001.o
[19/22] ar rcs /home/wink/prgs/test/gen_srcs/test/build/libs/libL004.a /home/wink/prgs/test/gen_srcs/test/build/obj/L004/L004.o
[20/22] ar rcs /home/wink/prgs/test/gen_srcs/test/build/libs/libL008.a /home/wink/prgs/test/gen_srcs/test/build/obj/L008/L008.o
[21/22] gcc -std=c99 /home/wink/prgs/test/gen_srcs/test/build/obj/testapp/main.o /home/wink/prgs/test/gen_srcs/test/build/libs/libL000.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL001.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL002.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL003.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL004.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL005.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL006.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL007.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL008.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL009.a -o /home/wink/prgs/test/gen_srcs/test/build/bin/testapp
[22/22] ar rcs /home/wink/prgs/test/gen_srcs/test/build/libs/libL003.a /home/wink/prgs/test/gen_srcs/test/build/obj/L003/L003.o
FAILED: gcc -std=c99 /home/wink/prgs/test/gen_srcs/test/build/obj/testapp/main.o /home/wink/prgs/test/gen_srcs/test/build/libs/libL000.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL001.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL002.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL003.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL004.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL005.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL006.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL007.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL008.a /home/wink/prgs/test/gen_srcs/test/build/libs/libL009.a -o /home/wink/prgs/test/gen_srcs/test/build/bin/testapp
gcc: error: /home/wink/prgs/test/gen_srcs/test/build/libs/libL001.a: No such file or directory
gcc: error: /home/wink/prgs/test/gen_srcs/test/build/libs/libL003.a: No such file or directory
gcc: error: /home/wink/prgs/test/gen_srcs/test/build/libs/libL004.a: No such file or directory
ninja: build stopped: subcommand failed.
```


oh my.. yes, they should be the same, my mistake. I made a couples tests with different value... and probably forgot..

I will make a news push request with that fixed and will see to make an -OR- for #1.


TODO/BUGS:
-remove images from repo? still no license
-refine randomized x&y positions of bubbles
---then remove draggable layers. BUG: drag a bubble, then expand, then condense. the bubble returns to orig position, not new dragged position
-handle longer posts. BUGS: width/height of initial bubble and textbox, misplaced text elements when expanded
---impose a char limit on posts?


> Note this would also prevent something like a "revive all parties" attack, if for some reason you wanted to select all the enemies so you could bring them back to life. Why would you do that?
Zombie enemies could change sides for the duration of a battle. That might be helpful, unless a roll of the dice made them very angry at your character and start taking vengeance for their own death.

Hmm, such a case could be handled by switching the battle actor type, anyway. Thanks for the fix!


TurnOffWarmupReportForRevDD (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1274 of 1274 tests passed)

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


UpdateCIsettings (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (0 of 0 tests passed)

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


UpdateCIsettings (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


UpdateCIsettings (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Tests Failed

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


UpdateCIsettings (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed

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


The default `Inherited` value is `true`. So this attribute remains inconsistent with `TargetElementAttribute`.


Yeah, this fixes Mode :smile:


TeamCity pull requests :: netty [Build 37](https://garage.netty.io/teamcity/viewLog.html?buildId=365&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5554, ignored: 20 Build time: 00:37:18


Hmm. shouldn't line 52 be returning episodesWithDateFormat  rather than exceptions?


The commit message is intended to indicate that if there is no trust line, then in addition to all the other ways you can submit a `TrustSet` that legally creates the line, you can now submit a `TrustSet` that only sets the `tfSetAuth` flag on a non-existent trust line (but note, the destination account still has to exist).


The commit message is intended to indicate that if there is no trust line, then in addition to all the other ways you can submit a TrustSet that legally creates the line, you can now also submit a TrustSet that only sets the tfSetAuth flag on a non-existent trust line, and the line will be created (but note, the destination account still has to exist).


Hello @Brit200313 could it be that I never did a pull request for this? I've done some work on this same branch, which I was going push to this branch.


ok, matisanba


@kostya : https://github.com/manastech/crystal/issues/1179#issuecomment-131267190


TeamCity ChameleonForms :: Continuous Integration [Build 251](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3647&buildTypeId=bt20) is now running


Duplicate line entry in Markdown file. How do I remove it?


Causes a dump of 1 diff shares to the pool at connect.


ok esses sao meus arquivos de teste


Line 45 seem unneeded
var identifierToken = typeDecl.Identifier;


You're right about the political stuff. No reason to introduce that kind of thing, especially at this point...


Não seria steinbeckian?


thanks


Works for me!


fix_intel_compile_linux (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (749 of 749 tests passed)

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


TeamCity ChameleonForms :: Continuous Integration [Build 2.0.0-PullRequest123+018](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3647&buildTypeId=bt20) outcome was **FAILURE**
Summary: Tests failed: 8 (8 new), passed: 1149 Build time: 0:0:0
### Failed tests
```
ChameleonForms.Tests.dll: ChameleonForms.Tests.FieldGenerator.DefaultFieldGenerator.BooleanTests.Use_correct_html_for_label_for_boolean_list: Test(s) failed. ApprovalTests.Core.Exceptions.ApprovalMismatchException : Failed Approval: Received file c:\TeamCity\buildAgent\work\81e762d276ae250c\ChameleonForms.Tests\FieldGenerator\DefaultFieldGenerator\BooleanTests.Use_correct_html_for_label_for_boolean_list.received.html does not match approved file c:\TeamCity\buildAgent\work\81e762d276ae250c\ChameleonForms.Tests\FieldGenerator\DefaultFieldGenerator\BooleanTests.Use_correct_html_for_label_for_boolean_list.approved.html.
   at ApprovalTests.Approvers.FileApprover.Fail() in c:\code\ApprovalTests.Net\ApprovalTests\Approvers\FileApprover.cs:line 50
   at ApprovalTests.Core.Approvals.Verify(IApprovalApprover approver, IApprovalFailureReporter reporter) in c:\code\ApprovalTests.Net\ApprovalTests\Core\Approvals.cs:line 17

ChameleonForms.Tests.dll: ChameleonForms.Tests.FieldGenerator.DefaultFieldGenerator.BooleanTests.Use_correct_html_for_label_for_boolean_list_with_overridden_label: Test(s) failed. ApprovalTests.Core.Exceptions.ApprovalMismatchException : Failed Approval: Received file c:\TeamCity\buildAgent\work\81e762d276ae250c\ChameleonForms.Tests\FieldGenerator\DefaultFieldGenerator\BooleanTests.Use_correct_html_for_label_for_boolean_list_with_overridden_label.received.html does not match approved file c:\TeamCity\buildAgent\work\81e762d276ae250c\ChameleonForms.Tests\FieldGenerator\DefaultFieldGenerator\BooleanTests.Use_correct_html_for_label_for_boolean_list_with_overridden_label.approved.html.
   at ApprovalTests.Approvers.FileApprover.Fail() in c:\code\ApprovalTests.Net\ApprovalTests\Approvers\FileApprover.cs:line 50
   at ApprovalTests.Core.Approvals.Verify(IApprovalApprover approver, IApprovalFailureReporter reporter) in c:\code\ApprovalTests.Net\ApprovalTests\Core\Approvals.cs:line 17

ChameleonForms.Tests.dll: ChameleonForms.Tests.FieldGenerator.DefaultFieldGenerator.EnumTests.Use_correct_html_for_label_for_enum_list: Test(s) failed. ApprovalTests.Core.Exceptions.ApprovalMismatchException : Failed Approval: Received file c:\TeamCity\buildAgent\work\81e762d276ae250c\ChameleonForms.Tests\FieldGenerator\DefaultFieldGenerator\EnumTests.Use_correct_html_for_label_for_enum_list.received.html does not match approved file c:\TeamCity\buildAgent\work\81e762d276ae250c\ChameleonForms.Tests\FieldGenerator\DefaultFieldGenerator\EnumTests.Use_correct_html_for_label_for_enum_list.approved.html.
   at ApprovalTests.Approvers.FileApprover.Fail() in c:\code\ApprovalTests.Net\ApprovalTests\Approvers\FileApprover.cs:line 50
   at ApprovalTests.Core.Approvals.Verify(IApprovalApprover approver, IApprovalFailureReporter reporter) in c:\code\ApprovalTests.Net\ApprovalTests\Core\Approvals.cs:line 17

ChameleonForms.Tests.dll: ChameleonForms.Tests.FieldGenerator.DefaultFieldGenerator.EnumTests.Use_correct_html_for_label_for_enum_list_with_overridden_label: Test(s) failed. ApprovalTests.Core.Exceptions.ApprovalMismatchException : Failed Approval: Received file c:\TeamCity\buildAgent\work\81e762d276ae250c\ChameleonForms.Tests\FieldGenerator\DefaultFieldGenerator\EnumTests.Use_correct_html_for_label_for_enum_list_with_overridden_label.received.html does not match approved file c:\TeamCity\buildAgent\work\81e762d276ae250c\ChameleonForms.Tests\FieldGenerator\DefaultFieldGenerator\EnumTests.Use_correct_html_for_label_for_enum_list_with_overridden_label.approved.html.
   at ApprovalTests.Approvers.FileApprover.Fail() in c:\code\ApprovalTests.Net\ApprovalTests\Approvers\FileApprover.cs:line 50
   at ApprovalTests.Core.Approvals.Verify(IApprovalApprover approver, IApprovalFailureReporter reporter) in c:\code\ApprovalTests.Net\ApprovalTests\Core\Approvals.cs:line 17

ChameleonForms.Tests.dll: ChameleonForms.Tests.FieldGenerator.DefaultFieldGenerator.FieldGeneratorTests.Use_label_text_when_without_label: Test(s) failed.   Expected string length 14 but was 0. Strings differ at index 0.
  Expected: "RequiredString"
  But was:  <string.Empty>

ChameleonForms.Tests.dll: ChameleonForms.Tests.FieldGenerator.DefaultFieldGenerator.ListTests.Use_correct_html_for_label_for_list_as_list: Test(s) failed. ApprovalTests.Core.Exceptions.ApprovalMismatchException : Failed Approval: Received file c:\TeamCity\buildAgent\work\81e762d276ae250c\ChameleonForms.Tests\FieldGenerator\DefaultFieldGenerator\ListTests.Use_correct_html_for_label_for_list_as_list.received.html does not match approved file c:\TeamCity\buildAgent\work\81e762d276ae250c\ChameleonForms.Tests\FieldGenerator\DefaultFieldGenerator\ListTests.Use_correct_html_for_label_for_list_as_list.approved.html.
   at ApprovalTests.Approvers.FileApprover.Fail() in c:\code\ApprovalTests.Net\ApprovalTests\Approvers\FileApprover.cs:line 50
   at ApprovalTests.Core.Approvals.Verify(IApprovalApprover approver, IApprovalFailureReporter reporter) in c:\code\ApprovalTests.Net\ApprovalTests\Core\Approvals.cs:line 17

ChameleonForms.Tests.dll: ChameleonForms.Tests.FieldGenerator.DefaultFieldGenerator.ListTests.Use_correct_html_for_label_for_list_as_list_with_overridden_label: Test(s) failed. ApprovalTests.Core.Exceptions.ApprovalMismatchException : Failed Approval: Received file c:\TeamCity\buildAgent\work\81e762d276ae250c\ChameleonForms.Tests\FieldGenerator\DefaultFieldGenerator\ListTests.Use_correct_html_for_label_for_list_as_list_with_overridden_label.received.html does not match approved file c:\TeamCity\buildAgent\work\81e762d276ae250c\ChameleonForms.Tests\FieldGenerator\DefaultFieldGenerator\ListTests.Use_correct_html_for_label_for_list_as_list_with_overridden_label.approved.html.
   at ApprovalTests.Approvers.FileApprover.Fail() in c:\code\ApprovalTests.Net\ApprovalTests\Approvers\FileApprover.cs:line 50
   at ApprovalTests.Core.Approvals.Verify(IApprovalApprover approver, IApprovalFailureReporter reporter) in c:\code\ApprovalTests.Net\ApprovalTests\Core\Approvals.cs:line 17

ChameleonForms.AcceptanceTests.dll: ChameleonForms.AcceptanceTests.ModelBinding.ModelBindingShould.Function_correctly_with_checkbox_and_radio_lists: Test(s) failed. TestStack.Seleno.Configuration.Interceptors.SelenoReceivedException : {"errorMessage":"Unable to find element with link text 'Model Binding Example with lists'","request":{"headers":{"Accept":"application/json, image/png","Connection":"Close","Content-Length":"64","Content-Type":"application/json;charset=utf-8","Host":"localhost:52387"},"httpVersion":"1.1","method":"POST","post":"{\"using\":\"link text\",\"value\":\"Model Binding Example with lists\"}","url":"/element","urlParsed":{"anchor":"","query":"","file":"element","directory":"/","path":"/element","relative":"/element","port":"","host":"","password":"","user":"","userInfo":"","authority":"","protocol":"","source":"/element","queryKey":{},"chunks":["element"]},"urlOriginal":"/session/0063d360-42ee-11e5-afb9-3f90c35fbdd9/element"}}
  ----> OpenQA.Selenium.NoSuchElementException : {"errorMessage":"Unable to find element with link text 'Model Binding Example with lists'","request":{"headers":{"Accept":"application/json, image/png","Connection":"Close","Content-Length":"64","Content-Type":"application/json;charset=utf-8","Host":"localhost:52387"},"httpVersion":"1.1","method":"POST","post":"{\"using\":\"link text\",\"value\":\"Model Binding Example with lists\"}","url":"/element","urlParsed":{"anchor":"","query":"","file":"element","directory":"/","path":"/element","relative":"/element","port":"","host":"","password":"","user":"","userInfo":"","authority":"","protocol":"","source":"/element","queryKey":{},"chunks":["element"]},"urlOriginal":"/session/0063d360-42ee-11e5-afb9-3f90c35fbdd9/element"}}
   at TestStack.Seleno.Configuration.Interceptors.CameraProxyInterceptor.Intercept(IInvocation invocation) in c:\ConsoleBuildAgent\work\f035b8dd08e41a61\src\TestStack.Seleno\Configuration\Interceptors\CameraProxyInterceptor.cs:line 52

```


newline.


Tomorrow I'll do the missing changes.


UUID as ID in SQLiteDatabase is not good idea


97215920---Fill-Gaps-in-HVAC-Components---Initial-Code (rraustad) - Win64-Windows-7-VisualStudio-12: OK (1792 of 1796 tests passed)

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


Of course. My usage might be a bit different from what you're thinking though. I end up doing an alloc-> copy to buffer->upload to gpu->free cyce so often that I even considered that a free list was overkill (planning to move to a stack allocator as soon as the allocator turns up in the profiler :D ).

For having fragmentation issues, I'd need varied alloc/dealloc patterns, which I don't have luckily. That's right now of course, I'll move to jemalloc eventually.

Also, now that you're here, I had hoped to come up with a way to do some kind of try-with-resources with the allocator, other than duck typing, the only way I could do it was with a wrapper object implementing Closeable like:

 ```java
BufferAllocator ba = new BufferAllocator(16); // 16 Mb or something.

try ( AutoBuffer tmp = ba.autoAlloc(64) ) // 64 bytes or something.
{
  ByteBuffer buffer = tmp.buffer;
  // Now do stuff with the buffer.
}
```
AutoBuffer then calls 'BufferAllocator.this.free(buffer)' on its 'close' implementation. The issue I have is that it seems that HotSpot cant do escape analysis on the thing, so it turns up on the memory profiler :disappointed: There is no way to hijack the allocateDirect from my side to provide a ByteBuffer that implements Closeable.

Any idea about that? Maybe its too much of a "high level" functionality to use with this kind of stuff though.

Having said this, I'm reeeally interested in the stack allocator you mentioned in the JGO thread :smile:


Add K-Family Picnic, Key Leader, Update Service Partners, Update CMRF, Add a few Governor's Blog posts


Hi Hanno,

I’m messing around with different ways of setting the pointers to see if I can gain back the 5%.  Just testing for now,

Dan

> On Aug 14, 2015, at 2:27 PM, Hanno Rein <notifications@github.com> wrote:
>
> Why do you need those?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/dtamayo/rebound/commit/13cf42f4b1764fd6c625f2ff5d1aae04bfb00ffe#commitcomment-12721472>.
>


97215920---Fill-Gaps-in-HVAC-Components---Initial-Code (rraustad) - i386-Windows-7-VisualStudio-12: OK (1792 of 1796 tests passed)

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


It's explained at the end of this post. Small perf improvement.
https://blog.polymer-project.org/announcements/2015/08/13/1.1-release/
On Aug 14, 2015 5:54 PM, "Sam Saccone" <notifications@github.com> wrote:

> just curious, why is this better @alexpaluzzi
> <https://github.com/alexpaluzzi> || @robdodson
> <https://github.com/robdodson>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/yeoman/generator-polymer/commit/0f6381326055f35aedb4f3214dcaa882a369bf95#commitcomment-12721605>
> .
>


@boris-p are you sure you're sending the latest version? GPL license was fine. I'm not sure why you revert back to CC license?


Here is the same! Current version is 0.0.60!


fyi @jackparmer @BRONSOLO @cldougl


@BRONSOLO - I'm leaving this commented out until I know what the `convertDate` function is


@BRONSOLO - is this dateformat supposed to match the dates above, i.e. `mm-dd-yyyy`?


101235614-Fix-AFN-Error-Output (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1797 of 1797 tests passed)

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


It [uses](https://docs.oracle.com/javase/8/docs/api/java/nio/file/Files.html#readAllLines-java.nio.file.Path-) UTF-8.


this commit could use a code review


91958608DOASEffectOnZoneSizing (wfbuhl) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


this is a line comment


OK, I've filed an issue:
https://github.com/Homebrew/homebrew/issues/42967

`-s` only removes caches of deleted packages by the manual.


That method is only overridden by `DarknetPeerNode`, and it calls the method from `PeerNode`. Unfortunately both those methods are so hopelessly complex that I couldn’t figure out what they do within 10 minutes. :/


test comment


Oh. I confused both offset hacks.


test comment asd


Cool, good job!


91958608DOASEffectOnZoneSizing (wfbuhl) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-76.17%25-red.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-0.0%25-red.svg)


Не работает подсветка. На основании каких исходников генеришь `highlight.pack.js` ?


spencer


@yui-knk in Spanish or Catalan that comma is not used, maybe you are familiar with a language that also does not and that's why you find it more natural. But the [Oxford command](https://en.wikipedia.org/wiki/Serial_comma) is common in English style guides and it's the preferred option for the Rails documentation.

Maybe the guidelines should be explicit about it.


ciao, non so se bisogna commentare in questa pagina quindi chiedo scusa preventivamente.

Ascolta ho appena riscontrato che il plugin genera un errore nelle sitemap create con diversi plugin (ne ho usati parecchi per provare, dal più importante seo yoast al meno conosciuto)...considerato che siamo in tanti ad usare plugin automatici per le sitemaps potresti controllare e magari risolvere? grazie


You're right, I apologise.


It can be enabled via device tree. Read DT documentation and you will know.


Is this correct fix? I don't see "audit_access" in files_dontaudit_write_all_mountpoints()?


Thanks! The only comment — one should check the return value of readBytes().


@robertpustulka This is yet another BC breaker. The old event names have to be kept for now.


91958608DOASEffectOnZoneSizing (wfbuhl) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-44.87%25-red.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-0.0%25-red.svg)


That doesn't sound correct at all. `var` is just another form of `:=`.


So are we sure it is created by kadmind?


Should this be "chance to stun on hit"?


91977808_LBNL_NewVRFModel (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


:+1:   可以在commit 加上对应的issue id .  比如 ‘fix #46'  可以自动关掉对应issue.


Taunt would make more sense here over attract.


Refer to https://github.com/OmgImAlexis/Olivia/wiki/SSL to add a ssl cert for express.


@gavinwahl, do you use a system tray? I'm trying out stalonetray and the only way I could get it to not cover up the xmobar is by just adding padding the to xmobar. I don't know what to do.


This somewhat implies that it only affects larger enemies.  Something like "Does more hits the larger the enemy is." might be more descriptive.


Update Readme


@gavinwahl, this doesn't actually work. I have no idea why. Do you know how to get to the output or logs of spawn?


Should we not us the stable mappings?


Without item.ipfsHash we have lost track of the image, no? This is why we have delayed the syncEditAction()


Please change this word "Dafter" --> "Daftar" :)


Thanks Isis!

This code doesn't actually block anything, just marks it as being lower priority than non-Tor traffic. It should never do anything unless there's an active DoS attack via Tor. So perfect accuracy isn't really needed here: Tor access still works fine and will do even if you run a Bitcoin node and Tor node on the same machine.

That said, I'll make a mental note to switch to the second URL when I work on this code again (might be soon, given the ongoing DoS attacks via Tor we're seeing).


I think there's a .gitmodules file missing here


This should either be Targe ( type of shield ) Smash or just Shield Smash.


inflicted -> inflicting


Maybe this is too short description. Extend to more informative.


Fixes issue #17


@rohitpaulk Why am I not able to do, `@user.following.first.user_items << @app1` ?


Thanks - that rolls out fine now.


:+1: :100:


resistence -> resistance


Thank you. Fixed by b6f199099ffc2f51858daebacc0e6b4097affd1b


Although PHPUnit fails, I needed CodeClimate to update. It is updating coverage only with *bigger* changes like adding new file.

Of couse, it is a good point, I may remove this file.


I will in a few days after my last exam


Created #12592 as a test for this


infront -> in front


@MacDada Thanks, didn't notice that!


Hmm, not sure. I guess ActiveRecord doesn't define a convienience method for polymorphic associations?


@MacDada Refactoring tool fart... I refactored Handle interface to DataHandle to be more self-explanatory... Didn't notice that IntelliJ has changed the name of the field itself too.


Should just say "Immolate burning enemies."


You renamed that table and didn't remove it from the schema.

The other table I just moved in alphabetical other like every other table in that file.


yes, i wrote it commented because i wasnt sure whether creating the `item.ipfsHash` property would break anything else. wanted to illustrate it. you can feed the hash (or a path) to get:

```js
ipfs.get(item.ipfsHash, function(err, data) {

})
// or
var path = "/ipfs/" + item.ipfsHash
ipfs.get(path, function(err, data) {

})
```


patched ok. thanks


@MacDada Unfortunately, while using JavaFX's FXML loader and defining all controls in *.fxml file, one can't pass any constructor arguments, hence the need to use setters. This issue, as far as I know, has been raised, and there is possibility that in the forthcoming JavaFX release it'll be added.


Good clean up @Haroenv.

I'm just curious about `…` what's wrong with this unicode ?


caldrops -> caltrops


I think openni2_launch is for Kinect. And this code is for Xtion.
This difference doesn't work your roslaunch.

Actually kalectro changed the original code to Xtion version.
You can see the detail here. (https://github.com/MalcolmMielle/openni2_camera/commit/2d07c8ef03a277f5a3b2be657f3c875b9849b1da)

In addition, roslaunch is group of ROS's nodes.
So roslaunch code will be hint for your work.

I wish this will be help for you.
GOOD LUCK!


Nothing, that's exactly why I picked it. But maybe it should be changed back to three dots, because it seems like that font doesn't have that glyph.


`AndroidApp` model doesn't have a field `item_type` specifying that it's an `AndroidApp`(which would be redundant anyway) but I guess that is what `UserItems` expects for this to work.


patched ok. thanks @jahntheman


Np ! I was wondering.


91977808_LBNL_NewVRFModel (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1799 of 1803 tests passed)

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


Could just say "stay hidden from enemies"


Can we please fix up the wording then in BIP68?  There is inconsistent use of `under` and `above` such that this is _not_ clear.


I was concerned with the children that could've been left around.


"I shows" -> "Shows"


To quote:
```
// Sequence numbers under SEQUENCE_THRESHOLD are not consensus
// constrained.
```

and

```
 // Sequence numbers equal to or above the SEQUENCE_THRESHOLD
            // are not treated as relative lock-times, nor are they given
            // any consensus-enforced meaning at this point.
```

These are entirely contradictory.
In both cases, the codes conditional operator is `>=`,  yet the comments are different.


Now, what is this? :-(


Could still say "causes bleeding effect."  'bleeds the enemy' is a bit strange.


Thanks for reporting


Blame Visual Studio :)


Можно вместо отдельного блока `.controls:hover {...}` написать **внутри блока** `.controls {...}`:
```css
...
cursor: pointer;
&:hover {
    background-color: @base_controls;
}
```
Результат тот же абсолютно, но логическая группировка уже другая.


Great. Thanks :)


Mostly, using 'px' units is not right.


I'll fix it at merge time, but this should be in the 4.03.0 changelog, not 4.02.3 which is already released.


Remove under.


"if they touching" -> "if they touch"


@khoaakt You wanted this one, right ? ;) Give it a test!


"increasing" -> "increase"


Before `:` there must be no spaces.


Important: Bug is now fixed in latest git version.
http://cgit.freedesktop.org/gstreamer/gst-plugins-good/commit/?id=64b06d1829493d9119022ed5e8476f3e9b486ec5
So when we switch to latest git or gstreamer 1.5.3 we should remove this patch.


Commented codes in production is _really_ bad!


This does not seem to be sufficient yet.

Implemented create and destroy context methods as below.

I call create before muilti-threaded use of bitcoinconsensus_verify_script, destroy afterwards. No change, the lib produces:

Assertion failed: (pkey != NULL), function CECKey, file ecwrapper.cpp, line 96.

randomly

```c
static pthread_mutex_t *ppmutexOpenSSL = NULL;
static int numOpenSSLLocks;

static void openssl_lock_callback_null(int mode,int type,const char *file,int line)
{
}

static void openssl_lock_callback(int mode,int type,const char *file,int line)
{
    if(mode & BITCOINCONSENSUS_CRYPTO_LOCK)
    {
    	pthread_mutex_lock (&ppmutexOpenSSL[type]);
   	}
    else if (mode & BITCOINCONSENSUS_CRYPTO_UNLOCK)
    {
    	pthread_mutex_unlock (&ppmutexOpenSSL[type]);
    }
}

JNIEXPORT void JNICALL Java_com_bitsofproof_bitcoinconsensus_LibBitcoinConsensusJNI_createLibconsensusContext (JNIEnv * env, jclass c) {
    ppmutexOpenSSL = (pthread_mutex_t*)bitcoinsonsensus_CRYPTO_locking_callback_malloc(sizeof(pthread_mutex_t), &numOpenSSLLocks);
    for (int i = 0; i < numOpenSSLLocks; ++i) {
    	pthread_mutex_init (&ppmutexOpenSSL[i], NULL);
    }
    bitcoinsonsensus_CRYPTO_set_locking_callback(openssl_lock_callback);
}

JNIEXPORT void JNICALL Java_com_bitsofproof_bitcoinconsensus_LibBitcoinConsensusJNI_destroyLibconsensusContext (JNIEnv * env, jclass c) {
	bitcoinsonsensus_CRYPTO_set_locking_callback(openssl_lock_callback_null);
	 for (int i = 0; i < numOpenSSLLocks; ++i) {
        pthread_mutex_destroy (&ppmutexOpenSSL[i]);
    }
	bitcoinsonsensus_CRYPTO_locking_callback_free(ppmutexOpenSSL);
 }
```


Remove "targets"


Either needs docs or should be private.


这么666，下次试试。


Error: syntax error, unexpected 'unclaim' (T_STRING)
File: /fp.phar/src/FactionsPro/FactionCommands
Line: 340
Type: E_PARSE

THIS CRASH WAS CAUSED BY A PLUGIN
BAD PLUGIN: FactionsPro v1.3.2

Code:
[331] 						$faction = $this->plugin->getPlayerFaction($sender->getPlayer()->getName());
[332] 						if($this->plugin->drawPlot($sender, $faction, $x, $y, $z, $sender->getPlayer()->getLevel(), $this->plugin->prefs->get("PlotSize")) == false) {
[333] 							return true;
[334] 						}
[335] 						$sender->sendMessage(TextFormat::YELLOW. "Claimed land for your faction from ". TextFormat::GREEN. " Wilderness);
[336] 					}
[337]
[338] 					/////////////////////////////// UNCLAIM ///////////////////////////////
[339]
[340] 					if(strtolower($args[0]) == "unclaim") {
[341] 						if(!$this->plugin->isLeader($sender->getName())) {
[342] 							$sender->sendMessage($this->plugin->formatMessage("You must be leader to use this."));
[343] 							return true;
[344] 						}
[345] 						$faction = $this->plugin->getPlayerFaction($sender->getName());
[346] 						$this->plugin->db->query("DELETE FROM plots WHERE faction='$faction';");
[347] 						$sender->sendMessage($this->plugin->formatMessage("Plot unclaimed.", true));
[348] 					}
[349]
[350] 					/////////////////////////////// MOTD ///////////////////////////////

Backtrace:


Ops! Should be private! Thanks!


Thanks for fixing `fork`!


The Openni2_launch says exactly the opposite. It says its for the Xtion and not for the Kinect. So im not sure why it does not work.


"if" -> "for"


A lot of (hopefully) helpful comments!


Well, since the commit message must signify the most important change in the commit, it must have been something like 'Merged all-activities and current-activities pages'.
So, in comparison to the ideal your mistake is not a biggie! :P


dasd


derped


@MacDada @FXML annotation is a part of the JavaFX API. Basically, you can define all your GUI in *.fxml file, and your controllers in *.java thus separating the logic from the definition of UI. For example, consider a button definition in fxml:
<Button text="Settings!" onAction="#editSettings" fx:id="settingsButton" />
Notice two things - onAction="#nameOfYourMethod"  and fx:id="nameOfYourField" attributes. When you create your java controller file, you can create @FXML annotated fields (have to be private, non-static) and methods (have to be private void, throwing no exceptions), which are then automatically injected/linked with appropriate UI control.


Can you send the PR?


It was not working without static linking before? When did it stop working?


This option doesn't do anything.

```elixir
iex(3)> Crutches.Format.Number.as_human(123456789, strip_insignificant_zeros: false)
"123 Million"
```


Блин, за что?
<< media-gfx/alchemy
<< media-plugins/gimp-wavelet-decompose
<< media-plugins/gimp-wavelet-denoise


I understand it, but I'm not sure how it would be implemented.


`--force` deletes everything and always has.


@xHFx Where is says
```php
sender->sendMessage(TextFormat::YELLOW. "Claimed land for your faction from ". TextFormat::GREEN. " Wilderness);
[336] }
```
After wilderness, put a <code>"</code>


This is cool :)


@MacDada 'toolbar' is automatically injected through the FXML initialization (check the comment below, about (at)FXML annotation and the one about constructor arguments).


I've tried using it but I didn't manage to make it work. When and how should the frameBased flag be set?

I tried something like this:
```
var tween = game.add.tween(sprite).to({x:200}, 200, Phaser.Linear.None, false);
tween.frameBased = true;
tween.start();
```

Not the exact code, but something similar and it didn't look like a frame based tween. What did I do wrong?


You should increase the precision. The default precision is 3, and significant is true, so it's correct. Use `precision: 4` to see a trailing zero.


Try rewriting it to `play = Hit(TARGET, Attr(Draw(CONTROLLER), GameTag.COST))`.


This was not supposed to be removed?


:+1: thanks.


It is not strange that a bundle having the Fragment-Host header also has the capability as host of the fragment? should a fragment host requirement not just match a bundle identity capability? The host of fragments does not have a specific header in it's manifest?


I can't agree, that's what your patch does: moves "gi--;" one level instead of two deeper than "--stopidx;" I've discovered it a day or two before your patch and fixed it myself.


Why not make this one `private` and store it as a constant?


Whoops, you're right. Spoke too soon!


In ES7, which we are using, you can write it this way:
```js
var rotate45;
if (this.state.isExpanded) rotate45 = {transform: 'rotate(45deg)'};
else rotate45 = {};
...
style={{...styles.fabStyle, rotate45}}
```


@alehander42 because some tokens can spawn multiple lines, does it make sense to convert the line information from `[line, column_begin, column_end]` into `[{line_begin, column_begin}, {line_end, column_end}]`?


> Creature_linking - this is hack.

... I have nothing more to say... :neutral_face:


This probably shouldn't be done unconditionally. Users may have configured other addresses already. An implementation of `MailAddressResolver` is probably the better approach.


@aMarCruz @cognitom we could update the package.json once npm 3 gets released [see also](https://github.com/karma-runner/karma-phantomjs-launcher/issues/71)


I'm not sure I understand the reason for duplicating readme.md


get in touch @jparicka - thanks!  :)


It actually already is a constant in `Texts`, as [`Texts.EMPTY`](https://github.com/SpongePowered/SpongeAPI/blob/master/src/main/java/org/spongepowered/api/text/Texts.java#L59)


While you are correct that this line needed to change I believe the correct line would be
"You will need to bind a shortcut to the 'CleanupFileAndNamespaces' action in order to use this plugin."


Ah , I see.. Anyway to add war zone?


qsbBar is written in Launcher.java
If there are no search provider package, it returns "null".

I changed these lines like this and re-build the Launcher3.

        if (qsbBar != null) {
            qsbBar.setVisibility(searchBarVisible ? View.VISIBLE : View.GONE);
            LayoutParams vglp = qsbBar.getLayoutParams();
            vglp.width = LayoutParams.MATCH_PARENT;
            vglp.height = LayoutParams.MATCH_PARENT;
            qsbBar.setLayoutParams(vglp);
        }

Now I get no error message.
I also tested install google search app and search bar setting on/off, it's OK.


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


Server works with MySql. H2 database is only for local testing.


@xHFx Probably. I can (hopefully) make it so you create  a war zone faction and put it on an alt account then if you're in the warzone claim you can't build


I don't understand 'A*'


Factions chat .. Can you add this http://dr-pvp.com/plugins/repo/factions/


ok, so which should I used?

Don't get proximity sensor working..


It is fixed in the multi-install-step branch.


Are you sure that this button send all params from form to server?


Actual changes at 4e6740f8863c7ce70fb4e735d1022a26acece81b 82b72723190ca5f50d85de8d1a55929c4d87093a


This is shorter, but I don't know if it is better:

```elixir
if precision < 0 do
  precision = 0
end
```


We should use post method there


Nice catch. I was looking for this too but did not find the time to dig into it ;-)


Far better. I used case because i simplified a previous version I wrote and didn't see it could be simplified more. :)


testtest


Look above.


fix a bug not allowing to have more than a signature per key


this is a test


What does it mean? What will we do with err in future? Where can we catch it? Or it will be like alert?


```elixir
if strip do
  strip_trailing_zeroes(number)
else
  number
end


`:strip_insignificant_zeros` is inconsistent with `strip_trailing_zeroes/1` and  `strip_trailing_zeroes/2`


why cat(1, ... ) and not vertcat ?


It will be great to use some datepicker there


if Args... is empty, this may be ambiguous with the unary overlaod.
if Args.. expadns ot a single type, this maybe ambiguous with the binary form.

You can start at horzcat(T, U, V, Args ... ) I think


Can I ask you why not? I realized it is now extra component available from Google Play, but you can still hook it, or not? I am just curious.


Avoid C-like casts. Use static cast or boost lexical cast. Windows does not like them.


Example:

```csharp
using System;
using System.Collections.Generic;
using System.Text;

namespace ListTests
{
    class Program
    {
        public static string ValueString(List<byte> bytesList)
        {
            StringBuilder sb = new StringBuilder();

            if (bytesList.Count <= 4)
            {
                byte[] valueBytes = new byte[4];
                bytesList.ToArray().CopyTo(valueBytes, 4 - bytesList.Count);

                // Reverse array if we are on little-endian machine
                if (BitConverter.IsLittleEndian)
                {
                    Array.Reverse(valueBytes);
                }

                sb.Append(BitConverter.ToInt32(valueBytes, 0));
            }
            else
            {
                foreach (byte b in bytesList)
                {
                    sb.AppendFormat("{0:x2}", b);
                }
            }

            return sb.ToString();
        }

        static void Main(string[] args)
        {
            List<byte> oneByte = new List<byte>(new byte[1] { 0xff });
            List<byte> twoBytes = new List<byte>(new byte[2] { 0xff, 0xff });
            List<byte> threeBytes = new List<byte>(new byte[3] { 0xff, 0xff, 0xff });
            List<byte> fourBytes = new List<byte>(new byte[4] { 0xff, 0xff, 0xff, 0xff });
            List<byte> fiveBytes = new List<byte>(new byte[5] { 0xff, 0xff, 0xff, 0xff, 0xff });

            Console.WriteLine("Length: {0}, Result: {1}", oneByte.Count, ValueString(oneByte));
            Console.WriteLine("Length: {0}, Result: {1}", twoBytes.Count, ValueString(twoBytes));
            Console.WriteLine("Length: {0}, Result: {1}", threeBytes.Count, ValueString(threeBytes));
            Console.WriteLine("Length: {0}, Result: {1}", fourBytes.Count, ValueString(fourBytes));
            Console.WriteLine("Length: {0}, Result: {1}", fiveBytes.Count, ValueString(fiveBytes));

            Console.ReadLine();
        }
    }
}
```

Program output:

```
Length: 1, Result: 255
Length: 2, Result: 65535
Length: 3, Result: 16777215
Length: 4, Result: -1
Length: 5, Result: ffffffffff
```


same here, you may want to start at T1, T2, T3, Args...


Ëîãèí ?


Ïàðîëü ?


Remove these two lines (8 and 9) because if they weren't deleted by `go template` this would always generate a map with `Key` type `string` and Value type `int`.


you can say ' a0... Pack of paramtrrs to check'


Sure thing. A bad old habit of mine.


Since you writed this
-Kieron for 1Behavior
 maybe you can add
-sebastianfra12 for alot behavior


This is what has been generated for me:

```go
// Template Concurrent Map type
package device

import "sync"

//template type ConcurrentMap(Key, Value)

type cmapDeviceMap map[string]Device

// A "thread" safe map of type Key:Value
type DeviceMap struct {
	items map[string]Device
	mutex sync.RWMutex
}

// Creates a new concurrent map.
func NewDeviceMap() *DeviceMap {
	return &DeviceMap{
		items: make(map[string]Device),
		mutex: sync.RWMutex{},
	}
}

// Sets the given value under the specified key.
func (this *DeviceMap) Set(key string, value Device) {
	this.mutex.Lock()
	defer this.mutex.Unlock()
	this.items[key] = value
}

// Removes an element from the map.
func (this *DeviceMap) Remove(key string) {
	this.mutex.Lock()
	defer this.mutex.Unlock()
	delete(this.items, key)
}

// Retrieves an element from map under given key.
func (this *DeviceMap) Get(key string) (Device, bool) {
	this.mutex.RLock()
	defer this.mutex.RUnlock()
	val, ok := this.items[key]
	return val, ok
}

// Retrieves an element from map under given key.
// If it exists, removes it from map.
func (this *DeviceMap) GetAndRemove(key string) (Device, bool) {
	this.mutex.RLock()
	defer this.mutex.RUnlock()
	val, ok := this.items[key]
	if ok {
		delete(this.items, key)
	}
	return val, ok
}

// Returns the number of elements within the map.
func (this *DeviceMap) Count() int {
	this.mutex.RLock()
	defer this.mutex.RUnlock()

	return len(this.items)
}

// Looks up an item under specified key
func (this *DeviceMap) Has(key string) bool {
	this.mutex.RLock()
	defer this.mutex.RUnlock()

	// See if element is within shard.
	_, ok := this.items[key]
	return ok
}

// Checks if map is empty.
func (this *DeviceMap) IsEmpty() bool {
	return this.Count() == 0
}
```


What's your idea with concurrent_test.go? Have a template to generate tests for concurrent map?

If the purpose is to test that the template generates valid code then this must go to a different directory.


What does it mean?


You can do this in one go in ES6:
```js
var {BarChart, AreaChart, PieChart} = rd3;
```
Also, you could do this in the import sight.


ok just bad cut and paste, I will correct it


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


yep!, tested and worked :+1:


It's not semantic to use `<H_>` tags to style.


Important: If last_retrieved = todays date and curr is submitted today => old one will add it duplicated.


Whoops, wrong commit message.  Ignore it. It belongs in the next commit. This was an accidental commit to clear up one errant space.  Yikes!


not sure if these should be removed. I tested on windows and it seemed to compile fine without them. any harm in leaving/removing these libs?


如果是浮点数呢？http://blog.csdn.net/v_july_v/article/details/8701148


The nullptr checks are not strictly necessary, `free(nullptr)` is a no-op.


What is the absolute path of img folder?


When is it necessary to change the list of packages to be installed?


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.494](http://teamcity.monogame.net/viewLog.html?buildId=9643&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 538, ignored: 6 Build time: 00:39:15


Lol didn't see that. I just copied my h815 folder anyway.

On Sat, Aug 15, 2015, 4:06 AM Erwan Leboucher <notifications@github.com>
wrote:

> https://github.com/genesixx/android_device_lge_h811
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lg-devs/android_device_lge_h811/commit/6657dc2dd4a21d489dd620a1f55c133882905862#commitcomment-12722700>
> .
>
--

Thanks,
Rashed Abdel-Tawab


I tried to run it, but there is something wrong with line 8. Not sure what's going on---it looks exactly like the line in the book.


The documentation of functions is great!  We have been using `roxygen2` to auto-generate manual pages.  Please reformat function documentation when you get a chance.


Argh, I meant to create a pull request for this, but it just went straight to master.  Hacking on a Mac for the first time, I must have done a step wrong.  Sorry!


With https://github.com/ZdrowyGosciu/device_g2/blob/lp511/system_prop.mk#L63 in D802 all sensors are working.. Maybe it's related to model?


English?


It will become customizable a bit later, so you can select the metrics and set a manual position etc.


Works for me, thanks.


`was signals` --> `signals`


91977808_LBNL_NewVRFModel (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (755 of 755 tests passed)

![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) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-7.3%25-green.svg)


That doesn't make a differenc because you have pure chat which overrides this


It has been for almost all the cases I've seen, but of course, that is a pretty limited subset. This is probably too strong a statement though. How about this:

"One should do their own benchmarks to determine the appropriate `chunk_size` for their problem. In many cases, the fastest option might be to not specify `chunk_size` at all. The `chunk_size` option is mainly useful in cases where performing the entire calculation at once would causes memory concerns."


A view *doesn't* have a lot of things -- no OpenMM calls, no `parameter_types` lists, no `strip()`, no *subsequent* selections... it really doesn't have *anything* except `atoms`, `residues`, `bonds`, `angles`, ..., and those are regular lists, not `TrackedList`s.  It's not even really a full view, since the lists of attributes are separate lists -- e.g., if you swap the order of two atoms in a view, that is *not* reflected in the original structure.  The only thing that a view guarantees for you is that the atoms, residues, and parameter/topology objects are references to the same ones used in the Structure.

Really all it's good for is collecting a set of atoms together that you can operate on to change the original structure.

The big thing here is that the `idx` attribute of all the types, atoms, and residues are only valid for a single list.  Which means that in a selection, then those indices can be valid for the original list (as in a view), or the list of the selection (as in a copy), but not both.  We've already discussed in a previous issue (#272) why a slice/selection returns a copy, and it's not something I plan on changing.


Finally this looks about right :)


resume instead of unpause looks better for me


I think this is an wrong behaviour, isn't?


Fyi, so far just compiling ffmpeg on a Core 2 Duo 4GB laptop, fast internet connection, got me
```
sandbox# time bash -ex ./ffmpeg_project.sh
real	31m43.887s
user	25m25.880s
sys	2m21.987s
```
Adding mpv
```
sandbox# time bash -x ./ffmpeg_project.sh mpv
real	2m41.057s
user	1m32.803s
sys	0m20.207s
```
Let's run some tests now.


How is casting the given `BaseClientPutter` to an arbitrary subclass of it more robust against internal changes in Fred? Oh wait, it’s not.


Translations traditionally aren't getting into changelog.


nice commit message :)


No . It doesnt sadly


Please put `where` and `output` on the same line because there's only one `where` clause


Well that's fucked up it should.


update


@riastradh-probcomp this change broke existing code in the `bdb-experiments` repo, and possible other places where `tests` was imported. I am going to restore.


No , While in factions chat it uses that format that you set , also the nametags do not work ... PureChat name tag options this overides it idk why


No functional change, just fixing up names plus code indentation.


I don't need tftp package on my CentOS7 host.
I use only /usr/sbin/in.tftpd


Yeah, there was a configuration that I had forgotten to change when I made a copy of the pull request job. Should work now. Let me know if it still does not build when you push to master


Usually you can either tell the linker to link against libraries in the project settings (or in cmake using `target_link_library()` which in our case is done by `build_addon()`) or through such a `#pragma comment(lib, ...)`. The idea behind this pragma is that the library author can add this information into a general header which will save the library user from having to do it himself.


9:53
77729351


Better set `overrideCursor=False` before the iface check and remove the iface check here


good


Change log please.

We should review the notes from the last upgrade and ensure we avoid issues we saw then


And I want to install selinux as well at the same time. So here are use-cases.


-lbsdxml should be replaced by -lexpat here


See issue #44


New page of Roboclub.


I changed my mind. Imho the error should be reported also without main. If the code "s->value - 1" is reached there is a null pointer dereference.


ok  that is ok


Will you send a working example of e grep so I don't have to think about it?


Zbędne spacje pomiędzy `=`

```html
type="submit"
```


Only looks that way because for the test php://stdout is used as the logging target.


### mpv test an old b&w movie
```
Playing: lgg.avi
 (+) Video --vid=1 (mpeg4)
 (+) Audio --aid=1 (ac3)
AO: [alsa] 48000Hz stereo 2ch float
VO: [opengl] 608x272 yuv420p
AV: 00:00:41 / 02:11:59 (0%) A-V:  0.000 Cache:  0s+73MB
[ffmpeg/audio] ac3: frame sync error
Error decoding audio.
AV: 00:00:48 / 02:11:59 (0%) A-V:  0.000 Cache:  0s+73MB
[ffmpeg/audio] ac3: frame sync error
Error decoding audio.
(Paused) AV: 00:01:06 / 02:11:59 (0%) A-V:  0.000 Cache:  0s+73MB
[ffmpeg/audio] ac3: frame sync error
Error decoding audio.
(Paused) AV: 00:01:18 / 02:11:59 (0%) A-V:  0.000 Cache:  0s+73MB
[ffmpeg/audio] ac3: frame sync error
Error decoding audio.
(Paused) AV: 00:01:28 / 02:11:59 (1%) A-V:  0.000 Cache:  0s+73MB
```
Playback looks good both windowed and full-screen, CPU 3.7%, memory 5.3% (4GB Core Duo Fatdog64 701). All frame sync and audio decoding errors are reported when pressing the arrow keys to seek a position. In spite of the error message, seeking works and audio/video remain synchronized.
Pass 10/10.


@Nadahar this description needs a little editing :).


Hmm... You can just check for the law computer again. It's less elegant, but it keeps things in the right place. I've sent a pull request to your repository showing how I'd do it.


Thanks for testing. Odd that it's not sufficient, as it matches what we do in Core.

- Are you sure your callbacks are being set? A quick printf in openssl_lock_callback() would verify that.
- Are you using any other libs which might be stealing the callback? I know that's a fairly common problem, I've dealt with it myself when using libcurl in my application, for example.

I'll try to find some time this weekend to setup a real-world threaded test-case.


This is the same logic used in [Jade's loader](https://github.com/jadejs/jade-load/blob/0dfc2f2de005d618466ee48703d4ec1fc62ad309/index.js#L29), except without support for include filters.


I think that this is a reasonable test to demonstrate the problem RT#106276


fixed #62


Пустое условие.


definitely a derp


Please use a .tar.gz archive instead.


Remove this blank line.


And this one.


Having `secondaryArchSuffix` is really pointless here, since there's no `SECONDARY_ARCHITECTURES` or any `$secondaryArchSuffix` on other PROVIDES/REQUIRES.


Do not add CSS before consulting the rest of the team


This is a single-page App. Do not add any other routes!!!


* 추가한 기능
 * 네이버 인증 로그인
 * 기존 튜토리얼이 바로 시작되는 구조에서 로그인 과정 후 버튼을 누르면 진행하는 것으로 임시 수정
 * action bar에 menu가 나오게 하고 싶었는데 아무리 해도 안된다ㅠㅠ
   * 코드 병합 전에는 보였는데 통합작업하면서 뭔가 바뀐듯... 추후 수정 예정


Please add this stuff to the README


no need


:clap:


lol xD


also you should support backwards compatibility. You should check the "post-message-api" element exists

I would do something like
var postApi = document.getElementById("post-message-api");
var allowedOrigins = (postApi && postApi.getAttribute("accepted-origins")) || '';


then you don't need these three lines

allowedOrigins = allowedOrigins.split(',');


no h8 m8 r8 8/8


You know that allowedOrigins is an array. You can just write
allowedOrigins.indexOf(origin) === -1


Why did you remove that and added an empty line?


I'm looking through your recent commits to LD.  This one looks like the right spirit, but should "sampe" be "sample"    I would like to commit this to my clone of LD.


Hi! should I write the code in the last line into the autoexec.ash file?
Can it BRICK my camera anyway?

Thanks for reply!


please make it more attractive


### Playing color episode matroska file x264/ac3
```
# ~/mpv ep1.mkv
Playing: ep1.mkv
 (+) Video --vid=1 (*) 'ep1' (h264)
 (+) Audio --aid=1 (*) 'ep1' (ac3)
File tags:
 Title: ep1
AO: [alsa] 48000Hz stereo 2ch float
VO: [opengl] 1280x640 yuv420p
(Paused) AV: 00:06:41 / 00:58:25 (11%) A-V:  0.000 Cache: 10s+73MB
```
Excellent playback. Memory 5.7%. Virtual memory 959M. This runs 13 (!) processes, the top process takes 30% CPU, the remaining ones sum up to 10%-20%.


Бэмовский элемент существует вне бэмовского блока, такого быть не может. Лучше сделать бэмовским блоком


sagar


То же самое элемент без блока


I don't think u need to check it here. The event won't be dispatched if the targetOrigin won't be compliant with the parent origin.


@njsmith @rgommers here is an attempt using metamodule. It fails the deprecation tests though.

These are the failing tests:

```python
def test_MODES_deprecation_new():
    def use_MODES_new():
        return pywt.MODES.symmetric

    assert_warns(DeprecationWarning, use_MODES_new)

def test_MODES_deprecation_getattr():
    def use_MODES_new():
        return getattr(pywt.MODES, 'symmetric')

    assert_warns(DeprecationWarning, use_MODES_new)
```

both fail with `AssertionError: No warning raised when calling use_MODES_new`.

@njsmith, is my implementation wrong?


Модификатор не может использоваться без основного класса. Скорее всего здесь имелся ввиду класс design__text


looks good plz add tests


looks good 10x!


Looks good. Plz add tests


Thanks, it's pretty useful to know that it's possible to call floor events multiple times by modifying that line :+1:

Are you sure that the camera is being updated every 1/60 time interval? I'm confused, we shouldn't need to use a flag in this case? As long as there's a timer set, we know not to switch the camera back to Lara. It seems that some timers are 0, so I'm going to assume they just ran the function every time Lara is on those specific camera sectors with 0 timer.


Also, plz add tests.


Thank you, will take a look at this :+1:


This causes lots of eval breakage for me, e.g. `nix-build nixos/release.nix -A tests.lightdm.x86_64-linux` results in:

````
error: The option value `system.modulesTree.[definition 1-entry 1]' in `/home/tmtynkky/nixos-configs/nixpkgs/nixos/modules/system/boot/kernel.nix' is not a path.
````

Looks like previously a derivation was considered a valid path, but now it isn't anymore.


 #[zstack Test Bot Report] P A S S
 Integration Test Result:
Test Summary:

 TestSuite|TestCase | Pass | Fail | Timeout | Skip
 ---------|---------| ---- | ---- | ------- | ----
 Total|||||


Oops, sorry. I'll revert the changes to `isPath`.


@rgommers r.e. test noise: there are no calls to deprecated functions in the tests except for those wrapped in `assert_warns`. Instead, the noise seems to come from the `inspect` module:

```python
/home/aaron/src/anaconda/lib/python2.7/inspect.py:153: DeprecationWarning: MODES has been renamed to Modes and will be removed in a future version of pywt.
  return isinstance(object, types.FunctionType)
```

Perhaps it comes when nose loads the module? I'm not sure.


I really dislike making the mempool depend on validation. I think it should remain a data structure whose responsibility is maintaining consistency, indexing and performing certain modification, but the logic of what changes are allowed to be performed should be outside of it, IMHO.


delete


Can these store indexed mempool iterators rather than hashes?
Are these recursive?


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (355 of 355 tests passed)

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


How is this different from mapNextTx?


@Tobschinski Agreed, and the same applies to TBC. Actually, I'm fairly sure it's 30% for WoTLK too. I believe they changed it to 40% with the BRD changes done in Cata. I just checked ingame, and the dungeon journal now says 40%. But I'm willing to bet that the video that @xfurry is referring to was made with 4.x or later. I'll try to find a find a fight with the general in my sniffs from the WoTLK era to confirm the 30%.


develop (lefticus) - i386-Windows-8.1-VisualStudio-14: OK (355 of 355 tests passed)

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


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/781d62d3a532788e945adb768f42d9d617e0112e/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/781d62d3a532788e945adb768f42d9d617e0112e/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/781d62d3a532788e945adb768f42d9d617e0112e/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/781d62d3a532788e945adb768f42d9d617e0112e/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/781d62d3a532788e945adb768f42d9d617e0112e/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (355 of 355 tests passed)

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


Please add `final static`


You forgot the `#` in the commit message.


@dmircevski It seems you haven't set up your personal info (Name & email) correctly? Github didn't correctly link this commit to your personal account.


Please add `static final`


version with more text.


:boom: A future thing. Documentation and guidelines are in progress. Text-OS is taking up most of my time anyways.


develop (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (355 of 355 tests passed)

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


resolves #57


While I see the use for this, I wonder if we shouldn't introduce a property on the termgroup element allowing people to set any termgroup as a site collection termgroup. Will bring this up with the team.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-AddressSanitizer: OK (355 of 355 tests passed)

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


break;?


How about

```
if (!res.isAutehnticData()) {
  // log
  return false;
}
```
?


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (355 of 355 tests passed)

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


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (355 of 355 tests passed)

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


develop (lefticus) - i386-Windows-8.1-VisualStudio-12: OK (354 of 355 tests passed)

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


`export CXX=g++-4.8` won't work, if `g++-4.8` isn't installed. Remove these 4 lines and check if it works


I'm not quite sure that we can specify metadata within the input and do not cause too much trouble. (They might not have been translated to ItemStack at all, but just Item instance). I'll find another time and look into this problem.


#jobless


No need to do this for each segment. Just chain check the full array access


Thanks for your help, this discussion disappeared after I recommitted my code & I thought there's was way to get back until I found a link to here on my homepage :)
Can you please help me with another git-related task - it says that my branch is 2 commits ahead, 9 commits behind ampache:develop - is there a way to synchronize those 9 commits without losing my code?


@WeAthFoLD get. Wait... That's u tell me to use metadata! Why didn't u tell me about the problem else!


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (355 of 355 tests passed)

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


It's my bussiness and I'll fix this, just keep it this way >)


No it doesn't too.
When the above 4 lines were not present, I got this error: https://travis-ci.org/Sumith1896/csympy/jobs/75725343#L200


@vkbansal just out of curiosity, any reason my name is removed here as the owner? :)


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


@pburtchaell Adding this documentation without updating the package in npm is confusing. Make sure NPM is updated. :) Cheers!


Then you should upgrade the `g++` compiler for this test case. Example is here, `https://github.com/isuruf/piranha/blob/travis/.travis.yml#L16`


It will be complete mess if you try to `pageStack.push( ... )` some page with its own actions.
How it looks like:
![actions_mess](https://cloud.githubusercontent.com/assets/5008338/9289139/dbb727de-436e-11e5-9f51-dd552c91b43e.png)

And how it should look:
![actions_correct](https://cloud.githubusercontent.com/assets/5008338/9289144/11c09cde-436f-11e5-80c1-cd963fb03aea.png)


typed_function_ordering (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (356 of 356 tests passed)

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


Dit is overigens de aanpassing van de notifications, ik was vergeten om de commit naam aan te passen.


Aggiunto il catch per le RuntimeException


@dmircevski I've done a force push to fix the author line in 767df24a6450eb780151684a5b7caa5c6b773038. It's Saturday. Hope it won't break something for anybody..


@mvalipour Sorry! It was a honest mistake. I didn't realize it. It must have slipped while fixing merge conflicts. I don't even use Gherkin.


I think IntelliJ is the better choice.


how can i instal it on android messenger?


accept.


Роли есть в /api/user/current

support - bool, если true то имеет доступ к области саппорта и может отвечать на вопросы, иначе только задать
moderator - bool, если true то имеет доступ к области модерации и может отменять бан.
сейчас в dev ветке распишу детально


develop (lefticus) - Win64-Windows-8.1-VisualStudio-12: OK (354 of 355 tests passed)

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


typed_function_ordering (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (356 of 356 tests passed)

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


@skeptical I was a bit tired when I did that, but .... :rage4:


It's cool man, thought maybe there's some sort of contribution rule or sth :)


@Kitke One more thing. Imagine the app grows. More and more windows are made, more and more things get configurable. Would u inject `Settings` object everywhere in that case?

I can see 3 options:

* inject params directly (`AboutWindow` needs a github account link? Inject the link, skip the middle `Settings` man): `+` clear and simple, `-` multiple arguments when multiple settings needed, `-` recreating `AboutWindow` or a setter when `Settings` change

* Interface segregation (instead of injecting `Settings`, u might have `AboutWindowSettings` interface; single `Settings` instance implementing many such interfaces, or many kind of `Settings` classes): `+` clients (like `AboutWindow`) need to "know" less (instead of a big class, they know the interface with only methods they need), `-` u might get maaaaany interfaces

* Generic settings class (instead of hardcoding getters for messages, create one "generic" method like `settings.get("messageKey")`): `+` the implementation can by anything (a hashmap, a switch/ifelse, could be easily stored in a file/database), `+` clients don't need to be recompiled every time `Settings` change, `-` very loose dependency, u can't easily see which keys are need / not needed (anymore)

Which option is best? None. You might even have all three of them in one project, but what's known for sure, is that "one-fits-all" `Settings` class with 100(s) methods is not cool in the long run ;-)