Hi Maxim,

Thank you for modifying files to support MySQL.

Query in *self.count_by_statuses* of *rails_workflow-0.3.0/app/models/rails_workflow/process.rb* has PostgreSql window function that is not supported by MySQL.

I have modified query to support MySQL (this query does not support PostgreSQL)

query = 'SELECT status, cnt FROM
               (
                    SELECT @row_number:=CASE
                                            WHEN @status=status THEN @row_number+1
                                            ELSE 1
                                        END AS row_number,
                           cnt,
                           @status:=status as status
                    FROM
                      ( SELECT count(*) AS cnt,
                               a.status
                        FROM rails_workflow_processes a
                        LEFT JOIN rails_workflow_processes b ON a.status=b.status
                        GROUP BY a.status,a.id
                      ) tmp,
                      (SELECT @row_number:=0,@status:=-1) AS t
               ) t1 WHERE row_number = 1'

Request you to take a look and update.

Regards,
Waleed


Thanks @atm-maxime, you're da man :+1:
Again, sorry for the harsh words. Had a tough day…


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

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


This process has since improved considerably — it's now the opening of a Pull Request on `everypolitician-data` that triggers a parallel PR on `viewer-sinatra`. This lets you see what the data will look like *before* merging it.


This is not necessary. There is a static public method
```java
com.izforge.izpack.installer.bootstrap.Installer.getInstallerMode()
```
available. Just pick the correct value to compare:
```java
Installer.getInstallerMode() == Installer.INSTALLER_AUTO
```


thankyu for your valuable suggestion i will try to improve .
On 30-Jun-2015 3:29 pm, "Sudhir" <notifications@github.com> wrote:

> Avoid code duplication. When you are duplicating logic like this you will
> end up having to fix it everywhere if you need to change the validity
> criteria. Instead introduce one method that will tell you whether the
> character is valid or not. That way you would have to change in just one
> place when needed.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/akashzemoso/Assign/commit/9f1de518c4b4878f8ed83a1cefbb01294db2cff7#commitcomment-11928035>
> .
>


use primitive type


Great!


`currentUserChanged` wasn't enough?


Err, hi - just checking out how this fork differs form the knopki version and saw this. I think this is a private API key - you probably will want to revoke and renew it, since you've just made it public on github.


That note makes this much more confusing, IMO. I don't understand what it's saying at all…


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


@author Smart


We should not use [TreatNullAs=EmptyString] for new features.


rename c to isCompleted


This is generally frowned upon to do this, but it seems common in the current codebase, so let's roll with this.


The data is published as soon as it's merged to `everypolitician-data`. The static website derived from that data is an entirely separate thing. (Naming it `data.everypolitician.org` was a big mistake — it should really have just been called `static.everypolitician.org`. I should probably also have renamed it ages ago, but I thought it would have ceased to exist by now by being merged into www.everypolitician.org.


"Let" is terminology for introducing a new variable. You can't use it like this.


74行目のMainの部分はcom.github.finder.Mainではないですか?


 libbitcoin_test_runner.sh
		@@ -10,7 +10,6 @@
		 #==============================================================================
		 BOOST_UNIT_TEST_OPTIONS=\
		 "--run_test=* "\
		-"--random=1 "\
		 "--show_progress=no "\


Shouldn't this be `libass4 | libass5`?


"--detect_memory_leak=0 "\


Yes, of course. Thanks for informing.

On Tue, Jun 30, 2015 at 3:05 PM, Hendy Irawan <notifications@github.com>
wrote:

> Changed package to `org.soluvas.socmed.tumblej` and license to Apache
> License. @daneshzaki I hope you're okay with this? :)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/soluvas/tumblej/commit/ae4567ce9516fb60669cfd0f704fb9d2c3c3ebe5
>


I think that's just because I was applying patches at some point to yasm which were unneeded. So this package could be simplified ;)


useless when completed[] is changed to primitive


`OptionReads` removed since play 2.4.0

https://github.com/playframework/playframework/commit/4fa5d3b2e4e


Confusing comment!


I am probably not gonna include this. I would be much more interested in a full fletched tutorial on the wiki, which discusses various options and the required post-installation tasks.


missing tests for error case and not completed case :)


It works but maybe it would be cleaner to use mb_strlen($content, '8bit');  (here and below) ?


@VenomVendor Great catch and approach! Making the change. Thanks for all your recommendations! Certainly teaching me a lot of new things and approaches.


Wow nice catch! Will you send a pull request with those fixes or should I port them manually?


No idea what this was doing here.


@SubJunk Have you reproduced same behavior? ;-)


Thanks :D


This is an [encryption](http://docs.travis-ci.com/user/encryption-keys/) version of our API key, so no worries.


:sweat: of course !


I am not understanding here.


Fixed.


You should add a comment to getNumPasses() method that it should cache the result if we do not want to have abysmal performance!


std::array?


is this change released in dropwizard-core 0.8.1?. I am using <0.8.1> version of dropwizard and can not see these changes.I want to use the the normal Logback facilities to be used to configure logging instead of dropwizard logging, is there any solution or work around for the same is available in version<0.8.1>??


Nice, thank you!


Please stay in the 80-char column size


Hi, I would like to know why too :)


This fork does not provide more functionality as far as I can remember. I does add automated test (Travis-CI), Vagrant boxes to experiment/test, some (idempotency) fixes and is well maintained (see our [other roles](https://github.com/Oefenweb?utf8=%E2%9C%93&query=ansible))


Copy-paste mistake. Will fix.


Ok, I'll submit a pull request.


@ulferts @NobodysNightmare not sure if my hypothesis is correct here, as this appears to work for `Api::V2::PlanningElementsController` (unless, of course, it's not being tested)

```ruby
@project.work_packages
  .includes([{ custom_values: [{ custom_field: :translations }] }])
  .find(params[:id])
```
https://github.com/myabc/openproject/blob/feature/20325-rails4-deprecated-dynamic-finders/app/controllers/api/v2/planning_elements_controller.rb#L87


Looks good


FYI; Some issues outstanding with this that are being worked on, not ready to merge!


Ah, sorry then :)


no need for this import to be inside the if.


btw. have you compared your implementation with minilzo?!?

* http://www.oberhumer.com/opensource/lzo/
* https://github.com/fengdh/minilzo-decompress.js

It seems that it is common in emscripten to named the mem file: ".mem.js"... So i would prefer to use the file ending: ".zmem.js" instead of ".js.zmem"... Yes, it's not JavaScript, but only then you will get a gzipped response from server...


Oh thank you for this one :)


@h3xstream Thanks for explanation, we had quite similar ideas. The difference is, that I am tracking "taintness" directly as going through the bytecode and I am marking the values on the stack. Therefore, I can just check the stack item before bug reporting, I am never looking to previous instructions (everything needed is remembered).

The most of improved ConstantPasswordDetector is already implemented, [see it in in my fork](https://github.com/formanek/find-sec-bugs/blob/557772cfea8574983db515b809e13a6a28c0c123/plugin/src/main/java/com/h3xstream/findsecbugs/password/ConstantPasswordDetector.java). Arrays and BigInteger classes are marked and tracked as hard coded too, even from fields. It can detect hard coded passwords in more flow variants than before and than FindBugs constant DB password detector can. I have not found any false positives yet. On the other hand, this proper analysis cannot found constant values with data flow going through an unknown method (which was possible before, of course with false positives), so I am going to add some heuristics with lower bug priority reporting.

 To avoid false positives in other detectors, I have planned to create a data flow taint analysis class by tuning (extending) ConstantDataFlowAnalysis for this purpose, so we could call something like `isTainted()` directly on the stack item. In comparison with your approach, I think this would be then easier to use and would allow us to use more functionality that is already implemented in FindBugs API.

In every case, we could coordinate our work and discuss changes (e.g. in issue tracking) to prevent merge conflicts.


Thanks for the explanation!


Why system-site-packages?


Also, why not use "python3 -m venv" rather than "virtualenv" ?


I agree with you, these files should be in separate directories, that is why I advised you to put it in `/cmake/toolchain/avr` or something like that.
Indeed, even if it should not be in the same directory, it remains a `CMAKE_TOOLCHAIN_FILE`, and I think it should be a subdirectory of `/cmake/toolchain`.


Creating a suffix array for the source text would enable us to find all matching substrings of the pattern that is above a certain length and does not overlap. Maybe this code could be that basis, depending on the license: https://github.com/tixxit/suffixarray (it needs a test suite)


```return -direction;```


```return direction;```


thnks.


3-times duplicated. Should move this to a method.


Yes, it's necessary to make extra request to the header value. The header value in `WWW-Authenticate` contains key information in creating the digest auth.

See ruby implementation here:
https://github.com/drbrain/net-http-digest_auth/blob/master/lib/net/http/digest_auth.rb#L8-L38


@waleedjaffar thank you for comment. I will add config setting to switch to mysql and will add that query in this case. Will add more information here shortly.


remove space before :


directionInput -> $directionInput


Disabling migrations for the docker image is not a good idea, since if someone wants to develop using this image, he might want to make new migrations and testing them will need the base migrations being applied.

Optimizing the time it takes to build the docker image is also irrelevant, since it can be done in Docker Hub and users can just fetch the result.


You're welcome. Anyway, thanks for contributing...


```
<filter>
    <filter-name>characterEncodingFilter</filter-name>
    <filter-class>org.springframework.web.filter.DelegatingFilterProxy</filter-class>
</filter>
<filter-mapping>
    <filter-name>characterEncodingFilter</filter-name>
    <url-pattern>/*</url-pattern>
</filter-mapping>
```

Maybe I do not get the point here, but shouldn't be the filter-class org.springframework.web.filter.CharacterEncodingFilter then?


bundle updateかけました。
もう一度pushすればいいですか？


Oops, my bad!
Thanks!


But there were no tests failing :(

The IIFE itself can be bound like `(function () {}.bind(something))()` :D I'll have a look after I'm back from holidays.


yeah, i agree with your point then the mutation is clearer and more obvious.  i think that `merge!` is probably to be avoided but it seemed to work ok here.  ironically, i've forgotten a ton of how i set up the state management and angular code on this site so I'll have to move some stuff around in the coming weeks.


this unit test fails, as the overflow condition is not handled in the Multiply extension


Why switch to `echo`, what's the value?


```ruby
if [:edit, :update, :destroy].include? action
```


Can you please stop messing up merging files?


@dan-nl could you extend this to also return if viewer is `null`?


www.everypolitician.org is more than a map of countries — it's where the CSV uploader etc lives too.

The important part here is simply that the map on the front page of www.everypolitician.org is automatically updated. Though since you wrote this the source of that data has changed too — it's now using the proper `countries.json` file from everypolitician-data that everything should use, rather than the old obsolete viewer-static version.


This is great, thanks. I see how this can work, and it definitely simplifies our string issues. Here are some comments and thoughts.

1) I agree that it makes sense to have UTF-8 as the default string encoding. So, we'll simply remove all other encodings? And add conversion functions from raw text if needed?

2) I'm also on board with using code units instead of code points. But, would it be possible for us to have a flag in the evaluation context that sets this? Just in case some Python user really hates it, or whatever.

3) What is the plan for ndt::fixed_string? We need to keep that to interface with NumPy, right? It remains unchanged, except for the removal of the other encodings?


