Could you explain what this fixes, and why? I saw earlier that it may have caused a crash, but it's unclear to me why changing that property value would do so (since, you know, it's a mutable request). The Apple Docs don't seem to mention anything about that, and a quick Google didn't seem to turn up any results either.


I really love the commit message :)


I am trying to install dosbox-x and I get this error when I run make: http://paste.debian.net/142779/. I am trying to install it on a Debian 7 computer.


JRuby is 2.2 compatible
```
irb(main):001:0> RUBY_VERSION
=> "2.2.0"
irb(main):002:0> RUBY_PLATFORM
=> "java"
irb(main):003:0> RUBY_ENGINE
=> "jruby"
```


I didn't forget, it's part of the scope of my `// TODO: add more test cases` comment at the beggining of the file ;-)
Now, while RETURN does close the dropdown, SPACE shouldn't (and doesn't) , at least in multiple selection mode where it serves a different purpose: toggling the selection of the navigated item.


Yay! could we push a point release to hackage? I could then stop add-source-ing it into my sandbox then. prettyplz?


Interesting... I see what you mean about it doesn't work because List#handleSelection() references `event.target`.  Yet somehow List (or KeyNav) correctly handles other keystrokes like DOWN_ARROW, even though that keystroke also depends on the current active descendant.   So perhaps List#handleSelection() can work the same way as however DOWN_ARROW is handled?


Why change from 12 to 16, out of interest? CPython on 64-bit uses 12 here.


Brilliant. Are there any other uses for binary coded decimal other than coins and money? Any reason for there to be a single digit version?


This line of code is also equivalent to `books.map {|book| book.print}.join(' ')`. And if you were using Rails you could use the symbol-to-proc trick and simplify it to `books.map(&:print).join(' ')`. Easier to read code is usually also easier to maintain.


This actually makes it worse because now everything is farther to the right instead of just the `<<` operators. Suggested:
```
WriteLog (lsERROR, LedgerMaster) <<
    "MISMATCH " << msg <<
    " without " << tx <<
    " metadata is " << metaData->getJson(0);
```


Testing closing: same answer as above.
end() calls: well, my understanding was that it's not a bad thing to clear the intern's stack. But if there's clear evidence that it would be better without these end() calls, I'll remove them. Do you have a strong opinion on it?


nitpick: add EOF newline? :)


Preconditions ftw


64-bit, it looks like? C++ std::min() expects both parameters to have the same type.


intentional.


Make sure that from here on out, the 1.0.0 gradle plugin (ie the one specified in the new build.gradle file) is used. IntelliJ IDEA should automatically pick this up, but just in case.


We need to say enough that the reference to a repository URI here makes sense to the reader. This section depends on the repository URI docs, so you should either add that content in this PR or add the part about `Datasets.list` when the repository URI content is added. I prefer the first option.


Still weak commit message :)
It's really far from beta version. Something like "Naive file list enumeration in custom menu was implemented" is better.


I also thought it was good (in a philosophical sense) to clear the stack, but I just don't think there's anything on the stack in this case.


Perhaps this should be a property rather than being hard-coded.


@mbabker in #5871 would be nice to use per template override together with the layout of form field media (The case is that the modal is rendering an iframe com_media/views/images that has some predefined javascript which is hard to be changed e.g. modal close on cancel and save)


It would be helpful to have some way to programmatically remove individuals via a config file. Occasionally employees are let-go and it would be best to keep something quiet.  Coaches, troubled employees, etc. If we could do the same thing and keep their uids out of the changelogs, that would be great.


File still presented under version control. Please remove it, it doesn't make a value for code base.


Yes, but still, there's a bit of hidden magic in this call. Or maybe everybody but me is perfectly aware that you can pass arguments like that via bind. Anyway, I can change it if you want.


Try to avoid comments like this, or remove them on commit.


http://coding-robin.de/2013/07/22/train-your-own-opencv-haar-classifier.html

2015-01-28 15:52 GMT-03:00 nisakhairil <notifications@github.com>:

> Okay thanks,about the banana classifier.xml file? How can i start to add
> another type of fruit? For example i want to add apple and so on
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Asaffe/fruits_detection/commit/34c3d76bf2badfee468eae80d160e10bbdc6569b#commitcomment-9484579>
> .
>


Try to avoid such messages. Use tracker or email to ask questions.


Looking good, thanks :)


This is awesome


Overall looks good.  Added a couple suggestions.


I don't really care; I agree it's a seldom used feature of `bind()` and I'm not sure everyone knows about it.


Aww thx :smile:


there should be an error with this commit because the demo website is broken right now :(


Although the Ruby interpreter doesn't mind unusual indentation, it makes it much easier for other people to read — including your future self!  Try to double check that your indentation is correct.

One way to do this is to run `git diff` before you add anything to a commit.  This will print out a "diff" or "changeset" much like what you see right here on GitHub.  You can use the arrow keys to scroll up and down through the diff if it's longer than a page and you can press the `q` key to return the the command line prompt.


Please, try to not ask question in code :) For example in this case you can just use commit comment on github, like this one. Also you can use email or hangouts, where you put link for a file like this https://github.com/drewpts/LPROJ-main/blob/trying_gmock/src/libprojplugin.h#L6.


The better approach, in my view, is to directly use mixK, mixH1, fmix here:

```java
// init:
int hashCode = Murmur3.SEED

// in the loop
hash = Murmur3.mixH1(hash, Murmur3.mixK1(hashCode))

// finally:
return Murmur3.fmix(hash, elements.size())
```


86714894-BoostSuperBuild (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2172 of 2190 tests passed)

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


Nice :+1:


Okay - I misunderstood your purpose here. It's kind of the opposite of what we typically do in tech writing. We tend to use longer variable names to add clarity and avoid collisions. I shortened all of the links, and will use similar abbreviations going forward.


FWIW, in my hacked up fork of this ugly component, I ended up wrapping this specific line in a `$app->isSite()` check and my git history doesn't tell me why anymore.  I imagine part of the reason is what you're trying to lead onto here.


it's weird that the indentation of longest_string.rb is off here. It was alright in my sublime.


Maybe. I also thought about it, but I tried to minimize the changes in List for our tomorrow deadline. I think we can keep that for examination in a later step - I can create an issue for it. On the other side, the focusDescendants=false mode has been introduced for Combobox' needs. Whether it makes sense to use a standalone List in this mode, and whether this is a use-case which deserves full attention in List, I'm not 100% sure.


