Hi, you might want to redownload & reapply the patch, since they released a version 2 of the patch. They forgot to patch the cart.phtml template in the 'rwd' package:
http://community.magento.com/t5/News-Announcements/Update-to-SUPEE-6285-patch-for-CE-1-9-0-0-1-9-1-1/m-p/10379


Can't we just use the original free font ("Perfect DOS VGA 437"), instead of a charset dump? I don't think we can redistribute the charset dump, but the free font looks OK to me.

If there are any missing characters, we could simply skip them, or substitute with a question mark, and throw a warning in the console


Offset is depended of each bot.


Excuse me for any profanity that could possibly in my comment. I'm sorry for any feelings I might have hurt. As I have stated, this is merely a suggestion for a possible improvement to the code base that I propose for recommendation. I posted this comment with no ill intention and I hope the team can see that.

If a meeting is required to discuss my action, I'm all ears


> Poza tym, czy ta logika nie mogłaby zostać wyrzucona poza controller? Konkretne funkcje mogłyby sobie same sprawdzać parametry, w controllerze zostałoby minimum, i chyba o to chodzi.

Tak, właśnie o tym piszę, że pobieranie parametrów specyficznych powinno odbywać się na etapie obsługi specyficznej metody analizy, więc hash z wszystkimi parametrami powinien być dostępny dla tej metody.

> Jeżeli moje zdanie jest brane pod uwagę, jestem przeciwnikiem zostawiania czegoś na potem, bo na ogół to "potem" się nie wydarzy albo do tego czasu się o tym zapomni.

W ramach pracy inżynierskiej ma być robiony duży refactoring, więc jest mniejsza szansa, że się o tym zapomni - @Andriu01 dodaj komentarz TODO w kodzie.


This should be `cassandra-driver`.


XXXはINFOに修正要、氏名は自身のgithubアカウント名を


True enough! What do you suggest?


> Should the encoding influence the ability to use the range function?

Using numeric encoding for a range facet request is a per-requisite. Using `FacetEncodingType.STRING` worked since the used faceting requests so far against this field were discrete (aka counting unique values). So provided you want to change the existing tests to use range faceting on this field you would have to change the encoding indeed. However, I am a bit confused about what I am looking at here. Are there any test changes as well?


@eliasdc in your call to `py2app`, add the `--no-chdir` parameter.


So, every link from xstream doesn't work on this system.?
I'm using win7 and OpenElec for rpi2, I'm not sure which player OE usually uses but I don't have this problem.
Perhaps this is really caused by xbian or the omx player.


The SQL file was generated by PHPMyAdmin. I created the database in PHPMyAdmin and then exported the empty database to get SQL code which sets up the database.


este es un autocomentario al pull-reques


o_O why is this ?!


AFAIS this is an external API call, but it is in the **protected** section. This is wrong...

Please move it to the public section, along with the rest of Public APIs. Documentation of parameters is missing.

NAK


Full namespace with description saying that it returns $this


Oh, my bad. I'll fix it. Thanks


because I was trying things for later releases `1.1.0`. Do you want it?


param1, param2, param3 odnoszą się do kolejnych parametrów pokazywanych na osiach. Słuszna uwaga, więc przy okazji refactoringu należy nazwać je param_x, param_y, param_z itd.

@Andriu01 - dodaj TODO

Obecna wersja to generalnie przepisanie starszej wersji w Node.js i żeby to działało jak najszybciej nie zmienia się interfejsów...


the address part of swissearch is updated once a week, the rest once a year.
I would just check if the requested search pattern ``raron`` without clicking on specific result.


+1 Understood the unnecessary table created


And be careful of the LOG(INFO)!!!


In compiled binary the constants and macros are replaced by their value so no symbols for them. BTW should this function and define goes here?
If this is intended to be called in another function you could even mark it as inline.
Anyway this comment is all ok.


This API call is synchronous, you don't need the promise, didn't you?


There might be some confusion with that. Maybe getNavigateBackLink should be intented for override and navigateBack() shoult be private  or final


Neither store the transaciton for this matter


Then []='s error should say "2 or 3", right?


IIUC, we should alway have the whole object at this point. and i am not sure we need to check the `_r` against `op.extent.length`. if we should, a scaring `assert()` in `FillInVerifyExtent::finish()` will serve the purpose better?

see https://github.com/ceph/ceph/blob/master/src/osd/ECBackend.cc#L1638,


grr


+1. Inner join can be used.


Or, `sprintf(MyMenu->Option[1].Str, "Selected Theme:     <  %d  > ", theme_num);`
I'm amused if the themes pass the amount of 9.


@Dani723 Where do they have onSmth example? Cann't find.
Agree we should get rid of two-way bindings. I will take a look at the calendar component to see if it's easily possible


And the initialization weight for linear combination is 1/n....


This PR needs some means of verification, like an CLI call


What are you 'grr'ing about?


Thanks Stef. Yes, more detailed data would be useful, that's what I meant here: http://forum.textpattern.com/viewtopic.php?pid=289515#p289515. For example, article_save() function could pass to update_lastmod() the whole article array: update_lastmod('article', $rs); Up to plugins to decide what actions should be taken based on the update type (article) and/or data ($rs).


This could be inlined if you would like it be called only in a particular function.


Genius. :)


Not sure about this. But seems like SliceColumnArray should be used as you have mentioned above the function.


new comment 1


In this case, it doesn't matter because in the end I get the same thing. Otherwise, I'd use the getter.


@SubJunk Going up now :)


That`s why I wanted to have such change separated into some other task. We could do it completely then.


Hi Mrcl1450, your a300_pfp.fw file is producing an error at sync. "error: unable to create symlink huashan/proprietary/etc/firmware/a300_pfp.fw (No such file or directory)"


when i execute the  ./build.sh it tells this :
checking for a supported radio type
# - ERROR : radio type must be specified
# usage: ./build.sh.sh radio-type (component-name)
# valid radio types are: SDR1, USRP1, B100, B110, B200, B210, N200, N210
# (optional) valid component names are: libcoredumper, liba53, subscriberRegistry, smqueue, openbts, asterisk, asterisk-config, system-config

what should i do ?


I'd add pagination. The method would then take the offset or page argument and the number of items per page.


This function name could be changed, cause no "DevMode" in menu. Maybe "bootSysNAND". Also about the region lock, there is "Ver." string found in decrypted+patched FIRM (ARM9). So i don't know if that could be done just with patches applied (this is currently in the MyThread.c).


Yep, it should.


+1
I will not do it next time


I don't think so, `[]` requires 2:

```
operator [] (coll: Collection, index: Int) -> Element {
  coll get(index)
}
```

And `[]=` requires three:

```
operator []= (coll: Collection, index: Int, el: Element) {
  coll set(index, el)
}
```

Whereas `-` can be implemented for both unary and binary minus:

```ooc
Vec2: class {
  x, y: Float
  init: func (=x, =y) {}
}

// unary
operator - (v: Vec2) {
  Vec2 new(-v x, - v y)
}

