I must say that I am a bit sad to see this interactive mode go. I guess that we can, in the not-so-distant future, refactor the code and adopt a model-view-controller design pattern so that we can have many ways to drive the code:

* the non-interactive mode.
* a GUI (perhaps in tkinter, as that comes built in with python).
* a text UI (like the current interactive move).

This will require turning the code more and more into a library (which is why I suggested the use of the LGPL, not the GPL), so that our code can be used as a library in the same sense that youtube-dl can.

I guess that I will file this as a bug for future enhancement.


Main issue I've noticed with this is that it spends most of it's time avoiding the phantom enemies and spending almost no time eating blobs unless the screen is completely clear for a while.


IMVHO, a non-interactive mode program should not print anything. Only log things.


no bueno


With your current routes, the link helper, `edit_summary_path` needs a `Summary` object to work correctly. eg. `edit_summary_path(@post.summary)`


wow! so we can redefine system`s string metatable!


Unfortunately, we don't take any numbers on how many people attend the job fair specifically, or even the conference, only on the number that register for the conference as a whole. The number that registers is a fair bit higher than the number that attends, though.


Well basically that is the meaning of ascent of descent, so it should have the basic necessary spacing also. That's the way it is also done in iOS (lately there's the `lineheight` property which is a synonym for "ascent+descent").
But I haven't seen this happen - can you post a picture?


(It fails locally for me on 4.0, so I guess it's not failing on Travis because Travis's time zone is set to UTC.)


Let me cherry-pick it.


No, it does not work. It might work with Linux binary compatibility enabled (disabled by default, usually never used for the open source). There are node binary packages distributed for various FreeBSD versions, so the best bet would be to use what is currently installed, if possible.


@Notsosof1 The "min_id" and 'max_id" were swapped between the `- (void) requestNewItemsWithCompletionHandler:(NewItemCompletionBlock)completionHandler` and `- (void) requestOldItemsWithCompletionHandler:(NewItemCompletionBlock)completionHandler`

That was the first issue I found and when fixed the data was being refreshed as expected. The photos are not downloading though - so that my next task.


Then it was my fault - may we can cherry pick it and integrate it in master! I am sorry for my criticism it was my fault.


Yep I'll change it back and post a screenshot, sec.


* TODO: Should this be an attribute?


Can you try 'mvn clean install' ?
maven-javadoc-plugin is insanely slow here. It seems it tries to (DNS?!) lookup something and after some time it times out.


Attenzione: Non tutti sono JavaDoc.
E ricorda (cancella quindi quelli inopportuni) che i javadoc vanno messi SOLO sui metodi public, sui private non servono a nulla.  Inoltre sui metodi di un interfaccia vanno messi sullìinterfaccia e non sull override


Why not ```_pData->tid = syscall (SYS_gettid)``` here?


Yes mi ricordavo grazie.
Per quello devo metterli in ordine, ma oggi avevo poco tempo
Il 31/Mag/2015 22:03, "igorravasi" <notifications@github.com> ha scritto:

> Attenzione: Non tutti sono JavaDoc.
> E ricorda (cancella quindi quelli inopportuni) che i javadoc vanno messi
> SOLO sui metodi public, sui private non servono a nulla. Inoltre sui metodi
> di un interfaccia vanno messi sullìinterfaccia e non sull override
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/igorravasi/Memo/commit/8d02ef816c0bb91be117adcc57022ac277bb683a#commitcomment-11445385>
> .
>


Sure, I'll look into that tomorrow.


I've used just global namespace.
in my output you can see:
`--[[local]] Array = {}`
so I wanted to use `local` but it caused dependency problem. I think your solution is better and more optimized.


this method would probably be moved to the VanillaMemeEdit and DemotivationalMemeEdit activities as well.


You want schum?


> 1. Not sure how I should do my routes here/what the best way to do this is. Currently I have my routes like so:
summaries/id.post_id
I read that I shouldn't nest my routes more than 2 deep. Should I try doing something like topics/id/post/id/summary? Or summaries/post_id/id? Or something else?

Consider this way of nesting your routes...

```ruby
resources :posts, only: [] do
  resources :summary
end
```

The `only: []` part of the first line there is to prevent any new endpoints being created for the `Post` object, it is just a way for us to get nesting within the `Post`.

This will give us routes that look like `posts/:post_id/summary`.

To access these routes, the helper methods will look like `post_summary_path(@post)`, or `edit_post_summary_path(@post)`.


Ah.. Now I see why you need to `@MapperConfig` and `@Mapper` to have retention RUNTIME.  Have you considered to generate a class (in the spirit of MapStruct) that is used by `Mappers` to determine the implementation? Then we would avoid MapStruct annotations ending up in runnable code.


`super` is a reserved keyword in Haxe, so no problem to use `super` also in Lua I think.


commit


I think this is acceptable to merge. It has Abass and Hoshiko's work. :+1:


we must review your version for performance drop


No idea ;) All I know is that it exist.

But it would be better then what we currently have right? ;)


Nice catch.


OK I see, it is because I have tested with a different font and different language.
Ascent/descent in android does not include the full ascent descent. We need to account for the full versions + spacing, as android separates those.

Please try this:
```java
float labelLineHeight = - mLegendLabelPaint.top() - mLegendLabelPaint.bottom() + mLegendLabelPaint.leading();
```

Not sure about the leading though - it might be included in top/bottom.
I'm not in front of an IDE right now so maybe you'll have to debug this - but the thing is that ascent/top are most likely negative, because they are negative Y relative to the baseline, and descent/bottom are positive. That's what the docs says. But for me flipping the sign on both of them worked. So I'm not sure if bottom should be subtracted or added...


Fixes https://github.com/PolymerElements/polymer-starter-kit/issues/74


This is not handling jmp/jcc properly.   I've seen `jmp r12` cause rip to be set to rsp and `jnz short` cause an infinite loop (maybe the breakpoint needs to be disabled for this one?), among other problems.

The Stop/pause button doesn't work while in the RunUntilRet handler.


e.g.
```java
package org.mapstruct;

MappingConstants {

public static final String implementationPostFix = "Impl";
public static final String implementationPackage = "org.example.impl"; // todo functionality;
}
```

Your mappers class could use that generated class (if present, because of deviations of the default values) to construct the right implementation class


An overall comment here. It seems to me that originally the PerformanceMonitor-related code was perhaps aiming to be a bit more generic than specifically monitoring execution times, allocated memory and so called "spikes" (see later comments). My suspicion is due to some of the javadocs and comments referring to "metrics" in a general sense rather than the specific metrics provided by the actual implementation.

I nevertheless decided to change these javadocs to reflect actual functionality than what was perhaps the originally intended functionality.


Ok I think that (mLegendLabelPaint.bottom() - mLegendLabelPaint.top())
should do the trick, without the leading. Leading seems to be "recommended"
vertical space between glyphs, which is used maybe when you manually draw
glyphs...

On Sun, May 31, 2015 at 11:04 PM, Philipp Jahoda <notifications@github.com>
wrote:

> [image: screenshot_2015-05-31-22-04-05]
> <https://cloud.githubusercontent.com/assets/6759734/7903572/076cbb42-07e1-11e5-81ca-f1a0ba18e429.png>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PhilJay/MPAndroidChart/commit/3e4ff0be58692142055ad40d5ba32c879cc666ae#commitcomment-11445392>
> .
>


Should it just be inline with the rest of the text? I kinda like it for posts like this ( http://eldh.co/blog/nexus-4/ ) but we should perhaps change the default way of displaying images...


{$DBPASS} is incorrect. Also...

It seems like this doesn't work right. The variables don't expand properly at runtime:

```
[root@summit-rhel-dev workspace]# atomic run mariadb
docker run -d -v /mysql:/var/lib/mysql --name mariadb -e DBUSER= -e DBPASS= -e DBNAME= -e NAME=mariadb -e IMAGE=mariadb mariadb
```


in JS - redefinition of built-in methods is bad practice. Also `toString` behaves differently in JS VMs.
lua.Boot.__string_rec also allows us to not generate Std.string (and Std class at all, if not used)
we can just keep any version - to your taste
but Std definetly will be used in non-hello-world projects, so another Boot class seems to be useless


Ааа. Понятно) Просто я тормоз.


@skyboy yes, that's true, but I think that needs a special marker myself- custom itemstacks don't quite fit the registry model.. Perhaps an additional argument for a custom itemstack name..


I reckon this is more 'missing functionality' then a bug. So it deserves a package on its own merits :+1: . What about `org.mapstruct.ap.test.customizing.mapper` ? We could add the customized implementation package to the same tests later..


@Notsosof1 The following code was missing from the `ImagesTableViewController.m` This fixed the images not showing up. Make this change and the previous one and see what you think. LMK.

```
- (void) tableView:(UITableView *)tableView willDisplayCell:(UITableViewCell *)cell forRowAtIndexPath:(NSIndexPath *)indexPath {
    Media *mediaItem = [DataSource sharedInstance].mediaItems[indexPath.row];
    if (mediaItem.downloadState == MediaDownloadStateNeedsImage) {
        [[DataSource sharedInstance] downloadImageForMediaItem:mediaItem];
    }
}
```


Cool!  Didn't think you'd merge it with so little discussion.


Looks great!


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

```


Mh, currently JAVA_HOME is not set right. I have to fix this first to run 'mvn clean install'


No need to change this now, but for future reference -- what I would do is change the INSERTs to specify the column names, so the next time a column is added the default value will be used and there will be no need to do this again.


i've just tested it on a rPi and it's indeed not an issue if you use onload.

sorry for the noise.


C'était juste une optimisation à faire à la fin, là du coup va falloir réfléchir tout le code différemment si on abandonne direct le arrivedPoint


in 4th commit:
1) the level jump iteration still needs verification, which involves subroutine c_2_f, initialization. For now just use level=1.
2) subroutine wall_finding and finding_dels are based on the geometry of a cylinder, need to be modified to add the configuration of a filter (currently working on this)


librtapi_math.so.0 is created but all references are to .so . I fear this does not work out as intended.


Well, I haven't merged it to master for a release yet (I'm evaluating #22 before doing a release), but it seems to simplify a lot of things.


C'était bien avant ça simplifiait des trucs comme j'ai dis #25


@msanroman Hola Mike! Gracias por contestar!! Pues he hecho esto y sé que falta un if (si todo esta correcto) load el otro popup no? pero lo he intentado de unas cuantas formas y no me sale... y es que me lío con lo de que tenga que hacer un load del otro popup que tiene su javascript y no sé cómo y dónde ponerlo... si me das un ejemplo por favor...

Gracias!;)


What has "html" got to do with this?


I think using different modifier keys is a good compromise.


When querying multiple tables it's best to specify the table name for each column, so that a future schema change can't cause ambiguity and mysterious breakage.  Also an AS clause is helpful for brevity.


No newline :)


I don't understand, but please fix it.  For packaging purposes, and in general, the `.so` symlink is for development, and the `.so.0` is the actual binary.  See `rtapi/shmdrv/Submakefile` for an example.


