Source/DOM classes/Unported or Partial DOM/SVGImageElement.m

Apple's analyzer in xcode 6.3 is claiming this is an over-release.

I am inclined to agree.

It looks like the intent wa that the UIImage's internal CGImage be replaced?

1. We created the UIImage (masked as ApplNativeImageRef to be compatible with OS X) on the fly here
2. We end the method by pulling the raw CGImage out of that UIImage and using it -- or by using a different CGImage we created - and we ignore the UIImage we started with.

Seems that instead of over-releasing the CGImage, we should be shoving it back into the UIImage, and then at end of the loop using whatever CGImage is now inside the UIImage (whether original or new).

Um?


this can be
return Iterables.getOnlyElement(inputProperties);


Are you talking about:
```Ruby
     payload = [json].flatten.map do |item|
       RecursiveOpenStruct.new item, :recurse_over_arrays => true
     end
```
Because that was all from blanket.


Agreed, that makes up for more readable code.


No, the whole file.


Saved a whole bunch of resources by removing the pointless code and logic. Used to call split functions and just basically concatenated everything but the first string returned. Instead in it uses strstr to return the first string returned: bonus, it's the same string so no need to actually make another copy. The last parameter, when passed, is handled just as elegantly, it keeps going until it finds the very last token, and returns that. Simple.


Cool! Comes in just fine. Even the little curves get properly segmented. Now just need to fix the materials thing and I think we'll be very close to a decent roundtrip


# blah blah blah blah


Does it make sense to make the number of bins and the fudge factor (0.95) configurable?  This brings up another question.  You are already using the left hand edge of the bin.  Do you need to make the conservative estimate of reducing the value by 5%?


Are you talking about the entire output of the response? For example:
```Ruby
<Blanket::Response:0x007fdd3425edb0 @payload={:objects=>
[{:name=>"foo", :key=>"object_1"}
]}>
```

There will be more than one object returned, so the array of objects is applicable.


Actually I take that back, after inspecting this I notice it returns the NULL pointer, so I'm clearly going to have to change this. I wonder how it passed the test.


Can you just add a comment to indicate why this is necessary?


This line has a typo and will not load ResponseException.


This change made it possible to actually modify the string passed to it by dereferencing pointer to the string, because C. It seems to work well enough, so I can't complain.


Fixes off by one error.


What's the benefit of writing out the type versus `auto`?


:+1: moving all these to one place


@isabarros @Bouckaert

mandei este commit direto pro master. Ele corrige as specs. Lembram que o banco de dados não estava sendo apagado entre duas execuções da suíte de testes? Isso acontecia por que a [configuração que faz com que o banco seja resetado](https://github.com/Bouckaert/donamaria/blob/master/spec/rails_helper.rb#L34) está no arquivo `rails_helper` e nas specs eu botei pra carregar o `spec_helper` ─ ele é carregado, inclusive, [dentro do `rails_helper`](https://github.com/Bouckaert/donamaria/blob/master/spec/rails_helper.rb#L5).


Join now joins the array with a given delimiter.


Why no string for this exception?


This is a bug. The original code splits on `":"` whereas you're splitting on `"://"` so this line isn't needed.

https://github.com/django/django/blob/1.6.11/django/core/cache/__init__.py#L57


Can you add the images below the steps because I reference below in the steps which will confuse the students


"Thank you Sir!"


This is something adapted from a need to do in a future project, basically returns a String of the contents between two substrings. Wonderful for parsing information from formatted text (I.E "<Object>int:32;string:`Hello world`;</Object>") for parsing information into structures sent across a network. Going to be used in network utils definitely


For whatever reason, it seems that Java 8 can't correctly infer the type of the transaction result (same for lines 124, 143, and AclJooqDao.java:227) and so we can't compile.  We could add an explicit cast to fix this, though it is a bit messay...


This error message seems unclear on the context. The caller is converting a string in the form of `currency+counterparty` to the `orderbook` argument object with `currency` and `counterparty` properties. I don't think we should resolve in this PR, but something to think about


should probably return `None` or `np.nan` for non-computed areas, `0` is generically a valid area.


"system" is ambiguous. I presume "bootstrap a system" means "activate a developer environment for a project".


__API Blueprint syntax error on line 1428__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


Its interesting you thought about using mutant internals that way. I wounder if mutant should report these explicitly. Whats your thought? //cc @plexus, @dkubb?


rax retry tests


Forgot to save vote_spec.rb ......


学习新知识。


also supports 'anywhere'


todo esta indentado con tabs, cambialo a espacios


Hmmm. True. Suggestions?


If you isolate the autotaunt in a commit, you can make a pull request instead of reuploading to workshop.


It is more than `activate a developer environment for a project`. Even at runtime, it is used to activate environment variables before starting servers etc...

So I use it as an interface between **command line** -> **bash.origin** -> **system scripts/commands**.

I use the term *system* because when calling these scripts I interact with on organized system of information which happens to be exposed via scripts callable on the command line.

## Conclusion

If we can remove the word **system** from all of bash.origin and use something that is appropriate for capturing the above and ideally more let me know. Not sure there is a better word if you keep reading the *Overview*?


@Jareechang Why are we ignoring just the minified file? I thought both were generated?


height <----------- por eso no esta aplicando esa propiedad


I'm having trouble making a good flaky test which works as expected when
you put it in a parent class (due to the static variable n being used by
both FlakyTest.testFlakyParent and FlakyParent.testFlakyParent), but from
what I can tell, inherited tests don't cause any problems for this
implementation and its ability to re-run tests.

On 13 April 2015 at 05:56, Tibor Digana <notifications@github.com> wrote:

> i need to investigate more because failedMethod.getClazz() has to fail
> with test inheritance as well.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/seanf/maven-surefire/commit/cb977633f0f1d2be4b2556a4743a5f52d8a7008a#commitcomment-10683965>
> .
>


Found BROKEN by windows-minimal.
Here's the log file for step 'testbot update':
```
Warning: Permanently added 'github.com,192.30.252.130' (RSA) to the list of known hosts.
Read from socket failed: The connection was aborted
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

```


Add rake task delete_old and take


May favourite deck involves lots of playing around with the (actually quite complex) rules around coin flipping. I'll use a separate issue to go into detail.


Wow I forgot stuff again.
Fixed #12


just an alternative, we could also initialize `commit` with NULL at line 1270. so this method will crash sooner if we run into a bug if `NDEBUG` is defined at compile-time.


You install it as a wordpress plugin. A new payment option will be available on woo commerce once you activate it.

> On Apr 12, 2015, at 4:01 PM, Profango <notifications@github.com> wrote:
>
> Where do you place the php on the server? I could really use the Verotel gateway. But does this entail al of the code needed and does wit work with the latest WooCommerce update?
>
> ―
> Reply to this email directly or view it on GitHub.
>


Found BROKEN by full.


@h-debin

take 方法是做什么的呢？

几天之前在Ruby，你可以直接这样写 2.days.ago


I don't think I understand this commit. You say in the commit message that you *enable* warnings in recent Trilinos versions. But you enable them only for versions *less than* 11.12. Is this an oversight?


again, i'd suggest init lock_state when defining this variable.


Here are some links that might be relevant for several sections of the industry:

They are from the EU, about the car industry and impact of electric cars on the economy and the grid.

http://acm.eionet.europa.eu/docs/ETCACC_TP_2009_4_electromobility.pdf
http://ec.europa.eu/enterprise/sectors/automotive/files/cars-21-final-report-2012_en.pdf


@melvinzhang The static actions were needed to check if they're null or not. How best would this be done?


那需要我重新提交吗


@metrue


"subscribing"


Tool


@DrKLO new beta version 2.7.1 have released in hockey app 2 days ago where you have written over there "Support for audio record, file upload, etc statuses".If you don't mind,Can you please clarify me this line.


hi


develop (lefticus) - x86_64-MacOS-10.9-clang-Debug: OK (337 of 337 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)


Can we not add opentok's "src/ios/OpenTok.framework" in it so PhoneGap build not show error of "File missing " ?


@metrue  你直接改得了


@h-debin 你改改然后重新发pull request好了。
take_news 的作用是什么，对生产环节有影响么？


@metrue 没有影响


users/hjl/binutils-2_25-branch  not work, I thing we has to use binutils 2.24.51.0.3 for now.


my muen


@casey-collab


Not a problem, that can be done.  However, the upload I pushed onto the workshop is an extension of the code used in objhunt.  ObjHunt Extended still requires the original to play and I still have many more changes that I wish to make the system.  Should I submit all of these as individual pull requests to the project?


@metrue   2.days.ago 我这边运行不了？ 我先按 2 × 24 。。。 提交，你再改成2.days.ago，好吧？


Parser is working, though outputs are slightly off. Incorrect comparisons is close enough, but correct comparisons is still around 300 comparisons off from expected.


You, Sir, have :football: :rugby_football:


Typo in commit message.


This actually needs a fix in the headless browser to work correctly. The label overlaps the checkbox causing an error in poltergeist. See below.

Firing a click at co-ordinates [48.5, 383.5] failed. Poltergeist detected another element with CSS selector 'html.mti-nexaw01light-n4-active.mti-active.js.no-flexbox.flexboxlegacy.canvas.canvastext.no-webgl.touch.no-geolocation.postmessage.websqldatabase.no-indexeddb.hashchange.history.draganddrop.websockets.rgba.hsla.multiplebgs.backgroundsize.borderimage.borderradius.boxshadow.textshadow.opacity.cssanimations.csscolumns.cssgradients.cssreflections.csstransforms.no-csstransforms3d.csstransitions.fontface.generatedcontent.no-video.no-audio.localstorage.sessionstorage.webworkers.applicationcache.svg.inlinesvg.smil.svgclippaths body#default.bg3.two-col div#content div#main_container.container div.one_column.row div#main_column.clearfix.col-md-12 form#contactForm div#continue-form div#sam-contact-form.amend-textarea div div#form-3572B654-E248-A5D9-D70B-8968CF7308EC.amend-form-container.loaded form#3572B654-E248-A5D9-D70B-8968CF7308EC fieldset div.checkbox div.control-group label' at this position. It may be overlapping the element you are trying to interact with. If you don't care about overlapping elements, try using node.trigger('click').


TeamCity Demo Green :: Build [Build 24](http:/24.246.71.137:8111/viewLog.html?buildId=62&buildTypeId=DemoGreen_Build) is now running


TeamCity Demo Green :: Build [Build 24](http:/24.246.71.137:8111/viewLog.html?buildId=62&buildTypeId=DemoGreen_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:00:21


GroundDomainUpdate (sushobhit) - 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)


好的。

正确的solution:

```
require 'date'

now = Date.today
three_ago = now - 3
```
如果require active_support 的话，可以直接

```
3.days.ago
```


remove static from EventAction and the actions. Static means only one copy is shared among all instances of MagicCoinFlipEvent. Say each card has its own MagicCoinFlipEvent object, but they will all use the action from the last copy created since it overwrote the shared MagicActions.


看到的都是测试用例 哈哈


develop (lefticus) - i386-Windows-8.1-VisualStudio-12: OK (336 of 337 tests passed)

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


should be #import "XYPoint.h"


remove `=` sign


see above


@WIZARD-CXY This was pointed out in a different thread - unless I am mistaken, 11.0.0.0/8 is a real IP range - not one that you can assign to your own purposes.  Can you please update these docs?


Correction: EventAction needs to be static as it is called from the super constructor, so it has to be a static factory method. Pushed a fix in 08430e262cee8337


I understand that, but would need a different constructor due to explicit invoking.


先生！


Testing unions of unrelated patches is usually frowned upon. However, a union of tuning patches is still a union patch. So I think testing this way is fine.

And I would like it if the H+3*CMH patch could be commited, because it fixes a silly inconsistency…


When I get more time I will look into this more. I suspect there's not really any bugs in my implementation, except I do want to go over the sched_init stuff again.


Ok, thanks!


Yes, you are right. that was a mistake, I thought the app.js was generating the app.min.js file.


typo: still a *union* path


ignore the funky commit message, that was an error on my part


“An execution environment for an application”


When I get more time I will look into this more. I suspect there's not really any bugs in my implementation of this, but I suspect any bugs are coming from the upstream 0.4 patch of VRQ itself. To get a booting implementation of the VRQ stuff you should go to the "BFS_vrq_less_exp" branch (which reverts the 0.4 changes, and some other stuff). There I've achieved varying degrees of stability - and any bugs are traceable. For now, I'm keeping this branch stashed waiting for the official VRQ 0.4 implementation.


typo again: still a *tuning* patch


PCSX2_FAQ.pdf still has the 'PCSX2 1.0.0' title. :-\


BIne :)