Nice!  Great use of [Enumerable#min_by](http://ruby-doc.org/core-2.2.0/Enumerable.html#method-i-min_by).  In Ruby, anything that's `Enumerable` supports these sorts of methods.  That includes both `Arrays` and `Hashes`.


Cleanup this please :fearful:


I'll double-check this point.


Changed the link to point at the Partition Strategy JSON Format page. Changed the text to "You define your partition strategy in partition strategy JSON format." Possibly redundant, but clear, which is the goal.


ça ira pour l'instant.


Right, I don't need the second print_space(), I was only thinking of symmetry.


That should work also pointing to the Gjust ithub repo. So run `bundle update spree`.


Great job, really thorough


It seems like you're comfortable with blocks, so I'll mention this now.  This is 100% a stylistic thing and is important if you want to write idiomatic Ruby.  However, it's not critical in terms of your learning, so don't take this feedback as me saying, "This code is wrong — please write it a different way."

In Ruby, there are two syntaxes for blocks.  One is the `do ... end` syntax that you used here, i.e.,

```ruby
list.min_by do |str|
  str.length
end
```

The other denotes the start of the block with `{` instead of `do` and the end of the block with `}` instead of `end`.  Idiomatically, you'll see this syntax for short, single-line blocks, often written on a single line, like so:

```ruby
list.min_by { |str| str.length }
```

So the stylistic rule of thumb is to use `do ... end` when your block is long enough to span multiple lines and to use `{ ... }` when your block is short enough to fit on a single line.


One more thing here.   I noticed in https://github.com/ibm-js/deliteful/issues/332#issuecomment-71850743 that you wrote a comment that:

> List/KeyNav selects the items already upon arrow down/up navigation

If that's really true, then you shouldn't need this call to setSelected().


Great hint, I will try to test it in #5871, thanks!


change this to logo2
it makes sense to change the filename to make sure we don't hit any caches but avoid "old" and "new" because that doesn't scale past one iteration


Did you mean `{error, any()} | any()`? Reading `any() | any()` seems awfully redundant to me.


Yes, we can imagine various changes / redisign for List, independently on Combobox. Again, this wasn't my objective for now, but it can be considered later, although, as already said, I'm not sure List's behavior with focusDescendants at false is a real use-case outside the Combobox context.


@Razzeee
not sure if you have picked up on this too.. but the watched time is also incorrect.

tested removing a full show collection/watched info from trakt.

re-sync and then the show is marked as watched (sync timestamp) trakt but not added back to collection.


69208874-LBNL_CoilVsSpacePeak (Myoldmopar) - x86_64-MacOS-10.9-clang: Tests Failed

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


This fix isn't going to work for the related Sandbox ticket due to complexities with that entity design.   The following would need to be addressed to accept this pull request.

- [ ] Use Logger for exception
- [ ] Add comment indicating that any usage of this validator for an entity with a large number of rows would likely require the implementor to add a custom index for the field being validated.
- [ ] The Validation message should be externalized


Okay, in this case I'd keep the code as-is.


For the load movie function, it would be better to have the dataset be an input parameter. I don't know if you have had a chance to look at the m file I sent earlier, but I am developing a file structure that enables all iterations of pre-processing steps to be stored in a single hdf5 file. Right now, the basic movie (before any processing other than concatenation) is in '/Data/Images'.

For the save movie function, the dataset name should also be an input parameter. Also, to be consistent with other scripts, we should have separate inputs for outputDir and outputName, right?.


Looks good


yes, if difference between your Z and land Z more then 8 yards, must be blink forward ( falling in abyss but on height ). And on this reason, if we will jump from bridge and orientation will at bridge - we will appear on it. But notice : need to take into consideration a time of cast, if you will a little lower, when Z coord of texture of angle of bridge - will collide with bridge ( or difference between your real Z and land will lesser, then 8 yards -> will appear on a ground )
p.s. Didn't checked Cyberium version of blink after new patch, but first his commit was bugged with this part of safeblink ( blink in falling on a ground ).


Am I reading this right that this is for the upgrade?


@spencems
you need to run a cleanup kodi does some funny things if don't do one


@pchi17 It's possible you don't have your Sublime set up to indent using two space characters and are instead indenting it using a single tab character (`\t`) with Sublime configured to render tabs with two spaces.  In Ruby, code is always indented using two literal space characters.

Visually, it's impossible to distinguish between a single tab character with a tab width of 2 and 2 literal space characters.  The different will only be obvious when you try to render the code in an environment where the tab width is set to something other than 2, e.g., GitHub's web interface which I believe uses a tab width of 4.

Here's my guess as to your current Sublime settings: http://cl.ly/image/3v2e0H3w270O

Here's what the "tab width" settings should look like: http://cl.ly/image/3P1H023z1y2j.  Notice that _Indent With Spaces_ is checked.


perty


@Club15CC The last major project we worked on that utilized `giternal` didn't end up pushing us to implement the freeze functionality I mentioned above.  I've been considering tackling it as a side-project, but if that happens it'll be in my upstream of this repo.  I just created jwhitley/giternal#2 to track this issue.  Please follow that issue if you're interested, and carry on any further discussion there.  Thanks!


> First the png, it looks a lot nicer than the gif, but it might be less compatible.

I haven't used a GIF image for anything in years. I would be surprised if any browser could not handle a PNG image. However, this will not work in IE8 or 9 because of the CSS animation (but neither would the `<progress>` element.

Personal opinion: I haven't seen this live yet, but I feel it is quite a bit of overhead for purely visual reasons when there is a built-in alternative in `<progress>`. I would at least want to discuss this in a devTalk before it is added to `master`.


Grz3s  it working? When i have tested, on my repo this way haven't worked.
( m_caster->m_movementInfo )
And on this reason i did 2 ways, special for players and special for units.


What was wrong with upsert() ?


What's this for? I've never seen the pattern. The sup should be a singleton so I'm not sure why you'd check for the existence of the table and instead just rely on a bad match if we ever did get a duplicate.


Would `composer install --no-dev` also not register the `autoload-dev` autoload sections in the dumped autoloader? I could just test this out but it's not clear from the docs.


My old https://github.com/ibm-js/deliteful/issues/332#issuecomment-71850743 is inaccurate, it does not select, only highlight (to select, you need the SPACE key).


Cyberium, you have tested it?


Oops, I accidentally used the wrong issue number in my prior comment.  That should have been jwhitley/giternal#9.


Should it be better to use partials? In the case of asyncio they prefer them: https://docs.python.org/3.4/library/asyncio-eventloop.html#asyncio-pass-keywords

I personally find them clearer: functools.partial(self._allocateUDPPortCallback, port_id, *args, **kwargs), other than that I don't know what other advantages they may bring?


Per discussion, since this is not as central a concept as it once was, I'll cover this in the URIs.md document. I need to open a JIRA, because I'm still not clear on the context for this. Pulling the comment about Datasets.list. I'll add it to the JIRA to be sure we cover it.

https://issues.cloudera.org/browse/CDK-894


Feel free to ignore me if this is out of scope for your windows branch, but it might be worth moving this logic to something like `os_pipename()`  (in keeping with libuv's generic name for domain sockets and pipes, [uv_pipe_t](http://docs.libuv.org/en/latest/pipe.html)).


I agree on the dataset being an input parameter for both load and save functions. I'll make the change.

In this case, when we're opening only one file, I'd prefer not having to segregate the output directory and the name. (I think the only case where we require a directory was to the folder containing all the XML/TIF recordings.) I find it cumbersome to have to provide multiple parameters to a function, unless necessary.


Really, why you can't ?:) There can be safely used forward declaration.


TeamCity xBIM :: XbimExchange :: Build [Build 63](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=806&buildTypeId=XBIM_XbimExchange_Build) is now running


I've released a new version that has the methods enabled


We definitely want to avoid collisions and be clear, but also as short as possible. For example, csv-tutorial is much shorter than using-the-kite-cli-to-create-a-dataset and has about as much chance of an accidental collision.

We also don't have to worry as much about collisions because each file is a separate scope. The config (accessed through `site`) is used for whole-project scope. So there's much less risk than a large XML-based project.


hello guys, this seems to be a bit problem for rails as the response is ActionDispatch::Response and does not implement an "empty?" method. Am I missing something?


TeamCity xBIM :: XbimExchange :: Build [Build 63](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=806&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:01:08


If BBT throws an error for non-generated keys, that's an error. Internally, it generates unique keys for every reference; the keys in the "extra" field override these if you want, but they should not be required for use.


Btw, I don't know Hystrix internals, but is it safe to just globally reset "everything" in HystrixPlugins? Users could register their own publishers in other Spring beans.


I can get rid of this.  Moment pls.


all tests pass:
success: http://build.servo.org/builders/linux1/builds/618
success: http://build.servo.org/builders/linux2/builds/617
success: http://build.servo.org/builders/android/builds/653
success: http://build.servo.org/builders/mac1/builds/834
success: http://build.servo.org/builders/mac2/builds/834


But it you look at the List code it's already using `this.navigatedDescendant` all over the place.  AFAIK the only time it uses `event.target` is in `handleSelection()`.   So this is more of a bug than a "redesign".


TESTS PASSED
 Mr.Roboto url : http://jenkins.plone.org/roboto/get_info?push=b29a5a39605940a6b89eaead5a68f809
 plone-5.0-python-2.7 [SUCCESS]


Fixes #41


The only other thing I would recommend for the write function (depending on how you see the function used) would be to generalize the size of the dataset. With the way it is now, you would not be able to append something to the dataset later on. I would instead use:

h5create(filename,dataset,[Inf,Inf,Inf],'ChunkSize',chunkSize,'DatasetType',...);

With this change you would also need to change the write line:

h5write(filename,dataset,movie,[1,1,1],[height,width,num_frames]);

For the default dataset name, that works for now! If we decide to have a different file structure, then we could change it to what the base directory is.

----- Original Message -----
From: "Tony" <notifications@github.com>
To: "inanhkn/analysis" <analysis@noreply.github.com>
Cc: "jmaxey" <jmaxey@stanford.edu>
Sent: Wednesday, January 28, 2015 3:37:19 PM
Subject: Re: [analysis] Generic save and load of movies from HDF5 (01711a6)

I agree on the dataset being an input parameter for both load and save functions. I'll make the change.

In this case, when we're opening only one file, I'd prefer not having to segregate the output directory and the name. (I think the only case where we require a directory was to the folder containing all the XML/TIF recordings.) I find it cumbersome to have to provide multiple parameters to a function, unless necessary.

---
Reply to this email directly or view it on GitHub:
https://github.com/inanhkn/analysis/commit/01711a6fe2d0464bb1258c612439ad2eb42b1db3#commitcomment-9489416


Yup, agreed. I'm just used to large XML-based projects, so your request didn't register properly. I'm happy to use short link names, even shorter than what you suggest, because they won't be that hard to locate, regardless, in the Markdown docs.


Just for context on the revert:

After deploying with this update we started seeing occasional subjobs just hanging when trying to post results back to the master. Looking at the stack trace dump, we could see that the slave's thread to post results to the master was actually hanging on the `requests.post()` call. There was no sign of this call being handled on the master.


Before we commit this, it might be nice to enshrine the guidance we landed on in a public place on the Wiki.  That way everyone understands what we are going after and why we are making these changes.


@Razzeee
tried the same as above, but cleaned kodi library first. still getting same issue.

some series/shows are getting watched update. - current sync timestamp.
The rest are getting collection update. - current sync timestamp and some future.


One concern on this - It appears that if you change names on the templates you lose a lot of your history on your host items as a result.  That's a reason why I've been very careful to change things in the past.  With the change to rabbitmq.queues, it's likely worth changing to V3 and taking the historical loss right now (I hit the same issues when I changed some data to float instead of int).  I'll probably look into creating a SQL script to migrate data to a V3  version of the template.


are yall tackling these in any specific order?


Iam a bit lost because the case on the video you posted perfectly work with that version of blink.


Some partner was complaining they have crash on the [request setURL:[NSURL URLWithString:url]];
I wasn't sure either because syntactically nothing is wrong. After some googling, found this.
https://groups.google.com/forum/#!topic/apportable-discuss/zkfPHq6c1Qs
And the partner said this solves the crash issue.


[SideCI/RailsBestPractices] remove trailing whitespace
reference: http://rails-bestpractices.com/posts/60-remove-trailing-whitespace


you need to replace 'mimetype ' to 'content_type' in Django 1.7 version


Нашёл, что это был лишний дубликат кода, расположенного выше.
Чёрт. Такие вещи документировать надо - отдельным сообщением или коммитом. Это же важные детали.


i'm wondering if there is something odd with your ruby setup/version because i cannot replicate.


fast-forwarding master to auto = 74e32d11


r=mbrubeck


Haha, I didn't realize making bullshit accusations was constructive.  I'd rather leave this branch here so I can keep a master between vanir and exodus.


@kvipka
```((Player*)m_caster)->m_movementInfo.HasMovementFlag(MOVEFLAG_FALLING)
and
m_caster->m_movementInfo.HasMovementFlag(MOVEFLAG_FALLING)
```
Is same in that case, no need for typecast.


@stevejameskent you should always do an identity check (===) rather than equality check (==) when comparing two values since the later does type coercion rather than strictly comparing the two values.


To keep it simple, my original idea was that if paging details are needed you simply create them via

        ApiUrl apiUrl = new ApiUrl(TmdbTV.TMDB_METHOD_TV, TmdbTV.TMDB_METHOD_ONTHEAIR);
        TvResults tvResults = tmdbTv.mapJsonResult(apiUrl, TvResults.class);

        // access paging results
        tvResults.getTotalPages()

However, I see your point and I've changed the api-wrapper to always return the resultspage including the paging information instead of just the results-list. This breaks existing usage (sorry!) but is way more flexible.

For increased convenience ResultsPage also implements Iterable now.


saw approval from mbrubeck
at https://github.com/pcwalton/servo/commit/9795ce0f7127b2d0a0c3b63ba57442cf0150eb8f


merging pcwalton/servo/text-align-justify = 9795ce0f into auto


pcwalton/servo/text-align-justify = 9795ce0f merged ok, testing candidate = cac54908


So does this happen automatically now? What's the migration path if you were using this flag? It isn't clear from the changelog.


Check arcane missile by player in move. Will interrupt or not.


I think you meant .Nm here


You kanged an idea with no credit.  Maybe you don't understand the word
"kang".  I suggest you look it up in a Google search.  I started binary
modifications and optimizations back in July of 2014.  So you expect me to
believe you came up with these ideas all by yourself?

https://github.com/SaberMod/android_build-OLD/commit/93d376380dffae1ec96eeb0ef00a7e663d107ca4


There's the [selfContained](http://6to5.org/docs/usage/transformers/#self-contained) optional transformer that supersedes the `includeRegenerator` and `runtime` options.


This is a major change and breaks code. Please do a patch release with this reverted and a new major version bump with this change.


No, sure, have a look at it first. Just as a heads up: That indicator is in addition to the `progress` element, because just having one progress bar slowly filling up felt like the script died. So, not a replacement.


Thanks, fixed via https://github.com/angular/material/commit/daa01312e74b6aa2d3652d5179b0fdf5cbf3aca4.

We really need automated testing in all browsers.


Thanks! The source code of both Vim-R-plugin and vimcom might be hard to understand because they have very few comments. Later, I may collect some of my comments in reported issues and put them in the vimcom's README.


Taste the rainbow?  :)


I think you'll always want reactivity otherwise the wrong selected value can get stuck showing. But maybe AutoForm can provide a more specific reactive variable that doesn't trigger reruns unless the options have changed.


2 x + y ではなく 2*x+y


Privileged != (uid=0). Privileged means you run with all kernel capabilities turned on.
Privileged + (uid=0) == Omnipotent


ここもおなじく `*` が漏れています.


I can't understand if this is needed or not. It seems it doesn't affect anything. Any idea @devisnik?


@mostaphaRoudsari ,
I have finished making sure that the results from these two systems can be brought into GH with the same result reader as the Write IDF component.
I wanted to also make sure that much of what I implemented on the Write IDF component gets implemented on the OpenStudio component.  I just found the OpenStudio SDK (https://s3.amazonaws.com/openstudio-sdk-documentation/index.html) and I will try to implement this soon.
-Chris


The problem with this approach is that you may end up with A LOT of overlap as you try to fill in these tags for every page that needs them. Further, the data isn't separated from the markup so if you want to change something individually or do something with the data elsewhere then there's no way to do it other than render the template, parse the tags, and grab what you want.

One potential alternative off the top of my head is you could create a method in the application controller that sets an instance variable with a hash of the meta tags. This hash could initially be set as a constant in the controller and the method setting the instance variable could be run in the before_filter. The application layout can then render a partial which in turn renders the markup for the meta tags hash (i.e. metatags.each do |tag| ....) Inherited controllers could then override this method, call super, and modify the relevant values for the specific controller. Individual actions in controllers could then further modify this instance variable because it was set as part of the before_action so you can have granular action-level control of the meta tags. This approach gives you global, controller-specific, and action-specific control of the meta tags while also allowing for inheritance of tags as you go from general to specific.

tl;dr you'll repeat yourself a lot with this approach


Could you add this for checkbox_nested too please? And maybe the radio/radio_nested ones? :3


> Does this "stay loaded"

Yes, and there is no speed penalty.  This is not normally done in node because it is nice to document the dependencies at the top.  But in Atom it helps load time.  There is a post in the forum somewhere that tells what to do to be a good citizen and speed up load.  This is one of them.  Although it usually makes more sense to do `@exec = require("child_process").exec` in the activate function and then use `@exec`.


@ehebert going back and forth on this -- should a liquidated order be removed from `current_data` (because that stock is no longer available -- if left in `current_data`, users can try to place orders for it that will never fill [this is the current behavior, effectively]) or should `update_universe()` be called, placing the Liquidation event into the universe but not processing it as a `trade`? Not sure there's a good answer -- and it could be neither...


Sorry, I don't know.  It looks like it was exported as of efd9745d1e3ecea5ef5a9ad0d5176b7aacc0cf15 "build docs from source", so if it has a function, it is for the docs... but I would think all the docs tasks take their input as parameters.


webstorm下使用github测试


#14 highlight tag can not be used in our blog.


Why not just adjust the `GOPATH` directly instead of installing the `godep` tool, especially since it's only used for this specific bit?  I can make a PR for this if you'd like.


Thank you very much for this fix! Folder creation for Google Drive was broken (creating 'Untitled' files) from Friday until this commit came through. I was actually planning on submitting a patch as soon as I returned home this evening.


unfortunately, this cannot be done automatically from command line: the MSVC solution is for VS2010, so if your machine has newer version, you have to manually convert it so your build tools can understand it.

the alternative method is using CMake (require CMake installed on your machine first) to build the core library (`capstone.dll`)

    mkdir build
    cd build
    cmake -G "NMake Makefiles" ..
    nmake


This blog raises an important question that will permeate through the class: who are the _consumers_ of open government data? Is it the public writ large? Is it trusted third parties like the press? Is it institutions such as universities, banks, etc.? My own sense is that it is, and will be, a combination.

As for the practical effect, I think it's still TBD. Lawyers are notoriously behind the curve when it comes to technology implementation. The question for lay attorneys is whether their lay clients are willing to accept PACER fees, costs for primary legal materials, technology bloat, etc. If not, are lawyers are willing to absorb those costs themselves? If the answer to those questions are _no_, dollars to donuts, lawyers will increasingly become interested...


but `[:100]` will not append the ellipsis sign "...". Maybe  `truncate(100, True)` would fix the issue: http://jinja.pocoo.org/docs/dev/templates/#truncate


why a `;` here?


OOPS!  :-)


@killpack :+1: ?


your test is crashing the framework. don't know why. stopped it.


1277852-CommandLineInterface (nealkruis) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (599 of 599 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-2.49%25-green.svg)


Each room in the room list response now has a users property, indicating how many users are currently in the room.

Then we call getRoomUsers for each room, but only if the user count is greater than 1 - whereas before we called it for every room regardless if it has users or not.


Is there a sample for collision detection of two convex polyhedrons? @danfis


Done!


ffffff


Not at my computer so pardon mistakes, but:
I think there's a check in the rails controller so you can't delete rounds if they've been started, so double check to make sure this change doesn't blow that up.
Also: is there already an ember property called isStarted on turns? If not, should probably add that too.
THANKS FOR FINISHING THIS BTW

> On Jan 27, 2015, at 10:11 PM, Geoff Schorkopf <notifications@github.com> wrote:
>
> @killpack  ?
>
> —
> Reply to this email directly or view it on GitHub.
>


Why hasn't the full history been imported from CVS?


OK, I see what is wrong. My master on GitHub has changed since I created that test. I will merge with latest master and submit later.


Perfect thanks!


este arquivo vai ser repensado, talvez removido, precisa verificar qual vai ser o futuro das conversoes TXT-XML XML-TXT, talvez um hangout pra discutir isso é bem vindo...


Hi Daniel,

If you know the number of wheel turns made each minute, you can then equate
that to the distance traveled in one minute.  From there, simply divide the
'wanted' distance by the distance traveled in one minute and you can find
the time required to run the motor.

On Mon, Jan 26, 2015 at 3:58 PM, dainelha1998 <notifications@github.com>
wrote:

> what do you mean by just using time? Time for distance?
> If so, then how do i incorporate time into the code?
>
> On Thu, Jan 22, 2015 at 3:51 PM, caffineehacker <notifications@github.com>
>
> wrote:
>
> > If you have an encoder you can do that, but you might not have an
> encoder
> > on all wheels. An encoder will always be optimal, but you might be able
> to
> > get away with just using time for now.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/FRC4461/Java/commit/6f4f3523da09defe2dabb00400db782722934394#commitcomment-9407626>
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/FRC4461/Java/commit/6f4f3523da09defe2dabb00400db782722934394#commitcomment-9450432>
> .
>


Test comment! :shipit:


Nah....Now it will be venture_mako-2.0--laladate.zip. I'll add a check for TAG.


I was also thinking about implementing 2d normal mapping in the past few days. I'm really looking forward for this.
Isn't Godot the first non-homebrew game engine that will bring 2D Light System to live?


Ya know... that variable name seems funky :P


update_sqlite_coverage (mbadams5) - x86_64-MacOS-10.9-clang: OK (1567 of 1604 tests passed)

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


Lots of good fodder here. My own two cents is that Brandeis would be far more concerned about data mining practices of--and inadequate protections/disclosure of security breaches--by corporations than he would be about the publication of a Solicitor General's non-privileged communications. To be sure, public officials are keenly aware of the gaze in the Internet age, but it certainly did not stop Governor Palin or Justice Kagan from seeking *higher* office.

Your broader thesis appears to be that the utility of data exists on a curve relating to data's availability is an intriguing one. I wonder whether you can expand on why that's so. In what way does new technology affect that curve? Is it just an arms' race?

On the question of judgment calls in the law. This is a core question about due process and the answer, at least I _think_ this is right, is rooted in your assessment about the level of indeterminacy in the law, and whether individual judgment is a prerequisite to fairness. I am still balancing it in my head... =)


@hemanth Would be cool if you can publish this.


Well, it does! ;) Apparently when :gameId is not provided it removes / as well


