I don't like this. It's fine for tooltips since tooltips only have body content, but in this case I'm specifying the popover body via the `popover` prop, while the popover title comes in via a `title` prop on the `PopoverTrigger`, which seems worse than the existing API where I just pass in a `Popover` element that has both.


destructure
```javascript
var {
  firstName,
  lastName,
  dobYear,
  ...
} = this.props.organization.bankInfo
```


Mmmm weren't you going to do a commit to put the derefence inside the g_agressive check? I can do it if you're too busy @gregory38 ... not likely to mess that up.


`array.push(thing)` cleaner than `array = array.concat(thing)` perhaps?


ㅇㅇ Subset으로 간주하고 이 테스트는 빼야겟다


No, there is any special reason.


Blank migration that isn't doing anything?  Why?


same as above


What is this `#if` business for? I could earlier run `liquid --cabaldir Liquid.hs` but now I get some weird error about

    liquid: liquid: phase `C pre-processor' failed (exitcode = 1)


This problem can be solved using SharedPreferences.


联络人：查新彧    南京大学
视频制作及上传：查新彧
参与人：
杨丽婧  重庆大学
李婧     西藏大学
查新彧 南京大学
吴书其  南京大学
王政     南京大学


Why pass whole request object as a param when you can just pass the appropriate IP address?


ㅇㅇㅇㅇㅇ


ffff


ㅇㄹㅇㅁㄻㅇㄴㄹ


the `try...catch` is unnecessary because you aren't calling `await` on any promises.  It won't actually ever trigger the catch AFAIK.


System.IO : LGTM :+1:


comentttt


تجربه


اللغة العربيه


Can you name this `createRemove` for consistency with the surrounding code (which I acknowledge is not very consistent)?

Thanks for the bug report and patch!


[These lines](https://github.com/Homebrew/homebrew/blob/30d953db45b70451d5aed0e2fe7fb2686b3590a3/Library/Homebrew/tap.rb#L18-L22) should be changed accordingly. How do get remote for boxen tap? Just use directory path?


It works... A bit hard to figure that you have to run compiler than websocketd java Counter .... But I guess people from Java world all know that?... I'd merge it unless someone disagrees in next couple days.


와우 굿럭! ㅋㅋ


Whoo, looks good.


"Word of caution" instead?


"create" is misspelled


Because it's part of the object we pass to the JSON formatter.  After this commit, it's set [here][1].  The non-UnixFS `ipfs ls …` sets a similar field [here][2], and you see that in its JSON output (they call it a Hash, but it's just the bare argument, hence my rename):

    $ ipfs --encoding=json ls /ipfs/QmXvEg83dD5PvaDUqHV17iXXtTpZMnPrpKR2cGyU3oCK5
    {
      "Objects": [
        {
          "Hash": "/ipfs/QmXvEg83dD5PvaDUqHV17iXXtTpZMnPrpKmR2cGyU3oCK5",
          "Links": [
            {
              "Name": "gentoo",
              "Hash": "QmXAuRiZKYpCvUmFFBBEzAMvvAgw9Aoxcf24DYaWnG43Er",
              "Size": 38775183,
              "Type": 1
            },
            {
              "Name": "protofs.md",
              "Hash": "QmWpn9xUDNNj3XvY4ixMgb8imiDZmGDr1qQLr1DBv8WAnG",
              "Size": 4953,
              "Type": 2
            }
          ]
        }
      ]
    }

[1]: https://github.com/ipfs/go-ipfs/commit/30f781984c387065a9c344ca8a86b6de0cd06451#diff-e292363c7bc442212de73a448093fad7R81
[2]: https://github.com/ipfs/go-ipfs/blob/v0.3.5/core/commands/ls.go#L80


and "system that authenticates via the AD" should be "systems that authenticate with AD"


I personally prefer

     _purgeTimerCount--;

but the original code put the `--` at the beginning.


@abhayachauhan Good pickup. I think the idea was to have a match element to tell what type of matcher to use, so the correct example should be: `{"match": "regex", "regex": "\\d+"}`

I'll wait for @bethesque to comment before making the change


huh, it just occurred to me that since the pop-up window is modal, the user shouldn't be able to click the button again while the timer is active.  I should probably remove this, but I might as well wait for other comments right now.


@MrApocalypse , when you added and built, did you do a dirty flash. I did and had to toggle both options a few times and started working normally. I have not tested a clean install.


"permission" mispelled. "User" should be lowercase, "minimal required permissions" instead of "minimal required"


probably mention "if you have `LDAPAuthenticator` enabled", since it makes more sense referred to as a class


On Thu, Jun 11, 2015 at 08:10:15PM -0700, Juan Batiz-Benet wrote:
> this commit means we _do_ keep the newline, correct?

No, the commit message just gripes about me having to remove from the
test's expected output, since I can't find an easy way to *add* a
trailing newline to the implementation.

This commit just makes sure we set the Argument attribute for both
files and directories, to avoid:

  $ ipfs --encoding=json file ls /ipfs/QmXvEg83dD5PvaDUqHV17iXXtTpZMPrpKmR2cGyU3oCK5/protofs.md
  {
    "Objects": [
      {
        "Argument": "",
        "Links": [
          {
            "Name": "/ipfs/QmXvEg83dD5PvaDUqHV17iXXtTpZMnPrpKmR2cGyU3oCK5/protofs.md",
            "Hash": "QmWpn9xUDNNj3XvY4ixMgb8imiDZmGDr1qQLr1DBv8WAnG",
            "Size": 4942,
            "Type": 2
          }
        ]
      }
    ]
  }


To whom?


I borrowed this patch from kerneldedup.org


"write attributes" instead of properties. Better to be consistent with terminology instead of use them interchangeably.


On Thu, Jun 11, 2015 at 08:12:28PM -0700, Juan Batiz-Benet wrote:
> the more we go this route, the more i think we need something that's
> universal + self-describing asap. let me give this some thought
> before we merge this PR.

I would love to have the option to drop a hash here with a link to a
spec describing our UnixFS directory / file / … type.  But I think
we'll still want the default to be a (lossy, collision-prone) common
name, becase a lot more folks will recognize “Directory” (even if
there are five possible implementations) then will recognize
QmSomeSpecDescribingMyDirectoryType.


it is possible that more work each of the jquery is not as performative as for native javascript. As he put it in an array of the DOM elements, we can simply scroll through these elements and disassociate the events to him and then yes remove not causing an obstruction and not overloading the memory


Fix code formatting.


Class level comments.


add test


@pimpmaneaton Yes, I did a dirty flash. I toggled both options multiple times too, I know the navbar after the customization commit needs a little nudge to get going, but no dice. I'll try it again tomorrow on a clean flash, and work through it slow.

Also, if your custom actions and chamber of secrets were broken like mine, I have fixes for them, if you needed them. Just FYI. =)


This should probably really be PROGMEM (best practice).


This is not correct (or very confusing/ambiguous).  It runs a zillion times.  What limits it to the frame rate is the guard clause below.  But the `loop` itself runs as often and as fast as it can.


Since we only sleep 1ms `loop` runs a little less than 1000 times a second.  The guard clause will return 65 out of 66 times or so... and on that 66th time (15fps) the entire `loop` code will run.


The refactoring looks good, thanks for fixing the bug in notify as well. The only change I'm interesting to know your reasoning for is moving the headers from notify into it's own method. It is specific to the notify request/method so at the very least I'd call it `notify_headers` to make that clearer.


@wking yeah, that's how JSON-LD solves the problem (it has a `@context` that maps "local" names to global names. we could do something similar. not sure how that would work yet. one thought is maybe we include the `@context` (ipfs) link in the root JSON document.


oo good catch. thanks


ok, thank you.


ok


Confirmed this is OK on Debian Jessie, Ubuntu 12.04, 14.04 but unfortunately **not on OSX** where python2 doesn't exist.  Any ideas? I suppose we could remove the shebangs from all of them and users could start them via "python -m piksi_tools.console.console" etc.


yes, there is a version for Windows in "base/files/file_util_win.cc".


원래 코드에서 새 코드로 변경한 이유가 뭔가요?


Oh, i understand now. in the raw `ls` case, it's not about being argument (was introduced here: https://github.com/ipfs/go-ipfs/commit/17c6907e1f6adca30ba482c3beecbba84f750c5a ). `Hash` is the hash of the ipfs object that's being returned. I believe it was needed somewhere... It was deemed ok to include the hash of the object itself... in the object ... since the hash describes the protobuf implementation and parsing -> protobuf would drop the hash. It's sort of self-describing in the same way TCP checksums are.

For this case, we should either drop it, or keep using `Hash` pointing just to the hash.

> (they call it a Hash, but it's just the bare argument, hence my rename)

maybe we should make it just the hash. otherwise, just remove it. an _argument_ to the call doesn't belong inside the the _first_ of possibly many objects in an array. if anything, it would belong in the root returned object.


done


yeah I tend to agree, it feels nicer for tool tip. I was originally worried that you wouldn't be able to just use the Popover components because they would need to be rendered somewhere in the trigger since they are the component responsible for rendering the overlay. Fortunately though I figure out a mildly hacky way of avoiding that in OverlayTrigger by rendering them into a disconnected DIV. so we can actually keep the old OverlayTrigger API if this doesn't work


Done


init


PrepTransitionFor84 (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1670 of 1670 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)


> 分分种注入攻击
主要是想了想本来也好不到哪里去……


Please bring thie feature back ---- pretty soon most content will be HTTPS, and then it'll work with HTTPS reddit again.    Until then, I'd rather keep using Reddit with HTTP with the toolbar than HTTPS without it.

It's not like reddit is some sort of banking site.

Recreational sites like this are perfectly OK for http.


Bug: Version should be 0093, when starting up it shows 0087


Shouldn't the above text come from a message properties file?


テキストファイル文字化けしてる・・・。
右側にあるViewボタンを押せば、本来の表示が見られます。


Yes! This is exactly what I wanted you to find.


Feature request: When changing values, make "long press" increase/decrease values rapidly after 500ms (like when changing altitute)


Just to make sure there's a log of this, @collinforrester wrote all of this, I happened to be the one who cloned it over. We'll make sure that doesn't happen again.


The first thing that came to mind was "File a bug with Apple," although I'm not sure how effective that'd be... I think that technically the OS X behavior counts as broken in this case by PEP 0394, so maybe they'll do something. It's bug #21352953 with Apple, but I don't know how to link to those. And their bug reporter page has weird UI.


Which is?


nice


I'm going to try installing this on an Android phone I have. I might be able to contribute to it.


I think this is incorrect, as the pentultimate should be the 5th last, not the 5th from the start :)


If MonSessions are recounted we need to put() the existing one when overwriting it.


Split getting the project file from parsing its contents.


Added class level comments : https://github.com/manoramahp/carbon-identity/commit/67df982ec3ad59aa6b864992c55c0c4a1d182a03


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


Do you clear this out later? Don't leave in commented code. ;)


Feature request: Different battery status views: 1. Status not shown (save some screen estate), 2. Only percentage shown in upper/lower left/right corner, 3. Battery bar.

Feature for all 3 viewing options: Only show battery if lower then X percent. X definable


what?


;p


Each tile generated is a Phaser.Sprite object, can we attach a body to those?


Feature request: When in menus, after klicking "back" don't always highlight the first item of the previos menu. Highlight the iten that you just came back from.


Thanks for the hint. Fixed.


Bug: When opening menu (setup or widgets), highlighting always switches to the second item.


Oops, just forgot about it. Commited by mistake.


please check background before post thanks


More commented code.


I took binary mingw64 package from Debian. The one these patches are is from jessie (the one from wheezy was even more broken). I don't really believe I can manage to get fixes released to it in reasonable time. Though I could try, for the new testing. Another option would be to search through ubuntu's ppas. Some of those also provide libraries, so probably they really should be tried.


*g*


please don't ever die thomas


`getaddresses` will return `[]` for unresolvable hosts, so it should be all good.


I worry about forking a project as active as this one.  What adds the bad parameter?  Can we alter that instead?


Just compiled without `.iter()` without problems. Release notes about this change can be found here:

 - https://github.com/rust-lang/rust/blob/master/RELEASES.md#version-100-alpha2-february-2015


Unknown -> Unsupported?


I don't understand which deleted lines you're referring, I'm afraid. What should this new macro display? Could you explain?


La magia està en esta linea, ya que al iniciar el botòn es nulo, pero cuando se presiona ya no lo es.


rat paci

![image](https://cloud.githubusercontent.com/assets/1418180/8123832/5a9fc29e-109e-11e5-94d1-04ed5ed5bbe5.png)


No. If this is needed I might as well not use it, it should be able to find it by its srg name in the dev environment.


This will start the server, which shouldn't occur until start.  I'm going to fix.


This is equally justifiable in both the new and the old location. It's fine here (but it was also fine there). In other words, sounds good to me.


wow, that's what I call fast turnaround time! :+1:


This file should not be pushed here.


84691154_AutomateBuildingOfDocs (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1674 of 1674 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)


The big font in the 1 layout and the autostart feature ist brilliant, thank you. Also, being able to see the time/flight time is great.


where's the like button on this thing


A small niggle that has been bothering me ;)

Should we add P's to V's or only add V's to V's; keeping the P's in P's?

So this would become view.addBox( newContainer( container ).getView(), serverContainer.getView() ).

If know in the impl of addBox you're doing the getViews().. just wondered if the API should reflect, for composite Widgets, we do not add P's to V's but keep the two separate.... just a thought (that happened to me whilst refactoring the Guided Decision Table editor).


@rjledger Gotta love shared mutable state. This bug dates back to 2012! Momentjs really needs to be immutable.


PrepTransitionFor84 (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1676 of 1676 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 fixes Granite Cave in Pokemon R/S/E where you'd magically fall through the floor when going down certain ladders.

Fun fact, this used to be a problem waaay back when with VBA.


I'm going to stop commenting on these now, but there are still some left on the global diff so you should go through and clean them up.


Bug: Battery status switches between full bar and empty bar when not charging. When charging it switches between 0 and 100%.


r+


84691154_AutomateBuildingOfDocs (lefticus) - 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)


Hi!
After 2 weeks of usage I'm quite happy with the result


@rjledger Looks like others agree: https://github.com/moment/moment/issues/1754


Yea we could make a gist of the difference in pivotal ui charts for comparison. there is a slight difference as well in the behavior. In this implementation each change to the cursor results in an update to the application state, the async version combines the changes and applies the full change in a single callback. There are ramifications for both implementations.


This div is not in use.


This looks like, line for line, the same semantic code as insertDoc1. Let's factor out the common code, either using a closure (in this case, a function that returns a `func(int) error`), or take an extra string parameter to facilitate the log message customization.


Ack. Fixed.


I'd be fine with any approach. I don't see leak in P touching V... I'd see problem in the oposite case (`third party` V touching P)


I like this section. It's much clearer than it previously was.


agreed, on both items. changes made.


saw approval from MonsieurNicolas
at https://github.com/graydon/stellar-core/commit/0689fdbd06e6ccc335aeec2ca388c142d5bf04fc


merging graydon/stellar-core/accelerate-history-probe-when-time-accelerated = 0689fdbd into auto


graydon/stellar-core/accelerate-history-probe-when-time-accelerated = 0689fdbd merged ok, testing candidate = 203bd07c


The point was to show how the buffered channel is providing the extra performance because the second insert can proceed. I would like to show somehow these are two distinct operations. Maybe this can be something else.


I think the proceeding line needs to flow into this example a bit more fluidly. Also, use `==` :)


... ~~that~~ "which" is ambiguous ...


I ran a test for you. That's because getActivity().findViewById returned null.


"introduce"
Should we use Javascript or EMCAScript throughout?


did you guys check to see if recreating the cursor with new data in this recursive manner an be collected by v8? It's not clear to me if this causes a leak or not.


yep, done.


Your commit log says "fix this", but the logic is unchanged...


No need to use parenthesis.


Was just wondering about authorship xD


surprisingly it worked. I need to test it a little more though.
Those fragments need a volunteer developer looking to learn about our source code so that he/she can create an AbstractMusicBrowsingFragment, I have a feeling there's a lot of repeated code, but not worth our time at this point.


@aldenml this `LoaderCallbacks`, I find this to be such an ugly, and not so fun to work with abstraction, although it might be great, it's like when you don't like a painting.


Thanks for the transition-only `loading-bar-directive`. Our designers were against the ever-present loading bars for everything. :+1:


Then we can fuzz the code of these two functions so they look distinct.


Each tile generated is not a Phaser sprite object in this branch. That is what this branch is all about.

This branch doesn't generate tiles, it generates sprites enough to fill the screen and just reuses those in a pool to make the map look like it is scrolling. There are never more sprites than there is screen space, and those sprites can refer to any tile at any time.


Ok, I will do that first before merging. Unless you have a good idea on how to do that now.


It can just be something pointless but visually distinct. maybe some fake low-level call to a "query" function -- we can pass "INSERT INTO A" in insertDoc1, and some other query string in insertDoc2


the random failure logic can also be pulled into that query function.


Ah, ok. Great Idea.


This condition was added to fix http://sourceforge.net/p/phpmyadmin/bugs/4927/ where disabling foreign keys in import was not working when the import file had only one statement.

When there is just one statement in import file, `$go_sql` is made to `true` and this turns the import into sql mode where it acts as if the user has submitted this single query through the SQL query box. As far as I can remember, this bypassed the disabling foreign keys and that is why I had to add the condition.

This condition might not be required since you have revamped this feature, extending this to SQL query box as well. Anyway, bug #4927 is the test case to check after removing this condition.


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/66491330


「出ているかのテスト」だと、
「エラーが出ればOK? エラーが出なければOK?」というのが分かりにくいかな。


c362642ad4706a27537cb916f7bc3e20df27b19e と同じく、空 is SUCCESS/FAILURE? が明示されていると意図がより明らかに出来て良いかな


this is really great!  Would you be interested in creating a PR of all this work?


Because I'm playing with an embedded server with minimal ram/cpu. Every time I (re)install the bot, it takes a *long* time for everything to download and for native dependencies to compile. I realize that I could just cache packages, but I've got no clue how to figure out how to cache *some* packages and not other packages. Instead, I'm hoping to cut that time down by simply `npm install steam-chat-bot --no-optional && npm install ueberDB sinon google ~/localnpm/steam-trade` if all I need is infobot and google and steam-trade.


`options.liveReloadPort` doesn't exist here. it should be `options.port`


Fantastic ! I like it!


Thanks :+1:


Looks good. Since these are no longer betas, the bug fix and new stuff list is pretty obsolete at this point.


fast-forwarding master to auto = 203bd07c


I new it


thinking we should add edit and update routes and views.


Lol this is the wrong way to do this.

http requests include error codes. You want to check that, and then print the message if it's an error.

http://en.wikipedia.org/wiki/List_of_HTTP_status_codes


The Specification is 192x192 pixel instead of 196x196.


PrepTransitionFor84 (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1676 of 1676 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)


84691154_AutomateBuildingOfDocs (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (644 of 644 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.32%25-green.svg)


I'm not generally a fan of commit logs that tell you nothing about the commit... like, I'm not saying "squash or I won't merge it", just that it'd be appreciated if you were more verbose in the future.


Cabal-1.22 (the version used by ghc-7.10) changed the type of `finalizePackageDescription` and the `#if` is there to ensure compatibility. It looks like the `MIN_VERSION_Cabal` macro is defined by cabal itself, so when liquid invokes GHC the macro is not defined and `cpp` fails.. sigh...


