Do you have Skype? And what problems, exactly, do you need? I need a little recap :P
> On Apr 7, 2015, at 7:01 PM, Joe <notifications@github.com> wrote:
>
> Sorry for the delay, been busy in the last 48 hours (and will likely for a while :/)
>
> It may not be as simple as player name changes, but I do realize though it's not really exhaustive, since all that's really necessary is a few changes to the specific player code and it should be working fine. I do wonder though how performance-efficient is getUniqueId().getString() though.
>
> As for the BukkitDev project addition, I'm waiting until you made all the changes necessary (besides the ones you had troubles on) and then me being able to fix some parts of the code (so you can submit another JAR file).
>
> Not entirely sure with partnerships right now, since the concept is private among friends & won't be for months.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/20zinnm/AntiBot/commit/eb1a8012199d5762991887116e050b6844493fbc#commitcomment-10614200>.
>


Git is 10 years old! Happy birthday! :birthday:


Dude, You rock. thx!


the #include string :P


aha, got it, will fix.


@marpo60 this is the fix, I'll create a new release with it


The formatting here is a bit wonky. Could probably be simplified:

```
 <li class="c-list__item">
    {.text}

    {?hasInnerItems}
       <ul class="c-list">
                    {#innerItems}
                        <li class="c-list__item"> {.text} </li>
                    {/innerItems}
                </ul>
            </li>
    {/hasInnerItems}
</li>
```

Although as I mentioned in chat, I think you could just not have a `{?hasInnerItems}` block at all and pass another instance of the `c-list` component into the value of `text` (which would probably be more generically named `content`)


ok, I must have missed the cmd


@pilerou I actually had a particular application in mind that required concaternation, which is why I made this alteration. I am aware this shouldn't be standard behaviour, which is why I didn't bother making a pull request to merge this into the main branch. Thanks for the input though!


Good note on commenting this method.  Hopefully it functions as should, all resources on StackOverflow were pointing to the same document for implementing this feature.


OK, the commit at which I forked was reverted later on. I should have merged the code before finalizing my pull request. Do you really want me to do the whole pull request thing for this, or can a committer just add the missing import? I'll be more careful next time!


I'll go ahead and add a README file. :-)


you still need to add back the non deterministic conjuncts


HeadlessFixes (axelstudios) - i386-Windows-7-VisualStudio-12: OK (2170 of 2197 tests passed)

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


Ah. I see. I guess we should move it to `travis_build_environment`, then. It doesn't belong here.


on that note, you should just use context.get()


Fixed. Thanks!


please update `.travis.yml` with the Go version requirement if it is not already set


The '/usr' part of the path should be added as the fallback PREFIX value rather than in the path definition (as per https://github.com/nemomobile/libconnman-qt/blob/master/libconnman-qt/libconnman-qt.pro#L9), otherwise if PREFIX is set the library will be built to /my/path/usr/lib instead of /my/path/lib.

This doesn't work for the bluez-qt.conf installation as that belongs in /etc not /usr/etc, but bluez-qt.conf isn't necessary anymore so it can be removed.


I don't see much here. Also, I can't find it, but I thought I left a comment about how the exclude param doesn't belong in the options; it belongs in the build job.


巧妇难为无米之炊。下面将给出一套H250的材料清单，对照清单为大家叙述在实际的DIY中我们应该如何选择材料，如何配置属于自己的H250四轴飞行器。


if we change this to

if (strncmp(tz_version, current_tz_version, strlen(tz_version - 2)) == 0) {

it will only check the first 2 digits of tz_version which are 01 in both cases and it won't fail.


Thanks for addressing this so quickly! It's no problem at all. A PR should be just fine. Please let me know if you don't have the time for it right now, and I can make one.


whats this for exacly? Thanks


here: https://github.com/SergioCrisostomo/grunt-mootools-packager/commit/9e06e02c8a2b265a3802b0d6030da2e32fd48f4c#commitcomment-10588117


@BanzaiMan I like that idea :+1:


This was probably related to my recent issues! In ``bower.json`` Bower was seeing ``"angular-ui-tinymce": "latest"`` but NPM is throwing this error ``npm WARN package.json tinymce@4.1.9 No repository field.``


Boa


"now also support" should be "now also supports"


this change...smh


`&#62` should be `>` (or `>`)


Hmm.. extending prototypes is usually not the safest idea, especially in this context. I might just make these separate functions?


You can remove comments when you are making a pull request


might as well just remove


Not necessary in the PR, would be more appropriate as a line comment rather than an addition to the code


This can just be node.getOriginalConstraint() since this code path implies no action on the TableScanNode was taken


HotFix-Regressions (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1683 of 1683 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)


:: notation doesn't work with -ms-input-placeholder, It should be :-ms-input-placeholder.


The comment is not correct :)


`plug-and-play, but for now, best results` should be `plug-and-play, but, for now, best results` (comma heavy but correct)


no, you need to ask manually. Most clients do this on every screen update received but on slow links you may want to poll more frequently


it'd probably be best to use [this syntax](https://github.com/cvan/mozvr-site/blob/fix-footnote-text/templates/layout.jade#L36) instead, so we don't have to rely on trailing whitespace (which could be stripped an editor)


Note that stderr isn't defined, so this will always call callback with undefined, which is equivalent to no error. `err` (from install) or `error` (from cache.clean) should be sent instead.


"thoughtfully pausing" should be "thoughtfully pauses"


Writing text in read me file


Could you please drop the second sentence about `meta`? I don't really want the spec to provide such loose guidance, even in non-normative notes.


Did you figure this out?  When I run it, the label is hiding and not hiding, but it looks like the problem is that the refreshPageView method is kicked off before the showLoading method


you can add a `title` too, if you'd like - like so:

```md
![Screenshot of Inspirit project](inspirit-1.jpg "Screenshot of Inspirit project")
```


Commenting on text in read me file


HotFix-Regressions (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)


This is a line note


Right...I understand your thinking.

But my thinking it is not a "simplification" but a necessity.  The coefficient for ordering moves and the lmr being in sync are mutually inclusive.  They MUST score the moves the same way..

If you take a step back and think about it....
We use history + cmh*3 to order moves.  We want the best moves to play first.
So why would we have a different ordering scheme at the lmr?  Common sense tells us to use the same exact formula to get the move score for lmr.  Otherwise they will not agree with each other...move ordering would say Moves 1,2,3 are very good.  While lmr will be "you are wrong, my formula tells me...1 and 2 should be reduce while move 8 (the last move) should not."

I even consider this a bug.  It's not about simplification it's about they must use the exact formula to score the moves.


@audiodude, would you mind taking a look at this and making sure it's sufficiently good JavaScript? This seems to be working when I monitor the network, but I just want to make sure it'll work reliably and that I haven't missed anything obvious before I release it.


This is a line at the bottom of the page


Ah, yes, that was an oversight when I added the code to remove the non-deterministic conjuncts (I added it to the other branch of the if)


lol


/cc @davidtheclark :)


Do you mean after a 'normal' plugin fail?


`.irbrc` basically already does this, anyhow


you might want to clarify a little more on what is intended with this field.


Cool! Hope it works for others.


Right, but we still want to record the first predicate that was pushed down (just for consistency).


New method of loading the Main page:
now originally gets all post text information, and populates list view. The posts that have a default image should then indicate a loading icon instead of the thumbnail picture. The images which do not have images will then display the no image indication.
As the images come in the list gets populated with the thumbnails and the loading icons show.


Maybe Gulp plugin need ome special way.

@w0rm but I think postcss-log-warnings format is awesome

/cc @nDmitry


I don't think that necessarily needs to go on the develop branch.  It's not a new feature, but rather a fix for incorrect build behavior.

Which `Makefile.am` would that get added to?


Fixed the loading icons on the main page: They all should appear and disappear appropriately.
No two requests should be able to be send at the same time.


If no action is taken, then for all the TableScan is concerned, no predicate was ever pushed down.


You do know that reverse duplicates the array right?


I don't remember exactly if these cleanups had an importance to reproduce the issue. Can we let them in the current test for now? I'll take care to retest this with/without cleanups and see how the test behave. I'll remove them if not.


http://stackoverflow.com/questions/1023146/is-it-good-style-to-explicitly-return-in-ruby please read up on functions vs procedures.


this is the dumbest pull request i've ever seen


are u an idiot or smth ????


The default should be false because only the player and enemies will have shadows. Walls, projectiles, and items won't.


Field validation for phone number and first/last name fixed
Initial loading for view post more smooth - contacts initially unable to be clicked, and title begins with "Loading information..."
From a new filter on viewdidappear refreshes the page


Scrollablilty between pages, and submit button from filter scrolls back to the main page


yep, this is optional in the documentation (http://git-scm.com/docs/git-branch).
didn't know that. =)


Or maybe it goes in `Makefile.in`?  I'm not completely familiar with how these files get written, but if you tell me where to put it, I'll add it to the macbuild pull request (and fix my script at the same time).


@changlong the issues you are having are related to not having linaro gdb as well.  Somehow I mistakenly made a branch called linaro-2.24 instead of checking it out. I was having a restless night at the time and randomly decided to try fixing LTO at 3 AM..... a mistake I don't plan on repeating. To avoid any issues in the future with building an LTO toolchain I've made a new toolchain branch (https://github.com/UBERTC/uber-manifest/tree/lto) which will track linaro-2.24 branch for binutils and linaro linaro-7.8 branch for gdb.  The toolchain builds fine when pairing the two linaro repos together.  I just finished uploading the toolchains only this time with the proper components at the same links as before or you are welcome to repo sync this branch https://github.com/UBERTC/uber-manifest/tree/lto  and build yourself.


Fair enough


Sorry, may fault - I was a little but confused and jumpled into conclusions too fast. The `ConfigurationProvider` is actually also bound explicitly just one line above, so it can be used for injection.


this means h1's won't be underlined, while h2's will be. GitHub underlines h1's and h2's, which is why I was doing the same here.


这里的revert包括直接把备份替换的操作，也即在妮妮开始作图之前，老A开始呱之前，发现图片被替换，则commit者负责revert他的commit同时接受五毛的罚款


this condition is no longer guaranteed to be true since the layouts may not have been generated until the AddExchanges optimizer.

In that case, we may need to do a single level predicate push down here instead.


Thanks Ian it is a while since I have seen a javap output! And I am shocked to see the output as the use of literal strings in a switch statement has been a common rookie mistake in Java :)

Normally I would check the javadocs for request.getMethod() - often these methods are kind enough to list the constants they return as part of the javadocs:

* http://docs.oracle.com/javaee/6/api/javax/servlet/http/HttpServletRequest.html#getMethod()

So this output throws me for a loop. Talking with @smithkm he can confirm that string literals is a non feature for Java. So what is going on ...


In ATF, we don't use any gpio interrupt feature. And we already have gpio_direction_input() function. At here, it seems that you only need a function to configure pin with gpio function. I think that pinmux_set_gpio() or other name is much easier to understand.


Okay found it - the use you are describing is apparently new to Java 7 (16 years after it was first requested).

* http://stackoverflow.com/questions/338206/why-cant-i-switch-on-a-string


MarkdownDocs (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1683 of 1683 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)


Also named a few things differently for better understanding.


If the purpose of this label is to wait until a new image is loaded then this line of code:        [[NSNotificationCenter defaultCenter] postNotificationName:kModelRefreshNotification object:nil];
needs to be in the completion block in getNewBackgroundImage,
right now it is just called after that method is called so it sets the notification before the image is actually loaded


- 我在本地提交后，mongo的server上查不到数据
- 职业：选择“其他”，要将输入框的值赋给occupation，还没测试
- 有车的情况下，加入汽车的具体信息，还没测试
- 上传照片的地方要分类一下：封面照，城市风景照(*2)，生活照(*2)
- 周芸的设计UI上说，自我介绍的textarea在非编辑情况下应该有不同的样式，还没加
- 最终提交前，我还是感觉最好有个review的页面，然后这些信息最好能直接发到咱们的邮箱
- 收集信息的项目可能还不够，比如学生的话是不是要加个毕业学校
- 部分项目的提供选项还不够，比如所在城市、兴趣爱好


This probably needs to be `exec taskkill //pid $process`. As-is, msys will likely interpret it as a path and try to do translation.


Because we cannot necessarily control what predicate is returned by the connector for this field, we don't know if any information that was used to generate this layout was lost. Thus, we need to intersect the node.getPredicate() with the node.getCurrentConstraint() (assuming both exist) to get the effective predicate.


Why was the new id necessary? Changing the id makes this actually a complete new app, leaving existing users with no way to update, but forcing them to a) notice the changes, b) deinstall the old app and c) install the new one.


