Markdown Link


This assumption may break if we decide to implement other ways of generating mazes.


Will this do the trick?


To be consistent with the rest of the coding style, prefer ++i instead of i++


Hey Rado: Is there anyway to take advantage of typed generics?

e.g.

```dart
class MyParentComponent {
  @Query()
  final QueryList<SomeDirective> childDirectives;

  MyParentComponent(this.childDirectives);
}
```

(Example is in Dart, but I assume TS generics will be supported soon)

Having `@Query(SomeDirective)` seems repetitive.

Just a thought, not a blocker.


Sure.


The link only shows the direct definition that bypasses braiding. But
yes I see that I could formalize it by saying braid xy o braid yx = id

I'll do it that way... Nevertheless what am I missing in the current
attempt? --Amr

On 03/25/2015 12:55 PM, Jacques Carette wrote:
> A symmetric monoidal category is a braided monoidal category with one
> more equation.
> http://en.wikipedia.org/wiki/Symmetric_monoidal_category.
>
> That is definitely how I would go about formalizing it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JacquesCarette/pi-dual/commit/e37c05c5d747403d7ca1770b0fe540375571ea58#commitcomment-10394608>.
>


While testing I noticed that the request fails when the SP is not requesting any context (unspecified).
In such cases I assume that any context used should still be allowed based on the previous version.

Can we revert the change on lines 355-376? Or is there any other way to get a successful login when the SP doesn't request a context?

Regards,
Alessandro


This feels like it was intended to be an &&, or the Distance check should be removed entirely, otherwise worms within a certain distance ignore the line-of-sight check.


Carbon is likely to be released the second half of 2016... You should probably be more optimistic than that (devel is currently slated to be Beryllium )


Could this be converted to a cursor? :)


Исправлено в 182b300a06f2502fd5cdb060387089402e4e5d8c


@jonschlinkert why you use double quotes in some places? I notice that also in *some* of `kind-of` tests.


Добавил ворнинги в код, чтобы не забыть.


Cache here really needed? Keystorage also caches the keys, but only per request. Do you take care to replace the key in the cache correclry if a key was updated?


Сделано в 424e703a384e4ba16d702e13ff7dc566b0e61b7f


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1163 of 1165 tests passed)

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


see comment here: https://github.com/rust-lang/rust/pull/23673#discussion_r27141708


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


03-26 01:02:10.691: E/AndroidRuntime(18951): java.lang.NoClassDefFoundError: android.support.v7.app.ActionBarActivityDelegateBase$2
03-26 01:02:10.691: E/AndroidRuntime(18951): 	at android.support.v7.app.ActionBarActivityDelegateBase.ensureSubDecor(ActionBarActivityDelegateBase.java:316)
03-26 01:02:10.691: E/AndroidRuntime(18951): 	at android.support.v7.app.ActionBarActivityDelegateBase.setContentView(ActionBarActivityDelegateBase.java:237)
03-26 01:02:10.691: E/AndroidRuntime(18951): 	at android.support.v7.app.ActionBarActivity.setContentView(ActionBarActivity.java:102)
03-26 01:02:10.691: E/AndroidRuntime(18951): 	at com.mao.testv7demo.SplashActivity_.setContentView(SplashActivity_.java:40)
03-26 01:02:10.691: E/AndroidRuntime(18951): 	at com.mao.testv7demo.SplashActivity_.onCreate(SplashActivity_.java:32)


Hooray for the rename.


3: M102 unrecognized bullet/signature


@jonschlinkert why you use double quotes in some places? I notice that also in some of `kind-of` tests.


Maybe we should remove the initial encryption completely. This creates a lot of trouble with timeouts for large installations. We should recommend to enable encryption in the beginning if the admin want to make sure that all files are encrypted. Maybe for large installations we can provide a occ script in the future


This isn't the right place for this. The update should be in the monitor/management/commands/readCheck.py file. Can you revert this? You're actually working on an older branch of code where the line you changed was already fixed.


The current attempt seems fine to me.  I just wouldn't go down to objects, but stay at the Functor level for the symmetry equation.


Either way, the requirement/dependency on these libraries should be strictly optional and have an easy-to-use configuration define to disable them. They are complicated dependencies to obtain and maintain, particularly on Windows where there is no apt-get or similar dependency fetching mechanism. A Doom source port should not logically require video encoding libraries.


oops, sorry for the silly mistake!


This is, however, a good time to make sure you're always doing a pull to sync up your local repo with the latest changes before modifying anything =P


And of course, Bimonoidal needs 2 monoidal structures (!), with a distributivity condition and coherence laws.


> Could this be converted to a cursor? :)
Maybe after the first merge ? I agree that the legacy code can be improved and cleaned-up, but it works and I think this is good enough for now.


if we decide to skip the initial encryption we can remove this method


Just to be sure: this change means that it won't be possible to install your extension on any XWiki instance older than 6.4.2.


I deleted `config.h` only because it conflicted with the generated one while adding CMake support. I think this change can be reverted.


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


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


@mpapis Just to make things clear for people who find this later:

Every single person on both the Bundler and Rubygems teams told you that rubygems-bundler was a bad idea, and you not only made it, you added it to rvm by default.

You know that rubygems-bundler has created a significant number of confusing and hard to debug tickets for Bundler, but you refuse to remove it as a default.

You know that Rubygems is not actually able to replace Bundler, but you insist on using it instead.


so funny


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


TeamCity xBIM :: Xbim Geometry :: Release [Build 44](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1208&buildTypeId=XBIM_XbimGeometry_Release) is now running


TeamCity xBIM :: Xbim Geometry :: Release [Build 44](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1208&buildTypeId=XBIM_XbimGeometry_Release) outcome was **FAILURE**
Summary: Exit code 2 Build time: 00:00:06


Indeed I agree with Thomas. Is there a strong reason to upgrade the minimal XWiki Commons required?


PS: please apply the xwiki formatting to the pom too if possible ;)


What would be the solution then? Stuck with 4.0rc-1 and all the deprecated calls?


Sure. Should I add this information to `README` as well?


@format: mvn xar:format did not do the job for pom.xml, xml format profile for eclipse?


ah I see it was part of the original test in #11591

Well in that case in the spirit of true regression testing, I recommend you encode it instead as `1i32 % 1i32`.


help your mod ae2 will not load on my server :( he is crashed


xar:format is for formatting XAR pages (wiki pages), not pom.xml files.


putting rotation speeds back to default value


Test fix on staging:
- [ ] Async request from https://staging.commcarehq.org/a/crs-remind/reports/case_activity/?emw=t__0&case_type=' should no longer 500


I too get the same error...


You should not create a HashMap (that is not case insensitive like the attributes map) and not do idName.toUpperCase()


@robertomiranda Hola, actualmente no hay forma de actualizarla. De hecho estamos haciendo algunos cambios en los puntos físicos, mientras tanto voy a actualizar la lista de los establecimientos soportados. Estamos preparando una nueva versión del API, me parece buena idea la de obtener una lista de los establecimientos.


@frrrances this is a change Brian made - when do we use `@include margin-right` versus just `margin-right`?


If you added a space after `signal,` you might as well add one after `2000,`...  :-)


summary is mandatory and 90% of them in other database's configure files are 'false'-->


I think my recommendation would be to look at the log messages to understand the changes, but to mostly review and comment on the overall diff, via the Files changed page.  But if you do want to review commits individually, you can of course look at the final diff to see whether those lines are changed further in a subsequent commit.

In this case, yes, the code to transfer values is gone, as is all of SdssShapeImpl.


No rush, just wanted to put a marker on it for future cleanup.


TeamCity xBIM :: Xbim Geometry :: Release [Build 45](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1209&buildTypeId=XBIM_XbimGeometry_Release) is now running


TeamCity xBIM :: Xbim Geometry :: Release [Build 45](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1209&buildTypeId=XBIM_XbimGeometry_Release) outcome was **FAILURE**
Summary: Exit code 2 Build time: 00:00:01


This looks off indentation


any reason you went this route rather than adding an annotation to `0`, which I infer is all you need ?  (Are you trying to hide the scope of the fallout here?  Conspiracy! ;) )

(Or wait, is that actually all you need?)


What should happen if the attributes map is empy and user whats to do model.toUpdate()?  I don't think calling appendEmptyRow() is the solution -- this will create something like `UPDATE students DEFAULT VALUES WHERE id = 1`, which is clearly wrong.


Hm.  `array` is not any more descriptive than `contents`, maybe less so.  How about `files`?


TeamCity xBIM :: Xbim Geometry :: Release [Build 46](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1210&buildTypeId=XBIM_XbimGeometry_Release) is now running


duh, `0` should fallback to `i32`, so I actually do not know what your new version is avoiding, if anything.


The analogRead returns a string also.


This doesn't compile for me, could you change it to:
```java
return getList(property, type, PathBuilder.class);
```

The same for all other creation methods that use PathBuilder.class.


(Do integral types on the RHS not fallback to `i32` in your PR?  I will go review again.)


Hi Clem - I happen to use this feature often. Take screenshot, edit it using Pinta from recent files in menu. Any chances of keeping it configurable? This was originally requested here.
https://github.com/linuxmint/Cinnamon/issues/1435


Oh crap i missed that line. thanks for pointing it out!


Can we add info about having resource constraints available to build now?
So... `docker build --cpu-shares=100 --memory=1024m <etc>` is now supported.


Cheers :)


So, we don't need the travis.sh script any more?  Then we should delete it as part of this merge.


If we need any model to have comments, then we need any model to paginate


This will work for any new model as long as an id field for it is added to the post


Also `--default-ulimit` option on the daemon which, which applies to all created containers (and overwritten by `--ulimit` on run).

`--default-ulimit` also includes build containers.


indent this line.


:trollface:


Why not just `make`?


@kyranstar Can you also change the namespaces in these files from `HuntTheWumpus.Platform.Desktop.Map` to `HuntTheWumpus.SharedCode.Map`?

P.S. It looks like you meant to commit a `Cave` class along with this code, which wasn't added to SC.


90693826-Central-cooling-coils-not-autosizing-to-correct-airflow-in-some-cases-#4810 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (654 of 657 tests passed)

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