@ExSport can you test it? For me it works.


Expanding and clarifying the javadoc for this method took me a long time. It turns out the problem is the term "spike". In my humble opinion this method doesn't actually return "spikes" which could be thought as sharp peaks in a graph displaying the activity's execution times over a number of frames. It returns the largest execution time of an activity discounted over time, so that more recent large execution times have the chance to replace the stored value. This is irrespective of the shape of the graph: an activity might have fairly uniform execution times, resulting in a smooth graph, and this method will still return a value per activity, weather it represents a sharp peak or not.

That been said, I couldn't find a concise way to rename this method to reflect what it does.


When I do not include like: @like in locals hash - throws undefined method error when utilizing in partial


Have Tried utilizing AJAX & HTML calls with Create & Destroy


T'as pas peur de péter toutes les icônes ?


I think this should be renamed getExecutionRunningMeans() and getAllocatedMean() should be renamed as getAllocationRunningMeans(), but I decided not to do it in this PR to keep its scope and the number of files it affects, more limited.


Utilise des variables :)


I don't think a name ending in "vm" is very telling. If it's because it's using dart:io, then call it "packages_io_impl.dart".


3) continuous calculation needs verified
4) the accuracy of the solution needs verified


this method would probably be moved to the VanillaMemeEdit and DemotivationalMemeEdit activities as well.


Currently I got some test failures - hope that they run fine for you?!:

Failed tests:
  TypeaheadConfigTest.assertValidHeaderScript:45->Assert.assertEquals:144->Assert.assertEquals:115 expected:<...d1').typeahead({},{"[source":engine.ttAdapter(),"name":"engine"]});> but was:<...d1').typeahead({},{"[name":"engine","source":engine.ttAdapter()]});>
  TypeaheadConfigTest.testComplexRemote:58->Assert.assertEquals:144->Assert.assertEquals:115 expected:<...espace,"remote":"{\"[wildcard\":\"%FOO\",\"url\":\"foo]\"}"}> but was:<...espace,"remote":"{\"[url\":\"foo\",\"wildcard\":\"%FOO]\"}"}>


Are the values from this controller what the partial has access to before the #create or #destroy action? Then after an action is called - what variables go back to the partial? The objects from the action or from the topics_controller?


I went out on a limb for this paragraph. I *think* this is the case and the paragraph is true.


this is hoshikos work i believe? besides the readme


I believe we can leave the button here for now, but we will have to move it to the VanillaMemeEdit and DemotivationalMemeEdit activity to allow the user to share the image once the meme is complete


method for calling sharing options looks good


I have disabled m-javadoc-plugin.

Yes, the tests pass fine here. The error looks like you run with JDK 1.7 with the new HashMap impl that shuffles the entries compared to pre-1.7.


You mean lastlog?


wenn etwas passiert wie das hinzufügen eins neuen Objekts zB einer Liste wird dies auf true gesetzt und auf der Seite oben erscheint eine Nachricht mit $alertType und $alertText (siehe unten)


FINALLY, thank you Naios! Truly grateful for your contribution! :)


:-) Again such an issue - I try to fix it.


Ich hab auch einfach ganz viel sauberer eingerückt, nicht verwirren lassen.


Du coup tu l'as fais relativement à la taille de la fenêtre...


Great! good thing you added theses permissions, i forgot to implement this in order to read a file from camera roll.


Hier werden dann die Texte und die Farbe (bestimmt durch den $alertType) gesetzt.


This seems to be a constant. Any reason to keep it at all? And timerTicksPerSecond?


Diese HTML Inhalte sind immer oben über allem anderen gelandet. Jetzt werden sie in die Variable $templateContent gesteckt, welche in der edit_list.inc.html verwendet wird.


C'est dangereux :smile:


Und Tabellen mag ich auch nicht, deswegen verwendet das jetzt die bootstrap Klassen "row" und "col-sm-3". Das hat auch den Vorteil, dass die sich auf kleinen Bildschirmen unter einander schieben :)


Why haven't you used `git merge rbenv/master`?


Hab meine Style Dateien hier geladen, damit sie auf jeder Seite geladen werden.


Resolves half of https://github.com/flux-framework/flux-core/issues/198


Das Menu ist jetzt keine Tabelle mehr. Diese Funktion spuckt jetzt li-Elemente aus.


There is one instance: https://github.com/reactor/reactor/blob/master/reactor-bus/src/main/java/reactor/bus/EventBus.java#L260 where you did not change to <Object , ..> which causes compile errors in our project, could you check that or shall I create an issue @smaldini


Wiederverwendbarer oberer Teil der Seite, wie wir das zusammen gemacht hatten.


Hi @hasandemircioglu
Sorry for my late reply, I have been out traveling.
I'll look into creating a new package tomorrow or the day after, when I have setup a testing ground

Regards,
Ted


In these type of calls, having a procedure provided as input, I interpreted the syntax forEachEntry(new procedure) as "instantiate a new anonymous procedure for every entry and then execute its code". I now suspect that's not the case and there isn't much of a performance improvement in using previously instantiated procedures. That been said, I hope the more concise rollCycle() method will still be seen as beneficial.


Implemented here: eebe3e148ba132f965df5d34f0b4d99e70f9c910

Being able to move the ball while having a player selected is definitely much more convenient. :+1:


Issue is at de.agilecoders.wicket.jquery.AbstractConfig which initializes a HashMap - We should convert it to LinkedHashMap


According to [Brandon Rhodes via Twitter](https://twitter.com/brandon_rhodes/status/605044030406303744), PyEphem uses the 1980 IAU precession and nutation. If I recall correctly, we might be able to set the IERS tables to the same numbers. More on this later!


Auf manchen Seiten wird hier die Funktion HeaderHTML() verwendet und auf manchen $MyHeaderHTML. Muss ich mir noch mal angucken. Auf jeden Fall wird $headContent auf jeder Seite erst mal mit einem von beidem gefüllt.


Can we switch these to use markdown style links? http://www.mkdocs.org/user-guide/writing-your-docs/#linking-documents


Ну я вижу, что переписана система приема инфы с сайта.


Markdown style bold would be preferred.


I only read after submitting this PR that the PerformanceMonitor is supposed to be used only from the main thread. I guess in this case we could just reuse the same instance of SetterOfActivityToRunningMeanMapEntry by adding setter methods to it for the activityToMeanMap and factor variables.


That's depends of the parser you are using.
Anyway is a expected behaviour, because in Markdown language you can inline HTML.

From the [markdown syntax page](http://daringfireball.net/projects/markdown/syntax#html):

> For any markup that is not covered by Markdown’s syntax, you simply use HTML itself. There’s no need to preface it or delimit it to indicate that you’re switching from Markdown to HTML; you just use the tags.

If you want to disallow some HTML tags, you will have to configure your Markdown parser.


As of this commit there is very little in the LOOT masterlist not in the BOSS masterlist as of commit https://github.com/boss-developers/oblivion/commit/b017ac64002a5708ce86bf237940728c4c6054b2


Make marker constant a private class member?


Same here.


The site has been up for almost a month, I should be linked by now :)

2015-05-29 12:46 GMT+01:00 Neil Lawrence <notifications@github.com>:

> Stop messing with the colors and get the site up so I can link to you ;-)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ric70x7/ric70x7.github.io/commit/df0d7016b3cbdf6cf7227e81de89186e06637573#commitcomment-11425539>
> .
>


`const`?


kephas, I appreciate the intent here, but I think this breaks 16-bit uncompressed images. I'd hate to break that just to clean the code up. If we want to support 16-bit for all of them and deduplicate the code, that would be great. Sorry for the delay!


OK, `DefaultConstructible` should just require `T{}`, and I can strike the prose. Thanks.


@tisto don't kill the messenger (i.e. the one try to make a pull request pass) but discuss directly with @keul

I agree with you actually, but the test case was like that, so we either fake a locale on the test itself or something else...


Locally created items should be version 0. The version only comes from the server.


Oh, I see the next commit... I'll keep reviewing.


I always use the latter, because I think it's easier to see that it's a continuation of the previous line that way.


Concluding this batch of comments: apologies for some highly verbose names: I initially tried to keep them shorter but I was having difficulties in keeping them also precise in their meaning.


sick typo bro ;d


@Debetux You're welcome. ;-) Remember to delete your old repo (https://github.com/Debetux/SkypeExport) since I don't want the super-old code out in the world, and I see that it's still up. If you're interested in having a copy of the new code, feel free to do a new clone the latest repo.


`length + 1` and no further null char?


Самокритика конечно решает, но лучше так не думать про себя ;)


Oh, if you are going to do that, I need to add a license to the project, otherwise it defaults to All Rights Reserved. One second.


The data-offset computation calls (ash k 1). What is k supposed to be?


It's an internal app, but the video just demos doing ctrl-cmd-z, pressing Inspect Element, and tapping around.


Oh oops.


And for 7.x branch https://github.com/l0rdn1kk0n/wicket-jquery-selectors/pull/13


Please avoid underscore functions, you can just use `prettyPrint()` or even `util.inspect()` which is clearer. In this case `testing.assert()` already does the pretty print so the whole thing is unnecessary.


`testing.assert()` already does the pretty print thing internally.


YOU NEEEEEEEEEEEEEEEED EMOJI


Please don't use functions that start with underscore.


Missing const qualifier here?


Please don't add unnecessary blank lines.


Need to ask Steffion about a few issues.


Oh really ? I was curious why you didn't have a .gitignore


No, the touchscreen bit may be added later.


I fucked up the emoji code and I was so disappointed in myself


Might want to use `[ $err -eq 0 ] || bail` instead, Not sure how compatible that not sign is. Although it might be good too.


Time to rename seems to have come. This shell is no longer a very simple shell. We should rename it to Legacy Minimal Shell, `lemsh`.


We might want to change this to OrderedSet with the natural ordering being the most recent to the least recent.


You need to add plenty of comments to explain what's going on with the Knockout stuff.  Without that, the effect is to shut out developers who are not familiar with it.  That's not in keeping with the project's philosophy.  For the same reason I think there should be some peer discussion before adding a new framework to the project.

Also is there a demo of this somewhere?


Nice, going to try out the R formats soon. I guess the BitMaskImageCodec to get ImageRaster support is the same as with Luminance16F/32F?


Normalement les fichiers baggers.js auraient dû être supprimés.
Tu as eu un pb lors du merge @nrichand ?


Using my `shellcheck` POSIX-compatibility checking program says it's fine. Don't see a reason to change it.


Testé, et fonctionnel.


