See my other comment about including private constructors. If we're not including them at all, should we rephrase this to make it clearer that private constructors should *never* be chosen (or add a test that confirms just the "never use private constructors" bit)?

Right now this test reads to me as a confirmation of the primary scope sort (which is good), but also makes me think it is *possible* for TinyIoC to choose a private constructor.

Might just be me reading too much into this though.


Looking back at the old code, we never provided the mount options through getAbsoluteMountPoints()... strange... I'll add it in


Correct, but I think there's more important issues to work on right now. I'm not aware of any issues with the manta audio HAL, so this is probably not too bad. I'm not saying it's not wrong, just that it should be a low priority bug.
Why don't you submit the fix to AOSP so we can see what they say and everybody can profit?


I'll look later today. I think it is an r-builder bug,  I guess it needs
some extra libcurl packages. G
On Aug 17, 2015 8:26 AM, "Jim Hester" <notifications@github.com> wrote:

> @gaborcsardi <https://github.com/gaborcsardi> You have any ideas why this
> is failing on r-devel?
> https://travis-ci.org/gaborcsardi/argufy/jobs/75812930#L225-L228 is the
> relevant travis output. I cannot reproduce the error locally using R-devel
> r69096, I have no clue why it is not able to download from the rstudio CRAN
> mirror.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gaborcsardi/argufy/commit/c5380e33368400aaff03abe4bea27ed0ea531f65#commitcomment-12737016>
> .
>


No need for these if-else blocks any more, you're already testing how many totalVotes there are and updating the text.


String should be in a resources file. Same for line 105.


What was the reason for this change?


Might this be used by BibleTime Mobile?

--Greg

On Mon, Aug 17, 2015 at 7:44 AM, Martin Gruner <notifications@github.com>
wrote:

> This is probably not needed with BibleTime.
>
> Am 17.08.15 um 14:31 schrieb Konstantin Maslyuk:
> > Why we have to supply Basic-Mobile.tmpl with BibleTime?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/bibletime/bibletime/commit/044d5374535944640108db7a6efd9916041357e9#commitcomment-12737062
> >.
>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bibletime/bibletime/commit/044d5374535944640108db7a6efd9916041357e9#commitcomment-12737296>
> .
>


File name.


comment pos 20


Es ist eig. eine Benachrichtigung "Notification" Kann natürlich auch passieren, dass es eine Notification bezüglich einer Einbindung in ein Projekt ist.


hi


There is no attribute `clientData`, it's just `data`.  Please see TR4 of http://docs.ably.io/client-lib-development-guide/features/ to make sure the attributes match exactly.  I think you are missing `encoding`, and `member_id` should be `member_key` and `id` is message ID


comment with token


consider avoiding local variables - enumerable is your friend :-)

```ruby
class CompressedSequence < Struct.new(:numbers)
  def to_s
    compressed.join(' ')
  end

  private

  def compressed
    numbers.inject([]) do |stack, number|
      number_exactly_as_previous?(number, stack) ? stack.last.increment : stack.push(Stats.new(number))
      stack
    end
  end

  def number_exactly_as_previous?(number, stack)
    stack.last && stack.last.number == number
  end

  class Stats
    attr_reader :number

    def initialize(number)
      @number = number
      @counter = 1
    end

    def increment
      @counter += 1
    end

    def to_s
      [@counter, number].join(' ')
    end
  end
end
```


That could be a good idea


ID is message ID, not member ID


`void 0`? Why not just use the `undefined` keyword?


Just had this issue resolved.

````
yum install ntp
chkconfig ntpd on
````

Did an initial sync, with

````
ntpdate pool.ntp.org
````
and started the time daemon with:

````
/etc/init.d/ntpd start
````


can I do it next time, pls, be merciful :)

2015-08-17 15:10 GMT+02:00 Simon Schäfer <notifications@github.com>:

> Sure, I just mean that when you do formatting changes you should put them
> in a separate commit. This makes reviewing easier.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/scala-ide/scala-ide/commit/0d0e3747296eedfd85758b0bb5e52b99af0bc2ac#commitcomment-12737734>
> .
>


This only passes because memberId is not defined


I don't think we do because we don't have the Google Dialer... I would try
to ask @mfonville.

On Mon, Aug 17, 2015, 8:09 AM Nicholas Buse <notifications@github.com>
wrote:

> I went to update this from the TK GApps package, but he doesn't have the
> com.google.android.dialer.support.jar file (nor the
> com.google.android.dialer.support.xml in /etc/permissions/).
>
> Do we have a dependency on this file, or can we remove it?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/opengapps/all/commit/193337b24ba3fc62c56690bcf4878d4fd86336d9#commitcomment-12737721>
> .
>

To unsubscribe click: <http://email.giesing.cf/u/eJwljkFrwkAQhX9NcpTZaTKzc9iDtlTRQim0kJvsTnZiqNYqplZ_fUP6Dg_eO3x8bSCKimUfHueb4cifzWp4vfPyd80fdqXT-_pLb8_3dPpZFrhwT_Wma6RZbV_GtQV-uxYVHGK_n3VT6_FQ7oKYgK9appgTURJDRGLvTcYzUyzP4Zy_97eRAcCeJbJy3RKIOWHyVWJTEoEW0Dw4MohOaktOBdXgP865LJLNC0ZHD5oBFGtNMipN-FnXX3ZDmqQuocD5HwYhRXI>


comment with token 2


it looks like you are deliberately allowing invalid values. Did you make this clear in the product code?


@Blackskyliner the pugx implementation is quite fragile, so it is not totally unexpected that it can be broken by changes in the core. It relies on some of its own code being called before doing any call to providers.
so IMO, this is something which needs fixing in the PUGX bundle.


HELLO WORLD


Sorry, but I'm not "getting" this. What is the difference between calling `#immutable!` directly or using `#__send__`?


Why are we removing this, is it because this hook is never called ? If so, should we also remove the `archive_room_packet` function?


please use plural names only for collections


vote 수를 계산할 때 지금처럼 계산하는 것이 아니라 self.board_content.vote_adult.count()를 사용하면 간단하게 개수를 계산할 수 있습니다.
len() 함수를 사용하게되면 QueryDict type을 list로 변환해야하기 때문에 추가적인 연산이 필요하다고 알고 있습니다.


Minor style thing - I believe in most cases we call this function `setup`.


Eh...?


Any particular reason why we don't do that already?

The query change should be simple - `status__in=[COMPLETION_STATES]`, right? Or am I missing something?


Three last tests are supposed to be valid indentation AFTER
indent-line-function. So haskell-indentation-mode is supposed to make them
valid.


What exactly do you mean 'multi-project support'? Starting multiple mods in single MC instance from IDE can already be done by just putting multiple projects in classpath.


Sorry to bother you guys again but I like to keep reading the firmware code to know what's going on, can someone explain to me what this is used for? Where I looked runtime_info_t type is known and fixed size so why do we need to store it's size? I'm guessing forward compatibility?


I've successfully interacted with a MS SQL Server instance from Haskell via ODBC (using the `HDBC` library). Immature is definitely the right label, but it can be done.


develop (lgu1234) - x86_64-MacOS-10.9-clang: OK (1271 of 1271 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)


Is this desirable?

If possible, we should try to catch a more specific exception to make it clear what kind of error we want to ignore - right now it looks like we simply ignore all the errors (potentially also mask programmer / typo errors hiding underneath).


This shows the wrong commit date due to the incorrect VM date settings. Fixed this using the following:

Just had this issue resolved.

````
yum install ntp
chkconfig ntpd on
````

Did an initial sync, with

````
ntpdate pool.ntp.org
````
and started the time daemon with:

````
/etc/init.d/ntpd start
````


humn... ok, I'll update the pr


Might be good to make additional arguments which are passed to `from_model` (notably `mask_secrets`) a module level constant (dict).


develop (lgu1234) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Third parameter is not aligned. This could be one line.
static CmsTopn * CreateCmsTopn(int32 topnItemCount, float8 epsilon, float8 probability);


Does pass by value with const make sense?


Unnecessary newline here.


Thanks for using HTML::SocialMedia and for spotting the typo which I will fix.  Interestingly your fix also has a typo!


It's pretty minor but I think I would've ordered `what_year?` under both `form_needed_for_1?` and `form_needed_for_2?`, as both of those questions can be asked before `what_year?`.


@dkuppitz i got the integration test back to just 13 requests. i guess times(2) ultimately creates an `IsStep` which is a `FilterStep` so adding the viz side-effect in the strategy after `FilterStep` created some added "decay" to gephi. That didn't seem right - I changed it to add the infrastructure after `HasStep` only for now - think we can make that any better? is it even good with just `HasStep`?


@stulees I think you need to add an `assert.isTrue(responded);` test to these event handler tests so that you can be certain that the event handler actually fired.


HELLO WORLD


That's not a bug report.  GnuPG bugs go here: https://bugs.g10code.com/gnupg/


mono_runtime_class_init was not correctly annotated, good catch.


simplify, `this.model.bind('change:geometry_types', this.refresh, this);`, no need to wrap it in yet another closure


It might be worth describing how you copied these files in the commit note. Particularly if you automated any of it.


use a hyphen


What do you think of also having constructors/methods for which the CodecRegistry is not required (and it uses CodecRegistry.DEFAULT otherwise), i.e.:

```java
public static TupleType of(ProtocolVersion protocolVersion, DataType... types) {
        return of(protocolVersion, CodecRegistry.DEFAULT, types);
}
```


alternative solution

```ruby
module CalculateFibonacci
  def self.fibonacci(number)
    number.between?(0,1) ? number : (fibonacci(number - 1) + fibonacci(number - 2))
  end
end
```


The comment about ignoring whitespace came in really handy :-) I probably would've spent ages trying to understand these changes if you hadn't mentioned that!


It's up to us as a design decision whether or not to include private ones. I'm sort of indifferent, but it sounded like other folks were in favor of just excluding them completely.


develop (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1274 of 1274 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)


when i use wildcards and and/or connectors i have this Error: Invalid argument supplied for foreach()

this is my model user.php

public $hasAndBelongsToMany =