You can also write

    resultMap.get("wards_chicago_mid_simp") must be ('defined)

I believe.


I don't like the loop structure you created.

You already tested that !attributes.isEmpty() before. So:

    for (;;) {
        // do stuff ...
        if (it.hasNext()) {
            query.append(", ");
        } else {
            break;
        }
    }
    query.append(" WHERE ")... // with the space before WHERE

Also, please mind the indentation (4 spaces). And don't use tabs.


yeah! sounds good.


http://winapi.freetechsecrets.com/win32/WIN32GetCurrentThreadId.htm ?


Thanks @ajduke :+1:


Well, it's in the config directory for something :P


You only need one of these two lines (either include jquery from code.jquery or from ajax.googleapis, but not both).

You have to put these BEFORE you put your jquery.js file.


> What would be the solution then?

It depends what is your goal, here you explicitly indicated that you need XWiki 6.4.2 or more because you have dependencies that use this version. The first thing to do is ask yourself what minimum version you want to support and then put that as dependency.


If that's not Windoze, it doesn't imply it's Linux ;)


Ok, but it is running now. I tried to stop and modify the test, and I am not allowed (gives an error message).


It that really one tab more indented?


Fantastic.  @chrisseaton Please let me know if you do plan on loading all of Rubinius kernel because it will conflict with my converter branch.


You are right, the goal is to get it running in all versions of xwiki. What is the mininum we want to support? (Changing back to 4.0rc-1)


yup =)


Quick note on this, because I ran into it today - this commit breaks formatted numbers put though `JText`.

```php
$formattedNumber = '$50,000';

// Before this commit
echo JText::_($formattedNumber); // $50,000

// After this commit
echo JText::_($formattedNumber); // $50
```
I encountered this in a round about way - I had select fields generated via `JHtml::_('select.options')` where the last parameter was set to `true`, which informed `JHtml` to request translated text values from `JText`, which breaks the formatting because it contains a comma and passes the safety checks in the code.

@mbabker @nonumber


@gmr, when I saw the use of write lock here, I held out hope that pika implementation might be thread-safe. However, upon further examination of the code in `adapters.BaseConnection._handle_write`, I noticed that `outbound_buffer` is not protected by the lock in the ioloop poll to  `adapters.BaseConnection._handle_write` code path is not protected.

So what is the purpose of locking with `_write_lock` here if the other code path that modifies the same `outbound_buffer` (popleft) does not guard it with the lock?


Cave is supposed to be somebody else's responsibility so I'm waiting for him to commit :palm_tree: but yeah I'll change it when I get home


3: L102 copyright year is outdated, expected 2015 but got 2014


Which JDK did you use for compilation?


I could use a comment here mentioning that AnalyzingNodeVisitor adds more stuff to self.names and self.base_classes.


It is almost a merge of my patch ;-)


I did not add `rubygems-bundler` to RVM, but yes I refuse to remove it as it actually is useful - it solves the problem that you refused to fix. As for the bugs - please point me to them, I am always eager to help in debugging and solving problems related or caused by my software.

As for replacing Bundler with Rubygems - I asked you earlier <https://twitter.com/mpapis/status/577956517388918786>:

> @rubytogether just curious why would you need to support Bundler when RubyGems does most of it's functionalities already?

and you blocked me on twitter for that, thanks for ignoring me. If you instead could spend some time to talk to me maybe you could explain it to me - I do it because I care, there is no selfish reason behind. Anyway I even advertised https://rubytogether.org/ in my talk about migrating to Rubygems - talk to me, explain me why, ignoring me helps nobody.


spawned off of an [earlier comment](https://github.com/rust-lang/rust/pull/23673#commitcomment-10395185) of mine: I might have guessed that we could try using `infcx().type_is_unconstrained_numeric(rhs_ty)` and map it to `i32` or `f64`, the same way that `fn default_type_parameters` does.

Why *not* do that?


here's a comment


No Direct just Pigeon mail :)-


you are right, but val mapRes is used elsewhere also.


lgtm


updated to build99


AF.getUnattacked() nicht PreferredExtensionCalculator.calculate()..


This commit broke some things because I change the AST. Just a heads up :)


Hi and thanks for your contribute. In the last version 1.6.0.14 in the views product packs, The system, in view a package products, in the tab to the 'interior of the package only shows a product.


Doesn't this make it EC2-specific?


you could put 4.0 to start with.


I think `self.class.new` is a little clearer than `Hocon::Impl::ResolveSource.new` since we're in the ResolveSource class.


creating a .csv file with data from root files and then making some simple plots with it


Hi Jonas, are the events you've implemented here not already part of the Dial Event?

DialEvent has a property, subEvent which contains either begin or end.

See: https://wiki.asterisk.org/wiki/display/AST/Asterisk+11+ManagerEvent_Dial

```
SubEvent - A sub event type, specifying whether the dial action has begun or ended.
* Begin
* End
```


@JulianFernando muito boa essa alteração, nós temos outro script que roda no servidor para realizar esses passos que você ta executando remotamente, achei interessante como você fez, tenho alguns pontos de estrutura no jboss que estamos fazendo mas já estamos analisando uma forma de melhorar isso. Na pasta deploy eu tenho uma pasta "versions" onde eu jogo o zip do gsan lá, e ai realizo os demais pontos que você colocou aqui. O detalhe que é legal jogar isso em uma pasta fora do jboss e vamos alterar o script e a estrutura e já vou pegar essas suas melhorias.


Same here


It depends what features you need in your extensions. The first step I guess and see what is the minimum version which which your jar is building and then look at the APIs you are using in your XAR.


talvez seja interessante a gente parametrizar o local onde vai ser jogado o backup, porque fica bem dependente da configuração de cada ambiente, o que você acha?


this isn't strictly speaking correct. Nothing stops me from creating a channel "#&foo" but the '&' wouldn't be handled correctly by our display code. My suggestion would be:
```js
var trimmedName = shortName.replace(/^(#|##|&|\+)/, '') || (shortName ? ' ' : null)
```


The next commit sort've changes it up so that it modifies a result object that's handed to it, may or may not still be necessary there.


We might also want to only have one instance of the `Cave` and pass it from Map to Wumpus on creation.


s/plath/path/


No problem, it's just that I found and fixed that issue myself and wanted to let you know what I believe is the correct fix.


This is very confusing due to the way you skipped curly braces around this. My suggestion is to always put {} around both branches.


Does it compile for you then?
I am using Zulu OpenJDK 1.8.0_31.

```
Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.1:compile (default-compile) on project querydsl-core: Compilation failure: Compilation failure:
com/querydsl/core/types/dsl/PathBuilder.java:[203,38] incompatible types: inference variable Q has incompatible upper bounds com.querydsl.core.types.dsl.SimpleExpression<? super A>,com.querydsl.core.types.dsl.PathBuilder
com/querydsl/core/types/dsl/PathBuilder.java:[327,32] incompatible types: inference variable E has incompatible upper bounds com.querydsl.core.types.dsl.SimpleExpression<? super A>,com.querydsl.core.types.dsl.PathBuilder
com/querydsl/core/types/dsl/PathBuilder.java:[357,31] incompatible types: inference variable E has incompatible upper bounds com.querydsl.core.types.dsl.SimpleExpression<? super V>,com.querydsl.core.types.dsl.PathBuilder
com/querydsl/core/types/dsl/PathBuilder.java:[411,31] incompatible types: inference variable E has incompatible upper bounds com.querydsl.core.types.dsl.SimpleExpression<? super A>,com.querydsl.core.types.dsl.PathBuilder

```


Feature implemented better than the default client, using combination of arrow keys to go all of the 8 directions.


This is not going to work if there's more than 1 subscription per channel


I wonder if this paragraph needs to be adapted also ;-) The interface make use of error_code and not error_condition.


Hey Jesse, we kinda decided to discard the mercurial implementation of our pre tested commit, we currently only have it for Git. We did quite a few design changes so we ended actually never publishing or creating the plugin.


Wayne and I agreed on a solution to the `bundle exec` problem for rvm, but you refused to implement it and wrote `rubygems-bundler` instead.

Ruby Together exists because Bundler _and_ RubyGems need support. No one is paid to work on either of them, but they need maintenance just to stay working.

RubyGems is not able to replace Bundler because its Gemfile functionality is not and never will be the same as Bundler's. Removing Bundler is asking your users to deal with both missing functionality and bugs that they would not encounter if they simply used Bundler.


This was a frustrating change...we had counted on the client being there already, and had to rework some logic or commit to adding those resources. Just a heads up


You can interpolate variables in a string in ruby. So for example, here you could do `"attempt to replace root #{root} with #{replacement}"`. This file is fine the way it is though, you don't need to change it.


Looks great!


I tried 1.7.0_51 which worked. But in 1.8.0 it fails also for me.


> Even if you put back code it still have issue :

What's the issue before this change?


I'm not sure why we should handle this at all really. # or ## or & or #& are all part of the channel name. Why not just keep them?


why is this asterisk red?


Ok, fair enough.


For reference, since @jeremyfelt has added this in the home development chain and that will filter up to the Spine parent theme.


Why was this removed?


This is good progress, but having information duplicated with `doc/stdlib` is going to get awkward quickly.


Can you include `upload-dsym.sh` as well, so we can just refer to it from Build Phases, like so?

```sh
if [ "${CONFIGURATION}" = "Release" ] || [ "${CONFIGURATION}" = "AdHoc" ]; then
sh ${PODS_ROOT}/TestFairy/upload-dsym.sh -d xxxxxxxxxxx
fi
```


I'm wondering if we should come up with a naming convention for implementing different versions of methods with the same name in the java code, as it seems like some differing naming conventions have sprung up.


@brifordwylie I see :-)
I've already rewritten test for decorator. Should I do the same If I see the "old style" somewhere else?


this commit could fixup


Formatted pom.xml with eclipse settings from http://dev.xwiki.org/xwiki/bin/view/Community/XmlCodeStyle


With latest code (this change) it always return true even if I use
removeflags.

Before change it set flag value to flag name instead of setting it to false

On Wed, Mar 25, 2015 at 6:38 PM, Branimir Karadžić <notifications@github.com
> wrote:

> Even if you put back code it still have issue :
>
> What's the issue before this change?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bkaradzic/genie/commit/13f511911d57586ca547aa2f1034a7647886e20e#commitcomment-10395649>
> .
>


That should trigger only when DeleteValue fails and that should happen irrespective of the finally block. Sorry, I might have misunderstood you but when I tried using a pattern like below I don't see any assert.
```C#
try
{
    Registry.CurrentUser.SetValue("Test1", expectedValue);
    Assert.Equal(expectedValue, (int)Registry.GetValue(Registry.CurrentUser.Name, "Test1", defaultValue));
 }
 finally
 {
     Registry.CurrentUser.DeleteValue("Test1");
 }
```


The solution you agreed on with Wayne was not solving the problem it was just a hack to avoid it, proper solution is Rubygems executable hooks and I'm asking to reopen https://github.com/rubygems/rubygems/pull/590#issuecomment-86134288 - so the proper way can be revisited.

