That seems like a working plan, it doesn't seem like we have quite enough games to make a fully informed idea on the best way to do this, so we can do a few more and revisit this later if it's a problem.  I've been toying with the idea of making an launcher/injector instead, that would handle this duty, and get us installed in a game more reliably, but it's also not clear yet if that's worth the effort.  Apparently the injector is the way most people (SweetFX e.g.) get around wrapping conflicts.

Just out of curiosity, how did you debug the Origin crash?  I took a brief look at Crysis3 with the VS debugger, and it was blocked/unhooked by Origin, so didn't pursue it from the logging.  Just curious if you had a different debug technique that was superior to logging.


Unit tests are broken.
This disable_automatic_console_logger () is the first breaking error (paver test|run).  It does not exist.


Thomas .. 维护他的 Git 仓库并且优雅地发布补丁到 ...


包含有通用的时钟源（clocksource），新的 `timeofday` 补丁，并且为一部分架构提供了真正的高精度定时器。


hwdwyghxhyhdh bchdncjdj ncjcjsd ncjcd jccdd


Looks good to me. Will this also go to Master?


In this file, `postgres_stanby.yml`, we define some tasks.

In general this is inconsistent, if we consider:

 * `/common.yml` - tasks
 * `/postgres.yml` - tasks
 * `/postgres_common.yml` - variables
 * `/postgres_master.yml` - variables
 * `/postgres_standby.yml` - tasks
 * `/postgres_master_post.yml` - tasks

To keep a more clear codebase, I propose one or a combination of these solutions:
 * Move this logic to a separated role, either a fork of the existing one or a wrapper role, which will configure a cluster.
 * Use groups in ansible as discussed in previous comments.
 * At least rename the files:
    * `/postgres_common.yml` =>  `/postgres_common_vars.yml
    * `/postgres_master.yml` =>  `/postgres_master_vars.yml`
    * `/postgres_master_post.yml` => `/postgres_master_post.yml` (the same)
    * `/postgres_standby_post.yml` => `/postgres_standby_post.yml` (the same)


can you add capital letters to the char space?


and updated...


Weird.. yes, I cloned your repo, ran `meteor update` and tried it locally. Just did it again and it works as expected! Have you tried doing a `meteor reset` or cloning the repo again? I have no idea what could be wrong.


This `.many?` check is not functionally equivalent to the length check it replaces. Previously the locale switcher was being shown if there was more than one locale, now it is only being shown if there are more than two, as the current locale is not included in `@locales[:available]`.


read.css


the regex is /i -> case insensitive


dese.css


redistribute()의 마지막에 이미 NULL인 temp_hashtable을 free하는 것은 치명적인 버그입니다.

```
    free(engine->assoc.temp_hashtable);
```


dam it. It did not fix #14 :-/


Neat!

Postgres stores JSON too, might be slightly more performant. It also makes sure that it’s valid.
http://www.postgresql.org/docs/9.4/static/datatype-json.html


高精度定时器工作是实验性并持续发展的，它首先聚焦于让 X86 先工作起来，如果这些胜任你的要求，那么推荐使用 2.6.16 内核。该项目同时会让我们知道有哪些问题和改进。相反地，回到 2.6.8~2.6.10 时期，George Anzinger 领导的早期实现中提供了相当复杂的一套功能，但是这个项目在当时并不活跃。


assoc_find()에서 아래 코드 대신에,
init 시에 expand_table이 제대로 된 초기 값을 가지도록 하는 게 좋겠어요..
예를 들어, expand_table은 2의 N승에서 N값을 가지도록 하면 될 것 같습니다.

```
    if (engine->assoc.expand_table[init_bucket] == 0) engine->assoc.expand_table[init_bucket] = 1;
```


how is it possible?! i tested with your output O.o .... well, will check it later, now i have some work to do.


You are not commenting on the latest code, and `find_by?` has been removed. We might remove `find?`, because it does not return a boolean (unlike `save?`).


@hepin1989 ,

Could you clarify one item/point in your request for reactive-streams;

Currently, IO handling via net 4.x branch provides not exactly but similar solution/behaviour in terms back pressure and other related IO scheduling issues.

What you are asking is to provide an abstraction layer which complies the reactive-streams contract, am I correct?

P.S: I do agree that Future / ScheduledFuture is ugly compared to Scala mechanics, just trying to understand your motivation.


Das Problem daran ist, dass die BurnTime nicht im Item selbst gesetzt wird, sondern extern. Schau dir die CustomFuels.xml an, da ist ein Beispiel für Diamanten drin. Vorteil: Jedes item kann jetzt als Brennstoff verwendet werden. Fluids sind wieder vollkommen anders, aber das geht ja mit Minetweaker denke ich


建议把配置用 \`\` 括起来，例如：`CONFIG_HIGH_RES_TIMERS=y` 记得删除中间的反斜杠。。。


subbucket 이란 이름 대신에, tablecount란 이름이 더 적절 ??


tested


Looking at history of mapogr.cpp, the call to msOGRTranslateMsExpressionToOGRSQL() was removed by @sdline in a large refactoring in 0a48731eeda16af3b276be11c92cbb151400a83e . So the function can be removed now. But with this removal, I'm wondering if MapServer filters (e.g. a WFS Filter) can now be translated to OGR SQL for the compatible subset.


Clock --> 时钟
Timers --> 定时器


如何检测你的定时器系统是否支持高精度？


Adding a class with inheritance adds a lot of complexity (serialization, dynamic_cast to use it, pointers everywhere).

On the other side, the only benefit of this subclass is to avoid an extra cost in-memory with an empty std::map. The memory size of an empty map with gcc is 48 bytes. Regarding the number of images we can manipulate in openMVG, this extra-cost is insignificant.
If this is a problem for you, we can use a pointer to the map and keep only 8 bytes per view.

Are there other benefits from using a separate class?


Hi Guys,

I’m currently in Pohnpei right now, starting a field expedition (I’ll send a link to blog posts soon)..  I’m not sure I’m tracking exactly what we’re talking about getting integers vs. UUIDs.  Can you clarify what sort of object (digital or conceptual) the IDs are representing?  Are they intended to be persistent?  One ID = one text-string?  One ID = one text string from one provider?  One ID = one TNU?  What is the (precise) basis of record?

Thanks!

Rich

From: Markus Döring [mailto:notifications@github.com]
Sent: Monday, July 20, 2015 9:02 PM
To: gbif/backbone-patch
Subject: Re: [backbone-patch] Importing existing ECAT provisional names list from google code (968749a)

Good point. Although I would expect most edits to be infrequent and small, thus potential id clashes could be solved in merging. Id generation would in most cases also be manual work. So it is likely UUIDs will be manually minted and therefore also not fully safe. I think I would leave it with ints for now

—
Reply to this email directly or view it on GitHub <https://github.com/gbif/backbone-patch/commit/968749ab10c316e23d7b57c4acfd297d330f8fed#commitcomment-12249637> .  <https://github.com/notifications/beacon/AEZSL95AGYTC8ebFBFaZHAgfUfm5jicAks5ofL6ggaJpZM4Fb0bR.gif>


default_engine.c에 수정한 코드는 불필요..

hashpower 값만 있으면, 나머진 assoc_init()에서 구할 수 있는 값들임..


可以类似用 \`\`括起来：

`Switched to high resolution mode on CPU0`


тут всё правильно? оба раза positiveButton?


查看 `/proc/timer_list`


this is quite good result.


assoc.h에서 WITHOUT_DAEMONTHREAD_EXPANSION_HASHTABLE define 해 둔 상태이고,
assoc.c와 default_engine.c는 assoc.h를 include하므로,
추가적으로 WITHOUT_DAEMONTHREAD_EXPANSION_HASHTABLE를 define할 필요가 없음.


OK, what about changing its default value to true?


PR: https://github.com/OData/WebApi/pull/372, issue #195


也可以查看 `timer_list`, 可以看到列出的时钟是否支持高精度。下面列出了一份在 [OSK](http://eLinux.org/OSK "OSK")(ARM系列开发板)上的 `/proc/timer_list`，显示了时钟被配置成了高精度。


Any advantage? gcc's info page doesn't really list any.

Any drawbacks? Does it work with code that requires -fPIC as Qt does? Does it work with clang, too?


`cat /proc/timer_list`


如果不存在$CURRENT_DIR/bundle/Vundle.vim，mv就是期望的结果；否则，$CURRENT_DIR/bundle/vundle被放入$CURRENT_DIR/bundle/Vundle.vim目录中。

当时的想法是如果存在$CURRENT_DIR/bundle/vundle/.git ，说明是老版本的考虑兼容，升级到 Vundle.vim；第一次升级后，以后不会再执行该if内的代码块。

这样判断实在太纠结了，如果存在 vundle ，不管文件、目录都直接删除，然后，git clone/update Vundle.vim，你看这样是否OK？


Thanks, 1.3rc1 is now on my phone, and everything works again :-)


코드 line의 길이가 가능하면 80자를 넘지 않도록 해 보세요..
안되면 어쩔 수 없고요..


I suggest we add a prefix to the name of these tasks like: `PostgreSQL Standby | `, as [they do in the postgresql role](https://github.com/ANXS/postgresql/blob/master/tasks/install.yml#L5)

This would ease analysing the output and troubleshoot issues in the cluster setup.


下面有一些要检查的事项：


那么 `.resolution` 将是多少个 ns。


to be updated


Great response text


Hmm, we should add POWER8...


Just noticed that the file needs emptying ( or moved to a backup file). Since these are the 1052 updates.


Broke https://github.com/dustin/seriesly :(


97648014-DocIssues4906 (rglazer2) - Win64-Windows-7-VisualStudio-12: OK (1754 of 1755 tests passed)

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


Yep, your taste is different from mine - and yours is even more popular.


Ah I think that was my mistake then. I tried to use this before as well, but never tried outside the function. I guess since the function is executed in the context of the caller and yields the path of that file instead of this.


This check isn't necessary since nick is always a key from nickUserMap since we use it to generate the list of nicks for this function at least.


Maybe nmem_hashes should be:
nmem_hashes = (uint8_t *)nmem_head + sizeof(ta_head_t) + sz_funcs + head->ro_size + head->rw_size;


Indeed, thanks


检查时钟设备的 `event_handler`。如果事件处理例程是 `hrtimer_interrupt`，那么时钟被设置成高精度。如果事件处理例程是 `tick_handle_periodic`，那么时钟被设置成了有规律的滴答。


`git clone --recurse-submodules` == `git submodule init && git submodule update`. The shorter, the better & easier to remember !!

But, we'll need to update `themes` submodule often or atleast base it on a stable release/tag otherwise, someone might end up generating the blaawg with old theme :wink:


/cc @ikokostya


检查 timers 的列表，看属性 `.hres_active` 的值是否为 1，如果是这样的话，高精度定时器的特性就已经被激活了。


This will modify the argument which will result in '-profile', self.profile_path being repeated ad infinitum as the method is called. You will need to duplicate the incoming argument list and extend the duplicate and use it below.


I could do away with dateScale completely, but thought better to keep it for convenience? Or is it a bit misleading?


But you said ADCO is adc continuous conversion bit-field, not adc0 :)


:kiss:


and i feel a strong urge to add curly braces every time i see something like this :)


好了，终于 Review 完了，@hduffddybz 同学，辛苦了，继续根据 Feedback 和最新的 zh/doc/README.md 修复后，基于最新 develop 分支发送 Pull Request 到 develop 分支吧。


Please fix indentation according to edx [guideline](https://github.com/edx/edx-platform/wiki/Python-Guidelines#breaking-long-lines)


采用markdown格式 ，写 虚变量部分


why not `MAPI_E_NOT_ENOUGH_MEMORY`?


ewrewrwerwe


True


`Hash#has_key?` is deprecated in favor of `Hash#key?`.