Still think that using the text from the currentPath is better than emptry string. See [this](https://github.com/atom/status-bar/pull/88#discussion_r33558677).


This will be confusing for user on untitled file as he will see message `Copied:` and then clipboard won't paste any text. See [this](https://github.com/batjko/status-bar/commit/0983a86e225a09e40ecd192911dfb955a6b80994#commitcomment-11928606).


This should still be "The lifetime constraint `'b`", not the other way around


Move the `apt-get clean && rm -rf /var/lib/apt/lists/*` part to same RUN command that does the "apt-get update" and "apt-get install". That'll make the intermediate image smaller.

Also move the `rm -rf /tmp/* /var/tmp/*` to same RUN command that leaves the cruft in /tmp and /var/tmp. Is it pip? Btw. are you sure that something really leaves cruft to both of these?


If we have `bla.ts` and `bla.tsx` in a directory we ignore both -- why?


Agree, will rename.


Will move it to .internal subpackage.


This is an oversimplification and only one of the few different ways one can hit coherence issues. The error is triggered for other cases where there are no local types too.

Either investigate this error's triggers thoroughly, or don't include it for now.


If you are using Gradle as your build system you can and should maintain version numbers in `app/build.gradle`.


Yea, I keep going back and forth on this one.
I suppose if people see "untitled" in the status bar and still want to click on it, they actually do want to have that word copied?
I might revert this one.


s/passed/declared


Лучше на верх вынести
```ruby
let(:params) { {} }
```
И специфицировать там где нужно. Здесь тогда можно было бы написать
```ruby
let(:params) { { id: account_owner.id } } if action == 'edit'
before(:each) { get action, params }


There are conflicting trait implementations for the same type.

(Second sentence unnecessary)


well we have spaces between parantheses and braces in most other places (methods, conditionals, ...)


白オーブの実装 #342


"this makes the trait implemented on all types", rather than "it means ..." (passive voice is better here)


```ruby
expect(User.find(id).role).to eq role
```
должно работать, так как сравнение моделей происходит по id


"this is considered ... implementation" -> "the implementations will conflict"


Niels,
1) it is a bad idea to change format for the schema - people already use it - it will break them all ;(
2) bad idea to save lastConfig value in config for two reasons:
2.1) anyway you don't save the config after changing the value, so it might be lost. So, simple this.lastConfig is ok too.
2.2) if you save it, it will make a lot of write cycles without any use (this is not really needed to save)


assuming that is open source project as https://github.com/netguru/help , nginx production conf (afaik Krzychu mentioned that) and especially domain name in public repo is not a good idea and shouldn't be public :)


One word: conditionscript


Actually my code was a little bit different yesterday, but today I changed it taking inspiration from yours and now multi DPI folders are supported. Do you confirm me that multiple DPIs are separated by space?


SKY LEARN HOW TO ROUND UP NUMBERS PLEASE


@lkraav because we use `echo` for the most part elsewhere, too. And `echo` is slightly fast. In fact, I was just filling in the `for` attribute of a couple labels when I saw `print` being used - and replaced it.


@kubetz Here we go.


Your always complaining about short variables and then `V`?


Trop hard pour moi ^^


Report the issue in https://crosswalk-project.org/jira/browse/XWALK-4517


Why is the explicit paragraph flush necessary here at the end? Without it the second page of this is not even generated, but when I tried creating a minimum test for that scenario I was unable to come up with one.


Added Scalding to DB section.


what's up?


I'm thinking here and the lack of \0 may be a problem for some uses.
In the other hand, expecting for \0 on blob-validator seems a problem, since some node types that handle blobs, like the file/reader won't add \0 in the end.


Yes we are. Will remove and make a note in the docstring


Can I ask you a question?


We are trying to *avoid polluting any global infcx* here.


Tiens oui, il manque DOMAIN_SUFFIX au moins.


euh, t'es sûr que c'est pas 1.3 plutôt ?


shoot


It was used in the various plugins, indeed. It has not been removed because this method adds only a few lines of code. Probably it should be used in several parts of the message_route heuristics instead of doing custom checks (hasattr) numerous times.


to dodalem


They are seperated by a dash (-) in the foldernames, e.g. the foldername of Maps is: 400-480-560-640


Are you free?


zmien na wielkie litery


We could add this behavior as an option for blob-validator, something like "expect_terminating_null_byte".
I'll wait for comments before working on a v2.


I remember a chat on The Loop at StackExchange with @toscho where people were measuring `print` vs `echo`, and `print` proved to be faster. Hence the confusion.


you port manually,  me a pull resquest I've never make it :(


Should be declared as a var, or change ending of above line to comma.


Alright, thanks for the contribution! :smiley:


Should be declared as a variable.


:D yes indeed


Może dodamy tą metodę do API?


Initial setup for displaying WooCommerce custom product fields. Everything goes into functions.php (CHILD THEME) file.


初始化


Still lolling, ftw


fixed in development branch


这里有向最早作者致敬的意思，建议适度保留。


@kubetz Got me convinced.
Happy now?


cpus() just returns the overall Cpus that are online. It does not mean that these cpus are all dense packaged regarding enumeration. An artificial example: cpus() may return 4, but the online cpus are 0,1 and 4,5. Better use "hip->cpu_max()" and check for a enabled cpu by calling "hip->is_cpu_enabled(i)".


OK then I'll have to change the code a little bit, I'll do it later.


Don't like this, an earlier hook can't know if a message will be filtered making behavior order dependent


What about target browser check in the condition?


@nfeske: Mmh, same applies here ;-)


91958608DOASEffectOnZoneSizing (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


maybe calling it callQueue would be better?


This one could be could called frameQueue, but this is bikeshedding, pick whatever you prefer.


The Adex2populations_2compartment_model.m ran smoothly, however, when two compartments where replaced with 1 compartment (Adex2populations_1compartment_model.m), model did not run. The error message is : reference to non-existent field 'g_l'. I looked at the source code of the simulator and I assume that this error is due to the fact that g_l (along with other passive properties) is only calculated when       NeuronArr(iGroup).numCompartments >1 , instead of >=1 , according to the VERTEX simulator script "calculatePassiveProperties.m". It would be helpful if someone can look at this too, thanks.


well what if an other browser also identifies them wrongly? but I can add a browser check if you like :)


Any chance that this gets merged into Dolphin soon?


91958608DOASEffectOnZoneSizing (mbadams5) - 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)


I stumbled *upon* a coding post...


That commit is not yours here is the original commit+author
https://github.com/CyanogenMod/android_system_qcom/commit/a51136993f7d87ffc69cfab833346cd3a99605c5


OK


Denne committen er litt for svært til at jeg klarer å fordøye den..


typo : I started *an internship* at Junto Studio...


stumbled *upon*...


«I dare say that…» имхо вообще лучше перевести как «Осмелюсь заявить»


Well, `echo` **is** faster, but only slightly. I assume @toscho just prefers `print` because it may be used in conditionals. But again, the main reason for this commit was not replacing `print` with `echo`. :)


Why add AutoCloseable? There is nothing to close.


a common mistake I've *made* myself.


А зачем тут запятую убрали перед «вы»? Этж сложноподчинённое предложение.


Looks like `props` is not used?


practically embedded *in* the curriculum


I've learned *on* this two-month adventure:


working very late hours (sin el *at*)


@dcarty01 This `check_search` method doesn't look like its being used - were you intending to use it?


@dcarty01 Well done on writing the iterative search method, looks good :+1:


no more bugs, no more features to add, literally nothing to touch anymore....
pretty much looks like this project is finished :p


Well yeah. Usually on someone else's repo, it's not a good idea to say anything about separating housekeeping commits from business logic commits, etc :)


I had a look at the cmdliner documentation for the variants returned by `Term.eval` (at http://erratique.ch/software/cmdliner/doc/Cmdliner.Term.html#TYPEresult ), and it looks to me like our response to `Help or `Version should be to return 1, so the caller knows that the expected work of selfsign didn't complete.


how can I get the patch??? I want to  get this patch,please help me !


`int` may not cover the full range, better use `size_t` or `unsigned int`


Me encanta como esta escrito este párrafo! <3


好的，那只保留第二个修改好了

> 在 2015年6月30日，19:03，bigeagle <notifications@github.com> 写道：
>
> 这里有向最早作者致敬的意思，建议适度保留。
>
> ―
> Reply to this email directly or view it on GitHub.
>


Squash this commit with the previous one. Git rebase -i will do that for you very easily!


Better use ´unsigned int` or `size_t`


@stefyb14,

Esta buenísimo! Me encantó el comparison con writing y el breakdown a 6 lessons!
:D


Could be: `uint256 hash(svstr[0])`


@bchavez :

  We are suddenly getting below exception in our Coinbase API integrated website. Please guide us to get over this.

![coinbase api exception](https://cloud.githubusercontent.com/assets/2277305/8429705/d7e9861a-1f47-11e5-8314-6d952c1b3b44.png)
![coinbase api exception2](https://cloud.githubusercontent.com/assets/2277305/8429706/d7ed05c4-1f47-11e5-864e-5eba66eb8d49.png)


Yep, that might be a good idea.

I'm also considering deprecating the other parameters:
* `pageName` replaced by `name` since passing spaceReference "X.Y" and pageName "Z" will not create "X.Y.Z", but "X.Y.Z.WebHome".
* `tocreate` (with value `space` or empty) reused, not deprecated, to value `terminal` or anything else (`space` or generally any other value that is != `terminal`).

This would be the result:

Existing/Deprecated parameters:

space
pageName
tocreate="space"

Backwards compatibility behavior:

1. document in space
space="X"                  < (final) space name
pageName="Y"               < document name
tocreate!="space"
=> "X.Y" document

2. space
space="X"                  < (final) space name
pageName=ignored         < replaced by WebHome
tocreate="space"
=> "X.WebHome" document

New parameters:

spaceReference
name
tocreate="terminal"

Nested Documents behavior:

1. terminal document in space (NS style)
spaceReference="X"         < final space reference
name="Y"                   < document name
tocreate="terminal"
=> "X.Y" document

2. Non-terminal document (ND or NS space)
spaceReference="X"         <  parent space reference, not final space reference
name="Y"                   <  space name
tocreate!="terminal"
=> "X.Y.WebHome" document


I hope it's not an insult :D because I really have no idea what this means :D


Once this is merged up to develop, we'll need to add a call to sync_mfocus in MCInterfaceExecCreateWidget as well.


Not sure how good (w.r.t. potential resource leaks) is the idea to return an open reader. Would it not be better to return a e.g. a URL that can be opened and safely closed using try-finally?


Given you reserve the order a few lines later, maybe we could just flip `<` and `>` here?


"VARIANT" seems to be a problem for at least openjdk,  when I try to make VARIANT=AAAA I have this error:

ERROR: Your VARIANT environment variable is set to AAAA. \n       Needs to be set to DBG or OPT \n
Exiting because of the above error(s). \n
make/sanity-rules.gmk:71: recipe for target 'post-sanity' failed


Ok, fixed in bac6694a18dc302f91b74cee527f0cc457d13b9c.


Wohoo, great work!


`step_X` shouldn't be used anymore


:clap: :clap: :clap: :clap: :clap:


`Properties.load()` does not close the reader, iirc. You should close it explicitly.


Thank you for finally giving credit. I still would've preferred cherry
picking but I'll settle with getting credit :)
On Jun 30, 2015 04:45, "Cheeppanon" <notifications@github.com> wrote:

> quincydcm: Add f2fs support
> Thanks to @javelinanddart for working on f2fs
>
>
> https://github.com/CandyDevices/device_samsung_quincydcm/commit/d717537426731a0c62fa15323ffb00d909188301
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/minelove/android_device_samsung_quincydcm/commit/bc175e991c400b91bdc0aeb48a20d59889ccfdec
>


Hello!
Pardon me if I interrupt while your framework is not even released but coming from a play/ninja, I wonder if you intend to also yamlize the pom.xml file.

The pom file is not readable, mix tasks and dependency definition and is not very welcoming to new developers.


The locks look fine! :)


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


potem dodam, jak ulepsze entity trackera by działał jak trzeba


Thanks for the heads up.


PhoneWindowManager.java:7723: error: reached end of file while parsing
}
 ^
1 error


@bachir151 Google -> c++ palindrome.


hi


Ty


Do skins too.


Thank you for the notice. In your example, would the physical affinity space have a width of 6 (cpu_max + 1)?


@afc163 不要在 componentDidMount 里再 setState


undefined behaviour here... ah, i'm such a dumbass sometimes...


This is already the default value and shouldn't be necessary


TeamCity xBIM :: XbimExchange :: Build [Build 362](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1977&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:03:02


This is the default value and isn't necessary


Hey,

Yamilize the pom.xml is not planned (yet). I may switch to another build system (e.g. gradle), but that is out of scope for 1.x.x. Besides, altough I don't like XML at all I didn't had a problem working with the pom.xml so far.


Please don't use `boost::to_string`, but `strprintf` instead. :)


@snc this is broken if i install the version. `\Predis\Profile\Factory` doesnt exists but `\Predis\Profile\ServerProfile` does...maybe composer resolved a wrong Predis version?


Good point, Heiner. I was also worried by the "magic number". I'll fix that


Hmm... The no names checkbox is being ticked, but the names don't go away until I uncheck it and re-check it again.


> We need an update metric endpoint if we want to, for example, let users
change retention policy for a specific metric. Or does such thing
already exist?

We need the endpoint. But first and more importantly, changes are needed to the task scheduler.


Correct my if I'm wrong. Shouldn't it be possible to not specify the expiration time at all? When implementing JWT-powered refresh tokens, being forced to specify expiration time makes no sense as refresh tokens do not need to expire.


this will fail for out of source builds


I don't think you'll need this method on the overall interface.  Only the test impl.


@kean how can we now tell if the response came from the network or the memory cache?


You need a directory for moose, not the global /var dir, unless ./configure is appending "something" to the end of WORKDIR (ie; WORKDIR is a prefix).

If not, then you'll need to set this to something like ```/var/db/${PORTNAME}```


Don't forget to update the fstab.qcom in your device tree and add kernel
support

On Tue, Jun 30, 2015 at 4:45 AM, Cheeppanon <notifications@github.com>
wrote:

> quincydcm: Add f2fs support
> Thanks to @javelinanddart for working on f2fs
>
>
> https://github.com/CandyDevices/device_samsung_quincydcm/commit/d717537426731a0c62fa15323ffb00d909188301
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/minelove/android_device_samsung_quincydcm/commit/bc175e991c400b91bdc0aeb48a20d59889ccfdec
>


These will need to be reserved and committed in ```/usr/ports/UIDs``` and ```/usr/ports/GIDs```


How about simplifying things to
```
		public MusicInfo[] Playlist { get; private set; }
		public bool Installed { get; private set; }
```


Yes, this should work, please make a pull request
thx


These will need to be reserved and committed (added) in ```/usr/ports/GIDs```


The thing is that predis is optional so you have to update your composer.json, too. See https://github.com/snc/SncRedisBundle/pull/172#issuecomment-117118218


This only works because of call order.  You want to actually watch the content of the parent and the args that you're tying together so that I could create A, B  and C, D and then chain them together A, B, C, D and recognize that chain.


Don't include the underscore in `@import` statements.


Can be `readonly`.


Any specific reason for using byte here? It ranges from 0 to 255, but we should still go with `int` here IMO (it's also noted as int in nativeDB)


This commit isn't very atomic, @gatorpazz. Commit more often in smaller increments.


I'm so sorry, for not giving credit to you at first time. :(
Thanks.


The naming standard is `WhatTheFuck?`. I'm not sure why you've decided it can by `null`.


Hui, please add UPLOAD_UUID to the CallHomeSettings, similar to CallHomeSettings.LAST_SUCCESSFUL_UPLOAD.


Oh my.  `authorizationFilter` produces an `http.Handler` based on a parent.  You ought to be able to use such a function (or interface with that function) the entire handler chain.


Except that none of these endpoints exist in the API...


Here's the problem, I think. Compiling now without this "{".


This is fine, but best practice is to use the angular $window injectable - one for the future


Should probably add the rest of your team to this list...


91958608DOASEffectOnZoneSizing (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (648 of 649 tests passed)

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


Myek @FroMage this isn't right.


@Robinlovelace Did you mean to add this as the third option? 'Cause this new layer - which looks great, btw, doesn't go with the title 'Black and White' set here:

https://github.com/npct/pct-shiny/blob/master/master/ui.R#L20


You already have the fstab.qcom commits, you just need to update the fstab.qcom here.
Here is the kernel commit: https://github.com/CandyDevices/kernel_samsung_msm8660-common/commit/4737ee54a5a470490a105bf8a1d82cd8d8165bef


'mfs' will need to be reserved and committed (added) in ```/usr/ports/UIDs``` and ```/usr/ports/GIDs```


This was tested on a system with a broken Virtual Box installation, i.e vboxdrv kernel module was missing. The libvirt provider was 'forced' by setting VAGRANT_DEFAULT_PROVIDER=libvirt on the shell vagrant commands were executed. Additionally, "Setting up of network" task in the ``common`` ansible play was commented out, since libvirt provider doesn't have a network interface plumbed by the name 'enp0s8'.


It is best to avoid this illegible coding habit. The code is hard to identify problems, if there exists one. You can improve this. Firstly is the code is unnecessarily lengthy. See mine below:
    prevChar = strO->str[(strO->index)-1];
    curChar = strO->str[strO->index];
    else if((curChar == ' ' || curChar  == '\n' || curChar == '\0') && isdigit(prevChar))


Am I counting right that this is the third exemplar of the same class within Hawkular code base?
Is there really ready-to-consume jar providing this functionality?

`TokenReplacingReader` should have a couple of tests if it is to stay here.


:+1:

merci


+# hello, Mark!


Not quite that simple mate, the code to trim the array for ```nFrom``` and ```nCount``` requires the array to be in a certain direction otherwise we're cutting from from the back (oldest transactions) rather than from the front (newest transactions) if that makes sense.


thank you for your advice. :)


How does this work? I always thought that „The only code construct allowed before a namespace declaration is the declare statement“


Ah I see, then don't bother.. :)


When you say "full range" what do you mean?  This variable is how many STO payments the wallet has received so I'm not sure what range we apply in that context :)  Happy to change it though :)


So you advise this?
```
cmake/
|_ toolchain/
  |_ avr/
  | |_ atmega128.cmake
  | |_ atmega2560.cmake
  | |_ atmega32.cmake
  | |_ avr.cmake
  |_ generic.cmake
  |_ sasiae.cmake
  |_ stm32.cmake
```
(sasiae and stm32 do not have their own directory as they currently target only one "mcu")

I think it would be better that way:
```
cmake/
|_ toolchain/
  |_ generic.cmake
  |_ target/
    |_ avr.cmake
    |_ avr/
    | |_ atmega128.cmake
    | |_ atmega2560.cmake
    | |_ atmega32.cmake
    |_ sasiae.cmake
    |_ stm32.cmake
```
This way, it clearly gets more and more concrete the deeper you go down the tree. And again, sasiae and stm32 do not have their own directory because they currently target only one "mcu".


You should never use empty commit messages ... and yes, `...` is an empty commit message.


Ahh nice - I didn't realize you could initialize a ```uint256``` directly from its hex like that :)


In future you can safely delete these blank `*.coffee` and `*.scss` files.


@Rayne: that warning occurs as soon as you do anything with datetimes. For me it is a requirement in the php installation to fill in `date.timezone` in `php.ini`.


Sorry, I don't understatement this comment.


For the time being yes. I think that it won't cause any performance issues. If it will turn out that, this is a bottleneck, we will think about more complicated solution, like implementing FIFO queue using SQLite table.


In future you can also safely delete blank `Helper`s like this.


PhoneWindowManager.java:5753: error: cannot find symbol
                        if (mVolBtnMusicControls) {
                            ^

mVolumeMusicControls instead ?


I think this commit added some wrong lines to ClientProxy.java
169 +<<<<<<< HEAD
173 +=======
176 +>>>>>>> 88764f8b1d81d853efe90854c844e99d0c479977


The routes at line 3 and 5 were probably added by `rails g controller ...`. In future, these generated routes be safely deleted when you separately define the routes yourself (as you did with `resources ...` on line 11).


Initially it was a replacement and is still needed as such: the original appears to be down:
https://leaflet-extras.github.io/leaflet-providers/preview/index.html

Problem: it's still loading the grey basemap for the cyclestreets layer, wasting resources.

Please fix if it's easy; otherwise I'll take a look in a bit.

Robin


Has this been reported upstream?


`t.tokens match { case (_: `(` +: _) :+ (_: `)`) => true; case _ => false }`


Done


done


```ruby
def ok?
  @stat[:status] == Occi::Probe::ReturnValues::OK
end
```


La habilitación o no de los eventos creo que debería estar en la config de cada evento en particular y no en el config general.


Below there is a error case.
I will add not completed test


trying...


@fissban Debería ser el máximo general, default. Lo mejor sería que exista un máximo en particular para cada evento (no es lo mismo 40 jugadores en un TvT que en un All vs All, por ejemplo).


hm, then we have a problem in case the user updates the file but none of the form fields..


@fissban Lo mismo que antes


Why did you add this back?


The result of `size()` is potentially larger than `int`, and it's unsigned.


It would be at least 6 (cpus() would return 6). However, the physical space could also be 8 (or any higher number),  but in this case is_cpu_enabled(i) for i >6 would return false and cpus() still would return 6.


Okay! :)


You sounds like sle :scream_cat:


> It would be at least 6 (cpus() would return 6)
Sorry, it would definitely return exactly 6.


Oops, m**O**difier :)


Checking the file now, i picked it too quickly before having lunch :D


@JanTvrdik It just switch a namespace (for example, [here](https://github.com/nette/tester/blob/47408644590556c5f794d3ba8c36d712159066a0/tests/CodeCoverage/parse/namespaces.php) or [here](https://github.com/nette/tester/blob/47408644590556c5f794d3ba8c36d712159066a0/tests/CodeCoverage/parse/namespaces.braces.php)). In this manner, the old Nette minificator worked.


you could add a hidden dummy field (`_changed` or sth like that) and set a value on it and trigger its `change` event once the file is changed. that way the button should be enabled automatically


dddd


Thank you @thecrazyskull for the heads up


Sure, let me have a bash.


Added not completed observer test


"namespace `$namespace` must be ends by \\" -> "namespace `$namespace` must be ends by " . self::NAMESPACE_SEPARATOR :bomb:


@fissban Puede usarse un ConcurrentHashMap y evitar el bloque synchronized


Yes ;)


@fissban Lo mismo que antes.


ffffffffffffff


Do not add .idea, or any other IDE nor editor files to the project.
Remove it from repo and make it globally ignored by git.
It totally blinds the code.


我在 `device ready` 事件里调用 `registerPush()`，在设备上运行时报 `Class not found` 错误，请问是为什么？

                xgpush.registerPush(function () {
                    console.log('XG success');
                }, function (reason) {
                    console.error('XG Error');
                    // 这里报 Class not found 错误
                    console.error(reason);
                });


No, it's needed, i forgot to place a } after the "break" (line 762)


Are we planning to maintain only a single Inventory implementation?

In case end users should have a real option to choose from several Inventory implementations via `ServiceLoader`, I vote for adding some code here that throws some informative Error messages in case (1) there is no Inventory implementation in the class path and (2) there is more than one implementation in the class path.


reack


No. I'm wrong, lets try again:

It would be at least 6 physical cpus (cpus() would return still 4). However, the physical space could also be 7,8 (or any higher number), but in this case is_cpu_enabled(i) for i >6 would return false and cpus() still would return 4.


Ога, вчера на скорую руку константу выделял. Спасибо!


Alright, thanks.


#349 [プレイヤー]無敵状態の実装
#347 [アイテム]緑オーブを実装する


@fissban @u3games Lo interesante podría ser que todos los mensajes debería pedirlos a una clase MessagesId o algo por el estilo, y que esa clase maneje qué mensaje le va a devolver (esto puede ayudar a cuando se haga el sistema de multilenguaje y a agilizar si se quieren cambiar los strings, la idea del engine es que sea lo más flexible y customizable posible).


@halojoy
Actually there is a field for displaying order that hasn't been yet programmed (as seen above)...

I want to create something like this: https://jqueryui.com/sortable/
Drag and drop will make thing a lot easier!

Another enhancement that I want to do is to not show any category when there's only one category or the others are empty. In this case, there isn't any division, the "category" is the community itself.


I had this problem when I updated cordova. You should update cordova-cli with $npm update cordova-cli or
$npm update -g cordova-cli


Get name from sender when sender's userID is nil like in the case of server-generated messages.


Well, maybe not really needed, but closing it after the break should fix the build (the original commit has the brackets too)


我勒个去，这么多格式的修改？你的编辑器这是干了什么？


Well, I don't want to add libraries to this repository (although I have already violated this once and I want to keep it that way). So it seems you have to *optionally* bundle it.


Use reinterpret_cast here


@tomislav unfortunately, there is no way to do that in v0.4.0. In previous versions you could check wether metadata is nil, but this behaviour was not documented. I should probably add a new `DFImageInfo*Key` that would explicitly tell wether the response was from memory cache. I'll fix it shortly, thanks for your feedback.


去掉了所有的无用的空格。。。。。。


ok to test


May need a comment to explain that r can never be zero, so the variable-time doesn't matter?


@fissban ¿Por qué el reward perdedor es de un sólo ítem? Es decir, ¿por qué se da por hecho eso?


Very nit: why 'wnaf' in the name of the function?


Attaboy


Thanks. This was a copy & paste job, so I'll need to fix it elsewhere as well.


Ye i had brackets too in my test branch


If you could make the subject a "squash!" one, that would be very much appreciated. That way, I won't forget to remove the respective commit during the next merging rebase. Thanks!


Look like you can refactor the following code out of the various `derivedExecuteCall` implementations:
```
self->preCall(buffer, execState);
if (execState->hadException()) {
    return JSValue::encode(jsUndefined());
}
```

and move it to `baseExecuteCall`.


You are right.


thanks @dzuelke!


批量替换么？是Tab不是空格


@mosic, I could not duplicate the error you mentioned regarding the boolean data, but reviewing the code I found and fixed the boolean vars initialization. Please test on your end and let me know if I can proceed ?


Can you use SECP256k1_SCALAR_CONST(0,0,0,0,0,0,0,0) instead? Makes it independent from the representation used by the implementation.


Is it necessary to `parseInt()` on every traversal? I guess this will be called per HTTP request, right?


Yep, there's just one } missing after the "break" (line 762)


不是不是

sublime project setting 里面加了：
"trim_trailing_white_space_on_save": true,

保存的时候就会去掉不小心加上去的无用空格，到广州用了 这个配置许久，没注意就提交了- -


So which value for the affinity width should I pick? Wouldn't cpu_max + 1 be just fine?


If it returns a hash, it's confusing to call it "point multiply". It's just doing ECDH, secp256k1_ecdh would be fine :)


Its basically country based
HM2014818- India
HM2014817- Singapore
HM2014816- Indonesia I think


Thanks


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


Would be nice to have more of a descriptive name here so that you can see what it's actually supposed to do. "Sending a message echoes it into the response" or something along those lines is nicer so that you can see how the functionality is supposed to behave.


这么棒，加到user setting里行么？


That won't work, since all the pathes inside those scripts are related to the 'pwd'.


Isnt this specific to the -cgi slave port?


Isnt this specific to the -cgi port?


Is that true though?

[fs.write(fd, data[, position[, encoding]], callback)](https://nodejs.org/api/fs.html#fs_fs_write_fd_data_position_encoding_callback)
> Unlike when writing buffer, the entire string must be written.

It looks like writing strings we don't have to worry about partial writes in the first place.

Introducing Buffer increases tsc's exposure to node by a great big chunk of the API. For a case of running in an emulated environment that's a painful hit.

At the moment tsc's dependency on node API is small and very mockable, but with Buffer it's gets really involved. Both in terms of code complexity as well as performance.


@xavijam :eyes: , please


I am going to :-1: this mainly because it is already possible to check the headers for this using the PSR-7 methods and this is something that you would want to check in your app when appropriate in middleware

```php
$acceptedHeaders = array('application/json');
$app->add(function ($req, $res, $next)  use ($acceptedHeaders) {
    $acceptHeader = $req->getHeader('ACCEPT');
    //do stuff.
    if (!iin_array($acceptHeader)) {
         $res->withStatusCode(400)->withBody("Incompatible media type");
    } else {
         $next($req, $res);
    }
});
```


@rafatower :eyes: , please


There will  be one new impl - based on Tinkerpop3 that is not backwards compatible with the current one.

And, frankly, I want to keep my options open if we realize later on that Titan and/or other Tinkerpop impl doesn't perform as well as we hoped for. This'd keep the ability to swap the impls out with for example SQL based impl for example which shouldn't be that hard to do.


貌似不行。。这个只是sublime的project配置。。

不知道怎么通用起来。。。


Just cpu_max(), which should be 64 as configured in the nova contrib sources in config.h


Just noticed, it "Sharry Stowell". Please update.


PROFIT


Doesn't this variable name throw an error?


Yes - I will redo this and go through the process.  And yes this is a good less why the process works.


Nice catch!


Thanks for the clarification. Intuitively, I expected cpu_max to be the highest CPU index.


Yup, that what exactly I was assuming.
And I've tried this script. The funny think that it was working :)
Anyway. I've removed this cd as well as the one above and tried the script again, it works.
@kevincon, pls, double check the script and try it your self. it's not difficult any more. You just need leptonica to download.
Actaully, we may even add a few lines to download and unpack leptonica inside the script. I've seen it's possible :)


Better: `Log.LOGGER.debugf(e, "Transaction failed due to: %s", e.getMessage());`

Otherwise, /me the developer would perhaps find it useful to see also the which attempt is that out of how many - therefore even better:
`Log.LOGGER.debugf(e, "Commit attempt %d/%d failed: %s", e.getMessage());`

BTW, is there a way for the developer to see what is being committed? Does `payload.toString()` show something meaningful? - that could be added too in some form.


Why does the Vector3i constructor use `{}`? We use `()` mostly if not all the time right?


Of course you're right ... I've reverted lots of my changes and add this.lastConfig instead - does the same:

https://github.com/Z-Wave-Me/home-automation/commit/e00525d867545382a2e4c9c077598b93b4999d05


What is the difference between lines 69 to 74 and this ?
view.ViewManager.on("search_done", self, function() {
    self.eval_tip(action_id, model, fields_view.type);
});

Also, are you sure to handle all cases that were previously handled ? Like the previous (fields_view.type === 'tree' || view.hasOwnProperty('editor')) and 'list_reset_tip' ?


If you have more comments please add them to the issue #21


Was the CMD ["logstash"] left on purpose ?


So the second contract in this file doesnt load and i really dont know why


Commentaire intelligent.

Enfin, surtout un test.


Unfortunately I am building the OS X binaries on an old Lion machine, here is the cc info:
````
$ cc -v
Apple LLVM version 4.2 (clang-425.0.28) (based on LLVM 3.2svn)
Target: x86_64-apple-darwin11.3.0
Thread model: posix
```

The error appears at the last linking stage of samtools_0.1.18:

```
cc -g -Wall -O2  -o samtools_0.1.18 bam_tview.o bam_plcmd.o sam_view.o
bam_rmdup.o bam_rmdupse.o bam_mate.o bam_stat.o bam_color.o bamtk.o
kaln.o bam2bcf.o bam2bcf_indel.o errmod.o sample.o cut_target.o
phase.o bam2depth.o -Lbcftools  libbam.a -lbcf -lm -lz #-lcurses
Undefined symbols for architecture x86_64:
  "___ks_insertsort_heap", referenced from:
      _ks_combsort_heap in libbam.a(bam_sort.o)
      _ks_introsort_heap in libbam.a(bam_sort.o)
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
```
Have no idea why it cannot find those symbols, using gcc instead works as expected so I did not try to fix it (I don't work on OS X and I have no experience with [old] clang quirks).


Great ! But we missed a column to do a click to call with the number :)


Do we need the "byGAV" in the name?


I know about pause and all that.. I did a quick fix.. sorry didn't finish!

yes great Idea... make it into a button but not reachable of course... maybe on the right!

good work.


PS didn't have a chance to look at next air will do later today.. i'm a little busy with life these days!


Great! I love it!


oh! i like it!!!!

good job.. let's polish it up but I already think it looks great.


I hate agar names too.  *shudders*


i thought to do the same :+1:  :P


Wait Unitl => Wait Until


Thanks Gabriel for your reply. For the moment, I've applied a patch by
migrating the method from the parent to the child class. I'll take a look
at the new class for sure. It just broke the client's website when LS had
updated at 3.2.9, but now its repaired.

By the way, this question may seems a bit odd, but where can we find the
documentation how to upgrade the webstore on a self-hosted environment (for
exemple, for our staging env)? Do we just copy an overwrite the new version
files over the old ones, or, there's an update tool? Looked over all the
wikis, but can't find any reference about the update process.

Thanks again.

On Mon, Jun 29, 2015 at 6:52 PM, Gabriel Guzman <notifications@github.com>
wrote:

> Hi @peteles <https://github.com/peteles> -- sorry about that, this
> function got refactored into the ProductGrid.php component as it makes more
> sense for it to live there than bundled with the general Controller. We
> don't have a deprecation strategy with regards to internal functions at the
> moment, but I'll bring it up during our next retrospective. Let us know if
> ProductGrid::createBookends() doesn't work for you.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lightspeedretail/webstore/commit/3984def4c6b2bbac84f7e3cde7a2ff39b8559f9a#commitcomment-11921067>
> .
>


@nyalldawson where's the backport?

Of course this must popup right after the release, again before the announcement and all packages being built. Although it's were just the LTR builds that were still running - but it's wasn't final-2_8_2 state anyway, but latest on the branch - ie. with your load of 2.10 backports and I just committed mine...
So there should probably be a 2.8.3 soon too.

Should we wait some more for feedback on 2.10?  People probably won't try qgis-ltr-dev / debian-nightly-ltr / ubuntugis-nightly-ltr either. :(


Unitl => Until


Newline eof


:+1: (wonderful how they change behavior :trollface: )


你现在每个项目都增加这个配置么？

On Tue, Jun 30, 2015 at 8:12 PM, linchao <notifications@github.com> wrote:

> 貌似不行。。这个只是sublime的project配置。。
>
> 不知道怎么通用起来。。。
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zhaoda/spa/commit/28ce1283c9454830093018d60478370148a23923#commitcomment-11930191>
> .
>


Well, [the doc](https://developer.gnome.org/gtk3/stable/GtkIconTheme.html#GtkIconInfo) doesn't tell me anything about GtkIconInfo suddenly becoming inherited from GObject since 3.8...


box-shadow 的值：
`x 方向偏移量` `y 方向偏移量` `模糊距离（视觉上表现为「纸片」和它的「父容器」的距离）` `阴影尺寸（一般不用）`


Why? What do you mean?


Feel free to create them. Thanks, Ned


Well, we have to know that it is a huge work to translate all descriptions in options.rpy and screens.rpy. But maybe one day I will do this.


It does!


Missing final newline. You can run `git diff --check` to be warned about this.


Does this check cover some extra properties, which I forgot? As for me, this is a bit less readable.


I guess new API is worth to be mentioned in docs, in `./docs/core.rst` file.


@hoholee12 left SH-OTA to finish this project... but i don't take help :/


@claudiosmweb can you cherry pick into 2.3 too?


The if is superfluous.


The name `remove_all` is inconsistent with other APIs in Mojolicious, something like `reset` would be a lot better. And the implementation is unnecessarily verbose, you could just as well use `sub reset { shift->pairs([]) }`.


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


`CGI` is a dead module, we do not reference it.


Looks like there even was a [ticket](http://hub.qgis.org/issues/12987) for it.


At the moment its pretty random. Some of us are moving over to using {}, whilst some are staying with (). We should probably decide on a style to use with c++11.


these two are *mutex_create* and *mutex_unlock*


ABU-757
- input width styled based upon component layout for full width style.
- input width styled if prefix is used.


在项目根目录添加 `.gitignore` 文件，写入内容
```
.idea
```
然后从 git 中删除 .idea 目录


:ok:


角色有点多，需要花点时间消化。


It doesn't crash if the collection of tokens is empty. Also, it relies on prettyprinting, which might be unreliable.


You should do your changed to the file in https://github.com/OpenRCT2/Localisation


@jef-n I don't think the original issue in that ticket is directly related - it refers to an issue in 2.8, and as far as I'm aware the offending commit wasn't backported to 2.8. But dash WAS reporting failures to us. It's just unfortunate that they got drowned in the noise of other failing OSX tests. I'm working on resolving that now...


sidali you are kavi


this looks weird.

return retrace::curPass + 1 >=  retrace::numPasses;

I don't like seeing -1 on an unsigned number :D


91958608DOASEffectOnZoneSizing (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1160 of 1166 tests passed)

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


OK thx for the review


Ditto


(done)


Bug when using GitHub local tool.


confidential file,  could be privately required.


Thanks for adding these filters. I'm excited to try them out.

I have one concern - that is the "Dry" parameter. I remember  being very confused about this when I first started using KDENLIVE. I would really hope that we could do  better in Shotcut. What is the possibility of replacing the "Dry" parameter with a "Level" parameter?

The Level parameter should be in units of dB
The Level parameter should be able to go both positive and negative (boost and cut).

Thoughts? I suppose you are getting ready to cut a release. So maybe there isn't time to figure this out. I won't have time before your next release to dig in. I might be able to make time next month to look into it.


ups, I wonder how that worked :) we should add use strict to tests :)


- Class added if prefix is displayed so text input field can have custom width.


I think you meant to replace the use of `parameters` with `materializedParameters` too, in order to get the benefit? ;)


This line's got a lot going on, maybe a good first step to resolving that'd be moving the use of `params` to its own method like `keywords` does (https://github.com/alphagov/finder-frontend/blob/master/app/lib/search_query_builder.rb#L68-L70)?


Dude, just move this one line up and return $options two lines above :_D


We should get only one edit tag, no need to keep "history" since that won't tell us much.


Already merged as commit #ebd4ebf


It does not compile.

[error] /home/dveim/github/scalameta/scalameta/src/main/scala/scala/meta/internal/parsers/Parsers.scala:1359: ')' expected but identifier found.
[error]       t.tokens match { case (_: `(` +: _) :+ (_: `)`) => true; case _ => false }
[error]                                     ^
[error] /home/dveim/github/scalameta/scalameta/src/main/scala/scala/meta/internal/parsers/Parsers.scala:1359: '=>' expected but '}' found.
[error]       t.tokens match { case (_: `(` +: _) :+ (_: `)`) => true; case _ => false }
[error]                                                                                ^
[error] /home/dveim/github/scalameta/scalameta/src/main/scala/scala/meta/internal/parsers/Parsers.scala:1360: ')' expected but identifier found.
[error]       lazy val isInBraces = t.tokens match { case (_: `(` +: _) :+ (_: `)`) => true; case _ => false }
[error]                                                           ^
[error] /home/dveim/github/scalameta/scalameta/src/main/scala/scala/meta/internal/parsers/Parsers.scala:1360: '=>' expected but '}' found.
[error]       lazy val isInBraces = t.tokens match { case (_: `(` +: _) :+ (_: `)`) => true; case _ => false }
[error]                                                                                ^

How about this change:
lazy val isInBraces = t.tokens.nonEmpty && t.tokens.head.is[`(`] && t.tokens.last.is[`)`]


For fixing a race condition bug.


yes but ive gone away from it now and learnt fragments from scratch and
will tackle this another time

On Mon, Jun 29, 2015 at 11:33 AM, Paolo Vagnini <notifications@github.com>
wrote:

> @martipello <https://github.com/martipello> did you create rootView?
> See line 104:
> View rootView = inflater.inflate(R.layout.fragment_main, container, false);
>
> (and then return it in line 110).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/udacity/Sunshine-Version-2/commit/1d424934d97c8c772978837c0eebb292cbc7f808#commitcomment-11907315>
> .
>


Useless now, as the code changed in 2.3.16 to use the first value under OID .1.3.6.1.2.1.47.1.1.1.1.11


@MarkusTeufelberger you may compile my fork, its nothing else than dolphin six days ago with this commit. Next time Ill pull upstream changes is when [PR 2675](https://github.com/dolphin-emu/dolphin/pull/2675) is pulled, its part of the 5.0 milestone. Or you may fork and pull this commit your self :)


path="/Users/rudi/...

Actually none of this has any effect with the new Eclipse except breaking the build, I guess.


Why? We don't have a special **zfs-dracut** package (although I have that in my native debs).


Forgive the drive-by comment, but shouldn't this be -= 360? so that if heading was 359 before, and then became 369, it would end up being 9 rather than 0?

It might be that the surrounding code makes this unnecessary.


The if modifier here seems inconsistent with the style guide since the body of the expression spans multiple lines


Inefficiency. If exists is set to true, we should break out of this loop.


You can write this as `trim(model.get('currentPage')?.get('title'))`


Wrong account, I was that :D


Hey, I tried to apply same spec to my monitor (slightly different model)
`cp -a ./db/monitor/DEL{4080,407F}.xml && make && sudo make install`
I get following in the output:
```sh
(plc)[dima@bmg ddccontrol-db-feuerrot]$ ddccontrol -r 0x12 -w 25 dev:/dev/i2c-5
ddccontrol version 0.4.2
Copyright 2004-2005 Oleg I. Vdovikin (oleg@cs.msu.su)
Copyright 2004-2006 Nicolas Boichat (nicolas@boichat.ch)
This program comes with ABSOLUTELY NO WARRANTY.
You may redistribute copies of this program under the terms of the GNU General Public License.

Reading EDID and initializing DDC/CI at bus dev:/dev/i2c-5...
Element value (id=standard) has not been found (line 48).
Element value (id=multimedia) has not been found (line 49).
Element value (id=movie) has not been found (line 50).
Element value (id=5700k) has not been found (line 43).
Element value (id=vga) has not been found (line 24).
Element value (id=dvi) has not been found (line 25).
Element value (id=hdmi) has not been found (line 26).
Element value (id=dp) has not been found (line 27).
Element value (id=off) has not been found (line 33).

EDID readings:
        Plug and Play ID: DEL407F [Dell U2713HM]
        Input type: Digital

Writing 0x12, 0x19(25)...
Control 0x12: +/25/100 C [Contrast]
```

Funny that I see the value you added in this commit `5700k` among not found.
Is that some sort of mismatch between ddccontrol and ddccontrol-db, a mismatch between monitors or did I just do something wrong?


This is not yet complete.
I think we would have to handle the case of PMA_DRIZZLE separately as it does not have the `mysql` db at all.


a typo in unknown


Is this `if` modifier conditionally rendering the partial or conditionally assigning the `locals`?


Python 2.6 support has been dropped : https://github.com/wackou/guessit/commit/7bbf9726b3a5009ccb942e5a8579ab3bad5b1950


After looking at the method bellow I thought this one would be also using an Assert.checkNotNullParam("map", map)


See my comments on the PR diff.


Paoloo,

Could you try compiling on linux now?  I've patched the official premake scripts.  Just do a clone and run the rebuild script.

Jim


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.315](http://teamcity.monogame.net/viewLog.html?buildId=8521&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 501, ignored: 6 Build time: 00:22:59


r+


Reminds me that we'd perhaps want to add a section for "Cluster tools" in the index.


If having a space is more easy for your script than having a dash, we could choose for that. I can easily change the add_sources and report_sources


Maybe above splitting up "Networking" into "Cluster" / "Cluster tools" / "Networking" on the TOC?


这样输入框就被预设上当前日期了，一般的需求是希望留空的。


Could you please be more precise, which race condition? Also why did you comment out out_standby() which is needed for Bluetooth audio connections.


This might be more readable as a private `paginated?` method?


vendor = meta.vendor


@hoholee12 the position is alfabetic, many love to you (L) xD


Maybe this giant method could be broken down into a few smaller (private?) methods?


we don't use that elsewhere, so why adding it here?


好。


I'd like to dimension the affinity space according to the actually present CPUs rather than based on NOVA's configuration. E.g., as a user, I would expect that if I divide the entire affinity space into two halves, that each half would contain the half number of CPUs. Hence, I think it is sensible to base the width of the affinity space on the index of the highest enabled CPU. See the patch https://github.com/nfeske/genode/commit/167380d3118fb04fa1d6f08daeec97dd7118d9c1 @alex-ab Do you agree?


@blundell Because we should use it everywhere :+1:


Fixed in 69ffe62a790f21746a3f9ccbce6fe0f8f729b8e2


Fixed in 524357c484801d8717a864d70ac355c1436e909a


Maybe `[[]]`


@nyalldawson if that was actually intended.  2.8.2 was the default for new tickets.  I removed the default and now it appears with _Please select..._


I understand.

Me, I don't like maven.
In fact, it dampens my mood so much that I'm working on a maven less version of ninja right now (with `pom.xml` replaced by a simpler yaml file - like play1) and a command line tool that would perform `ninja run`, 'ninja package` etc.

```
# Project
  name: glog
  dependencies:
    # Ninja
    - org.ninjaframework:ninja-standalone:5.1.3
    # Ninja modules
    # Application libraries
    - org.webjars:jquery:2.1.3
    - org.webjars:bootstrap:3.3.4
    - com.h2database:h2:1.4.186
```

Note that changing from standalone to servlet mode is just a matter of replacing the `org.ninjaframework:ninja-standalone:5.1.3`.

I love how open source let us to scratch us our personal itch.

Good luck with mangooio!


has_many :speakers <-- plural


in docs it's double

```
``DistributedPubSubMediator``
```


has_many :events


I have been thinking quite long about that, and the idea is very tempting. The initial PR more or less did this, but not to it's full extent. It merged the **dracut** and the **initramfs** parts because there is _a lot_ of duplication between those two and the **init** scripts.

How about a two stage process - I'll include the zfs-functions.sh script and remove anything thats in that from the initramfs script and use code from zfs-functions.sh as much as possible. I don't move code FROM the initramfs script TO zfs-functions.sh just yet. Only remove duplicate code from **initramfs** and use zfs-functions.sh code where they exist.

Then, after the **0.6.5** tag, I start moving stuff that's both in **init**, **initramfs** _and_ **dracut** into zfs-functions.sh and we use that for all three parts? For example, the import code (etc/init.d/zfs-import.in:do_import() and somewhere in the middle of the huge initramfs/scripts/zfs:mountroot() function) is now almost identical between the **init** and **initramfs** (something I quickly throw together before I pushed this last commit). It needed to be done, because the old **initramfs** code depended on the old USE_DISK_BY_ID, which is no more.

 The **dracut** code could benefit from more … 'common' code with the other two. That way all three parts, that does pretty much the same thing (code vice at least), actually **DO** exactly the same thing.

I _also_ propose (in this second stage) to move both the **dracut** and **initramfs** dirs into the contrib/ directory. Neither of those is really, really _part_ of ZFS or ZoL. It's "contributed code" to simplify to/for users...


Thanks for sharing!


You also need to tie users to the created events.  You can add user_id to events to signify which user created the event.

Then, in this model, add has_many :events


Cge ate. Dmi ko pla mali hehe. Ung login at sign up device di ko pa pwede
gwn ate?

has_many :events

—
Reply to this email directly or view it on GitHub
<https://github.com/MichelleNonato93/Agorae/commit/a804bb0b8113f574ce0d3616a898b052818d13fb#commitcomment-11930949>
.


Please add your validations inside the model.  Read more on validations here: http://guides.rubyonrails.org/active_record_validations.html

Score for this exercise: 90%


lgtm


Cge ate anu next na ggwn bukod sa wireframes?
On Jun 30, 2015 9:02 PM, "Maricris Nonato" <notifications@github.com> wrote:

> Please add your validations inside the model. Read more on validations
> here: http://guides.rubyonrails.org/active_record_validations.html
>
> Score for this exercise: 90%
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MichelleNonato93/Agorae/commit/a804bb0b8113f574ce0d3616a898b052818d13fb#commitcomment-11930973>
> .
>


Le 30/06/2015 13:31, jsanda a écrit :
> We need the endpoint. But first and more importantly, changes are needed
> to the task scheduler.

OK so it' doesn't exist. I wasn't suggesting it should be added in this PR.


add author such as:

```
author: cescoffier
```

(was not documented, just added the support ;-)


This commit could probably be cherry-picked straight to master, rather than having it require the conversion branch to be merged.


JavaEE instead of J2EE


If you change url to `git://github.com/FFmpeg/FFmpeg.git` instead, it will make the folder browseable on https://github.com/Microsoft/FFmpegInterop. The official mirror always stay in sync. Currently it is targeting https://github.com/FFmpeg/FFmpeg/commit/620197d1ffea20e9168372c354438f1c1e926ecd (16 days old, while their current master is at https://github.com/FFmpeg/FFmpeg/commit/5c047f3ad49c8a8a5ae126d134b1004a4dcbc39b). :)


Thanks for the review.
1. I believe the `_x` `_y` etc is only for .csv serialization. As far as I can tell this b91d2ea5d73dcaebbe9f47124772972a85168a88 tweak gets it right, at least for the case I'm interested in: skipping the detector when a ground truth face detection is provided.
2. I looked at classifiers/cascade.cpp and it wasn't immediately obvious where it would go?


I would replace IoT by _small_


That seems a bit in-elegant - but I see that you later removed this again.
One could argue that you should clean up your commits and not add and then remove again an argument like this, but Subsurface doesn't have strict rules about things like this and I won't view this as a reason to reject these commits


Given that the fake heading goes up 10 by 10, I don't think anything more complicated is necessary.


white space looks rather weird...


I understand the desire. Your commit does not help by such a case - cpus enabled: 0,1, 2, 3, 7,8 . (It's artificial anyway).

I think that you should keep the line with "_cpus = Affinity::Space(hip->cpus(), 1);" in the platform.cc, which exactly gives what you want, the number of available cpus for the affinity space. What is missing, is that the Genode affinity cpu numbers get converted back to nova cpu numbers in the cases where the syscall create_ec and assign_gsi are used in base-nova.


The first proposition was not exactly what I meant. It was more like that :
```
cmake/
|_ toolchain/
  |_ avr/
  | |_ atmega128.cmake
  | |_ atmega2560.cmake
  | |_ atmega32.cmake
  |_ arm/
  | |_ stm32f4.cmake
  |_ avr.cmake
  |_ arm.cmake
  |_ generic.cmake
  |_ sasiae.cmake
```

But your second proposition seems quite fine to me. May be with some modifications :
```
cmake/
|_ toolchain/
  |_ generic.cmake
  |_ target/
    |_ avr.cmake
    |_ avr/
    | |_ atmega128.cmake
    | |_ atmega2560.cmake
    | |_ atmega32.cmake
    |_ arm.cmake
    |_ arm/
    | |_ stm32f4.cmake
    |_ sasiae.cmake
```
The arm directory may be modified to separate "cortex-m3", "cortex-m4", ... Or separate constructors ("st", "atmel", "freescale", ...). But it is not important currently.

Any way, the difference between our propositions is to add the directory `target`. My question is : Is it useful to add the directory `target` if it only separates `generic.cmake` ? I think both are ok.


@pandalion uh, to stop overflow :trollface:


Might be nice to have a screenshot of the UI.


well now I feel special :-)


Why use a String here for the type? What if it is null?


They uses the same rom. Only difference is in the borad id. MIUI's init has a code to override values based on the board id. Just dont care about it. :)


I can't remember, soz. But it was likely for good reason, as is everything I write!


;D


d


@Mazda LOL..its still funny after all this time. LOL


this is becoming a pattern. One commit adds something, the next commit removes it.
This really indicates that maybe your commits SHOULD be refactored and combined.
Sometimes there is value in having the sequence of steps in the git history at times... but this doesn't seem like one of those cases. I leave this for Robert to decide...


Do we need this method anyway?


Maybe changing that will speed up the code a little bit. Let me know so that I push the new code (also with corrected typos).


@gokul1794 ah, I see now that we have a css rule for `.zci.is-active` which is taking precedent.

You'll need to keep it as it was then, sorry for the mixup.


This should be 1e-9! Fixed.


shouldn't this read list.add(idx, value)?


Personally I don't this adds much value to the test.


Thanks! I have a Lion or Mountain Lion machine at home and will try to reproduce this.


so the units were wrong on some of these? But two seem completely different.
Again, is this something that should just be fixed in the first commit? Or are there corresponding algorithmic changes?
The commit messages are overall a bit too sparse for me. They don't explain how you are doing things, just bullet style what the outcome might be


*I don't think this adds much value to the test


add 'downgrade' and autolock


at least the white space mess is fixed


A small issue: iio_device_get_poll_fd( ) is not defined in iio-private.h, so I get "implicit declaration" issues when compiling this branch.


Thanks for the info, I was hoping that this gets merged upstream sooner or later rather than having to maintain a patch/fork. After a quick glance it only seems to extend dolphin's capabilities and bring them on par with the Windows version, so I doubt it would be controversial. Is there anything missing?


Won't this always be called after the first handler on the very first frame of every connection, effectively closing the connection immediately?


Ah shit indeed I meant to change that import. thanks. How many times can I fuck this up?


-1 for returning a `String[]` - it cannot be made immutable.


@mosic, script now creates new vendor/default_model if does not exist already before starting data import


Overall this commit is a bit confusing. It seems to combine several changes that are fairly independent from each other. I'd much rather have one commit with just the change in the constants and an explanation WHY. Then a commit that adds the vpmb_start_gradient() function and explains how it is used.
And then a commit with the algorithmic changes that explains what was changed and why.
There is no way that YOU will look at this commit in a year and it will make sense. And certainly it will be even more impenetrable for Robert or myself.


Good first step, thanks Sascha!


This release contains batching as well, so should this be `0.1.0`?


Yet another completely unexplained change to the constants


I think, there's also a lot of gore missing in WoW.


Ok. Now I got it.


Overall comments are lacking in the code.


これも結局は「不安がなくなるまで」としか言えないのですが、例えば今回のケースだと

- 「100の倍数年はうるう年ではない」という事実を他のチームメンバーが知ってるか
- 「400の倍数年はうるう年である」という事実を他のチームメンバー知ってるか

が論点になりますね。あとはそもそもそこを変更する可能性がありうるか。例えばこのケースだとうるう年の定義の問題なので変更可能性は著しく低いからテストは薄くてもいいっていう判断もありです。（とはいえそれだと課題にならないので一応細かめに書いて欲しいんですが :sweat_drops: 実務においては適切に省くのはアリです。）


If this will typically be called per HTTP request, then I am -1 for returning a new array instance.


If this rescue really needed? If the code reaches this point it's already closing down, either because an error was raised or because there are no more frames.

I also believe the error raised is lost from the thread value, so any other thread that has #join'ed this one won't receive the true reason why the thread exited.


No explanation here (as comments) or at least in the commit message


English ordinals are a little different than in the code. I stuck some off-the-shelf ordinal method into the English language generator to correct it.

– Joe


things like this should be explained in the commit message.
in my mind this change alone might be one commit with explanation


reack


As far as I understand @Tilka wanted to Look into doing a different
implementation of this feature. Or he might have forgotten :-P
On 30 Jun 2015 3:14 pm, "MarkusTeufelberger" <notifications@github.com>
wrote:

> Thanks for the info, I was hoping that this gets merged upstream sooner or
> later rather than having to maintain a patch/fork. After a quick glance it
> only seems to extend dolphin's capabilities and bring them on par with the
> Windows version, so I doubt it would be controversial. Is there anything
> missing?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Tilka/dolphin/commit/6a4edcabe2b7114315f9285828ff3b76f9b05546#commitcomment-11931194>
> .
>


「場合分けをテストする」という感覚がそもそもホワイトボックス的なので TDD とはちょっと違うんですが、厳密な TDD をやるとそもそもテストコード書かないと場合分けするプロダクトコード書けないはずですね。


> On Jun 30, 2015, at 14:45, Volker Stolz <notifications@github.com> wrote:
>
> path="/Users/rudi/...
>
> Actually none of this has any effect with the new Eclipse except breaking the build, I guess.
>

Oops, fixing...


This commit is a fairly massive rewrite of the core algorithm, sprinkled in with a bunch of other, seemingly unrelated changes.
And no commit message at all.
That is not acceptable


I just changed the patch to https://github.com/genodelabs/genode/commit/f5e9a920f3e3b58fc1200ee1197d36879acddd9b


Oh you didn't release a new version when that was merged to master?


Finish the wireframes until Friday. Put up validations on the models.


Again, completely lacking explanation in the commit message. So what changed? And why?


Allright, let's go for it.


Fixes issue #22.


a new car list starts here


You're missing a line of commented LOGW.


@sotch-pr35mac Do we have to make a new project name unique?  If I have a private project, and you have that same private project, don't you think that the name could be the same?  The only difference would have to be the project ID.


I think we need this because in the rule-file the level is defined
https://github.com/forge/docker-addon/blob/master/sample_rules.yaml#L136.
Usage:https://github.com/forge/docker-addon/blob/d2dbef0b1d1366142d792cfddd3dbdab97fc808e/src/main/java/org/jboss/forge/addon/docker/validation/DockerfileValidationImpl.java#L268


Shouldn't this be in `setElementDirty`, a couple of lines below?


Conceptually this is the right way to do it. In the preferences, with an enum.


and again, a constant added earlier gets removed


あとこのへんは「パラメタライズドテスト」と呼ばれる方法があるのですよ。


За следващия път пускай всяка задача на отделен commit за да може да merge-вам нещата, които са хубави, а останалите да ги поправяш : )
Strings to integers - направо ползвай read тук.
За concat пък, може направо да конкатенираш с (++), в случая не пестиш нищо с (:) защото го извикваш за всеки елемент в списъка.
За reducer правиш дооста излишни обхождания по списъка, трябва да стане наведнъж.
Оправи задачките за които ти коментирах, за да ги merge-на и да кача аз останалите : )


