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


Still missing the same check for internal links though


Checkmarx scan results 2015-08-05T10:56:08Z

High
Medium
Low
Info

New
13
140
1649
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

5ef7ad106abf97d4e1844a0c0085fdd2d1a90978	@litals

068915702a6010a0c1f371e1a6d6c046e8039d87	@litals


aec and adp don't exist.


Agreed.


Having the table name here seems kind of redundant, as it can be fetched dynamically via `Thredded.user_class.constantize.table_name`.


Kjørte denne 100 ganger (helt sant!) lokalt uten å få den til å feile..


fine for now, but this might be a case where notifications are being used for one-to-one communication, which delegation is better suited for.


atomic-enterprise is the binary name. Right now there isn't an atomic binary but if there needs to be let me know and I'll update packaging.


Checkmarx scan results 2015-08-05T10:58:45Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
72
61
66
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

d5a07cb0b1744f15f637a4c7ffe8f2ac25f77bdd	@dudug

327d9f1dd663a94d99a8db8a9cf08c5d9cb48564	@dudug


I don't believe aedm and adm exist


@michalmuskala can you look at this? It feels like the traversing of the embeds do not belong here. As soon as a changeset is marked as delete, in the repo code we should traverse it and set its children to delete too. I have removed the lines below and no test failed. So I am not sure if this is covered or not tested.


Added
On Aug 5, 2015 5:08 PM, "Agustinus Kristiadi" <notifications@github.com>
wrote:

> @eckyputrady <https://github.com/eckyputrady> did you forget to add the
> actual files?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/timotiusnc/onboarding/commit/b09c7cf155d5116eab512a68628a86f5d39b9a82#commitcomment-12536974>
> .
>


We do have special code in the back end to deal with these expressions having a top-level scope. I expect this will have to be adapted. Don't remember where though...


There are already too many globals in vim ^^ I'm would prefer to let std_paths private, spec_orders (don't like the name) public and a public function which returns values from std_paths.


Like this :

{ SSpacer:{
		height:2
}},

Thanks for the info @AndreiLux , will change my kernels when have time.


change #else to #endif
This was a bullsh*t mistake by me, if GOVDEBUG would be activated, the whole code after the else would never be executed.


remove accordingly


A little utility library. Surprisingly handy.


I guess the convention for controller tests is to have controller as part of their names. Not so? :)


1: M111 unrecognized component name: webhooks


Actually should have made this change in waystone-io/justincloud :D


@APLANA-Alexey-Stolyarov The assemblies are published seperately with every update of the main package:

https://www.nuget.org/packages/OsmSharp.SQLServer/4.4.0.1046-beta


My impression was that we actually have that policy, but you always learn something new ;-)


Yeah, this is out-of date commit. I've just pushed rebased version.


@sclausen .d.ts files needs only on transpilation, so I exclude they from handler, but code not good. Please look, and if you want, may do better.

dTs = !!compileStep.inputPath.match(new RegExp(/.d.ts$/i));
if(dTs!==true) {
  compileStep.addJavaScript({
    path: compileStep.inputPath.replace('.ng', '').replace('.ts', '.js'),
    data: output,
    sourcePath: compileStep.inputPath
 });      }


Right, I'll remove it.


Don't you need some assertions before you handle the request, to confirm that things have actually changed after the request?


shouldnt there be a 1 as the first argument?


Undefined variable: offset
where is $pos used?


what should i do with these codes please tell me


What is "pre-sprint 69"?


On 5.8.2015 12:59, Stephen Milner wrote:
> I don't believe aedm and adm exist
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/miminar/origin/commit/df78234fe7465dc1b0a7d109a06fa91398c4121e#commitcomment-12537696>.
>
You're right. I've just pushed the PR.


Damn beat me to it!!


Yeah, it will read off the end of the entrance definition array, but nothing too severe. This is for those people that were trying to glitch the entrance tile to make it invisible.


this maybe should be a [variable from config](https://github.com/zikula/core/blob/1.4/src/app/config/config.yml#L18)


I currently have nothing better todo!


Should be lesson.md not plan.md or plan!

See the template: https://github.com/raspberrypilearning/teach-resource-template/tree/master/lesson-1


ffdf


sdfdf


Yeah it looks like RST doesn't like code blocks unless there's a blank line after the `::`. Another one of those nice features which is apparently just so intuitive that it doesn't need writing down anywhere.


Whoops, good catch - thanks!


Nice to see that standard 5.x is used in production :+1:


:D
Le 4 août 2015 20:03, "lief2" <notifications@github.com> a écrit :

> :D
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cSploit/android/commit/803a3f70ac80e1ec769e8d18d20ab2d37e3f2bfd#commitcomment-12523199>
> .
>


А если запустить `gradle build` из корневой папки?


Could you give an example of "garbage attributes"?
To optimize comparing parameters we can check also "hidden" attribute and work with only visible parameters.


Ah, right, I didn't realise the build dir part. Does cmake have a system to create a build dir automatically, or does it have to be done manually before running the cmake command?

Ah, I messed up the examples part. Should only be the binaries that are generated. Sorry about that. I'll fix it. If we do the build dir system, this entire commit won't be needed anyway :)


Sorry: was going by your email: "Lesson-1/lesson-plan-1/ should be /lesson-1/plan/" Wasn't convinced it looked right: changing back now.


Thanks for bringing this up as this was something I was considering to discuss in a broader scope. I am not sure if an assert is a good idea here for my understanding is that if we are to check on a precondition that is relatively inexpensive to verify, use of something like Preconditions.checkXXX(...) consistently seems a more natural fit and more robust. Another thing is that the assertion here seems too restrictive that I cannot load a vector given a buffer and metadata of a compatible type simply because vector names do not match or such. I am not sure if this is needed. I was in between removing assert or replacing it with a precondition check that captures the behavior of the method. To stay safe, I guess it makes sense to turn these asserts into precondition checks if no disagreement is raised.


Yes, I think you are right about removing the characters. The flag could be used to signal and log that characters are not in the 'default' alphabet. And make it possible to implement an auto mode that selects between the 'default' or ucs2. I am a bit allergic to lists:flatten so I will rewrite to code slightly :-)


sorry, please use overrides


Since every member of this struct is immutable, I don't see a problem with keeping it as a global.


The URL looks like that - but not the markdown file - sorry.

Really we shouldn't have links in text like that - we have navigation in the templates.


Just personal preference. If I had to give a reason, I'd say it's closer to the kwarg syntax for click.style.

On 4 August 2015 10:51:13 CEST, Jens Diemer <notifications@github.com> wrote:
>Why "dict()" here? and not just, e.g.: ```'warning': {'fg':
>'yellow'}```
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/click-contrib/click-log/commit/707fabd7f05c0613fbdb18f3c77349e4eb07062b#commitcomment-12510740

--
Sent from my phone. Please excuse my brevity.


or `"arm".equals(Base.getNativeArch())` is `getNativeArch()` might be returning `null`.


This comment shouldn't be removed when the homepage and download URL hostnames differ. The same applies even for ".gov" domain zones.


How/where is `nativeArch` used in practice?


Thanks for fixing this bug.


OK: it should all be fixed now. No damage done, I hope?

With regard to the 404s, haven't been finding much, but it looks as if the Networking lessons have non-standard filenames. Do you want me to go through those as well? They aren't on your list.


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


I did not want to create a directory unless nvim will definitely use it.


will do.


Just to note:  This commit does break the semantic versioning guidelines.


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


Oh, the irony http://git.661346.n2.nabble.com/GPG-signing-for-git-commit-td2582986.html (see why Linus thinks that GnuPG-signed commits are a bad idea)


Thanks, overriding works. In case someone is not familiar with the override mechanism here is what I needed to do in my bower.json:

```javascript
  "overrides": {
    "font-awesome": {
      "main": [
        "./css/font-awesome.css",
        "./fonts/*"
      ]
    }
  }
```


For the long run, I have a feeling the error label components can be updated more efficiently than by doing a add/remove cycle.

However, right now the labels are only added on demand - no error on init() and you get no error label. A better approach might be to have them added to all components all the time, and then use CSS visible to show/hide them as needed. That would mean there would be one more DOM element that would stay in place, reducing churn and flickering.


Amil


Perhaps sort these values in ascending order (I had a hard time determining if my list matched)


Really?
When i disable the setting
>Show the offline content on the website when logged in on Backend (only for the logged in user)

My menu doesn't display offline pages?


dsafdas


The Networking resource needs a lot of work. Teach resource lessons should have a lesson plan (lesson.md) and (optionally) a student worksheet (worksheet.md). The lesson plan should be formatted exactly like the [template](https://github.com/raspberrypilearning/teach-resource-template/blob/master/lesson-1/lesson.md) (note the headers and how they're displayed [on the website](https://www.raspberrypi.org/learning/sonic-pi-lessons/lesson-1/plan/)). They shouldn't have any additional files - any other content should be brought into the lesson plan or worksheet as appropriate.

If you can find a way to make it work, please feel free. Otherwise, we need to think about how we convert it to the new format. See the resource's [issues](https://github.com/raspberrypilearning/networking-lessons/issues).


You can use `#{version.to_f}` to get the same effect. Of course, it's a little bit hackish but looks much more cleaner.


No need to add the additional variable. Even if it duplicated in both releases.


What do you think of
``` javascript
dTs = !!compileStep.inputPath.match(new RegExp(/.d.ts$/i));
if (dTs) {
  return true;
}
```


@matethurzo, can you please review this change? it's just a jsp creating links to the staging site and the live site if those exist. thank you!


@timoweiss why select realtime here and not just the default connection?


:8ball: :pizza: :recycle:


Non je pensais juste qu'il s'agissait d'un oubli...


I removed And so we wont' get over some magical number :P


implement**ed** then


I am also getting the same error as @pktri-git.
It happens if the user has an account already and you want to link them.
I'm having a look for where the problem is.


二叉树


weird.


Reviewed, looks good.


This is not a type of a function - this is type of a return value of a function


I do not quite follow your question. Introducing `Hash()` and `#to_hash` in https://github.com/cucumber/cucumber-ruby/commit/925a6739677d8df0871274f200c45f43ffbf2d51 broke [the build](https://travis-ci.org/cucumber/cucumber-ruby/builds/72147511) (for Ruby 1.9.3). Using `Hash#try_convert` made [the build](https://travis-ci.org/cucumber/cucumber-ruby/builds/73328950) green again. If you know a better construct, please advice.


Still think this should be void*, not uint8_t, if this is supposed to carry generic data.


I'll see what I can do once I have finished the others. I'm trying to do the other checks as I go too (broken links, typos etc) so it might take a little while.


Please release this bug fix. Thanks.


This is actually not necessary as derivable from indexr!


This should probably be:

    char key[strlen("OPTION_000")]);

(note that on gcc strlen() is special and is resolved at compile time if possible, hence this is not even a VLA).


