Please revert this change to read.  It's just wrong.  We are not dealing with strings here, we are dealing with a buffer containing binary data.  Also, 3 allocations and 4 copies for absolutely no gain is just... bad.  In addition, it needs to return the number of bytes read, just like the original.

I do not understand why this change was even made.  There is no reason to use std::string here at all, and as you can see, many reasons not to.


That test doesn't look right?


Is this something that definitely should be supported? I saw it in jprops and decided a SyntaxError was better, but didn't test how java reacts.


Like read(), this change to write needs to be reverted as well.  It makes no sense to use std::string here as we are not dealing with strings.  In addition, this function adds a 0-byte to the underlying write call, which is just wrong.

I will be happy to provide a PR to undo these changes to read and write.


That makes *no sense*


Good Work @MaxXor :+1:  :+1:


Well I mean I guess it makes some. Empty key equals empty value. But why would you want that?

Furthermore, what's a line containing just a space (which is a separator character) then? Is it `"" = ""`, or ignored?

There's a lot we want to consider before we support these I think (that is, support in the form of tests).


sum1 revert dis shyt


Not really technologically realistic, but it has an interesting story. Basically the story of a first true A.I. "growing up" (as in, learning about the world and learning what's right and wrong). Great effects (although sloppy in a couple of scenes).

I will add the commentary to every movie I added to this list probably in the next 24 hours.


:boom:


Yep, I'll add some documentation.


To add a little bit more:
I feel manual garbage collection should never be used. It is just too easy to make a hit to CPU just for the extra memory that would have been done at a time better-determined by the garbage collector's algorithms.
After disposal of all the client's forms that it opened, memory should already be low enough again to the point where we shouldn't have to call GC.Collect(). A notable exception perhaps could be usage of the UnsafeStreamCodec. Even then, manual collection should be avoided and left for the Garbage Collector to decide. Before using the UnsafeStreamCodec, this is always a thing to help the Garbage Collector understand the situation to handle our memory usage better.


Protip: http://stevenharman.net/git-pull-with-automatic-rebase

an alwaysrebase keeps a @zenspider happy :+1:


Done in d414a12ad85355ff15204a64be0bebe0d5096842.


Didn't know `getaddresses` existed. Makes sense to check that all returned IPs are internal.

Done in d414a12ad85355ff15204a64be0bebe0d5096842.


I believe this should be %20, rather than %2B


See #504...


Ending note:
I get that you may really want, for any reason, to reclaim as much available memory as possible after disconnection of the client. It would  be the most resourceful and logical for the server to reclaim all available memory here instead of on each client object's disconnection. This way, garbage is collected after all of the clients desired are disconnected.


Er sorry, #500


service 패키지안에 각 서비스별로 패키지 따로 생성
- 서비스가 많아지면 너무 난잡해 질꺼 같아서 해당 서비스별로 패키지 생성.


Elasticsearch was mistaking my elasticsearch.yml playbook for a config file. Changing to the elasticsearch directory when installing bigdesk fixes the problem. See [https://stackoverflow.com/questions/29190666/what-does-this-ansible-error-mean](https://stackoverflow.com/questions/29190666/what-does-this-ansible-error-mean).


It has invalid characters at the end and for some reason the client displays the hover all to the left, which puts the trace off the screen, thus making it useless.


`#seek()`


By calling ..


`_id` and `guid` need to be omitted as well.


line break


The lines below should be modified accordingly:

```diff
mkdir -p $PKG/github.com/gopherjs/gopherjs
-cp $GOPATH/pkg/darwin_js/github.com/gopherjs/gopherjs/js.a $PKG/github.com/gopherjs/gopherjs/js.a
-cp $GOPATH/pkg/darwin_js/github.com/gopherjs/gopherjs/nosync.a $PKG/github.com/gopherjs/gopherjs/nosync.a
+cp $GOPATH/pkg/darwin_js_min/github.com/gopherjs/gopherjs/js.a $PKG/github.com/gopherjs/gopherjs/js.a
+cp $GOPATH/pkg/darwin_js_min/github.com/gopherjs/gopherjs/nosync.a $PKG/github.com/gopherjs/gopherjs/nosync.a
```


`else if`


that's true.  I'll go with that.


@vbrown608 This text should be conditional based on the API endpoint that says if membership is open or not. Checkout membership.html.haml and the use of `- unless @accepting_applications` there.


Just to throw this out there as well, the memory usage starts to pile up even when forms aren't opened.  After executing 100 clients on a VM, my memory usage spikes up from ~ 40k to 200-250k,  If I send a command to reconnect the clients, the memory usage goes up another 200k+


I opened [this issue](https://github.com/tknorris/script.module.urlresolver/issues/1) for those hosters. In the future it would be helpful to open an issue rather than commit on previous commits.


You can't really use the RED / RG formats though. Their usage here is just as a workaround since OpenGL 3 core doesn't support luminance / alpha textures.


The [unit tests](https://github.com/Alamofire/Alamofire/blob/master/Tests/ParameterEncodingTests.swift#L208-L217) and [Wikipedia](http://en.wikipedia.org/wiki/Percent-encoding#Percent-encoding_reserved_characters) both point to `%2B` as the percent-encoded octet representation of `+`. Is there something obvious that I'm missing?


The removal of x,y,z broke a lot of mods.  Just sayin.


Além disso, foram removidos os fflush e todas as leituras agora são com readline =)


Por que borrastes los usergroups?


ok, I see now that it is set in [runnableEntry()](https://github.com/matejk/poco/blob/d97dda2c30186686689a8b8d90fd9acb39704416/Foundation/src/Thread_POSIX.cpp#L451). I'm good with that. @obiltschnig


banco commitado


Is this necessary?


Meh, changed this ;)


What exactly is Pandora? Is it a Linux build?


Don't be so hopeful, hungry man...
try these ones:
(let (() ) (+ 2 2))
or
(let ((x 3) (x 4)) (+ x x))


Is there any way to add an option to remove the check to see if the version is the latest when installing the cia files?

I have a method of downgrading emunand that involves installing old cia files from the previous system updates. Currently I need to delete the cia files one by one and reinstall them in FBI and it just takes forever.

I would love to use your program but it skips files that are older by default. Is there a hot key to allow for this that you can implement?


Europe/Voronezh


Sorry took me so long to include the change. I was having trouble parsing and finding all specific libraries that needed to be added to the CLASSPATH. I still need to find a way to speed up the search that is taking a few seconds to populate the CLASSPATH every time a new file is open.

Your solution was adding all support, extra and android sdk jars (including all available versions) to the search path. That would really slow down things for syntastic and java-autocomplete2.


@adamwathan here's the commit with the changes in response to what we paired about


No tab please.


GetUnitByName = GameLib.GetPlayerUnitByName

There is a problem in variable name


why a new line here ?


Can you mark all of these as advanced as well. Also doesn't one of them need to be marked as ON ?


why a tabulation here?


tab too.


Shouldn't this go below the following OPTION commands ?


:+1: :+1: :+1:


This does not do anything. It can go away.


No I checked this. If I put it below, then its not setting the auto-detected value properly.


This should always be sm_20 for now. Why are we setting it to sm_32 for just one case ?


sm_32 is tegra. Thats going to remain constant right.


Aren't we encoding a space, rather than a `+`?


Where is COMPUTE_VERSIONS being used ?


Make sure this is prefixed with m as it is a member variable.


多少小时 改成 小时数 吧


Please see https://github.com/symfony/symfony/blob/2.6/UPGRADE-2.6.md#optionsresolver


吐槽你话 -》 吐槽你的话


Ah I was using that when trying to compile multiple PTX version. I'll remove it.


Ok that makes sense.


rax retry tests


Se encuentra disponible desde hace rato https://github.com/Wolftein/AoW/tree/master/X-Client/C-Main pero no esta completo, cuando tenga tiempo lo termino.


This should be replaced with an IntDef https://developer.android.com/reference/android/support/annotation/IntDef.html


`NSComparisonResult` 前空格


`.SearchBackwards` 前空格，后面的自己再看看吧~


Yep thought so


Remove default Javadoc


One of them will be marked on from CUDA_COMPUTE_CABILITY.
When setting manually (not running autodetect), the user can just set the option to ON.


Is it safe to ```erase``` from a std::set without first checking if the value (```*it```) exists?


also added date functionality with posting trips


Replace 'Fng' with 'Finger'


I think you probably want to keep this symlink here, since it's going to be on the /opt/app FS, which is the immutable one.  It'll be a dangling link until `launcher.sh` runs, but that's okay.


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


TeamCity xBIM :: Xbim Geometry :: Release [Build 84](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1779&buildTypeId=XBIM_XbimGeometry_Release) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 70 Build time: 03:32:33
### Failed tests
```
VSTest: GeometryTests.XbimModelExtensionsTests.CanGetRawGeometry: <no details avaliable>

```


Unless I'm misreading terribly (and I am tired! :smile:) this doesn't actually test the dispatching.

Shouldn't this read something more like this?:

```js
    function DummyList() {}
    var called = false;
    DummyList.prototype.forEach = function(fn) { called = true; };
    R.forEach(function() {}, new DummyList());
    assert.strictEqual(called, true);
```

That last line is because, as of #956, we stopped using plain `assert` statements, opting for the more verbose form.


Assume unintended paste


I thought we'd already done this?


created
SCCS-file: factory
SCCS-SID: 1.1


SCCS-file: factory
SCCS-SID: 1.1


Oh, sorry for that. I forgot to convert Tab to spaces. Now I figure out that VS can replace tab to 4 spaces when editing and I will avoid using >>.


Why were these commits made but never put into any release?


Hey, I'm happy to see that you found the code useful, and it looks like you made some great improvements, but you can't just drop everyone else's name from the license of an open source project and call it yours. I appreciate that you kept our names in the NOTICES file, but that's not really the right way to go about it. Why didn't you get in touch? it would have been cool to collaborate on some of this.


"comp.sources"


Nevermind seems that was on one of my own branches d'oh


Loving that you added 'forward' to it.

maybe even stock could be added for libraries.


saw approval from shino
at https://github.com/basho/riak_cs/commit/094b151119d095976734f102ec1995ad0a1812ee


merging basho/riak_cs/kuenishi-patch-1 = 094b1511 into borshop-integration-1156-kuenishi-patch-1


basho/riak_cs/kuenishi-patch-1 = 094b1511 merged ok, testing candidate = 5948798a


had to remove `dir` and then put in the `icon` to not display the default cat icon! 😖


LGTM


There was a discussion on this commit over at my fork, but I'm moving it back to this repo.

<pre>
@flight16 because it wasn't working the way I wanted it to. It would add this metadata to all entries that jrnl edited, rather than just the entries that lacked it. In a nutshell, jrnl was throwing away all existing metadata. I would like jrnl to keep existing metadata on entries. This is a feature I intend to come back to.
</pre>

I think this is what caused #358, but I haven't had the time to verify.


[this explains it](https://twitter.com/rvagg/status/604956690031755264):

> Maybe hold off on @official_iojs v2.2.0, a minor change broke npm, we'll get a v2.2.1 out within the next 24 hours to rectify it.


Better to split this into multiple lines in accordance with normal practice.


I'm just getting my self up to speed on this patch after having applied it to my latest pull request.

If this is supposed to act like the existing ```arc_do_user_prune()``` insofar as bumping refcounts is concerned, you'll need to restore the ```cp``` and ```np``` pointers (presumably current and next pointers) since this won't compile as-is.


Whups, thanks!


It was never pushed/commited to the main branch.


Add a comment here to explain why you're doing this.


I have tried many variations here (<%= form_for [@user, @item.create] do |f| %>, <%= form_for [@item.create] do |f| %>, etc) and I keep getting various errors. @bridgetlandis


I see some new modules going in as GPL3 and others as GPL2.  Might be good to have a discussion about this.


Okay, another question, I get this error
```
1) Project manager creates TODO Successfully
     Failure/Error: visit user_items_path
     ActionController::UrlGenerationError:
       No route matches {:action=>"create", :controller=>"items"} missing required keys: [:user_id]
```

It appears to be due to missing the associated user, how would I rectify this? Can I add a before action maybe?


Great question! Yes, it is (tested). :)


Test


Okay, this is obviously extremely unfinished, but I was wondering if this is an acceptable way to ask questions to you (since I am not pushing to master and just submitting to feature branch)? I wouldn't mind if you took a once over of the code I have so far and leave any comments you have, so I know if I am heading in the right or completely wrong direction.. :) Thank you! @bridgetlandis


self isn't necessary  here (Rubular would point this out to you).  In generate I use it when setting an instance variable = to something.  IE  `title.do_something` is google but `self.title = 'a new value'` require self so that you don't inadvertently create a local variable `title`


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1195


Missing break; CID 1296285 (fixed)


hello


default control loop, at least partially mapped out. i can add more detail as i remember it or someone else makes a correction
https://github.com/opennars/opennars/blob/volatile5/doc/opennars_default_loop.svg


justin how are you


Prerequisites shouldn't have a hyphen. :)


am i missing something or does this not work to begin with?


Where is this seen @mehulkar? Trying to see where this is to review styles


Does what not work? @rc420head


@sergeyshushlyapin, referencing `FieldIDs.FinalLayoutField` will only work in 8. In SCORE, in order to compile against and work with older versions, I had to get that GUID into a constant of my own. This way it runs just fine and if it's an older version of Sitecore the item just has no value in that field. Maybe FakeDb needs to do the same?


Internally we use "title" and "note".  I think we can pass $title, $note and $options in and that would cover all use cases.


My name is Shubham Tembhurkar,currently I am pursuing my engineering form mechatronics branch 6th sem  ,chattrapati shivaji institute of technology


`transport` is used below in
```c++
        if (player->TeleportTo(mapId, x, y, z, o))
        {
            if (transport)
                transport->AddPassenger(player);
        }
```

CID 1296285


attempt to convert all control activity to a unified frequency scale
https://docs.google.com/presentation/d/1_C95J2e5ATnswYubVfsBUcT1C0vOj5NNrtHRFxRjwnA/edit?usp=sharing


Working with static analysis software ansys
also working with dynamic analysis software adams.


Why did you remove the Setup section?

I was about to send a pull request with putting it back, as I didn't find in the readme and had to look for it in the repository and out on the internets.


That might be a tad bit confusing.  The plugin came from this commit https://github.com/boss-developers/oblivion/commit/f5a91d8cb7a6bfac6b99bf5e4dfc2245182f8cc0.  However, the LOOT masterlist is current with the commit in the description.


I did fix some of the automatic replacements, but it is now obvious that I overlooked many more.
Thanks a lot for the detailed review of the patch @masatake!


Also renamed HiddenShips :arrow_forward: OpponentBoard
and Main :arrow_forward: HiddenShips


TeamCity YaccConstructor :: YC.Core General Build [Build 414](http://teamcity.codebetter.com/viewLog.html?buildId=192490&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 114 Build time: 00:11:59


Since we are going to support both ifname and pid, I feel LIST_CLIENTS_ID makes more sense


Yes, they've been generated by ember-cli.


I'm just playing Lego with bricks you made.


Yes, they've been generated by ember-cli


gsgsgsdgdsgds


Transmuted into scope :smiling_imp: :octocat:

060e82ed0b222544bbf1395c28377f43fbc17d2e


Why not?
My pleasure.


Thanks Pierre. I guess It's a mean. According to Marc in the documentation, ion is the average of z the other is the average of depth of field. so yes it' can be computed from landmarks in world frame and local frame.


I don't think so,  but it might be worth checking.   Although,  you did write this code... ;-)


Yeah, I did :) Can't really remember much of it.


I think here should be `==` and `equals()`.
And sorry for the delay...


Oh, right. I somehow screwed up the search.
In any case, I will fix it together with another disadvantage:
>I am surprised that you didn't move the report geneartion to core/session too.


You don't use this scope in controller. So campaigns won't be properly sorted in view. I think.


drhtbv


This module needs `jQuery.ajaxTransport` to be defined and it's defined in `../ajax`. We have more dependencies like that; perhaps we could break some of it further into smaller components but that requires time & careful consideration of each case.


Looks good, misses a javadoc comment on setIsFrequencyColumn. Also I think we should consider making Column a final class.


^seconded, and tests missing.


I am new here


For navn som stammer fra latin burde jo us/a være en god prediktor, så siste to vokaler burde jo fange mye. En kunne kanskje også lagt på en regel for at postfix 'ine' indikerer et kvinnenavn. Jeg kommer ikke på like entydige postfix på nordiske/germanske navn. Språkteigen og språkrådet kan fort finne på å vite noe om vanlige trekk på jente og guttenavn.


@alexanderdean do we need to add license to the beginning of each file?


Complete 1 Task(s):
* Task [#ZG9d](http://localhost:8080/w/ZG9dap2PHReyQSfHoDvaiBmv)


sfaasfasfasfasfasfweqrqrqwrwqe


Good to hear that you did not forget about gamin.


Yes please - you can grab from any other Snowplow project.


Number => Intege


the implementation doesn't match https://git.kernel.org/cgit/bluetooth/bluez.git/tree/android/hal-msg.h#n1154 for here and below.


What about adding the commands, it allows for copy/pasting and is more precise. e.g.

cd /opt && git clone --recursive https://github.com/rcbops/rpc-extras


And i think that og vocab end point need vsite id in the query string.


I tried to trace it back using `git blame`, but it seems it has always been `<name>`.


That's included in node.js standad lib
https://nodejs.org/api/zlib.html#zlib_class_zlib_deflate
I'm not sure about dependency in package.json

btw as I see node.js 0.10 and 0.12 has slightly different zlib api


I think you should not commit local file path(s)


We still didn't marked 4.1 release nor pushed to greasy, so I saw no reason to jump versions.


I checked before commit on local server with ascending and descending
order. Worked fine.

2015-06-01 12:08 GMT+02:00 Jakub Kubacki <notifications@github.com>:

> You don't use this scope in controller. So campaigns won't be properly
> sorted in view. I think.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fantasygame/campaigns/commit/b6f8e2993bd5cd702060b24611418aadfab3d797#commitcomment-11450954>
> .
>

--
Pozdrawiam,

Małgorzata Grobelska


As far as I've read about this, the `_buffer` will be allocated in the Large Object Heap (LOH) because it's bigger than 85KB. If there's no more space for new allocations the GC frees unused objects in the LOH. Maybe we don't even have to worry about it, but it seems scary when no client is connected and the server eats 200MB RAM. :D


No it likely doesn't. Thank's for catching.


That is just fantastic to see this bug FINALLY quashed. Been waiting for a decade to see it get fixed in pcsx2. Now to test out the emulator and see for myself :).


OK, I think we're going to roll it back...


Please run `$ php app/console fos:elastica:populate` command


Ah, yes please add this to the release notes.


when the ajax module would actually be referenced in the below `function( jQuery, support, ajax )` and then used as `ajax.ajaxTransport` this dependency would be more obvious. Also it wouldn't rely on the inner workings of ajax to attach things to the global `jQuery` class.

But I guess you do this kind of "trick" to save some bytes, right?


Oh, you are using scope in view. Ok, that's fine. :+1:


No problem!
I will fix it with rebase


TeamCity YaccConstructor :: YC.Core General Build [Build 415](http://teamcity.codebetter.com/viewLog.html?buildId=192492&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


I think this should be the only AlterField in this commit.


Ah, right. Not sure why it was done this way but we rely on too many "globals" right now; some are currently being hidden in tickets like #2058 or #2224. `jQuery.ajax` will obviously stay a "global" but your idea seems like sth good to do.


I personally would not go the const int route in this case as I think the tradeoff (readability, maintainability) is not reasonable given the architecture layer (business domain) we are in.
But if you insist on it I will change it. It's your project after all :)


is there not a stable version you can depend on?


ok, will provide a PR then later this week.


Move permit() in one common method.


Since you now have a requirements file, you could read it from setup.py to build the requirements list.


Labai gerai!


I'd prefer if the check was inside the `transact_SetSignerList`.


You're correct, sorry. Slow day :).

Okay, will investigate the differences of the API and check to see what breaking differences there are.

Thanks again.

Jonathan.

> On Jun 1, 2015, at 13:12, Oleg German <notifications@github.com> wrote:
>
> That's included in node.js standad lib
> https://nodejs.org/api/zlib.html#zlib_class_zlib_deflate
> I'm not sure about dependency in package.json
>
> btw as I see node.js 0.10 and 0.12 has slightly different zlib api
>
> —
> Reply to this email directly or view it on GitHub.
>


Should use [`link_to`](http://api.rubyonrails.org/v3.2.21/classes/ActionView/Helpers/UrlHelper.html#method-i-link_to).

```erb
<%= link_to _("Browse all requests →"), request_list_all_path, :class => 'link_button_green' %>
```


额。。。原来改了这里，怪不得不能运行。。。


Could you provide screenshots/videos of Pandora running GLideN64? Just curious.


Message should be ```User updated successfully```. Move all messages into en.yml


TeamCity YaccConstructor :: YC.Core General Build [Build 416](http://teamcity.codebetter.com/viewLog.html?buildId=192493&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


Move all messages into en.yml


The problem is that the stable is quite old.

https://github.com/magnars/fold-this.el/compare/0.2.0...master

I can ask @magnars to make release. I doubt that he will have something against it.


Move messages into en.yml


OK. I can't see how the past names map might be used to compromise security at the moment, but I agree it's better just to use importNode and not worry about it. Might be nice to have a comment in the source code explaining why it was chosen; helps people coming in later (and indeed ourselves in a few months…).

OK for the options too; I take your point on consistency.


Move static labels into en.yml


@mcshaz Thanks for contributing to Knockout-Validation wiki with this custom validation rule. Regarding the `Array.isArray` method call, the rule can use the helper method `ko.validation.utils.isArray` so that any polyfill can be avoided.


@vcunat rtmpdump seems to depend on nettle http://hydra.nixos.org/build/22693861/nixlog/1


let's make dashes a standard? "-"
:)


TeamCity YaccConstructor :: YC.Core General Build [Build 416](http://teamcity.codebetter.com/viewLog.html?buildId=192493&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: Error during Nuget update. ..\tools\Build.Tools\NuGet\NuGet.exe install "Z:\Builds\work\4138ce7c6332a78c\src\FsYARD\packages.config" -OutputDirectory "..\src\packages" -ConfigFile "..\src\NuGet.config"   at FSI_0002.Packaging.restor... Build time: 00:02:28


Should these look more like links? On first look I wasn't sure that they were. I _assumed_ they were, but I wasn't really sure until I hovered over them.


@hodgestar ready for review


Move static labels into en.yml. common labels should be in common hierarchy
```
common:
    captions:
        first_name: 'First Name'
```


maybe you mean gcc-4.9 there? :)


Move all static labels, place-holders, button, disable_with into en.yml


谢谢大神！没问题了。


Hi,

When do you think will a version containing this fix be released? I'm depending on this fix.

regards,
totycro


Yes, good catch, thanks. I fixed that.


:-( schade, ist ja aber eventuell eh überflüssig


We usually write docstrings and commit message with imperative tone, hence Returns -> Return. I've applied locally, you don't need to resend.


TeamCity YaccConstructor :: YC.Core General Build [Build 415](http://teamcity.codebetter.com/viewLog.html?buildId=192492&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 114 Build time: 00:11:50


@creepylava, I see what you're doing there and I don't blame you for it I just don't agree with it. In the context of the dungeon generator source itself, that license demand is pretty heavy considering that the source it is used on is of a plugin type nature.


TeamCity OsmSharp :: GitHub - OsmSharp [Build 1030](http://build.osmsharp.com:8080/viewLog.html?buildId=4918&buildTypeId=bt16) is now running


No generally sheet_margins can be smaller than a range selection.


Please define uint128_t properly in stdint.h.


I see :smile: I didn't look at the divMod signature.


Might be slightly more efficient to have `pluck("DISTINCT public_body_id")` here.


Thanks


actually nevermind, it was a fault on my side!
my bad....


Can you please tell me when is this going to be included in stable version of spree?


This doesn't seem to link anywhere so we should only add it when it goes somewhere


That's true. Just mention in the usage that you can get the id from `--list`.


this is not a simplification. please schedule it as a tuning patch, if you think it can be a small elo gain.


@mswiderski This looks wrong to me. We are using the params before checking if they are null. If Params is null, the resolveTaskDetails() method will blow up! :)


Removing this code makes setup on node a little less obvious – I used to be able to just `require('moment-range')` and get the `moment` object back, but now I have to `require(moment)` first, then `require('moment-range')`. Not the best idea to silently break the interface in a minor version change...


>Are you sure it doesn't need QT_BEGIN_NAMESPACE/QT_END_NAMESPACE too?

I think we can never use it.
This is needed when Qt build uses some namespace (official Qt not use it). But even in such cases it is not enough to compile qBittorrent with "namespaced" Qt.


>break

Fixed.


Ok. Did a quick test of Baldurs Gate 1 (PAL). Seeing this in full screen with no half screen stuff is just AWESOME.

The game runs so slow in some areas that I'd prefer to wait till things speed up somewhat before I try a proper full play through in the emulator. However speeds were far better in the combat areas.

Speedhacks seem to have zero effect on this game. Noticed no difference with or without them on. (Using v1.3.1-425-g2cbde89).

In town - 3-15 FPS (30-40 when talking to people).
1st Dungeon - 25-35 fps


Hi gary,
        I am using rabbitmq in my application. But its increasing the java heap size.


主界面作为启动界面


Could you override the default layout and render the hero partial before we open the wrapper div?


TeamCity OsmSharp :: GitHub - OsmSharp [Build 4.4.0.1030](http://build.osmsharp.com:8080/viewLog.html?buildId=4918&buildTypeId=bt16) outcome was **SUCCESS**
Summary: Tests passed: 541 Build time: 0:0:0


登录状态的全局变量
使用方法：在activity中MyApp app = (Context) getApplicationContext();
app.isLogin();判断


we can add null check here, but do we have situation when we create task without parameters?


登录成功后修改状态的全局变量


Maybe a bit of transparency on this?


Done! `0.3.0` released.


Please add license header.


It seems that implementing this without reviewing commands leads to double escape issues ..

We definitively need an automated test for mage :)


This makes me want to click it, but its not a link.


The null check is already one line below.. and yes.. I had one situation for ad-hoc tasks, that's why I found the issue.


Popraw tego dowódce oddziału


While you're at it, please add a license header.


intendation is wrong


c'est pratique de pouvoir créé un commentaire


@pSub gnutls no longer propagates nettle is the cause


Why not just call `klass.validation_attributes_for(field)`?


I'll take some time and review pending pull requests then I think we'll be ready to go if there's no objections.


I mean in the view itself - unless I've misunderstood it doesn't look like we're saving any code here (me: goes to look at subsequent commits to see if I've missed something).


Are these `;` necessary?


We have to create new method commonDao.updateSafe(...), becouse commonDao.update() is already used in ResetPasswordResource.


This is what I mean: `SearchForm. validation_attributes_for(:postcode)`?


Beter solution

```javascript
$js = UTIL_JsGenerator::composeJsString('$("#" + {$uniqId}).suitUp({$buttons}).show();', array(
    "buttons" => $this->buttons,
    "uniqId" => $this->getId()
));
OW::getDocument()->addOnloadScript($js);
```


Int


__API Blueprint syntax error on line 358__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


@Underground78 @XhmikosR Plans to push it? I think we should redirect people to this page asap for the stable.

OT: It is a pity that we need to mark all crashes again in drdump for new version.


The build worked at this revision we comment.


Awesome


Yes, it'll be.


Shouldn't it be `#ifdef ARM64`?


I don't think we want to have --no-gpg-checks unconditionally..


啊啊啊啊啊啊，我全部数据都不能用了，哭哭哭


Ah sorry just noticed this is what you wanted linked. Try this:

```erb
<p class="intro__browse"><%= _(%Q(Browse <a href="#{ request_list_successful_path }">{{number_of_requests}} requests</a> to <a href="#{ list_public_bodies_default_path }">{{number_of_authorities}} authorities</a>),
    :number_of_requests => number_with_delimiter(InfoRequest.visible.count),
    :number_of_authorities => number_with_delimiter(PublicBody.visible.count)) %></p>
```


You're actually quite correct which is pretty confusing for me.  My local source was still using `$timeout`; so when I "fixed" it then built, I was really fixing my local problem.  Sorry for the mistake.  Will submit new PR to correct.


Is there another name we could use for this method? It looks like it's doing the same thing as the Rails `validate` declaration. But I assume this is specifically for setting client-side validation rules?


Not sure where you want this one to go?

```erb
<p class="intro__description">
  <%= _(%Q(Make a request for information to a UK public authority: by law, <a href="LINK_GOES_HERE">they have to respond</a>)) %>
</p>
```


I see no harm in adding these in arm64.h instead.


do you need to remove them from the model fields too? or how are you thinking that will work?


It will be placeDao.updateSafe(...)


@joeyespo

Nope that demo doesn't quite work, fails. If you meant for `[<directories>]` to be an optional positional argument, turns out it isn't, the easiest example being:
```sh
ptw -- foo`
```

Also, if it's a positional argument that represents a path/paths, it's a common unix philosophy that it should take those paths as space separated positional arguments (py.test does it, for one). One of the reasons for this is the way shell expansion works -- you would expect this to work:
```sh
ptw folder_*
```
and the shell would expand it to something like
```sh
ptw folder_1 folder_2 folder_3
```

So we really want something like this:
```
ptw [options] [<directories>...] [(-- <arg>...)]
```
but docopt doesn't seem to be able to handle it?


There's no goal to get rid of BOOST_FOREACH in GUI code. Please keep ones you can't replace with `foreach` as-is so they can be replaced with c++11 range-based for loops when that is enabled.


@moatazelmohtaseb Please update the comments to reflect this command.


the base product update channels are already in the image iirc.


Je pense là


Great! Sorry I didn't get the chance to work on it - it seems you've already fixed the issue - https://github.com/npct/pct-shiny/commit/7195c3457a28edbd6ab81bc24dd2c75733697639

Good work!


nice `Boolean` bro


There is reason for this. I've made docker image for cowrie:
https://registry.hub.docker.com/u/k0st/cowrie/

In order to make best possible image, it would be good to have some way to separate code and data. Currently, it is not possible without patching.

It would be good to have following structure:
data/conf
data/honeyfs
data/dl

It would make backup and any sandboxing easier.

Another thing which would help is addition of --config parameter in start, i.e.:
./start.sh --config /path/to/config

Hope it helps!


Agreed and reverted this one.


not sure if thats worth it .. maybe we should just include the missing package in our C:O:* projects


ici, done(null, toke) seulement si token est bon


the copy paste is real


How about

    def is_incoming_message?
        incoming_message_id

for minimum object loading?


Could do the same change for `is_outgoing_message?` and `is_comment?`


@jfirebaugh I need to do this to get the signs rendered on the first load, see http://mapillary.github.io/iD/#background=Bing&map=17.00/13.00070/55.60543 - is there any proper way to redraw the map after the objects have been added to make the signs show up at the right offsets?


Ya, it was obvious. The project responsible could consider integration of a linter in relation to git and building/packaging.


Aha! Ok, so in the future I will be able to keep the exec bits. :-)


You didn't include your PC specs so that isn't saying much...


Good catch, libvirt should be fixed too. It's probably better to do a separate PR for that.


Is the _count_ being used anywhere?


It would be nice to have these selectors defined in uppercase (constant-like) variables.


This line instantiates *all* classes.


Thanks!


My miss, sorry!


With this change, connections to web_admin when tls is enabled fail with:
```
12:51:44.517 [info] (#Port<0.7226>) Accepted connection 127.0.0.1:53677 -> 127.0.0.1:5280
12:51:44.518 [info] started: {p1_tls,{tlssock,#Port<0.7226>,<<>>}}
12:51:44.518 [error] CRASH REPORT Process <0.651.0>
 with 0 neighbours crashed with reason:
 no case clause matching {send,<<>>} in p1_tls:recv/3 line 185
12:51:44.519 [error] Supervisor ejabberd_http_sup had child undefined started with
 {ejabberd_http,start_link,undefined} at <0.651.0> exit
 with reason {case_clause,{send,<<>>}}
 in context child_terminated
```

Configuration:
```
listen:
  -
    port: 5280
    module: ejabberd_http
    certfile: "/etc/ejabberd/ssl.pem"
    tls: true
    web_admin: true
```


I use a test_parser and pass it to the new_parse_arguments function. however, i still need to pass the args. I'm still a bit confused about the API: we want to get only the sys.argv from the main script, and then making everything through the functions, right?


I was thinking they could stay in the models since it correctly describes what the final state will be. But perhaps a comment would be useful.


I'm doing that now actually! It was left out as it requires a little more work for some libraries with strange coding styles (*stares at Slice*).


The tests use it. It seemed a natural thing to return when converting from an API that deleted single values (and returned an error if they were not found) to one that supported wildcards...


Thanks.


I still don't understand. How can it instantiate more than one class?


:smile:


now


This is already protected against in the _isClobbered fn (and that's presuming SANITIZE_DOM is set to false by the user; otherwise all clobbering ids/names are removed). So there's no way you could clobber it unless inserting HTML that hasn't been sanitised, in which case you've lost anyway.

Having said that, it would be more consistent to not check for these in _isClobbered, and instead to do something like you've done here. I think it would be simpler and faster though to simply cache references to the methods themselves, e.g.:

    // Cache on load
    createNodeIterator = document.createNodeIterator;
    ...
    // Then during the sanitize call:
    createNodeIterator.call( doc, doc, etc.


Why not use ```unique``` instead of ```db_index``` ?
This would also make the field in the database unique instead of only putting an index on it.


> not sure if thats worth it .. maybe we should just include the missing package in our C:O:* projects

I guess that's about the Virtualization repository? I'll remove that commit...


This doesn't belong in filesys; that's supposed to be for file systems. We've talked a few times about a generic "services" plugin that can obtain similar information from systemd and SysV systems (and ideally is a broad enough abstraction to permit Windows services and OSX launchd services if we bring those platforms back into the maintenance fold).

I can't currently think of a better place to put this.


Ack


Can you document the traits, Mahmoud made a legitimate claim there


Ack - might consider adding a Copyright line if you wish.


Right. The status value is in the middle of the response structure. So I removed these helper functions and decode the message directly.


Use `add_cmd_outputs()`


maybe we should add a space after [Rails 4.2 release post]


add a space after [Rails log]


Ok I was not on this function at all.
I understand what you mean, the research function in _getItemById() is not very efficient.
But I can not find where the function renderItemToItem is called ? My fought is that this function is only used in the unit tests to verify if we can pass from a renderItem to an item correctly. In the overall process this function is not called. For this reason I do not think that renderItemToItem is called once per item.
What do tou think about it ?


creation of getter object is costly, cache them during startup.


Doesn't belong in block; the udev db is going to contain a tonne of other device data that has nothing to do with block devices.


Anyway, #8112.


Fixed this to use 'm' instead. The memory should already be allocated as part of the structure.


Also I've marked 1.7.8 as "Support has ended" and set default response to https://mpc-hc.org/ so the users will see message that they should update to new version and have link to our webpage.


Why switch to 'standard' `ps` syntax rather than BSD (that was 'standardised' by sysreport)?


This should still be installed when is_amazon is false


Added admin user and uncommented intl extension in php.ini


Pass lists to `add_cmd_outputs()` instead of making multiple calls.


`ansible_distribution` is equal to "Amazon" only when it's a CentOS on amazon? What about other distributions?

To me, this variable name is misleading: I would assume "is_amazon" means is a VM running on amazon, not a *CentOS* running on Amazon.


I think we need more discussion about why this belongs in the systemd plugin rather than cgroups (and whether the current organisation is "right").


Note, the game uses an internal upscaling so 2x upscale is equivalent to 4x.

Could you check memory consumption too?


Great forking there Fin...


Sure.
Not a video, because current framerate is between 1 to 0.2 fps (I have not found yet were the GPU is spending all his time).
But here are a few pic from Mupen64 running on the Pandora (using 800x480 display, and the "3" Adjust mode to fill the screen).
![gliden64_1](https://cloud.githubusercontent.com/assets/3250792/7911660/926ac7d2-0860-11e5-8799-197fe5f196e0.png)
![gliden64_2](https://cloud.githubusercontent.com/assets/3250792/7911666/9bbbc4da-0860-11e5-9870-a7289fcbb5ff.png)
![gliden64_3](https://cloud.githubusercontent.com/assets/3250792/7911670/a8f4f3a6-0860-11e5-94f0-01d462185d1d.png)


Use angular.isNaN


Can you also add it to Decodable?  This Encodable/Decodable implementation should be moved, but for now please just also cater for it in decodable. thx


I upgraded gcc and r. However, when I tried to install the **tm** package I get the following error:

```
Error in dyn.load(file, DLLpath = DLLpath, ...) :
  unable to load shared object '/Users/me/Library/R/3.2/library/slam/libs/slam.so':
  dlopen(/Users/me/Library/R/3.2/library/slam/libs/slam.so, 6): Library not loaded: /usr/local/lib/gcc/4.9/libgfortran.3.dylib
  Referenced from: /Users/me/Library/R/3.2/library/slam/libs/slam.so
  Reason: image not found
ERROR: lazy loading failed for package ‘tm’
* removing ‘/Users/me/Library/R/3.2/library/tm’
```

Any thoughts would be much appreciated.


comments?


Thanks for the forking encouragement Steve


This is a bit ugly and requires a lot of work. I've been thinking about how to fix it and while it's not easy I have an idea in mind. I will send you back the pull with an example for your review.


Commit message needs fixing to conform to the [Contributor Guidelines](https://github.com/sosreport/sos/wiki/Contribution-Guidelines). Specifically the subject should be something like:

```
[last] new plugin to capture login actions
```

Sorry for not spotting this first time round.

Also please use a list for reboot/shutdown/lastlog - these don't need a `root_symlink` so they can use a single add_cmd_outputs() call.


Which particular version of the compiler did you encounter this with?


@taylorotwell is there a specific reason why Cache Tags section was removed from 5.1 docs? It is a powerful feature and I think it would be nice to be documented.

Thanks


There is no 'transport' parameter in the earlier HAL protocol. I found it easier to read if there's one complete branch per variant.


What does this function do ?


@psampaz I assume it'll be coming back soon. The docs are still being rewritten atm.


Fixed.


I'm not a big fan of this if/else control flow, since the different branches don't do parallel things.  I'd suggest instead something like

````
if options[:force]
  drop_view ...
end

definition = definition.to_sql ...

command = "CREATE" ...
command += " OR REPLACE" if ,,,

execute "#{command} VIEW ..."
````


That is an interesting point. I assumed (and I cannot imagine where this would not be true on x86) that `sizeof(char) == 1` when I wrote that. I just checked and I don't think this is actually required by the language standard, so you are correct.

I am not sure how that managed to compile without including `cstring`. I will put out a pull request for both of these briefly.


Thanks for info! :)


¡Hola Mike! @msanroman Muchas gracias por responder, si al final lo e hecho con anclas ya que no había forma de que funcione. No se bien que es lo que sucedía!


Sorry, was used to linux kernel style naming. Will fix.
I will actually submit a new pull request of a new branch.

Thanks for feedback!


Build still not finished. I guess it might be stuck at some interactive user prompt..? Anyway, curious that they not terminate builds after some time..


Could put a `new_record?` clause before the association check so that we don't call it needlessly in cases where it's not going to exist because the `InfoRequestEvent` has a different type of association.


review_num == @ordinary_records.size, right?


Hi @ronen, thanks for that. I was actually thinking they are doing parallel things: if `options[:force]` is true, the `:allow_replace` option is redundant because there definitely won't be a view present to be replaced. That was the reason for going with the if/else.

If you still disagree, let me know and I'll change the structure to work as you suggested.


Fixed. The Bluedroid function expects a signed type. I hope this difference doesn't cause any trouble in the long run.


Merged


is not correct OpenMVG?


No worries.  Sorry about that.

On Mon, 1 Jun 2015 at 19:14 Sab <notifications@github.com> wrote:

> thanks!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/qgis/QGIS/commit/4b6ad0c237f82d0ca08784a8272980e14112c480#commitcomment-11450120>
> .
>


I have never used scales.
In my world scales don't exist.
Surely you are more competent than me.

Can you tell me, please, how can I reproduce the issue?
Thanks in advance.


Error messages should be proper.
![selection_060](https://cloud.githubusercontent.com/assets/5477412/7911867/e40be854-087f-11e5-9688-3cb4184149a5.png)


@magnars thanks! I was going to open issue for that, but I see that mentioning you is enough.


I approve!!!


Fixed.


Not sure if we really need this empty "if" branch, but then, this is a question of style


Otherwise LGTM


That comment was a bit of a flash-forward to a later PR in which setup dependencies _are_ added, I missed that when splitting it up into PRs. Sorry for the confusion.


Cool tier


tuning bounds are `(0,4)`, not `(-1.5,4.5)`:
https://github.com/glinscott/fishtest/wiki/Creating-my-first-test#parameter-tweaks


Hi,

* pls. use a neutral default error page.
* are u shure that /var/www/static exists? If not, we should create it upfront.

Best Regards,
Michael


perhaps state to TXIDLE? then the next call to recvDone will do the work?


Hm. Give Ontology it's own class and resource? (SRP)


No need to write routes for change password. Use inbuilt Device routes.


Browserify has a [standalone option](https://github.com/substack/node-browserify#usage) which generates a UMD bundle.

```
  --standalone -s  Generate a UMD bundle for the supplied export name.
                   This bundle works with other module systems and sets the name
                   given as a window global if no module system is found.
```

With that we could remove the checks for CommonJS or AMD and just do `module.exports = interact;` and then let browserify do the UMD for us in the bundling process.


I think the import for `provider` is missing.  (Shouldn't this be caught by a test?)

Also note that some people consider it bad style to have the same name (`provider`) refer to two different things within a single expression.  And you shouldn't do that for list comprehensions, since they leak the loop variable into the enclosing scope in Python 2 (but dictionary comprehensions don't.)


There is a spelling mistake, please re-do and commit.


Same as before. There's no 'transport' in the older versions. IIRC there are are some functions where we share code for decoding common fields. But it's quite unreadable.


I believe the 's should be `s


Please do proper refactoring....:)


@anurag09 Thanks :)


Originally, I did, but KISS. On the other hand, if you fancy taking a look at App.class so that we can add resources easily


Fixed.


Sorry forgot to add my PC specs to my prior post. Here ya go:

PC Specs:

i7 2.66 GHZ 3GB Ram. 32 bit.
Radeon HD 7900 3GB.


You can put Code review Comments here :)


I also did them with an Alliance NPC before noticing this, if this is what you are meaning.
But I think this is fair: Alliance was in Azeroth way before Horde pops in, so everything Horde gets (including quests) should also be available to Alliance first.


@ipa-fxm: this fix might be needed in the recent version as well:
```
/u/robot/git/care-o-bot/devel/lib/libaugmented_solver.so: undefined reference to `Eigen::MatrixBase<Eigen::Matrix<double, 3, 1, 0, 3, 1> >::cross_product_return_type<Eigen::Matrix<double, 3, 1, 0, 3, 1> >::type Eigen::MatrixBase<Eigen::Matrix<double, 3, 1, 0, 3, 1> >::cross<Eigen::Matrix<double, 3, 1, 0, 3, 1> >(Eigen::MatrixBase<Eigen::Matrix<double, 3, 1, 0, 3, 1> > const&) const'
collect2: error: ld returned 1 exit status
make[2]: *** [/u/robot/git/care-o-bot/devel/lib/cob_twist_controller/cob_twist_controller_node] Error 1
make[1]: *** [cob_control/cob_twist_controller/CMakeFiles/cob_twist_controller_node.dir/all] Error 2
make: *** [all] Error 2
```


One need to acknowledge the above comment before a significant amount of time


Method documentation would be nice :)


I used 'transport' because I thought this is the same as 'transport' in the other functions?


Hello ,

Iam trying to rebuild the droid-vnc-server native method libraries. i just followed the link https://github.com/oNaiPs/droidVncServer. for aosp download i followed this link http://source.android.com/source/building.html.

but iam not succeded.

I got this error message when try to build the droid-vnc-server( contained in nativeMethods) woth AOSP master branch:

PLATFORM_VERSION_CODENAME=REL
PLATFORM_VERSION=5.1.51
TARGET_PRODUCT=aosp_arm
TARGET_BUILD_VARIANT=eng
TARGET_BUILD_TYPE=release
TARGET_BUILD_APPS=
TARGET_ARCH=arm
TARGET_ARCH_VARIANT=armv7-a
TARGET_CPU_VARIANT=generic
TARGET_2ND_ARCH=
TARGET_2ND_ARCH_VARIANT=
TARGET_2ND_CPU_VARIANT=
HOST_ARCH=x86_64
HOST_OS=darwin
HOST_OS_EXTRA=Darwin-13.4.0-x86_64-i386-64bit
HOST_BUILD_TYPE=release
BUILD_ID=AOSP

OUT_DIR=out

make: *** No rule to make targetout/target/product/generic/obj/STATIC_LIBRARIES/libunwind_llvm_intermediates/libunwind_llvm.a', needed byout/target/product/generic/obj/SHARED_LIBRARIES/libdvnc_flinger_sdk22_intermediates/LINKED/libdvnc_flinger_sdk22.so'. Stop.

is this really a developer can built droid-vnc-server native libraries( flinger and gralloc wrapper libs) or which is author specific ?and one more doubt for building vnc wrapper libs ( flinger and gralloc) AOSP master branch needs atleast one total building or module specfic build is fine...?

Any helping hands would be great encourage to me.

Many Thanks for your posts

Best
Manikumar


Yes you are right cstring of course.
I wasn't sure about the char length required by the language standard, you are right, but then it is maybe a bit clearer to write strlen(string) + 1.


Yes, but I think a new/second model was added in TBC, that could explain why wowhead tells it was added in 2.0.1


Yeah, I get that.    But from a control flow perspective I found it a bit jarring.. the code is in the process of building up the sql command, and then one branch of the if modifies that command ("OK") but another branch executes some other command ("huh? I hadn't finished building the first command!").  I had to read a few times to be sure I understood what was happening, which one overrides which, etc.

Since they do completely two different things to the underlying sql, in some sense it's reasonable to leave it to the user: specify `:force` and we'll first delete the view;  specify `:allow_replace` and we'll use "OR REPLACE" modifier;  specify both and we'll do both!  Why would somebody want both?  I dunno, but that's what they asked for.  And it's harmless anyway so we needn't go out of our way to protect them.

Or we could go with something like `if_exists: :delete` or `if_exists: :replace`.  That'd arguably be the cleanest interface;  and `force: true` could become a deprecated alias for `if_exists: :delete`


Can any of this comment be converted into an assertion?


I was meaning that you have written ope(m)MVS instead of OpenMVS.


Uhh,  it is not the apache software license. Read the name of the license.


Hm, we should see, that we follow a strict schema for assignments to global values, for example:
- check resource files
- check environment
- check commandline
May be all in one place/function?
And at all, i would prefer an object/dict storing "globals", because globals() gives us all globals (including functions and so on) in an module, not only our specific ones.


Your code go in the good way,
I will base on that + at first glance :
- seems this.seqNo must be put to -1 before sending the error, or any subsequent query will be on error.
- i think the size comparaison must check the header size, will check that
- prepareStatement parameters size will not been checked at all.
I will take some time to make that right.

But " batch rewrites for prepared statements, it does not work as it is written" :  the problems have been corrected in the master version. have you seen anything in this version ?


I feel honoured! Thank you...


Oh, sorry about that.


Should we add another test case with loading sub resources or sub frames? Does the sub frames will also fire ```onDocumentLoadedInFrame```? Thanks.


// i was here


the model of I7 ?  anyway, if it was a older gen I7 it probably has a low single threaded performance and might perform somewhat slow. also, I'm awaiting to hear about the impact it has on memory consumption.


This placeholder is still not working; any idea why?


Projecting and filtering on embedded id properties...


@spMohanty Done! In addition to namespacing the user ID, I've prefixed it with a `#` (an invalid username character) to ensure that users can't create the same user ID. I've also deleted the `anonymous` user since it no longer serves any purpose.
Check out commit https://github.com/geotagx/geotagx-theme/commit/4c112cf41c273066528886ce4e67604b628fe288.


Not really I've just removed them.


I'd reference the follow-up OGM issue (if there is one).


Wrong indentation!


Fixed and pushed


not going to take the change


ack


Fixed in https://github.com/fallen/artiq/commit/8afd7382f9b62e434572f5e67eae10818bd209eb


Heh, I knew that when I wrote it, I'll change it to paging.conf.


Does atexit work in unittests? (note the s)


add another constructor, to take curator framework, and funnel old one through this.


description


This doesn't do any harm, but why is it necessary?


Same question here.


foo bar


Maybe 4CPUs is a bit too much :-)


Create new user functionality is not working. Please fix this
![selection_061](https://cloud.githubusercontent.com/assets/5477412/7911962/d66b42d4-0880-11e5-8bef-51be3d843cae.png)


There used to be systems where `sizeof(char)!=1`, but it has surely been several decades since the last one was introduced :-)

Anyway, a follow-up patch would of course be welcome!


Couldn't this `if` block and the final `saved` line be replaced with a single line:

    !! node.save

?


Thank you very much for these changes. All events i can handling now :).


There are a bunch of issues on with 2D rendering. And all of them looks
hardware specific (I see different set of issues on different laptops). May
be to revert that changes?

I just checked how spots sizes correspond to mouse click events. They are
not precise but looks of right size (so clicking inside of a spot lables
it, outside of the spot dislables). I don't thing there is a bug in how I
calculate distance (it's just dx^2 + dy^2 < r^2). We also should check that
previos SVG implementation was correct.

I'm unlikely will have a chance to fix it soon. Next weekend I'm likely
traveling to the US.

On Mon, Jun 1, 2015 at 12:24 AM, Theodore Alexandrov <
notifications@github.com> wrote:

> thanks!
>
> the spots look smaller than with the SVG vis. I guess they are 2 times
> smaller because for this example above earlier the whole space was filled
> when using spot border = 1.
>
> can you please increase visualized spot size x2?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ili-toolbox/ili/commit/2b2830f544562dc86784ce6a7576a9338f150994#commitcomment-11446119>
> .
>


I'm curious that you don't ensure code builds before committing it, but whatever.


You are right


So what are you doing exactly, adding a marker? Moving them in a different location? That information would make things easier to understand for the reader.


**User singin --> Change Password**
If I press ```save`` button with blank form, it should be redirect to Dashboard without changing password. Please fix this.

![selection_064](https://cloud.githubusercontent.com/assets/5477412/7912375/748c5a22-0884-11e5-9f63-83d3f8ca22dc.png)


Yes. Well, strictly speaking it should be `#if defined(ARM64) && defined(CFG_WITH_ARM_TRUSTED_FW)`. Currently ARM TF is always set when we build for AArch64, so we could just keep `#ifdef ARM64`, but since the "hack" is closely related to the behavior of ARM TF, I propose to keep both conditions.


I initially created `$.ajax.prefilter` and `$.ajax.transport` but it made a lot of plugins fail because they relied on duck-punching `$.ajax` (effectively nullifying those two). The "fix" was to use `$.ajaxPrefilter` and `$.ajaxTransport`. So it's not about byte count, it's about compatibility with duck-punching plugins.


Constants should not have trait setters.

```
scala> :paste -raw
// Entering paste mode (ctrl-D to finish)

trait U { val x = 42 }
trait T { final val x = 42 }

// Exiting paste mode, now interpreting.
scala> Class.forName("U").getDeclaredMethods.mkString("\n")
res5: String =
public abstract int U.x()
public abstract void U.U$_setter_$x_$eq(int)

scala> Class.forName("T").getDeclaredMethods.mkString("\n")
res6: String = public abstract int T.x()

scala> typeOf[T].decl(TermName("x")).isGetter
res7: Boolean = true
```


1, 2 ? a comment would help to understand what this is.


hello word !!


What's the rationale behind this?
Originally it was seen as a benefit to minimize DOM element creation/deletion.


Not related to this change, but by reading this I don't understand what's being prepared :)


Don't swallow the exception, prefer :
LOG.error("Error while migrating chat data : " + e.getMessage(), e);


I think i cannot understand this function.

Can you document what exactly is each parameter?


Right, better use tearDown() indeed. Fixing that. Thanks!


Will this work on JBoss ?


Well done!


Why static ?


its not its an attack named melee with 2 second interval...


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


==3322== Memcheck, a memory error detector
==3322== Copyright (C) 2002-2013, and GNU GPL'd, by Julian Seward et al.
==3322== Using Valgrind-3.10.1 and LibVEX; rerun with -h for copyright info
==3322== Command: ./main Documento Configuraciones Instrucciones
==3322==
Palabra: Augusto
Pagina: 2
Linea: 15
Posicion: 12

Palabra: B
Pagina: 1
Linea: 1
Posicion: 34

Palabra: Bolton
Pagina: 1
Linea: 9
Posicion: 63

Palabra: Cambridge
Pagina: 2
Linea: 16
Posicion: 1

Palabra: Canterville
Pagina: 1
Linea: 3
Posicion: 43

Palabra: Canterville
Pagina: 1
Linea: 5
Posicion: 26

Palabra: Canterville
Pagina: 1
Linea: 9
Posicion: 1

Palabra: Canterville
Pagina: 2
Linea: 18
Posicion: 1

Palabra: College
Pagina: 2
Linea: 15
Posicion: 58

Palabra: DIJERON
Pagina: 1
Linea: 1
Posicion: 10

Palabra: Dampier
Pagina: 2
Linea: 15
Posicion: 20

Palabra: Después
Pagina: 2
Linea: 16
Posicion: 12

Palabra: Estados
Pagina: 1
Linea: 2
Posicion: 1

Palabra: Hasta
Pagina: 1
Linea: 5
Posicion: 1

Palabra: Hiram
Pagina: 1
Linea: 1
Posicion: 28

Palabra: King's
Pagina: 2
Linea: 15
Posicion: 51

Palabra: LE
Pagina: 1
Linea: 1
Posicion: 7

Palabra: Lady
Pagina: 2
Linea: 17
Posicion: 62

Palabra: Lord
Pagina: 1
Linea: 5
Posicion: 21

Palabra: Lord
Pagina: 1
Linea: 8
Posicion: 56

Palabra: Me
Pagina: 1
Linea: 12
Posicion: 42

Palabra: Norteamérica
Pagina: 1
Linea: 2
Posicion: 19

Palabra: Nosotros
Pagina: 1
Linea: 8
Posicion: 2

Palabra: Otis
Pagina: 1
Linea: 1
Posicion: 37

Palabra: Otis
Pagina: 1
Linea: 13
Posicion: 17

Palabra: P
Pagina: 2
Linea: 15
Posicion: 9

Palabra: Rdo
Pagina: 2
Linea: 15
Posicion: 4

Palabra: TODOS
Pagina: 1
Linea: 1
Posicion: 1

Palabra: Unidos
Pagina: 1
Linea: 2
Posicion: 9

Palabra: a
Pagina: 1
Linea: 8
Posicion: 34

Palabra: a
Pagina: 1
Linea: 9
Posicion: 24

Palabra: a
Pagina: 1
Linea: 10
Posicion: 55

Palabra: a
Pagina: 1
Linea: 12
Posicion: 68

Palabra: a
Pagina: 2
Linea: 16
Posicion: 55

Palabra: a
Pagina: 2
Linea: 17
Posicion: 40

Palabra: a
Pagina: 2
Linea: 19
Posicion: 1

Palabra: adquirió
Pagina: 1
Linea: 3
Posicion: 18

Palabra: al
Pagina: 1
Linea: 1
Posicion: 18

Palabra: allí
Pagina: 1
Linea: 8
Posicion: 42

Palabra: anciana
Pagina: 1
Linea: 9
Posicion: 29

Palabra: apenas
Pagina: 2
Linea: 18
Posicion: 27

Palabra: asimismo
Pagina: 1
Linea: 14
Posicion: 34

Palabra: asociado
Pagina: 2
Linea: 15
Posicion: 38

Palabra: ataque
Pagina: 1
Linea: 10
Posicion: 11

Palabra: aun
Pagina: 1
Linea: 14
Posicion: 16

Palabra: biblioteca
Pagina: 2
Linea: 19
Posicion: 61

Palabra: castillo
Pagina: 1
Linea: 3
Posicion: 31

Palabra: causa
Pagina: 1
Linea: 10
Posicion: 57

Palabra: cena
Pagina: 1
Linea: 12
Posicion: 36

Palabra: cometía
Pagina: 1
Linea: 2
Posicion: 38

Palabra: como
Pagina: 1
Linea: 5
Posicion: 39

Palabra: como
Pagina: 1
Linea: 14
Posicion: 29

Palabra: comunicárselo
Pagina: 1
Linea: 6
Posicion: 36

Palabra: conciliar
Pagina: 2
Linea: 18
Posicion: 44

Palabra: condiciones
Pagina: 1
Linea: 7
Posicion: 1

Palabra: continuar
Pagina: 2
Linea: 17
Posicion: 30

Palabra: creo
Pagina: 1
Linea: 12
Posicion: 45

Palabra: creyó
Pagina: 1
Linea: 6
Posicion: 14

Palabra: criados
Pagina: 2
Linea: 17
Posicion: 16

Palabra: cuando
Pagina: 1
Linea: 3
Posicion: 11

Palabra: cuando
Pagina: 1
Linea: 6
Posicion: 51

Palabra: cuando
Pagina: 1
Linea: 12
Posicion: 1

Palabra: de
Pagina: 1
Linea: 1
Posicion: 53

Palabra: de
Pagina: 1
Linea: 2
Posicion: 16

Palabra: de
Pagina: 1
Linea: 3
Posicion: 40

Palabra: de
Pagina: 1
Linea: 5
Posicion: 51

Palabra: de
Pagina: 1
Linea: 6
Posicion: 33

Palabra: de
Pagina: 1
Linea: 7
Posicion: 13

Palabra: de
Pagina: 1
Linea: 9
Posicion: 60

Palabra: de
Pagina: 1
Linea: 14
Posicion: 1

Palabra: de
Pagina: 1
Linea: 14
Posicion: 59

Palabra: de
Pagina: 2
Linea: 15
Posicion: 66

Palabra: de
Pagina: 2
Linea: 17
Posicion: 9

Palabra: de
Pagina: 2
Linea: 19
Posicion: 41

Palabra: deber
Pagina: 1
Linea: 6
Posicion: 27

Palabra: debido
Pagina: 2
Linea: 18
Posicion: 65

Palabra: decirle
Pagina: 1
Linea: 13
Posicion: 1

Palabra: del
Pagina: 1
Linea: 10
Posicion: 19

Palabra: del
Pagina: 1
Linea: 10
Posicion: 63

Palabra: del
Pagina: 2
Linea: 15
Posicion: 47

Palabra: del
Pagina: 2
Linea: 16
Posicion: 21

Palabra: desde
Pagina: 1
Linea: 9
Posicion: 14

Palabra: dio
Pagina: 1
Linea: 10
Posicion: 4

Palabra: disparate
Pagina: 1
Linea: 3
Posicion: 1

Palabra: dos
Pagina: 1
Linea: 11
Posicion: 41

Palabra: duquesa
Pagina: 1
Linea: 9
Posicion: 46

Palabra: duquesa
Pagina: 2
Linea: 16
Posicion: 60

Palabra: el
Pagina: 1
Linea: 3
Posicion: 28

Palabra: el
Pagina: 1
Linea: 4
Posicion: 8

Palabra: el
Pagina: 1
Linea: 5
Posicion: 7

Palabra: el
Pagina: 1
Linea: 6
Posicion: 24

Palabra: el
Pagina: 1
Linea: 13
Posicion: 27

Palabra: el
Pagina: 1
Linea: 14
Posicion: 47

Palabra: el
Pagina: 2
Linea: 15
Posicion: 1

Palabra: el
Pagina: 2
Linea: 18
Posicion: 54

Palabra: embajador
Pagina: 1
Linea: 1
Posicion: 43

Palabra: embrujado
Pagina: 1
Linea: 4
Posicion: 24

Palabra: en
Pagina: 1
Linea: 6
Posicion: 21

Palabra: en
Pagina: 1
Linea: 14
Posicion: 20

Palabra: esqueléticas
Pagina: 1
Linea: 11
Posicion: 51

Palabra: estaba
Pagina: 1
Linea: 4
Posicion: 17

Palabra: estaba
Pagina: 1
Linea: 12
Posicion: 8

Palabra: expuso
Pagina: 1
Linea: 8
Posicion: 49

Palabra: familia
Pagina: 1
Linea: 14
Posicion: 7

Palabra: fantasma
Pagina: 1
Linea: 13
Posicion: 30

Palabra: galería
Pagina: 2
Linea: 19
Posicion: 47

Palabra: gran
Pagina: 1
Linea: 2
Posicion: 50

Palabra: gran
Pagina: 1
Linea: 5
Posicion: 54

Palabra: ha
Pagina: 1
Linea: 13
Posicion: 39

Palabra: hemos
Pagina: 1
Linea: 8
Posicion: 21

Palabra: hombre
Pagina: 1
Linea: 5
Posicion: 44

Palabra: hombros
Pagina: 1
Linea: 11
Posicion: 33

Palabra: honradez
Pagina: 1
Linea: 6
Posicion: 1

Palabra: incidente
Pagina: 2
Linea: 16
Posicion: 36

Palabra: la
Pagina: 1
Linea: 9
Posicion: 43

Palabra: la
Pagina: 1
Linea: 12
Posicion: 33

Palabra: la
Pagina: 1
Linea: 14
Posicion: 4

Palabra: la
Pagina: 1
Linea: 14
Posicion: 62

Palabra: la
Pagina: 2
Linea: 16
Posicion: 57

Palabra: la
Pagina: 2
Linea: 19
Posicion: 44

Palabra: la
Pagina: 2
Linea: 19
Posicion: 58

Palabra: lamentable
Pagina: 2
Linea: 16
Posicion: 25

Palabra: las
Pagina: 1
Linea: 6
Posicion: 67

Palabra: le
Pagina: 1
Linea: 10
Posicion: 1

Palabra: le
Pagina: 1
Linea: 11
Posicion: 5

Palabra: localidad
Pagina: 1
Linea: 14
Posicion: 65

Palabra: logró
Pagina: 1
Linea: 10
Posicion: 30

Palabra: logró
Pagina: 2
Linea: 18
Posicion: 37

Palabra: los
Pagina: 2
Linea: 17
Posicion: 12

Palabra: los
Pagina: 2
Linea: 19
Posicion: 3

Palabra: lugar
Pagina: 1
Linea: 4
Posicion: 11

Palabra: manos
Pagina: 1
Linea: 11
Posicion: 45

Palabra: mi
Pagina: 1
Linea: 9
Posicion: 26

Palabra: mismos
Pagina: 1
Linea: 8
Posicion: 11

Palabra: mismísimo
Pagina: 1
Linea: 5
Posicion: 10

Palabra: misteriosos
Pagina: 2
Linea: 19
Posicion: 14

Palabra: muchas
Pagina: 2
Linea: 18
Posicion: 13

Palabra: ninguno
Pagina: 2
Linea: 17
Posicion: 1

Palabra: no
Pagina: 1
Linea: 8
Posicion: 18

Palabra: no
Pagina: 1
Linea: 10
Posicion: 27

Palabra: noches
Pagina: 2
Linea: 18
Posicion: 20

Palabra: nuestro
Pagina: 2
Linea: 17
Posicion: 42

Palabra: nunca
Pagina: 1
Linea: 10
Posicion: 48

Palabra: obligado
Pagina: 1
Linea: 12
Posicion: 59

Palabra: ocurrido
Pagina: 2
Linea: 16
Posicion: 46

Palabra: pactaron
Pagina: 1
Linea: 6
Posicion: 58

Palabra: para
Pagina: 1
Linea: 12
Posicion: 28

Palabra: personas
Pagina: 1
Linea: 13
Posicion: 64

Palabra: por
Pagina: 1
Linea: 13
Posicion: 53

Palabra: por
Pagina: 1
Linea: 14
Posicion: 43

Palabra: porque
Pagina: 1
Linea: 4
Posicion: 1

Palabra: produjo
Pagina: 1
Linea: 11
Posicion: 8

Palabra: profesor
Pagina: 2
Linea: 15
Posicion: 29

Palabra: provenían
Pagina: 2
Linea: 19
Posicion: 30

Palabra: párroco
Pagina: 1
Linea: 14
Posicion: 50

Palabra: que
Pagina: 1
Linea: 2
Posicion: 34

Palabra: que
Pagina: 1
Linea: 9
Posicion: 20

Palabra: que
Pagina: 1
Linea: 10
Posicion: 23

Palabra: que
Pagina: 1
Linea: 11
Posicion: 1

Palabra: que
Pagina: 1
Linea: 13
Posicion: 23

Palabra: que
Pagina: 2
Linea: 19
Posicion: 26

Palabra: quiso
Pagina: 2
Linea: 17
Posicion: 24

Palabra: recobrarse
Pagina: 1
Linea: 10
Posicion: 37

Palabra: ruidos
Pagina: 2
Linea: 19
Posicion: 7

Palabra: se
Pagina: 1
Linea: 6
Posicion: 11

Palabra: sentir
Pagina: 1
Linea: 11
Posicion: 16

Palabra: servicio
Pagina: 2
Linea: 17
Posicion: 50

Palabra: señor
Pagina: 1
Linea: 1
Posicion: 21

Palabra: señor
Pagina: 1
Linea: 13
Posicion: 10

Palabra: si
Pagina: 2
Linea: 18
Posicion: 34

Palabra: sido
Pagina: 1
Linea: 13
Posicion: 42

Palabra: sobre
Pagina: 1
Linea: 11
Posicion: 23

Palabra: sueño
Pagina: 2
Linea: 18
Posicion: 57

Palabra: sus
Pagina: 1
Linea: 11
Posicion: 29

Palabra: también
Pagina: 1
Linea: 12
Posicion: 50

Palabra: terror
Pagina: 1
Linea: 10
Posicion: 67

Palabra: tía
Pagina: 1
Linea: 9
Posicion: 37

Palabra: un
Pagina: 1
Linea: 2
Posicion: 47

Palabra: un
Pagina: 1
Linea: 10
Posicion: 8

Palabra: varias
Pagina: 1
Linea: 13
Posicion: 57

Palabra: venta
Pagina: 1
Linea: 7
Posicion: 16

Palabra: vida
Pagina: 1
Linea: 14
Posicion: 23

Palabra: vistiéndose
Pagina: 1
Linea: 12
Posicion: 15

Palabra: visto
Pagina: 1
Linea: 13
Posicion: 47

Palabra: viuda
Pagina: 1
Linea: 9
Posicion: 54

Palabra: vivir
Pagina: 1
Linea: 8
Posicion: 36

Palabra: vuelto
Pagina: 1
Linea: 8
Posicion: 27

Palabra: y
Pagina: 2
Linea: 17
Posicion: 60

Palabra: y
Pagina: 2
Linea: 19
Posicion: 56

==3322==
==3322== HEAP SUMMARY:
==3322==     in use at exit: 1,656 bytes in 3 blocks
==3322==   total heap usage: 384 allocs, 381 frees, 56,988 bytes allocated
==3322==
==3322== LEAK SUMMARY:
==3322==    definitely lost: 0 bytes in 0 blocks
==3322==    indirectly lost: 0 bytes in 0 blocks
==3322==      possibly lost: 0 bytes in 0 blocks
==3322==    still reachable: 1,656 bytes in 3 blocks
==3322==         suppressed: 0 bytes in 0 blocks
==3322== Rerun with --leak-check=full to see details of leaked memory
==3322==
==3322== For counts of detected and suppressed errors, rerun with: -v
==3322== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)


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

VSO State: Done


Looks like the `This` node will be unpositioned.


@elstgav how can I update my SublimeLinter-slim-lint plugin in ST3 to use this fix?


I find the `Virtual` naming a bit awkward. Could it be "includeDeleted"?


The script must be always there. It is up to the script then to know if the migration should be done or not.


Me parece que la coma no está de más, porque hacía locales las variables `minDate` y `maxDate`, no?


I think we should not print output when the migration has already been done, or it will appear at each subsequent start, so it is very relevant. Maybe we can simply add a debug log.


lvguannan


What is record? Give proper naming convention.


@hvr I agree that this check should change (and I'm fixing that now), but why 1.24 rather than 1.23?


@codalicious .... first 3 qestions ....


@jaubourg this file does not define `ajaxTransport` but calls it. I won't change the name or namespace of any function involved.


@codalicious you missed "for" in Source code...  :), maybe there's unnecessary "and" in ...easily extracted from the application and into...


The MigrateService service is a startable service, so it will automatically be launch by eXo container. So it looks like it will be started twice, no ?


don any one leave comments here..


Attending and trying to solve #12


Falta configurar


Dont any one give comments here that understood or not worked or not


:-D


I had included this into configure.zcml file before.. should we take it out?


be consistent, `return fmt.Errorf(...)`


Oh I guess I was getting confused by `renderItemToItem()` and `itemToRenderItem()`.   The latter is presumably called once per item.   But I also don't know when `renderItemToItem()` is called.  @cjolif?


Doesn't opera use [the same Blink rendering engine as Chrome](https://en.wikipedia.org/wiki/Opera_(web_browser)#History)? Alloy works perfectly fine for me in Opera.


+ lokale Bilddateien durch Uploads ersetzt


@codalicious :clap: great work, really good style of writing, can't wait to read your restaurant reviews. :grinning:


Agreed. I will make the change as soon as I will get my hands on it.

Ali

On Sun, May 31, 2015 at 9:59 AM -0700, "Antonio Scandurra" <notifications@github.com> wrote:

Why can't we hardcode everything? I mean, I see this stuff as "assets", which are by no means different from a logo or a fancy graphic.

What about keeping it simple and hardcode URLs?

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


Editted above post with more CPU info.

Now onto test 2 to get the memory stats:

Tested inside inn. Outside of any conversations. Game running at 12 fps, in hardware mode.

Program using 594,000K+ of memory (and rising). Do you need the CPU usage etc?

Took a screenshot of the game, which you can find here: http://postimg.org/image/ph2sljllr/


Yes, it should use 1.23 - using `custom-setup` with unreleased versions of Cabal should be possible.


I'd really like to see this moved to `pkg/archive`. there's nothing very special about it that ties it to the executor, other than a reference to the data input (bytes) and data output (path)


See pull #572


TeamCity xBIM :: Xbim Geometry :: Release [Build 87](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1782&buildTypeId=XBIM_XbimGeometry_Release) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 03:47:44


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


Closes #28


I've update the message and created the issue


Another pull


Running `mvn test` on the same server I exectute the DPU passes without errors. Is it possible that if I compile the DPU on a different machine, then it may not work on the server?


A side effect of this is that bagpipe cannot anymore use exabgp as a lib.
Maybe this main could be put in the exabgp.application submodule ? This would still allow to start exabgp with python -m, it would simply be "python -m exabgp.application" instead of "python -m exabgp".
Possible ?


Вах!


Display Error messages properly. Please see below screen-shot. It creating confusion.

![selection_066](https://cloud.githubusercontent.com/assets/5477412/7912664/35b231b6-0887-11e5-8944-d9de0958bb0a.png)


We have to copy these to the destination first so that the `manifestclasspath` task can build the classpath property.


No need to dl only some files. Download the entire set of titles and it will downgrade only what needs to be downgraded. Titles with the same version are ignored. Check the source code if you can read it.


I macha immer en Commit,  Fetch und denn as Rebase und denn gradwider Push - bis jetzt ohni Problem...


I just noticed this difference to Blink's IDL files. Is there any public discussion around this change I should be aware of, any data on how common the previously optional filename argument is in the wild?


Why did the `-shell` get removed here? Was this intentional? See #40263.


The `indexjars` tasks was the crucial element here.  I took a look at the 1.8.7 version and noticed this `INDEX.LIST` file was different.  After quite a bit of research (ant's documentation doesn't provide examples), this is what was settled on. :)


The usual convention in ROS is to name the directory the same way the package is called. So, I would suggest you to either rename the whole repository to ``event_bullet_world`` or create a new directory ``event_bullet_world`` and put all the files in that directory. The second would be more preferable if you are planning to create more than one package throughout this project. Otherwise, the first alternative would be easier, of course, and your GitHub history won't get messed up.


For extra test post them in #262

The game use a very framebuffer so I suspect that GS memory is too small to contains various data (only 4MB). So the game does a lots of texture transfer from/to the memory, which can be translated to lots of slow transfer of texture on the GPU side. @refractionpcsx2 doesn't complain on the perf for BG2, can you test it to compare the perf.


These lines are unrelated to this commit really, just doing some last minute cleanup. ;)


It should not change anything for bagpipe - what do you see please ?.


Any chance to get this in 3.1?


Это паршел не для "формы для создания нового вопроса", а для всего вопроса, просто название я выбрал неудачное, т.к. "_question" был занят для другого паршела.


Double entry for ``cram_language``. Doesn't cause errors but doesn't look clean.


I compiled the DPU on the same machine I use it in a pipeline. I have restarted the pipeline and will let you know once it finishes if it changed anything.


This MATLAB software package is used to predict protein phosphoglycerylation sites.


Je t'envoie un mail pour t'expliquer.

Le 1 juin 2015 06:46, Maxime Thirouin <notifications@github.com> a écrit :

> Y'a des gens qui s'inscrivent à ça ? On le sait où ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/francejs/ToulouseJS/commit/aee3d5c40a2425280d5711038a9c61332c0a9353#commitcomment-11447560>
> .
>


OH GED


I'll defer to you on that one. The problem I saw was calls to rf12_canSend() returning false before any new calls to rf12_recvDone which were able to clear the stall.


Good, I planned to do so tonight.


this needs explanation.


Excelente, gracias por las correciones.


This is reproducible with cl.exe 18.00.31101 (x86/x64) from Visual Studio 2013. Decoding of xll_51_16_192_768_1.dtshd from the test suite breaks without this patch.


Hola Mariano, si estaría bueno lo del ticket. Si, work in progress. Lo
resolvemos en cuanto podamos. Muchas gracias.

Saludos
Franco

El 1 de junio de 2015, 5:59, Mariano Cerdeiro <notifications@github.com>
escribió:

> Prefieren haga un ticket con referencia a todos mis comentarios de este
> commit así no se pierden?
>
> No se que es un WIP, work in process? Algunos comentarios igual son válidos
> y prefiero que los vean ahora y no con el trabajo listo :)
>
> Saludos
> Mariano
>
> Am 31.05.2015 23:42 schrieb "FrancoSalinas" <notifications@github.com>:
>
> > Hola Mariano, es un WIP. Lo subí porque hay un nuevo integrante en el
> > equipo y él se va a encargar de este archivo. Es muy difícil entender la
> > función de este archivo porque falta todo el resto del código en el que
> > venimos trabajando. Espero subirlo durante esta semana. Perdón por el
> > desorden. Saludos.
> >
> > 2015-05-30 15:07 GMT-03:00 Mariano Cerdeiro <notifications@github.com>:
> >
> > > mejor seria hacer 2 asserts?
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/ciaa/Firmware/commit/d349ab31d26c50ba1914fd1a1425b77d891cf432#commitcomment-11440524
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/ciaa/Firmware/commit/d349ab31d26c50ba1914fd1a1425b77d891cf432#commitcomment-11445794
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ciaa/Firmware/commit/d349ab31d26c50ba1914fd1a1425b77d891cf432#commitcomment-11449911>
> .
>


hey @manucorporat

I finally got a chance to take a quick look at this and it looks pretty good, it does seem however to only support english. I haven't been able to take a look at all of the gin code yet, so please forgive me if this is a stupid question, but is there a way you can pass the error itself back to the handler? that way the user can determine how they want to display the message and you could just provide some helper methods or ways to combine errors like in your example with email, description... is required.

I am looking into a way to allow for the end user to map a function to the error calls of my error structs and I don't think it would be too difficult:

```go

//vars etc...

fn := func(err *FieldError) string {
   /// call whatever function to parse error into string or even translations
   // and can use vars etc... from above
}

validate.AddFieldErrorHandler(fn)

// within the FieldError Error() call fn
```

the only issue I have in one of my apps is that I determine the users locale on the fly and so like in the example above vars etc... would be a moving target and I don't know how to deal with that yet; maybe you have an idea?


Thanks! No need for a new PR/branch though - just amend the commit and use `git push --force` to update the branch & PR - that way GitHub keeps track of everything.


This seems off, the Haxe plugin shouldn't depend on internal?


When generating your own custom messages, the build dependency you need is on ``message_generation``, ``message_runtime`` is a run dependency, as the name suggests. See http://wiki.ros.org/ROS/Tutorials/CreatingMsgAndSrv.


License


@littleguy77 I just checked and cpufeatures isn't necessary.


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


remove comments or improve it


pourquoi `get_map` au lieu de `map`?


Интересная реализация. В плане, не понятно, зачем вам именно random * random.. ну да ладно. Мелочи


1) Use ```form_for`` instead of ```simple_form_for```
2) No need to use static url. Use route helpers.

You can check all routes using [http://localhost:3000/rails/routes](http://localhost:3000/rails/routes)


Habitué du c# haha, corrigé thank


I was told the inn was much slower, it's possible pcsx2 still needs the 4gb patch to run these games well.  I haven't tested  that far, I might have a go later


setUnselect?


Coder le soir c'est mal, fixed


Marking them for `sync()`ing, as described [here](https://github.com/mozilla-services/readinglist/wiki/Client-phases#local-data-storage-requirements). The idea is to track record status information so we know what to send precisely when syncing (g. batching operations).


oh, right, thx


About the new pages, I thought @XhmikosR told me we would merge the branch
when updating the website for the stable but I'm not so sure I got it right.


[Also](https://github.com/prezi/spaghetti/commit/47addc218d4fde9f416f03bde71412facddb896b#commitcomment-11452987) here I am not sure this is intended? Are the Gradle plugins intended to be part of core or users of Spaghetti core?


> Also, in a few place (at least localpref and med), implementing cmp would make sense.

I made an exception for RT as you use the feature, and will implement it if you need it for other objects but until then I would rather only implement it if/when needed: otherwise not all attribute are behaving the same.


@jmcphers


This is the indexedDB schema version number; we added an index, so it's good practice to bump that number.


Согласен


Change this URL pattern (/containers) to some static place.


A transaction. What other name could better reflect this intention?


follow naming conventions


voglende -> volgende
geüdatet -> geüpdatet


Definitely!


Same here.


@staabm Right but you cannot call `ajax.ajaxTransport` as it won't be defined as `jQuery.ajax.ajaxTransport` but as `jQuery.ajaxTransport`. So for it to make sense you'd need to break out `src/ajax.js` into sub-components, one defining `ajaxTransport` and then you'd put it in dependencies list.

I'm not sure how feasible such refactoring would be but it's certainly not a question of changing two lines.


К коду особых претензий нет.
Претенции к функционалу:
- любой пользователь может удалять сообещния любого пользователя, что неправильно (нжно было сделать так же как и редактирование, чтобы только автор мог удалить что-то)
- в localstorage неплохо бы добавить сохранение имени залогиненного пользователя. т.к. при перезагрузке оно пропадает


if (m_upscale_multiplier <= 6 && (m_width / m_upscale_multiplier) != (m_buffer_size * 64)){
fix at 6x native setting


A cleaner and more flexible rewrite.

Accept or reject at will, or ask me to make changes to the pull request.

Cheers!


typo?


asdf


good job !


reack.


С учётом сроков оценка "нормально-" (4)


Sorry if this sounded like something that should be answered here, I personally know the answer to this question but I believes it should be an addition of the readme.


I missed this on the first review; "Kilo" should be capitalised here.


These get improved in 1ce819573d3b8bf7d1d919900c613de7beaf41f2


ouch, horizontal scrolls :(

what this line does?


Good call


merging changes


Will do :+1:


Can we add this comment in the code?


These macros should be in utils ns. Not in taxi NS IMO.


Add doc string to both of them.


I suggest move this group of strings for a DTO.


Ah, my mistake.


No, in dotc positions are "filled in" form the top if they are undefined. So the outer withPos is enough.


зачем доктору имя пациента?..


`startTransaction`?


dto.doSomething(container)


зачем пациенту наименование анализа??


зачем анализу наблюдение?


Euhm, will this build, even against master ?


This is a section that can not be included in the initial landing page, but is of course important as soon as we have an official client


This is a bit problematic, because:
- I don't think that uint128_t is defined by a standard such as ISO C99 or later, so does it make sense to add such a type to stdint.h which is supposed to provide standard integer types?
- __uint128_t itself is a GCC extension and I found no way to detect its availability (and thus define uint128_t only when it makes sense)

So, what about:
```
struct vfp_reg {
	uint8_t v[16];
};
```
(all we need is the storage space, we don't care much about having an arithmetic type inside)?


Refs #1198


This is a follow-up to 078fb2ce11a7ae8a5ab83935275ddbdef355856d


yupp this one is correct !


变动里的“连续相同的警告信息不输出”是什么意思？感觉和“本组件暂不提供重复信息只输出一次的功能”矛盾了。


Please update variable name.


There is still quite some considerations and discussions needed to provide accurate description of pro versions, hence the initial landing page should probably not include this section.


Please update parameter name.


ezt nem szoktuk, igaz, hogy olvashatóbb, de nem szoktuk


The brackets are all messed up here. Emacs should be highlighting the closing brackets for you, so that you could fix them up. ``C-M-f`` and ``C-M-b`` will move the cursor to the end / start of the S-expression.


a requestStack, akkor ugye nem kell, ahogy megbeszéltük, mert várhatod a metódusban


Why is this check only performed for GET and not also for the other methods below?


yes, which was the whole purpose ;)
see https://github.com/xbmc/xbmc/pull/7221


itt se kell


Here, e.g., one closing bracket missing.


nice! thank you.


This commit addresses the following situation: executing `test.jx` packed with `extract : true` was failing while there was `test` file (e.g. previously compiled native package).


Token


Вроде работает.
НО:
- как по мне, грубая ошибка дизайна заключается в том, что у вас история сообщений выглядит как List<String> а не List<Line> как моглы бы быть в вашем случае. Захотите узнать почему так - спрашивайте.
- следующая ошибка проектирования заключается в том, что у вас реализована генерация id для новых сообщений на клиенте. по-хорошему, она должна осуществлять на сервер, а клиент должен только лишь считывать id приходящие с сервака.

Также с учётом сроков ваша оценка "нормально-" (4)


I think can be removed then? :P


ennek hol a tesztje? ez nem egy class notfound?


Yes. I wanna do a prototype of live-visual-diffs of screencast frames between multiple browsers like IE and Chrome in https://github.com/auchenberg/remotedebug-gateway

(Going to be announced tomorrow)


I don't think you need to make it a ``desig-package``. A normal Lisp package should suffice. Make it a ``defpackage``. Then it will compile.


sleep()とswtch()にコメントを記述。


Although i added that "slogan" as a sort of filler, we should consider a tag line that conveys the functionality of the service in a short sentence. I would propose:
mapalupa - "publish your (eo or earth observation) data with ease!" or
"share your eo data interactively!" or
"publish your georeferenced data online!"


`maven-compiler-plugin` should be defined only at the level of root pom.xml


@aheba ?


There's no benefit to the extra namespace in the .cpp file


is there any need to return an error?


Why do you put an \ at the end of this line? This isn't needed. Can you change that?


Add a more high level comment to indicate we are creating a tempory file named .xxx, to ensure atomicity, which will be renamed with its final name to ensure atomicity.


**[unitTests_js_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_js_branches/12826/testReport/) for this commit**


rekt


i think CreatePidFile func is very important, so should we check any error ret value? for example, os.MkdirAll?


foss merged


Hmm?


Это я сделал, как у них в моделях, чтобы POST'ы прошли некоторые.


Ok, I'll remove it.


to be replaced in

    if (!(flags & TEE_DATA_FLAG_OVERWRITE)) {
        res = TEE_ERROR_ACCESS_CONFLICT;
        goto clean;
    }


what is this?


It's to have an ability to mock creation of Geolocation instance and cover with isolated unit tests


create a separate pull request for this


update from master


Sass can handle this. If not, the compiler would throw an error @Xayer


@jauler Raspberry


```form_for``` should be common. It should be use in ```_form.html.haml``` file conditionally.


why is this a separate commit? squash it into "introduce metadata object update"


commit


would it be better to move `ConfigParameters` into `config.yml`

ENV VAR are ok but better to have a less invasive way to configure a service, without affecting the entire JVM.
I think this [doc](https://dropwizard.github.io/dropwizard/manual/configuration.html) may help


Move in-line CSS into common.css with proper naming convention.


Вроде бы даже работает.
Правда вы забыли библиотеку json-simple подложить к исходникам, но допустим, я это сделал сам.
Учитывая сроки и то, что ошибки дизайна приложения из 7 лабы перекочевали и сюда, оценка "нормально" (5)


You also need to ``shadowing-import-from #:cpl`` of the symbol ``#:fail``. Otherwise you will have conflicts.


reack.


No need to use ```else``` condition.


@JovicaSusa :+1:


The directory is named ``world-events``.


стиль a la "быдлокод"
публичный класс, поля без модификаторов доступа... а надо бы писать т.н. JavaBeans
public class Message {
  private String id;
  // other private fields ...

  public String getId() { return this.id; }
  public void setId(String id) { this.id = id; }

  // other accessor methods
}


Neither this nor the one below it are a partition, they need to be all separate metrics


when we do the above task  here, it throws the following exception.
Exception in thread "main" org.apache.flink.api.common.InvalidProgramException: Object org.apache.flink.graph.Graph$3@1363fb3 not serializable
	at org.apache.flink.api.java.ClosureCleaner.ensureSerializable(ClosureCleaner.java:97)
	at org.apache.flink.api.java.ClosureCleaner.clean(ClosureCleaner.java:59)
	at org.apache.flink.api.java.DataSet.clean(DataSet.java:181)
	at org.apache.flink.api.java.DataSet.map(DataSet.java:211)
	at org.apache.flink.graph.Graph.mapEdges(Graph.java:414)
	at org.apache.flink.graph.library.HITS.run(HITS.java:83)
	at org.apache.flink.graph.Graph.run(Graph.java:1332)
	at org.apache.flink.graph.example.HITSExample.main(HITSExample.java:55)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:497)
	at com.intellij.rt.execution.application.AppMain.main(AppMain.java:140)
Caused by: java.io.NotSerializableException: org.apache.flink.graph.library.HITS
	at java.io.ObjectOutputStream.writeObject0(ObjectOutputStream.java:1184)
	at java.io.ObjectOutputStream.defaultWriteFields(ObjectOutputStream.java:1548)
	at java.io.ObjectOutputStream.writeSerialData(ObjectOutputStream.java:1509)
	at java.io.ObjectOutputStream.writeOrdinaryObject(ObjectOutputStream.java:1432)
	at java.io.ObjectOutputStream.writeObject0(ObjectOutputStream.java:1178)
	at java.io.ObjectOutputStream.defaultWriteFields(ObjectOutputStream.java:1548)
	at java.io.ObjectOutputStream.writeSerialData(ObjectOutputStream.java:1509)
	at java.io.ObjectOutputStream.writeOrdinaryObject(ObjectOutputStream.java:1432)
	at java.io.ObjectOutputStream.writeObject0(ObjectOutputStream.java:1178)
	at java.io.ObjectOutputStream.writeObject(ObjectOutputStream.java:348)
	at org.apache.flink.util.InstantiationUtil.serializeObject(InstantiationUtil.java:314)
	at org.apache.flink.api.java.ClosureCleaner.ensureSerializable(ClosureCleaner.java:95)
	... 12 more


Nope. They install Biopython to a virtual env on top of your system's Python, which I can't access from R, at least not in any sort of robust way. Mainly because I don't know the path to that virtual env, but I'll look into It some more. It will all work in R, but I'm going to have some stuff that uses Biopython only if it exists on the system path to achieve some better results (i.e. it's optional). If Biopython is able to be found that message will go away. Easiest way to get it is to deactivate Anaconda and do `pip install Biopython`. I have a line in my shell profile that activates Canopy. I just comment it out, open a new shell, do the install, then uncomment the line to reactivate it.


This special casing seems fragile in the face of future variables, I'd suggest having each as a separate metric


use the following command for installing phylogeo package
install_github("zachcp/phylogeo")


If I'm not mistaken this cannot be built when you have kodi installed on your system and the includes are in /usr/include/kodi/xbmc_audioenc_dll.h


# 修正すべき場所
* dynamic_pointer_castの部分はtaskで関数を作ってその中でやってあげましょう。
* taskに各Behaviorのクラスが追加されてるけど、Behaviorはゲームオブジェクトではなく、コンポーネントです。
* なので、Playerに包含するなどにしておきましょう。
* プレイヤーのものなのにタスクから探すのは適切ではないと思います。

以上です。


I think I'm "seeing double"... can't catch the difference.


depth is not an UCI option! test deleted: it would have frozen all the workers into an infinite search.
you can run fixed depth=10 locally. it's very cheap.


First thing, i'm not sure if i get the git right,
But, ...
if this is last file? which i already build a sample over it, and test it, it's not going to be enough...

there should be these two option... otherwise saving this content is nonsense
1. Save Data To Cookie, and re-loadable ability (should be checked if the new grid columns are changed (due update of older system, so wont result in failure))
2. Ability to transfer state object to the remote server, and to do save it in DB / Or remote cookie. (BTW if this one exist then it's lack of my knowledge to translate java object to ASP Objects...)

And may i notified if it fixed: mimosh_pisholack@hotmail.com


ここの上二行は
'''
$data['users'] = $users[0];
'''
この一行で書けるのでは?


Another commit with a typo in the commit message. Wrote "reprase" when I ment to say "rephrase".


It is possible. There are two ways to start a node in roslisp, one is to use ``with-ros-node`` and put all the code inside this macro. The other one is to use the function ``roslisp:start-ros-node``. You call it once inside of the initialization function and close the node with ``roslisp:shutdown-ros-node`` in the destructor or something like that. Any code in between these two calls should be able to use any ROS communication functionality (``ros-info``, ``subscribe``, etc...)


Please remove code that is not yet working from the PR, thanks!


@aclark4life This might provide a workaround that avoids the circular import problem with setuptools pkg_resources, for now:  https://github.com/upiq/uu.formlibrary/blob/master/uu/__init__.py

Alternately, we could always just name the package upiq.task instead?

Had I known about this name conflict, I would have chosen a different namespace long ago, but the workaround has always worked for me without realizing it was a workaround.


Thanks, I'll fix those later today. I had some other options for radius but failed to do cleanup properly!


let's allow only

<key> : <value> with or without spaces


this is completely unrelated, create a separate pull request for this


Great thanks :D


Pushing sticky events when the pad is not linked is not a problem, they will be sent on the next opportunity then. Not sure how this fixes anything, can you provide a testcase? :)


Thanks ;)


this breaks backward compatibility for no reason. unmergable from my point of view.


The geoserver commit number looks wrong here ?


I'm calling `_showOpenPanel()` from `attachedCallback()` because I only need to call that method on the accordion initialization to show the first open panel.
I'm also calling it on `refreshRendering()` but only after the no attached panels are attached.
Actually I think I could call that method directly instead of using the `accordion.deliver()` on line 190 ?


The condition `!$patternselect` would never be true, as the parameter has a default value.

I would really like to see an option to set a pattern to use, as it looks like the pattern is never explicitly set in any call to this function.


Do we want to remove `SetHex` as a public member of `base_uint`?


Can you elaborate or give a pointer to more information? Just curious!


check for ```!params[1]``` should go first


the last lines can be merged to one line


I'll give it a try tonight thanks. My coding skills are so so but I'll give it a look.

Ill link you the guide when I am done in case you want to system transfer


A switch-case statement would probably more suitable here ;)


This dead code should be removed. Theres also the field definition at the top which is also commented out and can be removed.


Right, it's generic. Makes the actual change pretty small, nice.


> The update hooks are, I think, being called in exactly the same places here as in 1.51 (and previous).

> (Whether that's correct or not is another matter.)

I think this is how we got away with it before. I recon we should move this `performPostUpdateHook` call into `Settings.set`, so the hooks are performed for all updates.


two


It works because the value of `KODI_INCLUDE_DIR` used in `CMakeLists.txt` points to `/usr/include/kodi` and not `/usr/include`.


**All** functions throw badarg if the table does not exist
and you got the following section at the top:

"*Failure*
In general, the functions below will exit with reason
badarg if any argument is of the wrong format, if the
table identifier is invalid or if the operation is denied due to
table access rights."


you are mistaken. Binary addons read kodi's include dir from kodi-config.cmake


Ahh yeah good point I'll fix that :)


Added more comments to the code


This (or `Settings.clear`) should call `Settings.performPostUpdateHook`.


This is related to issue #7


Shouldn't an exception be thrown if the LOC doesn't exist as Mambu should return an HTTP response code 404?


Yeah, sorry, thanks... that's what I get for making changes to the build on a Friday night before heading out for dinner, and then for trying to fix it via in-line github editing on Sunday... :)

Will take a look now to fix whatever is left to fix...


Completed.


```
struct vfp_reg {
    uint8_t v[16];
};
```
sounds good. Please set some alignment too.


ok so It will no longer work with 14.x ?


This has now been taken care of by using DOM construction instead of HTML strings.


@otafuku
ここの上二行は
$data['users'] = $users[0];
この一行で書けるのでは?


Rebuilded/uploaded.


Sounds great @jhoguet, I had some family obligations to tend to this weekend and am planning on beginning and completing the section today. I'll definitely lean heavily on what you've done.


Hey Felix,
Thanks for the help. Moved the redirect line to the end to remove the redundency.

How does this version look.

Regards,
Blayne


Fair enough.


Thanks. Next time try to comment directly in the source code. It is easy to find.

@refractionpcsx2 your code :p GSRendererHW.cpp line 44