As for collecting money for Ruby related work and infrastructure I'm all in, just please don't make it about yourself and Bundler, join efforts with Rubygems and remove all duplicated code, it will help to minimise the time needed on supporting both projects... if you feel like you can explain it to me better why it's bad idea in a chat here is my skype: michal_papis and I use hangouts on `mpapis@gmail.com`.


Thanks @anthonysterling, good catch!


I'll do the changes.


At this point we already know that it was previously empty.


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1149
success: http://build.servo.org/builders/linux2/builds/1148
success: http://build.servo.org/builders/android/builds/1183
success: http://build.servo.org/builders/mac1/builds/1362
success: http://build.servo.org/builders/mac2/builds/1365
success: http://build.servo.org/builders/gonk/builds/331


@Saylenty yeah, in fact most of the test are based on the unittest framework, it's just one of those things I never got around to :)  I kinda feel bad having you fix all of them... if you want to great otherwise I'll get around to it in the next couple of days :)


imports should be sorted alphabetically


I'm not seeing what you changed to fix the hash problem.  Didn't we try this exact solution and get a different answer?  Yes, takes 2 minutes to run for me too.


exported function comments should start with the full function name, this helps godocs work better for the user


IdTracker is definitely null at this point.
Lines 24 and 25 should be removed.


Yes it doesn't show for `CurrentUser` :) Try with `LocalMachine`, `ClassesRoot`, etc.


lgtm :+1: feel free to merge


Yes, as I wrote in other comment, we shouldn't change that behavior not, at least not for FS1.0.

```
globallevel | projectlevel | resulting perm
    0         |      0          |      0
    0         |      1          |      1
    1    |      0     |      1
    1    |      1     |      1
```


That's a good point. We discussed this approach with Misko and Victor. Indeed, it would look better if we can support that syntax. There are however some technical issues when we try to tie ourselves too much to the type system. Victor mentioned some old dart2js bugs which escapes my mind right now. Also I want to wait on TS landing before we start depending on runtime information of generic types (which most likely would be different between Traceur and TS).


Cette virgule n'est pas facultative ?


I would create a struct for the return to encapsulate the relation, iface, and payload.

type Endpoint struct { rel string, iface string, json string }


fast-forwarding master to auto = da2231a8


Any argument validation needed? Also @mattwar and @jasonmalinowski should review.


I think this should be `--` instead of `++` since we're removing a cycle marker


Using $PATH properly would not have been a hack. The way `rubygems-bundler` monkeypatches RubyGems executables is a hack, and the entire RubyGems and Bundler teams have asked you politely to stop.

Furthermore, Ruby Together is not about myself and Bundler, it's about Ruby infrastructure. The Bundler and RubyGems are already working to remove duplicated code today. There is an open pull request right now for RubyGems to use [Molinillo](https://github.com/CocoaPods/Molinillo), the dependency resolver already shared by Bundler and CocoaPods. In the future, Bundler will become part of the RubyGems and RubyGems will include the `bundle` command.

Telling people to stop using Bundler now does not help with any of that, and instead just exposes RVM users to more bugs with less functionality.


Ditto.


Couldn't spell apparently. Should say for when user input does not match the name exactly


got it brah :~ montando um horário pra isso!


Might as well be a docstring, no?


As mentioned in https://github.com/letsencrypt/acme-spec/pull/97#issuecomment-85904635, we are converging on only checking ICANN.


Noindex is not a valid html tag. It is introduced by the Russian search engine, Yandex. Probably Google or other engines will not understand it.

If you still want to implement it and want it to be valid, you have to use "noindex" as html comment:
```
<!--noindex-->Content<!--/noindex-->
```
But Yahoo and Google implemented it different way.
See the article on Wikipedia: http://en.wikipedia.org/wiki/Noindex


Are these still needed? Since nothing is serializable anymore what are we testing?


Should this be removed?


slverwolf@gmail.com

On Tue, Mar 24, 2015 at 9:13 PM, Holl0wHeart <notifications@github.com>
wrote:

> Where can i reach u?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/holyangel/M8-Sense_L/commit/425e678f9c7198473376fcf6be60e2d528aaa43e#commitcomment-10381337>
> .
>


I'm glad I am predictable :) When you say "the other one" I assume you mean if the user specifies both sourceMap and inlineSourceMap? I think that also does not need to be an error. I feel like inlineSourceMap should cause sourceMap to just get ignored. And then if inlineSourceMap is not specified, we pay attention to the sourceMap option.


error messages should be lower case


@brifordwylie keep this routine for me, please ;-) Additionally, I will have a chance to learn more about project :)


Can we download to `/tmp`? I don't think we need these synced to the local machine.


lower case


docstring


Already reported #13961 and fixed.


Thanks mate!


hmm OK, maybe it should error. you can still use the ignore parameter to not touch those fields.


I see. Why didn't we need it before? Does the base64 encoding of the mappings not require padding?


Did you think about or try not backtracking here? That is, we already know, from the visitation method called, whether it's an "import from" or a "from import", so we may be able to avoid re-testing here.


Oh, we need unicode because one of your file names or scope names can have unicode?


Я бы сюда добавил <thead>


maybe use a const MinSplitResults instead of 2.


Я бы сюда добавил `<thead>`


We got 99 features but a :fish: ain't one


А сюда `
</thead>
<tbody>`


exports is optional? What's exports? You mean exportedMembers? If that's the case, just do the for..of if it exists


а эти `<td>` --> `<th>`


Move this to another test


Oh this commit was about not forcing users to use Bundler, I did not ask them to stop using Bundler ... at least not here. Anyway, for some people the Rubygems support for Gemfile already works - I am one of those people, I can also understand why you try to prevent the migration as I was in similar situation with version switchers, that's why I try ask questions and to understand the problem.


LGTM.


@MikeyNick Can you add this above Instructions as a requirement to work with this demo.  Also include where in where you would comment out code to get this working w/o Myo.


SymbolTable is just `Map<Symbol>`. So it doesn't have to be exposed for you to return it.


@domenic shouldn't this be a breaking change? For example, I'm pulling in chai 1.9.2, but with this change, my build now fails because you require chai 2.1.2.


HelloWorld.md


I thought my text might have been a little... convoluted


не уловил смысл p.price__name


LGTM


le pro license


there a reason you didn't just link /wjzeng/document/1916 ?
I guess locally it would still work, but I think it is not the best practice because people might use https exclusively..


Thanks, fixed.


this is shit, right?


Сразу вопрос, что это за переменная и чем отличается от локальной переменной метода? )


Ok ça marche


Good point. I didn't think about https will change

On Wed, Mar 25, 2015 at 6:01 PM David Furlong <notifications@github.com>
wrote:

> there a reason you didn't just link /wjzeng/document/1916 ?
> I guess locally it would still work, but I think it is not the best
> practice because people might use https exclusively..
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/willzeng/cograph/commit/fbe5307d2c7fad1a0b451477869de88ef6ce2349#commitcomment-10396171>
> .
>


lol how did we not see this.


yrdy


Stoi le fourre-tout


test


Ohouhoh...

I think this radical change should not be made in this situation.

I fear it has too many other dependencies on other areas outside get_task_list.
The pro and contra of project permsision override global permissions must be discussed extensively.
You can yet get all your requirements by moving the peoples from global to project level permissions without making such radical changes after 1.0 alpha.


@leofeyer This triggers for the homepage of the site because `$pageId` gets set to a nummeric value before.


Please accept this fix to make the command submission works.


I didn't knew this. Thanks a lot for pointing it to me.
I would have merged this pull request immediately, But I was thinking to come up
with a better way of making sure if the request is successful or not.
Something like this:

[code]
url = "https://httpbin.org/status/404"
try:
    response.raise_for_status()
except requests.exceptions.HTTPError as err:
    print("HTTP Error: ", err.message)
[/code]


Aunque ya lo probé en Safari y funciona bien. Según veo en caniuse.com .mp4 es soportado por todos, así que vamo' a ve'.


hey <repo>https://github.com/yol0swag/LeagueSharp</repo> is duplicated, know and contrib.


это же заголовок, может, `<h2>` – лучше будет?


Don't online if statements


In my opinion, this validation is equivalent to an `assert`. Its purpose is not to find defects in the code before it is shipped (which is what, among other things, tests are for, as you said). It is:
* To document the invariants of this code path.
* To trigger an error at the point where these invariants are not satisfied, instead of letting the program continue and have it fail later in another way that would be much harder to debug.

This is to me _complementary_ to tests, but still very useful.


Add "contains" ("A datafile contains a particular...")


array.first(2).map(&:to_i).include? ip


break up this line across a few lines. ie. expand the object


Non, puisque que c'est une incidente.

http://www.btb.termiumplus.gc.ca/redac-chap?lang=fra&lettr=chapsect6&info0=6.1.9


`<div>` --> `<header>`?)


return не нужен в конце метода


Don't one line


I am building again now.

Thanks,
	Dean

On 3/25/15, 10:35 AM, "Charles Doutriaux" <notifications@github.com> wrote:

>Merge pull request #1161 from chaosphere2112/vcs_anim_no_png
>
>VCS2D Animation Speedup
>
>all right since everybody is happy and it works for both sam and I.
>Merging. @williams13 please go ahead and build again
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/UV-CDAT/uvcdat/commit/c0aa782878dc0b2de062363a1fee6044e
>849ea4c


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1167 of 1168 tests passed)

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


Let's call it "utils" to match dxr.utils.


You need an intro paragraph above this section.


In the commit message, you said:

> do not force users to use bundler

In your last comment, you said:

> Oh this commit was about not forcing users to use Bundler

I am not trying to prevent a migration. Once Bundler has merged into RubyGems, I will ask everyone to migrate. RubyGems is not yet ready for the migration. Asking people to migrate now is irresponsible and adds more work to the already overloaded RubyGems team. Don't do it.


> Cave is supposed to be somebody else's responsibility so I'm waiting for him to commit :palm_tree: but yeah I'll change it when I get home

Ahhhh I see now. I think it's probably fine to create his empty object for him and then let him fill it out later.


This section should be a level `##` header. `#` corresponds to the page's title (which you also need).


You're not using this subsequently


Fantastic!  Thanks!


Simplify this sentence and move it to the top of the section.


It seems to me that the logic here is "inverted".  The name of the setting is `disable_full_inertia_tensor` and yet the full matrix is generated when this is `true`.