Shouldn't that be ``` `` `::function` `` ``` ?


This is a nice tab to space conversion.


Hmm, nevermind. :P


Should it also support `[text](::function)`]?


 STR_ALIEN_SHIP_ATTACK appears in the ruleset as the name of a mission, which would be displayed when UFOs are detected by hyper-wave. however, no UFOs are actually spawned for this mission type, so in practical applications, the string will never actually be displayed in-game. for all intents and purposes this is an internal variable name, and probably doesn't need the STR_ prefix.

HOWEVER:

assuming a modder decides to add some UFOs to the shipping lane mission waves, this string WOULD come into play on hyperwave detection screens, so it is conceivably justified.


I sign tags, which I believe are in line with the Torvaldian Way.
On Aug 5, 2015 4:20 AM, "Hugo Roy" <notifications@github.com> wrote:

> Oh, the irony
> http://git.661346.n2.nabble.com/GPG-signing-for-git-commit-td2582986.html
> (see why Linus thinks that GnuPG-signed commits are a bad idea)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/amoffat/masquerade/commit/9b0562595cc479ac8696110cb0a2d33f8f2b7d29#commitcomment-12538046>
> .
>


Code duplication, extract session to wrapper and use try-with-resources.


nein, das sind nur die änderungen, die ich noch vorgenommen habe. durch vorige änderungen kann mensch sich von dieser ansicht hier hangeln, indem oben den 'parent'-commits gefolgt wird. da sind nach zwei schritten dann eure.


Why are you ignore exceptions?!


Tss, tss, single line if blocks with {}...


How about correct naming?


Formatting.


The license here should be `:commercial`. It has a trial period.


DAOs MUST implement interface!


Formatting?


Wow, so long error message... I'll have a think about how to show such long error messages to users within Vim.


Ignored exception!


wouldn't `Iterable` be better?


@mrehanali Its best practice to never upload the user uploaded file with the same name, instead a unique timestamp should be generated and that should be the image filename.


> Yes it isn't strictly related to API 3.0.0, but without it, the changes I made don't work. Happy to split it into a separate PR if you prefer.

I'm afraid that both you fix and the initial implementation are not correct. Possibly your fix is working when both client and server are located in the same time zone. But what happens if not? Could you please try this IMO real fix here (https://github.com/ksooo/pvr.hts/commits/server-timezone) and report back? Beware, it compiles but I did not have the chance to test yet


to be defensive, you would filter null values from the list argument, just to be safe


Name convention?


yep, will update.


Dead code in comments.


missing `Objects.requireNonNull` to guarantee that `getLanguageKey` never returns `null`


Do you see any possible cases when using only `word-wrap` would cause problems?


properties should be before the constructor


could have an `@Immutable` annotation


missing, equals, toString and hashCode methods


teste


To be deleted


ask ui team if this line needed


The "(alternative)" does not seem to work though (here or elsewhere). I've checked other examples that work and after a quick look I'm not sure what I've done...


esse é um teste


missing `equals`, `toString` and `hashCode` methods


fixed this and the other one, thanks!


it looks good thanks!


Checkmarx scan results 2015-08-05T11:46:38Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
72
61
66
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

d5a07cb0b1744f15f637a4c7ffe8f2ac25f77bdd	@dudug

327d9f1dd663a94d99a8db8a9cf08c5d9cb48564	@dudug

a479af2dc60c5e3d47a4c0308f1dec16edf183ad	@dudug

fedf746368489ecbb0ee891d152befbab44db1f9	@dudug

27ad63073e3c2aa12ea263fb4b9914826dc2dfc5	@dudug

c6c10e625db5f31ccfe15e2d7029e07135f69039	@dudug

5f913f602916b1a5751f4be2742e6cf7c70f906c	@dudug

1ba42ffbd647b4b765baf787f9c8ddcb3e72c014	@dudug

cc8fb54ec5c3d0840f90fbebe2f62d90390aaac9	@dudug

e5077af18528bac6d3f8c636353e8923f84d4707	@dudug

8c243f37f0aed799d586c32a5b9400dc4102d020	@dudug

059fc0173cbb8a05fe212a0162a21afb499cd454	@dudug

c4fd57c54da5a74526a4b92ab6b8845736279988	@dudug

d4d93b42ce2890e4e7973d279acd50daead22f8f	@dudug

dfc68329bd02eeca9274805825156ceb8a43deef	@dudug

9a3fdd0be88e1597aab1c0d785182bb0783890c0	@dudug

a8f36e99056cb2126abb9eaee0f87e2a28671a56	@dudug

909ad4704c846df3febb2b65c0929e5530d45613	@dudug

049d7eeb2975dd58ed65adc7c32042c0667d2f14	@dudug

e84bad447da69baaf9314b2b41d8aabd25ac571c	@dudug

41ed5e8e316e2ab7f48cbb5ca3f82843b6e75a85	@dudug

5bd601d6b8fa952106c1afe1a06bd226c0959bfd	@dudug

ef5f142de7c6fcc94a0adb3bd469f7db8f2fbe91	@dudug

d870498c7f84672c6228a965cfa83aeb5eef5904	@dudug

f829b67a2028c7bff706338de1a6766c5918930a	@dudug

26be59b1dc65eaa49cdc9b73d28bcb2fbd074150	@dudug

61447f9f8435f4498b269aa56383aad86e5d1caa	@dudug

fe7fa9367d2ded0a6af660c19dafb0c3688d0aef	@dudug

dd6707de1fda624f4a55a0ad951bc5a998f3cb5d	@dudug

0e60f69d71dae0871b2895fb544c424c6bfd5dae	@dudug


Do you really need this?


You can set the version to `1.1.8_57`.


