Also remove productPriceCalculator


what happens if the string only contains spaces? Will the string be returned unmodified then?


Shouldn't this be r4 instead of r2 ?


done


how is this reachable?


@alexandros-s ... what was broken that warranted this fix? In our project, we see that this change broke our build.


It is valid to calculate MMP for running power !
This is also a classic "action at a distance" antipattern to avoid.


And moreover, maybe the change should have resulted in a new fix version (1.7.2) instead of modifying the existing version(s).


통상적으로 많이 사용하는 swap 함수의 개형입니다.

void swap(int *a, int *b)
{
  int temp;
  temp = *a;
  *a = *b;
  *b = temp;
}


Very minor point...

Go comment style is to put the name of the function first.  The linter enforces this for external functions, but it is a good idea to always comment them like that, eg

```go
// isAuthOwned checks if any of the item owners is the authenticated owner
```


Это html.


Shall I also change replaceSingleton(double newValue) to use Number?


TeamCity pull requests :: netty [Build 957](https://garage.netty.io/teamcity/viewLog.html?buildId=4478&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6364, ignored: 23 Build time: 01:23:45


Я ищу первый после auth_token.


- [ ] ↑に関連し、"JDK8 Date ..." は表現として正確ではない。"JDK8"を削るか or  "Java 8から追加されたDate and Time ..."という表現の方が適切。


`mods` is unused


I think this is alarming, AFIK `parents` should always contain a class (the superclass) as first member.


Why?


@dellis1972 - We're suddenly getting this error from Xamarin.Android:

```
Xamarin.Android.Common.targets(379,2): mandroid error XA0020: Unable to launch daemon. [Lidgren.Network.Ouya.csproj]
Xamarin.Android.Common.targets(379,2): error XA9010: Unable to determine license edition. [Lidgren.Network.Ouya.csproj]
```

What is going on?


Why would this fail travis?


Actually, I just moved this function here from the end of the file, I did not change its body -- and yes, it will return a string made of whitespace unchanged since in this case `s.find_first_not_of(" \t")` will return `std::string::npos`.


1. You mean "probably" ?
2. Where/when did Victor come from


NetEventSource is being included in multiple assemblies?  Does that mean that each assembly could end up having an EventSource with the same ID?


@KonajuGames - Note that I suspect a bunch of PR merges are about to fail to build over this error above.


This is UI porn.


put navigation in nav>ul structure


why `true` all the time? If always true, we don't need `clearParameter`


Can you add assert(device); here, other than that it seemed fine.


Do it like it is on the groovy version give a default value to amount


мы пишем string const &


Awesome. :+1:


One fewer "array_merge".....  ;-)


remove that class, you have different class for "MY CALENDAR" and "GROUP VIEW"


@tomspilman I think I've had that error on my development machine before, but I don't remember how I fixed it.  I'd recommend trying to restart the build agent to see if that fixes it (especially if this is Windows)?


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


ui-sref-active


Looks good


add class to a, insted of span


To save myself time experimenting, what should be done when drawing is done on screen? Doing gl.LoadIdentity in that case would make DrawScreen callins suddenly behave like DrawWorld :)
I have some WIP code for drawing units/features on textures, but maybe this example (when they're drawn on the actual screen) is easier to understand: https://github.com/gajop/Scened-Core/blob/master/scen_edit/view/view.lua#L80-L98


@bluedxca93
Please check if this theme works on Cinnamon 2.8 with starkmenu and no gap between menu and left border of screen:
http://b00merang.weebly.com/windows-7.html


Good point, actually it should probably go in `crowbar-openstack` if anything.


Лучше перенести вниз объявление переменной, а в ретурнах дальше возвращать string()


@tomspilman looks like an issue with the licensing.. might just be down to a network glitch.. try a reset/restart of the build see if that fixes it. If not we'll need to log in an see what errors we are actually getting


- [ ] サーバ側の視点だと「Webサービスを公開するアドレスを指定する」が正しい表現なのでは？ ＋アドレスといいつつ、実際はパス部分だけ指定しているので、簡単に説明があった方がよい。


1) that's not going to work, your custom shader will be overridden by the default when raw=false
2) side note from looking at c114550: it's a good idea to always wrap any API calls within a PushMatrix/PopMatrix pair


A good start :)


I did wonder about that.


If a restart doesn't fix it, it's possible that the Xamarin Android license has expired, in which case you'd need to apply again using the open source form.


Rather than passing around ```this```, getting object names, etc., how about just using [CallerNameAttribute] and other such attributes where the compiler fills in the caller information for you?


Fixed in update to your PR. Please *pull* before you carry no developing.


Yes definitely, I see that as part of the UI story though.


This question is still relevant.


remove class attribute


we should explain why this is not a default cli value.
reason: default template is *way* too long


I think `bufferLimit` would be a more logical name for the option.

Overall I'm ok with the size-based instead of the time-based buffering, though it seems like you've done all the heavy lifting to handle the races so I don't really see why the simple timer would be complicated to implement. It seems to be it would just be something like:

```
var self = this;
self.flushInterval = setInterval(function() {
    if (self.flushing) {
        self.logger.debug('flush already in progress');
    } else if (self.queue.length > 0) {
        self.logger.debug('nothing to flush');
    } else {
        self.flushing = self.flush();
    }
}, options.flushInterval || 1000);
```

You would of course need to cancel the flusher in `eof`.

In any event, it seems like both options are complementary since having both a size and a time bound on the amount of data to be buffered would be useful. So I'm ok with going ahead with the current approach and then possibly adding the timer later.


At the first read above, I thought that `self.flushing` would be a boolean and not a promise. How about calling it `flushPromise` or `activeFlush` or something like that.


/1/ for the script tag


Please add a comment explaining why you have to both wait for the currently pending flush operation to complete _and_ schedule an additional flush to get all the points out.

Also this is a nit but I would find this slightly more readable as:

```
var currentFlush = this.flushing || Promise.resolve();
currentFlush.then(function() {
    return self.flush();
})
...
```


True, but it's a bit harder to have it there (or maybe not in our case). It's just that we install this package in the non-remote case in crowbar-pacemaker already.


wrong classname.
user ul>li instead of this span , and wrap it in <nav>
use classname like "main-menu" etc.


Would not use the hash by default, just urlSync: true will work


Done


если `event_names.launch(pid)` является осмысленным для логики программы методом, его надо вынести из `console.log`, иначе его гарантированно кто-нибудь снесёт рано или поздно. я как-то раз отлаживал проблему - в режиме отладки работала программа, а в релизе нет. оказалось, полезный вызов был в отладочный макрос завёрнут, который в релизе просто вычищался под ноль вместе с содержимым.


remove that class, you got link-active on <a> element


Sorry I might be the one that did it this way, to support reloading of the page


@Valodim This is missing in the documentation above on ACTION_DETACHED_SIGN


remove that nav - it is more like user-option, not navigation


Fine with me (although I feel it belongs to this PR ;-)). I'll add a task to the UI card.


- [ ] 「Webサービスで使用するコンポーネントが格納されているパッケージを指定する」とかの方が適切な気がします。


yes it should be but this is some copy/re-malloc avoidance... TODO fix it somehow (buffer that fits the header?...).


Doesn't look like player is used? but I might just be missing it


Consider it to add a test for card2 too


I think using (non-breaking) spaces in the delta format would make it easier to read.


I managed to fix the collision between the spaceship and the enemy :D Can i send later this version?


ul is block element. don't put block element inside inline element (span)


Au weia, mir ist nie aufgefallen, dass `eZCLI` alles nach STDERR schreibt! Böse!


Here also


user-menu is better class name.
it should be a block element


It was causing crashing and I have no desire to debug it.


This question still seems relevant.


Sure, but you'll only be able to present the previous version. (don't worry about it too much though :relaxed: )


add class to this element - it is user-nav etc.


Please make `last_connect_time` line up with `self`.


I am always that diao~


And here for card1 and 3


Additionally is gl.LoadIdentity only needed when raw=false?


Looks cool, what if we would call the property `autoResize`?


Ball is a slightly misleading name, I'm guessing it's a cracker?
The class names/other names seem good


I think this line should not have been commited :)


This adds an undeclared dependency to ding_notifications. If this dependency is added we get a circular dependency since ding_message already depend on ding_notifications. That is unfortunate.


- [ ] 内容には関係ありませんが、このコミットからgithub上でレンダリングできなくなっているので、rstの文法に問題がないか確認してください。


Oh, my fault.
Awesome!


I believe `-march=native` implies `-msse4.1  -mavx  -mavx2` if the compiling host supports it. Also, if you include `<x86intrin.h>` and call AVX functions directly, the flags will have more or less impact. I'd suggest sticking to `-march=native`.


Here's what it looks like here:
https://framapic.org/NFt7MOsTW2mX/RtPOZBwWGHfE
I wonder what does this last line means? Apps which require play-services work on my phone.


Name this `last_rx_time`.


It is still that way. I just wanted them to see an example of one first. Then actually implement one on their own in pairs. In fact, i'd say most of the you do, self learning on plugins is still happening


Why?


dddd


all commit has a purpose, all lines refer to same change. Fix visual mode selection.

http://vim.1045645.n5.nabble.com/detecting-the-current-mode-td1178529.html

> running a script changes the mode

Try to visual select two words and use <Leader>ag. It will search only first