Wat ? (https://www.youtube.com/watch?v=fbGkxcY7YFU)


"Я поблагодарил за игру, за самоотдачу, за желание. На таком уровне в таких играх нет смысла говорить о самоотдаче, о том, что кто-то не выложился. Все сделали все, что могли, но так распорядилась судьба. Если бы в начале второго тайма все повернулось бы в нашу сторону, то "Севилья" уже не восстановилась бы. Вспомним групповой турнир и когда в Баку не засчитали гол "Карабаха" в ворота "Интера", мы бы вообще из группы не вышли. Но мы же не говорим о том, что было тогда, мы говорим о том, что есть сейчас. Мы были в шаге от золотых медалей, а получили серебро. Человек всегда максималист. Он хочет большего, и когда у него есть шанс, и он им не воспользовался, то об этом жалеет и не думает о том, что когда-то мог не выйти из группы. У каждого человека есть свой финал. У любого спортсмена есть стремление к максимальному результату.

Мы говорим о принципах Кубертена, важна не победа а участие, но никто в это не верит. Все стремятся и хотят победы. Для настоящего спортсмена есть только блеск золотой медали. Никто не ставит задачу занять второе место, ставят задачу стать чемпионом. Поэтому нет смысла утешать игроков, они настоящие спортсмены, профессионалы. Честь им и хвала, что они были близки к золотым медалям европейского турнира. Можем только сожалеть, что не сложилось в этот раз. Надеемся на будущее"


this is *technically* slightly inaccurate when we're talking about a parent and child class. It's actually the parent class that does the `this`-making, but `new.target` will point to the child class constructor.

I prefer to describe `new.target` as relating to the constructor at the call-site where `new` occurs... like literally, "`new.target` is the target of the `new` operator". Thus, it points at the child constructor, even inside of the parent constructor that's invoked via the `super(..)` delegation from the child constructor.


Also, this really isn't a mod at the moment. It doesn't really do any mod stuff. It's mainly just basic Java that just hashes stuff. I don't think I would do this in the same way. I should update it. I should be able to finish it next month, when the summer holidays start.


This commit message has an extra "cached" in it.


Hi, i guess that was the simplest way for me to fix it as I haven't dived into the workflow in all of the configurations and tls_init seemed to be called in all of them.
It caused us serious problems. We compile your library to a lib then use it in several of our products.
one of them is itself a dll which is loaded and unloaded periodically from the same process. each time the initializtion takes a tls slot - which are limited to 1024 if i'm not mistaken. then the process crash.
other than that the library is working very well. good job


Received msg have at last 2 char EG.: {}


yep :)

I definitely should've mentioned this, sorry about that.  This is how I'm managing static strings, etc.

If we don't alter the metatable in this way, then I think we'll have to use the same pattern I used with  lua.Boot.defArray({...}), which is to pass every static string declaration into a constructor.  E.g. :
```haxe
var x = 'a string';
```
becomes:
```lua
local x = lua.Boot.defString("a string");
```

... and we add the length field and set metatable, etc. within that method.

lua.Boot.defString("A lua string").


I think something like "GssapiAllowMechs krb5 ntlmssp" will make it easier in 'merge' cases (see AuthMerging directive).
Note though, that I haven't tested 'merging' logic in general with MAG.


no, not yet.  That will be helpful.  I really didn't want to focus too much on line count right now, just making it as many things work as possible.


What do you think of the message being `__METHOD__ . ' is deprecated. Use \Composer\Package\PackageInterface::getFullPrettyVersion() instead.`?


Ah, good point.  Is there a generic lua polyfill library for this sort of thing that we can use?  I'd hate to have to try to maintain this in our generator, that's too much complexity.


looks like I have full Array implementation. but some code in Lua-magic to be more performant... maybe...


fyi: the current <?, Consumer> return value of consumerRegistry gives:

"The method unregister(capture#1-of ?) in the type Registry<capture#1-of ?,Consumer<? extends Event<?>>> is not applicable for the arguments (String)"

consumer registry is internally a Registry <Object, Consumer>, any specific reasons you put in ? as type?


tuy


This should be translated to "Tyyppi".


This should be translated to "Ala".


This should be translated to "Kieli".


This should be translated to "Sukupuoli".


Добавьте пожалуйста поддержку MySQL


Could not


I think that's a great idea!


Is kubeletstarted still used?


como eu excluo esse commit?
e deixo o que vc enviou como principal?


After this commit TinnyMCE doesn't work.
http://prntscr.com/7bn2s7
On previous one works fine.


bit/bit32? I think polyfilling bits is useless, Lua is in 5.2 / 2.0 (JIT) version for quite a long time, and everyone must enable any of bit or bit32 extensions in embedded VM.
but `print` is not standard, and one can accidentally define it as `trace` or `console_log` or `message` etc


Next to the two small comments, LGTM


But why?! Some of us need SoftwareSerial! Any advice?


Я что-то здесь не понимаю?


I think it looks good, and I have not run it.
The only thing that get me confused is your m_inv. It looks like you might treat 2 variables as one , one local and one global with the program. for instance I am not sure about having to return a global variable
Anyways, if you have problems maybe you want to take a close look at m_inv.


ow! this is why I passed classes`s prototypes into _G, not local. Probably this must be optional `-D lua_livecode`


This looks like two unrelated commits — the one mentioned in the commit message, and then one about remove_open_past_parents?


markdown syntax isn't processed inside a block level HTML tag (e.g. table), so I had to go with <a> instead.


@CumpsD really glad you liked it. I've sent the PRs. :) If you have any requests, please open a issue on fsprojects/generator-fsharp.


Suggestion: s/>= 0x2/& MASK_INDEXED/ (rename the mask if you can come up with a better name).


@asutton seems to frown on using type traits to define concepts. Here's one way to avoid that:

```c++
constexpr struct _Target {
    template<class T>
    T const & operator()(T const &) const;
    template<class T, std::size_t N, typename _This = _Target>
    auto operator()(T const (&t)[N]) const -> decltype(_This()(*t));
} __target{};

template<typename T>
auto __destroy(T const &t) -> decltype(t.~T());

template<class T>
concept bool Destructible =
    requires(T const & t) {
        {__destroy(__target(t))};
    };

struct not_destructible {
    ~not_destructible() = delete;
};

static_assert(Destructible<int>, "");
static_assert(Destructible<int&>, "");
static_assert(Destructible<int(&)[4]>, "");
static_assert(!Destructible<not_destructible>, "");
static_assert(!Destructible<not_destructible&>, "");
```


yeah, I already have something similar to this in place (taken from php approach).  I'm sure it's slower, but I can't think of a better alternative.


Thanks, Paul!


There's too much indentation here.


Leaving it here would allow #2487 to generate smaller s8 displacements.


Drop from the previous commit.


I assume this feature isn't complete due to the last revert - it is broken as follows.

When not using cred_store this cause any Negotiate or NTLM to fail because we don't acquire credentials before calling 'gss_accept_sec_ctx', so it results:
[Sun May 31 22:23:24.089288 2015] [auth_gssapi:error] [pid 25567] [client 10.0.0.250:34624] gss_set_neg_mechs() failed: [A required input parameter could not be read, No credentials were supplied, or the credentials were unavailable or inaccessible ((null))]

I tried to fix it by acquiring default credentials anyway (like we do for basic auth) but then when I specify "GssapiAllowMechs ntlmssp" it fails (I think because we didn't acquire SPNEGO credentials):
[Sun May 31 22:26:48.459085 2015] [auth_gssapi:error] [pid 25793] [client 10.0.0.250:34626] gss_set_neg_mechs() failed: [The operation or option is not available or unsupported (Unknown error)]

At last, I tried to simply comment out "goto done" below but it still doesn't accept SPNEGO when I specify either GssapiAllowMechs krb5/ntlmssp, and I get:
[Sun May 31 22:29:12.269412 2015] [auth_gssapi:error] [pid 26000] [client 10.0.0.250:34631] gss_accept_sec_context() failed: [No credentials were supplied, or the credentials were unavailable or inaccessible (Unknown error)]

It is like my previous comment at issue #23 when we acquire credentials only for actual mechs the library will refuse to do SPNEGO with these creds.
So what seem to be needed (for Negotiate only!) to acquire the mechanism specified by GssapiAllowMech AND also acquire SPNEGO and restrict it again to the mechs specified by GssapiAllowMech with 'gss_set_neg_mechs'.


Same problem with the hyperlinks, markdown syntax isn't processed inside block level HTML.  I believe double underscore is translated to "strong" by markdown, c.f. http://daringfireball.net/projects/markdown/syntax#em


False


I would replace this with an assertion.


Only one empty line at the end of the file.


Delete


I'm sorry, but I don't follow. Could you elaborate your question, please?


This case "(let ((x 2) (y 3)) (let ((x 7) (z (+ x y))) (* z x)))" was returning 70 instead of 35.


merge/


First change from working folder.


create admin user


The underline needs to be a bit wider to fit the above line.


I think that the user should save rules explicitly and not when the service shut down.


Hi thanks for the update, fyi small error, i get this message:
```
./Rtorrent-Auto-Install-3.0.2-Debian-Jessie: line 858: syntax error near unexpected token `}'
./Rtorrent-Auto-Install-3.0.2-Debian-Jessie: line 858: `}'
```

Apparantly it can't handle the empty function..


To be removed from the next commit


Is there a reason for the `wins_resize_all()`?


I would rather make class B non-static and make doStep method modify its instance variable when calling doStep.
This way you won't have to modify&assert on some global variable.


Hola Mariano, es un WIP. Lo subí porque hay un nuevo integrante en el
equipo y él se va a encargar de este archivo. Es muy difícil entender la
función de este archivo porque falta todo el resto del código en el que
venimos trabajando. Espero subirlo durante esta semana. Perdón por el
desorden. Saludos.

2015-05-30 15:07 GMT-03:00 Mariano Cerdeiro <notifications@github.com>:

> mejor seria hacer 2 asserts?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ciaa/Firmware/commit/d349ab31d26c50ba1914fd1a1425b77d891cf432#commitcomment-11440524>
> .
>


Maybe this type could be parameterized?


Then don't enable the service.


Oh, that's why I couldn't find it. There are two .gitignore files in the repo. One here, and one in the root of the project. I guess that works, though, in case you ever decide to make the ktsg theme its own repo.


Thanks Anders G. It works well.
No problem now to split the float and wheel pointmass.
I'll do this to morrow.

*(Sorry...I believed having tried this after your post, but with I don't know what mistake...)*


LGTM. Tested with a pair of unos. All worked well including the repl.


Wow! The code is that much clearer now! I'm impressed... 10/10 would hire


You should add that the max and min parameters are optional...


It might be worth moving the output ("Windows tidied.", "No tidy needed.") outside of the `ui_tidy_wins()` function and have that function return the boolean, since otherwise when closing a window these messages are currently shown.


Replaced with "configuration" for better grammar. Let me know if you meant to use completely different word.


Pourquoi le destructeur est retiré ?


This should be translated to "Ladataan...".


I would name it getComputationStepParameters or something like it :)


Ah yes of course. Based on other tools - I think it is probably best to assume it won't already be encoded and do the encoding for the user.


This could be extracted to private method or even separate class.


That's even better, done.


According to the newest standards, indentation should be different in these ways:
- 1 space after while
- 2 spaces after if
- 3 spaces after an open brace
- 4 spaces after a closed brace
- tab after a cast (e.g. `(int)`)
- ideographic space between any sequences of letters that
have no punctuation in between (e.g. 'function randomInt')
- zero width space on both sides of arithmetic operators
I'll make a pull request with the changes mentioned above.


Why don't we pass this in constructor?


Correction: Moved just enums, not interfaces.


> `static_assert(!Destructible<not_destructible&>, "");`

All reference types are destructible. This C++11 TU compiles without errors:
```
#include <type_traits>

