BTW:
question to @xfurry:
we having spam with spell error there:
```quote
-- ERROR:Aura::TriggerSpell: Spell 37546 have 0 in EffectTriggered[1], not handled custom case?
```
its becouse we missing spell 37545 -- not in db.
Can something be done there?

P.S... for tests... i made empty new spell 37545 ... and that solves it... :) ..
any ideas?


I'm installing Eclipse and Android so I can help out and learn what's going on. Bluetooth seems to be the way to go since using a LAN would be way more complicated, unless there is something for android like iOS's GameCenter that you can add friends and play over the internet with


Oh, and is this from some other project? Do we have a license statement and copyright for it? If you wrote it, would be good to mark it BSL 1.0 so that it can get blended with the rest of my cmake modules.


Why not get('status') ? ParsedRequest perform a processing on parameters and group them by where, sort ...
Probably you can use this.


not sure if its a problem on your side, but I have to use 1.0.1 in Android Studio (Gradle). With 1.0.2 I get ClassNotFoundException.


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.165](http://teamcity.monogame.net/viewLog.html?buildId=6064&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependency failed: MonoGame :: Develop (Mac) Build time: 00:00:00


`<chrono>` doesn't have mapping to civil calendar dates except through `time_t` and `tm`.  Assuming that is what we need, I see two options:
1.  Travel via `tm` and `time_t`.
2.  I've been working on a `<chrono>`-compatible date library.  Header-only.  Hyper-efficient.  Currently requires C++14 `constexpr`, but that could be trivially commented out:  Description and implementation at: http://howardhinnant.github.io/date_v2.html

Syntax would look something like:
````
expectEquals (ts, day_point(may/2/2015) + hours(5) + minutes(42) + seconds(23));
````
Or once we get C++14 `<chrono>` literals:
````
expectEquals (ts, day_point(may/2/2015) + 5h + 42min + 23s);
````


Done.


Maybe remove this line too?


however erase method will change pointer in previous element to next element and returns correct next id


What exactly would you like to extract?


Fixed.


Yes.
Le 2015-04-06 10:42 AM, "Durand" <notifications@github.com> a écrit :

> Should it be in maintenance/4.7 ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/inverse-inc/packetfence/commit/ab750b40c7ae622bc0c98dbff81f7c4eb1e3e2ee#commitcomment-10584685>
> .
>


Okay, found the problem. There's an error in the specialization rewiring. PR coming soon.
Results:
```
quicksort_array_of_tuples:
   miniboxed_hardcoded-1000000 :   323.18417 +/-   2.54005
                     miniboxed :   726.79137 +/-   9.17921
                         ideal :   318.10269 +/-   1.98126
                       generic :   684.42501 +/-   5.65956
                   specialized :   322.53045 +/-   2.30520

```


what about uBlock-maintained ?


@vigneshng since MHA has many depts we would have to make separate documents for each of the depts. So if thats the case I will make a folder for the MHA which you can then work on.


Http://outlinenone.com - tego się nie tyka! Można usunąć jedynie w tych przypadkach, gdy style z `:hover` przepisujemy na `:focus`

Ciekawi mnie kiedy znienawidzicie moje marudzenie :D


This one is a bit trickier.

Since the DOGNZB attributes are not specified in the standard way I created a function to read the additional values.

So if we get for instance:

<newznab:attr name="category" value="7000"/>
<newznab:attr name="category" value="7030"/>
<newznab:attr name="size" value="196431487"/>
<newznab:attr name="usenetdate" value="Sun 05 Apr 2015 23:44:35 -0600"/>
<newznab:attr name="group" value="alt.binaries.comics.dcp"/>

Then the function will create a 'newnab' section in the entry that it returns so that you can use it like this:

to get the list of categories : entry['newznab']['categories']
But much more importantly:
to get the size : entry['newznab']['size']

The rest of the attributes should be there too, just not of much use.

So that _start_newznab_attr(self, attrsD) function is injected in the feedparser code with

feedparser._FeedParserMixin._start_newznab_attr = _start_newznab_attr

So it will be automatically triggered whenever feedparser comes across a newznab:attr tag. And with that there's no need to change the version of feedparser or anything.


First order of business after declaring the system variable is finding a way to see which columns are of interest, according to what we are trying to do and also according to the binlog_row_image variable.

Also, ignore commit title, it's mark_columns_per_binlog_row_image.


唔。。恩。。。


Ekran full HD - 85% jest tak rozciągnięte, że po prostu za trudno się czyta


This code is not used and prevents compilation when changing the prototype of binlog_write_row. As discussed with Sergei Golubchik, this whole code is up for the chopping block after this feature is complete.


Why not callbackDict


index.html
<!DOCTYPE html>
<html>
  <head>
    <meta charset="utf-8">
    <meta http-equiv="X-UA-Compatible" content="IE=edge">
    <meta name="viewport" content="width=device-width, initial-scale=1">

    <title>--- TODO ---</title>

    <link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/3.2.0/css/bootstrap.min.css">
    <link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/font-awesome/4.1.0/css/font-awesome.min.css">

    <!-- HTML5 Shim and Respond.js IE8 support of HTML5 elements and media queries -->
    <!-- Leave those next 4 lines if you care about users using IE8 -->
    <!--[if lt IE 9]>
      <script src="https://oss.maxcdn.com/html5shiv/3.7.2/html5shiv.min.js"></script>
      <script src="https://oss.maxcdn.com/respond/1.4.2/respond.min.js"></script>
    <![endif]-->
  </head>
  <body>

    <!-- TODO: Here goes your content! -->

    <!-- Including Bootstrap JS (with its jQuery dependency) so that dynamic components work -->
    <script src="https://ajax.googleapis.com/ajax/libs/jquery/1.11.1/jquery.min.js"></script>
    <script src="https://maxcdn.bootstrapcdn.com/bootstrap/3.2.0/js/bootstrap.min.js"></script>
  </body>
</html>


Utility function.


In case of doubt I tend to consult this:
https://google-styleguide.googlecode.com/svn/trunk/cppguide.html

If it is not included there then I am happy to go with what we agree is
best. From your comments the only one that confuses me slightly is with the
ampersand &. Is it the norm to pass arguments by reference? I would have
thought it is safer to not do so unless explicitly necessary, since the
function may edit the argument unexpectedly.

 For example, I send some stiffness values that I calculated for the joints
to perform (SetStiffness), and I wouldn't want the setting function to be
able to change *my *original stiffness variable.

 Does that make sense?

On 6 April 2015 at 15:25, Stan Manilov <notifications@github.com> wrote:

> You don't need to blindly follow them, if you disagree. In fact, I'll be
> happier to know your opinion about it, rather than having the code more
> readable (by my definition of readable).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/edinferno/edinferno/commit/e31cbec30ac70f996e3085dbb41110aebbddeeb9#commitcomment-10584504>
> .
>


Зачем нужны переменные для хранения констант? Почему не использовать константы.


Will be used in a later commit. Taken straight from MySQL


if anyone can think of a better way to determine if a block is 2 high let me know :)


Oops, I meant to merge this with the next commit and have a more meaningful commit message. Anyway, like the next commit this is only about style changes in this file.


There seems to be an error with the postfix  :joy:


That link (  at http://newznab.readthedocs.org/en/latest/misc/api/#predefined-attributes ,  ) in the commit comment was more meant for me than for you so that I could find it again later :)


Here is where the previous commit is useful.


Good catch. Likely because `callbackDict` was introduced in a second refactoring step and I missed this particular line. Will push another commit.


Why not callbackDict?


We will no longer pass the columns as a parameter. They should be held within the TABLE* table ptr, in read_set and write_set.


I feel like every ruby developer has created this bug at least once.


what about here?


It's an oversight, will fix


Might want to apply the same to `_.method`?


So, just to confirm, do we still want to allow users to have the option to input a negative sigma_t?


Why would you create a new Python script instead of rework the existing?


I think you can use "getBlockY()" for this.


For every kind of event, we remove the columns parameter. Instead we use the table->read_set and write_set according to the kind of event.

write_set in this case as we are writing a brand new row.


one assignment per line


@MaZderMind Sieht sinnvoll aus. Auch wenn das mit dem Tag nicht umbedingt notwendig wäre. ;-) Aber schadet ja auch nichts.


.......bot_version.cpp has been inserted to .gitignore


read_set as we're interested in finding the row to delete.


Here we actually use both the read_set (for the before image) and write_set (for the after image)


remove space


Highly educational video


Just update the function prototypes.


add space


already tried, that returns the Y of the bottom block as minecraft only deals with the bottom block as far as I am aware


space


We're no longer using columns as parameters.


@knightopress I'm pretty sure that will just return the Y value of the block you are trying to place, which is the bottom half of the door.


Naja ich hab halt immer dieses "Dieser gute Vortrag am zweiten Tag" im Kopf. Und du weist selbst wie lange die Fahrpläne so im Schnitt online sind. Außerdem hilft das z.B. beim Congress, wo die Vorträge schon ja so zusammen in Blöcken geplant werden, ähnliche Vorträge zu finden


@osworx :+1:
in my own implementation i have he() hd() (and _t(..) for language->get(..), _j() for javascript string) model/controller/view utility methods, but having methods with more semantic names like htmlEncode(), htmlDecode(), jsEncode(), ... with default encoding/decoding params would be good.


Hey there.  I added this null check as a quick and dirty fix for a problem I was experiencing with null parent associations in one of my ActiveRecord models.

I don't think this should be merged into master, at least not in this form.  It may be better to instead warn of a null parent reference if found and skip processing for that particular reference.


Here we change the actual logging functions to make use of the correct columns.


Logging functions now don't require the columns parameters.


Ack!  How did that work?  /me fixes.


This commit introduces ugly path results. It inserts a dummy obstruction square and then sometimes forces    the unit to detour. What is the intention of this commit?
![screenshot0028](https://cloud.githubusercontent.com/assets/9270128/7006039/c482bc3a-dcb8-11e4-9681-4e3ef9c3cecd.gif)


process has been killed


Haven't really dug into them yet. It's on my todo. You'll see some work when they start getting documented :)


Fixed. I forgot to push :)


This file just has the function parameters and the function they call updated.


Nothing hard :) Just transmit me your skype email adress, the one you used to register on skype


:clap:


extract const (see above)


Yeah, definitely Bluetooth
I've only skimmed the details, but basically we'll need code to manage
picking a device and whatnot and once there's a connection we gotta figure
out how to send the actions between devices.
For war, we can do a tostring and getbytes on the card to send it's data,
then reconstruct it.
Blackjack is a bit more complex cause we have to manage bets and stuff too

I'm gonna start working on it in a few min
On Apr 6, 2015 10:45 AM, "Tim Clyne" <notifications@github.com> wrote:

> I'm installing Eclipse and Android so I can help out and learn what's
> going on. Bluetooth seems to be the way to go since using a LAN would be
> way more complicated, unless there is something for android like iOS's
> GameCenter that you can add friends and play over the internet with
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TimothyJimothy/WarGame/commit/9aae0408ae26d7a27e5eed8fca0d2172cd19b9f2#commitcomment-10584713>
> .
>


Same of old log events, but I don't know exactly where these are used. I could use a bit of help with this part.


why not process error before?


As explained in a previous commit, when writing a brand new row, make use of the write_set.