values should be a list to be consistent.


This `when` expression does not evaluate to true. The reason is that `backups.stdout` is a string.

In order to work, we must cast the type using a jinja2 filter:

```diff
       shell: >
         su postgres -c '/usr/bin/envdir /etc/wal-e/ wal-e {% if wal_e_aws_instance_profile %}--aws-instance-profile{% endif %}
         backup-push /var/lib/postgresql/9.3/main/'
-      when: "backups.stdout < 2"
+      when: "backups.stdout|int < 2"
```


So we have three guys with three tastes - which already proves that there's for sure no single way to do it right ;-))


위의 수정을 마치면,
연속된 메모리 공간을 가진 1개의 hash table이 아닌 연속되지 않은 N개의 table table을 가지는
형태로 구현해 보기 바랍니다. 파일명은 아래와 같이
- assoc_v2.c
- assoc_v2.h


@lessless it is not good idea, because convention for token authentication is using "token" term in auth header. Using values other than token, digest and basic may break functionality of some frameworks.


I think it would be good if we made both actions - "discard", and "send to holding pen" - available as alternative values of an `INCOMING_EMAIL_SPAM_ACTION` config param - then site admins can preview what would be caught non-destructively by using the holding pen option.


Might be clearer to say "Integer or false" in the description.


catch only ImportError


what's the diffrence between `reflections` and `_reflections`?


I don't like how it's done right now and it's quite different from that we've discussed.


Having this be `false` rather than `null` would be consistent with the comment line above and with what gets set in the `configuration.rb` file.


@nickvergessen I'd like to see this implementation reusing this initSession hook to setup this special encrypted session handler - if possible - THX


@vreys agree! how about than returning 'token' instead of 'auth_token' in the authorization response?


Il manque le fichier **.gitignore** pour le dossier `tmp`


```import multiprocessing``` at the beggining
And then leave only one line ```MAX_THREADS = multiprocessing.cpu_count()```


d>

     <img src="//farm6.stati


The file  `templates/recovery.conf.j2` is missing! You might forget adding it. :(

```
TASK: [Create recovery.conf] **************************************************
skipping: [hector200715-tsuru-postgres-0]
fatal: [hector200715-tsuru-postgres-1] => input file not found at /Users/keymon/gds/templates/templates/recovery.conf.j2 or /Users/keymon/gds/tsuru-ansible/templates/recovery.conf.j2
```

Please, commit it.


http://ant.design/docs/getting-started

样式错了?


@danicampora you can define this to be 1 for cc3200 port to save some ROM and RAM.

@pfalcon should we default this to 1?  I didn't get a chance to test how it impacts performance...


the_content filterを削除。
他のプラグイン等でthe_content filterを使用していた場合、意図しない出力になってしまうので回避。


```js
 macOpenCmd: (path, pageUrl, cmd) => `open -n -a "${path}" --args ${pageUrl} ${cmd}`
```


@gokhangulbiz  I thought it would affect subscribe button's visibility but it didnt. Setting the default value to true and using as visible: !isSubscribable() works like a charm. I'm not able to test the subscribe button's visibility though. E-mail addres that I'm testing is already on the limit.


You're welcome!


```
老师，你的MarkDown好像没起作用啊！
```


Didn't know that. Will fix it eventually.


Can you remove this line and the empty ones also?


The ratio is incorrect now for FFmpeg it stretches


... and that a common code style guide is in order... :o)


This should be `headers.containsKey(USER_AGENT_HEADER)`, see #546.


Do you think the overhead of using a lambda function like this is in any way relevant here?


The idea is to have different values for both directions. This was needed after fixing the way these functions are called in 57c205ba1182e1dfe1bee7a15e80819109c530fe.


This is actually `X-Spam-Score` in `configuration.rb`, not `X-Spam-Level`.


Can you do this without introducing a state variable to the class? State variables are bad for several reasons:
 * they tend to break in unforseen scenarios (e.g. the state gets stuck somewhere)
 * they make the code much harder to understand
 * when new code gets written they might be misinterpreted or forgotten

In your case wouldn't an additional argument to the method call (can be default = false) suffice?


@RandQalan Provide logs in trace level for both versions you tested.
Btw. replacing ffmpeg itself influence AR? If not, change is in UMS code and not in ffmpeg itself.


Why there are ```|| new Date()```? if dateParser actually can't parse that should mean that this is empty/invalid input


97648014-DocIssues4906 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1240 of 1240 tests passed)

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


Check out https://github.com/cc65/wiki/wiki#This_cc65_projects_coding_guidelines :-)


Remove this from project repo too


I wasn't sure how else to handle this.
The alternatives that come to mind are
- a With(Sprite)BuildingBody inheriting WSB, but that's too close to the old system for my taste, or
- a separate generic WithBuildingAnimations trait
- somehow try to distribute as much of the stuff you listed above to other traits, away from this one.

Not sure what would be the best course of action here.


>The Mandarin neutral tone comes before the phonetic symbols. In vertical Zhuyin, the height of the tone mark is 10% of that of its base characters, while its width is the same as that of the base characters. In vertical Zhuyin, the width of the tone mark is 10% of that of its base characters, while its height is the same as that of the base characters.

@ethantw The term 'base character' usually refers to the hanzi character, not the 'phonetic' or zhuyin characters.  The base characters are those characters that are annotated. I recommend that we continue to use 'base character' only to refer to the hanzi, especially given that bopomofo tone marks are not combining characters, and that they are associated with the syllable, rather than a character. Therefore, i suspect that this should say the following. (Emphasis added just to show differences.)

The Mandarin neutral tone comes before the phonetic symbols. In vertical Zhuyin, the height of the tone mark is 10% of that of __the phonetic__ characters, while its width is the same as that of __the phonetic__ characters. In vertical Zhuyin, the width of the tone mark is 10% of that of __the phonetic__ characters, while its height is the same as that of __the phonetic__ characters.


This commit adds unprefixed functions.


rebase this branch on fixed issue1 branch and we are good to go here too


我改回来吧。。。


Rebase this branhc on fixed issue1/issue2 branch and we are good to go here too


I really like this change but could you split the new function into two parts? If it makes sense the first one should contain the two ifs, the second one the rest.


Sorry this is for #35


int b=10%2;


It has some common code with the `__is_bsd_rfc_3164()` function. We may that function in `__is_bsd_linksys()`, too.


I don't like this comment because it is the kind of thing that survives refactorings and tends to outlive its context. I would prefer to remove it.


d>

     <img src="//farm6.stati


I think we can directly write `find?(pk) || raise_not_found(pk)`. wdyt?


I still think we should atleast add the [recommended way](https://github.com/yyuu/pyenv-installer#github-way-recommended) of installing pyenv
```
$ curl -L https://raw.githubusercontent.com/yyuu/pyenv-installer/master/bin/pyenv-installer | bash
```


Benefit:
- keep the API unchanged.

Despite checking if a given string is present in a map provide a lot of flexibility i's not a solution for all the problem. Inheritance allow users to perform whatever he wants.
But for sure we can discuss on this point during hours...


this should be updated to `if (targetChangeKeys && targetPredecessorChangeLists)`


don't sudo, [fix npm permissions](https://docs.npmjs.com/getting-started/fixing-npm-permissions) instead :)


comment


values is a dict and not a list, right?


Hi Chris, I only change the values of the BC but not the type. It's still a Dirichlet BC (fixed value). Can you provide an example that shows how to change the parameter set to update a Dirichlet BC?


comment 2


sure: https://github.com/nviennot/nobrainer/commit/bbb452ae595976fd8be6bd3837c1be7326a6b6ca


Just try a different email address :) (As you know, you are able to create a PII and a wishlist and send a wishlist email through APIs. Or you can fake the api response to test UI functionalities as well.)


Use & instead of just & (ampersand)


added a PR #14


Those are IntelliSenseos, not typos.


Here is the fix.


So far it works for me. I haven't tested clang but I could do that.

This is the information I get from gcc's manpage:
```
       -pie
           Produce a position independent executable on targets that support it.
           For predictable results, you must also specify the same set of options
           used for compilation (-fpie, -fPIE, or model suboptions) when you
           specify this linker option.

       -fpie
       -fPIE
           These options are similar to -fpic and -fPIC, but generated position
           independent code can be only linked into executables.
           Usually these options are used when -pie GCC option is used during linking.

           -fpie and -fPIE both define the macros "__pie__" and "__PIE__".
           The macros have the value 1 for -fpie and 2 for -fPIE.
```
So from what I understand -fPIE is pretty much the same as -fPIC but specifically made to be used on executables instead of shared libraries. Since compiling on Linux requires position independent code, but no shared libraries are build, I though that using -fPIE might be more appropriate.


видимо нет, поправил


Hi Chris, I wasn't too sure about this until recently. I found it confusing that a general change of all existing code to camel case never happened. In the future, I'll use camel case.


Should these last two really throw? Why? I'd prefer them just to return false.

Also these three `if` statements are untested.


I think this function can be splitted into three parts:
```C
if (__is_bsd_pis_or_asa(src, left))
  __parse_bsd_pis_or_asa();
else if (__is_bsd_linksys(src, left))
  __parse_bsd_linksys();
else if (__is_bsd_rfc_3164(src, left))
  __parse_bsd_rfc_3164();
...
```


## This is a Markdown file


maybe use template function something like
::ToObject<EntityType> ?


Nechápu


This won't work since the test constructor is called on the main process during initialization and there won't be build information available until the test is actually being run.


I think @DefinitlyEvil is talking about PC, and you guys are talking about PE.


Well tried the newest ffmpeg and same

here is the debug log

http://www.mediafire.com/download/o2x81cf66i8dxtu/ums_dbg.zip


vice na issues #8


I think this good! Grammatical mistakes fixed & correct terms used :+1:


This may not be an adequate method of determining whether the entity has intersected with the floor.
Firstly, it checks the current position as opposed to the new position and secondly, by this point we haven't determined whether there was a collision along the y-axis.


> Technical details aside, GH issues and PRs are suffering from a split of communication streams.

+1

> All discussion on 0-N PRs should circle back into the issue.
> Removing the ability from PRs to contain any content (aside from code) would be positive.
> In turn, removing the ability to convert issues into PRs sounds like a necessary intermediate step to achieve that goal.

Sure.

Another solution would be to have an exclusive list of dependent issues or pull requests or how ever you would call some more prominent issues. E.g. on the right site instead of somewhere in the discussion.

The problem with mixing in discussion from the pull requests is that there it is about the solution detail, not necessarily about the problem anymore. Maybe pull requests should 'implement' an issue where the issue is the interface ;)


don't need "./"


We can also a link to the [mailing list](https://groups.google.com/forum/#!forum/nairobi-gnu). Otherwise, :+1:


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


Missed tests/webappstartup.py.

I'm unclear about how talostest.py fits into this picture and if attempting this generalization is appropriate at this point.


Is this okay?


Hi Chris, in finite_elasticity_routines.f90, "ENDIF" occurs almost three times as often as "ENDIF" (315 versus 114). So the few occurrences I changed don't really matter. I could do a 'search and replace' to change the entire file to the code style if desired.


this seems [PEP 366](https://www.python.org/dev/peps/pep-0366/) & [PEP 338](https://www.python.org/dev/peps/pep-0338/#import-statements-and-the-main-module) related...


I believe you'll need to map your fields in the `attributes` method within this class too.


WOW, Nice !


@RandQalan Please post logs from working and not working version to compare.
It will be easier to find the difference where the problem lies. Thx


I think youre missing another line here to set the appName back to the default name:
evData.appName = defaultAppName;


I belive this is specific to the videoconference room search. Maybe we could add a `minLength` parameter or similar?


I'm not sure this is wise. The original probably rightfully crashed here if there was no `content-type` (None doesn't have a `.split()`). Your newer version would return an empty string `''` instead.

Maybe throwing a more informative exception is the right thing to do? Can you provide a minimal example?


need to also check if ev type is "function", no?


LGTM :)


