The code that parses GBRs verifies that the payload is valid utf-8 (which a vCard must be), but it doesn't verify the validity of the vCard itself. This test case is both not a valid vCard and not valid utf-8 so it passes (and I believe has always passed), but the reason it passes isn't directly related to the name of the testcase. It's possible that this testcase should be split into two at some point (NotUTF8 and NotVCard), but in the meantime I agree that this NotVCard case should pass.


combine the db.Open and the error checking into one line

```golang
if e := db.Open(); e != nil {
      fmt.Fprintf(os.Stderr, "Error opening database: %v", e)
      os.Exit(1)
}
```


With xunit running tests in parallel, the script can be made to run as a Test class/assembly initialization step using fixtures or constructor-dispose pattern. However it depends on how the networking team is designing the infrastructure for the tests. @vijaykota can provide insight here.


7. hello


noice


Hey there.  I know it's been about a year since you did this code, but wanted to let you know, I've moved it into the Merchello core with a few alterations.  There have been a few features added to Merchello that did not allow me to do a direct port, most notably the extended product content - but I've followed your pattern and have that bulk inserting as well.

I also made a base repository that makes the batch update statement off of the DTOs ... your code had this as a TODO.

We'll release it in version 1.15.0 which will come out towards the end of the month / early next month.

Anyway, awesome job.


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


Free df bhai +I hate clowns

 Lock conversation


You could make the same argument for points, but those are still the same. Our default sort behavior is to sort ascending, which is applied to all fields. We could certainly talk about changing that, but I don't think it makes sense to just change it for a single field.

Besides, you're manually overwriting the sortAscending property and hence ignoring the sort order that is passed into through the URL. That way you're ignoring the users sort preference, which is why it's now impossible to reverse the sort order.


:+1: was experimenting (as with all these changes :blush:) using further plugins - and this was an over-zealous tidy up!

I'm terrifically impressed with your solution (thanks for sharing! Jenkins deployments are unruly swine), and need to level up on Jenkins Job Builder. Do you know of a tool to reverse engineer job configurations by chance?


This line needs to be where it originally was because no matter whether we are using path params or the request body, the arguments need to be built before invoking the method


Good job simplifying this


Got it. I did the bootstrap because I want to see how much does the list library benefit from the optimization, but yeah, I will remember to reset phase0 back before committing.


In which case there's no body? I assume a HEAD request. But in that case why do you access the body?

You can use `response.body_io?` for a nilable IO. If we make `body_io` nilable it's more boring to use (`response.body_io.not_nil!.gets`, etc.)


I think a better signature for this routine is
```go
func register(router *mux.Router, controllers ...controller.Controller)
```
This would allow you to call the method with:
```go
register(router, heatCtrl, tempCtrl, programCtrl)
```


Note to self: & is a thing...............


I'm on the fence about whether we need a has no params annotation since we detect that either way, but maybe I'm not thinking of the use case you are


You don't need the route argument to this method, since the route is already visible here.


I think instead of removing the given tests we should make this new one a separate test. it was a great test, maybe call it testAddBlocksWhenFull()


Remember to use complete words for variables


Hi John, thanks for pointing it out. I think these methods are generated by doctrine and therefore they are consistent with the attribute name `$versions'. But you are right, it is quite confusing, I will change them, thanks!


Sils, I cannot find any combinations or using Ignore keyword capable of
doing this.
I tried wrapping it around, giving Ignore in each line, and stuff.

Could you look at it ? At best case scenario, isort puts all imports above
the assertion, not below

On Thu, Feb 11, 2016 at 2:23 AM, Lasse Schuirmann <notifications@github.com>
wrote:

> you can define a range with # Start ignoring PyImportBear or you need to
> have this comment above *one* of the affected lines of the diff. Question
> is how that move looks in our diff so a range might be better?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/9696322524682912ec6c4c9df5ad25eb4dab3e98#commitcomment-16016934>
> .
>


> What’s dangerous other than file:?

I assume everything, until proven innocent. Even browsers ask before opening unknown schemes, and — as the bug has taught us — `openURL` is more powerful and trusted than opening the same URL in the browser.


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


Since I doubt anyone's pulled this branch yet, go ahead and undo this commit, commit only the modified files and not phase0, and force-push.


Daily reminder this should just be returning its output instead of actually doing anything.


You can use org.apache.commons.httpclient.HttpStatus constants instead of hardcoding the numbers


TeamCity FubuMVC Swank :: Deploy [Build 2.0.56.0](http://build.mikeobrien.net/viewLog.html?buildId=2509&buildTypeId=fubumvcswank&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 399 Build time: 00:01:41


唔，不行啊，这个仓库根本不是git仓库，没法发呀。。。

发自我的 iPhone

> 在 2016年2月11日，03:53，Mingye Wang <notifications@github.com> 写道：
>
> 给锅子 pot 人（我是说 pottering）发个 bug 吧。
>
> —
> Reply to this email directly or view it on GitHub.
>


@zhangwei1983ch , please check


## Task 1

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly demonstrates the use of **chained conditionals** to accomplish a task (2 pt)

## Task 2

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly uses recursion to sum 0 to 1000 [inclusive] (2 pt)

## Task 3

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly uses recursion to compute `e` (2 pt)

## Task 4

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly accepts (and uses) the user input value for the maximum of the range over which to compute `e` using the `input()` function (2 pts)

## Task 5

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly accepts (and uses) the user input value for the maximum of the range over which to compute `e` using the "argparse" module (2 pts)


I guess you can use:
```for(cons auto &i : scan_line_msgs->bins) {...}```
And moving the loop increments in the loop body. It's a little bit easier to read, simple syntax comment.


hi @cydh
I wanted the player can use a command like.
nocostume
Help mod for command
thank


I have my own LineItem class, we will merge later, do not worry about implementing any of this.


It should be userFound :P


Please avoid memcpy, there is no reason to use it in C++ with all funtions of the stdlib.
Here it can be replace by a simple `=operator`


This means that the request isn't being decoded, which is fine as long as that doesn't change. If it were being decoded at some point, then the test would fail.


I agree, let's do this, unless someone else has a different argument why we shouldn't return a 400 when the handling method explicitly specifies one parameter, but the user sends nothing or else the JSON is malformed


Also, to be consistent with the existing implementation of `heroku orgs`, I removed the headers.


no comnents again ?


113008903Provide-a-better-warning-message-for-a-triangular-window (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2111 of 2112 tests passed, 0 test warnings)

 * 550 tests had: AUD diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: ZSZ big diffs.
 * 1 test had: Table big diffs.

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


`i` is `isize`, why would it be less than zero?


We could use the `for(const auto &i)` notation here also.


Your javadocs are very detailed and I can see you applied some patterns and were able to integrate the knowledge gained during the bootcamp in this final project. The authentication mechanism is very rustic you could have used some spring configuration to handle it.
Anyway, good job Matias!


"Would be the first time in TRCL history that I have nothing to complain about."  - lol :D

I half-knew about the ??? powerup (POWER1.BIN) but didn't get around to investigate it. Thanks to your report I have a head-start. Looking into it I forgot to add random selection of powerup in PowerupSystem so it ended up using the filler data with no effect on the player. This should be easy to fix.

The most GPU-specific thing I have changed is the maximum number of portals in a scene, from 4 to 8 and that may have pushed things too far for that 512MB card. I have a few things planned in optimization which may enable me to cut this back down.

Are the vanishing polygons mostly from terrain? I see that happen a lot and may have set relevance culling too aggressive. TerrainChunks are big objects so when their centers become irrelevant the still-visible edges can cut out while still in view. This is especially nasty in satellite view.


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


@iLexiconn why do they sit up right after they sit down?


yay for drinking our own :cocktail:!


Octree (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1541 of 1541 tests passed, 0 test warnings)

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


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


I'm a bit wary about this change.

@nrc could you review this?


113008903Provide-a-better-warning-message-for-a-triangular-window (lgu1234) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1529 of 1529 tests passed, 0 test warnings)

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


Please let me know if you'd like to make any necessary modifications and resubmit, or if you don't have time and I'll try to pick up where we left off at.  Thanks again for contributing


@rpspringuel This line breaks `tex-output/symbols-only/symbols-only.tex`, which only loads `gregoriosyms.sty`, which doesn't define `\gre@debugmsg`.  Should I remove this line or add `\gre@debugmsg` to `gregoriosyms.sty`?


Looks okay to me, just needs a sanity check from @nrc


## Task 1

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly demonstrates the use of **chained conditionals** to accomplish a task (2 pt)

## Task 2

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly uses recursion to sum 0 to 1000 [inclusive] (2 pt)

## Task 3

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly uses recursion to compute `e` (2 pt)

## Task 4

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly accepts (and uses) the user input value for the maximum of the range over which to compute `e` using the `input()` function (2 pts)

## Task 5

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly accepts (and uses) the user input value for the maximum of the range over which to compute `e` using the "argparse" module (2 pts)


Looking at the code in `HiveClientModule`, it appears that we can only support two modes: sasl and simple.  Are there other modes that we might want to support or Hive might add in the future.  If so, maybe we should change this to a property like:
```java
public enum HiveMetastoreAuthentication {
    SIMPLE, SASL
}
private HiveMetastoreAuthentication hiveMetastoreAuthentication = SIMPLE;
```


Wouldn't the presence of interpolate more properly be a usage error?

Isn't it an error if neither glyphs-path nor ufo-paths is specified?


same for this one


As a follow-on I've also tested this change on my phone with Firefox 44.  Some more changes are need to set `button` to -1 for a pointermove triggered my a touchmove (and probably something similar for a pen move).


This confused me a bit.  I guess if you have both OpenFont and Font, you prefer OpenFont.  Not sure why you didn't search for them in preference order and stop once you'd found something.


Please log this at level 'warning'.


Does this still work with socks?


you forgot the curly braces


this method should be private, otherwise people can bypass the singleton pattern


This sort of renaming is done elsewhere in this PR too. Given that HIR hasn't received the same renaming; perhaps we should perform the same removal of reexports and renaming on the HIR?


Commit


1. add Horses
2. add Horses -- fails
3. poll -- should return Horses, right? did this test pass


Throwing runtimes instead of checked exceptions is a bad practice. Also whenever an exception is thrown you need to provide a message explaining the reason, and what happened.


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


@nadavrot Can you thumbnail those images? My browser struggles on the image decode when loading this page...


you miss "#!/usr/bin/env python"for all your task.


 public void setAccion(String direccion,String idCasa, int num) {
        this.accion1 = "call registraCasa('" + direccion + "','"+idCasa+"');";
    }


Since chance is already 1 'catch up' does nothing and should be false.


I'm confused, how is glyphs_path available here?


while (rs.next()) {
                            if (rs.getString("resultado").equals("ira men no existe ese numero de serie")) {
                                System.out.println(numSerie);
                                System.out.println(rs.getString("resultado"));
                                System.out.println(rs.getString("estaduki"));
                                out.print("<script> alert('ira men no existe ese numero de serie');</script>");
                                out.print(registro);
                            } else {
                                if (rs.getString("resultado").equals("no hay despensa")) {
                                    conectar.setAccion(correo, pass1, clave, nombre, aPaterno, aMaterno);
                                    result = conectar.modificacion();
                                    conectar.setAccion(direccion,numSerie,1);
                                    conectar.modificacion();
                                    conectar.modificacion1("insert into usu(correo,tipo) values('" + correo + "','" + tipoUsr + "')");
                                    //ingresando la despensa a la base
                                    conectar.modificacion1("insert into despensa(estatus) values('activo')");
                                    conectar.modificacion1("insert into relCasaDespensa(idCasa,idDespensa) values('" + numSerie + "','1')");
                                    //ingresando la despensa a la base

                                    //conectar.setAccion(2, correo, pass1, huella1);
                                    //conectar.consulta();

                                    if (result == 1) {
                                        if (tipoUsr.equals("Premium")) {
                                            out.print("<script> alert('Bienvenido " + nombre + "');</script>");
                                            sesion.setAttribute("sessionMail", correo);
                                            sesion.setAttribute("sessionName", nombre);
                                            sesion.setAttribute("sessionStat", "logueado");
                                            out.print(index);
                                        } else {
                                            out.print("<script> alert('Bienvenido " + nombre + "');</script>");
                                            sesion.setAttribute("sessionMail", correo);
                                            sesion.setAttribute("sessionName", nombre);
                                            sesion.setAttribute("sessionStat", "logueado");
                                            out.print(index1);
                                        }

                                    } else {
                                        out.print("<script> alert('Error');</script>");
                                        out.print(registro);
                                    }
                                }

                            }
                        }


Thanks Mamba !
I think this change is wrong. legacy_map_view is smarty function so $ is not required.
Can you check this change again ?


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


Octree (DeadParrot) - Win64-Windows-7-VisualStudio-12: OK (2090 of 2091 tests passed, 3 test warnings)

 * 6 tests had: AUD diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: Table big diffs.
 * 2 tests had: EIO diffs.
 * 2 tests had: ESO small diffs.
 * 1 test had: SHD diffs.

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


Its a bad practice to name a variable with a verb just like a method.
In this case is worse because the variable has the same name that the method to get a connection and it gets real confussing


You should use curly braces even for single lined blocks


I don't think it hurts to have an alias.


Weird that this worked with a hyphen.


Falta ingresar datos a la relación de la casa con el usuario y de un procedimiento para quitar los insert del jsp


Update the sed command to:
sed -i 's|bind-address.*|bind-address=0.0.0.0|g' /etc/mysql/my.cnf

.* any char multiple times
g is for grep


You shouldn't catch exceptions and do nothing with them, you'll get a message saying the user doesn't exist but in reality maybe you lost connection to the DB


@nk0leg so I propose above to set `-1` as default. It will be more clear. BTW it's not good to set `NULL` to `int` value. At least set `0`. And what about `GetPort()`?


TeamCity Ppconsul :: Build (Linux) [Build 90](https://136.243.151.173:4433/viewLog.html?buildId=712&buildTypeId=Ppconsul_BuildLinux&guest=1) is now running


@flavour This one is for retrieving other fields too like mime_type, size


Why do you suppress raw types and serialization warnings here? Does not look like a raw type.


CLI Distribution test passed.


that spin.js library caused a few problems.  I remember seeing a CPU going to 100% because of it, at least twice


Interesting; I'll add it as an alternative. I like it, but we'd still need to clarify some parts. E.g. are the following to equivalent:

    Order<@NotNull @Valid RetailCustomer> order;
    Order<@NotNull RetailCustomer> order;

I think yes as per "If a member, parameter or return value declaration presents an annotated type use, then @Valid is implied for that declaration".

We'd also need a way to disable cascaded validation if a constraint is present.


TeamCity Ppconsul :: Build (Linux) [Build 90](https://136.243.151.173:4433/viewLog.html?buildId=712&buildTypeId=Ppconsul_BuildLinux&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 110 Build time: 00:01:03
### Failed tests
```
null: catalog.services_blocking.service: CHECK: 4544082759ns >= 5s
at /var/TeamCity/buildAgent/work/b87bef787eb069fe/tests/catalog/catalog_tests.cpp:474