If the string is null (Other than the enum types) the conditional clauses won't execute.


Clearing of JobExecutionContext.Current should occur within a finally block.


Thanks, I missed this obvious stuff.
You'll have to stop being awesome ;)


I would prefer that the coding style / whitespace changes were done by fixing the earlier commits


What's the point on checking if the item in hand is null? won't that prevent player's just normally right clicking it with an empty hand?


Well not really I am a bit intimidated by C++. This just came as a
byproduct of the code structure and I thought it was interesting.

On Tue, Jun 30, 2015 at 3:10 AM, Toby Dylan Hocking <
notifications@github.com> wrote:

> If you want to use C++ instead of C that is fine too.
>
> On Tue, Jun 30, 2015 at 5:30 AM, Ishmael Belghazi <
> notifications@github.com>
> wrote:
>
> > @tdhock <https://github.com/tdhock> I though you might like this. This
> is
> > really starting to look like an object, just mentally replace &trainer
> with
> > "self" or "this". Beyond aesthetics and having fun, I think this
> interface
> > will make thing easy when we'll move to passing arbitrary C or R function
> > to SAG. It would be as simple as defining a new model containing pointers
> > to the relevant function while keeping all the remaining code unchanged.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/IshmaelBelghazi/bigoptim/commit/80cff82a26c65d073eb474c54ff2be0cd7943661#commitcomment-11923704
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/IshmaelBelghazi/bigoptim/commit/80cff82a26c65d073eb474c54ff2be0cd7943661#commitcomment-11925398>
> .
>