openToDoListFragment


I am curious about this change as the TILDE for android should use a Single Channel learned filter no?


I'm still mixed about these deeply nested conditional operators.  That's a lot of `? - :` pairs!

I'm not saying I dislike it.  In fact, I think I could grow to like it.  But I'm not used to doing more than two levels, and that's generally in a one-liner for a comparator:

    return a < b ?  -1 : a > b : 1 : 0;

It's certainly nice to deal with simple expressions here and not control-flow statements like `if-else` or `switch`.  Hmm.


Why did you release a 10.1 version with no changes against 1.0?


Hi Chris,
thanks for the hint. I'm actually aware of this - the thing here is, that we haven't really found a solution to this problem yet. This is still work in progress. I only committed these things because we wanted to have everything in the trunk for the major changes you plan. Once we have figured out how to solve this problem, I'll clean up the file and remove all comments and flags. Please ignore them for now.


taskToBeDone -> title
dateOfTask -> dueDate


isn't this method name too general ? where as you are using it for a specific purpose only. Give it a thought.


Isn't this cut&paste error? This is a copy of preceding test (`test_constraints_local_editable_install_causes_error`) with `-e` removed yet assert tests for **editable** package.


之前的 ＝ 输出失效是没有将 lastIndex 重置为 0 的原因


you should leave the dependency, it's a bad practice to depend of indirect dependency for library we use directly


Nice. Didn't know about underscores in number literals in Ruby :tophat:


unnecessary activity declarations to be removed.


Please name it BRACKETS


useless backslash and please align following pep8/pylint guidelines.


ah! good! :)


lgtm


Yeah, that is specific to the vc room search. Ok, I will change that.


Логічно правильно виносити такі функції в методи самої модельки, а в екшині вже викликати цей метод


This was reverted due to security concerns.


:sob:


done


Is it possible that `$3` is `NULL`? I don't know how it is handled.


@gokhangulbiz Thanks for the advice. Tests are clear. Should I commit my new changes directly, or should I create a new PR with a clean commit?


```c#
var utcOffset = timeZone.GetUtcOffset(DateTime.MinValue);
if (utcOffset >= 0)
{
    ...
```


Thanks for the fix! Is it possible to publish new gem version (0.9.1) that includes this fix?


тут просто конвертни json to dict
а далі просто зможеш писати:
if 'date_joined' in res:
    date = res['date_joined']


зліва - те що буде в базі
справа - те що буде видимо в адмінці


Still remember


 mods/xpanes/init.lua this broken all old panes in the world, bring xpanes:bar_[n] to "unknown node"
![screenshot_20150720_145001](https://cloud.githubusercontent.com/assets/3438730/8775206/a176ad68-2eee-11e5-9af0-20266f6b62af.png)


A cleaner solution would be:
           retries=17,
            retry_delay=timedelta(seconds=600),
            *args, **kwargs):
        super(ExasolOperator, self).__init__(retries=retries, retry_delay=retry_delay, *args, **kwargs)


@joka  can you maybe comment here what you explained to me?


Err, I meant #32


I thought that the maven.findbugs.failure.strict property was the "maven standard" way to do it.

It is used that way in the [literate plugin](https://github.com/jenkinsci/literate-plugin/blob/master/pom.xml#L74), the [credentials plugin](https://github.com/jenkinsci/credentials-plugin/blob/master/pom.xml#L74), the [Cloudbees endpoints pom](https://github.com/cloudbees/endpoints/blob/master/pom.xml#L46), and the [scm-api-plugin](https://github.com/jenkinsci/scm-api-plugin/blob/master/pom.xml#L75).


@burmanm last time we ran the Sonar checks, I had tried (without success) to find a way to ignore empty catch blocks with an exception named "ignored". It's common in Java to ignore a useless exception. Do you know if that's doable?


It is possible that the variable may change in future. It's better to keep the height in code rather than depend on NavigationBar.styles.General.TotalNavHeight unless it is officially exported.


Doesn't the < need to be surrounded by a {@... ?


"Arbetsbeskrivning", kanske?


Why atof and not lexical_cast?

Then you can wrap it in a try-catch block for safety... particularly useful if this is user-modifiable input.


출력할 수도 있습니다. UTF-8 BOM일 수도 있고, 공백 문자일 수도 있고요. 무엇보다도 `before_module_init` 시점에서 실행되는 애드온은 완전히 엉뚱한 내용을 출력하고 `exit()`해 버릴 수도 있죠.

만약 애드온이 PHP의 기본 출력 버퍼 용량을 초과하는 내용을 출력하고 `exit()`할 경우, `register_shutdown_function()`으로 지정한 세션 상태 체크 함수가 정상 작동하지 않게 됩니다. 세션을 시작하기 전에 이미 뭔가가 출력되어서 쿠키가 구워지지 않을 테니까요. 그래서 애드온 실행 전에 `ob_start()`를 호출하여, 애드온에서 뭘 출력하더라도 세션 시작에 지장이 없도록 했습니다.


"+= -padding" vs. "-= padding" ?


use `Charsets.UTF_8` to avoid handling exception. It should be in guava, or apache commons lib


_bueno_ :+1:


Thanks Kirill, I will take a deeper look at this. There's effort made to be threadsafe in these functions, and I have a vague recollection of a possible leak as a consequence.


Will this create a space between the Move and the closing tag > in the output?

Is there any chance that {@link Class<Move>} works? If I recall correctly, it doesn't, not at least in JDK 6 (and for now we still do our release builds with JDK 6, even though we want our builds to be JDK 8 compatible too).


    key.split("_").map(function(s){ return s.charAt(0).toUpperCase() + s.slice(1)}).join(" ")
reappears here after line 14. This can be optimized.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.387](http://teamcity.monogame.net/viewLog.html?buildId=8969&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 522, ignored: 6 Build time: 00:21:54


Move the logic to a private method.


Yeah I have some thoughts about that...
See #1595


shouldn't this be tns-ios?


Interesting. I wonder if there's another solution to that issue. Great work here btw.


@dpgeorge awesome! I'll enable it! Thanks!!

> On Jul 20, 2015, at 1:08 PM, Damien George <notifications@github.com> wrote:
>
> @danicampora you can define this to be 1 for cc3200 port to save some ROM and RAM.
>
> @pfalcon should we default this to 1? I didn't get a chance to test how it impacts performance...
>
> —
> Reply to this email directly or view it on GitHub.
>


Hi,

Adding the **--yes** flag removes backwards-compatibility for older versions of dh_make, hence the cheating by using echo to fake Enter ;)


I'd rather we not change indentation in an unrelated commit.  The indentation and white space handling in the plugin source files is a mess, but I'd rather have smaller diffs and retain the mess than have incremental changes where I need to activate my mental "white space filter".

I proposed a year or two ago to automate the formatting of files in the tree so that these types of white space changes would be less likely.  The other contributors felt that was too large a change.  I've tried to honor their wishes by generally reducing white space only changes.


As the repo usage grows, we might have to add guidelines for creating issues. Otherwise, :+1:


Yeah we appear to do this a few times.  It should probably be refactored into a mixin.


Good call. OpenShift doesn't work above 1.7, just so you know.


Very nice!


ㅋㅋㅋ요새 코딩 열심히 하는중!


I prefer having no empty line after a heading section. Anyways, empty line or not, the end result from markdown parser is the same :smile:


Я думал, здесь что-то интересное,  а оказалось просто реклама Snapster


$iCmbSearchMode = 2 crashes though, that will be a bit trickier


Fusion/Synchro/XYZ monster can be added to main deck in the puzzle mode


Use https://github.com/Kdyby/Doctrine/commit/b917799a185e395af4b2195537a6de70147b997c instead


Corrected in https://github.com/mattcollier/bedrock-issuer/commit/e8c8cdade15e3ffb9576ca4c52bbfd5230edb4e4


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

VSO State: Done


<a href="return" onclick="javascript:alert('ok');"/>


OK, let me first get a better idea of what is happening here:
~~ - Why do you implement `IPlaceBuildingDecoration` in the Info?~~
Ah I get it now - to draw the RangeCircles on the ConYards when placing the building. I think that would be best suited for inside the `PlaceBuilding` trait.
 - Why are you setting up the animation inside `BuildingComplete()` instead of the constructor?


Hello


I can't find a default value for this?


It's time to continue....


Add tests, such as '0001-01-01T00:07:59.0' for +08:00.


not sure why this was translated in the first place


I figured we set on a style, and since most of the original contributors didn't use them, I figured it's best to just have a consistent style then to battle over semicolon/not. I'm using standard which just works. Personally this no semicolon thing is new to me and I'm still getting used to it.

Although, if it's not something that you want to see, it's not too late to change that, but we'd have to pick an eslint style guide.


test


Add tests, such as '9999-12-31T21:59:59.9999999' for -08:00.


Damn. I totally missed this. The fix is incoming.


Nice idea!


make this easier to review

look for null pointer access


From the PECS (Producer extends, Consumer super) point of view, the usage of <? super T> is wrong here.


Hi.

Maybe you have files for Catalyst 15.7 for kernel 4.1.X?


good modify


fix commit message to explain why I did this.


спасибо! статья незавершена, в течении недели рассчитываю добить, заодно выложу сорцы терминала.


many test


yeah, it should be. I copy/pasted from the interwebs and didn't check that


Done.


There is no `baobab.min.js` in src.


I used a combination of SpyStudio to get an idea of what it was doing, and IDA to dig down into the details. Plus a bunch of trial and error in 3DMigoto to see how it's behavior changed as I tried different things.


assertThat(Iterables.size(objects), is(numberOfObjects));


I agree with your comment.  I tried the proposed change and it worked as expected.


assertThat(size, is(numberOfObjects));


wrap the line at 120 chars.


assertThat(obj, hasKey(key));


ça marche la. Tu penses pas qu'il faudrait aussi afficher une idée en cliquant sur le titre et non pas seulement sur le bouton + ???


assertThat(obj.get(key), is(value));


Found a workaround. all good for now. Thanks again :+1:


assertThat(obj.get(queryKey), is(elementValue));


Sapratu


Hi,

We have implemented this fix but still get the same error of "Cannot read property 'indexOf' of undefined" from line 105.
In addition, we see that you are still using angular.element(el).scope() in line 55, is this fix complete?

Thanks,
Yana


Thanks @beckettsean I will try to spend some time on it very soon


assertThat(obj.get(queryKey), is(elementDbObj.get(queryKey)));


Students/Draghici I. Vlad - Ioan/homework/src/homework/sum.java

Why `Object`?


why cast?


Take a look at [this post](math.stackexchange.com/questions/9259/find-the-sum-of-all-the-multiples-of-3-or-5-below-1000) and see how you can do this quite a bit more efficiently


There was an request on the german forum. My first reaction was that this will not make sense but someone showed and example: Schubert Impromptu Ges-Dur op. 90/3 bzw. D899/3: http://burrito.whatbox.ca:15263/imglnks/usimg/4/4d/IMSLP39751-PMLP02062-Schubert-Impromptu-Op90No3.pdf (last page in the middle).


Students/Draghici I. Vlad - Ioan/homework/src/homework/palindrom.java

why not return boolean? why return `0` or `1` and have the return type `double`


I would agree with this change as well.


why not return boolean? why return `0` or `1` and have the return type `int`


if you'd return a boolean value, you could just use `return (b==nr)`