This looks strange to me at first glance. It looks like the intent is to read the `eupsPath` entry from `env` if it exists and is `True` but it checks `eupsProduct` first when it should be checking `eupsPath`.


> Basically you want to wait for another goroutine to reach line 164, right?

yes

testutil.WaitSched is just for test usage. I will make a helper function to hold this.


The `versionString` variable doesn't seem to exist in this function.


not necessary to change.


yes. will delete it from the map.


The queue:work don't have the timeout option. You can try  ```php artisan help queue:work`` for detail.


media


销售


troubleshoot


The macros are defined in `dist/build/autogen/cabal_macros.h` so we probably just need to make sure that's on the include path.


comentado porque  "mod" lo resume en una linea


@sajus @gayatrisabade Please review and suggest changes if any


hello,
please Upgrade  to laravel 5.1
this is my errors when update
 Problem 1
    - Installation request for barryvdh/laravel-elfinder ^0.3.3 -> satisfiable by barryvdh/laravel-elfinder[v0.3.3].
    - Conclusion: remove laravel/framework v5.1.1
    - Conclusion: don't install laravel/framework v5.1.1
    - barryvdh/laravel-elfinder v0.3.3 requires illuminate/support 5.0.x -> satisfiable by illuminate/support[v5.0.0, v5.0.22, v5.0.25, v5.0.26, v5.0.28, v5.0.33, v5.0.4].
    - don't install illuminate/support v5.0.0|don't install laravel/framework v5.1.1
    - don't install illuminate/support v5.0.22|don't install laravel/framework v5.1.1
    - don't install illuminate/support v5.0.25|don't install laravel/framework v5.1.1
    - don't install illuminate/support v5.0.26|don't install laravel/framework v5.1.1
    - don't install illuminate/support v5.0.28|don't install laravel/framework v5.1.1
    - don't install illuminate/support v5.0.33|don't install laravel/framework v5.1.1
    - don't install illuminate/support v5.0.4|don't install laravel/framework v5.1.1
    - Installation request for laravel/framework == 5.1.1.0 -> satisfiable by laravel/framework[v5.1.1].