Can't think of a befitting test ATM.
Could you please suggest one.


hmm not sure if it was necessary anymore... an activity could have several components using different adapters... I'll revert this commit


Much cleaner!


deciding on if I should upgrade again for this feature, but can you clarify what was missing that was just restored please?


Redundant, no?


And you should'nt be allergic to Visual Studio and ASP.NET MVC.


Are you sure it will not throw a NPE?


Thanks.


Sounds reasonable. Do you know of anyway to profile R C extensions?

On Tue, Jun 30, 2015 at 3:04 AM, Toby Dylan Hocking <
notifications@github.com> wrote:

> I think it is fine to have lots of in/out arguments, as long as you add
> comments in the C code so you know what each of them are. You can use a
> struct if you think it will simplify things. I would recommend making
> design choices based on speed if you can (less reads/writes/function calls
> is usually faster). Also I talked to Mark yesterday and he said that out of
> the 3 versions of the SAG code the most complicated one is the fastest
> (*SAG_LipschitzLS)
> so I would recommend using that as the basis of your L2-regularized GLM
> implementation.*
>
> On Mon, Jun 29, 2015 at 8:23 PM, Ishmael Belghazi <
> notifications@github.com>
> wrote:
>
> > @tdhock <https://github.com/tdhock> So I started modularizing the code.
> I
> > broke-up the main SAG loop in an outer loop responsible that will call
> > _sag_constant_iteration at each visited example. I am a bit bothered by
> the
> > function above, it just seems really messy. It has in arguments, out
> > argument and it returns the nCovered variable that SAG updates after each
> > iteration.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/IshmaelBelghazi/bigoptim/commit/f5d2d3827e6b93182281dce1ebbcdbf0d682c466#commitcomment-11916057
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/IshmaelBelghazi/bigoptim/commit/f5d2d3827e6b93182281dce1ebbcdbf0d682c466#commitcomment-11925318>
> .
>