`+ void-elements@2.0.0` :)


Ahh your right (https://github.com/dc-js/dc.js/blob/develop/docs/coverage.js#L144), I'll add it back.


Is this supposed to be TodayOffset?


may be move to all.snippets?


how long is the content valid for?
http://www.mobify.com/blog/beginners-guide-to-http-cache-headers/


cleanup?


@hemanth thanks!

Also because `void-elements` is exported in Jade as an array (for some reason), I won't be able to update Jade to void-elements 2.0.0 until Jade v2.


why is this inline instead of nice params?


There's now a isStarted property on both rounds (checks if any scorecards are started) and scorecards (checks if any turns have been started). I *think* the methods on the Rails side mirror the computed properties on the Ember side.


cleanup


Agree, let's handle that in a new issue.


Make this more dynamic


After this change, `brew info` garbage will print out to the screen instead of being captured.


Actually I can just use `Object.keys`. Dumb me.


can this become 2 commits? one that changes the EOL,
and then one that makes your change?


Hmm, again we would have to compare the performance ;) So a new issue, something new will always pop up with perf stats.


You're actually waiting for the button held position and then using the first time its "held" and saying its the press vs the held.  What might be better is a counter for the number of times you checked and it was held.  That way you can say pressed is count 1 and say 3 is held that way there is a buffer between the 2.  This would give you more flexibility.


1277852-CommandLineInterface (nealkruis) - i386-Windows-7-VisualStudio-12: OK (1593 of 1596 tests passed)

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


QUnit notifications is an interesting project. I'll treat this as a regression until we find a better implementation for it on QUnit 2.0.0.

@jzaefferer, is that ok?


I used this fashion to pass tests.


ไม่ได้ใช้ ไม่เอา


I still believe using Timers is the right way to go.  Then you don't have to hack the time and update it.  Much easier to explain


r+


My tendency is to avoid:
Input.GetKey(KeyCode.UpArrow)

and to go for configurable input using something like:
Input.GetButton("Forward")

Thoughts?


@leobalter nice :) @rwjblue has a work around but it would be preferable to use the official API