Does this work? I mean my ARM's cpuinfo looks like this:
```
cat /proc/cpuinfo
processor       : 0
Features        : fp asimd evtstrm
CPU implementer : 0x50
CPU architecture: 8
CPU variant     : 0x0
CPU part        : 0x000
CPU revision    : 1

processor       : 1
Features        : fp asimd evtstrm
CPU implementer : 0x50
CPU architecture: 8
CPU variant     : 0x0
CPU part        : 0x000
CPU revision    : 1

processor       : 2
Features        : fp asimd evtstrm
CPU implementer : 0x50
CPU architecture: 8
CPU variant     : 0x0
CPU part        : 0x000
CPU revision    : 1

processor       : 3
Features        : fp asimd evtstrm
CPU implementer : 0x50
CPU architecture: 8
CPU variant     : 0x0
CPU part        : 0x000
CPU revision    : 1

processor       : 4
Features        : fp asimd evtstrm
CPU implementer : 0x50
CPU architecture: 8
CPU variant     : 0x0
CPU part        : 0x000
CPU revision    : 1

processor       : 5
Features        : fp asimd evtstrm
CPU implementer : 0x50
CPU architecture: 8
CPU variant     : 0x0
CPU part        : 0x000
CPU revision    : 1

processor       : 6
Features        : fp asimd evtstrm
CPU implementer : 0x50
CPU architecture: 8
CPU variant     : 0x0
CPU part        : 0x000
CPU revision    : 1

processor       : 7
Features        : fp asimd evtstrm
CPU implementer : 0x50
CPU architecture: 8
CPU variant     : 0x0
CPU part        : 0x000
CPU revision    : 1

```


They are, in the next commit https://github.com/oskopek/optaplanner/commit/fd03e24cc43f4fca7f66ff78f14b9f4bc4f5c71e. Sorry about that.


OK so this includes 2 functions from elsewhere and copies some functions from `autotest/client/base_utils.py`. There are some nitpicks, but they origin from autotest, so I guess this patch is OK.


Example: Instructor Razuvious

OLD:
90 * ((35.9047 + (268 / 14)) * 3.5) = 17339.9805 ( Bestiary 17299)
90 * ((1.5 * 35.9047 + (268 / 14)) * 3.5) = 22994.97075 ( Bestiary 22934)

NEW:
90 * ((35.775 + (268 / 14)) * 3.5) = 17299.125 ( Bestiary 17299)
90 * ((1.5 * 35.775 + (268 / 14)) * 3.5) = 22933.6875 ( Bestiary 22934)


i could set the ring tone a couple month ago already, but once i set the ringtone on android, i cannot change it anymore, also i cannot deactivate the ringtone without deactivating all the ringtones on my device


Example: C'Thun

OLD:
1 * ((35.9047 + (268 / 14)) * 2) = 110.0951143 ( Bestiary 110)
1 * ((1.5 * 35.9047 + (268 / 14)) * 2) = 145.9998143 ( Bestiary 146)

NEW:
1 * ((35.775 + (268 / 14)) * 2) = 109.8357143 ( Bestiary 110)
1 * ((1.5 * 35.775 + (268 / 14)) * 2) =145.6107143 ( Bestiary 146)


аааа, точно, я уже переписал функцию, поэтому и работало))


> is it okay to use a destructive operation here?

i guess yes, but you never know


Thanks!


I added this change to support full disactivation of the logger.


you should add scikit-learn.github.io, too, I thin. currently the site is failing again. There seems to be some weird redirect going on.


haha, really looks like a cat's handwriting


@atfornes please do not add to the scope things that are not used in the vew


hi @mitsuhiko - this commit seems to have killed the changelog - is there some place else to look for changes? - can't seem to find it :)

thanks!


Great! Thanks!


I give a tutorial in an hour, it would be good if at least the backup would not be down at that time.


What's up with 'elementOrFail'? Can't find the definition of it anywhere


but passing enable to `winston.Logger` makes no sense, it is a third party modules, so we cant change it, and it doesn't accept the parameter by default.


Related #159


    My bad, I hadn't noticed: normally I review the entire PR change,
    not the individual commits, but due to the <p/> verbosity, it
    was more practicable for me to review the individual commits (and
    take the <p/> commit on faith).
    In any case, it doesn't matter: as long as the code ends up in the
    correct state, it's all good.

    Ondrej Skopek wrote:

      They are, in the next commit fd03e24. Sorry about that.


Why did you leave `Access-Control-Allow-Origin` there?


@DefinitlyEvil The server is up now on my end :D


I though this will append those props to the returned object. It was not tested at all.


\<insert emoji in emoji here>


much emoji :+1:


I like this very, very much!  It places the exclusion directly into the source code so that I don't need to refer to a separate findbugs exclusion file.


Я бы написал
```ruby
AmazonAwsClient.try (is_public_bucket ? :delete_from_public_bucket, :delete_from_private_bucket), zip_file_url
```
Но это чисто мои заморочки)) Если понравилось, используй, нет, оставь, как есть)


@niryariv should be `have taken the`, right?


no as far as i saw in code, winston just takes the properties it supports from the object to configure logger


> I give a tutorial in an hour, it would be good if at least the backup
> would not be down at that time.

Darn, it was working for me a minute ago.


Easter egg should probably be removed


to fully disable logger, i think we will have to wrap the logging methods created by winston in our Qlogger module.


The flaw is when the value is encoded for JSON, not after the input. So you're fixing it at the wrong place.


Возможно есть смысл сделать отдельный метод, чтобы не дублировать код
```ruby
private
def self.delete_from_bucket bucket, key
  self.client.delete_object bucket: bucket, key: key
end
```


The defaults comment meant: This should go to Kappa defaults in KTuple_cff and this white list definitely should.


The whitelist should be a default.


all is good now.


thanks @ogrisel


defaults and commonly used settings still have to be identified


Well I had b34690bd656047d9718d3bf512610f0739512770 and it worked but now that I compile it again it is not working and that is the one before this so I still think something with FFmpeg


make: *** No rule to make target 'shader/compiled/lcd3x_vcg_gxp.o', needed by 'libvita2d.a'.  Cancelled.


Думаю, лучше сделать так
```ruby
  after_validation :schedule_theme_delete on: :create, if: "errors.present?"
```


right


What is jwesc? ;)


Looks good, thanks!


> all is good now.

Yes. DNS cache is playing tricks (as always) on some machines. We'll have
to wait for expiration of cache for things to work reliably for everyone,
but the fix is there.

Hurray!


Submitting a single fix for the purposes of demonstrating to Raphy. The compound in question isn't in a reaction so isn' relevant to the biochemistry.


I think it was meant to be... I believe the right XA protocol is to do an xaEnd before calling the commit, meaning the Sesssion shouldn't have an XID.


I found the issue with B1 Growing Degree Days


An extra string on the end of the amp; so we can unescape them again without doing any that were already amp;


Can i please get the instructions so that i can get this application to work properly. Thank you.


There are many places where you call some function and there are no spaces after commas between parameters. Please fix them.


move to ShortDecimalType?


ditto


No space after 'if' and there are few other similar cases in the changed code below.


You have to start the thread, it won't magically do that for you :P


@tridge already commented on this one. See https://github.com/emlid/ardupilot/commit/6d23ed5540d4751f82f246f2b9eab2c27e6b3597#commitcomment-11943732

If you think it does make sense, please reply there so we can discuss alternatives rather than re-sending the set with this patch.

The possible use of it is also commented on https://github.com/emlid/ardupilot/commit/0f7a7b76f45342cec485ce2832478101b174d7d8#commitcomment-11943832


## This is a Markdown file


Creating new file


Replace this one with one "else if" in line 255.


I've changed the most of the lines in the method, hence I decided to "properly" format the header as well


Typo here: `classs`, I guess it should be just `class`.


LOL, funniest commit.


>A WCHAR if UNICODE is defined, a CHAR otherwise.

https://msdn.microsoft.com/en-us/library/windows/desktop/aa383751(v=vs.85).aspx#TBYTE


That's an awful lot of changes. For which compiler does this help?


Where is this notmer used?
And isn't it possible to have it true by default and only check if we're running on mer instead of adding all the distros here?


Move line 382-400 to a private method, and then re-use with above `ApplyTo` method for `IQueryable`.


Seems wrong indent here and " ".'filter_jwplayer_media' can be rewritten as ' filter_jwplayer_media'