Sweet! Selecting by Regex! :+1:


Please leave the parsing in DockerFileValidationImpl only. I don't think this method adds any benefit to this class


No, we haven't made a release with that yet.


It will respond all the time with an array, whether it is empty to not. Because it's AJAX, template then should show a message if it sees that an empty response has been returned.

Another additional solution, would be to hide selects on the load, leaving just project select, and than use, for example, slideDown() from jQuery (http://api.jquery.com/slideDown) to show category select, when category is selected, to show the next select, etc.


Ah true.Fixing it.Should I remove the method all together and make changes in the linter impl ?


Yes, but nothing official. I changed jobs about six months ago-- so my next move is to ask OWASP for an official capacity in regards to PHP ESAPI. I have OWASP organization rights because of my official capacity in the Dev Guide project. I will do some research and see who I can get as a second set of eyes in the mean time.


This commit is useless. ;)


Should this be 800px?


@JAVA777 See https://github.com/freegate-release for free vpn.


Remember to correct this :)


@Sunilkumarr  code not formatted


Don't think so, because libass4 is not available on Jessie and libass5 not on Wheezy. Kodi linked against libass4 can't work with libass5 and vice versa. libass4 and libass5 have different libraries, that's the dilemma


Do you mean line 7 and 8 below? This is now possible, because the retrieval of the element in charge depends on who triggered the event. So this is right there where it belongs. Or did you mean something else?