I'm not a big fan of coupling between package directly into beans, in addition, here, it means `org.sonar.server.computation.component.Component` if part of the `ComponentImpl` class, therefor exposing `org.sonar.server.computation.component.Component` to the implementation


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.460](http://teamcity.monogame.net/viewLog.html?buildId=9413&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 538, ignored: 6 Build time: 00:25:31


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


How about logging?


Why should we use Integer here?


Minor nitpick: couldn't you avoid `memset` by just checking if `sol_spi_open failed` before?


Logging?


Is this sufficient for users that changed their email but did not yet verify it? Or can I get a forum to send notification emails to an unverified address this way? ("spam")


Remember to run checkstyle, this `|=` needs a space...


@Megatron007 bisi this ain't needed vendor mein he sab daalte hain :3 im reseting and removing


Repository?!


missing checks on null values in String array


Replace `1.1.8_57` with `#{version}` in the download URL.


Get rid off this var.


Fixes #1


@mrehanali These are two separate logical things, so essentially should be in two separate commits :(


score 500


Name convention?


Please remove this commit and don't add unrelated changes to the PR.

In this case you have a bug where decalsToDraw can contain the decal twice.


nice


t2 comentário


Ok for both point


Should be as util class.


really, it does not make sense to fail in the build method
the stacktrace will be useless to the developer if it fails in the `build` method, it won't tell her where she set an incorrect value and won't be able to investigate fast and easy


Bad naming


It's ignored -> I'll add it to the doc


I don't think we need this gibberish in every file, there's already a LICENSE file in the main directory.


Browserify is now compressing, true?


dead code


I hard coded these to match the test machines I use in production. If these scripts are useful to you as well, then yes, we should either pass in the machine architecture as an argument to the script or determine it automatically.


Ok


`ComponentImpl` should be instanced only once per `MeasureComputerImplementationContext` instance


propTypes should be in the end or in the beginning? I find propTypes on top more informative for reader what the component is accepting.


indeed !


logging!


Sorry, I always get proxy confused with reify.  Would it be possible to use reify here instead of proxy?


Oh, I didn't realize what you meant by "passing the widget instance as the argument".  Hmm, that seems strange.  You are tricking `computeProperties()` and `refreshRendering()` into thinking that many properties have been changed, whereas really they are all the default value.


Cannot return empty because it's not possible to set no output metrics in the setter of the builder


Don't define types in `.hrl` files. Define them in the respective module - if needed in an `.hrl` file, refer to them with `module_name:` prefix.


this set should be created only once per instance of MeasureComputerImplementationContext


Cannot return empty because it's not possible to set no input metrics in the setter of the builder


Other than that, looks ok. If no one else says anything, you can merge it - but remember to fix checkstyle issues before.


Use c:url


Thanks for the latest update - this process is complex (even without workarounds such as this), and now that there are 3 or more of us who need to be able to reliably reproduce these stylx files, we need to be extra vigilante about keeping the JMSML -> stylx creation process current/correct/repeatable/etc.


hahah


Great. I recompiled telephony-common.jar and dual sim now works. Is sim deactivation working on your build? I cant deactivate one sim, and I read somewhere that Sprout cm12.1 has this issue too. Thanks.


Bien vu !


Exctract  javascript


Indeed, it's case sensitive and it can return null


No, not exactly. However, to minimize work and get this thing going now, I would suggest the following:

Use your original implementation but rename the union variables. I don't like "native". We should name it "handle" as this is the real handle which should be used everywhere in the code. The "handle64" is only a placeholder and shall never be used. Therefore, we should not name it "handle64". I would suggest "padding" to show that it's only a placeholder.

Then, the struct could look like this:
```c
typedef struct
{
    union
    {
        tTimerHdl        handle;
        UINT64           padding;
    } timerHdl;
    union
    {
        UINT32          value;            ///< Timer argument supplied as UINT32
        void*              pValue;         ///< Timer argument supplied as void*
    } argument;                            ///< The timer argument the timer was initialized with.
} tTimerEventArg;
```

What do you think about? Is it clear now?


Well done.


`ComponentToMeasure` could be instanced only once, in the constructor


I was asking myself the same question, but as it's returned a sorted list, returning an iterable won't highlight that this is a sorted list. Or I am wrong ?


Shouldn't be module?


How about clear naming?


Feasability concerns have been resolved by real life tests :-)


This is little bit confusing. It looks like it has state but acts stateless. And it definitely have to be stateless.


ok


Seems that this is actually written down. Although as important as it is, it is usualy only the last thing mentioned....

[Sphinx doc](http://sphinx-doc.org/rest.html#source-code)

> Literal code blocks ([ref](http://docutils.sourceforge.net/docs/ref/rst/restructuredtext.html#literal-blocks)) are introduced by ending a paragraph with the special marker `::`. The literal block must be indented (and, like all paragraphs, separated from the surrounding ones by blank lines):

Also in the ref above http://docutils.sourceforge.net/docs/ref/rst/restructuredtext.html#literal-blocks

> Blank lines are required before and after a literal block, but these blank lines are not included as part of the literal block.


Anyway, I guess we can easily fix this.


No, because language key can be null


missing javadoc anyway


Je ne comprends pas la logique du Set(x,y) + Get() par rapport à un Get(x, y) tout court.

Et Get n'est peut-être pas forcément le meilleur nom possible du coup.


indeed


$('#overlay').remove() ???


Yes I could've but it would have been a lot more work. In FreeBSD devfs manages /dev, you can't just tromp around in there running ln -s and expect that to persist. You'd have to detect all the mpr cards and then modify devfs.conf before mounting devfs.

I also didn't mention that I've done this before. To sas2flash even. So it was more mundane than Herculean. ;)

Thanks,

Josh Paetzel

> On Aug 5, 2015, at 1:42 AM, David Gwynne <notifications@github.com> wrote:
>
> couldnt you have just created some symlinks?
>
> —
> Reply to this email directly or view it on GitHub.
>


Thank you. I've been afraid to mess with these because I don't understand the two different numbers and the the security implications of either.That said, let me describe a use case that is important to me.

I have several sites I use at work, one pretty much every day, the others rarely, maybe every other week. I would hope that my daily activity would keep me logged in day to day and even over weekends but maybe not vacation. Wouldn't I need activeDuration to be three days plus eight hours to have me logged in all day Monday if I last touched wiki the previous Friday morning?

At work we use a default data retention period of eight days. This handles the common case of looking back to the previous week over useful periods of interest week to week. That duration, eight days, might be a good number for us. It sounds generous but it doesn't last forever so forgotten sites will require reestablishing login when rediscovered.

So my suggestion then is to increase from 12 hours to 8 days unless the 90 day number is already covering the cases I describe even more generously.


Dead code


> @dead2k is relativizing here what you would expect?

Seems reasonable enough.


Just remembered we might be able to use Symbols to truly hide the special props. I'll look into that.


Sounds right.


> @deads2k and here

Seems less reasonable, though I could see a claim for consistency.  "openshift touches it and it will be relativized."


I don't understand -> ComponentImpl is not in the API, why do you say that here we're exposing internal class ?


Attempting to add signature support, most likely doing it wildly incorrect.


Make this final field.


Same.


remove


Right !


ok


Correct handling?


exposing an `Iterable` leaves no choice to the consumer of the method to consume the returned collection in the specified order (at lest until it is copied into another collection where the consumer can do whatever it wants), where a list leaves the possibility by contract to access computer randomly (http://docs.oracle.com/javase/7/docs/api/java/util/List.html#get(int))


\o~ \o/ ~o/


Possible NPE, in case root == null


In fact we should fail in the setters and in the build, as nothing prevent to directly called the build method


Exctract obtaining root to method


Indeed !


Same


Is this because https://github.com/pweil-/origin/commit/bfd12e9d1a2d824022b53fd655309910206f6cf2 is out of date?  If so, squash it in there.


i would have to grep all plugins to see which one uses it.
From what i understand is you do
   xbmcplugin.addSortMethod(int(sys.argv[1]), 1)  sort by label
   xbmcplugin.addSortMethod(int(sys.argv[1]), 19)  sort by date
and this adds two sort methods to that path


right


If it floats your boat, but go ahead and squash it into the existing carry commit.


then a `@Nullable` is missing on the argument of the constructor and a `@CheckForNull` on the getter of the class and the interface


exact


@joetsoi Cheers, maybe a proper PR would be better next time :wink:


Returning `true` from a timeout doesn't make it run again?


Seperate into 2 classes: RestClient and MainApp


Log


Ditto.


Constants?


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

VSO State: Done


Extract formatiing url.


Exctract obtaining response


Meta files must be added to verson control!
See [the doc](http://docs.unity3d.com/Manual/ExternalVersionControlSystemSupport.html).


C'est pour les bruits complexes. Avant, les bruits complexes devaient
transmettre x,y au bruit simple qu'ils utilisent. Ce qui est inutile car
leur seule rôle est de modifier l’échelle des bruits simples pour chaque
octave. En aucun cas ils ne doivent utiliser x et y.
Du coup, la conséquence avant c'est que les bruits complexes devaient être
écris pour chaque dimension, alors que c'est le même code juste avec un
paramètre de plus.

D’où la nouvelle architecture. Le bruit complexe contient une référence au
bruit simple. Pour avoir la valeur du bruit complexe tu appelle d'abord
Set(x,y) sur le bruit simple, puis Get sur le bruit complexe (qui en
interne appelle plusieurs fois le Get du bruit simple).

Donc désormais une seule source par bruit complexe quel que soit le nombre
de dimensions.

Le 5 août 2015 13:59, Jérôme Leclercq <notifications@github.com> a écrit :

> Je ne comprends pas la logique du Set(x,y) + Get() par rapport à un Get(x,
> y) tout court.
>
> Et Get n'est peut-être pas forcément le meilleur nom possible du coup.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Overdrivr/NazaraEngine/commit/44a46c994a67778809ba11834ffa2fa362f472ce#commitcomment-12538870>
> .
>


@mxaly Already pointed me out that while you are using ES6, you need to have class defined first :(


Exctract parsing response.


Constant?


Dead code?


Hmm... Thanks to POSIX->Windows path mangling, this will change the config *that is shipped with the installer* to include an absolute path *that is only valid on my machine*...

How about contributing a PR to https://github.com/git-for-windows/git that contains just this patch: https://github.com/drvink/MINGW-packages/blob/b5a56ef12b110cb788bc3a368f6755a7d826ff8a/mingw-w64-git/0001-msys2-usr-link-hack.patch.in ?


Okay, thanks for explaining in detail.
I will share the changes as per the comments.


how to use it?


Seperate again.


there is definitely no need to link `org.sonar.server.computation.measure.api` with `org.sonar.server.computation.component`, its a bean, KISS and SOC


Scala? :) I don't really see a lot of Ruby in there. Groovy or BeanShell would be a better fit on the JVM, I guess.

Clojure also borrowed some things from Ruby. It's cited as an influence here: http://cdn.oreilly.com/oreilly/booksamplers/9781449394707_sampler.pdf


"Atention" -> "Attention" or "Warning", also please add a `tr("Warning")` around it so it can be translated


Code duplication!


conect -> connect


you're right


Do you know in which unity is this? If so, say so. If it's a raw value without unity, doesn't it have a conversion to a standard unity?


if(inputStream == null){
return;
}


你好


Refresh


You've duplicated the protocol in the url here fyi :sweat_smile:


remove


Show error


fdfdfdfdf


aha, so it was


Thanks, I'll take a look when I get time


Тут же дублирование и ты в 356 проверил что это функция


Coment.


Exctract show error to method.


I also wonder if we could refactor out this part and have a central connection to the db. I assume this will be used by different parts of the app in the future.


Formatting


hello world!


as far as i can tell, the only difference was the order in which the duchies were listed


Added an if statement to read the footage type from userData.m - makes it more user-friendly


Fixed in commit https://github.com/owncloud/android/commit/7c9c4a4ca6db4ab2aa658ee119c9ccc4056a7273


@CheckForNull is already in Component.FileAttributes.getLanguageKey()
But the @Nullable si missing


Formatting?


ok, this argument I understand


They are pointers to const char, not const pointers to const char so they can be mutated (const char *const bla). But the problem is in the "global variable" part. 430 globals variables doesn't help us to understand how vim works.


Log?


OK, I see. Hope this will help me with my g2m camera...


Fixed in commit https://github.com/owncloud/android/commit/97b9932d811ab99e6adf74cd0937c2b874a9721a


Well, no link to device datasheet, this is sad =/


No link to the datasheet?


Fixed in commit https://github.com/owncloud/android/commit/3956a50855fa04621cf2ba105e1a716e9a501c08


you could use `ImmutableSet.copyOf(Sets.intersection(computerByOutputMetrics.keys(), measureComputer.getOutputMetrics))` and check if it is empty, if not, you can report all conflicting metric keys and not only the first one


Exctract creating image to method


thanks a lot! :+1:


Fixed in commit https://github.com/owncloud/android/commit/f2899989468411cc3040d8b49ebbda4b9a029154


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.461](http://teamcity.monogame.net/viewLog.html?buildId=9417&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 538, ignored: 6 Build time: 00:27:04


Please acquire `g++-5` from `ppa:ubuntu-toolchain-r/test`, as that is the version we intend to support (GCC >= 5), since earlier versions don't have full C++14 support


Added progress bar to layout in commit https://github.com/owncloud/android/commit/c2418d24d561e2988bac03a79f3390988d958386

@masensio how would I navigate to this screen on a phone? I couldn't find out, so that's the reason it has been untested :(


Well, what is a "byte string"? I mean, we just know it's "something", and in C one usually indicates that via void pointer, rather than a uint8_t pointer...

In C all pointers automatically convert to void pointers and back, which makes this nice here, as people using this don't have to cast explicitly into uint8_t* anymore then. Hence I am pretty sure this should be void* like everywhere else...

Compare this with memcpy() which also takes void* rather than uin8_t*, to make the explicit casting unnecessary, and all that even though one could say it just "copies bytes"...


might as well break out of the loop if hasIndex is true.


What was wrong with Maven? :)


LoL, thermal-engine & mpdesicion is RIl? :)
Also will help me if I need to update RIL...


OK, I'll look into doing that. But what should be done about this issue, then?


But you can't rely on `@props` inside a helper function.


Use equals method instead of ==


There is something wrong here:
// About Us
$locale['debonair_0410'] = "About Us";
$locale['debonair_0411'] = "Email:";
$locale['debonair_0412'] = "Please install the Official PHP-Fusion News Infusion";
The section is "About Us", but asks for "install the ... News Infusion"?


you have a problem with this implementation.

The contract of `org.sonar.server.computation.measure.Measure` is not the same as `org.sonar.api.ce.measure.Measure`, especially, there is no guarantee with the former that the type of the measure is the same as the type of the metric: it can be NO_VALUE.

Maybe you should ignore NO_VALUE measures in get context's `getMeasure` and `getChildrenMeasures` methods


I agree I wanted to add it to `input_mandala.coffee.cjsx`


`Objects.requireNonNull` missing here


Does it make sense then?


Where is this information provided now?


if(!restBookClient.decrementPage()){
showError();
return;
}


if(!restBookClient.incrementPage()){
showError();
return;
}


what about adding this step right before it is used (ie. before `ComputePluginMeasuresStep`)? This would avoid loading computers if any of the steps before `ComputePluginMeasuresStep` fails


.field() -> .expressionOrField() as of now ;)


Very well )))

I think ShMcK must comment .d.ts processing, before make decision about server side.
I'll try say about whishes in Angular2-Meteor-Demos hub, as I can ))


Can you explain what is "generation unit"?


Just pointing out, you missed the r in "strip".


oh, interesting !


Change throw to done(err), done takes one argument called err


ok


Ooops, forget it... With an expression it should not be active.


You could use `list.insert(index, extra-item)` to not only cut down num-of-lines but to be explicit about the order, since order seems to be important here.
No biggy, but highlighting just because similar logic is in `case_list.py`