GroundDomainUpdate (sushobhit) - x86_64-MacOS-10.9-clang: OK (1630 of 1633 tests passed)

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


Meant for staging-bringup branch?


why disable simons cat?


why not https://github.com/gizak/termui  ?


This defines the command with a hard coded IP address for 1 server.


Download libSBML version I used here from http://sourceforge.net/projects/sbml/files/libsbml/5.11.4/stable/MATLAB%20interface/


my first comment


delete


Choice commits right here. Step right up for choice commits. :+1: :skull:


My Heading comment


@richard42 This should fix [the link problem](https://github.com/mupen64plus/mupen64plus-core/issues/107#issuecomment-92135145) on windows.


GroundDomainUpdate (sushobhit) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (617 of 617 tests passed)

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


That was my thinking as well.


hello junhyukoh,when I do 'make runtest' I met this problem :[  FAILED  ] LstmLayerTest/2.TestGradient, where TypeParam = caffe::FloatGPU,what should I do?  Thank you!


@fhs, would you be able to tag a new version of ZipFile.jl?


Nice! Looks like what you need.  The author is really well known in the JS community.


In JavaScript capital letters are reserved for classes.  If you used lowercase r and c they would not conflict with the keys in the object {c:0, r:0}.  JavaScript knows that object keys are not variables or values.


Problems:

A - Pretty straightforward solution, just iterate on each kind of people and add your guests on demand, this solutions O(Smax).

B - First thing you should take into account is that for a case your answer is at most max {1 <= i <= n: Pi}, its easy just let the dinners to eat. Next you can try to divide each pancake stack so that each stack height is at most P, with P between 1 and Pi_max, its possible to determine the time you need to divide the stacks in O(n). Since in the large tests both the stack height and the number of plates is at most 1000 this O(N * Pi_max) solution is enough.

C - This problems was interesting because is related with quaternions, but after a bit of analysis we get an obvious solution: First we can compute a cumulative product of the input, so in an array Prod[ i ] we have the product of the first i terms and the base case is Prod[ 0 ] = 1, we can simulate the products by retrieving the results for each possible combination of terms with a map or hash table. After that the second point to take into account is that we need to split the array in three sections, the obvious should be i, the second and third are most challenging, since we have a cumulative prefix we need that the Prod[j] for some j should satisfy Prod[j] = i * j, checking the definitions we are finding for a j with Prod[j] = k, and last one is solved in the same fashion, Prod[n] = i * j * k, and knowing that i * j is k we have that Prod[n] = k * k = -1, and its done. Its enough to get all small test cases ok, but for large test cases I will let you think about the solution, and I will gift you two tips: 1. is possible that the pattern has a cycle?, 2. If the cycle is small enough (and it is) we can compute the preffix and suffix and check the previous conditions with that.


Thanks!

Using `.provider('$mdGestureProvider', MdGestureProvider);` throws an error, I'm using `.provider('$mdGesture', MdGestureProvider);` as in your gist.


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


Bye bye bugfixes for me. I won't update forge on my Beyond reality pack because I want to play Gregtech5 until Gregtech6 actually get significant content. Gregtech5 and likely other mods will not update to latest forge. Well, it is like a MC version change, without new features. If anyone is to blame. It is Forge for adding incompatible changes. Why could they not keep the old methods by deprecating and schedule remove for next MC version? I have not the skills to answer this. I only regret it.


You should always use { and } for single line if statements and don't put them on one line. So: if(range.e.c < C) range.e.c = C;  should be:

if (range.e.c < C) {
  range.e.c = C;
}


I have Master Quest rom with such internal name. Not sure that it is from good set though.


GroundDomainUpdate (sushobhit) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1636 of 1639 tests passed)

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


SBML specs here: http://sbml.org/Documents/Specifications

For SBML Level 3 Version 1, the spec document is at http://iweb.dl.sourceforge.net/project/sbml/specifications/Level%203%20Ver.%201/sbml-level-3-version-1-core-rel-1.pdf.

For SBML IDs vs kroneckerbio symbolic model nicestrs (the ones painstakingly autogenerated for uniqueness/compatibility with the symbolic toolbox), the BNF spec for SBML IDs is on page 12 as Type SId. It looks like it should be OK to use them directly.


Patch pushed to NPM. Sorry guys!


Invalid project


Debuggary :+1:


Unless we regexp for those prefixes in later code...


Not quite understanding this just yet


Still working on details of output.


:trophy: for today's best commit message


Thanks, I'll do!


wercker の記事も書いてみたので、どうぞ。
http://qiita.com/7kaji/items/416fac856888db49db8d


gitignore in 8d1b5a0


I hope you don't mind me taking inspiration from you. :smiley:


GroundDomainUpdate (sushobhit) - i386-Windows-7-VisualStudio-12: OK (1636 of 1639 tests passed)

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


Only admins should be modifying '<cib/>' attributes I'd have thought.
Likewise, is there any practical equivalent to being able to set attributes to `""` and deleting it the whole tag?  You can make a mess either way.

Not deleting child nodes is already possible by setting 'deny' on them.


#121 Not fixed yet


You are missing updating the dictionary part which we can't skip in case response is not None. Instead of removing the map entirely add a check for list physical network response.


Internally we translate all these into xpath anyway.
I'd encourage GUIs/CLIs to do the same sort of thing - expose user-friendly wrappers around the xpath capabilities.


this actually relates to #531 no #550


@strongville94 que te parecería usar este logo de FLISoL? http://flisol.info/Logo?action=AttachFile&do=get&target=FLISoL-2015.png

Me gusta más y es el que estan usando actualmente en http://flisol.info


Hi All,
Thanks for your feedback.
I'll split this commit to allow easy "pull request".


Glad to hear soft assert has been useful!


Indeed.

Thank you.


ほい


hai, this is for trial


I think this should be a minus instead of a plus. Right now the scroll wheel is going in the opposite direction it's supposed to. Changing it to a minus fixed it.


commit 에 대한 의견을 달 수 있음.


Hi,

why you create another language key, it already exists ?
```
	'SUPPORT_TOOL_KIT_PASSWORD'			=> 'Mot de passe',
```


Please help, email me at ctsdt@yahoo.com
my inquiry number is 15582539
date 12/5
thank you


daDa


androidweekly/Kotlin for Android (III): 扩展函数与默认值/readme.md

了解


GroundDomainUpdate (sushobhit) - Win64-Windows-7-VisualStudio-12: OK (1636 of 1639 tests passed)

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


任何疑问


I sort of did that on purpose because I'm used to it. But the code for this is changed entirely in my `gwen` branch since I'm using a different GUI library.


I'd say that it might be better to change Size's ``__str__`` to not return translated strings by default and to use ``xlate=True`` where we explicitly want it translated. However, that might be a subject for a follow-up patch.


我们可通过简单扩展函数语法将一个显示Toast的函数添加到Activity类中。


it's broken, needs fix before enabling


Hi This is the first commit I'm trying to do


具体类型


@Lollypo 自己再理一遍吧，句子通不通顺啥的。


This is very odd. I saw the bug you reported once but I can't create it again? The only global is app->title and I don't see that being set by these functions. Could it be the newer hash syntax? Seems crazy.

the #if 1 else was new version vs old version so I could compare them as I modified the new. It wasn't meant to be part of the commit.


Should be delegated to Concrete Factories , in real life code each factory might need to do its own pre-processing. before servicing the call, like say caching.


^Convert


I believe the following line:

    ExecStartPre=/usr/bin/etcdctl set /coreos.com/network/config '{"Network":"10.244.0.0/16", "Backend": {"Type": "vxlan"}}'

is missing the `--peers` options, so it should be something like:

    ExecStartPre=/usr/bin/etcdctl --peers <master_ip>:4001 set /coreos.com/network/config '{"Network":"10.244.0.0/16", "Backend": {"Type": "vxlan"}}'


@dgreisen-cfpb I added the `createTeam` method to the collection instead of the model (where everything else is). LMK if you think it belongs elsewhere.


This line needs to read <bool name="config_device_volte_available">false</bool>


`Function` -> `function` maybe...


I can recreate it.


I will update the doc and script and let the user fill in their unused ip range for service portal_net instead of being 11.1.1.0/24


Welcome back in Java 6!


Yes


develop (lefticus) - Win64-Windows-8.1-VisualStudio-12: OK (336 of 337 tests passed)

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


Methinks Travis is doing its job here. It's opinionated yeah.


@moonso: take a look at this. It seems like it just works after I reloaded the data. Now the compounds are sorted correctly in the output.


re


sa


Yes I don't think I needed to revert that one. Assume you fixed subsequently.


Nope - just wanted to get it working on the new data.


cool, will fix it in the next PR :)


친구들과 함께 특별한 녹음을 준비해보세요 이거 어떻게할거예요 ㅋㅋㅋㅋ