saw approval from glennw
at https://github.com/pcwalton/servo/commit/448a9f5f45c61a6a869a5b195301cc591c736de8


merging pcwalton/servo/font-stretch = 448a9f5f into auto


pcwalton/servo/font-stretch = 448a9f5f merged ok, testing candidate = 4b324fec


I cannot receive any via sms
please , can any one tell me how to request it via voice
thanks in advance :)


这特么是谁传上来的文件？！！！！特么都泄露学号了？！！！！！！！！！


@hemanth I can expose the Array generated with `Object.keys`, but internally use the object.

Also for some reason the index.js in the published version still uses the Array form.


Are there any devices on the bananapi that aren't on either cubie?  Or a
difference like the nand size or dram size?
On Jan 27, 2015 6:43 PM, "Jean-André Santoni" <notifications@github.com>
wrote:

> I would need your advice. I want to build an image that works for
> cubietruck, cubieboard2 and bananapi. Any idea how recognize the bananapi
> in u-boot to load my uEnv.bpi?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/patrickhwood/u-boot/commit/80915217f89adb2ee4b251a84ad55ba08f37c1c4#commitcomment-9469951>
> .
>


Sorry, my bad. I had issues with that chain getting it to compile or... use it. I assumed wrong and thought that you had the same issue.


使用绝对路径。因为crontab运行时，没有环境变量。php和while.php都要用绝对路径。


Yes. One spelling error and a lot of tab completion. I'll fix it in the morning.


This doesn't seem to be used anywhere?


It's so obvious now... if (item !="earthquakes")!!!!!!!!!


不知道打个.gitignore文件么。。。


这特么又是怎么回事？！


QUnit.url wasn't intended to be public, it is also not documented. I did this commit because I'm progressively trying to clean QUnit from exposed methods, but I believe a *good* implementation is enough to consider it a regression.


Incorrect leftover variable name.


It's late so the name doesn't accurately reflect what it does.

What it does is: "Show all content pubilshed by selected publisher for the current database (i.e. platform/system)"


One thing we will have to watch is the parameters we set on the textures as different targets support a different set of filters and wrap modes.


Credits to Vanir here


aha, docs/coverage.js... it is a standalone utility to discover which methods have been documented.

IIRC it does not work/is not useful because it doesn't comprehend "inheritance" properly. If we get it to work, it should probably receive its input the usual grunt way, rather than this weird export.


some tests failed:
failure: http://build.servo.org/builders/linux1/builds/602
failure: http://build.servo.org/builders/linux2/builds/601


This is causing a giant diff, when all you need to do is overload the method with an extra parameter. Talk to Madhura and Chris who recently made a similar complication, when there is a much simpler solution


Looks generally good to me.  Feel free to push after fixing the in_tex_target non-usage.


@TimothyGu Hmm..released a patch, looks like `pre-publish` isn't working...`+ void-elements@2.0.1`


Vanir and Slimroms here as well


yes. Take autoprefixer as an example.


Ack


Misleading name, isn't this just a scim.wirte token, and not a zone admin?


At operator.md it should be operator instead of filter, what you say?


Are yankactive freqs optimized?
I have set hispeed, sync and up_threshold = 1.497
Should there be any conditions like sync should be lower than hispeed or else?


Well I added it back...  We can remove when fixing the Coverage stuff.


This change broke some functionality for me.

I used to be able to do this:

```
ssh://hg@bitbucket.org/russellballestrini/my-private-repo:
  hg.latest:
    - target: /www/my-private-repo
    - rev: tip
    - user: russell
    - opts: --ssh 'ssh -i /etc/salt/bitbucket-private-key'
    - require:
      - file: /www
      - ssh_known_hosts: bitbucket-fingerprint
```


We really need it? (May be choice reactivity as option).


OKL4 implements `pthread_exit` in [lib/posix](https://github.com/kungliet/okl4/blob/master/libs/posix/src/pthread.c) as following:
```c
void
pthread_exit(void * value)
{
    /* Obtain the pthread state from the thread state in TLS
       This function is defined in threadstate.h */
    pthread_t pthread = thread_state_get_pthread();

    pthread_lock(pthread);

    /* Set termination status and exit value
       Termination status simply indicates that the thread has finished execution */
    pthread->termination_state = PTHREAD_STATUS_TERMINATED;
    pthread->exit_status = value;

    /* Unlock any exec'ing pthread_once's.
     * We must do this before calling cleanup functions, as they
     * could themselves call pthread_once with the same control block */
    while (pthread->once_exec_stack != NULL) {
        struct pthread_once_i * once_control_i = pthread->once_exec_stack;
        okl4_libmutex_unlock(&once_control_i->mutex);
        pthread->once_exec_stack = once_control_i->next;
    }

    /* Call any registered cleanup functions */
    while (pthread->cleanup_stack != NULL) {
        struct pthread_cleanup_routine * cleanup = pthread->cleanup_stack;
        cleanup->routine(cleanup->arg);
        pthread->cleanup_stack = cleanup->next;
        free(cleanup);
    }

    /* Destroy keys, calling any destructor functions first */
    // TODO: possibly keep a stack of used keys to avoid having to iterate entire list?
    for (int i=0; i<PTHREAD_KEYS_MAX; i++) {
        if (pthread->keys[i] != NULL) {
            struct pthread_key_instance *pthread_key_instance = pthread->keys[i];

            /* Call destructor, passing data as argument */
            if (pthread_key_instance->destructor != NULL) {
                pthread_key_instance->destructor(pthread_key_instance->data);
            }

            /* Free and clear */
            free(pthread_key_instance);
            pthread->keys[i] = NULL;
        }
    }

    /* IPC any joined threads, passing the returned value */
    if (pthread->joined_stack != NULL) {

        /* Set the thread to detached */
        pthread->detach_status = PTHREAD_DETACHED;

        /* Since we are now detached, we know we're about to die.
         * Remove ourselves from the list now before we wake up any joiners.
         * This ensures that those joiners will not continue and find us
         * in an unjoinable, but still valid (existent) state */
        pthread_list_remove(pthread);

        /* Create the joined return message */
        L4_Msg_t msg;
        L4_MsgClear(&msg);
        L4_MsgAppendWord(&msg, (L4_Word_t)pthread->exit_status);
        L4_MsgLoad(&msg);

        /* IPC each joined thread in turn */
        while (pthread->joined_stack != NULL) {
            struct pthread_joined_thread * joined_thread = pthread->joined_stack;
            L4_Send(joined_thread->joined->l4tid);
            pthread->joined_stack = joined_thread->next;
        }
    }

    /* If the thread has a detach status of "detached", delete it immediately
     * Otherwise, wait to be told to die. */
    if (pthread->detach_status == PTHREAD_DETACHED) {
        pthread_unlock(pthread);
        pthread_delete(pthread);
    } else {
        pthread_unlock(pthread);
        L4_ThreadId_t sender;
        L4_Wait(&sender);
        pthread_delete(pthread);
    }
}
```

What we can know from the implementation is:
* First, indicate that the thread has finished execution
* Invoke the registered cleanup functions
* Destroy keys iterally
* Inform any joined threads, passing the returned value
* If the thread has a detach status of "detached", delete it immediately. Otherwise, wait to be told to die

@georgekang, Please guide for utilize Pager specific operation to implement `pthread_exit`.


I think I understand, that sounds simple.


This means that users without `+colorcolumns` will not have `textwidh` defined, which is a regression.

You should put the `if exists('+colorcolum')` around the block down here.


You should probably also check that there's only one GstGLMemory so the possibility of having multi-planar/memory rectangle textures doesn't break when sampling the first plane as YUV in the shader when it's really a single Y plane.  Another option is to add the APPLE_422_YCBCR type to the GstVideoGLTextureType enum and check based on that.


hi


Eclipse decided to crash in the middle of that commit. Here are the rest of the details:
- Added chunk loading: ChunkTaskPool now scans for chunks within a defined world limit and ensures those that aren't needed are removed
- Textures can now be set to visible or not, as specified by JECoder0x2D
- Lots of bug fixes and minor tweaks


@problem - @thoys has a good point. This statement probably shouldn't be here. It came from the readme.txt that was in the gverb directory... but it doesn't make sense in the BUILD_WIN.md instructions.


comment is general question about the use of log in this file.
we havent really logged our output anything to date from the cfops or related libs like cfbackups.
just curious what prompted all the log output here?


express.oi!


Another question... why not add these instructions to the other platforms as well... you also need to follow these instructions to get it to build on the Mac for example.


CP paste from other files because i don't care.  speaking of.. don't you have anything better to do?


Excellent points on the role that the public and institutional actors play in determining the long-term effects of policy. I think the constitutional analogy is great! A protectionist document in concept; a stunningly elegant democratic document in practice. Open Government Data _could_ be another protectionist effort. Or it could promote democratic governance. Your point about the virtuous cycle of government legitimacy is an important one: open government not only helps the public; it necessarily helps itself.


nitpick: const GstGLFuncs *gl;


TESTS FAILED
 Mr.roboto url : http://jenkins.plone.org/roboto/get_info?push=64a6fed266184097854519be07d59971
 plone-5.0-python-2.7 [FAILURE]


コンフリクト実験.txt

test


So if I understand correctly, the main problem is that when using filter expression directly with a data provider, the expression instance is not initialized properly with call to prepare() and the request is not updated to accommodate extra requirements (may also request geometry, not just columns).

It makes sense to me to have a pointer to QgsFields if we need to do filtering in QgsAbstractFeatureIterator.

I would also suggest to have the QgsExpression instance inside QgsAbstractFeatureIterator instead of being in QgsFeatureRequest. That looks quite dangerous as one request could be used by multiple iterators (in multiple threads). Ideally QgsFeatureRequest::filterExpression() should only return QString instead of QgsExpression* but I guess that's something we have to keep for QGIS 3.


Did you create & run a "20150122024256_..." migration but not add/commit it?


@arteymix Right... I'm so smart... Have fixed it 06f9147c5d89a61b16397ab17cfd006fe850a0ac


some comment goes here


o(∩_∩)o 您好。请问：
Senparc.Weixin/Senparc.Weixin/Config 的 TIME_OUT 属性，
是否会考虑成外部调用时，允许修改？
目前使用SDK的过程中，发现在网络环境一般的场景里，10秒钟的超时时间，确实比较短。（特别是上传视频文件的时候，微信官方允许10MB以下。）

RequestUtility的HttpGet、HttpPost，有看到方法是暴露Timeout参数，
但是它的上一层，Senparc.Weixin.MP.AdvancedAPIs相关的类，目前没提供Timeout参数供调用。
因此后续是否会考虑提供呢？

希望收到您们的回复。


@mcostalba @lucasart I just had a look at the code. Sorry you are right, depth is accounted for.

However I am still unsure that making it blunder more is a good thing. I'll try playing a few games with it.

I also agree on a certain level that using time makes playing with it more fun.


fff


ff


YAY!


sdfgsdgf


while we havent defined any rules against it, i still feel compelled to say that im really not a fan of multiple returns points from a function. To me it its just an indicator that the function / method is doing too much, and there are other ways to have a function/method exit quickly.

just calling it out, but as i said...we havent defined any rules for or against, and its common enough in the wild.


Testing fot github


great point! I will update the flag now :)