// binary
operator - (v, w: Vec2) {
  Vec2 new(v x - w x, v y - w y)
}
```


+1 It's a good idea.


+1
i will remember it


I forgot to comment these lines, sorry.


+1
i will not do it again.


But overall, this collector shouldn't be doing this.

The published version shouldn't be determined from the working file, but rather from other published files. Actually, where is `os.walk` walking? How are you controlling the current working directory here?


I'll check.


Because we can! 顶！


No, it could be removed


There are tests already which use range faceting, which for some reason passed originally. With this change I could make this test happy again (`WebShopTest`) but two others are failing as I am seeing now. Still looking into it.


The find_each method uses find_in_batches with a batch size of 1000 (or as specified by the :batch_size option).


?
can u give me an example?


@DavideD I have the SolrAnalyzerTest passing now as well.


Why did you choose the Named Parameter Binding instead of the Prepared Statement Binding?


adding the content command


+1
i will remember it.


Can't get a specific class with generic type. Dozer can not get the type at runtime therefore the only solution is to set an hint as which type of object is inside the List in the mapping, and create a generic List converter.


哈哈，玩笑话，某一集《生活大爆炸》里 Howard 的台词


Please provide a javadoc comment @deprecated explaining why its deprecated


?
ABC was specified in the SQL induction queries.


The TableHeader instance could return an array of class names to be added.


Yussssss I'm so excited to have finally got PS4 working, with help from heaps of people


no space after if


Yeah, let's keep it as Basecamp. This framework was a collaboration between
@lifo and I conducted on company time (and with the additional help of
@javan). So that seems most fitting.

On Thu, Jul 9, 2015 at 1:34 AM, Rafael Mendonça França <
notifications@github.com> wrote:

> cc @dhh <https://github.com/dhh>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rails/actioncable/commit/d2ed66762b6272f49f075b1868eb6642c31f033b#commitcomment-12073889>
> .
>


dfsdfsdfsdfdfdsf


It is good, almost got one to test. It's been good seeing new guys helping out too


Sounds good. Lets go for that!


I am guessing this needs to go back in after the staff photographer work is done (in the UI changes branch)?


It's perhaps not provided by the minitest version we use.


My mistake. It should be timestamp.


@ektajaiswal3mar `brew update`


@NobodysNightmare fixed in 1d1d562


Please fix your email in your client so that github knows who you are. Are you hollander?


Yea I noticed that, fixed it.


Okee ;)


But we want to test the click. The click should zoom the map and go to the location.


I would make that 5s to be more on the safe side. But I guess this should be configurable on a per-controller basis and even could be disabled entirely.


For me, this commit works in Linux.
However, in Windows, using IPython 2.4 and 3.2 the ifsharp kernel crashes with
```
Unhandled Exception: System.IO.IOException: The process cannot access the file 'd:\Source\IFSharp\shell.log' because it is being used b
y another process.
   at Microsoft.FSharp.Control.CancellationTokenOps.Start@1234-1.Invoke(Exception e)
   at Microsoft.FSharp.Control.Trampoline.ExecuteAction(FSharpFunc`2 firstAction)
   at <StartupCode$FSharp-Core>.$Control.-ctor@520-1.Invoke(Object state)
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean
preserveSyncCtx)
   at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserve
SyncCtx)

   at System.Threading.QueueUserWorkItemCallback.System.Threading.IThreadPoolWorkItem.ExecuteWorkItem()
   at System.Threading.ThreadPoolWorkQueue.Dispatch()
```


Second commit


true


Do you think an error handling approach could have been used when dealing with database usage?


Then why the outer `numArgs!=2`?


[![LGTM](http://lgtm.in/p/VQ62)](http://lgtm.in/i/VQ62)

タイムラインに更新情報が表示されることを確認しました。

あなた宛のお知らせに出てこないようなので、そこのぶぶん調整してみてください。


Fixed Issue #1


it can take from very very fast to lots of seconds. It depends on the number of ticks you wanna move and the velocity profile you chose.
It can take 10 sec for instance.


Maximum number of theme is 9 so...


> However, I am a bit confused about what I am looking at here. Are there any test changes as well?

It's an existing test that was working with the encoding set to STRING.

The cause of the exception in this branch is
`LongRangeFacetCounts facetCount = new LongRangeFacetCounts(
				facetRequest.getFieldName(),
				facetsCollector,
				ranges
		);`
in `QueryHits.getFacetResultForLongRange(...)`.

The reason of the exception is that the new library now check that the facet doc value is of type NUMERIC.

@hferentschik I was confused because the test explicitly use STRING as encoding. Was it an error in the test or is it something that should continue to work?


Hmmm you're right, that would accept 2 arguments for `[]=`, which is bad.


Yes, I noticed that. So?


Ok.


I think that something is definitely weird. I spent  hours to debug jet_l3_correction_x.cc code, and I found that at line 150 you were using:
double refpt   =hrefpt->GetMaximumBin();
double jetpt   =hjetpt->GetMaximumBin();
which does not make sense. I also then had to changed the fit range and I did it manually to control.
Maybe I did not worked on the most update package. I rerun with:
double refpt   =hrefpt->GetMean();
double jetpt   =hjetpt->GetMean();
and the fits are good, so no need to change the code then.


> @DavideD I have the SolrAnalyzerTest passing now as well.

Cool, so now all the tests pass. Is there anything left?


Warning: you missed a spot and your change will get reverted when running `make test`
https://github.com/FStarLang/FStar/issues/276


现在出现暗藏


DevMode is inside "Advanced Options" Menu


seems to be missing a comma :)


Depends. The obvious thing is to how a modal or other UI element with error message. I'd also think about using local storage for unsynchronized tasks, so that user is not losing his work.


Forgets to unref ``mpi`` (so it'd leak).


Wouhou. Thanks


native scroll for android does not work after upgraded to 1.0.1,   the webview does not respond touch scroll event.
I enabled native scroll instead of Ionic js scrolling to improve scroll performance , it works well with last version,
.config(function($ionicConfigProvider) {
  if (!ionic.Platform.isIOS()) {
    $ionicConfigProvider.scrolling.jsScrolling(false);
  }
})

not sure It's an issue , anyone met the issue ?


The correct type here is VdpVideoSurface, though the generated code is the same.


@NobodysNightmare I think this mass-assigns the attributes, but I'm not sure.


> Cool, so now all the tests pass. Is there anything left?

There are quite a few test failures in the JGroup backend module. If you like, you might check out those unless @Sanne has gained any insight into these already?


It's a good idea. It would make the code cleaner.


@gunnarmorling ok, I will look at them


@NobodysNightmare [`ActiveRelation#merge`](http://api.rubyonrails.org/classes/ActiveRecord/SpawnMethods.html) helps here. I wasn't sure if it was _in scope_ (wortspiel) for this PR, so i created a separate PR #3225.


Nice, that seems to avoid the need to cast enum values. We should use that everywhere if it works on all compilers (I will try MinGW and MSVS tonight)


Yes I removed the commented out original lines, but why you do not want to include the new tau algorithm?
If we do not include it, we can't run with the tau.


test was the wrong way round; it rejects connections from peers we know.  has been fixed since.


PK_ for primary keys

UK_ for unique keys

IX_ for non clustered non unique indexes

UX_ for unique indexes

All of the index name take the form of
<index or key type>_<table name>_<column 1>_<column 2>_<column n>


Thanks!


Then it is okay.


please clarify/document the three possible states of checkconfig_result (constants? anyone?)


    return this || eval('this');

Would this work, or is *(1,eval)* necessary?


este es otro auto-comentario para el pull recquest de el cambieo de aciete


SaberMod toolchains are now on http://sabermod.com

To use them refer to this post
https://plus.google.com/+PaulBeeler/posts/MyeMTfMRMv8


Is there anyway to enable ipv6 again?


Discussion of using the shell for arithmetic in GNUmakefile's at http://stackoverflow.com/q/31302909


version 3


So either we put a very large value here, or we let it be configurable as well.


Updates are getting pretty ridiculous.


It should be used. It should throw an exception if something went wrong.


Hi Josh Bendavid,
                                         Since we changed the scheme from 4f to 5f, I am still wondring whether I am missing something or the cards are fine for production. Please do comment.

Cheers,
Sarmad


what about some constants for YADT_INIT_TYPE?


It didn't make it into the raspberrypi/firmware repo either, despite being included in the commit message there as well.

Perhaps there is a race condition - the vga666 commit only hit the tree at 17:27, and the firmware was pushed at 18:36, so it was probably close to the wire.


> I will try MinGW and MSVS tonight

From what I could read, it should be ok on MSVC > v10 but your tests will let me know more about it. :)


OK


Hi Matt, did you use any NSW plates in this training? Do you know what is the accuracy rate for NSW plates?


Woahoooo!


This can cause `alt="nil"` to be generated. Needs a `nil?` check.


s/``/$()/


Nice Nyall! Regards


On 07/08/2015 02:26 AM, Christophe Troestler wrote:
> Shouldn't a dependency be required /iff/ the lib/exec is built? In case a section build is conditional, the dependencies should be optional (if the default property is false) — the install field is still true but the lib/exec won't be installed anyway.
>

You are right, the `Install` field has nothing to do with build dependencies.

There might be a situation where the `Install` field should be taken into consideration: build vs runtime dependencies for libraries.
If a library (and its reverse deps) are not installed, it should only be a build dependency.


I created a new brach and made a new pull request for the binary display as
"main.c",,.... I hope its better and can be taken

On Mon, Jul 6, 2015 at 1:16 PM, David Muriuki <notifications@github.com>
wrote:

> Please clean up your repo before creating a pull request. You have a lot
> of back-up files. check how to ignore them. rectify this and may be...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MAKOCHIENG/sph336_june2015/commit/0d39c431a42364c58272d0e785d736da05b48842#commitcomment-12014836>
> .
>


I checked the code and changed the patch to simply remove the single case in the function where s is checked for non-NULL. This way covertity won't complain.


Yes, you can enable it on a per interface basis using connmanctl. The patch above only changes the defaults for newly detected/created interfaces.

For example to enable ipv6 auto-configuration on the Ethernet interface:

connmanctl> config ethernet_b827ebd04564_cable --ipv6 auto

(Replace ethernet_b827ebd04564_cable with the correct string for your system found using the services command)


This is a really nice feature file.  I hope you enjoyed writing it as much as I enjoyed reading it.


Just to inform, after this commit the command  `import csv` returned to giving warnings on every record.


@thibaultCha can this be `return false, err` ?


Should it be "How do you believe me?" And not something like "Don't believe me?"


The last sentence should be something like "It's been a really long since I've met someone new."


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

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


@chrisze any chance you remember why we need this? I'm considering to update to the latest version of the library.


972154-NewSimpleChilledCeiling (RKStrand) - i386-Windows-7-VisualStudio-12: OK (1682 of 1682 tests passed)

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


Nice, that makes the logic much clearer.


No because the BaseDao needs to know where is the issue coming from. Because an error from the second return value is a Schema issue for sure. But an error from a `self_check` might be an error of any type.

See https://github.com/Mashape/kong/blob/master/kong/dao/cassandra/base_dao.lua#L357


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


it is riffed off the language from the funnel image


Suggest: separate the two URLs in two lines.


Yes I agree it should, this was added before `GetRemoteTag` was added to the v1 session


Ok thanks for confirmation, will update shortly


Your joystick is probably broken then, there's no way 2 different buttons have the same value. I reverted the commit.

I tested here, if the Analog red light is off, then both left analog axis and d-pad have the same value. But, like I said, the config is only valid and make sense when the light is ON.

Someone need to update the wiki, about generic joypads. They need to have this analog light ON, to get the right config.


Yeah... but who knows, maybe we're on some other JVM operating system, of a totally unknown origin, somewhere in year 2476 on board of spaceship heading towards the galaxy of Andromeda, and which uses `"\\"` for its file separator too, as Windows does now...then what? Besides, this is more explicit. So I did that... but thanks for the advice! :D


Ah, coo coo.


Well, you're not using `map` function, so maybe a for-loop? Seriously.
Especially cuz in this commit you've lost your "one-lineniness"


I was wondering if there was a reason that the ABC1 logo really needs to be all-white. Technically, the logo has shading, and I understand that many logos could have shading and don't. But unless there is code that takes the white and changes its color for other uses, what's the harm in using the shading? https://en.wikipedia.org/wiki/ABC_(Australian_TV_channel)#/media/File:ABC_TV_2015_logo.png


The upload derped? Weird..


redundant


Right, I should have been more specific -- bare parens in patterns are
deprecated and will eventually result in errors.

On Thu, Jul 9, 2015 at 4:36 PM, Robert Virding <notifications@github.com>
wrote:

> Bare parens will keep working, it seems too idiomatic to drop it. Quoting
> it of course works. What will never work is using nil as the empty list,
> nil is the atom nil and never the empty list.
>
> Robert
>
>
>
> On 9 July 2015 at 22:04, Eric Bailey <notifications@github.com> wrote:
>
> > Using bare parens to represent an empty list is deprecated; in a future
> > version of LFE attempting to do that will actually result in an error.
> >
> > Ah, great to know!
> >
> > There are two guides for LFE
> >
> > Cool, I'll check them out and maybe fork locally.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/lfex/xlfe/commit/2c58947d39924372a477e2c7059c03884b5775da#commitcomment-12092595
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lfex/xlfe/commit/2c58947d39924372a477e2c7059c03884b5775da#commitcomment-12094298>
> .
>


@iam4x `atom.project.getPaths[0]` returns`/home/vramana/Documents/work/prizetracker` which is exactly my project directory. How do I find out `cwd` from devtools?


Great refactoring of onStart(). However, this commit has both code refactoring and added feature (consent form + email). Is it possible to separate them into two commits? Let me know if it's possible.


thanks for dropping in and asking though!

How ya doing?


1) on your mac
2) development
3) mac


I did it in my fork (but I need to test). BTW: it is not really different.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.328](http://teamcity.monogame.net/viewLog.html?buildId=8603&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:07:46


The tricky bit here is that you might not need `alien` depends on what your attempting to do.  For example building locally in tree doesn't require it nor is it needed to generate rpms.  It would probably be better to only perform these checks when running configure on a Debian style system.


It means: Storage in the cloud. The cloud is typically defined as accessed
over the Internet. In this case storage over the Internet.
On Jul 9, 2015 7:54 PM, "Steve Mao" <notifications@github.com> wrote:

> What does "cold storage" mean?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Famous/famous-react-DEPRECATED/commit/0620144dcc268dbdec8f42378b6904ced9d5d1f0#commitcomment-12096267>
> .
>


P good. Cascadia sounds pretty rad : )


We use `gulp-autoprefixer` as part of our build process. But Javascript-injected styles [are not managed by this process] must be manually prefixed. I think only `transform` is the issue... at least in this case.