struct indestructible { ~indestructible() = delete; };
static_assert(!std::is_destructible<indestructible>(), "");
static_assert(std::is_destructible<indestructible&>(), "");

struct foo { indestructible& bar; foo(); };
static_assert(std::is_destructible<foo>(), "");

void f(indestructible&) {}
void f(indestructible&&) {}
```
I also frown on using type traits to define concepts. Predicate constraints are barriers to normalization; they don't provide any useful information to the constraint language about the relationship between the concept being defined and any other concept that doesn't include the identical predicate. As far as the compiler knows, there may be a model of both `std::is_destructible<T>::value` and `!std::is_destructible<T>::value`. Expression constraints also provide useful documentation to clients by telling us what we can *do* with a `Foo`.

That said, the expression constraint `__destroy(__target(t))` is no better than a predicate constraint. It doesn't tell me what I can do with a model of `Destructible` - I can't use reserved names - and it's unlikely that expression will appear in another concept definition. I don't see how this is an improvement over `std::is_destructible`.


this should be initialized as a global accumulator variable.

```scala
	LongSumAggregator aggregator = new LongSumAggregator();
	public void preSuperstep() {
		// retrieve the Aggregator
		sum = getIterationAggregator("sum");
	}
```


This should be modulo 4, with normalization in the second and fourth branch accordingly.

There is no need to send an "empty" message in step 1 and 3 as we discussed last Wednesday, since the update function will be automatically invoked for each vertex that was updated in the last superstep.


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


In iptables and ip6tables services what happens on finish is a flush. I think arptables service should behave in the same way.


Можно просто `deque([head_coord])`


Всё равно наврал


defineProperty won't make the property not "undefined", it gives it extra behaviour as a getter & setter in this case. So after you actually set a value to width & height, typeof(this.width) should return number. Though I'm wondering if this change actually does anything. Not sure what the impact is of calling defineProperty for the same name multiple times.


Jesin compression brakes my website, any update for it? In my opinion, it is very nice extension... any updates?


Awesome, I'm currently making my own that does more or less the same thing.

On Sun, May 31, 2015 at 3:09 PM, Ned Hyett <notifications@github.com> wrote:

> Also, this really isn't a mod at the moment. It doesn't really do any mod
> stuff. It's mainly just basic Java that just hashes stuff. I don't think I
> would do this in the same way. I should update it. I should be able to
> finish it next month, when the summer holidays start.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nedhyett/ModSync/commit/0f6e63bfc32f15506e7800a0ca25bf304185b818#commitcomment-11445675>
> .
>


Create a graph by pre-processing the edges as a `union` of the original and the reverse ones, accordingly. Use a flatMap to attach a label ("H", "A") for the original and the reversed one.


I added this streamGetAttachmentResponse method as a parallel to the getAttachment method.

The getAttachment method ends up loading the body of the attachment into memory, and it was simpler (but a bit hackier I admit) to just download the whole response (in XML form) into the specified OutputStream then parse it from there.


Я как раз хотел спросить, а то я запутался в docstring.
Мне надо в Attributes указывать аргументы класса (в данном случае только head_coord) или всё, что идёт в __init__ в виде sefl._?


Я предлагал вынести только это условие в отдельную функцию. Имя этой функции недостаточно отражает её содержание


Нельзя, он tuple разбивает на 2 значения


The ServiceRequestBase.validateAndEmitRequest got a new overloaded form that now takes an OutputStream.

If that OutputStream is set, the entire Response's Entity (aka the content) will be dumped into it instead of loaded into memory.


Как здесь более гладко написать docstring?


If GssapiAllowMech is not specified then we could end up doing arbitrary gssapi mechanism over NTLM channel (prefixed).
Not that I think a client will ever do that but for sanity i'd rather restrict it.


Атрибуты -- это обычно те поля, которые могут использоваться снаружи. У тебя же все поля скрытые, так что ничего не требуется указывать


Хотя, именно в квадратных скобках вроде можно


Ты невнимательно прочитал мой комментарий


I'm not sure what this new logic is supposed to do, but don't you need it in the else section as well?


this is where a new method, "streamContent", was added for consumers (us) to call.

The load(OutputStream) method that already exists pulls the whole content into memory, so is unusable.


Thanks for the backport, btw please try to keep the code as much as possible in sync with master - or at least I think this would be a good idea. I have seen that you did not port the `//forward declaration` code comment removal and fixed the `creature is force to use MMaps` grammer;)


the new streamGetAttachmentResponse method is above.


Это внутреннее поле, его можно не описывать


Код сократился на 4 строчки благодаря словарю, но мутно выглядит вызов в 177 строке


Right below here - I think we would also want want to advertise NTLM support along with 401 when no authorization header was sent.


this is a custom xml parsing hack-job, implemented below.


Hey I'm just curious, you guys have any luck with T-Mobile WiFi Calling yet? I can't figure it out.


my sample xml files normally show the namespace as "t", so I could have used "<t:Content>" as the pattern here, but I don't trust MS to now change it on their end.


Объект #71641
Такой объект не существует, либо ещё не добавлен в базу данных.


На самом деле он выглядит довольно понятно


Well it does not really change things significantly, but should help preventing the issue i described, and looks like a cleaner way to do it (as per the JS API, a property can be redefined as long as the configurable option is set to true).

> On 1 juin 2015, at 01:05, Aaron McLeod <notifications@github.com> wrote:
>
> defineProperty won't make the property not "undefined", it gives it extra behaviour as a getter & setter in this case. So after you actually set a value to width & height, typeof(this.width) should return number. Though I'm wondering if this change actually does anything. Not sure what the impact is of calling defineProperty for the same name multiple times.
>
> —
> Reply to this email directly or view it on GitHub.
>


fancy FilterInputStream that overloads the "read()" method so that it signals end_of_stream when it encounters the "<" character.  This is only used above, and is Base64-encoding-specific (since the "<" character is not a valid Base64 character).


this is the lowest-level guts of the change - if the HttpWebRequest had its responseOutputStream set, a successful HttpResponse will dump its entity into that OutputStream here using the HttpEntity.writeTo(.) method.


this setter is used within the new ServiceRequestBase.validateAndEmitResponse(.) method below.


Переход на `deque` связан с тем, что это работает за O(n)?


By not catching an `ImportError` here, the `msgpack` library must be installed, even if the project doesn't use it.


this is a test of the 2nd half of the FileAttachment.streamContent method, the "FileAttachment.writeContentFromResponseFile" method.

Resource files that were added to support these cases are below.


Since @hunse was able to convince you earlier, here too it would be good to keep these imports alphabetized ;)


https://git hub . comment/defunkt/jquerp jax/commit/71ecfufgs

b31342dffddjgd63d99b99925ef7425fe


Should be `nengo_viz.Viz()`


EqualsVec?


verifyRepeatingResponse (below) is a method that verifies that the UTF-8 bytes of the input stream are repeated a specific number of times in the corresponding test resource file.


@davies any reason you didn't backport this into 1.4 branch?


Great Work.

I got some compile errors. was easy to fix. May be if you can update the repo it may be useful for others.

Thanks


binary verification was done by generating a file with an xml prefix/suffix surrounding the Base64-encoded contents (courtesy of the internet) of my picture of the SRAM Wireless Shifting rear derailleur.


rax retry tests


Ouch... that's certainly not desirable behavior in any firewall. The service should only exist to preserve the current set of rules, so when you reboot it'll come back with the rules you had before shutdown.


Updated Title


What's the point of this?


one space more after the 8


Isn't this exactly what would be thrown anyway if the file open failed? (except now you have a race, too)


I believe this constant isn't used anymore, well, at least not within the scope of this patch. The message text however *is* still in use, a few lines below. See my comment over there


I believe this line isn't proper english. What about changing it to something like:

`BUG: The following mutation didn't produce a single diff hunk. Please report a reproduction!`


а каким парсером пользуйтесь? npc data parser? или object парсер?


True


Удаление первого за константу, да


LGTM


Uhm, I haven't made that. It was already there when I forked what you've done before.


@snusnu Good catch. Thx.


Yeah, it was just for debugging. I will erase it.


@snusnu Do you want to submit a fixing PR, also including the unused constant above?


Для класса `Field` `width` и `height` тоже не надо указывать?


Import


You are fine not re-basing. I can remove it if it showed up. It is only one line.


Да


Support for case insensitive keywords
Added in support for case insensitive keywords as indicated by the
standard. This allows all the tests to pass without failure


If *fmt is zero, then strlen(fmt) is also zero.  So the strlen(fmt) is duplicate, and unnecessary.

It's better to just check if (!fmt || !*fmt)


reply = 'Welcome to San Francisco CrossFit SMS. Text "Subscribe" to receive daily workouts.'


Alphabetical?!?  I hadn't even realized that was part of the rules...  I'd just been grouping things into the 3 import sections.  But now that I know I'll try to be better about it...  :)


Тогда как пользователь разберётся, что туда подавать?


Одно дело описывать метод `__init__`, другое дело -- описывать весь класс


@cleiton-tavares parabéns cara! O Clinn.io é muito bacana. :dancers:


there was already a "getAttachment" method, I added this one so that I could minimize the number of changes to the internal code.


Added basic map prefabs.
Created dummy map for testing purposes.


this "validateAndEmitResponse" method got a new overload so that the OutputStream argument could be filled up with the Response's entity content during the http response handling.


:+1: I just wanted to make sure you don't forget.


Why is it still not in npm registry?