You've tested only one scenario. What about cases when there's no ```no_auto_import``` declaration, when functions in ```no_auto_import``` mentioned as tuples (see http://www.erlang.org/doc/man/compile.html), and may be some other cases?


Tested clang 3.6 and gcc 5.1 here.


Do you also have the formatter settings used to get this output? While I appreciate coding style cleanup I would appreciate having it reproducible, so that I can prevent this from being reintroduced later on.


Couple of general points here:

If you delete values that are null, you'd be allowing the underlying API to set the default value. After a quick pass in the API, it seems these aren't hard-coded and as such, would be returned as 'undef', I believe. This is functionally the equivalence of saying 'null' but has implications if attempting to use the data programmatically.

You're loading compartments, should we consider eliminating the compartment object from the biochemistry entirely?


thanks, fixing


> There are no known sysfs "timing bugs" since a long time.

Except for the trackpoint driver whose attributes are created a little bit later such that the following (common) rule does not work anymore:

```
SUBSYSTEM=="serio", DRIVERS=="psmouse", WAIT_FOR="/sys/devices/platform/i8042/serio1/serio2/sensitivity", ATTR{sensitivity}="150", ATTR{speed}="150"
```

The commit itself is ok (regarding clean code), but the assumption in the commit message is wrong (or I misunderstood it).


See https://docs.moodle.org/dev/Coding_style#Arrays syntax (I know, it was written like that and I did not spot before, but would be good to change)


you should check if result actually fits precision and throw ArithmeticExpression otherwise


We can't because the ID is not mandatory in `source` elements


@MacDada nad returnem dodałem to:

            $this->doctrine->insert('posts', array(
                'title' => $_POST['title'],
                'body' => $_POST['body']
            ));


No need for tab here.


`renderMTConfig` already gets called by passing `provider` as it's first argument which is available from the outer scope of the anonymous callback method. Is there really a need to pass `provider` along with `d.data.config` from backend?


good


No problem, I am glad you could isolate your issue!


QtCreator just replaced all spaces with tabs (to be uniform and don't mix spaces with tabs) and removed trailing whitespaces.
I would recommed using clang-format, astyle, uncrustify or something like this.


Thanks. I cherry-picked the patch to staging now.


Where are tests with negative scenarios?
Is this test "add blog" for connector?
Is it possible to check not only presence on UI but also in repository, cause this test covers only UI layer?


After reconnecting shouldn't login workflow calls like sendGetPrivacyBlockedList() and sendActiveStatus() be made as well? As in a normal login flow?


Should be `.event` not `.evnt`


help


this should be before the poll(), otherwise you'll be calling poll() even if timeout == 0.


Is Adam still actively developing nestedvm?


hello


If this prevents the use of the base class's `load()` then is the overload of `IO_stat load(IO_handle stream, uint32_t version)` needed?


Remote the empty line.


I believe the "mismatch" was probably just a "burp" caused by pressing the tab key or something. Some extraneous whitespace has crept in here which could be removed.


Beleza! Acho que usando o Paperclip, a gente nem precise da tabela Media… vou investigar


That's a good question, I think I claim temporary idiocy.


Why not just leave this out, then any use would cause a link time error rather than a runtime error.


I'm not sure where you are going with this...
Is the idea that the settings `forceHeight`, `forceWidth` and `autoSize` can be changed dynamically?
I'm trying to think of a use case for that...

In my mind, these settings are 'backward compatibility settings'. Normally people should be able to leave them at the default values, but if upgrading flip from the old version gave issues, they could use these settings to get back the old behavior. I'm not sure why you would want to change them dynamically.


Looks like this commit introduced a regression. See https://github.com/crsmithdev/arrow/issues/246


If you add tests with partial fill of form, of course you split such methods into 2. fillFormat() and fillLang().
It is a suggestion.


Yo!


sda


No it will not.

`{@link Class<Move>}` doesn't work in JDK7, and it actually raises an error in JDK8: `error: type arguments not allowed here`. It should probably be left as is.


- for find, s/add/get/ and insert


Chapter 03 - Sorting Algorithms/BitwiseSort2.Tests/BitwiseSortTest.cs

Why don't you just break from the loop after failed assert?


That's good to know. Thanks for testing it out.


Would it be helpful to put an assert here, so that if we add a new security method in the future we will be reminded to update this switch?


Actually, the tabbing is completely wrong, which is masking a bigger error. What you currently have is this:

```
.event {
  background-color: #fff;
  padding: 20px;
  border-radius: 10px;
  box-shadow: 0px 0px 10px rgba(0,0,0,0.5);

.event  h3 {
    margin-bottom: 0;
  }

.evnt  time {
    display: inline-block;
    font-size: 20px;
    padding-left: 10px;
  }
}
```

But what you want is this:

```
.event {
  background-color: #fff;
  padding: 20px;
  border-radius: 10px;
  box-shadow: 0px 0px 10px rgba(0,0,0,0.5);

  h3 {
    margin-bottom: 0;
  }

  time {
    display: inline-block;
    font-size: 20px;
    padding-left: 10px;
  }
}
```

This is because the `h3` and the `time` are nested inside the `.event`.


@jhoeller, I see you beat me to the fix, but I pushed my additional _polishing_ anyway (0153913ef4597e6a47517f763961a4c6cc232443). :wink:


Next time create new branch for new futures.


@nick22891 to change the version number of angular being used, please do it in the bower.json file.  As part of the build process, bower runs and installs the application dependencies.


Review complete, one minor question logged


So, this is check whether title is shown somewhere on page. I would rename it to isPostShownAmongSaved and change appropriate assert message to "Post wasn't found as saved on page"


I didnt know this syntax "data: _data" but it seems to compile, so fine by me !


임시저장은 stash를 활용해요 우리~


I don't like this, preferably we should have scale and precision computed all the way. Maybe we should add support for computed types in ParametricOperator constructor? We could also pass computed literals and return type to signature() method to avoid computations redundancy.


Add unit tests for the change to this method.


TeamCity Hyrise :: Unit Tests [Build 85](http://ares.epic.hpi.uni-potsdam.de/hyrise-build/viewLog.html?buildId=93&buildTypeId=Hyrise_UnitTests) is now running


Thanks, @dorileo. I'm going to write what's missing for sol-fbp-generator and I'll get in touch with you for future PRs, so we integrate everything at once as pointed by @ibriano.


@ksooo: Nice one - works well with my pvr.mythtv branch :)


음 그렇다면 이것은 `session`과 관계 없이 기존에 XE가 가지고 있던 버그였겠군요?

애드온 소스를 찾아보니 그 대표적인 예가 captcha네요....


Add unit tests for the change to this method, besides existing functional tests.


Agree. I'm going to work on what's missing for sol-fbp-generator and this is one of them.


Hi,
With this commit quilt are badly rendered here.
Bad:
![Uploading Capture du 2015-07-20 13:41:05.png…]()

Good with the commit removed:
![Uploading Capture du 2015-07-20 13:41:42.png…]()

Does it ring a bell?

Didier


You can just check for "no-window" here I think. By the time a request reaches Mixed Content "client" is normalized away.


I'll split the commit. Thanks.


@miq: can you check if that also fixes this no-frame-problem for you? If so, I'll release a bugfix version ASAP.


기존의 XE는 항상 세션을 먼저 시작하고 애드온들을 호출했기 때문에 문제가 없습니다.


You should also add the "(DEPLOYED)" comment :)


When enable relaxing, the service ...


@lucasdemarchi

It's been fixed in the next commits.


Hrm. Ok, so how do we distinguish a document's `fetch([insecure url goes here])` from `fetch(event.response)`?

I think the former starts with a `window` of "client", doesn't it?


On my Spony Xperia Z3 this is the correct naming for orientation.


@ldoktor true, he suggested something else that I don't remember right now


아하 이해했습니다~


@riobob29 Could you provide more detail? I don't see style.css referenced in http://learn.jquery.com/jquery-ui/environments/amd/.


Sure, it's kind of against the PEP8. I can fix it.


@lucasdemarchi You're right! Thanks. That's a typo. I'll update the PR accordingly.


Is this incomplete ?


typo bro


Yeah, you can check if client equals window. (Or you could explicitly check client's global object being a Window object.)


TeamCity Hyrise :: Unit Tests [Build 85](http://ares.epic.hpi.uni-potsdam.de/hyrise-build/viewLog.html?buildId=93&buildTypeId=Hyrise_UnitTests) outcome was **SUCCESS**
Summary: Running Build time: 00:09:17


hmm, you'll have to show me.  it looks to me like it is just getting lost as of now.   Since the handlers also just return a plain `error` type now anyways, we can just return the plain old `err` directly. although I'll need to look into what the newer `HTTPError` wrapper gives us


Back it


`EnableVersionConstraint`?


Use ' ' symbol


to/too


in fact last message received will be of greater id every time it is costly check


Use small size of image/ 1367x768


I did not find any rpm macro %define saying spec is executed in Mer (like suse_version or fedora).

Hm, now I do not see it is used anywhere here. So, maybe I took it from my WIP branch.


Just to be clear, these changes work locally on your machine because you've swapped in Vega 2 locally?


I don't know why I did that. Fixed and re-pushed.


Please delete this field from the struct. It's usage was removed by this commit so it shouldn't stay here.


I think it would be easier to review and understand the changes if you split this commit into multiple parts (with a little description why the refactor refactor is needed).

As far as I can see this commit contains at least the following changes:
* indentation fix in grammar
* swap `lo` and `hi` in `ipv6_parts`
* rename `_in6_addr_compare` to `_in6_addr_compare`
* rename `get_network_address` to `filter_netmask6_calc_network_address`
* split `filter_netmask6_new()`
* and those I missed :)

The function renames don't need much explanation but why `is_valid` was removed may worth some words.


why clone only shape[0]? There can be multiple output to this node.


@pratikbutani Please refactor your commit. It contains files from the `.idea` folder which should be excluded via `.gitignore`.


why do you clone it before calling scan_utils.infer_shape? I think you only need to clone it after it on line 154 and not here. This would speed up this method.


Correct. I changed [this line](https://github.com/johnmyleswhite/Vega.jl/blob/575a6274abad6309cc69c59e648f01fa83f02743/src/render.jl#L68) from

```julia
println(io," <script src=\"http://trifacta.github.io/vega/vega.js\"></script>")
```

to

```julia
println(io," <script src=\"http://vega.github.io/vega/vega.js\"></script>")
```

to get it to work.


RFC 7301 defines only "http/1" plus the spdy-related ones. So if this is really about "http/1.1", we should say so.


bug here, no adjustment for negative decimals, use unscaledValueToSlice maybe


I can't see your capture images.  btw is this with or without opengl?

Basically the easy way is to not use this commit, but then we waste a little time rendering to an area that will be covered anyway.

Can you try against my pending pull request:
https://github.com/OpenCPN/OpenCPN/pull/335

I want to reproduce it but I cannot see your shot.  I know there is a bug with this in opengl mode which is actually fixed in my projections branch but not in master dealing with the rendered chart bar, maybe it's the same.  I wasn't going to bother with it if the branch can be integrated before 4.2 stable release.


It feels, we have abstractions from multiple levels processed in one place.

We're building an ErlangTarget in some context, it currently means we have to compile sources and process app config files in some way.

But this code says we have to collect dirty files first. We'd better move this logic one level below where we do the actual compilation/.app files processing. One more point to that is if we add support for some other kinds of compilation (resources copying, running code generators, etc.), we will be able to simply add another build command which knows all about it's execution.


Struggled a whole day yesterday trying to push and update my Github files, but failed. Refork the repository from Roger's Github and pushed again. So all the files are after the deadline, and no updated SHA-1 hash number.


There should be ```plugins/``` too.


add tests for this


Oh, ok i understand now, i missed the  ```provider``` param in ```renderMTConfig```, this method can be polymorphic based on ```provider``` param.

Right


Good to see that this code is in now. The implementation is surprisingly difficult due to lack of appropriate support by OpenSSL.

When integrating non-trivial code sections like these, please acknowledge them as contributions by Siemens AG.


saxasxas


I don't like this class, we might want to use literal computations to always have scale and precision evaluated in ParametricFunction


xasxasx


I would put the `free()` call after `filter_netmask6_new()` and before `YYERROR`.


format code


hello2 = `hello2.html`
`'name'` = `hello2.html`::`name`


make sure that result fits


1. We build simple UI test only for **success** scenario. I want to run it every time when we deploy demo env from scratch. So i need a smoke test to answer to question - is demo evn ready or no.
2. Selenium doesn't have access to repository


/customfuels help
wirft bei mir nur fehler raus das er den command nicht kennt.


Dear @malk : we don't use Guava in atom-jaxb


Selbst /help funktioniert nicht mehr


Po co return na końcu funkcji nie zwracającej niczego?


nice new macro


Feed output files right after you copied them.


I think it would be easier to review the code if you split this commit into smaller ones:
* remove `is_valid`
* swap `hi` and `lo`
* function renames
* splitting `filter_netmask6_new()`
* and those I missed

If the change is not renaming I think a short description why it was needed would be also helpful.


It's with opengl, linux 3.13 32bits, radeon free driver

I have tried #335 it's the same.

Second try for screenshots....
Bad:
![capture du 2015-07-20 13 41 05](https://cloud.githubusercontent.com/assets/12138026/8776654/1c8ea5ec-2ef3-11e5-9e9d-74e312bc4ce7.png)

good:
![capture du 2015-07-20 13 41 42](https://cloud.githubusercontent.com/assets/12138026/8776674/3f318862-2ef3-11e5-8d22-2c1bdfcd53ba.png)


Well, I think the plan is to remove makefile from the repository but only keep makefile.boot, right? In that case, people won't be able to do "make all" on a newly cloned repository.


looks good to me; awesome !


Shouldn't this be "license"?


add negative tests too


typo: Shouldn't this be `g++`?


Why is the string name of this field gone ?


unscaledValueToConstant?


Take the logic from the `routes.php` to `PagesController.php`
Then tell the route to use method `home()` on the controller.


Building with clang 3.6 works too.


If a Feed contains an author, entries MAY have one also.
If a Feed doesn't have an author, all the entries MUST have an author.

So we can't validate each entry by it's own.


@mswiderski Just a sanity check: are you sure you want to move these classes to the public (kie-api) module?

Is there a reason that they can't start in the `kie-internal` api?

Also, IMHO, we should try to keep POJO's out  of `kie-api` -- but that's just IMHO. :)


This goes after `SOURCE_DIR` (or directly under `CHECKSUM_SHA256` if there isn't one).


Why is the string name of this field gone ? (Additionaly, I'd write default=True instead of default=1, even though it's the same result)


No `CHECKSUM`?


Why does this field still have a size parameter and not the others ?


Shouldn't this have a `the`, and use the same `${...}` constant used in the `SUMMARY`?


It costs an additional code path which
 1. lowers readability
 2. won't be tested (or we need to run the tests twice)
 3. confuses future maintainers (so what exactly is the reason to implement this part using numpy? esp. because it is the only use in the entire code base)
 4. slows down the code

In short, I don't agree, because of the [KISS principle](https://en.wikipedia.org/wiki/KISS_principle).


left some comment here it seems :)


I'm a bit lost here. Why is it possible to extend assigned request but not the unassigned requests?


Not a big deal but one think I learned is that it's usually better to use `for _ in xrange(download_retries)` and `break` when we finish. This way you don't end up in infinity loop when someone screws your code (eg. forgets about `dowload_failures += 1`, or adds `continue` somewhere). Anyway your code works fine, I just wanted to point this out.


typo: comma should be before space.


Why is this relevant?


Okno należy do menedżera stanów? Nie powinien to być jak już jakiś pointer inicjalizowany z zewnątrz?


Wouldn't it be better to throw a `LogicException` here? Using OwnershipAssertion without having implemented the `ProprietaryInterface` seems like a user error.


Damn, I forgot to remove unused imports.


Remove commas after years. (Not sure it's exactly possible to add a lint for this...)


OK


effecting -> affecting


Why didn't you keep the read ? By looping on the recordset like you do, there is cache prefetching which is done, so all the fields of all records of this model are prefetched in the cache. It may not be a problem in another method, be here we are in a name_get so we obviously only need the name. I'd revert to using a self.read.


No, the correct ordering is:
```
PROVIDES
REQUIRES

PROVIDES_devel
REQUIRES_devel

BUILD_PREREQUIRES
BUILD_REQURIES
```


ID isn't mandatory in `Source` ;)


We should use a ReactRouter helper to generate app links: https://github.com/rackt/react-router#whats-it-look-like


Ah, so you're doing this from a REPL then, not a notebook. Go ahead and add that change to your PR as well; there's a reason why I directly link against vega in the REPL command but not in Jupyter Notebook, but I don't remember why. Better to have a working version up now and make it more elegant later.


better throw console.error too.


We should use a ReactRouter helper to create app links: https://github.com/rackt/react-router#whats-it-look-like


Well given that I want to replace the (searches for a suitable word...) impedance-mismatched-cloud-api with one that is better suited for implementers and users alike... hacks will have to live until I get the time to fix.


These are neat tricks @Ogeon thanks for sharing!
I tried them. Though, at first sight, it does seem they do not improve performances compared to
```
     for row in self.board.iter() {
            for cell in row.iter() {
                if let Cell::Taken { player: _ } =  *cell {
                      //etc...
```


this should be squashed on the commit doing the mistake... I don't mind not getting cited on the commit.


The variable expand to "bepc-unknown-haiku" or something like that, which would not be very readable here. However "x86" will be wrong for all other ports, which use the same recipe (arm, ppc, etc). Maybe one of haikuports variables can be used?


* About to connect() to ipconfig.me port 80 (#0)
*   Trying 82.98.86.174... connected
> GET / HTTP/1.1
> User-Agent: curl/7.22.0 (x86_64-pc-linux-gnu) libcurl/7.22.0 OpenSSL/1.0.1 zlib/1.2.3.4 libidn/1.23 librtmp/2.3
> Host: ipconfig.me
> Accept: */*
>
* HTTP 1.0, assume close after body
< HTTP/1.0 200 OK
< Date: Thu, 08 Aug 2013 06:07:18 GMT
< Server: Apache
< X-Powered-By: PHP/5.3.3-7+squeeze15
< Expires: Mon, 26 Jul 1997 05:00:00 GMT
< Last-Modified: Thu, 08 Aug 2013 06:07:18 GMT
< Cache-Control: no-store, no-cache, must-revalidate
< Cache-Control: post-check=0, pre-check=0
< Pragma: no-cache
< Set-Cookie: tu=9131e48e9a1aa70037addb60b3bb2fa0; expires=Tue, 31-Dec-2019 23:00:00 GMT; path=/; domain=ipconfig.me; httponly
< X-Adblock-Key: MFwwDQYJKoZIhvcNAQEBBQADSwAwSAJBANnylWw2vLY4hUn9w06zQKbhKBfvjFUCsdFlb6TdQhxb9RXWXuI4t31c+o8fYOv/s8q1LGPga3DE1L/tHU4LENMCAwEAAQ==_ABA8dAgDBb344B7D83BGUT88vvELQ/5UJUR/xmOSiHARox5AO4aAgBLw3e575E6caj8OPvoTZeBATLsI2VBroQ==
< Vary: User-Agent,Accept-Encoding
< Content-Length: 1655
< Content-Type: text/html
< X-Cache: MISS from 20857
< Connection: close
<
<html xmlns="http://www.w3.org/TR/REC-html40" data-adblockkey="MFwwDQYJKoZIhvcNAQEBBQADSwAwSAJBANnylWw2vLY4hUn9w06zQKbhKBfvjFUCsdFlb6TdQhxb9RXWXuI4t31c+o8fYOv/s8q1LGPga3DE1L/tHU4LENMCAwEAAQ==_ABA8dAgDBb344B7D83BGUT88vvELQ/5UJUR/xmOSiHARox5AO4aAgBLw3e575E6caj8OPvoTZeBATLsI2VBroQ==">
	<head>
		<script src="//google.com/adsense/domains/caf.js" type="text/javascript"></script>
...


-- Design choice change -> we now don't run this function and check outside the function whether to run it


Consider InitializeDateTime gone.
The problem with
```cpp
bool bFirstDay(m_timerType && m_timerType->SupportsFirstDay());
```
is that it only uses the 'first day' time if it is supported by the timer type. I was using it as a means of storing 'start time' when 'anytime' was set. (we are doing this anyhow by using firstday as the time when start=end=0 for strSummary)

I think this work-arround is just addressing one more symptom of loss of start time due to 'anytime', and in a less than ideal way. I'll go with your suggestion for now (as it is better than returning a time in 1970 if first day is not set).

I'll try to come up with a separate PR which tries to address the issue properly (i.e. a separate anytime boolean or similar). Unfortunatly that will result in another PVR API change, so I will probably suggest incuding it at the same time as another idea of mine, including subtitle, season and episode number in 'recording' and 'timer' types (they already exist in the EPG type).


This is GCC4, though -- I thought `BePC` is only part of `uname` on the GCC2 ABI?


With m you could add support to line numbers again :)


All these handle clicks could be refactored to a single method that receives the selected block as parameter


This would mean we are not allowed to distribute a package built with patches, without agreement from the author (since he retains an *exclusive* right to do so). Adding a PATCH or PATCHES would violate the license so it is nice to have a reminder here?


Look like you misspelled canola


@TomasTomecek The existing code in the module uses this format. The inherited class will encode the url when it performs the underlying `request()`.


They are probably optimized to the same thing. I just wanted to present an alternative :)


Let's use [classnames](https://www.npmjs.com/package/classnames) to cleanup the view


Right, I don't remember what the machine triple is for gcc4 but it's certainly not just "x86"


@dudekm Commitnij dowolną wersję, która działa jak chcesz. Ale ma być wszystko co chcesz, żeby było i ma działać.


Looks fine :+1:


You're right, nothing is Mandatory.


I've added an autotest in mapserver/msautotest@51d6652d8f to check for WFS filters using OGR backends, and the results are identical to the direct shapefile access which would tend to show that the filtering is working correctly. However @sdlime could you check that all is OK inside the OGR driver concerning the RFC91 filtering?


Did you try google search?


```ContainerUtil.filter``` followed by ```ContainerUtil.mapNotNull``` can be replaced with a single call to ```ContainerUtil.mapNotNull```.


no change


That should be `src`, not `newsrc`.


@miniBill I see you've removed XSP classes in this patch. I'm trying to build monotools-server against xsp master and it requires XSPWebSource so is failing to compile. Was it your intention to remove these classes as a cleanup? If not could they be added back into Mono.WebServer.dll or if so can you suggest what I should be doing to fix the monotools-server build? Thanks


Also, can you place here the plain text of that code snippet screen shot you have above?


@MacDada wołam ;)


@metaron-uk: You're welcome.


this is second text


> 2. won't be tested (or we need to run the tests twice)

One will see it in code coverage.

> 3. confuses future maintainers (so what exactly is the reason to implement this part using numpy? esp. because it is the only use in the entire code base)

`git blame` and reasonable commit message will solve it

> 4. slows down the code

you can do it with minimal runtime performance penalty ...

>  I don't agree, because of the KISS principle.

That's all right. I will take my responsibility for it.


This class can be converted to an enum, in order to create it only once.


why moving back?   I was standardizing thing by moving to scripts   so we can use run `npm run deploy` everywhere.


I don't get it.

> keep the API unchanged
Adding a member in a class with some functions is not an API break.

> Inheritance allow users to perform whatever he wants.
People could still inherit from View if needed.


I’m reverting this change.


What's the point of mapping these dependencies every time you're asked for incoming edges? You can pre-compute it once.


This is a comment


@jonas why? is it because it needs next version as well?


This bit is shit


`_reflections` are the raw reflections. `reflections` has been postprocessed to handle HABTM. I'm not entirely sure I should be using one or the other.

Having said that, _reflections is private. Maybe I should override `reflections` instead?


Nice, I did not realize that `kprintf` was deprecated.


Even if you DO want to *import static*, you don't have to add the entire library.


I just noticed something interesting. If I checkout this branch using git extensions, it converts the newlines into \r\n so that I can open ini files with notepad (not notepad++) and see everything formatted properly. But if I download this branch as a zip, then the newlines aren't converted, and if I open the ini file with notepad, everything appears on the same line.

Its pretty easy for us to open the ini file in a better text editor, but would this cause problems for people just downloading the collector?


Made final in 95e8b0b3d24800d0471cf81612f07428a1e3f048.


dasdasdasdas


It's not 'real' PostInfo, mean it's always hidden and 'real' PostInfo is insdide PostHeader component (perhaps it's rudiment from previous versions)


sssss


Same thing seems to be happening here:
```
normalized_penalty.kin_Q8................................................................... FAILED (EXODIFF) [0.084s]
```

```
normalized_penalty.kin_Q8: Element variables:
normalized_penalty.kin_Q8:    stress_xx  rel diff:  0.0000000e+00 ~ -1.7022507e-08 = 1.00000e+00 (block 2, elmt 3)
```


xzczxczxczxcs


@suhailshergill Yes, `sbt-release` prompts for release version and next version, it then has steps that updates and commits `version.sbt` both before tagging and after to return to SNAPSHOT version or whatever "next version" is. Since we are using git-based version management in SBT we don't need those.

I might consider contributing something that facilitates this workflow to sbt-release.


I agree. We should start tagging the theme, https://github.com/nairobilug/pelican-alchemy/issues/38


Renamed in 7f907ead51285275e2b84fe05e659ee3092605e3.


These look interesting: `org.eclipse.jdt.internal.codeassist.ThrownExceptionFinder.getThrownUncaughtExceptions()`


private


Come on dude........if you're going to fork our shit and use it, then use it BUT don't kang stupid shit

https://github.com/DirtyUnicorns/android_device_samsung_hlte/commit/89ed2d360084815c840c17252253beb1f5a47576


That implies ```res1``` has a reference to ```OptController``` or do you mean that ```res1``` is the ```OptController``` itself?
What would we do with the cases like subsequent call ```res3 = bboptimize(res1; MaxTime = 2.0)```? The user might expect it continues, where ```res1``` has stopped, whereas it resumes from ```res2```, because it holds the reference to the same optimizer.
An alternative might be to have ```bbsetup()``` call that returns optimizer object, which could be ```run!()``` (and, eventually, resumed), while ```bboptimize()``` just returns the result, without any references to optimizer and possibility of resuming.


It's best not to use ui-view as ionic has wrappers around ui-view (ion-nav-view, ion-view, etc) which will keep track of history and caching of views.


czxcxcdssdcds


@arthurvr you reviewed https://github.com/jquery/themeroller.css-chassis.com/pull/8/files#diff-354f30a63fb0907d4ad57269548329e3 as well, where this is used. Did you miss it there?

> This means the build may fail on a freshly cloned repo and still pass on Travis because a sub-dependency got updated and Travis didn't catch it.

What does "didn't catch it" mean here? My understanding of this cache is: It keep the folder around and `npm install` will update it with any updated packages, just like you would locally. Your comment sounds like "cache the folder, never update it", which doesn't make much sense.

That said, I'm fine with dropping the cache is there's a valid concern (not sure), but then we shouldn't use it anywhere (see Chassis TR link above).


put an exec in front of sslh


sadsadas


You should not forget this, it would have prevent you to do the commit "Fix sending of active rules in analysis report" :)


Wouldn't it make sense to use the Mmio framework API for accessing the bits of the hardware registers? (also in ```usb_phy_init``` and ```clock_pwr_init```)


Unfortunately not (yet) supported by the most recent version of python-matplotlib which can be installed on SLC6 (0.99.1.2-1.el6)


   groupkey         dforeignkey,
   smtpkey          dforeignkey,
   msgsubject       VARCHAR(256),
   msgbody          VARCHAR(1024),
   filename         VARCHAR(64),
   exporttype       VARCHAR(5),

это все поля отчета, а не автозадачи. у нас будет возможность отсылать отчеты по почте не только через автозадачи, но и через нажатие кнопки. Имя файла тут вообще ни к чему. Оно должно генериться. Сабж, боди тоже сомнительно. Должны генериться. Кто тут что будет прописывать вручную.


Why do we need the entityName set in the configuration, when we already have access to it on the document?


I knew someone was hacking my phone, google , eveything. what can I do?


`removed events`.


@yehudasa, Updated as your comments, thanks! I feel difficult to write
tests for the code. Do you happen to have any examples to follow?

Sincerely, Yuan

On Sat, Jul 18, 2015 at 1:02 AM, Yehuda Sadeh <notifications@github.com>
wrote:

> we also want to avoid calling this multiple times. Should keep the result
> for this on the object and reuse it.
> Also, note on the object that we already read this, and later on in
> get_params() only get it if this was not called.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zhouyuan/ceph/commit/896aa427f7eb52668d347383cddc72556ef485ae#commitcomment-12229519>
> .
>


> you reviewed https://github.com/jquery/themeroller.css-chassis.com/pull/8/files#diff-354f30a63fb0907d4ad57269548329e3 as well, where this is used. Did you miss it there?

Seems like I did, yeah. Sorry.


TgdAutoReportTask = class(TgdAutoTask) -- обсуждалось, что за отсылку почты отвечает TgdWebClientThread. Автозадача тут ни при чем.


And again, this is an utility class whose only use is getSortedModulesCall. Make it private, forbid to instantiate it from outside, and remove logic like this ```return mySortedModules == null ? doGetSortedModules() : mySortedModules;``` - it's not going to be called again anyway.


Ah, whoops, missed that.


yep, I will dig more on it, once I go home I will put the code in plain text


sdasdassa


Thanks @taylorotwell! Makes things lot easier.


YOU MUST BE STOPPED


gjggjgh


Why deleted this function ?


No unit tests on this class ?


if you use the preg_match() directly, you don't have to add the method to the Generator class


Seems that this causes an error in travis.
```
error: while querying the derivation named ‘pypy2.6-rainbowstream-1.2.7’:
while evaluating the attribute ‘postPatch’ of the derivation ‘pypy2.6-rainbowstream-1.2.7’ at /home/travis/.nox/nixpkgs/pkgs/development/python-modules/generic/default.nix:65:3:
attribute ‘sitePackages’ missing, at /home/travis/.nox/nixpkgs/pkgs/top-level/python-packages.nix:7513:19
```
Or I am wrong?


The comment line is needed because in Android the onClick and onTouchStart are called in same time, resulting by calling the function twice.


Thanks, I assume this was a mistake on Meetup? It's a cronjob updating this list.


Alexander <Alexdunn220@yahoo.com>


> Incidentally, how stable is nrepl-client.el now? Any chance of pulling it out of CIDER, so that it can work as a replacement for beanshell.el?

Yes, this is actually in a our roadmap, but with low prio. @vspinu needs it for a project of his and he was planning to do the work. Overall the client is stable, but the problem is that it's highly coupled with CIDER as no one tried to use in another project. There's also another project called monroe, that was forked from a very old version of CIDER, when everything was implemented in Emacs Lisp, but I'd generally not advice using it, as pretty much everything `nrepl-client.el` is more reliable and robust.


Hello


Sorry, answer in your commit :)


sasa


What was broken?


ok, on my phone it wasn't a problem...humm..


You've re-implemented ```ContainerUtil.mapNotNull``` here. Also, avoid using specific types like ```ArrayList``` here - use more generic ones instead (```List``` in this case).


I think it may be Vizio update doing it they pushed version 1.60.10-020.000 and I think that is when it started
Wonder why mencoder works fine


Thanks!


@pynicolas from my point of view it is not very clear why your "increment" of -1 for accessors


Either `description_field_id` or `description_code` are required, depending on which `description_type` is chosen, so it is not useful to check `f_descr`.
If it `False`, `cur_rec[f_descr]` will be `False` but `line.description_code` will have a value.


Cool thanks.


Ikedam I intended to add Spanish translations as a pull-request but they were appended to master, I am sorry for that. This commit does not modify any existing file.


Did a complete restructuration of the code.


Modifying ```optctrl``` within the callback looks a little bit worrisome. When the callback finishes, all the assumptions that the calling function had about the ```optctrl``` state might be wrong (the callee could be innocent, but even ```stop_optimization!()``` might do a lot). I would rather pass some sort of ```OptimizationState``` and expect callback return ```true``` (continue) or ```false``` (interrupt).


+create HelloWorld.md


yes,provide a rs implement for the read/write side.

rs is composeable but not netty's auoread config.

thia is just a nice addition,if you would like😁


The original file uses `/proc/kallsyms`. The result should be the same, but IMO you should use copy the original behavior and update it later (at least following patch).

Also this method actually does not work at all, because the iso9660 module is loaded on-demand. So unless you inserted this module manually (or inserted CD into the CDROM), this reports no-support (booth, original and your version).

So I think you should try running `modprobe iso9660`, ignore the failure (it might be built in) and then trying the /proc/filesystems. (this should be fixed in autotest too)


log error when no error?


TeamCity DNN JavaScript Libraries :: CI [Build 41](http://teamcity.engagesoftware.com/viewLog.html?buildId=309&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


TeamCity DNN JavaScript Libraries :: CI [Build 41](http://teamcity.engagesoftware.com/viewLog.html?buildId=309&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:8


It will not be looged if no error


@jonas any way you could expand current explanation based on commend above and your comments on slack? it would help people like me. rest good, +1


TeamCity DNN JavaScript Libraries :: CI [Build 42](http://teamcity.engagesoftware.com/viewLog.html?buildId=310&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


TeamCity DNN JavaScript Libraries :: CI [Build 42](http://teamcity.engagesoftware.com/viewLog.html?buildId=310&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:6


Done.


This went away.


Again, this should IMO go to a separate commit to keep track of changes.


Ouch! You're modifying global state here. Do you really need platform prefix to be set?


My alignment wasn't correct but yours isn't either:
https://github.com/bbatsov/ruby-style-guide#indent-conditional-assignment


No, but the previous rest wasn't used after all. Fixed.


var sum=0;
var num=1000
for (var i=0; i<num;i++){
  if (i%3===0||i%5===0){
    sum+=i;
  }
}
console.log(sum);


> My understanding of this cache is: It keep the folder around and npm install will update it with any updated packages, just like you would locally.

If you depend on `"a": "1.2.3"` and `a@1.2.3` depends on `"b": "^2.1.4"` and then `b@2.7.1` is released, `npm install` on a fresh clone will get you `a@1.2.3` and `b@2.7.1`. But a previously cloned project will get no change on `npm install`. The same will apply to Travis - it will use `b@2.7.1` until the requested version of the `a` package is changed.

This means that Travis may pass when a fresh clone wouldn't pass tests.


But this doesn't change the behavior, so it's not that bad...


```ErlangModuleCompilationTest``` extends this class. Why would you use it here?


Cool!


var sum=0;
var num=1000
for (var i=0; i<num;i++){
if (i%3===0||i%5===0){
sum+=i;
}
}
console.log(sum);


If you are going to require the entityName is specified in the hook definition, then we should do the same here.


It didn't show up for me in Firefox or Chromium. The iframe just had "<html></html>" in it. Perhaps caused by an update to GitHub Pages.


Actually this is why autotest always fails without the isoinfo in my grid... I'll send pull request today to fix it upstream...


:grin:
Let me read those completely later.


I google the exception problem, not much help, got the try-catch on the google search with extend exception, thanks for post the question stack Overflow


what about `invenio/testsuite/test_utils_vcs_svn.py`?


@sunstorymvp Why not just?:
```javascript
if(_.isString(title.apply(currentRoute))){
   ...
}
```


Where is `time` defined? Is that a missing import?


Imo a greater log_level means more is printed, so I would say:
```
If the log_level of the printer is less than DEBUG, it prints only the message. If it is DEBUG or higher, it shows the message along with the traceback of the exception.
```


Thanks a lot !


why someone should place the traceback here? I think `log_level` is quite understandable :)


Ok, thanks Saen


Mm, seems to be the "tight_layout" option to better align multiple plots
in a canvas. Sorry about that, let me see how to do this with an older
SLC6-compatible version...

On Mon, 20 Jul 2015, Laurent Forthomme wrote:

>
> Unfortunately not (yet) supported by the most recent version of python-matplotlib which can be
> installed on SLC6 (0.99.1.2-1.el6)
>
> ?
> Reply to this email directly or view it on
> GitHub.[AEewx9evzIqI9-HJunhMdNCF_ArYNzEMks5ofPPQgaJpZM4Fb84C.gif]
>
>
>


You are missing `:return:`


It looks like you don't need a command here. Hence, you can do all your setup under single write action in ```setUp```


I think we have to use a closing </li> too?


@sunstorymvp  could you change to?:
```javascript
...
}else if(_.isFunction(title) && _.isString(title.apply(currentRoute))){
...
```


It's the log_level for the message. Not the traceback - the traceback always has the DEBUG log_level.


Confirming this as well, src not newsrc. But then again, the whole VT Module needs a complete clean up imo.


OK so again it's copied from `autotest/client/shared/iso9660.py` and there is one change in behavior, which I don't like. But if you update it upstream first, it can be merged...


I created a PR (https://github.com/lightblue-platform/lightblue-esb-hook/pull/22) to switch this back. We can discuss further there if needed.


所以? 输出命令不变,
只是多了一组输出?!


And `:param kwargs:`


DEBUG is the lowest log_level (0)
ERROR is the highest (3)


> If not showing multiline commands or output of steps, then don't include the $

not done


Return is removed in a later commit.


Upsert mode is not supported for tables without primary keys


I just used your old code. (See org/intellij/erlang/compilation/ErlangPrepareDependenciesCompileTask.java:195 in master). I realize now that I should review the old code before using it.


So the problem is that we don't have `SourceFile`s for all files, only for those with errors. Since we need to scrape the others as well (or read the content and then create a new SourceFile), I prefer to process everything in the same way. Also SourceFile doesn't expose the number of lines, so we'd just get an ArrayOutOfBoundsException when iterating over sf.lineContents(sf.lineToOffset(i)), or have to check whether line == nextLine(offset)... Not really easier.


ok gracias lo pidere apoyo aun tengo dificultad


This doesn't apply for this commit, but we really should document the logging methods.
I'll open an issue for this (if it doesn't exist already).


> @vspinu needs it for a project of his and he was planning to do the work.

Yeh. I need it for cider-like IDE for R, and was planning to implement that this summer, but then I suddenly got pulled into a bunch of commercial projects and my plans went haywire.

I am still very much interested in getting nrepl-client.el out but I have no idea when I will have time for it :(


It works！


Maybe you want supply all log_levels together with `DEBUG` that are higher and not `DEBUG` only?


Remove it. It's not used.


对，命令不变， out/ 下多了两个文件 `pycon.json` 和 `en/pycon.json`


It does not fail, so this PR is fine.


Hi,

you used the wrong print method from six.
The correct one is six.print_() instead of six.print()
https://pythonhosted.org/six/#syntax-compatibility

Cheers,
Stefan


I dont understand what you mean


Oh those calls were using `commands.getoutput` originally, so this actually doesn't change the behavior. I guess it should go directly into the commit where you add this.


Rename it to assertFilesExist, or assertContains, or something like that


Did we not had this discussion today when we did the travis changes?


You can get this information from Javadoc.
// -- Byte streams from channels --

    /**
     * Constructs a stream that reads bytes from the given channel.
     *
     * <p> The <tt>read</tt> methods of the resulting stream will throw an
     * {@link IllegalBlockingModeException} if invoked while the underlying
     * channel is in non-blocking mode.  The stream will not be buffered, and
     * it will not support the {@link InputStream#mark mark} or {@link
     * InputStream#reset reset} methods.  The stream will be safe for access by
     * multiple concurrent threads.  Closing the stream will in turn cause the
     * channel to be closed.  </p>
     *
     * @param  ch
     *         The channel from which bytes will be read
     *
     * @return  A new input stream
     */
    public static InputStream newInputStream(ReadableByteChannel ch) {
        checkNotNull(ch, "ch");
        return new sun.nio.ch.ChannelInputStream(ch);
    }


Worth to comment how `key_num` is computed.


yes 1 for each ui element sounds like a good idea :+1:


`operationInfoMultiple` does not contain memory allocated for `keyInformation`. This may lead to memory corruption


Can we check that `obuf` never overflows? or is it garanteed by construction? (not clear to me).


Don't use final unless you need it. Add a nullability annotation.


Better to have it ordered, that is after the definition of `CMD_GenerateRandom`


Would be good to add another test with only 1 required key, such as xtest 9115


Indicate in the commit message that xtest 9238 is modified


I move this code into the separate method because guess that the new tests will have to create new modules and then add new source root.


This seems to be missing the MetadataSource.java file. Did you forget to commit it?


Thanks for the explanation. We could use `npm update`, though that might end up being slower than just installing everything from scratch.


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1241 of 1241 tests passed)

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


a super la dificultades


а в разметке не помогает атрибут, для показа клавы?


ErlangModuleTextBuilder or ErlangFileTextBuilder


не обязательно у view вызывать, можно сразу findViewById


"Moment Diary" is the smartphone application.
You can manage diaries and search and see .
If you attatche #XXX in article, article is providede with the function of a hash tag for performing grouping.

-----------------------------------------------------------------------------
This application has 4 commands.

import
	Diary dates are imported. You can get diary date　from the "Moment Diary".
print
	Diary dates are displayed. This command has a one attribution.
	a. print	You can read all articles.
	b. print a, b, ...  (a,b is month) .  You can read articles wrote in a,b... month.
hash
	This command has 1 attribution.
	a. hash			Using this command, you can confirme all hash coads.
	b. hash XXX		You can read articles concernning XXX.

q
	this application has been quited.

------------------------------------------------------------------------------
スマートフォンアプリの瞬間日記(MD)で記録された日記の記事を検索、保存することができるこれはコンソールアプリケーションです。
基本的に、日記の使い方は決まっていません。
記事のどこかに、#XXXと書くと、その記事にXXXというハッシュタグを付けることができます。
このアプリケーションは以下の4つのコマンドを持ちます。

------------------------------------------------------------------------------
import
	アプリで記録されたデータをインポートし、このアプリケーション型に変換して保存します。
print
	保存されている記録を表示していきます。このコマンドは属性を持っています。
	print	コマンドのみを入力した場合、保存されているすべての記事が表示されます。
	print a,b,...	(a月、b月)　指定した月に書かれた記事のみを表示します。
hash
	日記につけられたタグに関するコマンドです。
	hash	コマンドのみを入力した場合、存在するタグの一覧を表示します。
	hash XXX	XXXとハッシュタグがつけられている記事のみ表示します。
q
	アプリを終了します。


why do you need that?


'antipattern' is the word most frequently used by the developer community


может стоит тернарный оператор применить?


I was thinking it would have a reference to OptController yes.

A bbsetup() might be a better solution. It's more similar to how it was currently done, when wanting to resume.


bei help bekomme ich den fehler
````````````````````````````````
[16:16:26] [Server thread/ERROR]: Couldn't process command: 'help'
java.lang.IllegalArgumentException: Comparison method violates its general contract!
	at java.util.ComparableTimSort.mergeLo(Unknown Source) ~[?:1.8.0_51]
	at java.util.ComparableTimSort.mergeAt(Unknown Source) ~[?:1.8.0_51]
	at java.util.ComparableTimSort.mergeCollapse(Unknown Source) ~[?:1.8.0_51]
	at java.util.ComparableTimSort.sort(Unknown Source) ~[?:1.8.0_51]
	at java.util.Arrays.sort(Unknown Source) ~[?:1.8.0_51]
	at java.util.Arrays.sort(Unknown Source) ~[?:1.8.0_51]
	at java.util.ArrayList.sort(Unknown Source) ~[?:1.8.0_51]
	at java.util.Collections.sort(Unknown Source) ~[?:1.8.0_51]
	at net.minecraft.command.CommandHelp.func_71534_d(SourceFile:84) ~[au.class:?]
	at net.minecraft.command.CommandHelp.func_71515_b(SourceFile:38) ~[au.class:?]
	at net.minecraft.command.CommandHandler.func_71556_a(CommandHandler.java:94) [z.class:?]
	at net.minecraft.network.NetHandlerPlayServer.func_147361_d(NetHandlerPlayServer.java:739) [nh.class:?]
	at net.minecraft.network.NetHandlerPlayServer.func_147354_a(NetHandlerPlayServer.java:718) [nh.class:?]
	at net.minecraft.network.play.client.C01PacketChatMessage.func_148833_a(SourceFile:37) [ir.class:?]
	at net.minecraft.network.play.client.C01PacketChatMessage.func_148833_a(SourceFile:9) [ir.class:?]
	at net.minecraft.network.NetworkManager.func_74428_b(NetworkManager.java:212) [ej.class:?]
	at net.minecraft.network.NetworkSystem.func_151269_c(NetworkSystem.java:165) [nc.class:?]
	at net.minecraft.server.MinecraftServer.func_71190_q(MinecraftServer.java:659) [MinecraftServer.class:?]
	at net.minecraft.server.MinecraftServer.func_71217_p(MinecraftServer.java:547) [MinecraftServer.class:?]
	at net.minecraft.server.integrated.IntegratedServer.func_71217_p(IntegratedServer.java:186) [bsx.class:?]
	at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:427) [MinecraftServer.class:?]
	at net.minecraft.server.MinecraftServer$2.run(MinecraftServer.java:685) [?:?]
[16:16:26] [Client thread/INFO]: [CHAT] An unknown error occurred while attempting to perform this command
````````````````````````````````````


No, now anything working without this section. In old version of this test case it was really needed. I will remove this code.


I don't see `IS_WRAPPED_NUMBER` defined in `rts.h` in `ghcjs` repo. Did you pushed this change?


Why rethrow it as a runtime exception? It will only make error output more cumbersome.


I agree it is dangerous but there might be more things the caller wants to do than just terminate. And I'd rather have only one callback rather than adding callback options for many possible situations.

And having both OptState and OptController seems unnecessary imho.


Looks good Erick


I'm a muppet, used the wrong source for this...
At least the test worked. Will need to fix this in the morning


тут можно у просто findViewById вызывать


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1244 of 1244 tests passed)

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


We're already doing the internal preg_replace on 358, is there any reason to do it again here (each time in this for loop?)
`if (preg_match("|^" . $s3path . '/' . "[^/]*/?$|", $file['Key'])) {`


It's really stupid mistake. I will fix it.


некорректное имя id


hm okay makes sense, keep it like this^^


Here is a related feature request where we also discussed OptState...

https://github.com/robertfeldt/BlackBoxOptim.jl/issues/4


пустая строка


Hi Stefan,

Thanks! I noticed it after committing, I was a bit too hasty with this. If you have the time fixing this and issuing a PR would be useful and awesome, I'm very busy at the moment and won't find the time to do this for a while.


OK


ah then I would write "(not **for** traceback)"


the code into evhtp_free()
    if (evhtp->server) {
        evhtp_safe_free(evhtp->server_name, evconnlistener_free);
   }


Hello Norman

I wrote the first version based on implementation of Arndale board. But
is possible rewrite the code using Mmio framework API.

Best Regards

Alexy

On 07/20/2015 09:48 AM, Norman Feske wrote:
>
> Wouldn't it make sense to use the Mmio framework API for accessing the
> bits of the hardware registers? (also in |usb_phy_init| and
> |clock_pwr_init|)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/xeos-team/genode/commit/2d503925e8fc1f3a963e96d81d2c5139b5fadb46#commitcomment-12253005>.
>


ah okay, but please document kwargs :D


will do :)


You have a ton of tests with this try finally block - why not create a ```CompilationRunner``` in ```setUp``` and dispose it in ```tearDown``` instead?


Same goes for here, on 357

`if (!$fname || $fname == $s3path || $fname == $s3path.'/') {`


I mean if we have now lower (so lower means < 0 like above), don't you want to pass this log-level instead every time `DEBUG`? I.e. for `VERBOSE`^^


-> #843


saw approval from christophermancini
at https://github.com/basho/riak-python-client/commit/aee07d1f5dda05352d855440c72ab6e458808de7


merging basho/riak-python-client/feature/bch/cleanup-pep8 = aee07d1f into borshop-integration-407-feature/bch/cleanup-pep8


basho/riak-python-client/feature/bch/cleanup-pep8 = aee07d1f merged ok, testing candidate = 723bd59b


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

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


Exactly - question everything you read. =)


Aber die fuel werte werden übernommen. Muss ich halt das Game neu laden.


I would recommend you to use a branching model strategy to go with feature development and merging what you need! Ánimo!


`if ($res = preg_match("|^" . $s3path . '/' . "(.*?)\/|", $file['Key'], $matches)) {`


`if (!in_array($s3path."/".$forlderName, $finalfiles)) {`


useless line return


+C on non-btrfs fails the tmpfile service. On some forums people even suggest others to delete the whole /var/lib/journal directory to "solve" this problem. Obviously the service should ignore the case where the attributes cannot be set.


OK


The indenting got screwed up during the checkin.. It looked fine in my local editor..I don't use iCmbSearchMode=2 so I didn't test that condition.  I didn't really suggest this one in the forums, because I know it's a hack :)  I didn't spend too much time debugging other than making sure it did what I needed..  If I get time to clean it up I'll fix it up and post in forums for others.


So the first time a new project loads the front-end will know?


`$finalfiles[] = $s3path."/".$forlderName;`


You can still have a protected method which you'd call from under write action.


nvm


Resolves #96


mohammad


с content на целых 2 итерации меньше


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.770.0460f5447c62e94d0378404734f75bab6943f42b](http://build.palaso.org/viewLog.html?buildId=59829&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 380, ignored: 3 Build time: 00:08:40


1) format your code (`ctrl+shift+f`)
2) put `e.printStackTrace();` to see the full stack trace of your error
3) notice that the problem is not reading from the file but in the actual multiplication 'algorithm' at line 93.

Have you tried debugging it?


oh and also, upload `.java` files, not `.txt`