> Here are the [AutoPrefixer Rules](https://github.com/postcss/autoprefixer-core/blob/master/data/prefixes.coffee) in you want to review these.


Take tomorrow off and come!!!


Cold storage as in we might take it out of the Famous repo


Yes, it caused me quite a bit of confusion when I was validating the rollback, as my local 3.1 deployment also had the bug. I doubt we will need to patch or update 3.1 again, but I agree it would be better to fix this proactively, regardless.


@vramana Published new version can you quit Atom and `$ apm uninstall linter-eslint && apm install linter-eslint` ?


This is breaking the check somewhere else when adding the plugin via the Admin API.


According to [Disable Dashboard in OSX 10.9 Mavericks](http://akenn.org/blog/disable-dashboard-mavericks/), setting ```mcx-disabled true``` since OS X 10.9 will actually perform the opposite goal.


develop (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


```
Unit* target = GetHitUnit()->GetCharmerOrOwnerOrSelf();
target->CastSpell(target, uint32(GetEffectValue()));


This brings /Url/Resolver up to 100% coverage.


This do-block looks wrong to me. Should't the print statement be wrapped in braces?


If I call `MediaPlayer#reset`, I get an exception thrown here because this function will be called twice (from `this.attachSource(null);` and `this.attachView(null);`) and on the second run through, `protectionController.protectionModel` will be `undefined`.


My mistake cold VS cloud ☺
On Jul 9, 2015 8:19 PM, "Myles Borins" <notifications@github.com> wrote:

> Cold storage as in we might take it out of the Famous repo
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Famous/famous-react-DEPRECATED/commit/0620144dcc268dbdec8f42378b6904ced9d5d1f0#commitcomment-12096470>
> .
>


`'offline'` is duplicated. Is it OK?


@tornoz : Hey, I tweaked the [br] files while removing Tipeee service. Can you check if I didn't broke a sentence in 'breton' while removing the link and changing the string ? Thank you :)


@iam4x I am still getting the errors If I open from Ubuntu Launcher. I guess I have put `console.log` statement and see find what the problem is...


mislim da je tu problem. +point ne radi sa svim formatima, SPLINE ima drukciji format tocaka nego lines. npr. ima razmak izmadju X i =


opt: `require` will not throw an error if the package is not installed. I generally prefer `library` but I will leave it up to you.


before_saveのコールバックにより、DBへの登録前に全て小文字にするので、
小文字のメアドが一意でなければ登録前に弾かれます。


Track uri feature in nginx.


develop (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (652 of 652 tests passed)

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


Well, I think it is too permissive. Editorconfig is a good way to force coding style in IDEs, if features are used.


This should be `if (yes) yes.click()`.  Otherwise it will throw an exception.


지나가다가 우연히 발견하고 몇글자 코멘트 적어봅니다 ㅋㅋ 저번 수업시간에 리뷰 생략했으니 그 리뷰 한다고 생각하고...

1. size_t의 경우는 제 개인적인 경험으로는 for문 돌 때 int로 쓰는게 유용한 경우가 많았습니다. 일단 size_t 든 int든 컨테이너의 원소 개수가 21억개를 넘어가기전에 일단 메모리가 터질게 거의 분명하기 때문에 오버플로우 문제는 크지 않고, size_t를 쓸 때 느낀 가장 큰 불편함은 역순으로 순회를 하는게 힘들다는 거였어요. for(size_t i = size-1; i >= 0; i--) 이런식으로 순회를 해야하는 경우들이 있는데 이렇게 짜면 size_t는 unsigned라 오버플로우가 일어나서 프로그램이 터져버리죠. 이건 그냥 참고삼아.. 상황따라 다른데 개인적으로 그냥 순회면 경고 안뜨게 size_t 쓰는게 좋다고 생각해요. 전 걍 int 쓰지만..

2. end(members)는 안되는게 당연합니다. C++에서 그냥 배열은 프로그램 실행중에 그 끝 지점의 위치를 파악할 수 없어요. end는 해당 컨테이너의 맨 끝부분을 가리키는 반복자(*포인터의 좀 더 일반화된 개념이라고 생각하시면 됩니다)를 리턴하는데 일반 배열은 그 끝 위치가 저장이 되지 않으니 끝 지점을 리턴할 방법이 없습니다. 이런 boundary check가 안된다는 배열의 문제점 때문에 C++ 11이상부터는 정적 배열의 경우 std::array를 쓰는걸 권장하고 있습니다. std::array<std::string, 10> arr = {...}; 같은 식으로 사용. 자세한 사항은 레퍼런스(http://en.cppreference.com/ 이 사이트가 표준 참고할 때 굉장히 좋습니다)를 참고하는걸로 ㅎㅎ

3. C에서는 포인터 p에 대해서 *p++이라는 코드를 많이 썼었는데 C++에서는 별로 권장되는 방법이 아닙니다. 특히 반복자의 경우 어떤 반복자 it에 대해 it++이라는 코드 자체가 효율이 떨어질 수 있어요. 보통 반복자 순회는 ++it로 쓰는 걸 더 권장합니다. 컨테이너 순회에 대해 제일 권장되는건 range-based for문 이라는 문법인데 이 내용도 한 번 찾아보시면 좋을 것 같아요. 타 언어에서 심심찮게 찾아볼 수 있는 문법이라 보면 금방 이해하실 듯. 그리고 it++보다 ++it가 권장되는 이유는 it++은 값을 증가시킨 뒤 기존의 값을 리턴하는 방식으로 동작하는데, 이렇게 하려면 '기존의 값'을 저장하기 위한 임시 객체가 생성되어야만 합니다. 기본 타입(int, char 등등)의 경우 이 부담이 크지 않지만 객체와 같이 큰 규모의 타입인 경우 이 임시 객체의 생성으로 인한 성능 저하가 발생할 수 있기 때문에 주로 ++it를 쓰는걸 권장해요(++it는 값을 증가시킨 후 그 값을 반환하기 때문에 임시 객체를 생성하지 않음)

4. sizeof(members)/sizeof(*members)는 보통 sizeof(members)/sizeof(members[0])으로 많이 쓰더라구요(*members 보다 members[0]이 원소 하나를 가리킴이 좀 더 명확히 보이기 때문인 듯). 그리고 Visual C++에서는 같은 역할을 하는 _countof 라는 매크로가 있습니다(Visual Studio에서만 사용가능함. 표준 X. 참고삼아..).


ip need not be mutable. Eg: let ip = if count == 2... else if count != 1.... else "0.0.0.0".to_string()

Additionally, if count != 1 case is reached, we should probably return early from the program


Isn't this line the label that is called by :ref:?


FYI, you can change the default branch https://github.com/NETMF/llilum/branches.
See https://help.github.com/articles/setting-the-default-branch/.


No problem

On Thu, Jul 9, 2015, 1:45 PM thequixotic <notifications@github.com> wrote:

> That is true, we can avoid a few non useful procedure calls. It will be
> changed. Thank you for noting. I just did it that way out of sheer habit of
> using collections.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/thequixotic/appinventor-sources/commit/f1918a4d38585fc2a21288d3151f1fdca5ae3854#commitcomment-12093364>
> .
>


I think that everybody knows what that button does, but including that info will not hurt.

In my case with the config you just updated:
-Analog ON: both analog works, d-pad not (as expected)
-Analog OFF: Neither analogs works (this is expected) but the d-pad doesnt work also.

With my config:
-Analog ON: both analog and d-pad works -> This is what I use.
-Analog OFF: neither works.

No problem, I will just use my config.


nit: Maybe s/way/much. Slightly less US colloquial but also less fun.


this is kinda wordy.. how about something like: 150 samples (10.0%) where 5 <= x < 10


Good to go. I noticed in some systems, the title gets rendered twice. Are you still seeing that?


지나가다가 우연히 발견하고 몇글자 코멘트 적어봅니다 ㅋㅋ 저번 수업시간에 리뷰 생략했으니 그 리뷰 한다고 생각하고...

1. size_t의 경우는 제 개인적인 경험으로는 for문 돌 때 int로 쓰는게 유용한 경우가 많았습니다. 일단 size_t 든 int든 컨테이너의 원소 개수가 21억개를 넘어가기전에 일단 메모리가 터질게 거의 분명하기 때문에 오버플로우 문제는 크지 않고, size_t를 쓸 때 느낀 가장 큰 불편함은 역순으로 순회를 하는게 힘들다는 거였어요. for(size_t i = size-1; i >= 0; i--) 이런식으로 순회를 해야하는 경우들이 있는데 이렇게 짜면 size_t는 unsigned라 오버플로우가 일어나서 프로그램이 터져버리죠. 이건 그냥 참고삼아.. 상황따라 다른데 개인적으로 그냥 순회면 경고 안뜨게 size_t 쓰는게 좋다고 생각해요. 전 걍 int 쓰지만..

2. std::end는 컨테이너의 제일 끝 원소가 아니라 제일 끝 원소의 다음 위치를 리턴합니다. 여러 가지 이유때문에 이런 방식으로 구현이 되어있는데, 일단 올바르게 고치자면 it < std::end(members)가 맞습니다. it <= std::end(members)라고 해버리면 boundary를 넘어가게 돼요.

3. C에서는 포인터 p에 대해서 *p++이라는 코드를 많이 썼었는데 C++에서는 별로 권장되는 방법이 아닙니다. 특히 반복자의 경우 어떤 반복자 it에 대해 it++이라는 코드 자체가 효율이 떨어질 수 있어요. 보통 반복자 순회는 ++it로 쓰는 걸 더 권장합니다. 컨테이너 순회에 대해 제일 권장되는건 range-based for문 이라는 문법인데 이 내용도 한 번 찾아보시면 좋을 것 같아요. 타 언어에서 심심찮게 찾아볼 수 있는 문법이라 보면 금방 이해하실 듯. 그리고 it++보다 ++it가 권장되는 이유는 it++은 값을 증가시킨 뒤 기존의 값을 리턴하는 방식으로 동작하는데, 이렇게 하려면 '기존의 값'을 저장하기 위한 임시 객체가 생성되어야만 합니다. 기본 타입(int, char 등등)의 경우 이 부담이 크지 않지만 객체와 같이 큰 규모의 타입인 경우 이 임시 객체의 생성으로 인한 성능 저하가 발생할 수 있기 때문에 주로 ++it를 쓰는걸 권장해요(++it는 값을 증가시킨 후 그 값을 반환하기 때문에 임시 객체를 생성하지 않음)

4. sizeof(members)/sizeof(members)는 보통 sizeof(members)/sizeof(members[0])으로 많이 쓰더라구요(members 보다 members[0]이 원소 하나를 가리킴이 좀 더 명확히 보이기 때문인 듯). 그리고 Visual C++에서는 같은 역할을 하는 _countof 라는 매크로가 있습니다(Visual Studio에서만 사용가능함. 표준 X. 참고삼아..).


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


Got it!


bind takes a type that implements `ToSocketAddrs` trait. It is probably nicer not to have to concatenate a string and port number together. See [this page](https://doc.rust-lang.org/std/net/trait.ToSocketAddrs.html). In particular, one good option is `(&str, u16)`. Ditto to wherever else you may be doing this.


ㅇㅕㅏㅕㅏ


래애러ㅇ


@ariya @Vitallium Please port this to master...


RadianceRefactor (macumber) - Win64-Windows-7-VisualStudio-12: OK (2178 of 2204 tests passed)

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


i know it's probably my fault :crying_cat_face:  :crying_cat_face:  :crying_cat_face:  :crying_cat_face:  :crying_cat_face:  :crying_cat_face:  :crying_cat_face:  :crying_cat_face:  :crying_cat_face:  :crying_cat_face:  :crying_cat_face:  :crying_cat_face:  :crying_cat_face:  :crying_cat_face:  :crying_cat_face:  :crying_cat_face:  :crying_cat_face:


Removing any other instances of `mono` or `mcs` throws the following after a `make`:

make --no-print-directory -s NO_DIR_CHECK=1 PROFILES='binary_reference_assemblies net_4_5 xbuild_12 xbuild_14   ' CC='gcc' all-profiles
mkdir -p -- build/deps
make[6]: mcs: No such file or directory
make[6]: *** [build/deps/basic-profile-check.exe] Error 1
*** The compiler 'mcs' doesn't appear to be usable.
*** You need Mono version 3.8 or better installed to build MCS
*** Check mono README for information on how to bootstrap a Mono installation.
/bin/sh: mcs: command not found
*** The version of 'mcs' is: .
make[5]: *** [do-profile-check] Error 1
make[4]: *** [profile-do--basic--all] Error 2
make[3]: *** [profiles-do--all] Error 2
make[2]: *** [all-local] Error 2
make[1]: *** [all-recursive] Error 1
make: *** [all] Error 2
fulvio@Fulvios-MacBook-Pro:~/mono$


This revision is clean.


Why did you remove  margin-bottom?


develop (kbenne) - i386-Windows-7-VisualStudio-12: Tests Failed

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


No, we would not hit a crash:

* The globset data_blob used in GLOBSET_parse is initialized to `buffer_length - 16`, hence 0 at https://github.com/openchange/openchange/blob/jkerihuel/fxbuffer-parser/libmapi/idset.c#L386

* In GLOBET_parse, we initialize parser with talloc_zero and immediately hit the first condition in the while loop, setting parser->error to 0 and then stepping out of while, setting range to NULL and exiting afterwards:
https://github.com/openchange/openchange/blob/jkerihuel/fxbuffer-parser/libmapi/idset.c#L252

We would show the OC_DEBUG message on https://github.com/openchange/openchange/blob/jkerihuel/fxbuffer-parser/libmapi/idset.c#L253 which is not relevant. A sanity check at the beginning of the function would indeed be much appropriate.


@iam4x Also when I am working in a project the `.eslintrc` unless it magically disappears in between there is no need to recompute it's file name every single time.


develop (kbenne) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


I presume this is to avoid compiler warnings - wouldn't removing the parameter name from the signature also fix the warning?
<pre>
} catch (Common::Exception &) {
 		delete[] compressedData;
 		throw;
 }
</pre>


This should be `alias /home/mywagtail/app/media/images/;`, going by the comment.


I just attempted to make the change to "Delete" and create another pull request, but something's not working right. I am new to Git, so likely I'm missing something here.. anyway, please just change it to "Delete" and you can go with that for now. Cheers.


rax retry tests


JSON files with the same content as the large/medium/small-dictionary.xml files


Thanks, fixed


Don't think sudo is required, is it?


why the change?
generally rhel images shouldn't be shared


why the indentation?


I just noticed this yesterday too! Thanks for fixing :)


extra white space on these two lines.
found by git diff --check


Why  removing margin-bottom?


Would it *kill* you to separate formatting changes into a separate commit? ;)


View this rendered... It doesn't display as expected. Eg $ dissapears.


Why md-6 ?


Steve,
  I got the delete player account for free, all I had to do was write the test.  I do want to make sure only the given player can delete his account, not another player.  We can talk about this on Saturday.  I am moving onto try to understand the Pusher gem because this is what I need to present a game screen to different players.


Uploaded Adobe Illustrator as source. I use illustrator but I tested it and it can be (very surprisingly) opened in Inkscape... :)


i believe it's fine. but would like to double confirm if there's any bad consequence of allowing this?


Try to get the vector's iterator (`iter()`), and see the `filter` method on `Iterator` trait.


This is not "backwards compatible" for me. `python trivial.py TrivialPgLoad` fails

```
# python trivial.py TrivialPgLoad
Traceback (most recent call last):
  File "trivial.py", line 58, in <module>
    luigi.run(main_task_cls=TrivialPgLoad)
  File "/usr/local/lib/python2.7/dist-packages/luigi/interface.py", line 345, in run
    cmdline_args.insert(0, main_task_cls.task_family)
AttributeError: 'NoneType' object has no attribute 'insert'
```
While `luigi --module trivial TrivialPgLoad` with the same code works OK, as long as I have trivial.py in my $PYTHONPATH.

I don't mind doing it with luigi cli, but a breaking change shouldn't be advertised as backwards compatible. I'd have to rewrite some cron jobs in order to use this version.


Код пока что повторяется в 3-х местах потому что ещё не сделала пару IP и порт.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.329](http://teamcity.monogame.net/viewLog.html?buildId=8607&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 516, ignored: 6 Build time: 00:23:13


I'm trying to understand how our process broke down here. This breaks the build environment, and was never tested. We need to ensure that we never modify master of any branches without testing it first.


and if `x-schema:` doesn't look like what we must allow. we may make it `x-x-schema:`. thoughts?


I have just come across this - it looks like a bug that this is `this.tools.spinnerToolItem...` as opposed to `this.spinnerToolItem...` when comparing to https://github.com/aodn/aodn-portal/commit/b01df3c147e43b360627ae18ce9ec011aaee8bd6#diff-ea5b70dadbc5d0197c651c4c74087164R66.

Likewise for https://github.com/aodn/aodn-portal/commit/b01df3c147e43b360627ae18ce9ec011aaee8bd6#diff-ea5b70dadbc5d0197c651c4c74087164R60.


You're right, it would. In fact, the UNUSED() macro does exactly that. I'd just, generally, like to have the UNUSED as a visible marker, to make clear that yes, this parameter is not used and that's correct.

In this case, it's not really even important that we're catching a Common::Exception &, it could just do catch (...). Originally, I probably thought I wanted to add something to the Common::Exception, but then didn't.

You see, I'm misusing exceptions a bit, as a container for a stack of user-visible error messages. Each time when one is caught, the catcher can add a string to the stack, and the final catch can then print the whole stack. It's not quite how they were meant to be used, but I think it is kinda neat.


I made the changeI mentioned, so this works great now without the possibility of a double loop running.


thank you!
by the way, you may want to write your signature in the `%changelog` section.


I have a question regarding your comment - the second output value is always `nil`, can't this exception be handled on the `BaseDao`, and keep the `validate_entity` function consistent with a return value of `ok, err` ?


Actually, take that back, looks like when the config is applied its add the tools to the `this.tools` object.


This is similar to the unmerged work I've done in #332 to work around the loss of `Option`.  This will have to be reconciled at some point, but this is the bugfix, so I'll deal with the merge.


@ItsMeAndrew This should now output a ROW_CLICKED event with the data of that row


@DrMcCoy I don't really think it's misusing; C# does similar with ``Exception.InnerException``.


This revision is clean.


> Would it kill you to separate formatting changes into a separate commit? ;)

Nope! Mostly because I'm already dead though :( (3AM here).

But in all honesty.. yeah, I should, and usually try to. Was pissed to find so many badly-formatted ifs and fors and just went `:%s/if(/if (/g` on the whole file.


I love when things are this easy to review


There are 180 Builds Since the first Expansion.
Cataclysm Should be the 3th expansion.

180 * 3 = 540,
12340+540 = 12880


If it's not available to download at all, kind of pointless to keep it in there.


Sure enough, very similar. Maybe you can just `git checkout` your code when it comes time to merge: it looks fine to me and I suspect it will be uniform across backends.


Before, I took advantage of the fact that Handlebars would render the array `tsInterfaces` as a comma separated list. But it didn't use any whitespace, and to get that I had to use the new `#join` helper, which is essentially a `forEach` on the array. That changes the context from the array to the array element, thus the need for `{{this}}`.


If you can't provide a link,
can you please confirm that it was tried with
a) double simplified
b) iso simplified
c) backward simplified
AND
d) psqt adjusted