Fixed


space


I actually don't understand why we need this. This was copy pasted from MySQL and is on my TODO list to figure out why.


exit -> exited


Even though I prefer this indentation this is not the indentation of this project. The indentation you had before was correct.


remove space


Why? This is identical to line 50 above, which acts as a fallback


With binlog_row_image=FULL This has the same effect as before, since read_set === write_set == all bits set.


done


I think its called "Play Games" for android's version of gamecenter

On Mon, Apr 6, 2015 at 10:45 AM, Tim Clyne <notifications@github.com> wrote:

> I'm installing Eclipse and Android so I can help out and learn what's
> going on. Bluetooth seems to be the way to go since using a LAN would be
> way more complicated, unless there is something for android like iOS's
> GameCenter that you can add friends and play over the internet with
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TimothyJimothy/WarGame/commit/9aae0408ae26d7a27e5eed8fca0d2172cd19b9f2#commitcomment-10584713>
> .
>


do you need \ here?


![screen shot 2015-04-06 at 11 01 56 am](https://cloud.githubusercontent.com/assets/4239858/7006108/6f744978-dc4c-11e4-86dd-4b3dc03d0ddd.png)

This is what I see when I go to my page, no mention of the waiver.


Error: no websocket


Ping.


Again, why? I suspect add_row_data changes the sets but I didn't get that far into debugging it.


When working with GDI, one needs to not take the "out of memory" too literally. This is well-known problem.


Not 100% sure on how gamecenter works, but play games is basically a leader
boards/ achievement system.
Don't think we can use it for multi-player? At least I think
On Apr 6, 2015 11:02 AM, "MitchMo" <notifications@github.com> wrote:

> I think its called "Play Games" for android's version of gamecenter
>
> On Mon, Apr 6, 2015 at 10:45 AM, Tim Clyne <notifications@github.com>
> wrote:
>
> > I'm installing Eclipse and Android so I can help out and learn what's
> > going on. Bluetooth seems to be the way to go since using a LAN would be
> > way more complicated, unless there is something for android like iOS's
> > GameCenter that you can add friends and play over the internet with
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/TimothyJimothy/WarGame/commit/9aae0408ae26d7a27e5eed8fca0d2172cd19b9f2#commitcomment-10584713
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TimothyJimothy/WarGame/commit/9aae0408ae26d7a27e5eed8fca0d2172cd19b9f2#commitcomment-10584981>
> .
>


Delete Row makes use of the read_set.


This line is the only one where I see something has actually changed.
The rest of the quotes and double-quotes were fine and according to WordPress coding standards.


technically it's 1.2+ now.   This will need updating.


https://developers.google.com/games/services/training/signin

On Mon, Apr 6, 2015 at 11:02 AM, Mitchell Moore <mamoore4@oakland.edu>
wrote:

> I think its called "Play Games" for android's version of gamecenter
>
> On Mon, Apr 6, 2015 at 10:45 AM, Tim Clyne <notifications@github.com>
> wrote:
>
>> I'm installing Eclipse and Android so I can help out and learn what's
>> going on. Bluetooth seems to be the way to go since using a LAN would be
>> way more complicated, unless there is something for android like iOS's
>> GameCenter that you can add friends and play over the internet with
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/TimothyJimothy/WarGame/commit/9aae0408ae26d7a27e5eed8fca0d2172cd19b9f2#commitcomment-10584713>
>> .
>>
>
>


Die Fahrpläne sind eigentlich immer online. Der Rest der Webseite ist dann wieder ne andere Sache. ;-)


finally :D


TODO: This function seems to make sense but I am not 100% confident I understand all it's implications.


You could check this and tell me what you think. Because this correction does not work for me.
http://pastebin.com/SnV1aafY


We no longer use this cols bitmap. It is safe to remove it. It previously acted as a "set all bits" bitmap.


Just some cleaning up. It was all in a loop that wasn't really necessary and added a lot of variables that were not really used.

I think it's cleaner that way but I have separated that in it's own commit so if you don't like it you can just ignore it but still have all the good bug fixing from the other commits.


Pues yo creo que es un poltergeist


Old typo, fixed.


Second commit was necessary to resolve a small branch problem.


Whole UserController is copied PMController, since PMs were only type of users before, so I reused some of those. There is more work on this controller in any case, since we will need to work on adding new users of different types, so I'll go over it again on the end.


From MySQL. TODO: Figure out what this actually fixes. I added it in order to make sure I get a working prototype that I can test.


Ah, I guess you can do multiplayer

Which approach should we take then?

I'll look into how complex it is to do via google play

On Mon, Apr 6, 2015 at 11:04 AM, MitchMo <notifications@github.com> wrote:

> https://developers.google.com/games/services/training/signin
>
> On Mon, Apr 6, 2015 at 11:02 AM, Mitchell Moore <mamoore4@oakland.edu>
>
> wrote:
>
> > I think its called "Play Games" for android's version of gamecenter
> >
> > On Mon, Apr 6, 2015 at 10:45 AM, Tim Clyne <notifications@github.com>
> > wrote:
> >
> >> I'm installing Eclipse and Android so I can help out and learn what's
> >> going on. Bluetooth seems to be the way to go since using a LAN would be
> >> way more complicated, unless there is something for android like iOS's
> >> GameCenter that you can add friends and play over the internet with
> >>
> >> —
> >> Reply to this email directly or view it on GitHub
> >> <
> https://github.com/TimothyJimothy/WarGame/commit/9aae0408ae26d7a27e5eed8fca0d2172cd19b9f2#commitcomment-10584713
> >
> >> .
> >>
> >
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TimothyJimothy/WarGame/commit/9aae0408ae26d7a27e5eed8fca0d2172cd19b9f2#commitcomment-10585007>
> .
>


Looks good. Thanks!


nu era mai bine daca era in format txt?


@brianpeiris good point, I'll revert. I think git/github makes ignoring whitespace changes easy, but it might still cause problems


This doesn't seem to make any sense for me. This is what MySQL does but does ha_update_row care?


Ok, sounds like adding an exception to iNoBounce isn't the right solution. Thanks for explaining the issue!


Closes #3363 #3096


Why not just have this one and leave out line 12451?


http://developer.android.com/google/play-services/setup.html

On Mon, Apr 6, 2015 at 11:04 AM, Mitchell Moore <mamoore4@oakland.edu>
wrote:

> https://developers.google.com/games/services/training/signin
>
> On Mon, Apr 6, 2015 at 11:02 AM, Mitchell Moore <mamoore4@oakland.edu>
> wrote:
>
>> I think its called "Play Games" for android's version of gamecenter
>>
>> On Mon, Apr 6, 2015 at 10:45 AM, Tim Clyne <notifications@github.com>
>> wrote:
>>
>>> I'm installing Eclipse and Android so I can help out and learn what's
>>> going on. Bluetooth seems to be the way to go since using a LAN would be
>>> way more complicated, unless there is something for android like iOS's
>>> GameCenter that you can add friends and play over the internet with
>>>
>>> —
>>> Reply to this email directly or view it on GitHub
>>> <https://github.com/TimothyJimothy/WarGame/commit/9aae0408ae26d7a27e5eed8fca0d2172cd19b9f2#commitcomment-10584713>
>>> .
>>>
>>
>>
>


The ultimate goal of the previous cleaning step: Now that all parsing is done the same way we can create a small _parse_feed(site, url) subroutine and use it everything, and that cuts down on a lot of the code, making it much easier to read (for me anyway).


Feels to me it would be much clearer to just say, units = (bytes > 1024*1024) ? 'MB' : 'KB'; and similarly compute a number. Also, the current code will return null if someone somehow has an image smaller than 1KB...will that cause problems? Could it happen?
Even the rounding seems unnecessarily complex..I can't see that for this purpose it's important whether we round up or down in the second decimal place.


This actually fixes the Update_Rows bug. Problem here is that we actually want the after_image from the write_set. Otherwise we'd get a HA_ERR_DUPP_KEY (or something simillar)


In `alignedBamUploadJob`, there is usage of /bin/gt-download-upload-wrapper-1.0.0/lib, which doesn't exist anymore since we're using 2.0.x. Also, there is a call to `/scripts/gnos_upload_data.pl`, but that script doesn't exist in `/workflow/scripts`. This code will only get called if there are two properties set, `bamUploadServer` and `bamUploadPemFile`. Since this code has not failed (as far as I know), is it even necessary? Should it be removed or updated?


может лучше
const freenect_resolution = BB

или
getopt


Can you explain the exact reason and further improvement plans?


Простой шаблон главной страницы


done


@andrsd - Let me know if this is the best way to retrieve the value of a nonlinear variable at an element centroid. I'm inside of a GeneralUserObject doing my own element loop so we can assume that we don't already have elements reinited.


Глобальные переменные это зло!


Also, another TODO related to this commit is removing the warning that binlog_row_image is a MYSQL_FEATURE that is not yet implemented in MariaDB.


The double 'return' here is awkward...All the other hanlder names start with 'Process'. I'm not sure that convention has value but I wondered about it.


@sbesson I think you cannot do that since you're not changing the session dict itself. It as there may be something about it not detecting the change though. This is only safe for reading


@patbos could you create test case and propose fix?


Nie można zrobić alternatywnego outline z :focus dla danych elementów ? Wydaje mi sie to sensowne z racji tego ze outline przeglądarki często nie dostosowywuje sie np do border-radius. Zamiast takiego outline można zrobić alternatywne style pod *:focus{}. Dobrze myślę czy źle ?


To be consistent it feels as if there should be a guard that there is a question mark in the RawUrl


Are their plans to formalized this API in the rstudioapi package?

As-is I could register lintr as a lint engine with

```r
as.environment(match('tools:rstudio', search()))$.rs.setLintEngine(lintr::lint)
```

And `lintr::lint` would be called instead of the internal RStudio linter.  The above does not quite work out of the box because the output formats are slightly different, but it would be fairly simple to translate.

Maybe exposing this was the idea eventually, but the API is still in flux?


Delegate  三种传值方式, 代理传值,属性传值,block传值


Hah. Yup, I was able to make a 563 byte image (1 x 1 pixel). Added "Bytes".


используйте Singleton pattern


In general it not enough to do good Form handling, duplicates that i already did https://github.com/KostyaSha/docker-plugin/commit/3cefa32c6a2eab9e72b3623c24eff7349b39ec9b


@enebo Check this please.  Adding this in fixed the Encoding#default_external specs using ShiftJS at start-up.


Out of curiosity: Why not `std::forward<Rng>(rng)` instead of `static_cast...`?


Is there an unwanted 'o' in oObject?


TeamCity ChameleonForms :: Continuous Integration [Build 226](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3540&buildTypeId=bt20) is now running


correct, I overlooked return assignment.


I think the "process" convention here tells us *nothing*. Not something to be emulated. I've changed this one to ReplyWithImageInfo


I would be willing to bet that it is much much simpler to import Googles "Play Games" API than it would be to try to figure out Bluetooth for the infinite number of devices.


@iam4x Please, don't forget to update notable changes in releases when you publish something new release and update release draft if you merge PRs. https://github.com/AtomLinter/linter-jscs/releases


``` this ``` ?