@zopyx Please make sure a package is in auto-checkout in buildout.coredev before you push changes. You should also have received an email from mr.roboto telling you that. It seems your change (or @davisagli's) broke some tests: http://jenkins.plone.org/job/plone-5.0-python-2.7/785/console


gutes zeug!


What updates were made?


Good. Will update: https://github.com/bash-origin/bash.origin/issues/2


Ok great. I'll test it out later this week, with a bit of luck tomorrow
On Apr 12, 2015 9:39 PM, "Benjamin ABEL" <notifications@github.com> wrote:

> I added *cells selection* in tests and all seems to work ipython 2/3 and
> nbformat 3/4.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/benjaminabel/pelican-plugins/commit/b05b0790c4959343e2ff1c9d9b8a38b9c32c7ef9#commitcomment-10683899>
> .
>


调皮


Contribution to #199


What about The Team page? That one is not on master.


I think Perish Song bypasses Substitute in this gen.


Looks good to me. Only possible change would be to use undefined instead of false for required. (same as if you hadn't defined required at all, which is actually easier). I think it's more clear than "false"

I'm also somewhat curious how you got your full selection of buildings here... It seems like more than was in the current script but I don't really understand which buildings you haven't included vs those you have.


싱글 스레드 비동기


this is not necessary in C++, neither, i believe.


Though the authentic flag is irrelevant for Perish Song.


GroundDomainUpdate (sushobhit) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1126 of 1129 tests passed)

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


@tchaikov. We must add good peer into missing_loc. For k=3/m=1, we met 1 replica CRC error,
 in ReplicatedPG::recover_replicas, the missing_loc.is_unfound will return true, But in fact, this object is not unfound(can recover).


haha that looks right! Let me test that now...


@acfoltzer @brianhuffman

FYI. SBV 4.4 is out. Shouldn't impact anything on Cryptol side; just some further work on floating-point support.


If you aren't using the `json` view, this is a good time to remove it.


No curly-braces needed, if they are empty.


hum hum, i thinking in the commit. you can rewrite any commit with rebase command.

well, I do it this time.


Could this be limited to show up only on the appropriate projects? *.doc I'm guessing?


What I do not understand is why I didn't get this error... have you got the time to test it? Are you really sure is a privileges problem?

I always try to do as less things as possible as root, so that being there means that all configuration (and its calls) is done as root. It's not bad, just not elegant.


Since the instance variable is defined already, it will be available in either view. This looks confusingly like you are trying to "pass" the instance var to the view, when really what's in the curly braces is a block of code that will run.


@cemak Russian Standard Vodka ?


Rather than a page reload, you could let this drop through to an `update.js` view file, which could replace the dom element with a newly rendered item.


@majianpeng agreed in the case of EC pool. but is it necessary for replicated pool as well?


合并新版本


Could this filter only projects that can be imported - hard to tell between the 3 projects per model which should be selected (when I suspect only .doc is valid)


@tchaikov. Can you give me a example? Thanks


It's entityName, not inEntityNamed


Why are you testing WRAM accesses against breakpoints on the SuperFX bus? The SuperFX can't even access the SNES WRAM.


I don't know my reasoning anymore. Using the official Node image makes sense though. Be my guest and submit a PR?


Some thoughts that can be raised: "What tiebreaker do we consider for competing maximums?"


@majianpeng i posted my understanding in a previous PR: https://github.com/majianpeng/ceph/commit/6be0d7935ce80dd98e2ebfc402321b3a50f0125a does it make sense ?


If you close a bank statement and at  a later date want to come back and update individual reconciliation lines on the bank statement there is no way of doing this except by cancelling the whole bank statement and loosing all previous reconciliation work on that particular bank statement. This is unacceptable where there is a large number of reconciled transactions on a bank statement involving the loss of a large amount of reconciliation work.
This change means when a reconciliation line is cancelled the bank statement is reset back to the "draft" state and makes the "Reconcile" button once again visible to that the reconciliation cancelled line(s) can now be updated.


This should always return 2, because you are creating a two-element list, the first element is `:listened`, and the second element is `song`. No matter how complicated a data structure `song` is, clojure counts it as one element, so this always returns 2. Test out in the REPL as the following:
```clj
(def song
  [{:title "Us and them" :listened true :link "https://www.youtube.com/watch?v=FKHArRwK6xU&spfreload=1"}
   {:title "Atom Heart Mother" :listened true :link "https://www.youtube.com/watch?v=UyRJzdKUho0"}
   {:title "Have a Cigar" :listened true :link "https://www.youtube.com/watch?v=LBfHpIRm5Qk"}
   {:title "Mother" :listened false :link "https://www.youtube.com/watch?v=lX3uCuFKlqw"}
   {:title "Shine On you crazy diam" :listened true :link "https://www.youtube.com/watch?v=3IUbncOv_-g"}
   {:title "Fat Old Sun" :listened false :link "https://www.youtube.com/watch?v=SKUfuS24O-I"}])
(count [:listened song])
```
No matter what you define song as, it will always be 2.

(Also, FWIW, you should name this variable `songs`, not `song`, since it is meant to represent multiple songs.)


my apologize, you did the both.


@arvindsv @srinivasupadhya

I'm not familiar with the requests that are made to the go server, but should this not include "HEAD" as well?


@foxlet Wrong. It's supposed to be "This has been left blank for now".


部分注释没有写完，解密过程中，对密文中CRLF的处理还需考虑


Looking back at this commit, I really don't understand how you are actually breaking the location updates into chunks -- you run one query in "queryLatestVersion" to find a cutoff point for the next batch, but then it's not actually used in `queryChanged()` or in `linkAdminEntities()` to limit the number of results, so as far as I can tell, rather than breaking a set of updated locations into, for example, 5 chunks of 50 locations (50, 50, 50, 50, 50) you're sending the full set of updates each time, but with the version that corresponds to the first chunk, which will yield batch sizes of (250, 200, 150, 100, 50), effectively multiplying the size of work to be done and volume to be transferred, or am I missing something?


This variable should be called `songs`, not `song`.


This pull request fix the following errors when running mix test:

1) test it's possible to delete a stored value by key (ExleveldbTest)
test/exleveldb_test.exs:30
     ** (MatchError) no match of right hand side value: {:error, {:db_open, 'IO error: lock /tmp/dbtest/dbtest4/LOCK: already held by process'}}
     stacktrace:
       test/exleveldb_test.exs:11: ExleveldbTest.mock_db/1
       test/exleveldb_test.exs:32

2) test it's possible to get a value from the datastore by key (ExleveldbTest)
     test/exleveldb_test.exs:25
     ** (MatchError) no match of right hand side value: {:error, {:db_open, 'IO error: lock /tmp/dbtest/dbtest3/LOCK: already held by process'}}
     stacktrace:
       test/exleveldb_test.exs:11: ExleveldbTest.mock_db/1
       test/exleveldb_test.exs:27

A reference to the database is required on the first assertion. Because the database is created into the functions put, the test cases fail due the existence of other databases called "dbtest3" and "dbtest4". This pull request intent to solve this issue using references to the databases.