Wow.  weird.  seeing weird test fail


Done like a pro! :+1:


on on


MarkdownDocs (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1173 of 1173 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.82%25-green.svg)


Let's call query parameters instead of values


On Apr 7, 2015 8:07 PM, "Marek Marczykowski-Górecki" <
notifications@github.com> wrote:
>
> IMHO it worth a comment somewhere that GLOBAL_CACHE is a temporary
variable "renamed" later in getFileLocations (anywhere else?).
>

No. It was repurposed.

I still have to commit qubes builder.  My system crashed while commenting
last commit and now I can not boot system.  I am using phone right now. I
have backup of most wines builder changed in my personal repo but not the
combined makefile i did today. I am going to work on rescue my disk now.
These changes need changes in builder.
> —
> Reply to this email directly or view it on GitHub.


Let's remove this line. This is for people looking to implement the API themselves and making a new library.


This sentence doesn't sound too good.


@paulrouget This was removing top results which I don't think is correct is it ?


The timer should be stopped when we call the callback, or we should check we haven't called the callback before calling it again.


svg, not sag.


Не вопрос, и понять, и простить можно.
Вопрос в другом: либо надо таки начинать понимать программирование (и тут мы можем помочь, по мере сил), либо выбирать несколько иное направление деятельности - дизайн там, или вёрстку, или системную интеграцию.


もっとちゃんとした説明入れてください


"specificed" -> "specified"


Should this be dc specific and go in inventory/group_vars/dc*?


abs?


ah, here


I'm surprised a node doesn't store its own height with a quick access function


Your comment made my day! I feel so totally clueless! My interface looks nothing like the movie???  I had to upgrade to Yosemite (which is awesome). I'm really stumbling through.


thanks I merged it into my fork dsmorse/gridster.js


「コミット」した後は、ローカルのものに更新が反映された状態です。
コミットすると「github for windows」のウィンドウ右上のところに「sync」というところが出てくると思います。
それをクリックしてもらうと「プッシュ」された状態になります。

他の人と同時に更新した場合、「競合」が発生してどちらの更新を反映するかなどを聞いてくる場合があります。
今後経験すると思うので、やってみてください。


Looks like CM added "ro.ril.ext.ecclist" as a backup to the radio's library/driver own offerings - presumably "ro.ril.ecclist".


I think this needs to be `data/FluxDispatcher.js` and `data/FluxStore.js`. :)


HotFix-Regressions (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (661 of 661 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-5.02%25-green.svg)


Instead of _add_team_name being inside of the return, and being awkwardly available to clients (under the python-esque _trailing_underscore_name), why don't you just put it at the top of the file, above `function Recap() {`?

Also, I don't think you need to return the formData, you can just modify it in your helper function.

Otherwise LGTM.


On 7 April 2015 at 19:42, Vladimír Čunát <notifications@github.com> wrote:
> xcb-util-cursor: autogenerate as other xorg pkgs
>
> @lovek323: is the HEAD version still needed on darwin after this bump?

Yeah, the -HEAD expression is still the only one that builds for me.


So apparently this worked @bsmithgall


I agree, separate function, no need to extend the prototype.


1era version


Should this be 1, 0, 0 instead of 1, 0, 1 ?


I don't see a real reason to move over to log4j2. I'd have to rewrite all the existing logging code for no real gain.
It already works and looks nice, which is enough for me.


Neat. Is the argument to `cache` just the name used to store the cache?


I implemented a very basic terrain with different textures according to normalized height. Transitions are taken care of by simple mix function. Currently everything is hard coded.


Nice!


Can't this be simplified to:
 if (!SpawnWebDriverServer() || !SpawnWebDriverServer())
     return false

?


make site better


This is a note about how to use memcached


do it


You can remove this space


Isn't it beautiful.


these patches won't be compatible with the current work that I am doing with contrail.


will modify the strings


Max, the logic now is to ensure that the state of the service instance can be locked so that other operation performed on the service instance will be blocked, this part has the same logic with the deleting of service instance, if you think is ok, we won't change the logic here


I really hope that is not the case.  Otherwise it would mean that any op that takes constants as input has a copy of its C code for each and every possible value of that constant (since the signature usually derives from the contents).

However, if that is the case, we can also go for the easy fix of disabling constant folding for GpuDnnConvDesc, but I feel like that is giving up.


yes, should be


yes, it's the required endpoint in the story


@maximilien, this is an old error string defined before, we are reusing it, so no need to translate


Hi Sean,

Many apologies for the late follow-up. I was away when you wrote and then
work intervened and it slipped my mind. Rasterio is very good work -- I
like all your stuff very much -- and I'm recommending it in my PyCon talk.
For pygaarst, I wanted to avoid another wrapper/dependency. The parts that
use fiona/shapely are well separated out, but the reading of rasters is
pretty central, and I'm not always using GDAL for it. The version as of
github right now uses h5py for HDF5, and I'm also moving to python-hdf4 as
the default for HDF4 files (with an option to use GDAL). The reason is that
GDAL is somewhat fiddly to install with HDF4 / HDF5  (less so) / NetCDF
support, and potential users are not unlikely to have python-hdf4 for
example if they need it.

See you in Montreal?

On Mon, Feb 2, 2015 at 10:37 AM, Sean Gillies <notifications@github.com>
wrote:

> Via your PyCon abstract and notebooks, I saw you're using Fiona and
> Shapely in pygaarst. Have you thought about Rasterio
> <https://github.com/mapbox/rasterio> instead of GDAL's own Python
> bindings? You'd get the same advantages -- modern, well-tested, more
> Pythonic APIs -- for raster datasets.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/chryss/pygaarst/commit/510e24af758d01f5a3d1ca54c427ef85fe91f4ab#commitcomment-9549030>
> .
>


OK


@maximilien, this is in a block called by each_with_object, so every time it returns an array and finally after the loop, the arrays will be merged together


ok


@programever, This might help: http://docs.appcelerator.com/titanium/latest/#!/guide/Installing_Titanium_SDK_Continuous_Builds


@maximilien , according to the story, the actor_name should be username(email address for non-admins)


not sure if this is right.


HeadlessFixes (axelstudios) - Win64-Windows-7-VisualStudio-12: OK (2171 of 2197 tests passed)

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


Variables are generally used when the same content will be used multiple times in a file. In this case, each "Temporary" variable is only being called once, so they're really not necessary. It also makes the code below difficult to follow as "var1" has little semantic meaning. For the sake of other developers reviewing your code, or even yourself when you revisit the code, using the actual strings would help make it more readable.


Looks like one of my previous fixes dealt with this already.


That seems like a good solution.  Not sure if having one more package dependency is worth the small gain, though.


Great use of comments throughout!


Yeah I like it, and it's trivial to add.

Any thoughts on whether we need extra spacing, particularly when there's multiple signatures?


Not sure why it even made it in. git am is really picky. . .The is the second time sometime like this merged when it should have. Thanks anyways!


sure, you are right, but I think this code may end up being a dead-end for my personal use only. I need to store the same files in p4 and git and server 2 masters at once, and this is the easiest way for me to manage it. I don't intend to issue a pull request.


To be honest I wasn't aware that we want POCO sagas;) If so, it should come on the context passed to the saga method but it should not be available to normal handlers.


test comment


This fixes #138


MDW看起来很像是P2的项目，放在xuemen目录之下的原因是？


`pkgdesc='Pattern-matching library for Lua (mingw-w64)'`


Fixed. https://git.gnome.org/browse/libglnx/commit/?id=16f02afb2b8a764ff697fbad133f319ab9dea97f


Can be changed, was only ADSP from me to define "audio digital signal processing", has used this "A" to prevent problems if something in the future comes with "V" for video or something else.


This closed #735 (Preview) rather than #648 (Drafts)


```
local luaver=$(pkg-config lua --modversion | sed -r 's/^([0-9]+[.][0-9]+)[.][0-9]+$/\1/')
install -Dm755 lpeg.dll "${pkgdir}${MINGW_PREFIX}/lib/lua/${luaver}/lpeg.dll"
```


It'd be nice to make this more future-proof, see suggestions above.


Fixed in 0fa11c17fba263f5664cfcd80e47aefa1e5ac74f.


I consider 'uppercase' slightly better than 'all-caps' too.

We try to get alpha 1 ready tomorrow. This obviously isn't a critical enhancement, so slipping to alpha 2 isn't really a problem.


Kill this goto


I see now, I didn't catch this because I'm using the default options to run specs, I believe. And the first errors mentioned make sense because the task is set up as a multi-task so when you place the configs outside the options wrapper it thinks you're setting up multiple Jasmine runs.

I'll try to take a look at this tomorrow, 3am now.


Test commment


Ah cool, forgot about that - my coffee hadn't settled in yet :wink:


== 0


lovesitdolls


Don't understand exactly what you mean. Do you mean to use 4 characters there? If yes no always two as tab space.


This from git


Muhwhahahhahaha!


You might want to remove this console.log


thx:)


It would also fix the `TypeError: watch() takes at most 3 arguments (4 given)` when I try to use INotifyWatcher with 2.3.2 (fixed in badf94b85dc8a7e4639d2b661daea576c2a5c2f1 and 698496ca78ed8612f6c9d9a76783423d855db4f0).


You misspelled canola


unsigned int size


Just a heads up that this commit doesn't seem to play well with the current responders gem 2.1.0 and master.


Its broke calls to keys without options param


Bail out before calculating anything. If ever


could copy the xlist using `all_ops.insert(sdata->ops_in_flight_sharded.begin(), sdata->ops_in_flight_sharded.end())`