Perhaps it could be:
```
function T.SkinNextPrevButton(btn)
	local normal, pushed, disabled
	local isPrevButton = btn:GetName() and (string.find(btn:GetName(), "Left") or string.find(btn:GetName(), "Prev") or string.find(btn:GetName(), "Decrement") or string.find(btn:GetName(), "Back"))
	local isScrollUpButton = btn:GetName() and string.find(btn:GetName(), "ScrollUp")
	local isScrollDownButton = btn:GetName() and string.find(btn:GetName(), "ScrollDown")

	if btn:GetNormalTexture() then
		normal = btn:GetNormalTexture():GetTexture()
	end

	if btn:GetPushedTexture() then
		pushed = btn:GetPushedTexture():GetTexture()
	end

	if btn:GetDisabledTexture() then
		disabled = btn:GetDisabledTexture():GetTexture()
	end

	btn:StripTextures()

	if not normal then
		if isPrevButton then
			normal = "Interface\\Buttons\\UI-SpellbookIcon-PrevPage-Up"
		elseif isScrollUpButton then
			normal = "Interface\\ChatFrame\\UI-ChatIcon-ScrollUp-Up"
		elseif isScrollDownButton then
			normal = "Interface\\ChatFrame\\UI-ChatIcon-ScrollDown-Up"
		else
			normal = "Interface\\Buttons\\UI-SpellbookIcon-NextPage-Up"
		end
	end

	if not pushed then
		if isPrevButton then
			pushed = "Interface\\Buttons\\UI-SpellbookIcon-PrevPage-Down"
		elseif isScrollUpButton then
			pushed = "Interface\\ChatFrame\\UI-ChatIcon-ScrollUp-Down"
		elseif isScrollDownButton then
			pushed = "Interface\\ChatFrame\\UI-ChatIcon-ScrollDown-Down"
		else
			pushed = "Interface\\Buttons\\UI-SpellbookIcon-NextPage-Down"
		end
	end

	if not disabled then
		if isPrevButton then
			disabled = "Interface\\Buttons\\UI-SpellbookIcon-PrevPage-Disabled"
		elseif isScrollUpButton then
			disabled = "Interface\\ChatFrame\\UI-ChatIcon-ScrollUp-Disabled"
		elseif isScrollDownButton then
			disabled = "Interface\\ChatFrame\\UI-ChatIcon-ScrollDown-Disabled"
		else
			disabled = "Interface\\Buttons\\UI-SpellbookIcon-NextPage-Disabled"
		end
	end

	btn:SetNormalTexture(normal)
	btn:SetPushedTexture(pushed)
	btn:SetDisabledTexture(disabled)

	btn:SetTemplate("Overlay")
	btn:SetSize(btn:GetWidth() - 7, btn:GetHeight() - 7)

	if normal and pushed and disabled then
		btn:GetNormalTexture():SetTexCoord(0.3, 0.29, 0.3, 0.81, 0.65, 0.29, 0.65, 0.81)
		if btn:GetPushedTexture() then
			btn:GetPushedTexture():SetTexCoord(0.3, 0.35, 0.3, 0.81, 0.65, 0.35, 0.65, 0.81)
		end
		if btn:GetDisabledTexture() then
			btn:GetDisabledTexture():SetTexCoord(0.3, 0.29, 0.3, 0.75, 0.65, 0.29, 0.65, 0.75)
		end

		btn:GetNormalTexture():ClearAllPoints()
		btn:GetNormalTexture():SetPoint("TOPLEFT", 2, -2)
		btn:GetNormalTexture():SetPoint("BOTTOMRIGHT", -2, 2)
		if btn:GetDisabledTexture() then
			btn:GetDisabledTexture():SetAllPoints(btn:GetNormalTexture())
		end
		if btn:GetPushedTexture() then
			btn:GetPushedTexture():SetAllPoints(btn:GetNormalTexture())
		end
		btn:GetHighlightTexture():SetTexture(1, 1, 1, 0.3)
		btn:GetHighlightTexture():SetAllPoints(btn:GetNormalTexture())
	end
end
```


@fedexin40 As explained by @legalsylvain the query migration script is called when the first time the module is installed. (And only the first time). Do you have tried to execute manually the 2 queries into the script to check if something is wrong wtih these queries on your db?


It's probably because those tests were never updated to compile on 7.10 and are now breaking because Applicative is a superclass of Monad. It's easy to fix, we just need to add Applicative and Functor instances. I didn't notice when I was working on 7.10 because the icfp benchmarks weren't being run on travis :(


コミットメッセージの `practice` 要るかなｗ
to create~ って書いてもそんな違和感ないと思う（1行目にsolves exercisesって書いてるし）


So if I were to call the phaser method of getTile, right now in the dev branch its returning the Phaser.Sprite object. What should it be returning in the end?


Best we're ever going to get ;)


Considering #2,
- is the `cmake` process a success ? If not: **why** ?
- have you compiled an application ? if not **why** ?


`to not` じゃなくて `not to` かな


あ、コミットメッセージの話です.


You thought this was wake and not water on windshield?


Is there a way left for maintainers to update/add information of tools?


Thank You sir

On Sat, Mar 28, 2015 at 6:30 PM, Chabardes <notifications@github.com> wrote:

> layout main.xml
> <https://github.com/pchab/AndroidRTC/blob/master/res/layout/main.xml>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pchab/AndroidRTC/commit/06236e784cd989e9aec212a49e23c40ef15f91e5#commitcomment-10449963>
> .
>


Can you try with the @dev stability?


@Naddiseo, I want to merge Overview and Motivation. "The goal" is mentioned twice.


Here we mention what the goal is again.


I am also having same problem. It is not working due to youtube API update.


Yes, test tweaking is not part of this one.

BTW: The above error is easily fixed with the usage of simply `select` from Capybara.

Here is the part of the documentation regarding multiple selects:

> If the select box is a multiple select, select can be called multiple times to select more than one option

http://www.rubydoc.info/github/jnicklas/capybara/Capybara%2FNode%2FActions%3Aselect

But as you said it's not part of this PR.


Yup :-)
But the test [failed](http://my.cdash.org/testDetails.php?test=19724037&build=779293) :'-(


Not sure what you meant by "flow into", but perhaps my last commit helped? I made some comments on 83291b012b5af03d94c5d51ff1feb1ca9c35f8bb about the Overview and Motivation sections.


Hi, Mine sharing your run.sqf, we run some of the same addons and the locking and unlocking stuff don't work on my server, thermals (2) kicks everyone


Btw this breaks on OSX. org.apache.fop.apps.FOPException: Failed to resolve font with embed-url '/usr/share/fonts/truetype/liberation/LiberationMono-Italic.ttf'


Since not everybody uses @, would quantity_price_symbol be more appropriate (or something else along those lines)?


Is the import above still necessary?


Shouldn't you replace the option key with "abrt_analyze_upload_coredump"?


asdf


Ah, this code part is moved code, let's keep it untouched.


@lukescott, I think we should keep them separate.
Overview -> what this proposal does.
Motivation -> what the current problem is.

But definitely need to reword that second "the goal" sentence.


Guter Indikator für einen Merge-Konflikt ^^
Versuch mal bitte immer mit `git pull -r orgin master` zu updaten. Wenn du dabei auf dem Branch bist den du weiterbringen willst und schon Änderungen gemacht hast, werden deine Änderungen zurückgerollt, die Änderungen aus dem Master angewendet und dann deine Änderungen obendrauf wieder ausgerollt. Dein remote origin im Beispiel sollte natürlich auf FlyHinotori/autopilot zeigen.


By flow into: There's a sentence, then there's a block of code. They're disconnected.


kernel aufs desteği aşağıdaki commit ile mevcut,  https://github.com/pisilinux/core/commit/d883c2773e1b54304613d3bbba8a392addb088e9

alabiliriz.


README.md

 master Last changed by rockinrex 40 minutes ago
finna-be-avenger

 Desktop version


I know this doesn't compile. It is a work in progress.


Why do you declare it "char *" instead of "const char *"?


ok, i played around with it and clearified it for me:
```php
<?php
echo 'hello!';
ob_start();
echo 'outer';
ob_start();
echo 'inner';

$a = array();
$all = '';
while ($l = ob_get_level()) {
	$out = ob_get_clean();
	$a[$l] = $out;
	$all = $out . $all;
}
var_dump($a, $all);
// array (size=3)
//  3 => string 'inner' (length=5)
//  2 => string 'outer' (length=5)
//  1 => string 'hello!' (length=6)
//
// string 'hello!outerinner' (length=16)
```


I regret that Medusa emulation couldn't make it in. I believe someone needs to think of another more clever way to emulate zone heap contents that doesn't require a 3 MB allocation :disappointed:


@loewenheim I have changed the SipProver class to return a SimpleInductionProof, is that enough?


It's a bummer that you're leaving, Jacob.  We could've really used your help next year.  Good luck in your math studies, though.


Do you really need this conversion? The old code simply returned "val". If you can call "string_to_bool(val)" here, you can also use this construction "string_to_bool(abrt_app_configuration_get_value(...))" anywhere else.


TFC Should implement methods to apply Damage reduction through configuration rather than use of the API since HP / Damage calculation are out of line with base game.


OK.  Also, I moved my appointment, so I can stay for a while after school tomorrow.


Why did you make this change


I have replaced winston for log4js in latest GW version, so new software is not using it anymore.


No, it isn't. I'll erase it.


I realy wondering why autor of this mail had problem with replace(), but not with before() and after()?
A.before(B) << how did they deducated what must be done here? A set before B or B before A (without reading the spec?)

Personally I don't like this change, all this new methods was short and clear enought, but now you put one longer and boom...camelCases. The only plus is that some other lib use this new name, but in practice I still prefer shorter name.


Nice test ;)


                                                                                We haven't changed our version of excon in a long long time. I forked and set to the version we've been using.I spent an hour in Fog this morning, and it's just a version mismatch between Fog and Excon. If we upped our version of Fog, we'd get rid of the warnings, but unsure about everything else.The two params passed to Excon it complains about are :host and :schema. We can stop passing :schema to excon and be okay, but if we stop passing :host we get a 403 from AWS.As well, we've forked Fog so we're version locked for the time being.                                                                                From: Kenneth BerlandSent: Thursday, June 11, 2015 9:57 PMTo: ConsultingMD/exconReply To: ConsultingMD/exconCc: Justin AhnSubject: Re: [excon] remove the validation that throws a warning (398f826)I worry about forking a project as active as this one.  What adds the bad parameter?  Can we alter that instead?

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


@everilae, thanks for teaching data tables to speak in text.

+1