If not, I will submit again.


Mailing-list link is dead


This is just to prove that our build will stay green, even when the accessibility test fails.


So yes.


確かに毎回作成するのは良くなさそうです！ 直します


Se alguém conseguir ver porque o pedido está bloqueado, eu vou vendo tb :)


One additional thought


Yeah, good point.


As a matter of fact, I did enjoy writing it. Glad to hear I was able to give you some joy.


Looks like the code further down ( https://github.com/chatid/iframe-transport/blob/aa07b3810f2a879d1e34dbdb6172cdb2bbd1fdbd/library/base/child-transport.js#L19 ) might be relying on it?


972154-NewSimpleChilledCeiling (RKStrand) - Win64-Windows-7-VisualStudio-12: OK (1682 of 1682 tests passed)

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


This doesn't make sense


:cry: @orta


There is no need to enable this, pull requests will always be tested by Travis since it is enabled in this repo.

Enabling it on a fork only makes it build branches before a PR is opened.


Those errors are semantically different. One is for the schema fields, the other can be any error thrown into a `self_check` function. Whether that be a DAO error, a schema error (relying on multiple fields being invalid), a UNIQUE or EXISTS error maybe, depends on the actual function and we want to keep `self_check` able to return any error type.

Hence, it is cleaner to separate them in the return value. It is not hard to handle 3 return values instead of 2 tho.


A fork's master branch doesn't matter, changes can very well go there. Doesn't warrant a warning IMHO.


2 teaspoons is not nearly enough


So no options for the XT1058 other than a custom rom? Bummer... Thanks for looking into it!


This is not a module name


This is not the syntax for Python comments


Cool! :)


Please consider adding
```
   - name: etcd.service
     mask: true
```

because if any service requires that unit it will stop etc2


They appear to have updated casey's reference (apparently ESPG::3857), but when I try converting it in qgis I get valid polygons located in france (http://bl.ocks.org/d/08356d6ea8bc158996a4). It's probably just early and I am insufficiently caffinated.


Non-admin users are not permitted to interact with this section of content, so I have included some logic to prevent it from being loaded in those cases


develop (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1169 of 1169 tests passed)

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


Much of this Javascript pertains directly to editing Goals, which non-admin users are not permitted to do.


This executes only the Javascript necessary to populate the Goals table, for non-admin users.


Nice work guys. So this now supports completely inline building via the make command?


@SubJunk All fixed, only affected Java 7/8 users and not the java 6 ones. Next time i'll tripple check wtf i'm doing :)


Fixed


Leftover from previous indention, fixed now


Avoid substr in favor of substring or slice; it is "nonstandard" (although now standardized in Annex B)


How about now?


charAt, not charCodeAt


I see that now, the PDF doesn't display correctly. I changed the format and it looks better now. What do you think?


LOL!

On Thu, Jul 09, 2015 at 04:42:50PM -0700, David E. Wheeler wrote:
> Should be Affero, no?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/davidfetter/pipelinedb/commit/357d8a5b2ae417e808c1580d3ecf2cab4d0298bb#commitcomment-12096128

--
David Fetter <david@fetter.org> http://fetter.org/
Phone: +1 415 235 3778  AIM: dfetter666  Yahoo!: dfetter
Skype: davidfetter      XMPP: david.fetter@gmail.com

Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate


`^` outside the `[]`, and probably add a `$` at the end


`.test` to avoid creating a whole unused result object


No, we want to match any other chars. `^` in character groups inverts choice.


`parts.pop()`


`raises` is already there in `symengine.utilities`


Test2  web


IT shouldn't be.  It was always supposed to be the GPLv3, which is why that's what's on Master branch now in the official repo.


Hmm not so sure about how e.g. parseIPv4Number uses `null` for failure whereas this function uses `throw new TypeError`. I wonder about `/* at the top of the file */ const failure = Symbol("failure"); /* here and elsewhere */ return failure`?


Is this going to be merged into the 1.8.x branch?  The lack of UUID support breaks a few things...


`numbers.pop()`


s/var/let


Actually just do `for (const n of numbers)`


`for (const part of parts)`


TeamCity Gensokyo Lobby :: Lobster Server [Build 0.0.1.4](http://pixelated.design/build/viewLog.html?buildId=19&buildTypeId=GensokyoLobby_LobsterServer&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 40 Build time: 00:00:08


Yeah, that's throwing is a bit of a non-spec shortcut at the moment. Everytime failure implies a failed parse (of the whole url) an exception is thrown. If failure is just something a function may return (which doesn't cause parsing to fail), it's encoded in the return value.


