Hi @frans-ebu,
I can't merge it anymore, could you please fix it and push the change.

Thank you!


Are eggGroups even needed for Megas? They're just duplicate, unused information.


Hmmmm. Thats a good question.
I would not expect someone to to do so but who knows. =s

Lets wait to see if someone open an issue reporting it. Or we can just create the new class again inheriting from this one. What do you think?


I also appear to have made the virtual host line up with the existing documentation


Thanks for this.
is there a way to global catch this? instead of console.error. I was trying to use `ErrorUtils.setGlobalHandler()`


@jabelar, I sent you a message on Skype - it'd be great if you could respond. :D


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11303](https://teamcity.mesosphere.io/viewLog.html?buildId=103455&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:46


I am a nice comment


LGTM ^_^


Hrm, not sure why my editor is fussing over white space.


TeamCity Civic Assembly :: Build backend + frontend [Build 155](http://54.229.176.143/tc/viewLog.html?buildId=430&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


For some reason Skype suspended my account. I have no idea why since I only used it to chat with you guys. I'll try to set up a new username. You can email me at julian_abelar@hotmail.com in the meantime.

On Jan 12, 2016 8:31 AM, gegy1000 <notifications@github.com> wrote:

@jabelar<https://github.com/jabelar>, I sent you a message on Skype - it'd be great if you could respond. :D

—
Reply to this email directly or view it on GitHub<https://github.com/TimelessModdingTeam/JurassiCraft2/commit/a0b67c77aa4fd85d79b64b16025ba1fb584387a4#commitcomment-15401713>.


Assertions are considered unformatted text. In this case I think it's clear enough that it's a function; it's using lowerCamelCase and has an imperative name.


Method `getRepoConfig` doesn't always exists and causes a fatal error on my end


> and then remove the ServiceBase inheritance from the old AugmentationLoop class so that
> the registerServiceProvider call was made by ZMQAugmentorInterface since it would be
> the one and only inheriting from ServiceBase

Yep, that's what I would have done.

> Sadly removing the ServiceBase inheritance
> does not make much sense since all those classes make use of the methods record* from
> the EventRecorder class that is inherited by ServiceBase

Well that sucks. Even though the AugmentationLoop could just inherit from EventRecorder and not ServiceBase, I think we can keep ServiceBase as a base class. It won't have any effect.

> So all came down to moving the registerServiceProvider call from the AugmentationLoop::init() method to the
> ZMQAugmentorInterface::init() method.

Yes that's perfect. You can also move the `ZmqNamedClientBus toAugmentors` declared in the header file in the ZmqAugmentorInterface as it should not be needed anymore in the AugmentationLoop.


Please don't accept options that are not implemented. This makes it unnecessarily hard to figure out what still needs to be implemented.


I think you can put QtObjects into the data slot of a Item.  That is a reparenting usecase.  The other one is the one in tests/test-paths.qml:line 33 above.  The assignment from the `QtObject` to property `obj` is reparenting object.  Maybe your change update cycle make this to a non-issue?


I won't drop PHP 5.3 if it's just for short array syntax ^^
Could you please comment on #18? As I'm force-pushing I'm going to lose track of commits.


Is there still a busy spinning case when m_messageQueue is empty and
ALLOW_AUDIO(m_speed) && !m_stalled && m_syncState != IDVDStreamPlayer::SYNC_INSYNC

We will not set m_stalled or sleep, but we'll still read from the m_messageQueue with timeout=0?
Is that combination possible?


I am working on a website. plz pull me the building blocks.


Well, but you don't inform him anywhere about the output. I mean, it should at least get some longer explanations in a comment in the sources next to the log lines really, so that people who can read the sources can figure this out at least!

(But I'd still prefer if we'd also log in the output what you see, as I suggested above)


You don't seem to use the regex crate?


why not opts.usage()?


yuuhuu


@djh- If you're changing the content in the HTML, all of the images need to be updated as well. I don't have the time to do at the moment unfortunately. Can it stay as "CV" at least for the HTML ?


TeamCity pull requests :: netty [Build 1017](https://garage.netty.io/teamcity/viewLog.html?buildId=4700&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6370, ignored: 23 Build time: 00:42:50


Please drop the `feature_` out of the method names and then this will be perfect.

`Feature.nv_overlay_enable unless`


@HaoH   Yeah, on my OS X just make it.

Python 2.7.10 + Java 1.7.80, and set HostName.

it worked.


I can't find anywhere where this is being used.  Is this function obsolete?


Updated extractor to use shortest names only for RU.
For other languages skills-override.tsv should be more simpler way to get rid of "bound" "veteran" "federation" and other trash in names.


Hey,

super idea. Yes, I was thinking to do it, also I just forgot about it.

Regarding the naming, what is a good prefix? You have choosen simpletr64_ isnt tr64_ or upnp_ a better one? Or do you think the package name is essential?

Cheers,
Ben


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1215.8328e61010d48ded59f788a0603ec8faf37fd3ff](http://build.palaso.org/viewLog.html?buildId=67868&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 824, ignored: 7 Build time: 00:19:52


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11304](https://teamcity.mesosphere.io/viewLog.html?buildId=103466&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:25


Why you're bumping the minor version? There's some change who breaks the existent interface?.


Please, revert the repository owner.


Fixed.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11305](https://teamcity.mesosphere.io/viewLog.html?buildId=103478&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:03:58


I have fixed it up manually in the vendor,so not a prob


That's why I mentioned this on old bug tracker, inconsistency of constants. xD


Hey, @diegomarcon

Any specific reason to change this to `mediaControlButton`?

I'm here totally without context but it seems like this new name is a bit generic, since we can have more than one button inside the media control area.


pull request for #9


> Does Borland have a problem with MIN_KEYLENGTH > 0? My apologies if we already talked about it.

@noloader yes the compiler (bcc32) is reporting:
```[bcc32 Error] seckey.h(194): E2313 Constant expression required```

I'm guessing that the compiler as a problem with this MACRO definition:
```cpp
    //! \brief The minimum key length used by the algorithm provided as a constant
    //! \details MIN_KEYLENGTH is provided in bytes, not bits
    CRYPTOPP_CONSTANT(MIN_KEYLENGTH=N)
```
The other solution that seems to work would be to change this in *config.h*:
```diff
diff --git a/config.h b/config.h
index 8a26dd0..db59454 100644
--- a/config.h
+++ b/config.h
@@ -449,7 +449,7 @@ NAMESPACE_END
 #endif

 // how to declare class constants
-#if (defined(_MSC_VER) && _MSC_VER <= 1300) || defined(__INTEL_COMPILER)
+#if (defined(_MSC_VER) && _MSC_VER <= 1300) || defined(__INTEL_COMPILER) || defined(__BORLANDC__)
 #	define CRYPTOPP_CONSTANT(x) enum {x};
 #else
 #	define CRYPTOPP_CONSTANT(x) static const int x;
```


111411352_ATDeprecateFields (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (954 of 954 tests passed, 0 test warnings)

![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-26.73%25-green.svg)


fixed: extracted as method


is this file committed intentionally? the file it references (libgpg.so) was removed in a previous commit..


@jabelar, I sent you an email :)


I don't know that it ever was being used. I'll remove it.


Is `username` the username from the `$path` ?


OK, this looks very good:

```
 // how to declare class constants
-#if (defined(_MSC_VER) && _MSC_VER <= 1300) || defined(__INTEL_COMPILER)
+#if (defined(_MSC_VER) && _MSC_VER <= 1300) || defined(__INTEL_COMPILER) || defined(__BORLANDC__)
 #  define CRYPTOPP_CONSTANT(x) enum {x};
 #else
 #  define CRYPTOPP_CONSTANT(x) static const int x;
```

Can you confirm it works on old and new Borland? There's nothing worse than breaking something that used to work...


Why do you need callback, this action is **synchronous**.

I mean you can simply do the following (pseudo-code):
```
adguardSelector.selectElement(node);
changeSliderPosition();
```


Ah, right, this is conditionalized on the source URL. :+1:


Merged to gh-pages


@unfulvio yeah I'm fine with either the global or singleton pattern for these


> This is too unclear!

What exactly is unclear? You pass the DOM node to be "selected" by selector lib, seems pretty easy to understand.


TeamCity Civic Assembly :: Build backend + frontend [Build 155](http://54.229.176.143/tc/viewLog.html?buildId=430&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 19 (6 new), passed: 1523, ignored: 12 Build time: 00:18:44
### Failed tests
```
Document type tests: Should not create new document type if cancelled: <no details avaliable>

Document type tests: Should add new document type: <no details avaliable>

Document type tests: Should not change document type: <no details avaliable>

Document type tests: Should change document type: <no details avaliable>

Document type tests: Should not delete document type if cancel pressed: <no details avaliable>

Document type tests: Should delete document type: <no details avaliable>

Media type tests: Should add new media type: <no details avaliable>

Media type tests: Should not change media type: <no details avaliable>

Media type tests: Should change media type: <no details avaliable>

Media type tests: Should not delete media type if cancel pressed: <no details avaliable>

Media type tests: Should delete media type: <no details avaliable>

##### there are 9 more failed tests, see build details
```


This should be just path, right?


I can cherry pick it into 3.4.3


Hey, @barbosa.

The idea was to have a enum with media control states (Playing, Paused, Stopped) and reflect this state on the button instead of creating a new 'PlayStopButton' or something like that when the media is a live streaming.

I also did NOT (:stuck_out_tongue: ) like the name but it's something that I thought we could improve once this branch evolves.

Anyway, if you have any suggestion about the name or even about the implementation please let me know.


:+1:, In Rails 4 the `avatar_url` was nil, but asset_url did not raise error for `nil` source in Rails 4.

It was added in https://github.com/rails/rails/pull/20669


removed


What's `231`?


有人反馈用FF在beta版的帖子頁面無法向下拉
![image](https://cloud.githubusercontent.com/assets/1846356/12270281/38d54c74-b990-11e5-8694-a9b325c2da2a.png)


Fred Nicolson is a lie


:+1:

Let's use un-minified versions in the future though.


> why aren't they installed by Pkg.build?

iirc they are there only for testing the output of generated images, and are not a requirement of Compose


Wrong commit. Working on making it right.


This is not needed for pdf and ebook generation, so I guess you found out issues on git-scm.com website.

This is a known issue with the deployment of the book's files to the web site.


Seems you are interested in contributing to this tool. I think we made significant contributions to the tool since your last contribution.  It would be nice to see you contributing this tool again.


Similar to my comment on grunt tasks, my conclusion is that the use of grunt-shell is a more appropriate pattern than breaking off the logic into child_process exec calls. The grunt-shell plugin is expressly designed to wrap child_process.exec in a way to define declarative feedback patterns and handle the async queueing of tasks. I think the key in Grunt is to think of queued tasks rather than imperative function flow. I don't think a switch from the use of grunt-shell is a better approach.


Me being an idiot. Fixed.


@om-nishu-trantor could you remove `(not SASS)`
it is breaking pattern `CodeFormatter: Format as <syntax name> Syntax`


Quick and clean fix. Neat...

Thanks for sharing this awesome library at all, it's a great piece of work.


Бля, зафейлил я миграции


Verified and it crashes on a C1 without nrpacks=1
Agree, I will add the line to the C1 config script


Hmmm, maybe not.  python_cmd can do things other than deal with syms.  I think this will fail *many* tests.


ben de evde biraz tekrar yapayım dedim. Scanner sınıfı hakkında okurken her defasında girdileri okumak için yeni bir Scanner objesi yaratmak yerine bunu en başta bir defa yaratıp program süresince kullanmanın ve bitiminde kapatmanın (.close()) daha doğru olduğunu öğrendim. Aynı mantıkla senin kodunda ufak tefek değişiklikler yaptım. Paylaşmak istedim. Aşağıdaki linkten görebilirsin, selamlar:

https://github.com/patateskafa/pattiz/blob/master/src/com/alp/alistirma/ev/EvDeneme.java


Fix #22


@mikegee I accidentally pushed to master.  You probably want to scrap this.


removed


Just a hint, line 93 till 102 don't need to be hashed out on sonar-desktop-settings package at all. Hashing it out in line 9 is totally fine and the only thing needed not to package that package yet.


i like!


I think we need to check if the queue is empty before we just transmit the incoming message.  Otherwise don't we run risk of transmitting messages out of order?


> Styles or function?

Function to draw the selection borders.

By "style" here I don't mean CSS. I mean how this selection looks to user. As I understand, currently selection is represented by four DOM nodes ("borders") with position:absolute.


You mean hide?


thanks, I pushed a new version. I have no clue for what reason the message query is done in Decode(). Does not make much sense to me. Returning return codes of msg.Get to the calling method of Decode make even less sense. I moved the timeout handling closer to msg.Get. Now is is more visible what happens.


Sorry about that. I'll revert the edits with issues.


tested method (createMailboxSession) has been moved from AuthenticationFilter to AuthenticationStrategy interface.
Also, the method do not throw anymore (now returns Optional)


Yes, its left over cruft comments, something like that.
I made one test build with sonar-common, not sure we need these two files at all. ;)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11306](https://teamcity.mesosphere.io/viewLog.html?buildId=103489&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:25


okis, removed.


Shouldn't this have more explanation


:+1: This might be from Rack but not sure.


Re-ordered to make it obvious that you can force params anywhere, not just at end.


ugh. I'm awful. Ok. I'll make a diligent effort to be the example of good commit messages going forward. I appreciate your reminders.


From what I can see in CodeSite, if you "copy as override" the INFO, the group for the children of DIAL is copied before/without the DIAL being copied first. If you "C A O" the DIAL it arrives properly as a children of QUST.  But if you then copy an INFO from that DIAL, it is added to a newly created group "Children of nnXXXXXX" directly under the QUST group and not inside the QUST record.


Picky picky. :)  Fixed.


On both O3ds/N3ds?


Offline commands have never behaved as expected. I tried supporting them originally, but I think there is a logic bug somewhere.


Apparantly you can't specify repository as a requirement when using install_requires. Let's not require GeoKey for now.


Note that if you write in es6 u dont need the `temp`

```
var x = 1;
var y = 100;

[ x, y ] = [ y, x ];

console.log( x ) // 100
```


@filfat why did you revert to the previous version of app.js?


no needc


yea


Would:
```
if (timeout == 0)
  Sleep(10);
```
be more explicit that we are sleeping when queue is empty and we are retrying without a timeout?


You're pre-1.0, so you don't need to maintain class-compatibility. I just raised it to see if you have any users you need to watch out for. Maybe bump the minor number instead of the patch?


@lorenzofar oups that were accidental. Will fix now.


((( idk how 2 add this


This was a BREAKING CHANGE!

Because in [hapi@12](https://github.com/hapijs/hapi/issues/2985) the querystring parsing [was removed](https://github.com/hapijs/hapi/issues/2983). This is used in kazana-raw-data attachment handlers, where we use querystring bracket notation like `scope[id]` to encode nested properties.


can probably be done in a @BeforeClass


@filfat Ok :)


Very nice. We need more patches with such good scaling.


Did you get the values somewhere or are you just trying to simplify?


assertThatThrownBy


Default value for `klass` is `None` and then this check assigns `logging.StreamHandler`. This can be shortcut with:

```
def add_log_handler(logger, klass=logging.StreamHandler, stream=sys.stdout, level='Info', fmt='%(name)s: %(message)s'):
```

This is preferable both for succinctness but also because if shows up in the docs.


We now need to add the [hapi-qs plugin](https://github.com/daf-spr/hapi-qs)


#sign-off


Ah you modified app.js and not index.ts :)


You might want to remove this section 😊


[R/exa.createScript.R:101:1:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L101) *style:* **Variable and function names should be all lowercase.**
```r
exa.createScript <- function(channel, name, func = NA,
^~~~~~~~~~~~~~~~
```
[R/exa.createScript.R:103:30:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L103) *style:* **Variable and function names should be all lowercase.**
```r
                             initCode = NA,
                             ^~~~~~~~
```
[R/exa.createScript.R:104:30:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L104) *style:* **Variable and function names should be all lowercase.**
```r
                             cleanCode = NA,
                             ^~~~~~~~~
```
[R/exa.createScript.R:105:30:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L105) *style:* **Variable and function names should be all lowercase.**
```r
                             inType = SET,
                             ^~~~~~
```
[R/exa.createScript.R:106:30:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L106) *style:* **Variable and function names should be all lowercase.**
```r
                             inArgs = list(),
                             ^~~~~~
```
[R/exa.createScript.R:107:30:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L107) *style:* **Variable and function names should be all lowercase.**
```r
                             outType = EMITS,
                             ^~~~~~~
```
[R/exa.createScript.R:108:30:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L108) *style:* **Variable and function names should be all lowercase.**
```r
                             outArgs = list(),
                             ^~~~~~~
```
[R/exa.createScript.R:109:30:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L109) *style:* **Variable and function names should be all lowercase.**
```r
                             outputAddress = NA,
                             ^~~~~~~~~~~~~
```
[R/exa.createScript.R:110:30:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L110) *style:* **Variable and function names should be all lowercase.**
```r
                             replaceIfExists = TRUE,
                             ^~~~~~~~~~~~~~~
```
[R/exa.createScript.R:111:30:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L111) *style:* **Variable and function names should be all lowercase.**
```r
                             mockOnly = FALSE
                             ^~~~~~~~
```
[R/exa.createScript.R:115:3:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L115) *style:* **Variable and function names should be all lowercase.**
```r
  initCode <- m$initCode
  ^~~~~~~~
```
[R/exa.createScript.R:115:17:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L115) *style:* **Variable and function names should be all lowercase.**
```r
  initCode <- m$initCode
                ^~~~~~~~
```
[R/exa.createScript.R:116:3:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L116) *style:* **Variable and function names should be all lowercase.**
```r
  cleanCode <- m$cleanCode
  ^~~~~~~~~
```
[R/exa.createScript.R:116:18:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L116) *style:* **Variable and function names should be all lowercase.**
```r
  cleanCode <- m$cleanCode
                 ^~~~~~~~~
```
[R/exa.createScript.R:118:3:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L118) *style:* **Variable and function names should be all lowercase.**
```r
  inType <- match.arg(inType, ALLOWED_UDF_IN_TYPES)
  ^~~~~~
```
[R/exa.createScript.R:118:23:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L118) *style:* **Variable and function names should be all lowercase.**
```r
  inType <- match.arg(inType, ALLOWED_UDF_IN_TYPES)
                      ^~~~~~
```
[R/exa.createScript.R:118:31:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L118) *warning:* **no visible binding for global variable ‘ALLOWED_UDF_IN_TYPES’**
```r
  inType <- match.arg(inType, ALLOWED_UDF_IN_TYPES)
                              ^~~~~~~~~~~~~~~~~~~~
```
[R/exa.createScript.R:119:3:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L119) *style:* **Variable and function names should be all lowercase.**
```r
  outType <- match.arg(outType, ALLOWED_UDF_OUT_TYPES)
  ^~~~~~~
```
[R/exa.createScript.R:119:24:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L119) *style:* **Variable and function names should be all lowercase.**
```r
  outType <- match.arg(outType, ALLOWED_UDF_OUT_TYPES)
                       ^~~~~~~
```
[R/exa.createScript.R:119:33:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L119) *warning:* **no visible binding for global variable ‘ALLOWED_UDF_OUT_TYPES’**
```r
  outType <- match.arg(outType, ALLOWED_UDF_OUT_TYPES)
                                ^~~~~~~~~~~~~~~~~~~~~
```
[R/exa.createScript.R:121:3:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L121) *style:* **Variable and function names should be all lowercase.**
```r
  inArgs <- do.call(paste, c(as.list(inArgs), sep = ", "))
  ^~~~~~
```
[R/exa.createScript.R:121:38:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L121) *style:* **Variable and function names should be all lowercase.**
```r
  inArgs <- do.call(paste, c(as.list(inArgs), sep = ", "))
                                     ^~~~~~
```
[R/exa.createScript.R:123:7:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L123) *style:* **Variable and function names should be all lowercase.**
```r
  if (outType == EMITS) {
      ^~~~~~~
```
[R/exa.createScript.R:124:19:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L124) *style:* **Variable and function names should be all lowercase.**
```r
    if (is.null(m$outArgs)) {
                  ^~~~~~~
```
[R/exa.createScript.R:127:5:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L127) *style:* **Variable and function names should be all lowercase.**
```r
    outArgs <- paste("(",
    ^~~~~~~
```
[R/exa.createScript.R:128:47:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L128) *style:* **Variable and function names should be all lowercase.**
```r
                     do.call(paste, c(as.list(outArgs), sep = ", ")),
                                              ^~~~~~~
```
[R/exa.createScript.R:131:5:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L131) *style:* **Variable and function names should be all lowercase.**
```r
    outType <- RETURNS
    ^~~~~~~
```
[R/exa.createScript.R:131:16:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L131) *warning:* **no visible binding for global variable ‘RETURNS’**
```r
    outType <- RETURNS
               ^~~~~~~
```
[R/exa.createScript.R:132:5:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L132) *style:* **Variable and function names should be all lowercase.**
```r
    outArgs <- as.character(outArgs)
    ^~~~~~~
```
[R/exa.createScript.R:132:29:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L132) *style:* **Variable and function names should be all lowercase.**
```r
    outArgs <- as.character(outArgs)
                            ^~~~~~~
```
[R/exa.createScript.R:135:30:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L135) *style:* **Variable and function names should be all lowercase.**
```r
  sql <- paste("CREATE", if (replaceIfExists) "OR REPLACE" else "", "R",
                             ^~~~~~~~~~~~~~~
```
[R/exa.createScript.R:136:16:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L136) *style:* **Variable and function names should be all lowercase.**
```r
               inType, "SCRIPT", name,
               ^~~~~~
```
[R/exa.createScript.R:137:21:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L137) *style:* **Variable and function names should be all lowercase.**
```r
               "(", inArgs, ")",
                    ^~~~~~
```
[R/exa.createScript.R:138:16:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L138) *style:* **Variable and function names should be all lowercase.**
```r
               outType, outArgs, "AS")
               ^~~~~~~
```
[R/exa.createScript.R:138:25:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L138) *style:* **Variable and function names should be all lowercase.**
```r
               outType, outArgs, "AS")
                        ^~~~~~~
```
[R/exa.createScript.R:140:18:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L140) *style:* **Variable and function names should be all lowercase.**
```r
  if (!is.null(m$outputAddress))
                 ^~~~~~~~~~~~~
```
[R/exa.createScript.R:144:24:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L144) *style:* **Variable and function names should be all lowercase.**
```r
                       outputAddress[[1]], "', ", outputAddress[[2]], ")",
                       ^~~~~~~~~~~~~
```
[R/exa.createScript.R:144:51:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L144) *style:* **Variable and function names should be all lowercase.**
```r
                       outputAddress[[1]], "', ", outputAddress[[2]], ")",
                                                  ^~~~~~~~~~~~~
```
[R/exa.createScript.R:157:16:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L157) *style:* **Variable and function names should be all lowercase.**
```r
  if (!is.null(initCode)) {
               ^~~~~~~~
```
[R/exa.createScript.R:159:10:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L159) *style:* **Variable and function names should be all lowercase.**
```r
    for (codeLine in deparse(initCode)) {
         ^~~~~~~~
```
[R/exa.createScript.R:159:30:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L159) *style:* **Variable and function names should be all lowercase.**
```r
    for (codeLine in deparse(initCode)) {
                             ^~~~~~~~
```
[R/exa.createScript.R:160:25:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L160) *style:* **Variable and function names should be all lowercase.**
```r
      sql <- paste(sql, codeLine, sep = "\n")
                        ^~~~~~~~
```
[R/exa.createScript.R:165:16:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L165) *style:* **Variable and function names should be all lowercase.**
```r
  if (!is.null(cleanCode)) {
               ^~~~~~~~~
```
[R/exa.createScript.R:167:10:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L167) *style:* **Variable and function names should be all lowercase.**
```r
    for (codeLine in deparse(cleanCode)) {
         ^~~~~~~~
```
[R/exa.createScript.R:167:30:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L167) *style:* **Variable and function names should be all lowercase.**
```r
    for (codeLine in deparse(cleanCode)) {
                             ^~~~~~~~~
```
[R/exa.createScript.R:168:25:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L168) *style:* **Variable and function names should be all lowercase.**
```r
      sql <- paste(sql, codeLine, sep = "\n")
                        ^~~~~~~~
```
[R/exa.createScript.R:173:8:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L173) *style:* **Variable and function names should be all lowercase.**
```r
  for (codeLine in deparse(code)) {
       ^~~~~~~~
```
[R/exa.createScript.R:174:23:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L174) *style:* **Variable and function names should be all lowercase.**
```r
    sql <- paste(sql, codeLine, sep = "\n")
                      ^~~~~~~~
```
[R/exa.createScript.R:178:8:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L178) *style:* **Variable and function names should be all lowercase.**
```r
  if (!mockOnly) {
       ^~~~~~~~
```
[R/exa.createScript.R:179:9:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L179) *style:* **Variable and function names should be all lowercase.**
```r
    if (odbcQuery(channel, sql) == -1) {
        ^~~~~~~~~
```
[R/exa.createScript.R:180:12:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L180) *style:* **Variable and function names should be all lowercase.**
```r
      stop(odbcGetErrMsg(channel)[[1]])
           ^~~~~~~~~~~~~
```
[R/exa.createScript.R:180:12:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L180) *warning:* **no visible global function definition for ‘odbcGetErrMsg’**
```r
      stop(odbcGetErrMsg(channel)[[1]])
           ^~~~~~~~~~~~~
```
[R/exa.createScript.R:185:36:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L185) *style:* **Variable and function names should be all lowercase.**
```r
  function(..., table, where = "", groupBy = "", restQuery = "",
                                   ^~~~~~~
```
[R/exa.createScript.R:185:50:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L185) *style:* **Variable and function names should be all lowercase.**
```r
  function(..., table, where = "", groupBy = "", restQuery = "",
                                                 ^~~~~~~~~
```
[R/exa.createScript.R:186:12:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L186) *style:* **Variable and function names should be all lowercase.**
```r
           returnSQL = FALSE, reader = NA, server = NA) {
           ^~~~~~~~~
```
[R/exa.createScript.R:189:5:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L189) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(is.character(table))
    ^~~~~~~~~~~
```
[R/exa.createScript.R:190:5:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L190) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(length(where) == 1)
    ^~~~~~~~~~~
```
[R/exa.createScript.R:192:5:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L192) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(is.character(where))
    ^~~~~~~~~~~
```
[R/exa.createScript.R:193:5:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L193) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(length(where) == 1)
    ^~~~~~~~~~~
```
[R/exa.createScript.R:195:5:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L195) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(is.character(groupBy))
    ^~~~~~~~~~~
```
[R/exa.createScript.R:195:30:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L195) *style:* **Variable and function names should be all lowercase.**
```r
    assert_that(is.character(groupBy))
                             ^~~~~~~
```
[R/exa.createScript.R:196:5:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L196) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(length(groupBy) == 1)
    ^~~~~~~~~~~
```
[R/exa.createScript.R:196:24:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L196) *style:* **Variable and function names should be all lowercase.**
```r
    assert_that(length(groupBy) == 1)
                       ^~~~~~~
```
[R/exa.createScript.R:198:5:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L198) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(is.character(restQuery))
    ^~~~~~~~~~~
```
[R/exa.createScript.R:198:30:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L198) *style:* **Variable and function names should be all lowercase.**
```r
    assert_that(is.character(restQuery))
                             ^~~~~~~~~
```
[R/exa.createScript.R:199:5:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L199) *warning:* **no visible global function definition for ‘assert_that’**
```r
    assert_that(length(restQuery) == 1)
    ^~~~~~~~~~~
```
[R/exa.createScript.R:199:24:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L199) *style:* **Variable and function names should be all lowercase.**
```r
    assert_that(length(restQuery) == 1)
                       ^~~~~~~~~
```
[R/exa.createScript.R:213:9:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L213) *style:* **Variable and function names should be all lowercase.**
```r
    if (groupBy != "") {
        ^~~~~~~
```
[R/exa.createScript.R:214:7:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L214) *style:* **Variable and function names should be all lowercase.**
```r
      groupBy <- paste(" GROUP BY", groupBy, sep = " ")
      ^~~~~~~
```
[R/exa.createScript.R:214:37:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L214) *style:* **Variable and function names should be all lowercase.**
```r
      groupBy <- paste(" GROUP BY", groupBy, sep = " ")
                                    ^~~~~~~
```
[R/exa.createScript.R:218:32:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L218) *style:* **Variable and function names should be all lowercase.**
```r
                 table, where, groupBy, sep = "")
                               ^~~~~~~
```
[R/exa.createScript.R:219:47:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L219) *style:* **Variable and function names should be all lowercase.**
```r
    sql <- paste("SELECT * FROM (", sql, ")", restQuery, sep = "")
                                              ^~~~~~~~~
```
[R/exa.createScript.R:221:9:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L221) *style:* **Variable and function names should be all lowercase.**
```r
    if (returnSQL) {
        ^~~~~~~~~
```
[R/exa.createScript.R:224:7:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L224) *style:* **Variable and function names should be all lowercase.**
```r
      execArgs <- list(channel, sql)
      ^~~~~~~~
```
[R/exa.createScript.R:226:9:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L226) *style:* **Variable and function names should be all lowercase.**
```r
        execArgs <- c(execArgs, reader = reader)
        ^~~~~~~~
```
[R/exa.createScript.R:226:23:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L226) *style:* **Variable and function names should be all lowercase.**
```r
        execArgs <- c(execArgs, reader = reader)
                      ^~~~~~~~
```
[R/exa.createScript.R:229:9:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L229) *style:* **Variable and function names should be all lowercase.**
```r
        execArgs <- c(execArgs, server = server)
        ^~~~~~~~
```
[R/exa.createScript.R:229:23:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L229) *style:* **Variable and function names should be all lowercase.**
```r
        execArgs <- c(execArgs, server = server)
                      ^~~~~~~~
```
[R/exa.createScript.R:231:12:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L231) *style:* **Variable and function names should be all lowercase.**
```r
      if (!mockOnly) {
           ^~~~~~~~
```
[R/exa.createScript.R:232:17:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L232) *style:* **Variable and function names should be all lowercase.**
```r
        do.call(exa.readData, execArgs)
                ^~~~~~~~~~~~
```
[R/exa.createScript.R:232:17:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L232) *warning:* **no visible binding for global variable ‘exa.readData’**
```r
        do.call(exa.readData, execArgs)
                ^~~~~~~~~~~~
```
[R/exa.createScript.R:232:31:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/exa.createScript.R#L232) *style:* **Variable and function names should be all lowercase.**
```r
        do.call(exa.readData, execArgs)
                              ^~~~~~~~
```
[R/EXADBI.R:6:2:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L6) *style:* **Commented code should be removed.**
```r
## First version written in 2015 by Marcel Boldt <marcel.boldt@exasol.com>
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:7:2:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L7) *style:* **Commented code should be removed.**
```r
## as part of the EXASOL R interface & SDK package. It may be used, changed and distributed freely with no further restrictions than
 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:58:25:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L58) *style:* **Variable and function names should be all lowercase.**
```r
  definition = function(dbObj) {
                        ^~~~~
```
[R/EXADBI.R:83:25:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L83) *style:* **Variable and function names should be all lowercase.**
```r
  definition = function(dbObj) {
                        ^~~~~
```
[R/EXADBI.R:97:44:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L97) *style:* **Commas should always have a space after.**
```r
    NextMethod(generic = "summary", object,...)
                                           ^
```
[R/EXADBI.R:146:15:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L146) *style:* **Commas should always have a space after.**
```r
  "dbGetInfo","EXAConnection",
              ^
```
[R/EXADBI.R:147:25:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L147) *style:* **Variable and function names should be all lowercase.**
```r
  definition = function(dbObj) {
                        ^~~~~
```
[R/EXADBI.R:148:10:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L148) *style:* **Variable and function names should be all lowercase.**
```r
    if (!dbIsValid(dbObj)) {
         ^~~~~~~~~
```
[R/EXADBI.R:148:20:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L148) *style:* **Variable and function names should be all lowercase.**
```r
    if (!dbIsValid(dbObj)) {
                   ^~~~~
```
[R/EXADBI.R:152:26:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L152) *style:* **Variable and function names should be all lowercase.**
```r
      db.version = paste(dbObj@db_prod_name, dbObj@db_version),
                         ^~~~~
```
[R/EXADBI.R:152:46:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L152) *style:* **Variable and function names should be all lowercase.**
```r
      db.version = paste(dbObj@db_prod_name, dbObj@db_version),
                                             ^~~~~
```
[R/EXADBI.R:153:16:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L153) *style:* **Variable and function names should be all lowercase.**
```r
      dbname = dbObj@db_name,
               ^~~~~
```
[R/EXADBI.R:154:18:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L154) *style:* **Variable and function names should be all lowercase.**
```r
      username = dbObj@db_user,
                 ^~~~~
```
[R/EXADBI.R:155:14:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L155) *style:* **Variable and function names should be all lowercase.**
```r
      host = dbObj@db_host,
             ^~~~~
```
[R/EXADBI.R:156:14:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L156) *style:* **Variable and function names should be all lowercase.**
```r
      port = dbObj@db_port
             ^~~~~
```
[R/EXADBI.R:165:44:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L165) *style:* **Commas should always have a space after.**
```r
    NextMethod(generic = "summary", object,...)
                                           ^
```
[R/EXADBI.R:189:47:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L189) *style:* **Put spaces around all infix operators.**
```r
                         fields = c(connection="EXAConnection",
                                             ~^~
```
[R/EXADBI.R:190:42:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L190) *style:* **Put spaces around all infix operators.**
```r
                                statement="character",
                                        ~^~
```
[R/EXADBI.R:191:45:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L191) *style:* **Put spaces around all infix operators.**
```r
                                rows_fetched="numeric",
                                           ~^~
```
[R/EXADBI.R:192:46:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L192) *style:* **Put spaces around all infix operators.**
```r
                                rows_affected="numeric",
                                            ~^~
```
[R/EXADBI.R:193:44:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L193) *style:* **Put spaces around all infix operators.**
```r
                                is_complete="logical",
                                          ~^~
```
[R/EXADBI.R:194:44:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L194) *style:* **Put spaces around all infix operators.**
```r
                                with_output="logical",
                                          ~^~
```
[R/EXADBI.R:195:40:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L195) *style:* **Put spaces around all infix operators.**
```r
                                profile="data.frame",
                                      ~^~
```
[R/EXADBI.R:196:40:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L196) *style:* **Put spaces around all infix operators.**
```r
                                columns="data.frame",
                                      ~^~
```
[R/EXADBI.R:197:48:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L197) *style:* **Put spaces around all infix operators.**
```r
                                temp_result_tbl="character",
                                              ~^~
```
[R/EXADBI.R:198:48:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L198) *style:* **Put spaces around all infix operators.**
```r
                                query_sent_time="POSIXct",
                                              ~^~
```
[R/EXADBI.R:199:39:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L199) *style:* **Put spaces around all infix operators.**
```r
                                errors="character",
                                     ~^~
```
[R/EXADBI.R:200:50:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L200) *style:* **Put spaces around all infix operators.**
```r
                                default_fetch_rec="numeric"
                                                ~^~
```
[R/EXADBI.R:210:1:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L210) *style:* **lines should not be more than 80 characters.**
```r
            "Add an int (the length of a newly fetched result set) to rows_fetched."
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:214:1:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L214) *style:* **lines should not be more than 80 characters.**
```r
#         close = function(commit=TRUE) {                              # dbClearResult is in row 880
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:215:1:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L215) *style:* **lines should not be more than 80 characters.**
```r
#           "Frees up all resources, in particular drops the temporary table in the DB."
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:218:17:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L218) *style:* **Commented code should be removed.**
```r
#               warning("Connection seems exipired.\n    ...it's gone...")
                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:219:17:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L219) *style:* **Commented code should be removed.**
```r
#               return(FALSE)
                ^~~~~~~~~~~~~
```
[R/EXADBI.R:223:19:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L223) *style:* **Commented code should be removed.**
```r
#                 if(commit) message("Changes commited.")
                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:224:1:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L224) *style:* **lines should not be more than 80 characters.**
```r
#             } else stop("Commit failed. Changes NOT commited. Closing aborted.")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:225:13:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L225) *style:* **Commented code should be removed.**
```r
#           message("Closing connection...")
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:226:15:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L226) *style:* **Commented code should be removed.**
```r
#             res <- try(odbcClose(connection),silent=TRUE)
              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:228:19:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L228) *style:* **Commented code should be removed.**
```r
#                 message("Connection closed.")
                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:229:19:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L229) *style:* **Commented code should be removed.**
```r
#                 return(TRUE)
                  ^~~~~~~~~~~~
```
[R/EXADBI.R:231:19:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L231) *style:* **Commented code should be removed.**
```r
#                 warning("Closing not successful.")
                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:232:19:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L232) *style:* **Commented code should be removed.**
```r
#                 return(FALSE)
                  ^~~~~~~~~~~~~
```
[R/EXADBI.R:234:23:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L234) *style:* **Commented code should be removed.**
```r
#                     warning(res)
                      ^~~~~~~~~~~~
```
[R/EXADBI.R:235:23:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L235) *style:* **Commented code should be removed.**
```r
#                     return(FALSE)
                      ^~~~~~~~~~~~~
```
[R/EXADBI.R:240:14:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L240) *style:* **Commented code should be removed.**
```r
            #close()
             ^~~~~~~
```
[R/EXADBI.R:247:15:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L247) *style:* **Commas should always have a space after.**
```r
  "dbGetInfo","EXAResult",
              ^
```
[R/EXADBI.R:248:25:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L248) *style:* **Variable and function names should be all lowercase.**
```r
  definition = function(dbObj) {
                        ^~~~~
```
[R/EXADBI.R:250:19:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L250) *style:* **Variable and function names should be all lowercase.**
```r
      statement = dbObj$statement,
                  ^~~~~
```
[R/EXADBI.R:251:19:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L251) *style:* **Variable and function names should be all lowercase.**
```r
      row.count = dbObj$rows_fetched,
                  ^~~~~
```
[R/EXADBI.R:252:23:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L252) *style:* **Variable and function names should be all lowercase.**
```r
      rows.affected = dbObj$rows_affected,
                      ^~~~~
```
[R/EXADBI.R:253:23:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L253) *style:* **Variable and function names should be all lowercase.**
```r
      has.completed = dbObj$is_complete,
                      ^~~~~
```
[R/EXADBI.R:254:19:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L254) *style:* **Variable and function names should be all lowercase.**
```r
      is.select = dbObj$with_output
                  ^~~~~
```
[R/EXADBI.R:262:44:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L262) *style:* **Commas should always have a space after.**
```r
    NextMethod(generic = "summary", object,...)
                                           ^
```
[R/EXADBI.R:272:3:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L272) *warning:* **no visible global function definition for ‘new’, Did you mean 'news'?**
```r
  new("EXADriver")
  ^~~
```
[R/EXADBI.R:277:3:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L277) *warning:* **no visible global function definition for ‘new’, Did you mean 'news'?**
```r
  new("EXADriver")
  ^~~
```
[R/EXADBI.R:287:25:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L287) *style:* **Variable and function names should be all lowercase.**
```r
  definition = function(dbObj) {
                        ^~~~~
```
[R/EXADBI.R:298:47:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L298) *style:* **Variable and function names should be all lowercase.**
```r
setMethod("dbDataType", "EXAObject", function(dbObj, obj, ...) {
                                              ^~~~~
```
[R/EXADBI.R:299:3:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L299) *style:* **Variable and function names should be all lowercase.**
```r
  EXADataType(obj)
  ^~~~~~~~~~~
```
[R/EXADBI.R:303:13:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L303) *style:* **Variable and function names should be all lowercase.**
```r
  vapply(x, EXADataType, FUN.VALUE = character(1), USE.NAMES = FALSE)
            ^~~~~~~~~~~
```
[R/EXADBI.R:316:13:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L316) *style:* **Variable and function names should be all lowercase.**
```r
  vapply(x, EXADataType, FUN.VALUE = character(1), USE.NAMES = FALSE)
            ^~~~~~~~~~~
```
[R/EXADBI.R:323:5:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L323) *style:* **Variable and function names should be all lowercase.**
```r
    dbGetInfo(drv, "connectionIds")[[1]]
    ^~~~~~~~~
```
[R/EXADBI.R:385:1:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L385) *style:* **lines should not be more than 80 characters.**
```r
                        exalogfile = tempfile(pattern = "EXAODBC_", fileext = ".log"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:390:1:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L390) *style:* **lines should not be more than 80 characters.**
```r
                        connectionlcctype = Sys.getlocale(category = "LC_CTYPE"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:391:1:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L391) *style:* **lines should not be more than 80 characters.**
```r
                        connectionlcnumeric = Sys.getlocale(category = "LC_NUMERIC"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:395:3:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L395) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
  {
  ^
```
[R/EXADBI.R:396:5:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L396) *style:* **Variable and function names should be all lowercase.**
```r
    EXANewConnection(
    ^~~~~~~~~~~~~~~~
```
[R/EXADBI.R:425:5:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L425) *style:* **Variable and function names should be all lowercase.**
```r
    EXANewConnection(drv = dbDriver(drv), ...),
    ^~~~~~~~~~~~~~~~
```
[R/EXADBI.R:425:28:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L425) *style:* **Variable and function names should be all lowercase.**
```r
    EXANewConnection(drv = dbDriver(drv), ...),
                           ^~~~~~~~
```
[R/EXADBI.R:436:5:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L436) *style:* **Variable and function names should be all lowercase.**
```r
    EXACloneConnection(drv, ...),
    ^~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:448:1:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L448) *style:* **Variable and function names should be all lowercase.**
```r
dbCurrentSchema <- function(con) {
^~~~~~~~~~~~~~~
```
[R/EXADBI.R:449:10:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L449) *style:* **Variable and function names should be all lowercase.**
```r
  res <- sqlQuery(con, "select current_schema")
         ^~~~~~~~
```
[R/EXADBI.R:449:10:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L449) *warning:* **no visible global function definition for ‘sqlQuery’**
```r
  res <- sqlQuery(con, "select current_schema")
         ^~~~~~~~
```
[R/EXADBI.R:450:44:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L450) *style:* **Commas should always have a space after.**
```r
  con@current_schema <- as.character(res[1,1])
                                           ^
```
[R/EXADBI.R:457:1:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L457) *style:* **Variable and function names should be all lowercase.**
```r
EXANewConnection <- function( # change defaults also above
^~~~~~~~~~~~~~~~
```
[R/EXADBI.R:481:11:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L481) *style:* **Place a space before left parenthesis, except in a function call.**
```r
        if(dsn!="") {
          ^
```
[R/EXADBI.R:481:15:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L481) *style:* **Put spaces around all infix operators.**
```r
        if(dsn!="") {
             ~^~~
```
[R/EXADBI.R:482:38:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L482) *style:* **Commas should always have a space after.**
```r
            con_str <- paste0("DSN=",dsn)
                                     ^
```
[R/EXADBI.R:484:25:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L484) *style:* **Put spaces around all infix operators.**
```r
        else if (exahost!="" & uid!="") {
                       ~^~~
```
[R/EXADBI.R:484:35:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L484) *style:* **Put spaces around all infix operators.**
```r
        else if (exahost!="" & uid!="") {
                                 ~^~~
```
[R/EXADBI.R:485:70:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L485) *style:* **Commas should always have a space after.**
```r
            con_str <- paste0("DRIVER={EXASolution Driver};EXAHOST=",exahost)
                                                                     ^
```
[R/EXADBI.R:488:1:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L488) *style:* **lines should not be more than 80 characters.**
```r
            stop("Connect failed. Either DSN, host & db_user or a connection string must be given.\n
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:489:1:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L489) *style:* **lines should not be more than 80 characters.**
```r
                 Hint: No lazy declaration of onnection parameters - these have to be stated ' dsn=...'.\n
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:493:11:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L493) *style:* **Place a space before left parenthesis, except in a function call.**
```r
        if(uid!="") {
          ^
```
[R/EXADBI.R:493:15:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L493) *style:* **Put spaces around all infix operators.**
```r
        if(uid!="") {
             ~^~~
```
[R/EXADBI.R:494:39:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L494) *style:* **Commas should always have a space after.**
```r
            con_str <- paste0(con_str,";UID=",uid,";PWD=",pwd)
                                      ^
```
[R/EXADBI.R:494:47:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L494) *style:* **Commas should always have a space after.**
```r
            con_str <- paste0(con_str,";UID=",uid,";PWD=",pwd)
                                              ^
```
[R/EXADBI.R:494:51:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L494) *style:* **Commas should always have a space after.**
```r
            con_str <- paste0(con_str,";UID=",uid,";PWD=",pwd)
                                                  ^
```
[R/EXADBI.R:494:59:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L494) *style:* **Commas should always have a space after.**
```r
            con_str <- paste0(con_str,";UID=",uid,";PWD=",pwd)
                                                          ^
```
[R/EXADBI.R:497:11:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L497) *style:* **Place a space before left parenthesis, except in a function call.**
```r
        if(exaschema!="SYS") {
          ^
```
[R/EXADBI.R:497:21:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L497) *style:* **Put spaces around all infix operators.**
```r
        if(exaschema!="SYS") {
                   ~^~~
```
[R/EXADBI.R:498:39:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L498) *style:* **Commas should always have a space after.**
```r
            con_str <- paste0(con_str,";EXASCHEMA=",exaschema)
                                      ^
```
[R/EXADBI.R:498:53:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L498) *style:* **Commas should always have a space after.**
```r
            con_str <- paste0(con_str,";EXASCHEMA=",exaschema)
                                                    ^
```
[R/EXADBI.R:501:35:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L501) *style:* **Commas should always have a space after.**
```r
        con_str <- paste0(con_str,";EXALOGFILE=",exalogfile)
                                  ^
```
[R/EXADBI.R:501:50:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L501) *style:* **Commas should always have a space after.**
```r
        con_str <- paste0(con_str,";EXALOGFILE=",exalogfile)
                                                 ^
```
[R/EXADBI.R:504:35:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L504) *style:* **Commas should always have a space after.**
```r
        con_str <- paste0(con_str,";LOGMODE=",logmode)
                                  ^
```
[R/EXADBI.R:504:47:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L504) *style:* **Commas should always have a space after.**
```r
        con_str <- paste0(con_str,";LOGMODE=",logmode)
                                              ^
```
[R/EXADBI.R:507:1:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L507) *style:* **lines should not be more than 80 characters.**
```r
        con_str <- paste0(con_str,";CONNECTIONLCCTYPE=",connectionlcctype,";CONNECTIONLCNUMERIC=",connectionlcnumeric)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:507:35:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L507) *style:* **Commas should always have a space after.**
```r
        con_str <- paste0(con_str,";CONNECTIONLCCTYPE=",connectionlcctype,";CONNECTIONLCNUMERIC=",connectionlcnumeric)
                                  ^
```
[R/EXADBI.R:507:57:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L507) *style:* **Commas should always have a space after.**
```r
        con_str <- paste0(con_str,";CONNECTIONLCCTYPE=",connectionlcctype,";CONNECTIONLCNUMERIC=",connectionlcnumeric)
                                                        ^
```
[R/EXADBI.R:507:75:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L507) *style:* **Commas should always have a space after.**
```r
        con_str <- paste0(con_str,";CONNECTIONLCCTYPE=",connectionlcctype,";CONNECTIONLCNUMERIC=",connectionlcnumeric)
                                                                          ^
```
[R/EXADBI.R:507:99:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L507) *style:* **Commas should always have a space after.**
```r
        con_str <- paste0(con_str,";CONNECTIONLCCTYPE=",connectionlcctype,";CONNECTIONLCNUMERIC=",connectionlcnumeric)
                                                                                ^
```
[R/EXADBI.R:511:35:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L511) *style:* **Commas should always have a space after.**
```r
        con_str <- paste0(con_str,";autocommit=",autocommit)
                                  ^
```
[R/EXADBI.R:511:50:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L511) *style:* **Commas should always have a space after.**
```r
        con_str <- paste0(con_str,";autocommit=",autocommit)
                                                 ^
```
[R/EXADBI.R:515:25:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L515) *style:* **Put spaces around all infix operators.**
```r
        while (length(d)>0) {
                       ~^~
```
[R/EXADBI.R:516:39:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L516) *style:* **Commas should always have a space after.**
```r
            con_str <- paste0(con_str,";", names(d[1]),"=",d[1])
                                      ^
```
[R/EXADBI.R:516:56:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L516) *style:* **Commas should always have a space after.**
```r
            con_str <- paste0(con_str,";", names(d[1]),"=",d[1])
                                                       ^
```
[R/EXADBI.R:516:60:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L516) *style:* **Commas should always have a space after.**
```r
            con_str <- paste0(con_str,";", names(d[1]),"=",d[1])
                                                           ^
```
[R/EXADBI.R:521:12:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L521) *style:* **Variable and function names should be all lowercase.**
```r
    con <- odbcDriverConnect(con_str)
           ^~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:521:12:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L521) *warning:* **no visible global function definition for ‘odbcDriverConnect’**
```r
    con <- odbcDriverConnect(con_str)
           ^~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:522:21:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L522) *style:* **Variable and function names should be all lowercase.**
```r
    exa_metadata <- odbcGetInfo(con)
                    ^~~~~~~~~~~
```
[R/EXADBI.R:522:21:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L522) *warning:* **no visible global function definition for ‘odbcGetInfo’**
```r
    exa_metadata <- odbcGetInfo(con)
                    ^~~~~~~~~~~
```
[R/EXADBI.R:524:25:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L524) *style:* **Commas should always have a space after.**
```r
    new("EXAConnection",init_connection_string = con_str,
                        ^
```
[R/EXADBI.R:526:52:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L526) *style:* **Commas should always have a space after.**
```r
        autocom_default = ifelse(autocommit == "Y",TRUE,FALSE),
                                                   ^
```
[R/EXADBI.R:526:57:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L526) *style:* **Commas should always have a space after.**
```r
        autocom_default = ifelse(autocommit == "Y",TRUE,FALSE),
                                                        ^
```
[R/EXADBI.R:527:56:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L527) *style:* **Commas should always have a space after.**
```r
        db_host = strsplit(exa_metadata["Server_Name"],":")[[1]][1],
                                                       ^
```
[R/EXADBI.R:528:67:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L528) *style:* **Commas should always have a space after.**
```r
        db_port = as.numeric(strsplit(exa_metadata["Server_Name"],":")[[1]][2]),
                                                                  ^
```
[R/EXADBI.R:530:1:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L530) *style:* **lines should not be more than 80 characters.**
```r
          regmatches(attributes(con)$connection.string, gregexpr("UID=[\\w]+?;", attributes(con)$connection.string,perl=TRUE)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:530:116:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L530) *style:* **Commas should always have a space after.**
```r
          regmatches(attributes(con)$connection.string, gregexpr("UID=[\\w]+?;", attributes(con)$connection.string,perl=TRUE)
                                                                                ^
```
[R/EXADBI.R:530:120:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L530) *style:* **Put spaces around all infix operators.**
```r
          regmatches(attributes(con)$connection.string, gregexpr("UID=[\\w]+?;", attributes(con)$connection.string,perl=TRUE)
                                                                                ~^~
```
[R/EXADBI.R:534:1:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L534) *style:* **lines should not be more than 80 characters.**
```r
              regmatches(attributes(con)$connection.string, gregexpr("UID=[\\w]+?;", attributes(con)$connection.string,perl=TRUE))[[1]]
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:534:120:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L534) *style:* **Commas should always have a space after.**
```r
              regmatches(attributes(con)$connection.string, gregexpr("UID=[\\w]+?;", attributes(con)$connection.string,perl=TRUE))[[1]]
                                                                                ^
```
[R/EXADBI.R:534:124:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L534) *style:* **Put spaces around all infix operators.**
```r
              regmatches(attributes(con)$connection.string, gregexpr("UID=[\\w]+?;", attributes(con)$connection.string,perl=TRUE))[[1]]
                                                                                ~^~
```
[R/EXADBI.R:551:1:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L551) *style:* **Variable and function names should be all lowercase.**
```r
EXACloneConnection <- function(drv, autocommit, ...) { # todo: parameters
^~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:551:54:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L551) *style:* **Opening curly braces should never go on their own line and should always be followed by a new line.**
```r
EXACloneConnection <- function(drv, autocommit, ...) { # todo: parameters
                                                     ^
```
[R/EXADBI.R:553:12:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L553) *style:* **Variable and function names should be all lowercase.**
```r
    drv <- dbCurrentSchema(drv) # update schema metadata
           ^~~~~~~~~~~~~~~
```
[R/EXADBI.R:553:12:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L553) *warning:* **no visible global function definition for ‘dbCurrentSchema’**
```r
    drv <- dbCurrentSchema(drv) # update schema metadata
           ^~~~~~~~~~~~~~~
```
[R/EXADBI.R:556:25:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L556) *style:* **Commas should always have a space after.**
```r
    d <- data.frame(...,stringsAsFactors = FALSE)
                        ^
```
[R/EXADBI.R:570:1:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L570) *style:* **lines should not be more than 80 characters.**
```r
        con_str <- paste0(con_str, ";", s[[1]][1],"=", s[[1]][2]) # take the s parameter
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:570:51:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L570) *style:* **Commas should always have a space after.**
```r
        con_str <- paste0(con_str, ";", s[[1]][1],"=", s[[1]][2]) # take the s parameter
                                                  ^
```
[R/EXADBI.R:586:31:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L586) *style:* **Commas should always have a space after.**
```r
    con_str <- substr(con_str,2,nchar(con_str)) # remove the initial semicolon
                              ^
```
[R/EXADBI.R:586:33:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L586) *style:* **Commas should always have a space after.**
```r
    con_str <- substr(con_str,2,nchar(con_str)) # remove the initial semicolon
                                ^
```
[R/EXADBI.R:588:12:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L588) *style:* **Variable and function names should be all lowercase.**
```r
    con <- odbcDriverConnect(con_str)
           ^~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:588:12:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L588) *warning:* **no visible global function definition for ‘odbcDriverConnect’**
```r
    con <- odbcDriverConnect(con_str)
           ^~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:590:1:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L590) *style:* **lines should not be more than 80 characters.**
```r
        stop(paste("EXACloneConnection error: failed to initialise connection.\nConnection String:", con_str))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:592:21:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L592) *style:* **Variable and function names should be all lowercase.**
```r
    exa_metadata <- odbcGetInfo(con)
                    ^~~~~~~~~~~
```
[R/EXADBI.R:592:21:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L592) *warning:* **no visible global function definition for ‘odbcGetInfo’**
```r
    exa_metadata <- odbcGetInfo(con)
                    ^~~~~~~~~~~
```
[R/EXADBI.R:596:1:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L596) *style:* **lines should not be more than 80 characters.**
```r
        autocom_default = ifelse(!missing(autocommit),ifelse(autocommit == "Y",TRUE,FALSE),drv@autocom_default),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/EXADBI.R:596:55:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L596) *style:* **Commas should always have a space after.**
```r
        autocom_default = ifelse(!missing(autocommit),ifelse(autocommit == "Y",TRUE,FALSE),drv@autocom_default),
                                                      ^
```
[R/EXADBI.R:596:80:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L596) *style:* **Commas should always have a space after.**
```r
        autocom_default = ifelse(!missing(autocommit),ifelse(autocommit == "Y",TRUE,FALSE),drv@autocom_default),
                                                                               ^
```
[R/EXADBI.R:596:85:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L596) *style:* **Commas should always have a space after.**
```r
        autocom_default = ifelse(!missing(autocommit),ifelse(autocommit == "Y",TRUE,FALSE),drv@autocom_default),
                                                                                ^
```
[R/EXADBI.R:596:92:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L596) *style:* **Commas should always have a space after.**
```r
        autocom_default = ifelse(!missing(autocommit),ifelse(autocommit == "Y",TRUE,FALSE),drv@autocom_default),
                                                                                ^
```
[R/EXADBI.R:597:56:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L597) *style:* **Commas should always have a space after.**
```r
        db_host = strsplit(exa_metadata["Server_Name"],":")[[1]][1],
                                                       ^
```
[R/EXADBI.R:598:67:](https://github.com/vsimko/r-exasol/blob/76b732f3d5e8256ab4465bf727a99267f830c25d/R/EXADBI.R#L598) *style:* **Commas should always have a space after.**
```r
        db_port = as.numeric(strsplit(exa_metadata["Server_Name"],":")[[1]][2]),


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11307](https://teamcity.mesosphere.io/viewLog.html?buildId=103491&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:11:07


111411352_ATDeprecateFields (Nigusse) - i386-Windows-7-VisualStudio-12: OK (2054 of 2054 tests passed, 0 test warnings)

![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)


ObjexxFCL41 (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed (1239 of 2094 tests passed, 41 test warnings)

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


why don't you use a mock ?


`karam` typo - also persisted below


Well all the functions in the C API are entirely private. The documentation is only for people working on the source anyway. IMO it won't matter if rdoc parses this or not.
-


Should remove the whole file, really 😉


you could test the case where no strategy is provided


Will do.


It's been documented in the latest commit


This isn't actually testing evaluate, test that the callback yields the correct parameters instead.


probably a lot of tests missing


Rename to `create`


This is in master, rebase needed I think?


TeamCity Civic Assembly :: Build backend + frontend [Build 156](http://54.229.176.143/tc/viewLog.html?buildId=431&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


hi @mfelczak. Thanks for the feedback. I will update the PR accordingly.


FYI 6.0 are very popular. Already over 2k downloads :-)


真是辛苦了~


Fix this formatting, one property per line, end with comma.


I kept it that way because FFTW uses that sort of convention for real numbers. Should I still change?


I cant to stablish any conection


private


ObjexxFCL41 (DeadParrot) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


@BeforeClass


LGTM.


Hmm. It could be. You think it worth to do so even that the patch version is already out there?


Please merge these edits for this morning's publication.


you are testing two classes at once. Is it intended ?


Dude, who hasn't wanted to have a static pizza cannon on-top of Lakka hill raining pizza goodness everywhere? =P


OK, so I know what's wrong, previously tests were not excluded from GeoKey upon install (this is what we wanted before), so I fixed that. However, because tests are excluded, you can't use them for your own tests...


It's not a DERPublicKeyProvider, it's a PublicKeyProvider


maybe we can get a less technical perfect but more user friendly message in here?


Scanner sınıfı hakkında yazılan çizileni okurken her defasında girdileri okumak için yeni bir Scanner objesi yaratmak yerine bunu en başta bir defa yaratıp program süresince kullanmanın (bu hesap makinesi örneğinde metotlara argüman olarak geçmenin) ve bitiminde kapatmanın (.close()) daha doğru olduğunu öğrendim. Aynı mantıkla senin kodunda ufak tefek değişiklikler yaptım (bir de ek olarak switch statement ile baştan yapmayı denedim). Daha doğru mu tam emin değilim (Taner hocaya da sormalı), öyle gibi geldi, paylaşmak istedim. Aşağıdaki linkten görebilirsin, selamlar:

https://github.com/patateskafa/patitiz/blob/master/src/com/alp/alistirma/ev/HesapMakinesi.java
https://github.com/patateskafa/patitiz/blob/master/src/com/alp/alistirma/ev/HesapMakinesiSwitch.java


better way to check if we are in the new form or the edit form is to check the record:

    f.object.new_record?


@pinepain can this change somehow be covered by a unit test?


I think `ErrorUtils.reportError` should be called for unhandled promise errors.


@Dominator008 I'm seeing some NullPointerExceptions on this line. I'll try to write up a minimal test case.


They gave us the translation for this, so you could set msgtr here to:

> Buscá o agregá información sobre los candidatos para las próximas Elecciones Municipales del 7 de febrero


Das schaut ja interessant aus :) Was es nur wird?


Sorry, that's for the one below, of course.


There should be a TODO to fix this so it's not dependent on /Users/alex.


See https://github.com/fitztrev/shuttle/pull/158

I noticed that Shuttle 1.2.5 still does not correctly handle the situation where you run a command while iTerm is running without any windows open. Although it does open a new tab, the tab shows up with the default theme and not the custom theme, if you specified one. This is a weird state that iTerm gets into that doesn't happen often in practice, although I have stumbled across it a few times already since upgrading to Shuttle 1.2.5.

To reproduce the problem:
1. Run iTerm.
1. Hit shift-cmd-w to close the window but do not quit iTerm.
1. Run a command with a custom theme from the Shuttle menu. The tab shows up with the default theme instead.

The solution is to check if `current terminal` succeeds and call `make new terminal` if not. Then use that terminal to launch the session.


But they've also given us this one:

> Para un uso óptimo del sitio, recomendamos seleccionar "Español" en la configuración de lenguajes de tu browser


How about turning the:
> sd_event_run() delay profiling enabled

logged once in sd_event_new() into

> sd_event_run() delay profiling enabled.
> Logarithmic histogram of event loop iteration durations 2^0 ... 2^63 us will be logged every 5s


Fix this formatting


Fix this formatting, 80 chars max.


Not sure this is the best way -- a preferable scheme would detect the version found by ``find_package(PythonLibs ..)`` and then enforce that for the ``find_package(PythonInterp ..)`` call. Can you modify the PR accordingly?


You should put that file in the Test bundle and use NSBundle to resolve its path dynamically at runtime.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1216.6779ee40cc1dce72c8cc6e5f43cace7ae911aff6](http://build.palaso.org/viewLog.html?buildId=67869&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 824, ignored: 7 Build time: 00:19:57


Would be nice if this was link to wrap it in an a tag.


you don't need to give MessageProperty to HashSet constructor


Shouldn't delete these lines in this commit


You could use subscripting syntax for these.


Das kann ich ihm verraten ;) Das ist der Quellcode für das RPG das ich für mein Rheinwerk Videotraining mache.


Я знаю, но оно у тебя так и было. Я поменял на \Fatmouse\Proxies\Agent и стал падать Drone тест поэтому я откатил назад.


Yeah it in theory can be done as soon as a function is created, although it can also be avoided if landing pads are turned off.


fixed in fa68dd695fb380842ca1e57161b2bbc45208b090


TeamCity pull requests :: netty [Build 1018](https://garage.netty.io/teamcity/viewLog.html?buildId=4701&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6370, ignored: 23 Build time: 01:22:22


Нужно было просто обновить fxp/composer-asset-plugin: composer global update
А потом обновить вендоров в проекте yii2.


very cool :+1:


Hm that's true yeah, this is used by `PartialEq` for `EarlyExitLabel` which is in turn used to find a cached label. It doesn't quite seem "smelly enough" to switch everything to manual impls using `same_kind` as well, but I could go either way.


Suggestion:

**Note**: Jobs can run for up to 60 minutes. When jobs exceed that limit, they fail. If a job is exceeding the limit, break it into multiple jobs. For example, if a job performs three tasks, you might break it into three jobs: one for each task.


Oh, yes! I'll fix that, thanks


done


Can you fix it?


В общем вот


There is no package named mingw in debian:
https://packages.debian.org/search?keywords=mingw&searchon=names&suite=all&section=all

Maybe is `mingw-w64`? This seems to depend on gcc and g++, I don't know if we need something more.


Thanks aasoror  ... Maybe I'll get it right on the third try ... :).


Checkmarx scan results 12 Jan 2016 17:24 UTC +00:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
0
4
33
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

ad030f7b673e3ef26e01d1ede8d18e7229e0908c	@macxil


Rename to `clear`, the protocol is verbose, node is not.


not sure if there's a word missing here:
> will tie the events that **would otherwise** be sent together in a batch.?


Test the callback, not the event


Thank you for this.  While I like the way the blaze-client is shaping up, it would be great to have an alternative, like we do for servers.

A more general comment on approach here: dispatch is a wrapper for async-http-client.  I think we could shed a layer by mapping directly to the underlying model in `toDRequest` and `fromDResponse`, and mapping the Java Future directly to a Task.


See https://github.com/janstovr/node-rdbg/commit/afbbd10f1e9cf41baef4cbade03c825d2f4a3ac5#commitcomment-15403109


From reading this I do not fully understand why the worker should produce more than the spiders consume. Won't this result in an overloading of the message bus after a long period of time? Isn't the ideal situation when exactly the same amount is consumed as is produced? Other than that I like it.
Also I can recommend [LanguageTool](https://languagetool.org/) for grammar and style checking. There is also e.g. a vim extension.


Ahh, I didn't realize you released the patch version. Probably not worth it then.


还有修复AI使用锁定视为技导致CardFinished时机内容出错


why do we need to control the PWM from inside this node?


Good catch. I'll make the change.


This is an attempt to standardize the location of the drush aliases file in projects. sites/all/drush is preferred because you distribute it with the repo, it comes "ready to go" (a design principle for the whole Vagrant box experience), and doesn't pollute your global alias list. An added bonus is that you can shorten the aliases, since they're site specific: ```@dev``` is always the dev alias for your current site.

The file copied still has local, dev, staging, and live aliases, and the latter 3 still need to be customized, chiefly with the name of your site... which makes me wonder if this step actually belongs in Yeoman instead of here.


@cbenz:
I tried to use the cache as we have used it for zone APL.
Could you tell me what I've done wrong ?
Thx


The problem being addressed is that the ```xmldom``` project released a new 0.1.20 version which has a bug/break. The broken version got picked up by this project since the version specified in package.json is 0.1.x.

This fix locks ```xmldom``` version at 0.1.19, and locks other ambiguous version numbers at current versions that work.


Молодец, возьми с полки пирожок


The extension `.xml` is too general and is not only used for mavlink messages description. So I don't think this is a good approach because the task `mavgen` will be created for any task generator that has XML files as part of its source files. Furthermore, any later call of `@extension('.xml')` will overwrite this one.

I propose using something like:
```python
@feature('mavgen')
@before_method('process_source')
def process_mavgen(self):
    inputs = self.to_nodes(self.source)
    outputs = []

    self.target = Utils.to_list(getattr(self, 'target', []))
    for t in self.target:
        if not isinstance(t, Node.Node):
            t = self.bld.bldnode.find_or_declare(t)
        outputs.append(t)
    # or we could just get the first target
    # self.target = Utils.to_list(getattr(self, 'target', []))
    # if not isinstance(self.target[0], Node.Node):
    #     self.target[0] = self.bld.bldnode.find_or_declare(self.target[0])
    # outputs = [self.target[0]]

    self.source = []
    self.create_task('mavgen', inputs, outputs)
```

What do you think?


This is the main culprit.


Sounds good. but maybe in one line, and with a full stop at the end, and with "sd_event_run() delay" replaced by "Event loop" or so... Combing this to:

```
Event loop profiling enabled. Logarithmic histogram of event loop iterations in the range 2^0 ... 2^63 us will be logged every 5s.
```


Got rid of all the ```x```'s and ```~```'s.


Not sure how this single quote (```) got in here, but this wont compile with it ;)


You did not create a pull request. Just a friendly review on code you have pushed into your remote. I do like seeing what people do with the code. Why the preference for the rsrd set over the core rule books?


Why?


> Note that we can't name it exactly as documented as it can mean different things depending on what's passed (because of `function-overloading) Naming it this way makes it more readable. Have a look at the putObject code you'll see why naming arguments this way is more readable.

Then its a bad idea to provide function overloading, the problem is Javascript doesn't support function overloading in its truest sense.


So when does the callback get called, when there is not an error? :grinning:


Installing nsis seems to bring all the mingw stuff, so we might not need the `install mingw` step.


About this interface, if hasNext() can throw, it kinda makes code more complicated, e.g.:

    while(hasNext()) {
    ...
    }

Has to become for example:

    while(true) {
        try {
            if (!hasNext()) break;
        } catch (...) {
        }
    ...
    }

What do you think about making next() not throw?
It makes the iterator implementation more complex, but I think it's doable.


Test the callback results, not the response event


oops, I forgot to return it


We are not depending on sleep time for correctness. This is nice to have and if we are luckily we may get a cache hit (vnid map). If we don't find it in the map, we query etcd to fetch/validate (Line#131).


The target should be passed too. See this comment https://github.com/bugobliterator/ardupilot/commit/6db2c5673c9f62df0e9f9160c3dae074e676c3d0#commitcomment-15400521 and the suggestion at https://github.com/bugobliterator/ardupilot/commit/6db2c5673c9f62df0e9f9160c3dae074e676c3d0#commitcomment-15403163 .


I *think* this notification is deprecated, this never gets sent from Chrome at-least


Bien visto! ahora envio la corrección.


TeamCity Civic Assembly :: Build backend + frontend [Build 156](http://54.229.176.143/tc/viewLog.html?buildId=431&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 19, passed: 1523, ignored: 12 Build time: 00:18:27
### Failed tests
```
Document type tests: Should not create new document type if cancelled: <no details avaliable>

Document type tests: Should add new document type: <no details avaliable>

Document type tests: Should not change document type: <no details avaliable>

Document type tests: Should change document type: <no details avaliable>

Document type tests: Should not delete document type if cancel pressed: <no details avaliable>

Document type tests: Should delete document type: <no details avaliable>

Media type tests: Should add new media type: <no details avaliable>

Media type tests: Should not change media type: <no details avaliable>

Media type tests: Should change media type: <no details avaliable>

Media type tests: Should not delete media type if cancel pressed: <no details avaliable>

Media type tests: Should delete media type: <no details avaliable>

##### there are 9 more failed tests, see build details
```


The examples directory is intended to be files that are generic in nature and would be customized on a per-project basis after being copied to the appropriate location. A number of those files should be dealt with by the Yeoman generator when we can get back to it.


const Vec3 t ;-) @simogasp


Так, сейчас будет много претензий по грамотным именам переменных, методов и прочего! :) (Есть программистская поговорка, что в программировании всего две сложных вещи. Так вот одна из них — грамотно выбирать имена)

Данные, которые мы тут передаём в initialize — это данные об _одном_ фильме, а у тебя тут множественное число (как будто фильмов много). То есть правильное название для входящих данных — либо `movie`, либо (чтобы не было тавтологии «initialize Movie with movie») — например, указание на то, какого типа данные ожидаются, то есть `movie_hash` или даже просто `hash` (читается «initialize Movie with hash»)


These probably shouldn't be the 'apps' versions of these labels. In general I tried to have those match the image's name, so org.iplantc.de.jobservices.git-ref etc. would match the pattern.