Indeed, I need to check that the type of the measure is the right type


referring to the comment below, you should enforce that no `MeasureImpl` can be created from a `org.sonar.server.computation.measure.Measure` of type NO_VALUE


Get rid of "else"


yes


remove


I think it depends on what exactly you mean by vertex data.  Point radii and stroke widths are related to vertex data as well.  In this case, it doesn't matter much to me whether this goes in `style` or a class method.


s/its/their/


same handling, move it into one section. Use IOException | ParseException.


Still refresh ;)


Ok, postupně to dám dohromady;) Teď toho mám zase hodně, abych tomu věnoval celý čas.


hum... that will be hard to test

especially not testing the same thing in the UT of the step and in the UT of `MeasureComputerImplementationContext`


Change all throw to done(err)


I know it's your change, but can you please change "func-dest" to "funcDest" for consistency?


One more thing, please put the `.hrl` in `src/` (and tweak `rebar.config` if needed for the tests to work). `include/` is for header files public to users of the library, which is not the case here.


Oh, right :D My fault, yes, But it should only merge if `category_names` not exist.


Haha pay no attention to the man behind the curtain! Just doing some experiments with Gradle that I probably am just going to scrap. Don't worry there will be no pull request coming from this.

Btw how come my builds always have runtime errors with image resources not being found? That is the only issue I have been having with Maven. Maybe Im doing something wrong with my IDE...


Fix issue #61 , #67


why ?


"constructor override used by Pico to instantiate the class when..." will be more helpful when we go back to this class next year


Just for my curiosity - what's that?


Fix issue #61 , #68


WOW NICE


add readme


quid if there is neither `Metric` nor `MeasureComputerProvider`?


There are several considerations that made me put it in Session:
1) It's very easy for Session to figure out whether a sample rate was set or not as it's handed that information on a silver platter. Any other class (e.g. View) would have to check if any of the signals contains at least one segment with a known sample rate.
2) From my point of view, single segments that have no sample rate information are very unlikely to appear in the field, so I wanted to keep the overhead low and not iteratively check for things that either will never be there or could/should be highlighted differently. A single segment without a sample rate could have a hatched background, for example.
3) I'm not sure switching between units within a measurement is user-friendly. Also, I'm assuming that we'll only ever deal with devices using a constant sample rate, so my assumption is that once we encounter a sample rate, this sample rate is valid for the rest of the data as well. Granted, a mixed known/unknown scenario would break this assumption but is that really a valid use case? I'm not quite sure.

From my point of view, using samples should be the last option if all else fails. For multi-device-measurement with one device not providing a sample rate, the user could be asked whether he/she wants to manually enter one or use the sample rate of the other device(s).

Thinking about it again, maybe 1) can use the assumption in 2) and only look at the first Segment instead of iterating over all of them or iterating over the ones currently seen. Maybe that route is more elegant than using Session after all.


Forgot to enter betrothal church


why "i+=2" (shouldn't it be "i+=1")?


Possible bug in resource loading? That would warrant a PR :)


I didn't know about it! Worth changing things up for?


Ok - never mind. I understand it.


why do you have to filter on `computers`?
it should be a set anyway


@pexcn 嗯，第一个是频率，第二个是电压μV，第三个不确定，目测是l2宽带开启数。


@ztc1997 嗯，谢谢~


ok


Can this be back ported to Wildfly 8 and 9?  This null check may be what is needed to have session-less JASPIC modules


missing `equals`, `hashCode` and `toString` especially because it is used in `Set`

`equals` and `hashCode` should ignore the `measureComputerImplementation` property for sure


Right ! Another constructor is missing


Haha okay I just assumed I might have been missing something and set out to solve it on my own. But I will go ahead and submit one. I'm surprised you're still watching this project amidst everything you're doing right now.


divider = height / 880;
imv.setFitHeight(880);
imv.setFitWidth(width / divider);


missing `equals`, `toString` and `hashCode` methods


When I try the shortcut, a window appears that says: The selected type contains no fields which may be initialized in a constructor. What am I doing wrong ?


We don't need to worry about overflow for ```start + accessSize```?


Oh yes, it's an 'old' code, because in the past I was adding a fake computer with core metrics, but in fact it's useless.
So no need to filter anymore !


Nit: could swap (`BigDecimal.ZERO.compareTo(amount)`), although it doesn't seem that `amount` could be `null`.


@natewarr I believe logback configuration should be done on the application level, not the box level. The reason is multiple play apps all running on the same box would all have potentially different configurations as play 2.4 requires you to set logging levels in the logback config file instead of the application.conf file as it has been done in the past.


Nit: indentation looks slightly off here and below


Why not use `AbstractBuild.getSensitiveBuildVariables()`? It does basically this, but is more comprehensive, as it looks at `BuildWrapper`s as well.


`allComputers` is useless, it is always equals to `computers`


It isn't actually needed. The annotation is for auto-generated documentation.

See the "Annotating your Endpoint" section on http://camel.apache.org/writing-components.html


> what should be done about this issue, then?

How about contributing a PR to https://github.com/git-for-windows/git that contains just this patch: https://github.com/drvink/MINGW-packages/blob/b5a56ef12b110cb788bc3a368f6755a7d826ff8a/mingw-w64-git/0001-msys2-usr-link-hack.patch.in ?


Not sure it's a good idea to effectively cache sensitive variables. Maybe they can still be added during the build?


ok


yeek. I still don't understand why you need this :-(


this could be replaced by a `addAll(computers)` if it exists
it would help put the rest of the loop into a method which could be named and documented to understand what it does


right !


I look at it time to time, since it is still used by people, according to GitHub statistics. You're right - pretty busy times, cannot find time even for ControlsFX. Hopefully results are worth the time spent.


> Which is unusual for Arabic, to say the least...

That is because the new Uyghur alphabet was introduced by Chinese based on an early Latin alphabet that wasn’t popular among Uyghur people (it is basically a one-to-one mapping of the Latin alphabet, including the hyphenation support). The historical Uyghur alphabet was more “Arabic” than this.


:+1: The fix looks simple enough.

Is it worth adding a real test (e.g. pure usage scenario without recurring section) or is the one you fixed (`testZeroDollarEvents`) enough?


@mrook: we'd have to tag this then in git as 1.4.1 probably


This caused #25866 (it didn't even consider if matched string is not literally equal to the regex pattern).


This comes with guice 2.0, which seems to only support the com.google.inject annotations. This unfortunately requires making camel-guice a hard dependency.


Секция, в которой ты напишешь объявление дружественной функции (public/private/protected) никак не влияет на её видимость


Why is this removed? This isn't where we would like to be.

My feeling is that if you set:
"api/v1" ""
in hack/update-generated-conversions.sh line 59 to something different, this would survive.

Maybe:
"api/v1" ""github.com/GoogleCloudPlatform/kubernetes/pkg/api"
instead?


Hi, I've install the tool from npm, run and it is working partially. I load the builder project from repository and try to edit something it is not working as expected: i press + and the console shows the error "cannot take frameEndpoint of undefined".

so I fix this and everything start working fine.
I suppose that frameEndpoint initialized only when it working on the web, not locally throught http://localhost:2222/builder/.


updated


I don't have the code to suggest, but as earlier it would be nice to report all conflicting metrics and not only the first one


review finished. looks good


I don't think this method is correct.
What if t is of type []int - this method will return true for it, but it should return false, right?

I think you should explicitly check it for Slice to (not sure about maps though).


`.sort()` method muted its original object, it's actually not necessary to put a `=` here. I consider it as a JavaScript problem though.


This is all about software that long it for human enjoyments


Don't have to be this long according to the docs:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/sort


First thing, this whole thing could been simply:

ActiveMQConnectionFactory cf = new URI("tcp://localhost:61616?type=CF");

No need for the parser