For clarity maybe `output += String(n % 256)`?


Do we not need transform? I thought it helped in cases of scaling.


Please keep your commits specific to the task at hand. If there's ever a reason to revert/rollback this change, we'd inadvertently be undoing the other change.

Just a tip from your Uncle Lar'!


Probably worth a spec bug to include this `floor`.


Dummheit erlauben😜 I didn't find a better term either...


Spec bug to return output.


added some extra information


These are mostly artifacts.


Oh, I see. I would still invert it. Use a regex to test if it *is* a radix-R number, then do `if (!regex.test(input))`. But nbd.


Sounds good, maybe file it as an issue to track?


1. Why not just use the React-Bootstrap Alert component I showed you?
2. Check out this element using Chrome Dev Tools.  You can [see in the DOM](https://www.evernote.com/l/AFVeSQxyIWNO3YC8JB0l5qyDh_ak3ENNeg0B/image.png) that the class name is not what you expected.  It looks like you are confusing the JSX `{}` to insert code with the Ruby's `#{}` string interpolation.


I think I mentioned it before, but you should be able to remove the `if..else` as long as you nest the `Alert` under a `div`.


@jtrmal, @danpovey: I added the KALDI_CYGWIN_COMPAT macro as you proposed. PTAL. Since we started this discussion on GitHub under a commit, w/o creating a PR, I cannot add a commit under this. I opened kkm000#1 with both commits listed. The post-discussion changes only are in kkm000@c026c89.


trabalho realizado por Jesué, Joalisson e Lídio


One new line between entries, please.


Can you order these alphabetically?


So, this broke the bootstrap.. diff of rock's rock_tmp

```
diff --git a/rock_tmp_safe/ooc/rock/rock/frontend/CommandLine.c b/rock_tmp_self/ooc/rock/rock/frontend/CommandLine.c
index 10db9b2..1a55832 100644
--- a/rock_tmp_safe/ooc/rock/rock/frontend/CommandLine.c
+++ b/rock_tmp_self/ooc/rock/rock/frontend/CommandLine.c
@@ -1681,6 +1681,8 @@ void rock_frontend_CommandLine__CommandLine_postParsing_impl(rock_frontend_Comma
                         #line 691 "/Users/amos/Dev/ooc/rock/source/rock/frontend/CommandLine.ooc"
                         _lang_array__Array __rock_frontend_CommandLine_arrLit242 = _lang_array__Array_new(lang_String__String*, 1);
                         #line 691 "/Users/amos/Dev/ooc/rock/source/rock/frontend/CommandLine.ooc"
+                        os_Process__Process_execute(os_Process__Process_new_fromArray(__rock_frontend_CommandLine_arrLit242));
+                        #line 691 "/Users/amos/Dev/ooc/rock/source/rock/frontend/CommandLine.ooc"

                         {
                             #line 691 "/Users/amos/Dev/ooc/rock/source/rock/frontend/CommandLine.ooc"
@@ -1688,14 +1690,14 @@ void rock_frontend_CommandLine__CommandLine_postParsing_impl(rock_frontend_Comma
                             #line 691 "/Users/amos/Dev/ooc/rock/source/rock/frontend/CommandLine.ooc"
                             lang_Memory__memcpy(__rock_frontend_CommandLine_arrLit242.data, __rock_frontend_CommandLine_ptrLit243, 1 * ((lang_types__Class*)lang_String__String_$
                         }
-                        #line 691 "/Users/amos/Dev/ooc/rock/source/rock/frontend/CommandLine.ooc"
-                        os_Process__Process_execute(os_Process__Process_new_fromArray(__rock_frontend_CommandLine_arrLit242));
                     }
```


One new line here as well.


i read some chines site day asking for baidu because google play is not working for they delete it if feel useless


I think I'll leave this as part of the current pull request, since it's not useful on its own. Will add myself to AUTHORS.


It is just a matter of taste, but I think this statement should not inside `with` statement because it no longer depends on contents of `cuda_deps_path` .


Move the assignment to the declaration on line 178.


@anti344 Because I need the index value.


Move this variable towards the top with the other variables.


Indentation is one level too deep.


Should I also add it for `session_slot_deleted`? Currently we have:

    `signals.event.session_slot_deleted.send(self)`


If you put a path into preferences at `Eslint Path` is it working? Might be something wrong with `atom-helpers.exec`


From an acceptance test pov it looks ok. Just a query about the Given I am on the create a new case page. Do we need the calls to the other steps:
+  step "I click on the 'Create a new case' button"
+  step "I am taken to the 'Create a new case' page"
in here as should these be run anyway when cucumber goes through the scenario. Just seems like those steps are being run twice in this scenarios, once in the above step and then again when it comes out of that step and runs the next two steps.


can't hurt to have it there, too


この辺りで、混乱してきました。`threshold[2] や lastpoint[0]` がなにを表しているんだっけ
「0 がx, 1 が y だよな。」と振り返ることが多くなってきました...:)

ですので、より 読みやすくするためには、lastpoint  や minmax, threshold 等も dot のように struct にしたほうが Readability は上がる気がします。


you're right, i was taking only the odd numbers, I got 697 this time . I only changed n to start from 2 and increment only by 1  if the ratio is not under 10%


The ratio is : 9.985632183908047
With maximum lenght: 697


return 書かないのが普通な気がします。


@TomasVotruba That's good for projects but I think it's bad to increase the required version of libraries unless needed or unless some version is really old. In my opinion abandoning 5.3 now is fine, 5.4 not yet unless you need generators.

I understand that you don't want to maintain support for older PHP but I can't use your libraries like this because ina few months you might push it to PHP 7 or something.

You don't need to run the tests for old PHP yourself since travis will do it for you. And if you don't want to bother with builds that work fine with new PHP but fail for older, I can offer you a deal. When that happens just send me an e-mail and I'll fix it for you. In return let me decide the supported PHP version for those Zenify projects that I'm using (for now that means migrations and fixtures). I'll gladly send you PRs to fix the compatibility with 5.4, set up travis and I'll maintain the support for you. What do you think?


Sorry to ask here (and I am probably being very dense) is this just a stylistic change?


Create the link as a click out from this page
i.e. instead of
here
use
here


Yes, just stylistic. It keeps it consistent with what I've done elsewhere.

I don't mind questions on code, or changes.  If you have more feel free to ask :)


See #8 , it should be fixed.


INSI Plugins Update-Checker


`dotholder` という名前よりは、コメントを変数名に反映させて `dots_in_area` などのほうがよい気がします。
コメントを書かなくてすみます。

`holder` からはあんまり情報が読み取れません。


I believe you cannot use the conditional blx in the middle of the conditional block, the ARM Architecture reference manual says:
>Instructions that can complete their normal execution by branching are only permitted in an IT block as its
last instruction, and so always result in ITSTATE advancing to normal execution.

The problem is that after returning from the blx, the ITSTATE would be already cleared.


See #8 , it causes "zend_mm_heap corrupted" error if we try to use smart_str_appendc without prior allocation of the space for the string.


Hmm, I don't think it's a good idea to change a setting's default value in this kind of PR.


какая-то жесть с роутами. зачем тебе какие-то Page? если тебе нужна постраничная разбивка таблицы с деталями, то это не так делается. нужно средствами грида самого пользоваться. посмотри про пейджинг - что это и как


If there are multiple subdomains: a.wikifarm.org, b.wikifarm.org, etc, they are correctly parsed. If they are wikifarm.org/a/, wikifarm.org/b/, it only gets one of them, the root.


When I give the path, it works but can't find `babel-eslint` for some reason. probably ignoring `.eslintrc`.
Currently ESlint is working for me but feels a lot less responsive about that. Can something be done about that ?? After this issue gets fixed.


Remove bracket ')' at the end of this line


I would also like to know why this has been changed. It causes the form to lose these values in case of a validation error.


не надо в 1 файл засовывать несколько public классов. ты к тому же смешал модели и вью-модели


このあたりで、`dotcounter` と `numcounter` の使い分けでちょっと混乱してしまったので、やはり `numcounter` はより適切な名前のほうがよい気が。


yes :)


This `ovi` can be dropped.


Well done! lovely work


i've modified the codes a bit , i found my mistake for the prime spiral and for the sum of Multipliers.
Thank  you very much for the feedback ! :+1:


dotcounter の宣言も、numcounter と同じく、ここのほうがよいです。なるべく使用されている箇所の近くのほうが振り返る距離がすくなくなってよい。


@iam4x How do I install the package from a local copy (I cloned the git repo) ?


Oh dear, thanks, fixed in a08a723.


I would use assert_exists instead of XPATH. It should be easier to comprehend as it would decrease complexity.

However, I see similar uses (of direct XPATH) in other tests as well. So this is just a comment.


Thank you for helping me solve the mistakes I've done ! :) :+1:


merge


I think I got `apm link`


It looks like we can actually use platform independent __debugbreak() since we are compiling with `-fms-extensions` option.


The diff you commented about is from Main-Regular. In @9cb8ac811049df247e5bb579ab7673fef1e7167c I've done the change you describe here, but the result affected some other line from the metrics file, adding skew correction as intended but at the same time setting the italic correction to zero. I must confess that I don't really know (yet) about the difference between these two numbers, and their respective effect. I assume that using the values from cmmi10 should indeed be preferable, but I'd be glad if someone with knowledge could confirm this.


Игра?)


direction を決めているぽいのですが、実際にやっていることは、threshold の更新ですので、ちょっとだけ混乱しました。:)

この場合、 `set` は使わないほうがよい気がします。


BlackList Information


You're welcome


@enumag I work on PHP 5.6+ projects, therefore it's the only place I can provide is 5.6+.
What versions do you work with?

I don't enjoy developing opensource for deprecated versions. Tried, but to much work and no fun => burnout => deleted projects. If you do enjoy this, I recommend forking.

Also there are older tags for PHP 5.3/5.4. I'd try installing package via composer before any further steps.


I would have expected something less tied to a specific language. And I thought you were going to tackle the `preserve comment` feature.

As I see it, there should be three settings:

1. `preserve comments` that make sure to only convert the text after the comment and then prefix every lines of the ascii version with the comment.
2. `generate as comments` that force the resulting ascii to be prefixed with the comments structure of the current language (`#`, `//`, etc.)
3. `use block comments` that, when `generate as comments`is enabled, will use the block comments form instead of the inline one, but only if the current language is actually supporting a block comment form.


请参考任意并发执行的结果和以其他顺序执行的结果相同
 ==》
任意并发执行的结果和以某种顺序执行的结果相同


можно просто селектором table пользоваться в стилях. а вот если потом отдельные таблицы надо будет как-то особенно сверстать, то тогда уже классы вводить. с чётким названием


You can clone the linter into `~/.atom/dev/packages` and start atom with `$ atom . --dev` it will look for packages in dev mode.


First, it dont think this commit broke the test suite, in fact it is probably the json.Marshal/Unmarshal stuff that brakes it.

I think there is a deeper issue here. If the first state update is containing valid values this code shouldn't be a problem. But if it is containing zeros, then I can understand the issue. The problem I had was that not all values are updated when I start up the aquarium node. So for example the pump state. If the pumps was running when the node was started it wont log a true value until the pumps was turned off and then on again. Without this, all states has to toggle once to show the real value in the graphs and that can take hours or days.

I propose that we should add a boolean of some kind telling the system if the state is valid or not. Or maybe change the nodes so they just send an empty state if the state is not valid yet.