I suggest we change the name to `use_full_inertia_tensor` (with an default value `true`).  This is also because I believe that setting names containing the words *disable* or *enable* are not appropriate if they are exposed to the user (it would be OK if these were private member variables that the user would control through some set function).  IMO, with a setting name such as `use_full_inertia_tensor` the intent is much more clear.


Review #1028  on proper markdown formatting.


@Maxwe11 This is happening because the test is getting into UnauthorizedAccessException code path. Using the following pattern should work

```C#
 Registry.LocalMachine.SetValue("Test2", expectedValue);
  try
  {
      Assert.Equal(expectedValue, (int)Registry.GetValue(Registry.LocalMachine.Name, "Test2", defaultValue));
  }
  finally
  {
     Registry.LocalMachine.DeleteValue("Test2");
  }


Will add.


I need to analyse that part a little bit. I ll send a PR/commit in the morning if that is the right way to do.


:tada:


tu vas voir kicékivasefaire fourrer !


Use .toString

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Number/toString


This isn't the man page, so don't list all these extra options.  They're just confusing.  Start with your example.


Add brace


This isn't correct the way it's worded


Thanks 👍


indent


show examples, not general syntax


@mogoodrich  I am not sure I am following.


More complete with focus method. :-)
```ruby
Shoes.app do
    stack margin: 10 do
        flow do
            @sl = slider fraction: 0.35, state: nil do |sd|
                @p.text = "value : #{sd.fraction}"
            end
            @p = para "", margin_left: 10
        end
        button "switch slider state" do
            @sl.state = (@sl.state == "disabled" ? nil : "disabled")
        end
    end.start { @sl.focus }
end
```


@deronnax Agreed! Would you like to submit a PR that cleans it up? It'd be much appreciated. =)


Attention j'ai modifié cette ligne dans la PR #127


Variables names can you know, be longer, specially in a dynamically typed language.


Instead of nesting if statements why not do:

    if (req.query === null) {
         // else content from below
         return; // stop execution of function
    }


None of your examples seem correct to me.  The main use of `env`, which you haven't listed is to launch a program with a modified environment without modifying your shell's environment.  For example:
```
$ env PS1="$ " /bin/sh
```
Will start a new shell with the prompt set to `$ ` instead of whatever settings you currently have.


fair enough. will do that then.


I bet there is a string builder in JS


When I try out the <inner-html/> techique described on the riot tag overview page, it works fine in 2.0.12 however the addition of dom.innerHTML = '' in this commit seems to have broken that capability. If I comment out dom.innerHTML = '' then all works as expected again. Have you tested this with against this technique?

<inner-html>
  var p = this.parent.root
  while (p.firstChild) this.root.appendChild(p.firstChild)
</inner-html>


You need a 1 sentence description about what the nice command does.


Use !== instead of !=

http://www.impressivewebs.com/why-use-triple-equals-javascipt/


LGTM.


If inception. (I don't know how to structure that much better though.)


I don't understand


@hadley this line is what caused the bug. The file had only NAs, 1s and 0s in a float column in the first 100 lines. This is how `psql` produced them. When I change this number to 1000 it works. I think that means this definitely needs to be user-configurable, but possibly it should always be set to n_max.


@Garagoth, updated code. It was checking if threads terminated successfully by looking at the queue size.

Now using Future, as you suggested. The [`future.get()`](http://docs.oracle.com/javase/6/docs/api/java/util/concurrent/Future.html#get%28%29) will throw an Exception if thread failed.


From https://github.com/xunit/xunit/issues/30
StrictEqual strictly relies on the built-in behavior of val1.Equals(val2). Assert.Equal attempts to do a lot of reconciliation to get the correct answer for collections, etc.


Oh no!  You had it right the first time:

```ruby
<%= form_for([@project]) do |p| %>
```


This addresses https://bugzilla.mozilla.org/show_bug.cgi?id=1145024 and is now live in production


Не ясно, что делает метод.

1. аргумент array надо переименовать, и всегда помнить о тех, кто будет читать твой код. Код должен быть написан как книга - четко, ясно и легко читаем)
2. puts я так понимаю для некоей отладки. Это хорошо для учебного примера. Но puts по-умолчанию выводит в консоль. Попробуй перенаправить вывод в файл для примитивного логгирования, чтобы все что выводит puts писалось не в консоль а в файлик типа lookup_ip.log
3. Методы возвращающие булевое значение принято сзади дописывать "?", пример:

```ruby
def ip_included?(current_ip, ip_list)
# ...
end
```

И если ты не определяшь класс, то instance переменные не надо объявлять


This was also being included as a sub-dependency of the library you added and ending up in the lib folder of the module--we needed to exclude it because it is also provided by core.


Nice job extracting this to the model.


Environment variable is must be string.
Otherwise, it will be implicit cast to string.


-1
A ideia do desafio é trabalhar a separação das camadas. A única coisa que deve ter de JS no HTML é a tag `<script>`, que incluir o arquivo. Dentro do arquivo, atrelar um evento ao form para submeter e criar a lista.


попробуй этот кейс [5, 6, 7, 8, 9, 1, 2, 3, 4, 5]


yes. The other one is slightlly diffrent as it is a base64VLQFormat. this one we are putting chars in digits, so you need a way to tell it that this is the end of the stream, and no more chars left.


Looks like the new code (in `BlendContext.__exit__`) already has better comments.


http killed on purpose?


-1
Mantenha o evento, mas adicione o listener no evento `submit` do form.


`fundinglevel` vs. `funding_level` is what caused your problems.  Rails _really_ wants it to be `funding_level`.


тут можно так написать ( на будущее):

```ruby
if array.first(2).map(&:to_i).include? ip
```


These examples seem to miss the point.  Let's say I'm on a shared computer (say hammer) and I have to run a computationally expensive operation.  I might run the command:
```
$ nice -n 19 expensiveoperation &
```
And this will work on the operation only when no one else is using the CPU.  If someone else comes along and runs something, `expensiveoperation` will effectively pause and wait for the other person's task to complete.  This is how programs like Seti@Home are able to use spare CPU cylces for scientific computing.

Nice is often combined with nohup.


La modernidad.

A mi en firefox me bota al dialogo de descarga de archivos.


what is going on? :cry:


Available Strategies
- Memoize (?) -
- Prune -
- Myopia -
Game View is already put in here.


So it seems like we should create another interface for `Run`. Thats good as we will be able to also find a better name for that which will make more sense. Keep in mind that this is optional as the code *works*.

So once you do: `public Run getBuildByNameAndNumber(String itemName, int buildNumber);`

We will return `CiRun` (just keep in mind finding a better naming convention).
`CiRun` will expose the method `startTime` (and maybe `endTime` if needed).
It will have to expose more methods which are actually being used by the code at the moment, e.g: `run.*`. (this might take some time to figure, so as said above - this task is optional at the moment).


Where's the "replace the sphinx docs with the new system" issue? Otherwise it's a little ridiculous that we have a full-on Markdown parser in Base if we're barely using it yet.


ff


`project_path` needs to know about the actual project, not the funding level.  You probably wanted

```ruby
redirect_to project_path(@fundinglevel.project), notice: 'Created!'
```


jahto, could you please only merge the parts of your great effort to FS1.0 master, that does not change the existing permission mechanics?

I hope I add the demo data sql queries soon on bugs.flyspray.org, so performance changes can be tested in a comparable way of different approaches.

The easiest way to drop memory on big results is to just fetch row by row. This could also be a cheap solution for 1.0 and do the perms in sql for 1.1 or later. It's just my fear we don't get the bug reports for 1.0 not to zero if we do too big things now.

Did you saw the discussion on adodb with buggy limit offset? ( ~ 2012)
And I saw you too made a pull request on adodb project . :-)


-1
Essa é forma antiga de atrelar eventos à um elemento. Assim você só consegue adicionar um callback por evento, por elemento.

O correto é usar `addEventListener`.


You would never run `nohup ls` because `ls` takes no time to finish.  You run `nohup` on things like servers that you want running for decades.


so each saved list has a name that you give it, is what we are doing here?


Would it be better to define "-D_POSIX_C_SOURCE=200809L" in the CMakefile instead? It is what we are doing in one other place to fix similar problem.


Closes 26


Yeah no idea. I thought about how to simplify this but it didn't seem any better....


No one would ever use timeout and ls together.  You need to think of a better example.


comment on commit!


-1
Não tem `;` no charset


-1
Não precisa do `type` para `link`, `style` e `script`.


внимательно посмотреть, что делает метод map ))


-1
Não usar ID para estilizar


Have you considered making these 3 functions methods of TreeAnalysisResult? The second 2 seems like gimmes, already taking an instance as their first arg. In the spirit of RAII, analyze_files could even be the constructor. Then TreeAnalysisResults (which could be renamed TreeAnalysis to better convey it) would no longer be a dumb bag of data but would be responsible for its own, in true OOP fashion.


NO HAPPY :{


That's another way to think about it. I thought that having that flexibility might be useful. If the FROM string needs to stay the same, the user need to just hit Return and proceed to TO.


:trolleybus:


This is another instance of code that I wrote in a non-idiomatic way so it would match the upstream version more closely.


Arg, I hate these dangling parentheses... This isn't a curly brace-base language :)


`#` should be used to indicate comments


A mí no, qué raro... debe ser por el header, el que está montado en heroku no es el que está acá en Github, por el tamaño del video no me recibe los push.


In efficient?


same here


Ha, yeah, good point.  What, you don't like `impl_impl`?


Just spent an hour trying to debug why my constants didn't have this... Only to realize it was added a week or so after I grabbed your library :smile:


Overall, you're going in the right direction.  Your material looks correct, but you need to work on your examples---You need to motivate why we would ever want to use these programs, not just give me technically correct invocations on meaningless examples.  Your writing style is fine on a sentence-by-sentence level, but your organization still needs a bit of work overall.  I can't give too detailed feedback on this yet because a lot of your writing is going to have to change to accommodate the improved examples.


and here


Also, the other one is just encoding numbers, whereas this one is encoding arbitrary text.


z


And what about avoiding the word "branch" here too?


QSDFGHJK


-1
Não lembro se te passei os links, mas use o querySelector para selecionar elementos.


Didn't think about this, but it seems it may be fruitless - http://stackoverflow.com/questions/1532461/stringbuilder-vs-string-concatenation-in-tostring-in-java


we are bad people, but not as bad as people who push without testing.


@mvaled thanks for the pypa pointer.

I _hope_ to make a release this weekend.


@vwadekar. Some questions and comments regarding this patch.


nit: you could define `par` along with `vaddr` and `type`