array(
        'Competence' => array(
            'with' => 'CompetencesUser',
            'className' => 'Competence',
            'joinTable' => 'competences_users',
            'foreignKey' => 'user_id',
public $filterArgs = array(

'competences' => array(

            'type' => 'subquery',
            'method' => 'findByCompetences',
            'field' => 'User.id',

         'connectorsAnd' => '+',
         'connectorOr' => ',',
          'before' => true,
           'after' => false)

);
public function findByCompetences($data = array()) {

$this->CompetencesUser->Behaviors->attach('Containable', array( 'autoFields' => false ) );

$this->CompetencesUser->Behaviors->attach('Search.Searchable');
//$data = array('competences' => 'Cake');

$data = array('competences' => 'First%+Second%, Third%');

$result = $this->CompetencesUser->parseCriteria($data);

$expected = array(0 => array('OR' => array(
    array('AND' => array(
        array('Competence.competence LIKE' => '%First\%%'),
        array('Competence.competence LIKE' => '%Second\%%'),
    )),
    array('AND' => array(
        array('Competence.competence LIKE' => '%Third\%%')
    )),
)));

$query = $this->CompetencesUser->getQuery('all', array(

   'conditions' =>array(
    $expected,
    $result),

    'fields' => array(
      'user_id'
   ),
    'contain' => array(
       'Competence'
)));

 return $query;
}
plz any idea ;
in my query
SELECT `User`.`id`, `User`.`username`, `User`.`nom`, `User`.`prenom`, `User`.`email`, `User`.`password`, `User`.`created`, `User`.`modified` FROM `dcsearch`.`users` AS `User` WHERE User.id in (SELECT `CompetencesUser`.`user_id` FROM `dcsearch`.`competences_users` AS `CompetencesUser` LEFT JOIN `dcsearch`.`competences` AS `Competence` ON (`CompetencesUser`.`competence_id` = `Competence`.`id`) WHERE ((((`Competence`.`competence` LIKE '%First\\%%') AND (`Competence`.`competence` LIKE '%Second\\%%'))) OR (`Competence`.`competence` LIKE '%Third\\%%'))) LIMIT 20


I know I picked name, but we don't use "empty" as a verb here. So could refactor function name to CmsTopnEmptySize()


Yah... we could have one that only have private ctors and assert that we throw or something like that; I think that's the best way to explicitly call out that we will never use the private ctors.


If you have never tried it before, try asking me to merge your changes into the original repo by making a pull request (the green icon that says 'Compare, review, make a pull request') :-)


Retiro então?


Why not `int len = 16`?


You can dynamically allocate memory at runtime, by: calling malloc()
directly, creating objects, calling String functions, opening TCP
connections, and numerous other ways.

This call tells you how much memory is left in the pool, the primary use
case is in tests to prove the presence of and subsequent fix for bugs.

It is not likely to be of much use in normal use.

On Mon, Aug 17, 2015 at 8:24 AM, Niklas Schnelle <notifications@github.com>
wrote:

> Sorry to bother you guys again but I like to keep reading the firmware
> code to know what's going on, can someone explain to me what this is used
> for? Where I looked runtime_info_t type is known and fixed size so why do
> we need to store it's size? I'm guessing forward compatibility?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/spark/firmware/commit/0f742b896b2b1f519a3aeb08c8b34abfca6f0cb3#commitcomment-12738000>
> .
>

--
Andy


Which command does it fail?


Hi you should abstract the access to the database via ORM. We use Hibernate for that right now. Have a look at  CMR/src/hibernate - here we have all mapping files. At info.novatec.inspectit.cmr.dao.impl you find classes dealing with accessing the database for a concrete data type, like MethodIdentDaoImpl.saveOrUpdate(MethodIdent).

The DAOs use a Spring class (HibernateTemplate) which allows to easily access the database and have the configuration outside of the code. To see the configuration check src/spring/spring-context-database.xml


Ack! Gerne, dann wäre
`foreach ($notifications as $notification)` IMHO der best-case?


I think it is better not to put this URL path here - it is not used for the other global stats page either (mirrorstats)


@sebix I think your changes are not correct. In this case if bot tries 3 times to start again, the bot will dump the same message 3 times. I think its better if you split your work. 1) improve functionality 2) clean and improve the code


@randall210 i'm not sure what's going on with your comments on this commit. if you need help with the SMS features of the FONA, please visit the Adafruit forums: http://forums.adafruit.com/


We should add Media_State_Flush before BB_END for all platforms.  Gen9 driver missed this.
Applied this patch,  SKL has obviously more stable when decoding.


nope - i'll fix this


@donnapep The test will timeout after 10 seconds if no call to `done()` occurs, so there isn't a need for that assertion in this case.


Well, ok. It is not that the code would get bad because of this.


Sorry should have worded that better. I see the point of the call but not of runtime_info_t.size which as far as I can see is always sizeof(runtime_info_t).


Bitte CamelCase


Oh yep that should be rm'd


feel free to look at how we have solved that at one of the first coder dojo

https://github.com/miksturait/coder-dojo-2015/blob/master/code_eval_180/knight_move.rb


Updated the readme to reflect your pull-up finding.   Thanks!


Ok, that sounds reasonable to me.

Super minor, but should we move these numbers to named constants so it's clear what scope they're associated with should this section ever change?


Totally. That's been on my mind to lock the list.


html语义标签


there are also references in mTeardown; I guess we can just remove the mTeardown function now


Commonwealth English :+1: :smile:


Die github-API-URL hart in einer Methode zu verdrahten ist nicht schön.

Im GitHub-Client ist die Konstante leider protected:
https://github.com/tan-tan-kanarek/github-php-client/blob/16526141c6e9273eb9669022b9ba4f3fea482b11/source/GithubClientBase.php#L9
Da kommen wir nicht ran.

Evtl. lagere die Konstante einfach genauso als Klassen-Eigenschaft in Deinen Kopf aus.


@zabawaba99 Well, does firego have to manage the messages and the medium? imho, there is no need. Look other libraries like [google/go-github](https://github.com/google/go-github/blob/master/github/github.go#L128). Its NewClient function accepts a client, and then configures some parameters. In my case, I need to work in a sandbox environment (like appengine), so i cannot use the default http.Client.


consider decomposing this line (e.g. what is `array.join('').length.to_f`

I just wonder what is the reason behind joining array in to string and counting letters, why not to check the size of array without converting it into string?


After someone clicks a external button conntected to Arduino board (Arduino Yún), it will send a restful command to https://pushover.net/ and then your iPhone/Android will get Push notifiatios.


use m->osd_features.  the connection features are the intersection of the osd's features and the forwarding mon's features.


``idgen-stamp`` ?

http://www.gnu.org/software/automake/manual/html_node/Multiple-Outputs.html


ich würde die Array-Elemente direkt hier mit Inhalt befüllen, das ist glaube einfacher :)


Ich finde ja die Begrenzung einer Beschreibung auf 255 Zeichen etwas knapp. Wir sollten ein LOB drauß machen.


all other stats work on specific files, this does not. So I thought I'd introduce something artificial.
Im open for better ideas :)


I actually agree, that the include shouldn't be in here anyways, considering that sometimes you already have code written, including the includes, but then want to add a function documentation.
It also helps making clear what actually is inside the snippet itself.
Maybe also switch the snippet name to being just `function_documentation_ace`?


How is this different than what it was previously? Also, we shouldn't be returning true on network errors.


Can you remove these unused parameters?


Felder die du noch nicht weißt, würde ich ncoh nicht setzen.
So kannst du später einfacher abfragen `issset($tasks->assignIssueToUser)`
Im Moment ist nicht klar: Soll das Ticket danach niemandem zugewiesen werden, oder wissen wir noch nicht, wem es zugewiesen werden soll. Das ist konzeptionell schlecht.


add use_gmt_hitset to pg_pool_t::dump() ?


s. o.


consider avoiding local variables when not needed ( enumerable module is your friend :-) e.g.:
```ruby
def longest_word(words)
  words.max { |a, b| a.length <=> b.length }
end
```


I like this.


Good catch!

2015-08-15 9:36 GMT+02:00 Andrey Khozov <notifications@github.com>:

> You need escape > as E<gt> inside code block.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/oliwer/mango/commit/d4221f56a7f5449688debc1a61f45c54bffde012#commitcomment-12722650>
> .
>


Wie besprochen: Erscheint mir sehr wirr.
Wenn Du Dir sicher bist, dass das nicht in einem rutsch geht, teste ich das mal gegen.
In diesem Fall bitte Issue an mich.


If you look at the function body they never return without passing a value. It seems like the named return is really not doing anything here.


嗯，这个更新不错


Yes.  It was discussed then, and we all forgot it in the changelog.


Einzeilige Kommentare werden in PHP idR mit `//` eingeleitet


As far as I remember based on hook metrics this one was never called. As for the function, base on our coverage result here: https://coveralls.io/files/716536462#L231 it is called so we cannot remove it.


You used `parseInt` on L68. Why not `tryParseInt`?


It's both to keep the connection alive and as a status check to make sure that the hyperdeck is still alive on the network. I still found the connection to be a little temperamental and there is also some code upstream of this to reconnect to the hyerdeck before sending a command if there is no current connection.


Oh, OK - I'll defer implementation questions to Mat - I'm just happy it is
there as another tool in our toolbox.

On Mon, Aug 17, 2015 at 8:36 AM, Niklas Schnelle <notifications@github.com>
wrote:

> Sorry should have worded that better. I see the point of the call but not
> of runtime_info_t.size which as far as I can see is always
> sizeof(runtime_info_t).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/spark/firmware/commit/0f742b896b2b1f519a3aeb08c8b34abfca6f0cb3#commitcomment-12738266>
> .
>

--
Andy


Can you tell me which parameter it was? I think you're right that we should be doing some encoding instead, this doesn't seem like the right solution.


Fixed


You need the repo to run inigo, you're right it should probably not have been in this section though. As for the bosh.io error, I thought that would upload latest, clearly that didn't do it, thanks for pointing it out and sorry for writing this without checking first.


Ok, sorry, am very busy this days :((

Regards

2015-08-15 10:39 GMT+02:00 Dario Di Maio <notifications@github.com>:

> I will care, no problem.
>
> From Mobile.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/souliss/souliss/commit/96df5efb4407b198a93261f0f6518114bc94cd8a#commitcomment-12722757>
> .
>


@Alexqw just to make sure this is tested correct: Have you deleted your cookie file when tested it without the User-Agent?
I was working on this 'long' time ago, but always had to use this User-Agent to get it to work. The problem was, that sometimes it works for 5 minutes, sometimes for 15 minutes, but it crashes some time ;)


@serhiy-storchaka I'm afraid `os.name == 'win32'` is always False. Shouldn't it be `os.name == 'nt'` instead?
`'win32'` is actually the value returned by `sys.platform`.


You're quite right! I'll get it fixed.


ich glaube du fängst bisher keine Kommandos im Issue selbst ab (sondern nur in seinen comments)
Weiterhin: Willst Du wirklich den ganzen Text in `sourceText` laden? Reicht nicht die Zeile mit dem Kommando?


Can you fix this jshint errors?

```
$ jshint -c .jshintrc ./src/*.js
./src/patch.js: line 40, col 58, Missing semicolon.
./src/patch.js: line 58, col 104, Line is too long.
./src/patch.js: line 71, col 63, Missing semicolon.
./src/patch.js: line 84, col 19, 'firstPath' is already defined.
./src/patch.js: line 85, col 18, 'restPath' is already defined.
./src/patch.js: line 89, col 32, 'firstPath' used out of scope.
./src/patch.js: line 89, col 43, 'restPath' used out of scope.
./src/patch.js: line 101, col 2, Missing semicolon.
./src/patch.js: line 11, col 5, 'parsePath' is defined but never used.

9 errors
```

Aside these minor code issues, it looks good! Thanks for spotting and fixing this :)


You are correct. However, the current master does only dump the last message when the error limit has been reached and not all the messages, where an exception occurred, see https://github.com/certtools/intelmq/blob/master/intelmq/lib/bot.py#L123
With this changes it is guaranteed, that every message resulting in an Exception, will be logged. Probably multiple times, yes. But all information can be found in the logs.

I tried to clarify and simplify (flatten, deduplicate) the error handling. As the behavior is undefined and undocumented, see #289.


IMHO falsche Reihenfolge: Die Comments müssen von unten nach oben durchgegangen werden.
Der Vorgang muss beim ersten Kommando an den GitReminder beendet werden


Any particular reason this doesn't read $argv?


This is my commit!! :D


Compare integers rather than dates because `provider.last_refresh_date` could be nil


Our tests pass with "master".


This is also my commit!! :D


Strings in .NET aren't zero terminated. x)


Since the system modules and applications are updated independently, the caller and callee may be compiled at different times and against different versions.  The `size` field is a simply way for the callee to know what the struct definition is when the caller was compiled.


Hmm, that's actually something I need to fix. I currently have an unreachable line of code in the template that I need to remove.


Den Namen des GitReminders sollten wir in einer Klassen-Eigenschaft vorbelegen


А сделайте пожалуйста pull request!