So you should actually remove the 'tool_insecure_warning' key from the translation here.  Providing keys for translations is now a deprecated practice.  The problem with leaving it the same here would be that the updated string would never be translated because it wouldn't be recognized as a change by the translators.  So, just removing the key should solve the problems.


1. 点击后马上移开鼠标再回来会出问题
2. 不停点击某个小按钮会有问题


done


혜정이 ㅎㅇ


Same as the comment on url_show.html.erb


No, the spring template is not used. We are using the one named prosilver.
The spring template is the template that was being used for the old forum.
It is not compatible with the new version of phpbb.
On Apr 6, 2015 6:35 AM, "abma" <notifications@github.com> wrote:

> wrong file?
>
> should be wwwroot/phpbb/styles/spring/template/overall_header.html ?!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/spring/spring-website/commit/a244582311b1fed6ad92d9b6a844400926f040b9#commitcomment-10583932>
> .
>


no


1. 不停点击@+按钮会有问题
2. 一个按钮在获取时其他按钮仍可以点击


You should mention the exact changes done in V2.


I'm not sure the extra complexity is warranted...but we've had reason to be somewhat concerned about memory usage for images, and it occurs to me that this class almost certainly has or soon will create an image from this file so it can be displayed. Is there any reasonably straightforward way to cache the dimensions at that point so we don't have to make the image again?


The white space changes are unacceptable.  We can not review this.


1. 显示不合要求
2. 不停点击@+按钮会有问题
3. 某个按钮在获取时其他按钮仍可点击


Here I think we should use a helper function to get all the units: https://github.com/ionutbalutoiu/juju-powershell-modules/blob/25c94438ddde02a81662eba26f91e34aadcfa230/CharmHelpers/Juju.psm1#L192


Yeah, but it has an API in android.
Google play is probably simpler overall, but it tries to generalize things
like automatching for 8+ people.  So for our purposes it may add complexity.
Don't know for sure yet, it does seem to handle syncing pretty nicely.  And
with this we can play with people we aren't right next to.

On Mon, Apr 6, 2015 at 11:19 AM, Tim Clyne <notifications@github.com> wrote:

> I would be willing to bet that it is much much simpler to import Googles
> "Play Games" API than it would be to try to figure out Bluetooth for the
> infinite number of devices.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TimothyJimothy/WarGame/commit/9aae0408ae26d7a27e5eed8fca0d2172cd19b9f2#commitcomment-10585241>
> .
>


Is there any particular reason why you changed the visibility?


I'd agree with that - and no problem!


I suppose since you return false, the JavaScript will see something like a 404 error, and since there's no error handler it will just quietly never execute the promise, which is OK?


I'll test out changing test.yml to `{ role:jdauphant.sublimetext, sudo: Yes }`. That should work around the bug.

I prefer to not create a duplicate ansible galaxy role. There are too many competing roles that do the same thing already.


we can put it as an optional parameter, with no default. then test it out with !peerrelationname


1.不断点击@+按钮会有问题
2. 一个按钮在获取时其他按钮仍可点击


Same applies here, if we already have a helper


TCP （传输控制协议） ：通过序列确认和包重发机制提供可靠的数据流发送和应用程序的虚拟连接服务。TCP和IP相结合，构成了Internet协议的核心。

TCP提供应用程序和IP之间的中间层通信服务。当应用程序使用IP在网络上发送大量的数据时，只要交一个请求到TCP，由TCP来处理IP的细节。

IP是 一个字节序列（由包头和包体组成）。包头包括：目的地址和用来转发到最终目的地址的路由。包体中包括IP层要传输的数据。TCP可以检测网络冲突、流量负载均衡和其他不可预见的问题，并要求重新传输的数据包、重新组织被打乱的数据包，甚至可以帮助最小化网络冲突，减小发生其他问题的概率。


TeamCity ChameleonForms :: Continuous Integration [Build 2.0.0-PullRequest117+010](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3540&buildTypeId=bt20) outcome was **SUCCESS**
Summary: Tests passed: 3004 Build time: 0:0:0


We plan to eventually expose this however it is definitely **not** ready to be exposed yet. Eventually we will allow layered addition of linters to the core linter. @kevinushey We should probably remove this entry point so that folks don't think this API exists when it really doesn't.


@dgcliff this fixes the issue but duplicates the solr_query method - is there a way we can reference that method without having to duplicate it?


OK I was able to make this work locally with the following example.

```
>>> a = {'b': {'c': 1}}
>>> s= a['b']
>>> s['c']=2
>>> s
{'c': 2}
>>> a
{'b': {'c': 2}}
>>> s['d']=1
>>> a
{'b': {'c': 2, 'd': 1}}
```

I will have a look at the validity of this for different versions of Python,


> probably rarely used

This confused me a lot just now. Getting rid of the Logger Console is fine, but using `-min.js` rather than `-debug.js` in `hpi:run` is very unexpected.


This is called a package, not a module


:ok:


Please follow the consistent coding style!


Do you see any difference with and without this? On my computer (OS X), I don't. I'll double check in the office though.


Is this work for 5.0 ?


Which macro is better for that purpose - ?CONSOLE or ?WARN?


The commit message could use some work.


I can't find imageInfoLink anywhere else in this change so I'm unable to figure out the purpose of this. A comment might be helpful.


1. 并没有单独处理每个button
2. 不断点击@+按钮会有问题
3. 一个按钮在获取时其他按钮仍可点击
4. 没有异常处理


is there a changelog somewhere? =)


@jasonlewis should there be a semicolon here?


make has options --directory specially for calling descent Makefiles. The same applies to "src" and "clean" targets


Ok, thanks! Amazing library by the way.
I am thinking about looking into some kind of Pareto version of Factorplot.


It's not just Andika...that's only prominent because we recommend and ship it. Any font (and Microsoft ships many, to say nothing of 3rd parties) that doesn't have real Bold or Italic faces will have the same problem.


In C++11, `std::forward` is not `constexpr`.


91244204-HeatExchangerAirToAirSensibleAndLatent-Supply-Air-Outlet-Temperature-Control-problems-#4794 (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1167 of 1169 tests passed)

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


it s my project i work on 3 weeks on this project.


whaiiiiiiiiiiiiiii :scream:


I'm happy to see `ReaderUserActions` killed :)


That sounds good.


Might these be better off as constants?


Is it worth a mention that if download size is at a premium, you can also use GhoStcript on Windows (something like 13M vs about 77 IIRC). Again, if it's installed (and Reader is not) it is automatically used.


Shouldn't this be "Starlane Speed"?


Maybe throw an exception since the file is unusable in following code? That would also leave the code and take error handling to the user.


You can handle a number of additional filetypes that knitr supports using the functions defined in https://github.com/yihui/knitr/blob/206d8bb6b594c40dbb990360e7a09990b8279c45/R/pattern.R

Also doing the above will cause this function to properly work if the user changes the delimiters from their defaults using `knitr::set_pattern()`.

But maybe you want to avoid the knitr dependency, in which case you can at least grab the defaults from there.


use --std=c++11 instead of --std=c++0x, and also add -pedantic flag. And why do you use "__cplusplus=201103L" explicitly?