Could we not define these accessors using `DEFINE_SYSOP_TYPE_PARAM_FUNC` like TF defines the cache maintenance by set/way instructions in `arch_helpers.h`


@kruppel Haha get it! I was wondering when I changed it to DarkGray. Anyway can you please change it back? It seems my local branch is broken or something wrong with my Xcode.


need a `isb` before the PAR_EL1 can be read. `AT` instrcutions require explicit synchronization.


nit: magic number `3`. Is it possible to use a constant instead?


for reals


It would be more scalable if we used the `SCR_EL3` from the stashed normal world context?


doesn't datepicker need a container element with an class of `datepicker`


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


Okay - that was a tougher than ordinary merge - @carlfeberhard and @guerler you may want to double check my  math on this one.


yet is the operative word!


I might be being a bit thick here but even after going through the discussion in tf-issues#222, I cannot completely understand why would S-EL1 want EL3 to translate secure world address? S-EL1 and S-EL0 addresses can be translated at S-EL1? Is there a need to throw a EL3 VA back to EL3 from S-EL1 for conversion?


опять же return не нужен в конце методов


90946736_WSHP_#4567 (kbenne) - x86_64-MacOS-10.9-clang: OK (1668 of 1670 tests passed)

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


@tardyp I am convinced :)

:+1:


Any chance you would want to own tf-issues#248 and make the address translation functionality available as a generic library either now or in the future :-) ?


trying another comment....


This could go to `arch_helpers.h` being a standard function?


I guess you could just move the WriteToFile function and the static variable to a cpp file if you prefer.


За это задание :+1:


They’re coming from upstream, in [nfarina/calloutview](https://github.com/nfarina/calloutview).


-1
Coloque o script logo antes do fechamento do `</body>`


That's what I get for cutting and pasting code. @shahraj81, I think it would be better to ensure that QUANTITY is set, rather than testing for it at line 1387. How hard would that be?


тут  оформлен if плохо


I'm pretty sure i have something similar to submit forms with ctrl+Enter somewhere (for comments and wikis). But I guess the submit is simple enough there is no use in trying to combine the two now.


The unit tests still serialize data and use these methods.


Yes, I removed it on purpose because it was hiding errors during testing (the ConfigError was bubbling out instead of the "real" Exception / cause - and ruby was only printing the ConfigError).  This doesn't seem to break anything and it seems to be an improvement since the actual exception will be logged, instead of our wrapper.


Falta limpar o input quando adicionar um novo item.


Good point, I changed the name to use_full_inertia_tensor


-1
Se tiver vários itens, o layout quebra. Não precisa desse estilo com fundo azul e borda.

![selection_026](https://cloud.githubusercontent.com/assets/487669/6832196/a7802b54-d303-11e4-9afe-ef07bdb99799.png)


Your editor seems to be adding a lot of trailing whitespace.

As I commented on E, we'll want to flag the account rather than the address in cases where we do - members will often reactivate their current address when notified.


input_array - слишком абстрактно. называй всегда так, чтобы ясно было о чем речь.

например:

```ruby
def dots_in_circle(dots, center, radius)
# ....
end
```


Algum motivo do Almir mudar o tema da palestra?


90022444-FewReleasePrepThings (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (0 of 0 tests passed)

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


Nice job, me.


e[0] - e.first
circle_center[0] - cricle_center.first


console is undefined in IE8 if Developer Tools are not opened, maybe replace with 'cdf/Logger' module?


> assuming it doesn't try to write to the .ks file

What do you mean by that?


90693826-Central-cooling-coils-not-autosizing-to-correct-airflow-in-some-cases-#4810 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1164 of 1167 tests passed)

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


maybe replace with 'cdf/Logger' module?


will do!

On Wed, Mar 25, 2015 at 10:20 AM, Brian Goff <notifications@github.com>
wrote:

> Also --default-ulimit option on the daemon which, which applies to all
> created containers (and overwritten by --ulimit on run).
>
> --default-ulimit also includes build containers.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jfrazelle/docker/commit/746e83023052e32b2cba8fccf67b6232e5c86886#commitcomment-10395225>
> .
>

--

Jessie Frazelle
4096R / D4C4 DD60 0D66 F65A 8EFC  511E 18F3 685C 0022 BFF3
pgp.mit.edu <http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3>


просто на будущее, если в массиве только строки без пробелов, то можно писать вот так:

```ruby
%w(Banner1 Banner2 Banner3 Banner4)
```


sava


Tercer commit


f


In fact, [0,4] will increase the chance for 1 elo patch to pass (41%), more than twice as [0,6] (20%), so I would prefer [0,4]


hey @DonatoB - You need to change var t = videotag.currentTime / 1000; // secs to ms to var t = videotag.currentTime * 1000; // secs to ms to fix an issue with buffering.


definitely not on-purpose and I thought I had put it back during rebase. Sorry.


I believe that the default should be `true` as that is the general case (i.e., even if the inertia matrix is actually diagonal the solution is correct -- albeit less efficient -- when using the full matrix; the reverse is not true)


This definition could go  into `arch_helpers.h`?


duh, will fix


зачем от Array наследоваться?)


 - __warning__: [include/chaiscript/language/chaiscript_eval.hpp:542](../blob/a75117c007ff665e03600ce43209c129b83a1720/include/chaiscript/language/chaiscript_eval.hpp#L542) Member variable 'Equation_AST_Node::m_oper' is not initialized in the constructor. (uninitMemberVar)
 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/a75117c007ff665e03600ce43209c129b83a1720/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/a75117c007ff665e03600ce43209c129b83a1720/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/a75117c007ff665e03600ce43209c129b83a1720/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/a75117c007ff665e03600ce43209c129b83a1720/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/a75117c007ff665e03600ce43209c129b83a1720/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


Modernizar workflows. Ainda vemos pessoas fazendo coisas que poderiam ser automatizadas e não são. Desde um simples minify, uglify, concat até build com deploy na aplicação.


что-то тут не то... метод в методе и методом погоняет))))


Yeah, I caught the whitespace and recommitted a cleaned up file - it's from where I added, then removed my breadcrumbs.

Here's a question: do you want a "dirty" flag to prevent just notification emails from going out, or "control" emails (e.g. password resets, etc..) as well? I'd personally lean towards the former, just to prevent a full lockout that needs admin intervention, but, on the other hand, the situation would really be barely different from forgetting the password and having lost access to the email account situation we've already got.


Не, так пойдет, задание не засчитаю, пока нормально не напишешь))


Nice.  Don't even need to eat those bytes, just hop right over them.


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/4fe536e65b2135bfb4604e1684b153b529251b71/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/4fe536e65b2135bfb4604e1684b153b529251b71/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/4fe536e65b2135bfb4604e1684b153b529251b71/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/4fe536e65b2135bfb4604e1684b153b529251b71/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/4fe536e65b2135bfb4604e1684b153b529251b71/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


Yes, I think so.


Yes


Gostei, já conheço o GruntJS, Gulp entre outros, é que também acho interessante o padrão Throttle e Debounce, e falei pra minha equipe que teria isso, rs, mas legal muito bom que estão procurando as coisas que estão funcionando no mercado e trazendo para nós.


I comment here to claim credit for the idea. :twiface:


Hi Psycho, if you think it is better to have composer.phar in the same folder like composer.json, it is ok for me.
(my tries to to workaround my hosters limitation by doing it all in setup/ failed, hoster still automatic blocks flyspray root dir by ".htaccess deny from all" after installing composer vendors over webbrowser)