:cry: you're totally right -- thanks, will fix...


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.910.f1fd869dc4e45693271969178de9f30504a31357](http://build.palaso.org/viewLog.html?buildId=61316&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 410, ignored: 4 Build time: 00:04:09


if you use it only once: [Data.List.lookup](http://www.stackage.org/lts-3.1/hoogle?q=lookup)


Yup. I'll update the commit message.


I'm on 3-0-stable. Okay.


LGTM


Right now I have the error msg printing so I can see it but I'll clean that up once it's working.  The --terminate flag sounds good.  To plugin I give the path to the executable in the config.json "hooks" section. For now I do 'go build' in the register dir to get the executable before runc is run.  The 'go build' will be added to Makefile eventually.

----- Original Message -----
From: "Daniel J Walsh" <notifications@github.com>
To: "sallyom/runc" <runc@noreply.github.com>
Cc: "sallyom" <somalley@redhat.com>
Sent: Monday, August 17, 2015 8:41:32 AM
Subject: Re: [runc] WIP: add RegisterMachine (8147d5c)

Should check print the error message to stderr?  If this works correctly, it looks awesome.  Should we have one executable with a --terminate flag?  How does this plugin to the runc hooks?

---
Reply to this email directly or view it on GitHub:
https://github.com/sallyom/runc/commit/8147d5c62a9c8bc76df343f6194ce9034c9ed3d8#commitcomment-12737249


you think setting innerHTML using userinput is a good idea without doing anything to the message?


Upper line #20 was removing it.
This is the same. Just refactoring.

Why it is removed in the first place - I don't know :cherries:


is this really necessary?


Fine by me.


The function doesn't look good when I try to separate this part. Let's leave it. I have reduced this function to 40 lines which should be good.


I'm still concerned because with the currently checked in code, as you and I have discussed I get a segfault due to a stack overflow, and that call stack shows this:
```
...
00007FFFD0AFB7D8 00007FFFF689A1DA [InlinedCallFrame: 00007fffd0afb7d8] Interop+libcurl.curl_multi_socket_action(SafeCurlMultiHandle, Int32, Int32, Int32 ByRef)
00007FFFD0AFB7D8 00007FFFF3431D61 [InlinedCallFrame: 00007fffd0afb7d8] Interop+libcurl.curl_multi_socket_action(SafeCurlMultiHandle, Int32, Int32, Int32 ByRef)
00007FFFD0AFB7C0 00007FFFF3431D61 DomainBoundILStubClass.IL_STUB_PInvoke(SafeCurlMultiHandle, Int32, Int32, Int32 ByRef)
00007FFFD0AFB8C0 00007FFFF3431945 System.Net.Http.CurlHandler.CheckForCompletedTransfers(SafeCurlMultiHandle, Int32, Int32, IntPtr)
00007FFFD0AFB9D0 00007FFFF3432773 System.Net.Http.CurlHandler.CurlSocketCallback(IntPtr, Int32, Int32, IntPtr, IntPtr)
00007FFFD0AFBAF0 00007FFFF34323D8 DomainBoundILStubClass.IL_STUB_ReversePInvoke(Int64, Int32, Int32, Int64, Int64)
00007FFFD0AFBD88 00007FFFF689A1DA [InlinedCallFrame: 00007fffd0afbd88] Interop+libcurl.curl_multi_socket_action(SafeCurlMultiHandle, Int32, Int32, Int32 ByRef)
00007FFFD0AFBD88 00007FFFF3431D61 [InlinedCallFrame: 00007fffd0afbd88] Interop+libcurl.curl_multi_socket_action(SafeCurlMultiHandle, Int32, Int32, Int32 ByRef)
00007FFFD0AFBD70 00007FFFF3431D61 DomainBoundILStubClass.IL_STUB_PInvoke(SafeCurlMultiHandle, Int32, Int32, Int32 ByRef)
00007FFFD0AFBE70 00007FFFF3431945 System.Net.Http.CurlHandler.CheckForCompletedTransfers(SafeCurlMultiHandle, Int32, Int32, IntPtr)
00007FFFD0AFBF80 00007FFFF3432773 System.Net.Http.CurlHandler.CurlSocketCallback(IntPtr, Int32, Int32, IntPtr, IntPtr)
00007FFFD0AFC0A0 00007FFFF34323D8 DomainBoundILStubClass.IL_STUB_ReversePInvoke(Int64, Int32, Int32, Int64, Int64)
00007FFFD0AFC338 00007FFFF689A1DA [InlinedCallFrame: 00007fffd0afc338] Interop+libcurl.curl_multi_socket_action(SafeCurlMultiHandle, Int32, Int32, Int32 ByRef)
00007FFFD0AFC338 00007FFFF3431D61 [InlinedCallFrame: 00007fffd0afc338] Interop+libcurl.curl_multi_socket_action(SafeCurlMultiHandle, Int32, Int32, Int32 ByRef)
00007FFFD0AFC320 00007FFFF3431D61 DomainBoundILStubClass.IL_STUB_PInvoke(SafeCurlMultiHandle, Int32, Int32, Int32 ByRef)
00007FFFD0AFC420 00007FFFF3431945 System.Net.Http.CurlHandler.CheckForCompletedTransfers(SafeCurlMultiHandle, Int32, Int32, IntPtr)
00007FFFD0AFC530 00007FFFF3431846 System.Net.Http.CurlHandler.CurlTimerElapsed(IntPtr)
00007FFFD0AFC590 00007FFFF343176D System.Net.Http.CurlHandler+<>c.<CurlMultiTimerCallback>b__18_0(System.Threading.Tasks.Task, System.Object)
...
```
which shows curl_multi_socket_action calling back to CurlSocketCallback synchronously.  Has something changed with regards to how this is all set up that would cause curl_multi_socket_action to behave differently?


Checkmarx scan results 17 Aug 2015 16:54 UTC +03:00

High
Medium
Low
Info

New
36
31
37
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

c9fe25f9884708c258e7020b958e7b510ecad620	@dudug


Okay


すみません、まちがいました。かいけつしました。


We should look at this, it got placed in sources/all, but actually has x86 in the version name...

/cc: @mfonville @bgiesing


@Majkl578 are you able to see the problem with this first patch? https://gist.github.com/laruence/848e69d0a43846b61d26


Right, let me fix it.


Question, why was this originally a pointer and not just a member data? It seems to have the exact same lifetime as the module.


*
+12 - 16th August


Well, in fact that does work:

```
ruby 2.1.6p336 wasabi in ~/workspace/bosh-lite
± |master ✓| → bosh upload release https://bosh.io/d/github.com/cloudfoundry-incubator/garden-linux-release
Acting as user 'admin' on 'Bosh Lite Director'

Using remote release `https://bosh.io/d/github.com/cloudfoundry-incubator/garden-linux-release'

Director task 88
  Started downloading remote release > Downloading remote release. Done (00:00:25)

  Started extracting release > Extracting release. Done (00:00:01)

  Started verifying manifest > Verifying manifest. Done (00:00:00)

  Started resolving package dependencies > Resolving package dependencies. Done (00:00:00)

  Started creating new packages
  Started creating new packages > btrfs-progs/71889c437be1137b98aa094a52e664cfe8e36cd3. Done (00:00:00)  Started creating new packages > busybox/1593c41beb57c12d7f2b82dff61b47d4d119913d. Done (00:00:00)
  Started creating new packages > garden-linux/df3c1c28468687a40690906ebe076b12b5dc8d50. Done (00:00:00)
  Started creating new packages > golang_1.4/f57ddbc8d55d7a0f08775bf76bb6a27dc98c7ea7. Done (00:00:01)
  Started creating new packages > iptables/7226d311e90f49b05287e79f339581a1de9ea82e. Done (00:00:00)
  Started creating new packages > shadow/ffd1741bd9e0a176e67c61ef70cd0ed76b0ec285. Done (00:00:00)
     Done creating new packages (00:00:01)

  Started creating new jobs > garden/e96f642a9e5899326f538e5a38e325bf16c8d24f. Done (00:00:00)

  Started release has been created > garden-linux/0.294.0. Done (00:00:00)

Task 88 done

Started         2015-08-17 13:54:50 UTC
Finished        2015-08-17 13:55:17 UTC
Duration        00:00:27

Release uploaded
```


In the future, I suggest using ```std::make_unique<...>()``` instead.


Denke hier kannst Du auch eine Referenz senden?


Doesn't matter what you think you are returning.
The closure is created when modelErr == nil so thats what we will always see.


how to run?? help me. I'm beginners


Crap I probably need to change that. I believe the unless was from my first
iteration and testing then decided a facter would make it a little more
clean. Let me update and remove the unless statement from there.

On Sun, Aug 16, 2015 at 6:35 AM, Ger Apeldoorn <notifications@github.com>
wrote:

> Hi,
>
> Thanks for your update; how does adding the facter fact add to the 'unless
> => ' statement in the exec?
> It should already prevent you beiing re-registered.
>
> Ger
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bsdtux/puppet-rhsm/commit/748f452cd09d7f3cdb61005678bbb496b23022f9#commitcomment-12727419>
> .
>


This is what I raised before. We never had network issues with these calls other than underlying issues to etcd. Now we have network issues related to the invocation which is different and the default value is false.


OK.


I put it in `Xapi_vm` for reliability (so we do the test even if not called from `Message_forwarding`).

I put it in `Message_forwarding` to avoid annoying the user with a misleading error: `forward_vm_op` has
```Xapi_vm_lifecycle.assert_power_state_in ~__context ~self:vm
				~allowed:[`Running; `Paused];```
so if `xenprep_start` in `Message_forwarding` did not check for ``Running` then it could raise a misleading error message saying that the VM must be either running or paused.


Old tr.txt (that generated from wikipedia articles) has lot of non-Turkish craps, so i generate new wordlist based on [Zemberek wordlist](https://code.google.com/p/zemberek/downloads/detail?name=full.txt.tr.tar.gz&can=2&q=). This wordlist is smaller than original because original one is too huge to use.

This wordlist is random generated and auto-transcripted.

**Note: It has too few functional POS yet. I'll add they in my free times.**
 ####If it has problems, please make pull request.####


Move params to separate method so you can reuse it.


I'll probably omit this commit just to keep it easier to port this PR forward if we need to. I assume `DigestUtils.md5Hex(email.toLowerCase())` returns the same thing as the POJ `generateMD5hash()` function you wrote.


What does the input json look like?  Are you seeing any errors in journalctl?


Maybe we should have a smarter selection of filter steps. That is: Only consider the filter step if the filtered element (the incoming element) is a vertex or an edge or rather if the previous step is a `VertexStep`, an `EdgeStep` or an `EdgeOtherVertexStep`.


Checkmarx scan results 17 Aug 2015 17:00 UTC +03:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
36
31
37
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

073fa3e64979a1e5b9c06fc4c06fe80a4f1ad057	@dudug


> Weird - for me it simply prints a bunch of colored boxes with question marks in them.

This is **True**, as *linux framebuffer* can only display 256 or 512 characters, it cannot display some character.

> Can you open an issue about this so we can take a look and solve it for more than one prompt?

You finally asked that. It is already an issue, look at #2070, it is still an unsolvable, mystery puzzle.


@Majkl578  anyway, I can not see the segfault whiling running www/index.php


Yeah, may want to raise some sort of malformed response error in that case?


Noted


Use decent exposure here


MALICIOUS_CODE : EI_EXPOSE_REP2


PERFORMANCE : URF_UNREAD_FIELD


Decent + params in separate method


Trying to minimize impact on the rest of the code, which is maintained by not me.
As to why the original author didn't just make it member data, beats me.


yes, that is why order of iteration over containers has/had to be made deterministic


Diese Zeile ist überflüssig.


Why do we enter a new scope if we're not lazy? That seems wrong.


Move it to decent exposure and remove this method from controller


woops, typed `--save-dev` out of muscle memory...


Das sieht mir noch nciht fertig aus =)


hmm, seeing now..


Why empy class param?


Checkmarx scan results 17 Aug 2015 17:03 UTC +03:00

High
Medium
Low
Info

New
2
1
103
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

3b7156b3a7c92a8537121cda42302dfda9bc8ece	@BearSpace


Aqui vai ser um integer mesmo? A estrutura do processo vai ser somente uma máscara?


@BaumSchorle I didn't test it while clearing my cookie cache. :-/ But this restores the User Agent, so it should all be kosher again.

I hate servers that care about user agents...


Is this path ok? Try to extract this to partial and reeuse in  app/views/admin/foods/index.haml  with params if needed.


為什麼要連執行檔一起上傳呢


Out of curiosity, why ```stream``` instead of ```global```?


Hmmm, you might need to delete the things in temp/cache to see it, because the error happens when cache for the application is being built.

I forgot to mention the caching in the bug report, sorry!


Is this essential to a leaser?  I'd rather take the time to think about this one separately.


Checkmarx scan results 17 Aug 2015 17:04 UTC +03:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
36
31
37
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

81c3f30f10a9a8c4a086f574473b8fc4856ca18a	@dudug


That's a fiar point. The initial intent was for it to behave like Firebase's iOS/Android/Web SDKs where it just takes a URL and operate against a particular instance. However, that may not be the majority use case for go end of things. I'd be more than happy to have some back and forth on redesigning the package to work for these type of use cases.


develop (lgu1234) - i386-Windows-7-VisualStudio-12: OK (1274 of 1274 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)


Yay! BigInt pra armazenar e a mascara pra melhor a UX.


If zero and -1 do the same thing, why do we have both?


:+1:  lgtm


* Create Table hat an der Stelle nichts zu suchen :)
* Bitte Datenbank bauen, Struktur exportieren und einchecken


Так можно сделать, но клиент этой ошибки не увидит. Не в этом месте нужно кидать Exception.


I agree on not including the include, I think the string should remain as is


develop (lgu1234) - Win64-Windows-7-VisualStudio-12: OK (1274 of 1274 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)


Hieraus könnte man auch Stored Procedures machen :-)
Aus Übungsgründen würde ich Dich hierum bitten, dies ist aber Prio 3