Is that a typo? Feels like whichId.


Thanks for this fix, Erik.  If it is OK, I will just incorporate this fix in my latest DRM pull request to be sumbitted.


making a gateway specific to connections to the bosh director is a good idea. I wasnt thinking in this direction, good call.


You'll want to do the same for the IDs below as was done for the buttons on beta-preference


@zathras-crypto: are you going to use the alert system now? ;)


I like to occasionally drop by and see what kangs your working on Dave :D
On Jan 27, 2015 9:00 PM, "Dave Kessler" <notifications@github.com> wrote:

> CP paste from other files because i don't care. speaking of.. don't you
> have anything better to do?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/VanirAOSP/build/commit/40b58b227de5082cd65cd335137aceab126771d2#commitcomment-9472240>
> .
>


working on a possible fix, will submit PR.  Going to implement an "identity" parameter similar to git.


The verb tense is incorrect here (ran should be run). Correcting it, though, still sounds odd - I think it should read something like, "If this is your the first time running the application [...]"


I really have to fix this hack lol


I think this should read, "run the following command to install [...]"


i dont understand it, whats the deal with it?


I'm merging this right away as it is a minor change and does not affect other parts of the library.


Backbone.historyのnavigate, fragmentメソッドを、ContactManager.{navigate,getCurrentRoute}で呼び出せるようにする。


Love it!


Sorry for being pedantic, but it would apply here, too => "[...] run the following command:"


Yo, these are really good editions! One thing tho. Keep  in mind that the state is a delta list. Ight. keep it up and you'll (I think) be done by lab tomorrow!


Check to see if its safe to downcast lol


In my test, when switch dns servers, python threads is faster than gevent, I don't have enough time to test  both of them.


shouldn't "Automatic" be quoted here?


Sadly this time there is no value to it - we need to tell 0.0.8.2 & below clients to upgrade, but they don't have alert code :(


Speaking of kangs...

On Tue, Jan 27, 2015, 9:19 PM Paul Beeler <pbeeler80@gmail.com> wrote:

> I like to occasionally drop by and see what kangs your working on Dave :D
> On Jan 27, 2015 9:00 PM, "Dave Kessler" <notifications@github.com> wrote:
>
>> CP paste from other files because i don't care. speaking of.. don't you
>> have anything better to do?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/VanirAOSP/build/commit/40b58b227de5082cd65cd335137aceab126771d2#commitcomment-9472240>
>> .
>>
>


Proposed edit: https://github.com/leyarotheconquerer/hivemindproto/commit/87fe4ba12c91156c35986def16cbe9615c239463


git remote add origin https://github.com/mougf/layout.git
git push -u origin master


Misspelled word


fixed here https://github.com/ministryofjustice/atet/commit/dd5cbfcf4d7f0041e2afd79048f8c50e36689170


Are the info messages polluting the log? Could there be value in storing the messages in the log - for an instance - such as a power failure where the log file may be the only way to recover the lost console messages?


is this boshdirector stuff called anywhere in the code? i cant seem to find it.

Also, while i know it's a pretty widely used approach, and you and Joe where talking about it.... I'm not sure i like the internal unit tests. To me it feels like that its an indicator of some things we should change rather than code around. like:
ex.
* we dont have a testable exported interface, so we should write more testable code.
* we are maybe not defining an appropriate exported interface, and should rethink what is being exported.
* perhaps some of the 'testable' non-exported parts would make sense as a standalone package or lib and then exported there and used to compose a different packages functionality?
* etc.

Perhaps a team discussion around this so we can just finalize a standard approach and unified opinion?


What are `null|this|base`?


See comment above about language specific namespaces.


Just so you know putting a copyright on kanged code can lead to a lot of
legal trouble.

On Tue, Jan 27, 2015, 9:31 PM Paul Beeler <pbeeler80@gmail.com> wrote:

> Speaking of kangs...
>
> On Tue, Jan 27, 2015, 9:19 PM Paul Beeler <pbeeler80@gmail.com> wrote:
>
>> I like to occasionally drop by and see what kangs your working on Dave :D
>> On Jan 27, 2015 9:00 PM, "Dave Kessler" <notifications@github.com> wrote:
>>
>>> CP paste from other files because i don't care. speaking of.. don't you
>>> have anything better to do?
>>>
>>> —
>>> Reply to this email directly or view it on GitHub
>>> <https://github.com/VanirAOSP/build/commit/40b58b227de5082cd65cd335137aceab126771d2#commitcomment-9472240>
>>> .
>>>
>>


How did you find out about language specific namespaces?

We support this for backward compatibility with some legacy code but the feature was a mistake in retrospect and we don't recommend using it, don't document it or show in examples. As such I would probably remove it from syntax highlighter.


Okay, I understand now.
Since Pager haven't support indication of thread stop, so I have to add that first?


It should be Installation and not Intallation for Windows, Linux and OSX.


Supported as of PHP 5.3.0


I never knew what that was for :)


ijiiji


s/flexibel/flexible/
Although decibel (db) ends in "bel" - what to do about English stealing words from many languages and spelling being such a debacle (that one ends in "cle" not "kel"!!!)


Is this really the intended behavior?

```.haskell
λ import Data.Aeson
λ import Control.Applicative
λ newtype A = A (Maybe Int) deriving (Show)
λ instance FromJSON A where parseJSON = withObject "test" $ \o -> A <$> (o .:? "a" <|> o .:? "A")
λ eitherDecode "{ \"a\" : 1 }" :: Either String A
Right (A (Just 1))
λ eitherDecode "{ \"A\" : 1 }" :: Either String A
Right (A Nothing)
λ
```

Shouldn't it be rather something along the lines of the following example?

```.haskell
λ instance FromJSON A where parseJSON = withObject "test" $ \o -> A <$> (Just <$> (o .: "a" <|> o .: "A") <|> pure Nothing)
λ eitherDecode "{ \"a\" : 1 }" :: Either String A
Right (A (Just 1))
λ eitherDecode "{ \"A\" : 1 }" :: Either String A
Right (A (Just 1))
λ eitherDecode "{ \"b\" : 1 }" :: Either String A
Right (A Nothing)
```


@dawilster: Does `notify!` need to call `save` at the end as well?


we should just use German ;-) hard enough but at least you write mostly as you pronounce


Suggestion - You could load these values from a JSON config file - which may make it easier to swap out configurations for doing things like mocking objects for unit testing and inversion of control if you decide to implement it. And in my experience, it is a great deal easier to maintain with properties in a JSON file. Think of the gruntfile for example. Even though it is a JS file, also, it is mostly just a container for all the grunt properties. To me, it is much more intuitive to configure grunt this way. Well, IMHO, anyway.


@antonivanopoulos dont_aggregate! also performs a save after the transition.


Huh?


I just noticed this was wrong, it's suppose to be ```livereload.changed``` not ```livereload.change```, overlooked the typo


all tests (except for kt and lrkt) should be passing for you. Now we can go back to doing TDD which should make development a little easier.


Maybe just call it `which`, then. I'd mirror the official SDL docs as much as possible. https://wiki.libsdl.org/SDL_Event https://wiki.libsdl.org/SDL_ControllerDeviceEvent


ah I see, I missed your commit and thought it was my typo


I will fix


1277852-CommandLineInterface (nealkruis) - Win64-Windows-7-VisualStudio-12: OK (1593 of 1596 tests passed)

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


For those looking for an example, check this comment on the initial issue: https://github.com/khan4019/tree-grid-directive/issues/14#issuecomment-71780860


Why?


Gotcha


Except you wear out the keyboard because of the long words :)


libpq.sgmlの8693行にtabが入っていてcheckに引っかかりました。
```
diff --git a/doc/src/sgml/libpq.sgml b/doc/src/sgml/libpq.sgml
index d350507..e1d3673 100644
--- a/doc/src/sgml/libpq.sgml
+++ b/doc/src/sgml/libpq.sgml
@@ -8690,7 +8690,7 @@ int eventproc(PGEventId evtId, void *evtInfo, void *passThrough)
         ensure that event procedures are declared <literal>static</>.  If the
         procedure's address must be available outside its own source file,
         expose a separate function to return the address.
-	-->
+-->
 Windowsにおいて、関数は２つの異なるアドレスを持つことができます。
 外部から可視のDLLと内部から可視のDLLです。
 <application>libpq</>のイベントプロシージャ関数ではこれらのアドレスのうちの１つだけが使用されることに注意してください。
```


Should we avoid using symlinks for the poor folks having to work with msysgit? I haven't tried it in a while, but I thought symlinks used to cause trouble.


Is anyone opposed to using bower/gulp-bower to manage frontend deps?


Thanks.


You can simply Berge and upload them ;)


With TC of 20 secs all tests with level above 10 have no value.


Your Jenkins server is getting a bit weird...
Maybe WorldEdit has vanished from the repo?


Thanks :-/


Nope, just forgot to turn it on after an OS update...


I apologize for the breakage!  I'm writing up unit tests so that I will hopefully not break this again in the future.


1277852-CommandLineInterface (nealkruis) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1098 of 1099 tests passed)

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


shouldn't be a problem if you buy proper german quality.
But it would be an interesting study to find out if german computer users are wearing out more keyboards and if they have more problems with their fingers.


Confirmed reverting these two commits fixed it for me.

7d1a9149e2a1a1854d74699276e2d9bf9536c9a7

37fe35d40f98b3d5728a4097bc3ab2ffb4880536


Can we just omit the isEmpty check and just never print the value?


Why not v2.0.0?


Oops, forgot to take this out.  Yes.


Metrics changed orgs.  This doesn't match our others:

    "io.dropwizard.metrics" % "metrics-json" % "3.1.0"


Any particular reasoning behind the removed animations for the checkboxes? They looked so good before!

I'm unable to find anything in the Material Design specifications that provides rules for this one way or the other, despite it being quoted as the reason for this change. In fact, if anything, the 'Delightful Details' page from the spec seemed to justify having a nice effect like this.

Was it Performance issues? Cross-browser compatibility? User Testing found them to be annoying in practice?