TeamCity LabkeyTrunk :: CDS postgres 9.3 [Build 35](https://teamcity.labkey.org/viewLog.html?buildId=252871&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 0 Build time: 00:05:43
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

```


this is the call to the new method above.  It causes the specified OutputStream to be filled with the full content of the HTTP Response Entity, which in our case is a big XML file that has a "Content" element inside it that contains a string that is the Base64-encoded content of the file attachment.

This just streams that big XML response to disk.


Upon further research, the comment wasn't entirely accurate.

Only 12 entries are used by the vertex loader. The other 4 are used by DMAs to xfmem.


This method (coded below) reads the XML file and streams out the Base64 String into the outputStream argument that the caller provided.


@gitster Can I pull this already, or did you have more to add before opening a pull-request?


Я думаю, понял. Спасибо!


this "responseFile" is a temporary file that can be deleted now that it's been parsed.


Do you plan to make any functional changes as well?


step 1 is to position the "responseInputStream" object to the location where the Base64-encoded string lives.


Should we use `getaddress` (the first address) or `getaddresses`?


then just stream from "is" into "outputStream" until the "<" character is found (that is not a Base64-encoding character).


Probably should split this into two methods `internal_host?` and `internal_ip?` to separate steps.


NOTE: Base64ValueStream is a custom hack class defined below.


the "readUntilPatternMatch" method is defined below.


```/ 8```, not ```/ 9```?  Back-referencing 1 byte at -w 4 -l 3, or 2 bytes at -w 11 -l 5 is stil worth it.


this is one of the more internal changes.

If we have a successful response status, and if the "HttpWebRequest.responseOutputStream" property is non-null, just use the "HttpEntity.writeTo(OutputStream)" method to dump the response contents into it here.


this is where the responseOutputStream property is set.


Хотя, контрольный вопрос:
то, что с `self.` - атрибуты класса, их надо описать,
то, что с `self._` - скрытые атрибуты класса, их можно не описывать
Так?


this is a new overload of validateAndEmitResponse that sets the responseOutputStream property on the HttpWebRequest.


Typo in commit message. I meant to say that I *removed comma between copyright years and copyright holder*, not just that I "removed comma between copyright years".


lightning fast, as in the prototype

but the coloring got wrong, please try SDPS_2D_Map-no-grid7_nocmap.png, coords_intensities_50percent3.csv:

![ili](https://cloud.githubusercontent.com/assets/3242749/7904322/006d7432-07bc-11e5-8764-235237c539ca.png)

why are the colors red and green?


test resource files are in this commit to verify these text content files.
verifyRepeatingResponse just verifies that the xml files contain a repeating pattern of a specific text string (in UTF-8 format).


verifyBinaryContent decodes a response XML file that contains base64-encoded binary content and compares it to the original file (also in this commit).


When we get to 2016, I can update copyright years and the above commit message will be mostly invisible except for in the full commit log.


just test resource files from here on down.


is this meant to be commented out?


if i have an element like:
<input id="test-invalid" type="email" is="iron-input" invalid="{{invalid}}" />
and i type a wrong email the invalid property is false as expected.
Then if i clear the field then call validate method the result is true and the invalid property remain true.


Да, в большинстве случаев скрытые атрибуты можно не описывать, так как это внутреннее устройство класса, не предназначенное для непосредственного использования кодом снаружи. В самом же коде можно добавить комментарий для подобных полей, чтобы самому не забыть об применении и ограничениях их использования.


Unless you specify the integration, AKA `--use minitest` you get the noop / null integration that never kills a mutation.


Or (stable) `--use rspec` (seems you use minitest through).


I don't know if that's the correct behaviour; it's what happens in debian/ubuntu/archlinux/voidlinux (for my experience).

```
[Unit]
Description=ARP Tables

[Service]
Type=oneshot
RemainAfterExit=yes
ExecStart=/usr/lib/systemd/scripts/arptables start
ExecStop=/usr/lib/systemd/scripts/arptables stop

[Install]
WantedBy=multi-user.target
```
This is what happens on archlinux for example
```
#!/bin/bash

CONFIG_FILE='/etc/arptables.conf'

case $1 in
  start)
    arptables-restore < "$CONFIG_FILE"
  ;;
  stop)
    arptables-restore < /dev/null
  ;;
  save)
    arptables-save > "$CONFIG_FILE"
  ;;
  *)
    echo "usage: ${0##*/} {start|stop|save}" >&2
    exit 1
  ;;
esac

# vim:set ts=2 sw=2 ft=sh et:
```


We need to find a way to stop doing these hacks for containers without a parent.


ok - updated PR and removed extfraneous debug stuff I forgot before


The problem with your rules is that if you wrongly put a rule that brake your network and you have some script that reset your server when it's not reacheable it goes in boot loop since the rule you added end in the arptable configuration when the service goes down.


TeamCity LabkeyTrunk :: CDS postgres 9.3 [Build 36](https://teamcity.labkey.org/viewLog.html?buildId=252874&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 0 Build time: 00:04:53
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

```


Верно ли так описывать метод, ведь он же сам не делает все эти действия, а вызывает другие методы объектов.
Может, так и написать: вызывает метод разбрасывания яблок, запроса пользователя...
А то и вообще ничего здесь не писать кроме 208 строки


There is still a minor bug in the mute button behavior - the first two taps the volume is muted / unmuted, but the icon is not updated correspondingly. After that the icon is updated well.


@mikeroelens
Need to repeat the `<li>` tag here

```
<ul>
    <li ng-repeat="game in recentGameMessages track by $index">{{game}}</li>
</ul>
```


@keithbostic I can't see revision 32de75 in the commit history - could you point me at the change that introduced the bug?


Yup silly me.  Good catch!


I see SEGV with the SQLite binding under SGen and it happens during GC. I wonder why you had to add a GCHandle there to make something to work on MonoTouch?


Ooooh yes, it was linked to an issue i noted while updating the space invaders, and when shooting all the ships (and winning the game)

Not sure though how to remove this test, it's kind of like removing the condition in a recursive function, at some point when reaching the top of the tree there is no more ancestor.

> On 1 juin 2015, at 06:48, Jay Oster <notifications@github.com> wrote:
>
> We need to find a way to stop doing these hacks for containers without a parent.
>
> —
> Reply to this email directly or view it on GitHub.
>


Thanks. It gives me an error
```
bundle exec mutant --include lib --require surrounded --use minitest ::Surrounded
Could not load integration "minitest" (you may want to try installing the gem mutant-minitest)
```


Can you make this a pull request?


Почему ты не рекомендовал использовать `lambda` на занятии. Это кажется удобным: объявлять выражения вместо функций?


# Edit 01/06/2015


You guys are killing it

On Sun, May 31, 2015, 6:55 PM mikeroelens <notifications@github.com> wrote:

> Yup silly me. Good catch!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PureHacks/ponghacks/commit/d24ee81868205f0f629a34bc1d62b3cefa92a20f#commitcomment-11446036>
> .
>


Fixed

On Sun, May 31, 2015 at 11:40 PM, Theodore Alexandrov <
notifications@github.com> wrote:

> lightning fast, as in the prototype
>
> but the coloring got wrong, please try SDPS_2D_Map-no-grid7_nocmap.png,
> coords_intensities_50percent3.csv:
>
> [image: ili]
> <https://cloud.githubusercontent.com/assets/3242749/7904322/006d7432-07bc-11e5-8764-235237c539ca.png>
>
> why are the colors red and green?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ili-toolbox/ili/commit/2b2830f544562dc86784ce6a7576a9338f150994#commitcomment-11446001>
> .
>


I'm gonna test it, @miguelarauj1o


Fixes #6.


Minitest is not released right now. https://github.com/mbj/mutant/pull/330

I assumed you use this branch.


Please be careful with your commits - this wiped out the git blame for azi


Debian, Fedora and SuSE are not doing this, and what Arch is doing is their problem.

IP(6)tables, ebtables and arptables aren't daemons that cease to exist when stopped. They run in the kernel and are active as long as the relevant modules are loaded, or when built into the kernel, active at all times. What happens when you flush it on the stop event is that best case, for a brief period of time during shutdown the host is running without a firewall but still has network connectivity. Worst case, shutdown hangs somewhere and the host is exposed for longer.

It's no problem to leave the firewall rules behind when shutting down Linux, it's in fact the sane thing to do because removing the rules just makes the shutdown process take a little bit longer at the additional cost of having the firewall be dysfunctional for a longer period of time than when you don't flush the rules.

If your firewall lives in userspace and has some bookkeeping to do, it does make sense to start and stop it with the service manager du jour. But it'll also likely manage *tables itself, so nothing has to be done here to support it (if such a thing even exists for Linux).


To use examples from this repository you **should** to install dependencies with Composer first:

i think in fact, we don't have to install composer to use codes .
composer is only useful for people who what to install dependencies for unit test or contributing.

why not just add “how to install composer” in the section `Contribute`?
if we remove ``composer.phar``, i think we don't need ``composer.lock`` for the same reason

just one propose


You can correct this to:

```markdown
If you are using [Spring IoC](http://docs.spring.io/spring/docs/current/spring-framework-reference/html/beans.html) for dependency injection, you should configure your beans like the following code:
```


Lol thanks


Than -> Then


Also, we should link to 2.4.x version of Play docs.


Enqueued Google fonts in functions.php.  The "platform.twitter.com/widgets.js" was already in the theme, but I enqueued it in functions.php as well -- wasn't sure about any dependents, however, so I left that parameter blank.


Done!


Aren't lines 321-5 the same as this?:

```ruby
@@nameservers = @@hints.keys
```

?


I think we can just completely remove any mentions to the sync implementation. It was deprecated in the previous version.


Add a ":" to the end of line. ;-)


Dammit


this is hacky, we should just not mutate the original object.


There is no need to keep this as a static variable. Users can decide if they want this provider to be a singleton or not.


See #24 -- You're probably right, but this is a reminder that better benchmarks will make the trade-offs clearer.


Maybe "module" is better than "library".


Cool, thanks. I will double check on a few non-Arduinos to make sure nothing is broken there.


We can remove this plugin too.


Oops, a typo.


This must be updated to play 2.4.0.