[R/coveralls.R:99:10:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/coveralls.R#L99) *style:* **Put spaces around all infix operators.**
```r
      sep="\n",
        ~^~
```
[R/coveralls.R:101:11:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/coveralls.R#L101) *style:* **Put spaces around all infix operators.**
```r
      text=system_output("git", c("log", "-n", "1",
         ~^~
```
[R/coveralls.R:102:52:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/coveralls.R#L102) *style:* **Put spaces around all infix operators.**
```r
          paste0("--pretty=format:", paste(collapse="%n", formats)))
                                                  ~^~
```
[R/covr.R:183:84:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/covr.R#L183) *style:* **Put spaces around all infix operators.**
```r
                     coverage <- run_tests(pkg, tmp_lib, dots, type, quiet, use_try=use_try)
                                                                                ~^~
```
[R/covr.R:186:73:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/covr.R#L186) *style:* **Put spaces around all infix operators.**
```r
          coverage <- run_tests(pkg, tmp_lib, dots, type, quiet, use_try=use_try)
                                                                       ~^~
```
[R/covr.R:201:80:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/covr.R#L201) *style:* **Put spaces around all infix operators.**
```r
                 coverage <- run_tests(pkg, tmp_lib, dots, type, quiet, use_try=use_try)
                                                                              ~^~
```
[R/covr.R:204:69:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/covr.R#L204) *style:* **Put spaces around all infix operators.**
```r
      coverage <- run_tests(pkg, tmp_lib, dots, type, quiet, use_try=use_try)
                                                                   ~^~
```
[R/covr.R:289:13:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/covr.R#L289) *style:* **Place a space before left parenthesis, except in a function call.**
```r
          if(isTRUE(use_try)) {
            ^
```
[R/data_frame.R:24:63:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/data_frame.R#L24) *style:* **Commas should always have a space after.**
```r
    df <- df[order(df$filename, df$first_line, df$first_byte),]
                                                              ^
```
[R/exclusions.R:103:8:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/exclusions.R#L103) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    for(i in seq_along(starts)) {
       ^
```
[R/R.R:50:5:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/R.R#L50) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.na(Sys.getenv("NOT_CRAN", unset = NA))) {
    ^
```
[R/rtools.R:151:6:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/rtools.R#L151) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  for(i in seq_along(keys)) {
     ^
```
[R/rtools.R:261:6:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/rtools.R#L261) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  for(i in rev(seq_along(version_info))) {
     ^
```
[R/shiny.R:35:62:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/shiny.R#L35) *style:* **Put spaces around all infix operators.**
```r
        shiny::tabPanel("Files", DT::dataTableOutput(outputId="file_table")),
                                                            ~^~
```
[R/shiny.R:75:62:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/shiny.R#L75) *style:* **Put spaces around all infix operators.**
```r
        shiny::tabPanel("Files", DT::dataTableOutput(outputId="file_table")),
                                                            ~^~
```
[R/shiny.R:201:19:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/shiny.R#L201) *style:* **Place a space before left parenthesis, except in a function call.**
```r
                if(coverage == 0) {
                  ^
```
[R/shiny.R:204:26:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/shiny.R#L204) *style:* **Place a space before left parenthesis, except in a function call.**
```r
                } else if(coverage > 0) {
                         ^
```
[R/shiny.R:211:37:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/shiny.R#L211) *style:* **Put spaces around all infix operators.**
```r
                shiny::tags$tr(class=cov_type,
                                   ~^~
```
[R/shiny.R:212:39:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/shiny.R#L212) *style:* **Put spaces around all infix operators.**
```r
                  shiny::tags$td(class="num", data[row_num, "line"]),
                                     ~^~
```
[R/shiny.R:213:39:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/shiny.R#L213) *style:* **Put spaces around all infix operators.**
```r
                  shiny::tags$td(class="col-sm-12", shiny::pre(class="language-r", data[row_num, "source"])),
                                     ~^~
```
[R/shiny.R:213:69:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/shiny.R#L213) *style:* **Put spaces around all infix operators.**
```r
                  shiny::tags$td(class="col-sm-12", shiny::pre(class="language-r", data[row_num, "source"])),
                                                                   ~^~
```
[R/shiny.R:214:39:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/shiny.R#L214) *style:* **Put spaces around all infix operators.**
```r
                  shiny::tags$td(class="coverage", cov_value)
                                     ~^~
```
[R/subprocess.R:57:24:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/subprocess.R#L57) *style:* **Put spaces around all infix operators.**
```r
    output <- R(options=c("-f", tmp_source, "--slave"),
                      ~^~
```
[R/summary_functions.R:134:6:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/summary_functions.R#L134) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  for(i in seq_along(x)) {
     ^
```
[R/trace_calls.R:23:11:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/trace_calls.R#L23) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      if ((!is.symbol(x) && is.na(x)) || as.character(x) == "{") {
          ^
```
[R/trace_calls.R:32:9:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/trace_calls.R#L32) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if ((identical(x[[1]], as.name("<-")) || identical(x[[1]], as.name("="))) &&
        ^
```
[R/trace_calls.R:49:7:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/trace_calls.R#L49) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(!is.null(fun_body) && !is.null(attr(x, "srcref")) &&
      ^
```
[R/trace_calls.R:104:29:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/trace_calls.R#L104) *style:* **Put spaces around all infix operators.**
```r
  rm(envir = .counters, list=ls(envir = .counters))
                           ~^~
```
[R/utils.R:14:46:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/utils.R#L14) *style:* **Commas should always have a space after.**
```r
  rex::re_substitutes(x, rex::rex(list(start,spaces) %or% list(spaces, end)),  "")
                                             ^
```
[R/utils.R:32:5:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/utils.R#L32) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(file.exists(file.path(path, "tests"))) {
    ^
```
[R/utils.R:169:26:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/utils.R#L169) *style:* **Put spaces around all infix operators.**
```r
      structure(list(file=file, coverage=coverage), class = "line_coverage")
                        ~^~
```
[R/utils.R:169:41:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/utils.R#L169) *style:* **Put spaces around all infix operators.**
```r
      structure(list(file=file, coverage=coverage), class = "line_coverage")
                                       ~^~
```
[R/with.R:176:16:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/with.R#L176) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      } else if(length(loc) == 1) {
               ^
```
[R/zzz.R:10:5:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/zzz.R#L10) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(any(toset)) options(op_covr[toset])
    ^
```
[tests/testthat/test-coveralls.R:81:58:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-coveralls.R#L81) *style:* **Commas should always have a space after.**
```r
      `covr::system_output` = function(...) paste0(c("a","b","c","d","e","f"), collapse="\n"),
                                                         ^
```
[tests/testthat/test-coveralls.R:81:62:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-coveralls.R#L81) *style:* **Commas should always have a space after.**
```r
      `covr::system_output` = function(...) paste0(c("a","b","c","d","e","f"), collapse="\n"),
                                                             ^
```
[tests/testthat/test-coveralls.R:81:66:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-coveralls.R#L81) *style:* **Commas should always have a space after.**
```r
      `covr::system_output` = function(...) paste0(c("a","b","c","d","e","f"), collapse="\n"),
                                                                 ^
```
[tests/testthat/test-coveralls.R:81:70:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-coveralls.R#L81) *style:* **Commas should always have a space after.**
```r
      `covr::system_output` = function(...) paste0(c("a","b","c","d","e","f"), collapse="\n"),
                                                                     ^
```
[tests/testthat/test-coveralls.R:81:74:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-coveralls.R#L81) *style:* **Commas should always have a space after.**
```r
      `covr::system_output` = function(...) paste0(c("a","b","c","d","e","f"), collapse="\n"),
                                                                         ^
```
[tests/testthat/test-coveralls.R:81:88:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-coveralls.R#L81) *style:* **Put spaces around all infix operators.**
```r
      `covr::system_output` = function(...) paste0(c("a","b","c","d","e","f"), collapse="\n"),
                                                                                ~^~
```
[tests/testthat/test-coveralls.R:83:44:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-coveralls.R#L83) *style:* **Put spaces around all infix operators.**
```r
      res <- coveralls("TestS4", repo_token="mytoken"),
                                          ~^~
```
[tests/testthat/test-coveralls.R:103:58:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-coveralls.R#L103) *style:* **Commas should always have a space after.**
```r
      `covr::system_output` = function(...) paste0(c("a","b","c","d","e","f"), collapse="\n"),
                                                         ^
```
[tests/testthat/test-coveralls.R:103:62:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-coveralls.R#L103) *style:* **Commas should always have a space after.**
```r
      `covr::system_output` = function(...) paste0(c("a","b","c","d","e","f"), collapse="\n"),
                                                             ^
```
[tests/testthat/test-coveralls.R:103:66:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-coveralls.R#L103) *style:* **Commas should always have a space after.**
```r
      `covr::system_output` = function(...) paste0(c("a","b","c","d","e","f"), collapse="\n"),
                                                                 ^
```
[tests/testthat/test-coveralls.R:103:70:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-coveralls.R#L103) *style:* **Commas should always have a space after.**
```r
      `covr::system_output` = function(...) paste0(c("a","b","c","d","e","f"), collapse="\n"),
                                                                     ^
```
[tests/testthat/test-coveralls.R:103:74:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-coveralls.R#L103) *style:* **Commas should always have a space after.**
```r
      `covr::system_output` = function(...) paste0(c("a","b","c","d","e","f"), collapse="\n"),
                                                                         ^
```
[tests/testthat/test-coveralls.R:103:88:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-coveralls.R#L103) *style:* **Put spaces around all infix operators.**
```r
      `covr::system_output` = function(...) paste0(c("a","b","c","d","e","f"), collapse="\n"),
                                                                                ~^~
```
[tests/testthat/test-exclusions.R:11:76:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-exclusions.R#L11) *style:* **Put spaces around all infix operators.**
```r
  expect_equal(do.call(parse_exclusions, c(list(t1), exclude_ops, recursive=F)), numeric(0))
                                                                          ~^~
```
[tests/testthat/test-exclusions.R:115:48:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-exclusions.R#L115) *style:* **Put spaces around all infix operators.**
```r
  expect_equal_vals(normalize_exclusions(list(a=c(1, 1, 1:10))), list(a = 1:10))
                                              ~^~
```
[tests/testthat/test-exclusions.R:117:48:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-exclusions.R#L117) *style:* **Put spaces around all infix operators.**
```r
  expect_equal_vals(normalize_exclusions(list(a=c(1, 1, 1:10), b = 1:10)), list(a = 1:10, b = 1:10))
                                              ~^~
```
[tests/testthat/test-exclusions.R:122:48:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-exclusions.R#L122) *style:* **Put spaces around all infix operators.**
```r
  expect_equal_vals(normalize_exclusions(list(a=c(1:10), a=c(10:20))), list(a = 1:20))
                                              ~^~
```
[tests/testthat/test-exclusions.R:122:59:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-exclusions.R#L122) *style:* **Put spaces around all infix operators.**
```r
  expect_equal_vals(normalize_exclusions(list(a=c(1:10), a=c(10:20))), list(a = 1:20))
                                                         ~^~
```
[tests/testthat/TestUseTry/R/notry.R:6:10:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/TestUseTry/R/notry.R#L6) *style:* **Put spaces around all infix operators.**
```r
    error=function(e) TRUE
        ~^~
```


The panel could take a user context and be configured with context based caching. Would that be sufficient?


this is really just administrative. But files are defined as we go through the lesson, but i can make a note underneath for a general overview.


This merge is out of date


change color app default is #08ae9e

แก้ในไฟล์  src/scss/ionic.app.css ดีกว่าป่ะ ให้มันแก้ code สีของ positive หมดเลย ประกาศไว้ข้างบน ยังไงก็ใช้สีนี้อยู่แล้ว แต่ทุกส่วนที่ใช้ค่าสี positive เดิมจะถูกเปลี่ยนหมด ตอนนี้เราใช้แค่ตรงส่วน bar กับปุ่ม connect

เค้าแก้อยู่ ทำได้แล้ว พึ่งเห็นว่าทำหน้า home ถ้าวิธีเค้าจะแก้ทุกส่วนเลย

แล้วแต่นะ ถ้าใช้อันนั้นไปแล้ว เค้าจะได้ย้อนกลับ

$positive:                        #08ae9e !default;


Also for the constructor argument.


Yeah, it needs to be a alot to see the fixed content plugin to work, i could just put a gist here instead.


This link seems to be broken.


I see that you just moved that, but what does it actually do?
What is a "description", what is an ExtensionTag, what do you split there and what does it then return?
Do you maybe want to sprinkle in some comments?


Please document this fucntion


Carefully,you should configure your project rightly, or you will fail to achieve the result just as you want.


why is this allowing null values for either of these columns?


Ubunutu? ;)


add readme


Ubunuta? :D


I will show you the upgraded version when I'll present the project, so i don't bother you anymore :D


To show it, I have opened #79. Now there's a test for visual selection.

To show it failing, I cherry picked that commit on a branch before this commits. I have marked test as failed to denote is expected to fail.

https://travis-ci.org/albfan/ag.vim/builds/100124021
https://travis-ci.org/albfan/ag.vim/builds/100123632

I will remove that branch when you feel, problem is well defined


Should this be the `provider` role?


If the join table is only for the 2 foreign keys, and there's no other reason to consider it a "first-class" citizen, why have a model for that? In effect, this whole class should not be present.


whut?


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


main.py requires kivy 1.1.1 but trash.kv needs kivy 1.0.9 I don't know if that effects the code or anything? but seems like they should both need the same version


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


Premier commentaire


Forgot to mention I changed the Decimal literal prefix from i to d. I don't know what I was thinking with i.


IV. Spring Boot features/23.3. Application property files.md

这里不应该用"句号"吧?个人建议还是用"."好~


Im not 100% sure if "files" is actually the right name for a list of triples of source roots, file paths and fully quallified class names?


This is required to avoid an error on a second build with no changes to the C code in Python 3.5. That version's distutils' new support class for Visual Studio 2015 performs late initialization of some data, including the paths to various build tools. They will be set only if anything has been compiled by the current `setup.py` invocation, otherwise, this change performs the missing initialization.


I now realize I should've added a more descriptive commit message, but I believe axon was raising an error with the Node version being used. IIRC replacing it with Redis didn't change the behavior of the app.


code for tinkering audio/random_audio_generation.py

play note has no return value bro so you do not need to define them as letters.
e.g:  a = playNote(56, dur, intensity) etc...
your code should work fine if you remove a,b,c.


This does seem cleaner so far than what I have at #11817, but it doesn't solve #11740 at all.

```
# LIBGL_DEBUG=verbose glxinfo
name of display: :0.0
libGL: AtiGetClientDriverName: 15.20.3 fglrx (screen 0)
libGL error: OpenDriver: failed to open /usr/X11R6/lib64/modules/dri/fglrx_dri.so, error[/usr/X11R6/lib64/modules/dri/fglrx_dri.so: cannot open shared object file: No such file or directory]
libGL error: OpenDriver: failed to open /usr/lib64/dri/fglrx_dri.so, error[/usr/lib64/dri/fglrx_dri.so: cannot open shared object file: No such file or directory]
...
```

`fglrxinfo` seems to work, although I'm not sure whether that indicated a problem for me in the first place.


Please add a DocBlock for this function.


This completely overrides the variables set on lines L801-802. Is this intentional?


Great catch! Thanks for that. :)


The using of function literal here is awesome!


@lukego from [The Unofficial DynASM Tutorial](http://corsix.github.io/dynasm-doc/tutorial.html):

> Note that lines which do not start with a vertical bar are untouched by dynasm.lua, and this remains true between .macro and .endmacro. As such, C/C++ code that is meant to be emitted when the macro is invoked needs to be prefixed with two vertical bars rather than no vertical bars.


Is the log required here since the message is already logged by `le_node` before the throw?


I didn't know about PHP_MAXPATHLEN, so thanks for that! That would definitely be the best fix.


The reason for this fix is because we are now supporting two track-jet collections (R=0.2 and R=0.4) and the new naming scheme for the operating points is universal across jet collections (FixedCutBEff_70). As a consequence, the extrapolations for the R=0.2 and R=0.4 track-jets had the same name without this fix.

The exact error, visible in e.g. build `#1190`, is

`More than one extrapolated analysis to apply (bottom-MV2c20-FixedCutBEff_70): MCcalib-bottom-MV2c20-FixedCutBEff_70-AntiKt2PV0TrackJets MCcalib-bottom-MV2c20-FixedCutBEff_70-AntiKt4PV0TrackJets`


Can this ever be called with time_2 < time_1?


try it. thanks.


hi there,
i need help, please!
how can i fix it in signal version 3.8.1?
thx


New contact and maintainer for platform/jn516x has been added


Maybe ignore classes with magic methods (`__call`) ?


Thanks! I searched that document for the literal characters `||` and had no hits but this was a bit simplistic.


Should `comma-separated` be replaced by `key1,value1,key2,value2`? I find it clearer.


Missing space after `,`.


Awesome! I will work from home this week and eager to help! Would be nice if we can get this into Galaxy by end of the week :)


I'd like to point out that `f` is not defined anywhere in the scope of this method.


use ul. please ;)


I think this should be `time.time()`.


@bgruening That's the plan!  I'm finishing up the API addition for easily adding and reloading tours via a POST to /api/tours right now.


Add a warning about not to change this class since it may be used by other products


Тогда лучше так:
```
string const kValue = "value=\"";
auto start = body.find(kValue, pos);
if (start == string::npos)
  return string();
start += kValue.size();
auto const end = body.find('"', start);
if (end == string::npos)
  return string();
return body.substr(start, end - start);
```


**each** is standard handlebars, **collection** is a thorax addition, see http://thoraxjs.org/api.html#template-helpers


I'd prefer 16k.... why did you choose that?


IndexName and TypeName?


Ok, Je suis d'accord
Romain


swig does not rely on yodl anymore


Can we rethrow if err is not `instanceof` `LogentriesError` as we don't want to swallow unexpected behaviour?


Oh, I'm mistaken. Maybe it should be called `now_fn` to make clear it is a function that returns a timestamp, not a timestamp itself.


Can't remember, sorry


I think it should be in a separated function (e.g.: _free_patterns)


Yeah its a distro I have been working on ;-). Fix coming (along with a few other improvements to the readme)....


If we make this behaviour configurable, it should be more acceptable as a PR. We can then use turn it on by default in our config


Awesome!


consider a more general implementation:

let firstM f = first f >>> arr (\(a, b) -> do { a' <- a ; return (a', b) })


The `device` arg is required for format helpers and not used for device helpers. I'm open to making it a kwarg in the base class or leaving it as-is (non-kw-arg only in format helper classes).


#สามารถจั่วเรื่อง simple design, tdd ไว้แถวๆนี้ได้นะ


What about the pull request to revert to the old defaults?

I think I agree, but it must be noted that puppyapps is part of a
greater scheme that includes config files like the puppypin, etc, for
it work to actually work.

In fact I have a greater plan for puppyapps, with a mime associations
db, placed in /var/lib/puppyapps/mimedefaults.db
with a command puppyapps will generate loads of mime associations for
rox and pcmanfm ready to use

My plan is to place the latest version of the freedesktop mime database here:
/var/lib/puppyapps/freedesktop.xml
with a command it can be copied/moved to /usr/share/mime/packages and
run update-mime-database


> The screenshots are not 16:9, but neither is SSR itself, so I don't know what to do about that. Resizing SSR just makes the screenshots look weird.

I think the screenshots are good.
@hughsie, do you have any suggestions for improvements?


Also, the function signature can simply say `now=time.time` since it doesn't matter when it is evaluated.


it shouldn't have.. I think its using one of mine.


According to our styleguide, `Foreign` was the better choice (one or two letter abbreviations should only be used for Thentos modules).


breaks && chain.


[R/coveralls.R:99:10:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/coveralls.R#L99) *style:* **Put spaces around all infix operators.**
```r
      sep="\n",
        ~^~
```
[R/coveralls.R:101:11:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/coveralls.R#L101) *style:* **Put spaces around all infix operators.**
```r
      text=system_output("git", c("log", "-n", "1",
         ~^~
```
[R/coveralls.R:102:52:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/coveralls.R#L102) *style:* **Put spaces around all infix operators.**
```r
          paste0("--pretty=format:", paste(collapse="%n", formats)))
                                                  ~^~
```
[R/covr.R:183:84:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/covr.R#L183) *style:* **Put spaces around all infix operators.**
```r
                     coverage <- run_tests(pkg, tmp_lib, dots, type, quiet, use_try=use_try)
                                                                                ~^~
```
[R/covr.R:186:73:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/covr.R#L186) *style:* **Put spaces around all infix operators.**
```r
          coverage <- run_tests(pkg, tmp_lib, dots, type, quiet, use_try=use_try)
                                                                       ~^~
```
[R/covr.R:201:80:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/covr.R#L201) *style:* **Put spaces around all infix operators.**
```r
                 coverage <- run_tests(pkg, tmp_lib, dots, type, quiet, use_try=use_try)
                                                                              ~^~
```
[R/covr.R:204:69:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/covr.R#L204) *style:* **Put spaces around all infix operators.**
```r
      coverage <- run_tests(pkg, tmp_lib, dots, type, quiet, use_try=use_try)
                                                                   ~^~
```
[R/covr.R:289:13:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/covr.R#L289) *style:* **Place a space before left parenthesis, except in a function call.**
```r
          if(isTRUE(use_try)) {
            ^
```
[R/data_frame.R:24:63:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/data_frame.R#L24) *style:* **Commas should always have a space after.**
```r
    df <- df[order(df$filename, df$first_line, df$first_byte),]
                                                              ^
```
[R/exclusions.R:103:8:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/exclusions.R#L103) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    for(i in seq_along(starts)) {
       ^
```
[R/R.R:50:5:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/R.R#L50) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(is.na(Sys.getenv("NOT_CRAN", unset = NA))) {
    ^
```
[R/rtools.R:151:6:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/rtools.R#L151) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  for(i in seq_along(keys)) {
     ^
```
[R/rtools.R:261:6:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/rtools.R#L261) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  for(i in rev(seq_along(version_info))) {
     ^
```
[R/shiny.R:35:62:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/shiny.R#L35) *style:* **Put spaces around all infix operators.**
```r
        shiny::tabPanel("Files", DT::dataTableOutput(outputId="file_table")),
                                                            ~^~
```
[R/shiny.R:75:62:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/shiny.R#L75) *style:* **Put spaces around all infix operators.**
```r
        shiny::tabPanel("Files", DT::dataTableOutput(outputId="file_table")),
                                                            ~^~
```
[R/shiny.R:201:19:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/shiny.R#L201) *style:* **Place a space before left parenthesis, except in a function call.**
```r
                if(coverage == 0) {
                  ^
```
[R/shiny.R:204:26:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/shiny.R#L204) *style:* **Place a space before left parenthesis, except in a function call.**
```r
                } else if(coverage > 0) {
                         ^
```
[R/shiny.R:211:37:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/shiny.R#L211) *style:* **Put spaces around all infix operators.**
```r
                shiny::tags$tr(class=cov_type,
                                   ~^~
```
[R/shiny.R:212:39:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/shiny.R#L212) *style:* **Put spaces around all infix operators.**
```r
                  shiny::tags$td(class="num", data[row_num, "line"]),
                                     ~^~
```
[R/shiny.R:213:39:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/shiny.R#L213) *style:* **Put spaces around all infix operators.**
```r
                  shiny::tags$td(class="col-sm-12", shiny::pre(class="language-r", data[row_num, "source"])),
                                     ~^~
```
[R/shiny.R:213:69:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/shiny.R#L213) *style:* **Put spaces around all infix operators.**
```r
                  shiny::tags$td(class="col-sm-12", shiny::pre(class="language-r", data[row_num, "source"])),
                                                                   ~^~
```
[R/shiny.R:214:39:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/shiny.R#L214) *style:* **Put spaces around all infix operators.**
```r
                  shiny::tags$td(class="coverage", cov_value)
                                     ~^~
```
[R/subprocess.R:57:24:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/subprocess.R#L57) *style:* **Put spaces around all infix operators.**
```r
    output <- R(options=c("-f", tmp_source, "--slave"),
                      ~^~
```
[R/summary_functions.R:134:6:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/summary_functions.R#L134) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  for(i in seq_along(x)) {
     ^
```
[R/trace_calls.R:23:11:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/trace_calls.R#L23) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      if ((!is.symbol(x) && is.na(x)) || as.character(x) == "{") {
          ^
```
[R/trace_calls.R:32:9:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/trace_calls.R#L32) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if ((identical(x[[1]], as.name("<-")) || identical(x[[1]], as.name("="))) &&
        ^
```
[R/trace_calls.R:49:7:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/trace_calls.R#L49) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if(!is.null(fun_body) && !is.null(attr(x, "srcref")) &&
      ^
```
[R/trace_calls.R:104:29:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/trace_calls.R#L104) *style:* **Put spaces around all infix operators.**
```r
  rm(envir = .counters, list=ls(envir = .counters))
                           ~^~
```
[R/utils.R:14:46:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/utils.R#L14) *style:* **Commas should always have a space after.**
```r
  rex::re_substitutes(x, rex::rex(list(start,spaces) %or% list(spaces, end)),  "")
                                             ^
```
[R/utils.R:32:5:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/utils.R#L32) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(file.exists(file.path(path, "tests"))) {
    ^
```
[R/utils.R:169:26:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/utils.R#L169) *style:* **Put spaces around all infix operators.**
```r
      structure(list(file=file, coverage=coverage), class = "line_coverage")
                        ~^~
```
[R/utils.R:169:41:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/utils.R#L169) *style:* **Put spaces around all infix operators.**
```r
      structure(list(file=file, coverage=coverage), class = "line_coverage")
                                       ~^~
```
[R/with.R:176:16:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/with.R#L176) *style:* **Place a space before left parenthesis, except in a function call.**
```r
      } else if(length(loc) == 1) {
               ^
```
[R/zzz.R:10:5:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/R/zzz.R#L10) *style:* **Place a space before left parenthesis, except in a function call.**
```r
  if(any(toset)) options(op_covr[toset])
    ^
```
[tests/testthat/test-coveralls.R:81:58:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-coveralls.R#L81) *style:* **Commas should always have a space after.**
```r
      `covr::system_output` = function(...) paste0(c("a","b","c","d","e","f"), collapse="\n"),
                                                         ^
```
[tests/testthat/test-coveralls.R:81:62:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-coveralls.R#L81) *style:* **Commas should always have a space after.**
```r
      `covr::system_output` = function(...) paste0(c("a","b","c","d","e","f"), collapse="\n"),
                                                             ^
```
[tests/testthat/test-coveralls.R:81:66:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-coveralls.R#L81) *style:* **Commas should always have a space after.**
```r
      `covr::system_output` = function(...) paste0(c("a","b","c","d","e","f"), collapse="\n"),
                                                                 ^
```
[tests/testthat/test-coveralls.R:81:70:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-coveralls.R#L81) *style:* **Commas should always have a space after.**
```r
      `covr::system_output` = function(...) paste0(c("a","b","c","d","e","f"), collapse="\n"),
                                                                     ^
```
[tests/testthat/test-coveralls.R:81:74:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-coveralls.R#L81) *style:* **Commas should always have a space after.**
```r
      `covr::system_output` = function(...) paste0(c("a","b","c","d","e","f"), collapse="\n"),
                                                                         ^
```
[tests/testthat/test-coveralls.R:81:88:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-coveralls.R#L81) *style:* **Put spaces around all infix operators.**
```r
      `covr::system_output` = function(...) paste0(c("a","b","c","d","e","f"), collapse="\n"),
                                                                                ~^~
```
[tests/testthat/test-coveralls.R:83:44:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-coveralls.R#L83) *style:* **Put spaces around all infix operators.**
```r
      res <- coveralls("TestS4", repo_token="mytoken"),
                                          ~^~
```
[tests/testthat/test-coveralls.R:103:58:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-coveralls.R#L103) *style:* **Commas should always have a space after.**
```r
      `covr::system_output` = function(...) paste0(c("a","b","c","d","e","f"), collapse="\n"),
                                                         ^
```
[tests/testthat/test-coveralls.R:103:62:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-coveralls.R#L103) *style:* **Commas should always have a space after.**
```r
      `covr::system_output` = function(...) paste0(c("a","b","c","d","e","f"), collapse="\n"),
                                                             ^
```
[tests/testthat/test-coveralls.R:103:66:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-coveralls.R#L103) *style:* **Commas should always have a space after.**
```r
      `covr::system_output` = function(...) paste0(c("a","b","c","d","e","f"), collapse="\n"),
                                                                 ^
```
[tests/testthat/test-coveralls.R:103:70:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-coveralls.R#L103) *style:* **Commas should always have a space after.**
```r
      `covr::system_output` = function(...) paste0(c("a","b","c","d","e","f"), collapse="\n"),
                                                                     ^
```
[tests/testthat/test-coveralls.R:103:74:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-coveralls.R#L103) *style:* **Commas should always have a space after.**
```r
      `covr::system_output` = function(...) paste0(c("a","b","c","d","e","f"), collapse="\n"),
                                                                         ^
```
[tests/testthat/test-coveralls.R:103:88:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-coveralls.R#L103) *style:* **Put spaces around all infix operators.**
```r
      `covr::system_output` = function(...) paste0(c("a","b","c","d","e","f"), collapse="\n"),
                                                                                ~^~
```
[tests/testthat/test-exclusions.R:11:76:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-exclusions.R#L11) *style:* **Put spaces around all infix operators.**
```r
  expect_equal(do.call(parse_exclusions, c(list(t1), exclude_ops, recursive=F)), numeric(0))
                                                                          ~^~
```
[tests/testthat/test-exclusions.R:115:48:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-exclusions.R#L115) *style:* **Put spaces around all infix operators.**
```r
  expect_equal_vals(normalize_exclusions(list(a=c(1, 1, 1:10))), list(a = 1:10))
                                              ~^~
```
[tests/testthat/test-exclusions.R:117:48:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-exclusions.R#L117) *style:* **Put spaces around all infix operators.**
```r
  expect_equal_vals(normalize_exclusions(list(a=c(1, 1, 1:10), b = 1:10)), list(a = 1:10, b = 1:10))
                                              ~^~
```
[tests/testthat/test-exclusions.R:122:48:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-exclusions.R#L122) *style:* **Put spaces around all infix operators.**
```r
  expect_equal_vals(normalize_exclusions(list(a=c(1:10), a=c(10:20))), list(a = 1:20))
                                              ~^~
```
[tests/testthat/test-exclusions.R:122:59:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/test-exclusions.R#L122) *style:* **Put spaces around all infix operators.**
```r
  expect_equal_vals(normalize_exclusions(list(a=c(1:10), a=c(10:20))), list(a = 1:20))
                                                         ~^~
```
[tests/testthat/TestUseTry/R/notry.R:6:10:](https://github.com/jimhester/covr/blob/fac89de61592f7ce5b5ceb17a6391851a2fbf1c7/tests/testthat/TestUseTry/R/notry.R#L6) *style:* **Put spaces around all infix operators.**
```r
    error=function(e) TRUE
        ~^~
```


You need the LoadIdentity if calling gl.*Shape in DrawWorld with raw=false.

The current code doesn't really support calling gl.*Shape in DrawScreen with raw=false, it implicitly assumes to be in DrawWorld.


你是谁？呵呵


Man I wish I was on the desktop! No can do right now.


Combo box and button operationnals


Do you think it would be better that the XmlMetadataAnnotations interface allows to set the lineNumber instead of doing it in the factory? It seems that it would be useful for all XmlMetadataAnnotations to support it and then have an standard way to set the lineNumber for all implementations


-แบ่งออกเป็น Subset ย่อยๆ ที่อิสระต่อกัน
+แบ่งออกเป็นส่วนย่อยๆ ที่อิสระต่อกัน


> Maybe the state should be renamed to PVR_CONNECTION_STATE_ACCESS_DENIED? (Standard display text: "Access denied") to get generic and backend-independent?

sounds good

> Do you think we need another state PVR_CONNECTION_STATE_USER

it's not really a state, is it?


This is a part of antlr Lexer I use to parse metadata of RAML actions to support java types in generated code. See `src/main/antlr3/com/hubrick/raml/mojo/antlr/ExtensionTags.g`


```.to match_array``` would have been better (rspec3 friendly ;p)

Also the ```.map { |t| t.url }``` can be replaced by ```.map(&:url)``` :)


This seems like nitpicking....that and address.com is an actual domain if you are doing to argue semantics....


The fix looks good - thanks for making the update.  I'll merge it.


I propose `CompilationUnitTopologyEntries` :)


This is embarrassing.


In this code, the || sequences are for stylistic reasons rather than funky macro-ness, as I like each stanza to have an unbroken | spine.


Maybe "address OR subnet"?


lol, yeah it's 2016 but Vim is from 1991


Est-ce que on parle plutot de musique ou de chansons ? Sachant que une chanson est une musique mais que l'inverse n'est pas vrai.


Hmmm, how do I see which line these comments apply to?


Why would you move this typedef to the middle of al this includes?


Hi @Zenedith - yes, this is working and good to go.  Thanks,

Pat


Default was True in the config file: is there a reason for changing?


Could `"ting/object/" . $id` be replaced with `entity_uri('ting_object`, $obj)`?


不是特别懂 Arrow。。
以及居然有人来围观 =w=


Perhaps it could be used the g_ptr_array_new_with_free_func when creating the array at line 831


-ไม่ได้ถูกแยากจากกันเพื่อให้เลือกอย่างใดอย่างหนึ่ง
+ไม่ได้ถูกแยกจากกันเพื่อให้เลือกอย่างใดอย่างหนึ่ง

-การเลือกใช้มีประโยชน์ที่ได้และค่าใช้จ่ายที่จ่ายออกไปในรูปแบบที่แตกต่างกัน
+แต่ละวิธีมีประโยชน์ที่ได้และค่าใช้จ่ายที่จ่ายออกไปในรูปแบบที่แตกต่างกัน


Empty comment?


@shlevy Hm, this file isn't used anywhere, as far as I can see...


I couldn't get this to connect to the sound files when I downloaded it but that may be an issue when at my end.


Если бы работал  я бы это  не делал.   Просто все могут  и просто git://  а bitbucket  ssh or  https


Why not at least log the exception?


This is for rest buttons not REST buttons :)


Why not at least log the exception? Do we expect them to occur under non-error conditions?


Perhaps add a comment here as to where random is used?


Hm. I'm not sure about this line... I feel like you should add an `undefined` check, as you access it's properties and couldn't be sure that `component` as well as `component.component` is defined? On the other hand, this will rise an error and fail the test (as intended) and the current error message is kinda ok. What do you think?


Sure. ``key1,value1,key2,value2`` would be more clear if users just don't like read docuementation first.


I don't see any modification to the add_contest.html template. Is there a reason?


This question is still relevant?


Hi,prasanthj
Now we use hive-2.0.0-SNAPSHOT,we meet an error when starting the service metastore .
The error is :ERROR StatusLogger No log4j2 configuration file found. Using default configuration: logging only errors to the console.
If we configure the variable HADOOP_CLIENT_OPTS  as
export HADOOP_CLIENT_OPTS="$HADOOP_CLIENT_OPTS -Dlog4j.configurationFile=hive-log4j2.xml " ,the error disappear,but entering hive cli slower.
Please help me how can avoid the ERROR.
Thank you very much.


-ที่ไม่มีสูตรตายตัวบอกได้ว่าควรจะเตรียมตัวไว้เยอะแค่ไหน
+ที่ไม่มีสูตรตายตัวว่าควรจะเตรียมตัวไว้เยอะแค่ไหน

-ดังนั้นการออกแบบระบบจะต้อง trade off ระหว่างความรวดเร็วในการพัฒนาระบบขึ้นในไปช่วงแรก และการขยายระบบออกไปในอนาคต
+ดังนั้นการออกแบบระบบจะต้องชั่งน้ำหนักระหว่างความรวดเร็วในการพัฒนาเพื่อเอาระบบขึ้นไปในช่วงแรก กับการขยายระบบออกไปในอนาคต


We should put a version on the migration code. For example, this code would be for v1.1.0.

We should also version the database. Maybe add a table info including a database schema version. Then we can migrate databases based on the different versions. We can also write a small script to run different migration sql code based on version difference. Maybe an RFC to the larger group?


2333，其实我是code码不动了到处瞎逛~ww


Maybe? I went with the full class name because I didn't want to use `format.py` and `device.py` for `FormatPopulator` and `DevicePopulator`.


Can't seem to manage the language files to work within Wordpress (Dutch). I have copied the code above in a new pot file and exported it to an .mo file. Than uploaded it to /tgm/langauges/. Language keeps being set to English.

I have also tried to setup custom language strings in my functions.php but after that there are still several words not being translated.

I use the plugin to include some plugins from the Wordpress.org respository so my theme always picks the most updated version of the plugins.


TeamCity pull requests :: netty [Build 958](https://garage.netty.io/teamcity/viewLog.html?buildId=4485&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6365, ignored: 23 Build time: 00:42:00


If magic methods are documented with @ method block - there will be no warning highlighting here. In other cases it will be highlighted like a method call ($obj->someMagicMethod()).


Hmmm, well, I want bootstrap.css eventually, and that puts css in the same
category as this problem with js, for one.

As far as the caching problem, here is the lay of the land:

#1 Perfect, but requires auto html generation or manual:
<script src='nw2016-01-04.js'>
<script src='nw2016-02-07.js'>
#2 Proxy caching problems:
<script src='nw.js?date=2016-01-04'>
<script src='nw.js?date=2016-02-07'>
#3 Deep caching problems
<script src='nw.js'>

What I did:
For jQuery, I used a file with a version string in the file name, and
manually bumped the reference to that file.
For jquery-linkify, I ignored the issue, we've only ever used HEAD of the
file from the bitlucid repo.
For jquery-timeago, I manually updated the caching a few tim
For nw.js, I manually updated a date cachebusting string.
For style.css, I manually updated a date cachebusting string in the
template.

*shrugs*

On Mon, Jan 4, 2016 at 9:16 AM, Beagle <notifications@github.com> wrote:

> Not sure there are any left. Also, how did you handle cache busting for
> our own scripts like nw.js et. al.? From: Roy R.Sent: Monday, January 4,
> 2016 08:41To: BitLucid/ninjawarsReply To: BitLucid/ninjawarsCc:
> BeagleSubject: Re: [ninjawars] Remove vendor javascript from source control
> (52fc2ce)Sure, just wanted to make you aware before more assets were
> handled that
> way.
>
> On Mon, Jan 4, 2016 at 8:39 AM, Beagle <notifications@github.com> wrote:
>
> > That's a separate issue that can now more easily be solved since it's a
> > feature of the source-code but a feature of the build process. From: Roy
> > R.Sent: Monday, January 4, 2016 07:47To: BitLucid/ninjawarsReply To:
> > BitLucid/ninjawarsCc: BeagleSubject: Re: [ninjawars] Remove vendor
> > javascript from source control (52fc2ce)This approach causes caching
> > problems. I'm sorry, I should have made it extra super clear in the
> issue,
> > but in the past cached version confusion ensued from symlinked assets,
> > otherwise we would already be using the symlink approach on that system.
> >
> > Everything caches via url string -> url string doesn't change while the
> > version underneath does -> browsers have unbreakably cached old versions
> of
> > js and css.
> >
> > For jQuery specifically, since we use the cdn by url string, this problem
> > will be heavily suppressed, but With most other assets that we serve
> > directly, we'll develop caching problems.
> >
> > —Reply to this email directly or view it on GitHub.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/BitLucid/ninjawars/commit/52fc2ce892365e2b0f942ce41b3f601a07c8b513#commitcomment-15243014
> >
> > .
> >
>
>
>
> --
> +++++++++++++++++++++++++++
> BitLucid.com <http://bitlucid.com/> - BitLucid, Inc. - Web & Programming
> Consultants
> ninjawars.net - My webgame project
> github.com/tchalvak/ninjawars - My open source projects
> dnaexmosn.deviantart.com - Art Gallery
> 585-519-7658 - Cell
> "Never compare your inside with somebody's outside."
>
>
> —Reply to this email directly or view it on GitHub.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BitLucid/ninjawars/commit/52fc2ce892365e2b0f942ce41b3f601a07c8b513#commitcomment-15243651>
> .
>

--
+++++++++++++++++++++++++++
BitLucid.com <http://bitlucid.com/> - BitLucid, Inc. - Web & Programming
Consultants
ninjawars.net - My webgame project
github.com/tchalvak/ninjawars - My open source projects
dnaexmosn.deviantart.com - Art Gallery
585-519-7658  - Cell
"Never compare your inside with somebody's outside."


gg


There are code formatting issues here.


Is `border-radius` a leftover from when we were using the hack to generate circles? It doesn't seem like it should be necessary.


On second thought, maybe not. I'm confused about why we are installing some of those requirements (`unittest2` and `mock`) on travis since they are also in requirements.txt. It looks like there are a fair number of test requirements in requirements.txt, so that seems like the right place to put this.


@arichiardi Before going on with this I need you to check it out. Is this what you mean by adding the `:enable-stacktrace` flag to `read-eval-call` (issue https://github.com/ScalaConsultants/replumb/issues/83)?

Because in order to do so we need to forward it from `replumb.core/read-eval-call` to `replumb.repl/read-eval-call`, than to `call-back!` and than add it the result map (`error-map` function).

This is exactly what I did: `:enable-stacktrace` is the flag passed to `read-eval-call` and `print-stack?` is in the result map (different names to avoid confusion). Then I modified `result->string` to check this flag and print the stack accordingly... but it was there as a parameter already, so I am asking.


I don't remember if we had ever decided a convention for this, but I'd like the new fields to be added, with their default value. (Even better would be to read the config file and give them the value they have there, but that's probably not feasible). In that way an version-18 dump updated to a version-19 dump looks exactly like a version-19 dump. (Moreover, some successive update script may assume those fields to be present).


@rouson This is the commit that tripped you up. It appears the critical step that you missed was that this branch, `fix-inconsequential-stack-sh-errors`, was not merged into master *before* updating it to be merged into `devel`. From what I can tell this is the order in which things happened:

 1. Create branch `fix-inconsequential-stack-sh-errors` based on `master` :heavy_check_mark:
 2. Make changes and commits to implement bug fix/enhancement :heavy_check_mark:
 3. Open PR against `master` :heavy_check_mark:
 4. Open PR against `devel`
     - this is OK, but the messages about how it can't yet be merged tripped you up. More explanation below.
 5. Merge `devel` into `fix-inconsequential-stack-sh-errors` to resolve merge conflicts for the `devel` PR :heavy_multiplication_x:
     - Once a PR is opened (i.e., the one against `master`) changes can still be pushed to the branch as the PR is reviewed and discussed. Because the PR against master had *not* been merged yet, the changes you made to the branch to allow it to be merged into `devel` also made their way into PR against `master`. The commits also get listed in chronological order, so when I reviewed the PR from the plane, you had not yet merged `devel` into `fix-inconsequential-stack-sh-errors`, so everything was kosher. Then the branch was updated to get merged into `devel` after I was off-line, which is where the beginning of the problems arose.
 6. Merge `fix-inconsequential-stach-sh-errors` into `master` :heavy_multiplication_x:
     - This needed to happen **_before_** step 5
 7. Merge `fix-incnsequential-stack-sh-errors` into `devel` :heavy_check_mark:
     - This needs to happen after step 5, which it did.

So to recap, this is the order in which things need to happen:

 1. Branch *from* (i.e., based on) `master`
 2. Open PR against `master` (and `devel` if you like, but **_ignore any messages about merge problems with `devel` for the time being_**)
 3. Discuss, change, etc. anything that comes up in code reviews by making more commits to the branch and pushing the branch to Github.
 4. If any other changes occurred on `master` (from other people's work, for example) merge `master` into your branch to ensure it is up to date---otherwise this step can be skipped. Merge conflicts may arise that need to be manually inspected and resolved. Github provides rough command line instructions for doing this, from the PR window.
 5. Merge branch into `master`, closing PR against `master`
 6. Only *after* 5, merge `devel` into your branch. (This makes it look like you are applying your changes against `devel` even though they were originally applied against master.) Merge conflicts could arise, but most of the time they are going to be small and easy to resolve due to the common history and proper git workflow. Fix these manually if needed.
 7. If you haven't opened a PR against `devel` yet, do this now. This can be done at almost any point **so long as you make sure step 6 happens _after_ step 5.**
 8. Merge your branch into `devel` closing the PR against `devel` opened in step 7 or earlier.

I'll try to updated CONTRIBUTING.md with these instructions since the CLA is hosted elsewhere now.


HI levak.
You can help me, fix wb.sh for version VN

dump launcher by OSPY
https://drive.google.com/file/d/0B5RUNjxNVSWFTks3dXZyTEx5ams/view?usp=sharing


The closed parenthesis is never opened :)


Why are you making a `clone` here if you are reassigning `variant.attributes` later anyway?


oto muj sóper duper nie op tymalny kod


`detect` and `find` are the same method: http://ruby-doc.org/core-2.2.3/Enumerable.html#method-i-detect
I think using `find` within a merhod called `find_by_slug` is clearer. It's also more common.


LGTM so far.


I briefly looked for average java class size on the internet, and stumbled upon this: http://programmers.stackexchange.com/questions/66523/how-many-lines-per-class-is-too-many-in-java/66598#66598

I eye-balled the mean of line numbers, which was 95.3, and multiplied it by (previously) maximum code line length, which felt a good balance between short lines and presently allowed 120+ character lines. That gave me (when rounded) 8000 ASCII characters. Unless the file is full of non-ASCII unicode characters (which it in most case is), that should be about right. And the closest power of two was 8192 :) This number is also widely used for byte buffers in java io/nio and other libraries, afaik. So, also a reference.


표준 C (ansi C)에서는 반복문 안에서 변수 정의가 원칙적으로는 되지 않게 되어 있습니다.
따라서 scope 밖에서 반복문에 사용할 변수 i를 정의해주는 것이 원칙입니다.


Please use `_.each` for this


Sorry didn't notice this before, it should be `3.4`


@addaleax as you can see, I added the test case from Node's commit (it is outdated though), but it doesn't add more coverage to what I already had before. In short, the `if (!fired)` path is still not taken. I'm not sure if the author of that test really tested or just wrote a naïve one...


Let's add a feature to our templating engine to generate URLs for these
types of resources (the composer world is calling them "components") that
for now just generates the exact same link as what we have but does do
programmatically. Then we can enhance it by adding an NginX rule to ignore
cache-busting strings, then we can add cache-busting strings to the
template function

On Mon, Jan 4, 2016 at 10:10 AM, Roy R. <notifications@github.com> wrote:

> Hmmm, well, I want bootstrap.css eventually, and that puts css in the same
> category as this problem with js, for one.
>
> As far as the caching problem, here is the lay of the land:
>
> #1 Perfect, but requires auto html generation or manual:
> <script src='nw2016-01-04.js'>
> <script src='nw2016-02-07.js'>
> #2 Proxy caching problems:
> <script src='nw.js?date=2016-01-04'>
> <script src='nw.js?date=2016-02-07'>
> #3 Deep caching problems
> <script src='nw.js'>
>
> What I did:
> For jQuery, I used a file with a version string in the file name, and
> manually bumped the reference to that file.
> For jquery-linkify, I ignored the issue, we've only ever used HEAD of the
> file from the bitlucid repo.
> For jquery-timeago, I manually updated the caching a few tim
> For nw.js, I manually updated a date cachebusting string.
> For style.css, I manually updated a date cachebusting string in the
> template.
>
> *shrugs*
>
>
>
> On Mon, Jan 4, 2016 at 9:16 AM, Beagle <notifications@github.com> wrote:
>
> > Not sure there are any left. Also, how did you handle cache busting for
> > our own scripts like nw.js et. al.? From: Roy R.Sent: Monday, January 4,
> > 2016 08:41To: BitLucid/ninjawarsReply To: BitLucid/ninjawarsCc:
> > BeagleSubject: Re: [ninjawars] Remove vendor javascript from source
> control
> > (52fc2ce)Sure, just wanted to make you aware before more assets were
>
> > handled that
> > way.
> >
> > On Mon, Jan 4, 2016 at 8:39 AM, Beagle <notifications@github.com> wrote:
> >
> > > That's a separate issue that can now more easily be solved since it's a
> > > feature of the source-code but a feature of the build process. From:
> Roy
> > > R.Sent: Monday, January 4, 2016 07:47To: BitLucid/ninjawarsReply To:
> > > BitLucid/ninjawarsCc: BeagleSubject: Re: [ninjawars] Remove vendor
> > > javascript from source control (52fc2ce)This approach causes caching
> > > problems. I'm sorry, I should have made it extra super clear in the
> > issue,
> > > but in the past cached version confusion ensued from symlinked assets,
> > > otherwise we would already be using the symlink approach on that
> system.
> > >
> > > Everything caches via url string -> url string doesn't change while the
> > > version underneath does -> browsers have unbreakably cached old
> versions
> > of
> > > js and css.
> > >
> > > For jQuery specifically, since we use the cdn by url string, this
> problem
> > > will be heavily suppressed, but With most other assets that we serve
> > > directly, we'll develop caching problems.
> > >
> > > —Reply to this email directly or view it on GitHub.
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/BitLucid/ninjawars/commit/52fc2ce892365e2b0f942ce41b3f601a07c8b513#commitcomment-15243014
> > >
> > > .
> > >
> >
> >
> >
> > --
> > +++++++++++++++++++++++++++
> > BitLucid.com <http://bitlucid.com/> - BitLucid, Inc. - Web & Programming
> > Consultants
> > ninjawars.net - My webgame project
> > github.com/tchalvak/ninjawars - My open source projects
> > dnaexmosn.deviantart.com - Art Gallery
> > 585-519-7658 - Cell
> > "Never compare your inside with somebody's outside."
> >
> >
> > —Reply to this email directly or view it on GitHub.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/BitLucid/ninjawars/commit/52fc2ce892365e2b0f942ce41b3f601a07c8b513#commitcomment-15243651
> >
>
> > .
> >
>
>
>
> --
> +++++++++++++++++++++++++++
> BitLucid.com <http://bitlucid.com/> - BitLucid, Inc. - Web & Programming
> Consultants
> ninjawars.net - My webgame project
> github.com/tchalvak/ninjawars - My open source projects
> dnaexmosn.deviantart.com - Art Gallery
> 585-519-7658 - Cell
> "Never compare your inside with somebody's outside."
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BitLucid/ninjawars/commit/52fc2ce892365e2b0f942ce41b3f601a07c8b513#commitcomment-15244760>
> .
>


I still do not see where `field_related_content` is defined.


Yeah, I have to change that to get Intellij to work due to a bug. Maybe at some point we can move this to a profile.


Nice !


There is no Python2 in the short_desc maybe append Python3.4


I still do not see where `field_use_serendipity` is defined.


You know that this really does not belong here, but instead into the GBAtemp thread? :) Anyways, you only need to use the START button to reboot... you can aletr the code to use something different or just force shutdown (long press POWER) your 3DS.


چچچچچچ


intention


Done


can this case happen with this code? Look on `TypeSignature::createNamedTypeParameters` line 220 in this commit.


数据库是mysql


- [ ] このNoteで伝えないことが、いまいちわからない。Webサービスのデプロイ・公開方法を伝えたい？


I still do not see where `field_materials ` is defined.


I suppose it does (by its special treatment of layers 0 and 1) such that this only changes the loading order in order to conform to the already shuffled output order. Let me think about this a bit more.


this should depend on `python3.4` and not on itself.


> what happens when lodash releases a new version with different/new mappings -- we have to wait for an update?

Yes, assuming you want to jump on lodash `v{new, breaks paths}` the day it was released. However, I do not consider this a major drawback because:

* You are free to continue to use the lodash `v{version that works}`.
* It is an open source project. You can contribute a patch or make your own branch that fixes it.
* The `lodashVersion` Babel plugin makes sure that the behaviour of the plugin does not change in unexpected ways.


@garvinhicking I assume this is okay for you?


Why did you change this to 99 and removed the Doc parameter?


Fixed.


Thanks


Axon was being used for the pub/sub between the web server and redis. Since it was a small project, and I was worried about it's long term support, we did the pub/sub with redis natively using the redis node package.


This code only works for arrays with 3+ rows and 2+ columns.


... but this is removed in the next commit anyway.


"asdd444" and related terms are (not) trademarked by the Gamemanj Consortium.
Please remove this immediately.


LGTM.


Nope~


I still do not see where `field_taxonomy_ting_objects ` is defined.


yes


I believe the changed paragraph is an explanation for the commands above that use 'email@address.com' (not visible in the diff here, check the full view).
I agree it is a detail, it is up to you if you want it or not.


No worries man no worries.
On Jan 4, 2016 8:19 AM, "Agent Revolution" <notifications@github.com> wrote:

> fcdf809
> <https://github.com/A3Wasteland/ArmA3_Wasteland.Altis/commit/fcdf80901697534fb8569f11be77c2d22fe7d9fc>
> - will wait on BIS until further fixes, because it's getting FUBAR at this
> point...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/A3Wasteland/ArmA3_Wasteland.Altis/commit/9beb8365f6739f9d46357c89c327b4af58397348#commitcomment-15242676>
> .
>


The line graph is effort points, whereas the bars are an incident count. I think that is a good point that the bars in the first graph could have the points listed on them.


I was about to ask the same thing.


Oh, look, you're going to remove it
see, saying "nope"


I actually got rid off the _server_path in conf in a later commit. The connection pool management is handled in a similar way to mongoengine via global state. That'd definitely make the string work prettier, thx


Done.


Yes, it is in both PRs.


vf


Looking good


> Yes, assuming you want to jump to using the lodash `v{new, breaks paths}` the day it was released.

It becomes tricky when npm defaults to `^` and base/internal paths are involved. The whole point of a plugin like this is to not worry about paths which is exactly what folks would end up worrying about.


Yes, `--theme` on the command line and `theme_directory` in the config take the same args.

I am starting to doubt my decision to tie the config file keys to property names instead of command line args.


So I see that you have this branch for your app specific needs, but I think there's a better way to do this without modifying the core lib.

I intentionally exposed the templates & classification settings on `$.fn` so you could easily create your own app specific config, i.e.:

```
$.fn.classify.markings.classification.U = {}

$.fn.classify.templates.main = '...'
```
You just have to set those before you initialize the plugin.

Also, you don't necessarily need to change the CSS selectors either. You could just provide an overriding CSS file, i.e.

```
.bs-classify .label-success {
  /* your styles */
}
```


Sorry! I miss type option "Create a new branch for this commit and start a pull request" and I did a direct commit by error! What should I make to document this?


Gg


Leaving it was unintentional. It gets removed in a later commit so that loop is handled the same as the rest.


Shouldn't this be `update_type`? https://github.com/alphagov/gds-api-adapters/blob/master/lib/gds_api/publishing_api_v2.rb#L15


- [ ] このNoteは、参照先(xxx-webの説明)の中に記載した方がよいと思われる。


My guess is that with `@Defaults`, you can miss components that are created later.


<html lang="en-US">
  <head>
    <meta charset="UTF-8">
    <meta name="viewport" content="width=device-width,initial-scale=1,user-scalable=no">
    <title>Data Driven Documents Playground</title>
  </head>
  <body>
    <script src="minified.js" charset="utf-8"></script>
  </body>
</html>


this is just never true?


Should we just do this to the original dictionary instead?  (Some emacs regex trick would probably do it).


I see what you mean.

Sorry I couldn't see how else to do it?


Yeah, I saw that as well. Pretty much all of the run methods are just copied from the old code. We can go back and refine/rewrite them as a separate exercise.


I think the question is still relevant.


Yes, I have figured that out in the middle of work, but changing it now would be extremely costly, because most of the commits from parametric_varchar and decimal branches would conflict with such change.

I will rename those values as separate pull request on top of the decimal.


No worries! Saved me the time of figuring out the length and event codes.


:heart: these


what about values such as `007` or `-0`? are they acceptable integers?


Please remove dead code from the codebase.


@copenhas I'm starting to wonder if it would be better to instead introducing ets, introduce a game process and let the game process add the user to the game and it just store a map of user_id -> process_id in it's state.

So instead of Player being in the channels you have something like Game.add_player(...) and the game process receives the requests checks the active player map and handles terminating the stale player process.

I feel like the ETS table is here because you need central place for state between the player processes, and game process would serve the same purposes and take this out of the supervisor letting the supervisor just worry about starting the player.


The label type is already logged above by log_method_call.


s/i's/its/


When I was using printk for debugging I was concerned that a name passed in from the VFS could be 255 chars long  and the printk might run off the end of the string.


ah yeah. That's a problem I had forgotten. PropEr isn't in mix, and the plugin cannot be packaged along with the app because its own dependencies cannot be complete.

This works fine for rebar3 which can build the plugins in this case (it can mix source and packafe deps), but mix won't be able to handle them. Without getting the guys behind PropEr to upload their stuff to Hex, it seems we'll need to either make it so mix can skip source dependencies or plugins in rebar3 projects, or get rid of the plugins tests in `backoff`, which I'd rather not do.


Why not leave this in?


How much time will it consume to port that to a valid curl solution?


Ah, I stumbled upon jingo 0.8.1 import weirdness while working on olympia the other day, looks like I'll steal this :p


Just the commented block at the end, right?


GWT also supports system cursors. Will send a PR later.


This is not related to removing the table schema. Can we move this to another issue? In the current stage of development it's no big deal to include some minor unrelated fixes most of the time, but this makes reviewing this a whole lot more complicated :)


Do we need to finish this loop or can we just return 1 already?


@pejakm GSF is used by the C2DM client library for push notifications (C2DM is the predecessor of GCM and open-source). Older applications and some open source ones thus require GSF to be installed. As it uses the same infrastructure as GCM, GmsCore can be used to provide the push notifications. If you want this, you can install [GsfProxy](https://github.com/microg/android_packages_apps_GsfProxy/releases).


haha what? did this not just crash every time?


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


Ah ok.


yo


Hi, @kmike

I'm a newbie on jupyter.
I have installed splash and jupyter (both use manual way).
But seems there is a problem on my server.
For example:  splash:go('http://www.example.com')  on notebook not work.
The status is busy all the time and cannot get the result.

![image](https://cloud.githubusercontent.com/assets/5621825/12092846/90508434-b33a-11e5-91ba-aa904cb79358.png)


Nit: match is guaranteed to be 0


Alternatively, I think you only have one test to fix. I did this for olympia: https://github.com/diox/olympia/commit/a1fcd48a34b5bb8795bfe1e3ac7ee3e652b9a79f


Done.


leave errors behind?


why is this better? isn't explicit ~usually better


Would we not at least log the exception?


See above.


Please add a newline at the end of the file.


Please remove dead code from VCS.


I'll take a peek, but easiest way to filter out the call upstream (around RideCache.cpp:722)  if its a run.


update--


you ask for this template here : https://github.com/epitre/ansible-role-php_pecl/blob/master/tasks/main.yml#L22


Ok, let's leave it then. Could you comment on the other issues or fix them?

Thanks!


seems like we could drop this altogether


I also wanted to ask, as this is the kind of magic our users will have a hard time understanding.

Though it's definitely not about missing components I think. `@Defaults`/`@Mutate` affects only the state of the _subject._ The _inputs_ of the rule (in this case the `TestSuiteContainer`) should be closed already in both cases.

The only thing this change should affect is the order in which this rule is being executed compared to other rules applied to `BinaryContainer`. For example, after the change above, this rule will always execute earlier than `copyTestBinariesToGlobalContainer()`:

https://github.com/gradle/gradle/blob/3d77d71cf7b6c0fecc630a94ef89fc851e0c349d/subprojects/platform-base/src/main/java/org/gradle/platform/base/internal/registry/BinaryTasksModelRuleExtractor.java#L112-L121

I'm not sure what that means, though.


and this


@martinb3 :+1:


ueditor

ueditor

UEditor富文本web编辑器


haha


Can these `debug()` and `undebug()` statements be removed from the examples as well? Or at least put wrap them in a `\dontrun{...}`? It would make the examples depend less on infrastructure which is specific to GNU R and thus help those of us working on other interpreters for the R language ;-)


ok


It looks lit it is dealing with rides between two dates so there isn't a single ride to check isRun on.


I think the 0.9.0.1 was included in the merge to master by mistake


Does this mean you can drag an mvd file to the ezquake window and it will play ? Not only supporting clicking the mvd file in Finder.


I think proper is on hex by now: https://hex.pm/packages/proper

If that doesn't solve the problem, from my perspective as a user of bypass, I'd like to just use bypass in my Elixir/mix project by declaring it a hex dependency. This clashes with your point regarding getting rid of the tests, but I think that leads to a real problem, where Elixir projects can't depend on Erlang/rebar3 hex dependencies any more. I don't know what to do about that.


I'll make the change :)


add comment here


yea


Fine, Although I don't see the point in introducing this collection iterator. Code is less straight forward.


oh, I think you're right.  Not sure how we overlooked this.  @pezmaster31, is Nigel correct?


.menu-background {
        color: #fff;
        -boxpointer-gap: 0px;
    -arrow-border-radius: 24px;

there you have to set it for the theme from B00merang theming.

maybe its more intelligent to insert arrow border radius code in
.menu-background.
Here two screenshots to prove that it really worked.

Also found out that a value about 24-32 has the highest changes if not try
16. 50 doesn`t work don`t know why.

Cauise this issue merely affects windows 7 and 10 users we might focus on
providing i9nfiormationsfor those only.

regards bluedxca93

On Mon, Jan 4, 2016 at 4:02 PM, cinnamoner <notifications@github.com> wrote:

> Man I wish I was on the desktop! No can do right now.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bluedxca93/starkmenu/commit/af9a8d440444b373a32d658605d471ce863ffd2b#commitcomment-15244612>
> .
>


Hi. I don't know what you're trying to achieve, but this should work just as well:
```viml
autocmd SessionLoadPost * call g:on_session_loaded()
```


	Is there any correlation (or a causal relationship) between (adverse) weather shock/variability and crime rates?

	There exists empirical evidence which shows that adverse weather shocks increase the level of stress hormone cortisol among humans. In Kenya, Haushofer, De Laat  and Chemin (2012) [https://www.dartmouth.edu/~neudc2012/docs/paper_195.pdf] find that farmers' cortisol level increases further than non-farmers. This empirical articulation can be taken a step further to hypothesize an increased incidence of violence due to weather shocks, possibly through the channel of stress.

	In this document [https://www.dropbox.com/s/oarer53jm1abra3/Use%20of%20satellite%20based%20micro-data%20for%20geographically%20disaggregated%20economic%20analysis.docx?dl=0] I have detailed some of the interesting 'big data' sources that have been used in the empirical Economics literature in the top  journals in recent years. In our case, we can look at 'minor' weather shocks, such as higher than average rainfall in a particular week or month as well as 'major' shocks such as flood exposure and try to see if there is any statistically significant correlation with crime rates that we can get from the newspaper data.

	Generally, in the empirical Economics literature, researchers use different statistical methods to go beyond showing correlations and attempt to establish a causal link between different factors. But it is generally accepted that incidence of different weather shocks is 'random' and such randomness makes it easier to make the case of a causal relationship.

	In our case, if we can construct a panel/longitudinal data across several years and locations (say, districts or upazila) we can control for the location and time trend (they call it the panel fixed effects methods in the econometric literature) and other time-varying variables to look at the effect of a weather shock on crime rates. Of course, we can disaggregate the both the crime and weather shock into different types to locate any interesting pattern.

	I think we can come up with interesting questions using these data sources.

	I also have access to several nationally representative data from Bangladesh Bureau of Statistics on different socio-economic dimensions such as education, health, migration, etc. I think several of these 'traditional' data sources can be merged with the interesting new data sources we are talking about to produce interesting research.


This number is duplicated in LESS too. Can we make this div a class (logo?) and reuse the variable?


Does the order matter here?

If not I think we should alpha-sort the list to make it a little easier to read/maintain.


My comment to this lines has nothing to do with visual selection, so don't reference to it.
It's independent issue.
It simply you have introduced second mapping for same purpose, when you could directly reuse existing:
```viml
nnoremap <silent> <Plug>(ag-group)  :<C-u>call ag#bind#f_tracked('AgGroup', 0, 'grp', [], [], '')<CR>
xnoremap <silent> <Plug>(ag-group)  :<C-u>call ag#bind#f_tracked('AgGroup', 1, 'grp', [], [], '')<CR>
```
All works as is and there no need to introduce Plug with another name.
Then in mapping table below also no need to split mappings and you can revert changes.


review again and i'll squash and push -f again


Confirmed that we also build on RHEL 5 with gcc 4.1. I will try to set up a dashboard on this machine.


Nitpick: Nodejs is not a javascript engine. The js engine for node is V8.


This has been causing some confusion for my team and it took a bit of troubleshooting to find that the problem was not actually a problem.
It outputs the console.warn even if there are no options (or only undefined options) specified.

A nice improvement to this would be:

    if (invalid_option) {
        console.warn(...);
    }


we definitely should


Is this a potential vulnerability? As far as I can see the scriptArguments are written directly to the script file from the HTTP request so someone could escape out of the current command and run arbitrary commands?

Is there a PowerShell cmdlet to escape/encode special characters?


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


Should these be `favouritesCtrl.selection` instead? And perhaps it would be better to move this logic into the controller and out of the HTML.


would it help to also pluralize this? I'm wondering if we should switch to YAML and provide a flag for plurals (so we can add an "s" to the trigger) or to also allow plural forms (when an "s" doesn't suffice).

Certainly isn't required, but would make the triggering a little more flexible


For #7


Oops I meant constructor....


saw approval from hmmr
at https://github.com/basho/riak_pb/commit/a7832c1a15507ee49db92b613213a0b64f4b07d5


merging basho/riak_pb/feature/jrd/ts-spark-coverage = a7832c1a into borshop-integration-173-feature/jrd/ts-spark-coverage


basho/riak_pb/feature/jrd/ts-spark-coverage = a7832c1a merged ok, testing candidate = 077b1be8


I support this change, if we would make it backwards compatible for one release. Please do have a go!


should be `!==`, but maybe it should be in the controller anyway?


test this please


good


TeamCity Civic Assembly :: Build backend + frontend [Build 135](http://54.229.176.143/tc/viewLog.html?buildId=368&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


Should be single quotes.


all these should use NODE_SET_PROTOTYPE_METHOD


For #7, #2


@michalmuskala I am now passing the fields you should be selecting on select as second element here. It may be nil if there is no `schema` so you can handle it on Mongo accordingly.


งั้นเอาแบบเจ๊ก็ได้ ว่าจะหาที่ไว้คลาสอยู่ด้วย


html_keep_url is called in plaintext2html . Probably we should see how it was done previously to the addition of html_keep_url (inline code ?).


Done.


Why is it necessary to update the CryptoHash benchmark? As long as the example from the benchmark does not make use of the new instances, this should not be necessary.


>> Do you think we need another state PVR_CONNECTION_STATE_USER

> it's not really a state, is it?

 No, it's not a state. That's why I suggested to NOT introduce it. => "Kodi cannot do anthing else with this 'state'. It has no real "value" for Kodi, because no semantics can be applied. I would suggest to use existing QueueNotification callback for this kind of "just display a message" 'states'." => Would this be okay for you?


Why the name "tailed"?


deserializedData is ok like this for now
gParser we should not do.


I also detected this column calendarorder (see https://github.com/owncloud/calendar/issues/585#issuecomment-168708924).

I´m not brave enough for editing it directly on the database because of possible issues. Any time update on this?


Interesting use case. It brings up questions about user management for the future digital system that will support the Omgevingswet. E.g. What is meant with the 'Stakeholder'? Is that the current owner of a BAG Pand of can it be everyone? Is/will it possible that an user 'claims' that her or she is owner of a BAG Pand? How and who will validate that someone really owns the Pand? And can an 'Authority' be responsible for one or more Woonplaatsen (e.g. based on BAG Woonplaatsen)? In that case an Authority only receives notifications for the Woonplaats(en) he/she is responsible for.


you are right, sorry for the trouble


set to nullptr here pls


![bildschirmfoto vom 2016-01-04 16_28_43](https://cloud.githubusercontent.com/assets/4351663/12093198/d5cec8f6-b301-11e5-90af-c40ecda0a71b.png)
![bildschirmfoto vom 2016-01-04 16_29_53](https://cloud.githubusercontent.com/assets/4351663/12093199/d5f84780-b301-11e5-8df6-ba413d4b9656.png)

proof here, need to view the css , normally changing boxpointer radius in .popup-menu-boxpointer works if not try to define in .menu-background .


it needs to be `https://github.com/ChibiOS/ChibiOS.git`


Thanks.


Also, for reference, you had emailed this description:

```
For reference:
I = Insert
A = Adapter
F1 = Forward barcode
RC1 = Reverse complement of F
F2 = Forward barcode 2
RC2 = Reverse complement of F2

Symmetric:   A -- F1 -- I -- RC1 -- A -- F1 -- I -- RC1 -- A -- ...
Asymmetric:  A -- F1 -- I -- RC2 -- A -- F2 -- I -- RC1 -- A -- ...
Tailed:          A -- F1 -- I -- F1 -- A -- RC1 -- I -- RC1 -- A -- ...
```


Thanks for the hint. Thought one couldn't have a private constructor, that's why I tried the namespace approach, only getting to know TypeScript, so I appreciate your help.


My plan was to have primitive hash functions not only for CULong but also CUInt, CInt and so on. Then call these primitive hash functions in the instances of LargeHashable. Should also be faster.


The PropEr copy on hex is an old one that doesn't support 18.x, maybe not even 17.x. I see the new beta tag, it's still a bit old-ish, but possibly good enough to work.

I can try to look at it when I get more time, getting `rebar3_proper` into hex is something I've wanted to do for a while now and was blocked on this.

The clash of dependencies within Elixir and Erlang is a thing that Erlang users have never had a solution for, but Elixir was able to keep things working given how none of the build tools for Erlang really worked end-to-end. Rebar3 took over a lot more of the workflow and that's a more natural consequence of that. I believe Rebar3 hoped that eventually hex could host pre-compiled packages for basic .beam dependencies and this would solve the problem, but this seems to have gone on the backburner.


yes.


@polythene1337 fixed in v5.5.4


This script is no longer used, so can be removed.


I get a django error when editing speaker shapes in admin sometime:

`Polygon must have at least four points in each ring.`

Not sure what I did to get that and I am unable to get it consistently, but wanted to report it anyway as it did happen numerous times. I'll add to this comment if I am able to reproduce in any consistent way.


Sorry Mark, seems I am not as good as I thought I was.....

I'll have a look at what you do and see if I can learn from it.


I disagree @jdalton.

As a user, you shouldn't concern yourself with the implementation. Your concern is the API of the tool and that it does what it is supposed to do. The concern of making sure that the tool works is the responsibility of the maintainers of the tool. Therefore, your argument:

> The whole point of a plugin like this is to-not-worry-about-paths which is exactly what folks would end up worrying about.

is superficial.

I will continue to use `babel-plugin-lodash-modularize` safe from worrying about what-if. When it breaks, I will fix it. Or, as the saying goes:

> If it ain't broke, don't fix it


I think `this.props.isInSignup || ! this.props.selectedSite` is redundant here. `! this.props.selectedSite` should be enough since no site will be selected in signup.


`name is defined but never used (no undef)` fraglint
was intended to add something below like `check the component with name ${name}`? Would it be useful anyway?