All the plugins were upgraded to 3.0.0 regardless of what version they were at previously so their major version is the same as `6to5` makes everything a bit easier.


I assume you meant to write `prefix + "$project.version" - suffix` here, right? I.e. *adding* the dynamic prefix (rather than subtracting...) instead of the hard-coded `v`.


no


-1 to such a versioning.

For example, we should release `4.0.0` version when some breaking changes occur in Metalsmith API, though `6to5` is still v3.x.


I think that mixes responsibilities. The writer only knows about writing pages to an ORC file. It doesn't (and doesn't need to) know about shards. Also, this says the writer class is badly named and should be called `OrcFileWriter`.


@shinnn I guess. [6to5ify](https://github.com/6to5/6to5ify/blob/master/package.json) is at v4.0.0 because it was already at v3


Yay! So nice to have the panels back!


Related discussion: https://github.com/kevva/elm-platform-bin/issues/2


What's your thinking behind all this?


ОШИБКА! Не та косая черта, срочно исправить!


shit off


I'm in support of using Bower.


this works great


onDestroy should be placed next to the other lifecycle methods


that nice..


@slpbaby, exactly, you have to ensure the indication of the lifetime of a given thread.


I just copied your syntax in getTagName ans was surprised that you use a "-". Never thought about groovcy beeing able to substract a string Form another one :smiley:

I need those prefixes for my projects. This way I can tag a commit for my app project and create another tag for my server project. They all geht a separater version string.


I will sync and build today... I hadn't time yesterday...


It's useless now, I would remove them. Thanks.


GasHumidifier (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1094 of 1094 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)


@MayhemYDG - Yeah, mistake on my part. Thanks for pointing that out.


D'oh, yeah.. right. >_<


色吃


GasHumidifier (kbenne) - 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)


Makes sense to me. I did what I should have done yesterday and searched for outside discussions on the Zork TGZ format, and came across this thread: http://forum.xentax.com/viewtopic.php?f=18&t=3511

It ends with a reference to an open source LZSS decoder from Apple - http://opensource.apple.com/source/BootX/BootX-59/bootx.tproj/sl.subproj/lzss.c - that from a quick look seems to do almost the same thing.

Not quite, though. Their 'text_buf' is 4096 + 18 - 1 bytes, where our '_window' is 4096 bytes. But they both initialize the first 4096 - 18 bytes to 0x20 / ' ', they both start on index 4096 - 18, they both "wrap around" the index the same way (which would seem to make their extra buffer space unnecessary?), and they both add 2 to the 'length' when copying data from the buffer.

There may be other differences - I didn't look that closely - but considering how well ours work even without this patch, I'm not going to lose any sleep over that.


This seems unrelated to the refactoring. The `move()` call will fail if the staging file does not exist (albeit with a confusing error message).


managed to screw up some things accidentallly by not doing pull, code, commit, merge, push


Good idea, I was thinking about that but forgot to add it


emm, it's not strong required here. I would remove it here and in similar scenarios with non-derivable class.


namedBindings


너만잘되냐


ㅇㅇ


Nice


Lol.

> Hiding something

Look.

Anyway. They do look sloppy and remove the beauty of the code.


typo


How does this type check? You are returning an ImportStatement, when the stated return type is ImportEqualsDeclaration? Seems incorrect


Can be this solved with 'git revert bed928b6472456508ecba02c7552bfd145c267bb'?


Remove this comment


o["r2"]...Default set to 203, and max set to 200...illogical


Coolio. Comments forthcoming in email :-)


I'm not sure if you were implying you already knew this, but I ran `asmdump -S` on nvim, and the contents of `main_msg` turn out to be the function `mch_msg`, not the macro.

> That makes me think we maybe shouldn't replace calls to mch_msg with puts until fully understanding what it's doing.

Indeed, although after looking at `mch_msg` and `mch_errmsg` (which is called by `mch_msg` if stderr isn't a tty or `UNIX` is undefined), it looks like this[0] might be a purpose for it, as mentioned by the comment above `mch_msg`:
```
/*
 * Give a message.  To be used when the screen hasn't been initialized yet.
 * When there is no tty, collect messages until the GUI has started and they
 * can be displayed in a message box.
 */
```

This isn't exactly elegant, so I think it's worth looking into alternate solutions.

[0] https://stackoverflow.com/questions/17546111/how-to-remove-the-start-message-of-gvim-and-why-my-mark-function-does-not-work


r=kmcallister


Right here.


Remove space before semicolon


looks great. please update bnuoj.sql in bnuoj project :)


saw approval from kmcallister
at https://github.com/glennw/servo/commit/e936349cdfa7fd9b40dcbd5d269070a29562e24b


merging glennw/servo/poll-events = e936349c into auto


glennw/servo/poll-events = e936349c merged ok, testing candidate = e122d76f


no need for this


Got it. Thanks.


Is this because of https://github.com/sphinx-doc/sphinx/issues/1585?


Thx. Will apply in the next version.


strncasecmp


我喜欢都加空格。。。恶趣味。。。


Hi Ahmed,

so, I have compared H_cal_etot from hcana and hcal_et from Engine, see
attached plots. They are in very good agrrement. Let me know if you have
questions.

Vardan.

On Mon, 26 Jan 2015, Zafar Ahmed wrote:

> Hi,
> When I checked fEtot, to me it does not look like hcal_et. I did a
> comparison with ENGINE. Then I came up with this new variable, "fDEBlock".
>
> Ahmed
>
>
>> I think the fEtot variable, as introduced and calculated by a previous
>> pull request is equivalent to the hcal_et variable of Engine. I think it
>> would be better to add a Get function for fEtot in THcShower, instead of
>> modification in this pull request.
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>> https://github.com/zahmed9/hcana/commit/07379e45b499779e615e1d1389f53e3ee325a2ff#commitcomment-9436847
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/zahmed9/hcana/commit/07379e45b499779e615e1d1389f53e3ee325a2ff#commitcomment-9438480


Got it.


same here


Acutally you can do $(this).parent("#start_time") i think.


Does a multiple line print statement start with three `"""` and also end with three `"""`. Also can this method be used with other methods and functions I am guessing so. Not just the print function.


I think so. The waitForEvents requests the list of event to complete. It's used to make sure when all the event list is ready.


Maybe I'm just not used to it, but I'm not convinced either of these are more readable:

```java
shardUuids.forEach(OrcStorageManager.this::writeShard);

shardUuids.forEach(uuid -> writeShard(uuid));


I'd mention in this description that it depends on our CSS structure (e.g., it is very specific and ties to our internals).


And what type of arguments can be used in the `exit()` method. Using `0` I guess means the program exited with no error. And `1`?


@CyrusNajmabadi, one branch is parseIdentifierName and the other is parseIdentifier. So you might need to look ahead to know which one to use.

An alternative is perhaps to just parseIdentifierName regardless. Then check if the thing you parsed is indeed an identifier. If either the current token is "as" or the thing you parsed is not an identifier, then parse the "as" keyword.


And what is the meaning and use of line 97 and line 98.


This should be moved to the `get_payment_method` method, keeps it more readable IMO.


Yeah, I'm changing the PlayN style for 2.0 as well, so I'll definitely want to update that.


I'd take this whole logic and put it in a separate function in rgw_rest.cc, next to lowercase_underscore_http_attr()


I think you can just call this `nextTokenCanFollowImportKeyword`


hmm, not sure if we want to change this one as it affects external interfaces (albeit inaccurate in the bucket case)


the "lo:<port>" should probably be documented, I think


@diurchenko can you shorten the commit title? maybe add some content describing it.


else {
  break;
}


Flott. Da er vi oppdatert til en godkjent arketype.


@diurchenko shorten the title of this one too


Add a case with `{ a, x }` and `{ a as b, x as c}`. Also maybe test for two commas next to each other, or a comma at the beginning


Sorry about adding the sqlite3 file...I changed computers and accidentally included it when I pushed


`.npmignore` 是 `.gitignore` 的 subset，用来忽略发布时不需要的文件，比如 CI、测试相关的配置和文件，你现在这些内容和 `.gitignore` 重复的，完全没意义。


oops, thanks for removing these


Dump of judgment enrichment tags?


@Xon I'm a bit curious, you mentioned that you were benchmarking the parser. What did were the results?


V8 binding will auto generate a derived class based on WebCLCallback named V8WebCLCallback. and override the handleEvent() function. Following is its code:

```
bool V8WebCLCallback::handleEvent()
{
    if (!canInvokeCallback())
        return true;

    if (!m_scriptState->contextIsValid())
        return true;

    ScriptState::Scope scope(m_scriptState.get());
    v8::Handle<v8::Value> *argv = 0;

    v8::TryCatch exceptionCatcher;
    exceptionCatcher.SetVerbose(true);
    ScriptController::callFunction(m_scriptState->executionContext(), m_callback.newLocal(m_scriptState->isolate()), m_scriptState->context()->Global(), 0, argv, m_scriptState->isolate());
    return !exceptionCatcher.HasCaught();
}
```

our case is a little simple. This mechanism also supports to pass argument to JavaScript through the handleEvent() function.


s/did//, phone added that for a reason.


If lookAhead ends up expensive, perhaps you can parse the identifier before checking this condition. At that point, you would have the identifier, and not know where to put it, but then depending on the following token, you would know whether continue parsing an ImportStatement or ImportEqualsDeclaration. If you did not get an identifier, you would check all the other cases (like *, {, etc)


Proxy.url is set by default and we run proxy mode with option ---proxy
So it can be more logical if when we launch gulp with proxy mode html_dir and twig_dir are set to null.


I kind of like the idea of just parsing a list here. It seems simpler, and we can give a better error.


To @mysegfault
Hi, Maxime Alexandre, I am from crosswalk project. From your foil(http://retour-experience-crosswalk-cordova.gopagoda.com/index_en.html), I heard that you had pain with white screen on Android devices.

If you have no necessity to animate the XWalkView, you can add the following line at Activity::onCreate:
***************************Code Start***************************
XWalkPreferences.setValue(XWalkPreferences.ANIMATABLE_XWALK_VIEW, false);
***************************Code End***************************
This may fix the white screen issue.

If you need to animate the XWalkView,  we do have a white screen or black screen at arm based devices and we are working on this issue.

BTW, If any questions,  you can contacts to me or my colleagues directly.

Hoping this letter not offending you.


GasHumidifier (kbenne) - i386-Windows-7-VisualStudio-12: OK (1583 of 1590 tests passed)

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


참 잘했습니다. 짝짝짝


you need to be a qualcomm licensee to build this module.


Basically the was it does layout seems to be broken (and really slow). Just moving things around to a different page in the output seems to work around it.


Thanks !


You'll see this one in your email :) The (in)famous string constructor!


We do not build as user


GasHumidifier (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Coverage Too Low

![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-2.32%25-red.svg)


It's better to use ErrorStatus here to get the String const value


Good point. I'm fine with that. One might think about making the destination argument optional and default to store it in the instance member, but that would just add another layer of complexity.


I left a comment!


these assertions aren't really related to creating a game, i moved them to the homepage spec


One (possibly more elegant) alternative to the below condition `if (!importClause.defaultBinding || parseOptional(SyntaxKind.CommaToken))` is to `parseOptional(SyntaxKind.CommaToken)` after the default binding. If you don't see a comma, parse a "from" keyword and return the importClause. Then below this `if(isIdentifier())` block, you assume you are just parsing a namespace binding, or named bindings. You will only get to the part below if you parsed a default binding followed by a comma, or if you didn't start with an identifier.


Should be renamed to: removeStepWithForm


rbenv and rvm use this file for ruby version


Method too long. Maybe it could be removed to SiteModel and also divided into private sub functions in SiteModel?


removed some other unused gems...


Wouldn't this be better for users to put in their own `.git/info/exclude` files (or a [global gitignore](http://stackoverflow.com/questions/7335420/global-git-ignore))?  Various editors use various backup filenames; why put them in the repository's `.gitignore`?  (FWIW, Julia's `.gitignore` does not include editor-specific filenames.)


Method too long. We should look at what we can do togheter.


requires were out of order, your own scripts should be after the vendor ones, tree sometimes requires too much also or uses the wrong order


You could have kept method name as it was, and just added optional index?


probably


@deadman96385 what other repo has the default settings?


if you include the application css / js files it will include these files automatically, no need to explicity require them in the html


js includes should be at bottom besides modernizr


includes(:winner, :loser) will eager load the winner/loser player models so you dont have to use Player.find in the view or load them one by one when you are rendering a lot of games on the same page


Ah, I see the problem - I needed specific mysql configuration, so I volume mount /etc/mysql for my config file, and that overwrote the container's my.cnf. So the new way for me would be to mount my configuration files inside `/etc/mysql/conf.d/` ? but I'd have to do it for each file separately so as not to hide docker.conf.


![screenshot from 2015-01-28 11 50 38](https://cloud.githubusercontent.com/assets/10736889/5933292/7ed749a8-a6e4-11e4-816f-3621cfdbad1b.png)
i have this error


@game is not being used in the view, unnecessary variable


@player being used in view now


setting flash notice/alerts on success/fail


needs feature test still


Reason I ask is there's probably some build tags somewhere in a Android.mk file that just needs to be modified to make it work..


emm, I don't think so.
```
struct WebCLCommandQueueHolder {
    WeakPtr<WebCLObject> commandQueue;
};
```

Here I leverage already existed weakptr encapsulation in WebCLObject.


models should not be called directly from view, also changed to use easier style of interpolation


first argument for label needs to match field name, ex :winner_id


@smfn3321 you get an open file descriptior here and don't close it. Please push a fix for review.


using the @player variable is more powerful because you can set properties on it beforehand or use it when a form submit fails to preload the data in the fields. you can access errors on it also from the view


Previously, the resultPosition value was incorrectly getting set to `rawResults.length` for every result. This change should fix that.


all tests pass:
success: http://build.servo.org/builders/linux1/builds/603
success: http://build.servo.org/builders/linux2/builds/602
success: http://build.servo.org/builders/android/builds/638
success: http://build.servo.org/builders/mac1/builds/819
success: http://build.servo.org/builders/mac2/builds/819


this should not be set in the view, i moved it to the controller but model/db default value would be best. i could change value of field in my browser even though the form is hidden and set my rating to whatever i want.


yes, you are right. will add more check here. Thanks


Also the 4 methods called on Game should be moved to a scope inside the model, such as Game.all_for_homepage


Why `EOL` is not working here?


@jordonscnlm looks like this function is deprecated and I assume this could be a replacement http://php.net/manual/en/imagick.setiteratorindex.php


fast-forwarding master to auto = e122d76f


I think just `transitMode`


<3
On Jan 28, 2015 8:14 AM, "Troy Howard" <notifications@github.com> wrote:

> update dates to avoid saturday apr 11 per @synhershko
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/dotnetfringe/dotnetfringe.github.io/commit/a347259e3eef369e1c1f3a1d21da09f467a1f7b3
>


your scripts are amazing!


That commit message should say:

> Replace the placeholder sentence with intro that includes what this is and who it is for.


most awesome commit ever...


no need for return at all if you're not returning anything :)


hehe nice one thx! :D


It might be better to move the `mdNoBar` check in front of the more expensive `ttabsCtrl.count() < 2`.


From them comments extra bytes for text_buf only for aid string comparison. And yes, this lzss looks like more identical to original ZGI/ZNEMESIS with 0x5000 bytes blocks reading from file


That covers it pretty much. QString instead of QgsExpression sounds also good.

Not sure about the best way to get QgsFields it in there yet.

Maybe the constructor can be modified and an empty QgsFields version can be used as default param and filterExpression() will complain to the logs if its required fields are not found.


lgtm


All I did for this was copy what was from the vanilla file for Redstone torch. Seeing that this is, in fact, a recoloring of that torch and nothing more.


I assume you refer to the `size_t` arguments and not the grid pointer. While it has been unusual to `const` non-reference types, I think it is quite common these days. Preventing well_i and well_j from being changed and communicating that to the reader is nice I think. I do not think they should be treated any different from `surface_z` down below.

I'll remove the const from the arguments if you want me to. What about `surface_z` on line 102?


Does it matter that the name used in setup (oscar-commerceconnect) is different from the github name (django-oscar-api) ?


Yes, makes sense. I was thinking more about having to maintain a two non final fields that need to satisfy the same invariant. But the code is simple enough, so it should be ok.


This returns nil for some reason


I forgot to add an assert below, with that it should make sense.


This seems kind of hacky. Couldn't `require: '^^Xyz'` be used instead ?


Same here.


Unfortunately that doesn't look like it did it :( Now it just takes 120 sec to time out! lol

I tried a few different servers, no change.


更新


Yes, this is unrelated. Just something I noticed.


I need to swap the one returned into the front spot, so I either need the index or iterator (otherwise I don't know where to put the one currently at the front). Just the completion is not sufficient.


yes is right


Looks good ! Thanks :smile:


This is my commit comment.


I'm ok with removing one set of docs, I copy the docs to the cpp file for convenience. However, I think that the header file is the natural place for it, not the implementation file. The header file will often be viewed by people who do not bother to look at the Doxygen docs, and I think it is the traditional place to put docs. The only problem is that sometimes lots of docs can obscure the contents, but that can be handled by including a synopsis (as in opm/core/grid.h). What are the advantages of putting the interface docs in the cpp file? Implementation details are of course documented in the cpp file only.


ergh. Shouldn't have merged that. Search is now breaking.


yup, a base class could help here!


It's because you consider a one dim array. try with a two dim array! (thus the ndim==2)


This is just not enough. Will add some other ingredients as well.


no, that's just needed for meta programming and getting the max of a compile time list of integers


I think it is better than nothing (which was the alternative here...), I admittedly put less work in documenting internal interfaces.


You are looking at the first commit ;)


Pull? :-D


No worries, thanks for the good work!


because the goal is just to give it the right shape. Obviously, you don't want to convert the *input* arrays. the result has a merged type as the direct result of float + int yielding a float


> Would it be possible to mark the std::vector<const CompletionPtr> of CompletionSet mutable, and thereby avoid the copy operation here? Or is that to much abuse of the mutable keyword?

I think so. Reordering is very much observable outside the CompletionSet class, and should therefore not be a const method.


Hey, should have probably mentioned this earlier, but one comment. The first place where you've replaced 0 with touchslope, it should be zero. I understand why you set it before we start swiping, but in the first few conditional statements, I'm only using that to update the dragEdge, and that should be instant.


ember-cli just updated to 0.1.11 and now this check fails because you are doing a string  compare.  it should use semver to check the version is good. :)


Hi, I think this should be `xdg-app-repo-update.1`.


umm.. explain to me how this is a kang again?  Also, you do realize that this branch is unused and hasn't been contributed to or built with in probably well over a month right?  Like i've said, maybe you should find something more productive to do.


that was an interface issue due to the way std::conditional works. I fixed this.


yeah!


With the current algorithm it would not have any less asymptotic complexity than the full ordering, so I have not implemented it. It can be done however, and the time for inserting a single perforation would be reduced a bit.


You know. I just started reading through commits in sequence without thinking about that. Sorry.


Ping @RReverser


that's not possible at the instruction level. You need to transliterate between types and that's costly


@zloirock Upgraded `core-js`, I made it a caret again so you can push minors and have it autoupdate.


At what point do you get this error? During scanning and postproc?


이 메서드 _(프라빗)으로 표기해도 괜찮지 않을까용?


Just during regular scan, using scan.py update

2015-01-28 06:51:11 INFO scan: starting update...
2015-01-28 06:51:11 INFO group: alt.binaries.moovee: scanning group
2015-01-28 06:53:11 ERROR server: could not connect to news server: The read operation timed out
2015-01-28 06:53:11 ERROR server: couldn't send group command
2015-01-28 06:53:11 INFO scan: processing binaries...
2015-01-28 06:53:11 INFO binary: processed 0 parts and formed 0 binaries in 0.06s
2015-01-28 06:53:11 INFO scan: processing releases...


Cool - I would probably use optional argument "const QgsFields* fields = 0" in the constructor. That way we could make difference between two different problems: 1) fields were not passed and 2) field does not exist.

For most iterator implementations that wouldn't require any changes as QgsAbstractFeatureIteratorFromSource template class could directly pass "mSource->mFields" to the constructor.


@Artazor I like it, but I am a little confused by async constructors. The idea of async functions to return a promise, whereas constructors must return the object they are constructing. Are those two functionalities not in conflict with one another?


Server request method check was redundant. A more exclusive check is performed in contentSelector.php.


Try now.


Using `some` would be more efficient.


> After composer update without minimum-stability dev, the Predis\Client will be removed which is required in Illuminate/Redis/Database.php.

That's not due to this change. We've removed redis from the default install. Add the dependency youself.

> This also seems to break with illuminate/html.

No it doesn't, you need to add `@dev` to your version constraint.


NB


@grepollo Please actually read the suggestions composer gives you of packages to install.


@zmughal :+1: :smile:


select shape should select the index of the largest of all shapes, and std::get just pick it to pass to the copy constructor


This 'if' never succeeds in Xcode 5.1.1 (iOS 7), so there are no pages added to the 'pages' list. I do not get any '<key>' element in the xml that holds 'WIRTypeKey' has its value in Xcode5.1.1. It works in Xcode 6.1 and Xcode 6.0 though.


Done!


@SubJunk Tested mate and loads fine with
Loading MediaInfo library
Loaded MediaInfoLib - v0.7.71


Looks good to me, they are anyway labeled as a WIP so nobody will expect miracles. I will, without further ado, start packaging a release.


What you missed is that the first arg of select_shape is the dim of the largest of all shapes. In fact select_shape is somehow a... find_if


oh and is it true that we're not deleting our tempfiles ever?


It used to be a perf trick, I am unsure it is relevant anymore...


Yay! :+1:


Is this style really needed with `md-content` ?


How can you make such a ghastly mistake!

and other such fun code reivew stuff


please update your plugins documentation page at https://wiki.jenkins-ci.org/display/JENKINS/Bitbucket+Approve+Plugin?utm_source=dlvr.it&utm_medium=twitter with some useful information


@ExSport it would make our lives a lot easier if there was a way to detect that, but unless MediaInfo have added detection for whatever the issue is since I checked, I don't see a way to detect it.


Better use `ngSwitch` (instead of multiple `ngIf`s checking the same property.


why this code update?


Sounds nice


You mean ``purity_tag`` right?


Maybe `hide-sm hide-md` could be placed here (instead of each child)


Just curious why it is in local.conf.append why not in sample.local.conf so that its gets included in main local.conf?


I'm really unhappy with the code duplication here. Again, you could make it a numpy_expr, once the variadic version is merged in.


We should create a meta-trait for all this traits ;-)


Thanks for your work, I'll take a look at ppt tomorrow and then will comment you by tomorrow morning.


Ok makes sense, thanks.


GasHumidifier (kbenne) - Win64-Windows-7-VisualStudio-12: OK (1583 of 1590 tests passed)

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


Can you add a comment to say why it's ok to leave @payment_url to nil for other types of payment gateways?


Do not work anymore with squid3.
Please revert to previous patch or test for squid version.

Actually, squidguard rewrite squid.conf at reboot with bad values.
For Squid 3 the only integration working is :
url_rewrite_program /usr/pbi/squidguard-squid3-i386/bin/squidGuard -c /usr/pbi/squidguard-squid3-i386/etc/squidGuard/squidGuard.conf;
url_rewrite_bypass off;
url_rewrite_children 16 startup=8 idle=4 concurrency=0


I would prefer to use the `signum` implementation in `java.lang.Long`. It does the same thing.
(It is also in `scala.math.signum` but there the return type is `Long`.)


This might warrant some discussion on whether or not *all* `-port` should be deprecated for `-address` settings that accept ports......


Just a note on github conventions.... it's common to have a `readme.md` in the root of your project with a little info about the project. Even if it is just a personal experimentation project, if someone browsing your profile takes a peek, you'll want them to know what they are looking at.

Also, get familiar with markdown syntax. Code highlighting is especially important when collaborating on github.


No change, no error yet though. Perhaps its astraweb being weird? I will give it a bit and try again.


modifid baseurl


好像還是一樣糟(?)


If this were checking the local filesystem it wouldn't necessarily be a big deal, but making multiple calls to s3 during the rendering of the page html with several images introduces a large performance penalty, in addition to an increased risk of failing to render the page at all and raising an error if any one of those calls to s3 fails.


ident 好像也不一樣？


With the new `setup` helper, some dependencies don't need to be injected in some tests.


hi


我再一個 commit 改正縮排好了


you're right. Change variables are local, no need for that (and the test).

What about updating the code sample in the comment ?


this change allows the extension to on `localhost`


Please review again. This is needed because there are multiple renderers now


That would take a while to load in a website with many images. I would suggest to make this check optional. I would not prefer it on my site personally.


> I assume you refer to the size_t arguments and not the grid pointer.

Yes - absolutely.

> Preventing well_i and well_j from being changed and communicating that to the reader is nice I think

I guess; the point is that to me the `const` in the method signature incorrectly screams reference type. I would actually prefer it removed.

> What about surface_z on line 102?

Here I am not inclined to misreading it as a reference type and the `const` communicates the intent nicely!


Boolean values should be written in lowercase (false)


teszt


I got this error too, it's strange that it lasts for so long ...


congratulation, this breaks the windows build when using bullet by creating a redefinitin of uint64_t, i have deduced that the inclusion of stdint.h was probably the source, note that bullet has a keyword to skip their definition.


That was the 42,000th commit :+1:


@h4cc Good catch. Would you mind creating a PR against the official composer/satis repo for it?


jdfsa


recommended -> typo


thanks for your amazing work.


It's bad design to have a class that is only mockable with PowerMock,
not bad design to mock with PowerMock. You did not design that interface
/ class, right? So...

Am 27.01.2015 um 15:31 schrieb Benjamin Schandera:
> Thank you mate! Also considered PowerMock. Only thing is that I think
> its bad design when you have to use this lib. And actually in this
> case the bad design is on Library side. Or I might geht things
> wrong... I guess, I'll stay with my work around and rather dont use
> PowerMock.
>
> > Am 27.01.2015 um 10:33 schrieb Frank Rosner <notifications@github.com>:
> >
> > I think there is no way to do it with Mockito. You should try
> PowerMock in this case.
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Husterknupp/github-api-ninja/commit/ab73e921ea4a2d07788185382e97284a94d0020d#commitcomment-9459854>.
>


I would expand the sentence: Activate the templates module within the Smart Proxy instance __serving the TFTP module__.


Subir, what is the reason to break video capture API this way?


write me please to mojak.svoboda at gmail com
i'll send you back the project file


Missing @author and class code sections.


Issue #193


Shouldn't the user fork first? As they do not have push access


- delete nullable in IPromise.done definition for compatibility with WinJS.IPromise and JQuery Promise.
- Add definition method for get folder ítems with range in interface 'getItemsAsync' (the class already contains):

https://msdn.microsoft.com/en-us/library/windows/apps/br227287.aspx?cs-save-lang=1&cs-lang=javascript#code-snippet-1


commented after discussing with malaya. presently there is no support for videocapture. Will be enabled after supporting it.


Yes, because the class was copied once by Janek from pbn. But OK, I'll add it.


Styling detail often the cause of nasty bugs: always put {} with if / else unless you put it on one single line:

if (condition) return // is OK, but otherwise do

if (condition) {
    return;
}

I know there were a few without {} but I systematically add them or put it on one line.


@Optimusprimeums can you please test again with the latest version of the "OSX-Java7" branch? I want to check that I updated the right file.


da


caeda0d merged the topic branch and this commit of yours disappeared. You might be able to re-add this at a later point but it will look different now.


see User.ADMIN


Hi,

I have finally removed the empty files from the last commit. Sorry about the dead links!


This breaks existing code that's using SparkServer directly. Only known consumer is SparkBase's private init method, so perhaps not a big deal. Better would be to add an overload or bump the major version as a part of accepting this pull request.


>The only problem is that sometimes lots of docs can obscure the contents

Well - that is my point also; and I even take it to the rather extreme limit that inline documentation is generally noise; at the very least i consider autogenerated doxygen formatting a complete turn off.

For the current question I would be happy to have the documentation in the .cpp or the .hpp file - your call.


@Memphiz : This sucks. Why is this a problem all of a sudden?


If we have a line like this:

    Unattended-Upgrade::Origins-Pattern
        {

the `origin=Debian` would be inserted before the bracket which probably breaks the config file.
I'd propose to either just leave the match with the brackets as you had it originally. Or we try to parse the config file in a more 'fault-tolerant' way, taking care of the next line too - but only if it's possible without too much hassle.


Did you do "git pull" from master? Works fine for me and now we have football.

![image](https://cloud.githubusercontent.com/assets/1864277/5934242/5fd73ec2-a681-11e4-9d37-7f38001e6d64.png)


Looks good for me


OK, const removed.


Well - to me nothing is actually better; the method is private and the only reason it exists in the interface definition is that the language requires it.


@lmr I was wondering mainly about this commit. It's kind-of hacky with the `eval` and `exec`, but I haven't think of better solution. The main issue is that when `tearDown` fails before reverting the correct functions, nose dies (because when I modify `remote.os.path.exists`, due to python nature I actually modify whole process' `os.path.exists` including nose. I'm not sure how nose solves multiprocessing, but if it's just threaded, I guess it'll mess-up the other threads too. If it's fork then it might be probably isolated).

On the other hand I haven't think of better solution to execute existing module which contains other standard python module and faking their results. I used similar functions in `autotest` (`virttest.qemu_devices_unittest`), where I use the autotest's `mock`. Anyway it's more complicated so I came up with my simpler version. The autotest version used different way to express things you want to modify which I didn't like. On the other hand it didn't use `exec`, nor `eval`. Booth does pretty much the same thing and booth affects the whole process' modules so in case `tearDown` is not executed, nose dies pretty terribly and multiprocess checks are in question.


@ioanalianabalas, is this still failing in IE8 as the comment indicates?


>I think so. Reordering is very much observable outside the CompletionSet class, and should therefore not be a const method.

I agree


I'll leave it in the hpp file then.


GasHumidifier (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Coverage Too Low

![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-68.89%25-red.svg)


... still for testing something which requires access to system's resources and other machines it's the easiest way to mock function to return some other value and check payload if expected calls were in the correct order and correct values.


> It can be done however,

Maybe we will do it one day ;-)


@Jckf, ved dispense av lava og water buckets så ødelegger dette rails med BP, dette er en midlertidig løsning


Thanks!


Yea works fine now. Not sure what was up

On Wed, Jan 28, 2015 at 12:05 AM, Blake Parkinson <notifications@github.com>
wrote:

> Did you do "git pull" from master? Works fine for me and now we have
> football.
>
> [image: image]
> <https://cloud.githubusercontent.com/assets/1864277/5934242/5fd73ec2-a681-11e4-9d37-7f38001e6d64.png>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/blakeparkinson/sports2/commit/2ce3e840fbd1760ad27eebc8f0befce24bdebb2a#commitcomment-9474159>
> .
>


Где скобочки? Где они?


oops


Почему здесь закоментировано? Зачем вообще этот класс? @sevenov


This is dangerous because we may not be able to override the cache with the latest views. Is saving of a findViewById worth that risk? Why does having multiple renders require this change?


@sevenov Не называй переменные с цифрами. Пожалуйста


The calculation in the data can be resolved before compilation:

X = -200.95
Y = -1199.1726015625

Also, the SetDungeonMapLevel seems to be important for multi-level maps. We should probably have a closer look at that. Let us spoof the function to see if we can grab level changes from it.