Would love some [commentary](https://github.com/sindresorhus/awesome-scifi/issues/12) on this if anyone watched it. I haven't watched it yet.


Keep in mind that globals were deprecated in Play 2.4.0.


When we upgrade to play 2.4.0, this `@` will be removed in favor of configure the project to use injected routes.


we're in a .c file, so _cplusplus should never be defined.


This gets to the core of my objection.  For the poor developer trying to figure out how the ACO selector got on this page, there are no comments as to the purpose of this div.  Worse, the method of getting the selector in here is totally different from the existing methods used for the other fields in this same template.  The way to introduce a new templating system is to replace the old one (Smarty in this case), not to inject components of the new one from a totally separate piece of code into the existing and presumably outdated methods.

I just wasted 2 hours of my life trying to figure out what's going on here.  And I like to think I have a bit of experience as a developer.


There are some concepts that aren't adequately captured by an expression
(unfortunately). I'd just use type traits for the implementations of those.
On May 31, 2015 5:27 PM, "Eric Niebler" <notifications@github.com> wrote:

> @asutton <https://github.com/asutton> seems to frown on using type traits
> to define concepts. Here's one way to avoid that:
>
> constexpr struct _Target {
>     template<class T>
>     T const & operator()(T const &) const;
>     template<class T, std::size_t N, typename _This = _Target>
>     auto operator()(T const (&t)[N]) const -> decltype(_This()(*t));
> } __target{};
> template<typename T>auto __destroy(T const &t) -> decltype(t.~T());
> template<class T>
> concept bool Destructible =
>     requires(T const & t) {
>         {__destroy(__target(t))};
>     };
> struct not_destructible {
>     ~not_destructible() = delete;
> };
> static_assert(Destructible<int>, "");static_assert(Destructible<int&>, "");static_assert(Destructible<int(&)[4]>, "");static_assert(!Destructible<not_destructible>, "");static_assert(!Destructible<not_destructible&>, "");
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ericniebler/stl2/commit/17994b7ee068885f667cbe3489770aea3ae94e39#commitcomment-11445744>
> .
>


Why do we need this?


Since Play 2.4.0 only deprecated plugins, instead of completely remove then, we can keep this plugin (and also deprecate it).


Of course, the plugin should be adapted to use the DI support where it is possible.


C'est étrange... le merge avait fait ça ?
Parce qu'à priori, ça ne peut pas fonctionner sans donc...


thanks!

the spots look smaller than with the SVG vis. I guess they are 2 times smaller because for this example above earlier the whole space was filled when using spot border = 1.

can you please increase visualized spot size x2?


I'm going to try all of them. I'm pretty sure debian behaves like archlinux and archlinux behaves like fedora.


My network dropped before I could comment again. I must have misread the docs before. I'm trying the branch now.
Thanks for your help


Code reviews are never wasted time... It's like CME(Comtinued Medical education) for developers... Now you know how knockout works :)
To keep in line with smarty, would it be best to use a smarty function then?
Or more comments?


replaced some 'may' with other phrases where it didn't align with RFC2119's 'may'. RFC2119 'may's are now in uppercase.


NP, was just curios. I search GH for mutant users that do stuff "obviously" wrong, and comment in case.


Je l'ai vu en testant, puisque ça ne fonctionnait pas sans, c'est peut-être moi en recopiant des lignes.


fixed locally


I already fixed that once and even thought I checked it after the rollback of the positioning of the gui menus. I wonder how it got set back to name again?


If you don't like the convenience of having your rules saved and restored after a reboot, feel free to ignore the service and pretend it was never there.


added locally, needs review


@vlj nice. cube_mrt.elf also renders completely correct now.


Yes, that would make it easier to have the final list of cards in the
readme and cubes. There should be a better method of creating the release
which doesn't block changes, I'll look into improving this next round.
Thanks for checking.

On 22:46, Sun, 31 May 2015 ShawnieBoy <notifications@github.com> wrote:

> Off-topic, but shall I hold fire on submitting new cards until this build
> is completed?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/magarena/magarena/commit/162fac6917d78a502018480133177add71f8e423#commitcomment-11443811>
> .
>


I don't like Smarty, but since this page uses it then new fields added to the page should also use it.  Either that or the page should be totally re-written in a consistent and sensible way.


Hi sir,right now these hosts all cannot be resolved: gorillavid.in,movdivx.com,putlocker.com,sockshare.com,thevideo.me,tubemotion.com,uploadc.com.


Cool, I can imagine it being annoying!


You'll need the development version of grappelli_safe from Github.


@beefcurtains RTFS (read the fucking source) :-)

Fedora/RH behaviour is exactly like archlinux
http://pkgs.fedoraproject.org/cgit/arptables.git/tree/arptables-helper
I think that opensuse use the same service.

The sysv service in debian/ubuntu, also contained in the default package does what archlinux does; it flushes the rules.
```
#!/bin/bash
#
# init script for arptables
#
# Original by Dag Wieers <dag@wieers.com>.
# Modified/changed to arptables by
# 	Rok Papez <rok.papez@arnes.si>.
#
# chkconfig: - 16 84
# description: Arp filtering tables
#
# config: __SYSCONFIG__/arptables

source /etc/init.d/functions
source /etc/sysconfig/network

# Check that networking is up.
[ ${NETWORKING} = "no" ] && exit 0

[ -x __EXEC_PATH__/arptables ] || exit 1
[ -x __EXEC_PATH__/arptables-save ] || exit 1
[ -x __EXEC_PATH__/arptables-restore ] || exit 1

[ "$1" != "save" -o -r __SYSCONFIG__/arptables ] || exit 1

RETVAL=0
prog="arptables"
desc="Arp filtering"

start() {
	echo -n $"Starting $desc ($prog): "
	__EXEC_PATH__/arptables-restore < __SYSCONFIG__/arptables || RETVAL=1

	if [ $RETVAL -eq 0 ]; then
		success "$prog startup"
		rm -f /var/lock/subsys/$prog
	else
		failure "$prog startup"
	fi

	echo
	return $RETVAL
}

stop() {
	echo -n $"Stopping $desc ($prog): "
	__EXEC_PATH__/arptables-restore < /dev/null || RETVAL=1

	if [ $RETVAL -eq 0 ]; then
		success "$prog shutdown"
		rm -f %{_localstatedir}/lock/subsys/$prog
	else
		failure "$prog shutdown"
	fi

	echo
	return $RETVAL
}

restart() {
	stop
	start
}

save() {
	echo -n $"Saving $desc ($prog): "
	__EXEC_PATH__/arptables-save > __SYSCONFIG__/arptables || RETVAL=1

	if [ $RETVAL -eq 0 ]; then
		success "$prog saved"
	else
		failure "$prog saved"
	fi
	echo
}

case "$1" in
  start)
	start
	;;
  stop)
	stop
	;;
  restart|reload)
	restart
	;;
  condrestart)
	[ -e /var/lock/subsys/$prog ] && restart
	RETVAL=$?
	;;
  save)
	save
	;;
  status)
	__EXEC_PATH__/arptables-save
	RETVAL=$?
	;;
  *)
	echo $"Usage $0 {start|stop|restart|condrestart|save|status}"
	RETVAL=1
esac

exit $RETVAL
```


show de mais....


Пытаюсь сделать так, чтобы работало параллельно в терминале и на tk-окне. Получается, если закрывать окно вручную после отображения.


Съедается половина квадратика змеи, видимо, надо задавать координаты с 1 до 11, а не с 0 до 10


I like your changes. :)


fun variableLookup(variable :: String, env :: Binding) -> Value:
  cases (List) env:
    | empty => "enviornment does not contain variable"
    | else =>
      if variable == env.first.name:
        env.first.val
      else:
        variableLookup(variable, env.rest)
      end
  end
end


I don't remember this ever happening with iptables, and it's just plain wrong. As stated, feel free not to use the service and manage arptables yourself. Not flushing it when shutting down is not going to bite you at all.


Because the sys call returns TID of the current thread. One would get the thread id of the thread that created new one and not the ID of the new thread.


TeamCity LabkeyTrunk :: CDS postgres 9.3 [Build 37](https://teamcity.labkey.org/viewLog.html?buildId=252877&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 0 Build time: 00:04:47
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

```


No, it's ready to go. Thanks.


@MovingtoMars


You should submit this as a pull request to the original project. I'd like to have this in the main extension myself.


I think that the committers should comment on this. It's a quite critical behaviour. I also created a pull-request to fix the behaviour in ebtables.


Nice work finding this.    I'm checking with Google to see if there is a different fix.  But this one seems ok for now, except for a couple of tiny points.


Okay so past commit + this, thoughts on the attribute bit too I think it's kinda nice

@MovingtoMars


Oh god.. /etc/ is as directory for configs, `/etc/config.conf` is super redundant and non-program specific. I suggest reconsidering this name.


Forced garbage collection is a painful hit; it forces collection of every generation. As by Microsoft's suggestion, it may be appropriate after a large amount of memory is known to be needed. However, in a situation where a client has opened no forms or sent only few packets, a client that, on disconnect, would call garbage collection every time would be a huge hit to CPU usage each time at the benefit of only a few megabytes of memory given back that could have otherwise been given back by the garbage collector's schedule at a time that was likely better.


This was very good work.  Good job!


It's not critical at all, the service is just a convenience! And this is fucking arptables, I'm merely packaging it because it still exists in the kernel. It'll be replaced with nftables and when that day arrives, good riddance. There are ~6 forks of iptables in the kernel in total and they're all out of sync.


They're changes from ruby-build, not rbenv.


Documentation/2) Card Library - Effects.md

`whilePresent` is not technically a trigger. It's more a kind of "constant effect". It is not possible at all to use `heal` and `damage` effects inside `whilePresent`.


OK, so that's unfortunate. I'll try to figure out whether there's a file that exists on both machines.


I don't think you need to include jQuery since it's already being included by the bones theme (this will probably include it twice). The reason you can't access it is because Wordpress uses jQuery's [noConflict method](https://api.jquery.com/jquery.noconflict/) which is good if you're using multiple libraries that use the `$` selector. Since you don't, you can probably just Google how to remove that from the WordPress core code (see [this](http://stackoverflow.com/questions/17687619/is-there-a-way-turn-off-jquery-noconflict-mode-in-wordpress) question/answer on SO). Or, easier yet, just surround your JS with an anonymous function and pass in jQuery at the end. Like so:

    (function($) {
        // jQuery code
    })(jQuery);

More info here: https://codex.wordpress.org/Function_Reference/wp_enqueue_script#jQuery_noConflict_Wrappers


The few lines above could be written in a more pythonic way as:

```
index = int(index) - 1 # lists are 0-based in Python
if 0 <= index < num_sections:
    return [sections[index]]


Good catch!


yes this was a hack workaround a long time ago.

It requires core support for proper system.

Once core part is done the system works really good though! So removed scripts now to force projects someday to actually implement system.


To keep alignment, I would suggest `%02d`, but that nitpicking.


TG-8 #Closed


Looks OK, apart from the long list comprehensions.


Ok ça semble correct, j'ai pas fait tourner.


I would like to relax this in the future to allow users to use `.netrc` as an authentication method (and not specify any password or anything else on the command line, since those are visible in output of commands like `ps`).

(coursera-dl already does this and I have merged a contribution that uses the native keyrings of the operating systems---I have not yet tested that, but I plan on transplanting that to this code, if nobody beats me to it).


Oh, the code looks OK otherwise.


I understand, the code is far from perfect, but I prefer equivalent changes either to be very small or easily and quickly testable. There's a bit of an issue here, the test infrastructure is virtually non-existent. Until there's a good and reliable test infra with lots of tests, I'm gonna be very cautious about changes, especially those that aren't required. IOW, let's keep it as-is. But I'm OK with a change like this:
~~~
#ifndef PATH_PREFIX
#define PATH_PREFIX "/usr/local"
#endif
...
pinclude = SystemFileExists(PATH_PREFIX "/smlrc/include/", 0, NULL, "limits.h");
~~~
and a similar complementary change for library path.


@ethanlee16 No problem! Just learned that myself, so thanks for the lesson.


English nitpick: "who" is for people; "which" (or "that") may be the more appropriate here. BTW, I love documentation. :) Thanks for writing!


Oooh, I may have missed the changes in my earlier reviews, but shouldn't the person specify the platform that they are using to get the appropriate list of courses?


Great initiative!


Excellent.


Perfect. Thanks!


There might be a better way to write this,

but the ebonus +=0 is intentional.
It's just a was to make the if/else more readable, and avoid some compiler errors on empty braces.


Would this commit be suitable for devices with 3gb RAM?
Thx in advance.


solved with compile of swftools on redhat


OK. Will add them from here on in.