Do you plan to add more buttons to the triple switch widget :)? I would replace this for loop with 3 ifs
```
   if (gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(option->radio_button_widget[ABRT_RADIOBUTTON_OPT_ALWAYS])))
   ...
   if (gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(option->radio_button_widget[ABRT_RADIOBUTTON_OPT_NEVER])))
   ...
   if (gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(option->radio_button_widget[ABRT_RADIOBUTTON_OPT_ASK])))
````
Or you can go crazy and associate the value with the widget using *g_object_set_data* like "abrt-option".


This should be 0.4.1 since its mostly a bug fix and testing.


You do not need to create another shared pointer just to test that existing pointer is non-null. Shared pointers look very much like a raw pointer and statement `if (ptr)` works fine on shared pointers.


There is a GCC method for that. Maybe it will perform better


`if (mergeStmt)`


`return  std::shared_ptr<query::SelectStmt>()`


习惯都小写了


Do not instantiate any object unless it is actually used.


最後の2行余分でした.
後で修正用のコミットをしておきます


(More of a question than feedback, hope that's fine.) Why did you move to brace-initialization here? Won't this be an ambiguity for types having an `initializer_list` overload?


Why not +=12 the string pointer?


Does the default port have to be handled here?
how about set the default port in core/config?


@philwebb is this means Spring Boot will require Java 8? I will be really glad that If Spring Boot will require JDK 8 :)


Can you please provide link to the spec?


Oh god. I totally overlooked that. Will work on it asap, along with a few other quirks.


Kanged xD
Kidding
https://github.com/k2wl/android_device_samsung_serranodsdd/blob/cm-12.0/device.mk#L30


It appeared to be collected when we tested.

On Thu, Jun 11, 2015, 22:15 Ryan Dy <notifications@github.com> wrote:

> did you guys check to see if recreating the cursor with new data in this
> recursive manner an be collected by v8? It's not clear to me if this causes
> a leak or not.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pivotal-cf/pui-cursor/commit/d28a1088c434efb8855abf3d824e355a46057cb4#commitcomment-11647718>
> .
>


Ah, thanks - well spotted. Will fix that.


Instead of array please use the vector or others data structures.


This good idea should pass LTC if tuned at larger time controls.


@sizzlemctwizzle
Non-existent url after remove e.g. opened the discussion in another tab, removed my goo self on dev as usual during testing, then refreshed the tab with that discussion in it and it bombs dev:

``` sh-session
/home/user/repo/git/OpenUserJS.org/martii/OpenUserJS.org/controllers/discussion.js:282
          discussion.save(callback);
                     ^
TypeError: undefined is not a function
    at Query.<anonymous> (/home/user/repo/git/OpenUserJS.org/martii/OpenUserJS.org/controllers/discussion.js:282:22)
    at /home/user/repo/git/OpenUserJS.org/martii/OpenUserJS.org/node_modules/mongoose/node_modules/kareem/index.js:177:19
    at /home/user/repo/git/OpenUserJS.org/martii/OpenUserJS.org/node_modules/mongoose/node_modules/kareem/index.js:109:16
    at process._tickDomainCallback (node.js:381:11)