93653360-ResetVarsAtEndOfWarmup (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (744 of 744 tests passed)

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


`ToComputerByKey` is the same for every computer, it could be instanced only once per call to `sortComputers`
you could change the second argument to be a `ToComputerByKey`


Ok, i'll do that if time allows, otherwise I'll put a TODO :(


This is not required as mio returns IoErrors in which the `Error::Raft` already covers.


ok


Fixes #4 and #12.


this is wrong, a Metric always have a key


@ghislaineguerin I don't see the 'btn-white' class being built in the /dist output..
Is there something off with this?  If it helps I don't mind using static colors (ie. 'color: white') for now, since it's needed for me to deploy Featured Product.  Thx!


You don't remove the "/tmp" directory after your change.


Source is not found

  GET      libcec
converted 'http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz' (ANSI_X3.4-1968) -> 'http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz' (UTF-8)
--2015-08-05 14:40:59--  http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz
Resolving mirrors.xbmc.org (mirrors.xbmc.org)... 89.16.176.16
Connecting to mirrors.xbmc.org (mirrors.xbmc.org)|89.16.176.16|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2015-08-05 14:40:59 ERROR 404: Not Found.

converted 'http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz' (ANSI_X3.4-1968) -> 'http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz' (UTF-8)
--2015-08-05 14:40:59--  http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz
Resolving sources.openelec.tv (sources.openelec.tv)... 82.220.2.33
Connecting to sources.openelec.tv (sources.openelec.tv)|82.220.2.33|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2015-08-05 14:40:59 ERROR 404: Not Found.

converted 'http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz' (ANSI_X3.4-1968) -> 'http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz' (UTF-8)
--2015-08-05 14:40:59--  http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz
Resolving mirrors.xbmc.org (mirrors.xbmc.org)... 89.16.176.16
Connecting to mirrors.xbmc.org (mirrors.xbmc.org)|89.16.176.16|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2015-08-05 14:40:59 ERROR 404: Not Found.

converted 'http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz' (ANSI_X3.4-1968) -> 'http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz' (UTF-8)
--2015-08-05 14:40:59--  http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz
Resolving sources.openelec.tv (sources.openelec.tv)... 82.220.2.33
Connecting to sources.openelec.tv (sources.openelec.tv)|82.220.2.33|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2015-08-05 14:40:59 ERROR 404: Not Found.

converted 'http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz' (ANSI_X3.4-1968) -> 'http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz' (UTF-8)
--2015-08-05 14:40:59--  http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz
Resolving mirrors.xbmc.org (mirrors.xbmc.org)... 89.16.176.16
Connecting to mirrors.xbmc.org (mirrors.xbmc.org)|89.16.176.16|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2015-08-05 14:40:59 ERROR 404: Not Found.

converted 'http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz' (ANSI_X3.4-1968) -> 'http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz' (UTF-8)
--2015-08-05 14:40:59--  http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz
Resolving sources.openelec.tv (sources.openelec.tv)... 82.220.2.33
Connecting to sources.openelec.tv (sources.openelec.tv)|82.220.2.33|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2015-08-05 14:40:59 ERROR 404: Not Found.

converted 'http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz' (ANSI_X3.4-1968) -> 'http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz' (UTF-8)
--2015-08-05 14:40:59--  http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz
Resolving mirrors.xbmc.org (mirrors.xbmc.org)... 89.16.176.16
Connecting to mirrors.xbmc.org (mirrors.xbmc.org)|89.16.176.16|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2015-08-05 14:41:00 ERROR 404: Not Found.

converted 'http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz' (ANSI_X3.4-1968) -> 'http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz' (UTF-8)
--2015-08-05 14:41:00--  http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz
Resolving sources.openelec.tv (sources.openelec.tv)... 82.220.2.33
Connecting to sources.openelec.tv (sources.openelec.tv)|82.220.2.33|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2015-08-05 14:41:00 ERROR 404: Not Found.

converted 'http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz' (ANSI_X3.4-1968) -> 'http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz' (UTF-8)
--2015-08-05 14:41:00--  http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz
Resolving mirrors.xbmc.org (mirrors.xbmc.org)... 89.16.176.16
Connecting to mirrors.xbmc.org (mirrors.xbmc.org)|89.16.176.16|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2015-08-05 14:41:00 ERROR 404: Not Found.

converted 'http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz' (ANSI_X3.4-1968) -> 'http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz' (UTF-8)
--2015-08-05 14:41:00--  http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz
Resolving sources.openelec.tv (sources.openelec.tv)... 82.220.2.33
Connecting to sources.openelec.tv (sources.openelec.tv)|82.220.2.33|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2015-08-05 14:41:00 ERROR 404: Not Found.

converted 'http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz' (ANSI_X3.4-1968) -> 'http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz' (UTF-8)
--2015-08-05 14:41:00--  http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz
Resolving mirrors.xbmc.org (mirrors.xbmc.org)... 89.16.176.16
Connecting to mirrors.xbmc.org (mirrors.xbmc.org)|89.16.176.16|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2015-08-05 14:41:00 ERROR 404: Not Found.

converted 'http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz' (ANSI_X3.4-1968) -> 'http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz' (UTF-8)
--2015-08-05 14:41:00--  http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz
Resolving sources.openelec.tv (sources.openelec.tv)... 82.220.2.33
Connecting to sources.openelec.tv (sources.openelec.tv)|82.220.2.33|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2015-08-05 14:41:01 ERROR 404: Not Found.

converted 'http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz' (ANSI_X3.4-1968) -> 'http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz' (UTF-8)
--2015-08-05 14:41:01--  http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz
Resolving mirrors.xbmc.org (mirrors.xbmc.org)... 89.16.176.16
Connecting to mirrors.xbmc.org (mirrors.xbmc.org)|89.16.176.16|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2015-08-05 14:41:01 ERROR 404: Not Found.

converted 'http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz' (ANSI_X3.4-1968) -> 'http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz' (UTF-8)
--2015-08-05 14:41:01--  http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz
Resolving sources.openelec.tv (sources.openelec.tv)... 82.220.2.33
Connecting to sources.openelec.tv (sources.openelec.tv)|82.220.2.33|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2015-08-05 14:41:02 ERROR 404: Not Found.

converted 'http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz' (ANSI_X3.4-1968) -> 'http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz' (UTF-8)
--2015-08-05 14:41:02--  http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz
Resolving mirrors.xbmc.org (mirrors.xbmc.org)... 89.16.176.16
Connecting to mirrors.xbmc.org (mirrors.xbmc.org)|89.16.176.16|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2015-08-05 14:41:02 ERROR 404: Not Found.

converted 'http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz' (ANSI_X3.4-1968) -> 'http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz' (UTF-8)
--2015-08-05 14:41:02--  http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz
Resolving sources.openelec.tv (sources.openelec.tv)... 82.220.2.33
Connecting to sources.openelec.tv (sources.openelec.tv)|82.220.2.33|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2015-08-05 14:41:02 ERROR 404: Not Found.

converted 'http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz' (ANSI_X3.4-1968) -> 'http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz' (UTF-8)
--2015-08-05 14:41:03--  http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz
Resolving mirrors.xbmc.org (mirrors.xbmc.org)... 89.16.176.16
Connecting to mirrors.xbmc.org (mirrors.xbmc.org)|89.16.176.16|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2015-08-05 14:41:03 ERROR 404: Not Found.

converted 'http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz' (ANSI_X3.4-1968) -> 'http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz' (UTF-8)
--2015-08-05 14:41:03--  http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz
Resolving sources.openelec.tv (sources.openelec.tv)... 82.220.2.33
Connecting to sources.openelec.tv (sources.openelec.tv)|82.220.2.33|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2015-08-05 14:41:03 ERROR 404: Not Found.

converted 'http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz' (ANSI_X3.4-1968) -> 'http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz' (UTF-8)
--2015-08-05 14:41:03--  http://mirrors.xbmc.org/build-deps/sources/libcec-3.0.1-6.tar.gz
Resolving mirrors.xbmc.org (mirrors.xbmc.org)... 89.16.176.16
Connecting to mirrors.xbmc.org (mirrors.xbmc.org)|89.16.176.16|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2015-08-05 14:41:03 ERROR 404: Not Found.

converted 'http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz' (ANSI_X3.4-1968) -> 'http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz' (UTF-8)
--2015-08-05 14:41:03--  http://sources.openelec.tv/mirror/libcec/libcec-3.0.1-6.tar.gz
Resolving sources.openelec.tv (sources.openelec.tv)... 82.220.2.33
Connecting to sources.openelec.tv (sources.openelec.tv)|82.220.2.33|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2015-08-05 14:41:04 ERROR 404: Not Found.


Didn't you mean an opposite? If there is no `endpointConfig.payload`, return `endpointConfig` as is, otherwise wrap?


Could you tell what OS and browser you where using when errors had appeared ?


Fixed some issues and rerun the CI.


this is wrong, since this function is used a `transformAndConcat` call, you should make sure no null is returned, I'm not sure how `transformAndConcat` behave


Type-o here should be "A re-build is required because a weaver has changed."


Wrong code. You cant deal with Win64 PE executable in Win32 way. And didn't compiler told you that 0x1415FF790 and 350 are too big for DWORD and char?


@petter-fidesmo  as per your suggestion.


Why function is not enough?


When using `Hash(@options)` construct, you get an exception if `@options` cannot be converted into hash. With `try_convert` you get `nil`. So at some place you risk to get an exception when operating with that unexpected `nil` object. It might be easy to debug but also it might be not depending on code path. Still it will always be easier if exception is

So to retain early failure to convert to `Hash`, I'd suggest using `@options.to_hash` to replace `Hash.try_convert(@options)`. Also `user_options.to_hash` instead of `Hash.try_convert(user_options)`, etc.

The ruby docs read that `#try_convert` uses `#to_hash`. So I think that's the best option.


Good idea.


:green_heart: Good work creating the `env` directory.


You need a space between the function name and the parentheses.


NPE when Component arg is null is not tested


Interfaces always include only public functions. Remove the ones that became protected.


For example, there is a character card, each character 16x16, the base address of the texture is slided to the upper left corner when printing the text, while TW/TH is set to 10, which results in a 4MB texture in the cache for each.


Yep! It was fixed: https://github.com/JetBrains/imperative-nodejs/blob/master/playground/todo/http-api/index.js#L92


Can we use pure Ruby here?
```rb
pid = File.read(pidfile).to_i
Process.kill("QUIT", pid)
FileUtils.rm_f("#{cache_path}#{db_filename}")
```


The manual states that:

"If the program priority is medium high when the time interval expires or if an attempt is made to set the priority to medium high when it is not allowed, the PRI field is set to medium."

Perhaps you could rephrase that.


In your earliest research, you discovered the "jsMql" attribute in PIR. this attribute would be my prime example, but there might be others. This was back in June as part of BACKLOG-3984.


Ok, as discussed during lunch it is probably OK for now to remove it..


:red_circle: This is *not* a `mongodb` URL! A `mongodb` URL looks like this: `'mongodb://localhost/hairven-db-staging'`.


This file download from http://www.sqlite.org/download.html


:yellow_heart: You do not need the filetype here. It should just be `./config`.


I don't understand the purpose of this method. Some Javadoc would certainly help


Yes. Saves keeping a bunch of temp files around. You can then move the zip file to a downloads directory and then delete the temp directory.

You might also look at doing a cleanup job -- say once a week -- to remove any of the old files that have been created. I did one for another project; the code is here:

https://github.com/music-encoding/customeization/blob/master/task.py#L68


missing assertion that `fileAttributes` is `null`


Because server is configureated at a build time.

It's possible to configure this at server startup, but this requires custom server logic and templating. With current approach, the app is just a buch of static files and can be served by any server, i.e `python -m http.server`


Well, that does not set CMP0054 to NEW. See http://www.cmake.org/cmake/help/v3.3/policy/CMP0054.html

> This policy was introduced in CMake version 3.1. CMake version 3.3.0 warns when the policy is not set and uses `OLD` behavior. Use the `cmake_policy()` command to set it to `OLD` or `NEW` explicitly.

The best is not to set the policy, and never write any code with a semantic that depends on the `OLD` behavior of `CMP0054`. That's what you did.


:green_heart: Good work creating the application structure.


As Baldurs gate is constantly firing loads of small textures at the cache, presumably there will be many repeat ones, I'm wondering if it does a similar method which is tricking the texture cache and causing the massive memory usage.  It would be interesting if it helps that.


@RV-Adrian yes, feel free to change .btn-white, it is not being used outside of store.


you should test all types by FILE, use `DataProvider`


Hmm :) Well at least we have a job doing auto-checkin of the live server: https://github.com/SUSE-Cloud/automation/commits/jenkins


I had a problem with `IJulia` not receiving Julia kernel messages, after a `git bisect`, this commit was the first bad commit... Issue JuliaLang/IJulia.jl#331


If we had a job automatically uploading config from git `master` to the server, then that would effectively enforce that policy...


you should also test when `setInputMetrics` is not called at all


Looks good!


you should also test when `setOutputMetrics` is called with no argument


@Robin-Svenhard we needed to remove master branch from the repository, could you please re-PR this fix to 0.0.3-release branch?

Thanks!


so multiple extruder printing is now fully functional?


calling set methods with only one argument is never tested


Error at PDFBoxIntegration
final PDRectangle character = fontObj.getCharacterBoundingBox(string, i, codeLength);
getCharacterBoundingBox() method is not in PDFont class.


good catch, thanks!


:red_circle: The URL for the production database should be obtained from the environment variable `process.env.MONGOLAB_URI`.


Does the given item, have a constraint set up for the property with the given value.


you can put these arguments in a constant and use them in both set method and containsOnly


:red_circle: The URL for the staging database should be obtained from the environment variable `process.env.MONGOLAB_URI`.


putting null values in the string vararg arguments is not tested


Fixes bug #474290.


i'd inset a _ between SAVE and EXTRACT...


similar here.


@sloriot: You [introduced the test that is being removed](1787a34308b964bfbaa3b34d7164d9124237d868). Meanwhile, the number of test has decreased a lot (divided by two), thanks to that merge:

commit 9f32b687b9e98b727467613f712514118248f3f5
Merge: 5b78e7f 8720500
Author: Philipp Möller <Philipp.Moeller@geometryfactory.com>
Date:   Thu Jun 14 09:13:24 2012 +0000

>    Merge the branch Installation-replace_cfg_macros-pmoeller into next

What do you think of the removal of the test?


hehe


`@CheckForNull` should also be on the class method


Do we need to notify for this? If so. just add `scoped_debug!("new connection accepted: {:?}", conn)` before the insert.


Could you please fix the second link in the paragraph as well?


Wait, why is [bcrypt](https://npmjs.org/package/bcrypt) an optional dependency? Aren't you using it?


Just noticed that you already removed these sentences. Please, disregard my previou comment. :)


Sort of, however, what it means is; The initial call of `computeProperties()`/`refershRendering()` are for letting widget update its DOM with widget’s initial state. Given the nature of “initializing”, widget typically should go through all properties associated with its DOM. That’s why it’s called with the widget instance as the argument.


`/usr/lib/libGL.so` is now missing, since that commit. I do not know exactly why the dependency was pulled, and why it is no longer pulled.

The package `mesa-libGL-devel.i686` should be installed. Should I fill an issue?


Acabo de fazer o primeiro comentário no commit.


An other idea. Create some textures view (layer) of the GS mem based on the format. Each format has its dedicated memory, this way you just need an offset instead of a texture duplication. The difficult part is to keep everyone in sync with some valid/dirty bits. (well there are probably others issues). However it might require to do the index lookup in the pixels part. Dunno.


drop this?


Looks fine.


logic of the test could just be to all computers, in order, of the MeasureComputerHolder, how this call is made could be delegated to another class: SOC
it's might be easier to test those to logics separatly


Thank you Andreas.
What do you think about using an expression on QGIS side instead? That could be easier translated.


Looks fine, but needs man page...


I think we should use hobject_t::get_max() here, or else this sorts *before* any objects iwth hash 0xffffffff


I would do the following to avoid a '//' in the beginning of the path

```
$uri = "/" . $uri unless $url =~ m#^/#;
$uri = $self->protocol."://".$self->host.":".$self->port.$uri;
```


This block should be placed after the "solo mining" test... else you submit multiples nonces on the wallet which cannot work... only one solution accepted per block ;) not a big deal, should not happen until the coin death