Are the extra parens added in this commit necessary?


@truongmkevin, this is the pdf version of the illustrator file i created last week. you should be able to pull this in to your program as a vector and modify it from there.


No need for this, we can push the existing translations to crowdin to save some effort


I'm not that good with CoffeeScript, but line 135 looks like the Callback is mandatory. Is it?


Keep EOF newline.


Oh, yes. Thanks


I think we could go without the `target` directory like in your proposition if we create a `sasiae` to move `sasiae.cmake` within so it sits in a subdirectory like any other targets and is not next to the over "factorised" cmake files. We should even call this subdirectory `simul` (for simulation) so it has a more generic name and we could eventually add more simulator support later on.

I think I will go with your first proposition in your previous message.


That fails, the template line (template<typename T, std::size_t Size>) needs to have a compile time known "Size" value. Any version of a runtime variable fails. Example, this works because the size is known at compile
```c
int foo[1];
misc::MurmurHash::hashCode(foo, 1);
```


OK, no need to change anything of there should be only a single Inventory implementation at any point in time.


Hey, sorry for contacting you this way, do you have, by any chance, interest in giving up this account? We would be interested in the account name for our foundation.


jQuery word match, no regex, but yes :)


I think I found the cause of the problem (no solution though):

While we're instantiating a new `Map` of people and passing it to a new `Server` instance for each test, the HTTP requests (2 `GET`s, 1 `POST`) executed in the tests are always hitting the first instance of `Server`. I don't understand all the ins and outs of `createServer` and how the framework is dealing with the HTTP requests but they are not hitting the correct servers.