It looks ok, although the semantic fix would have been to revert max() into min() again, as it used to be before 74e50c4.


woohoo :D :dancer:


I'm not quite familiar with Groovy, but seems there is no setOriginalResource() method with no parameter. Should be setOriginalResource(uriResource) ???


Well it might be better to check travis for the context; this conversion starts from ammonia_fit_example.py line 75. The equivalency converting from km/s to GHz can be seen when I print out the equivalency list but still it can't convert it. Same thing happens in the ammonia_vtau_fit_example.py


lgtm


;) stalker hehe


Such an essential part of the site - its a wonder why it hasn't been merged


Could you link to the specific line in the travis failure?


@MarcelGerber Sorry I should have commented it before but function name should not be brace specifically as we are doing it for closing brackets/parentheses as well. Can we rename it to something common like handleWhitespaceForElectricChars


@tmortagne: could it be the cause of: http://ci.xwiki.org/job/xwiki-enterprise-test-ui/881/org.xwiki.enterprise$xwiki-enterprise-test-ui/testReport/org.xwiki.test.ui/VersionTest/testRollbackToFirstVersion/
Now the message in the history is "Rollback to version [1.1]" instead of "Rollback to version 1.1".
Either we fix the test of the code.


please use prefix `++` for non-trivial iterator for better performance.


@aslakhellesoy the `comments` property of the `astBuilder` struct gets initialized with `nil` per default.

You can explicitly set it to a zero-length slice in `func NewAstBuilder()`:
``` go
builder.comments = []*Comment{}
```


probably broken since zmq, only other time i see it getting called is charutils::LoadChar() (which is always called on zoning now)


Review the mask


Fixes #1.


Thanks for the updates to this. I've integrated most of your changes now.


or
``` go
builder.comments = make([]*Comment, 0)
```


Align `)` with `(`.


depot_index
depot_offset


Git is 10 years old! WHAT!
Happy birthday! :birthday:


we already aligned required_size


Zrovna jsem chtěl vytvořit pull-request. :)


What's happening when openssl does not support an option? If I see at this moment the latest openssl documentation does not contain SSL_OP_NO_TLSv1_1 and SSL_OP_NO_TLSv1_2 options. (https://www.openssl.org/docs/ssl/SSL_CTX_set_options.html)


Ahh damn...


ack


This is kinda stupid.

Tabs > Spaces


could use `all_ops.insert()` for this.


Sorry somehow I thought it keeps global/class-local.


Byron, deberias excluir la carpeta .idea de los commits


print 'hello world'


This should be `@Named("urls")` I think.


`all_ops.size()`'s complexity is constant. maybe we could use it instead?


@fritsch sorry, hadn't reloaded the page to see your latest comments. While these notes are not wrong, for labels that are generic enough to be reused elsewhere the description IMO should also be written in a more generic way that reflects the used context. So something like:

```
#. Label of a setting to limit the number of fps used for updating the GUI while playing videos. This is useful for slow systems that have problems rendering GUI and video at the same time in full speed.
#: system/settings/settings.xml
msgctxt "#38013"
msgid "Limit GUI updates during playback"
msgstr ""

#. Description for the setting Video -> Acceleration -> Limit GUI updates when playing video with label #38013
#: system/settings/settings.xml
msgctxt "#38014"
msgid "Limits the speed (fps) used to update the GUI while playing videos. This can reduce CPU load and fix playback issues while the GUI is shown."
msgstr ""

#. Label of a settings value to indicate the "disabled" state of an otherwise limiting setting (like fps limit, bandwidth limit, ...)
#: system/settings/settings.xml
msgctxt "#38015"
msgid "No Limit"
msgstr ""

#. Used to format framerate values. %d will be replaced with the according number/integer, like "24 fps", "50 fps"
#: system/settings/settings.xml
msgctxt "#38016"
msgid "%d fps"
msgstr ""
```

btw - contextual help strings use `#.` prefix instead of `#:`
Also we probably could  use "Unlimited" instead of "No limit", what do you think?

@jjd-uk mind checking wording and grammar? thanks.


I'm not sure this is the correct way to implement.


... determines which _Optional_ this will ...


This shouldn't be needed, as Ballots is always initialised to an empty list in the constructor for Poll.


I think this is so necessary, so I wouldn't please :)


This is it: https://travis-ci.org/pyspeckit/pyspeckit/jobs/57612369


Oh, I use NetBeans :)


Thank you .
What if pthread_mutex_unlock  freed the lock, then immediately  other thread locked it  before pthread_mutex_destory is called .  This point puzzled me.  The count variable makes sure that no other pthread hold before, but cant make sure another thread is waiting for the lock after the  --fp->f_count == 0.


you're right.


Soul-guiding Statue


Nice @dpogue! I've been itching to do the same thing :smile:

How well is this working for you?


Why static? you use it below non-statc btw.


because it doesn't need self.


TODO: switch to __memcmp when memcmp is instrumented


How does this prevent division by zero, if division is performed in previous line?


от так вот


Че делаем?


Пиши селекторы сразу


Such an empty string.


s


Старайся следовать стилям оформления кода
https://git.snpdev.ru/frontend/study_frontend/blob/master/basics/standarts.md#coffee


That's fine. but, you are automatically allowing the caller origin here. that's no different from *


Call find_stack again for double-check locking


@looopTools I would only use this line as the class description. The parameters should stay with the constructor.


Hey there. You are correct. There is indeed an issue for numbers between 11 - 13. Just added a new release. Thanks for spotting this. Cheers


I updated and improved the node source analysis on etdataset in quintel/etdataset@2b6912780c76abb9634a953d8853dcd4e203f7d1


You sure to not use an abstract function `open` to let the user open a stream? If the implementor does non-sense this could mess up things when using it too early.
So I would
```python
def __init__(self):
  self._open()

def _open(self):
  self.open()
  self.__closed = False

@staticmethod
def open():
  # Override here
  pass
```


s183736 er studnr mitt


Yes it should do - I'll do that.


@looopTools And let's copy the same parameters to this second constructor.


Такой способ подключения шрифтов не желателен
https://github.com/snphq/snp-component-sass-mixins/blob/master/index.sass
Используй mixin font-install


static method, no need to invoke via instance: `self._close()` -> `_close()`


No. There is no any changes in this file.


I created https://github.com/OPM/opm-parser/issues/445 for this.


https://www.youtube.com/watch?v=BpJQGyY1KhI :sun_with_face:


возможно нужно использовать позиционирование


Well with jnr-posix support in the VM and escape analysis, I would hope that the stat structure would end up stack allocated and then we would get an inline system call, which may be more efficient than what Java can currently do. Not sure if we're there yet.


The same here: This line is enough for the class description.


Не оставляй закоментированный код. Все изменения есть в git


How can clean all options, because there is a default value. I think it is not possible, because TSO_NONE means NOT modify any options.


Thanks. I amended it to the commit (still squashed).


I have an AC68U btw...
well as it stands now... the nat loopback is unusable... either asus or merlin option don't work...
merlin doesn't work at all because of WRS and asus works for 5-10 min then everything related to nat loopback dies as I said above... all of this makes the firmware unusable for people like me that have sites hosted behind the router...
I think WRS needs to be removed since its closed source and we have no control over it....  or at least separated into 2 version with or without it.... I'm pretty sure besides some technologically inept people no one needs it...


Well, my patch works for me yes, but I never tried it against git, only
2.2.0 at the latest. Had a quick look and it seems they removed the
"activity" callback in favor of "input-current"[0], so I'd suggest
trying that one instead if you haven't already.

If that doesn't work though, I don't know. As I said I only worked
against 2.2.0 so I don't know what else might have changed...

[0]
https://github.com/videolan/vlc/commit/2d6e21de8306826313f11b50ec235abea1671351


I think it's possible that someone could modify an instance of `Stat` to include other instance variables before calling this, so it could see multiple shapes.


Please use better name of variable.


Dont think so..


what kind of nonsense could an implementor do?


bin -> bucket


I should really make it so that `@CoreClass` creates as many modules as needed.


but the method can be overridden nonstatically!


Думаю тебе нужно почитать о нейминге стилей
https://git.snpdev.ru/frontend/study_frontend/blob/master/basics/standarts.md#css-html


You now get an error at the point of use.


amazing!


Some debugging code? See https://sourceforge.net/p/phpmyadmin/bugs/4846/


I think using TSO_NONE and other options are contradictory.


TODO: fix double-check locking synchronization


Will use Unlimited and your strings. thx much @da-anda

```
<section id="videos">
  <category id="videoacceleration">
  <group id="3">
    <setting id="videoplayer.limitguiupdate" type="integer" label="38013" help="38014">
      <level>2</level>
      <default>10</default>
      <constraints>
        <minimum label="38015">0</minimum> <!-- No Limit -->
        <step>5</step>
        <maximum>25</maximum>
      </constraints>
      <control type="spinner" format="string">
       <formatlabel>38016</formatlabel>
      </control>
    <control type="edit" format="integer" />
   </setting>
</group>
</category>
</section>
```
```
#. Label of a setting to limit the number of fps used for updating the GUI while playing videos. This is useful for slow systems that have problems rendering GUI and video at the same time in full speed.
#: system/settings/settings.xml
msgctxt "#38013"
msgid "Limit GUI updates during playback"
msgstr ""

#. Description for the setting Video -> Acceleration -> Limit GUI updates when playing video with label #38013
#: system/settings/settings.xml
msgctxt "#38014"
msgid "Limits the speed (fps) used to update the GUI while playing videos. This can reduce CPU load and fix playback issues while the GUI is shown."
msgstr ""

#. Label of a settings value to indicate the "disabled" state of an otherwise limiting setting (like fps limit, bandwidth limit, ...)
#: system/settings/settings.xml
msgctxt "#38015"
msgid "Unlimited"
msgstr ""

#. Used to format framerate values. %d will be replaced with the according number/integer, like "24 fps", "50 fps"
#: system/settings/settings.xml
msgctxt "#38016"
msgid "%d fps"
msgstr ""
```


:dancer:


Ah... :)

Could you add an exemplar command line in the notes, possibly?  Thanks!


Зачем заворачивать @ui.videoTag в jquery? Это итак jquery-объект