```


can't you simply use the class fa-spin?


[Uploading 菜单目录.zip…]()


@anosach-luxoft , yes. event handler is set only for one case - in threaded_socket_connection. There are other use cases for the socket. For streaming, and for "bye socket". In these cases events are not used at all.


ack :D


共通的に使用するメソッドは切り出してクラスを作ってcommonパッケージに入いれておいたほうがいいね。


ack


unack
1. about the commit message, we follow the `tag: Short description` format, where tag is generally the file name we changed :)
2. AFAIK the backslashes here are supposed to escape the underscore in a restructuredtext viewer? did you manually check after making this change that it still works?


Do any more users of your software (besides me) care for [interface stability](https://en.wikipedia.org/wiki/Interface_%28computing%29#Software_interfaces_in_practice "Software interfaces in practice")?


@malirod ok


The test is not working on a release build, because of the optimizations. It seems the compiler is taking a shortcut when using propagate and kills the program without rewinding correctly. Try commenting it out and run `cargo test --release uncatched`. It panics correctly, but cargo test does not detect the panic, before the program fails. This might be worth reporting on https://github.com/rust-lang/rust.


I am not sure about this fix. What does it actually fix? There is just another wrapper function which just passes the argument. Can you give an example in which this solves a problem?


looking good


> It used to work for Reversi.scala

Ah, right.. Makes sense...


@aldavidson the closing bracket is gone too, guessing we want to close this function


`assert (self.use_stdin and file is not None) or (not self.use_stdin and filename is not None)`?


Even when forcing the /system to mount as f2fs by modifying kernel initramfs, the rom wont boot. I think the rom itself doesn't support f2fs on the system partition, so it's pointless to keep this line.


good


[广告启动页面.zip](https://github.com/chengongwen/iOS-component-integration/files/126153/default.zip)


This is the reason behind implementing `#getDisplayName()`:
![java_2016-02-11_11-02-44](https://cloud.githubusercontent.com/assets/3450050/12973753/0a56958e-d0af-11e5-812b-923f84c61e36.png)


@vilchik-elena No unit test fail if I remove this line (same thing for the 2 lines above).


TeamCity Ppconsul :: Build (Linux) [Build 91](https://136.243.151.173:4433/viewLog.html?buildId=714&buildTypeId=Ppconsul_BuildLinux&guest=1) is now running


@a3095194  回答もらってるけど、コメントを編集されると気が付かないので、Add a line note を使って欲しいです。


I think we should merge `apiKey` and `password`.


TeamCity Ppconsul :: Build (Linux) [Build 91](https://136.243.151.173:4433/viewLog.html?buildId=714&buildTypeId=Ppconsul_BuildLinux&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 00:01:04


is your commit description complete enough to make a new build? otherwise, that's a cool commit.  I am just wary of website themes using other jquery ui effects.  I'll ask qsm to test just to be sure


I think the test separation could go in an own commit


unack


lenuta


changes are made


TeamCity Ppconsul :: Build (Linux) [Build 92](https://136.243.151.173:4433/viewLog.html?buildId=716&buildTypeId=Ppconsul_BuildLinux&guest=1) is now running


unack, `_read_stdout` and `_read_stderr` can be easily simplified to one function that takes `sys.stdout` or `sys.stderr` as an argument


[滑动菜单.zip](https://github.com/chengongwen/iOS-component-integration/files/126156/default.zip)


블랙리스트 추가
mno 정보 추가 set 할 수 있도록 변경.


TeamCity Ppconsul :: Build (Linux) [Build 92](https://136.243.151.173:4433/viewLog.html?buildId=716&buildTypeId=Ppconsul_BuildLinux&guest=1) outcome was **UNKNOWN**
Summary: Canceled (Tests passed: 49; exit code 137 (new); number of tests is too small: 49 (new); failed to parse xml report (new)) Build time: 00:00:36


2 comments added


I'm not quite sure that this test does what you think it's testing.

When auto_create_bucket is False (the default), then in the real non-test code, get_bucket(validate=False) is called inside _get_or_create_bucket(), and with validate=False, no actual call ever goes to S3 that could return the S3ResponseError. Thus this is a no-op, but I think in this case you actually /do/ want it it actually check the buckets existence.


It's good practice to return the response status as we working with an API in this case.


the conversion seems simple...


TeamCity Ppconsul :: Build (Linux) [Build 93](https://136.243.151.173:4433/viewLog.html?buildId=718&buildTypeId=Ppconsul_BuildLinux&guest=1) is now running


@vilchik-elena is there a test relying on that line?


good


TeamCity Civic Assembly :: Build backend + frontend [Build 94](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=974&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


TeamCity Civic Assembly :: Build backend + frontend [Build 95](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=976&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


这是我的一个测试的留言


tirsk, heti typo!


Good.... good.....


Ah! So there is an app (Firefox) that relies on the scripts registered in the font's GSUB or GPOS to determine the font's script coverage. Very interesting. I did think about it and even postulate such an approach on the OT list -- that the script records (at least with the dflt langsys) are used as an informative basis, even if the font does not have any features. The extreme case might be monospaced or CJK fonts.


why not `results = self._read(sys.stdout) if ... else ...` - you don't need to read both so you even get a performance benefit plus those variables aren't really helpful


shouldn't it be true ?


унеси в свой локальный)


AFAICS, a `Curl_convert_whcar_to_UTF8()` can return an `argv[]` with a `,` (comma) in it.
Does `parse_args()` and `getparameters()` handle that where it's not expected now? (ref. the *--range* option).


I'm not sure this is the correct strategu totake in this issue


No, it's not the same visual effect


Good idea.  I'll give that a shot.


yep


TeamCity Ppconsul :: Build (Linux) [Build 93](https://136.243.151.173:4433/viewLog.html?buildId=718&buildTypeId=Ppconsul_BuildLinux&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new); number of tests is too small: 0 (new) Build time: 00:01:38


my apologies I will correct it


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


Duplicate
```
notifications:
  slack:
```
in .travis.yml.


can't we set this in a way that it throws an exception if the user doesn't set it?


They don't use it for script coverage detection per se, but to reject
certain scripts that are otherwise considered supported, yes.

On Thu, Feb 11, 2016 at 5:08 PM, Adam Twardoch <notifications@github.com>
wrote:

> Ah! So there is an app (Firefox) that relies on the scripts registered in
> the font's GSUB or GPOS to determine the font's script coverage. Very
> interesting. I did think about it and even postulate such an approach on
> the OT list -- that the script records (at least with the dflt langsys) are
> used as an informative basis, even if the font does not have any features.
> The extreme case might be monospaced or CJK fonts.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/behdad/fonttools/commit/74e629e6004c71a9eeebbb94189b0b910dcac098#commitcomment-16028827>
> .
>

--
behdad
http://behdad.org/


TeamCity Ppconsul :: Build (Linux) [Build 94](https://136.243.151.173:4433/viewLog.html?buildId=720&buildTypeId=Ppconsul_BuildLinux&guest=1) is now running


TeamCity Ppconsul :: Build (Linux) [Build 94](https://136.243.151.173:4433/viewLog.html?buildId=720&buildTypeId=Ppconsul_BuildLinux&guest=1) outcome was **FAILURE**
Summary: Exit code 1; number of tests is too small: 0 Build time: 00:00:03


looking good


!== is the same as != except that in the first no type conversion is needed for the check, and I use it by default. But != should also be fine since we are checking if the object is null


Leading `_` have special meaning in ghc.  what about `aUID`?  or `id_`?  or `auid`?


ok in this line you change the meaning of the `results` variable, that's bad. Please give it a new name or IMO rather give the old meaning a new nae.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (981 of 981 tests passed, 0 test warnings)

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


why add the file contents here? That's really unneeded you have an explicit invocation anyway


see #40 for details


I've tried this and it didn't fix the bug ?
Did this work for you ? Please, a jsfiddle will be welcome.


> Would a dispatch_group_t have any advantage? Cleaner code?

Cleaner code, not really, any advantage, I don't know honestly, was hoping you would/could tell me ;)
I guess I tend to lean toward dispatch_group_t when I'm forcing an async block-based API to sync, and I always think of resource handling with a semaphore.


Nightly passed. Tagging.


unack


handleMsg设计的很奇怪不是？


TeamCity Ppconsul :: Build (Linux) [Build 95](https://136.243.151.173:4433/viewLog.html?buildId=722&buildTypeId=Ppconsul_BuildLinux&guest=1) is now running


TeamCity Ppconsul :: Build (Linux) [Build 95](https://136.243.151.173:4433/viewLog.html?buildId=722&buildTypeId=Ppconsul_BuildLinux&guest=1) outcome was **FAILURE**
Summary: Exit code 1; number of tests is too small: 0 Build time: 00:00:03


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


is `-webkit-animation` necessary? According to caniuse.com it's just for Safari 8 (desktop and mobile), which is about 2% of global use.

Also the spinner PNG can be recompressed to 570 bytes (from the current 1369), or replaced by an animated gif (which may or may not be heavier, but will make the animation unnecessary)


TeamCity Civic Assembly :: Build backend + frontend [Build 95](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=976&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:07:08


webwxsync接受一个handleMsg来处理消息。。好奇怪的设计，消息处理难道不应该分离到webwxsync外吗？


Someone from Microsoft mentioned last year that they were going to use the [meta.dlng](https://developer.apple.com/fonts/TrueType-Reference-Manual/RM06/Chap6meta.html) structure from 🍎's spec to denote the "scripts and languages that the font is intended to use" (so some stub glyphs for certain scripts that exist in some fonts but aren't really considered "good" would be omitted).

They said they'd deploy this in Windows 10 but I looked at the fonts in the shipping W10 and found no "meta" tables.


nope, pybossa doesnt't have a password, but should theses checks reside in the constructor of the classens and thro an exception then?


这个文件和cacheContent(准确来说是cache user info)归类为util


@gunnarmorling no actually they are not equivalent. It took me a while to think this one through but here is what's happening.

`Order<@NotNull RetailCustomer> order;` is equivalent to `@Valid Order<@NotNull RetailCustomer> order;`

But when you put a `@Valid` on a type_use (in your case `RetailCustomer`,) it means that the references to the type parameter `RetailCustomer`  inside `Order` is marked as `@Valid`

```
class Order {

    // equivalent to adding the following two annotations
    @Valid @NotNull
    private RetailCustomer retailCustomer;
```

To come back to your example:

* `Order<@NotNull @Valid RetailCustomer> order;` will validate order properties, validate that the RetailCustomer property is not null and cascade the RetailCustomer property
* `Order<@NotNull RetailCustomer> order;` will validate order properties and validate that the RetailCustomer property is not null (without cascading further that is).

That is actually in the strict spirit of your proposal.


it's OK to add _FORTIFY_SOURCE=2


ack


Nice! I'll refactor to use it.


what he said


Many thanks, it works perfectly now. Finally I have a native FreeBSD 64-bit wipfc executable. :-)


TeamCity Civic Assembly :: Build backend + frontend [Build 94](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=974&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 148 (1 new), passed: 24, ignored: 4 Build time: 00:10:19
### Failed tests
```
Blog categories tests: Should delete blog category new blog category 1455185726716: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


@anosach-luxoft  it's Qt recommended life cycle for the qtcpsocket
see -> http://doc.qt.io/qt-5/qtnetwork-blockingfortuneclient-example.html


This solution also does not require a way to disable cascading AFAIR.


Наследование используем в DAO.
В моделях это может усложнить разработку.


looks good on first sight


This special case is needed in order to convert a property value for a TimeSpan.


That isn't the question. The logrotate settings are not interesting - it is the format of the names that matters.. I just had to install the redis package and start the daemon to find this out:

```
$ sudo systemctl status redis.service
● redis.service - Redis persistent key-value database
   Loaded: loaded (/usr/lib/systemd/system/redis.service; disabled; vendor preset: disabled)
  Drop-In: /etc/systemd/system/redis.service.d
           └─limit.conf
   Active: active (running) since Thu 2016-02-11 10:14:50 GMT; 1s ago
  Process: 23250 ExecStop=/usr/bin/redis-shutdown (code=exited, status=127)
 Main PID: 23566 (redis-server)
   CGroup: /system.slice/redis.service
           └─23566 /usr/bin/redis-server 127.0.0.1:6379
$ ls /var/log/redis/
redis.log
$
```
So redis uses a single `redis.log` by default and rotates by number using logrotate.

Please change this to use the idiom I described earlier:

```python
        if self.get_option("all_logs"):
            self.add_copy_spec_limit("/var/log/redis/redis.log*",
                                     sizelimit=self.limit)
        else:
            self.add_copy_spec_limit("/var/log/redis/redis.log",
                                     sizelimit=self.limit)
```

This will capture the main log by default and all of the rotated logs with `--all-logs`, subject to the normal size limiting rules.


TeamCity Ppconsul :: Build (Linux) [Build 96](https://136.243.151.173:4433/viewLog.html?buildId=724&buildTypeId=Ppconsul_BuildLinux&guest=1) is now running


TeamCity Ppconsul :: Build (Linux) [Build 96](https://136.243.151.173:4433/viewLog.html?buildId=724&buildTypeId=Ppconsul_BuildLinux&guest=1) outcome was **FAILURE**
Summary: Exit code 1; number of tests is too small: 0 Build time: 00:00:02


lgtm, won't be able to reack anyway as of changes to previous commits that'll result in conflicts here


> viewport sizes but this can be mitigated via use of

Should read:

viewport sizes. This can be mitigated via use of


Tagged source map tests pass.
Publishing.


`BindingFlags.IgnoreCase | BindingFlags.Public | BindingFlags.Instance` is used to allow for case insensitivity on the property names.


Hmm. That would mean Closure Library - which we use but only a tiny fraction of - would need to be fully parsed each build (much slower). (In a new project, is CL still recommended for modern JS in evergreen browsers, if not what?)


Интерфейсы и абстрактные классы используем в DAO
В моделях это может усложнить разработку.


Thank you for your comments, @sindresorhus. Soon I will correct! :smiley:


Yes, it's another possibility.


TeamCity Civic Assembly :: Build backend + frontend [Build 96](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=980&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


Seeing that `ui.rb` and `ui_spec.rb` grew quite a bit already and technically will grow for every new "screen" that might be added. Do you think it would be worthwhile having screen/action specific UI classes?


@jsuchal vies co pouzivam to radsej s ```nil``` lebo nepochopitelne 1 z 10 pripadov vyhodi chybu, ze je zle zadefinovane volanie, takto to funguje 10 z 10. Ale nie je to myslene v logike ako ```nil```, len forma/styl zapisu.


Can you explain why you chose `scan` over `match` and why the positive lookahead in the regular expression is needed?


@jsuchal jasne, bolo to v plane, zbytocne volania


@Robotia so it makes no sense, removed, and there are no problems from it. :P


@xmo-odoo I can't use trhobber because the background is not transparent.
Yes I can remove -webkit- if the backend does not support this browsers.
I'll recompressed the image.


OK, then let's put down the hate and ignore the hate.

I must admit I cherish elitism. Whoever does not have the knowledge and pretends to have is whom I hate, and you two aren't, so why should I hate you?

Anyway, thanks @aodzip I'll add PeratX about that


@twardoch Most of the Windows 10 fonts I’m looking at have 'meta.dlng' and 'meta.slng', at least in Windows 10 10240.


@jsuchal ono tam dava len z release date, cize ked mas 10 dokumentov a pridali nejake 2, tak ti tam da len tie 2. Je tam porovnavanie datumu.


@jsuchal good point


Why do you need `isPaused` and `isShared` methods? They are not used in production code. YAGNI violation.


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


Can format entire string using Kotlin's `"${...}"`.


Wouldn't that be too JS specific? I tried to be as agnostic to languages as possible, maybe I can broaden it up a bit by saying it should be possible to convert to JSON and back?


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1365](http://teamcity.monogame.net/viewLog.html?buildId=14765&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 584, ignored: 6 Build time: 00:19:08


Removed useless comments according to what was said previously.


Ok, makes sense. I feel these subtleties will be very tough on users :/


You have to ask for consent to film in a public place? Thats completely f*ked up.


Is this not a duplication of https://github.com/HaxeFoundation/haxe/blob/development/std/haxe/macro/ComplexTypeTools.hx#L50 ?


Исходя из соображений о потокобезопасности не стоит этот метод делать статическим.


Spelling errors make for bad coding errors.


TeamCity Ppconsul :: Build (Linux) [Build 97](https://136.243.151.173:4433/viewLog.html?buildId=725&buildTypeId=Ppconsul_BuildLinux&guest=1) is now running


TeamCity Ppconsul :: Build (Linux) [Build 97](https://136.243.151.173:4433/viewLog.html?buildId=725&buildTypeId=Ppconsul_BuildLinux&guest=1) outcome was **FAILURE**
Summary: Exit code 1; number of tests is too small: 0 Build time: 00:00:02


Пример аннотации можно посмотреть в классе MapObject. Как-то так:
```
@Retention(RetentionPolicy.SOURCE)
@IntDef({ OPEN_NOW_UNKNOWN, OPEN_NOW_YES, OPEN_NOW_NO })
public @interface YesNoUndef {}
...
public Description(String featureType, String region, String distance, String cuisine, int stars, @YesNoUndef int openNow)
```


Перенеси инициализацию коннекшена в метод initConn()


I initially wrote that thinking you'd prefer this for it's readability :D WIll change it to the longer assert


OK, fine!


TeamCity Civic Assembly :: Build backend + frontend [Build 96](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=980&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 148 (1 new), passed: 24, ignored: 4 Build time: 00:07:36
### Failed tests
```
Blog categories tests: Should delete blog category new blog category 1455186449309: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


with gimp I have the same size when I recompress :-/


Nice idea, will do


Исходя из предыдущих комментариев нужно будет убрать static


OK, pull request on its way. Thanks for the heads up.


You could extend `BrooklynAppUnitTestSupport` in place of `setUp` and `tearDown`.


I know you're trolling me Phil but this is actually a good suggestion.


Can we open a new issue for this? I'd like to try and use the pip from https://github.com/pypa/pip/pull/3450 by doing:

```
$ pip install git+git://github.com/esc/pip.git@fix/install_target_c_ext
```

And check if the dependency is included correctly then.


This will not work. For more than one removal the indexes will be off after the first removal. Either remove from the back first or maybe use  void beginBatchedUpdates () to batch these.


help


@dalesit: I think it would be reasonable to assume the configuration for multiple nodes in a cluster be the same which is why in the template I fall-back to using the original single-server settings. I didn't see the harm in allowing per-node configuration though, which is why I added that flexibility. I wouldn't be adverse to simplifying the patch if people think the extra flexibility is unnecessary. As it stands, the ticket for this functionality has been open for over a year and so unless there is further comment from one of mergers I'm unlikely to do anything more with this as it Works For Me.


Assert something like `equals(dynamicRequiredOpenPorts, ImmutableSet.of(9999))` rather than size.


That was used before the project was hosted at Eclipse.
This file was used for our previous sandbox. (I'm not sure this was usable by someone else)


Source: http://opendata-download-metobs.smhi.se/explore/


Surprised I didn't have this feature implemented before, thanks.


@nk0leg, why we have to call this `setEnabled`? It it's really must be called why not to set one in c-tor?


Will the clarification on the name selection be continued with [the bug report "Improve application programming interface design around match management functions"](https://github.com/coccinelle/coccinelle/issues/64 "Better identifiers for Coccilib?")?


Hello,

Could you explain what is the situation where I need the complete `(args)` (with parentheses) selected ?

Thank you


That's true. For example, I guess if we renamed a table at some stage, then migrations which use a model might break because the model would reference the new table. Good thinking :)


I'm pretty sure this won't work. :-)

Btw, `-cvs` and `-snapshot` are the same thing in Emacs 24.4+.


OK, that was meant to be an experiment anyway.


else, instead of exit 0?


ohoooo


Ah, I guess I looked too early! :)


todo esto se hizo en clase ?


Argh, naming is difficult! :smile:


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


Will change to suggested snippet.


You'll have to depend on Melpa's version number.


`$date`는 XE 특유의 `YYYYMMDDHHIISS` 포맷으로 들어오는 것 아닌가요? 그렇다면 `ztime()`은 유지해야 할 것 같은데요...


Seems like workaround. Is it possible to find out more stable solution for this problem?


remove comma?


TeamCity Ppconsul :: Build (Linux) [Build 98](https://136.243.151.173:4433/viewLog.html?buildId=727&buildTypeId=Ppconsul_BuildLinux&guest=1) is now running


TeamCity Ppconsul :: Build (Linux) [Build 98](https://136.243.151.173:4433/viewLog.html?buildId=727&buildTypeId=Ppconsul_BuildLinux&guest=1) outcome was **FAILURE**
Summary: Exit code 1; number of tests is too small: 0 Build time: 00:00:11


@minecrafter Does the 5ms include commands which may execute SQL queries for example? If so, that'd be more than 5ms. Just a thought but might be misunderstanding.


fixed both


Just because of ignorance, I think I tried `match` in pry but I couldn't make it do what I wanted.


Yes, I already thought about splitting the UI and having one UI for every action :+1:
They could be subclasses of a class that has the common behaviour (common methods) so that all of them can share it.


This change seems to cause a bug:
platform: chrome browser
If I edit the daterangepicker input then click outside to close the datepicker, the changed value is not updated in the  'hide.daterangepicker' event's datepicker value.


I ran out of names :scream:


I guess it is an either an `&&` or `||` in the for condition. Otherwise with a simple comma the first is ignored

Clang actually throws this warning
```
warning: inequality comparison result unused [-Wunused-comparison]
    for(; it != intrinsics.end(), otherIt != other.intrinsics.end(); ++it, ++otherIt)
```


Done


fixed (whoops)


(and sorry, i didn't notice it during the PR...)


Home work from 05.02.2016


I cannot reproduce the jitter bug. On chrome it just works. The margins are uneven, though.
![selection_028](https://cloud.githubusercontent.com/assets/1159999/12974551/0d3b7b7e-d0b5-11e5-8f1d-62e9003ee240.png)


Works. Thanks


This looks cool - I always wanted to add a RatFun! I guess division would use the ApproxFun special functions methods, but `(f::Fun)//(g::Fun) = h::RatFun` in analogy to rational promotion of integer division!


Yeah, you're right. Then I guess it's fine as it is and conversion to JSON and back is implied.


I would use $WORKSPACE here (we cd to it in other parts of the script, but this can't rely on that)


OK. Actually I'm not sure `:nth-last-child(n+2)` is equivalent to `:not(:last-child)`. You should verify that.

Anyway, I think the more common way to handle this is to do the following:
```css
table > tr > td::after {
  content: '⠆';
}
table > tr > td:last-child::after {
  content: none;
}
```


Haha, well I clarified it either way: https://github.com/wooorm/unist/pull/6#discussion_r52584723


TeamCity ReOpenLDAP :: devel [Build 234](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1820&buildTypeId=ReOpenLDAP_Devel&guest=1) outcome was **SUCCESS**
Summary: Tests passed Build time: 01:46:08


This feels confusingly named. [Semver](http://semver.org/) is `x.y.z` whereas this is just a single number. I also don't really understand why this is a separate model rather than an attribute on the `ContentItem`? What's the difference between this and the `Version` model?


don't use gimp, image design software don't do heavy compression. Try using something like ajslater/picopt or toy/image_optim (I used [ImageAlpha](https://pngmini.com) and [ImageOptim](https://imageoptim.com) but those are OSX applications)


Yeah, I was a bit unsure I should do that... but then I got a little confidence and it looked OK.


Reverted to old pattern with delimiter added


*"some platforms"* includes Winsock2. Hence my previous message re. this very little helpful `OPENSSL_assert()`. Much better now.


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


lgtm :+1:


Firefox
![selection_029](https://cloud.githubusercontent.com/assets/1159999/12974621/b3ea8532-d0b5-11e5-85ad-6eedd963b855.png)


yo tengo uno la semana que viene ya empezare a subir mis proyectos ya que eh tenido problemas con le disco externo la placa que lleva integrada el disco externo se me jodio cuando se produjo un apagon en la casa jajaja


here's the recompressed 570b spinner:

![spin](https://cloud.githubusercontent.com/assets/7571158/12974639/d0de1d98-d0b5-11e5-9bd0-024ee485edf1.png)


Chrome:
![selection_030](https://cloud.githubusercontent.com/assets/1159999/12974659/ee7cae46-d0b5-11e5-9971-638e89d33ff1.png)


Not sure why this version was changed at all.  Depending on cider 0.10 is enough IIRC.


We could do that too, though I like the semantics of the current implementation: every child in reverse order with index greater than 1. But maybe this is confusing?

Van: Bert Frees [mailto:notifications@github.com]
Verzonden: donderdag 11 februari 2016 11:49
Aan: snaekobbi/pipeline-mod-dedicon <pipeline-mod-dedicon@noreply.github.com>
CC: Davy Kager <DavyKager@dedicon.nl>
Onderwerp: Re: [pipeline-mod-dedicon] Initial supprot for tables. re #11 (247dbbd)

OK. Actually I'm not sure :nth-last-child(n+2) is equivalent to :not(:last-child). You should verify that.

Anyway, I think the more common way to handle this is to do the following:

table > tr > td::after {

  content: '⠆';

}

table > tr > td:last-child::after {

  content: none;

}

—
Reply to this email directly or view it on GitHub<https://github.com/snaekobbi/pipeline-mod-dedicon/commit/247dbbdcb804d0136dc7b6416a5834da88ef91b6#commitcomment-16029483>.

DISCLAIMER:
De informatie verzonden met dit e-mail bericht is uitsluitend bestemd voor de geadresseerde. Indien u niet de beoogde geadresseerde bent, verzoeken wij u vriendelijk dit aan de afzender te melden (of via: info@dedicon.nl<mailto:info@dedicon.nl>) en het origineel en eventuele kopieën te verwijderen.

The information sent in this e-mail is solely intended for the individual or company to whom it is addressed. If you received this message in error, please notify the sender immediately (or mail to info@dedicon.nl<mailto:info@dedicon.nl>) and delete the original message and possible copies.


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 107](https://teamcity.labkey.org/viewLog.html?buildId=302583&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 32 (17 new), passed: 260, muted: 1 Build time: 01:49:22
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


Indentation


(minor) Obsolete space before `(`.


Addresses #3830


would this be better?
```diff
-dontShare = hasattr(self, 'DontShare')
+dontShare = getattr(self, 'DontShare', False)
```


@mcollina Good point, I'll add a mapping here


already fixed


Wrong parameter order: http://php.net/strtr


Now I have:

```
Caused by: org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'tomcat.server' defined in class path resource [jee-container.xml]: Invocation of init method failed; nested exception is java.lang.NullPointerException
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.initializeBean(AbstractAutowireCapableBeanFactory.java:1578)
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.doCreateBean(AbstractAutowireCapableBeanFactory.java:545)
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBean(AbstractAutowireCapableBeanFactory.java:482)
	at org.springframework.beans.factory.support.AbstractBeanFactory$1.getObject(AbstractBeanFactory.java:306)
	at org.springframework.beans.factory.support.DefaultSingletonBeanRegistry.getSingleton(DefaultSingletonBeanRegistry.java:230)
	at org.springframework.beans.factory.support.AbstractBeanFactory.doGetBean(AbstractBeanFactory.java:302)
	at org.springframework.beans.factory.support.AbstractBeanFactory.getBean(AbstractBeanFactory.java:220)
	at org.springframework.beans.factory.support.DefaultListableBeanFactory.getBean(DefaultListableBeanFactory.java:351)
	at org.springframework.beans.factory.support.DefaultListableBeanFactory.getBean(DefaultListableBeanFactory.java:332)
	at org.springframework.context.support.AbstractApplicationContext.getBean(AbstractApplicationContext.java:1066)
	at org.red5.server.ContextLoader.afterPropertiesSet(ContextLoader.java:113)
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.invokeInitMethods(AbstractAutowireCapableBeanFactory.java:1637)
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.initializeBean(AbstractAutowireCapableBeanFactory.java:1574)
	... 16 more
Caused by: java.lang.NullPointerException
	at org.apache.catalina.startup.Tomcat.getLoggerName(Tomcat.java:717)
	at org.apache.catalina.startup.Tomcat.silence(Tomcat.java:695)
	at org.apache.catalina.startup.Tomcat.addWebapp(Tomcat.java:541)
	at org.apache.catalina.startup.Tomcat.addWebapp(Tomcat.java:522)
	at org.red5.server.tomcat.TomcatLoader.addContext(TomcatLoader.java:203)
	at org.red5.server.tomcat.TomcatLoader.addContext(TomcatLoader.java:185)
	at org.red5.server.tomcat.TomcatLoader.start(TomcatLoader.java:300)
	at org.red5.server.tomcat.TomcatLoader.afterPropertiesSet(TomcatLoader.java:171)
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.invokeInitMethods(AbstractAutowireCapableBeanFactory.java:1637)
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.initializeBean(AbstractAutowireCapableBeanFactory.java:1574)
	... 28 more

```

:(


typo: `scrol` -> `scroll`


TeamCity Ppconsul :: Build (Linux) [Build 99](https://136.243.151.173:4433/viewLog.html?buildId=728&buildTypeId=Ppconsul_BuildLinux&guest=1) is now running


thanks :)


Constructors shouldn't know anything about the configuration.


I don't mind the duplication per se (we have several of those between Context and the various Tools classes) but both should use the same implementation.

However, I don't think this new implementation has any advantage over the ComplexTypeTools one. Also the naming is questionable, "resolve" usually has a String -> Something signature. Overall I'm in favor of reverting this to avoid API confusion.


так, вытащи эту кнопку из форму. она просто элемент по нажатию на который будет появляться форма)


TeamCity Ppconsul :: Build (Linux) [Build 99](https://136.243.151.173:4433/viewLog.html?buildId=728&buildTypeId=Ppconsul_BuildLinux&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 00:01:04


added


In general: why is there a limitation on possible values?

If limitation is necessary then increase origin coordinates to negative values too.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1366](http://teamcity.monogame.net/viewLog.html?buildId=14769&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 584, ignored: 6 Build time: 00:19:34


shouldn't it be 300000


было бы неплохо классы для li задать или для ul. проще будет прописывать стили.


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


The simplified version looks like it's working fine.


One question: do we know that this is complete? I.e. is it ever possible for an administrator to configure `libvirt` in such a way that even read only access requires authentication?

If that can happen then for now it may be worth lowering the timeout value used for this command.


Reverted to previous code as in if condition already used preg_replace and we don't need to replace else code

missed if block :)


Airbnb style uses trailing commas for all items, similar to PEP8. It means you can add new things and not worry about accidentally breaking it for lack of a comma


AFAICT this class is not deprecated, but renamed


@vilchik-elena Shouldn't we also update `UndefinedShadowingCheck` in the same way?


It confused me a bit, but maybe that is just because I'm familiar enough with that approach. But now having looked at https://www.w3.org/TR/css3-selectors/#nth-last-child-pseudo I think it should work indeed. I was looking at https://www.w3.org/wiki/CSS/Selectors/pseudo-classes/:nth-last-child which has a different (wrong) definition.


@kijin  `strtotime()` 함수도 그런형식을 인식하던데요?


:+1: Makes sense so far.


This method signature is same as the implementation in .OSX.cs .To make it compile.


Done


채팅방 목록에 사용할 Adapter 만들고 값 넣는 데까지 구현하고 테스트.
변수 이름 변경 필요. 아직 Merge하지 않는 것을 추천.


could just add an unique class to the body like `csDev` and then just have your css target that instead of the generic body tag which will affect everyone


It's not that I'm advocating `match` over `scan` but I would like you to be aware of the differences. Even more so about the positive lookahead (which seems overkill here).


Alright, good to know! :+1:


I agree that you should not remove packages.

And the change history should have some indication of the severity of the fix.  You thought it was a bad enough bug to remove the package, but the changelog is a small one-sentence mention.  Why not, "This means you should definitely not use 0.18.1, it's very broken, and we've removed the packages from PyPI."  I came looking for a reason why the packages might have been removed, and didn't connect that mild mention to the problem.


**Nitpick - Code Quality**
What's the idea behind naming the variable `command` here? The method `AuthenticateUser.call` returns a user's token.
You should always use appropriate variable names so that it's clear to someone else reading the code base to understand what's going on in a line of code.


I’ve tested with with tables of 4 columns wide including some empty cells and it appears to work. jStyleParser also has a small test for it I think, but not for this exact scenario.

Van: Bert Frees [mailto:notifications@github.com]
Verzonden: donderdag 11 februari 2016 12:02
Aan: snaekobbi/pipeline-mod-dedicon <pipeline-mod-dedicon@noreply.github.com>
CC: Davy Kager <DavyKager@dedicon.nl>
Onderwerp: Re: [pipeline-mod-dedicon] Initial supprot for tables. re #11 (247dbbd)

It confused me a bit, but maybe that is just because I'm familiar enough with that approach. But now having looked at https://www.w3.org/TR/css3-selectors/#nth-last-child-pseudo I think it should work indeed. I was looking at https://www.w3.org/wiki/CSS/Selectors/pseudo-classes/:nth-last-child which has a different (wrong) definition.

—
Reply to this email directly or view it on GitHub<https://github.com/snaekobbi/pipeline-mod-dedicon/commit/247dbbdcb804d0136dc7b6416a5834da88ef91b6#commitcomment-16029690>.

DISCLAIMER:
De informatie verzonden met dit e-mail bericht is uitsluitend bestemd voor de geadresseerde. Indien u niet de beoogde geadresseerde bent, verzoeken wij u vriendelijk dit aan de afzender te melden (of via: info@dedicon.nl<mailto:info@dedicon.nl>) en het origineel en eventuele kopieën te verwijderen.

The information sent in this e-mail is solely intended for the individual or company to whom it is addressed. If you received this message in error, please notify the sender immediately (or mail to info@dedicon.nl<mailto:info@dedicon.nl>) and delete the original message and possible copies.


Ah, nuts - was sure I sorted those...

On 11 February 2016 at 08:20, Seb James <notifications@github.com> wrote:

> Sorted in commit 93bfe05
> <https://github.com/SpineML/SpineML_2_BRAHMS/commit/93bfe05ad7c8e0dc3b32f79586379aac2e173109>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SpineML/SpineML_2_BRAHMS/commit/92fc74b09a986c390d5bf230f40a3246a861df56#commitcomment-16026807>
> .
>

--
Alex Cope
Research Associate
Behavioural and Evolutionary Theory Lab
Department of Computer Science, University of Sheffield
www.alexcope.co.uk


Thanks

On 11 February 2016 at 11:06, Alex Cope <a.cope@sheffield.ac.uk> wrote:

> Ah, nuts - was sure I sorted those...
>
> On 11 February 2016 at 08:20, Seb James <notifications@github.com> wrote:
>
>> Sorted in commit 93bfe05
>> <https://github.com/SpineML/SpineML_2_BRAHMS/commit/93bfe05ad7c8e0dc3b32f79586379aac2e173109>
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/SpineML/SpineML_2_BRAHMS/commit/92fc74b09a986c390d5bf230f40a3246a861df56#commitcomment-16026807>
>> .
>>
>
>
>
> --
> Alex Cope
> Research Associate
> Behavioural and Evolutionary Theory Lab
> Department of Computer Science, University of Sheffield
> www.alexcope.co.uk
>

--
Alex Cope
Research Associate
Behavioural and Evolutionary Theory Lab
Department of Computer Science, University of Sheffield
www.alexcope.co.uk


Hey remember to set your high thread test priority to 1 - don't want to be stealing your thunder!


Yes. It is fixed


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


**Nitpick - Code Quality**
There should be a new line after this line.


Unfortunately due to small api changes we can not depend on 0.10.x anymore. We have to depend on 0.11.x


Do we have something to restart sync if the initial sync fails (e.g. user is offline and comes online during the session)?


space typo?


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 105](https://teamcity.labkey.org/viewLog.html?buildId=302585&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **SUCCESS**
Summary: Tests passed: 218 Build time: 00:17:35


if  InitIntegrity and  AcceptIntegrity  are set  and not Confidentiality. only the flags in this if block  will be set  and not the one in below if block. Why would that be considered an error ?  What error you are referring too.. ?


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


@pynicolas indeed! thanks!


Which is not ideal as we it will be PITA to release clj-refactor before CIDER 0.11


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


review of code


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1533 of 1533 tests passed, 0 test warnings)

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


removed limitations for origin coordinates. i am keeping them for all other values because we need non-negative values there. and if you require a min-value you also have to give all the other params for QValidator.


yyyy


Looks good


@sapegin, FYI:

![image](https://cloud.githubusercontent.com/assets/3027415/12975014/caa02374-d0b8-11e5-9054-8d0e828823c8.png)


We should try it!


인식은 합니다만, 서버의 표준시간대와 라이믹스의 `internal_timezone`이 다른 경우 `strtotime()`은 서버의 표준시간대를 따르기 때문에 잘못된 결과를 반환할 수 있습니다. `ztime()`은 항상 라이믹스의 `internal_timezone`을 따릅니다. (최대한 똑같이 맞춰주려고는 하지만, 일부 시간대에서는 안 맞을 수도 있어요. 예를 들어 `internal_timezone`이 GMT+05:30인 경우...)


**Nitpick - Code Quality**
When breaking an argument list, each argument should be place on different lines.

In this case, you could have something like this
```
render json: {
  feedback: "Item not created"
  },
  status: :unprocessable_entity
```


 @thezoggy how many body tags do you set in a single HTML document?
An inline style is the correct place to put such important style param. If its part of class it can be overwritten while if you add it on debug-bar loading as inline style it can't be overwritten by anything (except other JS..) and at the same time its dynamic (if the toolbar has dynamic height).


why no switch here?


These Environment variables need to be documented in the readme. They also need to be prefixed with 'EQ_' just so they are clearly namespaced.


I think we need to tell our input operator which type we expect, i.e., `translateToPlan(config, Some(inputTypeInfo))`, to ensure we receive a `Row` type.


All fixed now ;) Reverted in https://github.com/ntwb/stylelint-config-wordpress/commit/5f77cf747e91f85431d2ee7c35a063601ac69fee


ack


if we can't have this all on one line, can you instead do it where the dict is split over the lines, e.g.
```
method({
  key: value,
}, {
  otherKey: value
}
```
This is how we do it in most of the other TC code bases iirc


Why would it clash ?


The idea of a semantic version as a thing is bigger than just the implementation used for ruby gems, but I accept that there can be some confusion there.

It's not an attribute on the content item for the same reason that location, translation, and state aren't, to stop the content item model ballooning as we add more concerns.  We'll write up the approach soon and present it at a show & tell.

The difference between it and the `Version` model is that the `Version` is incremented by system actions and is used for optimistic locking, while the `SemanticVersion` is incremented when a user makes a conceptual change (like redrafting a document) and is presented visually to the user as the version number.

`Version` will shortly be renamed `LockVersion` to reduce this confusion.  I'm open to discussions on the name of `SemanticVersion`.


why not make it simpler: ```while (line = r.readLine() != null) {```?


@vilchik-elena `enterScope` calls `getScopeFor` which could be improved to avoid iterating over all scopes.


crap thanks


Good stuff :)


thanks for fixing the commented out code


Closes #32


Gotcha, just a wrong copy-paste from the above `str_replace_array` docblock.


Octree (DeadParrot) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


say you have
```

pass

```

if you first correct spacing, then remove the unusd pass it'll end up wrong, not if you do it the other way round


use NIO here?


this looks like an 8-space indent on top of a 4?  can we go with exclusively 4?


Thanks, but already did this myself.


Check indentation/soft-space setting in the editor is using 2 soft-spaces per indent. (Line 3 has a bit too much indentation)


Glad we could resolve this :-)


Ah, I see


ack


Reverted regex changes and Z&H portal changes
https://github.com/practiceprovider/openemr/commit/28ed081e30690a7184a9da63180146404186f850


```Files.newBufferedReader```?


We're using `--max-line-length` in autopep8 - that shouldnt happen


Should it be ```Files.copy```?


Great idea :) did it help?


@joojic said after this commit, ActorSDK is failed to build.
I suggest refactor or new feature be done on feature/xxx branches, keeping master and release branches stable.
A good git collaborating workflow tutorial: https://www.atlassian.com/git/tutorials/comparing-workflows/forking-workflow


Should it be ```Files.newBufferedReader```?


Could you share a use case. Where do use this function in the source of your app. Some sample code will be appreciated. Thanks


@hungerburg Can you be a bit more precise about what it doesn't like. If I just compile and run `new URI("xmldb:exist:///db")` then all is fine. Is this something specific to FOP perhaps?


The semver also occupies one of the four pillars of uniqueness for a content item, as you can have multiple superseded versions with the same locale and at the same location.


Why in the end you remove the other events?


it happens


Looks good to me.  If we do something like this again, having a single commit that's exclusively indentation changes and *nothing* else would be really appreciated.  It makes the indentation changes trivial to review.


@kijin 어자피 $_SERVER['REQUEST_TIME'] 얘도 서버 표준시간대를 따릅니다. `getTimeGap()` 에서는 그 시간차이만 얻으면 될꺼라고 생각해요.


Could you share a snippet where and how you use this?


change the for to :

for c.running.Load() == true {
}


autopep8 is based on different rules, some it can fix, some it can't always fix. So if it decides to reorder a line and then fails to fix the line length this happens


Maybe we want to file a bug in autopep8 ?

On Thu, Feb 11, 2016 at 4:53 PM, Lasse Schuirmann <notifications@github.com>
wrote:

> it happens
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/634ff63cc0ecfe95af2afa4b0186949af9c90dd5#commitcomment-16030102>
> .
>


Ah, so you want to raise an error if autopep8 cannot fix that - got it (I thought autopep8 rearranges lines so its too long sometimes)

ack


I can revert the api change. Or you can tweak your code to understand both "" and nil.


Remove comments.


I've been looking for solution for this. The problem is that we are working on live element and outer middleellipsis-container doesn't have fixed width. It can be stretched a bit with the text sometimes. With trivial algorithm when width is decreasing by one it works fine, but with binary search when width of inner element changes heavly, outer element also is resized.

We have:
Inner : 500px
Outer : 300px

It will try to resize inner to 250px, Outer will change to 280px. Then it will try to adjust to new size. This is the problem.


Remove ongoing work.


@sils1297 dont you think SpaceConsistency should be called after  pep8 too ?
I mean if pep8 can fix it, why let spaceconsistency ask me to fix it ?


my zip file


> Why? It's working, isn't it?

I didn't test this on OS X yet but I assume it's working. However it's impossible to link with the particular version of static library.
It's a minor flaw but usage of framework on OS X is now forced. In fact, it's the same dynamic library however with hard-coded directory structure.


Good catch, this was for testing only and was commented out in earlier commits.


that doesn't really matter I think we could even take spaceconsistencybear out assuming autopep8 gets that done correctly


no problem :+1:
One more thing
On first Boot , Do you have the Dexopt window getting stuck on "preparing Contacts Storage" ? for about 10 mins and then it boots


I've tried to set min-width on container to 100% but it doesn't work. Width is already set to 100%. CSS is really painful.


I think move the whole struct into the c file and create getter functions, would be better, and cleaner


Hi Olmo,

Is there a change in the way Lite(s) are retrieved from the database ?

I have the feeling that when using a Database.Query on an entity the related Lites are retrieved without the ToStr presentation. When I debug, and for instance the name attribute is supposed to be the ToString presentation, the VS debugger give “Name is not defined” as its value and ToStr = <null>

Regards,

Marcel.

Van: Olmo [mailto:notifications@github.com]
Verzonden: zaterdag 15 augustus 2015 23:01
Aan: signumsoftware/framework
Onderwerp: Re: [framework] add missing [ExpressionField] (a7e3699)

Expression method changes (mandatory ExpressionField)

I'm glad to another another important step in making Signum Framework a code-aware library<https://channel9.msdn.com/Events/dotnetConf/2015/NET-Compiler-Platform-Roslyn-Analyzers-and-the-Rise-of-Code-Aware-Libraries> and alleviating one pain points of working with Signum.Framework. Unfortunately this one is not easy to explain, so pay attention!

Expression field are an old feature that allow you to define the body of a method or property using a static field of type Expression<T> so the LINQ provider can understand it and translate it to SQL.

The OLD Rules

The old rules to define an expression field for :

  *   You define a field with the same name than the property of method but with "Expression" at the end.

static Expression<Func<AlbumEntity, string>> ToStringExpression = a => a.Name;

public override string ToString()

{

        return ToStringExpression.Evaluate(this);

}

  *   Or you reference the field manually using [ExpressionField("CustomField")] attribute.

static Expression<Func<AlbumEntity, string>> ToStringExpression2 = a => a.Name;

[ExpressionField("ToStringExpression2")]

public override string ToString()

{

        return ToStringExpression2.Evaluate(this);

}

Problems

This syntax is long and complicated, and users struggle to be able to write the Expression<T> declaration, understand the magic convention, the relationship between the member and the static field types amd keep them synchronized, avoid misspellings, finding the ExpressionFieldAttribute in the rare cases when it's needed (overloads), etc...

Solutions

Unfortunately we can not simplify the syntax as we have done in other cases, but we can improve tooling around:

  *   Using Signum.MSBuildTask we can explore the (simple) body of the member and find the usage of the static field (in the example ToStringExpression2), then rewrite the ExpressionFieldAttribute in the top with the right name. This way we can keep it strongly typed and avoid redundancy. [:+1:]
  *   Using Signum.Analyzer we can support the user detecting error cases (missing field evaluation, wrong number or parameters, wrong order, wrong types, etc...) or even Auto-generating the missing field when [ExpressionField] is writing in the top [:+1:] [:+1:]

The NEW Rules

The new rules to define an expression field for :

  *   You ALWAYS need to declare ExpressionField attribute in your member, Signum,MsBuildTask will write the field name for you looking into the member body Evaluate.

static Expression<Func<AlbumEntity, string>> MyToStringExpression = a => a.Name;

[ExpressionField]

public override string ToString()

{

        return MyToStringExpression.Evaluate(this);

}

  *   If the member requires an special body, then a mandatory field name is necessary, but will be checked by Signum.Analyzer.

static Expression<Func<AlbumEntity, string>> MyToStringExpression = a => a.Name;

[ExpressionField("MyToStringExpression")]

public override string ToString()

{

        return a.Name ?? "Fill name please";

}

Note that ToStringExpression is now just a human convention, not understood by the framework in any way.

How to update

  1.  In MyApp.Logic.csproj include Signum.MsBuildTask (as we did before in the MyApp.Entities.csproj projets)

   <UsingTask TaskName="SignumTask" AssemblyFile="..\Framework\Signum.MSBuildTask\Binaries\Signum.MSBuildTask.dll" />

   <Target Name="AfterCompile" DependsOnTargets="CoreCompile">

     <SignumTask Assembly="@(IntermediateAssembly)" References="@(ReferencePath)" KeyFile="$(KeyOriginatorFile)" />

   </Target>

 </Project>

1.      Also Right-Click in your MyApp.Entiites.csproj, select Add Analyzer and pick \Signum.Analyzer\Binaries\Signum.Analyzer.dll.

2.      Remove all the explicit ExpresionFields names that you have (now are probably automatic, otherwise you should get compile-time warnings)

FIND:    ExpressionField\(".*"\)

REPLACE: ExpressionField

  1.  Add the missing ExpressionField<now%20is%20mandatory>

FIND:    (static (readonly )?Expression([^;]|[\r])*;)(\s*)(public|internal|private|\[Unit)

REPLACE: $1$4[ExpressionField]$4$5

Conclusion

Even if this change was not able to simplify the pattern (as the previous changes did) I hope at least will simplify the usage of it and allow caching errors earlier.

Enjoy!

—
Reply to this email directly or view it on GitHub<https://github.com/signumsoftware/framework/commit/a7e369966d99dc8a6fc36eb7a0b7227d50223f30#commitcomment-12725175>.


Can `@Ignore` be removed since with rebased on Timo's codeGen commit?


Z&H portal changes https://github.com/practiceprovider/openemr/commit/cd1a203510d46d0e3a860c87cb6e620772fb44ce


I do not. It happens instantly


This causes a:
```
crypto/bn/rsaz_exp.c:47:1: error: expected '=', ',', ';', 'asm' or '__attribute__' at end of input
 NON_EMPTY_TRANSLATION_UNIT
 ^
```

Hence this is needed:
```
--- a/crypto/bn/rsaz_exp.c 2016-02-10 10:11:55
+++ b/crypto/bn/rsaz_exp.c 2016-02-11 12:32:09
@@ -40,6 +40,7 @@
 * (2) University of Haifa, Israel                                            *
 *****************************************************************************/

+#include <openssl/opensslconf.h>
 #include "rsaz_exp.h"

 #ifndef RSAZ_ENABLED
```


Can be removed?


 Weird Most CM based Roms have this but none have been able to find the solution
only workaround is Hitting power button to force reboot
Anyways thanks


https://github.com/actorapp/actor-platform/commit/11aa4b2e6b4deff1409ebd4473f65b1e917d796c - I think this merge is fixed this problem, but it not in master branch


Now everything looks OK. Please add a doc string to the module (after the license) and commit to UIA.


The code section was simplified and its duplicated due to the debugging; before we are done, it will be refactored into its own method. I was thinking a do-while would probably work here as well.


I am a gitter fan and i don't understand IRc


The unit test is updated and packet 7 or 8 breaks.


В библиотеке есть более лучшая функция: hasItem


This problem is also present on Windows.


directory /var/tmp is listed twice.


I see a lot of hashmap operations here whose return values are not checked. Is this really safe? If it is, this should maybe get a comment (or even an explicit cast to (void)).

But I have serious trouble grokking whats going on here...

I think it would help if these hashmap operations would be split out into functions of their own?

Also, opening code blocks with {} in the middle of code is something we didn't typically do so far...


Upgrading a 5.1 server is impossible though ? Or yes ? Or we do not know ?


I should have known that they'd change something; I'll fix this asap.


so much WOW
so GITHUB


> Thomas Altenburger (@mrhelmut on github) - already part of the OpenTK project for a long time
>
> Andy Korth (@andykorth on github) - already part of the OpenTK project for a long time
>
> Christopher Lees (@leezer3 on github) - actually has an outstanding Pull Request to improve OpenTK right now
> http://www.bvecornwall.co.uk
>
> Benjamin Ward (@WardBenjamin on github) - actually has an outstanding Pull Request to improve OpenTK right now
> ward.programm3r@gmail.com
> http://voxxongames.github.io
>
> Paul Scharf (@amulware on github) - actually writing a new platform to run on top of OpenTK
>
> Adam (ilexp on github) - very enthusiastic OpenTK supporter
> http://www.adamslair.net
>
> Harry (cra0zy on github) - very enthusiastic OpenTK supporter
> cra0zy@gmail.com
>
> @wcdeich4 (myself), math programmer who uses OpenTK - I don't know if I'm worthy of being a full admin, but if you give me Pull Request approval permission, I could help out.

Did you ask everyone if they wanted to be maintainers/if they had time?


Remind of global permission : some user can read all survey.

````
if(!Permission::model()->hasGlobalPermission('survey', 'read'))
{
//
}
````

I don't remind if it's surveys or survey


@mhugo I would like to draw your attention to [Redmine #14281](https://hub.qgis.org/issues/14281) for a regression introduced by this commit.


# Rubocop:
——————
rtpfe1@agent-lab11-ws:~/smigopal/feb_interface/cisco-network-node-utils$ rubocop tests/test_interface.rb
warning: parser/current is loading parser/ruby22, which recognizes
warning: 2.2.3-compliant syntax, but you are running 2.2.1.
warning: please see https://github.com/whitequark/parser#compatibility-with-ruby-mri.
Inspecting 1 file
.

1 file inspected, no offenses detected
rtpfe1@agent-lab11-ws:~/smigopal/feb_interface/cisco-network-node-utils$ rubocop lib/cisco_node_utils/overlay_global.rb
warning: parser/current is loading parser/ruby22, which recognizes
warning: 2.2.3-compliant syntax, but you are running 2.2.1.
warning: please see https://github.com/whitequark/parser#compatibility-with-ruby-mri.
Inspecting 1 file
.

1 file inspected, no offenses detected
rtpfe1@agent-lab11-ws:~/smigopal/feb_interface/cisco-network-node-utils$ rubocop lib/cisco_node_utils/interface.rb
warning: parser/current is loading parser/ruby22, which recognizes
warning: 2.2.3-compliant syntax, but you are running 2.2.1.
warning: please see https://github.com/whitequark/parser#compatibility-with-ruby-mri.
Inspecting 1 file
.

1 file inspected, no offenses detected
rtpfe1@agent-lab11-ws:~/smigopal/feb_interface/cisco-network-node-utils$

# Minitests:
——————

# N9K I2 platform:
——————————
rtpfe1@agent-lab11-ws:~/smigopal/feb_interface/cisco-network-node-utils$ ruby tests/test_interface.rb -v -n test_interface_fabric_forwarding_anycast_gateway -- 10.122.197.125 admin admin
Run options: -v -n test_interface_fabric_forwarding_anycast_gateway -- --seed 7445

Running:

Node under test:
  - name  - agent-lab11-nx
  - type  - N9K-NXOSV
  - image - bootflash:///nxos.7.0.3.I2.1.bin

conf : ["interface vlan11"]
conf : ["feature fabric forwarding"]
conf : ["nv overlay evpn"]
conf : [" fabric forwarding anycast-gateway-mac 1223.3445.5668"]
conf : ["interface vlan11", "no fabric forwarding mode anycast-gateway"]
conf : ["interface vlan11", " fabric forwarding mode anycast-gateway"]
conf : ["interface vlan11", "no fabric forwarding mode anycast-gateway"]
conf : ["feature fabric forwarding"]
conf : ["nv overlay evpn"]
conf : ["no fabric forwarding anycast-gateway-mac "]
conf : ["interface vlan11", " fabric forwarding mode anycast-gateway"]
conf : ["interface vlan11", " fabric forwarding mode anycast-gateway"]
conf : ["interface ethernet1/1"]
conf : ["interface ethernet1/1", " fabric forwarding mode anycast-gateway"]
TestInterface#test_interface_fabric_forwarding_anycast_gateway = 7.90 s = .

Finished in 7.899277s, 0.1266 runs/s, 0.7596 assertions/s.

1 runs, 6 assertions, 0 failures, 0 errors, 0 skips
Coverage report generated for MiniTest to /home/rtpfe1/smigopal/feb_interface/cisco-network-node-utils/coverage. 678 / 1282 LOC (52.89%) covered.
rtpfe1@agent-lab11-ws:~/smigopal/feb_interface/cisco-network-node-utils$

# N9K I3 platform:
——————————
rtpfe1@agent-lab11-ws:~/smigopal/feb_interface/cisco-network-node-utils$ ruby tests/test_interface.rb -v -n test_interface_fabric_forwarding_anycast_gateway -- 10.122.197.222 admin admin
Run options: -v -n test_interface_fabric_forwarding_anycast_gateway -- --seed 13486

 Running:

Node under test:
  - name  - dt-n9k5-1
  - type  - N9K-C9396PX
  - image - bootflash:///nxos.7.0.3.I3.1.bin

conf : ["interface vlan11"]
conf : ["feature fabric forwarding"]
conf : ["nv overlay evpn"]
conf : [" fabric forwarding anycast-gateway-mac 1223.3445.5668"]
conf : ["interface vlan11", "no fabric forwarding mode anycast-gateway"]
conf : ["interface vlan11", " fabric forwarding mode anycast-gateway"]
conf : ["interface vlan11", "no fabric forwarding mode anycast-gateway"]
conf : ["feature fabric forwarding"]
conf : ["nv overlay evpn"]
conf : ["no fabric forwarding anycast-gateway-mac "]
conf : ["interface vlan11", " fabric forwarding mode anycast-gateway"]
conf : ["interface vlan11", " fabric forwarding mode anycast-gateway"]
conf : ["interface ethernet1/1"]
conf : ["interface ethernet1/1", " fabric forwarding mode anycast-gateway"]
TestInterface#test_interface_fabric_forwarding_anycast_gateway = 15.55 s = .

Finished in 15.548825s, 0.0643 runs/s, 0.3859 assertions/s.

1 runs, 6 assertions, 0 failures, 0 errors, 0 skips
Coverage report generated for MiniTest to /home/rtpfe1/smigopal/feb_interface/cisco-network-node-utils/coverage. 678 / 1282 LOC (52.89%) covered.
rtpfe1@agent-lab11-ws:~/smigopal/feb_interface/cisco-network-node-utils$


Hi.
By changing `LINK_PUBLIC` to `LINK_PRIVATE` again, you unfortunately broke (part of) what 282a7977809409818ac904f6fdcf9acd032606cf fixed


Fixes #1.


Nerozumiem.


@pixeltrix What's the question? We should probably leave this as-is for now, then change it next story when we pull in territory.openregister.org and uk.openregister.org


There are no changes. Can you paste the ToString of the entity you're trying to retrieve Lites from?


Octree (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (993 of 993 tests passed, 0 test warnings)

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


Thanks for the info, guys :-)


Good thing I use AOSP ;-)  Maybe it's a SQLite error?


`node index.js` won't work as its ES6 file. It should ideally be `gulp serve` or `node dist\index.js` but in the latter case developer should know that gulp build needs to be executed first. So lets stick to `gulp serve`.


Can anybody help with broken build? I take a look at the log, but I have no experience to find out what I did wrong.


Hello, this fix actually breaks the project when I use the demo project from [http://www.dhtmlx.com/x/download/regular/dhtmlxConnector/laravel_dhtmlx.zip](url) and update the dependencies with 'composer update'.


@flavour This seems to work perfectly fine for me? Refreshes layer automatically too


mmap_cache_unref() like most our destructors is happy to take a NULL ptr, and becomes a NOP then. No need to check f->mmap first here...


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


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


Updated, I'm not sure why sepolicy manpage generated it twice :-/ .


a bug in IE, document.write does not work in context:
http://joji.me/en-us/blog/how-to-resolve-sec7111-https-security-is-compromised-by-null-error-in-ie11


Yep, huge table lines... But my suggestion was indeed to use the literal values directly and get rid of the macros as they aren't used elsewhere. I'm in favor of using literal values instead of macros here because we're defining a table, and I think that way is easy to identify the meaning of the values.


You need everything in getCrossPlatformDependencies() unless its truly not available (like libQt5WebChannel.so.5), but you should start again with getLinuxOnlyDependencies() and not assume that you need the things that I needed when building on CentOS.

In case you weren't aware, you don't need to run "make portable" just to re-run the "copy-libs" file. Once you have run "make portable" once and have a "MuseScore.AppDir" inside "build.release" you can do:
<pre>
cd build.release/MuseScore.AppDir
./../../build/Linux+BSD/portable/copy-libs .

# Test it:
./AppRun
./AppRun check-depends

# Make some changes to copy-libs and then reload:
rm -r lib
./../../build/Linux+BSD/portable/copy-libs .

# Test it again:
./AppRun
./AppRun check-depends

# Make some changes to copy-libs and then reload, etc...
</pre>


query_params 를 보고 language 를 추출하는 부분 불필요.. 그냥 header Accept-Language 를 통해 알아낼 수 있음.


Ok, here we go.
First, get rid of our last attempt.
Second, in organ_stump.dm add `var/origin` to the declaration and in new() add `origin = limb.type` underneath `wounds = limb.wounds`
now add the following here
`for (var/obj/item/organ/external/stump/stump in target.organs)`
   `if (stump.origin == new_limb.type)`
      `stump.removed(target)`
      `break`

If that doesn't work I'll be ready to cut my own stump off! (Ironically, I do have one lol)


Would you like to make a PR for this.


front 에서 그렇게 처리해줌.


Can we store this in `mbgl/util/constants.hpp`?


이 부분을 통해 locale 를 django 에서 인식함. _() text 등도 아마 이부분과 관련이 있을지도?


Can we check if the packages (`puppet=3.8*` or `puppet-agent`) are installed before we do this stuff? I'd rather not wait for all this apt-get stuff if I forget to do a `--provision-with=puppet` or I just want to update the modules.


Ditto.


hahha :dancer:


LGTM


yes. this  is non- padded encryption . Added assert . and more comments


This is much better than a list of IDs as it will show new alerts now, which is the critical thing  :)
However it still requires a server side refresh to hide newly-expired alerts.
How could it not?
JS only sees a string here...which doesn't change.
So every 900s the exact same request is sent to the server about showing all alerts which didn't expire at an abritrary past time.

Anyway, this is a major step forward from the previous solution & it is then up to JS to finish the job by updating the timestamp in the URL. Since this is much lower priority I would be happy for this to be an @ToDo


Seems the apply button from CraueConfigBundle's panel can't be translated. :(


Thanks, this is useful info.

I'm in the middle of redoing all my steps, but with older wheezy (but I'm having to include repo https://twolife.be/raspbian which has the important Qt5 libraries)


`$_SERVER['REQUEST_TIME']`은 서버의 표준시간대와 무관한 유닉스 타임스탬프입니다.


TeamCity Azmq :: Build (Linux) [Build 157](https://136.243.151.173:4433/viewLog.html?buildId=729&buildTypeId=Azmq_BuildLinux&guest=1) is now running


interesting. I wonder if we're doing this in other places such as the scheduler plugin.


@flavour thanks...will do that
And I was flying thinking this is a hidden treasure in EDEN ;)


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1550 of 1550 tests passed, 0 test warnings)

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


TeamCity Azmq :: Build (Linux) [Build 157](https://136.243.151.173:4433/viewLog.html?buildId=729&buildTypeId=Azmq_BuildLinux&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 30 Build time: 00:01:27


1) I think we also should clean the keys as well as the values in dict.....
2) maybe clean at the self.name level (if it happens) rather than the md.update() level, just in case?
3) What happens to the **kwargs when they are passed in?  Please double-check they are handled.  I see it in   I see it in Sample but not other places.  Please check....


could you explain why we need to clean up the session from original context? that doesn't look right?

FTR in the scheduler plugin we do:

        XWikiContext scontext = context.clone();
        scontext.setWiki(context.getWiki());
        context.getWiki().getStore().cleanUp(context);


Maybe ask this guy to contribute his dockerfile? https://github.com/mc-b/IoTKit/blob/master/docker/leshan/Dockerfile


works for me (tm). Two things:
1. This was kind of a proof of concept. It don't update the current project, it creates a new one sending a zip file.  I think the correct way would be delete all files from the project and re-upload them.
2. I gave up developing this once I learn that overleaf (former writelatex) has a straight forward to pull/push from their repositories. So I decided to share in case you are interested as a implementation idea.


I agree with @tobixen. You just go ahead and edit the `.jade` files here on GitHub, @haukbjork. Good luck and thanks! :smiley:


the plotting logic appears to be still in save_tiff....is it so?  It should be deleted.


@kijin 아... 근데 `strtotime()` 얘도 Unix timestamp를 반환하지않나요?


Rails now provides a finder `find_by(...` which can be adopted.
In this case we will have something like `Item.find_by(id: params[:id])`.


Ah, thanks!


Add your ~/.emacs.d to load-path and also use `(provide 'my-gui)` in the `my-gui.el` so you can do `(require 'my-gui)`

Also, rather than `my-` use a prefix unique to you, it's a nice habit to get into, which I have not yet done much.

e.g. `linclee-` or something you like more!


Cómo mola! :D


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


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


well then, as mentioned in the tc issue tracker:
with these changes it will be very easy for tanks to aggro all summoned adds from bosses no?


Test


test


I'm wondering why these http calls are returning 200 in case of negative scenario, ideally your rest call should return non 200 for negative scenarios.


This looks weird... is this temporary until a fix is found? If not, there's no reason to keep ```#if 0``` in the code...


Hmmm. Inline interfaces? Would put and `ID?` after `typeparams?` make it ambiguous? I would allow something like `{ add(U64, U64): U64 }` which makes for a generic one-fun-interface inline definition.

Taking it further makes me doubt about how useful `{ add(U64, U64): U64; sub(U64, U64): U64 }` would be versus union and intersection types (e.g.: `({ add(U64, U64): U64 } & { sub(U64, U64): U64 })`), or the regular interface definition.


Is this supposed to say " will cause fetchPromise to fulfill, thus derivedPromise will be **rejected**"?


IIRC there are some standard folder names for lisp under emacs.d which are automatically added to `load-path` we can discover what they are by doing `M-x describe-variable` <kbd>RET</kbd> `load-path` (only when starting with `emacs -Q` though, or they may be polluted)


TeamCity Azmq :: Build (Windows) [Build 184](https://136.243.151.173:4433/viewLog.html?buildId=731&buildTypeId=Azmq_Build&guest=1) is now running


TeamCity Azmq :: Build (Linux) [Build 158](https://136.243.151.173:4433/viewLog.html?buildId=730&buildTypeId=Azmq_BuildLinux&guest=1) is now running


TeamCity Azmq :: Build (Linux) [Build 158](https://136.243.151.173:4433/viewLog.html?buildId=730&buildTypeId=Azmq_BuildLinux&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 30 Build time: 00:00:05


TeamCity Azmq :: Build (Windows) [Build 184](https://136.243.151.173:4433/viewLog.html?buildId=731&buildTypeId=Azmq_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 30 Build time: 00:00:11


оно и было реализоавано через before_action


This prints the users password in the console. Can this be removed?


I was Thinking the same thing.
maybe some api shit
Thanks ill check it out


When I read this for the first time, it wasn't obvious why derivedPromise should reject - on a second read I saw `doSomeeething`, sloppy reading on my end but I wonder if it can be made more obvious somehow.