Good idea.  Done.


That functionality is all in this function though... It creates a uuid-named folder in media-root/temp/, then uploads files into this folder and/or unzips files into this folder. Then once all the files have been renamed and attached to the piece, it deletes the tree in media-root/temp/uuuid-name


how about just returning hobject_t() ?  Or, we could set the pool_id... I don't think it matters either way except that the operator<< will say MIN for hobject_t().


What's the preicese rationale for moving this?


I'm using a code,

There was a problem parsing  the package. Error during downloading update


Ah, you mean we should use the current tab instead of opening a new one, so the toolbox is present?


It's true, but `#define`ing a constant is not inherently different in terms of reducing the global surface area.


@GianlucaGuarini oh, got it. Thanks a lot.😄


それだめじゃん


Or - actually - the functionality is in the calling function. upload_files() is only ever called by handle_attachments(), which deletes all the temp files after they've been attached to things.


Concept makes sense, but we should port "import-tar" over to this, no?


I suspect the problem is that the copr repositories do not set the dependecies of libQGLViewer correctly. Please file an issue or fix the dependencies in copr.


copy-paste error here


I thought about it as well, but the manhole label is a lot more complex than the label for channels/reaches.

There are multiple 1:n relations (covers, incoming, outgoing pipes) and pipes have to be ordered by function and direction. I don't know how this can be solved with QGIS expressions and still perform well.


And what would be the dimension? For the shaders you need to match the normalized coordinates with TW/TH.


why did you remove the is_fee parameter?
this broke the fee computation; the fee is no longer updated when more inputs are added to the tx


Thanks chap!


The scan was canceled by the user admin admin on 05/08/2015 16:06:34
Included commits

Commit ID	Owner

f4545b47b9f24835acdb9ec38420d31bea849110	@dortal

abb4c31a1e965ba5fa14857ae8935a2269a7a0b6	@dortal

565af35d4610101d7f0d7a9df6153063c12d0d3f	@dortal

bb37aa4384e3495bddd16447868529612cc14d22	@dortal

0467f9dc03aa49abf3742cdea0e547b1c9cf8179	@dortal


Would it be more python-y to make handle_attachments() a class and then have the upload_files() and unzip_files() and such be class functions?


Thanks @technickle


should this be 0 so we chown to root (no-op), or is the idea that chown to -1 will fail silently?


I have no idea why I did this commit. I suspect I did it while discussing with one of you...


@NaikSoftware поосторожнее с переименованием ключей, сломал мне две заметки этим коммитом :smiley_cat:


Meanwhile we can further reduce the number of tests by determining compiler versions instead of running tests (which is also much more portable).

We can also have a look over some of the config files and reduce the number further.


Checkmarx scan results 2015-08-05T13:09:41Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

2df19aa46451ad26f4dc72164d23cf438903565b	@cxehud

ad58b128e2fede39820ae668970bbfab38b56195	@cxehud

6b05867d4fe80a4c3b55cb36e0b6f76abc0a59f9	@cxehud

accf7e51faac6fffc26f426e5afeeaf3ed561731	@cxehud

7917c1e1165bbb3f8b67a7c2943b68d3780e2a79	@cxehud

74262a0e47571dc6ed4aa4c3a03f759fee5bd42c	@cxehud

5b96953244ae55ea53f4983dc1a9fa4d4eb872a6	@cxehud

2166d1ea7164e801468b1b1f20ff0a3ecf4c67dd	@cxehud


I see...
Can we add pure data without translation and human readable text to the database and do the translation at runtime? I am afraid that this function will be very hard to maintain for future translation.


Sorry, I forgot to put that in the commit message.

I use it later in import-dkr because it's exactly what I wanted to do, but previously it was only used in systemd-pull and I needed to use it in systemd-import, and both systemd-pull and systemd-import use import-common, but systemd-import doesn't use pull-common.


the paste is removed


@LorenzMeier checking for the existence of ```/dev/pwm_output1``` looks like it will always fail since ```/dev/pwm_output1``` doesn't exist until ```fmu mode_pwm``` is executed.

Perhaps this check can be done further down before the mixer loads or just do with using the ```SYS_USE_IO``` parameter?


Riight, I'm receiving quite a few notifications that the build instructions don't work, I'll see if i can find some time to review them one of these days.

In this particular case, @mlsankar , it seems that you are building against pdfbox without my patch applied. Confirm that you checked out the mentioned revision, that it applies cleanly, and that mvn install worked before you try to build pdfextract.


extract method isReturnTypeOrAnyArgumentTypeCalculated


I'm not sure, what's the reify equivalent to `proxy-super`?


Deze drie lijnen mag je weg doen, die worden ook beschreven door de `resources :learning_materials`


Dus voor de duidelijkheid: de drie `get /learning_materials/...` lijntjes :)


saw approval from jvoegele
at https://github.com/basho/yokozuna/commit/095c125fee03b6c523be959e1a62bf23189e43a6


merging basho/yokozuna/handling/zl/better-handle-of-wm-errors-yz_wm_search = 095c125f into borshop-integration-532-handling/zl/better-handle-of-wm-errors-yz_wm_search


basho/yokozuna/handling/zl/better-handle-of-wm-errors-yz_wm_search = 095c125f merged ok, testing candidate = 9c3bfba5


Un mot en trop.


We use the test suite to find bugs. The most severe bug is that there are two python txt2tags implementations in the repo.

If you want to test txt2tagslite you can change the lib.TXT2TAGS variable locally. Everybody else should only have to worry about fixing **one** implementation.


I guess you should check for cpy here, not value.


We will need to have a local storage of random names to pull from rather than a webservice, but for now this will do.


`DurationBuilder` should be just `Duration`


Why the version is `dev`?


Shit.  You're right.  I guess the way you have it is as good as we can get for now then.


Let's imagine you have a GsMemView [PSM] [4MB]. A big atlas texture that contains all the real textures.

You just need the parameter (offset, TW, TH... AKA TEX0) to compute the texel location. Hum not sure of the impact of TBW.
real_texture_data = &GsMemView[current_psm] [ current_offset];

Hum, just remember that you have texture overlapping in the HW TC and it is potentially the same in the SW TC. So it is maybe not a good idea.


The scan was canceled by the user admin admin on 05/08/2015 16:15:53
Included commits

Commit ID	Owner

f4545b47b9f24835acdb9ec38420d31bea849110	@dortal

abb4c31a1e965ba5fa14857ae8935a2269a7a0b6	@dortal

565af35d4610101d7f0d7a9df6153063c12d0d3f	@dortal

bb37aa4384e3495bddd16447868529612cc14d22	@dortal

0467f9dc03aa49abf3742cdea0e547b1c9cf8179	@dortal


Just as far as the code goes, can we call this `FreeEventTickets` rather than `Events`? I realise reconfiguring Zuora is probably too much effort, just thinking about the Scala code.


It needs to be like this because the whole object needs to be in a function, so it can be evaluated each time it is printed.


Because load isn't a loop menu


TBW is tricky, it can be different than the real width and then the next row of pages can wrap to the end.


Let's discuss this at the meeting on Monday. I am open for other solutions - maybe just translate the characters with a QGIS expression and always store the english version in the database.


Sounds OK.


If the width is too small (because the rest of the table is too big), you may end up with a character.


I do have a 'build bot' created for building Win32 binaries.  It runs on my box, queries git and builds, tests, archives binaries and sftps the resultant files.     It's in perl, and pretty simple.  I may look at doing a build bot on one of our sparc servers, even if it only builds once a day from a cron task.  This would give us a BE/NON-ALIGNED build machine.