Here is what I did to figure this out:

```java
        get("/", (req, res) -> {
            System.out.println("-- GET  on " + System.identityHashCode(this));
            Map<String, Object> map = new HashMap<>();
            return new ModelAndView(map, "index.mustache");
        }, new MustacheTemplateEngine());

        post("/shouts", (req, res) -> {
            System.out.println("-- POST on " + System.identityHashCode(this));
            Person user = people.get(req.queryParams("name"));
            user.shout(req.queryParams("message"));
            return "shouted to someone in " + people.hashCode();
        });
```

Also, if you only run the `postShoutsMessageFromTheGivenUser` test, you'll see it is passing as the server instance is the right one.


Muy bueno! Ya estamos 1.5x mejor. Me juego las pelotas que podés llegar a muuucho más en CPU single-core.
Otra cosa, anotá con el hash de commit la versión que estás benchmarkeando. Por ejemplo: 5c519269742e19824321fd69eeda94c8606274dd


I think the id and label's `for` attribute has an extra `$*_checked` in it:
`id='left-$revision->ID'$left_checked` should probably be `id='left-$revision->ID'`


Looks good, but I’d like to fix this slightly differently.

I’ve created a Phabricator task for it — let’s track this there: http://phabricator.freedesktop.org/T122


No, arguments in Javascript are never mandatory, they are always optional :)


Same here.


Hi,
did you install another version of ddccontrol previously? The "Element value has not been found" errors sound like ddccontrol is using a different version of the options.xml file.
I'm using `./autogen.sh && ./configure --prefix=/usr && make && sudo make install` to build `/usr/share/ddccontrol-db` from this git repository.


BCPC-Headnode set back to previous version and BCPC-Headnode-Ephemeral added.


"we assume that our data object" -> "This assumes that the data object has a method getOwner() [...]"


там есть и GPLv3+ и LGPLv3+


OK, I will implement this in a separate commit, I'll just amend my commits and push to upstream


It gave a lot of errors, libuv is not present(perhaps you forgot to add *).