```

This will probably kill production with continual restarts if active.


Array element count does not equal sizeof array !!!
array_element_count = sizeof(array) / sizeof(array_element_type)


Please add ', " brackets type


You can use ``getFlatData()`` since we use only user attributes and want all users of facility. Users are then located elsewhere (look at API JavaDoc in perun-core - ServicesManager).


@sizzlemctwizzle
Great that you picked up on this... think it's missing last commentator change and last commentator time correction I think. Effectively I can't remove anyone at the moment without causing some DB corruption... does seem smooth in the moderator tools though. :)


I was not able to do any of the above, i am not used to working on the terminal(using commands) so everything looks a little bit new to me and i am trying hard to familiarize myself with it, But i  will manage this.


just return open_brackets.empty()


@s8321414
```bash
git commit --amend -m "catalog: zh_TW translation"
git push -f
```


A retry of 10000 is too much!

Please put the default to something like 10 instead.


This is addressed in the next commit cdfc0bf5a66e199d3a85339e11be4e6f33eaf608.


Please print also header of csv file, where you will use friendlyName parameter of attribute.


Why  0?


Thanks :-)


Why was this resetStopOnReset here then?


sir, how can I output the graph representation after convert it from smiles?


It would be nice, if you could sort all users by their login before printing (put them to map, get all keys, sort them, then iterate and print). Thanks.


I guess the offsets should be easy to remove. Maybe we want to keep the sentence number to reduce the time it takes to calculate it from the CAS... not sure. In any case, I believe these values are currently not all being set in the same place. Instead of having six different setters, I think it would be better to have a
setWindow(Sentence first, Sentence last) and calculate the other information in there. We could still keep the individual getters.


```C#
!await UserManager.IsEmailConfirmedAsnyc(user.Id))
```
looks wrong to me (mainly the `!await`). I'd prefer the brackets around the await.


承載


license add


How about use

```
$ git remote add upstream https://github.com/Keanux/keanux-personal.git
```


Thanks. It helped a lot. :)


`String` should be used when referring to the `System.String` class (i.e. `String.IsNullOrEmpty`), whilst `string` should be used when referring to the type (i.e. when referring to an object, as in this case).


result pylab image is below.
![kalman_result](https://cloud.githubusercontent.com/assets/4533295/8124764/e4c8735a-1119-11e5-88b1-3a8fa7781bdb.png)


Rather than use this 'magic number', 3, make this loop dependent on the length of the sentence you're translating. This principle will make your code more reusable.


I optimize like that too. We define out of bounds array access to return null so it's fair to discard it like this in my opinion. Can you give an example of that special getter/setter behavior?


Zambia


This function should return "chirp chirp chirp" and not log it. If you run the tests, (with `npm test`) you will see that they're not passing because they're not returning the right value.


btw, this will not raise an IndexError if no `(` is in the string `id`:

```
>>> id = 'hi'
>>> id.split('(')
['hi']
```


Nice!


Create a text file called HelloWorld.md


Does this want to be embedded into the assembly? Would simply having it as a file on disk be better, as we don't have to redeploy just to change something on the email template?


We want to return a string here, not an array. And again don't forget to `return` not `console.log` the value you want to come out of this function.


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


Should be `String.IsNullOrWhiteSpace(...`.


なるほどです(英語ﾑｽﾞｶｼｲ)。
次回以降、似たような問題が出たら気をつけます〜


Applies to #126


czesc,
ten skrypt nie bedzie dzialac do zapisywania
ogolnie  w tt++ jest opcja ze mozesz caly konfig zapisac do jednegeo pliku. takie rozwiazanie tu jest ze nadpisujesz swoj konfig z informacjami z bazy [zeby dodawac przemdioty]


Type might seem to be obvious, but it's a method call, so better to be explicit. Type should be `Assembly`.


flere funktioner?


reason is so I don't have to update the year, lol


どちらが正解なのか迷ってました。
`not to` なのですね。ありがとうございます...!


@chelmuth, @nfeske: please add to staging


I don't understand why we don't use our actual zone heat?


Math.random() gives you a number between 0 and 1 (not including 1). If you take this number and multiply it by the length of the array and round that down (with Math.floor), you'll get a number in the range between 0 and the last index of the array (length - 1). With this number, you can access an item at a random index and eliminate all these if/else clauses. Let me know if you have questions about this.


If ipv6 is not supported in the zone, please create a sort of Ipv6NotSupportInZoneException, do not return null.


PrepTransitionFor84 (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Why is there a clone and cast here?


I think you have the right answer here but the test is failing because you're not returning this value.


Specifying a returner on the minion is completely supported. I thought it
was documented, but if its missing or incomplete I'd definitely open an
issue for that-- since it is a feature that a lot of people use :)

On Tue, Jun 9, 2015 at 10:00 AM, Konrad Mosoń <notifications@github.com>
wrote:

> @jacksontj <https://github.com/jacksontj> We are working on our returner
> to do this (local_cache returner doesn't matche our needs).
> I was able to hack what we need using return in /etc/salt/minion and
> ext_job_cache in /etc/salt/master and /etc/salt/minion and scheduler + my
> own returner ;)
>
> The only problem with this is… return in /etc/salt/minion is
> undocumented… but as I saw if source code it should works (and it probably
> worked since always).
> I guessed it from salt-call --return myreturner state.highstate, and code
> says that salt-call loads /etc/salt/minion to load config, and then parse
> commandline arguments to apply cli config (that's why return in minion
> config it works).
>
> I just want to be sure this will be documented and will work later too…
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/saltstack/salt/commit/93c7e3ec397d840f4702e2db8e3ae67be87ce4e6#commitcomment-11593546>
> .
>


No I see why you added 10k as default number.

This retry mechanism is just wrong. In the worst case scenario, you will run the query 10k times agains the DB.

Please change it and implement a retry based in time instead.

For example:

retry 5 times; 1st retry in 5 seconds; 2nd retry in 10 seconds; 3rd retry in 20 seconds; ... something like that or better.


LGTM


In general I recommend filing issues rather than commenting on commits. The reason we changed the name here is because these names were inspired by existing libraries, that do use `after()`/`before()`, but not `replace()`.


@mthli I updated third-party download link, finally. you can check out this commit: https://github.com/pcqpcq/open-source-android-apps/commit/81e42cbf90ba845d2fbd71e26fa7c9a9ce874ce6


Why do you set max-height?


Should be `String.IsNullOrWhiteSpace(...`


This can be `var`.


Again as above. `String.Copy` can be used, but neither are necessary.


dude. this will break existing installations. :-/


And should go to line 84, after the `None` check.


Leftover.


it is normal, it is a reproducer for a bug in RT0


s/value/Value and `.` at the end. :P


Ugh what?


It's not clear what the `string code` parameter is for. It looks to be the confirmation code, but it's not 100% obvious from the method signature.


Can you... not make entire commits for this kind of junk


You know, I can't remember why I was doing that (&nbsp replacement) anymore... there was some reason.  I thought it had to do with select/copy/paste or the cursor css or something, but I'm not sure

I did notice a very slight bug with this commit:  the cursor doesn't show up when it's on a space right before a line wraps around.  Fix that and then this is good to go.


Uh right. Shame on me.
Added those calls as well as removed 75L everywhere.


Fixes #1.


Not sure on the name "RegistrationConfirmation". Whilst that conveys what the page _is_, I see it as more of an action, and would suggest "ConfirmRegistration".


This should now be better readable.


"metrics" left here


Fixes #2.


You should raise a config.error, and put the import statement on top.


needs to handle nil users in array


Fixes #3 with hubot-scripts/tasks.coffee instead.


plain except are a no-go. If you expect multiple exceptions, put them here all, or split your try statement.


get error :
packages/services/Telecomm/src/com/android/server/telecom/TelecomService.java:456: error: method does not override or implement a method from a supertype
        @Override


Nice! Interesting implementation.


"If not, ensure you check your spam filters and click _the button_ below if you still can't find it."

"If you don't see an email after a few minutes, check your spam folder." The button being labelled "Resend Email" should be obvious enough that we don't need to spell out what it does. Possibly label it "Resend Confirmation Email" if we want to be more explicit.


reviewed and sent to hugo


Whilst this is just fixing a test, the value should be accessible in all the tests in case it's required at some point. In fact, why do these tests now need it, and should there be tests that now use it?


ack :)


Public TODO means we're more likely to bother with it eventually. And ... I did what I noticed, nothing more >.> I'll patch it with my next commit I guess


Good work. Clean and straightforward. You might want to think about what happens if we the average never drops below 4 but then again returning undefined here isn't the worst thing.


@dca 發 PR 修正 ＸＤ


Also as it just comes to me, should I make `remote` a separate method to reduce the shell out overhead in `initialize`?


we should filter nil users here instead of doing it in Players


As we're now explicitly using the email as the user identifier, we should update the database models to enforce uniqueness in the database (as currently email is non-unique, although we don't allow it to be set separately from the username).


sorry, fixed it now.


對齁，就在想好像有指令可以用... 來個PR吧~


If someone attempts to resend a confirmation for an account that hasn't signed up, do we want to let them know they've not signed up? This mechanism allows people to see if a given email has an account on the site, which isn't really good practice.


Looks good, clean and straightforward with no unnecessary variables.


No it doesn't.


Great, you could even just return favoriteAnimals[randomIndex] and skip this assignment


I thought the same but I can't get it worked. I will try it once again maybe I will have more luck this time.


i would rather have a ```10000``` and a ```>=``` in the code.

Please modify vusion-deployment to add this line. find the file in puppet/modules/cakephp/templates/vusion.php.erb


Two calls to `GoToBaseUri(driver);`.


Yes, you're totally right. This was my mistake. I wanted to refactor these, because every number gets converted into an integer the old way. I will figure a way out to do this correctly.


@sathiyas, please link with RHEL script/conf you working on. initctl is a systemd stuff so, maybe there a different approach.


BringToFront() on one window does do nothing if the target window is already at the front, but here it's being called twice on two different windows.

In the most common case the AutoHideWindow will already be at the front and the Document will be next behind it. In this case the original code does this

DockWindows[DockState.Document].BringToFront(); // brings document to front *over* the autohide window
AutoHideWindow.BringToFront(); // brings autohidewindow back to the front over the document

This causes me a whole load of flicker in the autohidewindow as it gets its layout re-calculated and controls re-drawn.

The code above is basically checking to see if the current window zOrder is the desired zOrder (Autohide window at front, just document behind) and in that case doing nothing. In all other cases it runs the original 2 lines of code.

Hope that's a bit clearer now


I don't think we need this variable if we're only using it in the next line, just `+=` mealsPerDay[i] directly


I can't honestly believe that we are the first one to needs such a functionality,. Have you looked around in the code base how others are doing ?


I think it's best avoid nameless subjects in specs, especially if you're going to explicitly refer to them later on (like you do in the `db:sample_data` spec).

Reckon we could make this `subject(:task)` instead?

Then in the other thing you could use `task.invoke` rather than `subject.invoke`, which is an improvement in clarity, IMO.


This is a link to the edit form, not a submit button!


This is a header, not a submit button!


Llevas razón, muchas gracias por comentarlo


I would probably just return at this point and save the extra iterations because we only care about the first tooHungryDay but this is fine too.


PrepTransitionFor84 (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (646 of 646 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.32%25-green.svg)


Yeah, those doesn't need to be included. Want to file an issue (or send a PR)? :smile:


Hi, we have similar problems with nulls first in a order by query. Unfortunately this patch did not help. Hibernate generates invalid sql for mssql server 2008. Aliases in case-then-when statements in order by did not work. We get a "invalid column" Message.


Search for _.countBy on this page: http://4waisenkinder.de/blog/2013/10/12/you-use-underscore-slash-lo-dash/


I would say this is:

    // TODO: read from DynamoDB, update, write to DynamoDB


I had the problem where a full reindex would complete while a partial reindex would exhaust all my memory. Google brought me here and this patches finally fixed it. However now the search in my store (frontend) isn't working anymore. Always show the correct number of result numbers but only show one product, which isn't actually a product but just a product item without any data (name, price, image). Reverting to the original Search.php bundled with 1.6.14 works fine. Did you notice this?


I wanted to prepare the function to make it simple to add additional buttons but you are right, It will be clearer with IFs.


With this, wouldn't it make more sense to make an empty AtomGroup evaluate to false?

Then we could use

```
if self.filter_first and self._s2:
```


Thanks for explanation @snicoll


+1

Thank you thank you :D


@tremenyi I have just knocked this up, would you mind commenting before I pass it on to Rog.


You dont want solely the size because some squares are more important than
others (central squares for instance)

Le jeu. 11 juin 2015 22:46, Navid <notifications@github.com> a écrit :

> And for mobility I need the number of attacked square, so simply the
> vector size for each piece. There must be a way to find it (or store it ?)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/RomainGoussault/Deepov/commit/5d075bdd91a505712485c0b0749750a0b951cf40#commitcomment-11642798>
> .
>


This shouldn't be very hard. Everything is same, except the crypt algorithm. We just need to replace the md4_crypt with md5_crypt along with the format-name of course.
Currently hash checking performance is very poor and currently I'm in the process of fixing it. Maybe we should try it afterwards.


Do while works here but what if you were processing an empty string?


We need C89 support so no // comments please. And also please use a proper commit message!


On 12 Jun, 2015, at 00:24, Richard Gowers wrote:

> With this, wouldn't it make more sense to make an empty AtomGroup evaluate to false?

That's a good idea. Are there any double-underscore magics one can play with?

I don't think it will have negative consequences, just allowing for more concise code as you point out. I suggest you open an issue.

>
> Then we could use
>
> if self.filter_first and self._s2:

--
Oliver Beckstein * orbeckst@gmx.net
skype: orbeckst  * orbeckst@gmail.com


We should look at "attack and defence map' on chess programming wiki

Le ven. 12 juin 2015 09:24, Goussault Romain <romain.goussault@gmail.com> a
écrit :

> You dont want solely the size because some squares are more important than
> others (central squares for instance)
>
> Le jeu. 11 juin 2015 22:46, Navid <notifications@github.com> a écrit :
>
>> And for mobility I need the number of attacked square, so simply the
>> vector size for each piece. There must be a way to find it (or store it ?)
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/RomainGoussault/Deepov/commit/5d075bdd91a505712485c0b0749750a0b951cf40#commitcomment-11642798>
>> .
>>
>


seems not used XD


thanks @halkyon


aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa


Actually I'd see this as more of a bug in the tests. Spring doesn't create a web container unless the classpath is primed for it, and what is Tomcat or Jetty doing on the classpath in Ratpack tests? It might even be good for users to have the option to build a parallel Servlet container in the same app listening on a different port, and this change seems to prevent that (or at least make it inconvenient).


@nnector25 Hi, Karl. If you open `BluelivAlienVaultPlugin.py` and paste:

```python
import sys
sys.path.append('lib')
```

Before the line:

```python
from lib.dateutil import parser
```

Does it work? If not, can you please send us an email (joao.alves@blueliv.com) with details such as Operating System and Python version?

Thank you very much,
Blueliv Engineering Team


Is this line necessary?


Rather than using if/else, what if we multiplied randomNumber by the length of the array (rounding down) to get a number we could use as a list index ? This would make this function more flexible for arrays of different lengths. See what you can come up with.


bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbb


should be >= I think


can we use flash memory of arduino mega 2560 for getting image for gt511c3


Linting error here, day has already been declared.


I have the same issue, and didn't found any solution yet, except this.
Thanks for django rest framework by the way. :)


another variable that's already been declared


Andrew, can Mach7 help with this? I can help you guys with patterns that would be useful to your domain and interact with the rest of Mach7 library.


now, you should not need the [hack](https://github.com/jashkenas/ruby-processing/compare/jruby-9000#diff-79dc060bab7120ee8a2f26520b22d9e1R184) as jruby and core will be loaded by the same class-loader ...


		+function get_insta_photo($username, $count, $link = TRUE) {
     $ins = '';
     $html = file_get_contents('http://instagram.com/'.$username.'/');
     $html = strstr($html, '["lib');
@@ -28,7 +28,9 @@ function get_insta_photo($username, $count) {
             if($secure):
                 for($i = 0; $i<$count; $i++){
                     $ins .= '<div>';


Thank you jj, for the tip.


Maybe just return here? If the average is never less than 4, what will tooHungryDay return?


Thanks @mshibuya!!  :smile:


I'd strip this functions section away, functions change. The glob syntax however is useful for the user of coala which doesn't need to care about internals. IMO.


TODO: add this file to mkdocs.yml at root


@plemarquand,

I am glad you ask because it gives me the opportunity to reflect on this in more detail. Originally, I had the same idea as you, i.e. providing a way to keep session state in a htird party cache/persistent store to be able to provide for fail over scenarios. However, after thinking about this for some time, my current take on it is that it probably does not make much sense to set up a system like that simply because of the additional latency this would create. The session object needs to be updated for each each and every message going through the DTLS stack. If you decide to keep that state in a cache, say Redis, you will introduce two extra remote calls for each message coming in our going out in order to retrieve the current session state and then sync back the updated session (received records, current sequence number/epoch). Depending on your use case, this might even be acceptable regarding latency. However, since a DTLS session is not a big object you can easily keep millions of them in a single Scandium node. So, in order to provide for fail over, my approach would be to set up some (independent) Scandium nodes behind a reverse proxy that makes sure to route packets from the same IP:port to the same node and then rely on the client re-establishing the DTLS session if a node fails.
So, in order to sum it up, I think the anticipated convenience of transparent failover between Scandium nodes is not worth the decrease in throughput created by introducing shared session state.
Does this make sense to you? I would be glad to hear other opinions on this, however. We are still in development towards a 1.0 release so we can revert any changes I have made if we feel that I was premature in making them ...


:heart:


you don't need the log printer so we can strip it away here :)


same here, good bye evil log printer


I think we'll need #if on HAL_CPU_CLASS so we don't blow up cpu usage and memory on AVR for plane


Well both are possible.


well... you know it already


nada, fijate que adentro deje los comentarios y me da paja escribirlos devuelta


I'd rather point to the documentation here later so we don't double that and forget one of them when it changes


Removing the getter is less verbose, it was the way we were using in previous API.
But as we are using getters in the CE, I will use them to keep the consistency


Hi, it seems like `data instanceof Readable` is not working with `fsFile.createReadStream()` for the gridfs adapter. The way `gridfs-stream` uses to create a readable stream doesn't make `instanceof Readable` true. Now I'm having trouble attaching the readstream of a gridfs file to another file. I'm trying to copy a file from one gridfs collection to another. Any suggestions?


yep it seems like it's doing the same thing, so it should work fine without that.


huh what does `= yield` do?


shouldn't fieldValues be /Out/?


On en a discuté avec guillaume, si seul capitaine est ticker ça doit marcher
Le 11 juin 2015 20:22, =?ISO-8859-1?Q?In=E8s_Kapp?= <notifications@github.com> a écrit :Pourquoi pas laisser en Ticker ? Plus rien ne va respecter la notion de pas, du coup.

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


You could also just return `favoriteAnimals[num]` here


why the stdio.h addition?


This equals to 200h. Is this really what you want to do? 12h == 12 * 3600


初始化创建


yes the very same issue happened to me . (speedup, and then frontend only showing matched count, but no actual items showing)

I think it is related to other modifications too (i have seen some table alternations around this commit)
..so to answer my question, i dont think this patch can be applied to a clean 1.6.0.14


7,790 additions and 77,023 deletions!

WOW, good job. Took me a while to scroll down here.


Minor note:

You could use the @note tag
There is also an @param (and @return) tag


This also equals to 200h default flush interval


test


Har dette noen konsekvenser for `migration`-prosjektet?


You're doing the same thing in the if and the else. Maybe just move it outside?


You could also just return here


Fixed in master.


Yes, that's exactly what I meant. Thanks!


Also, I didn't consider the regular expression `match` result being NULL because of the lack of a telephone-event attribute in the SDP (FF does not offer it, for example). The corresponding check is missing then.


阿里巴巴-淘宝-资深Android开发工程师面试总结
面试时间：2015年6月10日
1、问题一：介绍你最近你做的项目、写的blog、看的书
我答：最近做了XX公司“后来1.0”；最近how-old比较火仿照此做了一个小项目；由于公司的需要自己做了一个“智能机器人”的小项目；
2、在简历上面看到有一个“通过移除Activity背景来提高Activity加载速度”
我答：对具体操作，突然就不记得了，哎，然后我说我们app都为每个Activity设置了大图，由于大图比较大，影响Activity的加载速度；描述了如何查找到影响Activity加载速度的因素？DDMS-heap-GC；回答问题没有回答到点子上，所以挂了！
3、然后面试官问到了Bitmap相关的东西
（1）Bitmap是加载到内存还是cpu问题搞糊涂了；
（2）Bitmap的加载通过BitmapFactory.Options处理，结果讲的不清楚就挂了；
（3）Bitmap加载使用到了LRUCache类，LRUCache的原理是什么？我讲到了底层是通过LinkedHashMap实现，结合线程池实现，软引用存储图片（其实是用WeakReference），结果就讲错了，哎！！！接着他就问了SoftReference与WeakReference的区别；这个区别讲清楚了，可是就是说错了！哎！后来面试官问到了，LRUCache实现的细节，结果没回答上，忘了，掌握不透啊！！！
4、然后面试官看到我简历上的ListView相关的知识
面试官问到了ListView相关的知识，结果讲的思路很不清楚！也挂了！！！
5、然后面试官问到了Activity生命周期和Fragment比较
Activity声明周期讲了哈！Fragment生命周期也讲了哈！然后对比了Activity跟Fragment；
6、然后问到了选择屏幕是Activity声明周期的变化
讲的不清楚，还有就是在设置了软键盘隐藏、onConfigurationChanged属性后，生命周期的问题，onConfigurationChanged属性详细知识也没有讲清楚；第三种情况，在生命周期变化时，有一个属性值，进行处理，没有回答上就挂了！
总结：这次面试很失败！！！浪费机会！！！问到了的问题之前都遇到过，可是就是实践太少，掌握不清楚！知识只有自己实践过了，才是自己的知识，切记！！！自己项目利用的太少了！！！


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 6](http://build.prxm.it:8811/viewLog.html?buildId=2984&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) is now running


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 6](http://build.prxm.it:8811/viewLog.html?buildId=2984&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) outcome was **SUCCESS**
Summary: Tests passed: 106 Build time: 00:01:10


@aembler see #2589


Does this also remove el_space between words?


I did the DB changes aswell before applying, which is just adding an index anyway, nothing that would impact the result running a correct query. It is possible that the data nedeed by Smarty to generate the page changed with this patch? I don't know well Smarty and PS codebase because I'm a new user, just wondering.


np: I'd prefer to write `@user && @user != ""` and t's a common idiom for this kind of situations. The only problem is that if `@user` is `false`, it could behave diferrent.

So if that's important to you, I'd write `!@user.nil? && @user != ""`. Anyway I guess it's a matter of taste.


Same as above.


不错 啊


.join() is a little more efficient but this certainly works


previous patch caused crash when toggle floating punctuation on a large epub file


This resolves https://github.com/chocolatey/choco/issues/194


This resolves https://github.com/chocolatey/choco/issues/290


This statement can be first.
If count of brackets not equal we can stop algorithm.


Thanks.
you can add information before National City. examples: He's from Texa- USA currently room.
add option city.


Shouldn't this take `empty_data` into account?

http://symfony.com/doc/current/reference/forms/types/choice.html#empty-data


Interesting approach, it does mean you have to iterate all the way through to the end.


J'approuve !


return after assignment


TeamCity Jarvis Framework :: CI [Build 72](http://build.prxm.it:8811/viewLog.html?buildId=2985&buildTypeId=CqrsFramework_Ci) is now running


@keanyc PHPCof 網站總招 XD 今年要用 React 寫嗎？


Irrelevantized via https://github.com/mcmonkey4eva/Denizen/commit/5cffab6d61a802acf5fda472f619ba35fcd3954c


I have thought about possible smooth animations in future, and since height cannot be animated with CSS - max-height will work just fine with CSS animations.


transfert effectuer vers Github


now that this is a class method i almost just want the function to be called fetch so it just reads like

     SlackInfoRetriever.fetch(handle)


TeamCity Jarvis Framework :: CI [Build 72](http://build.prxm.it:8811/viewLog.html?buildId=2985&buildTypeId=CqrsFramework_Ci) outcome was **SUCCESS**
Summary: Tests passed: 117 Build time: 00:01:07


@thomascube I guess there is no need for this special package definition any more. Net_SMTP is published on packagist.org => https://packagist.org/packages/pear/net_smtp


Hi J

First of all, awesome stuff getting onto this today.
Looks good, I have a few comments but can't remember my GitHub details right now on the bus...
I'll review hopefully tonight and add bits in, especially something about what scientists can offer you... I'll have to think hard!
I'll put some comments here anyway.

This will massively improve uptake of imos products.
If it improves scientists workflows, use of imos data would increase.
It will inform imos techs of the computer literacy of scientists (what they are up against).
Be good for building social / collaborative ties, bring imos techs into doing research ( authors of papers?)
We could aim to build a teaching resource - a single database that all teaching staff could use containing different kinds of data...
Converting scripts into generalise useful functions ( ie add to the imos toolbox )

That's I for now.
Great idea. I am very supportive and want to be involved.
Keep in mind though, I am at the higher level of computer literacy, almost a perfect candidate for such a program. Not sure how recruitment past me would work / who to target.

Have a great weekend.

Sent from my iPhone

> On 12 Jun 2015, at 5:24 pm, jkburges <notifications@github.com> wrote:
>
> @tremenyi I have just knocked this up, would you mind commenting before I pass it on to Rog.
>
> —
> Reply to this email directly or view it on GitHub.
>


If I overwrite a `getter` method and call save inside it like

```
  def some_attribute
     update_attribute :some_attribute, "value"
  end
```

`cache_changed_attributes` will raise a **instance variable @cached_changed_attributes not defined** exception.


fixed the broken first chinese character in a line. this line is changed by this patch
https://github.com/koreader/crengine/commit/d1f7f7b36ff769d8c1c5a6197c40d0f0327e3d90.
 I just changed it back and it works.


Wouldn't the lowercasing be less efficient though since all commands currently registered are already capitalized?


As long as there is space, it seems OK.
Are we trying to support any LCDs on 64k processors? Prom is very tight there.

From a scan of the other language files, this change appears to add up to 12 characters to each display line.
¿Cómo se dice en alemán ?


@medington I actually started with this, but I opted for correctness over consistency, since it actually only accepts 1 argument in this case, I removed the plural.


ajout d'une ligne


Well I don't think it is more correct than the previous discussed implementation; for example if someone orders tickets on the website but does not pay, attendeees stay in draft, but they are not expected at all ... I would have kept the confirmed + done computation instead.


+1 to this change


uipdate


update


What does this do?


学んでいる


Isn't it inherited from `models.Manager`? I think it is.


Merci @MoOx  ;)


Rubyは関数じゃなくてメソッドかな


The same fix is needed for opencl_oldoffice_fmt_plug.c.


I mean sharedStringsFileCache not sharedStringsReader ;-)


Not needed anymore with this: https://github.com/ezio84/frameworks_base/commit/07fe11ab3ae849c18264189c5319816f1722e122


@sshumake @mattnolting: (@ardasgup) Aritro pointed out that this div testing is not required and is erroneous. We may have to remove this.


> you can add information before National City. examples: He's from Texa- USA currently room.
add option city.

This information is not always available and when it is, not always reliable when not in the USA. Adding such information will lead to accurate results only for USA, which is not in my sight.

If you want to add it to your build, you can since the city_name field is already filled. Just print it.

FYI: You should create github issues instead of commit comments. Indeed, this commit has nothing to do with geoip features.


PrepTransitionFor84 (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1162 of 1162 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)


ありがとうございます。以後気をつけます！！


I see it under StoreMap.js on my side. Are you sure ?


Vaya, si no edito más texto no puedo hacer un nuevo commit :-1:  muerte!!!!!!


This works but I'm not sure we need to change `str` and build `sp`. One or the other would probably be sufficient.


@Morphan1 who cares about load time microeffic?


It's not undocumented: https://docs.djangoproject.com/en/1.8/ref/migration-operations/#django.db.migrations.operations.RunSQL

The issue was that my tests were failing on the Travis, because `sqlparse` was not installed. Somehow it got installed in my local environment, though.


@dnngll
please use https://github.com/fhorinek/SkyDrop/issues
instead of comments after the commit, so I wont forget about your suggestions.


Nice!


One more question @durran: does this change affect embedded documents (`embeds_many`), or are those automatically returned in order by mongo?


is PERFORMANCE your concern to move properties to ivars?


I know this is total and average meals but generally you'll want to name variables as clearly as possible.


So just looking for the first tooHungryDay? Nice logical expressions. I might just return when we hit this the first time but that's up to you.


I think we can remove this.


We can remove this too.


test github


	<Achievement id="51" kind="damage_done" MS_side="0" amount="100000">
		<UI name="@destroyer1" desc="@destroyer1_desc"/>
		<Filters>
			<Filter kind="enemy" param="1"/>
		</Filters>
		<BannerImage image="challenge_mark_03" type="mark" />
	</Achievement>
//////////////////////////////////////////////
I tried to change ID mark
4294967295, 51,
4294967295 , session.experience
but but nothing happens


Wiw


Please add license ;-)


What do you mean by 'productionize'?


yes. thank you.


should probably be ${ETC_DIR} in the target?


For the first point the fact is that I send an event in fetchRange of WrapObservableArray that needs to be emit from the delite/Store. That is why I had a reference to this.
You are right for the two other points, I will do the modifications.


Error handling. Could have checked if there are exact 2 args.


Sorry for incorrect user comments. v_levonyan's comments must be assigned to hamletgrigoryan.


What is the purpose of this instance? Presumably people call `cast` if they want to move something to another type. Does it solve a practical problem?


Please, remove `std::ostringstream os` as well.


What happened to handling the POST method call to _result_post_handler()? _result_post_handler handles the delivery notices from SMSSync. It uses the same ?task=result as the GET version.


Why not rather just copy the yaml file into ${ETC_DIR}/conf.d/ ?


Here too.


Don't forget var. It won't usually hurt you but it makes your code a little clearer.


isn't a call to Unsubscribe() now missing in each Destructor?


You decided to remove order test, since we added `static_assert`s in `sfinae.h`, right?


```typescript
        private notInLiveEdit(): boolean {
            if (wemjq(this.getHTMLElement()).parents(".inspection-panel").length > 0) {
                return false
            } else {
                return true;
            }
        }
````
can be a one-liner:
```typescript
        private notInLiveEdit(): boolean {
            return (wemjq(this.getHTMLElement()).parents(".inspection-panel").length > 0);
        }
````


Nice! We could even skip this assignment and return here directly.


This seems like an overly complicated solution. Either the imports should be rearranged such that the real `id` is in scope, or the instance should be written as:
```
instance Cast a a where
  cast x = x
```


If you add global_overrides: to the yaml, then you can just copy the file into ${ETC_DIR}/conf.d/ instead of appending it.


Ah, you did it :-)


-sigh- why do you insist on using "base.OnStart(start);" when the method really wants it to be called with "base.OnStart(state);" ?


Ah, I see you added a new one below :wink:


Mini-solar system. 4 planets, implemented Gouroud and Phong Lighting.


> I tried to change ID mark

This will only affect the popup info box, not the one seen ingame or in lobby.
The popup is sent by p2p where as the lobby/room infos are givent by the server.

Thus, it is completly useless to change info in that box besides being caught / reported by other jalous guys.


Your fork removes all the license comments from the minified file. If you use this in production, you may actually violates the software licenses. (e.g. [selectivizr](https://github.com/keithclark/selectivizr) is licensed under [the MIT license](http://choosealicense.com/licenses/mit/) that requires license and copyright notice.)


Would it be possible to rebase against master, or do whatever to ensure that the compare branches function doesn't show all the other cruft? Maybe a squashed commit to make it clearer what's changed?


It seems to actually need the fully obfuscated names. See my later commit.


IMHO, a FIXME added to this comment would have been appropriate. Otherwise, it is pure luck that someone reads this comment...


I do not think this patch does anything useful.  Pretending success without writing any data is IMO worse than letting the program honestly crash.  Please either handle the failure of EVP_DigestInit() as failure (and propagate it back to the caller) or just revert the patch.


Thank You


Wow


true :)

On Fri, Jun 12, 2015 at 11:11 AM, Alan Semenov <notifications@github.com>
wrote:

>         private notInLiveEdit(): boolean {
>             if (wemjq(this.getHTMLElement()).parents(".inspection-panel").length > 0) {
>                 return false
>             } else {
>                 return true;
>             }
>         }
>
> can be a one-liner:
>
>         private notInLiveEdit(): boolean {
>             return (wemjq(this.getHTMLElement()).parents(".inspection-panel").length > 0);
>         }
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/enonic/xp/commit/58f90d174009c1762a051bebb7b98a7398bec9ea#commitcomment-11649422>
> .
>


It looks like I forgot it. I will repair it


手误，应该是 "bytes=0-{}"，不过好像没有造成额外伤害，我就不改啦


Hmm, this is crying out for a loop. Try to write one that iterates through the days in mealsPerDay with a variable that tracks total meals and average meals per day. If the average dips below 4, return the number of that day. Let me know if you have questions.


how about using build-in Array.prototype.**forEach()** instead of a jquery function?


```.empty?``` ?


I guess this fix is needed for opencl_gpg_fmt_plug.c as well.


Align comments horizontally?


 Following CMDx around is a bit hard.
How about putting it all in a .bat/.sh file ? Also, since the .bat file doesn't use anything dynamically detected by cmake, can it be added to the source repo ?


```proxyuser << ":" << @pass if @pass && !@pass.empty?```


Boxen's tap doesn't have a remote, technically. Boxen just puts files in this directory and versions them itself.


It Breaks the build :P


Finally RFC6761 in action somewhere ;-)


OK, then I will fallback its remote to `"https://github.com/#{@user}/homebrew-#{@repo}"`. The change is already in #40608.


So I wrote the tests for this... and they passed already!

I think this is because AtomGroups are based on lists, (including their `__len__`) so Python was clever enough to figure out the rest.

This functionality is now covered by tests, so is "safe" to use in the future


Be careful after using erase.
Position of close/open bracket changed and you can go out of range or get wrong substring value.
Would be more preferable gets substring after that erase if needs.


I think so yes, but do note that there is now an additional dependency to install: inlinestyler==0.2.0


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 7](http://build.prxm.it:8811/viewLog.html?buildId=2987&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) is now running


Sorry, my bad I didn't run `make check` on Mac :(


You are right. This way it would be complete, I think.


No problem with learning Ruby, I just guess it might be less concise than the current situation. I will try my best, anyway :)


I agree!

But there's more to cleanup here:

1. libssh2_md5() doesn't seem to be used from anywhere! We can simply remove it.
2. these symbols are not exported and should therefore be prefixed _libssh2_


@craigds Your point about keyword arguments makes sense for languages that accept them (Python, Perl?). But that wouldn't work for Java (and probably CSharp as well). So the current approach is quite generic, but I agree this could be improved in the Python case.

@fazam A possibility would be to override the generated gdal.Info() with a hand-made Python gdal.Info() implementation in a %pythoncode %{ %} block

That should be something like (to be completed and fixed ...) :

```
%pythoncode %{

def Info(ds, format = gdal.INFO_FORMAT_TEXT, deserialize=True):
     options = gdal.InfoOptions()
     options.format = format
     ret = _gdal.Info(ds, options)
     if format = gdal.INFO_FORMAT_JSON and deserialize:
            import json
            ret = json.loads(ret)
     return ret

%}
```
There are override examples in swig/include/python/gdal_python.i and swig/include/python/ogr_python.i


No clearly the differences between a dstore/Store and an Array is too big to be able to make it function with the exact same way.
A solution to make this if disappear is to implement the function queryStoreAndInitItems in the WrapObservableArray.
With that we can replace computeProperties by :
```
computeProperties: function (props) {
			if ("store" in props || "query" in props) {
				if (!Array.isArray(this.store)) {
					this.wrapStore = this.store;
					this.queryStoreAndInitItems(this.processQueryResult);
				} else {
					this.wrapStore = new WrapObservableArray();
					this.wrapStore.store = this;
					this.wrapStore.queryStoreAndInitItems(this.processQueryResult);
				}
			}
		},
```
The factorization I mentionned was just the commun part of queryStoreAndInitItems with Array and dstore. But this part is not very big so maybe it is not important.


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 7](http://build.prxm.it:8811/viewLog.html?buildId=2987&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) outcome was **SUCCESS**
Summary: Tests passed: 106 Build time: 00:02:35


Yes, I was experimenting to see how much size difference the comments made.
In the end it was only 1.5Kb, in the scheme of things not much I guess.
What are your thoughts otherwise? Is it worth me sending through a pull request?


Should this be a colour variable?

I think it needs a bit more contrast. Its good against the white of the How it Works section, but against the off white (under the "recent requests" list on the homepage) its pretty hard to see its a button


@ryantenney This is really exciting. When this code will be landed in master?


It is to ensure that.  I'll update to make it clearer...


I think we need this twice. When you add another Graylog server to the cluster without Elasticsearch on the same node it still needs to be added to the list of ES nodes.


@rsutphin Have you checked this with Pouch?


Yeah - I noticed; and removed them


I have actually already fixed this in the event_broadcasting branch


`gen-docu.sh`?


ah, you've spotted a bit where we're out of sync with the new colour scheme. When the new colour scheme is merged I'd planned to replace this with the variable $colour_light_grey. It also sorts out the contrast, which will be much stronger then


##This is secondary Heading.
###This is Tertiary Heading


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 8](http://build.prxm.it:8811/viewLog.html?buildId=2988&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) is now running


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 8](http://build.prxm.it:8811/viewLog.html?buildId=2988&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) outcome was **FAILURE**
Summary: Compilation error: Jarvis.NEventStoreEx\Jarvis.NEventStoreEx.csproj (new); compilation error: Logging\Jarvis.Framework.MongoAppender\Jarvis.Framework.MongoAppender.csproj (new); compilation error: Jarvis.Framework.ElasticLogPoller\Jarvis.Framework.El... Build time: 00:00:20


so it is recommended to do it with and without docker???


TeamCity Jarvis Framework :: CI [Build 73](http://build.prxm.it:8811/viewLog.html?buildId=2989&buildTypeId=CqrsFramework_Ci) is now running


it is worth adding a ns to the event : destroy.apip-from for instance.


That fix is also required for opencl_pbkdf2_hmac_sha1_fmt_plug.c (moving come functions into a separate file to be reused by CPU and GPU formats seems like a good idea).


TeamCity Jarvis Framework :: CI [Build 73](http://build.prxm.it:8811/viewLog.html?buildId=2989&buildTypeId=CqrsFramework_Ci) outcome was **SUCCESS**
Summary: Tests passed: 117 Build time: 00:01:10


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 9](http://build.prxm.it:8811/viewLog.html?buildId=2990&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) is now running


Is oci8 ported to php7?
I`ve got error during make:
error: too many arguments to function call, expected 2, have 3
                zend_register_resource(value, column->stmtid, 0); /* XXX type correct? */


Effectivement, j'avais pas pensé au fait que le bateau n'agit que quand le capitaine lui demande .


you can also let setuptools install the script in your python environment


sure, i have left moving all such messages to the message properties file to be as well part of https://issues.openmrs.org/browse/CSM-89 Under Refactoring, this would help me save time and localize all messages that appear in the UI when dealing with that ticket


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 9](http://build.prxm.it:8811/viewLog.html?buildId=2990&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) outcome was **FAILURE**
Summary: Tests passed: 106; exit code 1 (new) Build time: 00:00:59


thanks


Please don't add a new `getInstance` signature for the sake of extra `@Nullable` parameters. If necessary, these parameters should be set via `withListener(LbListener)` method.


This is just a way of writing (that was common in C).
For both both silent and vibrating it will return false.

On Fri, Jun 12, 2015 at 4:40 AM, Stephen Black <notifications@github.com>
wrote:

> @tzachi-dar <https://github.com/tzachi-dar> Since you fall through on
> silent shouldnt this be false? Or should it return false on silent instead
> of falling through?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/StephenBlackWasAlreadyTaken/xDrip-Experimental/commit/07061a816e7e59268fb4c47d094814561d077677#commitcomment-11646309>
> .
>


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 10](http://build.prxm.it:8811/viewLog.html?buildId=2991&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) is now running


Means, return true if m_backgroundColor, m_borderColor and m_color found on dbc. Where is the problem?


@Jalle19 I can take care to open the respective PVR addon version bump Kodi-PR today. Okay?


> Is it worth me sending through a pull request?

Not needed, for the moment. This fork has both merits and demerits, so I'm considering whether this change should be merged to the original project or not. Thanks for trying to improve lt-ie-9 anyway.


Ascending still works. it's functionalty is being done through the line:

if (timeFromStartPlaying >= MAX_VIBRATING) {
...
}

On Fri, Jun 12, 2015 at 11:34 AM, Tzachi Dar <tzachi.dar@gmail.com> wrote:

> This is just a way of writing (that was common in C).
> For both both silent and vibrating it will return false.
>
> On Fri, Jun 12, 2015 at 4:40 AM, Stephen Black <notifications@github.com>
> wrote:
>
>> @tzachi-dar <https://github.com/tzachi-dar> Since you fall through on
>> silent shouldnt this be false? Or should it return false on silent instead
>> of falling through?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/StephenBlackWasAlreadyTaken/xDrip-Experimental/commit/07061a816e7e59268fb4c47d094814561d077677#commitcomment-11646309>
>> .
>>
>
>


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 10](http://build.prxm.it:8811/viewLog.html?buildId=2991&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) outcome was **SUCCESS**
Summary: Tests passed: 106 Build time: 00:01:13


TeamCity Jarvis Framework :: CI [Build 74](http://build.prxm.it:8811/viewLog.html?buildId=2992&buildTypeId=CqrsFramework_Ci) is now running


This commit also caused a deviation between CPU and OpenCL format implementations.
This needs to be taken care of in case the common code gets moved into separate source files.


This isn't needed here. There already exists functions in [linear](https://hackage.haskell.org/package/linear-1.18.1.1/docs/Linear-Projection.html#v:lookAt) and [diagrams](https://hackage.haskell.org/package/diagrams-lib-1.3.0.1/docs/Diagrams-ThreeD-Projection.html#v:lookingAt).

My plan was to add a projection function to diagrams `:: Diagram V3 -> Diagram V2` so hopefully we won't have to worry about projections here. But I guess that's something to think about later.


Removed again after https://github.com/enaqx/awesome-react/commit/529ffa33872665d4e286bcf864b1d199e204bb59


The output is amazing. Thank you, @lunaryorn, for the CSS.


TeamCity Jarvis Framework :: CI [Build 74](http://build.prxm.it:8811/viewLog.html?buildId=2992&buildTypeId=CqrsFramework_Ci) outcome was **SUCCESS**
Summary: Tests passed: 117 Build time: 00:01:10


Hvorfor er det bedre på en linje egentlig? Så mer lesbart ut slik det var?


udah dihapus sama saya yaa


Did you test that this works properly ? I think that the correct typemap for the char** must be explicitely specified. You'd need to refer to the char** options typemap. See for example the binding of gdal.OpenEx() in swig/include/gdal.i

And you shouldn't directly affect the variable as   infoOptions->papszExtraMDDomains = papszExtraMDDomains;  but rather CSLDestroy(infoOptions->papszExtraMDDomains) and then infoOptions->papszExtraMDDomains = CSLDuplicate(papszExtraMDDomains);  It would actually be cleaner to have a C function GDALInfoOptionsSetExtraMDDDomains(  GDALInfoOptions *psOptions, char** papszDomains ) to do that and use it in the bindings

This setter is currently untested by test_gdalinfo_lib. That would be worth being tested


Suggestion of flipping `Comment` and `Discussion`... but that portion will need to be retested and is how I do it manually to attempt to avoid DB corruption... e.g. remove comments first then twiddle with the discussion if owned.


Overhode ikke bedre, men wercker / yaml som tuller seg :(


For me, it doesn't break the build, but causes too many warnings. Plus lag when flashed, so removed it.


Well this should go into /usr/bin when packaged, right?


OK


Fixed


Kim, just do it !!


@wlbragg It was mixed with code moving the contact points I had removed and I was probably in a hurry. Is the current merge request code ok for windshield splashes? They are enabled if any part of the floats are in water and computed using your formula.


Please test the new prices. Logically they should be symmetrical now, at least. But I fear I need to separate LARGE_PROFIT and SMALL_PROFIT and double prices into buying and selling (since Merchant should always make a small profit).


Hello @ifreecarve, thanks for submitting your pull request!
Looking at your change, seems that you will always have clang and its binaries (plus the compiled plugins) in /tmp, but that folder gets cleaned every time the system starts, thus breaking the execution of Infer.
Furthermore, the rest of the code that points to clang and the plugins (which are important for the execution of the analysis), currently assumes that they are in the parent folder of the Infer repo (as you pointed in your comment); this means that all these relative-paths should be built using the content of your `.facebook-clang-plugin-dir` file.


I think we need a separate if/else branch for type "infiniband", ie instead of catching all device types not matched before in this final else: branch for type "802-3-ethernet" we need to actually check the type of the device and for infininband eg set type to 'infiniband' (I think this is enough, check https://developer.gnome.org/NetworkManager/1.0/ref-settings.html.

Also for the infiniband branch I think we could even leave out the ifname= check and mac-address setting because I guess dracut doesn't support IB devices renaming or IB devices at all? (we store mac address in ifcfg only if ifname= dracut boot option is used). Or at least if we keep the ifname check, we need to make sure it will not traceback.


@pbauer Please add a changelog entry. This is an important change for users.


Thanks


No worries, your comment didn't come across that way. It's a valid criticism & I agree.


It Breakes the build for me :-/ weird


We should probably again handle the inifiniband device type here (type setting below), but I am not sure how IB slaves are supported in general so I'd probably leave this out completely as out of scope of the BZ.


dev-master? Waarom niet zonder versie. Dan wordt de laatste stable (dus tag) binnen gehaald.


Any chance in open sourcing the new snowflake?


Are you by mistake mixing up session state database and the term distributed cache here (and also in the sql server "cache" implementation)?
Shouldn't this be totally separated from each other, I don't really see SQL Server as a distributed cache at all ;)


Magic numbers are hard to read.


idem


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 11](http://build.prxm.it:8811/viewLog.html?buildId=2993&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) is now running


This improves the loading time of 100 empty modules from 75ms to 25ms.
On the Cuteness app I see no real difference.


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 11](http://build.prxm.it:8811/viewLog.html?buildId=2993&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) outcome was **SUCCESS**
Summary: Tests passed: 106 Build time: 00:01:12


ik vind dat eigenlijk geen vebetering. Die "yoda comparisons" hebben wel hun voordelen (zijnde, je kan geen assignment doen), maar ik vind dat het je code minder leesbaar maakt.

Standaard code `if ($term != '')` lees je als "als $term geen lege string is"
Yoda comparison `if ('' != $term)` lees je als "als geen lege string gelijk is aan $term".

Ik vind dat een goed genoeg reden om geen yoda comparisons te gebruiken.


Resloved IRL. The reason is line 100 were `raw_tty_putc` is  passed to `tputs`.


ik zou die q hernoemen naar 'term' ofzo. Dit zou veel meer zeggen over het veld, zonder een over dreven lange get parameter te worden.


Could you please fix this on maint as well?


al die class stuff is eigenlijk iets dat in de template moet gebeuren, zeker zoiets met logica achter (ieder child is een sub-menu-item). Kan dit niet met css of door de template te overschirjven opgelost worden?


@sizzlemctwizzle
Also not entirely sure if it is accounted for this but if two spam accounts come along and work in tandem in the same discussion and the discussion is removed it will create an orphan of the 2nd non-owned account comment.


idem (zie examplebundle)


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 12](http://build.prxm.it:8811/viewLog.html?buildId=2994&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) is now running


Jp2 image file upload and preview using imagemagick in liferay


`'mapped' => true` lijkt me de default, dus die optie moet er volgens mij niet bij. Die `'data' => array()` mss ook, maar dat zou eens getest moeten worden.


TeamCity Jarvis Framework :: Publish Framework to MyGet [Build 12](http://build.prxm.it:8811/viewLog.html?buildId=2994&buildTypeId=CqrsFramework_PublishFrameworkToMyGet) outcome was **SUCCESS**
Summary: Tests passed: 106 Build time: 00:01:11


fixed in commit e9536eded


Maybe, but I'm not sure how. Could you give an example?

The complexity is that if we read ESC and *then* '[' or '0' and *then* 'A', 'B', 'C' or 'D'. If we at some point do not read any of these we bail and return `ch`.


We currently don't have the endianness conversion macros in OP-TEE or TomCrypt. Another issue is that `val` may not be 64-bit aligned.

Actually, I tried two variants of this code:
1. Incrementing 64-bit quantities (hi/lo), see https://github.com/jforissier/optee_os/commits/aes-ce-a64-ctrhilo
2. Using NEON assembly similar to what is in `ce_aes_ctr_encrypt()`, see https://github.com/jforissier/optee_os/commits/aes-ce-a64-ctrneon

In both cases, the code is longer and arguably more difficult to understand than the naive per-byte increment. Also, I did not measure better performance. Not sure how reliable this is, but the Neon and the byte versions need 0 or 1 interval of the cntpct_el0 counter to perform the increment, while the 64-bit version takes 1 to 3 intervals (counter frequency is 1.2MHz AFAIK).


You could check the [forks](https://github.com/jparkie/Aizoban/network) for an actively developed one. My one looks to the most recently updated but I think @chogos might be actively developing his version (though you may want to ask him yourself) whilst I've put it on hold as I'm still a beginner and Aizoban's architecture still makes little sense to me.


Oh, yeah! Good catch.


ul li list item click selected  Example ??


これは分かりました。有難う御座います。


nice catch.

There is no such issue #111


Just put "2015 June" or something. alpha is our old version system which was abandoned quite a while ago.

I assume those TODO will be fixed?


Going to take a crack on the mono speaker stuff on the Galaxy tab S 8.4

Did you ever have any luck @sigma-1 ?


thanks . good job.!!


item != null should be checked at the very start. Given null, you'll pointlessly traverse all recipes.


@manuel-uberti You're welcome.


I think we like to end our files with a blank line.


Is selectedApp used ? I only see another affectation in this method.


masuk gk ?


@taiyun :flushed:


We could only use the load balancing option because websockets doesn't work with a single instance. What do you mean with instance typ?