```Perl
#!/usr/bin/perl

# this is my auto-compile/package tool for making Windows builds.
# it builds 4 files.
#   1 A 'base' package. This is DLL's, chr files, conf files, etc
#   2 a 'SSE2' package. This is just exe's for SSE2 build
#   3 a 'XOP' package.  Same exe's but for XOP
#   4 a 'AVX2' package. Again, exe's but for AVX2

my $v="";
my $first=1;
for (;;) {
	$new_dat = `git fetch --dry-run 2>&1`;
	if (length($new_dat) > 0 || $first) {
		$first = 0;
		$new_dat = "";
		chdir ('src');
		my $jnk = `git pull`;
		`make -s distclean`;
		$v = `git rev-parse --short HEAD`;
		chomp $v;
		my $dt = `date`;
		print ("Starting run on version $v at: $dt");
		make_base_pkg();
		my $s = `CPPFLAGS=\"-mno-ssse3\" ./configure --disable-opencl 2>&1 && make -sj3 2>&1 && ../run/john -test=0 -form=cpu 2>&1 && ../run/john -test=0 -form=dynamic-all 2>&1`;
		my $ret = $?;
		if ($ret != 0) { print "failure running configure/build/test for SSE2 version\n"; sleep(120); next; }
		make_exe_pkg('sse2');
		$s = `./configure --disable-opencl 2>&1 && make -s clean && make -sj3 2>&1 && ../run/john -test=0 -form=cpu 2>&1 && ../run/john -test=0 -form=dynamic-all 2>&1`;
		$ret = $?;
		if ($ret != 0) { print "failure running configure/build/test for XOP version\n\n$s"; sleep(120); next; }
		make_exe_pkg('xop');

		#$s = `CPPFLAGS=\"-mavx2 -DHAVE_AVX2\" ./configure --disable-opencl 2>&1 && make -s clean && make -sj3 2>&1 && ../run/john -test=0 -form=cpu 2>&1 && ../run/john -test=0 -form=dynamic-all 2>&1`;
		#$ret = $?;
		#if ($ret != 0) { print "failure running configure/build/test for AVX2 version\n"; sleep(20); next; }
		#make_exe_pkg('avx2');

		`make -s distclean`;
		chdir('..');
		$dt = `date`;
		print ("Completed run on version $v at: $dt");
	} else {
		my $dt = `date`;
		#print ("waiting, time is: $dt");
	}
	sleep(120);
}