Is this test ignored on purpose? Maybe add a TODO?


I think this could be an issue but not certain


This calls the model `.copy()` method, so if we decide to any copying of the nested data we'd have to do it there. Right now we don't modify the fields inside the draft service data and only the new draft is saved to the DB, so it should be ok.


Usar nombre de variables con significado ejemplo: $requirements


It might make sense to test if the response entity contains expected values. Or is this tested somewhere else?


Corregir el nombre de la función: getRequirementsAndDependenciesWitihSequences


全体として、ちょっとプロっぽい感じ (ほめてる）の code で steper にしては珍しいなーと思いました。
完全に独自のアイデア？ 素晴らしいと思います。
 :+1:


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


json.Marshal/Unmarshal is not in master.

This is the problem:
metrics_test.go:177: Expected Log to have ran 3 time got: %!s(int=17)

is expects 3 updates but gets 14 (your initial update) and the 3.


I don't enjoy it but I still need 5.4 for some projects. Forking is what I'm doing now with your projects (https://github.com/enumag/DoctrineMigrations/commits/php-54). But if I'm doing the work regardless there is no reason to not have it in the official repository.


I like that the event is now published _after_ the group has been updated :+1:


инлайн стили? ай-ай. надо всё в css выносить. общие стили для всего сайта - в один файл. особенные по страницами - в отдельные файлы для каждой страницы


this code displays a list of packages from the database and also edits and deletes the packages


We've decided against doing it for now. Depending on how the user research goes the draft copy creation might start with the "enter the service name for the draft copy" form, but for now it's just a "Make a copy" button that creates an identical copy


this basically means that the logic in DeploymentPlan#dependencyOrderedSteps regarding restarting an application is broken, correct?


Why are you assigning the strategy in the environment? We discussed letting the algorithm do this. The only thin you needed to do was work on a single method in the algorithm. Can you address that and then I will look through.


Have you tried installing package? Version 2.0 that supports it might do all you need.


But options still there just moved to battery options


don't keep a reference to the loop in a variable as it is a singleton - the reference is just a waste. call asyncio.get_event_loop wherever necessary


For now probably yes but I'm not looking for short-term solution.


宣言と初期化が同時にできるならできるだけ同時にしたほうがよいです。
```c
double distance = sqrt( pow(current.x - lastpoint[0], 2.0) + pow(current.y - lastpoint[1], 2.0) );
```


Having second thoughts about this... Why do we want to let developers know that they can run controller on PC?  That would leave no use of Ci20 and thus could hamper the sale of Ci20 boards.  I checked with Abhijit, he said that the Linux PC side controller app was developed for debugging.  So, he also agrees that this section should be removed.  In that case, the line in Device Setup "Note that these instructions can also be followed on a Linux PC if you prefer." and the PC side binaries for controller in the release zip will also need to be removed


As far as I can see this V2AppDefinition stuff is rather new and maybe WIP – just funny to see that the body is parsed into a V2AppDefinition, converted to an AppDefinition, and then re-converted into a V2AppDefinition.


好みの問題かもしれませんが、
```c
 total -= heap[route[0]].distance;
```
のほうがよいです。total を2回書かなくてすみます。


что опять за непонятная конструкция. Сейчас у тебя в lot.summary_price записывается ничего, если nil?, или расчет суммы.
Если как я предлагал, то нужен return
```
return lot.summary_price if @selected_lots_initial.nil?
```
метод вернет `lot.summary_price`, если @selected_lots_initial будет nil.
Дальше без `else`
```
lot.summary_price + @selected_lots_initial.
  group(:company_id).
  sum(:summary_price).values.first.to_f
```


But there can be situation where users wants to use a different implementation of the event loop.


вроде получается, а откуда оно берется?


hello
There is only one double quote on this line!
const ESI_INCLUDE_CLOSE = "' />";


из контроллеров удобнее везде ActionResult возвращать, т.к. в качестве него можно всё что угодно возвращать


для такие случаев есть специальное слово `unless`


Don't actually leave empty lines with hyphen like this it will think this a list of nothings, and break.


@mcostalba: If this test passes
http://tests.stockfishchess.org/tests/view/559efea50ebc590abbe1b979
Can you write the patch ? (and submit a pull request)
I don't understand how to make your template stuff work. The only way I can write this code w/o the ugly `reinterpret_cast` is by carving our CMH completely into a separate class.


а что у тебя корзина-то везде с клиента приходит? она должна тупо в сессии висеть, а с клиента только айдишники всякие приходить или типа того


Why use lodash's `keys` here?


оно там чуть выше написано


and here


избавляемся от подчёркиваний


подчёркивание...


я думал ретурн не обязательно если любой метод последний расчет возвращает


Forget it, I've read the PR


Wording can be further changed -
open the terminal and run the following commands.


Paso 3 del proyecto Quiz


![screenshot from 2015-07-10 15 59 02](https://cloud.githubusercontent.com/assets/3112509/8616913/9c266b48-271c-11e5-9681-321dd085635b.png)

npm-which is actually throwing an error.


Using deprecated, unsupported, unsecured PHP and packages versions is not long-term solution either.

If your project matters in long term, [upgrading to PHP 7.0 when out is necessity](https://tideways.io/profiler/blog/essential-macro-optimizations-to-improve-php-performance) (tip 1).


опять `return` забыл? и не надо писать `return nil`, можно просто `return`


пробелы для позиционирования надписей - не гут. надо стилями


не надо var злоупотреблять. он уместен только там, где тип очевиден, либо анонимный.


Yes, I will.

On Fri, Jul 10, 2015 at 12:27 PM, lucasart <notifications@github.com> wrote:

> @mcostalba <https://github.com/mcostalba>: If this test passes
> http://tests.stockfishchess.org/tests/view/559efea50ebc590abbe1b979
> Can you write the patch ? (and submit a pull request)
> I don't understand how to make your template stuff work. The only way I
> can write this code w/o the ugly reinterpret_cast is by carving our CMH
> completely into a separate class.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lucasart/Stockfish/commit/542c28683270f44b0ac81506ca807d7c41b24818#commitcomment-12103013>
> .
>


typo: egc


Thank you @Patrick, will merge and make a new release soon.


Instead of creating a new issue, I put this note here. From the MSVC compilation:
```
print-bgp.c(2214): warning C4457: declaration of 'atype' hides function parameter
print-bgp.c(1322): note: see declaration of 'atype'
```
I think we can just do:
```
@@ -2211,7 +2211,7 @@
                 len -=4;

                 while (len) {
-                    u_int aflags, atype, alenlen, alen;
+                    u_int aflags, alenlen, alen;

                     ND_TCHECK2(tptr[0], 2);
                     if (len < 2)
```


to find the correct paths for the resources, use `rake routes` for example:
```

$ rake routes
   Prefix Verb   URI Pattern               Controller#Action
    todos GET    /todos(.:format)          todos#index
          POST   /todos(.:format)          todos#create
 new_todo GET    /todos/new(.:format)      todos#new
edit_todo GET    /todos/:id/edit(.:format) todos#edit
     todo GET    /todos/:id(.:format)      todos#show
          PATCH  /todos/:id(.:format)      todos#update
          PUT    /todos/:id(.:format)      todos#update
          DELETE /todos/:id(.:format)      todos#destroy
     root GET    /                         welcome#index
```


All methods will need some level of docs and a doctest. Even if the test is functional in nature.


а Title ниже для чего?


When is this going to be released?


@ лишнее


Why are you printing things?


docstring


дважды одно и то же


character-encoding-filter update to 2.1.6.4-SNAPSHOT


Yeah. Currently doing that. This was just to show the current implementation of the algorithm in its working state including the generation of the bipartite graph.


>... my wrappers have 0 negative effects and 1 positive effect, so why not use them regardless.

I just want to help them improve the same way I improve. If you disable their warnings, probably nobody will notice. If you strive for "error free" code, you should care about that. Fortunately there's still people who do care.

>But we still have to wait until 5.5.1 release in SEPTEMBER to actually get the fix. ...

Yeah, we're all people making mistakes. And we don't even pay money for it. So what's the deal? I went through much heavier rain than waiting 2 months for a fix ... :smile:


jrflwrlglgjl


docstring.


Hi,

I just created a SYNCRONIZERS file.
Can you update it reflecting which syncronizers you updated
so that they can upload new sport types ?

/Jonas


use jadira usertype


$(document).ready( можно писать короче как $(


docstring - all methods also.


character-encoding-filter update to 2.1.6.4-SNAPSHOT, use jadira usertype


Well.. the problem with `isResolved` is that some nodes think they are resolved, then suddenly they aren't anymore. The AST mutates around way too much unfortunately. One of the goals of oc was to separate ooc AST & C ast. There could be ooc resolution, then generation of a C ast (with some additions, like being able to generate variables anywhere), then that could be simplified to valid C, then written.


Also, there are some debugging stuff which I need to remove as well.


Forget about the "fixme", I do not know what came over me that moment


What I believe are some errors in the code in the book:
QuickCheck: in check1, numberOfIterations is undeclared
 for func area(size: CGSize) -> CGFloat, CGSize is not Arbitrary


https://github.com/voat/voat/commit/2320c3848c4c3137bc547750a4b5697a12977ead#diff-9b58bead434d983ca03bf9f39671caafR91

I know this is making the endpoint do a it was originally meant to do (only return 200) but could there be some way of getting a list of all subverses? I used this endpoint to get the info I need for my stats app and now I'm restricted to only show stats from the top200.


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


a bit o' segment mishap)


typo & cosmetics


можно Session в переменную венести, чтобы каждый раз тут длинные строки не писать


//test


typo


@vramana Okey, eslint does exists into `/home/vramana/Documents/work/prizetracker/node_modules/.bin/eslint`


Does @choice need to be an instance variable? Can it be a local variable instead? (i.e. is it only ever used in this method?)


What's this method?


I would actually make this as a new Screen, like GenerateRandomContactsScreen, that the user can choose at any time.


"is this Java-idiomatic"? Unlike case classes, it is supported in Java at all.


Oh dear... this will only ever work on your machine


Oh I stuipid. I translate but I don't make PR)


Yes, I agree I just didn't know how to do that.
This problem arose because I added Matt's suggestions for better error messages


Thanks,  I have been using Perlcritic recently for another project and this form of unpacking is picked upon.

eg."Always unpack @_ first ....  See page 178 of PBP.  (Severity: 4)"

I not suggesting to change it or to change any coding styles, constancy is usually a lot better than doggedly adhering to PBP .


Pardon? I didn't get your question/statement.


Thx! will edit


@iam4x Hey I am getting warning for the files in `.eslintignore` that `File ignored because because of your .eslintignore file. Use --no-ignore to override`. What sorcery is this?


Is the `eslint` binary present into `/home/vramana/Documents/work/prizetracker/node_modules/.bin/` ?


/meaningless/..


Yes.


@vramana It's the normal behaviour when a file is ignored from `.eslintignore` eslint will print a warning.


Use `/` for Linux compatibility


Is there any downloads of the code in the book?


Okey so `npm-which` isn't finding it correctly.


But you seem to never pattern match on `Map`, so why do we need a case class?

The optimization done in this commit should also work as:

```scala
def map(p, f) = new Parser {
    // default implementation of map
    override def map(g) = p map (f andThen g)
}
```


For tests remember you should use that TempFile object rather than specific file paths. Tests should never use 'actual' operating system resources, because it means test runs are not independent. Using a TempFile object means you always get a random file that no other process will have access to. It gets given a random name and will be deleted once your process ends.


@iam4x But but I am never had warnings before. When I am using previous version. Can you give an option to supress these? Slightly annoying but will live with them.


@Kirow Is your app even launched on the AppStore?


May be you mean `AND default_setting_value is not null` ?


almost there


Missing double quote?


Thanks, I manually included it in the latest release


hm, even with it, I still get a deep exception when using `smoutput` - but it still prints the result. Weird. Will post the stack trace when you change path separators so we are on equal ground.


would a few `if emptysrc() <: ` statements work to differentiate the cases where it's `BoundsError` vs `ArgumentError` ?


I know, and my boot.img has this command line option already set to permissive. I wanted to make "double" sure that selinux is running permissively so that others forking my project for custom ROMs don't have to take care of thinking about boot params.


Ok. Thank you for your answer.


![screen shot 2015-07-10 at 12 52 27](https://cloud.githubusercontent.com/assets/893837/8617282/8ad30f3c-2702-11e5-8932-f5b5687bef4b.png)

I'm trying to match the same behaviour of the CLI, so they won't be an option to remove this. If an option comes on `eslint` to ignores this with params I will.


You are correct thanks for bringing this to my attention.


ここでItemStack、NBTTagCompoundの作成、ワールドにドロップする


This patch is broke call recording for us.
On call event <<"RECORD_START">> and <<"RECORD_STOP">> JObj dont have <<"Application-Response">> key. As result it cast messages {'record_start', 'undefined'} and {'record_stop', 'undefined'}.

wh_media_recording:363 (<0.27532.3>) unhandled cast: {record_start,undefined}
wh_media_recording:363 (<0.27532.3>) unhandled cast: {store_recording,undefined}

And finaly It dont set is_recording='true' in record State, and dont send command to store recording.


This is not just about feeling. Codata simply doesn't support pattern matching.


这里的话， 使用 ansible_hostname  很可能会有问题的。
很多主机的hostname 都是 localhost /   ubuntu  这样的， 不是一个完整的域名。


I don't know how, but it would be cool to add `°` as equivalent to `Degree` but I couldn't find out how to do it.


@vramana I you replace npm-which with this: https://github.com/isaacs/node-which

 Does it finds the correct binary?


rp_IsValidDoor ***


@yfcai: can you drop the redirect?


どういう分けか拾ったアイテムのItemStackのstackTagCompoundはnullになっている
ただし、描画するにはこのNBTのテクスチャの情報がいるのでnullだったら描画されないはず


Jason, could you do me a favor and see if that fix also needs to be applied to the event code in cb-event-forwarder?  Thanks.


putSubDirectory ?


@mikekirchem
I've tried to reproduce your problem, but could not:

> Before this commit, I could go down a list of files, pressing D or ds against any files I like, and I always keep 3 windows, the Gstatus one and the two diffs. Now it seems to get confused with the windows...

I've also tried it without any other plugin.

Check it yourself using `vim -u NONE -N --cmd 'source ~/.vim/neobundles/fugitive/plugin/fugitive.vim'` (adjusting the path to fugitive of course).

I'd say that another setting / plugin is interacting here, and the former behavior of fugitive was hiding it.  Maybe you have even some old workaround for the old behavior in place?


Adam advised that the reason that all features pass but multiple_validations run by itself  fails is that some feature is implicitly loading the ValidationController requirements that are not loaded by multiple_validations by itself


DRY


My issue really is these warnings add to the issues of the total project but they are not really issues. Also we are getting warning because we trying to run eslint on ignored file. So, that is kind of a mistake on our side. We should avoid that in the first place right? not try to lint the ignore files at all.


Somewhat by pure fluke when we integrated this with our codebase an error was thrown.

After a whole lot of digging it appears Nokigiri calls `Document#characters`  either when it hits an element or after a certain amount of characters/whitespace (it didn't call repeatedly when I had `<COLUMNS>\t#{"A"*1_000_000 }\tB\t</COLUMNS>` but does with this example).


The patch originated in http://trac.sagemath.org/ticket/852 which is quite ancient. It is surprising that at least this branch of singular didn't adapt to it. Testing needed.


Cool!


\n


finals


Perfectly fine, explanation in the forum thread.


这里两行重复了。
而且直接用管道不行吗？
```
shell:  " echo '{{item.password}}' | ocpasswd -c /usr/local/etc/ocserv/passwd {{item.username}} "
```


是啊，所以我在这个PR里顺带把 `ansible_hostname` 改成  `inventory_hostname`了。


das ist ein sehr interessantes Feature!


yay!


![screenshot from 2015-07-10 16 34 03](https://cloud.githubusercontent.com/assets/3112509/8617439/8146b792-2721-11e5-81c5-fc47a74a99dd.png)

This time `eslint not found`.


好吧。。diff 左右看错了。


what does this message mean?


Spaces are used in PPULLVMRecompiler instead of tabs, for some reason. Probably would be nice to tabify all PPU LLVM related files to make them be in-line with all the other files.


必须要两行啊，ocserv里面需要输入password，然后repeat password。


Player input name need to be extracted to separated method


why this check again


Remove this unnecessary change.


>commind

typo


Is this change required?


should there be 'else' or ternary?


@rfabes21 Hi bro,

Can you please tell me how to create a html5 video slider like on www.blitzagency.com.

Thank you,

Regards


remove space


Can we remove all the findbugs-excludes.xml


handlingmethod要从参数传进来，不能gethandlingmethod


Shouldn't this be a proper java doc.


flamegraph


extract var


nice work, if you rename the file to .abap instead of .txt, then github will do syntax highlighting and recognize the repository to contain ABAP code


@beledouxdenis I do not think that this really solves the problem in an ideal way:

Granted - One problem is that the e-mail is only send before this patch after the user comes back from the payment provider which is solved by this BUT the other problem is that the sales order is still in the draft state before the user returns from the payment provider - because of this the user could still change his shopping cart if he did not return correctly from the payment provider - even worse the sales order could never be finished because it may be that the number is already known to the payment provider which is the bigger problem in my opinion!

To solve all of this problems at once i simply change the sale order to the state "Send" after the user presses on the Pay Now button and clear the context so that the shopping cart is empty again - with this both problems are solved:

- user can not change the shopping cart (if no or no correct or a late answer comes from the pp or he pressed the back button)
- Sales order will be send by mail
- user can start a new shopping cart an he will get a new sales order number

I already made all this changes as an addon:
https://github.com/OpenAT/odoo_v8.0/tree/master/addons-own/website_sale_payment_fix


We share the same idea, at least on templating grammar


no EOF line


Hey @locks, this looks great, we may have to tidy some things up with the markdown format, but it's worth getting something out there and refining rather than holding out.

In terms of linking recipes, Jekyll should expose a `url` property on the collection item:

```
{% for recipe in site.recipes %}
  <li>
    <a href="{{recipe.url}}">{{ recipe.title }}</a>
  </li>
{% endfor %}
```
See the [Jekyll Collection docs](http://jekyllrb.com/docs/collections/) for the lowdown.


Usually such resource freeing is done using labels:

                if (foo) {
                        .......
                        goto droplink; // replace 'goto out' by 'goto droplink'
                }
                ......
        out:
                // curent logic
                return;
        droplink:
                ss_droplink();
                goto out;


这个控件在layout中无法看到视图，发现有一下BUG:
Exception raised during rendering: Index: 0, Size: 0 java.lang.IndexOutOfBoundsException: Index: 0, Size: 0   at java.util.ArrayList.rangeCheck(ArrayList.java:653)   at java.util.ArrayList.get(ArrayList.java:429)   at com.exutech.lanjing.ExWidget.PickerView.drawData(PickerView.java:214)   at com.exutech.lanjing.ExWidget.PickerView.onDraw(PickerView.java:198)   at android.view.View.draw(View.java:15114)   at android.view.View.draw(View.java:15022)   at android.view.ViewGroup.drawChild(ViewGroup.java:3404)   at android.view.ViewGroup.dispatchDraw(ViewGroup.java:3198)   at android.view.View.draw(View.java:15017)   at android.view.ViewGroup.drawChild(ViewGroup.java:3404)   at android.view.ViewGroup.dispatchDraw(ViewGroup.java:3198)   at android.view.View.draw(View.java:15017)   at android.view.ViewGroup.drawChild(ViewGroup.java:3404)   at android.view.ViewGroup.dispatchDraw(ViewGroup.java:3198)   at android.view.View.draw(View.java:15117)   at android.view.View.draw(View.java:15022)   at android.view.ViewGroup.drawChild(ViewGroup.java:3404)   at android.view.ViewGroup.dispatchDraw(ViewGroup.java:3198)   at android.view.View.draw(View.java:15017)   at android.view.ViewGroup.drawChild(ViewGroup.java:3404)   at android.view.ViewGroup.dispatchDraw(ViewGroup.java:3198)   at android.view.View.draw(View.java:15017)   at android.view.ViewGroup.drawChild(ViewGroup.java:3404)   at android.view.ViewGroup.dispatchDraw(ViewGroup.java:3198)   at android.view.View.draw(View.java:15017)   at android.view.ViewGroup.drawChild(ViewGroup.java:3404)   at android.view.ViewGroup.dispatchDraw(ViewGroup.java:3198)   at android.view.View.draw(View.java:15117)   at android.view.View.draw(View.java:15022)   at android.view.ViewGroup.drawChild(ViewGroup.java:3404)   at android.view.ViewGroup.dispatchDraw(ViewGroup.java:3198)   at android.view.View.draw(View.java:15117) Copy stack to clipboard


Is this ready to merge?


ты же пропертю делал для такой конкатенации


и тут


Maybe its better to remove this


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


@blueyed :+1: You're right, it does work so another plugin must be causing the problem. I'll investigate.

Thanks very much for the tip on how to run with a single plugin, that'll be really helpful.


"the other problem is that the sales order is still in the draft state before the user returns from the payment provider - because of this the user could still change his shopping cart if he did not return correctly from the payment provider"

This would be another issue. This fix concerns only the fact the email wasn't set, not the fact you can edit your cart until the SO is confirmed.

Besides, if you configure your payment provider to activate the server to server feedback, your SO will be confirmed without needing the user to come back from the payment provider. This way, the SO is confirmed immediately, the email is sent and the cart can no longer be changed.

"even worse the sales order could never be finished because it may be that the number is already known to the payment provider which is the bigger problem in my opinion!"
Again, this would be another issue, not the fact the email wasn't sent.


Debugging


Oops :frowning: I committed as root.


@danwall please, upgrade to https://github.com/dericofilho/sublime-phpfmt/releases/tag/4.21.0


wrong rule patterns, it is `/sms/+33-6-*` and `/sms/+33-6-*/send$`


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


LGTM


I know that:
if (mDataList.size()>0)
	canvas.drawText(mDataList.get(mCurrentSelected), x, baseline, mPaint);
So you should add that:   if (mDataList.size()>0)


используй вычисляемое свойство


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


We should split result with symbol that can't be contained in the title. So, we use '\n'


random returns a long


![blog-teamcheesy](https://cloud.githubusercontent.com/assets/6314687/8617656/32174166-2706-11e5-90ed-5ce52deb1b03.jpg)

@alicetragedy Here is the second photo! All ready to go :-)
Could you please doublecheck if the Fun Facts formatting (blockquote + italics) works?


not correct ParamName in ArgumentException


тут по идее @ViewBag.Title должно быть вместо Index. чтобы заголовки окон менялись между страницами


Forgot to edit some. Done now!


это очень странно, что ссылка на админку как отдельный cshtml сделана. да и вообще - зачем ссылка? сделать админку по роуту /admin и ограничить доступ


assertIsInstance won't work before 2.7

https://docs.python.org/2/library/unittest.html#unittest.TestCase.assertIsInstance
https://travis-ci.org/Scondo/purepng/jobs/70363276


In addition, I am not even sure the fact you can still edit your cart is an issue.

In most ecommerce, like Amazon, you can change your cart as you want until you actually paid. It's the payment that confirms the order and empty the cart so you cannot change it.

I believe you should simply activate the server to server feedback, so your sale orders are confirmed immediately once the order paid, if you do not want to rely on the user coming back from the payment provider.

See:
- Paypal IPN: https://developer.paypal.com/docs/classic/products/instant-payment-notification/
- Ogone Feedback: http://payment-services.ingenico.com/int/en/ogone/support/guides/integration%20guides/e-commerce/transaction-feedback


Hi, yesI thought it would be easier to assign the strategy when creating the agents. But I also wanted your opinion, because I even though the  original test passes the assigning is not tested.


по умолчанию итак get вроде


роль 0?) ды ну нафиг)


for me it seems that the fixes for this are interconnected and can not simply be seen as as separate things but ... you are the professional.

I do strongly disagree with your comment that someone should simply enable server to server communication - if it is flawed without S2S than server to server should be the only option or all of the options should work correctly.


зачем тут true/false? итак булево значение имеем в условии. надо это с выражением выше слить


All fine now. :)


что это строка даёт?


Since POST is usually used for creating a resource and PUT for updating, in the context of this paragraph they should be stated the other way around.


скобки конструктора лишние, когда инициализатор пишем


Mistakenly committed code in master but undo last commit.


@yanns here we go https://github.com/sphereio/sphere-node-sdk/commit/a4f2addc00f75af38ed26c3c58c7572068f8713a


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


Now we do not free the skb. It's removed from the list and passed to higher layer, so the higher layer is responsible for it's freeing. That's good.

The higher layer should is hooked by ```connection_drop()``` on the socket closing, so the callback must free the skb. Server socket handling is fine: it calls ```tfw_sock_srv_connect_failover()``` which in turn calls ```tfw_connection_destruct()``` which deletes the skb via ```conn_destruct``` callback. However, client code does not delete the skb at all.


When it doesn't find the binary it's when you start Atom from launcher not CLI?


@NobodysNightmare unfortunately it's not possible to move `includes(:project)` into `Project.allowed_to`, as we use `Project.allowed_to` in places (e.g. `Message`) where there is direct `belongs_to :project` association defined.


Yeah so make the algorithm assign the strategy.


Yeah, but “begin” is a verb, `beginningPos` sounds odd and `startPos` is already used in the code with a different meaning. So I’ll stick with `savedPos`.


Thank You, ilius!


For a company having thousands of orders everyday, not relying on the fact the payments have actually been received or not, through the server to server feedback, is just unbearable.

In your case, this would mean that every sale order within the "Send" state should be checked one by one, to see if we actually received the payment or not. Besides, the customer would receive the confirmation of the order, while he may not have paid for it. I strongly believe that, if this one by one sale order confirmation is done in your company, the confirmation email should be sent at the moment your employee confirms the "Send" SO to "Confirmed" SO, and not at the moment the SO comes from 'Draft' to 'Send', as, at this moment, you are not yet sure the payment has been actually received.


Chilean university


"In most ecommerce, like Amazon, you can change your cart as you want until you actually paid. It's the payment that confirms the order and empty the cart so you cannot change it."

Sorry but this is comparing apples with pears.

Let me show you an example why i think it is flawed:
- User presses PayNow
- Redirected to the PaymentProvider he pays
- The odoo controller gets the answer from the payment provider BUT has any small problem with it (already happend with ogone because of wrong date format)
- The user gets no answer on the odoo landing page or no ok - the odoo cart is still full
- The user tries to finish the process again > FAILS because ogone tells him that there is already a payment
- The user clears the shopping cart and adds new products > Payment STILL fails because the sales order Number and so the ogone id is already taken


When this plugin will be ready ? I can't wait :O


just for fun!


@openframeworks/macos @danoli3 can we silence `brew update` somehow? I think it's this command which spews some 2k useless lines of new/updated formulae names in the [build log](https://travis-ci.org/openframeworks/openFrameworks/jobs/70361260). I researched this myself, but it seems there's no manpage of brew online, where I could see any CLI args.


I don't think this is needed.


right. Thx


А ты знаешь, что скорее всего, это условие у тебя никогда не выполнится, т.к. если ничего не найдено, то в `@selected_lots_initial` будет типа пустого массива?


ok, but never commit to develop, I know is a simple change, but even a simple change could break javascript syntax.

We use develop as our source of truth when we deploy cartodb


Sweeet! I was getting these errors


![you are awesome!](https://s-media-cache-ak0.pinimg.com/236x/9d/11/f2/9d11f26dba54f32873befcc4a0c12fc7.jpg)


зачем в массив переводить, если first и так тебе первый элемент вернет?


Curriculum Vitae


abı niye sildin?=


you are right, but I am not @ home so cannot fix it atm


looks good


Game.createItem(460, 1, toPosition) ?


You could add pipelining here https://github.com/spray/spray/blob/master/spray-can/src/main/resources/reference.conf#L33, for batching requests (a value of 10 is a good start before investigating further)


That'd be a great exercise if you want to learn more about the Mathics internals. Are you familiar with parsers? If you feel like a challenge take a look in `mathics/core/parser.py`.


If your connection to 'weather' is persistent (keep-alive) you should be fine, otherwise this might be a bottle neck


I'd have thought `zope.verify` would have caught this, but the signature in the interface should match the signature in the implementation, so the interface ought to have `force=False`.


And maybe add a note about the default behaviour to the docstring.


Beware this does not apply to the client (10s default)


/me opens email
Ah! finaly another one that can into bash[isms]!

/me opens github
*cringe*

What I do approve of:

- Changing checking of exit status, though I think there was some case where it did not work but I seem to have forgotten.

What I do not approve of:
- merging 4 seperate commits into one
- changing coding style for the heck of it
  - at mknod, you *could* have replaced tabs with spaces (so everything is aligned regardless of tab/space setting of ones editor) which I would approve of, but it seems you did not replace everything consistently with one space? e.g. line 143
  - since you went for style changes, you could have also replaced ${var} with $var where it is not needed and harms readability (e.g. line 554)
  - replacing [[ with [. Even if [[ is not strictly needed, it is much better as it is guaranteed to be a shell builtin (I think bash can be made to call /usr/bin/[ under certain circumstances), plus [ is *deprecated* as far as Bash is concerned.
  - removing ;; from 'exit 1;;'. Every case statement must end with a break (;;) even if that case statement is the default statement. While bash's syntax checker will not complain about this, it is the right thing to do (and consistent!).
- changing which with hash. using hash is faster, but using command -v would be both faster and more correct.
- above all, it seems you did not even bother checking if your changes actually work (see e.g. line 200)


или еще вариант
```
def трулляя
  next_lot =  @next_lot.first

  aaa = next_lot.store_type.try(:title) || ''
end
```


или еще
```
def трулляя
  next_lot =  @next_lot.first

  aaa = online_store_type_next(next_lot)
end

def online_store_type_next(lot)
 lot.store_type.try(:title) || ''
end
```


Yes


Should removed this yesterday !


oh, yes. You are right!


I need to say that this change makes it a bit difficult to write a software that needs to work with both GitPython < 0.3.5 and GitPython >= 0.3.5.
The common practice is to set the defaults in such way that the original behaviour is preserved because of compatibility reasons.


Should this be `:tracking_user_id => @display_user.id`?


Sorry, the problem actually appears after this commit: 9fa4dbb699b3a71115a5d2e498f59712886a23ff.


Spaces may be included in window name(title) and process name, so we're using newline as separator between window name and process name when returning them via stdout from our window finding module.


E ok rezolvarea, dar incearca sa implementezi o sortare manual. Gasesti de exemplu, explicat in linkul cu Selection sort din follow-up-ul de la curs.

Daca nu reusesti, e ok si daca gasesti un exemplu in PHP pe Google si-l adaptezi pe array-ul tau. Dar incearca sa intelegi exact ce se intampla. Ia pas cu pas, poti incerca chiar ca in filmuletul ala de pe youtube, cu niste bucati de hartie pe care iti scrii numerele.


This seems wrong.
We should add a User to the Upload/Download actions and pass it through.

VCAP isn't a thing for Docker.


Ditto.


This is the readme.


It would be nice to have a constant array with list of supported flags. Then you can use the array and check if the 2 characters match any of the array's elements.


Codul functioneaza.

Incearca o solutie si cu FOR.

Hint: Sirurile sunt egale, deci elementul de pe pozitia `$i` din primul array are un echivalent in celalalt array, dar si in array-ul final. Poti parcurge primul array si sa folosesti `$i` pentru a scoate elementul echivalent din al doilea array si pentru a pune suma lor in array-ul suma.


Tja, zo heet ie nou eenmaal. Ik heb het aangepast naar suggestie


this constructor should remain protected, public one should look like this:
```java
public UserPage(WebDriver wd, String login) {
    this(wd, false, login);
}
```


тут форма не нужна по идее - по аналогии с Edit должно быть


`Given` steps are usually written in past tense - `Given I am signed in as a user`.


а зачем values.first.to_f ?
если у тебя будет пусто, тебе sum должен вернуть ноль и так


Não entendi a ligação. Olhei muito rapidamente e não vi "fatos e imagens".


опять игра) откуда у тебя игры, когда тут детали должны быть?


Nice, this makes the code a little easier to read.


Again, use generated user, not hard coded one, create it before test, and delete after


As @darkofabian said, it makes more sense to hardcode this in scenarios.


Try-catch block are the worst thing in Minecraft modding, never ever put a general try-catch block, if you expect a certain exception to be thrown handle it separately, and log any other exception.


Is this used anywhere in the code?


.searchUser(**"ia"**, userName );	- **bold text** is hard code


Nice! :+1:


My fault :(
Sorry. What should I do?


>If you strive for "error free" code, you should care about that.

I'm sure any warnings in the Qt headers that made their way into release are harmless. They're not error messages, they're just noise. And since it's not up to me to fix the warnings, I'm removing the noise the only way I can.

> And we don't even pay money for it. So what's the deal?

So it's OK for free software to not work properly because it's free? I don't think so.

>Do you have a link to the issue?

https://bugreports.qt.io/browse/QTBUG-46833


Fixed #2


Can this feature be enabled for Graph2d Groups as well? Meaning add the `groupTemplate` property to optionsGraph2d.js as well. That should be it?


Alguém corrija o erro ortográfico ~~images~~ **imagens**, por favor. Não quero fazer novo PR pra isso.


Well, I've created a PR to address this: https://github.com/wvteijlingen/Spine/pull/15
And you silently rejected it without giving me the chance to contribute properly. Since that's the only reward I can get in OSS, I'm disappointed. Just to let you know.


sdfsfdsfds


имхо естественнее смотрится, когда разметка формы в явном виде описана. без всяких фокусов с метаданными


Remove sleep - use explicit wait mechanism


Jk


Output might be a bit nicer if you use contexts here. Although you will have few lines more...


What about negative tests?