paolo@daath:~/Public/dfg/node9-master$ ./rebuild.sh
sh: 0: Can't open autogen.sh
./rebuild.sh: line 1: ./configure: No such file or directory
make: *** No targets specified and no makefile found.  Stop.
rm: cannot remove ‘.libs/lib*.so*’: No such file or directory
==== Building LuaJIT 2.0.4 ====
make -C src
make[1]: Entering directory `/home/paolo/Public/dfg/node9-master/LuaJIT-2.0.4/src'
DYNLINK   libluajit.so
LINK      luajit
OK        Successfully built LuaJIT
make[1]: Leaving directory `/home/paolo/Public/dfg/node9-master/LuaJIT-2.0.4/src'
==== Successfully built LuaJIT 2.0.4 ====
make[1]: lib9.make: No such file or directory
make[1]: *** No rule to make target `lib9.make'.  Stop.
make: *** [clean] Error 2
./rebuild.sh: line 7: premake5: command not found
./rebuild.sh: line 8: premake5: command not found
==== Building lib9 (debug) ====
make[1]: lib9.make: No such file or directory
make[1]: *** No rule to make target `lib9.make'.  Stop.
make: *** [lib9] Error 2
paolo@daath:~/Public/dfg/node9-master$


Yep, that's what I meant. Thanks for the clarification!


"Download" is a bit misleading here.


урл некорректный


Fixes the issue that was caused here:

Jun 30 13:18:12 ip-10-0-1-131 twitter-listener: AttributeError: 'NoneType' object has no attribute 'strip'
Jun 30 13:18:12 ip-10-0-1-131 twitter-listener:     stream.userstream(replies='all')
Jun 30 13:18:12 ip-10-0-1-131 twitter-listener:   File "/var/lib/authorbee/venv/lib/python2.7/site-packages/tweepy/streaming.py", line 379, in userstream
Jun 30 13:18:12 ip-10-0-1-131 twitter-listener:     self._start(async)
Jun 30 13:18:12 ip-10-0-1-131 twitter-listener:   File "/var/lib/authorbee/venv/lib/python2.7/site-packages/tweepy/streaming.py", line 346, in _start
Jun 30 13:18:12 ip-10-0-1-131 twitter-listener:     self._run()
Jun 30 13:18:12 ip-10-0-1-131 twitter-listener:   File "/var/lib/authorbee/venv/lib/python2.7/site-packages/tweepy/streaming.py", line 255, in _run
Jun 30 13:18:12 ip-10-0-1-131 twitter-listener:     self._read_loop(resp)
Jun 30 13:18:12 ip-10-0-1-131 twitter-listener:   File "/var/lib/authorbee/venv/lib/python2.7/site-packages/tweepy/streaming.py", line 298, in _read_loop
Jun 30 13:18:12 ip-10-0-1-131 twitter-listener:     line = buf.read_line().strip()


why don't you use `$this->isGranted('view', $post)` or `$this->denyAccessUnlessGranted('view', $post)` ?


Пофиг


I know, the line tells apt to install one of them.


嗯。。明天发给你。

插件加配置，非常爽


why not get the visibility of the note (`.is(':visible')`) instead?


If you want to automatically determine the size from the array argument, you'll only be able to use arrays with a size known at compile time. If the size isn't known at compile time, you'll need to add a third argument to this method containing the size.


не хватает
```
BuildRequires: pkgconfig(capstone)
BuildRequires: pkgconfig(openssl)
```


не нужно


This isn't right.


`%{name}%{?isa} = %{version}-%{release}`


I see it here. Yes.


Maybe add a test to check that `input_name` isn't part of the output? Just to make sure we indeed rename and do not simply copy the values.


%autosetup


Thx


Really? I didn't knew this. So I will change this for RPI and IMX6


The source code for the original English spelling is here:
https://github.com/fritzing/fritzing-app/blob/bd779a5b3a799582efc8a04a30962e87c7b93196/src/items/partlabel.cpp#L455

It should be translated, since it's encapsulated by the tr() function. Maybe try using the same special character notation as done here.


okay, i see. please look at the next screenshot from my webstorm debugger:
![this error is not an error](https://cloud.githubusercontent.com/assets/13100598/8432144/7ece2aee-1f3e-11e5-83d6-91696a34f49b.PNG)


```
%license COPYING COPYING.LESSER
%doc doc/* README.md
```


Your changes in this commit add a lot of useless new lines in the console log(specially at the end after session aborted). Is that really what you want ? Here is a before/after screenshot.
![beforeafter](https://cloud.githubusercontent.com/assets/9955284/8432123/17fd632a-1f0c-11e5-8a1d-24ab90a00058.png)


This seems a bit dangerous and not really generic. So far, it seems to work for all of them so let's not dig too much into this. However, bear in mind that the proper design would be to have Metric fully virtual and then having a reimplementation per backend (AMD_perfmon_Metric for example).


You need to check `self.object.canUserAccess(session.user)`.
Also, EventNoteBase is for modification so you probably want to inherit for a different RH.


Yes, you are right.


добавить в `%install`: `rm -f %{buildroot}%{_libdir}/*.a`, а отсюда убрать это говнецо


Yep, see latest commit, I fixed the old variable name to the current


нет newline


r+


I'm not sure what happens, can you please investigate a bit?


箇条書きの最初に不要なタブ文字が挿入されている


A little comment here to tell you are checking if the id and gid and valid would be appreciated.


собирать с:
`--with-syscapstone`
`--with-syszip`
`--with-openssl`


ヘッダを表す記号はタイトルの文字より長くないといけない。マルチバイト文字の場合いくつ入れればよいかよく分からないのでとにかくたくさん書く


Network and Security capabilities?


What is this glGetError() for?


Yay!


Hi @antirez , `jobReplicationAchieved()` may have a side effect that it may free the job j.
And `acknowledgeJob()` may access freed job. :-)


@vkarpov15 Why aren't you using semver ranges? e.g. `"mocha": "~2.2.5"` See also http://semver.npmjs.com/


リンクが一度しか出てこない場合はテキストに直接埋め込む方法を採用する方がよいと思います。GitHub レポジトリのURLを rokkopy のレポに変えておきました


слишком короткое описание:
```
The %{name} is a reverse-engineering framework that is multi-architecture,
multi-platform, and highly scriptable.  %{name} provides a hexadecimal
editor, wrapped I/O, file system support, debugger support, diffing
between two functions or binaries, and code analysis at opcode,
basic block, and function levels.
```


remove this.


通し番号付きの箇条書き


не уверен, что все эти библиотеки нужны, проверь. собирается ли без них?


can you make comp.pre configurable? Most I am using comp.pre.jbalcas


Super! With this, you can even `.include(:nested_elements)`.


edty ert erty ert yerty ery er


Also arch configurable?


erty erterty e


Method should be renamed to renderSearchTemplate


нет, не пофиг


Thanks Zach! Do you have more on your mind?


Hmmm yeah missed that. I'm trying to write a test to verify that it doesn't close after one frae


You added your instructions in the wrong place, now the command to set a custom server is at the bottom instead of before the Dokku instructions.


если там есть питонятина, то нужно ещё python-devel в BuildRequires. надо смотреть что там за бидон используется, от этого зависит python2-devel или python3-devel


@FHenry  is it your module Lead/affaire that is being integrated ?


Actually removing the padding creates an issue when you use borders for blocks. This is why we moved it to the theme. Any suggestions to solve this otherwise?


Is 777 really needed?


This actually removes all the logic that used `accept_empty`. So far a request to the locate API's without a body (for example a get request) was fine, but doing the same to the submit API's returned a 400.

I'm fine with changing this behavior, but then we should just remove the accept_empty flag everywhere and change the `test_error_completely_empty` test to do a self.app.post(self.url, '', status=200).


@certik This is the c wrapper code for `Basic::get_args()`. As you can see it is very similar to `basic_free_symbols`. I have tried printing the values in `(*array)` inside the function, and they are correct. But the [test](https://github.com/abinashmeher999/symengine/commit/b32926674800476176d23200439c7e66c9324b26#diff-8491b7e47a679403840abc38c4e23cf4R75) fails throwing a segfault, whenever I try to access `basic_array[i]`. I have tried comparing the pointer that is passed to the function and that is used in the function. They are the same. My only guess, after the function returns, the data at pointer `(*array)[i]`/`basic_array[i]` is being freed. That might explain why this shows correctly inside the function but not in tests.


Yes, you have to create the tuple with the inner set of parens, then add it to the items. I tried to remove them thinking they weren't necessary and broke all the navs locally.


What ? I don't understand your explanation.


What's up with the APP_KEY? Might be best to change it to something like `SomeRandomKey ` so that users know they have to change it.


Is this your actual db password?


Ach! Sorry, there is an underscore after the close brace: `[1]_`

One more push!


What happens if I remove the hidden in the DOM and submit a comment ?


Hey Sascha -- yes, this would relate back to https://github.com/uikit/uikit/pull/948


Thanks, sorry, I missed that Zhang-Suen was already a non-reference in the existing docstring.


http://www.oxforddictionaries.com/definition/learner/sweep --> past tense is `swept`


Why did you remove this test?


Looking again, it seems that this commit hasn't been merged. @thomasf , did you submit a pull request for this? If not, you have my most eager encouragement to do so, as I find issue #189 supremely irritating and am very appreciative that you wrote this up.


Don't worry man, I'm just using the nice people that I can buy beer to apologise to as guinea pigs in an automated PR contributor assessment system :)

You may unsub (or you can watch the future unfold before YOUR VERY EYES!!!)


Is it checked in by mistake?


I did, but that was not the cause.
Cleared everything out
Checked options.xml was gone
rebuild ddccontrol-db, installed
rebuild ddccontrol-0.4.2, installed

same "not found" items.

```sh
(plc)[dima@bmg ddccontrol-db-feuerrot]$ grep -r 5700 .
./db/monitor/DEL407F.xml:                       <value id="5700k" value="0x0B"/>
./db/monitor/DEL4080.xml:                       <value id="5700k" value="0x0B"/>
./db/options.xml.in:                            <value id="5700k" name="5700K"/>
```

Looks like `options.xml` is not re-generated from `options.xml.in`


NEAT!
Now let's use it everywhere.
However I wonder if we would rather move it to the translation package to allow others to use it as well. Thus eventually getting rid of the globals array access all over the place.


ok lgtm, the exe confused me thought :-) sounds like some windows thing... git_command ?


Por favor não use palavrões.


Why?


No, randomly generated for the README. When you run ```mariadb:create``` it will generate a password.


gore? yeah you are right :D


The previous three lines are redundant, I think. You don't have to ask twice if element's folded.


We are not allowed to compile the "operation" project ?


I figured it out.
it's a side-effect of `git clone` -- all files had same timestamp.
I did `touch db/options.xml.in` and re-built.

Now I have correct `options.xml` system-wide.


Yay!!  :smiley:


As we have discussed, to my mind the \0 is part of the data in this case, it's part of the string and we are converting all data. For now, the idea of the option sounds good enough.


fdddddddddddddddddddddddddddddddd


Just curious here: why do we need to cast to string?


`<%==` ? One `=` more than necessary, or am I missing something?


> _yada yada_ **BEER** _yada yada_

You were saying?


test guthub 1


잘했어. 희수야.
근데, $로 li을 만들고 html을 다시 만든 이유는 뭐니?