On Mon, Apr 06, 2015 at 08:28:08AM -0700, Laurent Raufaste wrote:
> is there a changelog somewhere? =)
`git log' should be it.

--
Eduardo Bustamante
https://dualbus.me/


Would be same as above^^


add signal handler (SIGTERM)


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1299
success: http://build.servo.org/builders/linux2/builds/1297
success: http://build.servo.org/builders/android/builds/1331
success: http://build.servo.org/builders/mac1/builds/1512
success: http://build.servo.org/builders/mac2/builds/1515
success: http://build.servo.org/builders/gonk/builds/475


From what Steve said this morning, the Linux change is not quite accurate...more like, we're switching to the system's default way of printing PDF (which internally happens to use GhostScript).


I won't accept commits with scripts in the tree, too messed up.


This breaks http://archlinuxarm.org/platforms/armv7/nvidia/trimslice and all other tegra2 based devices.


Ping?


please, use git rebase in those cases


Really close stream after write? The user maybe makes an intermediate backup copy^^


2 different ebuilds touched in the same commit, and please digest before commits and push.
Descriptions in commits usually follow a not-defined standard, read the commits history to have an idea.


Yeah, looks like google play is probably safer too

I'm good to go with that if you guys are

On Mon, Apr 6, 2015 at 11:22 AM, Anthony Quigley <aquigley1777@gmail.com>
wrote:

> Yeah, but it has an API in android.
> Google play is probably simpler overall, but it tries to generalize things
> like automatching for 8+ people.  So for our purposes it may add complexity.
> Don't know for sure yet, it does seem to handle syncing pretty nicely.
> And with this we can play with people we aren't right next to.
>
> On Mon, Apr 6, 2015 at 11:19 AM, Tim Clyne <notifications@github.com>
> wrote:
>
>> I would be willing to bet that it is much much simpler to import Googles
>> "Play Games" API than it would be to try to figure out Bluetooth for the
>> infinite number of devices.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/TimothyJimothy/WarGame/commit/9aae0408ae26d7a27e5eed8fca0d2172cd19b9f2#commitcomment-10585241>
>> .
>>
>
>


1. 不断点击@+按钮会出错
2. 如果先点击了一个按钮获取后再点击@+就无法进行该按钮后面的自动获取过程


Can you rebase and digest the ebuild as well?


Hmm, what do you mean exactly by "bad strategy"? One generated from code, or one written in another spec?

One generated from code should never be able to hit the assert, since it would hit the TypeError first. On generated from a spec might hit the assert, indeed, which is the "must ensure..." clause.


I just noticed the screenshot command got removed. On purpose?


fast-forwarding master to auto = 92b6a0b8


Canonical name for c++ compiler is CXX, and also it would be nice to use "?=" instead "=".


> or possible it could be part of the automated frame rate test.

I've found that you need about 2 seconds to reliably benchmark a certain frame size. Maybe we could have an activity that runs once at first run and benchmarks these? We could then store three resolutions that work and offer them to the user as "low", "medium" and "high" quality.


Debian recommends using apt-get for the upgrade to jessie, not aptitude (there is a difference in dependency resolution).  There is no discussion of checking disk space, it does not follow the release notes recommendation of doing a minimal upgrade before doing the full dist-upgrade, or what to do when things go wrong.

The release notes for Jessie contain an entire chapter of over 6000 words on upgrading from Wheezy.   While the release notes could be boiled down into something more concise (with a "see the release notes for more details"), this is a bit too concise to be of any value.


Honestly I totally lost feeling for coala code style, but this seems okay for me, ack.


如果先点击了某个按钮获取后再点击@+会出错


Okay, your "fix" shows that no, the code was fine before (I was missing the "-2" in the condition). The "fix" doesn't change any behavior.


sounds good to me

On Monday, April 6, 2015, aQuigs <notifications@github.com> wrote:

> Yeah, looks like google play is probably safer too
>
> I'm good to go with that if you guys are
>
> On Mon, Apr 6, 2015 at 11:22 AM, Anthony Quigley <aquigley1777@gmail.com
> <javascript:_e(%7B%7D,'cvml','aquigley1777@gmail.com');>>
> wrote:
>
> > Yeah, but it has an API in android.
> > Google play is probably simpler overall, but it tries to generalize
> things
> > like automatching for 8+ people. So for our purposes it may add
> complexity.
> > Don't know for sure yet, it does seem to handle syncing pretty nicely.
> > And with this we can play with people we aren't right next to.
> >
> > On Mon, Apr 6, 2015 at 11:19 AM, Tim Clyne <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
> > wrote:
> >
> >> I would be willing to bet that it is much much simpler to import Googles
> >> "Play Games" API than it would be to try to figure out Bluetooth for the
> >> infinite number of devices.
> >>
> >> —
> >> Reply to this email directly or view it on GitHub
> >> <
> https://github.com/TimothyJimothy/WarGame/commit/9aae0408ae26d7a27e5eed8fca0d2172cd19b9f2#commitcomment-10585241
> >
> >> .
> >>
> >
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TimothyJimothy/WarGame/commit/9aae0408ae26d7a27e5eed8fca0d2172cd19b9f2#commitcomment-10585480>
> .
>


Nevermind, but this is to a reminder to replace all `through` with `though` in the language file except for `Strike-through text`.


rax retry tests


That is great idea!


I think if someone messed with Object.prototype.value, this might fail.

The ES spec would probably handle this with some kind of Record (i.e., a spec type that doesn't correspond to a real object). But, I am not sure how to integrate that with promises, which can only hold real objects. Maybe this is related to your GenerateChunk idea, and we need to refactor more.


including this recipe here will distributes the krb5 admin and master passwords to client nodes. I think we don't want to.


saw approval from jdm
at https://github.com/Ms2ger/servo/commit/b50b21d1da6154bd87a6cca86c61cd9f553d92df


merging Ms2ger/servo/int-net = b50b21d1 into auto


LiquidSmooth/android_frameworks_base@5e93c21
LiquidSmooth/android_packages_apps_Settings@0043457
LiquidSmooth/android_frameworks_base@db5bb9e
LiquidSmooth/android_frameworks_base@791b66c
LiquidSmooth/android_frameworks_base@ce1ea8c


Are you sure you want to ignore all those exceptions and not log any of its occurrences?


Fixed. Thanks for reporting.


Yes, I was. I was actually hoping to do some more regarding this. I basically would like to move DB specific code optimizations into their own files, so that we don't have this file getting clouded up by statements like these and the ones that are like

```javascript
if(<some specific db implementation>){
optimizedQuery = <some really long optimized string for this specific db>;
}
return that.knex.raw(optimizedQuery);
```

I was hoping to make it so it would end up with a require statement like such at the top of the file

```javascript
var dbDialect = require('dbs/' + knex.client.dialect);
```

and then have this part of the file look like
```javascript
dbDialect.setExpiredTimestamp(new Date(expired).toISOString());
```

That way someone could an oracle or an mssql connector, & this file wouldn't get ugly.

However after thinking about it this weekend, I think I will split them into different PR's.


```
Effect 1: Id 6 (SPELL_EFFECT_APPLY_AURA)
BasePoints = 1
Targets (1, 0) (TARGET_SELF, TARGET_NONE)
Aura Id 23 (SPELL_AURA_PERIODIC_TRIGGER_SPELL), value = 1, misc = 0 (0), miscB = 0, periodic = 12000
Trigger spell (37545) Not found, Chance = 101
```
Unless you have a sniff that will tell you what should happen in 12 secs after the spells was cast, I have no idea on how to fix this, so just leave the error as it is.


ack


1. 不停点击@+会出错
2. 不点@+的情况下点击某个按钮会自动进行下去
3. 离开@+并没有充值随机序列，F5才可以


merging Ms2ger/servo/int-net = b50b21d1 into auto failed


Yeah, @yutakahirano and I discussed this briefly in person in Tokyo. We think symmetry is important. For example for the non-byte-stream-case, the structured clone of `const r = new RegExp(); r.foo = "bar"` does not have a `foo` property.

For the byte stream case, especially a UA-generated byte stream where we know the Uint8Arrays don't have expandos or anything, we could optimize by only creating one clone and re-using the original. But that's an optimization.

I will add a note about this in the spec.


Makes sense, ack


By inspection, I'm pretty sure that it works correctly, but it was not immediately obvious to me that duration would return `00:00:00` for executions in the `new` or `errored` states.  I don't think any code logic changes need to be made, but perhaps it's worth a comment somewhere what the intended behavior is.


jo - ez wirds guet.


What I'm missing here is the ability for OpenWebRTC to mix audio with other applications without interrupting/disrupting that audio.

This can be done with the following option:

    withOptions:AVAudioSessionCategoryOptionMixWithOthers

Without this option, e.g. audio in Bowser would possibly stop working the moment OpenWebRTC is enabled.


why did you place it in separate foleder?


1. 没有单独处理每个button
2. 先点击某个button获取后再点击@+会出错


Can you say more? After making the change, the tests were still green for me, but maybe something was skipped?

Also, I haven't personally encountered that approach to configuring tests, so it would be great if that was documented in the `README`.


Consider making `fmtstring` default to `nullptr`, and then null-check in `timestamp()`.


`./coala` prints me messages:
```
Usage of __del__ method is deprecated thus each printer needs to be closed manually now!
```
Do you intend to fix it later?


Ben,

Have you tested it before? When I was playing a bit with vmplan the tripleo picked up the custom value for the Undercloud, and made a same size flavor for all Overcloud instances, independently of vmplan settings (picked up the settings from 2nd line). I did it with vm deployment mode only, not the baremetal.


Just to make function parameter compatible with base class.


@maurycyp thanks for spotting this


thank you for this example. I haven't yet catched up with the latest Spring additions; nice to see this update. I will give it a try later this week


I think this is wrong place to fix this. I'm going to reject the PR but please make sure that there is an issue well documented so that we can tackle it


Why does this file has different extension?


Nothing


Is this ready for deployment now?


Please add a test for at least one additional format.


meaningless comment.


and again...


and again


This isn't supposed to return anything.


Hi!
It seems that

WCHAR *pUtf16ExePath = nullptr;
_get_wpgmptr(&pUtf16ExePath);

always asserts. See https://msdn.microsoft.com/en-us/library/z2zbk054.aspx


`parentLeft` is never referenced anywhere else.


fair enough


`offsetLeft` is always undefined, so `positionData.offset` was always 0.


Ésta fue la única línea que difiere del DescribeStep original.


Looks like a typo to me, given that doctrine-bridge is included twice...


Is there any reason why all this stuf isn't const?


really


I haven't tested, but I would expect you to need to use lxc_list_for_each_safe() here.


May as well make this `const`.


It's basically a workaround to a header circular dependence issue.


Also `const`.


spelling, _blah_


Yes! you are right! It's a bit archaic :anguished: ! I will make the change.


I believe this will leak the actual sorted_cgroup_settings entry itself.  The deletion loop will start at sorted_cgroup_settings->next (deleteing that one), and stop when iterator == sorted_cgroup_settings.  So sorted_cgroup_settings won't ever be freed.


高兴了。。。


@davidjb - related to http://stackoverflow.com/questions/29454544/where-to-put-automatic-setup-schema-for-colanderalchemypyramid

Are we missing `from sqlalchemy.orm import mapper` in there?


These closing ```a``` tags were removed, just not showing in this diff because I used line-by-line staging and didn't add the deleted lines.


The original issue is fixed in misoc 176b924. Please try to correct problems instead of implementing workarounds.


Use else if


Please remove this. bower_components needs to remain there because if we add third-party scripts, and enqueue them then can be version controlled.


Hum, yes, I preferred a clean makefile with features that worked.

But screenshot worked, so...  I don't know.. :hammer:


Please check allocation results (here and below).


1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
Running Seastar on OSv
======================

1. Compiling Seastar for OSv
----------------------------

Before compiling Seastar, configure it with the following command:

./configure.py --so --disable-hwloc \
    --cflags="-DDEFAULT_ALLOCATOR -fvisibility=default -DHAVE_OSV -I../osv/include" \
    --mode release

Or more easily, use the "--with-osv=..." shortcut for all the above settings:

./configure.py --mode release --with-osv=../osv

Explanation of these configuration options:
   * The "--so" option is needed so that the Seastar applications, such as
     httpd, are built as shared objects instead of ordinary executables.
     Note the "--pie" option also exists, but because of bug #352 in OSv,
     and the fact that Seastar uses thread_local in one place, these PIEs
     cannot be run on OSv.
   * The "--disable-hwloc" option is needed so that Seastar does not attempt
     to use the complex NUMA-discovery library, which isn't supported on OSv
     (and isn't relevant anyway, because VMs with NUMA are not (yet) common.
   * The "-DEFAULT_ALLOCATOR" uses in Seastar the system's regular malloc()
     and free(), instead of redefining them. Without this flag, what seems
     to happen is that some code compiled into the OSv kernel (notably,
     libboost_program_options.a) uses the standard malloc(), while inline
     code compiled into Seastar uses the Seastar free() to try and free that
     memory, resulting in a spectacular crash.
   * The "-fvisibility=default" option disables the "-fvisibility=hidden"
     option which is hard-coded into Seastar's build file. Supposedly
     "-fvisibility=hidden" provides some better optimization in some cases,
     but it also means OSv can't find main() in the generated shared object!
     So either we use "-fvisibility=default", as suggested here, or
     alternatively, make *only* main() visible - for example, to make httpd's
     main() visible add in apps/httpd/httpd.cc, before the main() definition,
     the chant: [[gnu::visibility("default")]]
   * The "-DHAVE_OSV" conditionally compiles code in Seastar that relies
     on OSv APIs (currently, this enables virtio device assignment).
     This OSv-specific code relies on some OSv header files, which is
     why the "-I../osv/include" is needed (where "../osv" is where the
     OSv source tree is open).
   * The "--mode release" is to compile only the release build. You'll
     usually not want to debug Seastar on OSv (it's easier to debug on Linux).

2. Building a Seastar-httpd module for OSv
------------------------------------------

As an example, we'll build a "seastar" module in OSv running Seastar's
httpd application.

In the OSv working directory, create a directory apps/seastar in it put:

* a link to the httpd binary in the Seastar working directory. I.e.,
  ln -s ../../../seastar/build/release/apps/httpd/httpd httpd

* A usr.manifest file, adding only this single "httpd" executable to the image:
  /httpd: ${MODULE_DIR}/httpd

* A module.py file with a default command line:

  from osv.modules import api
  default = api.run(cmdline="/httpd --no-handle-interrupt")

The "--no-handle-interrupt" is needed so that Seastar does not attempt to
use signalfd() to capture ^C. signalfd is not yet available on OSv, and
the ^C handling is not a very important feature of Seastar anyway.

Also note that currently we cannot specify "--network-stack=native", because
neither vhost nor a more efficient mechanism for "virtio assignment" is yet
complete on OSv. So we must keep the default (which is "--network-stack=posix")
which uses the OS's Posix networking APIs, which OSv fully supports.

3. Running the seastar module on OSv
-------------------------------------

To run the Seastar httpd application, using the module defined above, do,
as usual, in the OSv working directory:

$ make image=seastar -j4
$ sudo scripts/run.py -nvV

This will open an HTTP server on port 10000 of the VM. For example, if the
above creates a VM with an IP address of 192.168.122.89, we can test it as
following:

$ curl 192.168.122.89:10000
<title>this is the future</title>
Future!!

4. Debugging OSv with the Seastar application
---------------------------------------------

If you want to debug OSv (not the Seastar application) in relation to the
way it runs Seastar, you'll want the "httpd" shared object to be available
to gdb.
Unfortunately, the object lookup code in "osv syms" (translate() in loader.py)
does not seem to look for objects in apps/, so until we fix this, we need
to put a link to httpd in a well-known place, such as build/release. So
do this in the OSv top directory:
 ln -s ../../apps/seastar/httpd build/release/httpd

Note you'll need to repeat this if you do "make clean" (as "make clean"
removes everything in build/release).


I've thought about it for a while, since as you say it is a big step. However, Addresses are 32 bytes with this change rather than 24, which is kind of neat.

Anyway, if the mapping starts at a null value, how would it find the children?


Why not just use a single array of materials (probably a static one) and you can use .contains for event.getBlock()? Those lines of code are too big tbh and if they add another two tall block you shouldn't have to deal with changing all these arrays and stuff.


use const reference


we might want to make this configurable in the future (or at least document that there need to be enough space)


Note that the test failure here is an issue with the test, not SVGO. The test expects `#ff00` when it should be `#ff0`. I'll have a look at this soon when I do another round of updates.