sub make_exe_pkg {
	chdir('../run');
	my $type=$_[0];
	my $zip = "JtR-${type}-${v}.zip";
	`strip *.exe`;
	chdir('..');
	`zip -m $zip run/*.exe`;
	`./sftp_file $zip`;
	chdir ('src');
	print("file $zip has been created\n");
}
sub make_base_pkg {
	unless (-e '../JtR-win64-base.zip') {
		chdir ('../run');
		mkdir ('x');
		mkdir ('x/run');
		mkdir ('x/doc');
		`cp -p /bin/cygbz2-1.dll /bin/cygcrypt-0.dll /bin/cygcrypto-1.0.0.dll /bin/cygwin1.dll /bin/cyggmp-10.dll /bin/cygssl-1.0.0.dll /bin/cygz.dll /bin/cyggcc_s-seh-1.dll /bin/cyggomp-1.dll * x/run 2>/dev/null`;
		`cp -p ../doc/* x/doc`;
		`cp -p ../READ* x`;
		chdir ('x');
		`zip -mr ../../JtR-win64-base.zip *`;
		chdir ('../..');
		`./sftp_file JtR-win64-base.zip`;
		print("base package has been created\n");
		chdir ('src');
	}
}
```


The scan was canceled by the user admin admin on 05/08/2015 16:19:04
Included commits

Commit ID	Owner

f4545b47b9f24835acdb9ec38420d31bea849110	@dortal

abb4c31a1e965ba5fa14857ae8935a2269a7a0b6	@dortal

565af35d4610101d7f0d7a9df6153063c12d0d3f	@dortal

bb37aa4384e3495bddd16447868529612cc14d22	@dortal

0467f9dc03aa49abf3742cdea0e547b1c9cf8179	@dortal


Hmm, the explicit *p = NULL sounds unnecessary, no? It's the first thing you do on the finish label after all?


Line is too long. [140/100]


Yes, exactly.


Great! Will you tag a new release? Currently, `Pkg.add("JuMP")` doesn't give access to this feature.


Making it

Sorry for this, but I don't even remember from where I picked it


Looks good.


The common casing for API set names is either all lower case or all upper case. Could you please update it?


Why don't you use FlatSpec or FunSuite test styles? IMO they look much better than "java" style tests


Fixes #2


see 959620d


const & ?


You can either create this Changesets in sql, xml or yaml.


Weird, it's still there for me. Anyway, pasted it here.
<pre>
    21:51:53.701 TypeError: window is null
    Stack trace:
    @resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/content/worker.js:127:7
    dispatch@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/method/core.js:119:12
    initialize@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/content/worker.js:61:7
    constructor@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/core/heritage.js:146:23
    Worker@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/tabs/worker.js:11:16
    Tab<.attach@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/tabs/tab-firefox.js:197:12
    openTab/<.onReady@chrome://devtools-prototyper/content/backend/code.js:34:22
    emitOnObject@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/core.js:112:9
    emit@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/core.js:89:38
    tabEmit@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/tabs/tab-firefox.js:235:25
    tabEventListener@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/tabs/tab-firefox.js:289:23
    @resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/tabs/tab-firefox.js:308:39
    emitOnObject@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/core.js:112:9
    emit@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/core.js:89:38
    emitOnObject@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/core.js:112:9
    emit@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/core.js:89:38
    emitOnObject@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/core.js:123:45
    emit@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/event/core.js:89:38
    frameMessageReceived@resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/remote/parent.js:152:32
    1 core.js:106
</pre>


The scan was canceled by the user admin admin on 05/08/2015 16:21:52
Included commits

Commit ID	Owner

f4545b47b9f24835acdb9ec38420d31bea849110	@dortal

abb4c31a1e965ba5fa14857ae8935a2269a7a0b6	@dortal

565af35d4610101d7f0d7a9df6153063c12d0d3f	@dortal

bb37aa4384e3495bddd16447868529612cc14d22	@dortal

0467f9dc03aa49abf3742cdea0e547b1c9cf8179	@dortal


Тут везде нужны ссылки:
type ReportWriter interface {
	Start() (*models.Report, error)
	Finish(*models.Report) error
	Commit(*models.Report, []*models.Problem) error
}


@ksooo - good point - I hadn't considered server and client being in different timezones.
Unfortunately it still isn't quite right (part of the problem may be that Hamburg is currently on UTC time ;-)

This solution works in the UK (currently UTC-1): https://github.com/metaron-uk/pvr.hts/commit/8a381dda55d43adfae69a257f9b77a1effce40ca

Note that the message that TV Headend returns minutes west of UTC/GMT seems incorrect. My server returns *hours* west of UTC/GMT...


Yeah, from all the reading I did it seems this is the one use-case where proxy is still a suitable correct choice over reify, and explains the odd mix-and-match in styles over in the 'donor' codebase.


можно как-нибудь эту переменную и покрасноречивее назвать))


Thanks, I always forget the ins and outs of proxy vs reify because I don't use them that often.  Sorry for the confusion.


Ah, excellent!!


У этого репорта не должно быть Finished, он же Unfinished


Notification names always need to be constants and declared in the header, see my previous comment:
https://github.com/wikimedia/wikipedia-ios/commit/ec2a4a191a7befea197b6a2383be43a796d33004#commitcomment-12500597


shouldn't this method be called addMeasure_long_create_measure_of_type_long_with_right_value or something of the like? (same comment for other methods named in the same fashion)


@nsarno plase make sure thath the icon will be visible with all the theme styles (darker,default and lighter). You can use one of the color used for others icons, flat and monocolor.


all tests pass:
success: http://buildbot.bos1/builders/test-yokozuna/builds/1203


so i discovered the cmake build system isn't available in the version of protobuf I'm using. anyways, the autogen.sh stuff is for linux/osx, and compiles protobuf just fine on osx

for windows, i need to use the visual studio project files here: https://github.com/google/protobuf/tree/bba8365/vsprojects

the cmake for this is straight-forward I think

```
include_external_msproject(libprotobuf vsprojects/libprotobuf.vcproj)
include_external_msproject(libprotoc vsprojects/libprotoc.vcproj)
include_external_msproject(protoc vsprojects/protoc.vcproj)
```

but this doesn't work; it takes 0 seconds to build with the line `Built target protobuf`, and then gives me this error message: http://pastebin.com/1cZF0UJX

is building the VS project files with cmake the right approach? do you know of any other projects that use include_external_msproject()? i'm running out of ideas :(


Почему Problems: Problems? Тебе нужен пустой список проблем, и он у нас уже создан.


Statistics: StatisticS


While you are on it, could you please also update the commit message (ms and win swapped, first line is too long to make it a good title, ...).


You missed one `data-ng-if="isKiezkasse"` in the first meta item. Unfortunately this means that if we stick to the plan the first item will be empty (which would look strange). Maybe we can convice the designers that comment count should always  be the first item?


Good work Jana. Thanks.


Statistics: BadStatisticS. И у тебя тотал не равен сумме остальных каунтов. То же касается и статистики ниже.


it is certainly the right approach, but i do not have any experience with it. @montellese give us a hand ?


It's really annoying that there is both *input* and *inputs* (and the latter can't just take a single node).


Excuse me can you tell me how does the recognition happen?
Because I've been trying to run this app multiple times on different datasets and each time it puts most of images or all of them in "Unknown section" to be tagged and nothing is left to be recognized when I click guess!
Am I running it correct ? or Is there any hint to run it properly ?

Thank you in advance ..


you could use `assertThat(underTest.getChildrenMeasures(NCLOC_KEY)).extracting("intValue").containsOnly(10)` I believe


you could use `assertThat(underTest.getChildrenMeasures(NCLOC_KEY)).extracting("intValue").containsOnly(10, 12)` I believe


thanks for quick merge and release!


Why is OC_TEAM_IDENTIFIER_PREFIX renamed to OC_SOCKETAPI_TEAM_IDENTIFIER_PREFIX?


@Bartuz , for performance reasons, i think Kernel methods not needed here. But I agree with you, it looks, that without ```BasicObject``` everything will works fine too.


it could be usefull to display something for the implementation in toString, even if it is just the `getClass().getName()`


Thanks for looking into this. What is not clear to me whether you are saying you consider your modification of my code scribble 100% working or do you think it still has issues?


On CentOS 7 this evaluates to false, which means the "else" block on line 93 gets executed instead, causing the module to try to install a package called "puppetdb-termini", which doesn't exist.... and there are no puppetdb-3.x packages available in the puppetlabs stable yum repos. The readme suggests the "correct way" to deal with this is to pin $::puppetdb::globals::version to 2.x, but this isn't great either because of https://tickets.puppetlabs.com/browse/PUP-1519 and https://projects.puppetlabs.com/issues/12179.


Checkmarx scan results 2015-08-05T13:20:32Z

High
Medium
Low
Info

New
0
33
22
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

f4545b47b9f24835acdb9ec38420d31bea849110	@dortal

abb4c31a1e965ba5fa14857ae8935a2269a7a0b6	@dortal

565af35d4610101d7f0d7a9df6153063c12d0d3f	@dortal

bb37aa4384e3495bddd16447868529612cc14d22	@dortal

0467f9dc03aa49abf3742cdea0e547b1c9cf8179	@dortal


double slashes are still valid though so I would leave it as is personnaly


Here's how it could be the result with hypothesis installation on site: [Peeragogy hypothesy](https://via.hypothes.is/http://peeragogy.github.io), and I also recommend to work also on jkelly theme design and stylesheet.

1.*Translations*: I do not think they are actually feasible in ongoing projects like this. Many parts are written in "English for English" and for many sections, properly adapted translation require a complete revision and linguistic adaptation. A literal translation of **Case study: 5PH1NX** for example, it really make no sense for an Italian reader. Also, the amount of handbook contents in order to be translated it require no less of < 350 volunteer working hours. Considers that in Italy a professional translator asks € 27h for translation and proofreading. By now the automatic translations section on dashboard even if is not a perfect solution results for me the simplest and practicable way.

2. *Roadmap, Next steps, Carrying capacity... *: Are all related topics, but it is missing the most important part for my point of view: "Community Identity" and more important "the why". Peeragogy in Italy is perceived just as a nice Howard Rheingold publishing project. Is really just this the "why" of peeragogy?

3.*Wrapper*:  We should try to find a wrapper... It  would be really helpful for all the community. Maybe we should think about making a public call and support & empower members (or a group) to try this role in a project.

4.*Newcomer*: We should write a handbook dedicated only to this subject.  Newcomers need time to get adjusted, understand principles and set-up the peeragoical system. Unfortunately, it is precisely at this stage that most of them drop out from the project. Can we do something about it?

***Developing a really accessible DIY tool-kit to make it easy for people to get started right away***
I think is one of the most important milestones to be achieved all together, and the dashboard development it could be just a part of this challenge.


ok.


Agreed with Brian on his comment… this should likely be a delegate or target action.

I think this is ok for now as well, because we haven't decided where the edit pencil will go, so no need to spruce this up just yet.

To elaborate i the reasoning…
When considering communication patterns, you should always ask "how many objects need to know about this event?", and if the answer is "1", that is a strong hint that it should not be a notification.

Another piece of the puzzle is the fact that this is a "view". Views are "singular" objects on screen and are generally managed by a single view controller. It is unlikely that multiple view controllers or views need to be informed when something happens in this view.

Notifications are most useful in the model layer - when a model object changes - where multiple views on screen could be representing that single model.

It's also useful to look at out best practices for communications strategies:
https://www.mediawiki.org/wiki/Wikimedia_Apps/Team/iOS/BestPractices#View_-.3E_View_Controller


Давай везде множественное число в имени репорта)


Checkmarx scan results 2015-08-05T13:19:41Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

2df19aa46451ad26f4dc72164d23cf438903565b	@cxehud

ad58b128e2fede39820ae668970bbfab38b56195	@cxehud

6b05867d4fe80a4c3b55cb36e0b6f76abc0a59f9	@cxehud

accf7e51faac6fffc26f426e5afeeaf3ed561731	@cxehud

7917c1e1165bbb3f8b67a7c2943b68d3780e2a79	@cxehud

74262a0e47571dc6ed4aa4c3a03f759fee5bd42c	@cxehud

5b96953244ae55ea53f4983dc1a9fa4d4eb872a6	@cxehud

2166d1ea7164e801468b1b1f20ff0a3ecf4c67dd	@cxehud


It's only to be able to set it in XCode while testing (since it gets passed as an argument to xcodebuild in the normal build). The variable should have been renamed earlier but I needed it so I just fixed it. To be clean this should be a different commit, you want it or is it fine?


By vertex data I meant vertex positions. These values are directly used for that. Radius could be one of those and very well could be part of style as well. Style to me is material attributes. Scale (radius or any other) are exception for now. If you don't have strong objections, I would prefer we change to this.upperRight(...) etc. Or may be just have one method that takes three parameters.


LGTM.


Use python-requests


i don't understand either -- i was just mirroring Rudi's change! :)


Thanks! After that I'll merge, since @barbieri, who needs it, said he takes responsibility on the rest of the review :)


That was deliberate I put in a ```<li data-ng-if="!isKiezkasse"></li>``` to balance it out


This change should not be in. You really don't want to modify the signature or Irmin.AO.


Private members suggested.


Checkmarx scan results 2015-08-05T13:38:21Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

2df19aa46451ad26f4dc72164d23cf438903565b	@cxehud

ad58b128e2fede39820ae668970bbfab38b56195	@cxehud

6b05867d4fe80a4c3b55cb36e0b6f76abc0a59f9	@cxehud

accf7e51faac6fffc26f426e5afeeaf3ed561731	@cxehud

7917c1e1165bbb3f8b67a7c2943b68d3780e2a79	@cxehud

74262a0e47571dc6ed4aa4c3a03f759fee5bd42c	@cxehud

5b96953244ae55ea53f4983dc1a9fa4d4eb872a6	@cxehud

2166d1ea7164e801468b1b1f20ff0a3ecf4c67dd	@cxehud


I spoke with Diogo about this specifically and he said that we do not want to update the param object on the co training panel, text, or even the current component. The valuesArray is the most important.


promotion -> production


Just don't store the name/version in the opam file!


I'm also confused why the change is necessary at all. Is it just so that the tagged tarball has a matching version in opam file?


love it


It shall be mandatory so:

$PDUPercent=intval($pduDraw/$maxDraw*100);
$PDUPhase1=intval($lastreading->Wattage1/$maxDraw*100);
$PDUPhase2=intval($lastreading->Wattage2/$maxDraw*100);
$PDUPhase3=intval($lastreading->Wattage3/$maxDraw*100);

...otherwise meter-wrap bar is filled completely, irrespective of value of PDUPercent.


need to also test when a child has a NO_VALUE measure


Line 33 of StudentManager.java I have issues with, it calls XMLManager.getXML().getDocument() unsure how to fix


ge zou hier nog /translations kunnen achterzetten. Niet de volledige cache moet gecleared worden om je translations up te daten


serves -> serve


Camel case for class names.


the commit message doesn't explain the change but only the changed file. Could you describe what you change and in the body why you do this plus a full URL to tthe github issue it fixes? (If there's nos uch issue yet, please create one.)


Continuous


I'm open to that.

How would that appear in opam if you have cohttp pinned?


@samoht the version is useful since it acts as a sensible default when pinning the development version. i.e. "opam pin cohttp --dev" just works


Currently, yes.
Sorry about that, this patch series has been through a few rebases, since `unquote_first_word()` has changed a few times since I started working on this.
It slipped through erroneously because later in the series we can end up with s being allocated to an empty string, at which point the later code to set `*p = NULL` isn't triggered.


Definitely working. (Tested as a replacement for my original fix commit in the master3 branch)

I've now created a branch including this, my API3.0.0 changes and your fix-autorec-start-stop-handling changes. Still needs tweaks but looking extremely encouraging :-).


looks good


what about ensuring that two computers sharing input metric is supported?


Checkmarx scan results 2015-08-05T13:32:46Z

High
Medium
Low
Info

New
1
1
2
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

2df19aa46451ad26f4dc72164d23cf438903565b	@cxehud


This document describes the following ways to promote images


Do we really need such a complex source here? We are not testing the exactly once semantics here so it would be fine to just output some elements (making sure the job fails before finishing) and then shut down without checkpointing.


93653360-ResetVarsAtEndOfWarmup (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1262 of 1262 tests passed)

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


Continuous

Are you saying Go is a CI tool?


unit test missing for `MeasureImpl`


[Code style standard](https://www.python.org/dev/peps/pep-0008/)


The value of `updateResult` is just going to be the object you defined above, so I'm not sure there's much value in checking it. I think the real value of this test case is in making sure that the correct functions are called in each case, with the correct parameters.


Please, use 4 spaces instead tab


It was pretty late in the evening that I noticed the bug; I forgot how to brain. I've implemented that method into the next commit.


Just realized that the default case in normal decode_char is not needed since the case cover all cases :-)
But a special test is needed for bad escape sequences. I will fix that anyways.


Nice work with the rapid prototype of hypothesis integration :-)

Let's share that with others in the G+ and see if they like it.  I
definitely do!

About translations: Yes, there's also cultural context (e.g. in Taiwan,
expectations for a book about peeragogy would be different).   Maybe the
principle for us to follow now is "make the best possible really useful
book in English, and only translate it later."   If Paola really wants to
do a Spanish translation soon, maybe that will work out.  But that gets to
the central question:

>  "Community Identity" and more important "the why". Peeragogy in Italy is
perceived just as a nice Howard Rheingold publishing project. Is really
just this the "why" of peeragogy?

That's where I think the Charter will help a lot.  It's where we can all
try our best to express the "why."

> 3.*Wrapper*: We should try to find a wrapper... It would be really
helpful for all the community. Maybe we should think about making a public
call and support & empower members (or a group) to try this role in a
project.

At the moment I think you're doing an amazing job with the Dashboard, and
starting work on "look and feel" for the site.  We can collaborate to
describe other tasks for the wrapper to do, and share them (I'm happy to
help with news items or anything else).

> 4.*Newcomer*: We should write a handbook dedicated only to this subject.
Newcomers need time to get adjusted, understand principles and set-up the
peeragoical system. Unfortunately, it is precisely at this stage that most
of them drop out from the project. Can we do something about it

> *Developing a really accessible DIY tool-kit to make it easy for people
to get started right away *I think is one of the most important milestones
to be achieved all together, and the dashboard development it could be just
a part of this challenge.

Yes, I think many others agree with this as well.  Can we define the steps,
probably working with Paola?


This is not actually a rich function :)


no need to be public


Cool. I'd like to separate the server timezone stuff from the API 3.0.0 changes and will submit a PR for this myself later today. Okay?


I like this much better than my attempt at a fix with sorting.


constants should be at the beginning of the file


@andela-ooduye your commit message doesnt state the type of corrections. That could be misleading. Pls rephrase.


Also maybe we can get rid of the PrefixCount-s in this case as they really don't serve any purpose, as we don't check the counting logic.


why do you have meanmap here?


Using proxy can solve ip issue to some extent.


I don't understand, everything works fine for me.
Can you send a screenshot?


Why do you have a url referencing meanmap?


i would make this a void* that is opaque.. so that the cursor *could* be an hobject_t


as far as your test is concerned, all three `NewMeasureComputerProvider` instances could share the same MeasureComputerImplementation` instance


One example of CSS peeragogy.io possible design with Hypothesis integration:
1. [site1](https://via.hypothes.is/http://www.autodidacts.io/openbci-brain-basics-neurons-structure-and-biology/)


Can be merged with [Line 20](https://github.com/ChenJianhao/crawler/commit/92ed8d0ca621c907dbaa1dc1101ef8094ed6e796#diff-9af5ece624158f7b04cfbaf9eac3e77dR20)


I'm not sure whether we've standardized on which quote characters we should be using, but I'm not convinced these are the right ones.