Такая работа с моделью не приемлема.
Нужно использовать методы `set` или `get` для изменения атрибутов модели.
Прочитай еще раз документацию [по моделям](http://backbonejs.ru/#Model)


@dead-horse this is broken change?


Test comment


According to the Dell documentation, API in the Header is preferred.


Fixed it [here](https://github.com/xwiki/xwiki-enterprise/pull/65).  Sorry for not checking the other functional tests. Thanks!


i can create pull request today but without tests. I am now working on tests but it is new for me so may take some more time. Also today we have few big meetings so don't have that much time to implement anything. Please let me know if you want to have pull request without tests or we wait until full quality is achieved :)


@KunalKene1797 Copyright 2012-2014,The BlackBox Android Project
Whaaaaaat?
Shouldn't it just be 2015 :P


Желательно путь до файла вычислять в модели


Yes, agree, modulo the warning message...


This comment does not add any plus information. Please remove it.


No, the keyword was "centrally" ;-)

Seriously though, I realize that puts me as a "bottleneck" for this machine, but its not going to work if more people are going to attempt this either.

Still waiting on a response from STFC to open the port on the firewall unfortunately ...


What is this file about? It seems to be an outdated version of the omn ontology. Could you please remove this file and use the version bundled with the omn translator?


@adarshmishra remove bluetooth, this looks to be client project code


I asked this question to @martinfurmanski and @apcj. They also felt it was a bad idea to try and catch `Error`, and it's best to let it die (in this case it would kill the thread, the JVM would report an uncaught exception, and the rotation flag would remain set such that it would never try rotating again until after a restart).


@DDuarte i tested and... Old locales like frFR are not working, the name/subname are always taken from creature_template and not from locales_creature, the same for the new locales. So i think that this implementation is correct, but there is something other that have to be fixed but i don't know where to look, can this be an opcode error? Or what?


Good question. From ssl v 1.0.1 those are also defined. The docs seemed a bit out of sync.
I checked also the 0.9 tree, and 1.0.0 tree. They do not contain the 1_1, and 1_2.
However, I can protect those two options with an #ifdef indeed.


@adarshmishra remove "build" folder


svctags: Single or Multiple Service tags
Note: Each tag must be separated by "|".


I wonder if we need a CTASSERT() that (PAGE_SIZE / CHERICAP_SIZE) evenly divides by (8 * sizeof(uint64_t)) -- as this is integer division, we could get an array that is too small if, for example, capability size grew further -- or page size shrunk.


I think we are OK losing the comment here.


When I think over it again you can't do something wrong. I thought maybe the implementor just initializes the stream to late and it gets accessed earlier. But anyway he is responsible for that to handle that correctly^^


Likewise here, I think we're OK losing the comment.


Change to "1A2B3C4" to match Dell documentation.


Разумнее было бы выделить отдлельный класс   `.btn` и для него определить всей свойства соответствующие кнопке и `cursor: pointer` в том числе


I wonder if an XXX comment suggesting the future use of CSETTAG() might be desirable here.


Python really allows that?


ack


As far as my knowledge goes, there isn't a simple repeat loop in Python that doesn't require a count variable, and I can't see another efficient way to structure the code.

I picked _ because I thought it was the only variable name that would stop the 'unused variable' warning. I'm quite happy to revert the _ changes in all non-test code if you'd like, and we can just live with that continued warning.


Rather than 11, should we be using CHERI_CR_CTEMP0 here?


@andisadewi : isn't this part of one of the omn ontologies?


Take a look at the new code setting up the offset/etc in cheri.c -- we might need to reorder these a bit. Also, an XXX about future use of CSetBound might be appropriate.


Thanks, seems to do the job!


Likewise CHERI_CR_CTEMP1 here?


@hmerk Ein guter Beginn wäre das Anlegen von Test-XML Dateien (via PR) an dieser Stelle: https://github.com/4thline/cling/tree/master/core/src/test/resources/invalidxml


Isn't this a duplicate of e.g. https://github.com/open-multinet/playground-rspecs-ontology/blob/master/ontologies/omn-service.ttl#L154 ?


fair enough: loading 2.7.2-epd gives a deprecation warning, and is expected to disappear.  The explicit python interpreter path gave trouble during last weeks MD testing, as that has changed on stampede, and we thus had to push a hotfix release.  It seems to work with the normal python module at this point.


TeamCity Toolbox :: Pull requests [Build 59](http://10.48.34.86:8111/viewLog.html?buildId=8183&buildTypeId=Toolbox_PullRequests) is now running


I wonder if there are any cache considerations here.


right


Thank you :)


left


Why not let SectionFiller handle interactor/printer closure?


TeamCity Toolbox :: Pull requests [Build 59](http://10.48.34.86:8111/viewLog.html?buildId=8183&buildTypeId=Toolbox_PullRequests) outcome was **SUCCESS**
Summary: Tests passed: 168 Build time: 00:00:43


Could use ffs() or such?


Thanks, good news.


Likewise, I wonder if there are any cache concerns here.


 Нельзя к модел добавлять произвольные свойства таким способом.
По все признакам здесь не модель должна быть не модель, а коллекция.
Кроме, того фрагмент из stub дублируется еще раз.
Разумнее всего здесь использовать ServerApi


A few comments on the CHERI swap-bit preservation/restore code.


lol


lol :smile:


Not like this?
```python
new_section = self.uut.fill_section([
            LocalTestBear,
            GlobalTestBear,
            "an inappropriate string object here"])```


may this be a problem?


Этот фрагмент нужно будет переписать


This is unnecessary, because ```setState()``` already forces it to update. ```forceUpdate()``` is for cases where your ```render()``` depends on something besides ```this.state``` or ```this.props``` and you change that something.


Yes, sorry. I'll remove this and I think it's worth releasing 4.4.1.1.


I am very disappointed at git that the move into the src/ subtree broke the log history for individual files -- so I also did not dig further to see when this got introduced.  I am sure slurm worked in the past though, on that level.

I'll try to look at the hist again...


Choses faites :
- Bundle Acme retiré.
- Ajout des flash-messages autogérés (+ Bootstrap) directement dans le layout.
- Correction des routes.
- Correction du controlleur.
- Correction de la template de base Bootstrap (notamment retirer le toggle navigation et remettre le footer en place)
- Correction de la page d'inscription (et de son flash message)
- Indentation corrigé sur la plupart des travaux listés ci-dessous.
- Suppression des commentaires inutiles.
- Code commenter pour le contrôleur. (+ Mise en place des TODO)

=> Plusieurs points seront cependant à revoir pour les routes, les ressources, la vérification de l'inscription, le renvois de message d'erreur, etc. Je laisse @lpera s'en charger.


@smarterclayton I am not sure about what Kube aliases to define here...


A little language-style comment:

Loads all configuration files, **retrieves bears and all needed settings**, saves back if needed and warns about **non-existent** targets.


I'm searching why my Due stop while printing. I started realy simple with debugging and send every 2.5s a serial. While the print is waiting for something, the debug came every 2.5s. After a while ~20s (or less or more) the print continues. Could this patch help with that? Or any other idea where to look at it? I'm running out of ideas.


Please do.

It's a matter of coding style, but when writing Python I'm more in favor of "forgiving" (`try` `catch`) rather than "ask permission" (`if`).

Maybe you won't see this in all topydo code, but it's an insight I gained along the way.


@adarshmishra  need to change to welcomeMessage


@akissa Thank you for the submission.

Can you please clean up the commit? This is a multi-feature and formatting commit. A single feature per pull request?


Parameter alignment!
```
:param arg_list: CLI args to use
:return:         A tuple with with the following contents:
                 ...
```


touple -> tuple


Ok.

Sent from my iPad

> On Apr 7, 2015, at 9:31 PM, Joe <notifications@github.com> wrote:
>
> Take a look at the GitHub comments...they're there for a reason.
>
> —
> Reply to this email directly or view it on GitHub.
>


This is not the part you are looking for. This part is processing our internal option flags / transform them to openssl options.


This changed post-renaming.
It's hidden in the merge on March 30.

> On 08 Apr 2015, at 13:02 , Andre Merzky <notifications@github.com> wrote:
>
> I am very disappointed at git that the move into the src/ subtree broke the log history for individual files -- so I also did not dig further to see when this got introduced. I am sure slurm worked in the past though, on that level.
>
> I'll try to look at the hist again...
>
> —
> Reply to this email directly or view it on GitHub.
>


TeamCity Ybackend :: synteny/ybackend [Build 17](http://192.168.30.225:8111/viewLog.html?buildId=1092&buildTypeId=Ybackend_SyntenyYbackend&guest=1) is now running


COMPUTER*


ack


There are four possible measurment states: 0 (to near), Infinity (to far), NaN (invalid) and everything else (valid).
Since 0 is interpretated as to near it is the right thing to use here.


TeamCity Ybackend :: synteny/ybackend [Build 18](http://192.168.30.225:8111/viewLog.html?buildId=1093&buildTypeId=Ybackend_SyntenyYbackend&guest=1) is now running


:+1: thanks


@AlexanderWillner which part?

Am 08.04.2015 um 12:57 schrieb Alex <notifications@github.com>:

> @andisadewi : isn't this part of one of the omn ontologies?
>
> —
> Reply to this email directly or view it on GitHub.
>


@janodvarko I agree that it is now switching the sidebar to "Packet Details" too often, I've pushed a fix in #27 (but if it still switches to the Packet Details too often then I will revert this change completely)


@adarshmishra how did the language changed here?


LGTM

@kolloch the `sed -e 's/ $//'` in the original version removed the trailing whitespace that was earlier added to the string as part of awk's `printf("%s ", $i)`. In some implementations of sed it also adds a newline (e.g. on OSX BSD sed it does, on Linux GNU sed it doesn't).

The `END {print("")}` in the new version simply prints a newline. However the trailing space is retained. So in that regard this commit differs from what the old code was doing.

That being said in the context of this script both, `sed -e 's/ $//'` and `END {print("")}` are unnecessary because the for loop will remove any newlines and trailing whitespace anyways. So in the end it doesn't make a difference and if you wanted to optimise it further the awk END part could be removed as well.


Ah okay I get it, this is a base class.


Could you explain why we don't need it anymore and what the connection block is?


got it, thanks :)


How does this iterator name differ the one in eg. the "j" as the first local variable in tls_get_x509_digest?
Do you have any suggestion?


Use `%r` around regular expression.


@adarshmishra remove this.


Исправила все ошибки, кроме equals. Посмотрите, пожалуйста


just delete


OIDC session management iFrame communication flow


Let me know if you have any questions...


its not that complicated.


Your commits should explain the actual changes to the code you made. If there are many unrelated changes then you could do multiple commits


Логи запросов завтра вечером закину


(your commit messages, I mean to say)


we must make sure that `SnapshotOffer` and `RecoveryFailure` are not treated as domain events


It looks like this change hasn't been published yes and it still shows the old version which prevents the successful build:
/Users/user/.cargo/registry/src/github.com-1ecc8888db9ec888/ncurses-5.71.1/src/ncurses.rs:24:25: 24:39 error: unresolved import `std::ffi::c_str_to_bytes`. There is no `c_str_to_bytes` in `std::ffi`
/Users/user/.cargo/registry/src/github.com-1ecc8888db9ec888/ncurses-5.71.1/src/ncurses.rs:24 use std::ffi::{CString, c_str_to_bytes};
It looks like this change hasn't been published yet and it still shows the old version which prevents the successful build:
/Users/user/.cargo/registry/src/github.com-1ecc8888db9ec888/ncurses-5.71.1/src/ncurses.rs:24:25: 24:39 error: unresolved import `std::ffi::c_str_to_bytes`. There is no `c_str_to_bytes` in `std::ffi`
/Users/user/.cargo/registry/src/github.com-1ecc8888db9ec888/ncurses-5.71.1/src/ncurses.rs:24 use std::ffi::{CString, c_str_to_bytes};

rustc-1.0.0-Beta


@AlwinEsch np, I was trying to make sense of your OSDDSPAudioOnNF.png OSDDSPAudioOffNF.png OSDDSPAudioOffFO.png

as  these just look the same to me over a black background, but without actually testing this, cant see them in action, so I'll wait till this is merged to see what its suppose to be. too many glyphs in one image as is for one thing.

Once merged I can see what they suppose to be and if you want I can help you with those.


@rpl it works great now, it doesn't switch to the packet-details at all ;-)
I don't know if this is by mistake, but I believe it's the best behaviour.

Thanks Luca!
Honza


But CoreClass cannot know if they are module or classes, so the current approach looks good to me.


if all steps have a description it could be made second positional parameter of Step(), less typing


Ok, thanks! I wasn't aware of the changes that were made to deprecate the 'ng' module name in favor of the 'angular' module name.


Why do you use ' instead of " ?


Travis doesn't let himself get cheated that easily :( Must find underlying issue :weary:


> Please do.

Okay. I will push rewritten commit later today. By the way: can You give me some insights on #2?


@adarshmishra there is an activity create MobiComActivity, please see if we can use it.


aesthethical: I would switch the two asserts because of the index order :)


Am not sure i understand, you want me for format the old code in one commit, exclude private addresses in another and then add support for return values in another ?

The last two are closely linked.


Right, but the logic would still be simpler than handling primitives I believe.


Pushed from Git on local PC


TeamCity Ybackend :: synteny/ybackend [Build 18](http://192.168.30.225:8111/viewLog.html?buildId=1093&buildTypeId=Ybackend_SyntenyYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:09:30


TeamCity Ybackend :: Feodorov/ybackend [Build 132](http://192.168.30.225:8111/viewLog.html?buildId=1094&buildTypeId=Ybackend_FeodorovYbackend&guest=1) is now running


i love EST


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


The pthread_tryjoin_np() interface is non-portable.  Is there any reason not to replace it with a normal pthread_join()?


debug purposes, pushed by mistake. will revert at next commit


But this is the case only when reading data. Here the data (~50.000 values) is copied. However I don't see any other way, the data is aquierd column-wise and later on stored in row major order while the amount of colums is a-priori unknown.


Are you sure? What I meant here:
`If the last rates were fetched less than one hour ago, return them. Otherwise fetch new rates.`
Please note that I am returning the last rates here, so the condition should be `if their timestamp is greater than one hour ago`. I think time comparison works on `Time` in UNIX format (integer).
```2.2.0 :015 > Currency.last.created_at
  Currency Load (1.3ms)  SELECT  "currencies".* FROM "currencies"  ORDER BY "currencies"."id" DESC LIMIT 1
 => Tue, 07 Apr 2015 16:01:02 UTC +00:00
2.2.0 :016 > 1.hour.ago
 => Wed, 08 Apr 2015 09:40:09 UTC +00:00
2.2.0 :017 > Currency.last.created_at > 1.hour.ago
  Currency Load (1.3ms)  SELECT  "currencies".* FROM "currencies"  ORDER BY "currencies"."id" DESC LIMIT 1
 => false
2.2.0 :018 > Currency.last.created_at.to_i
  Currency Load (1.3ms)  SELECT  "currencies".* FROM "currencies"  ORDER BY "currencies"."id" DESC LIMIT 1
 => 1428422462
2.2.0 :019 > 1.hour.ago.to_i
 => 1428486077
2.2.0 :020 >
```
As you can see, I created the `Currency.last` record yesterday which in UNIX format is less than one hour ago (unix counts seconds from midnight 1970-01-01, correct me if I'm wrong). So, if it is less than hour ago, I want to have new rates fetched. If it is more than one hour ago, I want the rates from the db. I hope it's right?


ack


This is not an appropriate way to update gems in our catalinker setup (as we run ruby and bundler inside Docker). I have committed a fix in `1f74e59a6e7f70acd80dd74fe794a08b8fd7ffce`.


I had same problem. To solve it, I added two parameters to initialize uploader:
```javascript
data: {"_csrf": "XXXXXXXXXXXXXXXXXXXX"}},
multipart: true,
```
This code is generated on server, but you can use JavaScript yii.getCsrfParam() and yii.getCsrfToken() on client-side:
```javascript
data: {yii.getCsrfParam(): yii.getCsrfToken() }},
multipart: true,
```


TeamCity Ybackend :: synteny/ybackend [Build 17](http://192.168.30.225:8111/viewLog.html?buildId=1092&buildTypeId=Ybackend_SyntenyYbackend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 72 Build time: 00:12:31
### Failed tests
```
ru.biocad.ybackend.backend.MetaServiceTest.MetaActor should list config if user has RO access: class java.nio.charset.MalformedInputException: Input length = 1
java.nio.charset.MalformedInputException: Input length = 1
      at java.nio.charset.CoderResult.throwException(CoderResult.java:277)
      at sun.nio.cs.StreamDecoder.implRead(StreamDecoder.java:338)

```


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 294](http://192.168.30.225:8111/viewLog.html?buildId=1095&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


Formatting shouldn't be part of the commit when the feature is something else


Haha lol i'll fix that! Used Default Android Dates xD


no indentation by 4 spaces?


indeed it does :)


Hm... nu prea merge asa


Here is a change unrelated to the original feature


I think SQLite-related comment does not belong here. I would put it in Configuration guide and reword it to emphasise that SQLite intended *only* for development use:

>Python ships with SQLite DB library that can create lightweight file-based databases. This makes it very easy to use in development environment: you don't have to setup a database instance! OpenTAXII will create database files ``/tmp/auth.db``, ...


because the SectionFiller gets the printers as an argument, thus they are
maybe used outside it and should be closed there.

2015-04-08 13:01 GMT+02:00 Makman2 <notifications@github.com>:

> Not like this?
>
> new_section = self.uut.fill_section([
>             LocalTestBear,
>             GlobalTestBear,
>             "an inappropriate string object here"])```
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/3a4a41467437adaf1ee0ee7cf1e7b3adfb4c8ae4#commitcomment-10621168>
> .
>


actually it is, but I think it is insignificant in most situation and maybe consider as a bug.


Actually this code isn't relevant for getting the smallest image. It will however fix the easy part of https://github.com/ResponsiveImagesCG/wp-tevko-responsive-images/issues/48.


Okay, for the moment at least, I've removed the _ changes from the non-test code.


no reason.


ack


I added some comments to what is bothering me about this request


NP, will reformat when i get some time.


like this? > cat /sbin/dnsmasq
#!/bin/sh
d=/usr/sbin/dnsmasq
if [ "$1" = "--help" ] ; then
    $d "$@" |grep -v bind-dynamic
else
    exec $d "$@"
fi


a bit tricky, but ack


I don't like this implicit fallbacks. I think this should result in an error instead like on regular queueing systems. It will avoid possible confusion.

Also, I don't quite understand why you would want a different core allocation logic for profiling and non-profiling mode.


Should be

> Include editor config

for consistency in the prose with the `travis` option


wrong indentation.


what's with log_printer?


low byte first ;)


missing newline right in the editorconfig lol


Same as above. I really don't think this is a good idea.


`editoconfig`? I would go for `editorconfig` ;)


PEP8, extra trailing space here.


TeamCity Ybackend :: Feodorov/ybackend [Build 132](http://192.168.30.225:8111/viewLog.html?buildId=1094&buildTypeId=Ybackend_FeodorovYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:09:35


Why spending time duplicating work? I had an open PR for this.


you commented on the wrong one :) but okay


> Include EditorConfig


"SSL_CTX_set_options() adds the options set via bitmask in options to ctx. Options already set before are not cleared!" How can clear all (default) option(s) ?


Yea, can't quite crack it and don't want to spend all day on it.


PEP8 should have new line at the end


it is. Plus bracket.


you mind to change to "?


FYI: https://github.com/bhanuvrat/junction/blob/master/.editorconfig


yeah... thats missing...


I don't understand why we don't want to compile with -O2 (or at least -O).


TeamCity Ybackend :: biocad/ybackend [Build 176](http://192.168.30.225:8111/viewLog.html?buildId=1097&buildTypeId=Ybackend_BiocadYbackend&guest=1) is now running


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


@smarterclayton I have changed the alias from a slice to a simple string because passing multiple types separately in `ResourceTypeOrNameArgs` means we have to pass all the resource names as well. Instead, the alias being a comma separated string, works like a charm since all the parsing is handled by `ResourceTypeOrNameArgs`:

```
[vagrant@openshiftdev sample-app]$ openshift cli new-app https://github.com/openshift/ruby-hello-world -l name=hello-world
ruby-hello-world
ruby-hello-world
ruby-hello-world
ruby-hello-world
Service "ruby-hello-world" created at 172.30.17.198:8080 to talk to pods over port 8080.
A build was created - you can run `osc start-build ruby-hello-world` to start it.
[vagrant@openshiftdev sample-app]$ openshift cli delete all -l name=hello-world
buildConfigs/ruby-hello-world
deploymentConfigs/ruby-hello-world
imageRepositories/ruby-hello-world
services/ruby-hello-world
```


So not like:
```
self.assertEqual(
    ("[" + _("ERROR") + "][" + ts.strftime("%X") + "] " +
        StringConstants.COMPLEX_TEST_STRING, "test")
...
```


This line shouldn't be here.


no...

2015-04-08 13:27 GMT+02:00 Makman2 <notifications@github.com>:

> you mind to change to "?
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/a9fe864e5d45713f17c740f4e746a8f59f4e6e11#commitcomment-10621580>
> .
>


remove this assignment


The problem is that the equivalencies are not being instantiated correctly.  The xarr in this case:
```
In [1]: sp.xarr
Out[1]: SpectroscopicAxis([SpectroscopicAxis([array(-247.97050528800003)], unit=Unit("km / s"), refX=23694500000.0, refX_units='Hz', frame=None, redshift=None, xtype=None, velocity convention=None),...,SpectroscopicAxis([array(384.547314282)], unit=Unit("km / s"), refX=23694500000.0, refX_units='Hz', frame=None, redshift=None, xtype=None, velocity convention=None)], unit=Unit("km / s"), refX=23694500000.0, refX_units='Hz', frame=None, redshift=None, xtype=None, velocity convention=None)
```
actually looks very odd... I think it's 3 `SpectroscopicAxis`s stitched together?  That shouldn't be, I don't think.

But in any case, the problem is that this has everything specified correctly to have a velocity equivalency, but no velocity equivalency.  I think this comes down to an interaction between the FITS reader and the quantities.

In the FITS reader:
https://github.com/pyspeckit/pyspeckit/blob/master/pyspeckit/spectrum/readers/__init__.py#L55
we need to have pyspeckit determine the appropriate spectroscopic convention.  This is one of those cases where specutils should do the work for us in the future, but for now we have to deal with it.

Copy this function:
https://github.com/radio-astro-tools/spectral-cube/blob/master/spectral_cube/spectral_axis.py#L6
into the header parser (`make_axis`) and have it look at the `CTYPE1` header keyword to determine which velocity convention to use.


I am not sure about the way I use `ResourcesForAlias` in `ResourceTypeOrNameArgs` though but I guess it's ok to do so.


Je me suis rendu compte que ce n'étais pas ça en effet, je vais modifier les valeurs et vérifier si la barre bouge. Si c'est le cas, c'est juste que pour une raison inconnue **cette** barre a bougé. Si la barre ne bouge pas, alors c'est qu'un autre mod le touche, ou que la valeur n'est pas initialisée pour une raison quelconque. La dernière idée que j'ai, c'est que hunger utilise des fonctions qui sont altérées/absentes du minetestforfun_game. À terme je mettrai l'ancienne version de hud en supprimant hunger et je déboguerai sur minetest_game.


no, this is aligned with the previous line. Otherwise it would be
```
(
    first part
    second part)
```


This is more a thinking out loud here: If prep events have a `terminal_event` value and all terminal events don't have a `terminal_event` value, can we now remove the `is_prep` field. The one problem I can see this causing is a creation of a terminal event would then have to occur prior to the creation of prep events despite prep events happening prior to terminal events chronologically.

Presumably though if the user was to have one or more prep events and create those first, they would then have to go back through all the prep events for the terminal event once it has been created and add the terminal event data to the event - seems like a time consuming annoyance at the very least.

If all events have a terminal event at some point and presumably they do? (@cecilia-donnelly can you confirm) then maybe we should not allow prep events not to have terminal events. If it's more convenient for the data entry person we could still display an `is_prep` type checkbox that we have today and if checked but no terminal event selected we could create one for the user without certain attributes (date, venue, etc.), this would mean in the future they have all prep events always linked to the terminal event and also all they have to do is update the terminal date when/if they know the data we're not able to fill in based solely on the prep event.

Sorry for rambling, just thinking outloud here.


The intent of "none" was to clean all _our_ default option list, not ssl's, since our default for ssl_options(no-sslv2).
This was hardcoded into the syslog-ng's code, before the patch. Now it can be cleared if you give only the "none" option.
If you want to clear that out, than we will not modify the openssl's default options.


okay


then ack


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.176](http://teamcity.monogame.net/viewLog.html?buildId=6088&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 481, ignored: 6 Build time: 00:14:40


> Why spending time duplicating work? I had an open PR for this.

:+1: https://github.com/yeoman/generator-node/pull/96

This commit also has a lot of obvious mistakes...


I follow the BEM syntax; in this case it would be: ``ng-inline-edit__button--save``


You removed the extra word, but you kept the `exsits` typo ;)


TeamCity xBIM :: XbimExchange :: Build [Build 199](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1316&buildTypeId=XBIM_XbimExchange_Build) is now running


Yes, like that.

You could move the original binary to `/usr/sbin/dnsmasq.dist` and then put the wrapper in its place.  However I think the wrapper should check whether it is being invoked via libvirt, and only change behaviour in that case.


The thing is that people rarely report IDEA version they use, so adding ".Idea13" and ".Idea14" might be a good idea.


You are assigning nodejs to python.


sys/mips/include/cpuregs.h says:

#ifdef __mips_n64
#define	MIPS_DIRECT_MAPPABLE(pa)	1
#define	MIPS_PHYS_TO_DIRECT(pa)		MIPS_PHYS_TO_XKPHYS_CACHED(pa)
#define	MIPS_PHYS_TO_DIRECT_UNCACHED(pa)	MIPS_PHYS_TO_XKPHYS_UNCACHED(pa)
#define	MIPS_DIRECT_TO_PHYS(va)		MIPS_XKPHYS_TO_PHYS(va)
#else
#define	MIPS_DIRECT_MAPPABLE(pa)	((pa) < MIPS_KSEG0_LARGEST_PHYS)
#define	MIPS_PHYS_TO_DIRECT(pa)		MIPS_PHYS_TO_KSEG0(pa)
#define	MIPS_PHYS_TO_DIRECT_UNCACHED(pa)	MIPS_PHYS_TO_KSEG1(pa)
#define	MIPS_DIRECT_TO_PHYS(va)		MIPS_KSEG0_TO_PHYS(va)
#endif

I wonder if CHERI uses __mips_n64 and whether this goes through the cache like a normal access? If so, would there be a problem?


This would fail to replace file that already exists with socket that we are trying to extract


Georgeous! :heart:


`.withOptions({travis: false})`


Can you enable kexec in titan_defconfig too ?
Thanks ;)


On Tue, Apr 07, 2015 at 09:46:12PM -0700, nrgaway wrote:
> I back in an AppVM on USB stick.  I will comment that when I get my system up and running.  I can get you qubes-builder tomorrow, just let me know how fast you need it; it won't take long to re-create the commits since most all the changes are in `nrgaway` branch already.  The only issue I have right now is I can not sign it until I get my system restored since I do not have access to my keys and I don't have enough space to restore a backup on USB stick... I want to try to recover my HD first since it been a few weeks since last backup.  If I can't get it recovered by tomorrow, I will just over-write it with restore image.

Take your time, nothing urgent.

> This sucks :)  It's the third time I lost my data; every time it has been when building.  The reason it happens is since I am using BTRFS and neglected to partition properly for VM images.  Number 1, each VM should have its own BTRFS share, and COW disabled.  What happens is because of COW, disk space runs out and when this happens during a write, the system locks up; and 1st superblock get corrupted along with other data.  It has pointers to data that exceeds the disk size, which is an error and reason for failure to boot and is very difficult to recover from.  With BTRFS, its hard to tell when you running out of space; a `df` told me I have over 500GB free, but I forgot I created a snapshot last week before updating to newest package releases and have been building qubes on and off for about a week testing all my changes.

Even on standard LVM, lack of disk space in dom0 on Qubes is quite
catastrophic (no way to start any VM, in corner case - no way to login).
If BTRFS gives you broken filesystem in such case, its even worse.

--
Best Regards,
Marek Marczykowski-Górecki
Invisible Things Lab
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?


Marged


@janodvarko Oh my... yes there are a couple of typos in the componentWillReceiveProps check, sorry for that.
I've pushed on #28 a commit which fixes this typos (and it does switch the sidebar tab)

Let me know if the #28 behaviour is reasonable (if it's not I'll push on #28 a new commit which doesn't refresh the sidebar tab... but without my typos :-P)


`initializing: function () {`


Não esta correto


TeamCity xBIM :: XbimExchange :: Build [Build 199](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1316&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:03:30


TeamCity Ybackend :: biocad/ybackend [Build 176](http://192.168.30.225:8111/viewLog.html?buildId=1097&buildTypeId=Ybackend_BiocadYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:09:34


TeamCity Ybackend :: biocad/ybackend [Build 177](http://192.168.30.225:8111/viewLog.html?buildId=1098&buildTypeId=Ybackend_BiocadYbackend&guest=1) is now running


@aslakhellesoy Gherkin.NuGetPackages.csproj is not a .NET assembly project, so it cannot depend on libraries. What was the issue?


TeamCity Ybackend :: biocad/ybackend [Build 178](http://192.168.30.225:8111/viewLog.html?buildId=1099&buildTypeId=Ybackend_BiocadYbackend&guest=1) is now running


remove space


This: https://travis-ci.org/cucumber/gherkin-dotnet/builds

My change didn't seem to fix it though...


Ah I see, the xarr looks the same as usual I think.
from __repr__ if self.shape is not () it prints:
```
rep = ("SpectroscopicAxis([%r,...,%r], unit=%r, refX=%r, refX_units=%r, frame=%r, redshift=%r, xtype=%r, velocity convention=%r)" %
                (self[0], self[-1], self.unit, self.refX, self.refX_units, self.frame, self.redshift, self.xtype, self.velocity_convention))
```


A couple of things @cecilia-donnelly

1. if we could be entering addresses from out of state the first question does the limitations on `type` and `direction` always hold true? I recall verifying that call Chicago addresses have both a `type` and `direction` but do all addresses in the US (presuming we allow addresses nationwide).
2. There aren't an infinite number of states we could also provide a set of choices to this.
3. Cities also don't have an infinite number but i presume getting a list of all cities in all states would be huge and probably should be left as you have created the field but it's worth noting.


Can I leave it to you?


@fuzzyami This code smells a bit. I know it's not yours but maybe a slight fix up would help.
Maybe add the original author's name to the file's docstring?


else


That is where it becomes tricky

Am Mittwoch, 8. April 2015 schrieb Szymon Pobiega :

> To be honest I wasn't aware that we want POCO sagas;) If so, it should
> come on the context passed to the saga method but it should not be
> available to normal handlers.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Particular/NServiceBus/commit/5d7ace2bcfc7eafbeefb9c317c448e9c53eff157#commitcomment-10620342>
> .
>

--

Daniel Marbach

Solution Architect, Particular Software

Email:  daniel.marbach@particular.net

Web:    www.particular.net
<http://t.signauxneuf.com/e1t/c/5/f18dQhb0S7lC8dDMPbW2n0x6l2B9nMJN7t5XX4RzDjlVS9QB88pTc_dW3Ljg7x56dWyBdgBXs402?t=http%3A%2F%2Fwww.particular.net%2F&si=4896787407044608&pi=33cc6202-1f31-4040-8d6c-e70641f44bba>


yep, i will take care.


Moving the original binary to /usr/sbin/dnsmasq.dist is not so different from patching it anymore... and I guess, the wrapper script would even break libvirt's process-kill logic that seems to check if it is going to kill the correct binary (so might even break when you upgrade the dnsmasq package).
The only better way I could imagine to solve this is to add a source patch to libvirt to make this configurable, but that would mean running a custom-patched libvirt (actually two, since we have SLE11 and SLE12 hosts) for a long time...


pep8.conf and test.py together create a working PEP8 test that ignores things unique to Docbuilder.


@Zangetsu38 You seem to completely miss my main point in the last response. You did back up your first 2 claims though, so that's nice.

So, here we go:
Yes, DX12 is quite a bit faster if you measure it in draw calls/s and push it to 100% usage. The catch with RPCS3? We wont push it to 100%! We can't execute interpret and emulate all the instructions fast enough to actually get any acceptable number of frames out. DX12 is not gonna benefit us. We currently use OpenGL, so here's a really bad example of RPCS3's main loop:
read and execute the instruction (roughly 50-75% of the time it takes to render a frame) -> do other stuff -> do syscalls and more stuff -> render a new frame with OpenGL (takes roughly 5-10% of time) -> rinse and repeat

So, what did we learn? We aren't limited by the amount of draw calls we can make in a second. Currently we're limited by how fast we can interpret and execute all the other stuff. If we have a homebrew/game that runs @ 60fps, we might be able to push a one or couple extra frames if we use DX12, but even that's optimistic.

"But, but, but... so many draw calls, it must be faster!"
No. That benchmark used for DX11 vs DX12 renders as many primitive and easiest to render shapes possible so the only limiting factor was the amount of draw calls? Do we render primitive shapes all the time? Nope. I haven't seen a game that consists of simple cubes and fog.

TL;DR: DX12 is not going to change anything for RPCS3 in short-term. In 3-5 years, once we have more processing power, it might actually start to be useful.


нету же знака равно


Seems correct, please rather open a new issue than comment here.


Would this error out if socket.bind errored out earlier?


@nmav: is this update abi compatible or do the dependant packages require a rebuild?


newline
ensure_newline_at_eof_on_save
http://sublime-text-unofficial-documentation.readthedocs.org/en/latest/reference/settings.html


newline


On Wed, Apr 08, 2015 at 04:43:06AM -0700, Markus Unterwaditzer wrote:
>    Seems correct, please rather open a new issue than comment here.

I just published and hadn't commented yet. Things should be fine now. If not, as said, please open a new ticket instead.


>magikarp has -1.

Plz revise.


wrong hash style
should be
`about: 'welcome#about'`


Пардон, что не написал в прошлый раз: JUnit предполагает, что инициализация происходит в методе `setUp`.


I can't tell if this is evil or just pointless.


wrong old hash style


I'll tuxify this later :)


has style


I was meaning to fix that. Thanks.


hash style


new line


Probably my cat's fault.


remove empty line


Нормс


TeamCity Ybackend :: biocad/ybackend [Build 177](http://192.168.30.225:8111/viewLog.html?buildId=1098&buildTypeId=Ybackend_BiocadYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:09:32


TeamCity Ybackend :: biocad/ybackend [Build 179](http://192.168.30.225:8111/viewLog.html?buildId=1100&buildTypeId=Ybackend_BiocadYbackend&guest=1) is now running


Looks like there is still a problem with `make push-subtrees` and `gherkin-dotnet`. I'll take care of that if you do your work in the `gherkin3` repo.


Maybe add the link https://bugs.eclipse.org/bugs/show_bug.cgi?id=463002 to the comment in the catch block, to document that it's a workaround?


That's how `life.js` is. True story.


TeamCity xBIM :: Xbim Geometry :: Release [Build 49](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1317&buildTypeId=XBIM_XbimGeometry_Release) is now running


TeamCity xBIM :: Xbim Geometry :: Release [Build 49](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1317&buildTypeId=XBIM_XbimGeometry_Release) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:02


data: {target: "#navbar-collapse", toggle: 'collapse' итп


Yeah, not needed.


а чо за файл ваще? (copy)


On Wed, Apr 08, 2015 at 04:42:15AM -0700, Bernhard M. Wiedemann wrote:
> Moving the original binary to /usr/sbin/dnsmasq.dist is not so different from patching it anymore...

I strongly disagree!!  Patching a binary is a very brittle and risk-prone activity, plus it makes the change in behaviour invisible, which has the potential to cause great confusion later on.

> and I guess, the wrapper script would even break libvirt's process-kill logic

Why?

> that seems to check if it is going to kill the correct binary

It depends on how it does the check.

> (so might even break when you upgrade the dnsmasq package).

The monkey-patched binary would also break when you upgrade.

> The only better way I could imagine to solve this is to add a source patch to libvirt to make this configurable, but that would mean running a custom-patched libvirt (actually two, since we have SLE11 and SLE12 hosts) for a long time...

Right - that's not a nice option :-/  Are you sure there is not a different strategy regarding the networking?


Cool thanks. I see ``_`` is a common throw away variable name but I prefer _unused or something more descriptive.


TeamCity Ybackend :: biocad/ybackend [Build 178](http://192.168.30.225:8111/viewLog.html?buildId=1099&buildTypeId=Ybackend_BiocadYbackend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 72 Build time: 00:11:55
### Failed tests
```
ru.biocad.ybackend.backend.MetaServiceTest.MetaActor should list config if user has RO access: class java.nio.charset.MalformedInputException: Input length = 1
java.nio.charset.MalformedInputException: Input length = 1
      at java.nio.charset.CoderResult.throwException(CoderResult.java:277)
      at sun.nio.cs.StreamDecoder.implRead(StreamDecoder.java:338)

```


TeamCity Ybackend :: biocad/ybackend [Build 180](http://192.168.30.225:8111/viewLog.html?buildId=1101&buildTypeId=Ybackend_BiocadYbackend&guest=1) is now running


hi @fmbiete,
i'm not using z-push-top/push connection (i've a custom proxy in front of zpush for that)/changesink,
that's why this part is so buggy, sorry :(

I'm calling DoForcePingTimeout in the constructor to timeout as soon as possible other ping request.
I really don't see why it loop forever, please tell me your php/php-redis/redis version


@RAlexis made my day ^^


I have defined these properties temporarily here because they are required for SSH service.


Commented Line fails here


@eitanavgil  I guess it's just for testing...?
We will use core playlist plugin eventually, right?


I guess than if there is no more object creation involved, we can rename this plugin to a more meaningful name, like hand of god or magnetic grasp?


private[scalismo]?


PHPCS: Expected 1 space after FUNCTION keyword; 0 found


No. I mean ONLY store the height of the tree. Then you can call this.left.height and this.right.height and compare the values


PHPCS: Line exceeds 120 characters; contains 175 characters


Route::get('deployment/{deployments}', [ // FIXME Should this be on the deployment controller?


thanks, i've pushed a fix 1b0289bd66ea6012d2f0ef6e37f5f201ef136de7


little fix
-int lock_trylocklock(LOCK lock);
+int lock_trylock(LOCK lock);


@Shandu :+1:


Is this a good name?


We could revert #290 and just not use libvirt's dnsmasq


I wouldn't fug magikarp


What's the difference between scipy.special.{expit,[logit](http://docs.scipy.org/doc/scipy/reference/generated/scipy.special.logit.html)}?


@mfulgo I put it in because this is a "premature end" and we should call ```startStopListenerDelegate.stop()``` (as plainly stated in the code you deleted :-).  It doesn't necessarily need to be achieved via ```setAttachment()``` though, I just used that because it seemed like a handy feature that fit the bill.  The issue at the time I put it in was that we were missing some stop events, i.e. the gui would continue showing items as playing after a connection error, but I've since found/fixed another bug that may also account for that problem, so it could be that removing this bit is harmless.


I'll check that today and let you know. We're hoping that we can be off of 5.3 completely in a few months and then we would ditch our fork.

On Wednesday, Apr 8, 2015 at 03:06, Benjamin Zikarsky <notifications@github.com>, wrote:

How does PHP5.3.3 behave, when you completely remove the abstract function definition, instead of defining it with an empty body?

—
Reply to this email directly or view it on GitHub.


## TSQL Table Hints

This commits ads initial support for writing [TSQL Table hints](https://msdn.microsoft.com/en-us/library/ms187373.aspx) to optimize some queries.

That means that a query like:

```C#
Database.Query<AlbumEntity>()
.WithHint("INDEX(IX_idLabel)")
.Select(a => a.Label.Name)
.ToList();
```

```SQL
SELECT l.Name
FROM dbo.Album AS a WITH(INDEX(IX_idLabel))
LEFT OUTER JOIN dbo.Label AS l
  ON (a.idLabel = l.Id)

```

Query Hints and Join Hints are not supported jet.


LoL


Strange that this is going to throw a message of "EnumNotStarted" when this could also be the case where enumeration has finished, but I see that's existing behavior.


28:1: I100 Imports statements are in the wrong order. from invenio.utils.shell should be before from invenio.version


I think it would have been clearer to call this commit something like "add control connection heartbeating, independent of gossip", since there was no specific TCP heartbeat previously


4: L102 copyright year is outdated, expected 2015 but got 2014
29:1: I100 Imports statements are in the wrong order. from invenio.ext.legacy.handler should be before from invenio.ext.logging
31:1: I101 Imported names are in the wrong order. Should be CFG_ACCESS_CONTROL_LEVEL_SITE, CFG_DEVEL_SITE
31:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.utils
32:1: F401 'webinterface_proxy' imported but unused
47:13: E731 do not assign a lambda expression, use a def
59:9: B901 blind except: statement
69: D209 Multi-line docstring closing quotes should be on a separate line
69: D203 1 blank line required before class docstring (found 0)
69: D204 1 blank line required after class docstring (found 0)
69: D400 First line should end with a period (not 'e')
69: D205 1 blank line required between summary line and description (found 0)
73:5: E301 expected 1 blank line, found 0
73: D102 Missing docstring in public method
78:13: E731 do not assign a lambda expression, use a def
85:23: E127 continuation line over-indented for visual indent
92:9: B901 blind except: statement
100: D209 Multi-line docstring closing quotes should be on a separate line
100: D203 1 blank line required before class docstring (found 0)
100: D204 1 blank line required after class docstring (found 0)
100: D400 First line should end with a period (not 'f')
100: D205 1 blank line required between summary line and description (found 0)
104:5: E301 expected 1 blank line, found 0
104: D102 Missing docstring in public method
108:13: E731 do not assign a lambda expression, use a def
116:23: E127 continuation line over-indented for visual indent
123:9: B901 blind except: statement
132:1: B901 blind except: statement
138:1: B901 blind except: statement
144:10: E126 continuation line over-indented for hanging indent
145:1: B901 blind except: statement
151:10: E126 continuation line over-indented for hanging indent
152:1: B901 blind except: statement
162:1: E302 expected 2 blank lines, found 1
162: D203 1 blank line required before class docstring (found 0)
162: D204 1 blank line required after class docstring (found 0)
165:5: E301 expected 1 blank line, found 0
165: D102 Missing docstring in public method
171:1: E302 expected 2 blank lines, found 1
171: D210 No whitespaces allowed surrounding docstring text
171: None: None
171: D209 Multi-line docstring closing quotes should be on a separate line
171: D203 1 blank line required before class docstring (found 0)
171: D400 First line should end with a period (not 'l')
171: D205 1 blank line required between summary line and description (found 0)
176:16: E127 continuation line over-indented for visual indent
208: D102 Missing docstring in public method
245: D102 Missing docstring in public method


1: D100 Missing docstring in public module
83:1: E402 module level import not at top of file
84:1: I101 Imported names are in the wrong order. Should be IndentedHelpFormatter, Option, OptionError, OptionGroup, OptionParser
84:1: E402 module level import not at top of file
86:1: I100 Imports statements are in the wrong order. import os should be before from optparse
86:1: E402 module level import not at top of file
87:1: I201 Missing newline before sections or imports.
87:1: E402 module level import not at top of file
88:1: I100 Imports statements are in the wrong order. import random should be before import pkg_resources
88:1: I201 Missing newline before sections or imports.
88:1: F401 'random' imported but unused
88:1: E402 module level import not at top of file
89:1: E402 module level import not at top of file
90:1: E402 module level import not at top of file
91:1: E402 module level import not at top of file
92:1: F401 'string' imported but unused
92:1: E402 module level import not at top of file
93:1: E402 module level import not at top of file
94:1: E402 module level import not at top of file
95:1: I201 Missing newline before sections or imports.
95:1: E402 module level import not at top of file
96:1: I100 Imports statements are in the wrong order. from warnings should be before from six
96:1: I201 Missing newline before sections or imports.
96:1: E402 module level import not at top of file
97:1: I201 Missing newline before sections or imports.
97:1: F401 'exc' imported but unused
97:1: E402 module level import not at top of file
105: D210 No whitespaces allowed surrounding docstring text
105: None: None
105: D400 First line should end with a period (not 'o')
116: D202 No blank lines allowed after function docstring (found 1)
116: D400 First line should end with a period (not 'g')
116: D205 1 blank line required between summary line and description (found 0)
122:5: E266 too many leading '#' for block comment
125:5: E266 too many leading '#' for block comment
137:5: E266 too many leading '#' for block comment
153:5: E266 too many leading '#' for block comment
158:5: E266 too many leading '#' for block comment
164:5: E266 too many leading '#' for block comment
168:5: E266 too many leading '#' for block comment
219:5: E266 too many leading '#' for block comment
226:5: E266 too many leading '#' for block comment
267:5: E266 too many leading '#' for block comment
272:5: E266 too many leading '#' for block comment
277:5: E266 too many leading '#' for block comment
285:5: E266 too many leading '#' for block comment
289:5: E266 too many leading '#' for block comment
311:5: E266 too many leading '#' for block comment
315: D103 Missing docstring in public function
321: D400 First line should end with a period (not 's')
321: D205 1 blank line required between summary line and description (found 0)
337: D205 1 blank line required between summary line and description (found 0)
344:5: E266 too many leading '#' for block comment
346:21: E127 continuation line over-indented for visual indent
347:5: E266 too many leading '#' for block comment
351:50: E502 the backslash is redundant between brackets
352:25: E127 continuation line over-indented for visual indent
352:66: E502 the backslash is redundant between brackets
353:25: E127 continuation line over-indented for visual indent
364: D205 1 blank line required between summary line and description (found 0)
371:5: E266 too many leading '#' for block comment
373:21: E127 continuation line over-indented for visual indent
374:5: E266 too many leading '#' for block comment
377:5: E266 too many leading '#' for block comment
393: D400 First line should end with a period (not '6')
393: D205 1 blank line required between summary line and description (found 0)
410: D400 First line should end with a period (not 'r')
410: D205 1 blank line required between summary line and description (found 0)
418:5: E266 too many leading '#' for block comment
420:14: E127 continuation line over-indented for visual indent
421:5: E266 too many leading '#' for block comment
424:16: E126 continuation line over-indented for hanging indent
425:13: E266 too many leading '#' for block comment
439:45: E128 continuation line under-indented for visual indent
440:45: E128 continuation line under-indented for visual indent
449: D400 First line should end with a period (not 'd')
449: D205 1 blank line required between summary line and description (found 0)
477: D209 Multi-line docstring closing quotes should be on a separate line
477: D400 First line should end with a period (not 'n')
477: D205 1 blank line required between summary line and description (found 0)
503:32: F841 local variable 'err' is assigned to but never used
523: D209 Multi-line docstring closing quotes should be on a separate line
523: D400 First line should end with a period (not 'n')
523: D205 1 blank line required between summary line and description (found 0)
537: D200 One-line docstring should fit on one line with quotes (found 3)
552: D400 First line should end with a period (not 'N')
552: D205 1 blank line required between summary line and description (found 0)
562:5: E266 too many leading '#' for block comment
564:5: E266 too many leading '#' for block comment
566:5: E266 too many leading '#' for block comment
569:9: E266 too many leading '#' for block comment
570:9: E266 too many leading '#' for block comment
586:9: E266 too many leading '#' for block comment
592:49: E128 continuation line under-indented for visual indent
597:9: E266 too many leading '#' for block comment
603:82: E231 missing whitespace after ','
618: D400 First line should end with a period (not 's')
618: D205 1 blank line required between summary line and description (found 0)
637: D209 Multi-line docstring closing quotes should be on a separate line
637: D205 1 blank line required between summary line and description (found 0)
662: D103 Missing docstring in public function
672: D103 Missing docstring in public function
707:72: E502 the backslash is redundant between brackets
708:20: E127 continuation line over-indented for visual indent
708:44: E231 missing whitespace after ','
712:35: E261 at least two spaces before inline comment
731:21: E128 continuation line under-indented for visual indent
731:93: E231 missing whitespace after ','
749:50: E231 missing whitespace after ','
769:48: E231 missing whitespace after ','
822: D400 First line should end with a period (not 'e')
822: D205 1 blank line required between summary line and description (found 0)
844: D400 First line should end with a period (not 's')
844: D205 1 blank line required between summary line and description (found 0)
859: D400 First line should end with a period (not 'r')
859: D205 1 blank line required between summary line and description (found 0)
869:10: E122 continuation line missing indentation or outdented
882: D200 One-line docstring should fit on one line with quotes (found 3)
897: D400 First line should end with a period (not 'y')
897: D205 1 blank line required between summary line and description (found 0)
916: D400 First line should end with a period (not 'L')
916: D205 1 blank line required between summary line and description (found 0)
931: D400 First line should end with a period (not 's')
931: D200 One-line docstring should fit on one line with quotes (found 3)
945: D400 First line should end with a period (not 's')
945: D200 One-line docstring should fit on one line with quotes (found 3)
959: D400 First line should end with a period (not 'd')
959: D200 One-line docstring should fit on one line with quotes (found 3)
973: D200 One-line docstring should fit on one line with quotes (found 3)
987: D202 No blank lines allowed after function docstring (found 1)
990: D203 1 blank line required before class docstring (found 0)
990: D204 1 blank line required after class docstring (found 0)
990: D400 First line should end with a period (not 'l')
1106: D210 No whitespaces allowed surrounding docstring text
1106: None: None
1106: D400 First line should end with a period (not 's')
1145:9: E266 too many leading '#' for block comment
1201:21: B901 blind except: statement


22:1: E402 module level import not at top of file
23:1: E402 module level import not at top of file
24:1: E402 module level import not at top of file
30:1: E302 expected 2 blank lines, found 1
30: D202 No blank lines allowed after function docstring (found 1)
36: D202 No blank lines allowed after function docstring (found 1)
41:1: E302 expected 2 blank lines, found 1
41: D210 No whitespaces allowed surrounding docstring text
41: None: None
41: D202 No blank lines allowed after function docstring (found 1)
47:1: E302 expected 2 blank lines, found 1
47: D103 Missing docstring in public function
50:1: E302 expected 2 blank lines, found 1
50: D202 No blank lines allowed after function docstring (found 1)
56: D210 No whitespaces allowed surrounding docstring text
56: None: None
56: D202 No blank lines allowed after function docstring (found 1)
64: D210 No whitespaces allowed surrounding docstring text
64: None: None
64: D202 No blank lines allowed after function docstring (found 1)
72: D202 No blank lines allowed after function docstring (found 1)
72: D400 First line should end with a period (not 's')
77:1: E302 expected 2 blank lines, found 1
77: D202 No blank lines allowed after function docstring (found 1)
77: D400 First line should end with a period (not 't')
82:1: E302 expected 2 blank lines, found 1
82: D202 No blank lines allowed after function docstring (found 1)
82: D400 First line should end with a period (not 't')
87:1: E302 expected 2 blank lines, found 1
87: D202 No blank lines allowed after function docstring (found 1)
87: D400 First line should end with a period (not 'y')
92:1: E302 expected 2 blank lines, found 1
92: D202 No blank lines allowed after function docstring (found 1)
92: D400 First line should end with a period (not 't')
97:1: E302 expected 2 blank lines, found 1
97: D202 No blank lines allowed after function docstring (found 1)
102:1: E302 expected 2 blank lines, found 1
102: D202 No blank lines allowed after function docstring (found 1)
102: D400 First line should end with a period (not 't')
107:1: E302 expected 2 blank lines, found 1
107: D202 No blank lines allowed after function docstring (found 1)
107: D400 First line should end with a period (not 't')
112:1: E302 expected 2 blank lines, found 1
112: D210 No whitespaces allowed surrounding docstring text
112: None: None
112: D202 No blank lines allowed after function docstring (found 1)
117:1: E302 expected 2 blank lines, found 1
117: D210 No whitespaces allowed surrounding docstring text
117: None: None
121:1: E302 expected 2 blank lines, found 1
121: D210 No whitespaces allowed surrounding docstring text
121: None: None
121: D202 No blank lines allowed after function docstring (found 1)
124:151: E231 missing whitespace after ','
126:1: E302 expected 2 blank lines, found 1
126: D202 No blank lines allowed after function docstring (found 1)
126: D400 First line should end with a period (not 'e')
132: D202 No blank lines allowed after function docstring (found 1)
132: D400 First line should end with a period (not 'e')
141: D202 No blank lines allowed after function docstring (found 1)
141: D400 First line should end with a period (not 'e')
151: D202 No blank lines allowed after function docstring (found 1)
151: D400 First line should end with a period (not 'e')
160: D202 No blank lines allowed after function docstring (found 1)
160: D400 First line should end with a period (not ':')
177: D202 No blank lines allowed after function docstring (found 1)
177: D400 First line should end with a period (not ':')
188:32: E128 continuation line under-indented for visual indent
189:32: E128 continuation line under-indented for visual indent
190:32: E128 continuation line under-indented for visual indent
191:32: E128 continuation line under-indented for visual indent
192:32: E128 continuation line under-indented for visual indent
193:32: E128 continuation line under-indented for visual indent
196: D209 Multi-line docstring closing quotes should be on a separate line
196: D202 No blank lines allowed after function docstring (found 1)
196: D400 First line should end with a period (not ',')
196: D208 Docstring is over-indented
208: D202 No blank lines allowed after function docstring (found 1)
215: D210 No whitespaces allowed surrounding docstring text
215: None: None
215: D202 No blank lines allowed after function docstring (found 1)
222: D210 No whitespaces allowed surrounding docstring text
222: None: None
222: D202 No blank lines allowed after function docstring (found 1)
229: D209 Multi-line docstring closing quotes should be on a separate line
229: D202 No blank lines allowed after function docstring (found 1)
229: D208 Docstring is over-indented
245: D209 Multi-line docstring closing quotes should be on a separate line
245: D202 No blank lines allowed after function docstring (found 1)
245: D208 Docstring is over-indented
261: D209 Multi-line docstring closing quotes should be on a separate line
261: D202 No blank lines allowed after function docstring (found 1)
275: D210 No whitespaces allowed surrounding docstring text
275: None: None
275: D209 Multi-line docstring closing quotes should be on a separate line
275: D202 No blank lines allowed after function docstring (found 1)
275: D400 First line should end with a period (not ':')
275: D208 Docstring is over-indented
292: D209 Multi-line docstring closing quotes should be on a separate line
292: D202 No blank lines allowed after function docstring (found 1)
292: D208 Docstring is over-indented
303:37: E128 continuation line under-indented for visual indent
304:37: E128 continuation line under-indented for visual indent
305:37: E128 continuation line under-indented for visual indent
306:37: E128 continuation line under-indented for visual indent
309: D209 Multi-line docstring closing quotes should be on a separate line
309: D202 No blank lines allowed after function docstring (found 1)
309: D400 First line should end with a period (not 's')
309: D205 1 blank line required between summary line and description (found 0)
309: D208 Docstring is over-indented
335: D209 Multi-line docstring closing quotes should be on a separate line
335: D202 No blank lines allowed after function docstring (found 1)
335: D400 First line should end with a period (not 'a')
335: D205 1 blank line required between summary line and description (found 0)
344: D210 No whitespaces allowed surrounding docstring text
344: None: None
344: D202 No blank lines allowed after function docstring (found 1)
349:1: W391 blank line at end of file