Additionally, I would recommend the following option to optimize the audio routing on iOS for voice chat:

    setMode:AVAudioSessionModeVoiceChat

See the following Apple docs for more information: https://developer.apple.com/library/prerelease/ios/documentation/AVFoundation/Reference/AVAudioSession_ClassReference/index.html#//apple_ref/doc/constant_group/Audio_Session_Modes and https://developer.apple.com/library/ios/qa/qa1803/_index.html


fix indentation.


噢 我的心好痛 我当时没有发现这个问题QAQ


https ? not costly to do now before we forget about it :)


This shouldn't be necessary, but doesn't hurt.


I think this is fine, but it almost seems like overkill compared to a config struct.  Given a struct with sensible defaults, you can just as easily do

```go
type Config struct {
    // required options
    server string
    name   string
    // optional params
    password  string
    user      string
    tlsConfig *tls.Config
}

type Bot struct {
   config Config
   //etc
}
```

And then, for init:

```go
ircx.New(ircx.Config{..})
```

Which seems just as easy and more straightforward.  Maybe? I dunno.  I think there are different preferences and paradigms.  I, personally, like breaking configs out into a struct with exported fields for most cases because then they're documented alongside everything else in a clear way, and you can't then provide a long string of (potentially conflicting) mutators to an init function.

With this method, you could conceivably do something like:

```go
b := New(l.Addr().String(), "test-bot", WithLogin("test-user", "test-password"), WithTLS(botConfig), WithLogin("foo", "bar"))
```

Which, I mean, isn't a big deal I guess, but It's confusing. What do you think?


issue http://jsfiddle.net/pvL4jpLs/2/


Nice link, thanks.

You are right - passing objects by reference is dangerous, as they might be modified by the callee as in:
```c++
#include <iostream>

using std::cout;

void f(int& x) {
    ++x;
}

int main() {
    int x = 0;
    f(x);
    cout << x << '\n';
}
```
This program outputs 1. However, we are using const references as in:
```c++
#include <iostream>

using std::cout;

void f(const int& x) {
    ++x;
}

int main() {
    int x = 0;
    f(x);
    cout << x << '\n';
}
```
This does not compile. g++ gives the following error:
```
tmp.cpp: In function ‘void f(const int&)’:
tmp.cpp:6:7: error: increment of read-only reference ‘x’
     ++x;
       ^
```
clang++ gives the following error:
```
tmp.cpp:6:5: error: read-only variable is not assignable
    ++x;
    ^ ~
1 error generated.
```
Basically, the compiler does not allow you to modify such a variable, as it is declared constant which means read-only. On the other hand, there are both **memory** and **performance** benefits of using references. Consider the following program:
```c++
#include <iostream>
#include <vector>

#include <malloc.h>

using std::cout;
using std::vector;

void f(vector<int> v) {
    cout << "v.size: " << v.size() << '\n';

    cout << "stats 3\n";
    malloc_stats();
}

int main() {
    cout << "stats 1\n";
    malloc_stats();

    vector<int> v(1048576);

    cout << "stats 2\n";
    malloc_stats();

    f(v);

    cout << "stats 4\n";
    malloc_stats();
}
```
This outputs the following:
```
stats 1
Arena 0:
system bytes     =          0
in use bytes     =          0
Total (incl. mmap):
system bytes     =          0
in use bytes     =          0
max mmap regions =          0
max mmap bytes   =          0
stats 2
Arena 0:
system bytes     =          0
in use bytes     =          0
Total (incl. mmap):
system bytes     =    4198400
in use bytes     =    4198400
max mmap regions =          1
max mmap bytes   =    4198400
v.size: 1048576
stats 3
Arena 0:
system bytes     =          0
in use bytes     =          0
Total (incl. mmap):
system bytes     =    8396800
in use bytes     =    8396800
max mmap regions =          2
max mmap bytes   =    8396800
stats 4
Arena 0:
system bytes     =          0
in use bytes     =          0
Total (incl. mmap):
system bytes     =    4198400
in use bytes     =    4198400
max mmap regions =          2
max mmap bytes   =    8396800
```
Notice stats 3: the memory for the vector is copied every time, so in this current setting that means 4MB of memory for every function call. Now change the function signature to
```c++
void f(const vector<int>& v)
```
Running the program gives:
```
stats 1
Arena 0:
system bytes     =          0
in use bytes     =          0
Total (incl. mmap):
system bytes     =          0
in use bytes     =          0
max mmap regions =          0
max mmap bytes   =          0
stats 2
Arena 0:
system bytes     =          0
in use bytes     =          0
Total (incl. mmap):
system bytes     =    4198400
in use bytes     =    4198400
max mmap regions =          1
max mmap bytes   =    4198400
v.size: 1048576
stats 3
Arena 0:
system bytes     =          0
in use bytes     =          0
Total (incl. mmap):
system bytes     =    4198400
in use bytes     =    4198400
max mmap regions =          1
max mmap bytes   =    4198400
stats 4
Arena 0:
system bytes     =          0
in use bytes     =          0
Total (incl. mmap):
system bytes     =    4198400
in use bytes     =    4198400
max mmap regions =          1
max mmap bytes   =    4198400
```
What?! No memory used at all? Well, this is not true, there is some bytes stored on the stack to save the reference, but since ```memory_stats()``` shows us only heap memory it is not reflected here. But still, ```8B``` compared to ```4MB``` is a difference. Okay, maybe we don't care about a few megabytes, let's look at performance:
```c++
#include <iostream>
#include <vector>

#include <malloc.h>

using std::cout;
using std::vector;

void f(vector<int> v) {
}

int main() {
    vector<int> v(1048576);

    const int ITER = 1 << 10;
    for (int i = 0; i < ITER; ++i) {
        f(v);
    }
}
```
Here, I removed the output from the function call, because we call it ```ITER``` times (```1024```). Measuring the time the program takes gives the following:
```
real	0m0.458s
user	0m0.450s
sys	0m0.008s
```
Let's see how the program performs if we use references. Change the function signatures again (like before). The output of ```time``` is:
```
real	0m0.008s
user	0m0.008s
sys	0m0.000s
```
Wow, is this correct? Let's increase the ```ITER``` to ```1 << 20``` (a million calls to the function). The output of ```time``` is:
```
real	0m0.009s
user	0m0.005s
sys	0m0.005s
```
Still no measurable time. Let's increase the ```ITER``` to ```1 << 30``` (a billion calls to the function). The output of ```time``` is:
```
real	0m2.194s
user	0m2.187s
sys	0m0.008s
```
Let's go back to using a copy of the vector and see how many calls we can make for the same amount of time. Setting ```ITER``` to ```1 << 13``` (8000 calls):
```
real	0m3.050s
user	0m3.046s
sys	0m0.000s
```
So as a conclusion it is more than 100 times faster to pass a variable by reference than by value.

One last thing: you might think our vectors are not as long as 1M elements. You're right. Let's try with a vector of 4 elements:
```c++
#include <iostream>
#include <vector>

#include <malloc.h>

using std::cout;
using std::vector;

void f(vector<int> v) {
}

int main() {
    vector<int> v(4);

    const int ITER = 1 << 24;
    for (int i = 0; i < ITER; ++i) {
        f(v);
    }
}
```
Output of ```time```:
```
real	0m2.340s
user	0m2.342s
sys	0m0.000s
```
Using references:
```c++
#include <iostream>
#include <vector>

#include <malloc.h>

using std::cout;
using std::vector;

void f(const vector<int>& v) {
}

int main() {
    vector<int> v(4);

    const int ITER = 1 << 30;
    for (int i = 0; i < ITER; ++i) {
        f(v);
    }
}
```
Output of ```time```:
```
real	0m2.149s
user	0m2.143s
sys	0m0.004s
```
So we can still make more than 30 calls more for the same time if we are using references rather than values.

So there is absolutely no reason to pass objects by value if you can achieve the same thing by passing them by reference (there are exceptions), unless you like to save the typing a "```const```" and an "```&```" :)