That was supposed to be the CPU bus (obviously, given that it's the CPU debugger...) but was testing some stuff with the SuperFX debugger at the same time and apparently wasn't paying attention. Thanks for the heads up, fixed now.


__API Blueprint syntax error on line 11__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


fixes #1


If we are relying on exceptions, its common practice to catch the specific exceptions and not global catching.


Shouldn't you be taking the absolute value of `xml_memory_diff` not `diff`? Otherwise, every single call being less than 64k will be enough to avoid the send.


chap nhan


first commit


@tchaikov . For the reason is right. This commit fix ec unrecover object. In ReplicatedBackend::recover_object, before call prepare_pull, it check get_parent()->get_local_missing().is_missing(hoid). This only primary object lost, we can add in repair_object.
I mean if we can't add goog beer into missing_loc, there is a bug. If add, is there a bug for replicated pool?


I see, a manual setup. I've got it working by removing `@WebAppConfiguration` from the test and `@Scope("request")` from `SWORDServiceResource`. Do you discourage using dependency injection in unit tests?


fitellipse_example_error_fix : vector out of range occur when some threshold value is '106'


https://github.com/lavabit/libdime/search?utf8=%E2%9C%93&q=dmime_header_keys

On line 49 of `parser.c`, the variable is *used*, not *defined*. And in `dmime.h` it is *declared*, not *defined*.

Were you really able to run `make check` at the top-level directory, and it worked?


This puzzles me a bit. As I understand Spring, removing `@Component` would render the bean undiscoverable by the container.


Updated to master


nvnvb


I cannot see where the this dependency is used directly. Is it a dependency of Fedora Test Utils? Why is it not pulled in from there?


IMO free entries in the scheduler table should have all bits zero. No extra flag to mark them. Simplifies the generated code.


@Olink well aware of that, but I don't want to break existing rest functionality if someone is string matching


oh, in hindsight, i guess its not throwing different exceptions than previously, whelp, rip.


The scheduler resides in its own sub folder but periodic doesn't. Why?


Nice.

However I'd like to modify the filter function slightly since I don't see why we have to pass the same filename back into the callback.

How about:

```js
if (config.livereload.enable) {
  watch(file.path, function(filename) {
    config.livereload.filter(filename, function(shouldReload) {
      if (shouldReload) {
        gutil.log('Livereload: file changed: ' + filename);
        config.livereload.io.sockets.emit('reload');
      }
    });
  });
}
```

And then the default `filter` function would look like:

```js
function(filename, cb) {
  cb( !(/node_modules/.test(filename)) );
}
```


Numan, will it be good to make the FloatingIPGetHandler to inherit from ResourceGetHandler and other actions like FloatingIPUpdateHandler, DeletHandler, CreateHandler to inherit from their respect Resource*Handler and make FloatingIPHandler to inherit from FloatingIP*Handler?

By the way, I could see a FloatingIPHandler with this same implementation in contrail_res_handler.py file as well.


Actually, enfore LF line endings on every text files.


Lack of an integer is not `-1`. Its `Nothing`. Same with `""`. No magic constants please.


I think the make uninstall on this place is not a good idea. Sometimes it can make problems when the build/configure process found libs and include files in the system. What you think, when we save the make file for 32 and 64 bit separate and run the for example ./build32/make uninstall or ./build64/make uninstall. Many packs also work with configure in a build folder, this is also a option. For example we make a build32/64 folder, cd in, and run configure .. --prefix=[...], same like cmake.


Level 25 Big knights, that was masochistic!


If we can make all {Resource}Handler to inherit their {Resource}*Handlers which inturn inherit their respective ResourceHandler, We can generalize the following apis

The DBInterface can just have a __getattr__ function, and from the attriibute, we can identify the type of resource and the action on that resource. Based on that can chose the {Resource}Handler api and send the resource_{Action} method. By doing so, we call avoid writing all these API's which more or less do the same thing.
Please correct me if I am wrong


I'm inclined to merge this.

@alanoe, what do you think about conforming to PEP 303 ?!


[Shadow Mapping](https://en.wikipedia.org/wiki/Shadow_mapping) is a technique you could use at some point to make this support eclipses and stuff.


I think connection pooling caused high memory usage in the Go server, too. :crying_cat_face:


评论


Typo in commit message.


Whoops!

Something must have happened between my testing it and committing, which resulted in me accidentally deleting the table. Really sorry about that, I pushed the fix.


That's not valid JSON


Hi and thank you for your fast feedback. That is useful additional information you're giving. So far I've suggested this to the author already per email and just saw he accepted that change.It will be in the next release. And there were no official steps, I created patch-files (I keep a list of commits here: https://github.com/hakre/HTMLawed/commits/stable-fixes) and mailed it to the author.


The Product Import module allows the admin to import content from the json file.

Installation
------------
Standard module installation applies.

Menus
-----
admin/config/content/product_import

Settings
--------
The settings page is at Administer >> Config >> Content >> Product Import.

Paste all the content from the json file and click import button to import all the content from the json file.

This is where you can import all the content.


I think you need a blank line here for correct formatting


@rschmukler this commit has broken the md-select directive. On the demo pages when you open the select the page is scrolling down approx. 20-30px. On my app it is scrolling down 300-400px, adding a min-width to the select of 653px even thought the widest text is only 80px. It also seems to be doing some odd stuff to background colors?


Yes you're right. But the combined commit is still very small.


This assignment is one example when you can do re-factoring and employ that DRY (Don't Repeat Yourself) principle I mentioned back in Chapter 5.

In `arrayOfNumbersBetweenNumber:` method, you have the same code repeated twice, the whole `for` loop. So you can rewrite this like this:

```objective-c
- (NSArray *) arrayOfNumbersBetweenNumber:(NSInteger)number andOtherNumber: (NSInteger)otherNumber {

    if (number > otherNumber) {
		return [self arrayBetweenLowNumber:otherNumber andHighNumber:number];
    }

	return [self arrayBetweenLowNumber:number andHighNumber:otherNumber];
}

- (NSArray *)arrayBetweenLowNumber:(NSInteger)lowNumber andHighNumber: (NSInteger)highNumber {

	NSMutableArray *arrayofNumber = [[NSMutableArray alloc] init];
	for (NSInteger i = lowNumber; i <= highNumber; i++){
		NSNumber *nextNumber = [NSNumber numberWithInteger:i];
		[arrayofNumber addObject: nextNumber];
	}

	return [NSArray arrayWithArray:arrayofNumber];
}
```

This way, the code that actually does the job is written only once. This, in some real iOS app, makes it much easier to maintain in the long run, as you have only one place to fix potential bugs.

Thus every time you see that you have code written twice, think about doing this.


Dear lord


index >= 0 ?


www


No problem, I’m glad that it works again.


Slick. I've only previously used https://github.com/github/gitignore.


Notice that I used `i` instead of `firstNumber` as the variable. It's good practice to use general-sounding words for variables that change over time, especially in loops. So you can use `i` or `counter` or something else. `firstNumber` is too specific sounding and as soon as you move to second number is no longer the "correct" name.


Created a Video Provider interface and now has Vimeo and Youtube. Both have a function called "recognize" that returns true/false if the video URL will work with that provider.


Shouldn't it read `ACL.users` instead of `users_ACL`?


with this patch, for a replica pool:

- case 1: the bad peer is primary: we put the soid into pglog.missing, and we put append the the good peers to `missing_loc.missing_loc`. so we can pull good copies from them in `ReplicatedBackend::recover_object()`. yes, all good and sound. this is the scenario you are talking about. and i agree with you.
- case 2: the bad peer is a replica: we also put good peers into `missing_loc.missing_loc`. this is not a bug, i'd say. but not necessary. because we *only* use `peer_missing` for the push targets who need the good object from primary. and the `missing_loc.missing_loc` is not used here.


`ACL.groups` instead of `groups_ACL` ?


have a meaningful docstring, use pep257 linter


http://127.0.0.1:8086/proxy.pac?flush
访问这里强制更新

出现错误提示
unsupport gfwlist regex: '^https?:\\/\\/[^\\/]+blogs
pot\\.(.*)'
unsupport gfwlist regex: '^https?:\\/\\/[^\\/]+googl
e\\.(.*)'
unsupport gfwlist regex: '^http:\\/\\/[^\\/]+line\\.
naver\\.jp'
update_pacfile failed: AttributeError("'NoneType' obje
ct has no attribute 'lstrip'",)

PAC文件已经很久没有更新了。请phuslu修复下吧！


use keyword style for passing such long list of arguments


new


document the option


@RamonGilabert not at the moment


document fixture


Issue opened here : https://github.com/angular/material/issues/2283


meaningful docstring, use pep257 linter


Guys, I see what you're doing here and I have a few issues:
This feature should be implemented within the current framework, not as a separate web-site.
We want this to be a single page website, with a consistent look-and-feel and URL scheme.
So this goes both to the styles, use of the current code infrastructure and build system.

If you must, you can use Jade, stylus, etc., but it needs to be a part of the current build system (grunt)
If you must, you can use angular, but it has to interact with the current Backbone models and base methods

Thanks!


we use get request for most, if not all, item drop actions (post/comment, photos, etc.)... i don't think csrf tokens are implemented anywhere except for account and channel deletion where we use a one-time token...


@tchaikov . Yes, you are right. But if we can't add, there is a bug for ec unrecoveralbe object. So my patch not better but can work. Are you have other option which can resolve this bug and looks good?


Does this mean that people can now use RetroPie commercially?


Hehe, hi @banderson. The article is over here: http://christianalfoni.com/articles/2015_04_01_Functional-Reactive-Applications. Though it is not using "observable-state" currently. It needs some more work, but will make it compatible ;-)


Ok thanks for the feedback. Please go ahead and re-format. Looking forward to an official release.


The MAXPLAYERS macro is called the same in all four games -- and only in Choco Hexen, the maxplayers variable takes its logical place. I am really not sure if we should rename it.


well, I fixed the code to use knnRadius. Can you please try out trunk ? Please let me know if that works for you and I'll release that. Thx.


短发的份上


We can do that
And we can start working with issues/bug reports.
Can you upload your working code?


Looks nice!
So it works now?


You can get stall velocity from another function.


to take the coverage information into account


i'm going to say no. on the premiss that i have an error and cannot find a symbol, unlike find the abominal snowman or a wookie this one will be found;

frameworks/base/packages/SystemUI/src/com/android/systemui/qs/tiles/LocationTile.java:162: error: cannot find symbol
            label.setText(getStateLabelRes(getItem(position)));
                          ^
  symbol:   method getStateLabelRes(Integer)
  location: class LocationTile.AdvancedLocationAdapter


一技能那个不实际检测装备区的牌的数量大丈夫？
吴国太 换装备 应该也触发move 吧？


The g inf is a column data, please using g(1,1)


Clever.


some missing methylation values are set to zero in this case


https://issues.apache.org/jira/browse/COMPRESS-184 and https://issues.apache.org/jira/browse/COMPRESS-183 seem to resolve the issue in commons-compress, which is a subject for adoption as a part of https://issues.jenkins-ci.org/browse/JENKINS-10629


You can also put that in your ~/.npmrc


the specified number of decimal places, you can also make it 3, 4...


I don’t understand what this refers to or what this does and I *wrote* the old code. Please use the comment to explain the issue in sufficient detail so that the reader can understand what is going on.


Is you exp branch stable enough for daily usage?


@majianpeng sure, we should add the good loc for EC pool, definitely. but can we guard the "add" code with something like following?
```c++
if (is_ec_pg() || bad_peer == primary) {
 // we'd better collect all shards for EC pg, and prepare good peers as the sources of pull in the case of replicated pg.
```
and accompany it with some comments for new developers like me.

thanks for your patience jianpeng. =)


@thibaultCha yes if the Nginx Plus Lua module has been compiled with LuaJIT (which doesn't seem to be the case). Will follow up with the nginx team to see if there is a way to enable LuaJIT.


이건 대체 왜!?


> I am actually getting an error on my dataset still with your changes unfortunately. Investigating.

@struktured Please open a bug report when you find the cause.


OK


삼항연산자화


전역변수로 변경


전역 렌더러 사용


Combining ligatures沒有被替換，而且Safari好像直接把U+030D吃掉了⋯⋯

![2015-04-13 15 00 29](https://cloud.githubusercontent.com/assets/31068/7111198/3446cd8c-e1ef-11e4-80ce-254fdcb06d7a.png)


헤더파일에 포함됨


What is new Thread() and e.printStackTrace? Is this how we're able to perform two separate programs at once? When it comes to writing the closeProgram() method, will there any specific code regarding this?

I'll check online for some definitions and examples as well.


Why not using the _manifest.lua for this?


In the coding guide we specified only the suffixes _l, _p and _g.
I would suggest to document the unit in the comment like for the other parameters and rename the variables: e.g. switchOverTimeMn, reducedSwitchOverTimeMn, delayedSwitchOverTimeMn


maybe we shouldn't use `std::vector`, there's no reason this shouldn't work on other containers. I'll test an iterator-based version later.


Кажется здесь должна быть строка return true;


@stevage would you mind making changes like this via a pull request in the future?


@tchaikov .  I'm not sure that, because  this :pg_log.missing_add(soid, oi.version, eversion_t()); IMO, pg_log.missing_add only for local missing object. Frankly, those code looks complex. @liewegas Can you give some suggestion. Thanks!


Oops


If this is public just for testing, perhaps it is better to use InternalsVisibleTo on the project. This will keep the public API cleaner.


Il faudra vraiment penser à passer les DAL du mode connecté au mode déconnecté.
Ca évitera une utilisation excessive en terme de temps de la base de donnée.
Et ca donnera une raison d'être au Dataset que tu as créé : )
( Et ca permettra d'avoir un code plus lisible, plus objet et moins 'SQLisé'. )
( Et ca uniformisera la manière de faire au sein du code  ^^  et c'est plus C# et proche du cours )


This is a poor choice of name — it is unclear what it is for and what it does. Let’s use `installerArguments` instead.

I do wonder if it shouldn’t be in a (new) winsparkle-specific namespace instead, though, so that Sparkle’s NS isn’t polluted by it…


41:1: I100 Imports statements are in the wrong order. import os should be before import sys
42:1: I100 Imports statements are in the wrong order. import inspect should be before import os
44:1: I100 Imports statements are in the wrong order. import cgi should be before import traceback
47:1: I101 Imported names are in the wrong order. Should be current_app, has_app_context
48:1: I100 Imports statements are in the wrong order. from operator should be before from flask
48:1: I201 Missing newline before sections or imports.
49:1: I201 Missing newline before sections or imports.
50:1: I201 Missing newline before sections or imports.
52:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from werkzeug.utils
53:1: I101 Imported names are in the wrong order. Should be CFG_BIBFORMAT_CACHED_FORMATS, CFG_BIBFORMAT_DISABLE_I18N_FOR_CACHED_FORMATS, CFG_BIBFORMAT_HIDDEN_TAGS, CFG_SITE_LANG
53:1: F401 'CFG_BIBFORMAT_CACHED_FORMATS' imported but unused
53:1: F401 'CFG_BIBFORMAT_HIDDEN_TAGS' imported but unused
54:6: E126 continuation line over-indented for hanging indent
59:6: E126 continuation line over-indented for hanging indent
60:1: I101 Imported names are in the wrong order. Should be create_record, record_empty, record_get_field_instances, record_get_field_value, record_get_field_values, record_xml_output
61:6: E126 continuation line over-indented for hanging indent
69:1: I101 Imported names are in the wrong order. Should be gettext_set_language, language_list_long, wash_language
69:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from .engines
69:1: I201 Missing newline before sections or imports.
70:6: E126 continuation line over-indented for hanging indent
74:1: I101 Imported names are in the wrong order. Should be CFG_BIBFORMAT_FORMAT_JINJA_TEMPLATE_EXTENSION, CFG_BIBFORMAT_FORMAT_OUTPUT_EXTENSION, CFG_BIBFORMAT_FORMAT_TEMPLATE_EXTENSION, CFG_BIBFORMAT_OUTPUTS_PATH, CFG_BIBFORMAT_TEMPLATES_DIR, InvenioBibFormatError
74:1: I201 Missing newline before sections or imports.
74:1: F401 'CFG_BIBFORMAT_TEMPLATES_DIR' imported but unused
74:1: F401 'CFG_BIBFORMAT_FORMAT_OUTPUT_EXTENSION' imported but unused
74:1: F401 'CFG_BIBFORMAT_OUTPUTS_PATH' imported but unused
75:6: E126 continuation line over-indented for hanging indent
81:1: I100 Imports statements are in the wrong order. from invenio.modules.formatter.utils should be before from .config
82:6: E126 continuation line over-indented for hanging indent
83:1: I101 Imported names are in the wrong order. Should be CFG_HTML_BUFFER_ALLOWED_ATTRIBUTE_WHITELIST, CFG_HTML_BUFFER_ALLOWED_TAG_WHITELIST, HTMLWasher
84:6: E126 continuation line over-indented for hanging indent
87:1: I100 Imports statements are in the wrong order. from invenio.modules.knowledge.api should be before from invenio.utils.html
88:1: I100 Imports statements are in the wrong order. from invenio.ext.template should be before from invenio.modules.knowledge.api
89:1: I100 Imports statements are in the wrong order. from HTMLParser should be before from invenio.ext.template
89:1: I201 Missing newline before sections or imports.
90:1: I201 Missing newline before sections or imports.
98:27: E261 at least two spaces before inline comment
99:28: E114 indentation is not a multiple of four (comment)
99:28: E116 unexpected indentation (comment)
100:28: E114 indentation is not a multiple of four (comment)
100:28: E116 unexpected indentation (comment)
101:28: E114 indentation is not a multiple of four (comment)
101:28: E116 unexpected indentation (comment)
104:28: E114 indentation is not a multiple of four (comment)
104:28: E116 unexpected indentation (comment)
118:33: E225 missing whitespace around operator
203:1: E731 do not assign a lambda expression, use a def
206:1: E402 module level import not at top of file
209: D203 1 blank line required before class docstring (found 0)
213: D401 First line should be in imperative mood ('Return', not 'Returns')
213: D400 First line should end with a period (not 's')
225: D401 First line should be in imperative mood ('Return', not 'Returns')
250: D102 Missing docstring in public method
289: D103 Missing docstring in public function
295: D401 First line should be in imperative mood ('Format', not 'Formats')
295: D400 First line should end with a period (not 'f')
295: D205 1 blank line required between summary line and description (found 0)
342:45: W503 line break before binary operator
342:45: E127 continuation line over-indented for visual indent
358:8: W503 line break before binary operator
485:27: E225 missing whitespace around operator
488:5: E303 too many blank lines (2)
506:82: E225 missing whitespace around operator
527: D103 Missing docstring in public function
543: D401 First line should be in imperative mood ('Return', not 'Returns')
543: D202 No blank lines allowed after function docstring (found 1)
543: D400 First line should end with a period (not 'g')
543: D205 1 blank line required between summary line and description (found 0)
564:64: E261 at least two spaces before inline comment
564:65: E262 inline comment should start with '# '
571:38: E261 at least two spaces before inline comment
571:39: E262 inline comment should start with '# '
572:48: E261 at least two spaces before inline comment
572:49: E262 inline comment should start with '# '
575:24: E127 continuation line over-indented for visual indent
584: D103 Missing docstring in public function
587: D400 First line should end with a period (not 's')
587: D200 One-line docstring should fit on one line with quotes (found 3)
600: D210 No whitespaces allowed surrounding docstring text
600: None: None
600: D400 First line should end with a period (not 'a')
600: D205 1 blank line required between summary line and description (found 0)
632:57: E126 continuation line over-indented for hanging indent
639:5: E303 too many blank lines (2)
698: D401 First line should be in imperative mood ('Evalute', not 'Evalutes')
718: D401 First line should be in imperative mood ('Analyse', not 'Analyses')
718: D202 No blank lines allowed after function docstring (found 1)
787: D401 First line should be in imperative mood ('Return', not 'Returns')
787: D400 First line should end with a period (not 't')
787: D205 1 blank line required between summary line and description (found 0)
886:13: B901 blind except: statement
917:78: E225 missing whitespace around operator
918:54: E225 missing whitespace around operator
950:47: E261 at least two spaces before inline comment
965:27: E225 missing whitespace around operator
1025: D401 First line should be in imperative mood ('Filter', not 'Filters')
1035: D401 First line should be in imperative mood ('Searche', not 'Searches')
1035: D400 First line should end with a period (not 's')
1035: D205 1 blank line required between summary line and description (found 0)
1061:9: E303 too many blank lines (2)
1075:5: E303 too many blank lines (2)
1078:1: E302 expected 2 blank lines, found 1
1078: D401 First line should be in imperative mood ('Return', not 'Returns')
1092:5: F841 local variable '_' is assigned to but never used
1095:12: E127 continuation line over-indented for visual indent
1104:16: E126 continuation line over-indented for hanging indent
1126:5: B901 blind except: statement
1138: D401 First line should be in imperative mood ('Return', not 'Returns')
1138: D400 First line should end with a period (not 's')
1158:16: E126 continuation line over-indented for hanging indent
1165:1: E302 expected 2 blank lines, found 1
1165: D401 First line should be in imperative mood ('Return', not 'Returns')
1165: D400 First line should end with a period (not 'e')
1197:13: E303 too many blank lines (2)
1211: D401 First line should be in imperative mood ('Return', not 'Returns')
1245:16: E127 continuation line over-indented for visual indent
1255:31: E126 continuation line over-indented for hanging indent
1282:9: B901 blind except: statement
1294:17: E126 continuation line over-indented for hanging indent
1301:1: E302 expected 2 blank lines, found 1
1301: D401 First line should be in imperative mood ('Return', not 'Returns')
1301: D400 First line should end with a period (not 'e')
1345:1: E302 expected 2 blank lines, found 1
1345: D401 First line should be in imperative mood ('Return', not 'Returns')
1345: D202 No blank lines allowed after function docstring (found 1)
1472:1: E302 expected 2 blank lines, found 1
1472: D401 First line should be in imperative mood ('Return', not 'Returns')
1472: D202 No blank lines allowed after function docstring (found 1)
1499:17: E225 missing whitespace around operator
1578:28: E712 comparison to False should be 'if cond is False:' or 'if not cond:'
1582: D401 First line should be in imperative mood ('Return', not 'Returns')
1582: D202 No blank lines allowed after function docstring (found 1)
1582: D400 First line should end with a period (not '}')
1599:56: E225 missing whitespace around operator
1635:1: E302 expected 2 blank lines, found 1
1635: D203 1 blank line required before class docstring (found 0)
1635: D204 1 blank line required after class docstring (found 0)
1635: D400 First line should end with a period (not 'n')
1635: D205 1 blank line required between summary line and description (found 0)
1666:15: E261 at least two spaces before inline comment
1668: D401 First line should be in imperative mood ('Create', not 'Creates')
1700:31: E261 at least two spaces before inline comment
1725: D401 First line should be in imperative mood ('Return', not 'Returns')
1725: D400 First line should end with a period (not 'e')
1740: D401 First line should be in imperative mood ('Return', not 'Returns')
1740: D400 First line should end with a period (not 'd')
1763: D401 First line should be in imperative mood ('Return', not 'Returns')
1763: D400 First line should end with a period (not 'e')
1763: D205 1 blank line required between summary line and description (found 0)
1801: D401 First line should be in imperative mood ('Return', not 'Returns')
1801: D202 No blank lines allowed after function docstring (found 1)
1910:36: E127 continuation line over-indented for visual indent
1913: D401 First line should be in imperative mood ('Return', not 'Returns')
1970:33: E127 continuation line over-indented for visual indent
1981:60: E251 unexpected spaces around keyword / parameter equals
1983:54: E251 unexpected spaces around keyword / parameter equals
1993:37: E127 continuation line over-indented for visual indent
2004:64: E251 unexpected spaces around keyword / parameter equals
2006:58: E251 unexpected spaces around keyword / parameter equals


Deze kan je weggooien.

```php artisan serve```


Should the script be disabled as well?


I've rewritten this. See the (force) update branch: https://github.com/LeSpocky/libcgi/tree/redirect


This completely ignores the commonly used handling of arguments *with spaces in them* by quoting them.

More importantly, I don’t understand why are you doing all this complicated stuff to begin with. Read [wxExecute docs](http://docs.wxwidgets.org/trunk/group__group__funcmacro__procctrl.html#gaa276e9e676e26bafeec3141b73399b33) and observe that you could just simply append the arguments to a single string; t would take care of parsing (& in a better way to boot). Would make the other if branch unnecessary as well.


1: M111 unrecognized component name: assets


In fact, on Aukro.ua you can buy one item without adding it to the cart.


I worked locally from what I could tell. But you need to re-populate the database - existing data isn't sorted :sweat_smile:


mongo에 shopper-session db를 수작업으로 만들어줘야 하나요?? 지금 show dbs하면 shopper-session이라는 db가 안나오네요..


*par


Thanks @Steell, @aosyatnik has since then made [a pull request to remove these files](https://github.com/DynamoDS/Dynamo/pull/4223).


What working code?
Issues are great


Bravo ^_^


@F03SD, russian RSM-56 Bulava


@majianpeng sorry, i should explain my suggestion more clear. this is the patch in my mind:

```
@@ -3665,15 +3665,20 @@ void PG::repair_object(
     assert(waiting_for_unreadable_object.empty());

     pg_log.missing_add(soid, oi.version, eversion_t());
+
+    pg_log.set_last_requested(0);
+    dout(10) << __func__ << ": primary = " << primary << dendl;
+  }
+
+  if (is_ec_pg() || bad_peer == primary) {
+    // we'd better collect all shards for EC pg, and prepare good peers as the
+    // sources of pull in the case of replicated pg.
     missing_loc.add_missing(soid, oi.version, eversion_t());
     list<pair<ScrubMap::object, pg_shard_t> >::iterator i;
     for (i = ok_peers->begin();
-        i != ok_peers->end();
-        ++i)
+       i != ok_peers->end();
+       ++i)
       missing_loc.add_location(soid, i->second);
-
-    pg_log.set_last_requested(0);
-    dout(10) << __func__ << ": primary = " << primary << dendl;
   }
 }

```


Hey Kordian,

would you please revert this commit. It's not about that I don't have the proper knowledge how to style the checkboxes. It's simply the wrong time for doing this.

This changeset add another layer of complexity to possible required code changes in relation to the checkboxes, since it's now required to take care of the CSS as well.

I don't see that any feature/functionality is fixed by now. That's why I have the strong opinion that we should focus on feature/functionality before starting to style something.


So SAP is actually slower? :) Interesting, I'll test it myself on my devices this week.

Also, is there a way to deploy it on Android?


Would you please write a few lines why it's necessary to introduce a new function. I don't see the benefit of doing this. We should focus on using existing stuff instead of bloating the code.


@carlocamilloni can you check if actually this is true? The change in 0dd0e3f381e475e2280095213b637719382ffd0f is
-    setDerivative(v,cv,fact);
+    setDerivative(v,i,fact);
The second argument should be the number of the cv (0,1,2, etc), whereas it was the value of the cv.
I think even with a single argument, this was going to work only when by chance the value of the cv was rounded to zero. Unfortunately c++ was converting the double to int without detecting the problem.


这些都要翻译啊 ，比如Marerial主题，新的控件，自定义阴影，Vector Drawables和自定义动画。


这些标题都要翻译


if you have some time, can you add some debug and retry
```
if ($res === false) {
    ZLog::Write(LOGLEVEL_DEBUG, "DoForcePingTimeout(): set just failed, retrying");
    ZLog::Write(LOGLEVEL_DEBUG, "DoForcePingTimeout() key: ".$this->key);
    ZLog::Write(LOGLEVEL_DEBUG, "DoForcePingTimeout() reqtime: ".$_SERVER['REQUEST_TIME']);
    ZLog::Write(LOGLEVEL_DEBUG, "DoForcePingTimeout() last: ".self::$redis->getLastError());
    continue;
} else {
```


The function name is a little weird now that it doesn't require an at.  Maybe `s/At/And/`?

Also, this function is used in a couple other places (notably parsing the `tool` in a release manifest) though I guess it's OK to have this change there too.


That's not actually true: `utils.splitReleaseName` is used for `.meteor/release`.


It's fine, only an error on line 34, it should be

```objective-c
numberDictionary[5] = "five"
```


전역 사용


const는 왜 붙인겨


> Lack of an integer is not -1. Its Nothing. Same with "". No magic constants please.

Do you mean `undefined`? Can we explicitly add `Nothing` while using `fmap`.
 Or should `>>=` be preferred as in

```hs
    ...
    , creationDate = getInt    =<< creationDate'
    , comment      = getString =<< comment'
    , createdBy    = getString =<< createdBy'
    , encoding     = getString =<< encoding'
    }
    where
        getString :: BVal  -> Maybe String
        getString (Bstr bs) = Just (unpack bs)
        getString _         = Nothing
        getInt :: BVal     -> Maybe Integer
        getInt (Bint val)   = Just val
        getInt _            = Nothing
mkMetaInfo _ = Nothing
```

> Also, no need of 2 functions. Its just one unpack

How to?


이름 더 잘 구분되게


Please use single quotes


#define으로 대체


Regarding this commit - can you explain a little bit what is intention of it and what exactly is written into sub_slot header field? As far as I know the sub_slot is for half rate traffic channel to identify halves of bursts. We can of course use it for something non-standard but it need to be definded.


else 있을 필요 없음


Should this be a return 0 ?


Isn't this try block superfluous ? There's another one outside doing the same thing...


Please use single quotes.


노쓸모


oık


@giovannibussi, yes I am quite sure its true because I used it for a couple of toy models
with tens of ENSEMBLE keywords with just one ARG, this is why I never noticed the bug.

> On 13 Apr 2015, at 09:36, Giovanni <notifications@github.com> wrote:
>
> @carlocamilloni <https://github.com/carlocamilloni> can you check if actually this is true? The change in 0dd0e3f <https://github.com/plumed/plumed2/commit/0dd0e3f381e475e2280095213b637719382ffd0f> is
>
> setDerivative(v,cv,fact);
> setDerivative(v,i,fact); The second argument should be the number of the cv (0,1,2, etc), whereas it was the value of the cv. I think even with a single argument, this was going to work only when by chance the value of the cv was rounded to zero. Unfortunately c++ was converting the double to int without detecting the problem.
> —
> Reply to this email directly or view it on GitHub <https://github.com/plumed/plumed2/commit/2c39afa79f0adc371733c1f6d3c2973680fdd32f#commitcomment-10687415>.
>


@tchaikov . Look good .updated & plz review again. Thanks!


@rakuco pls dont push to master without review.


I followed this account on WeChat. :smile:


@artpol84 Does this code work correctly? For me -o option doesn't work as expected, output still goes to stdout. BTW, why did you need to redirect stdout to /dev/null? It looks like you did it just if verbose is disabled. But in this case there is no output from clients at all so no need to hide it.


Would change this also to Clap Clap or something Freeletics like (no excuses) but other than that it looks good!


:smile::smile::smile:


sure:D i hate russian lang :D english better :D


:+1:  Only the project description says `Library for accessing Gittip APIs`.


ship it


Oh wow! Can’t wait for the YouTube video. :+1:


commit title should be "even out spacing of request comments"


41:1: I100 Imports statements are in the wrong order. import os should be before import sys
42:1: I100 Imports statements are in the wrong order. import inspect should be before import os
44:1: I100 Imports statements are in the wrong order. import cgi should be before import traceback
47:1: I101 Imported names are in the wrong order. Should be current_app, has_app_context
48:1: I100 Imports statements are in the wrong order. from operator should be before from flask
48:1: I201 Missing newline before sections or imports.
49:1: I201 Missing newline before sections or imports.
50:1: I201 Missing newline before sections or imports.
52:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from werkzeug.utils
53:1: I101 Imported names are in the wrong order. Should be CFG_BIBFORMAT_CACHED_FORMATS, CFG_BIBFORMAT_DISABLE_I18N_FOR_CACHED_FORMATS, CFG_BIBFORMAT_HIDDEN_TAGS, CFG_SITE_LANG
53:1: F401 'CFG_BIBFORMAT_CACHED_FORMATS' imported but unused
53:1: F401 'CFG_BIBFORMAT_HIDDEN_TAGS' imported but unused
54:6: E126 continuation line over-indented for hanging indent
59:6: E126 continuation line over-indented for hanging indent
60:1: I101 Imported names are in the wrong order. Should be create_record, record_empty, record_get_field_instances, record_get_field_value, record_get_field_values, record_xml_output
61:6: E126 continuation line over-indented for hanging indent
69:1: I101 Imported names are in the wrong order. Should be gettext_set_language, language_list_long, wash_language
69:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from .engines
69:1: I201 Missing newline before sections or imports.
70:6: E126 continuation line over-indented for hanging indent
74:1: I101 Imported names are in the wrong order. Should be CFG_BIBFORMAT_FORMAT_JINJA_TEMPLATE_EXTENSION, CFG_BIBFORMAT_FORMAT_OUTPUT_EXTENSION, CFG_BIBFORMAT_FORMAT_TEMPLATE_EXTENSION, CFG_BIBFORMAT_OUTPUTS_PATH, CFG_BIBFORMAT_TEMPLATES_DIR, InvenioBibFormatError
74:1: I201 Missing newline before sections or imports.
74:1: F401 'CFG_BIBFORMAT_TEMPLATES_DIR' imported but unused
74:1: F401 'CFG_BIBFORMAT_FORMAT_OUTPUT_EXTENSION' imported but unused
74:1: F401 'CFG_BIBFORMAT_OUTPUTS_PATH' imported but unused
75:6: E126 continuation line over-indented for hanging indent
81:1: I100 Imports statements are in the wrong order. from invenio.modules.formatter.utils should be before from .config
82:6: E126 continuation line over-indented for hanging indent
83:1: I101 Imported names are in the wrong order. Should be CFG_HTML_BUFFER_ALLOWED_ATTRIBUTE_WHITELIST, CFG_HTML_BUFFER_ALLOWED_TAG_WHITELIST, HTMLWasher
84:6: E126 continuation line over-indented for hanging indent
87:1: I100 Imports statements are in the wrong order. from invenio.modules.knowledge.api should be before from invenio.utils.html
88:1: I100 Imports statements are in the wrong order. from invenio.ext.template should be before from invenio.modules.knowledge.api
89:1: I100 Imports statements are in the wrong order. from HTMLParser should be before from invenio.ext.template
89:1: I201 Missing newline before sections or imports.
90:1: I201 Missing newline before sections or imports.
98:27: E261 at least two spaces before inline comment
99:28: E114 indentation is not a multiple of four (comment)
99:28: E116 unexpected indentation (comment)
100:28: E114 indentation is not a multiple of four (comment)
100:28: E116 unexpected indentation (comment)
101:28: E114 indentation is not a multiple of four (comment)
101:28: E116 unexpected indentation (comment)
104:28: E114 indentation is not a multiple of four (comment)
104:28: E116 unexpected indentation (comment)
118:33: E225 missing whitespace around operator
203:1: E731 do not assign a lambda expression, use a def
206:1: E402 module level import not at top of file
209: D203 1 blank line required before class docstring (found 0)
213: D401 First line should be in imperative mood ('Return', not 'Returns')
213: D400 First line should end with a period (not 's')
225: D401 First line should be in imperative mood ('Return', not 'Returns')
250: D102 Missing docstring in public method
289: D103 Missing docstring in public function
295: D401 First line should be in imperative mood ('Format', not 'Formats')
295: D400 First line should end with a period (not 'f')
295: D205 1 blank line required between summary line and description (found 0)
342:45: W503 line break before binary operator
342:45: E127 continuation line over-indented for visual indent
358:8: W503 line break before binary operator
485:27: E225 missing whitespace around operator
488:5: E303 too many blank lines (2)
506:82: E225 missing whitespace around operator
527: D103 Missing docstring in public function
543: D401 First line should be in imperative mood ('Return', not 'Returns')
543: D202 No blank lines allowed after function docstring (found 1)
543: D400 First line should end with a period (not 'g')
543: D205 1 blank line required between summary line and description (found 0)
564:64: E261 at least two spaces before inline comment
564:65: E262 inline comment should start with '# '
571:38: E261 at least two spaces before inline comment
571:39: E262 inline comment should start with '# '
572:48: E261 at least two spaces before inline comment
572:49: E262 inline comment should start with '# '
575:24: E127 continuation line over-indented for visual indent
584: D103 Missing docstring in public function
587: D400 First line should end with a period (not 's')
587: D200 One-line docstring should fit on one line with quotes (found 3)
600: D210 No whitespaces allowed surrounding docstring text
600: None: None
600: D400 First line should end with a period (not 'a')
600: D205 1 blank line required between summary line and description (found 0)
632:57: E126 continuation line over-indented for hanging indent
639:5: E303 too many blank lines (2)
698: D401 First line should be in imperative mood ('Evalute', not 'Evalutes')
718: D401 First line should be in imperative mood ('Analyse', not 'Analyses')
718: D202 No blank lines allowed after function docstring (found 1)
787: D401 First line should be in imperative mood ('Return', not 'Returns')
787: D400 First line should end with a period (not 't')
787: D205 1 blank line required between summary line and description (found 0)
886:13: B901 blind except: statement
917:78: E225 missing whitespace around operator
918:54: E225 missing whitespace around operator
950:47: E261 at least two spaces before inline comment
965:27: E225 missing whitespace around operator
1025: D401 First line should be in imperative mood ('Filter', not 'Filters')
1035: D401 First line should be in imperative mood ('Searche', not 'Searches')
1035: D400 First line should end with a period (not 's')
1035: D205 1 blank line required between summary line and description (found 0)
1061:9: E303 too many blank lines (2)
1075:5: E303 too many blank lines (2)
1078:1: E302 expected 2 blank lines, found 1
1078: D401 First line should be in imperative mood ('Return', not 'Returns')
1092:5: F841 local variable '_' is assigned to but never used
1095:12: E127 continuation line over-indented for visual indent
1104:16: E126 continuation line over-indented for hanging indent
1126:5: B901 blind except: statement
1138: D401 First line should be in imperative mood ('Return', not 'Returns')
1138: D400 First line should end with a period (not 's')
1158:16: E126 continuation line over-indented for hanging indent
1165:1: E302 expected 2 blank lines, found 1
1165: D401 First line should be in imperative mood ('Return', not 'Returns')
1165: D400 First line should end with a period (not 'e')
1197:13: E303 too many blank lines (2)
1211: D401 First line should be in imperative mood ('Return', not 'Returns')
1245:16: E127 continuation line over-indented for visual indent
1255:31: E126 continuation line over-indented for hanging indent
1282:9: B901 blind except: statement
1294:17: E126 continuation line over-indented for hanging indent
1301:1: E302 expected 2 blank lines, found 1
1301: D401 First line should be in imperative mood ('Return', not 'Returns')
1301: D400 First line should end with a period (not 'e')
1345:1: E302 expected 2 blank lines, found 1
1345: D401 First line should be in imperative mood ('Return', not 'Returns')
1345: D202 No blank lines allowed after function docstring (found 1)
1472:1: E302 expected 2 blank lines, found 1
1472: D401 First line should be in imperative mood ('Return', not 'Returns')
1472: D202 No blank lines allowed after function docstring (found 1)
1499:17: E225 missing whitespace around operator
1578:28: E712 comparison to False should be 'if cond is False:' or 'if not cond:'
1582: D401 First line should be in imperative mood ('Return', not 'Returns')
1582: D202 No blank lines allowed after function docstring (found 1)
1582: D400 First line should end with a period (not '}')
1599:56: E225 missing whitespace around operator
1635:1: E302 expected 2 blank lines, found 1
1635: D203 1 blank line required before class docstring (found 0)
1635: D204 1 blank line required after class docstring (found 0)
1635: D400 First line should end with a period (not 'n')
1635: D205 1 blank line required between summary line and description (found 0)
1666:15: E261 at least two spaces before inline comment
1668: D401 First line should be in imperative mood ('Create', not 'Creates')
1700:31: E261 at least two spaces before inline comment
1725: D401 First line should be in imperative mood ('Return', not 'Returns')
1725: D400 First line should end with a period (not 'e')
1740: D401 First line should be in imperative mood ('Return', not 'Returns')
1740: D400 First line should end with a period (not 'd')
1763: D401 First line should be in imperative mood ('Return', not 'Returns')
1763: D400 First line should end with a period (not 'e')
1763: D205 1 blank line required between summary line and description (found 0)
1801: D401 First line should be in imperative mood ('Return', not 'Returns')
1801: D202 No blank lines allowed after function docstring (found 1)
1910:36: E127 continuation line over-indented for visual indent
1913: D401 First line should be in imperative mood ('Return', not 'Returns')
1970:33: E127 continuation line over-indented for visual indent
1981:60: E251 unexpected spaces around keyword / parameter equals
1983:54: E251 unexpected spaces around keyword / parameter equals
1993:37: E127 continuation line over-indented for visual indent
2004:64: E251 unexpected spaces around keyword / parameter equals
2006:58: E251 unexpected spaces around keyword / parameter equals


I am not sure if we should ignore this messages per default. As far as I could find out the target is not allowed to send Async Event Messages of Type 0 unless its explicitly activated.

"If the target supports SCSI asynchronous event reporting (see SAM-2) as
indicated in the standard INQUIRY data (see SPC-3), its use may be enabled
by parameters in the SCSI Control mode page (see SPC-3)."


@andrejoshua  masih ada yang bermasalah, di check login..


done


@kostiakoval Any way to remove `(error)` compiler screams help if you do it


I like to have included the unit in the variable names. This improves the readability of the code. If the unit is included in the variable name (e.g. switchOverTimeMnUs) or appended as suffix doesn't matter.


1. For me it works well, I test it on our
2. Clients MUST pring OK in any case. But I don't want to see it in "quiet" mode. However in quiet mode and redirection ("-o prefix") I want to see it in the "prefix.<rank>" file. In your implementation this wasn't the case but I need it for automate testing.


Okay, I think I got it.

You need to address the parent label of the checkbox, which isn't possible with CSS. IMHO it would be better to change the Helper\form_checkbox, to apply the class to the label instead of the checkbox.

Existing code can be changed to something like:

```
label.class input[type="checkbox"] {
  // style the checkbox
}
```


I want to make hg metrics tab with rails-like ui, may be problems with active tabs will disappear.


Why should I use `el.Listener.Close()` directly


Yey elia! <3


We can also have smaller (CN) node numbers for a redundant MN. Even if the spec says: "If the configuration tool does not manage RMN priorities, the RMN shall have their Node-Id from 241 to 250", we cannot be sure the the node number is > 240. I would suggest to check for node numbers < 241 and set the priority to (250 - 240) + nodeId in this case to avoid invalid priorities.

I also would use C_ADR_MN_DEF_NODE_ID instead of the constant 240 and introduce two new definitions
e.g. C_ADR_MN_FIRST_NODE_ID -> 241, C_ADR_MN_LAST_NODE_ID -> 250


Yes, i've checked this. We don't need this on current customer environment.


facebook po'o.... -_-

On Mon, Apr 13, 2015 at 2:56 PM, alvarisi <notifications@github.com> wrote:

> @andrejoshua <https://github.com/andrejoshua> masih ada yang bermasalah,
> di check login..
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/alvarisi/KIJ-chat/commit/ae92500e51abe2d3a462c36a2d7a02de1a36c559#commitcomment-10687615>
> .
>


Kleiner bugfix, hat sich irgendwie eingeschlichen ^^


Most repositories don't run pinned revisions, so I'd rather keep my local settings in line with projects I'm likely to contribute to instead of accidentally breaking their convention


Looks good!


I checked. What you see on the screen is the output of the server. Server is not affected by the "-o" option. In the files you can see what each client has printed.


there was one lgtm in the pull request, even if from someone who's doesn't normally work on oz-wl, and it was not clear at all if you were going to review it


Fair enough!


A nice catch ! I will integrate this in my own customization patches at some point.


This is now fixed upstream.


Approved access.

-------Roberto J Villegas

On Mon, Apr 13, 2015 at 3:09 AM, t2t2 <notifications@github.com> wrote:

> Fix tests
> Removed scrutinizer until @vincent404 adds permission for it to access repositories
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/cosmicradiotv/livehub/commit/8b553c6138a1d326de40fd1b43434e74cc1bd159


@jensrutschmann
"'Psuedo-element' and 'Justice/Law'"
This is not always true, according CCS3 -
"UAs that allow wrapping at punctuation other than spaces should prioritize breakpoints. "
For more details please refer to
http://www.w3.org/TR/css3-text/#line-break-details


added.


removed.


sometimes it's cheaper to check than try to open. but if you insist..


Looks a lot better now, thank you Ysurac.


I did not know that vim now includes an append mode, neat!
That requires a recent version of vim though, and for instance the vim shipped in Ubuntu 14.10 (7.4.273) is too old. The README should be updated to reflect that neomake requires vim >= 7.4.503.


is it possible that the `reweight` is 0, but `kb_used` is greater than 0 ? or an item whose `kb_used` is 0, but its `reweight` is non-zero?  i mean, shall we check `reweight` instead of `var` here?


Please, Please revert it. The same as written in https://github.com/kordianbruck/miniflux/commit/44b9652c5cc8e99af6fe6ecfe6ca51d370fede6c#commitcomment-10687364 applies to this.

First of all I don't like the idea of having coloured tags and it isn't a strict required to use tags. IMHO that isn't the lightweight solution that should be implemented.

It breaks the complete miniflux design and I doubt that it is mergable. With the current miniflux design in mind, It looks like a kid used a crayon on top of the miniflux UI.

The coloring should go into a custom theme. We can add little hints to the label, which allows the theme designers to add per tag styling. To pick up https://github.com/kordianbruck/miniflux/commit/44b9652c5cc8e99af6fe6ecfe6ca51d370fede6c#commitcomment-10687659 we could add the class ```btn``` and another class with an incrementing counter (```btn-i++```)  to the label. Similar is used by the cards theme to add different background colors to different feeds (https://github.com/miniflux/theme-cards/blob/master/css/feed-color.css).

Furthermore you added styling related stuff to the Model. It took me a lot of time to remove similar mistakes from the miniflux code (check the miniflux commit history for details).

And the last note on this: Your commit includea formating only changes again. It's a real pain in the ass to review such patches. Open source projects I'm aware, reject patches because of unrelated code change regularly.


thank you!


readability reduced as requested.


Yeah, but individual objects will probably take much more space and reading from many individual objects might be much slower than reading from a single pack file. That's because Git compresses objects in pack files really well.


Yes, this could improve readability but it isn't consistent. We have a lot of time values in the struct and none of them includes the unit.


Hi,

On Fri, 10 Apr 2015 05:48:32 -0700
Rafaël Carré <notifications@github.com> wrote:

> On 04/10/2015 02:16 PM, Tomas Novotny wrote:
> > Hi,
> >
> >> Hello,
> >>
> >> I wish I had seen your work prior to starting this :)
> >
> > Yes, it's pitty that we did it twice...
> >
> >> Merging would be fine, I'll need to get your branch and see how it works for me,
> >> and then send you any changes I need. How does this work?
> >
> > I'm not sure if I understand correctly, are you asking for current status? From the first quick tests it seems that everything for ODROID-C1 works as expected - both u-boot and linux are build from sourcecodes (Hardkernel's github repos) and bootable SD card image is created. It can be flashed using dd and it boots.
>
> Ok, same status here, although I am not 100% sure of the u-boot fix:
> https://github.com/funman/meta-odroid/commit/93d1b80ed98ef44bbb6ba2e3277aec2fdd7a44ea

We did also some tweaking of the compiler arguments needed for u-boot. See
that line:

```
SRC_URI_append = " ${@bb.utils.contains('TUNE_FEATURES','callconvention-hard',' file://0002-added-hardfp-support.patch ','',d)}"
```

in recipes-bsp/u-boot-hardkernel/u-boot-hardkernel_2011.03.bb

> > There is just one issue - with core-image-minimal there is no u-boot.ini and DTB in the /boot directory of the image. I added
> >
> > ```
> > IMAGE_INSTALL += " \
> >         u-boot-ini \
> >         kernel-devicetree \
> >         "
> > ```
> >
> > to my image's definition and it works as expected.
> >
> > What about you layer? Is it "production ready" or it still needs some more work?
>
> Definitely would need some testing.
>
> I started with core-image-minimal target, but I made some changes since
> and I'd need to reverify that
> core-image-minimal still works.
>
> >> BTW we should get in contact with openembedded to have meta-odroid listed somewhere,
> >> so people can find some authoritative source.
> >
> > That's true, probably layers.openembedded.org is the best place. I was just asking in the Yocto thread on ODROID's forum and found that there is no C1 support yet. I found one guy (Christian) also interested in the support for C1 so we made the current layer. Next time I will do better marketing ;)
>
> Ok, I wanted to announce it on ODROID forums too but my activation link
> had failed.

oh :( ... there is "Yocto support" thread.

> > But the problem is that I have only C1. Other devices in the repository are here only because of the fork, I didn't test them... So it is somehow "production ready" only for C1.
> >
> >> Which devices do you have? I have a XU3-lite and a C1.
> >
> > I have C1 only and I won't have any other in the near future.
>
> OK well I can test on XU3 and hopefully we can find some people on the
> ODROID forums.

XU3 isn't supported in my meta-odroid, I just added C1 to the forked
meta-odroid.

> >> Thanks for the heads up
> >
> > Thanks for you answer,
> >
> > Tomas
>
> I will be gone for a couple of weeks so not sure if I'll give you
> updates on this, but I'm definitely interested into
> working on your branch rather than duplicating the work!

ok, thanks for the info. It would be nice to have more maintainers.
	Thanks and best regards,

	Toams

> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/funman/meta-odroid/commit/c1bdd426ca6143a36ebf921590310e7978aec819#commitcomment-10665440


done.


@kordianbruck Bitte nimm die Kritik nicht persönlich. Ich will uns nur zum Bestmöglichen Ergebnis pushen.


yes, I am going to review patches going to master


reworded.


I don't submit a patch because i have not enough knowledge of this part, so i explain the issue, how thing should behave, suggest some options, for those with the competant skills to fix (and/or) implement it.
I see two ways (but maybe there is more) :
Either not overiding the position of SSA/ASS calculated by the renderer if "Overide SSA" in subtitles option is not selected, or add a 6th option in the positions, like "Automatic", for which behavior should be :
If subtitles is SSA/ASS, not overide the the calculated position, if subtitles is not SSA/ASS, use the "fxed".


Well done :D


new


hmm. logically it should but there was a reason for the true (likely a hack/TODO). i need to look into it..


well, i'm not yielding my copyright.


This commit introduces major bugginesses!
Outputted gcode contains short paths with lots of extrusion, outsets which have weird geometrical properties (45 degrees angles everywhere), infill connects wrong lines.
etc.


done


@yuyuyu101 . I'm only replace the real data copy(not include hole data copy) w/ _do_copy_range.


,m,m


I'm confused, shouldn't the Window Title be updated to the number of selected clients after a selected client is disconnected?  lstClients.SelectedItems.Count will get the clients that are still selected if a client that is also selected disconnects


ytyyttyt


I put it after the foreach-loop, so the disconnected client is already removed.


fix issue https://github.com/zurb/foundation-rails/pull/117


Just tested print functionality from my home computer. Worked great for me! Nice work!


It was to make sure that the heroes would kill the enemy dungeon if you do nothing ^^
But I agree 4 level 25 BigKnights are not easy to beat


jedisSentinelPool.returnBrokenResource(resource) & jedisSentinelPool.returnResource(resource) are deprecated.
And what about jedisSentinelPool.returnResourceObject(resource)?


Make sure the port you connect to is opened and the url/ip is correct


Hi KalleDK, I'd like to say it's some tool that generates the datasheet automagically, but it's a hard-written Word document. I made it a template for you, it includes the font used.

https://drive.google.com/file/d/0B087zoyPh32WWFNtV083R2NFRTg/view?usp=sharing

Cheers.


http://lists.freedesktop.org/archives/mesa-announce/2015-April/000150.html

python and mako is not needed anymore :)


Straight forward. I had not tested with console ^^


The function name is awkward. You should rename `spacemacs/useful-buffer-p` to `spacemacs/useless-buffer-p` because you are checking current buffer is useless, if so continue looping until a useless one is found, not continue looping when a useful one is found. Also this could cause infinite loop when no useful buffer is found, and I fixed it in previous PR, but by mistake I overridden my develop branch and hence the commit in PR. You should fix `spacemacs/next-useful-buffer` like this:

```elisp
(defun  spacemacs/next-real-buffer ()
  (interactive)
  (let ((start-buffer (current-buffer)))
    (next-buffer)
    (while (and (useful-buffer-p (buffer-name (current-buffer)))
                ;; keep looping until returns to starting buffer
                (not (eq start-buffer (current-buffer))))
      (next-buffer))))

(defun  spacemacs/previous-real-buffer ()
  (interactive)
  (let ((start-buffer (current-buffer)))
    (previous-buffer)
    (while (and (useful-buffer-p (buffer-name (current-buffer)))
                ;; keep looping until returns to starting buffer
                (not (eq start-buffer (current-buffer))))
      (previous-buffer))))
```


Yes. I would prefer `>>=`.

Its ok to have `get<type>`. There is already a `bstrToString` in `Bencode.hs`. Please add all getters there together and import them. This might get used in a few other places, so no need of limiting it inside the where clause.

Good job by the way ;)


Maybe here you could use the new Swift 1.2 syntax (Firefox for iOS requires Xcode 6.3 with Swift 1.2 as of April 9):

```swift
if let scheme = url.scheme,
       index = find(ignoredSchemes, scheme)
{
    return false
}
```


My bad!  That works too


No problem!


I don't recall at what point the values are read back from the entries
in the monitor centre gui. If I attached them to a callback for when the
entry changes then this is fine (because the entry won't count as having
changed). But if I set it to read back all the info from every field
when the save button is pressed (which is effectively what I did with
dialog boxes in the Builder) then the zeros will get saved.

On 13/04/2015 00:01, Jeremy R. Gray wrote:
>
> I am unsure if those lines are only setting the value for display
> purposes, inside the relevant |ctrls| (in which case it should be no
> problem? or at least not the problem you raise), or if those changes
> are later propagated back to |self.currentCalib['distance']| (in which
> case it might be an issue).
>
> |mon.getDistance()| just returns |self.currentCalib['distance']|.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jeremygray/psychopy/commit/449e422a8e6d100f93ac096f75e6a40ab4a4d9da#commitcomment-10684604>.
>


I think that will not work.

StartScan is not a function of self , but more of RaidCore object.


You should change the name of `useful-buffer-p` to `boring-buffer-p` to signify that we are checking for boring buffer. If we find one, we return `t` to continue looping, otherwise returns `nil`, like this:

```elisp
(defun boring-buffer-p (buffer-name)
  (catch 'found
    (mapc (lambda (regexp)
            (when (and (not (member buffer-name spacemacs-interesting-buffers))
                       (string-match regexp buffer-name))
              (throw 'found t)))
          spacemacs-boring-buffer-regexp-list)
    nil))
```


you can get Xcode UUID in the Applications/Xcode/Contents/Info.plist file, DVTPlugInCompatibilityUUID key's value.


@maximilien, yeap, will confirm this with Shannon via tracker


Does it? Seems to me it's default when type is nil.


아뇨. 자동으로 생성되는데, 로그인을 해서 세션이 DB에 저장되도록 해보시겠어요?


This is test repo.


Increasing the timeout makes both of these pass for me...

        protected override string GetConfiguration()
        {
            return "127.0.0.1:6379,syncTimeout=10000";
        }


Hm, You are right. I added a new commit that corrects this mistake.


hey, we need to keep markup out of the JS - it's much harder to maintain and design for - how about duplicating an existing bit of markup in the DOM instead of this.


`error in ` .. @3lvis


Sorry, we use github for code discussions only. If you don't have the skills to comment on code please refrain from posting here. Use forum or trac instead.


no, you will try to fix it with rebase command in dev3 branch :)

That will be a good training. :)


kube-dns is a service (see https://github.com/GoogleCloudPlatform/kubernetes/tree/master/cluster/addons/dns). It is enabled by default for example on GCE. I changed this example to use dns instead of HOST, PORT env vars, for example instead of using REDIS_MASTER_SERVICE_HOST as an address of redis master, I use just redis-master which will be resolved to the redis host by kube-dns.


js should be targetting js specific classes (js-whatever)


This should stay in setParameter() as it will need to be run every time a parameter changes (not yet implemented)


From what I read, this is how STI behaves by default. For base class it always set nil.


Hehe


each character hero/monster could have different abilities?
max health for monsters/hero
each monster could level up i e get more difficult as the game progresses
on the different places of map, could place different bonus for map, armor increase, weapon increase, etc


FYI, you can just invoke every task in a role with sudo when including it in a playbook, like this:

```yaml
roles:
  - { role: JasonGiedymin.sbt, sudo: true }
```


50 should come from a parameter stored as a variable.


Why do you need to nest it in BuildUtil?


lastAccess time is not updated


Looks good.


Builds return the following error for VS10 at least:

101>C:\dev\AUTO_PACKAGE\head\OpenMS\src\openms\include\OpenMS/FILTERING/TRANSFORMERS/SpectraMerger.h(312): error C2668: 'log' : ambiguous call to overloaded function
101>          c:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\math.h(575): could be 'long double log(long double)'
101>          c:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\math.h(527): or       'float log(float)'
101>          c:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\math.h(120): or       'double log(double)'
101>          while trying to match the argument list '(int)'
101>          ..\..\..\OpenMS\src\topp\SpectraMerger.cpp(150) : see reference to function template instantiation 'void OpenMS::SpectraMerger::average<OpenMS::PeakMap>(MapType &,OpenMS::String)' being compiled
101>          with
101>          [
101>              MapType=OpenMS::PeakMap
101>          ]
101>
101>Build FAILED.


Would like to see the entire contents of the for loop into another function.
Then use `defer unitFileCache.mu.Unlock()`


We found some smaller issues, mostly cosmetic. I will correct them by myself before integrating the change.


>Why do you specify `(error) -> Void` ? remove it

Is there a way to drop `error` at all?


Are you sure the Cozy can handle it?


typo?


fixed issue #18 and #19


all green


Yes! didn't know that. :) comiting with it


This should be

var offerid = user.profile.current_offer_id;
Meteor.users.update({_id: user['_id']}, {$push:{"profile.failed_offer_ids":offerid}});


The replacement log(2) -> log(2.0) should do the trick, right? I will prepare the fix.


so should i add it to the cart?There will be no difference between buy one product and buy all the products from your cart.In my opinion,there are no sense of doing it.


@taku0 short answer is "no".
```
11:42 < the-kenny> i_rony: if stuff like that happens, who guarantees two hashes are enough? We always might have to add another one. And then another one. And still another one. This is NOT the way to go for deterministic builds.
11:43 < the-kenny> i_rony: A mirror would be the correct solution.
...
11:47 < Lethalman> i_rony, doesn't make sense to specify multiple hashes
11:48 < i_rony> Lethalman: even if one decides to server different versions of same file for different regions?
11:49 < Lethalman> i_rony, if it's the same file, it has the same hash
11:50 < i_rony> Lethalman: contents may differ a bit. I don't know, some legal stuff
11:51 < Lethalman> i_rony, provide two different packages, then
11:51 < Lethalman> the user chooses based on the region
```