Then enable chdir('..'); again --working-dir=. instead of --working-dir=..
in the setup/* files.


When you create a list you give it a name. It will use that name when storing and loading from localStorage. If two list objects are given the same name they will overwrite each other in localStorage, which might be useful or not depending on the context.


yeah I was looking at the coverage report, started refactoring this out (since it was duplicated).. and then realized we're not even using them in the first place :smile:


Thanks Al!


[Pretty-printed literals section at this commit](https://github.com/cdzombak/objective-c-style-guide/blob/cdz/multiline-literal-commas/README.md#literals).


It could be. As a relatively new collaborator I just wanted to make sure my patch had the smallest possible impact on the code being compiled, so I went with the absolute bare minimum.

It's your call really.


You are right :(


Removed this one because this functionality is already shown by the first test


checking to find the edit button


@vwadekar. Some more comments regarding this patch. Please let me know if anything is unclear.


Could you use `DEFINE_SYSOP_TYPE_PARAM_FUNC` to define these functions in `arch_helpers.h` and make them available to the rest of the code as well.


Need a `isb` before the `PAR_EL` read. `AT` commands need explicit synchronisation.


Magic number `3`. Can a constant be used?


I think it would be more scalable to pick up the `SCR_EL3`  from the non-secure context instead.


I might being a bit thick here but even after going through tf-issues#222, I cannot completely understand why there is a need to translate S-EL1 and S-EL0 addresses in EL3. Can this not be done in S-EL1 itself or is there a case where an EL3 VA is passed to S-EL1 and the latter needs to translate it to a PA.


Any chance you want to own tf-issues#248 and export this functionality as a generic library now or sometime in the future :-) ?


Removed this test because this functionality is already shown by the first test


a + 1


Canola?


, (space) a + 1


Can you add a assert() for the incremented &decoder.TotalDroppedDims ?  Otherwise looks good


parenthesis are unnecessary


Neat.  You could also write this using the array `sample` method (http://ruby-doc.org/core-2.2.0/Array.html#method-i-sample):

```ruby
def show_background
  ["background1.png",
   "background2.png",
   "background3.png", "background4.png",
   "background5.png",
   "background6.png"].sample
end
```


remove useless spaces at the end of line.


So instead of M-q as now, the user should press M-q RET all the time for the 5% chance that she will do M-q -> correct -> RET?

And take into account that the correction in that step may produce zero candidates for replacement, since ivy has exited at that point. I don't like this idea.

See instead the last commit for the option to cancel the query-replace with C-g, edit
the input, and then M-q again.


umm! thinking about it more, i don't this is really an issue with CPU_SUSPEND since the architectural state of S-EL1 will be stashed during the last exit from TLK. EL3 will flush it out and it should be just available for use upon a subsequent power up after a suspend.  Does that sound reasonable?


Check off once tested on staging:
- [ ] Async request from https://staging.commcarehq.org/a/crs-remind/reports/case_activity/?emw=t__0&case_type= no longer 500s
- [ ] Async request from https://staging.commcarehq.org/a/crs-remind/reports/submit_errors/?submitfilter=1&submitfilter=4 no longer 500s


Update to match source


I've got some trouble accepting
```
globallevel | projectlevel | resulting perm
    1       |      0       |      1
```
It practically means there's no easy way to turn off access if already given on global level, even if it would be needed in some special cases. Of course there's always the hard way to achieve that, which needs a lot more thinking how to manage groups, does a user belong only to a global group or to a project specific group too. Like putting almost any user first to a global group that has no rights whatever, and then in every project add again a new group that has the correct permissions for that project. I must say that I'd like a lot more a solution where I could do with a little less of that kind of group management acrobacy. So my opinion is: if there is a project level group, that wins over global. Project level groups are in the most simple use cases even not needed, just having globals is enough. And especially, because it's going to be FS 1.0, we could, can and should fix that deficiency. It's not like 0.9.9.7 in many other ways too.


To check once this is on staging:
- [ ] Async load of builds on https://staging.commcarehq.org/a/newproectsettingsfeatures/apps/view/409250b03e827a9349b29d240d32d6cd/releases/ does not 500


rax retry tests


This project is deprecated...not sure that we want it to clutter the Project Monitor.


Thanks! :)


que capa :) igual ojo que esa docu es de FreeOSEK y no de cuando lo integramos en Firmware, esta muy desactualizado... estoy armando un libro apunte en espaniol de FreeOSEK pero vengo re atrasado... :(


i = 1


Logs are now filled with:
```
ERR (3): Strict Notice: Only variables should be passed by reference  in fooman/googleanalyticsplus/app/code/community/Fooman/GoogleAnalyticsPlus/Block/Remarketing.php on line 272
```


Why not
x = SCR_HorizontalAlignForString( x, align, width );
?


y = {min = 1


Changed in 944bb7f24009ba230b67d50ac6066f93aeb5c333.


You could use your background helper method here if you wanted, or (even better) add a `body_style` helper method in your helpers first:

```ruby
# in your helper file
def body_style
  "background-image: url(" + asset_path(show_background) + ")"
end
```

And then in your application.html.erb...
```
<body style="<%= body_style %>">
```


Sass looks good! Looks like the edX settings are in the commit as well though...


Sorry @PonteIneptique it's back now in  3e414bf


styling: function($) {


To avoid a useless calculation of the length when the string is aligned to the left. This is what the commit is supposed to do per your request.


Landing in GCLI in 1860ace and in gecko-dev in a changeset that will be rebased, but not forgotten.


You can remove the same thing from python3-meh too


Is that testing needed though? Aren't we just testing .NET serializer and not our code?


styling: 0) {


I'm fine with either implementation. All I wanted was not having to type the same search string in the FROM field :)


styling: parameters: [],


I am learning so much from reading your code @lbeschastny.

I have a question: I see how this `when` prevents the call to `func` until `delta` has elapsed. What I do not understand is: where are the args accumulated in the mean time? Each time the `fn` is called before `delta` has elapsed, it appears to me that the args are lost.

I believe the code works of course, because I watched it run in the terminal. But I expect to see the args (to `tick`) being accumulated somewhere, then after `delta` elapses, I expect to see a call to `tick-by`. But I don't see that.

Where is the magic?


styling: function() {


missing ; in the end


sdfsdf


As discussed we need rpcbind command in dockerfile


Sorry to react so late (unfortunately after it has been merged) but I was off the office this afternoon.

There is something I still don't agree (because I don't understand) in Product::priceCalculation():
Assume use_tax is true.
Until line 2858 price is tax excluded.
After line 2859, price becomes tax excluded + taxes i.e. tax included.
At line 2880, assuming other conditions are satisfied (it is possible because they are independant of use_tax, isn't it?), price becomes tax included + taxes i.e. tax included twice.

Another point, less important: I personally find the block 2872-2883 difficult to understand compared to what I proposed in #2548, with the same result I think:
if taxes are requested and reduction excludes them, then add them.
if taxes are not required and reduction includes them, then remove them.
Here I am subjective, surely, and partial, maybe.


:) Glad to see you also got one, let's make this device awesome


It compiled in my linux mint 17 computer perfectly. Are you sure that it happened at clean trunk?
OBS: If the issue happens only at your edited ot, simply try to add #include <math.h> at your combat.h or combat.cpp file.


Unlicensed forking of empty repository? Not on my watch!


Quick question: What does the comma represent? I've never seen that syntax before.


Sorry about that -- there should be a fix shortly for the automatic selection. This is happening because cuDNN is selecting the fastest but memory consuming algorithm so that the GPU runs out of memory.


Its in clean trunk, msvc doesnt have round in default gcc does.
25 mar 2015 19:52 "assassina" <notifications@github.com> napisał(a):

> It compiled in my linux mint 17 computer perfectly. Are you sure that it
> happened at clean trunk?
> OBS: If the issue happens only at your edited ot, simply try to add
> #include at your combat.h or combat.cpp file.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/opentibia/server/commit/0b74e3837b063c280c8fd2b950db65ccdddb8abe#commitcomment-10397371>
> .
>


Also fixed the troll table view


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1163 of 1167 tests passed)

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


oh, good catch. they should all be single quotes. some of these I copied from the template-helpers lib - and in templates, I prefer using double quotes. I tried using `to-single-quotes` but it messes up code sometimes.


va_list is not necessarily char* - it's implementation defined and might as well be struct ;)
But if you want to keep the current code - be that way.


Ye but line 323..


Tu ne modifi que les test, je ne vois rien qui touche au model client


develop (mjwitte) - 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)


belongs also to Ticket #2


Windows will complain in the GUI view. No where else.


Oops.


Pietia, it doesn't make any sense. The round function is a totally cross plataform function. It is defined in math.h, a standard C library.
I can't look at this now. Could you try to compile it in linux, to see what happens? Even if it is very unlikely, I want to be sure that I didn't change any code AFTER compiling and commited the code without testing.
All I can say is that, if it doesn't work in windows, it shouldn't work in linux. It is a standard library!


Complain...


Nitpick: we use 8 space indent or a reasonable alignment for line brakes


something odd about this - do you know why the order would make this kind of difference? or why class_eval is the savior here? i'm probably doing something horrid...


@xray7224 yes, that's what happened.  I wanted the "add" and "change" links to be closer to the "Participants" and "Events" names on the /swoptact root page.  Thanks for checking back.


on peut enlever cette methode


Vision should be reverted :(
I liked it


@kasper93 Why don't you just make static GetFilename() method? And if you don't want to allocate memory each time, you can place static array (or whatever) in there. At I would do it like that :p


Cool. (I think I'll be adding a `.vimrc` in another pull request soon.)


Hello! Thanks again for your help translating the app.
It would be better to leave out the strings marked as `translatable="false"`: they don't appear in the app, and translating them causes compilation warnings.

For this particular case, do you think it would be better to provide a google.pl link instead of google.com?


Ok, I won't send a PR then, and I'll delete the branch.


I [changed the conventions](https://github.com/ligershark/side-waffle/blob/autoupdate/TemplatePack/Tooling/TemplateLocalInfo.cs#L21) for dynamic templates to the following.

 - ```project-templates``` - for existing "TemplateReference" projects
 - ```project-templates-v0``` - for the legacy project template format
 - ```item-templates``` - for item templates

I made these changes so that the SideWaffle vsix can have templates and we can have dynamic templates in the same repository.


Unfortunetly it isnt the case I compiled fine on my linux box too ,google
the error if yoy dont belive me and cant test it ;-)
25 mar 2015 19:57 "assassina" <notifications@github.com> napisał(a):

> Pietia, it doesn't make any sense. The round function is a totally cross
> plataform function. It is defined in math.h, a standard C library.
> I can't look at this now. Could you try to compile it in linux, to see
> what happens? Even if it is very unlikely, I want to be sure that I didn't
> change any code AFTER compiling and commited the code without testing.
> All I can say is that, if it doesn't work in windows, it shouldn't work in
> linux. It is a standard library!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/opentibia/server/commit/0b74e3837b063c280c8fd2b950db65ccdddb8abe#commitcomment-10397466>
> .
>


You forgot change to .com in users.js


Even including math.h doesnt helps
25 mar 2015 20:00 "Piotr Walaszczyk" <pwalaszczyk11@gmail.com> napisał(a):

> Unfortunetly it isnt the case I compiled fine on my linux box too ,google
> the error if yoy dont belive me and cant test it ;-)
> 25 mar 2015 19:57 "assassina" <notifications@github.com> napisał(a):
>
>> Pietia, it doesn't make any sense. The round function is a totally cross
>> plataform function. It is defined in math.h, a standard C library.
>> I can't look at this now. Could you try to compile it in linux, to see
>> what happens? Even if it is very unlikely, I want to be sure that I didn't
>> change any code AFTER compiling and commited the code without testing.
>> All I can say is that, if it doesn't work in windows, it shouldn't work
>> in linux. It is a standard library!
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/opentibia/server/commit/0b74e3837b063c280c8fd2b950db65ccdddb8abe#commitcomment-10397466>
>> .
>>
>


Okay cool. I must have missed it somehow. sorry.


This is a really clever way of maintaining a stack. Hard to screw up. Kudos.


Just in case.. : http://www.mixxx.org/wiki/doku.php/coding_guidelines


Thanks! But aren't we missing the tests for RHEL 6.X with this patch?
AFAIK, the service only changes its name when using RHEL 7 and above (and derivatives) due to systemd.
I'm gonna merge your code and add my patch on top of yours.
Thank you again!


Extra line


Though it will not be truly static, but each allocation for each compilation unit. Logger.cpp approach is probably the best after all.


Fixed in 323c8a9a5cf454536103c0f955414e2ef4425635.


si, vi que estaba algo desactualizada, :) pero no pude con mi genio.

Te hago una consulta: Estuve viendo el issue #169
https://github.com/ciaa/Firmware/issues/169 y creo que es mas importante de
lo que pensaba.
Si van a empezar a usar la CIAA-NXP con OSEK, es necesario definirle al
usuario el numero máximo de recursos, tareas, eventos que puede crear. Así
como existe el numero máximo de interrupciones ( 32 de 0 a 31) .
Realmente no se cuan importante son estas limitaciones. si estamos hablando
de 10 tareas máximas o de 1000.

No supe como darme cuenta de esto. Supongo que depende de la memoria
disponible en el hardware y de la velocidad del clock y no como una
limitación de software. Me gustaría saber que es lo que realmente limita la
cantidad de recursos, tareas, etc.

Además como ya existe la declaración en el MAKE de que board estas usando,
si la limitación viene dada por el hardware, se podría hacer que al
compilar, si el numero de tareas es superior al máximo muestre un mensaje
de warning. De esta forma no estaríamos obligando al usuario a leer toda la
documentación, pero estaríamos notificándolo.

Ojo quizás estoy diciendo pavadas.

Saludos.-

2015-03-25 15:46 GMT-03:00 Mariano Cerdeiro <notifications@github.com>:

> que capa :) igual ojo que esa docu es de FreeOSEK y no de cuando lo
> integramos en Firmware, esta muy desactualizado... estoy armando un libro
> apunte en espaniol de FreeOSEK pero vengo re atrasado... :(
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ciaa/firmware.modules.rtos/commit/2ce1e86928ebccfe16fdfaa2a81eaadb0433f50a#commitcomment-10397219>
> .
>


I like this mixin as well. Good factoring!


Hey peter... Sorry you don't see me a lot lately, i'm pretty bad sick... Have been keeping up the merges, but that's about it. Having the .phar file into root dir isn't a problem imo, but it would need to bring back my temp hack/workaround I made last time, Will read Composer doc to find if there's hidden commands somewhere


Recommend making this a constant with the name of the file, so you’re not constructing it from pieces of string up in `main()`, which would go back to taking `org_sources` as an argument.


…see [`store_const`](https://docs.python.org/dev/library/argparse.html#action).


I suggest to merge developing branches into **dev** branch, making the dev branch as the latest updated branch, and start a new feature branch from **dev** branch. Only merge dev branch to master when some level of completeness is achieved, so we can reduce the complexity and conflict on the master branch.


Didn't notice this, sorry. Should it be changed back to before or kept public?


This should be fixed in the latest commit


missing changes file


since the codes are used later, it might make sense to use module constants here (TCP_MAP_CODE_OK... or something). It also would make it a bit more readable.


@justinnoah Why should "source" be missing from properties?


Ok, I googled it and round was introduced only at c99, it is not ANSI C. I might be kind of busy at the next days, but I will try to fix it soon.


Why didn't the plugin get approved?


So we had to set it to None.! I still don't understand why it wouldn't delete when it had to but well at least it works correctly now.


That looks silly. Have you considered using what BuildCraft does (a TileBuffer)?


don't forget the `,` :-)


Was MushroomStewHeal a joke?


Which JDK do you use?


fixes #7


document couchdb type


document key type


Do we really need a full copy of minified jQuery in our code tree to test this?


`Tinfo` => `TInfo`


How about 'Sorry, there was a problem processing your answer. Please try again.'


so the previous code is wrong?
can you document the meaning of the codes? (maybe with the constants change I proposed is self-evident).
I also would prefer if this change is not made in a [refactor] commit, since this is probably a bug fixed.


`string` => `String`


`PersonalNumber`, `Integer`


`Hours`


`Rate`


ZP -> `Salary`


Thanks for posting this - I also need support for long long, you've saved me some time.


For security I think this should error out if the the survey's user isn't the current user.


This is a bit vague. Would it be accurate to say "Indicates that an exclusive lock could not be acquired for a lock file."?


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1167 of 1168 tests passed)

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


I played around to have this splash screen scaled on HighDPI displays.

My idea was:
- access the splash screen as soon as "SplashScreenHelper" is instanciated
- get image URL
- draw scaled image to graphics ressource of the splash screen

This would display the splash screen in its original size for only a very few seconds and then scale it to a souitable size so the start-up information can be displayed with a larger font size and everything looks well.

problem #1 that occured:
- dimension of splash screen is fixed

solution:
- put a big transparent border around the splash.png so the splash screen frame is larger than it appears
- start to draw the resized image a hand full of pixels off-screen (-x, -y) so the visible part is still centered

Until this point everything worked fine.

Show stopping problem #2:
- you dont really have access to the splash screen image layer but to a graphics layer above it
- the main graphic layer is using a SRC_ATOP rule. Under this rule, the alpha value of a pixel on the overlay remains whatever it was in the original image. Because of that, every pixel laying over a transparent one of the original splash screen is also displayed translucent (tested it with a 50% transparent border)

Idea:
Once the SplashScreenHelper is instanciated, display a decorationless frame where all the splashscreen magic can be put into. The original splash screen could be the "laying eight"-Logo. After the above mentioned few seconds (~2-3 on a modern system) this new splash screen is displayed in a frame and therefore can be resized depending on a scale factor (see https://github.com/arduino/Arduino/pull/2776 and http://forum.arduino.cc/index.php?topic=263838).

I would try to contribute some code if developers thing this idea is a good one.


This doesn't fix UV-CDAT/uvcdat#1124, 7f09cf54 does.


I Sign Off


`clj-progress` keeps the state of a progress bar internally, so `(apply func args)` do nothing except for updating an actual progress bar (printing the new one over the old one). In other words, `args` already holds all information about progress state, while `func` is simply printing it. So, there is no need to manually track all ticks.


Ah, nice catch. This should be
```
    survey = get_object_or_404(Survey, id=survey_id, user=request.user)

```


I think this does not need to post and the endpoint should be removed.


Thanks for sharing :-)


open issue with crash, your crash has nothing to do with this commit probably


ah, since the remaining blockface reservations are not being mutated, they don't need to be recalculated and we can rely on the mapAnotherPopup context being brought in when the page is loaded.


Ok I see the issue.


This is about more than just the pipe next to us. It's about every pipe between our routing pipes. So Unrouted LP pipes as well as BC pipes and TE ducts. [Maybe more in the future]. The previous way was the main performance usage LP had and that usage was regardless of the fact that the system was used or idle.


No, I was just lazy and copied the other test project that had it in it, so have made it twice as bad.  Will blot it out with gibberish.


Yep - different use cases :)

Cheers!


'influcing that influence consensus' ?

Maybe just: "Parameters that influence chain consensus"


Spent hours figuring out why half of my Foundation was not working until I fixed it locally and found this commit. Bad day..


We might want to call this `exists_setting?`.


That is why I wanted to make permissions view and management clear and really easy to manage, but keeping the existing permissions mechanics. No group management acrobacy and that is what is still open in the bug tracker for 1.0 but yet had not priority time to do it. But I promise management of groups and permissions is easier then, but keeping the full power of permission system.

Don't forget, currently you can add several project group levels too, their naming is absolutely independent from global groups. I really see too much new problems in changing it now!


Can you elaborate on your surprise :)


Why not just make an internal cache of tile entities that works this way? If you request a tile, it adds it to the cache, and refreshes it only if the tile is invalidated. I feel that hooking into those two classes is very, VERY intrusive.


Yep :-) Definitiv :-))


The comments I left inline are minor. This is :+1:


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (654 of 657 tests passed)

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


If Jetty tries to write to the .ks, it will fail because Program Files is
write-protected. It shouldn't though, it should just read from it. Unless
the Java crypto libraries do something funky

On Wed, Mar 25, 2015 at 2:30 PM Tres Finocchiaro <notifications@github.com>
wrote:

> assuming it doesn't try to write to the .ks file
>
> What do you mean by that?
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/qzind/qz-print/commit/e4668116143785da92ae4c31cf82c8530e886bff#commitcomment-10396870>
> .
>


This is maybe a stretch, but if you pass in an empty string as xforms, it'll use a newly instantiated `[]` instead of the one you passed in. Probably doesn't break anything now but it'd be a little surprising but not shocking, and horrible to debug! I'd go with the more verbose `xforms if xfroms is not None else []` just to not have to think about it


styling for h1 headings, links, navigations


@Ekaseo You don't seem to have properly updated the submodules. Try running "git submodule update --init" as described in the building instructions for RPCS3.


Same comment here as in previous commit, we might want `setting` to be part of the name here. I get that this is just copied from another place though, so we can just roll with it for now.


Nice! I think I structured some of this code that way hoping this would happen one day :)


we are interested in keeping the services library limited to only executing OCF scripts in the /usr/lib/ocf/resource.d directory. that's why the wrapper scripts live there in their own namespace "containers".  I don't hate the idea of renaming the namespace though. "isolation" might be better.


There should be no comma before ,


That SO question is on Java, though according to [this](http://stackoverflow.com/questions/7299010/why-is-string-concatenation-faster-than-array-join) plus/concat is super fast.


also, the plan is to make this available for people to create their own custom isolation wrappers in the future. we'll make that possible someday.. just not immediately. i want to see this stuff settle out before i give people the ability to do their own.


Made the biggest mistake of leaving home in 2008, which created a wall between us. Bitches (especially that one bitch) and Bad friends entered into his life and We got completely separated. I wish that my brother was with me now. December 2009 changed everything. Our 10.6 years of close friendship came to an end. If he was with me like before, then life would have been better and tasks would have been finished faster. Now, more than 5 years has passed. I miss our old times bro!!! Do we deserve to be deadly enemies forever?? I couldn't find a buddy like you bro to adjust with my stubborn nature. I really want to leave this house soon. I can't live as a stranger in my own home anymore. </3


yep. did that in a future commit.


The gramatical error wasn't corrected. It's `reupload to`, not `in`. And the dot in the path isn't correct either.


@Underground78 @kasper93

https://github.com/mpc-hc/mpc-hc/compare/a7620372a7485f336b1c146387ebca60d0fea1d8...XhmikosR:develop

About the installer change:

```irc
[21:12] <vanila> Hi all :)
[21:14] <vanila> regarding this https://twitter.com/mcclure111/status/580593190039896064
[21:14] <vanila> I think you should change the program to say "This program is free software distributed under the GPL" with a next button
[21:15] <vanila> you don't need to display the liscence or ask people to read it (it's huge and complicated)
[21:15] <vanila> There is no need to 'accept the terms' of the GPL when using the software
[21:16] <vanila> I hope that info helps improve your nice program a tiny bit! :)  It's great that you all create a free software nice video player for windows
```

Now, not sure if we should not show it at all, or if we should show something else, but this seems like a good compromise in the meantime.


No, I didn't see that discussion. But if it works for mysql and postgresql it's enough for the time being. Worked for me with postgresql. Don't know about mysql. You can test.

I'm not sure about this, should examine more adodb code. But I think that when not using offset and limit, it internally just fetches all the result set into memory, so trying to loop over it row by row does not solve any problem.

About the permissions stuff, please remember that what I've got finished now is not the final version. I will change it to match to what will come out of the discussion of correct permission mechanism. Did we have that discussion already or should we start it? What we propose now to release for the users, is in my humble opinion, indeficient big pile of shit.

And of course doing the permission checks on database side must also first be tested with big data sets. It could have some several efficiency drawbacks and the best solution might be to do just some pre-filtering in database, the rest in php.

Yes, I have made 2 pull requests to adodb, 1 still waiting to be made, it's not general enough. Only 1 merged. I had to have some fixes made to that project too. It couldn't handle upgrading Flyspray if using PostgreSQL... and that's why we have setup/upgrade/1.0/datadict-postgres.inc.php. It can handle at least this case correctly enough. Don't know what I'll have to do with setup/exportdb.php, that one too is currently broken when using PostgreSQL. Don't know even if it's related to Flyspray or adodb.


So just to be clear, you didn't update any of the tests to actually use jasmine or remove the node code comments in any of the files?


Is this just undefined behavior or do we want to enforce the reverse of this test? I think either a comment above this or actually replacing this with `self.assertNotEqual(case.type, 'newtype')` would be much appreciated by future you or me visiting this code


@bnoordhuis this patch gives me an error when #1152 is applied on it:

```
Jeremiahs-MacBook-Pro:io.js Jeremiah$ ./iojs_g test/parallel/test-timers-unref.js

#
# Fatal error in ../deps/v8/src/api.h, line 386
# CHECK(allow_empty_handle || that != __null) failed
#

==== C stack trace ===============================

 1: ??
 2: ??
 3: ??
 4: ??
 5: ??
 6: ??
 7: ??
 8: ??
 9: ??
10: ??
11: ??
Illegal instruction: 4
```


@kitsao this date is causing a restriction that only counts the results till the fourth day of the month, the reason the counts dont tally with the infection report


This 80-col reformatting in this file really hurts readability.


genial, en el response seria bueno tener tambn el icon de la tienda


DERF thanks for the catch!


this should be ``iuc`` in this repo


this function doesn't seem too useful but i'll give it a pass.

Maybe the genercize this function, have it take an array of key's to extract. Or see the `Validate` module's `ensure` function. Whatever, doesn't really matter.


What does the backtrace look like in gdb or lldb?


I must have totally missed that project group level feature, didn't realize something like that exists. It might change the whole picture.


`Registry.LocalMachine.SetValue("Test2", expectedValue);` also throws `UnauthorizedAccessException`


It doesn't matter is it com or pl, com redirecting to pl so it's the same.
I just saw fr so changed it to pl but to be honest I don't even know is it
affect on anything.

2015-03-25 19:59 GMT+01:00 Ivan Kwiatkowski <notifications@github.com>:

> Hello! Thanks again for your help translating the app.
> It would be better to leave out the strings marked as translatable="false":
> they don't appear in the app, and translating them causes compilation
> warnings.
>
> For this particular case, do you think it would be better to provide a
> google.pl link instead of google.com?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JusticeRage/ApkTrack/commit/6d7ed1769b4e3847b9c96001e371ea7c32415f3e#commitcomment-10397489>
> .
>


skillz


This WILL throw NPEs if the key is `null`.
I'd suggest switching around `LibMisc.BLACKLIST_ITEM` with `message.key`.


We don't really do this elsewhere in the code; I'd be happy with a literal 401 here.


Hmm I have never quite understood the license stuff but if it's not useful then I guess better skip it completely and just show "This program is free software distributed under the GPL" at some point as a link to the license text.


r+


reason for global scope here?


much better. thanks for removing the offset! :+1:


there is the same `#if defined` in src/statTimer/statisticalTimer.extern.h — not sure how useful it is, but I guess it should be renamed too :-)


same here, scope?


Grab the latest and try it.


OAuth


Great use of comments to avoid having to maintain line numbers. We should copy this for some of our C tests.


:ok: It works and fixes the issue. @mchelem, feel free to ship it.


Oh and quick question, I saw that strings layout is different in main file
than in polish version, should I make mirror but translated copy of it??

2015-03-25 20:36 GMT+01:00 Chamelleon <chaamelleon@gmail.com>:

> It doesn't matter is it com or pl, com redirecting to pl so it's the same.
> I just saw fr so changed it to pl but to be honest I don't even know is it
> affect on anything.
>
> 2015-03-25 19:59 GMT+01:00 Ivan Kwiatkowski <notifications@github.com>:
>
>> Hello! Thanks again for your help translating the app.
>> It would be better to leave out the strings marked as
>> translatable="false": they don't appear in the app, and translating them
>> causes compilation warnings.
>>
>> For this particular case, do you think it would be better to provide a
>> google.pl link instead of google.com?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/JusticeRage/ApkTrack/commit/6d7ed1769b4e3847b9c96001e371ea7c32415f3e#commitcomment-10397489>
>> .
>>
>
>


Ah, I missed that. I'll push it out.


saw approval from jdm
at https://github.com/glennw/servo/commit/7163a3c580b3467340a246d22890310819cc3efe


merging glennw/servo/moz-events = 7163a3c5 into auto


glennw/servo/moz-events = 7163a3c5 merged ok, testing candidate = 644dc405


Check to see if device is type 'iOS', if so, default to Apple Maps for directions; otherwise, use Google Maps for directions.


Well, I'm not sure either if we should do that, that is why I said my solution should do the job in the meantime. But if anyone else has any proof about the opposite let me know.


Thanks for looking into this :smile: ...sending out a PR now...


3: L102 copyright year is outdated, expected 2015 but got 2013


rax retry tests


Did you leave this in for testing?


Nice!


Ye may want to rename the file for writing on the server disk and save the original file name in the appropriate db column. Our system could have the files named 1 .. n e.g. This is to avoid us accidently overwriting an already existing file that had the same name as some other. Since we don't have the data bases implemented yet just put in obvious TODO comments in the appropriate places where this data access/storage would take place so we can add it later.


I think @hayd did some work on porting the old docs a while ago. Tomorrow I'll take a look at some outstanding features that would help with that.


__API Blueprint syntax error on line 884__: symbol 'Provider Collection' already defined (Autogenerated message from apiary.io)


we need to call `split.destroy()` in the else case, right?


same comment about destroying the split


It's needed! I tried loading just the aggressive-indent.el and it gave this error

> Loading /home/kmodi/.emacs.d/elpa_25_0/aggressive-indent-20150325.1131/aggressive-indent...
Symbol's value as variable is void: aggressive-indent-


Much nicer!


in the spirit of supporting nucleotide only, I have made aa_muts and nuc_muts. auspice looks for aa_muts


1: M111 unrecognized component name: base


I think it's brittle to try to exclude the directory from everywhere that might want to list providers or agents (pcs, crm_resource, but then get crmsh on board, and someone somewhere probably has custom scripts), but if we do want to follow that approach, I'd recommend prefixing the name with a dot or underbar to more clearly indicate that it's not a provider and should be hidden. That way the tools can be future-proofed by ignoring any such pattern rather than hardcoding this one directory.


if you destroy the split like I mentioned above, then this won't be necessary


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

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


In my opinion we should keep `var app` here to avoid polluting the global namespace.


I'm actually working on fixing this now (and will hopefully clean up case processing a tad), but I didn't want to couple them. Added a comment for the time being.


@sizzlemctwizzle

This change creates a **big security issue**. One from the [OWASP top 10 violation](https://www.owasp.org/index.php/Top_10_2013-Top_10): [OWASP A10 - Unvalidated Redirect](https://www.owasp.org/index.php/Top_10_2013-A10-Unvalidated_Redirects_and_Forwards).

You publicly store the redirect url without validating it, which is valuable for phishing and spam. The session redirect wasn't perfect too without validation, but at least it was hidden from the public.


Now everything is fine. Thank you!


All inside this folder will be imported into the style-imports.css file. Grunt watch will take this file and generate two more files – style.css and style.min.css.


Escaping the "-" sign for hostname field


Good job!
Thanks dude! ;)


This gives me this error:

> Debugger entered--Lisp error: (wrong-type-argument stringp nil)
  file-name-nondirectory(nil)
  lm-version()
  (progn (lm-version))
  (progn (progn (require (quote lisp-mnt))) (progn (lm-version)))
  eval((progn (progn (require (quote lisp-mnt))) (progn (lm-version))) nil)
  #[128 "\301\302\303B\"D\207" [lexical-binding quote eval progn] 5 1653732]((progn (require (quote lisp-mnt))) (progn (lm-version)))
  (eval-when-compile (progn (require (quote lisp-mnt))) (progn (lm-version)))
  (progn (eval-when-compile (progn (require (quote lisp-mnt))) (progn (lm-version))))
  (defconst aggressive-indent-version (progn (eval-when-compile (progn (require (quote lisp-mnt))) (progn (lm-version)))) "Version of the aggressive-indent.el package.")
  eval-buffer(#<buffer  *load*-459051> nil "/home/kmodi/.emacs.d/elpa_25_0/aggressive-indent-20150325.1131/aggressive-indent-autoloads.el" nil t)  ; Reading at buffer position 9208
  load-with-code-conversion("/home/kmodi/.emacs.d/elpa_25_0/aggressive-indent-20150325.1131/aggressive-indent-autoloads.el" "/home/kmodi/.emacs.d/elpa_25_0/aggressive-indent-20150325.1131/aggressive-indent-autoloads.el" nil t)
  load("/home/kmodi/.emacs.d/elpa_25_0/aggressive-indent-20150325.1131/aggressive-indent-autoloads" nil t)
  package-activate-1([cl-struct-package-desc aggressive-indent (20150325 1131) "Minor mode to aggressively keep your code always indented" ((emacs (24 1)) (names (20150125 9)) (cl-lib (0 5))) nil nil "/home/kmodi/.emacs.d/elpa_25_0/aggressive-indent-20150325.1131" ((:keywords "indent" "lisp" "maint" "tools") (:url . "http://github.com/Malabarba/aggressive-indent-mode")) nil] nil)
  package-activate(aggressive-indent)
  package-initialize()
  eval-buffer(#<buffer  *load*-38543> nil "/home/kmodi/.emacs.d/setup-packages.el" nil t)  ; Reading at buffer position 1914
  load-with-code-conversion("/home/kmodi/.emacs.d/setup-packages.el" "/home/kmodi/.emacs.d/setup-packages.el" nil t)
  load("/home/kmodi/.emacs.d/setup-packages.el" nil :nomessage)
  eval-buffer(#<buffer  *load*> nil "/home/kmodi/.emacs.d/init.el" nil t)  ; Reading at buffer position 4654
  load-with-code-conversion("/home/kmodi/.emacs.d/init.el" "/home/kmodi/.emacs.d/init.el" t t)
  load("/home/kmodi/.emacs.d/init" t t)
  #[0 "\205\262

emacs 25.x built yest evening


ast_optimizations (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (335 of 335 tests passed)

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


Nice, smart addition! Thanks for the pull request.