@jbrooksuk Need some help filling out the email contents lol. We need to translate all emails and missing some strings.


`#order`는 sql에서 정렬하는 거고, `#+`는 sql에서 받아온 애들을 다 array에 넣은 다음에 루비에서 `Array#sort_by`로 정렬하게 되는데, 후자가 전자보다 압도적으로 느림. 여기에 pagination을 한다고 하면, `.order.page`는 해당하는 페이지에 필요한 애들만 가져오는데 `reverse!.page`는 다 가져와서 뒤집고 필요한 애들만 걸러내는 방식이 되어서 퍼포먼스가 더 안 좋아지는 문제가 있음


`.order(created_at: :desc)`는 공통으로 쓰이는 거니까 `Story`의 scope으로 만들 수 있지 않을까?


간단하게 pagination도 추가해봅시당


> removes the user from

?!


`"creates the user's like on the story"` 로 쓰면 어떨까?


right


lol


그러면 story의 scope로, sql 차원에서 by 또는 of가 user인 스토리를 다 가져와서 정렬하는 걸 만들어서 쓰는게 더 낫겠군?


How would I retrieve a dummy `replyHeader`?


@gibiansky Would this work?


Hi, don't know what you did (or someone else), but as least the error is fixed now :-) Thanks


update myfork change display output


음 나는 `User`에 association으로 정의하는 걸 생각했어


some small이 아니라 그냥 다 뜯어고친것같은ㄷ..


@Hanarin **착각**입니다


User#favorites 처럼 하는거군. 이름은 feeds 정도로 할까?


feed는 swindler 내에서 이미 다른 뜻으로 쓰이고 있으니 `User#all_stories` 정도가 어떨까?


Just looked at that "New analysis" link.  Looks like it covers the 6th rank stuff reasonably.


new commit


should be able to run this through the same `imageRepoReference` filter as the build strategies, and the tag is baked into the name, depending on the `kind` of `from`


since `setup` calls `setup_environment`, this is redundant


IT ONLY TOOK 4 YEARS

BUT WE FINALLY FINISHED URIST'S ORES FOR HIM


Sorry, I didn't knew it. I'll fix it.
Thank you!


the js file is not injected into index html causing issue https://github.com/jhipster/generator-jhipster/issues/1549


This was done for #19


Finally I have the build working! This is ridiculous. I can sleep right now.


the lifespan of an entity is measured in ticks and every second has 20 ticks so ...

6000/10 = 300 seconds just like vanilla.


Up


@JanTvrdik I'll fix the version in config.h but otherwise I'd suggest making a reproducing case and submitting it upstream if there was some BC break in pcre, since we just bundle it, we can't really do much if something has changed upstream.


hello love


This is wrong, beacuse you have 39 of life, and the mob attack's you with 20, this is what the code do
39 - 20 = 19 // it means set life to 19
Then check if ((19-20) < 0) // because now curlp is 19... so get -1 and error...


Awesome


hm, librtapi_math needs to remain linked into rtapi, that was the point (src/rtapi/Submakefile)


here's a comment


and another


My bad! Glad to see they added it at some point, though


Shoot, sorry Dan, I notice that I cherry picked this one out of my master and I didn't get the subsequent commits that were relevant.  It looks like I've got a similar problem with the machines.exe.  I didn't mean to make work for you.


I meant interactive resume app   :-)


<!-- PMABOT:SOB -->
This commit is missing Signed-Off-By line to indicate that you agree with phpMyAdmin Developer's Certificate of Origin. Please check [contributing documentation](https://github.com/phpmyadmin/phpmyadmin/blob/master/CONTRIBUTING.md) for more information.


these should be rspec tests, I should've pointed to only follow the chapter in regards to creating the user.


Cool. Glad to have wasted time doing them.


I’m sorry you wasted time writing those test unit specs. I would’ve thought with the resources I posted regarding rspec both in slack and in email that it was clear we were using the rspec test suite.


Using `noHeader = error "No header created"` results in a user error.


And how could I do that and not notice when I did the build.  Well, I didn't do all of the builds after the cherry pick, just the windows one.  Which was a mistake, but particularly stupid because I was picking up code that had #ifdef platform's in it.  :-(


@maxogden
Thank you for your comment. :)
I also hope some hacker enable sound and bluetooth function on x205ta with linux!


It's fine, I should have read over the email again.


Referenciando os Casos de Uso 1, 2 e 3 #2 #3 #4


Referenciando o Caso de Uso 5 #6


https://github.com/DMOJ/hephaestus/commit/0948da9d8129c4f339c2276e2be6990e6ce5bba4 grr


Yeah, that causes warnings in the package since `rtapi_app` itself doesn't use those symbols.  Tests failed, and faced with the alternatives of quick fix vs. figure out whether linking `rtapi_math` to each comp even makes sense, I chose the former.


dldi*ddd* **dfsklfjdsl**


A lot better on the food priority. Next beneficial tweak I could see is not treat virus as blobs. It'll try to keep them at distance, rather than try and just not touch them.


pretty sure `tap` isn't on 1.8.7?


shockingly, it is:

```

$ chruby 1.8.7

$ rspec ./spec/runtime/inline_spec.rb:52
Run options:
  include {:focused=>true, :locations=>{"./spec/runtime/inline_spec.rb"=>[52]}}
  exclude {:sudo=>true, :realworld=>true, :ruby=>"=< 1.8.7", :rubygems=>"=< 1.6.2", :rubygems_master=>true}

bundler/inline#gemfile
  requires the gems

Finished in 1.58 seconds (files took 0.22469 seconds to load)
1 example, 0 failures
```


shockingly, it is:

```
$ chruby 1.8.7

$ rspec ./spec/runtime/inline_spec.rb:52
Run options:
  include {:focused=>true, :locations=>{"./spec/runtime/inline_spec.rb"=>[52]}}
  exclude {:sudo=>true, :realworld=>true, :ruby=>"=< 1.8.7", :rubygems=>"=< 1.6.2", :rubygems_master=>true}

bundler/inline#gemfile
  requires the gems

Finished in 1.58 seconds (files took 0.22469 seconds to load)
1 example, 0 failures
```


This change breaks at least `uri` skin. @horror -- is there a reason for it?


Done. Improved json EventStoreSerializer is here: https://github.com/pawelkaczor/akka-ddd/blob/master/eventstore-akka-persistence/src/main/scala/pl/newicom/eventstore/Json4sEsSerializer.scala


is -200 for the poison stay poisoning the player, this is not the hit that the cobra will give.


Not a problem.


@mary-kate: Pet peeve of mine but it will still work properly if I change the ```$wgBlacklistSettings``` array to be ```'//meta...'``` (protocol-relative) instead of using ```http:``` correct?


We'll see.


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


Es un commit importante, pero estuve laburando principalmente en tratar de armar algo parecido a lo que se mostro en clase sobre Batalla de Botes. Estuve leyendo mucho sobre como crear las funciones equals y hashcode. De paso agrege tests para lo que no se habian creado, al menos un par. Hay codigo que faltaria verificar en esta subida. En cuanto a Mapa, cambie un constructor sin parametros por uno con parametros, porque me parece que todo mapa se le necesita pasar una cantidad predeterminada de filas y columnas. Puede que podamos tener un constructor por default, pero tengo mis dudas. Aun asi, si creamos uno por default, lo unico que tenemos que hacer es usar el otro constructor pasandole un valor determinado como cantidad de filas y cantidad de columnas.

Por otro lado, si bien dijimos que ibamos a usar celdas, me estoy preguntando, ¿Son realmente necesarias?. Tenemos los casilleros que es la tabla de hash, que guarda un par de coordenada como clave de la misma, y en el valor, podriamos directamente guardar un edificio o una unidad, ya que por casillero solo va una cosa. Aun asi, es un tema debatible.


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


스크롤 해야하는 높이가 실제 스크롤 가능 높이인지 확인을 한번 더 했습니다.


r+


Source: https://github.com/SFML/SFML/wiki/Tutorial:-Create-and-Use-SetIcon


"matches" -> "match", but maybe clearer as "should return matches for a single condition for full-text conditions in ANY mode"


`chown -R $USER:$USER` 是错误的用法 比较严谨的写法应该是 `chown -R $(id -u):$(id -g)`
1. UID和GID一定相同
2. $UID和$GID存在一定可能获取不到 因此需要用 $(id -u) $(id -g) 的方法获取UID和GID


Thanks @pwendell, this isn't necessary for branch-1.4. I'll update this one.


ד


bfg clobbered repo, did not save much space yet but might eventually.


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


Hi @jmezetin I think I figured it out. For line 743, answering no for 570 should lead to 581
For line 663 answering no to 540 should lead to 554


Actually for line 663 answering no to 540 should lead to 584.


- Added small UI/UX Design enhancement. User can now scroll back to top


XGetWindowAttributes(), XWindowAttributes.depth


FYI `xml2` has been temporarily removed from CRAN. See my edit to `.travis.yml` in my refactoring branch to install it from github in the meantime. This will keep travis builds from passing unless you do something about it.

https://github.com/jennybc/googlesheets/commit/4c7d22543db679c35f1d5eb50b3574f4e7d34a14


Couldn't it be simplified to just (re)throw exception in a catch block?


can you upload your vsproj for this? cannot get this to compile for the life of me after rafi released the lib..


Couldn't it be done as an import?


looks like a spelling mistake


Encoding an encoded string will result in something different, since the percentage signs will now be encoded. That is, if I understand your last question correctly.


Nice!


Dont know if we want todo this,also
GetVersionAsString returns 3.1.0
GetVersionLongAsString returns 1.3.0

confusing....

And the openzwave version can be displayed from the hardware tab, no need to use this in the constructor of the webclass


1. $USER:$USER 这里属于手误，不过只要是owner了，组其实无所谓。
2. 测试下来暂时没发现有系统不支持$UID的，平时看见判断是否root的脚本大多也是直接用$UID，不过既然id -u更有保证，我已经改为id -u了，感谢建议。
3. 我目前尽可能做到不需要root也可以运行相关的脚本。


I want to mention that if you remove these getter and setters, then the ability to get the head and get the tail are not available outside of this class.  This is okay though because everything in the interface is still implemented and chances are that's how they will be accessing the data.


```
function chkenv() {
    stathub='/var/stathub'
    owner="$(id -u):$(id -g)";
    if [[ ! -d "$stathub" ]]; then
        echo "run \`sudo mkdir -p $stathub && sudo chown -R $owner $stathub\` fix";
        exit 1;
    elif [[ $(stat -c '%u:%g' "$stathub") != "$owner" ]]; then
        echo "run \`sudo chown -R $owner $stathub\` fix";
        exit 1;
    fi
}
```


Not sure 1.8 supports this, FWIW


Should be ATLConversationViewController instead of ATLConversationListViewController, am I right?


其实你上面提示中的命令跟脚本的是一样的，所以脚本失败了之后，就算用户马上执行这个命令也不会成功的。