Can you name this?  I find it confusing to read `l.Start()`, when I'm looking for `l.Plug.Start()`


Read http://blog.althafkbacker.com/2013/10/android-and-microsoft-sql-ms-sql-server.html


The scan failed.
Included commits

Commit ID	Owner

6cf2249d7346f481528f7a0b8bfa94d149571d67	@dudug


Olá, bom dia!
Gostaria de esclarecer uma dúvida. Quando o cliente sai da minha loja e é direcionado ao CheckOut Cielo para efetuar o pagamento, ele é direcionado para a página que eu configurei na minha URL de Retorno. Eu gostaria de saber como eu faço para recuperar os dados enviados pelo formulário. Exemplo:
Quando eu envio o nome do cliente, ele enviaria o "customer_name" em método POST, para eu recuperar, eu iria declarar a variável:
$customer_name = $_POST["customer_name"];

E para exibir a mesma, eu iria colocar o:

echo $customer_name;

Contudo, fazendo isso, retorna em branco, como se não estivesse enviando o parâmetro. Como fazer para recuperar essas variáveis enviadas pelo formulário do CheckOut Cielo?

Att,
Junior


You need to also remove the actual params ([L281](https://github.com/kugel-/geany/commit/6db5c24a33e949d26f03afb09170bdf618eee5dd#diff-a6498e6de1fbc3174b0ce344a1e3b758R281))


I think the Layer will need something along the lines of `tms_url`, which is the ultimate product of the processing that happens on the composite of a Layer's Images.


You guys have different URLs.  Matt has:

http://bosh.io/releases/github.com/cloudfoundry-incubator/garden-linux-release

Luan has

https://bosh.io/d/github.com/cloudfoundry-incubator/garden-linux-release

Generally, bosh.io URLs for *d*ownloading assets like releases and
stemcells have are extensions of https://bosh.io/*d*/

On Mon, Aug 17, 2015 at 6:56 AM, Luan Santos <notifications@github.com>
wrote:

> Well, in fact that does work:
>
> ruby 2.1.6p336 wasabi in ~/workspace/bosh-lite
> ± |master ✓| → bosh upload release https://bosh.io/d/github.com/cloudfoundry-incubator/garden-linux-release
> Acting as user 'admin' on 'Bosh Lite Director'
>
> Using remote release `https://bosh.io/d/github.com/cloudfoundry-incubator/garden-linux-release'
>
> Director task 88
>   Started downloading remote release > Downloading remote release. Done (00:00:25)
>
>   Started extracting release > Extracting release. Done (00:00:01)
>
>   Started verifying manifest > Verifying manifest. Done (00:00:00)
>
>   Started resolving package dependencies > Resolving package dependencies. Done (00:00:00)
>
>   Started creating new packages
>   Started creating new packages > btrfs-progs/71889c437be1137b98aa094a52e664cfe8e36cd3. Done (00:00:00)  Started creating new packages > busybox/1593c41beb57c12d7f2b82dff61b47d4d119913d. Done (00:00:00)
>   Started creating new packages > garden-linux/df3c1c28468687a40690906ebe076b12b5dc8d50. Done (00:00:00)
>   Started creating new packages > golang_1.4/f57ddbc8d55d7a0f08775bf76bb6a27dc98c7ea7. Done (00:00:01)
>   Started creating new packages > iptables/7226d311e90f49b05287e79f339581a1de9ea82e. Done (00:00:00)
>   Started creating new packages > shadow/ffd1741bd9e0a176e67c61ef70cd0ed76b0ec285. Done (00:00:00)
>      Done creating new packages (00:00:01)
>
>   Started creating new jobs > garden/e96f642a9e5899326f538e5a38e325bf16c8d24f. Done (00:00:00)
>
>   Started release has been created > garden-linux/0.294.0. Done (00:00:00)
>
> Task 88 done
>
> Started         2015-08-17 13:54:50 UTC
> Finished        2015-08-17 13:55:17 UTC
> Duration        00:00:27
>
> Release uploaded
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cloudfoundry-incubator/diego-release/commit/ffd653f15059398d9795b1b8181c230ac74ee4f4#commitcomment-12738827>
> .
>


HELLO WORLD comment on pos 0


I think I'd thought of that at one point, but it's kinda complicated because it's not always a check of the previous step, like:

```text
g.V(123).out().has('name','stephen').has('age',20)
```

Is there a nice way to do that? and how complicated does that get?


Is this URL the path to the tms/tiled output that is produced on input?


:+1: cool to have insurance, and I like the refactors


Just so I'm sure the path here.  Your intent is shoot yourself in the head and let systemd restart you?  If so, can you comment to make the intent clear to everyone.


`WaitForStop`?


Название селектора отбивается от блока правил пробелом
.e-circlelabel {
P.S. Сложное название для блока
Названия в разработке целая отдельная тема
Чем оно короче, очевидней и проще, тем лучше.
Если название состоит из нескольких слов, разделяем дефисом
Не стоит в названиях использовать стилевые характеристика (это будет видно и так по коду) лучше характеризовать смысловую нагрузку компонента.
(Пример .red-bordered-box { color: red; border: 1px, solid, #FF0000 } || .error { color: red; border: 1px, solid, #FF0000 } - Воторой вариант намного понятней, более читабельный, и не режущий глаз
e-circle-label
Предложения
Блок в основном содержит инфу о преимуществах, фичах компании
Как вариант назвать его e-feature, и модификаторы для отображения соответствующего фона


I'm using Metin2CMS CLI when I push changes to repository and I forgot to change the commit message before hitting "Enter". My bad.


NetIDAllocator treats the "max" value as being part of the range, so this should be `(1 << 24) - 1`


Not to be terribly annoying, but if we remove the GModule argument, shouldn't the function rather be called `geany_load_plugin()` or something maybe?  There's no more reference to a "module" except for this function name, might be odd.


Настрой табуляцию в редакторе
Символы табуляции должны заменяться пробелами, 1 таб - 4 пробела


And that's why I added an `else if (rsc == bestReachingScore) { if (get(vertex_index, g_, v) < get(vertex_index, g_, bestVertex)) bestVertex = v; }` in bfcbe492d832556f1e61b1224a9f5018303a9a0a


`Xapi_vm_helpers.update_vm_virtual_hardware_platform_version` is called from `Xapi_vm.start` and a few other functions in the `Xapi_vm` module. I'm not sure if I want to rely on that for the long term, but perhaps it makes sense. Or if we do want to do the update here, we could use Xapi_vm_helpers.update_vm_virtual_hardware_platform_version so as to avoid repetition of logic (even though that function will do needless checking of whether auto-update is switched on).

Oh, and also I wanted to update the virtual hardware platform version _before_ switching on auto-update, but perhaps that's not so important.


This works when I tested on Duckpan, but I'm not sure how to plug the updated `pm` file so that the triggering works on DDH10 too.


You should use `Array.isArray($scope.videoList)`


develop (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1274 of 1274 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.95%25-green.svg)


PlantOperationSchemes (kbenne) - i386-Windows-7-VisualStudio-12: Build Failed

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


@dalexander, I don't think you need to review this anymore. I gave it an alternative implementation inae962ffb4b7e48ac2ba582a559f5238676f70247


Is calling stop on a non-started plug allowed?  After a stop happens, am I allowed to call start again?


Aggiunto thread per effettuare il seleziona tutto


This is quite strange to me - there was already a test here that was supposed to check that it's reaching the end state - not sure why the test passed.


Required d3 since the library needs a global reference. This line shouldn't be done at the top, however, but rather within an if-statement for checking if we're in a CommonJS environment.


just use `Array.isArray` to avoid error-prone code and I will merge it


this is no longer a problem anymore since `2.0.0`, very nice guys!


1141 строка
Compaigns, а должно быть Campaings (опечатка)

2480 строка
 #: common/php_functions.php:946 common/php_functions.php:1282
 msgid "ХЗ что"   msgid "ХЗ что"
-msgstr "god knows what"  +msgstr "God knows what"

скорее вопрос, это сообщение об ошибке, стоит ли формализирровать?
Спасибо, что исправили ошибку в 612 строке


I was getting an error on this line of `TypeError: tspan.node(...).getComputedTextLength is not a function`. I commented out the if-statement to get it working.


This might be a dumb idea because I've never worked with react contexts and might misunderstand what you are trying to solve but anyway:

What if you define the 'tos' context to be an immutable stack. Then getChildContext on Springs would create a stack if there is none and add to stack if there is. Child would have a getChildContext that return a popped stack from its context.


You can simply write `if (ok)`. The `== true` is slightly redundant as the variable is already a boolean


@imsickofmaps :+1:


i don't think we would want to ++ the SpanningReads of the null_vertex


mAkeVector & makeCacheMAtrix files uploaded...


Put some class level javadoc explaining what this class is doing/the rule of the name generator.

The point of the game is that the code does the manipulating of the strings . So rather than asking for the first,second and third letters of a name you should be asking the user just for their first name and using the methods on String ( such as substring ) to pull extract the 1st ,2nd 3rd letters and build the name up like that.


consider looking at the flow - why you need to run unique twice (it also means that the calculation would be done twice)?

what about return value of method `result` sometimes it would be String and sometimes it would Number (we should avoid this kind of situations). What's more the class is calculating position - and position should be as number (as long as we are not displaying value to the user, then we are converting value to string). Please look at the following solution:

```ruby
class LowestUniqueNumberPostion < Struct.new(:numbers)
  def to_s
    lowest_unique_position.to_s
  end

  def lowest_unique_position
    (numbers.index(lowest_unique_number) || -1) + 1
  end

  private

  def lowest_unique_number
    unique_numbers.min
  end

  def unique_numbers
    stats.select { |_, counter| counter == 1 }.map(&:first)
  end

  def stats
    numbers.each_with_object(Hash.new(0)) { |number, memo| memo[number] += 1 }
  end
end
```


Отступы
И здесь написано буквально
Чистая чистота
Комментарии нужны в следующих случаях
Для разделения блоков кода,
Для объяснения неочевидного решения, и причины его применения
P.S.
Всё очевидное намного лучше не очевидного


I did automate it but I'm afraid it's too setup-specific. I'm using zsh and did this:

```
autoload -U zmv
noglob zmv -W lib/smartdown_dlows/student-finance-forms/snippets/*.txt  lib/smart_answer_flows/student-finance-forms/_*.govspeak.erb
```

I'm pretty sure there's a nicer way to do this, but it's a one-off task and it does the trick.


If you could take the time to look at our XDA thread, you would see that thanks is given to CM which includes you @uberlaggydarwin

As for your second comment, if you didn't want credit (like you said) then you wouldn't have made this post. Don't try to play the cool guy card and seem like you're whatever about it because the truth is that you thrive on glory hahaha

The commit will be corrected shortly so no worries there.........thanks, I guess


It looks like the npm version is a little behind the github one https://www.npmjs.com/package/d4. When I pulled it the latest was 8.10, as of today it's 8.15. Perhaps the newer version solves the odd tspan error below.


Цвет лучше кодом


oh ok let me ignore this then


Can you explain what is completes<-complete(directory, 1:332) ? since i'm unable to find complete function in R library?


I wanted to do a CSS-only solution for this, but I couldn't figure out a way to do it that didn't break the dark theme style.


teste 2,1


Won't this cause a memory leak if you hit either of the first two returns?


in the future you should probably `rebase -i` to remove false starts from history


I hadn't gone and manually checked if TransientTrackBuilder was OK to use like that, but looking at the code again it's used directly and not copied. I guess there is no reason to not go global.

Will fix.


(non-public history, of course)


Checkmarx scan results 17 Aug 2015 17:06 UTC +03:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
36
31
37
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

16c53fe90ca89f1849313c9da368b5490ac97f1d	@dudug


return std::move is bad practice. Just return the temporary and let the compiler decide to either elide the copy or call the move-ctor


Содержимое объекта внутри
{
    elem: 'title',
    ...


Yea, unfortunately at this point I had already made it public. =(


I commented out all intermediate anonymous functions and `use strict` declarations.


OK will go for a release today/tomorrow.


gjdsgvjdksjckdzkc dx


:mushroom:

Maybe we should take a more "Read on to get a clear and detailed explanation" approach here? (Thinking of folks who are freaking out about why they just got a bandwidth overage bill and such).


else what ? assert ?


kkkkkkkkkkkkkkkkk


You don't have to use .replace('px', '') when you're using parseFloat(), only if you use Number(). Also be aware that parseFloat will parse anything whereas Number will give you NaN if the string is invalid.


why did you remove that?


Oh, was the intent here to have a `LeasedPlug` that conforms to `Plug`?  A few extra letters on a name might've helped. :)

If so, please add to the godoc.


Не нужно писать объекты в одну строчку
mods: {
    upper: true
},


Check out [this post](http://summerjava.boards.net/thread/12/assignment-zoowsome). It's nothing mandatory btw


MALICIOUS_CODE : EI_EXPOSE_REP2


PERFORMANCE : URF_UNREAD_FIELD


Если пишеш так, то отбивай отступом содержимое массива
[
    {
        ...


Am 17. August 2015 16:10:04 MESZ, schrieb Colomban Wendling <notifications@github.com>:
>You need to also remove the actual params
>([L281](https://github.com/kugel-/geany/commit/6db5c24a33e949d26f03afb09170bdf618eee5dd#diff-a6498e6de1fbc3174b0ce344a1e3b758R281))
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/kugel-/geany/commit/6db5c24a33e949d26f03afb09170bdf618eee5dd#commitcomment-12739168

Fuck me! I'm getting mad for this PR. I was 110% sure I did that...but was late also :-(


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

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


Yep:-)


If anyone is asking "hey where are the tests" I'm working through a fairly big/painful refactor from controllers to components that when complete will show how I came across this bug. For now I used this components/pods example to fail/pass (using 1.0.3 and 1.1.0 as the before and after)

https://github.com/toranb/pods-with-components-example


I think `NewImage` should be `NewGif` ?


Читается тяжело
Поправь однотипные замечания в коде
И изначально пиши по правилам хорошего тона


Also `NewGif` ?


what is this code suppose to do ?


'Tis ok


:stars:

Love thinking about astrophysics... but same feedback as below: the people reading this might be in a sensitive state, having just gotten a huge overage bill.


you forgot the character ``?``
if route => if route?


when i use wildcards and and/or connectors i have this Error: Invalid argument supplied for foreach()

this is my model user.php

public $hasAndBelongsToMany =

array(
'Competence' => array(
'with' => 'CompetencesUser',
'className' => 'Competence',
'joinTable' => 'competences_users',
'foreignKey' => 'user_id',
public $filterArgs = array(

'competences' => array(

        'type' => 'subquery',
        'method' => 'findByCompetences',
        'field' => 'User.id',

     'connectorsAnd' => '+',
     'connectorOr' => ',',
      'before' => true,
       'after' => false)

);
public function findByCompetences($data = array()) {

$this->CompetencesUser->Behaviors->attach('Containable', array( 'autoFields' => false ) );

$this->CompetencesUser->Behaviors->attach('Search.Searchable');
//$data = array('competences' => 'Cake');

$data = array('competences' => 'First%+Second%, Third%');

$result = $this->CompetencesUser->parseCriteria($data);

$expected = array(0 => array('OR' => array(
array('AND' => array(
array('Competence.competence LIKE' => '%First\%%'),
array('Competence.competence LIKE' => '%Second\%%'),
)),
array('AND' => array(
array('Competence.competence LIKE' => '%Third\%%')
)),
)));

$query = $this->CompetencesUser->getQuery('all', array(

'conditions' =>array(
$expected,
$result),

'fields' => array(
  'user_id'

),
'contain' => array(
'Competence'
)));

return $query;
}
plz any idea ;
in my query
SELECT User.id, User.username, User.nom, User.prenom, User.email, User.password, User.created, User.modified FROM dcsearch.users AS User WHERE User.id in (SELECT CompetencesUser.user_id FROM dcsearch.competences_users AS CompetencesUser LEFT JOIN dcsearch.competences AS Competence ON (CompetencesUser.competence_id = Competence.id) WHERE ((((Competence.competence LIKE '%First\%%') AND (Competence.competence LIKE '%Second\%%'))) OR (Competence.competence LIKE '%Third\%%'))) LIMIT 20


Maybe useful - but beyond the scope of this change.
IP + xpath are the main 2 that will reduce the amount of traffic/results coming back from the server.
The client can still post-filter.


I stand corrected! Could be worse for my first authored man-page :laughing:
Now let's hope I don't make this mistake again, lesson learned!


Thanks a lot! :+1:


Final newline is missing here. It's quite [important](https://unix.stackexchange.com/questions/18743/whats-the-point-in-adding-a-new-line-to-the-end-of-a-file/18789#18789).


Thanks, i'll fix


I tagged this and released as 0.3.1 (https://github.com/convexopt/gpkit/releases).

Do you like this way of doing minor releases, or is this too much clutter on the releases page? I worry it might be the latter.


In the UK it's generally not possible to get above 1Mbps up on an ADSL connection anymore - and that's the sales literature figure, in reality it's usually 800kbps.
I did originally sign up with a better quality service, but it was acquired and they quietly removed my extra uplink speed!! I've shop around and don't have any better options.
Only a fibre or cable connection will provide better than that.

Maybe 1 carrot could have 1 Mbps up, and a higher uplink speed could be in 3 carrot?


@dalexander, I would appreciate it if you would review the following line and the changes to expectedDot in TestPoaConsensus.cpp for correctness.


No Vietnamese in source code


Hmm, now tests won't run in the browser anymore, right?


I was thinking about this, it could be useful and probably not difficult to do. Could be useful for passing json body to post request.


I think this should have been a revision bump, so users get the benefit of the slot operator.


I want to keep the server-side getLocks simple, i.e. just deal with locks as xpaths.
logical file xpath locks are just a specialized client (DFS) use of a lock.

Besides, anything under /Files *should* be a file xpath, so the !dfln.setFromXPath() is only a cautionary check to exclude anything that happens to be under /Files but not a logical file..


dalidiag "filelocks" wasn't documented (not in command usage) and it duplicated the functionality of daliadmin dalilocks files + had a different output format that had no advantages, so I dropped it, but kept the simple "locks" variety.

There's a separate old JIRA open, where any dalidiag functionality not in daliadmin was to be merged into daliadmin and dalidiag deprecated.


You can also add an appcast for easier maintenance of this cask in the future:
```ruby
appcast 'http://www.eltima.com/download/commander-update/settings.xml'
```


After looking at this import of having hibernate in a UI controller, i think the best approach is create a custom exception, and throw it from the service layer. Then this controller catches the custom exception and display the error message like you are already doing. Do you agree? :)


I just wanted to do it :)


I have to use wildcards and and/or connectors ;I have this Error: Invalid argument supplied for foreach()

this is my model user.php

public $hasAndBelongsToMany =

array(
'Competence' => array(
'with' => 'CompetencesUser',
'className' => 'Competence',
'joinTable' => 'competences_users',
'foreignKey' => 'user_id',
public $filterArgs = array(

'competences' => array(

        'type' => 'subquery',
        'method' => 'findByCompetences',
        'field' => 'User.id',

     'connectorsAnd' => '+',
     'connectorOr' => ',',
      'before' => true,
       'after' => false)

);
public function findByCompetences($data = array()) {

$this->CompetencesUser->Behaviors->attach('Containable', array( 'autoFields' => false ) );

$this->CompetencesUser->Behaviors->attach('Search.Searchable');
//$data = array('competences' => 'Cake');

$data = array('competences' => 'First%+Second%, Third%');

$result = $this->CompetencesUser->parseCriteria($data);

$expected = array(0 => array('OR' => array(
array('AND' => array(
array('Competence.competence LIKE' => '%First\%%'),
array('Competence.competence LIKE' => '%Second\%%'),
)),
array('AND' => array(
array('Competence.competence LIKE' => '%Third\%%')
)),
)));

$query = $this->CompetencesUser->getQuery('all', array(

'conditions' =>array(
$expected,
$result),

'fields' => array(
  'user_id'

),
'contain' => array(
'Competence'
)));

return $query;
}
plz any idea ;
in my query
SELECT User.id, User.username, User.nom, User.prenom, User.email, User.password, User.created, User.modified FROM dcsearch.users AS User WHERE User.id in (SELECT CompetencesUser.user_id FROM dcsearch.competences_users AS CompetencesUser LEFT JOIN dcsearch.competences AS Competence ON (CompetencesUser.competence_id = Competence.id) WHERE ((((Competence.competence LIKE '%First\%%') AND (Competence.competence LIKE '%Second\%%'))) OR (Competence.competence LIKE '%Third\%%'))) LIMIT 20


The need for broadband, electricity and other costs that the taxman prevents us from allowing staff to expense is already factored into salaries.
It's a minimum requirement for being able to 'get to work' - think of it like commuting to work, but much much cheaper.


The above misses the stack trace. Instead use log.error(ex.getMessage(), ex)


we usually put the docs in the implementation side


This looks good to me if someone actually want it.


Posting an updated commit message, because it this info may get lost in the `rebase/merge` process.

    Translate scenarios to integration tests for student-finance-forms

    These integration tests are a naive direct conversion of smartdown scenarios
    /lib/smartdown_flows/student-finance-forms-old/scenarios/.

    Conversion was rather trivial, given a scenario:
    - question_1: answer_1
    - question_2: answer_2
    - outcome_1

    the integration test equivalent is:

    should "lead to outcome_1" do
      assert_current_node :question_1
      add_response :answer_1
      assert_current_node :question_2
      add_response :answer_2
      assert_current_node :outcome_1
    end

    One gotcha is that in scenarios, the value on the left might not directly
    correspond to the node name e.g. what_year_eu_students.txt node can have a
    question 'what_year', not 'what_year_eu_students'.

    There are 3 integration tests fewer than scenarios, because three of them where
    duplicated.

    This commit is a part of the student-finance-forms conversion from smartdown to
    ruby smartanswers.


you don't need to quote this, it's a variable assignment


I captured some notes on recommended equipment here: http://way.wunder.io/working-at-wunderkraut/health-and-safety/work-comfortably/equipment/

maybe those can be merged in here, or just linked to?


Note to self: if we want to delete 0.3.0 or 0.3.1, https://help.github.com/articles/editing-and-deleting-releases/


Updated fcrepo-webapp and fcrepo-messenger to 4.3.0


I have created an issue with your notes. thank you!

https://github.com/sendgrid/sendgrid-php/issues/159


(Change suggested by Ümit Aslan.)


General comment. I love this three carrot approach.
I've made a couple of minor comments, but otherwise it's good to go and I'm happy to merge once they've been dealt with


Yeah, I just figured since the similar compile-time version is documented here (for obvious reasons), it would make it easier to read when looking at the source code (as opposed to using HTML version).


Probably shouldn't have made a commit for this file. wups.


@Majkl578  @pavelkouril okey, seems we missed another place to fix. I already get it now. thanks... making test.


probably. just submit another one-letter fix PR like I did :smile:


Why is this being cast to a `time.Duration`?


I think we should wait until we have an actual use case, but it's fine by to add if someone already has a use planned.


After some more discussion with Rob, I've replaced this code with a comment.


After a while in the console we have:

ABRT has detected 2 problem(s). For more info run: abrt-cli list --since 1439798550
[root@opendcim ~]# abrt-cli list --since 1439798550
id 75db4f525312d563c6aeac7f4de5ad51ed12caee
reason:         dot killed by SIGSEGV
time:           Mon 17 Aug 2015 10:38:21 AM EEST
cmdline:        /usr/bin/dot -Tpng -o/tmp/graph_l6wZEA /tmp/dot_lgud2Z
package:        graphviz-2.30.1-19.el7
uid:            48 (apache)
count:          21
Directory:      /var/tmp/abrt/ccpp-2015-08-17-10:38:21-13609

id 65440442808f5875b4df6eff55384ab51d3f209c
reason:         packagekitd killed by SIGTRAP
time:           Mon 17 Aug 2015 01:03:04 PM EEST
cmdline:        /usr/libexec/packagekitd
package:        PackageKit-0.8.9-11.el7.centos
uid:            0 (root)
count:          1
Directory:      /var/tmp/abrt/ccpp-2015-08-17-13:03:02-32239

Any idea?


`ContainerListener` (`NullContainerListener`, `Analyzer::ContainerListener`, `Builder::ContainerListener`) sounds good to me.

I think I could merge you list vs nested idea into something like this:

```rb
class CompositeListener
  def initialize(*listeners)
    @listeners = listeners
  end

  # example
  def started(image, name)
    @listeners.each { |l| l.started(image, name) }
  end
end
```

CLI could just send in `Analyzer::ContainerListener` as-is, but Builder could send in `CompositeListener.new(Builder::ConatinerListener.new, Analyzer::ContainerListener.new)`.

WDYT?


Thanks for your comment


This might be a bit too ambitious at first actually!

The paths you've outlined are definitely not wrong or bad, but I think it would be good if you added them as you create them. For example, add a single route, add it's template, prototype a bit. When you need another path, ad that.

It's hard to do big design like this up front, as you will likely want to change it a lot as it goes along. My advice is to start one route/handlebars template at a time :)


Can't modify a liquibase if it's been merged -- and thus run by others. I'll merge it anyway, but avoid that.


Having said that, it's not bad if you merge it in and start with that. You just might find you'll change it around a bit :)


Would it be possible to make this a mixin, rather than a fixed value? It would be nice to be able to reuse this throughout the site and have the ability to change color, border-widths, etc.


This package might have requirements that will conflict with versions installed on the Fuel Master. Could you please verify which version of zabbix-client can be installed on the Fuel Master node and pin that version in requirements file?


@zenvdeluca :beers:


the `\` at the end of the line is obsolete


Remeber to add some Javadoc at the class level.

This is a bit of a mean question to put in so early on the course as there are a few ways to do it ( including yours) that could have some gotchas ( the reason I'm bringing this up is that this gotcha is also used as a discussion point in the suggested questions for interviewing java developers )

The easiest way to do this would be to make use of the StringBuffer class, you can just pass it a string and call StringBuffer.reverse(String).

You could also shorten this by using a recursive method
  e.g.

public static String reverseString (String str)
{
	if (str.length <= 1 )
	{
		return str;
	}

	return reverseString(str.subString(1)) + str.charAt(0);
}

This doesn't avoid the gotcha I mentioned, and it could also result in a Stack Overflow Exception if this string was really long as there are as many levels of recursion as there are characters in the string.

A potential issue with your solution is that in Java chars are always 16 bits, but some character sets have characters that require 32bits to represent the character.  By using String.charAt() you would only retrieve half of a wide character, and the the value of the character would be jumbled up by your reverse method.


The usage of `waitFraction` doesn't seem to match the godoc.  When we do the subtract for `last`, we get the inverse of the time for the `waitFraction`, right?  `ttl - (ttl * waitFraction)`


I'll merge it too, but we should adopt a class name that won't be 10 times on the classpath ;-)

Should have XmlAdapter in its name to make its purpose clear, and the fact that it works both ways.


add some javadoc here to explain what this class does.


yep


Thanks for updating it Nick. We can remove the dialer indeed. Since we didn't add the Google dialer go any package (yet).


I have just suppressed the if an hour ago and pushed the modifs !
Are you looking at older code ?


Sorry about that.  I guess my grepping skills aren't up to scratch ;)


One more question, what about cloud-starters? Do they fallow the same naming convention?


Change it for ```@return 0 or greater.```


I don't think this doing what people would expect.  I think just controlling it via the `waitFraction` will be easier to understand.  Given usage, I could see eliminating it from the renew logic and just having `releaseRetries`


Mediafire links aren't matched correctly against Regexp because ID expects 15 characters.
Links within the hoster don't seem to have that number of characters in the ID. For instance : http://www.mediafire.com/?bymzjtndprr (11 char)
==> I replaced {15} with * in the pattern in MediafireCom.py
Tested, seems to be working, my downloads finally start


You could initialise this array on the same line you declare it, there isnt any benefit in declaring it then initialising in the line below


nit: unnecessary () around return expression.


Check argument values.


@Soll ещё проще можно сделать :)


Why is this "assertion" going away?


Should this keep have the NumberFormatException as a cause? Might provide info on how to fix problems. At the least it should have the limitString as party of the exception message


个人觉得
local str4 = [=[string have a "[[]]"]=]  --只要双方括号左右等号相等即可
修改成
local str4 = [ =[string have a [[]]  ]=]
更能说明问题


I'm not being rude, I'm just being straightforward xD


@Soll новый метод добавил, а тестов нет, не порядок :)


the above are already granted via logging_send_syslog_msg(blkmapd_t)


Why not? I don't think it causes problems and it makes it possible at least to figure out to which declaration the expression belongs, which is otherwise impossible.


Yes sorry


the above is already granted via logging_send_syslog_msg(blkmapd_t)


Fixed in commit 3a07115


Using int here rather than Integer (for both the bonus ball and other balls) might  hide bugs later on in the code.  An int by default will have a value of 0, whereas you need to initialise the integer with using the new keyword.  This might be a better approach here as if you forgot to set one of lottery balls it would have a value of 0, whereas if you were using Integer objects a null pointer exception would be thrown if you tried to use any integer object that had not be initialised.


Do this will be simpler if table had a (...) ctor ?


It's not, well spotted :)


Here you can use: /var/run/blkmapd.*
or if you want be more strict use:  /var/run/blkmapd\.pid


I don't really like passing the user's data back to them. I think the error
message is clear enough. The exception cause won't be returned to the user
so I didn't think it was worth providing it.

On Mon, 17 Aug 2015 15:41 idlewis <notifications@github.com> wrote:

> Should this keep have the NumberFormatException as a cause? Might provide
> info on how to fix problems. At the least it should have the limitString as
> party of the exception message
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Azquelt/tool.lars/commit/34a3d895d0c079de853d64014cdfc1ab833db955#commitcomment-12740032>
> .
>


Due to regexp use: /etc/rc\.d/init\.d/blkmapd


---> Fix flash dialog not showing issue


getSpace change will undo and re-pull


Well someone beat me to it.

```
[ 60%] Building CXX object pcsx2/CMakeFiles/PCSX2.dir/gui/MemoryCardFolder.cpp.o
cd /tmp/buildd/pcsx2-1.3.1-971-gf9e6a02+dfsg/obj-i586-linux-gnu/pcsx2 && /usr/lib/ccache/c++   -DDISABLE_BUILD_DATE -DDOC_DIR_COMPILATION=/usr/share/doc/pcsx2 -DGAMEINDEX_DIR_COMPILATION=/usr/share/games/pcsx2 -DPLUGIN_DIR_COMPILATION=/usr/lib/i386-linux-gnu/pcsx2 -DWXUSINGDLL -DWX_PRECOMP -DXDG_STD -D_ARCH_32=1 -D_FILE_OFFSET_BITS=64 -D_M_X86=1 -D_M_X86_32=1 -D__WXGTK__ -g -O2 -fstack-protector-strong -Wformat -Werror=format-security -D_FORTIFY_SOURCE=2  -msse -msse2 -march=i686 -pipe -std=c++11 -fvisibility=hidden -pthread -fno-builtin-strcmp -fno-builtin-memcmp -Wall -Wno-attributes -Wno-missing-field-initializers -Wno-unused-function -Wno-unused-parameter -Wno-unused-variable -Wno-unused-value  -Wstrict-aliasing -Wstrict-overflow=4  -D_FORTIFY_SOURCE=2  -Wformat -Wformat-security -DNDEBUG  -O2 -Wno-invalid-offsetof -pthread -I/usr/include/soundtouch -I/usr/include/freetype2 -I/usr/include/glib-2.0 -I/usr/lib/i386-linux-gnu/glib-2.0/include -I/usr/include/atk-1.0 -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/cairo -I/usr/include/pango-1.0 -I/usr/include/gtk-2.0 -I/usr/lib/i386-linux-gnu/gtk-2.0/include -I/usr/include/SDL -isystem /usr/lib/i386-linux-gnu/wx/include/gtk2-unicode-3.0 -isystem /usr/include/wx-3.0 -I/tmp/buildd/pcsx2-1.3.1-971-gf9e6a02+dfsg/common/include -I/tmp/buildd/pcsx2-1.3.1-971-gf9e6a02+dfsg/common/include/Utilities -I/tmp/buildd/pcsx2-1.3.1-971-gf9e6a02+dfsg/common/include/x86emitter -I/tmp/buildd/pcsx2-1.3.1-971-gf9e6a02+dfsg/obj-i586-linux-gnu/common/include -I/tmp/buildd/pcsx2-1.3.1-971-gf9e6a02+dfsg/pcsx2/gui -I/tmp/buildd/pcsx2-1.3.1-971-gf9e6a02+dfsg/pcsx2/x86 -I/tmp/buildd/pcsx2-1.3.1-971-gf9e6a02+dfsg/obj-i586-linux-gnu/pcsx2/gui -I/tmp/buildd/pcsx2-1.3.1-971-gf9e6a02+dfsg/pcsx2/.    -fno-omit-frame-pointer -fno-strict-aliasing -Wno-parentheses -Wstrict-aliasing -Wno-char-subscripts -Wno-missing-braces -fno-guess-branch-probability -fno-dse -fno-tree-dse -fno-omit-frame-pointer -fno-strict-aliasing -Wno-parentheses -Wstrict-aliasing -Wno-char-subscripts -Wno-missing-braces  -fno-omit-frame-pointer -fno-strict-aliasing -Wno-parentheses -Wstrict-aliasing -Wno-char-subscripts -Wno-missing-braces -DWX_PRECOMP -fno-guess-branch-probability -fno-dse -fno-tree-dse -fno-omit-frame-pointer -fno-strict-aliasing -Wno-parentheses -Wstrict-aliasing -Wno-char-subscripts -Wno-missing-braces -DWX_PRECOMP -DXDG_STD -o CMakeFiles/PCSX2.dir/gui/MemoryCardFolder.cpp.o -c /tmp/buildd/pcsx2-1.3.1-971-gf9e6a02+dfsg/pcsx2/gui/MemoryCardFolder.cpp
/tmp/buildd/pcsx2-1.3.1-971-gf9e6a02+dfsg/pcsx2/gui/MemoryCardFolder.cpp: In member function 'void FileAccessHelper::CloseFileHandle(wxFFile*, const MemoryCardFileEntry*)':
/tmp/buildd/pcsx2-1.3.1-971-gf9e6a02+dfsg/pcsx2/gui/MemoryCardFolder.cpp:1418:70: error: taking address of temporary [-fpermissive]
   fn.SetTimes( nullptr, &entry->entry.data.timeModified.ToWxDateTime(), &entry->entry.data.timeCreated.ToWxDateTime() );
                                                                      ^
/tmp/buildd/pcsx2-1.3.1-971-gf9e6a02+dfsg/pcsx2/gui/MemoryCardFolder.cpp:1418:117: error: taking address of temporary [-fpermissive]
   fn.SetTimes( nullptr, &entry->entry.data.timeModified.ToWxDateTime(), &entry->entry.data.timeCreated.ToWxDateTime() );
```


This looks like it might be subtracting 1 from x as well?


Same here, seems like it might be modifying x?


Corrigir endentação.


Looking good. May I suggest one more change? sys.path in Pythonista by default contains ".". While this is a good idea for the Python prompt, it is a bad idea for stash. For instance, if you run a wol.py without the sys.path modification in the stash/bin directory, it will not work because "import copy" (used in argparse) will pick up copy.py in bin instead of the python module copy. In stash, you should remove "." from sys.path, or move it to the end of path entries.
Georg


@coreation Thx :)


Should this be "Repeat property is ignored for event workspaces."?


It would be no worse than it currently is. When it's possible to just move the append, that's obviously better, since it doesn't do inefficient list ops.


why not os.path.join?


Matt, is the usage of "every?" here critical in the new "AND" style filter?


`__send__` bypasses visibility checks, in case a subclass wants to "hide" `immutable!` from it's public interface. Without it, the added spec will fail with `protected method 'immutable!' called` exception.

In this case the "spirit" of `protected` should allow `immutable!` to still be called here, since it "belongs" to the `Immutable` module.

Per the commit message, I recognise what I want to do here is pretty weird but this is the only change needed and it seemed pretty harmless.


This should be in optional_policy block.


yes i know isort


the above seems questionable to me


Matt, what is the best part about the refactoring of cell rendering?


Could you attach AVC related to this allow rule?

Thank you


This can be refactored to use a mapping

```python
model_map = {
    'registration': RegistrationApproval,
    'embargo': Embargo,
    # ...
}
Model = model_map.get(kind, None)
if not Model:
    raise UnsupportedSanctionHandlerKind
```


I'm pretty sure jsp is still null if it was null before?


The first argument to an `Exception` should be its error message.


I agree: /etc/rc\\.d/init\\.d/blkmapd


There will always be a session won't there? Just `req.session.token` should be fine.


Oh darn, I see now, @hsiliev fixed it on a subsequent commit, that's why I got the correct URL.


The powerpoint suggests using java.math.Random, rather than java.util.random. They act differntly


Merge branch 'release/1.0.0' into develop


@Cauldrath yeah I missed return statements and cause a memory leak.

I published a version to NPM.

Thank you for review.


Here is missing parameter (role)
You use 2 parameters in this interface.


Ooh yes - good catch. Although the x value would be added back on straight after. I'll change it to definitely only be affecting y though.


when i use wildcards and and/or connectors i have this Error: Invalid argument supplied for foreach()

this is my model user.php

public $hasAndBelongsToMany =

array(
'Competence' => array(
'with' => 'CompetencesUser',
'className' => 'Competence',
'joinTable' => 'competences_users',
'foreignKey' => 'user_id',
public $filterArgs = array(

'competences' => array(

        'type' => 'subquery',
        'method' => 'findByCompetences',
        'field' => 'User.id',

     'connectorsAnd' => '+',
     'connectorOr' => ',',
      'before' => true,
       'after' => false)

);
public function findByCompetences($data = array()) {

$this->CompetencesUser->Behaviors->attach('Containable', array( 'autoFields' => false ) );

$this->CompetencesUser->Behaviors->attach('Search.Searchable');
//$data = array('competences' => 'Cake');

$data = array('competences' => 'First%+Second%, Third%');

$result = $this->CompetencesUser->parseCriteria($data);

$expected = array(0 => array('OR' => array(
array('AND' => array(
array('Competence.competence LIKE' => '%First\%%'),
array('Competence.competence LIKE' => '%Second\%%'),
)),
array('AND' => array(
array('Competence.competence LIKE' => '%Third\%%')
)),
)));

$query = $this->CompetencesUser->getQuery('all', array(

'conditions' =>array(
$expected,
$result),

'fields' => array(
  'user_id'

),
'contain' => array(
'Competence'
)));

return $query;
}
plz any idea ;
in my query
SELECT User.id, User.username, User.nom, User.prenom, User.email, User.password, User.created, User.modified FROM dcsearch.users AS User WHERE User.id in (SELECT CompetencesUser.user_id FROM dcsearch.competences_users AS CompetencesUser LEFT JOIN dcsearch.competences AS Competence ON (CompetencesUser.competence_id = Competence.id) WHERE ((((Competence.competence LIKE '%First\%%') AND (Competence.competence LIKE '%Second\%%'))) OR (Competence.competence LIKE '%Third\%%'))) LIMIT 20


As above


The zabbix-client=0.1.0 package has an optional dependency on the requests>=2.2.0 package. So, you just should to pin a version of the zabbix-client.


Right.  We want a match only if every filter word is present in the source.


You didn't need an updated version of Py after all?


There is no blkmapd_var_run_t type directory to search and rules are expensive:

allow $1 blkmapd_var_run_t:file read_file_perms;


This function seems wrong in the first place. I think it's interpreting tileOrigin as the index of the tile (0, 1, 2 etc) rather than as the coordinate of the bottom left of the tile (0, 64, 128 etc). Let's either remove it (seems like it's not used much) or fix it to use the same meaning of tileOrigin as we use elsewhere (including the tileOrigin() method below).


I think it should be removed, because it misses its 'lib'-folder and classes.dex.
So this apk is damaged/incomplete. One who has a computer could probably confirm this by checking the signature files in META-INF and which files it there describes that *should* be in the archive


@ameenkhan07 PARTAYAYAYYYAYAYAYAYAYYAYA!!!! :tada:


BTW I offer to review slides etc to clean that easy to get wrong facts right. Feel free to ping me on the email address in my GH profile.


it's first time l use it.


Nice! Thanks!


My issue was that the render-cell function had too many parameters, especially since render-body-cell was calling a bunch of them with nil.  By switching to a map, we not only eliminate the need for nil parameters, but we get labels for each parameter.


I haven't looked at this branch, but previously, the filter used to be case sensitive. Is that still so?


It's been made case insensitive.

Hold up for a sec, I just thought of something that might be broken.


Thanks for the comments. Good suggestions. I think some more work to `flock` the file will also be necessary. I'll add a patch shortly.


Ok, I will add it.


This line caused me some serious headache.

````
propsToChange = {someProp: 100}
TweenMax(domEl, propsToChange)
````

propsToChange then becomes :
````
{someProp: 100, css: {someProp: 100}}
```

Another part of my app was modifying someProp (As below), expecting gsap to then render it differently, but gsap defaults to the css prop if its present, which contained the pre modified value that gsap added to it.

````
{someProp: 50, css: {someProp: 100}}   <--- gsap taking the css.someProp value

````

Inputs should be treated as immutable, modifying them is asking for problems.


You are creating a new instance of random  every time you go round this loop. It would be better to create an instance of Random outside of this loop and then call random.nextInt inside the loop.

I'm not entire sure how java.util.Random works, could nextInt() randomly return the same int as the previous call?


i think this has to go to TGRootViewController


@wesner0019 It is a hardware Serial so yes it should work for that.  However the system firmware may need to have it added to the WiFiCredentialsReader.cpp to transfer firmware YMODEM protocol.  If you could confirm this is what you want, would you please add an issue requesting it?  Thanks!


The `appengine` use case is very valid and if we allow to set the http client, we may be able to defer the `TimeoutDuration` to the client they provide.


Agreed with what Silent said.


nice


Some useful dockerfiles


Why not wait for the dial from `serve`?  Do we really not want to block in that case?


quick question: are you aware that n++ is different to ++n ? It doesn't matter in this case, but I just wanted to check that you were aware of that as the use of incrementing is a little inconsistent in this method.


Yes, that would be a better solution, I will look at it!


Fixed!  Only the Photon is reversed.


Ok sounds good, how do we make this explicit? I don't remember this when joining, as it clear to others when joining?


That will cause an unnecessary rebuild.


Да, в новой версии работает.


I'll move this in a separate commit as I don't trust git to not cause me any additional work when rebasing :)


I opened issue #92 because of this situation


there is no blkmapd_var_run_t to manage files in:

allow blkmapd_t blkmapd_var_run_t:file manage_file_perms;

Wonder why policy builds/loads so slow? it's also because of redundant rules that its needs to process.


I'm following what's here! https://github.com/gideonthomas/thimble.webmaker.org/commit/b1b1055028dec817dc46f83dc22e2256c9f1be80#diff-d7dfcb350720309421bbb465055aafdfR502
I think there won't be a session when you logout and redirect to `/` because it'll be set only after middleware is resolved? Not sure what the logic behind https://github.com/gideonthomas/thimble.webmaker.org/commit/b1b1055028dec817dc46f83dc22e2256c9f1be80#diff-d7dfcb350720309421bbb465055aafdfR502 is


yes they do go well together


:+1 - although we should have the next release be at least a minor bump so that I remember to adapt at https://github.com/mapbox/mapnik-vector-tile/blob/f254c4a02a55add203d50f5bd1e721d272019e64/test/vector_tile_pbf.cpp#L5 (where we're overriding the assert to silence it so that even in DEBUG mode we can ensure that exceptions are thrown in the right places).


PlantOperationSchemes (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2191 of 2236 tests passed)

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


How do you mean? What "same naming convention" ?

`spring-cloud-starter-xyz`; more info [here](https://github.com/spring-cloud/spring-cloud-starters)


This needed to be done because the SearchBase.java Interceptor class requires the "get" and "is" methods of *VO classes to be equal to the fieldname with the first letter after "get" and "is" set to lower case.

I will look into this because there maybe circumstances where you do not want this.

Example:

Method: getIPv4Address()
Wanted field: ipv4Address
Searchbase requires: iPv4Address


Checkmarx scan results 17 Aug 2015 18:01 UTC +03:00

High
Medium
Low
Info

New
2
1
103
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

3b7156b3a7c92a8537121cda42302dfda9bc8ece	@BearSpace


Okay, it's working, but I just added another commit to make the regex better.


Because memcad tests work better without it.


Javadoc!  I know we both know whats going on here, but its worth getting into a habbit.


I noticed that in mobile and tablet landscape views only, the dim screen overlay was not covering the entire screen and that about a quarter of the screen was not covered.

I know this fix came in after the due date, but I figured it would be best to fix this issue rather than leave it as is.


perhaps yes, I don't know really if we have to make from pack or initializer_list ?
However these ctor can be writen calling mk or sized_mk or is it inefficient ?


Layer should have an `organization` field.  For example:

Name: Landsat 8
Org: USGS


OMG, this is why I hate comments! xD


So shortly I would like to create my custom cloud started, so should I name it similary to boot
name-spring-cloud-starter?

Something like feign-vnderror-spring-cloud-starter.


minor but since i am on a roll: brace expansion is only useful if there is something expand:

files_pid_filetrans(blkmapd_t, blkmapd_var_run_t, file)


Isn't there already a stop condition on `i <= MAX_LEVEL` in the above for loop?


this changes the existing behavior, where it would still try to read from a non-writable cache


not 0.0.0.0?


Open question:  Can images in a layer be sourced from multiple providers, as for as v1 is concerned?  I suspect, yes, possibly.  We may need another provider/organization attribute here.


In this case, it has small impact on security, but @doverride  is right, there is no blkmapd_var_run_t directory.


@vilchik-elena I would have used a "if / else if" with tree.is(...) instead of a switch.


Make sure to handle if the user is not found.


I am not entirely sure of the use case for using this method with `$argv`, I am using it to get regular key bindings for the vi key bindings in insert mode. Which potentially conflict with this, but if this is not the main purpose this should probably also take the `$argv`. As well as the two lines bellow:

```fish
	# Ignore some known-bad control sequences
	# https://github.com/fish-shell/fish-shell/issues/1917
	bind \e\[I 'begin;end'
	bind \e\[O 'begin;end'
```

What do you think?


Same here; handle when the sanction is not found.


Looks good!
Thx for your contributing


Thanks, I'll pull it.


return next((o['id'] for o in question.field_data['options'] if o['option'] == option), None)


Just looking through old branches to see what needs to be kept. This looks interesting @mohitbhoite - do you know if it helped with stability on the CC3000?


Agree


dangling new line that I removed in this commit:
https://github.com/lucasdemarchi/ardupilot/commit/765390cbe4fd182ded523be80b3e6beaa7a2320e


Desse modo você ta pegando a collection mesmo que tenha dado erro na função ```startDatabase```. Joga essa chamada pra dentro do ```else``` que resolve o problema :)


Ah, ok. Sorry. That's what I would do yeah but you should probably ask the boot team.

ping @dsyer @spencergibb


Should be share/man/man1?


if NOT?


Looks good.


develop (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (749 of 749 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.21%25-green.svg)


How big is enormous?


update 'yer bundler :P


Olá, bom dia!
Gostaria de esclarecer uma dúvida. Quando o cliente sai da minha loja e é direcionado ao CheckOut Cielo para efetuar o pagamento, ele é direcionado para a página que eu configurei na minha URL de Retorno. Eu gostaria de saber como eu faço para recuperar os dados enviados pelo formulário. Exemplo:
Quando eu envio o nome do cliente, ele enviaria o "customer_name" em método POST, para eu recuperar, eu iria declarar a variável:
$customer_name = $_POST["customer_name"];

E para exibir a mesma, eu iria colocar o:

echo $customer_name;

Contudo, fazendo isso, retorna em branco, como se não estivesse enviando o parâmetro. Como fazer para recuperar essas variáveis enviadas pelo formulário do CheckOut Cielo?

Att,
Junior
E-mail: junior@dimarkdigital.com.br


PlantOperationSchemes (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


None of it, 100% hand made ;)

The OVF spec refers to the CIM here and there, which I have added selectively (by hand) in the file `cim.go`.


Hm, sounds reasonable.


could you use rst way to format code? http://docutils.sourceforge.net/docs/user/rst/quickstart.html#preformatting-code-samples


I probably should have created a config object here instead of calling fs.readFileSync.


sounds reasonable.


[MALICIOUS_CODE] : EI2: May expose internal representation by incorporating reference to mutable object (EI_EXPOSE_REP2)
This code stores a reference to an externally mutable object into the internal representation of the object.  If instances are accessed by untrusted code, and unchecked changes to the mutable object would compromise security or other important properties, you will need to do something different. Storing a copy of the object is better approach in many situations.


[PERFORMANCE] : UrF: Unread field (URF_UNREAD_FIELD)
This field is never read.  Consider removing it from the class.


Discussed in person: there is a [practical limit to URL lengths](http://www.boutell.com/newfaq/misc/urllength.html). You might need to reduce the size of these tokens. One way to do this is to encode tuples instead of dictionaries.


we removed the description


This introduces a bug... we need to release the lock **before** doing a read. I fixed it in a separate commit in my pr:
https://github.com/lucasdemarchi/ardupilot/commit/55272375dd5cb61f7e741c9c8a5d24097a4aac66


I agree that is has little direct impact on security. The above macro expands to two rules though. One for the directory and one for the actual file. That means there is one rule added to the policy that has no meaning. One rule is not a problem but if you consistently do this then this adds up and makes the policy unnecessarily bigger.

Besides it make the policy hard to read because it conflicts. Policy should be easy on the eye' s things should make sense. If there are no dirs then there should not be any rules implying there are dirs.


## Hello world.md


Cool, thanks!


This utility can be private... the only place is used outside testing is in PerReadAlleleLikelihoodsMap just before calling AlleleBiasedDownsamplingsUtils.selectAlleleBiasedReads; the returned value is passed as a parameter to the latter. That latter call also is passed the only argument for totalReads thus that call can be private within selectAlleleBiasedReads.

Also with the current approach selectAlleleBiasedReads is exposed to a potentially invalid totalReads estimate and is not checking for its validity.


You need no this delete &&  can remove variable `defaultValue` complete,
if you replace
```
defaultValue = refract::IElement::Create(LiteralTo<bool>(parameter.defaultValue));
```
by
```
     if (!parameter.defaultValue.empty()) {
            value->attributes["default"] = refract::IElement::Create(LiteralTo<bool>(parameter.defaultValue));;
        }
```


In general, we try to use interfaces/patterns. I'm OK with manage_files_pattern.

On the other hand, you are right with slow load/build, but it's too late to talk about slow build/load due to redundant rules in m4. We use CIL in rawhide, which is faster as you know.


While not completely necessary in this PR, it was noticed while running the CI tests for this PR and figured I would fix it so that way we could make sure that absolutely nothing was affected by the PR changes.


MALICIOUS_CODE : EI_EXPOSE_REP2


PERFORMANCE : URF_UNREAD_FIELD


This may be here in the case of older installs but ```/etc/atomic-enterprise``` and ```/etc/openshift``` won't exist any longer.


Checkmarx scan results 17 Aug 2015 18:13 UTC +03:00

High
Medium
Low
Info

New
2
1
103
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

3b7156b3a7c92a8537121cda42302dfda9bc8ece	@BearSpace


Not really. We need to do that all the types and it would be just duplicating code.


请在任务管理器终结 ss_privoxy 之后尝试最新版


Same as ```/etc/{atomic-enterprise,openshift}```. Only ```/etc/origin``` will be used moving forward.


Will I ever want to start controllers without starting their related help?


mixing indent fixes with comment changes is bad... I didn't add this change on the cherry-pick I did:
https://github.com/lucasdemarchi/ardupilot/commit/d28601067c18369f80b66742399b9dc3dba0bb27


@wesner0019 sorry about the regression, glad you are playing with fire in the `develop` branch though ;-)  I fixed this up and tested this morning https://github.com/spark/firmware/commit/94bcb08fd354745846c8cb72e941e3aba68afe73


Should we base this on `orgsync/base`?


```/etc/sysconfig/{origin,atomic-openshift}-{master,node}``` will be used moving forward.


i think this npc should restart it's path if it's reached it's homepositions

me->GetMotionMaster()->MovePath(me->GetDBTableGUIDLow() * 10, true);


gdfgdfg


Being picky here :stuck_out_tongue_closed_eyes:
As we have the `_INTERVALS` array above, what about adding an attribute `if_external_source:true/false` to each interval item and then write this as `var disabled = from_external_source && !interval.if_external_source;` (or something similar, naming is not my best skill)


Hello
On peut aussi demander à huma-Num ce qu'il y a comme outils de stats déjà prêts là bas :)


Name updated in metrics branch to clarify.


this reopens
https://github.com/springlobby/springlobby/issues/491

i guess fabs will do the trick? :-|

i'm at the point of writing our own abs function... wtf


0 gets turned into the default (in the future).  -1 means off.


I could see this command failing if the argument list is too long, especially since the filenames of uploads can be quite long. I've seen `xargs ` used to limit the number of arguments, although I'm not sure if this is present in all systems.


Because go requires you to match types to do arithmetic.


Shouldn't be there, I moved it to a new changeset :/


@juliaelman how about adding a mixin in addition to a variable? Just curious, where else in the site do we need a different box shadow for focus state?


waitFraction 0.75 results in waiting 75% of the lease, then beginning retries.


Just so you know, the only place this method appears to have been used was in GafferImageTest.ImagePlugTest, so I've removed it altogether.


another commit reverted it


Yes, I'll do that. I didn't like this var assignment here https://github.com/CartoDB/cartodb/commit/d508f8968a45cb9825764496e94656b6cfd42187#diff-720d90956f0c4ed69b601c5a7e067038R72


I don't want to do unbounded retries - I considered a rate limited, but was aiming for minimal outcome.  But since you can control via minimumRetryInterval that's fine to me.


Some of these interpolations are unnecessary, no? Prefer sites[0].page.slug to "#{sites[0].page.slug}"?


type=AVC msg=audit(1439379617.370:678): avc:  denied  { read } for  pid=5048 comm="blkmapd" name="log" dev="devtmpfs" ino=10298 scontext=system_u:system_r:blkmapd_t:s0 tcontext=system_u:object_r:devlog_t:s0 tclass=lnk_file permissive=1

type=AVC msg=audit(1439379617.370:680): avc:  denied  { write } for  pid=5048 comm="blkmapd" name="dev-log" dev="tmpfs" ino=10297 scontext=system_u:system_r:blkmapd_t:s0 tcontext=system_u:object_r:devlog_t:s0 tclass=sock_file permissive=1


I don't understand why your compiler doesn't throw warning on these.


You can define a top level variable that will hold all supported plugins like this:

    PLUGINS = {
        "external-zabbix-snmptrap": transfer_zabbix_snmptrap,
    ...
    }

and only one argument with a set of all supported plugins:

    parser.add_argument('plugins', choices=PLUGINS.keys(), nargs="+")

After that, the command will accept the next format:

    $ octane upgrade-plugins ORIG_ID SEED_ID plugin [plugin ...]

A default values is also accepted.


permission sets are there to be used as well, where it makes sense to use them.

it is never too late to write more efficient policy.


@maya thinking ahead. The mixin would allow the ability to pass in a color variable, yes (and possible a different blur/border values)


That is not unnecessary. The subslot is not in the VDB otherwise.


thanks. yeah, as long as OSD and the mon lead support it, we are good.


This project was fully described at https://hackaday.io/project/5588-hardware-password-manager


will do.


> waitFraction 0.75 results in waiting 75% of the lease, then beginning retries.

ok, I see it.