(-: Stan


why not to use vector?


use const reference.


不过还是谢谢指出啦～


Editado e em fase final


          case (colRef@ColumnRef(_, SoQLFloatingTimestamp),
                cast@FunctionCall(MonomorphicFunction(TextToFloatingTimestamp, _),
                Seq(StringLiteral(ts, _)))) =>
            for {
              parsedTs <- SoQLFloatingTimestamp.StringRep.unapply(ts)


I've duplicated Profile twice. And left out the Create Visualforce Component. Will fix in next commit.


I'd also change "Set the amount of idle time required before displaying the screensaver." to something like:

"Set the time the application waits for any activity to occur before displaying the screensaver."


I've pulled in one from gentoo directly along with its manifest .
repoman is biting me in the but,  badsamba also not wanting to digest i've put it and the yara bumped templates in comm overlay for now.  going to kill this pull and kill yara bumbs for now ...


Clever :)


@chrissprague please use descriptive branch names.  It took me a full day to realize that the numbers are referencing issue numbers.


There is std::generate_n function in algorithm header.


Agreed, and ReplyWith is much better than either Process or Return for this purpose. I like it so much I've put a card on the technical debt Trello to improve the others!


I barrel roll every day.


Thanks xD.


Hey FYI:

http://raymondhill.net/ublock/popup.html

Something in this commit causes #7 to be broken where I see:

```
Example Domain

This domain is established to be used for illustrative examples in documents. You may use this domain in examples without prior coordination or asking for permission.

More information...
```

When I git checkout to the previous commit to this, the popup is blank.  #6 must have been broken a lot longer than this and shows the same page.  Let me know if you guys see the same thing.


Was this really the reason why urls weren't working?? D:


This example is actually different than the iteration example.  So removing it completely is not what I would recommend.  Rather I think the following can be added to the new combined example so we continue to show how to reference a child by name:

    tenantMo.BD['default'].dn

That will print out the MO dn to the stdout and provide an example of accessing a child by name.


I can't merge a generic except statement.


Division with 0 is possible. Example: number divided by 0 always equals the same number and number divided by the same number equals 0.


If you need the value from the center of an element regardless of FE type, this is how we do it.


there remains an error in checked files which cannot be fixed until I get answer for my question (yKomarov). Comments will be deleted (as variables that are to be renamed) only when the problem gonna be solved, right now it is not final version thus I dont see any reason to keep code clean. getITube will be removed after the problem is fixed 'cause it demands some architecture reorganisation. It is not done 'cause the first priority task is to make it all work. That was a comment for your comment for previous commit.


Thanks!


instead -> **instead**


How can I make it into a work-able .jar file so I can put in plugins folder? I've tried rename .zip to .jar, convert .zip to .jar using online converter, and extracted player head v3.9 into a folder and paste all the data in the zip file into that folder and zip it again and rename to .jar, but none of them is working. When I put the self-made .jar into my server /plugins folder it just doesn't load. Can anyone teach me how? I'm using win 8 btw.


there is std::for_each...


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


I think it would be better to put this in a middleware.py file (or something like that) instead of a long, very specific, one-time use filename.


Je contribue, je contribue (y'a pas moyen que ma PR ne soit pas merged).

Pourquoi tu regardes GitHub un lundi de paques ?

2015-04-06 15:28 GMT+02:00 Bruno <notifications@github.com>:

> Ca bosse le lundi de Pâques ??
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/idlead/scikit-learn/commit/acdb400c288fd15a9e28d3d887735ada398c71f6#commitcomment-10583865>
> .
>


Good to know, thanks @ben-eb.


(Refer to my other comment re: middleware.py.)


This actually does still allow negative ``sigma_t``. The conditional checks whether the total cross-section is "close enough" to zero (*e.g.*, within the interval (-1E-10, 1E-10)) and if so, assigns it to ``ZERO_SIGMA_T``. Hence, to set a negative total / transport cross-section, it must be < -1E-10.


Maybe add line continuation here:

    >>> myClassQuery.propFilter = 'eq(fabricPathEpCont.nodeId, \
    ...                               "{0}")'.format(nodeId)
    >>>


      case fc@FunctionCall(MonomorphicFunction(fnType, bindings), _)
          if (fnType == Gte || fnType == Lt) && bindings.values.forall(_ == SoQLFloatingTimestamp) =>


up to 3db406f


What browser? I tried with Chromium and it worked as expected.


:+1:  my french background...


@RaineForest Nice.


Well, nested callig is one of the advantages of modern PHP... I found it nice this way but you are free to change it


@akatrevorjay OK. So I've been trying to diagnose https://github.com/akatrevorjay/pyzfscore/issues/10 but installing an "editable" version of pyzfscore from pypi and ran into exactly the problem you foresaw. It seems that if you just install from pypi normally, it all works fine as the sources have been pre-compiled.

I've read the cffi docs, and it seems that the ``get_extension()`` call is the way you're supposed to do it. The reason I tried to not use this, is it causes the cyclic dependency of cffi before it's installed.

Reverting this is probably for the best. I'll "fix" the setup.py and make a new PR and release.


@twalling soon


When we process a Logout Request, the Service Provider must send if the SAML Request is valid a SAML Response, so the method build the HTTP-Redirect values that includes the SAML Response.

When we process the SAMLResponse, we want to validate it, but after that there are a lot of actions that could be done, that is why we let the developer that uses the code the ability to do a redirection or other action.
In the [django-demo](https://github.com/serpulga/python-saml/blob/django-fixes/demo-django/demo/views.py#L69) yo can see that after process the SLO we do the redirection only in the case of sending a SAMLResponse (Logout Response), if not we show the "success_slo" page.

I don't see any benefit with this change.

P.S Notie that we do the same in the php-saml toolkit:
https://github.com/onelogin/php-saml/blob/master/lib/Saml2/Auth.php#L128
And I created several SAML plugins without problems.


It is incorrect to install the binaries directly. We want to build, release and install the Pacman packages instead: https://github.com/lygstate/build-extra/blob/220ffc10a00c66cc1796fad3dceb7fbc55ba2f08/installer/HowToRelease.txt#L56-L63


Вот чтобы так не писать, луче использовать let!


It seems to work as-is:
```
# let x : (int,string) Rwo_core2.Result.t = Core.Std.Ok 1;;
val x : (int, string) Result.t = Rwo_core2.Result.Ok 1
```


ㄴ


Sounds like you are (much) more familiar with audio management on iOS than I am. I'd appreciate a PR! Thanks.

Another thing I have been thinking about is if we should have an API for setting which speaker that should output audio. Apps could the decide if they support handsfree mode.


Обычно, при тестировании методов делают `describe` с его названием и внутри `describe` уже тесты и нужные данные. Причем, для инстанс-методов используют `#` в названии метода, а для методов класса - точку. Это соглашение в ruby. В твоем случае будет `describe "#make_best"`


@spastorino note this doesn't help compatibility with older Rails -- we're not calling the method: we're overriding it


There was a merge issue, so I fixed it


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


Correction: nut.bar.Paint now uses the HUDShouldPaintBar hook to actually decide if the bar should be painted.


Bah, I lied; was moving too quickly, sorry.  Your change actually did NOT make it out with the deploy today. (I neglected to pull master before doing the deploy.)  It'll be a few weeks before this gets out.

SORRY!  :S


Use the parser's error mechanism here rather than throwing an exception. That means changing the return time to `Parser [Long]`, and I think you'll just want to fold this match expression into `number` below to achieve that.


Just wondering about the generalisation of ARG.IT

It is both SN and RN, how is this not destroying self as well as other?


Is it safe to call drmModeFreeEncoder with NULL?


When you commit, you no longer get a requests response object back.  Instead you now get a list (usually an empty one unless you ask the apic to send more info):

    >>> cr = ConfigRequest()
    >>> cr.addMo(b)
    >>> modir.commit(cr)
    []
    >>>


抓到一枚程序媛 lol


When you commit, you no longer get a requests response object back. Instead you now get a list (usually an empty one unless you ask the apic to send more info):

    >>> cr = ConfigRequest()
    >>> cr.addMo(b)
    >>> modir.commit(cr)
    []
    >>>


Haha, it seems [you got some inspiration](https://github.com/IonicaBizau/I-Love-Octocats)! :smile:


nice!


Duplicate line.


@BenBernardCIS just noticed that the first two constructors for this command have pretty similar signatures, but do different things. If someone means to call the first one (using inches), but they pass an integer value, the class would use the 2nd constructor, correct? We may want to think about whether we truly need both, or if there is a way to make them less similar.


Yeah that sounds good to me too

On Mon, Apr 6, 2015 at 11:38 AM, Tim Clyne <notifications@github.com> wrote:

> sounds good to me
>
> On Monday, April 6, 2015, aQuigs <notifications@github.com> wrote:
>
> > Yeah, looks like google play is probably safer too
> >
> > I'm good to go with that if you guys are
> >
> > On Mon, Apr 6, 2015 at 11:22 AM, Anthony Quigley <aquigley1777@gmail.com
> > <javascript:_e(%7B%7D,'cvml','aquigley1777@gmail.com');>>
> > wrote:
> >
> > > Yeah, but it has an API in android.
> > > Google play is probably simpler overall, but it tries to generalize
> > things
> > > like automatching for 8+ people. So for our purposes it may add
> > complexity.
> > > Don't know for sure yet, it does seem to handle syncing pretty nicely.
> > > And with this we can play with people we aren't right next to.
> > >
> > > On Mon, Apr 6, 2015 at 11:19 AM, Tim Clyne <notifications@github.com
> > <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
> > > wrote:
> > >
> > >> I would be willing to bet that it is much much simpler to import
> Googles
> > >> "Play Games" API than it would be to try to figure out Bluetooth for
> the
> > >> infinite number of devices.
> > >>
> > >> —
> > >> Reply to this email directly or view it on GitHub
> > >> <
> >
> https://github.com/TimothyJimothy/WarGame/commit/9aae0408ae26d7a27e5eed8fca0d2172cd19b9f2#commitcomment-10585241
> > >
> > >> .
> > >>
> > >
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/TimothyJimothy/WarGame/commit/9aae0408ae26d7a27e5eed8fca0d2172cd19b9f2#commitcomment-10585480
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TimothyJimothy/WarGame/commit/9aae0408ae26d7a27e5eed8fca0d2172cd19b9f2#commitcomment-10585542>
> .
>


Nice one!!!
 Very good explanation, please let's go through this and other great tips
during our next RoboCup meeting. At the end of the day, we do want our
robots to be as fast as possible, and designing it well and efficiently
from the ground up will go a long way.

 Cheers,
      Alex

On 6 April 2015 at 16:58, Stan Manilov <notifications@github.com> wrote:

> Nice link, thanks.
>
> You are right - passing objects by reference is dangerous, as they might
> be modified by the callee as in:
>
> #include <iostream>
> using std::cout;
> void f(int& x) {
>     ++x;
> }
> int main() {
>     int x = 0;
>     f(x);
>     cout << x << '\n';
> }
>
> This program outputs 1. However, we are using const references as in:
>
> #include <iostream>
> using std::cout;
> void f(const int& x) {
>     ++x;
> }
> int main() {
>     int x = 0;
>     f(x);
>     cout << x << '\n';
> }
>
> This does not compile. g++ gives the following error:
>
> tmp.cpp: In function ‘void f(const int&)’:
> tmp.cpp:6:7: error: increment of read-only reference ‘x’
>      ++x;
>        ^
>
> clang++ gives the following error:
>
> tmp.cpp:6:5: error: read-only variable is not assignable
>     ++x;
>     ^ ~
> 1 error generated.
>
> Basically, the compiler does not allow you to modify such a variable, as
> it is declared constant which means read-only. On the other hand, there are
> both *memory* and *performance* benefits of using references. Consider
> the following program:
>
> #include <iostream>
> #include <vector>
>
> #include <malloc.h>
> using std::cout;using std::vector;
> void f(vector<int> v) {
>     cout << "v.size: " << v.size() << '\n';
>
>     cout << "stats 3\n";
>     malloc_stats();
> }
> int main() {
>     cout << "stats 1\n";
>     malloc_stats();
>
>     vector<int> v(1048576);
>
>     cout << "stats 2\n";
>     malloc_stats();
>
>     f(v);
>
>     cout << "stats 4\n";
>     malloc_stats();
> }
>
> This outputs the following:
>
> stats 1
> Arena 0:
> system bytes     =          0
> in use bytes     =          0
> Total (incl. mmap):
> system bytes     =          0
> in use bytes     =          0
> max mmap regions =          0
> max mmap bytes   =          0
> stats 2
> Arena 0:
> system bytes     =          0
> in use bytes     =          0
> Total (incl. mmap):
> system bytes     =    4198400
> in use bytes     =    4198400
> max mmap regions =          1
> max mmap bytes   =    4198400
> v.size: 1048576
> stats 3
> Arena 0:
> system bytes     =          0
> in use bytes     =          0
> Total (incl. mmap):
> system bytes     =    8396800
> in use bytes     =    8396800
> max mmap regions =          2
> max mmap bytes   =    8396800
> stats 4
> Arena 0:
> system bytes     =          0
> in use bytes     =          0
> Total (incl. mmap):
> system bytes     =    4198400
> in use bytes     =    4198400
> max mmap regions =          2
> max mmap bytes   =    8396800
>
> Notice stats 3: the memory for the vector is copied every time, so in this
> current setting that means 4MB of memory for every function call. Now
> change the function signature to
>
> void f(const vector<int>& v)
>
> Running the program gives:
>
> stats 1
> Arena 0:
> system bytes     =          0
> in use bytes     =          0
> Total (incl. mmap):
> system bytes     =          0
> in use bytes     =          0
> max mmap regions =          0
> max mmap bytes   =          0
> stats 2
> Arena 0:
> system bytes     =          0
> in use bytes     =          0
> Total (incl. mmap):
> system bytes     =    4198400
> in use bytes     =    4198400
> max mmap regions =          1
> max mmap bytes   =    4198400
> v.size: 1048576
> stats 3
> Arena 0:
> system bytes     =          0
> in use bytes     =          0
> Total (incl. mmap):
> system bytes     =    4198400
> in use bytes     =    4198400
> max mmap regions =          1
> max mmap bytes   =    4198400
> stats 4
> Arena 0:
> system bytes     =          0
> in use bytes     =          0
> Total (incl. mmap):
> system bytes     =    4198400
> in use bytes     =    4198400
> max mmap regions =          1
> max mmap bytes   =    4198400
>
> What?! No memory used at all? Well, this is not true, there is some bytes
> stored on the stack to save the reference, but since memory_stats() shows
> us only heap memory it is not reflected here. But still, 8B compared to
> 4MB is a difference. Okay, maybe we don't care about a few megabytes,
> let's look at performance:
>
> #include <iostream>
> #include <vector>
>
> #include <malloc.h>
> using std::cout;using std::vector;
> void f(vector<int> v) {
> }
> int main() {
>     vector<int> v(1048576);
>
>     const int ITER = 1 << 10;
>     for (int i = 0; i < ITER; ++i) {
>         f(v);
>     }
> }
>
> Here, I removed the output from the function call, because we call it ITER
> times (1024). Measuring the time the program takes gives the following:
>
> real    0m0.458s
> user    0m0.450s
> sys 0m0.008s
>
> Let's see how the program performs if we use references. Change the
> function signatures again (like before). The output of time is:
>
> real    0m0.008s
> user    0m0.008s
> sys 0m0.000s
>
> Wow, is this correct? Let's increase the ITER to 1 << 20 (a million calls
> to the function). The output of time is:
>
> real    0m0.009s
> user    0m0.005s
> sys 0m0.005s
>
> Still no measurable time. Let's increase the ITER to 1 << 30 (a billion
> calls to the function). The output of time is:
>
> real    0m2.194s
> user    0m2.187s
> sys 0m0.008s
>
> Let's go back to using a copy of the vector and see how many calls we can
> make for the same amount of time. Setting ITER to 1 << 13 (8000 calls):
>
> real    0m3.050s
> user    0m3.046s
> sys 0m0.000s
>
> So as a conclusion it is more than 100 times faster to pass a variable by
> reference than by value.
>
> One last thing: you might think our vectors are not as long as 1M
> elements. You're right. Let's try with a vector of 4 elements:
>
> #include <iostream>
> #include <vector>
>
> #include <malloc.h>
> using std::cout;using std::vector;
> void f(vector<int> v) {
> }
> int main() {
>     vector<int> v(4);
>
>     const int ITER = 1 << 24;
>     for (int i = 0; i < ITER; ++i) {
>         f(v);
>     }
> }
>
> Output of time:
>
> real    0m2.340s
> user    0m2.342s
> sys 0m0.000s
>
> Using references:
>
> #include <iostream>
> #include <vector>
>
> #include <malloc.h>
> using std::cout;using std::vector;
> void f(const vector<int>& v) {
> }
> int main() {
>     vector<int> v(4);
>
>     const int ITER = 1 << 30;
>     for (int i = 0; i < ITER; ++i) {
>         f(v);
>     }
> }
>
> Output of time:
>
> real    0m2.149s
> user    0m2.143s
> sys 0m0.004s
>
> So we can still make more than 30 calls more for the same time if we are
> using references rather than values.
>
> So there is absolutely no reason to pass objects by value if you can
> achieve the same thing by passing them by reference (there are exceptions),
> unless you like to save the typing a "const" and an "&" :)
>
> (-: Stan
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/edinferno/edinferno/commit/e31cbec30ac70f996e3085dbb41110aebbddeeb9#commitcomment-10585903>
> .
>


look for drop High Five part 5 http://l2db.info/


@khipkin lol, yes, I agree. Just couldn't think of name for the file at the time


I think the `Table` class will generally be useful, so it should be lifted out of SchemaParser. Then you can add a method to `Schema` that adds a table.


we'll have to update the whole gamemode to this new currency.


This is not right. SuiteSparse version should probably be 4.2+. You are probably referring to the CHOLMOD version. Each component within SuiteSparse has its own version number.


I think following should be supported:
1. DSCP - Configure a value, copy from inner, map based on payload DSCP/802.1p
2. TTL - Configure a value, copy from payload header
3. Flow Label - Configure a value, mapping


uBlock Core


@shoyer mentioned before (and he was right) that this really should be:

```
if type(c) != np.ndarray:
    c = np.asarray(c)
return c
```

`asarray` always returns `ndarray`, even if the input is an `ndarray` subclass.


Please take some time to write meaningful commit messages. I am sure that not even you would understand what this commit is about, six months from now.

We not only want commit messages [in a certain form](http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html), and not only want [a sign-off](http://stackoverflow.com/questions/1962094/what-is-the-sign-off-feature-in-git-for), too, but also want them to contain all relevant information that cannot be deduced from the diff itself.

In this particular case, the following questions are left wide open by the commit message:
- what problem does this fix?
- what problem in `gettext` are you referring to?
- which #65 do you refer to? (I am *guessing* that you mean https://github.com/git-for-windows/git/issues/65, but that is only a guess, and I would have trouble thinking of that link so much as two weeks from now)
- why does `/share/locale/` have to be *excluded*?
- what alternative solutions did you try?
- what better solutions could you think of?


thats wrong, it is used!


It is SuiteSparse 4.1.0


good


I don't like this.
Leave tournament as it was and add a field `tournamentStats` to the `data` structure.

~~~go
		data := struct {
			User         mdl.UserJson          `json:",omitempty"`
			Teams        []mdl.TeamJson        `json:",omitempty"`
			TeamRequests []mdl.TeamRequestJson `json:",omitempty"`
			Tournaments  []tournament          `json:",omitempty"`
			TournamentStats []stats           `json:",omitempty"`
			Invitations  []mdl.TeamJson        `json:",omitempty"`
		}{
~~~


I think the tests will be clearer if they are organized as:

"Parsing a number should" - {
  "handle decimal" ...
  "handle hexadecimal" ...
  "yield an error" ...
}

"Parsing an identifier should" - ...


perhaps `task[0] in (getitem, getarray)`?


Hum, is all this supposed to be in this commit ?


I dont get you question.
One is title other is text under it in pedia.
https://github.com/Cjkjvfnby/freeorion/blob/master/default/encyclopedia.txt#L204


@khipkin it's not necessarily that the urls weren't working; the issue was that if you had a trailing "/" character at the end of the URL, the `updateStyle()` js function (see below) wouldn't properly recognize it because the string we were expecting in `updateStyle` was not _exactly_ the same as the URL coming in. Additionally, without the `/$` I added, you could straight up type anything in as long as the first part of the string read "resources", or "pictures" (etc.) So like, `csc.cs.rit.edu/resources/whatever` was being registered as a valid URL, redirecting you to the resources page, but `updateStyle` didn't highlight the active page correctly.

Make sense? That's what I made of it.


Something I did made this code nonfunctional.


Is there a reason you don't support `(getarray, (getitem, x, a), b)`


Imho is not possible. Proof:
4 : 2 = 2  =>  2 * 2 = 4
0 : 2 = 0  => 2 * 0 = 0 (actually possible if 0 is the first parameter)
2 : 0 = impossible  =>  0 * (?) = 2


Would it be possible to make the tolerances exactly identical. In @trigtech/cumsum.m the tolerance is
based on the coefficients not the sum command. Since you're assuming it's a trigfun here, could you not do the exact same test on the coefficients?


This should be called `SchemaParserSpec`, since it's testing `SchemaParser`.


Yes.


If it's functioning correctly - How about:
```
public static final String IT = "(?<it>(sn|rn|it|((this|that) (artifact|creature|enchantment|instant|land|planeswalker|permanent|sorcery)))";
```

To allow scripts to not need RN


But GAP events don't tell you that any data is lost at all or that there is an actual error. There might just not be data continuously.
If you want to handle something in videorepair, handle the packet-lost events. That would make more sense :)


WTF??? this is server->client message(UserAvailabilityChangedIndication)


Of course. Fixed in cd653bb04ffe00cc12b6e3a732332f5d70a5873f.


BSD style comments, here and below:

    /*
     * multi-line comment
     */


I checked and it works because the definition in `Core.Result.t` already rexports the type:
```
    type ('ok, 'err) t =
      ('ok, 'err) Core_kernel.Result.t =
        Ok of 'ok
      | Error of 'err
```
If you used `Core_kernel.Result` instead, it would not work:
```
type ('ok, 'err) t = Ok of 'ok | Error of 'err
```
would not be shared with `Core_kernel.Result.t`.


it doesn't fixed issue


@stefanalund I've been poking around in gst-plugins-good's sys/osxaudio code and am wondering if it isn't a better idea to move this code into there. What do you think?


ATTR_INTF is set as IP? Could we make the description and the type consistent.


WTF??? what is purpose of this message?


@khipkin right, I should have informed contributors of the style for branch names I was going to be using. Although I want to keep using this style because it mandates the use of the issue tracker (which helps the repo -- and its problems -- be more organized) plus the taskbranches themselves will be very clear what work is to be done in them, so long as people know why TBs have the name they do.

This is the style we used at my first co-op and I found it effective. Can we keep doing it this way?


Who and why uses this flag?


I guess that there is a bug fix here, intermingled with a rearranging of lines, which makes it doubly hard to see either.

The commit message would have to do provide a convincing argument why rearranging the lines is desirable.

As to the HTML pages, I verified and fixed the bug myself. Please have a look at e3cfa57 and imitate the verbosity and amount of content in future commit messages, as well as the sign-off and the columns/line limit.


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


I would rather not have this param, but modify the semantics of uri_prefix:

    /foo
    /foo=/
    /foo=bar

(forward as is, trim, rewrite /foo to /bar)


@escowles, it looks like "user" is never used. Am I missing something?
If indeed it is not used, we should probably remove the field.


server will be informed by client, that client wait server response. Genious!

Server know that, client is waiting, if it is waiting.


I dont'know what to say...


Definitely; it should also expect a `String`, not a `URL`, to handle invalid addresses properly.

For example, empty string will throw a `ConversionException`


Can you also add Hoopla to `README.md` and `access-granted.html`?


Why not return T?


I just created a new account for Skype, instead of using my Facebook account. But my name and email is still the same.


@damirius

Ok so make sure to remove any code not used to prevent building up dead code.
Thank you.


"Java" --> "Python" (here and other places)


On Wed, Apr 01, 2015 at 04:18:35AM -0700, Dimitri John Ledkov wrote:
> "However, it looks to me like it sets the new file's perms to be the perms on the open file." - I don't follow this comment. If commonio_open is called with O_CREATE option, yet said database does not exist on disk, the database is "opened" but no file is created on disk

Oh, I see.  I was worried that umask is omly set at commonio_close() (well, in
fopen_set_perms) but I see that commonio_open doesn't open with O_CREATE.
So this could be fine.


Line not needed...? (Already within same)


According to wikipedia (see http://en.wikipedia.org/wiki/Digital_Living_Network_Alliance) it's `DLNA` and not `dlna` even though the logo says `dlna`.


Resolved in
https://github.com/mrocklin/dask/commit/3af964101472f39a72aad1cc2b6ce9ae9beba078

On Mon, Apr 6, 2015 at 9:20 AM, Jim Crist <notifications@github.com> wrote:

> Is there a reason you don't support (getarray, (getitem, x, a), b)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mrocklin/dask/commit/338d5570a5eb145fa95033b13dab9c89f210aed2#commitcomment-10586255>
> .
>


You don't like 4 spaces? :-) .


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


@chrissprague Please don't use this style. Anyone should be able to look at the branch and know why it exists and what it is trying to do.  You shouldn't have to reference a million different places to figure that out.  Just add a comment in your commit(s) referencing the issue(s) addressed.  This acts as a direct link to the issue, AND then the branch names are descriptive.

We shouldn't use gimmicky, convoluted means to get people to use the issue tracker... anyone who is contributing to the CSC's website *will* use the issue tracker.

Since it seems that we both feel very strongly about this, maybe there should be a vote?  Either way, we should be consistent and put something in the README for future reference.


yes its possible to do in that way but still want to do it in different way :) give me some time.


public inheritance represents "IS" relation, in other words, Point3D IS Point2D... Does it makes sense?


Yeah, now just accept my invitation...


No, it shouldn't assert. pValue in docs is &pUtf16ExePath and cannot be
null. Do you really run the code?

понедельник, 6 апреля 2015 г. пользователь SmartCat написал:

> Hi!
> It seems that
>
> WCHAR *pUtf16ExePath = nullptr;
> _get_wpgmptr(&pUtf16ExePath);
>
> always asserts. See https://msdn.microsoft.com/en-us/library/z2zbk054.aspx
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cocos2d/cocos2d-x/commit/653bf1549d930ef4026a07716c6d4bbd7430e258#commitcomment-10585721>
> .
>

--
Vladimir Timofeev <vovkasm@gmail.com>


Is this the wrong way around? -It looks to me like it's setting it to NT_INT if the field contains a point and is an integer. Shouldn't it be NT_FLOAT in that case? Isn't it only NT_INT if n is an integer and does *not* contain a point? Or am I missing some trick in the 'pick random' test below...?


@LK4D4 let's add `stringutils.Int64()` function, I did that for `Bool` in one of my branches. WDYT?


Bumped version to v0.6.7


@tpetr it looks like the parse method isn't being fired when we get a 400 because the response content-type is text, not json. A potential workaround could be to just listen for `change:present` on the ajaxError model and then run a custom method in the model with what is currently in the parse method. I can take a run at this.


If I'm right about above, then shouldn't this be returning an integer only if both are NT_INT? - i.e. "&&" rather than "||"...?


`return directory.exists() && directory.isDirectory()` ?


I think this is a good approach. Our other conference types will work fine with the change.

Go ahead and leave the {active?} check off of the permission. The controller action should check that and return an error, though. We don't want to allow closing an ended conference on the backend or api users might overwrite the conference {ended_at} erroneously.


vote for 3


This should not use "vagrant" this should use /var/www/assets.


Either way is not a big deal, but this block can actually go before line 233, because flagVals is a struct of pointers (wouldn't work otherwise), all it's fields will get their values once f1.Parse() is called.


发现一个错别字：/s/多余/多于


ok, take your time.  :+1:


Why don't you implement an efficient `puts` instead?


@chrissprague ahh, you're right! I never realized that... nice catch!  I'll try out your changes tonight, but the URL resolution on the live site is definitely weird in the way you described.


Good to know. Thanks again for your help! :smile:


hm, reviewed your code and well its good idea to execute queries directly in loop - it does not appends stringstream for every achievement, every query tooks aproximally 25-30 symbols. But still want to play with single stringstream, just i want to know why mysql doesn't accepts query whyle query syntax is good to use.


Awesome. That's a great idea!


Does $(document).ready() break anything? Its actually good practice to make sure the dom is ready before selecting.


Let's debate with the team....


Also why did you remove the metadata?


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


@dbarashev fyi `exists` call is redundant: http://docs.oracle.com/javase/7/docs/api/java/io/File.html#isDirectory()


Woo, this is pretty awesome. However, it's considered bleeding edge. We should make sure @oldfoagie uses a supported browser ([as listed in the css docs](http://www.w3schools.com/cssref/css3_pr_perspective.asp))


if you pass configuration as an argument, you will be able to make this method static and initialize write in constructor like `myWriter = createWriter(configuration)`


I think that the commands should be unified to `create:something`. Right now you have `create:user` and `role:create` (backwards).

Also, I think that each command's class name should reflect the order used, so `create:user` should be `CreateUserCommand`, as you had it before.


No problem, I suggested a less generic name only because users can only installed plugins with the same name once in CLI.


after that will rewrite some multimaps data containers in objectmgr


@awoods: here's a PR to remove just that unused field: https://github.com/fcrepo4/fcrepo4/pull/764


Still work in progress here...! :)


Right. I will fix this ASAP. It should be INT_MAX.


Can this code be in the routine so we don't copy it all over?


Can the name of this be `allowed_role_types` or something?


No. Its right with ?= ....


Thank you for including an alt tag. It's very important for SEO. Remember that the actor trying to consume this img tag can't "see" the picture to extract distinguishing features. I always try to write them from the perspective of a search engine or a blind person.


Yeah, great point.  The routine should have a role_id argument vs the current role then it can do the lookup if it needs to.


Otherwise looks like it is the type of the passed in role


In this case no - as soon as jQuery is ready we can fire this off. You're right though, sometimes you do want to wait till DOM and even `$(window).load()` too.


Right, sorry :smile:


The seed should be random by default


@ykhodorkovskiy


Meta data is in a local commit still - my bad!


But I'll settle with entity:verb here, nevertheless.


std::array


oh no. not me i am not the author. all credit to #flar2


Back to μBlock (muBlock), I guess ?
Or simply just "gorhill's uBlock" is fine.


`ourInstance`
But in fact, unless you use IndexerConfiguration in many places (which doesn't seem to be the case), I suggest to just create a new instance and pass it to any client where it is required. There are a few reasons to do it:
1. You explicitly declare a dependency: you can't create an instance of client or call its method without creating an instance of IndexerConfiguration
2. Provided that IndexerConfiguration has an abstract interface, you can pass another implementation or mock object to test the logic of the client


why do we not create these all under the /var/www directory?


Yeah, that makes sense. Any thoughts on whether this would play nicely with an API (in owr) for setting the active speaker?


Did you intend to remove the JsonProperty attribute?


oh no. not me i am not the author. some mistake


Using bare js can be verbose and annoying to write. Using JQuery is so much easier.
`document.getElementById("foo")`   ->  `$("#foo")`


oh no. not me i am not the author. some mistake. all credit to #flar2


@mostaphaRoudsari ,
Thanks for the input and I agree.  It's probably best to let the Ladybug components be.  I will try to implement the increased timestep on the comfort map after I implement the ability to view the outdoor temperature distribution off of the E+ results.
-Chris


FondoJuegoSombras


Assert throws an AssertionError exception if condition is not true :)


oh no. not me i am not the author. some mistake.


Probably better, now that I think about it.


This looks good, thank you. Just a question: shouldn't all this happen in the ````parse```` method?


are we not incorporating search by City Block for this iteration?   @gsf @atogle @kdemi


The preferred properties from the parent do not necessarily fit the preferred properties for the semijoin. I haven't spent the time to think about how this one should work (especially given that I need to adapt the PR that introduces partitioned semijoins on top of all this logic). For now, the conservative approach is to ignore parent preferences.


The script works because you are loading it at the end of the page, if you put it at the top, it will break, because it will fire before the elements exist. If you always use ready() it makes the script more portable. It will work no matter where you put it on the page.


We don't use the confwriter after this statement anymore and the user of this module has no real access to it plus we need to guarantee it gets closed


In general yes (see PHP files), just an oversight here within .json, will address that in .editorconfig soon (so my IDE doesn't go haywire anymore ;))


You may want to loop through IndexedField.values() and add enum instances like `put(v.name().toLower(), v)`

Or even create static method `IndexedField.findByName()` which encapsulates this logic


Yes. I wouldn't push this commit without the following ones.
Am 06.04.2015 17:43 schrieb "Makman2" <notifications@github.com>:

> ./coala prints me messages:
>
> Usage of __del__ method is deprecated thus each printer needs to be closed manually now!
>
> Do you intend to fix it later?
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/de55c56f022bc9d2afdf939ef1aadd6ab1a7d814#commitcomment-10585640>
> .
>


不知道现在大家能不能看懂这份代码和文档…………


Ještě bych k tomu výpisu přidal i poměr RW, ať to je zpětně dohledatelné.


be careful w/ your indentation


The missing "A" in "RGB" refers just to transparency.... but for now only "colbrDefault" has transparency, all others are "FF" color [not transparent] ... I think we can implement some bit-bit function to force transparency...

Example:

transpGreenColor: TARGBColorBridge;

transpGreenColor:= DoColorTransparency(AA, colbrGreen) ;

Note: Well, maybe I did not understand the question ...


I'll add a TODO to introduce a config option/session property for this


Logical/Perfect


This won't fix the issue, cause
	-e '^/usr/share/doc/' -e '^/mingw../share/doc/' \
doesn't removed!


`
^/mingw../share/doc/
`

This line will exclude all docs.


should be @extends p
Also, p is not a mixin - doesn't take a parameter


rejection sampling


You might be able to do something with that, see http://stackoverflow.com/questions/16557371/how-to-set-speaker-configuration-programatically-using-core-audio-api-on-mac-os

This would probably require some interoperability with the osxaudio driver though.


Przepraszam za ten bezsensowny bezlad. Też nie mam co powiedzić
Musiałem zapytać najpierw u Ciebie, co jest potrzebne, a czego w ogóle nie
ma być.
Już nie mam pytań dotyczących tego, co napisałem.
Mam pytaina dotyczące tego, co jest potrzebne w zakresie "States",
"Messages"?
Opróć "Logged" i "Idle" inne stany nie są potrzebne?

2015-04-06 19:24 GMT+03:00 Tomasz Frydrych <notifications@github.com>:

> I dont'know what to say...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ksanderon/SKCIR/commit/4635b86b00b9fb6a6049f8afe0def9e500eca7fc#commitcomment-10586329>
> .
>


Correct, this is not okay.


After updating I noticed that I actually use this API to be able to hook into the `ProcessScriptFile` and `ProcessMarkdown` as discussed here: https://github.com/tpetricek/FSharp.Formatting/issues/251.
See also #282 for a possible solution.


you can do "AOERadius" "%area_of_effect" instead of having a static value there


2 divided by 0 is 2.


@brendan-ward I see a trailing comment you might delete. The rest of this looks great to me.


Is there any specific reason for DQ endpoint changed instead of reading from Property?


A link to Settings is unneeded now