You're a genius!
Thank you for making pocketmine and supporting it!!!
When I some extra money I will donate :)


Why is the author changed?


I was under the impression that *all* messages needed a header. However, headers usually have a `parentHeader` field which indicates what they're a reply to; a fresh message that isn't a reply probably should not have a `parentHeader` (that is, `parentHeader = Nothing`).

So, don't use `createReplyHeader`; instead just make  a new function that is just `createNewHeader` (and refactor `createReplyHeader` to use it). The difference between the two functions is that `createNewHeader` makes `parentHeader = Nothing` and `createReplyHeader` changes `parentHeader` from `Nothing` to `Just` the header of the parent.


xxx


Hey I would take this commit but it needs to be broken down into a smaller commit, k?


Hmm. Do line 98 and 99 actually work?
```javascript
.then(parseEpisodeCounts)
.then(markSeasonsWatched)
```
Both these functions take parms and we are not passing any?
```javascript
 function parseEpisodeCounts(counts) {
 function markSeasonsWatched(seasons) {
```


I've put them back as is, while I find a better solution.


Something is still weird.  After pulling I tried this, and it doesn't print anything:

```haskell
testVectorPipe :: IO ()
    -- Test that 'toVector' is polymorphic enough to allow
    runEffect $ each [1..5::Int]
            >-> (runToVectorP toVector >>= yield)
            >-> (await >>= lift . print)
```


I prefer having them on the same level because those loops mostly appear together.


I'm not sure what the purpose of a widget manager is. Could you explain what a widget manager is and why we need one?

Also, `widgetSendClose`, `widgetSendOpen`, and `widgetSendData` all share a lot of code. You could refactor them as such:
```haskell
widgetSend :: UUID -> Value -> MessageType -> IO ()
widgetSend uuid value message = do
  replyHeader <- getReplyHeader
  header <- dupHeader replyHeader message
  send $ case message of
    CommCloseMessage -> CommClose header uuid value
    ... -> ...
```
Then you just use it as
```haskell
widgetSend uuid value CommCloseMessage
```


Built-in Exceptions don't contain FileNotFoundError. Then need to use the IOError. From documentation:
"exception IOError
Raised when an I/O operation (such as a print statement, the built-in open() function or a method of a file object) fails for an I/O-related reason, e.g., “file not found” or “disk full”."


да, коммиты крутые!)


Wow, this looks really nice!


have changed


But it was not happening every time right? Becausebl when I checked last time some were correct...I guess it also depends on who wrote the docstring...
But yes with this fn you solve it once and for all :))


Uh uh uh :+1: :)


I see. Well, I'm using it to port some OpenBSD code to a 16-bit environment, so not really interested in fancy features.

BTW, I didn't have too much time to get familiar with it yet, but why does it produces redundant jumps around functions and data? seems it could do better...


hi sir,first thanks very much for your work on this project.And I find the host thevideo.me cannot be resolved right now.Can you fix this problem?Thanks very much.


why in system packages ?


adding note


@Kelketek Some data are exported into js using `name` - I decided it is way simpler to alias name to title than go through all those placees using `name`.


I'll take a look at it.


@Kelketek it's not a docstring - it's excuse for not making it a iterable :) It was an iterable initially, and than it constantly failed `if resources` check - empty iterable still evaluates to true


Please rebase and test against the latest...


master


@Kelketek same response here - I think I'll remove it in the upcoming commits to avoid rebasing everything


@Kelketek yep, it's removed in the following commit.


FYI licenses and readme are automatically included by npm


missing newline.


加入台東縣


/*
 * To change this license header, choose License Headers in Project Properties.
 * To change this template file, choose Tools | Templates
 * and open the template in the editor.
 */
package tbsandynaufalrizqon;

import java.util.Scanner;

/**
 *
 * @author Muhammad Sandy Alfianoor
 */
public abstract class Analisis_CCTV implements Warisan_jumlah_kondisi_posisi_sarana {

    Scanner input = new Scanner(System.in);
        private int jumlah;
        private String kondisi;
        private String posisi;

        public void setJumlah(int jumlah){
            this.jumlah = jumlah;
        }
        public int getJumlah(){
            return jumlah;
        }
        public void setKondisi(String kondisi){
            this.kondisi = kondisi;
        }
        public String getKondisi(){
            return kondisi;
        }
        public void setPosisi(String posisi){
            this.posisi= posisi;
        }
        public String getPosisi(){
            return posisi;
        }

    @Override
    public int analyticJumlah() {
        System.out.println("\nMasukkan Jumlah CCTV:");
            jumlah = input.nextInt();
            if(jumlah == 2){

            return 1;
            }
            else {

            return 0;
            }
    }

    @Override
    public int analyticKondisi() {
        System.out.println("\nMasukkan Kondisi CCTV : ");
            kondisi = input.next();
            if(kondisi.equals("BAIK")){
            return 1;
            }
            else{
            return 0;
            }
    }

    @Override
    public int analyticPosisi() {
        System.out.println("\nMasukkan Posisi CCTV : ");
            posisi = input.next();
            if (posisi.equals("dibelakang") || posisi.equals("disamping")){

            return 1;
            }
            else{

            return 0;
            }
    }

    @Override
    public void show() {
        System.out.println("\nJumlah CCTV :"+jumlah);
        System.out.println("Kondisi CCTV : "+kondisi);
        System.out.println("Posisi CCTV : "+posisi);
    }
}


okay


/*
 * To change this license header, choose License Headers in Project Properties.
 * To change this template file, choose Tools | Templates
 * and open the template in the editor.
 */
package tbsandynaufalrizqon;

import java.util.Scanner;

/**
 *
 * @author Ahmad Naufal
 */
public abstract class Analisis_KipasAngin implements Warisan_jumlah_kondisi_posisi_sarana{

    Scanner input = new Scanner (System.in);
    private int jumlah;
    private String kondisi;
    private String posisi;

    public int getJumlah() {
		return jumlah;
	}
	public void setJumlah(int jumlah) {
		this.jumlah = jumlah;
	}
	public String getKondisi() {
		return kondisi;
	}
	public void setKondisi(String kondisi) {
		this.kondisi = kondisi;
	}
	public String getPosisi() {
		return posisi;
	}
	public void setPosisi(String posisi) {
		this.posisi = posisi;
	}

    @Override
    public int analyticJumlah() {
        System.out.println("masukkan jumlah Kipas Angin : ");
		jumlah = input.nextInt();
		if(jumlah >= 2){
			return 1;
		}
		else{
			return 0;
		}
    }

    @Override
    public int analyticKondisi() {
        System.out.println("masukkan kondisi Kipas Angin : ");
		kondisi = input.next();
		if(kondisi.equals("baik") && jumlah == 2){
			return 1;
		}
		else{
			return 0 ;
		}
    }

    @Override
    public int analyticPosisi() {
        System.out.println("masukkan posisi Kipas Angin: ");
		posisi = input.next();
		if(posisi.equals("atap_ruangan")){
			return 1;
		}
		else{
			return 0;
		}
    }

    @Override
    public void show() {
       System.out.println("jumlah Kipas Angin : "+jumlah);
		System.out.println("kondisi Kipas Angin : "+kondisi);
		System.out.println("posisi Kipas Angin : "+posisi);
    }

}


/*
 * To change this license header, choose License Headers in Project Properties.
 * To change this template file, choose Tools | Templates
 * and open the template in the editor.
 */
package tbsandynaufalrizqon;

import java.util.Scanner;

/**
 *
 * @author Ahmad Naufal
 */
public abstract class Analisis_LCD implements Warisan_jumlah_kondisi_posisi_sarana {
    Scanner input = new Scanner (System.in);
	private int jumlah;
	private String kondisi;
	private String posisi;

	public int getJumlah() {
            return jumlah;
	}
	public void setJumlah(int jumlah) {
            this.jumlah = jumlah;
	}
	public String getKondisi() {
            return kondisi;
	}
	public void setKondisi(String kondisi) {
            this.kondisi = kondisi;
	}
	public String getPosisi() {
            return posisi;
	}
	public void setPosisi(String posisi) {
            this.posisi = posisi;
	}

    @Override
    public int analyticJumlah() {
        System.out.println("\nMasukkan Jumlah LCD : ");
            jumlah = input.nextInt();
            if(jumlah >=1){
            return 1;
            }
            else{
            return 0;
            }
    }

    @Override
    public int analyticKondisi() {
        System.out.println("\nMasukkan Kondisi LCD : ");
            kondisi = input.next();
            if(kondisi.equals("BERFUNGSI")){
            return 1;
            }
            else{
            return 0;
            }
    }

    @Override
    public int analyticPosisi() {
        System.out.println("\nMasukkan Posisi LCD : ");
            posisi = input.next();
            if(posisi.equals("dekatdosen")){
            return 1;
            }
            else{
            return 0;
            }
    }

    @Override
    public void show() {
        System.out.println("\nJumlah LCD : "+jumlah);
	System.out.println("Kondisi LCD : "+kondisi);
	System.out.println("Posisi LCD : "+posisi);
    }
}


/*
 * To change this license header, choose License Headers in Project Properties.
 * To change this template file, choose Tools | Templates
 * and open the template in the editor.
 */
package tbsandynaufalrizqon;

import java.util.Scanner;

/**
 *
 * @author Muhammad Sandy Alfianoor
 */
public abstract class Analisis_StopKontak implements Warisan_jumlah_kondisi_posisi_sarana{

     Scanner input = new Scanner (System.in);
	private int jumlah;
	private String kondisi;
	private String posisi;

	public int getJumlah() {
            return jumlah;
	}
	public void setJumlah(int jumlah) {
            this.jumlah = jumlah;
	}
	public String getKondisi() {
            return kondisi;
	}
	public void setKondisi(String kondisi) {
            this.kondisi = kondisi;
	}
	public String getPosisi() {
            return posisi;
	}
	public void setPosisi(String posisi) {
            this.posisi = posisi;
	}

    @Override
    public int analyticJumlah() {
        System.out.println("\nMasukkan Jumlah Stop Kontak : ");
            jumlah = input.nextInt();
            if(jumlah >= 4){
            return 1;
            }
            else{
            return 0;
            }
    }

    @Override
    public int analyticKondisi() {
        System.out.println("\nMasukkan Kondisi Stop Kontak : ");
            kondisi = input.next();
            if(kondisi.equals("BAIK")){
            return 1;
            }
            else{
            return 0;
            }
    }

    @Override
    public int analyticPosisi() {
        System.out.println("\nMasukkan Posisi Stop Kontak : ");
            posisi = input.next();
            if(posisi.equals("dekatdosen") || posisi.equals("dipojokruangan")){
            return 1;
            }
            else{
            return 0;
            }
    }

    @Override
    public void show() {
        System.out.println("\nJumlah Stop Kontak : "+jumlah);
	System.out.println("Kondisi Stop Kontak : "+kondisi);
	System.out.println("Posisi Stop Kontak : "+posisi);
    }
}


i want to change this sentense